Modify CI jobs to include application credentials installations#339
Modify CI jobs to include application credentials installations#339morenod wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/88873985cf374f93b41e587ce8ad517d ❌ openstack-meta-content-provider-master FAILURE in 19m 34s |
morenod
force-pushed
the
watcher_operator_appcred
branch
3 times, most recently
from
872cdf9 to
525eeaf
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/769018796f004f18aa6a60c1f923ce9c ❌ openstack-meta-content-provider-master NODE_FAILURE Node request 100-0008160421 failed in 0s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e1448f23cef54cdc86d81e8540e05654 ❌ openstack-meta-content-provider-master NODE_FAILURE Node request 100-0008160423 failed in 0s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/98f4139b27ec423f8fd013058a2a1eef ❌ openstack-meta-content-provider-master NODE_FAILURE Node request 100-0008160526 failed in 0s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/81e0772711214007a086539de2b310f8 ❌ openstack-meta-content-provider-master NODE_FAILURE Node request 100-0008160603 failed in 0s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/85ebe1dec50e42b68b52c081e9535a71 ❌ openstack-meta-content-provider-master FAILURE in 17m 02s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fdd6fb6e7fcc4194aa9ab0e3e859b803 ✔️ openstack-meta-content-provider-master SUCCESS in 2h 37m 59s |
morenod
force-pushed
the
watcher_operator_appcred
branch
2 times, most recently
from
4bbf39b to
d0572bc
Compare
|
Master Job appcreds passing:
|
morenod
force-pushed
the
watcher_operator_appcred
branch
2 times, most recently
from
2dc5980 to
ca6292e
Compare
.zuul.yaml
Outdated
| It will test current-podified control plane EDPM deployment with openstack watcher | ||
| master content. It deploys watcher using TLSe, and creates the certificates to use. | ||
| Neither Nova nor Cinder notifications are not enabled on this job. | ||
| No dedicated RabbitMQ notifications broker is enabled on this job. This job validates |
There was a problem hiding this comment.
I think this is wrong, watcher-operator-validation-epoxy is not validating deployment with AC.
.zuul.yaml
Outdated
| watcher-operator-validation qualification with OCP 4.16 | ||
| watcher-operator-validation qualification with OCP 4.16. | ||
| No dedicated RabbitMQ notifications broker is enabled on this job. This job validates | ||
| Watcher deployment with Application Credentials authentication. |
ci/README.md
Outdated
| | Job Name | OpenStack Version | OCP Version | Notifications | App Credentials | NFS Backend | | ||
| |----------|-------------------|-------------|---------------|-----------------|-------------| | ||
| | `watcher-operator-validation-master` | master | 4.18 | ✅ Yes | ✅ Yes | ✅ Yes | | ||
| | `watcher-operator-validation-epoxy` | antelop (all) - epoxy (watcher) | 4.18 | ❌ No | ❌ No | ❌ No | |
There was a problem hiding this comment.
The right OpenStack version name is antelope.
ci/README.md
Outdated
| |----------|-------------------|-------------|---------------|-----------------|-------------| | ||
| | `watcher-operator-validation-master` | master | 4.18 | ✅ Yes | ✅ Yes | ✅ Yes | | ||
| | `watcher-operator-validation-epoxy` | antelop (all) - epoxy (watcher) | 4.18 | ❌ No | ❌ No | ❌ No | | ||
| | `watcher-operator-validation-epoxy-ocp4-16` | antelop (all) - epoxy (watcher) | 4.16 | ❌ No | ❌ No | ❌ No | |
morenod
force-pushed
the
watcher_operator_appcred
branch
3 times, most recently
from
fed0dec to
21dfa39
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/17e5b4191ffc4e888f67c553b27c8783 ✔️ openstack-meta-content-provider-master SUCCESS in 1h 23m 26s |
morenod
force-pushed
the
watcher_operator_appcred
branch
from
21dfa39 to
45f3366
Compare
SeanMooney
left a comment
SeanMooney
left a comment
There was a problem hiding this comment.
this looks good to me too thanks for addign the ci readme
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Modified CI Jobs watcher-operator-validation-master: Now tests Application Credentials Documentation Added new Continuous Integration section to README.md referencing detailed CI documentation. Created ci/README.md with a comprehensive matrix table showing all CI jobs and their configurations (OpenStack version, OCP version, notifications, Application Credentials, NFS backend). Coverage Strategy This approach ensures we test all critical combinations with minimal job overhead: Master OpenStack: Tested with AC and notifications enabled Epoxy OpenStack: Tested without AC and notifications Different OCP versions: Covered through the epoxy-ocp4-16 job Kuttl tests: Already include AC rotation testing Assisted-By: claude Signed-off-by: morenod <dsanzmor@redhat.com>
morenod
force-pushed
the
watcher_operator_appcred
branch
from
45f3366 to
b81fb4c
Compare
|
New changes are detected. LGTM label has been removed. |
|
recheck |
4 participants
This PR adds comprehensive Application Credentials (AC) testing to the CI pipeline while minimizing the number of new jobs required.