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

Allow user to configure annotations on infra #1115

Merged
merged 3 commits into from
Mar 9, 2023

Conversation

arkodg
Copy link
Contributor

@arkodg arkodg commented Mar 8, 2023

This PR allows the user to add annotations to the
managed Envoy service and well as the Envoy pods using the EnvoyProxy resource

This PR allows the user to add annotations to the
managed Envoy service and well as the Envoy pods using
the EnvoyProxy resource

* Fixes envoyproxy#377

* Closes envoyproxy#648

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested a review from a team as a code owner March 8, 2023 23:24
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@codecov-commenter
Copy link

codecov-commenter commented Mar 8, 2023

Codecov Report

Merging #1115 (d22678c) into main (c903255) will increase coverage by 0.11%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1115      +/-   ##
==========================================
+ Coverage   62.33%   62.44%   +0.11%     
==========================================
  Files          72       72              
  Lines        9449     9509      +60     
==========================================
+ Hits         5890     5938      +48     
- Misses       3151     3157       +6     
- Partials      408      414       +6     
Impacted Files Coverage Δ
...rnal/infrastructure/kubernetes/proxy_deployment.go 91.66% <100.00%> (+0.28%) ⬆️
...nternal/infrastructure/kubernetes/proxy_service.go 85.00% <100.00%> (+2.30%) ⬆️
api/config/v1alpha1/validation/envoyproxy.go 68.18% <0.00%> (-21.82%) ⬇️
internal/provider/kubernetes/controller.go 49.54% <0.00%> (-0.12%) ⬇️
internal/provider/kubernetes/helpers.go 83.78% <0.00%> (+5.40%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@arkodg arkodg added kind/enhancement New feature or request area/api API-related issues area/infra-mgr Issues related to the provisioner used for provisioning the managed Envoy Proxy fleet. priority/high Label used to express the "high" priority level labels Mar 9, 2023
@arkodg arkodg added this to the 0.4.0-rc.1 milestone Mar 9, 2023
@arkodg arkodg added the provider/kubernetes Issues related to the Kubernetes provider label Mar 9, 2023
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested a review from zirain March 9, 2023 01:39
@arkodg
Copy link
Contributor Author

arkodg commented Mar 9, 2023

thanks @zirain , hoping to get another approval before merging this API change

Copy link
Member

@Xunzhuo Xunzhuo left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@Xunzhuo Xunzhuo merged commit 38470e0 into envoyproxy:main Mar 9, 2023
@arkodg arkodg deleted the annotations branch March 14, 2023 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api API-related issues area/infra-mgr Issues related to the provisioner used for provisioning the managed Envoy Proxy fleet. kind/enhancement New feature or request priority/high Label used to express the "high" priority level provider/kubernetes Issues related to the Kubernetes provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Support for Managing the Envoy Proxy Service Add k8s annotations to the Envoy Proxy Service
4 participants