-
Notifications
You must be signed in to change notification settings - Fork 49
[zuul] Add a CloudKitty CI job #720
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
base: main
Are you sure you want to change the base?
[zuul] Add a CloudKitty CI job #720
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: elfiesmelfie The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Zuul encountered a syntax error while parsing its found duplicate anchor; first occurrence |
0f6903d to
2734457
Compare
|
Zuul encountered a syntax error while parsing its found undefined alias |
2734457 to
594af57
Compare
|
Zuul encountered a syntax error while parsing its found undefined alias |
594af57 to
4609689
Compare
|
Zuul encountered a syntax error while parsing its found undefined alias |
4609689 to
a7ad3f4
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a04ae0818bb6479dbc9f067e9c96ee05 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 32m 43s |
|
recheck |
a7ad3f4 to
b083962
Compare
|
Zuul encountered a syntax error while parsing its while scanning a simple key |
b083962 to
522ead5
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1b4fbfe0345c4bd9b1baf25bce920b50 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 31m 37s |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d480b4fa1b29468d857750e3e995f034 ❌ telemetry-operator-multinode-cloudkitty FAILURE in 1h 07m 15s |
1cc7fcf to
2c52232
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6f02549e96de45a0897ab3f328aa2f46 ❌ telemetry-operator-multinode-cloudkitty FAILURE in 17m 08s |
2c52232 to
f0582c3
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0181658ddc5342bbbcd3835f9876b740 ❌ telemetry-operator-multinode-cloudkitty FAILURE in 1h 10m 01s |
f0582c3 to
15750b1
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ba2c7f058c5a41228c7dc67d04cd1e2c ❌ telemetry-operator-multinode-cloudkitty FAILURE in 1h 10m 12s |
11cf1ce to
000e9f3
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
8b65140 to
fb2e08a
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
recheck |
To workaround this issue, the regular content provider is used and hardcoded images are used for deploying CK ref: openstack-k8s-operators#720 (comment)
c876fc5 to
847e944
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a7610f8bccc74eeba479ff81a6173627
|
|
recheck |
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/openstack-operator#1614 is needed. |
|
recheck |
3 similar comments
|
recheck |
|
recheck |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/5192dc27c15545e09f1ed6abafb6d891 ❌ telemetry-operator-multinode-cloudkitty FAILURE in 22m 12s |
86c8506 to
7059594
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fcd5e5d3b1054ae6b75c4fcd9d785676 ❌ telemetry-operator-multinode-cloudkitty FAILURE in 21m 59s |
7059594 to
16c4dd0
Compare
| - name: Deploy loki operator | ||
| ansible.builtin.shell: | ||
| cmd: | | ||
| oc apply -f {{ ansible_user_dir }}/{{ zuul.projects['github.com/openstack-k8s-operators/telemetry-operator'].src_dir }}/deploy-loki-for-ck.yaml |
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.
I need to move this file into ci/
|
recheck |
Add a pre_deploy hook to install loki-operator for CloudKitty deployment Add a pre_deploy hook to create the oscp kustomization to enable cloudkitty * add s3storageConfig secret * Enable metricsStorage in telemetry to provide prometheus for storage * Configure storageClass for CloudKitty in configure-cloudkitty hook Set the storageClass to crc-csi-hostpath-provisioner for CloudKitty deployment in CRC-based CI jobs. This prevents the need to increase PVC allocations in install_yamls by using the default storage class available in CRC, which allocates storage as required.
Loki-operator needs to be pinned to 6.3.0, because 6.3.1 has this issue: https://issues.redhat.com/browse/LOG-7752
16c4dd0 to
62ecbbc
Compare
Last change in CK
Depends-On: https://review.opendev.org/c/openstack/cloudkitty/+/961609
Depends-On: openstack-k8s-operators/install_yamls#1088
Depends-On: openstack-k8s-operators/tcib#328
Add cloudkitty-tempest package to openstack-tempest-all (Requires using tempest master image)
https://review.rdoproject.org/r/c/openstack/tempest-distgit/+/58015
Related change: openstack-k8s-operators/test-operator#340 (merged)
telemetry-operator change: #741
openstack-operator change:
Depends-On: openstack-k8s-operators/openstack-operator#1614
The openstack-operator is getting updated to reference the wrong version of telemetry-operator, which doesn't have CK available. This cifmw change will hopefully fix it. This is only an issue because we're using Depends-On to add a dependency on multiple changes from the same operator repo (i.e. telemetry-operator)
Add CK to patch-openstack-versions
Depends-On: infrawatch/feature-verification-tests#315
openstack-k8s-operators/ci-framework#3400