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

Merging of policies results in not applying policy on some outbound #6426

Closed
jakubdyszkiewicz opened this issue Mar 31, 2023 · 1 comment · Fixed by #6460
Closed

Merging of policies results in not applying policy on some outbound #6426

jakubdyszkiewicz opened this issue Mar 31, 2023 · 1 comment · Fixed by #6460
Assignees
Labels
kind/bug A bug triage/accepted The issue was reviewed and is complete enough to start working on it

Comments

@jakubdyszkiewicz
Copy link
Contributor

What happened?

Reported by Brent Yarger. Apparently when applied this policy

apiVersion: kuma.io/v1alpha1
kind: MeshRetry
metadata:
  name: retry-all
  namespace: kong-mesh-system
  labels:
    kuma.io/mesh: default 
spec:
  targetRef:
    kind: Mesh
  to:
    - targetRef:
        kind: Mesh
      default:
        http:
          numRetries: 1
          backOff:
            baseInterval: 5s
            maxInterval: 1m
          retryOn:
            - "5xx"

and

apiVersion: kuma.io/v1alpha1
kind: MeshRetry
metadata:
  name: retry-echo-to-mockserver
  namespace: kong-mesh-system
  labels:
    kuma.io/mesh: default # optional, defaults to `default` if unset
spec:
  targetRef:
    kind: MeshService
    name: echo-service_default_svc_8080
  to:
    - targetRef:
        kind: MeshService
        name: mockserver_mockserver_svc_1080
      default:
        http:
          numRetries: 5
          backOff:
            baseInterval: 10s
          retryOn:
            - "5xx"

results in the other upstreams in the echo service sidecar get retry config removed.

@jakubdyszkiewicz jakubdyszkiewicz added triage/pending This issue will be looked at on the next triage meeting kind/bug A bug triage/needs-reproducing Someone else should try to reproduce this and removed triage/pending This issue will be looked at on the next triage meeting labels Mar 31, 2023
@github-actions github-actions bot added the triage/pending This issue will be looked at on the next triage meeting label Mar 31, 2023
@jakubdyszkiewicz jakubdyszkiewicz removed the triage/pending This issue will be looked at on the next triage meeting label Apr 3, 2023
@jakubdyszkiewicz jakubdyszkiewicz added triage/accepted The issue was reviewed and is complete enough to start working on it and removed triage/needs-reproducing Someone else should try to reproduce this labels Apr 4, 2023
@jakubdyszkiewicz
Copy link
Contributor Author

I confirm the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug triage/accepted The issue was reviewed and is complete enough to start working on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant