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

Upgrade tests failing in master from file test pod already existing #8843

Closed
travisn opened this issue Sep 27, 2021 · 0 comments · Fixed by #8848
Closed

Upgrade tests failing in master from file test pod already existing #8843

travisn opened this issue Sep 27, 2021 · 0 comments · Fixed by #8848
Assignees
Labels
bug test unit or integration testing

Comments

@travisn
Copy link
Member

travisn commented Sep 27, 2021

Is this a bug report or feature request?

  • Bug Report

Deviation from expected behavior:
The upgrade tests are failing in the master branch since #8758 was merged. They must have a different permutation than what is run for PRs. For example, see this build.

Error from server (AlreadyExists): error when creating "STDIN": persistentvolumeclaims "file-test" already exists
Error from server (AlreadyExists): error when creating "STDIN": pods "file-test" already exists
2021-09-27 13:47:40.956120 E | utils: Failed to execute stdin: kubectl [create -f -] : exit status 1
2021-09-27 13:47:40.956149 E | testutil: Failed to execute kubectl [create -f -] -- Failed to run stdin: kubectl [create -f -] : Error from server (AlreadyExists): error when creating "STDIN": persistentvolumeclaims "file-test" already existsError from server (AlreadyExists): error when creating "STDIN": pods "file-test" already exists
    ceph_base_file_test.go:291: 
        	Error Trace:	ceph_base_file_test.go:291
        	            				ceph_upgrade_test.go:255
        	            				ceph_upgrade_test.go:102
        	Error:      	Received unexpected error:
        	            	failed to create pod -- 
        	            	apiVersion: v1
        	            	kind: PersistentVolumeClaim
        	            	metadata:
        	            	  name: file-test
        	            	  namespace: upgrade-ns
        	            	spec:
        	            	  accessModes:
        	            	  - ReadWriteOnce
        	            	  resources:
        	            	    requests:
        	            	      storage: 1Gi
        	            	  storageClassName: file-upgrade
        	            	---
        	            	apiVersion: v1
        	            	kind: Pod
        	            	metadata:
        	            	  name: file-test
        	            	  namespace: upgrade-ns
        	            	spec:
        	            	  containers:
        	            	  - name: file-test
        	            	    image: busybox
        	            	    command:
        	            	        - sh
        	            	        - "-c"
        	            	        - "touch /tmp/testrook/csi.test && sleep 3600"
        	            	    imagePullPolicy: IfNotPresent
        	            	    env:
        	            	    volumeMounts:
        	            	    - mountPath: /tmp/testrook
        	            	      name: csivol
        	            	  volumes:
        	            	  - name: csivol
        	            	    persistentVolumeClaim:
        	            	       claimName: file-test
        	            	       readOnly: false
        	            	  restartPolicy: Never
        	            	. Could Not create resource in args : [create -f -] -- Failed to run stdin: kubectl [create -f -] : Error from server (AlreadyExists): error when creating "STDIN": persistentvolumeclaims "file-test" already existsError from server (AlreadyExists): error when creating "STDIN": pods "file-test" already exists
        	Test:       	TestCephUpgradeSuite/TestUpgradeRookToMaster

Expected behavior:
Passing integration tests.

How to reproduce it (minimal and precise):

Trigger the CI in the master branch

@travisn travisn added bug test unit or integration testing labels Sep 27, 2021
leseb added a commit to leseb/rook that referenced this issue Sep 28, 2021
Since we were pointing at CephUpgradeSuite test only this was running
all the tests and thus some resources were created by others tests.
Let's just run the test we need.

Closes: rook#8843
Signed-off-by: Sébastien Han <seb@redhat.com>
leseb added a commit to leseb/rook that referenced this issue Sep 28, 2021
Since we were pointing at CephUpgradeSuite test only this was running
all the tests and thus some resources were created by other tests.
Let's just run the test we need.

Closes: rook#8843
Signed-off-by: Sébastien Han <seb@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug test unit or integration testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants