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

[release-1.19] Fix telemetry templates #48436

Closed
istio-testing opened this issue Dec 19, 2023 · 1 comment
Closed

[release-1.19] Fix telemetry templates #48436

istio-testing opened this issue Dec 19, 2023 · 1 comment
Assignees

Comments

@istio-testing
Copy link
Collaborator

Manual cherrypick required.

#47894 failed to apply on top of branch "release-1.19":

Applying: Fix telemetry templates
Using index info to reconstruct a base tree...
A	manifests/charts/istio-control/istio-discovery/templates/telemetryv2_1.20.yaml
A	manifests/charts/istio-control/istio-discovery/templates/telemetryv2_1.21.yaml
A	manifests/charts/istiod-remote/templates/telemetryv2_1.20.yaml
A	manifests/charts/istiod-remote/templates/telemetryv2_1.21.yaml
Falling back to patching base and 3-way merge...
Auto-merging manifests/charts/istiod-remote/templates/telemetryv2_1.17.yaml
Auto-merging manifests/charts/istiod-remote/templates/telemetryv2_1.16.yaml
CONFLICT (content): Merge conflict in manifests/charts/istiod-remote/templates/telemetryv2_1.16.yaml
Auto-merging manifests/charts/istio-control/istio-discovery/templates/telemetryv2_1.17.yaml
Auto-merging manifests/charts/istio-control/istio-discovery/templates/telemetryv2_1.16.yaml
CONFLICT (content): Merge conflict in manifests/charts/istio-control/istio-discovery/templates/telemetryv2_1.16.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Fix telemetry templates
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

@hanxiaop
Copy link
Member

I think this has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants