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 15 commits into
base: main
Choose a base branch
from
Open

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

wants to merge 15 commits into from
+369 −978

Commits on Nov 13, 2024

  1. Install Tekton Results through operator 

    This only changes development overlay as a start.
    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    80e1405 View commit details
    Browse the repository at this point in the history

  2. test no DB SSL verification 

    The option to provide path to cert is not available in the OSP oprrator.
Until it is added, test with no verification.
    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    70b0373 View commit details
    Browse the repository at this point in the history

  3. Removing Results profiling configuration

    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    2e1b0bb View commit details
    Browse the repository at this point in the history

  4. More changes

    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    230f178 View commit details
    Browse the repository at this point in the history

  5. Remove Results prometheus config 

    Still not supported by the OSP operator. Removing temporary to allow
testing.
Also remove S3_HOSTNAME_IMMUTABLE, set to default value (false) anyway.
    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    07f8103 View commit details
    Browse the repository at this point in the history

  6. Disable tls_hostname_override, overriden by the operator 

    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
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    2a33ede View commit details
    Browse the repository at this point in the history

  7. Rename db_* postgres_* variables

    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    2908ea5 View commit details
    Browse the repository at this point in the history

  8. Remove Postgres Application 

    It is now deployed by the OSP operator as part of the Tekton Results
deployment.
    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    1bccc74 View commit details
    Browse the repository at this point in the history

  9. Rename postgres secret name 

    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
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    855b87f View commit details
    Browse the repository at this point in the history

  10. Bump OSP to newest version

    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    3bbcbe5 View commit details
    Browse the repository at this point in the history

  11. Enable DB TLS verify-full and prometheus_histogram 

    With operator now supporting those configuration options, we add them
back to configuration.
    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    40524eb View commit details
    Browse the repository at this point in the history

  12. Expose TLS key and cert to Postgres

    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    2eaebd2 View commit details
    Browse the repository at this point in the history

  13. Revert "Remove Postgres Application" 

    This reverts commit 47a55cf.
    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    c4e9601 View commit details
    Browse the repository at this point in the history

  14. Reintroduce external DB 

    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
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    07a4ea3 View commit details
    Browse the repository at this point in the history

  15. Fix Results deployments patching

    @enarha
    enarha committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    281e4df View commit details
    Browse the repository at this point in the history