thoth-station / thoth-application

Thoth-Station ArgoCD Applications
GNU General Public License v3.0
12 stars 22 forks source link

Migrate cronjobs to Argo CronWorkflows #654

Open pacospace opened 3 years ago

pacospace commented 3 years ago

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

Migrate cronjob in Thoth to CronWorkflow:

Describe the solution you'd like Use Argo Cronworkflow for flexibility, monitoring and extensibility: https://github.com/thoth-station/thoth-application/issues/429#issuecomment-727820282

Describe alternatives you've considered

Additional context

goern commented 3 years ago

/triage accepted

harshad16 commented 3 years ago

constraint: this would require workflow-controller to be deployed in other namespaces as well.

goern commented 3 years ago

/milestone 2021.07.19

goern commented 3 years ago

/milestone 2021.08.02

sesheta commented 3 years ago

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

/close

sesheta commented 3 years ago

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle rotten

sesheta commented 3 years ago

@sesheta: Closing this issue.

In response to [this](https://github.com/thoth-station/thoth-application/issues/654#issuecomment-905047287): >Rotten issues close after 30d of inactivity. >Reopen the issue with `/reopen`. >Mark the issue as fresh with `/remove-lifecycle rotten`. > >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
VannTen commented 2 years ago

constraint: this would require workflow-controller to be deployed in other namespaces as well.

Is a cluster-scoped controller not possible as an alternative ?

harshad16 commented 2 years ago

As we work on shared cluster, cluster-scoped controller are most of the time hard to get deployed.

VannTen commented 2 years ago

As we work on shared cluster, cluster-scoped controller are most of the time hard to get deployed.

Could operate-first offer that as a cluster-level service (aka, this would become part of the operate-first platform offer) ?

I don't mean technically, but would that work with the operate-first organization ?

harshad16 commented 2 years ago

Yes correct, operate-first platform can provide these cluster-scoped resources.

Gregory-Pereira commented 2 years ago

/cc @Gregory-Pereira

VannTen commented 2 years ago

So to resume the discussion yesterday, although operate-first could spin up an argo-workflow controller as a cluster service, we deploy to other environments (I didn't know that ^) where it would be more complicated.

Gregory-Pereira commented 2 years ago

Yes, I guess I should follow your example to document the result of our discussion. The issue prompting this discussion in the first place is that by migrating our Cronjobs to Argo Workflows, we would want the components to run in their correct namespaces. As such there we would need access to the Argo workflows controller in those namespaces. It is possible to deploy a single cluster scoped instance in operate first, which would eliminate the need to redeploy the controller in every namespace but this comes with its own issues, mainly that this is not reproduce-able in other non-operate-first environments, particularly our staging OCP4 environment. This would in turn decouple our deployment for our staging and production clusters which honestly seems like more trouble than its worth. As a result, it seems like a smarter idea to simply deploy an instance of the workflow controller in the namespace that dont currently have it (this would be a good first step). Let me know if I misunderstood something, or am missing anything.