Made some changes so this runs on 4.12+ cluster#64
Made some changes so this runs on 4.12+ cluster#64afouladi7 wants to merge 45 commits intorcarrata:mainfrom
Conversation
rcarrata
left a comment
rcarrata
left a comment
There was a problem hiding this comment.
Kindly rollback to use the OpenShift Pipeline checks again, adapt to the post_ci to use the main branch approach.
Thanks for your contributions @afouladi7!
| until: crds.resources|length > 0 | ||
| retries: 30 | ||
| delay: 10 | ||
| # - name: Adapt to the openshift_cluster_version LESS than 4.9 |
There was a problem hiding this comment.
@afouladi7 why we are removing these checks? I've tested in 4.13 and worked like a charm.
There was a problem hiding this comment.
Are you running this on BM? I tested this on both BM and RHPDS on 4.12.
There was a problem hiding this comment.
VMs (Hetzner + libvirt + 4.13) and I'll do it in RHPDS 4.11 as well :)
| namespace: "{{ ocp4_demo_pipeline_namespace }}" | ||
| type: Opaque | ||
|
|
||
| # - name: Get Service Account objects |
There was a problem hiding this comment.
I've fixed (check the main branch) grabbing directly the secret token itself, instead of grepping for the sa pipeline token. Shall we move to the main branch proposal and avoid to use the shell commands?
run without failure.
changing the ubi to 8.8
doc edits for ACS policy change and Fedora prerequisite installs
We cannot specify both keycloak and dex as SSO providers. When I tried enabling just keycloak the gitops-operator hit some segfault errors so switch to just using dex for now
Fixing ArgoCD CR: Cannot specify both keycloak and dex
Using Keycloak breaks the gitops operator as of v1.9.1
Removing update to add keycloak for now
Adding Quay to doc and status.sh
adding the argo dex fix
Tell it to ignore cert validation for the gogs url
Create trust_quay_from_another_cluster.md
adding charges from org
5 participants
Adding all these changes to ensure this demo still runs on 4.12+ cluster. Also tested on 4.13.