-
Notifications
You must be signed in to change notification settings - Fork 62
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(chart): expose all services through ALB #2713
Conversation
ArgoCD Diff for commit
|
Legend | Status |
---|---|
✅ | The app is synced in ArgoCD, and diffs you see are solely from this PR. |
The app is out-of-sync in ArgoCD, and the diffs you see include those changes plus any from this PR. | |
🛑 | There was an error generating the ArgoCD diffs due to changes in this PR. |
Yes, it should be served by S3+CloudFront if I am not wrong |
@severo can you please tell me if |
Yes, we need it somewhere. Not necessarily at that URL though, if it's an issue. re 404: not important at all |
re: openapi.json. Ideally, it would be served from the docs. See the previous discussions here: cf8042d and here: huggingface/doc-builder#312 (comment). So: not a problem at all if we move its URL, we would just have to update it in the docs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering: would it make sense to have all the paths in the same ingress config file? or is it better to have one file per service?
We can do the redirect with a custom annotation. So let me know if you want to move it or not ;) |
Yes we can. Do you think it will be easier for you? |
Please let's keep it unchanged if it's possible, and we can move it later in a dedicated PR |
This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Please note that issues that do not follow the contributing guidelines are likely to be ignored. |
We should finish this PR. Happy to give you a hand @rtrompier |
Thx for the reminder. I plan to finish it in the week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's good now, right?
I think so yes :) |
/
onapi
instances/admin
onadmin
instances/rows
onrows
instances/search
onsearch
instances/filter
onsearch
instances/sse
onsseApi
instances/openapi.json
on github