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

test/system: Bump secondary fedora image from 29 to 32 #780

Merged

Conversation

HarryMichal
Copy link
Member

The fedora-toolbox:32 image is the first of images in the renamed
toolbox image repository[0]. With the change we can drop the
pull_image_old() function because it was kept only for the old image.

[0] #615

@HarryMichal HarryMichal added 6. Minor Change Should not cause breakage 3. Refinement Reduces technical debt of the codebase 2. Testing Related to testing of Toolbox - unit, system,.. labels May 26, 2021
HarryMichal added a commit to HarryMichal/toolbox that referenced this pull request May 26, 2021
The fedora-toolbox:32 image is the first of images in the renamed
toolbox image repository[0]. With the change we can drop the
pull_image_old() function because it was kept only for the old image.

[0] containers#615

containers#780
@HarryMichal HarryMichal force-pushed the test/system/bump-minimal-fedora-image branch from 758f829 to c6eb3ee Compare May 26, 2021 22:39
@softwarefactory-project-zuul
Copy link

Build failed.

HarryMichal added a commit to HarryMichal/toolbox that referenced this pull request May 28, 2021
The fedora-toolbox:32 image is the first of images in the renamed
toolbox image repository[0]. With the change we can drop the
pull_image_old() function because it was kept only for the old image.

[0] containers#615

containers#780
@HarryMichal HarryMichal force-pushed the test/system/bump-minimal-fedora-image branch from c6eb3ee to 24ff3f9 Compare May 28, 2021 09:43
@softwarefactory-project-zuul
Copy link

Build failed.

@HarryMichal
Copy link
Member Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

@HarryMichal
Copy link
Member Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

@HarryMichal
Copy link
Member Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

@TristanCacqueray
Copy link
Contributor

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

HarryMichal added a commit to HarryMichal/toolbox that referenced this pull request May 31, 2021
The fedora-toolbox:32 image is the first of images in the renamed
toolbox image repository[0]. With the change we can drop the
pull_image_old() function because it was kept only for the old image.

[0] containers#615

containers#780
@HarryMichal HarryMichal force-pushed the test/system/bump-minimal-fedora-image branch from 24ff3f9 to bf3c911 Compare May 31, 2021 09:58
The fedora-toolbox:32 image is the first of images in the renamed
toolbox image repository[0]. With the change we can drop the
pull_image_old() function because it was kept only for the old image.

Seems like newer version of ShellCheck checks the validity of variable
names (SC2153). This caused a false positive, so I silenced it.

[0] containers#615

containers#780
@softwarefactory-project-zuul
Copy link

Build failed.

@HarryMichal HarryMichal force-pushed the test/system/bump-minimal-fedora-image branch from bf3c911 to c5b9ea6 Compare May 31, 2021 10:12
@softwarefactory-project-zuul
Copy link

Build succeeded.

@HarryMichal HarryMichal merged commit a24c2f6 into containers:main May 31, 2021
@HarryMichal HarryMichal deleted the test/system/bump-minimal-fedora-image branch May 31, 2021 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. Testing Related to testing of Toolbox - unit, system,.. 3. Refinement Reduces technical debt of the codebase 6. Minor Change Should not cause breakage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants