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

[Tech Debt] Follow up on helm-locker repo reorg #78

Closed
7 tasks done
mallardduck opened this issue Sep 25, 2024 · 4 comments
Closed
7 tasks done

[Tech Debt] Follow up on helm-locker repo reorg #78

mallardduck opened this issue Sep 25, 2024 · 4 comments

Comments

@mallardduck
Copy link
Member

mallardduck commented Sep 25, 2024

Moving the follow up actions into this issue from the original PR.

  • Tag a v0.3.0 release here - which will release both images to GHCR
  • Add package/helm-project-operator label to existing issues in this repo
  • Shutdown usage of the helm-locker repo:
    • Move over the issues from helm-locker repo & label as package/helm-locker,
    • update helm-locker repo readme,
    • archive helm-locker repo, - created issue to track
  • [n/a] Implement new branching strategy (maybe after Alex PR merge, need to decide optimal order)
    • Realized this is getting ahead of ourselves since we want to merge this codebase into prometheus-federator code base,
    • The goal to use new branching was slated for after that process completed and this was just the first half of that process.
    • For now we will do minimal dependency bumps keeping Rancher 2.8 as our target; this should keep these projects more updated and maintain compatability with all necessary k8s versions.
  • [n/a] Rebuild Alex's PR to update helm-locker on this repo; will update both sets of dependencies at once.
    • I've realized a lot of the changes that happened since the PR was started supersede it,
    • At this point I think we'll forego that PR (reevaluate later to decide if needed at all or if we close it),
  • Update prometheus-federator to use patched versions of this repo,
  • [n/a] subsequently ensure rancher-monitoring and rancher-project-monitoring are updated and in charts

This PR's n/a steps will be followed up on later in rancher/prometheus-federator#109 or other work in the prom-fed repo.

@mallardduck
Copy link
Member Author

Having created an initial 0.3.0 tag, i'm working on the prometheus-federator PR to use the new repo structure now. Last time I worked on it I was able to make some progress fixing CI issues there. However ran out of time on friday so I will get back to that task this week.

@mallardduck
Copy link
Member Author

New release with CI fixes that were needed: https://github.com/rancher/helm-project-operator/releases/tag/v0.3.1

@mallardduck
Copy link
Member Author

PR to update Prom Federator here: rancher/prometheus-federator#95

@mallardduck
Copy link
Member Author

Closing as superseded by rancher/prometheus-federator#109

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant