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

fix: explicitly set resourceFieldRef.divisor #1655

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

Jaydee94
Copy link
Contributor

@Jaydee94 Jaydee94 commented Dec 6, 2024

Description of the change

In the helm chart the divisor is not set and kubernetes mutates the Deployments to "0" which is the same as "1".

Benefits

When deploying with ArgoCD the application will not be displayed as "OutOfSync"

Possible drawbacks


In the helm chart the divisor is not set and kubernetes mutates the Deployments to "0" which is the same as "1".

Signed-off-by: Jan <55805868+Jaydee94@users.noreply.github.com>
@Jaydee94 Jaydee94 changed the title fix: explicitly set resourceFieldRed.divisor fix: explicitly set resourceFieldRef.divisor Dec 6, 2024
Copy link
Contributor

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the Stale label Dec 22, 2024
@Jaydee94
Copy link
Contributor Author

Hey, can you consider to accept this change to the helm chart?

This would help us so that our Argocd does not show out of sync.

Best Regards :)

@github-actions github-actions bot removed the Stale label Dec 23, 2024
Copy link
Contributor

github-actions bot commented Jan 8, 2025

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the Stale label Jan 8, 2025
Copy link
Contributor

Due to the lack of activity in the last 7 days since it was marked as "stale", we proceed to close this Pull Request. Do not hesitate to reopen it later if necessary.

@github-actions github-actions bot closed this Jan 16, 2025
@alvneiayu alvneiayu added backlog Issues/PRs that will be included in the project roadmap and removed Stale labels Jan 16, 2025
@agarcia-oss agarcia-oss reopened this Jan 16, 2025
@alvneiayu alvneiayu merged commit a78a582 into bitnami-labs:main Jan 16, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Issues/PRs that will be included in the project roadmap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants