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: add API for Deployment Mode #1413

Merged
merged 3 commits into from
May 17, 2023
Merged

Conversation

arkodg
Copy link
Contributor

@arkodg arkodg commented May 10, 2023

Relates to #1117

apiVersion: config.gateway.envoyproxy.io/v1alpha1
kind: EnvoyGateway
gateway:
  controllerName: gateway.envoyproxy.io/gatewayclass-controller
provider:
  type: Kubernetes
  kubernetes:
    watch:
      namespaces:
      - "ns-1"
      - "ns-2"

Relates to envoyproxy#1117

```
apiVersion: config.gateway.envoyproxy.io/v1alpha1
kind: EnvoyGateway
gateway:
  controllerName: gateway.envoyproxy.io/gatewayclass-controller
provider:
  type: Kubernetes
  kubernetes:
    mode:
      watch:
        namespaces:
	- "ns-1"
	- "ns-2"
```

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested a review from a team as a code owner May 10, 2023 05:01
@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Merging #1413 (5a9065d) into main (b90946c) will decrease coverage by 0.68%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1413      +/-   ##
==========================================
- Coverage   62.63%   61.95%   -0.68%     
==========================================
  Files          79       79              
  Lines       11116    11293     +177     
==========================================
+ Hits         6962     6997      +35     
- Misses       3699     3836     +137     
- Partials      455      460       +5     
Impacted Files Coverage Δ
api/config/v1alpha1/envoygateway_types.go 100.00% <ø> (ø)
api/config/v1alpha1/zz_generated.deepcopy.go 0.00% <0.00%> (ø)

... and 7 files with indirect coverage changes

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Copy link
Contributor

@haq204 haq204 left a comment

Choose a reason for hiding this comment

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

This looks reasonable to me

@zirain zirain added the cherrypick/release-v0.4 cherrypick to release/v0.4 label May 17, 2023
@zirain
Copy link
Contributor

zirain commented May 17, 2023

add a label for testing

@zirain zirain merged commit efe575a into envoyproxy:main May 17, 2023
17 of 18 checks passed
tanujd11 pushed a commit to tanujd11/gateway that referenced this pull request May 20, 2023
* Add API for Deployment Mode

Relates to envoyproxy#1117

```
apiVersion: config.gateway.envoyproxy.io/v1alpha1
kind: EnvoyGateway
gateway:
  controllerName: gateway.envoyproxy.io/gatewayclass-controller
provider:
  type: Kubernetes
  kubernetes:
    mode:
      watch:
        namespaces:
	- "ns-1"
	- "ns-2"
```

Signed-off-by: Arko Dasgupta <arko@tetrate.io>

* rm top level mode

Signed-off-by: Arko Dasgupta <arko@tetrate.io>

* fix

Signed-off-by: Arko Dasgupta <arko@tetrate.io>

---------

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg deleted the mode-watch-ns branch May 22, 2023 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherrypick/release-v0.4 cherrypick to release/v0.4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants