Skip to content

DO NOT MERGE: Run tests with Zstd:chunked compression default#21903

Closed
mtrmac wants to merge 6 commits intocontainers:mainfrom
mtrmac:try-zstd_chunked-default
Closed

DO NOT MERGE: Run tests with Zstd:chunked compression default#21903
mtrmac wants to merge 6 commits intocontainers:mainfrom
mtrmac:try-zstd_chunked-default

Conversation

@mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Mar 1, 2024

Inspired by #20633 ; now that #21571 is succeeding, let’s see what happens in the more complex case.

Does this PR introduce a user-facing change?

do not merge this

@openshift-ci openshift-ci bot added release-note do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Mar 1, 2024
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 1, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mtrmac

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 1, 2024
@rhatdan
Copy link
Member

rhatdan commented Mar 1, 2024

WhooHoo Passed tests.

@github-actions
Copy link

github-actions bot commented Apr 1, 2024

A friendly reminder that this PR had no activity for 30 days.

@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch from 0e174ef to 7b39108 Compare April 3, 2024 16:21
@github-actions github-actions bot removed the stale-pr label Apr 4, 2024
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch from 7b39108 to 84b528f Compare July 17, 2024 17:41
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch from 84b528f to 2e43b0a Compare October 14, 2024 20:20
@mtrmac
Copy link
Contributor Author

mtrmac commented Nov 4, 2024

@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch 3 times, most recently from 6488a88 to 39a5f78 Compare November 28, 2024 20:56
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch 3 times, most recently from 683d90a to 29b9c44 Compare December 14, 2024 00:21
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch 3 times, most recently from bcd6040 to a41ef14 Compare January 7, 2025 18:40
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch 3 times, most recently from dafd485 to 48ed8d1 Compare January 14, 2025 21:13
@packit-as-a-service
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch 2 times, most recently from 94b7437 to 0a33fda Compare January 14, 2025 21:57
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch 2 times, most recently from 0926906 to 9cbd4af Compare January 21, 2025 20:54
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
The Go image is too old, at least in the CI images

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac mtrmac force-pushed the try-zstd_chunked-default branch from 9cbd4af to 0d4d473 Compare January 23, 2025 21:54
@mtrmac
Copy link
Contributor Author

mtrmac commented Feb 6, 2025

We now (e.g. with #25007 ) always run at least some tests with Zstd, and there are no immediate plans to switch to it as default.

@mtrmac mtrmac closed this Feb 6, 2025
@mtrmac mtrmac deleted the try-zstd_chunked-default branch February 6, 2025 21:56
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label May 8, 2025
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators May 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants