This repository has been archived by the owner on Feb 8, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 47
Use selectors for StatefulSet lookups #278
Merged
osowski
merged 2 commits into
Seagate:CORTX-28968_server_pod_scale_out
from
keithpine:CORTX-28968_start-cluster-fix
Jun 8, 2022
Merged
Use selectors for StatefulSet lookups #278
osowski
merged 2 commits into
Seagate:CORTX-28968_server_pod_scale_out
from
keithpine:CORTX-28968_start-cluster-fix
Jun 8, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Keith Pine <[email protected]>
Not really a necessary change since the problem was fixed elsewhere, but it's a good a time as any to switch to using selectors instead of grep, since the resource type has also changed. |
osowski
approved these changes
Jun 8, 2022
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.
lgtm
osowski
added a commit
that referenced
this pull request
Jun 8, 2022
* CORTX-28968 - Initial pass at STS changes needed Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Initial pass at STS changes needed Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Final draft with e2e impl, passing FQDN all the way through Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Draft revision for server pod scale with recent cortx-k8s changes Signed-off-by: Rick Osowski <[email protected]> * (SQUASH) Houeskeeping updates Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - merge in integration Signed-off-by: Rick Osowski <[email protected]> * Use release namespace for server domain names (#269) Signed-off-by: Keith Pine <[email protected]> Co-authored-by: Rick O <[email protected]> * CORTX-28968 - Convert bash defaults to yq queries Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Convert bash defaults to yq queries Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Integrating changes for CORTX-30726 (1/n) Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Integrating changes for CORTX-30726 (2/n) Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Integrating changes for CORTX-30726 (3/n) Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Integrating changes for CORTX-30726 (4/n) Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Integrating changes for CORTX-30726 (5/n) Signed-off-by: Rick Osowski <[email protected]> * Fix status script for StatefulSet, label updates, etc. (#277) * Remove unused loadbal service Signed-off-by: Keith Pine <[email protected]> * Changes Signed-off-by: Keith Pine <[email protected]> * Rename headless service file Signed-off-by: Keith Pine <[email protected]> * Headless service `app` label Signed-off-by: Keith Pine <[email protected]> * Update pvc checks in status script Signed-off-by: Keith Pine <[email protected]> * Update app label Signed-off-by: Keith Pine <[email protected]> * Fix solution_stubl.yaml lint failure Signed-off-by: Keith Pine <[email protected]> * Fix shellcheck lint errors Signed-off-by: Keith Pine <[email protected]> * Fix chart lint error Signed-off-by: Keith Pine <[email protected]> * Fix chart docs lint error Signed-off-by: Keith Pine <[email protected]> * CORTX-28968 - Integrating changes for CORTX-30726 (6/n) Signed-off-by: Rick Osowski <[email protected]> * Use selectors for StatefulSet lookups (#278) Fix start script Signed-off-by: Keith Pine <[email protected]> * CORTX-28968 - PR review code changes Signed-off-by: Rick Osowski <[email protected]> * Update charts/cortx/templates/_cluster.yaml.tpl Co-authored-by: Keith Pine <[email protected]> Signed-off-by: Rick Osowski <[email protected]> * Update charts/cortx/templates/_cluster.yaml.tpl Co-authored-by: Keith Pine <[email protected]> Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - PR review code changes Signed-off-by: Rick Osowski <[email protected]> * CORTX-28968 - Updated solution.example.yaml with required build number Signed-off-by: Rick Osowski <[email protected]> * Update chart appVersion Signed-off-by: Keith Pine <[email protected]> Co-authored-by: Keith Pine <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Use new selector to get statefulset and pod lists.