Improve backups/jobs reliability and observability - #1776
Conversation
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
| echo " Message: $COND_MSG" | ||
|
|
||
| # The condition should be set due to backup creation failure | ||
| if [[ "$COND_REASON" != "BackupCreationFailed" && "$COND_REASON" != "BackupFailed" ]]; then |
There was a problem hiding this comment.
Hmm, I think I need to fix this
egegunes
left a comment
There was a problem hiding this comment.
I think the general intent is good.
But according to tests operator panics on cluster deletion, @Kajot-dev could you please check?
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
Sorry, should be fixed now, I forgot a guard |
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
|
|
Files that use TestAssert, should be named like |
Signed-off-by: Jakub Jaruszewski <jjaruszewski@man.poznan.pl>
commit: 9e49866 |
CHANGE DESCRIPTION
This MR adds:
Degradedcondition to the PerconaPgUpgrade resource and error handling on the path where it was not handled previously (found when debugging K8SPG-1058), so the user can see if there is some kind of error preventing the upgrade from progressingScheduledBackupDegradedcondition on the PerconaPGCluster - which allows observability when backup/job creation failed for some reasonProblem:
This MR adresses general backup reliability and jobs observability such as:
CHECKLIST
Jira
Needs Doc) and QA (Needs QA)?Tests
Config/Logging/Testability