Skip to content
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

Return proper full hash for component repos #2189

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

raukadah
Copy link
Contributor

@raukadah raukadah commented Aug 1, 2024

In Component repos, we dump current-podified hash. We need the same hash needs to be set during hash info set_fact. So the content provider this hash to dependendent job otherwise it will fail.

As a pull request owner and reviewers, I checked that:

  • Appropriate testing is done and actually running

Copy link
Contributor

openshift-ci bot commented Aug 1, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

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/acabe132f2a04e15bf469c3881b5e599

openstack-k8s-operators-content-provider FAILURE in 4m 38s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ adoption-standalone-to-crc-ceph-provider SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 38s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 02s
cifmw-molecule-build_openstack_packages FAILURE in 4m 17s
cifmw-molecule-repo_setup FAILURE in 4m 23s

@raukadah raukadah force-pushed the component_hash branch 2 times, most recently from d937006 to ff6987b Compare August 2, 2024 01:59
In Component repos, we dump current-podified hash. We need the same hash
needs to be set during hash info set_fact. So the content provider this
hash to dependendent job otherwise it will fail.

Signed-off-by: Chandan Kumar <[email protected]>
@pablintino
Copy link
Collaborator

/approve

Copy link
Contributor

openshift-ci bot commented Aug 5, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pablintino

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Aug 5, 2024
@rebtoor
Copy link
Contributor

rebtoor commented Aug 5, 2024

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Aug 5, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit dee1f78 into main Aug 5, 2024
6 of 7 checks passed
@openshift-merge-bot openshift-merge-bot bot deleted the component_hash branch August 5, 2024 08:03
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.

3 participants