[DO-NOT-MERGE] Install Tekton Results through operator #4600
+373
−978
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
enarha
force-pushed
the
switch-results-operator
branch
3 times, most recently
from
c9362ad
to
40a19d3
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: enarha The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
enarha
force-pushed
the
switch-results-operator
branch
2 times, most recently
from
cbadd40
to
051849c
Compare
|
/test appstudio-e2e-tests |
enarha
force-pushed
the
switch-results-operator
branch
3 times, most recently
from
982ac02
to
5e0ef49
Compare
|
/test appstudio-e2e-tests |
enarha
force-pushed
the
switch-results-operator
branch
from
65a62f1
to
c71ce06
Compare
|
/test appstudio-e2e-tests |
3 similar comments
|
/test appstudio-e2e-tests |
|
/test appstudio-e2e-tests |
|
/test appstudio-e2e-tests |
enarha
force-pushed
the
switch-results-operator
branch
5 times, most recently
from
e6669a7
to
4753aec
Compare
enarha
force-pushed
the
switch-results-operator
branch
from
68ccf35
to
8f8dc9b
Compare
enarha
force-pushed
the
switch-results-operator
branch
2 times, most recently
from
1377caa
to
47c269e
Compare
enarha
force-pushed
the
switch-results-operator
branch
from
47c269e
to
51eb934
Compare
enarha
force-pushed
the
switch-results-operator
branch
from
199c04f
to
4c9a967
Compare
enarha
force-pushed
the
switch-results-operator
branch
from
2fd6169
to
1b967bb
Compare
This only changes development overlay as a start.
The option to provide path to cert is not available in the OSP oprrator. Until it is added, test with no verification.
Still not supported by the OSP operator. Removing temporary to allow testing. Also remove S3_HOSTNAME_IMMUTABLE, set to default value (false) anyway.
The option has been depricated and should be completely removed. The operator overrides this value, so it and argocd fight over it blocking the deployment.
It is now deployed by the OSP operator as part of the Tekton Results deployment.
The operator expects secret named `tekton-results-postgres` while orinal secret name is `tekton-results-database`. The name of the fields in the secret also changed.
With operator now supporting those configuration options, we add them back to configuration.
This reverts commit 47a55cf.
enarha
force-pushed
the
switch-results-operator
branch
from
1b967bb
to
d0e2b7a
Compare
With the DB installed through the operator, we can't provide TLS certificate and key, thus we can't enabled TLS verification in the API.
enarha
force-pushed
the
switch-results-operator
branch
from
d0e2b7a
to
07a4ea3
Compare
|
/test appstudio-e2e-tests |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This only changes development overlay as a start.