-
Notifications
You must be signed in to change notification settings - Fork 10
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 software template to add gitlab integration #92
Conversation
ded0335
to
63f0616
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few questions; looks very good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Is there any related PR to be opened to the orchestrator-helm-operator repo? Maybe the tekton https://github.com/parodos-dev/orchestrator-helm-operator/blob/main/helm-charts/orchestrator/templates/tekton-pipeline.yaml#L37
I opened this PR to the operator about the tekton pipline, do we need to update that pipeline with every change to the templates/operator? I haven't done so for the create Quay story |
I guess not but I was wondering if GH and GL had different behaviour when we want to fetch and push or if all the tokens are passed correctly |
On rhdh I tested e2e by adding the gitlab integration to the app-config-rhdh-auth. I haven't tested my operator changes, but that is not related to the veracity of the templates changes |
With gitops enabled? So the tekton pipeline was executed fine and argoCD deployed the workflow on the cluster as expected as well?
yes, that's unrelated to the pipelines, just the deployment of RHDH |
I may have not tested that part to completion. Maybe we should hold until |
5651b72
to
3c22f80
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need some changes for better UX and avoid manual change of URL or secret name if on GH or GL
Is the scope limited to the basic workflow?
scaffolder-templates/skeletons/basic-workflow-skeleton/catalog-info.yaml
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as basic and advanced workflow templates were E2E tested, it's good to go for me
convertToFlat: false | ||
targetPath: workflow | ||
- id: addCIPipeline | ||
name: Adding the gitlab pipeline for the Workflow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
generally speaking, we wanted shorten titles for steps (more concise so they fit into the screen)
Closes https://issues.redhat.com/browse/FLPATH-1823
New Software Template to allow creating a Advanced and basic workflows with Gitlab integration (instead of github).
The user will provide as input the gitlab group, instance (host), and new project name of their choosing,
the workflow will use the software template to create the new software project in the provided gitlab location,
It will also create a new API token and gitlab variable to run a CI pipeline.
Actual Code changes:
adding the gitlab-workflow/ dir, gitlab-workflow/template.yaml, build/.ci-gitlab.yml, catalog-info.yaml files
There were some directory tree changes