-
Notifications
You must be signed in to change notification settings - Fork 824
Support simple dumping of Argo manifests #2338
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
Comments
I need this feature!! |
Re-opening this after having spent some time using Metaflow. I'm quite convinced now that there ought to be some way to integrate the state of my code base to what should be on the cluster. Right now, if one adds, then removes a The only way I can see to do this from the design of Metaflow is to keep some ability of the code state and allow something like ArgoCD to reconcile required updates. I see maybe even the To be clear, I think it's great one can run ad-hoc / development flows in a development cluster that gets cleaned up regularly, but in production we need a more declarative approach. |
Support a
--dump-manifests
flag or subcommand that only constructs the argo-workflows manifest(s).We use ArgoCD extensively for basically all our resources. I'm not very keen on using the existing deployment of argo-workflows create to deploy. I'd rather dump the manifests out into a kustomization app that ArgoCD then evaluates for actual deployment.
I know about the
--only-json
but it seems clear it's trying to access and modify cluster things:I could go down the road of providing proper access, but the point is I don't want it doing any of this; metaflow will eventually exist on our cluster thru Helm installation (happy to provide a PR for a helm chart pre-configured for on-prem installs later if this works out), and I only want the argo workflow manifest(s) which I will then feed to ArgoCD like all our other resources/services.
I'm happy to dedicate some time to implement this if it's not already and I just missed it.
I would be willing to contribute this feature with guidance from the community.
The text was updated successfully, but these errors were encountered: