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

New repo: argocd-pull-integration #173

Closed
chenz4027 opened this issue Oct 25, 2022 · 10 comments
Closed

New repo: argocd-pull-integration #173

chenz4027 opened this issue Oct 25, 2022 · 10 comments
Assignees

Comments

@chenz4027
Copy link
Contributor

We would like to create an additional repo argocd-pull-integration for enhancing ArgoCD and OCM integration with the pull model resource delivery mechanism. Eventually, we are hoping to donate this repo to the ArgoCD Lab organization https://github.com/argoproj-labs by going through the process provided in https://github.com/argoproj/argoproj/blob/master/community/ecosystem-projects.md

Related issue: open-cluster-management-io/ocm#70
Design doc: https://docs.google.com/document/d/1X1Sce55bGUsyFCJkcefb8E4_K7tcpwRTwKD8aXnKS7Q/edit
Proposed repo content: https://github.com/mikeshng/argocd-pull-integration
Related videos: https://www.youtube.com/watch?v=eNDJy58gIJ0 and https://www.youtube.com/watch?v=SLdcNgs603I and https://www.youtube.com/watch?v=z7mEz8i91xM

/assign @qiujian16 @mikeshng

@mikeshng
Copy link
Member

+1

@qiujian16
Copy link
Member

qiujian16 commented Oct 26, 2022

could we clarify the diff from https://github.com/open-cluster-management-io/multicloud-integrations? Could we just rename the multicloud-integrations to argocd-integration? Or that will impact our plan to move it into argocd ecosystem?

@mikeshng
Copy link
Member

Hi @qiujian16

could we clarify the diff from https://github.com/open-cluster-management-io/multicloud-integrations?

The main difference is we are not planning to migrate that repo to the Argo lab org. The newly proposed repo is hopefully destined to be migrated. If the ArgoCD community refuses the new repo then we will merge the new proposed repo into the existing multicloud-integrations repo.

Could we just rename the multicloud-integrations to argocd-integration?

I think we always meant for that repo to have more integrations and not just ArgoCD. We are also hoping to get the argocd-pull-integration repo accepted into the Argo lab org first and then we can evaluate what else make sense to migrate over.

Or that will impact our plan to move it into argocd ecosystem?

Yes, exactly. We think it will impact our plan to move to Argocd ecosystem. We want to start with the less Openshift centric pull controller first.

Once we are successfully onboarded to the argocd ecosystem, we can archive the argocd-pull-integration repo.

@qiujian16
Copy link
Member

ok, that makes sense. Thanks. I am on board.

@mikeshng
Copy link
Member

Forgot to mention the OWNERS file can start with this one https://github.com/mikeshng/argocd-pull-integration/blob/main/OWNERS

If you think that is too many users then:

approvers:
- jnpacker
- mikeshng
- xiangjingli
reviewers:
- jnpacker
- mikeshng
- xiangjingli

Thanks.

@mikeshng
Copy link
Member

mikeshng commented Nov 3, 2022

@qiujian16 could you please create the repo whenever you have some cycles? Thanks. CC @xiangjingli

@qiujian16
Copy link
Member

my bad, will do it today..

@qiujian16
Copy link
Member

https://github.com/open-cluster-management-io/argocd-pull-integration created. @mikeshng would you help to finish the logistic part? owner files, security.md, dco github action etc..and close this one when they are done. Thanks!

@mikeshng
Copy link
Member

mikeshng commented Nov 7, 2022

Done as required.

/close

@openshift-ci
Copy link

openshift-ci bot commented Nov 7, 2022

@mikeshng: Closing this issue.

In response to this:

Done as required.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci openshift-ci bot closed this as completed Nov 7, 2022
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

No branches or pull requests

3 participants