Skip to content

Conversation

@raukadah
Copy link
Contributor

@raukadah raukadah commented Jan 16, 2025

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

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1730a260e8344041aca6f62018345698

✔️ openstack-meta-content-provider-master SUCCESS in 54m 56s
watcher-operator-validation-master RETRY_LIMIT in 0s
✔️ noop SUCCESS in 0s
watcher-operator-validation RETRY_LIMIT in 2s
watcher-operator-kuttl RETRY_LIMIT in 2s

@raukadah
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f87171ca6a324ee6af825ea0ba59de9b

✔️ openstack-meta-content-provider-master SUCCESS in 2h 15m 49s
✔️ watcher-operator-validation-master SUCCESS in 1h 21m 15s
✔️ noop SUCCESS in 0s
watcher-operator-validation FAILURE in 18m 35s
watcher-operator-kuttl RETRY_LIMIT in 15m 26s

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/be9fe455e2d7496ababc7079b8311b8c

✔️ openstack-meta-content-provider-master SUCCESS in 2h 18m 44s
✔️ watcher-operator-validation-master SUCCESS in 1h 21m 33s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-validation SUCCESS in 1h 21m 33s
watcher-operator-kuttl RETRY_LIMIT in 16m 14s

@raukadah raukadah force-pushed the zuul_job_main branch 2 times, most recently from b4b9fb6 to 4b637fb Compare January 20, 2025 06:15
@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#2658 is needed.

@raukadah
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/524b3ac564c747afb547af3e7cc3336d

✔️ openstack-meta-content-provider-master SUCCESS in 50m 42s
watcher-operator-validation-master RETRY_LIMIT in 0s
✔️ noop SUCCESS in 0s
watcher-operator-validation RETRY_LIMIT in 0s
watcher-operator-kuttl RETRY_LIMIT in 0s

@raukadah
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6425b0194e3343179c9ece4b28dcde08

✔️ openstack-meta-content-provider-master SUCCESS in 53m 56s
watcher-operator-validation-master RETRY_LIMIT in 0s
✔️ noop SUCCESS in 0s
watcher-operator-validation RETRY_LIMIT in 0s
watcher-operator-kuttl RETRY_LIMIT in 2s

@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#2658 is needed.

@raukadah
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#2658 is needed.

@raukadah
Copy link
Contributor Author

recheck

@raukadah raukadah force-pushed the zuul_job_main branch 2 times, most recently from 351b547 to a23ced4 Compare January 22, 2025 03:55
@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6885467cbedd4835a295de1655606d5f

✔️ openstack-meta-content-provider-master SUCCESS in 1h 57m 24s
watcher-operator-validation-master FAILURE in 1h 03m 18s
✔️ noop SUCCESS in 0s
watcher-operator-validation FAILURE in 1h 05m 09s
watcher-operator-kuttl RETRY_LIMIT in 20m 59s

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/aec5f860a20e48ee9729c77bbbcb8c29

✔️ openstack-meta-content-provider-master SUCCESS in 1h 59m 19s
watcher-operator-validation-master FAILURE in 1h 06m 26s
✔️ noop SUCCESS in 0s
watcher-operator-validation FAILURE in 1h 03m 29s
watcher-operator-kuttl RETRY_LIMIT in 22m 24s

PATH: "{{ cifmw_path }}"
tasks:
- name: Fetch dlrn md5_hash from master DLRN repo
when: fetch_master_dlrn_hash | default(true) | bool
Copy link
Contributor

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

Copy link
Contributor Author

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
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor

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...

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/51cdf39b90f3462cb205eafef77dfccf

✔️ openstack-meta-content-provider-master SUCCESS in 1h 30m 07s
watcher-operator-validation-master FAILURE in 1h 18m 58s
✔️ noop SUCCESS in 0s
watcher-operator-validation FAILURE in 1h 18m 13s
✔️ watcher-operator-kuttl SUCCESS in 35m 45s

@raukadah
Copy link
Contributor Author

raukadah commented Feb 4, 2025

recheck

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/20b6ee03d9554cf2a02cee22229ec316

✔️ openstack-meta-content-provider-master SUCCESS in 2h 42m 58s
watcher-operator-validation-master FAILURE in 1h 28m 22s
✔️ noop SUCCESS in 0s
watcher-operator-validation FAILURE in 1h 33m 02s
✔️ watcher-operator-kuttl SUCCESS in 39m 37s

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bc6d1816b74440efbb44a13a4a7cc1f3

✔️ openstack-meta-content-provider-master SUCCESS in 1h 38m 54s
watcher-operator-validation-master FAILURE in 1h 25m 38s
✔️ noop SUCCESS in 0s
watcher-operator-validation FAILURE in 1h 26m 50s
✔️ watcher-operator-kuttl SUCCESS in 39m 02s

Copy link
Contributor

@marios marios left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Feb 5, 2025
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>
@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/8b151ae6c83d4bd3b0c2df5bb1e90e3e

✔️ openstack-meta-content-provider-master SUCCESS in 1h 41m 20s
✔️ watcher-operator-validation-master SUCCESS in 1h 29m 21s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-validation SUCCESS in 1h 25m 24s
watcher-operator-kuttl RETRY_LIMIT in 32m 53s

@raukadah
Copy link
Contributor Author

raukadah commented Feb 5, 2025

recheck

@amoralej
Copy link
Contributor

amoralej commented Feb 7, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Feb 7, 2025
@marios
Copy link
Contributor

marios commented Feb 11, 2025

/lgtm

@cescgina
Copy link
Contributor

/approve

@openshift-ci
Copy link

openshift-ci bot commented Feb 11, 2025

[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

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Contributor

@marios marios left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-merge-bot openshift-merge-bot bot merged commit 7cabd03 into openstack-k8s-operators:main Feb 11, 2025
6 checks passed
@raukadah raukadah deleted the zuul_job_main branch February 11, 2025 10:32
cescgina added a commit to cescgina/watcher-operator that referenced this pull request Feb 11, 2025
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
openshift-merge-bot bot pushed a commit that referenced this pull request Feb 11, 2025
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants