Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

v1.17: Deprecate bz2/gzip/none/tar snapshot compression types (backport of #33484) #33492

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Oct 2, 2023

This is an automatic backport of pull request #33484 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

@mergify mergify bot assigned steviez Oct 2, 2023
These options are now disallowed on the command line for
solana-validator and solana-ledger-tool, which effectively means no more
snapshots will be created with this types in normal usecases. However,
support for reading the deprecated types is still in place.

(cherry picked from commit 73e9e6d)
@steviez steviez force-pushed the mergify/bp/v1.17/pr-33484 branch from 59585c4 to 533ae74 Compare October 3, 2023 16:49
@steviez
Copy link
Contributor

steviez commented Oct 3, 2023

Force pushed to rebase on top of v1.17 branch

@codecov
Copy link

codecov bot commented Oct 3, 2023

Codecov Report

Merging #33492 (533ae74) into v1.17 (3a49a00) will decrease coverage by 0.1%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##            v1.17   #33492     +/-   ##
=========================================
- Coverage    81.7%    81.7%   -0.1%     
=========================================
  Files         802      802             
  Lines      217786   217777      -9     
=========================================
- Hits       178110   178090     -20     
- Misses      39676    39687     +11     

@steviez steviez requested review from mvines and brooksprumo October 3, 2023 17:22
@steviez
Copy link
Contributor

steviez commented Oct 3, 2023

@mvines & @brooksprumo - Me adding the v1.17 label is my inherent support for backporting this. What do you all think; technically, I missed the v1.17 cut by a day but I think this is a low-risk change and will get rid of something that has been a support headache for a while

Copy link
Contributor

@brooksprumo brooksprumo left a comment

Choose a reason for hiding this comment

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

@mvines & @brooksprumo - Me adding the v1.17 label is my inherent support for backporting this. What do you all think; technically, I missed the v1.17 cut by a day but I think this is a low-risk change and will get rid of something that has been a support headache for a while

Yeah, I think it makes sense to have this in v1.17.

@brooksprumo brooksprumo requested a review from willhickey October 3, 2023 17:26
@steviez
Copy link
Contributor

steviez commented Oct 3, 2023

Given that Brooks already agrees with pushing this into v1.17, I'm going to take the "ask for forgiveness instead of permission" approach in regards to mvines + willhickey and assume they are fine with this too

@steviez steviez merged commit b90f08f into v1.17 Oct 3, 2023
@steviez steviez deleted the mergify/bp/v1.17/pr-33484 branch October 3, 2023 20:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants