[patch] patch postgredb-tekton-results-postgres-0 PVC with storageclass when no default available #67
Conversation
This file contains hidden or 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
anilprajapatiibm
requested review from
alequint,
andrercm,
durera,
leo-miran,
terenceq and
whitfiea
as code owners
durera
requested changes
May 3, 2025
Contributor
durera
left a comment
durera
left a comment
There was a problem hiding this comment.
This has been sitting around for a month .. please resolve the merge conflicts with the May catalog update branch that I just merged into stable and we will get this done
Contributor
|
After resolving the merge conflicts the PR is showing no actual changes, so I assume the changes in this branch were already introduced via a different PR somewhere and I am closing this PR as a result. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
External Issue - ibm-mas/cli#1511
Internal issue - https://jsw.ibm.com/browse/MASCORE-6225
When Openshift-pipelines operator is installed as part of tekton pipeline,
the postgredb-tekton-results-postgres-0 PVC was not getting ready since there's no default storageclass set in cluster.
since this PVC doesn't come up and was in pending state, it causing the tekton-results-postgres statefulSet also to be in pending,
due to these resources not coming up, the MAS pre-install-check in the pipeline times out checking the health of this statefulSet, causing failure in pipeline.
Solution - with this fix, we will determine the default storage class and patch the PVC. If for some reason, there's no recognizable storage class to use, the patch will fail
fvt test:
#2715 and #2716 tested with this changes