-
Notifications
You must be signed in to change notification settings - Fork 327
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
chore(kuma-dp) refactor dataplane resource loading #2252
Conversation
Refactor the dataplane resource loading to make it oblivious to the underlying resource types. This will later allow us to add support for more dataplane resource types without changing any code. Signed-off-by: James Peach <james.peach@konghq.com>
Codecov Report
@@ Coverage Diff @@
## master #2252 +/- ##
==========================================
+ Coverage 51.77% 52.05% +0.28%
==========================================
Files 909 913 +4
Lines 41004 41051 +47
==========================================
+ Hits 21229 21371 +142
+ Misses 17741 17621 -120
- Partials 2034 2059 +25
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Refactor the dataplane resource loading to make it oblivious to the underlying resource types. This will later allow us to add support for more dataplane resource types without changing any code. Signed-off-by: James Peach <james.peach@konghq.com> (cherry picked from commit b2e7522) # Conflicts: # app/kuma-dp/cmd/run.go
Refactor the dataplane resource loading to make it oblivious to the underlying resource types. This will later allow us to add support for more dataplane resource types without changing any code. Signed-off-by: James Peach <james.peach@konghq.com> (cherry picked from commit b2e7522)
Refactor the dataplane resource loading to make it oblivious to the underlying resource types. This will later allow us to add support for more dataplane resource types without changing any code. Signed-off-by: James Peach <james.peach@konghq.com> (cherry picked from commit b2e7522) Co-authored-by: James Peach <james.peach@konghq.com>
Summary
Refactor the dataplane resource loading to make it oblivious to
the underlying resource types. This will later allow us to add
support for more dataplane resource types without changing any
code.
Full changelog
N/A
Issues resolved
N/A
Documentation
N/A
Testing