Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[DO-NOT-MERGE] Install Tekton Results through operator #4600

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

[DO-NOT-MERGE] Install Tekton Results through operator #4600

wants to merge 18 commits into from

Conversation

enarha
Copy link
Member

@enarha enarha commented Sep 25, 2024

This only changes development overlay as a start.

@enarha enarha force-pushed the switch-results-operator branch 3 times, most recently from c9362ad to 40a19d3 Compare October 27, 2024 14:17
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Oct 27, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: enarha
Once this PR has been reviewed and has the lgtm label, please assign simonbaird for approval. For more information see the Kubernetes Code Review Process.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@enarha enarha force-pushed the switch-results-operator branch 2 times, most recently from cbadd40 to 051849c Compare October 27, 2024 16:07
@enarha
Copy link
Member Author

enarha commented Oct 28, 2024

/test appstudio-e2e-tests

@enarha enarha force-pushed the switch-results-operator branch 3 times, most recently from 982ac02 to 5e0ef49 Compare October 29, 2024 12:31
@enarha
Copy link
Member Author

enarha commented Oct 29, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member Author

enarha commented Oct 30, 2024

/test appstudio-e2e-tests

3 similar comments
@enarha
Copy link
Member Author

enarha commented Oct 30, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member Author

enarha commented Nov 4, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member Author

enarha commented Nov 4, 2024

/test appstudio-e2e-tests

@enarha enarha force-pushed the switch-results-operator branch 5 times, most recently from e6669a7 to 4753aec Compare November 5, 2024 12:17
@enarha enarha force-pushed the switch-results-operator branch 2 times, most recently from 1377caa to 47c269e Compare November 10, 2024 09:40
@enarha
Install Tekton Results through operator
80e1405 
This only changes development overlay as a start.
@enarha
test no DB SSL verification
70b0373 
The option to provide path to cert is not available in the OSP oprrator.
Until it is added, test with no verification.
@enarha
Removing Results profiling configuration
2e1b0bb
@enarha
More changes
230f178
@enarha
Remove Results prometheus config
07f8103 
Still not supported by the OSP operator. Removing temporary to allow
testing.
Also remove S3_HOSTNAME_IMMUTABLE, set to default value (false) anyway.
@enarha
Disable tls_hostname_override, overriden by the operator
2a33ede 
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.
@enarha
Rename db_* postgres_* variables
2908ea5
@enarha
Remove Postgres Application
1bccc74 
It is now deployed by the OSP operator as part of the Tekton Results
deployment.
@enarha
Rename postgres secret name
855b87f 
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.
@enarha
Bump OSP to newest version
3bbcbe5
@enarha
Enable DB TLS verify-full and prometheus_histogram
40524eb 
With operator now supporting those configuration options, we add them
back to configuration.
@enarha
Expose TLS key and cert to Postgres
2eaebd2
@enarha
Revert "Remove Postgres Application"
c4e9601 
This reverts commit 47a55cf.
@enarha
Reintroduce external DB
07a4ea3 
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
Copy link
Member Author

enarha commented Nov 13, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member Author

enarha commented Nov 18, 2024

/test appstudio-e2e-tests

@enarha
Fix watcher logs sending
c8c61d5 
Rerquired with the newest version which has it set to false by default.
@enarha
Copy link
Member Author

enarha commented Nov 19, 2024

/test appstudio-e2e-tests

@enarha
Create bucket automatically
aa61ba1 
Currently created "manually" by init container.
@enarha
Copy link
Member Author

enarha commented Nov 19, 2024

/test appstudio-e2e-tests

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Nov 19, 2024

@enarha: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/appstudio-e2e-tests aa61ba1 link true /test appstudio-e2e-tests

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@enarha @openshift-merge-robot