Skip to content

Conversation

@ammachado
Copy link
Contributor

What does this PR do?

This PR enables ArgoCD's Progressive Syncs feature on the Openshift GitOps operator.

This feature enables basic application deployment ordering by the ArgoCD controller, when specified.

Checklist

  • You have completed the described test plan and included screenshots in the PR or comments showing the results
  • Relevant documentation has been updated
  • You have squashed commits (including commits to test from your branch) to be logical and reduce unnecessary commits

Test Plan

Deploy the AI Accelerator to an Openshift cluster and then check if the openshift-gitops ArgoCD instance has the parameter added:

$ oc get argocd/openshift-gitops -n openshift-gitops -o jsonpath='{.spec.applicationSet.extraArguments[0]}'
--enable-progressive-syncs

There is no changes on ArgoCD's UI or current application deployment.

Documentation

@ammachado ammachado requested a review from a team as a code owner September 18, 2025 18:11
@strangiato
Copy link
Member

@ammachado is progressive sync turned on for "new" argo instances these days?

If it is, I would say let's keep it in the base.

If not, maybe we configure it as a component. That would make it easier for people to choose to use it or not in the future.

@ammachado
Copy link
Contributor Author

No, @strangiato, progressive syncs is still an "alpha" feature, so its not enabled by default on ArgoCD, but usage is opt-in, requiring explicit configuration on the ApplicationSets.

So, should I move it to a different component?

@strangiato
Copy link
Member

Yeah, let's make it a component so it is easy to disable if a user wants to

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@ammachado ammachado force-pushed the argocd-progressive-sync branch from 83cc36a to 7a77bb7 Compare October 2, 2025 17:08
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 this pull request may close these issues.

2 participants