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

Add test for optional address value in httproute #3687

Open
EyalPazz opened this issue Mar 18, 2025 · 3 comments · May be fixed by #3689
Open

Add test for optional address value in httproute #3687

EyalPazz opened this issue Mar 18, 2025 · 3 comments · May be fixed by #3689
Assignees
Labels
area/conformance-test Issues or PRs related to Conformance tests. triage/needs-information Indicates an issue needs more information in order to work on it.

Comments

@EyalPazz
Copy link
Contributor

In continuation to:
#3616

@EyalPazz EyalPazz added the area/conformance-test Issues or PRs related to Conformance tests. label Mar 18, 2025
@EyalPazz
Copy link
Contributor Author

/assign

@shaneutt shaneutt added the triage/needs-information Indicates an issue needs more information in order to work on it. label Mar 18, 2025
@shaneutt shaneutt moved this to Triage in Gateway API Pipeline Mar 18, 2025
@EyalPazz EyalPazz linked a pull request Mar 18, 2025 that will close this issue
@mlavacca
Copy link
Member

Hi @EyalPazz, would you mind adding some context to the issue? What it is about, what we want to achieve, etc.

@EyalPazz
Copy link
Contributor Author

Sure! Basically it's a follow up to #3616, we need a test to check that implementations that will support an empty address value in Gateway.Spec.Addresses.Value behave correctly (accepted and assigned an address)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/conformance-test Issues or PRs related to Conformance tests. triage/needs-information Indicates an issue needs more information in order to work on it.
Projects
Status: Triage
Development

Successfully merging a pull request may close this issue.

3 participants