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

Inform on and use Gateway API v1 types where possible #6153

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

sunjayBhatia
Copy link
Member

Only files that set up an API client scheme or use ReferenceGrant should have the v1beta1 import

Fixes: #6150

@sunjayBhatia sunjayBhatia added do not merge Don't merge this PR until this label is removed. release-note/major A major change that needs more than a paragraph of explanation in the release notes. labels Feb 2, 2024
@sunjayBhatia
Copy link
Member Author

Based on #6094 as that makes it a little easier to do the find/replace

Set as do not merge for now so we wait until after the 1.28.0 release

Copy link

codecov bot commented Feb 2, 2024

Codecov Report

Attention: 68 lines in your changes are missing coverage. Please review.

Comparison is base (f131882) 79.47% compared to head (8c84a44) 79.46%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6153      +/-   ##
==========================================
- Coverage   79.47%   79.46%   -0.02%     
==========================================
  Files         142      142              
  Lines       16250    16254       +4     
==========================================
+ Hits        12915    12916       +1     
- Misses       3021     3024       +3     
  Partials      314      314              
Files Coverage Δ
internal/dag/builder.go 93.18% <100.00%> (ø)
internal/dag/cache.go 97.03% <100.00%> (ø)
internal/dag/policy.go 95.52% <100.00%> (ø)
internal/gatewayapi/listeners.go 100.00% <100.00%> (ø)
internal/k8s/kind.go 65.21% <100.00%> (-1.95%) ⬇️
internal/k8s/statusaddress.go 79.11% <100.00%> (ø)
internal/provisioner/scheme.go 83.33% <100.00%> (+1.51%) ⬆️
internal/status/backendtlspolicyconditions.go 91.37% <100.00%> (ø)
internal/status/cache.go 93.22% <100.00%> (ø)
internal/status/gatewayclassconditions.go 100.00% <100.00%> (ø)
... and 14 more

@sunjayBhatia sunjayBhatia marked this pull request as ready for review February 12, 2024 22:29
@sunjayBhatia sunjayBhatia requested a review from a team as a code owner February 12, 2024 22:29
@sunjayBhatia sunjayBhatia requested review from tsaarni and skriss and removed request for a team February 12, 2024 22:29
@sunjayBhatia sunjayBhatia removed the do not merge Don't merge this PR until this label is removed. label Feb 12, 2024
@sunjayBhatia sunjayBhatia requested review from a team, davinci26 and izturn and removed request for a team February 12, 2024 22:29
Only files that set up an API client scheme or use ReferenceGrant
should have the v1beta1 import

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
Copy link
Member

@skriss skriss left a comment

Choose a reason for hiding this comment

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

Nice, LGTM

@sunjayBhatia sunjayBhatia enabled auto-merge (squash) February 12, 2024 22:57
@sunjayBhatia sunjayBhatia merged commit bd003fb into projectcontour:main Feb 12, 2024
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/major A major change that needs more than a paragraph of explanation in the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to reconciling specifically Gateway API v1 resources (from v1beta1)
2 participants