Skip to content

Conversation

@Rajalakshmi-Girish
Copy link
Collaborator

The job postsubmit-os-origin-build-golang-master-ppc64le started throwing below error since openshift/origin#27570

+ openshift-tests run kubernetes/conformance --dry-run
  May 31 22:43:04.270: INFO: Enabling in-tree volume drivers
Attempting to pull tests from external binary...
  goroutine 1 [running]:
  runtime/debug.Stack()
  	runtime/debug/stack.go:24 +0x6c
  github.com/openshift/origin/test/extended/util.FatalErr({0x1703cfe0, 0xc0030baf90})
  	github.com/openshift/origin/test/extended/util/client.go:922 +0x3c
  github.com/openshift/origin/test/extended/util.(*CLI).AdminConfig(0x0?)
  	github.com/openshift/origin/test/extended/util/client.go:750 +0x58
  github.com/openshift/origin/test/extended/util.(*CLI).AdminConfigClient(0xc0030f5680)
  	github.com/openshift/origin/test/extended/util/client.go:663 +0x30
  github.com/openshift/origin/pkg/test/ginkgo.extractBinaryFromReleaseImage({0x17a59910, 0x9}, {0x17a88905, 0x12})
  	github.com/openshift/origin/pkg/test/ginkgo/external.go:77 +0xa8
  github.com/openshift/origin/pkg/test/ginkgo.externalTestsForSuite({0x18689750, 0x1bf09bc0})
  	github.com/openshift/origin/pkg/test/ginkgo/external.go:33 +0x58
  github.com/openshift/origin/pkg/test/ginkgo.(*Options).Run(0xc000cf4c40, 0x1b4de700, {0x17a75661, 0xf})
  	github.com/openshift/origin/pkg/test/ginkgo/cmd_runsuite.go:168 +0x210
  main.newRunCommand.func1.1()
  	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:330 +0x298
  main.mirrorToFile(0x0?, 0x0?)
  	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:467 +0xf0
  main.newRunCommand.func1(0xc000ce1800?, {0xc00063eb80?, 0x4?, 0x17a4af0b?})
  	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:311 +0x88
  github.com/spf13/cobra.(*Command).execute(0xc000ae1800, {0xc00063eb60, 0x2, 0x2})
  	github.com/spf13/cobra@v1.6.0/command.go:916 +0x740
  github.com/spf13/cobra.(*Command).ExecuteC(0xc000ae1500)
  	github.com/spf13/cobra@v1.6.0/command.go:1040 +0x370
  github.com/spf13/cobra.(*Command).Execute(...)
  	github.com/spf13/cobra@v1.6.0/command.go:968
  main.main.func1(0xc000aa2600?)
  	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:96 +0x8c
  main.main()
  	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:97 +0x498

panic: �[1m�[38;5;9mYour Test Panicked�[0m

For now, exporting OPENSHIFT_SKIP_EXTERNAL_TESTS=true is avoiding this error.

But the code at https://github.com/openshift/origin/blob/master/pkg/test/ginkgo/cmd_runsuite.go#L166-L168 has to be modified and made better to look for env variable and the corresponding adminconfig file for the openshift cluster required.

@ppc64le-cloud-bot ppc64le-cloud-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 8, 2023
Copy link
Contributor

@mkumatag mkumatag left a comment

Choose a reason for hiding this comment

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

/lgtm

@ppc64le-cloud-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mkumatag, Rajalakshmi-Girish

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

@ppc64le-cloud-bot ppc64le-cloud-bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jun 9, 2023
@ppc64le-cloud-bot ppc64le-cloud-bot merged commit 731d862 into ppc64le-cloud:master Jun 9, 2023
@ppc64le-cloud-bot
Copy link
Contributor

@Rajalakshmi-Girish: Updated the job-config configmap in namespace prow at cluster default using the following files:

  • key openshift-postsubmit.yaml using file config/jobs/ppc64le-cloud/builds/openshift-postsubmit.yaml
Details

In response to this:

The job postsubmit-os-origin-build-golang-master-ppc64le started throwing below error since openshift/origin#27570

+ openshift-tests run kubernetes/conformance --dry-run
 May 31 22:43:04.270: INFO: Enabling in-tree volume drivers
Attempting to pull tests from external binary...
 goroutine 1 [running]:
 runtime/debug.Stack()
 	runtime/debug/stack.go:24 +0x6c
 github.com/openshift/origin/test/extended/util.FatalErr({0x1703cfe0, 0xc0030baf90})
 	github.com/openshift/origin/test/extended/util/client.go:922 +0x3c
 github.com/openshift/origin/test/extended/util.(*CLI).AdminConfig(0x0?)
 	github.com/openshift/origin/test/extended/util/client.go:750 +0x58
 github.com/openshift/origin/test/extended/util.(*CLI).AdminConfigClient(0xc0030f5680)
 	github.com/openshift/origin/test/extended/util/client.go:663 +0x30
 github.com/openshift/origin/pkg/test/ginkgo.extractBinaryFromReleaseImage({0x17a59910, 0x9}, {0x17a88905, 0x12})
 	github.com/openshift/origin/pkg/test/ginkgo/external.go:77 +0xa8
 github.com/openshift/origin/pkg/test/ginkgo.externalTestsForSuite({0x18689750, 0x1bf09bc0})
 	github.com/openshift/origin/pkg/test/ginkgo/external.go:33 +0x58
 github.com/openshift/origin/pkg/test/ginkgo.(*Options).Run(0xc000cf4c40, 0x1b4de700, {0x17a75661, 0xf})
 	github.com/openshift/origin/pkg/test/ginkgo/cmd_runsuite.go:168 +0x210
 main.newRunCommand.func1.1()
 	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:330 +0x298
 main.mirrorToFile(0x0?, 0x0?)
 	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:467 +0xf0
 main.newRunCommand.func1(0xc000ce1800?, {0xc00063eb80?, 0x4?, 0x17a4af0b?})
 	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:311 +0x88
 github.com/spf13/cobra.(*Command).execute(0xc000ae1800, {0xc00063eb60, 0x2, 0x2})
 	github.com/spf13/cobra@v1.6.0/command.go:916 +0x740
 github.com/spf13/cobra.(*Command).ExecuteC(0xc000ae1500)
 	github.com/spf13/cobra@v1.6.0/command.go:1040 +0x370
 github.com/spf13/cobra.(*Command).Execute(...)
 	github.com/spf13/cobra@v1.6.0/command.go:968
 main.main.func1(0xc000aa2600?)
 	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:96 +0x8c
 main.main()
 	github.com/openshift/origin/cmd/openshift-tests/openshift-tests.go:97 +0x498

panic: �[1m�[38;5;9mYour Test Panicked�[0m

For now, exporting OPENSHIFT_SKIP_EXTERNAL_TESTS=true is avoiding this error.

But the code at https://github.com/openshift/origin/blob/master/pkg/test/ginkgo/cmd_runsuite.go#L166-L168 has to be modified and made better to look for env variable and the corresponding adminconfig file for the openshift cluster required.

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.

@Rajalakshmi-Girish
Copy link
Collaborator Author

Raised openshift/origin#27975

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. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants