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

fix(meshtrafficpermission): nil pointer for autoreachableservice when no top targetRef (backport of #12152) #12160

Closed
wants to merge 1 commit into from

Conversation

kumahq[bot]
Copy link
Contributor

@kumahq kumahq bot commented Dec 3, 2024

Automatic cherry-pick of #12152 for branch release-2.6

Generated by action

cherry-picked commit ec8b2bb
⚠️ ⚠️ ⚠️ Conflicts happened when cherry-picking! ⚠️ ⚠️ ⚠️

On branch release-2.6
Your branch is up to date with 'origin/release-2.6'.

You are currently cherry-picking commit ec8b2bb5d.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   pkg/plugins/policies/meshtrafficpermission/graph/reachable_services_graph.go
	modified:   pkg/plugins/policies/meshtrafficpermission/graph/reachable_services_graph_test.go

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   pkg/plugins/policies/meshtrafficpermission/graph/backends/reachable_backend_refs_graph.go
	deleted by us:   pkg/plugins/policies/meshtrafficpermission/graph/reachable_backend_refs_graph_test.go

@kumahq kumahq bot added the conflict label Dec 3, 2024
@kumahq kumahq bot requested a review from a team as a code owner December 3, 2024 17:01
@kumahq kumahq bot added the release-2.6 label Dec 3, 2024
@kumahq kumahq bot requested review from lobkovilya and lukidzi and removed request for a team December 3, 2024 17:01
@lukidzi lukidzi closed this Dec 3, 2024
@bartsmykla bartsmykla deleted the chore/backport-release-2.6-12152 branch December 11, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant