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

utility: Fixes backfillreplication script to use manifest blobs (PROJQUAY-2218) #826

Merged
merged 1 commit into from
Jul 9, 2021

Conversation

ibazulic
Copy link
Member

@ibazulic ibazulic commented Jul 8, 2021

utility: Fixes backfillreplication script to use manifest blobs (PROJQUAY-2218)

Fixes an issue where the backfill replication script was using the image table and not the manfiestblob table which is being used since Quay 3.4.x exclusively.
JIRA: https://issues.redhat.com/browse/PROJQUAY-2218

@app-sre-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@ibazulic ibazulic changed the title Fixes backfillreplication script to use manifest blobs utility: Fixes backfillreplication script to use manifest blobs (PROJQUAY-2218) Jul 8, 2021
@ibazulic ibazulic force-pushed the fix-backfill-replication-worker branch 3 times, most recently from c19591b to 6f89900 Compare July 8, 2021 17:49
…QUAY-2218)

Fixes an issue where the backfill replication script was using the `image` table and not the `manfiestblob` table which is being used since Quay 3.4.x exclusively.
JIRA: https://issues.redhat.com/browse/PROJQUAY-2218
@ibazulic ibazulic force-pushed the fix-backfill-replication-worker branch from 6f89900 to 127f514 Compare July 8, 2021 17:51
@ibazulic
Copy link
Member Author

ibazulic commented Jul 9, 2021

@kleesc can I get a quick review of this? Thanks!

Copy link
Member

@kleesc kleesc left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for doing this.

@ibazulic
Copy link
Member Author

ibazulic commented Jul 9, 2021

LGTM. Thanks for doing this.

No problem. I don't have write permissions on Quay so I can't merge it. Can you do it? Thanks!

@kleesc kleesc merged commit 6977730 into quay:master Jul 9, 2021
@ibazulic ibazulic deleted the fix-backfill-replication-worker branch July 9, 2021 15:00
harishsurf added a commit to harishsurf/quay that referenced this pull request Jul 22, 2021
…QUAY-2273) (quay#826)

Fixes an issue where the backfill replication script was using the `image` table and not the `manfiestblob` table which is being used since Quay 3.4.x exclusively.
JIRA: https://issues.redhat.com/browse/PROJQUAY-2218

(cherry picked from commit 6977730)
harishsurf added a commit that referenced this pull request Jul 22, 2021
…QUAY-2273) (#826) (#845)

Fixes an issue where the backfill replication script was using the `image` table and not the `manfiestblob` table which is being used since Quay 3.4.x exclusively.
JIRA: https://issues.redhat.com/browse/PROJQUAY-2218

(cherry picked from commit 6977730)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants