-
Notifications
You must be signed in to change notification settings - Fork 26
Test opendev watcher master content in CI job #39
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
Test opendev watcher master content in CI job #39
Conversation
378b1f4 to
8398c75
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1730a260e8344041aca6f62018345698 ✔️ openstack-meta-content-provider-master SUCCESS in 54m 56s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f87171ca6a324ee6af825ea0ba59de9b ✔️ openstack-meta-content-provider-master SUCCESS in 2h 15m 49s |
8398c75 to
8fab0d4
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/be9fe455e2d7496ababc7079b8311b8c ✔️ openstack-meta-content-provider-master SUCCESS in 2h 18m 44s |
b4b9fb6 to
4b637fb
Compare
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/ci-framework#2658 is needed. |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/524b3ac564c747afb547af3e7cc3336d ✔️ openstack-meta-content-provider-master SUCCESS in 50m 42s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6425b0194e3343179c9ece4b28dcde08 ✔️ openstack-meta-content-provider-master SUCCESS in 53m 56s |
4b637fb to
427418e
Compare
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/ci-framework#2658 is needed. |
|
recheck |
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/ci-framework#2658 is needed. |
|
recheck |
351b547 to
a23ced4
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6885467cbedd4835a295de1655606d5f ✔️ openstack-meta-content-provider-master SUCCESS in 1h 57m 24s |
a23ced4 to
aede059
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/aec5f860a20e48ee9729c77bbbcb8c29 ✔️ openstack-meta-content-provider-master SUCCESS in 1h 59m 19s |
aede059 to
c9f4756
Compare
9964062 to
1510abd
Compare
| PATH: "{{ cifmw_path }}" | ||
| tasks: | ||
| - name: Fetch dlrn md5_hash from master DLRN repo | ||
| when: fetch_master_dlrn_hash | default(true) | bool |
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.
we should rename fetch_master_dlrn_hash var at some point, since if we provide a different watcher_dlrn_namespace, we could fetch from a different release too
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.
Good point, renamed the var to fetch_dlrn_hash.
.zuul.yaml
Outdated
|
|
||
| - job: | ||
| name: watcher-operator-base | ||
| override-checkout: main |
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.
why at job level? is zuul still defaulting to master? Just wondering that when we start to running against different ranches, we will need to always add the override-checkout
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.
Dropped it from all jobs.
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.
right, but we'll also need to see if some projects are require the branch-override...
1510abd to
2c3c5bb
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/51cdf39b90f3462cb205eafef77dfccf ✔️ openstack-meta-content-provider-master SUCCESS in 1h 30m 07s |
|
recheck |
2c3c5bb to
8d46bc7
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/20b6ee03d9554cf2a02cee22229ec316 ✔️ openstack-meta-content-provider-master SUCCESS in 2h 42m 58s |
8d46bc7 to
83ef22c
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bc6d1816b74440efbb44a13a4a7cc1f3 ✔️ openstack-meta-content-provider-master SUCCESS in 1h 38m 54s |
83ef22c to
edc6428
Compare
marios
left a comment
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
In order to test opendev watcher master content in CI job, this pr adds following things: * opendev-master-watcher-operator-pipeline project template which will run meta content provider from master and then Deploy EDPM job watcher-operator-validation-master from master content. It will help to test any opendev watcher patches with master edpm job. * It changes the existing job watcher-operator-validation (still deploying the antelope content) with update_containers: false to avoid using meta content provider containers. This change will help to test master content with antelope job as in downstream, we are going to backport lots of watcher master patches downstream. It will help to avoid breakage in advance. * It also updates the existing watcher-operator-validation and watcher-operator-kuttl job to use openstack-meta-content-provider-master as a job dependencies. Depends-On: https://review.opendev.org/c/openstack/watcher/+/938893 Depends-On: openstack-k8s-operators/ci-framework#2675 Related-Jira: https://issues.redhat.com/browse/OSPRH-13660 Signed-off-by: Chandan Kumar (raukadah) <raukadah@gmail.com>
edc6428 to
f9daa75
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/8b151ae6c83d4bd3b0c2df5bb1e90e3e ✔️ openstack-meta-content-provider-master SUCCESS in 1h 41m 20s |
|
recheck |
|
/lgtm |
|
/lgtm |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cescgina 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 |
marios
left a comment
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
7cabd03
into
openstack-k8s-operators:main
In a recent change [1], we introduced a self-referencial use of some vars in the `run-with-webhook` target. This change modifies the variable with the default urls slightly so the target is functional again. [1] openstack-k8s-operators#39
In a recent change [1], we introduced a self-referencial use of some vars in the `run-with-webhook` target. This change modifies the variable with the default urls slightly so the target is functional again. [1] #39
In order to test opendev watcher master content in CI job, this pr adds following things:
opendev-master-watcher-operator-pipeline project template which will run meta content provider from master and then Deploy EDPM job watcher-operator-validation-master from mastter content. It will help to test any opendev watcher patches with master edpm job.
It changes the existing job watcher-operator-validation (still deploying the antelope content) with update_containers: false to avoid using meta content provider containers. This change will help to test master content with antelope job as
in downstream, we are going to backport lots of watcher master patches downstream. It will help to avoid breakage in advance.
It also updates the existing watcher-operator-validation and watcher-operator-kuttl job to use openstack-meta-content-provider-master as a job dependencies.
Depends-On: https://review.opendev.org/c/openstack/watcher/+/938893
Depends-On: openstack-k8s-operators/ci-framework#2675
Test results: #39 (comment)
Related-Jira: https://issues.redhat.com/browse/OSPRH-13660