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

IsLikelyNotMountPoint issue when upgrade k8s.io/mount-utils from v0.29.4 to v0.30.2 #1975

Open
andyzhangx opened this issue Jul 12, 2024 · 2 comments

Comments

@andyzhangx
Copy link
Member

andyzhangx commented Jul 12, 2024

What happened:
need to find out whether it's ok to upgrade to k8s.io/mount-utils, what's the change and impact in vendor/k8s.io/mount-utils/mount_linux.go

I0712 03:39:30.760769   14023 driver.go:93] Enabling controller service capability: EXPAND_VOLUME
    nodeserver_test.go:1040: 
        	Error Trace:	/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/nodeserver_test.go:1040
        	Error:      	Received unexpected error:
        	            	rpc error: code = Internal desc = Could not mount target ./target_test: not a directory
        	Test:       	TestNodePublishVolumeIdempotentMount
    nodeserver_test.go:1042: 
        	Error Trace:	/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/nodeserver_test.go:1042
        	Error:      	Received unexpected error:
        	            	rpc error: code = Internal desc = Could not mount target ./target_test: not a directory
        	Test:       	TestNodePublishVolumeIdempotentMount
    nodeserver_test.go:1056: 
        	Error Trace:	/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/nodeserver_test.go:1056
        	Error:      	Not equal: 
        	            	expected: 1
        	            	actual  : 0
        	Test:       	TestNodePublishVolumeIdempotentMount
--- FAIL: TestNodePublishVolumeIdempotentMount (0.01s)

986ddc9#diff-9d276589000ecf169e4bb915a05a8c339bcb5e6c09c3d7734f915eedc246a404

What you expected to happen:

How to reproduce it:

Anything else we need to know?:

Environment:

  • CSI Driver version:
  • Kubernetes version (use kubectl version):
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:
@andyzhangx
Copy link
Member Author

/assign @cvvz

@andyzhangx
Copy link
Member Author

duplicated to issue mentioned here: #2055 (comment):

the reimplemented IsLikelyNotMountPoint func in mount-utils would make one unit test fail, current IsLikelyNotMountPoint would return error on an empty dir with not a directory, I think that's a bug, we need to fix it in upstream k8s main repo.

=== RUN   TestNodePublishVolumeIdempotentMount
I0817 02:55:14.201401   14208 driver.go:93] Enabling controller service capability: CREATE_DELETE_VOLUME
I0817 02:55:14.201441   14208 driver.go:93] Enabling controller service capability: EXPAND_VOLUME
    nodeserver_test.go:1040: 
        	Error Trace:	/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/nodeserver_test.go:1040
        	Error:      	Received unexpected error:
        	            	rpc error: code = Internal desc = Could not mount target ./target_test: not a directory
        	Test:       	TestNodePublishVolumeIdempotentMount
    nodeserver_test.go:1042: 
        	Error Trace:	/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/nodeserver_test.go:1042
        	Error:      	Received unexpected error:
        	            	rpc error: code = Internal desc = Could not mount target ./target_test: not a directory
        	Test:       	TestNodePublishVolumeIdempotentMount
    nodeserver_test.go:1056: 
        	Error Trace:	/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/nodeserver_test.go:1056
        	Error:      	Not equal: 
        	            	expected: 1
        	            	actual  : 0
        	Test:       	TestNodePublishVolumeIdempotentMount

@andyzhangx andyzhangx changed the title TestNodePublishVolumeIdempotentMount failure when upgrade k8s.io/mount-utils from v0.29.4 to v0.30.2 IsLikelyNotMountPoint issue when upgrade k8s.io/mount-utils from v0.29.4 to v0.30.2 Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants