Test output: Add links to hcp logs #3633

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

Open

janboll wants to merge 8 commits into main from log-links

Collaborator

janboll commented Dec 17, 2025 •

edited

Loading

What

Create links to Hosted Control Plane logs

Needs a bit more polishing, just wanted to show the progress made

Why

Special notes for your reviewer

openshift-ci bot requested review from bennerv and deads2k

December 17, 2025 13:35

openshift-ci bot commented Dec 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: janboll

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:

~~test/OWNERS~~ [janboll]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci bot added the approved label

stevekuznetsov reviewed

View reviewed changes

test/cmd/aro-hcp-tests/custom-link-tools/options.go Show resolved Hide resolved

stevekuznetsov reviewed

View reviewed changes

test/cmd/aro-hcp-tests/custom-link-tools/options.go Outdated Show resolved Hide resolved

stevekuznetsov reviewed

View reviewed changes

test/cmd/aro-hcp-tests/custom-link-tools/options.go Outdated Show resolved Hide resolved

stevekuznetsov reviewed

View reviewed changes

test/cmd/aro-hcp-tests/custom-link-tools/options.go Outdated Show resolved Hide resolved

stevekuznetsov reviewed

View reviewed changes

test/cmd/aro-hcp-tests/custom-link-tools/options.go Outdated Show resolved Hide resolved

stevekuznetsov reviewed

View reviewed changes

test/cmd/aro-hcp-tests/custom-link-tools/options.go Outdated Show resolved Hide resolved

janboll force-pushed the log-links branch 2 times, most recently from dadc26e to fe07c66 Compare

December 18, 2025 15:51

janboll added 3 commits

December 19, 2025 08:19


          Add links for each test

bc23610

Parse out the resource group affected and then create links for each affected resourcegroup per test


          Add some tests, refactor queries to be faster/more useable

b99f222


          rebase

bb0dde6

janboll force-pushed the log-links branch from fe07c66 to bb0dde6 Compare

December 19, 2025 08:20

janboll added 4 commits

December 19, 2025 09:28


          Apparently metadata is written to shared dir, look it up there

afdf6bd


          Fix filename lookup

fdfbd76


          Fix fixture

36b6bf5


          Fix tests

6420b80

janboll requested a review from stevekuznetsov

December 19, 2025 12:20


          Less data

5c55c80

openshift-ci bot commented Dec 19, 2025

@janboll: 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/e2e-parallel	`5c55c80`	link	true	`/test e2e-parallel`

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-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

Labels