Skip to content

[release-1.29] add statsCompression field to MeshConfig API, deprecating per pod annotation#3640

Merged
istio-testing merged 3 commits intoistio:release-1.29from
istio-testing:cherry-pick-3628-to-release-1.29
Jan 27, 2026
Merged

[release-1.29] add statsCompression field to MeshConfig API, deprecating per pod annotation#3640
istio-testing merged 3 commits intoistio:release-1.29from
istio-testing:cherry-pick-3628-to-release-1.29

Conversation

@istio-testing
Copy link
Collaborator

This is an automated cherry-pick of #3628

…otation (#48051)

* if set to true Envoy applies HTTP compression for metrics / stats via brotli,
zstd or gzip if client sends appropriate `Accept-Encoding` header.

* deprecate `sidecar.istio.io/statsCompression` annotation

xref: istio/istio#48051
Signed-off-by: Christian Rohmann <christian.rohmann@inovex.de>
@istio-testing istio-testing requested a review from a team as a code owner January 21, 2026 23:06
@istio-testing istio-testing added the release-notes-none Indicates a PR that does not require release notes. label Jan 21, 2026
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 21, 2026
@fjglira
Copy link
Contributor

fjglira commented Jan 23, 2026

Waiting for TOC approval: https://istio.slack.com/archives/C018VK0T9JT/p1769079792259339

@fjglira fjglira added the do-not-merge/hold Block automatic merging of a PR. label Jan 26, 2026
Copy link
Contributor

@fjglira fjglira left a comment

Choose a reason for hiding this comment

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

Approved by TOC members during WG last meeting. Adding a hold until istio/istio#58717 on master is merged and cherry-pick is created

Manually edit generated files :gross:
@PetrMc
Copy link
Contributor

PetrMc commented Jan 27, 2026

@keithmattix / @fjglira - is there a reason for the hold label?

@frittentheke
Copy link
Contributor

@keithmattix / @fjglira - is there a reason for the hold label?

@PetrMc I suppose #3641 should be part of this backport here?

@frittentheke
Copy link
Contributor

@keithmattix when this one is merged and also updates in istio/istio accordingly (automator?) I could then get to finalizing istio/istio#58717 . The only thing left is the removal of the go.mod override because automator is currently stuck with CI failures I suppose.

@fjglira
Copy link
Contributor

fjglira commented Jan 27, 2026

Removing the hold, the master istio/istio is approved and ready to merge, and we need the update in the release branch to be able to run the test on cp PR when is created

@fjglira fjglira removed the do-not-merge/hold Block automatic merging of a PR. label Jan 27, 2026
@fjglira fjglira requested review from keithmattix and removed request for keithmattix January 27, 2026 14:40
@istio-testing istio-testing merged commit 3f3a4e4 into istio:release-1.29 Jan 27, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes-none Indicates a PR that does not require release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants