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

Registry operator should handle environment variable updates #784

Closed
kim-tsao opened this issue Feb 25, 2022 · 0 comments
Closed

Registry operator should handle environment variable updates #784

kim-tsao opened this issue Feb 25, 2022 · 0 comments
Assignees
Labels
area/registry Devfile registry for stacks and infrastructure kind/bug Something isn't working
Projects

Comments

@kim-tsao
Copy link
Contributor

/kind bug

Which area this bug is related to?

/area registry

Bug Summary

Describe the bug:

The DevfileRegistry CRD allows telemetry properties to be specified (registry name, key) and these are mapped to environment variables in the deployment but if a user updates these properties, they do not take into effect (i.e. no live updates). Users would need to re-create the pods in order to see the updates.

To Reproduce:

  1. Deploy a DevfileRegistry with Telemetry configuration specified
  2. Update the deployment with modified properties (registryName and/or key)
  3. The environment variables in the pod are not updated

Expected behavior

Changes should occur on existing pod

Additional context

Looks like it's just the telemetry properties that are affected but I'll review the other ones to make sure we have everything covered.

Any workaround?

Delete the deployment. New pods should have the changes

Suggestion on how to fix the bug

Need to update this method to handle the environment variables:

@openshift-ci openshift-ci bot added the kind/bug Something isn't working label Feb 25, 2022
@kim-tsao kim-tsao self-assigned this Feb 25, 2022
@kim-tsao kim-tsao added this to Under consideration in Sprint 214 via automation Feb 25, 2022
@kim-tsao kim-tsao moved this from Under consideration to To do in Sprint 214 Feb 25, 2022
@kim-tsao kim-tsao added the area/registry Devfile registry for stacks and infrastructure label Feb 25, 2022
@johnmcollier johnmcollier removed this from To do in Sprint 214 Feb 28, 2022
@johnmcollier johnmcollier added this to Under consideration in Sprint 215 via automation Feb 28, 2022
@johnmcollier johnmcollier moved this from Under consideration to To do in Sprint 215 Feb 28, 2022
@kim-tsao kim-tsao moved this from To do to In progress in Sprint 215 Mar 2, 2022
@kim-tsao kim-tsao moved this from In progress to Review in progress in Sprint 215 Mar 2, 2022
@kim-tsao kim-tsao closed this as completed Mar 9, 2022
Sprint 215 automation moved this from Review in progress to Done Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/registry Devfile registry for stacks and infrastructure kind/bug Something isn't working
Projects
No open projects
Development

No branches or pull requests

1 participant