-
Notifications
You must be signed in to change notification settings - Fork 5
chore: remove optional backend-api service from tower-svc.yml default manifest #843
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
base: master
Are you sure you want to change the base?
chore: remove optional backend-api service from tower-svc.yml default manifest #843
Conversation
✅ Deploy Preview for seqera-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Asked in ticket, but I'll ask here too: "What was the benefit of exposing this API endpoint publicly in the first place? (beyond just specific subdomain control)" |
gwright99
left a comment
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.
IMO there should be placeholder text explaining that we used to have a service there but it was removed (with a link to this issue), along with some guidance on why sites might wish to keep it around (if at all).
Let's do a favour to future us and any client doing an upgrade to help explain within the manifests why something is different rather than having to answer it one-off via support tickets.
|
@markpanganiban I would prefer not to remove this and instead add notes into the manifest explaining why it exists. The historic documentation and deployment configuration should not change for customers, as they may use these manifests to restore their environment and these changes may not be transparent for them. Complementing and explaining the configuration feels like a more appropriate approach. Adding something like. Along with something like the following. Note - we are reworking some of these aspects and looking to add appropriate warnings and disclaimers for the examples. |
Summary
The backend-api Service was included by default even though it’s only needed when a deployment intentionally exposes the backend to API/CLI users via a dedicated subdomain. Keeping it enabled by default caused confusion and could conflict with the frontend’s wildcard routing.
What changed
Rationale
Impact / Compatibility
Security