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.

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

[Backport] Fix oauth SARs for interactive login #617

Closed
wants to merge 30 commits into from
Closed

Commits on Oct 21, 2022

  1. Move the SNMP trap delivery checks (#381) (#384)

    * Move the SNMP trap delivery checks
    
    Move the SNMP trap delivery checks as where they are situated now seems
    to cause false positives. Moves the checks closer to the end of the
    smoketest run seems to result in a better change that the logs the check
    is looking for have been provided.
    
    * Use a loop to check for SNMP status with break and max time
    
    (cherry picked from commit 1dc0808)
    leifmadsen authored Oct 21, 2022
    Configuration menu
    Copy the full SHA
    b0289e5 View commit details
    Browse the repository at this point in the history

Commits on Oct 28, 2022

  1. Lock the bundle to OCP v4.10 (#385) (#386)

    (cherry picked from commit 1560d3c)
    leifmadsen authored Oct 28, 2022
    Configuration menu
    Copy the full SHA
    0c4d8aa View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2022

  1. Fixes for 17.0 ir script (#380) (#383)

    (cherry picked from commit 69d73b7)
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    Co-authored-by: Matthias Runge <mrunge@redhat.com>
    3 people authored Nov 8, 2022
    Configuration menu
    Copy the full SHA
    9c5b82a View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2022

  1. Make all certs 8yr expiry (#387) (#389)

    * Make all certs 8yr expiry
    * Use certificate_duration and test against generated cert
    * Better messages during CI cloning
    
    (cherry picked from commit 4b5d7a1)
    csibbitt authored Nov 14, 2022
    Configuration menu
    Copy the full SHA
    907dbc4 View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2022

  1. Update metadata to reference stable-1.5 (#397)

    Update the metadata to reference stable-1.5 so that we can perform
    nightly builds against upstream stable-1.5 repository and result in an
    index image that can be referenced by a CatalogSource manifest to allow
    installation of STF 1.5 content.
    leifmadsen authored Nov 30, 2022
    Configuration menu
    Copy the full SHA
    90ae8ee View commit details
    Browse the repository at this point in the history

Commits on Feb 28, 2023

  1. GHA checkout action v2 is deprecated (#407) (#408)

    The GitHub Actions checkout action v2 is deprecated and needs to move to
    version 3.
    
    (cherry picked from commit 7687cd7)
    leifmadsen authored Feb 28, 2023
    Configuration menu
    Copy the full SHA
    c3868c4 View commit details
    Browse the repository at this point in the history

Commits on Mar 1, 2023

  1. CI change to pre-clean cert-manager-operator (#409)

    * not 100% sure this is 4.12 related, but it's new and first seen during testing 4.12
    
    (cherry picked from commit 37dceed with
    modifications)
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    leifmadsen and csibbitt authored Mar 1, 2023
    Configuration menu
    Copy the full SHA
    1dbc112 View commit details
    Browse the repository at this point in the history

Commits on Mar 7, 2023

  1. STF 1.5.1 release ops (#413)

    * Fixes for 17.0 ir script (#380)
    
    * Move the SNMP trap delivery checks (#381)
    
    * Move the SNMP trap delivery checks
    
    Move the SNMP trap delivery checks as where they are situated now seems
    to cause false positives. Moves the checks closer to the end of the
    smoketest run seems to result in a better change that the logs the check
    is looking for have been provided.
    
    * Use a loop to check for SNMP status with break and max time
    
    * Lock the bundle to OCP v4.10 (#385)
    
    * Make all certs 8yr expiry
    
    * Revert "Make all certs 8yr expiry"
    
    This reverts commit af35714.
    
    * Make all certs 8yr expiry (#387)
    
    * Make all certs 8yr expiry
    * Use certificate_duration and test against generated cert
    * Better messages during CI cloning
    
    * Expand support for OCP 4.11 (#391)
    
    * Expand support for OCP 4.11
    
    Allow installation to be done on OCP 4.11 while updating the smoketest
    jobs to support later versions of the client. Also migrate to using
    community-operators CatalogSource instead of OperatorHub.io. Only enable
    community-operators when the use_community strategy is enabled.
    
    Update the token request syntax when requesting a service account token.
    Add checks to look for oc client version and fail if we're using a
    version that's too old.
    
    * Make passwords safer in smoketest job template
    
    Encapsulate the password values with double quotes to help make them
    safer for consumption in the template. I had an odd situation where the
    password contained a bunch of extended characters and caused the
    smoketest to report an error on the template having an issue with yaml
    to json.
    
    The password contained several characters such as . and : which confused
    the template. Wrapping the contents in the double quotes allowed the
    smoketest to apply the job.batch template and result in a working
    smoketest run.
    
    * Force SGO checkout during build (#388)
    
    * Replacing the placeholder namespace during the build results in a "there are local changes" error on next build
    * This forces the checkout to discard that (and other!?) local changes
    * Quicker dev/test loop
    
    * Update oc to 4.11 in jenkins agent (#393)
    
    * Update oc to 4.11 in jenkins agent
    
    Need 4.11 for new token handling changes
    
    * Remove OperatorHub.io as a CatalogSource (#394)
    
    Remove the OperatorHub.io CatalogSource and instead use the
    community-operators CatalogSource which is available with an OCP
    installation. Ideally this will avoid some of the conflicts we've been
    seeing in our CI environment. This is a short term fix as future
    development will likely make use of Observability Operator to provide
    the metrics data store and alert delivery mechanism.
    
    * Changes for 4.12 (#401)
    
    * Catalog changes
    * CI change to pre-clean cert-manager-operator
      * not 100% sure this is 4.12 related, but it's new and first seen during testing 4.12
    
    * Remove Loki from stf-run-ci (#405)
    
    * Remove Loki from stf-run-ci
    
    * Return "Get new operator sdk" to stf-run-ci
    
    * GHA checkout action v2 is deprecated (#407)
    
    The GitHub Actions checkout action v2 is deprecated and needs to move to
    version 3.
    
    * Implement SNMPtrap delivery controls (#404)
    
    * Implement SNMPtrap delivery controls
    
    Implement ability to override the default values for the SNMPtrap
    alertmanager receiver via prometheus-webhook-snmp component.
    
    Closes: STF-559
    
    * Run operator-sdk generate bundle
    
    Run the following command to update the bundle artifacts:
    
    operator-sdk-0.19.4 generate bundle   --metadata   --manifests   --channels unstable   --default-channel unstable
    
    * Build out the remaining SNMP options
    
    Build out the remaining options for prometheus-webhook-snmp to allow for
    finer grained controls and delivery of SNMP traps via alertmanager
    alerts.
    
    * Generate bundle contents with operator-sdk
    
    * Implement changes for operator-sdk-1.26.0 testing (#411)
    
    * Implement changes for operator-sdk-1.26.0 testing
    
    Implement changes that allow testing validation via operator-sdk-1.26.0
    without bumping the entire bundle generation process from
    operator-sdk-0.19.4 to post-operator-sdk-1.x.
    
    These are the same tests run for validation during product pipeline
    verification.
    
    * Adds test to verify building of the bundle image works.
    * Adds KinD deployment to allow executing scorecard checks.
    
    Related: STF-1252
    
    * Fix properties.yaml
    
    * Simplify use of RELEASE_VERSION variable (#412)
    
    * Add note about why we're copying files in
    
    * Expose ability to set certificate renewal target times (#406)
    
    * Adds duration param for CA and endpoint certs
    
    Replaces certificate_duration for ca_certificate_duration
    and endpoint_certificate_duration. Set default value for those
    to 70080h (previous value)
    
    Removes the certificate_duration param from the Issuer
    resource since it's not actually needed (see [0])
    
    [0] https://cert-manager.io/docs/reference/api-docs/#cert-manager.io/v1.IssuerConfig
    
    * Exposes CA and endpoint certificate duration config
    
    Exposes certificate duration config for both ElasticSearch
    and QDR
    
    Keeps the default value in use for now. Better default values
    should be discussed to be included in a follow up change.
    
    * Fix identation for certs duration param in servicetelemetry crd
    
    * Adds cert duration to the OLM catalog
    
    Includes cert duration params in the OLM catalog
    for both ElasticSearch and QDR
    
    * Changes snake_case to camelCase to yaml case
    
    Fix to match style convention
    
    * Adds pattern expresion for certs duration
    
    * Add certificates param to events and transport
    
    * Exposes duration parameter in the CI script
    
    Adds the duration parameter for both ElasticSearch and QDR
    in the CI script
    
    Also updates the OLM Catalog with the latest changes (certificates object)
    
    * Corrects naming to certificates params in CI script
    
    * Fix snake cae in the CI script params for cert duration
    
    * Fix identation for transports in the deploy_stf CI script
    
    ---------
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    
    ---------
    
    Co-authored-by: Leif Madsen <lmadsen@redhat.com>
    Co-authored-by: Jaromír Wysoglad <jwysogla@redhat.com>
    Co-authored-by: Victoria Martinez de la Cruz <victoria@redhat.com>
    4 people authored Mar 7, 2023
    Configuration menu
    Copy the full SHA
    ecc5e5e View commit details
    Browse the repository at this point in the history

Commits on Mar 8, 2023

  1. Fix default CA cert lifetime values in stf-run-ci (#414)

    (cherry picked from commit e3a1125)
    csibbitt committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    59f0442 View commit details
    Browse the repository at this point in the history
  2. fix/client version mismatch (#415)

    * Fix PROMETHEUS_K8S_TOKEN to account for oc version mismatch
    
    * Fix to use correct variable name in test
    
    (cherry picked from commit 7c66ed8)
    leifmadsen committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    2085c15 View commit details
    Browse the repository at this point in the history

Commits on Mar 9, 2023

  1. Allow oc client version override for Jenkins agent (#416) (#417)

    (cherry picked from commit 4d2f348)
    leifmadsen authored Mar 9, 2023
    Configuration menu
    Copy the full SHA
    c1fc1db View commit details
    Browse the repository at this point in the history

Commits on Oct 26, 2023

  1. Configuration menu
    Copy the full SHA
    2403533 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8693f46 View commit details
    Browse the repository at this point in the history

Commits on Oct 27, 2023

  1. Drop .zuul.yaml for the stable-1.5 branch

    Drop .zuul.yaml for stable-1.5 since it's not setup for non-main testing
    at this point. In the future we may develop a separate set of tests for
    the stable-1.5 branch during merge, but not for this initial import.
    We'll rely on Jenkins testing for our functional validations.
    leifmadsen committed Oct 27, 2023
    Configuration menu
    Copy the full SHA
    e50dc5f View commit details
    Browse the repository at this point in the history
  2. Merge pull request #509 from infrawatch/import/stf153

    Import master to stable-1.5
    leifmadsen authored Oct 27, 2023
    Configuration menu
    Copy the full SHA
    03a5873 View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2023

  1. Fix qdr auth one_time_upgrade label check (#518) (#521)

    * Fix qdr auth one_time_upgrade label check
    
    * Fix incorrect variable naming on one_time_upgrade label check
    
    * Adjust QDR authentication password generation (#520)
    
    Adjust the passwords being generated for QDR authentication since
    certain characters (such as colon) will cause a failure in the parsing
    routine within qpid-dispatch. Updates the lookup function to only use
    ascii_letters and digits and increases the length to 32 characters.
    
    ---------
    
    Co-authored-by: Leif Madsen <lmadsen@redhat.com>
    csibbitt and leifmadsen authored Nov 6, 2023
    Configuration menu
    Copy the full SHA
    b063b92 View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2023

  1. QDR Auth in smoketest (#525) (#527)

    * QDR Auth in smoketest
    
    * Added qdr-test as a mock of the OSP-side QDR
    * Connection from qdr-test -> default-interconnect is TLS+Auth
    * Collectors point at qdr-test instead of default-interconnect directly
    * Much more realistic than the existing setup
    * Eliminated a substitution in sensubility config
    * Used default QDR basic auth in Jenkinsfile
    
    (cherry picked from commit 37b6f03)
    csibbitt authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    c896687 View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2023

  1. QDR Auth for infrared 17.1 script (#517) (#528)

    * QDR Auth for infrared 17.1 script
    
    * Fix missing substitution for AMQP_PASS in infrared script
    
    (cherry picked from commit d12aa38)
    csibbitt authored Nov 13, 2023
    Configuration menu
    Copy the full SHA
    94e343c View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2023

  1. Restart QDR after changing the password (#530) (#534)

    * Restart QDR after changing the password
    
    * Fixes bug reported here: #517 (comment)
    * Avoids an extra manual step when changing password
    * Would affect users who upgrade from earlier STF and subsequently enable basic auth
    * Also users who need to change their passwords
    
    * Fixing ansible lint
    
    * Update roles/servicetelemetry/tasks/component_qdr.yml
    
    * Adjust QDR restarts to account for HA
    
    * [smoketest] Wait for qdr-test to be Running
    
    * [smoketest] Wait for QDR password upgrade
    
    * Remove zuul QDR auth override
    
    (cherry picked from commit 16b8197)
    csibbitt authored Nov 14, 2023
    Configuration menu
    Copy the full SHA
    edebfb5 View commit details
    Browse the repository at this point in the history

Commits on Nov 16, 2023

  1. Support OCP v4.12 through v4.14 (#535) (#536)

    Support STF 1.5.3 starting at OpenShift version 4.12 due to
    incompatibility with 4.11 due to dependency requirements. Our primary
    target is support of OCP EUS releases.
    
    Closes: STF-1632
    (cherry picked from cba3874)
    leifmadsen authored Nov 16, 2023
    Configuration menu
    Copy the full SHA
    03269cd View commit details
    Browse the repository at this point in the history

Commits on Nov 27, 2023

  1. Adjust Operator dependency version requirements (#538) (#543)

    * Adjust Operator dependency version requirements (#538)
    
    Adjust the operator package dependency requirements to align to known
    required versions. Primarily reduce the version of
    openshift-cert-manager from 1.10 to 1.7 in order to support the
    tech-preview channel which was previously used.
    
    Lowering the version requirement allows for the
    openshift-cert-manager-operator installed previously to be used during
    the STF 1.5.2 to 1.5.3 update, removing the update from being blocked.
    
    Related: STF-1636
    (cherry picked from commit 77dea87)
    
    * Only require Interconnect and Smart Gateway (#541)
    
    * Only require Interconnect and Smart Gateway
    
    Update the dependency management within Service Telemetry Operator to
    only require AMQ Interconnect and Smart Gateway Operator, which is
    enough to deploy STF with observabilityStrategy: none. Other Operators
    can be installed in order to satisfy data storage of telemetry and
    events.
    
    Installation of cert-manager is also required, but needs to be
    pre-installed similar to Cluster Observability Operator, either as a
    cluster-scoped operator with the tech-preview channel, or a single time
    on the cluster as a namespace scoped operator, which is how the
    stable-v1 channel installs.
    
    Documentation will be updated to adjust for this change.
    
    Related: STF-1636
    
    * Perform CI update to match docs install changes (#542)
    
    * Perform CI update to match docs install changes
    
    Update the stf-run-ci scripting to match the documented installation
    procedures which landed in
    infrawatch/documentation#513. These changes are
    also reflected in #541.
    
    * Update build/stf-run-ci/tasks/setup_base.yml
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    * Also drop cert-manager project
    
    The cert-manager project gets created with workload items when deploying
    the cert-manager from the cert-manager-operator project. When removing
    cert-manager this project is not cleaned up, so we need to delete it as
    well.
    
    ---------
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    (cherry picked from commit ba9c918)
    leifmadsen authored Nov 27, 2023
    Configuration menu
    Copy the full SHA
    b321394 View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2024

  1. Add optional spec.replaces field to CSV for update graph compliance (#…

    …572)
    
    The way we generate our CSVs uses OLM's skipRange functionality. This is fine,
    but using only this leads to older versions becoming unavailable after the
    fact -- see the warning at [1].
    
    By adding an optional spec.replaces to our CSV we allow update testing as
    well as actual production updates for downstream builds that leverage it.
    
    Populating the field requires knowledge of the latest-released bundle,
    so we take it from an environment variable to be provided by the
    builder. If this is unset we don't include the spec.replaces field at
    all -- leaving previous behavior unchanged.
    
    Resolves #559
    Related: STF-1658
    
    [1] https://olm.operatorframework.io/docs/concepts/olm-architecture/operator-catalog/creating-an-update-graph/#skiprange
    
    (cherry picked from commit 99221fb)
    compi-migui authored Feb 7, 2024
    Configuration menu
    Copy the full SHA
    4ffd1ab View commit details
    Browse the repository at this point in the history

Commits on Feb 14, 2024

  1. STF 1.5.4 release ops (#574)

    * Add gitleaks.toml for rh-gitleaks (#510)
    
    Add a .gitleaks.toml file to avoid the false positive leak for the
    example certificate when deploying for Elasticsearch.
    
    * [stf-collect-logs] Move describe build|pod from ci/ to the role (#505)
    
    * [stf-run-ci] Fix check to include bool filter (#511)
    
    Update the check to use bool filter instead of a bar var.
    By default, ansible parses vars as strings, and without the | bool
    filter, this check is invalid, as it will always resolve to true, since
    it is a non-empty string. Other instances of the same check did this,
    but this one was missed.
    
    * [allow_skip_clone] Allow skipping of the cloning stages (#512)
    
    * [allow_skip_clone] Use <repo>_dir instead of hardcoding all directories relative to base_dir
    
    This will allow configuration of the repo clone destination, so we can
    use pre-cloned dirs instead of explicitly cloning the dirs each time.
    
    This is essential for CI systems like zuul, that set-up the repos with
    particular versions/branches prior to running the test scripts.
    
    * [zuul] List the other infrawatch repos as required for the job
    
    * [zuul] Set the {sgo,sg-bridge,sg-core,prometheus-webhook-snmp}_dir vars
    
    Add in the repo dir locations where the repos should be pre-cloned by
    zuul
    
    * Replace base_dir with sto_dir
    
    * set sto_dir relative to base_dir is it isn't already set
    
    * [ci] use absolute dir for requirements.txt
    
    * [ci] Update sto_dir using explicit reference
    
    zuul.project.src_dir refers to the current project dir. When using the jobs
    in another infrawatch project, this becomes invalid.
    Instead, sto_dir is explicitly set using
    zuul.projects[<project_name>].src_dir, the same way that the other repo dirs
    are set in vars-zuul-common
    
    ---------
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    
    * Fix qdr auth one_time_upgrade label check (#518)
    
    * Fix qdr auth one_time_upgrade label check
    
    * Fix incorrect variable naming on one_time_upgrade label check
    
    * Adjust QDR authentication password generation (#520)
    
    Adjust the passwords being generated for QDR authentication since
    certain characters (such as colon) will cause a failure in the parsing
    routine within qpid-dispatch. Updates the lookup function to only use
    ascii_letters and digits and increases the length to 32 characters.
    
    ---------
    
    Co-authored-by: Leif Madsen <lmadsen@redhat.com>
    
    * Add docs for skip_clone (#515)
    
    * [allow_skip_clone] Add docs for clone_repos and *_dir vars
    
    * Align README table column spacing (#516)
    
    * Align README table column spacing
    
    * Update build/stf-run-ci/README.md
    
    ---------
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: Leif Madsen <lmadsen@redhat.com>
    
    * [zuul] Add STO to required repos (#524)
    
    It appears that STO is not included explictly when running jobs from
    SGO [1]. This will be the case in all the other repos.
    This change explicitly add it, in case it's not already included by
    zuul.
    
    [1] https://review.rdoproject.org/zuul/build/edd8f17bfdac4360a94186b46c4cea3f
    
    * QDR Auth in smoketest (#525)
    
    * QDR Auth in smoketest
    
    * Added qdr-test as a mock of the OSP-side QDR
    * Connection from qdr-test -> default-interconnect is TLS+Auth
    * Collectors point at qdr-test instead of default-interconnect directly
    * Much more realistic than the existing setup
    * Eliminated a substitution in sensubility config
    * Used default QDR basic auth in Jenkinsfile
    
    * QDR Auth for infrared 17.1 script (#517)
    
    * QDR Auth for infrared 17.1 script
    
    * Fix missing substitution for AMQP_PASS in infrared script
    
    * [zuul] Define a project template for stf-crc-jobs (#514)
    
    * [allow_skip_clone] Use <repo>_dir instead of hardcoding all directories relative to base_dir
    
    This will allow configuration of the repo clone destination, so we can
    use pre-cloned dirs instead of explicitly cloning the dirs each time.
    
    This is essential for CI systems like zuul, that set-up the repos with
    particular versions/branches prior to running the test scripts.
    
    * [zuul] List the other infrawatch repos as required for the job
    
    * [zuul] Set the {sgo,sg-bridge,sg-core,prometheus-webhook-snmp}_dir vars
    
    Add in the repo dir locations where the repos should be pre-cloned by
    zuul
    
    * Replace base_dir with sto_dir
    
    * set sto_dir relative to base_dir is it isn't already set
    
    * [ci] use absolute dir for requirements.txt
    
    * [ci] Update sto_dir using explicit reference
    
    zuul.project.src_dir refers to the current project dir. When using the jobs
    in another infrawatch project, this becomes invalid.
    Instead, sto_dir is explicitly set using
    zuul.projects[<project_name>].src_dir, the same way that the other repo dirs
    are set in vars-zuul-common
    
    * [zuul] Define a project template for stf-crc-jobs
    
    Instead of listing all the jobs for each preoject in-repo, and needing to update the list every time
    that a new job is added, the project template can be updated and the changes propogated to the
    other infrawatch projects
    
    * [zuul] don't enable using the template
    
    * Revert "[zuul] don't enable using the template"
    
    This reverts commit 56e2009.
    
    ---------
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    
    * Restart QDR after changing the password (#530)
    
    * Restart QDR after changing the password
    
    * Fixes bug reported here: #517 (comment)
    * Avoids an extra manual step when changing password
    * Would affect users who upgrade from earlier STF and subsequently enable basic auth
    * Also users who need to change their passwords
    
    * Fixing ansible lint
    
    * Update roles/servicetelemetry/tasks/component_qdr.yml
    
    * Adjust QDR restarts to account for HA
    
    * [smoketest] Wait for qdr-test to be Running
    
    * [smoketest] Wait for QDR password upgrade
    
    * Remove zuul QDR auth override
    
    * [zuul] Add jobs to test with different versions of OCP (#432)
    
    
    * Add crc_ocp_bundle value to select OCP version
    * zuul: add log collection post-task to get crc logs
    * Add ocp v13 and a timeout to the job
    
    * Update README for 17.1 IR test (#533)
    
    * Update README for 17.1 IR test
    
    Update the 17.1 infrared test script README to show how to deploy a
    virtualized workload on the deployed overcloud infrastructure. Helps
    with testing by providing additional telemetry to STF required in
    certain dashboards.
    
    * Update tests/infrared/17.1/README.md
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    
    * Update tests/infrared/17.1/README.md
    
    ---------
    
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    
    * Support OCP v4.12 through v4.14 (#535)
    
    Support STF 1.5.3 starting at OpenShift version 4.12 due to
    incompatibility with 4.11 due to dependency requirements. Our primary
    target is support of OCP EUS releases.
    
    Closes: STF-1632
    
    * [stf-collect-logs] Add ignore_errors to task (#529)
    
    The "Question the deployment" task didn't have
    ignore_errors: true set, so when the task fails, the play
    is finished. This means that we don't get to the
    "copy logs" task and can't see the job logs in zuul.
    
    ignore_errors is set to true to be consistent with other tasks
    
    * Mgirgisf/stf 1580/fix log commands (#526)
    
    * update stf-collect-logs tasks
    * Update log path
    * solve log bugs in stf-run-ci tasks
    * create log directory
    
    * Adjust Operator dependency version requirements (#538)
    
    Adjust the operator package dependency requirements to align to known
    required versions. Primarily reduce the version of
    openshift-cert-manager from 1.10 to 1.7 in order to support the
    tech-preview channel which was previously used.
    
    Lowering the version requirement allows for the
    openshift-cert-manager-operator installed previously to be used during
    the STF 1.5.2 to 1.5.3 update, removing the update from being blocked.
    
    Related: STF-1636
    
    * Clean up stf-run-ci for OCP 4.12 minimum version (#539)
    
    Update the stf-run-ci base setup to no longer need testing against OCP
    4.10 and earlier, meaning we can rely on a single workflow for
    installation. Also update the deployment to use
    cluster-observability-operator via the redhat-operators CatalogSource
    for installation via use_redhat and use_hybrid strategies.
    
    * [zuul] Add job to build locally and do an index-based deployment (#495)
    
    * [zuul] Add job to build locally and do an index-based deployment
    
    * Only require Interconnect and Smart Gateway (#541)
    
    * Only require Interconnect and Smart Gateway
    
    Update the dependency management within Service Telemetry Operator to
    only require AMQ Interconnect and Smart Gateway Operator, which is
    enough to deploy STF with observabilityStrategy: none. Other Operators
    can be installed in order to satisfy data storage of telemetry and
    events.
    
    Installation of cert-manager is also required, but needs to be
    pre-installed similar to Cluster Observability Operator, either as a
    cluster-scoped operator with the tech-preview channel, or a single time
    on the cluster as a namespace scoped operator, which is how the
    stable-v1 channel installs.
    
    Documentation will be updated to adjust for this change.
    
    Related: STF-1636
    
    * Perform CI update to match docs install changes (#542)
    
    * Perform CI update to match docs install changes
    
    Update the stf-run-ci scripting to match the documented installation
    procedures which landed in
    infrawatch/documentation#513. These changes are
    also reflected in #541.
    
    * Update build/stf-run-ci/tasks/setup_base.yml
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    * Also drop cert-manager project
    
    The cert-manager project gets created with workload items when deploying
    the cert-manager from the cert-manager-operator project. When removing
    cert-manager this project is not cleaned up, so we need to delete it as
    well.
    
    ---------
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    * [stf-run-ci] Explicitly check the validate_daployment was successful (#545)
    
    In [1], the validate_deployment step is successful, despite the
    deployment not being successful.
    This causes the job to timeout because the following steps continue to
    run despite an invalid state.
    
    To get the expected behaviour, the output should be checked for a string
    indicating success.
    i.e. * [info] CI Build complete. You can now run tests.
    [2] shows the output for a successful run.
    
    [1] https://review.rdoproject.org/zuul/build/245ae63e41884dc09353d938ec9058d7/console#5/0/144/controller
    [2] https://review.rdoproject.org/zuul/build/802432b23da24649b818985b7b1633bb/console#5/0/82/controller
    
    * Implement dashboard management (#548)
    
    * Implement dashboard management
    
    Implement a new configuration option graphing.grafana.dashboards.enabled
    which results in dashboards objects being created for the Grafana
    Operator. Previously loading dashboards would be done manually via 'oc
    apply' using instructions from documentation.
    
    The new CRD parameters to the ServiceTelemetry object allows the Service
    Telemetry Operator to now make the GrafanaDashboard objects directly.
    
    Related: OSPRH-825
    
    * Drop unnecessary cluster roles
    
    * Update CSV for owned parameter
    
    * Remove basic-auth method from grafana (#550)
    
    * Only openshift auth will be allowed
    
    * Adjust Alertmanager SAR to be more specific
    
    * This matches recent changes in prometheus[1] and grafana[2]
    
    [1] https://github.com/infrawatch/service-telemetry-operator/pull/549/files#diff-2cf84bcf66f12393c86949ec0d3f16c473a650173d55549bb02556d23aa22bd2R46
    [2] https://github.com/infrawatch/service-telemetry-operator/pull/550/files#diff-ae71801975adb4f8dd4aa5479a66ad46e46f17de40f9d147b2e09e13ce26633eR45
    
    * Revert "Adjust Alertmanager SAR to be more specific"
    
    This reverts commit 0f94fd5.
    
    * Auth to prometheus using token instead of basicauth (#549)
    
    * Auth to prometheus using token instead of basicauth
    
    * Add present/absent logic to prometheus-reader resources
    
    * s/password/token in smoketest output
    
    * [zuul] Make nightly_bundles jobs non-voting (#551)
    
    ---------
    
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    
    * Fix branch co-ordination in stf-run-ci (#555)
    
    I think it got broken by an oops recently[1].
    
    Since that change, working_branch (`branch` at that point) is never used because version_branches.sgo has a default value.
    
    This breaks the branch co-ordination in Jenkins[2] and in local testing[3].
    
    [1] https://github.com/infrawatch/service-telemetry-operator/pull/512/files#diff-c073fe1e346d08112920aa0bbc8a7453bbd3032b7a9b09ae8cbc70df4db4ea2dR19
    [2] https://github.com/infrawatch/service-telemetry-operator/blob/0f94fd577617aee6a85fc4141f98ebdfc49a9f92/Jenkinsfile#L157
    [3] https://github.com/infrawatch/service-telemetry-operator/blob/0f94fd577617aee6a85fc4141f98ebdfc49a9f92/README.md?plain=1#L62
    
    * Adjust Alertmanager SAR to be more specific (#553)
    
    * This matches recent changes in prometheus[1] and grafana[2]
    
    [1] https://github.com/infrawatch/service-telemetry-operator/pull/549/files#diff-2cf84bcf66f12393c86949ec0d3f16c473a650173d55549bb02556d23aa22bd2R46
    [2] https://github.com/infrawatch/service-telemetry-operator/pull/550/files#diff-ae71801975adb4f8dd4aa5479a66ad46e46f17de40f9d147b2e09e13ce26633eR45
    
    * Add optional spec.replaces field to CSV for update graph compliance
    
    The way we generate our CSVs uses OLM's skipRange functionality. This is fine,
    but using only this leads to older versions becoming unavailable after the
    fact -- see the warning at [1].
    
    By adding an optional spec.replaces to our CSV we allow update testing as
    well as actual production updates for downstream builds that leverage it.
    
    Populating the field requires knowledge of the latest-released bundle,
    so we take it from an environment variable to be provided by the
    builder. If this is unset we don't include the spec.replaces field at
    all -- leaving previous behavior unchanged.
    
    Resolves #559
    Related: STF-1658
    
    [1] https://olm.operatorframework.io/docs/concepts/olm-architecture/operator-catalog/creating-an-update-graph/#skiprange
    
    * Stop using ephemeral storage for testing (#547)
    
    Update the __service_telemetry_storage_persistent_storage_class to use CRC PVs
    Use the default value (false) for __service_telemetry_storage_ephemeral_enabled
    
    * [zuul] Use extracted CRC nodes in stf-base (#531)
    
    * [zuul] Update base job for stf-base
    
    * Add in required projects: dataplane-operator, infra-operator, openstack-operator
    
    * Remove nodeset from stf-base
      it overrides the nodeset set in the base job.
      The nodeset is going to be used to select the OCP version
    
    * [zuul] define nodesets for easy reuse
    
    * Define the nodeset
    * Rename the base
    * Select OCP version with the nodeset
    
    * [zuul] Add a login command to get initial kubeconfig file
    
    * [stf-run-ci] Add retries to pre-clean
    
    * Update galaxy requirements
    
    * [ci] Add retry to login command
    
    * [ci] Configure kubeconfig for rhol_crc role
    
    * Apply suggestions from code review
    
    * Zuul: Update how we get the initial kubeconfig (#558)
    
    * use ci-framework infra playbook
    * add make targets to do set-up
    * link the kubeconfig files
    * Remove pre-get_kubeconfig.yml; the script is no longer used
    
    * [ci] Add common-tasks.yml to cover the tasks that setup every play (#556)
    
    * [zuul] Update the labels used for extracted CRC
    
    * Remove non-default cifmw_rhol_crc_kubeconfig value
    
    * Implement support for Grafana Operator v5 (#561)
    
    * Implement support for Grafana Operator v5
    
    Implement changes to support Grafana Operator v5 when the new
    grafana.integreatly.org CRD is available. Use the new CRDs as default
    when they are available. Fallover to deploying with Grafana Operator v4
    when the Grafana Operator v5 CRDs are not available, thereby providing
    backwards compatibility to allow administrators time to migrate.
    
    Additionally, the polystat plugin has been removed from the rhos-cloud
    dashboard due to compatibility issues with grafana-cli usage when
    dynamically loading plugins. Usage of Grafana Operator v5 is also a
    target for disconnected support, and dynamically loading plugins in
    these environments is expected to be a problem.
    
    Related: OSPRH-2577
    Closes: STF-1667
    
    * Default Grafana role set to Admin
    
    In order to match the previous (Grafana Operator v4) role, set
    auto_assign_org_role to the Admin value. Default is Viewer.
    
    * Remove old vendored operator_sdk/util collection (#563)
    
    Remove the old 0.1.0 vendored collection operator_sdk/util from the
    upstream Dockerfile and repository. Instead use the default
    operator_sdk/util in the base image which is a newer version of 0.4.0.
    
    We only use the util collection for one call to k8s_status when
    ephemeral storage is enabled. The newer collection also provides a
    k8s_event module which could be useful in the future.
    
    Closes: STF-1683
    
    * Add nightly_bundle jobs to periodic pipeline (#564)
    
    The nightly_bundle jobs will run once a day
    
    * Remove hard-coded Prometheus version in template (#565)
    
    Remove the hard-coded Prometheus version in the Prometheus template when
    using observabilityStrategy use_redhat, which uses Cluster Observability
    Operator to manage the Prometheus instance requests.
    
    Previously this value was hard-coded to prevent a potential rollback
    when moving from Community Prometheus Operator to Cluster Observability
    Operator.
    
    Resolves: JIRA#OSPRH-2140
    
    * Set features.operators.openshift.io/disconnected to True (#570)
    
    STF can now be deployed in disconnected mode. This change updates
    the features.operators.openshift.io/disconnected annotation to
    reflect this.
    
    * [stf-run-ci] Update validation check for bundle URLs (#571)
    
    * [stf-run-ci] Update validation check for bundle URLs
    
    An empty string passed as the bundle URL will pass the existing test
    of "is defined" and "is not None" and still be invalid.
    
    The validation for the bundle URL can be done in one check per var:
    
    * If the var is undefined, it becomes "", and the check fails, because of length
    * If the var is None, there's an error because None does not have a length
    * If the var is an empty string, the check fails because of the length
    
    This simplifies the check and improves readability
    
    * Prefer Grafana 9 workload (#575)
    
    Prefer usage of Grafana 9 container image from RHCC. Grafana 7 is EOL
    upstream and receives no security support. Prefer use of Grafana 9 which
    is still supported.
    
    ---------
    
    Co-authored-by: Leif Madsen <lmadsen@redhat.com>
    Co-authored-by: Emma Foley <elfiesmelfie@users.noreply.github.com>
    Co-authored-by: Chris Sibbitt <csibbitt@redhat.com>
    Co-authored-by: Marihan Girgis <102027102+mgirgisf@users.noreply.github.com>
    Co-authored-by: Miguel Garcia <migarcia@redhat.com>
    6 people authored Feb 14, 2024
    Configuration menu
    Copy the full SHA
    ad468f2 View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2024

  1. Configuration menu
    Copy the full SHA
    cf20fdf View commit details
    Browse the repository at this point in the history

Commits on Feb 16, 2024

  1. Add related images for Prometheus and Alertmanager (#578) (#580)

    * Add related images for Prometheus and Alertmanager
    
    Add support for RELATED_IMAGES for other artifacts we need in a default
    deployment when installing via disconnected.
    
    These images are required to be specifically called out in disconnected
    environments as the Prometheus and Alertmanager controllers in the
    Cluster Observability Operator (COO) don't use the RELATED_IMAGE
    environment variables specified in the downstream CSV for COO, so we
    need to specify them ourselves and then deploy Prometheus and
    Alertmanager using the spec.image reference in the Custom Resource.
    
    Closes: JIRA#STF-1713
    
    * Place related images in correct deploy manifest
    
    * Use grafana_path_image by default
    
    Remove the baseImage configuration for Grafana from the example CR so
    that in the OLM UI (OCP console) the baseImage isn't a populated value
    by default.
    
    Use the value from grafana_image_path as the preferred default as it
    will be populated from the RELATED_IMAGES_GRAFANA_IMAGE environment
    variable, which then itself has a default of the base_image value in
    defaults/main.yml. Only if the administrator sets a specific value for
    baseImage should we deploy with that image.
    
    We do this to allow for a better disconnected environment experience out
    of the box.
    
    * Drop setting Grafana related images
    
    For upstream, there isn't a good option that is on quay.io (which
    doesn't have quotas), and since it's out of scope for this effort, I'm
    just going to drop it and revert back to how it worked before.
    
    * Revert removal of Grafana default image path
    
    * Fix issue with alertmanager related image tag var
    
    Co-authored-by: Leif Madsen <lmadsen@redhat.com>
    vkmc and leifmadsen authored Feb 16, 2024
    Configuration menu
    Copy the full SHA
    f39dd05 View commit details
    Browse the repository at this point in the history

Commits on Feb 26, 2024

  1. Fix env var naming issue (#581) (#582)

    * Fix env var naming issue
    
    Fix an environment variable naming issue in the CSV for STO when
    attempting to install alertmanager disconnected. The env var being
    looked up should have a postfix of _IMAGE to match the other env vars.
    
    Found in testing by vkmc.
    
    * Run operator-sdk generate bundle
    
    Run the generate bundle command to make sure everything is in sync.
    Fixes CI found issue in previous commit.
    
    (cherry-picked from commit 6568039)
    leifmadsen authored Feb 26, 2024
    Configuration menu
    Copy the full SHA
    5f3f9a0 View commit details
    Browse the repository at this point in the history

Commits on Mar 1, 2024

  1. Update smoketest.sh (#583) (#584)

    Changed default-alertmanager URL to v2
    
    Co-authored-by: Alex Yefimov <126113326+ayefimov-1@users.noreply.github.com>
    elfiesmelfie and ayefimov-1 authored Mar 1, 2024
    Configuration menu
    Copy the full SHA
    eb7d5b0 View commit details
    Browse the repository at this point in the history

Commits on May 31, 2024

  1. [zuul] Remove OCP 4.13 jobs (#602)

    OCP 4.13 jobs were added as a way to test ocp-latest before 4.14 was released
    elfiesmelfie authored May 31, 2024
    Configuration menu
    Copy the full SHA
    cbefc68 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2024

  1. Update nodesets label (#614)

    The CI jobs does not need to be force use one cloud provider in the CI.
    This commit set the label that is defined in multiple cloud provider in
    Zuul. Available labels you can find [1].
    
    [1] https://review.rdoproject.org/zuul/labels
    danpawlik authored Jul 17, 2024
    Configuration menu
    Copy the full SHA
    3b6b1e1 View commit details
    Browse the repository at this point in the history

Commits on Jul 18, 2024

  1. Fix oauth SARs for interactive login (#612)

    - Doesn't work unless resource name is plural form
    - The "group" property is called "resourceAPIGroup" in SARs
      -"to avoid confusion with the 'groups' field when inlined"[1]
    
    [1] https://docs.openshift.com/container-platform/4.14/rest_api/authorization_apis/subjectaccessreview-authorization-openshift-io-v1.html
    csibbitt authored and vkmc committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    2daaf95 View commit details
    Browse the repository at this point in the history