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 client packages #5206

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

dprotaso
Copy link
Contributor

@dprotaso dprotaso commented Feb 5, 2025

Commits

  • git add tools for generating clientset, informers, and listers
  • drop manual registration and annotate k8s crds types
  • add make targets to generate clients
  • wire client gen into make generate call
  • annotate EnvoyGateway

What type of PR is this?

  • "feat(client): add new feature"

What this PR does / why we need it:
This adds client sets allowing end-users to create envoy gateway resources programmatically.

Which issue(s) this PR fixes:

Fixes None

Release Notes: No

@dprotaso dprotaso requested a review from a team as a code owner February 5, 2025 04:10
Copy link

codecov bot commented Feb 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.89%. Comparing base (36fdc0b) to head (9a4e5ca).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5206      +/-   ##
==========================================
+ Coverage   66.87%   66.89%   +0.01%     
==========================================
  Files         210      201       -9     
  Lines       32998    32980      -18     
==========================================
- Hits        22067    22061       -6     
+ Misses       9592     9582      -10     
+ Partials     1339     1337       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dprotaso dprotaso force-pushed the add-client-packages branch from dd697e0 to 13bb812 Compare February 5, 2025 15:37
@dprotaso
Copy link
Contributor Author

dprotaso commented Feb 5, 2025

repushed commits for DCO

@dprotaso
Copy link
Contributor Author

dprotaso commented Feb 5, 2025

Don't really understand the failure given this PR doesn't touch that package

coverage: 54.9% of statements
FAIL	github.com/envoyproxy/gateway/internal/provider/kubernetes	37.792s

Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
@dprotaso dprotaso force-pushed the add-client-packages branch from 13bb812 to 54a26d3 Compare February 5, 2025 16:19
@dprotaso dprotaso closed this Feb 5, 2025
@dprotaso dprotaso reopened this Feb 5, 2025
@dprotaso
Copy link
Contributor Author

dprotaso commented Feb 5, 2025

rebased on latest main

@dprotaso
Copy link
Contributor Author

dprotaso commented Feb 5, 2025

gotta sign

Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
Signed-off-by: Dave Protasowski <dprotaso@gmail.com>
@dprotaso dprotaso force-pushed the add-client-packages branch from c320541 to 9a4e5ca Compare February 5, 2025 18:37
@arkodg
Copy link
Contributor

arkodg commented Feb 5, 2025

thanks @dprotaso this PR looks good, would ideally like to have not have the patch committed, lets wait a day to see if kubernetes/kubernetes#129307 gets merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants