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

XServices should only contain 1 helm release #114

Closed
stevendborrelli opened this issue Sep 7, 2023 · 3 comments · Fixed by #127
Closed

XServices should only contain 1 helm release #114

stevendborrelli opened this issue Sep 7, 2023 · 3 comments · Fixed by #127
Labels
enhancement New feature or request

Comments

@stevendborrelli
Copy link
Member

What problem are you facing?

Currently XServices installs both a Prometheus and Universal-Crossplane chart.

How could Upbound help solve your problem?

  • Update XServices to only deploy 1 generic Helm release
  • Update API for helm parameters
  • Split out provisioning of each chart into a separate XR.

Epic #

@stevendborrelli stevendborrelli added the enhancement New feature or request label Sep 7, 2023
@ytsarev
Copy link
Member

ytsarev commented Sep 7, 2023

What's wrong with multiple Releases? XServices semantically matching multiple services to install

@haarchri
Copy link
Member

With functions, we can create a versatile input fields for Helm releases, allowing users to add as many releases as they need.

Additionally, we open the door to deploying GitOps systems like Flux, which enables users to install additional required Helm charts through GitOps. #116

@ytsarev
Copy link
Member

ytsarev commented Oct 16, 2023

In addition to the above, I tend to deprecate XServices and provide a separate dedicated abstraction per Service like Prometheus , Flux or ArgoCD as separate optional configurations to install

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants