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

Refactor buildextend compression and add zip #3363

Merged
merged 1 commit into from
Apr 18, 2023

Conversation

baude
Copy link
Contributor

@baude baude commented Feb 16, 2023

buildextend --compress supports gzip as an alternate compression over xz using a key called "gzip" where the value is a bool. This is now refactored to a key called "compression" with a value of "gzip", "skip", or "zip" (see below). The "skip-compression" key is no longer applicable.

HyperV images which are used on Windows need to be compressed with zip because xz and gzip are not supported natively. Added zip support to buildextend and set the hyperv platform compression to "zip".

Supports:
* coreos/fedora-coreos-tracker#1411
* coreos/fedora-coreos-tracker#1421

@openshift-ci
Copy link

openshift-ci bot commented Feb 16, 2023

Hi @baude. Thanks for your PR.

I'm waiting for a coreos member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@baude baude changed the title jcjdjd Refactor buildextend compression and add zip Feb 16, 2023
@cgwalters
Copy link
Member

image

🤔 https://xkcd.com/1530/

Copy link
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

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

Looks good generally, just a couple small things.

src/cosalib/qemuvariants.py Outdated Show resolved Hide resolved
src/cosalib/qemuvariants.py Outdated Show resolved Hide resolved
@bgilbert
Copy link
Contributor

/ok-to-test

src/cosalib/qemuvariants.py Outdated Show resolved Hide resolved
src/cosalib/qemuvariants.py Outdated Show resolved Hide resolved
@baude baude force-pushed the zipcompression branch 2 times, most recently from c0eefde to e36e059 Compare April 12, 2023 14:37
buildextend --compress supports gzip as an alternate compression over
xz using a key called "gzip" where the value is a bool.  This is now
refactored to a key called "compression" with a value of "gzip", "skip",
or "zip" (see below).  The "skip-compression" key is no longer
applicable.

HyperV images which are used on Windows need to be compressed with zip
because xz and gzip are not supported natively.  Added zip support to
buildextend and set the hyperv platform compression to "zip".

Supports:
  * coreos/fedora-coreos-tracker#1411
  * coreos/fedora-coreos-tracker#1424

Signed-off-by: Brent Baude <[email protected]>
@bgilbert
Copy link
Contributor

Deferring merge until after Tuesday's releases, out of an abundance of caution.

@bgilbert bgilbert added the hold waiting on something label Apr 14, 2023
@bgilbert
Copy link
Contributor

/retest

@bgilbert bgilbert removed the hold waiting on something label Apr 18, 2023
@bgilbert bgilbert merged commit 56f5334 into coreos:main Apr 18, 2023
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