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

[Discussion] Unify GitSource CRD & PiplelineResource of type Git #749

Closed
sbose78 opened this issue Apr 11, 2019 · 8 comments
Closed

[Discussion] Unify GitSource CRD & PiplelineResource of type Git #749

sbose78 opened this issue Apr 11, 2019 · 8 comments
Labels
kind/question Issues or PRs that are questions around the project or a particular feature

Comments

@sbose78
Copy link
Contributor

sbose78 commented Apr 11, 2019

Hi,

I'm Shoubhik from the OpenShift Developer Console team.

The DevConsole operator team is using a CRD called GitSource which represents a Git repo. Here's how it looks like

https://github.com/redhat-developer/devconsole-operator/blob/master/deploy/crds/devconsole_v1alpha1_gitsource_crd.yaml

apiVersion: devconsole.openshift.io/v1alpha1
kind: GitSource
metadata:
  name: example-gitsource
spec:
  url: "https://github.com/sclorg/nodejs-ex"
  ref: master
  contextDir: /cmd/manager
  httpProxy: http://proxy.example.com
  httpsProxy: https://proxy.example.com
  noProxy: somedomain.com, otherdomain.com
  secretRef:
    name: mysecret
  flavor: github

We use this for in our suite of CRDs which also consists of Component CRD and GitSourceAnalysis CRD. The GitSource CRD has a controller which verifies the Git 'connection' for the repository and writes an assertive/negative to the status field.

The structure/spec has been inspired from the BuildConfig type and the reason we went for a new CRD altogether is because we wanted to store Git Source information in a way that we are able to use this outside the context of BuildConfig as well. Example, a user can import an application in OpenShift Developer Console and then launch it in Che in which case we could use the GitSource belonging to the associated app Component in Che for cloning code into Che ( and other Che to Git operations )

I feel the goal of PipelineResource of type 'git' and GitSource is similar - and should try unifying the effort :)

Let me know your thoughts!

@sbose78 sbose78 changed the title [Discussion] Unify GitSource CRD & Git Resource CRD [Discussion] Unify GitSource CRD & PiplelineResource of type Git Apr 11, 2019
@sbose78
Copy link
Contributor Author

sbose78 commented Apr 11, 2019

( edited the CRD type :) )

@dlorenc
Copy link
Contributor

dlorenc commented Apr 17, 2019

Thanks for opening this! Would you be interested in coming to one of the upcoming Tekton weekly calls to discuss some goals/ideas for unifying? I wholeheartedly agree with the idea of fewer CRDs :)

@sbose78
Copy link
Contributor Author

sbose78 commented Apr 17, 2019

Sure, I would love to, @dlorenc .

@bobcatfish
Copy link
Collaborator

Nice, I've added this to the agenda @sbose78 ! And here's the info on our working group meeting just in case you hadn't seen it yet. Talk to you soon!

@sbose78
Copy link
Contributor Author

sbose78 commented Apr 23, 2019

Sorry, will add this to my calendar and join the next one!

@vdemeester vdemeester added the kind/question Issues or PRs that are questions around the project or a particular feature label Nov 7, 2019
@bobcatfish
Copy link
Collaborator

@sbose78 is this something you're still interested in ? (Note that in #1673 we're redesigning PipelineResources)

@bobcatfish
Copy link
Collaborator

Gonna close this for now, please re-open if still interested!

@sbose78
Copy link
Contributor Author

sbose78 commented Jan 9, 2020

Sounds good, @bobcatfish !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/question Issues or PRs that are questions around the project or a particular feature
Projects
None yet
Development

No branches or pull requests

4 participants