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.

Sign up for GitHub

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

Jump to bottom

controller: updates the status of CRDs with reconciliation result #130

Closed
1 of 3 tasks
mathetake opened this issue Jan 18, 2025 · 4 comments · Fixed by #436
Closed
1 of 3 tasks

controller: updates the status of CRDs with reconciliation result #130

mathetake opened this issue Jan 18, 2025 · 4 comments · Fixed by #436
Assignees
@mathetake @daixiang0
Labels
enhancement New feature or request
Milestone
v0.2.0

Comments

@mathetake
Copy link
Member

mathetake commented Jan 18, 2025 •
edited
Loading

@mathetake mathetake added the enhancement New feature or request label Jan 18, 2025
@daixiang0
Copy link
Member

/assign

@mathetake
Copy link
Member Author

@daixiang0 are you still interested in this? i think we could use your help

@daixiang0
Copy link
Member

Sure, I will do it

@mathetake
Copy link
Member Author

Great!

@daixiang0 daixiang0 mentioned this issue Jan 27, 2025
Merged
@mathetake mathetake added this to the v0.2.0 milestone Jan 29, 2025
mathetake added a commit that referenced this issue Feb 25, 2025
@daixiang0 @mathetake
controller: update AIGatewayRoute status when reconcile (#187)
525a79f 
**Commit Message**

Update AIGatewayRoute status during reconcile.

**Related Issues/PRs (if applicable)**

Contributes to #130

---------

Signed-off-by: Loong <[email protected]>
Co-authored-by: Takeshi Yoneda <[email protected]>
@mathetake mathetake self-assigned this Feb 26, 2025
@mathetake mathetake mentioned this issue Feb 26, 2025
Merged
mathetake added a commit that referenced this issue Feb 26, 2025
@mathetake
controller: finishes reconciliation result updates (#436)
6978f83 
**Commit Message**

This adds the status fields to BackendSecurityPolicy and
AIServiceBackend just like we already have for AIGatewayRoute since
#187.

The output will look like

```
$ k get aiservicebackends.aigateway.envoyproxy.io
NAME                                  STATUS
envoy-ai-gateway-basic-aws            Accepted
envoy-ai-gateway-basic-openai         Accepted
envoy-ai-gateway-basic-testupstream   Accepted

$ k get backendsecuritypolicies.aigateway.envoyproxy.io
NAME                                     STATUS
envoy-ai-gateway-basic-aws-credentials   Accepted
envoy-ai-gateway-basic-openai-apikey     Accepted

$ k get aigatewayroutes.aigateway.envoyproxy.io        
NAME                     STATUS
envoy-ai-gateway-basic   Accepted
```


**Related Issues/PRs (if applicable)**

Follow up on #187 and closes #130

---------

Signed-off-by: Takeshi Yoneda <[email protected]>
daixiang0 pushed a commit to daixiang0/ai-gateway that referenced this issue Feb 27, 2025
@mathetake @daixiang0
controller: finishes reconciliation result updates (envoyproxy#436)
f900468 
**Commit Message**

This adds the status fields to BackendSecurityPolicy and
AIServiceBackend just like we already have for AIGatewayRoute since
envoyproxy#187.

The output will look like

```
$ k get aiservicebackends.aigateway.envoyproxy.io
NAME                                  STATUS
envoy-ai-gateway-basic-aws            Accepted
envoy-ai-gateway-basic-openai         Accepted
envoy-ai-gateway-basic-testupstream   Accepted

$ k get backendsecuritypolicies.aigateway.envoyproxy.io
NAME                                     STATUS
envoy-ai-gateway-basic-aws-credentials   Accepted
envoy-ai-gateway-basic-openai-apikey     Accepted

$ k get aigatewayroutes.aigateway.envoyproxy.io        
NAME                     STATUS
envoy-ai-gateway-basic   Accepted
```


**Related Issues/PRs (if applicable)**

Follow up on envoyproxy#187 and closes envoyproxy#130

---------

Signed-off-by: Takeshi Yoneda <[email protected]>
daixiang0 pushed a commit to daixiang0/ai-gateway that referenced this issue Feb 27, 2025
@mathetake @daixiang0
controller: finishes reconciliation result updates (envoyproxy#436)
1c186b5 
**Commit Message**

This adds the status fields to BackendSecurityPolicy and
AIServiceBackend just like we already have for AIGatewayRoute since
envoyproxy#187.

The output will look like

```
$ k get aiservicebackends.aigateway.envoyproxy.io
NAME                                  STATUS
envoy-ai-gateway-basic-aws            Accepted
envoy-ai-gateway-basic-openai         Accepted
envoy-ai-gateway-basic-testupstream   Accepted

$ k get backendsecuritypolicies.aigateway.envoyproxy.io
NAME                                     STATUS
envoy-ai-gateway-basic-aws-credentials   Accepted
envoy-ai-gateway-basic-openai-apikey     Accepted

$ k get aigatewayroutes.aigateway.envoyproxy.io        
NAME                     STATUS
envoy-ai-gateway-basic   Accepted
```


**Related Issues/PRs (if applicable)**

Follow up on envoyproxy#187 and closes envoyproxy#130

---------

Signed-off-by: Takeshi Yoneda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mathetake mathetake

@daixiang0 daixiang0

Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.2.0
Development

Successfully merging a pull request may close this issue.

controller: finishes reconciliation result updates envoyproxy/ai-gateway
2 participants
@mathetake @daixiang0