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

Improve SHA query for different registries #493

Merged

Conversation

viroel
Copy link
Contributor

@viroel viroel commented Sep 28, 2023

It now compare with the proper registry before sending api query. For registry != from quay.io (e.g. quay.rdoproject.org) don't support 'filter_tag_name' but might support 'limit', so we can retrieve more older tags.

@viroel
Copy link
Contributor Author

viroel commented Sep 28, 2023

from results[1], using the proper api call for quay.io now, in content-provider job:
++ curl -s 'https://quay.io/api/v1/repository/openstack-k8s-operators/cinder-operator-bundle/tag/?onlyActiveTags=true&filter_tag_name=like:1b9a7de3d088'

It now compare with the proper registry before sending api query.
For registry != from quay.io (e.g. quay.rdoproject.org) don't support
'filter_tag_name' but might support 'limit', so we can retrieve more
older tags.
@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://review.rdoproject.org/zuul/buildset/31db556dcfc54464ab53425f8fb76dc5

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 06m 33s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 52m 31s
cifmw-crc-podified-edpm-baremetal FAILURE in 32m 52s

@viroel
Copy link
Contributor Author

viroel commented Sep 29, 2023

/test openstack-operator-build-deploy-kuttl

@viroel
Copy link
Contributor Author

viroel commented Sep 29, 2023

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://review.rdoproject.org/zuul/buildset/749ab5089f594915a52cde9959d3d056

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 31m 50s
podified-multinode-edpm-deployment-crc TIMED_OUT in 3h 08m 08s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 51m 46s

@stuggi
Copy link
Contributor

stuggi commented Sep 29, 2023

recheck

@stuggi
Copy link
Contributor

stuggi commented Sep 29, 2023

/test openstack-operator-build-deploy-kuttl

@cjeanner
Copy link
Contributor

/lgtm

content-provider job passed. Let's unblock CI :)

@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://review.rdoproject.org/zuul/buildset/e4ba701c0b694a278c1c683be247f83e

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 22m 30s
podified-multinode-edpm-deployment-crc TIMED_OUT in 3h 08m 24s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 54m 12s

@pablintino
Copy link
Contributor

recheck

@rebtoor rebtoor self-requested a review September 29, 2023 09:02
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 29, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rebtoor, viroel

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-merge-robot openshift-merge-robot merged commit b40eccb into openstack-k8s-operators:main Sep 29, 2023
3 checks passed
@viroel viroel deleted the quayio_filter branch September 29, 2023 13:53
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