Skip to content

Cirrus: Improve test synchronization with c/skopeo #1202

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

Merged
merged 1 commit into from
Apr 26, 2021

Conversation

cevich
Copy link
Member

@cevich cevich commented Apr 16, 2021

The test_skopeo_task must be substantially duplicated between this
repository and containers/skopeo. Make this easier to maintain by
implementing a runner script to contain any/all differences. This
allows the test_skopeo_task definition to remain 100%
identical across the two repositories.

Signed-off-by: Chris Evich cevich@redhat.com

@cevich cevich force-pushed the mirror_skopeo_changes branch 2 times, most recently from 9c760d0 to 1d06cae Compare April 16, 2021 18:32
@cevich
Copy link
Member Author

cevich commented Apr 16, 2021

Need to wait for containers/skopeo#1249 to merge, than can strip the 'DO NOT MERGE' commit from here.

The `test_skopeo_task` must be substantially duplicated between this
repository and containers/skopeo.  Make this easier to maintain by
implementing a runner script to contain any/all differences.  This
allows the `test_skopeo_task` definition to remain 100%
identical across the two repositories.

Signed-off-by: Chris Evich <cevich@redhat.com>
@cevich cevich force-pushed the mirror_skopeo_changes branch from 1d06cae to 8363e0d Compare April 21, 2021 14:47
@cevich cevich marked this pull request as ready for review April 21, 2021 14:48
@cevich
Copy link
Member Author

cevich commented Apr 21, 2021

This is ready.

@rhatdan
Copy link
Member

rhatdan commented Apr 21, 2021

LGTM
@vrothberg @mtrmac PTAL

@TomSweeneyRedHat
Copy link
Member

LGTM

@rhatdan rhatdan merged commit d319da1 into containers:master Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants