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

Policy table should handle omitted top-level targetRefs #2934

Closed
lobkovilya opened this issue Sep 13, 2024 · 0 comments · Fixed by #2935
Closed

Policy table should handle omitted top-level targetRefs #2934

lobkovilya opened this issue Sep 13, 2024 · 0 comments · Fixed by #2935
Assignees
Labels
kind/feature New feature triage/accepted The issue was reviewed and is complete enough to start working on it
Milestone

Comments

@lobkovilya
Copy link
Contributor

Description

Since Kuma 2.9 it's possible to create policy without the top-level targetRef:

type: MeshTimeout
name: mt1
mesh: default
spec:
  to:
    - targetRef:
        kind: MeshService
        name: redis

which is an equivalent of:

type: MeshTimeout
name: mt1
mesh: default
spec:
  targetRef:
    kind: Mesh
  to:
    - targetRef:
        kind: MeshService
        name: redis

In policy table Target ref we want to see Mesh instead of :
image

@lobkovilya lobkovilya added triage/pending This issue will be looked at on the next triage meeting kind/feature New feature labels Sep 13, 2024
@slonka slonka added the triage/accepted The issue was reviewed and is complete enough to start working on it label Sep 16, 2024
@slonka slonka added this to the 2.9.x milestone Sep 16, 2024
@slonka slonka removed the triage/pending This issue will be looked at on the next triage meeting label Sep 16, 2024
@johncowen johncowen self-assigned this Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature 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.

3 participants