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

Add Helm Support to Release Manager #866

Closed
serverhorror opened this issue Mar 14, 2022 · 4 comments
Closed

Add Helm Support to Release Manager #866

serverhorror opened this issue Mar 14, 2022 · 4 comments

Comments

@serverhorror
Copy link
Contributor

Is your feature request related to a problem? Please describe.

There are a quite a few cases where helm is needed/helpful to deploy and to be able to deploy via release manager

Describe the solution you'd like

The ods-infra type can already deploy helm charts. Right now this is deliberately prohibited:

Describe alternatives you've considered

Nothing alternative considered

Additional context

This has been discussed in #554 already so this issue is to have a focused discussion about specifically helm and release manager.

  1. I have a branch ready that allows helm charts to promote to P from D and can send it as a PR.
  2. I haven't found a good place for the docs yet, any hint is appreciated
@serverhorror
Copy link
Contributor Author

This is also in relation to these: #537, #554

@serverhorror
Copy link
Contributor Author

@s2oBCN -- ping! A little help how a good test would look like is much appreciated

@serverhorror
Copy link
Contributor Author

related #916

jafarre-bi pushed a commit that referenced this issue Jan 27, 2023
Add Helm to possible methods for Component and Release Manger rollouts

* Fixes #916, #866
* Relates #922

Co-authored-by: Martin Marcher <serverhorror@users.noreply.github.com>
Co-authored-by: Martin Marcher <martin.marcher@boehringer-ingelheim.com>
@braisvq1996
Copy link
Contributor

Closed by PR #916

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