Skip to content

Conversation

tplavcic
Copy link
Member

@tplavcic tplavcic commented Aug 15, 2023

K8SPS-73 Powered by Pull Request Badge

CHANGE DESCRIPTION

This adds chaos mesh self healing tests.
Async self healing test is located here and depends on some code fixes to be merged: #428

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?
  • Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
  • Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

  • Is an E2E test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

Config/Logging/Testability

  • Are all needed new/changed options added to default YAML files?
  • Are the manifests (crd/bundle) regenerated if needed?
  • Did we add proper logging messages for operator actions?
  • Did we ensure compatibility with the previous version or cluster upgrade process?
  • Does the change support oldest and newest supported PS version?
  • Does the change support oldest and newest supported Kubernetes version?

@tplavcic tplavcic requested a review from nmarukovich as a code owner August 15, 2023 06:55
@pull-request-size pull-request-size bot added the size/XXL 1000+ lines label Aug 15, 2023
passwords="$(for i in $secrets; do base64 -d <<< $i; echo; done) $secrets"
pods=$(kubectl -n "${NAMESPACE}" get pods -o name | awk -F "/" '{print $2}')
local secrets=$(kubectl get secrets -o json | jq -r '.items[].data | to_entries | .[] | select(.key | (endswith(".crt") or endswith(".key") or endswith(".pub") or endswith(".pem") or endswith(".p12")) | not) | .value')
local passwords="$(for i in $secrets; do base64 -d <<< $i; echo; done) $secrets"
Copy link
Contributor

Choose a reason for hiding this comment

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

[shfmt] reported by reviewdog 🐶

Suggested change
local passwords="$(for i in $secrets; do base64 -d <<< $i; echo; done) $secrets"
local passwords="$(for i in $secrets; do
base64 -d <<<$i
echo
done) $secrets"

@tplavcic tplavcic force-pushed the K8SPS-73-self-healing branch from 7ca1f88 to 2a26b60 Compare August 15, 2023 10:08
@tplavcic tplavcic force-pushed the K8SPS-73-self-healing branch from 0de6bd1 to 5107f97 Compare August 15, 2023 16:16
@JNKPercona
Copy link
Collaborator

Test name Status
async-ignore-annotations passed
auto-config passed
config passed
config-router passed
demand-backup passed
gr-bootstrap passed
gr-demand-backup passed
gr-haproxy passed
gr-ignore-annotations passed
gr-init-deploy passed
gr-one-pod passed
gr-scaling passed
gr-self-healing passed
gr-tls-cert-manager passed
gr-users passed
haproxy passed
init-deploy passed
limits passed
monitoring passed
one-pod passed
operator-self-healing passed
scaling passed
service-per-pod passed
sidecars passed
smart-update passed
tls-cert-manager passed
users passed
version-service passed
We run 28 out of 28

commit: 70568ae
image: perconalab/percona-server-mysql-operator:PR-424-70568ae

@tplavcic tplavcic requested a review from hors August 18, 2023 06:48
@tplavcic tplavcic merged commit ed35937 into main Aug 18, 2023
@tplavcic tplavcic deleted the K8SPS-73-self-healing branch August 18, 2023 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XXL 1000+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants