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

INF-1890: switch to use ingressClassName #44

Merged
merged 1 commit into from
Apr 19, 2022

Conversation

jerryc8
Copy link
Contributor

@jerryc8 jerryc8 commented Apr 18, 2022

No description provided.

@jerryc8 jerryc8 requested a review from cep21 April 18, 2022 10:59
@linear
Copy link

linear bot commented Apr 18, 2022

INF-1890 voice-staging: switch rest to ingressClassName

Problem we want to solve: follow up from INF-1789 to switch to use ingressClassName. However, flux1 helm operator has a bug and complains about existing resource conflict when ingress goes from v1beta1 to v1 . Therefore, set annotated: false AFTER a namespace/cluster has moved to flux2

What we want to do:

still to be either deleted or upgraded to ingressClassName:

agglomerator                      agglomerator-agglomerator                                                   <none>    
argo                              argo-ui-ingress                                                             <none>    
deepgram-batch                    deepgram-api                                                                <none>    
deepgram                          deepgram-api                                                                <none>    
flux-system                       flux-deployments-receiver                                                   <none>    
gitdb                             gitdb                                                                       <none>    
gitdb                             gitdb-public                                                                <none>    
multi-tenant-test                 ai-agent                                                                    <none>    
pgadmin                           pgadmin-pgadmin4                                                            <none> 

When we will know we are done:

ingresses in all namespaces are no longer using annotation to specify ingress controller

@jerryc8 jerryc8 requested a review from cresta-robot April 19, 2022 04:55
Copy link
Collaborator

@cresta-robot cresta-robot left a comment

Choose a reason for hiding this comment

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

Accepted from slack

@jerryc8 jerryc8 merged commit f340cc1 into master Apr 19, 2022
@jerryc8 jerryc8 deleted the inf-1890-voice-staging-switch-rest-to branch April 19, 2022 04:56
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.

2 participants