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

feat(kuma-cp) add Gateway resource type #2428

Merged
merged 5 commits into from
Jul 30, 2021
Merged

feat(kuma-cp) add Gateway resource type #2428

merged 5 commits into from
Jul 30, 2021

Conversation

jpeach
Copy link
Contributor

@jpeach jpeach commented Jul 26, 2021

Summary

Add a Gateway resource. This just defines the type, which is not registered or used in any way. When enough PRs land, the type will be registered from the gateway plugin, which will be enabled by a build tag for a good long while yet.

Full changelog

N/A

Issues resolved

N/A

Documentation

N/A

Testing

  • Unit tests
  • E2E tests
  • Manual testing on Universal
  • Manual testing on Kubernetes

Backwards compatibility

  • Add backport-to-stable label if the code is backwards compatible. Otherwise, list breaking changes.

@jpeach jpeach requested a review from a team as a code owner July 26, 2021 06:35
@jpeach
Copy link
Contributor Author

jpeach commented Jul 26, 2021

Arguably, this proto definition should be in the gateway plugin (see #2412). Happy to move it there if that's the consensus.

@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2021

Codecov Report

Merging #2428 (e8ff428) into master (bb07923) will decrease coverage by 0.01%.
The diff coverage is 46.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2428      +/-   ##
==========================================
- Coverage   52.61%   52.59%   -0.02%     
==========================================
  Files         885      887       +2     
  Lines       47971    48394     +423     
==========================================
+ Hits        25238    25453     +215     
- Misses      20689    20891     +202     
- Partials     2044     2050       +6     
Impacted Files Coverage Δ
pkg/core/resources/apis/mesh/resources.go 78.21% <12.82%> (-2.76%) ⬇️
api/mesh/v1alpha1/gateway.pb.go 33.57% <33.57%> (ø)
pkg/core/resources/apis/mesh/validators.go 90.97% <91.66%> (+0.15%) ⬆️
pkg/core/resources/apis/mesh/gateway_validator.go 95.00% <95.00%> (ø)
pkg/xds/cache/once/cache.go 87.17% <0.00%> (-7.70%) ⬇️
pkg/insights/resyncer.go 66.38% <0.00%> (+0.84%) ⬆️
pkg/plugins/resources/memory/store.go 82.06% <0.00%> (+4.34%) ⬆️
pkg/core/bootstrap/autoconfig.go 54.46% <0.00%> (+8.03%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb07923...e8ff428. Read the comment docs.

@jpeach jpeach marked this pull request as draft July 28, 2021 23:25
@jpeach
Copy link
Contributor Author

jpeach commented Jul 28, 2021

Back to draft while I add validation and tests.

@jpeach jpeach marked this pull request as ready for review July 30, 2021 04:57
Signed-off-by: James Peach <james.peach@konghq.com>
Signed-off-by: James Peach <james.peach@konghq.com>
api/mesh/v1alpha1/gateway.proto Outdated Show resolved Hide resolved
api/mesh/v1alpha1/gateway.proto Outdated Show resolved Hide resolved
pkg/core/resources/apis/mesh/validators.go Outdated Show resolved Hide resolved
pkg/core/resources/apis/mesh/validators.go Outdated Show resolved Hide resolved
Signed-off-by: James Peach <james.peach@konghq.com>
Signed-off-by: James Peach <james.peach@konghq.com>
Signed-off-by: James Peach <james.peach@konghq.com>
@jpeach jpeach merged commit 4d7e7c0 into kumahq:master Jul 30, 2021
@jpeach jpeach deleted the feat/proto-add-gateway branch July 30, 2021 10:53
mergify bot pushed a commit that referenced this pull request Jul 30, 2021
Signed-off-by: James Peach <james.peach@konghq.com>
(cherry picked from commit 4d7e7c0)

# Conflicts:
#	pkg/core/resources/apis/mesh/validators.go
jpeach added a commit that referenced this pull request Jul 31, 2021
Signed-off-by: James Peach <james.peach@konghq.com>
(cherry picked from commit 4d7e7c0)
jpeach added a commit that referenced this pull request Aug 1, 2021
Signed-off-by: James Peach <james.peach@konghq.com>
(cherry picked from commit 4d7e7c0)

Co-authored-by: James Peach <james.peach@konghq.com>
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.

None yet

3 participants