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

[main] helm-project-operator and helm-locker repo reorg updates #95

Merged
merged 54 commits into from
Oct 9, 2024

Conversation

mallardduck
Copy link
Member

Produced a mallardduck/prometheus-federator:reorg-demo image and pushed to dockerhub.

(This PR is based on a local repo branch equal to: #93)

@mallardduck mallardduck requested a review from a team as a code owner October 5, 2024 00:53
@mallardduck mallardduck force-pushed the dpock/repo-updates branch 2 times, most recently from d408ff6 to 8210a5c Compare October 7, 2024 17:24
@mallardduck
Copy link
Member Author

mallardduck commented Oct 7, 2024

So I just realized that updating the chart in this PR is a chicken and egg problem.

We want to update the chart dependencies to make sure it uses the new helm-project-operator chart. However in doing so it is "proper process" to bump the version otherwise we end up clobbering over the exiting chart files. However the image tag for the new chart will not be prepared yet so cannot be pulled to install.

That said, I don't think anything in our CI/tests will use the new image tag for testing, since it should use "local" (to the test runner) builds of the images for testing. In these cases the tag in the values.yaml should be overwritten anyway.


Do take note that the current E2E test failures stem from a mixture of issue. Mainly:

scripts/pull-scripts Outdated Show resolved Hide resolved
@mallardduck mallardduck merged commit c86bc81 into main Oct 9, 2024
4 checks passed
@mallardduck mallardduck deleted the dpock/repo-updates branch January 30, 2025 19:33
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 this pull request may close these issues.

2 participants