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

Consider integration with Flux via a "Ytt Controller" #802

Open
ThomasVitale opened this issue Jan 30, 2023 · 8 comments
Open

Consider integration with Flux via a "Ytt Controller" #802

ThomasVitale opened this issue Jan 30, 2023 · 8 comments
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution enhancement This issue is a feature request

Comments

@ThomasVitale
Copy link

Describe the problem/challenge you have
I really like working with ytt and I usually use it together with kapp-controller. Still, I'm afraid many organisations might miss out on the great power of ytt because of the challenge to integrate it in their already-established GitOps workflow based on Flux. And that would limit a wider adoption of ytt.

Describe the solution you'd like
Flux is based on highly modular toolkit. In particular, the Source Controller provides support for different types of artefacts (Git, OCI, buckets), the Kustomize Controller provides support for Kustomize-based deployments, and Helm Controller for helm releases. Perhaps it could be an idea to introduce a Ytt Controller component in the GitOps Toolkit used by Flux. What do you think?

Anything else you would like to add:
In a recent community meeting, there was a conversation around integration possibilities with Flux and some of the resulting ideas are gathered in carvel-dev/kapp-controller#1048.


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@ThomasVitale ThomasVitale added carvel triage This issue has not yet been triaged for relevance enhancement This issue is a feature request labels Jan 30, 2023
@github-actions
Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label Mar 12, 2023
@ThomasVitale
Copy link
Author

Dear bot, please keep the issue open :)

@github-actions github-actions bot removed the stale This issue has had no activity for a while and will be closed soon label Mar 14, 2023
@github-actions
Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label Apr 23, 2023
@praveenrewar praveenrewar reopened this May 4, 2023
@praveenrewar praveenrewar reopened this May 10, 2023
@praveenrewar praveenrewar reopened this May 16, 2023
@praveenrewar praveenrewar removed the stale This issue has had no activity for a while and will be closed soon label May 16, 2023
@100mik
Copy link

100mik commented Jun 9, 2023

Thanks to Praveen for valiantly fighting off the bot, I am going to add a discussion tag here so that it so that we can have a truce 😆

@100mik 100mik added discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution and removed carvel triage This issue has not yet been triaged for relevance labels Jun 9, 2023
@100mik
Copy link

100mik commented Jun 9, 2023

I think it would be interesting to compare the pros and cons of having fine grained controllers vs having a "use flux to deploy a Carvel package" controller.
Maybe this is something we should bring up in the next community meeting and then summarise here 🤔

@ThomasVitale
Copy link
Author

ThomasVitale commented Jun 14, 2023

@100mik this is the community Ytt Controller I mentioned earlier that makes it possible to use ytt with Flux: https://projectsveltos.github.io/sveltos/ytt_extension/ Project: https://github.com/gianlucam76/ytt-controller

It works as a post-processor outside Flux. What I had in mind was a native ytt controller built using the GitOps Toolkit provided by the Flux project (similar to how the kustomize and helm controllers work).

@100mik
Copy link

100mik commented Jun 14, 2023

Would this mean we would contribute these bits to the Flux project directly?

@warroyo
Copy link

warroyo commented Oct 4, 2023

this would be a great addition to natively support in carvel. I recently created a similar issue here.

this would not be a contribution to the flux project but rather a carvel maintained controller that could reference flux objects. just like the community project mentioned above.

I would like to see both the kapp integration and the ytt controller make it into official carvel features

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution enhancement This issue is a feature request
Projects
Status: In Progress
Development

No branches or pull requests

4 participants