-
Notifications
You must be signed in to change notification settings - Fork 48
[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
[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 |
a2cf2ee to
2e117cc
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/db0d2ed7e31140aa90f720c45e50adbd ❌ telemetry-operator-multinode-cloudkitty FAILURE in 1h 14m 00s |
ci/vars-cloudkitty-tempest.yml
Outdated
| service_available.sg_core false | ||
| service_available.aodh false | ||
| service_available.cinder false | ||
| telemetry.prometheus_service_url "https://metric-storage-prometheus.openstack.svc.cluster.local:9090" | ||
| telemetry.ceilometer_polling_interval 120 | ||
| telemetry.prometheus_scrape_interval 30 | ||
| telemetry.alarm_threshold 50000000000 |
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.
| service_available.sg_core false | |
| service_available.aodh false | |
| service_available.cinder false | |
| telemetry.prometheus_service_url "https://metric-storage-prometheus.openstack.svc.cluster.local:9090" | |
| telemetry.ceilometer_polling_interval 120 | |
| telemetry.prometheus_scrape_interval 30 | |
| telemetry.alarm_threshold 50000000000 |
| cifmw_test_operator_tempest_include_list: | | ||
| ^tempest.*\[.*\bsmoke\b.*\] | ||
| cloudkitty_tempest_plugin.* | ||
| telemetry_tempest_plugin.* |
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.
| telemetry_tempest_plugin.* |
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.
Do we want the telemetry tests here or not? I'm OK with either, but I'm confused by the suggestion still being here.
57284e3 to
898f69d
Compare
|
Zuul encountered a syntax error while parsing its mapping values are not allowed in this context |
898f69d to
56ac70d
Compare
|
/retest telemetry-operator-build-deploy-kuttl |
|
@elfiesmelfie: The The following commands are available to trigger optional jobs: Use In response to this:
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-sigs/prow repository. |
|
/test telemetry-operator-build-deploy-kuttl |
| zuul: | ||
| projects: | ||
| github.com/openstack-k8s-operators/telemetry-operator: | ||
| src_dir: "{{ telemetry_operator_dir | default('telemetry-operator/') }}" |
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.
This is so that the playbook can be run locally.
529af33 to
9e56d12
Compare
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.
9e56d12 to
dbb3429
Compare
|
/test telemetry-operator-build-deploy-kuttl |
1 similar comment
|
/test telemetry-operator-build-deploy-kuttl |
|
recheck |
|
/test telemetry-operator-build-deploy-kuttl |
1 similar comment
|
/test telemetry-operator-build-deploy-kuttl |
|
|
||
| - name: Set the loki-operator version to pin the version | ||
| ansible.builtin.set_fact: | ||
| loki_operator_version: "v6.3.0" |
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.
Small thing. I see this variable actually doesn't influence what version will get installed and deploy-loki-for-ck.yaml has the 6.3.0 hardcoded. Can you add a comment somewhere here noting that? I don't want to get into a situation in the future, where we'll want to use a different version of the operator and we'll be surprised that just changing the version here doesn't achieve the goal.
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.
This is true. There should be some template rendered with the vars in it, if we're to use this.
I can address this in a follow-up so that we can merge this and get the test coverage.
I'll add a comment for now, and push a follow-up MR later today.
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.
Sounds good.
Loki-operator needs to be pinned to 6.3.0, because 6.3.1 has this issue: https://issues.redhat.com/browse/LOG-7752
dbb3429 to
54fb11e
Compare
|
/lgtm |
873139a
into
openstack-k8s-operators:main
Add in a job that deploys CloudKitty and runs tempest tests.
Depends-On: openstack-k8s-operators/install_yamls#1120
Depends-On: #796