Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Behavior change when modifying environment variables #2262

Closed
1 task done
mbrancato opened this issue Jan 6, 2022 · 1 comment · Fixed by fluxcd/kustomize-controller#527
Closed
1 task done

Behavior change when modifying environment variables #2262

mbrancato opened this issue Jan 6, 2022 · 1 comment · Fixed by fluxcd/kustomize-controller#527

Comments

@mbrancato
Copy link

Describe the bug

I think I'm noticing another behavior change after a recent upgrade to 0.24.1 from pre-0.18 versions. While applying a number of changes to environment variables in several deployments, none were applied when committed to git. Some other, related changes were made, but resources with only env var changes were not updated. Previously, these were updated when merged into git.

Specifically, the changes were made in a Deployment in .spec.template.spec.containers[1].env[] where entries were removed.

I looked thru the docs and couldn't find anything about how arrays or lists are handled when a single item is removed. Does it seem that by removing the entry it's no longer managed by Flux but wasn't pruned either? Is this expected behavior?

Steps to reproduce

Remove an environment variable from a list of other environment variables and commit to the flux source.

Expected behavior

Reconciliation should apply the changes.

Screenshots and recordings

No response

OS / Distro

n/a

Flux version

v0.24.1

Flux check

► checking prerequisites
✔ Kubernetes 1.19.14-gke.1900 >=1.19.0-0
► checking controllers
✔ helm-controller: deployment ready
► fluxcd/helm-controller:v0.14.1
✔ image-automation-controller: deployment ready
► fluxcd/image-automation-controller:v0.18.0
✔ image-reflector-controller: deployment ready
► fluxcd/image-reflector-controller:v0.14.0
✔ kustomize-controller: deployment ready
► fluxcd/kustomize-controller:v0.18.2
✔ notification-controller: deployment ready
► fluxcd/notification-controller:v0.19.0
✔ source-controller: deployment ready
► fluxcd/source-controller:v0.19.2
✔ all checks passed

Git provider

GitHub

Container Registry provider

GCR

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants