extended: log registry pod to artifacts directory #16859
Conversation
To preserve it for post-CI-job-debugging. Signed-off-by: Michal Minář <[email protected]>
openshift-ci-robot
added
the
size/XS
|
@bparees PTAL |
|
/test extended_image_registry |
|
/assign bparees |
| path, err := oc.Run("logs").Args("dc/docker-registry").OutputToFile("pod-" + pod.Name + ".log") | ||
|
|
||
| ocLocal := *oc | ||
| ocLocal.SetOutputDir(exutil.ArtifactDirPath()) |
There was a problem hiding this comment.
is the artifacts dir gathered? it's not obvious to me that it is: https://ci.openshift.redhat.com/jenkins/me/my-views/view/DevEx/job/test_branch_origin_extended_builds/238/s3/
There was a problem hiding this comment.
I believe exutil.ArtifactDirPath() returns a root dir for the artifacts - the one you can see in this view: https://ci.openshift.redhat.com/jenkins/view/Origin%20Test%20Jobs/job/test_branch_origin_extended_builds/238/s3/
I can see the artifacts dir in the view of prow runs: https://gcsweb-ci.svc.ci.openshift.org/gcs/origin-ci-test/pr-logs/pull/16859/test_pull_request_origin_extended_image_registry/36/
There was a problem hiding this comment.
ok, i was thinking it was the testadata fixture path, i guess i'm wrong.
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bparees, miminar The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
openshift-merge-robot
added
the
approved
|
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
Automatic merge from submit-queue. |
To preserve it for post-CI-job-debugging.
Follow-up for #15807
Addresses comment #15807 (comment)