Skip to content

Conversation

@wking
Copy link
Member

@wking wking commented Sep 19, 2022

So that it's easier to guess at the fraction of requests that are CVOs vs. other clients. I'd like to set this for our signature requests too (vendor/github.com/openshift/library-go/pkg/verify/store/sigstore), but that would take some library-go requests first.

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 19, 2022
@wking
Copy link
Member Author

wking commented Sep 19, 2022

I tested this in cluster bot with launch 4.12.0-0.nightly-2022-09-19-161258,openshift/cluster-version-operator#839 gcp, set channel, and pointed ClusterVersion's spec.upstream at the invalid https://api.stage.openshift.com/api/upgrades_info/wking-cvo-839. That gave logs like:

$ oc -n cincinnati-stage get -o name pods | while read POD; do oc -n cincinnati-stage logs -c cincinnati-policy-engine "${POD}" | grep wking; done
[2022-09-19T23:03:09Z ERROR policy_engine] Error serving request 'Method: 'GET', Request: '/api/upgrades_info/wking-cvo-839', Query: 'arch=amd64&channel=wking-cvo-839&id=d9d35431-c35b-4425-bb47-35001cc4da70&version=4.12.0-ec.3', User-Agent: 'Go-http-client/1.1', Accept: 'application/json'' from '10.131.28.30:53010': Incorrect Endpoint
[2022-09-19T23:05:57Z ERROR policy_engine] Error serving request 'Method: 'GET', Request: '/api/upgrades_info/wking-cvo-839', Query: 'arch=amd64&channel=wking-cvo-839&id=d9d35431-c35b-4425-bb47-35001cc4da70&version=4.12.0-ec.3', User-Agent: 'Go-http-client/1.1', Accept: 'application/json'' from '10.131.28.30:35020': Incorrect Endpoint
[2022-09-19T23:07:54Z ERROR policy_engine] Error serving request 'Method: 'GET', Request: '/api/upgrades_info/wking-cvo-839', Query: 'arch=amd64&channel=wking-cvo-839&id=d9d35431-c35b-4425-bb47-35001cc4da70&version=0.0.1-0.test-2022-09-19-221623-ci-ln-l86zy4b-latest', User-Agent: 'ClusterVersionOperator/1.0.0-899-ge3101963-dirty', Accept: 'application/json'' from '10.131.28.30:34674': Incorrect Endpoint

As I updated back and forth between 4.12.0-ec.3 and the cluster-bot-built commit to check the behavior vs. the current dev branch. It shows my change improving the User-Agent from ec.3's Go-http-client/1.1 to my PR's ClusterVersionOperator/1.0.0-899-ge3101963-dirty. With ART builds, we'll get the same version number they compile us with, like:

$ curl -s https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/periodic-ci-openshift-release-master-nightly-4.12-e2e-aws-sdn-serial/1570267348072402944/artifacts/e2e-aws-sdn-serial/gather-extra/artifacts/pods/openshift-cluster-version_cluster-version-operator-7789fd6cc6-gknjr_cluster-version-operator.log | head -n1
I0915 04:53:34.063931       1 start.go:23] ClusterVersionOperator 4.12.0-202209132338.p0.g583c3e2.assembly.stream-583c3e2

for 4.12.0-ec.3.

@wking wking force-pushed the set-user-agent-for-cincinnati branch from cc32403 to 7d0e9bd Compare September 19, 2022 23:15
@wking
Copy link
Member Author

wking commented Sep 23, 2022

/retest

@openshift-bot
Copy link
Contributor

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci openshift-ci bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 23, 2022
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 23, 2022
So that it's easier to guess at the fraction of requests that are CVOs
vs. other clients.  I'd like to set this for our signature requests
too
(vendor/github.com/openshift/library-go/pkg/verify/store/sigstore),
but that would take some library-go requests first.
@wking wking force-pushed the set-user-agent-for-cincinnati branch from 7d0e9bd to 1dd2378 Compare December 23, 2022 19:36
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 23, 2022
@wking
Copy link
Member Author

wking commented Dec 23, 2022

Rebased onto master with 7d0e9bd -> 1dd2378.

/remove-lifecycle stale

@openshift-ci openshift-ci bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 23, 2022
@petr-muller
Copy link
Member

/retest

Copy link
Member

@petr-muller petr-muller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

openshift-ci bot commented Jan 3, 2023

[APPROVALNOTIFIER] This PR is APPROVED

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

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-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD d3a03ab and 2 for PR HEAD 1dd2378 in total

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 3, 2023

@wking: 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-merge-robot openshift-merge-robot merged commit cdc1b51 into openshift:master Jan 3, 2023
@wking wking deleted the set-user-agent-for-cincinnati branch September 18, 2023 21:45
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. 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