Skip to content

[release-1.29] adding e2e test for downstream baggage telemetry#6803

Closed
istio-testing wants to merge 3 commits intoistio:release-1.29from
istio-testing:cherry-pick-6796-to-release-1.29
Closed

[release-1.29] adding e2e test for downstream baggage telemetry#6803
istio-testing wants to merge 3 commits intoistio:release-1.29from
istio-testing:cherry-pick-6796-to-release-1.29

Conversation

@istio-testing
Copy link
Copy Markdown
Collaborator

This is an automated cherry-pick of #6796

@istio-testing istio-testing requested a review from a team as a code owner January 29, 2026 15:34
@istio-testing istio-testing added the release-notes-none Indicates a PR that does not require release notes. label Jan 29, 2026
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 29, 2026
@keithmattix keithmattix added cherrypick/release-1.29 Set this label on a PR to auto-merge it to the release-1.29 branch do-not-merge/hold Block automatic merging of a PR. and removed cherrypick/release-1.29 Set this label on a PR to auto-merge it to the release-1.29 branch labels Jan 29, 2026
@keithmattix
Copy link
Copy Markdown
Contributor

Can come after the rc

@PetrMc PetrMc mentioned this pull request Jan 30, 2026
96 tasks
Copy link
Copy Markdown

@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.

LGTM, let's run the test again and check if we need to do something

@fjglira
Copy link
Copy Markdown

fjglira commented Feb 24, 2026

/retest

@PetrMc PetrMc removed the do-not-merge/hold Block automatic merging of a PR. label Mar 2, 2026
@keithmattix
Copy link
Copy Markdown
Contributor

/retest

@PetrMc
Copy link
Copy Markdown
Contributor

PetrMc commented Mar 17, 2026

@grnmeira looking at the error:
Got:

k8s.deployment.name=ratings-v1,service.name=ratings,service.version=version-1,app.version=v1,k8s.namespace.name=default,k8s.cluster.name=server-cluster,k8s.instance.name=ratings-v1-84975bc778-pxz2w

Want:

k8s.deployment.name=ratings-v1,k8s.namespace.name=default,k8s.cluster.name=server-cluster,service.name=ratings,service.version=version-1,app.name=ratings,app.version=v1,k8s.instance.name=ratings-v1-84975bc778-pxz2w

it seems (a) missing fields, (b) different order. how those two are addressed in master?

@fjglira
Copy link
Copy Markdown

fjglira commented Mar 18, 2026

/retest

@istio-testing
Copy link
Copy Markdown
Collaborator Author

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
test-arm-arm64_proxy_release-1.29 506cd3d link true /test test-arm-arm64
test-asan_proxy_release-1.29 506cd3d link true /test test-asan
test_proxy_release-1.29 506cd3d link true /test test
Details

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-sigs/prow repository. I understand the commands that are listed here.

@grnmeira
Copy link
Copy Markdown
Contributor

@PetrMc added a manual cherry-pick here: #6914

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/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants