Skip to content

Conversation

@petr-muller
Copy link
Member

@petr-muller petr-muller commented Jun 26, 2023

OTA maintains staging (identical to production) and integration (running on engineering candidate OCP clusters) Cincinnati instances. We are searching for traffic that we could route to especially the integration one, so that we can find possible problems with engineering candidate early.

All the instances are serving identical data (up to some minimal skew coming from when individual instance scrape their source data) so we should be able to easily use the integration instance first, then the staging one, and if needed, use the production one as a backstop.

After:

$ ci-operator --org stackrox --repo stackrox --branch master --target=[images] --git-ref=stackrox/stackrox@master
...
INFO[2023-06-26T16:11:20+02:00] Requesting 4.12 from https://api.integration.openshift.com/api/upgrades_info/v1/graph?arch=amd64&channel=fast-4.12 
INFO[2023-06-26T16:11:21+02:00] Resolved release latest to quay.io/openshift-release-dev/ocp-release@sha256:ba7956f5c2aae61c8ff3ab1ab2ee7e625db9b1c8964a65339764db79c148e4e6

Before:

$ ci-operator --org stackrox --repo stackrox --branch master --target=[images] --git-ref=stackrox/stackrox@master
...
INFO[2023-06-26T16:14:56+02:00] Requesting 4.12 from https://api.openshift.com/api/upgrades_info/v1/graph?arch=amd64&channel=fast-4.12 
INFO[2023-06-26T16:14:57+02:00] Resolved release latest to quay.io/openshift-release-dev/ocp-release@sha256:ba7956f5c2aae61c8ff3ab1ab2ee7e625db9b1c8964a65339764db79c148e4e6

We can see that the release was resolved to the same pullspec after querying both api.integration.openshift.com and api.openshift.com/api endpoints.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 26, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 26, 2023

@petr-muller: This pull request references OTA-905 which is a valid jira issue.

Details

In response to this:

  • cincinnati client: cleanup deprecated ioutil usage
  • cincinnati client: query integration & staging instances

After:

$ ci-operator --org stackrox --repo stackrox --branch master --target=[images] --git-ref=stackrox/stackrox@master
...
INFO[2023-06-26T16:11:20+02:00] Requesting 4.12 from https://api.integration.openshift.com/api/upgrades_info/v1/graph?arch=amd64&channel=fast-4.12 
INFO[2023-06-26T16:11:21+02:00] Resolved release latest to quay.io/openshift-release-dev/ocp-release@sha256:ba7956f5c2aae61c8ff3ab1ab2ee7e625db9b1c8964a65339764db79c148e4e6

Before:

$ ci-operator --org stackrox --repo stackrox --branch master --target=[images] --git-ref=stackrox/stackrox@master
...
INFO[2023-06-26T16:14:56+02:00] Requesting 4.12 from https://api.openshift.com/api/upgrades_info/v1/graph?arch=amd64&channel=fast-4.12 
INFO[2023-06-26T16:14:57+02:00] Resolved release latest to quay.io/openshift-release-dev/ocp-release@sha256:ba7956f5c2aae61c8ff3ab1ab2ee7e625db9b1c8964a65339764db79c148e4e6

We can see that the release was resolved to the same pullspec after querying both api.integration.openshift.com and api.openshift.com/api endpoints.

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/test-infra repository.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 26, 2023

@petr-muller: This pull request references OTA-905 which is a valid jira issue.

Details

In response to this:

OTA maintains staging (identical to production) and integration (running on engineering candidate OCP clusters) Cincinnati instances. We are searching for traffic that we could route to especially the integration one, so that we can find possible problems with engineering candidate early.

All the instances are serving identical data (up to some minimal skew coming from when individual instance scrape their source data) so we should be able to easily use the integration instance first, then the staging one, and if needed, use the production one as a backstop.

After:

$ ci-operator --org stackrox --repo stackrox --branch master --target=[images] --git-ref=stackrox/stackrox@master
...
INFO[2023-06-26T16:11:20+02:00] Requesting 4.12 from https://api.integration.openshift.com/api/upgrades_info/v1/graph?arch=amd64&channel=fast-4.12 
INFO[2023-06-26T16:11:21+02:00] Resolved release latest to quay.io/openshift-release-dev/ocp-release@sha256:ba7956f5c2aae61c8ff3ab1ab2ee7e625db9b1c8964a65339764db79c148e4e6

Before:

$ ci-operator --org stackrox --repo stackrox --branch master --target=[images] --git-ref=stackrox/stackrox@master
...
INFO[2023-06-26T16:14:56+02:00] Requesting 4.12 from https://api.openshift.com/api/upgrades_info/v1/graph?arch=amd64&channel=fast-4.12 
INFO[2023-06-26T16:14:57+02:00] Resolved release latest to quay.io/openshift-release-dev/ocp-release@sha256:ba7956f5c2aae61c8ff3ab1ab2ee7e625db9b1c8964a65339764db79c148e4e6

We can see that the release was resolved to the same pullspec after querying both api.integration.openshift.com and api.openshift.com/api endpoints.

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/test-infra repository.

@openshift-ci openshift-ci bot requested review from hongkailiu and jmguzik June 26, 2023 14:20
@petr-muller petr-muller force-pushed the ota-905-integration-cincinnati-in-ci-operator branch from 9de6e03 to acf2256 Compare July 11, 2023 16:48
OTA maintains staging (identical to production) and integration (running
on engineering candidate OCP clusters) Cincinnati instances. We are
searching for traffic that we could route to especially the integration
one, so that we can find possible problems with engineering candidate
early.

All the instances are serving identical data (up to some minimal skew
coming from when individual instance scrape their source data) so we
should be able to easily use the integration instance first, then
the staging one, and if needed, use the production one as a backstop.
@petr-muller petr-muller force-pushed the ota-905-integration-cincinnati-in-ci-operator branch from acf2256 to 886a7af Compare July 11, 2023 16:51
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 11, 2023

@petr-muller: all tests passed!

Full PR test history. Your PR dashboard.

Details

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/test-infra repository. I understand the commands that are listed here.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 11, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 11, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hongkailiu, petr-muller

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 11, 2023
@openshift-merge-robot openshift-merge-robot merged commit 2020667 into openshift:master Jul 11, 2023
@petr-muller petr-muller deleted the ota-905-integration-cincinnati-in-ci-operator branch July 12, 2023 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants