Skip to content

Conversation

@sunjayBhatia
Copy link
Member

What type of PR is this?

/kind documentation

What this PR does / why we need it:

Adds conformance report for new Contour 1.33.0 release which supports Gateway API 1.3.0

Which issue(s) this PR fixes:

N/A

Does this PR introduce a user-facing change?:

NONE

Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/documentation Categorizes issue or PR as related to documentation. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 9, 2025
@sunjayBhatia
Copy link
Member Author

cc @kubernetes-sigs/gateway-api-maintainers

Copy link
Member

@robscott robscott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sunjayBhatia!

Comment on lines 69 to 72
skippedTests:
- GatewayStaticAddresses
- HTTPRouteInvalidParentRefSectionNameNotMatchingPort
- HTTPRouteRedirectPortAndScheme
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of skipping tests can you just change the set of features you claim to support?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robscott I did re-run conformance with unsupported features rather than specific skipped tests, though it does mean some tests that do pass are not run/documented as such any longer

Hoping to cover all of these in our next release and with 1.4.0 conformance so not a huge deal

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't able to stop skipping the HTTPRouteHTTPSListener test as there is no specific feature associated with the test, it would mean skipping all HTTPRoute tests which is not ideal of course

Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sunjayBhatia
Once this PR has been reviewed and has the lgtm label, please assign kflynn for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

- core:
result: partial
skippedTests:
- HTTPRouteHTTPSListener
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

had to leave this as a skipped test since there is not a specific enough feature to disable this one test (otherwise we would have to disable testing HTTPRoute in general), this should be addressed in the next release however to remove the skip entirely

unsupportedFeatures:
- GatewayStaticAddresses
- HTTPRouteParentRefPort
- HTTPRoutePortRedirect
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: passing HTTPRouteParentRefPort and HTTPRoutePortRedirect as unsupported means we're now skipping running some tests that do pass, the specific tests that were previously skipped are HTTPRouteRedirectPortAndScheme and HTTPRouteInvalidParentRefSectionNameNotMatchingPort

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants