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

SADS switchover #368

Merged
merged 2 commits into from
Oct 19, 2023
Merged

SADS switchover #368

merged 2 commits into from
Oct 19, 2023

Conversation

c0c0n3
Copy link
Owner

@c0c0n3 c0c0n3 commented Oct 17, 2023

This PR make the SADS "offline" service the only SADS service accessible to external users.
In detail:

  • SADS "online" is now set to have no replicas. The deployment descriptors are still there, but there's no service running in the cluster. Ditto for the Argo CD app---we keep it around just in case.
  • SADS "offline" now handles all incoming requests to the /sads path.
  • The /sads-offline path is unavailable.

Breaking change notice

@c0c0n3 c0c0n3 self-assigned this Oct 17, 2023
@c0c0n3 c0c0n3 added the AI / app services Platform application services label Oct 17, 2023
@c0c0n3 c0c0n3 linked an issue Oct 17, 2023 that may be closed by this pull request
@kostasgrevenitis kostasgrevenitis merged commit 7cf9ce9 into main Oct 19, 2023
@c0c0n3 c0c0n3 deleted the sads-switchover branch January 2, 2024 15:43
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI / app services Platform application services
Projects
Status: 🔍 QA
Development

Successfully merging this pull request may close these issues.

Rename SADS-Offline to just SADS
2 participants