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.

Sign up for GitHub

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

Jump to bottom

Add more CEL Validation in BackendTrafficPolicy #2066

Closed
arkodg opened this issue Oct 25, 2023 · 2 comments · Fixed by #2110
Closed

Add more CEL Validation in BackendTrafficPolicy #2066

arkodg opened this issue Oct 25, 2023 · 2 comments · Fixed by #2110
Assignees
@slayer321

Comments

@arkodg
Copy link
Contributor

arkodg commented Oct 25, 2023

Description:

Describe the issue.

  • Disable targetRef.sectionName for now
  • Make sure consistentHash is non nil when type of load balancer is set to ConsistentHash

[optional Relevant Links:]

Any extra documentation required to understand the issue.
@zirain zirain added the help wanted Extra attention is needed label Oct 25, 2023
@arkodg arkodg mentioned this issue Oct 25, 2023
Merged
@zirain
Copy link
Contributor

zirain commented Oct 26, 2023

Also need tests for those things.

@slayer321
Copy link
Contributor

/assign

@slayer321 slayer321 mentioned this issue Oct 29, 2023
Merged
@Xunzhuo Xunzhuo removed the help wanted Extra attention is needed label Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slayer321 slayer321

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: CEL Validation in BackendTrafficPolicy slayer321/gateway
4 participants
@zirain @arkodg @Xunzhuo @slayer321