concourse-chart icon indicating copy to clipboard operation
concourse-chart copied to clipboard

should global resources be enabled by default?

Open iedsapala opened this issue 4 years ago • 4 comments

I noticed that global resources are enabled by default when using this chart, but are disabled by default when using the "off-the-shelf" concourse binary. Should this feature be enabled by default when using this chart and if so why?

iedsapala avatar Apr 27 '21 17:04 iedsapala

additional: I noticed this flag was introduced in an "enabled" state in commit 26cb45b72d0fc72c7b7270d2b980ff538b7623be but the comment doesn't indicate why.

iedsapala avatar Apr 27 '21 17:04 iedsapala

It's off by default because it breaks some exotic workflows like EC2 IAM profiles, I'm not sure if it was an intentional decision to enable it for the helm chart. We did notice that the chart had it enabled, but nobody reported any issues regarding it so we kinda just let it be. 🤷

We would like it to be on by default because it gives a decent performance boost to resource checking and at this point the feature itself is stable. But until we have workarounds to unblock the exotic workflows, we have to keep it opt-in, this is better tracked at https://github.com/concourse/concourse/issues/4870

chenbh avatar Apr 27 '21 18:04 chenbh

Looks like this was added before we changed tactics to avoid this very confusion. The concourse binary should be the source of truth for default values. This wasn't being enforced in PR review until the last year-ish.

taylorsilva avatar Apr 28 '21 15:04 taylorsilva

Our team got bitten by this particular feature flag differing in the chart defaults vs the concourse defaults.

We're using two different git resources, one for input used as the trigger for the job and one for output used to push a release tagging commit, that are pointed at the same branch with the same config. When we did a put to the out resource it created a new version (decorated with [skip ci]) which should have been ignored by the input resource, but instead it got picked up and caused the job to do a new build. Please see the attached image of the pipeline below.

image

Considering this kind of possible situation, and the general feature flag discrepancy between the chart, should this feature flag be disabled by default?

iedsapala avatar Apr 28 '21 15:04 iedsapala