Skip to content
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.

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

[BUG]: Gopowermax unit test Pipeline fails to run tests #701

Closed
delldubey opened this issue Mar 6, 2023 · 1 comment
Closed

[BUG]: Gopowermax unit test Pipeline fails to run tests #701

delldubey opened this issue Mar 6, 2023 · 1 comment
Assignees
Labels
area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@delldubey
Copy link
Contributor

Bug Description

Gopowermax unit test Pipeline fails to run unit tests

Logs

[root@csipowermax gopowermax]# make unit-test go clean -cache APIVersion="" \ go test -v -coverprofile=c.out unit_test.go unit_steps_test.go authenticate.go interface.go system.go sloprovisioning.go VolumeSnapshot.go VolumeReplication.go metrics.go -args "format=pretty" "outfile=" "test-tags=" "test_paths=unittest" go: downloading github.com/sirupsen/logrus v1.4.2 go: downloading golang.org/x/sys v0.1.0 go: downloading gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c command-line-arguments [command-line-arguments.test] ./authenticate.go:185:42: cannot use &Client{…} (value of type *Client) as Pmax value in assignment: *Client does not implement Pmax (missing method CreateMigrationEnvironment) ./authenticate.go:206:9: cannot use &client (value of type *Client) as Pmax value in return statement: *Client does not implement Pmax (missing method CreateMigrationEnvironment) ./authenticate.go:212:9: cannot use c (variable of type *Client) as Pmax value in return statement: *Client does not implement Pmax (missing method CreateMigrationEnvironment) FAIL command-line-arguments [build failed] FAIL make: *** [Makefile:41: unit-test] Error 1

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

make unit-test

Expected Behavior

Successful unit test run

CSM Driver(s)

Gopowermax v2.2.0

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

NA

Operating System

RHEL 7.6

@delldubey delldubey added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. labels Mar 6, 2023
@delldubey delldubey added this to the v1.6.0 milestone Mar 6, 2023
@delldubey delldubey self-assigned this Mar 6, 2023
@csmbot
Copy link
Collaborator

csmbot commented Mar 6, 2023

@delldubey: Thank you for submitting this issue!

The issue is currently awaiting triage. Please make sure you have given us as much context as possible.

If the maintainers determine this is a relevant issue, they will remove the needs-triage label and assign an appropriate priority label.


We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at container.storage.modules@dell.com.

@delldubey delldubey added the area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax label Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

2 participants