Skip to content

Commit

Permalink
B: make caching streamed volumes opt-in
Browse files Browse the repository at this point in the history
previously, this feature was enabled by default. we found that the disk
usage on our darwin/windows workers increased significantly though,
since we now keep volumes around for much longer. to avoid users running
into issues around this, lets make the feature off by default

Signed-off-by: Clara Fu <fclara@vmware.com>
  • Loading branch information
clarafu committed May 20, 2021
1 parent 5c2e643 commit f9f02e3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions atc/atccmd/command.go
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ type RunCommand struct {
EnableAcrossStep bool `long:"enable-across-step" description:"Enable the experimental across step to be used in jobs. The API is subject to change."`
EnablePipelineInstances bool `long:"enable-pipeline-instances" description:"Enable pipeline instances"`
EnableP2PVolumeStreaming bool `long:"enable-p2p-volume-streaming" description:"Enable P2P volume streaming"`
DisableCacheStreamedVolumes bool `long:"disable-cache-streamed-volumes" description:"By default, streamed resource volumes will be automatically cached on the destination worker. This flag opts out of that behaviour"`
EnableCacheStreamedVolumes bool `long:"enable-cache-streamed-volumes" description:"When enabled, streamed resource volumes will be cached on the destination worker."`
} `group:"Feature Flags"`

BaseResourceTypeDefaults flag.File `long:"base-resource-type-defaults" description:"Base resource type defaults"`
Expand Down Expand Up @@ -521,7 +521,7 @@ func (cmd *RunCommand) Runner(positionalArguments []string) (ifrit.Runner, error
atc.EnableBuildRerunWhenWorkerDisappears = cmd.FeatureFlags.EnableBuildRerunWhenWorkerDisappears
atc.EnableAcrossStep = cmd.FeatureFlags.EnableAcrossStep
atc.EnablePipelineInstances = cmd.FeatureFlags.EnablePipelineInstances
atc.EnableCacheStreamedVolumes = !cmd.FeatureFlags.DisableCacheStreamedVolumes
atc.EnableCacheStreamedVolumes = cmd.FeatureFlags.EnableCacheStreamedVolumes

if cmd.BaseResourceTypeDefaults.Path() != "" {
content, err := ioutil.ReadFile(cmd.BaseResourceTypeDefaults.Path())
Expand Down

0 comments on commit f9f02e3

Please sign in to comment.