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

runtime error on "blobfuse2 mount all" #1032

Closed
williamohara opened this issue Jan 13, 2023 · 2 comments · Fixed by #1034
Closed

runtime error on "blobfuse2 mount all" #1032

williamohara opened this issue Jan 13, 2023 · 2 comments · Fixed by #1034
Assignees
Labels
Milestone

Comments

@williamohara
Copy link

Which version of blobfuse was used?

2

Which OS distribution and version are you using?

NAME="Ubuntu"
VERSION="20.04.5 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.5 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

running through WSL

If relevant, please share your mount command.

blobfuse2 mount all /mnt/azstorage

What was the issue encountered?

runtime error

panic: runtime error: index out of range [4] with length 4

goroutine 1 [running]:
github.com/Azure/azure-storage-fuse/v2/cmd.mountAllContainers(0xc00009e960, 0x2, 0x2, 0x0, 0x0, 0x7ffe7fbe3956, 0xe, 0x0, 0xe79982, 0x9)
        /mnt/vss/_work/1/s/azure-storage-fuse/cmd/mount_all.go:321 +0xf12
github.com/Azure/azure-storage-fuse/v2/cmd.processCommand(0x0, 0x0)
        /mnt/vss/_work/1/s/azure-storage-fuse/cmd/mount_all.go:180 +0x599
github.com/Azure/azure-storage-fuse/v2/cmd.glob..func9(0x149b920, 0xc00034b270, 0x1, 0x1, 0x0, 0x0)
        /mnt/vss/_work/1/s/azure-storage-fuse/cmd/mount_all.go:82 +0x9f
github.com/spf13/cobra.(*Command).execute(0x149b920, 0xc00034b250, 0x1, 0x1, 0x149b920, 0xc00034b250)
        /home/cloudtest/go/pkg/mod/github.com/gapra-msft/[email protected]/command.go:882 +0x472
github.com/spf13/cobra.(*Command).ExecuteC(0x149c100, 0x3, 0x3, 0xc00040a7c0)
        /home/cloudtest/go/pkg/mod/github.com/gapra-msft/[email protected]/command.go:1000 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
        /home/cloudtest/go/pkg/mod/github.com/gapra-msft/[email protected]/command.go:928
github.com/Azure/azure-storage-fuse/v2/cmd.Execute(0xffffffff, 0xc000042118)
        /mnt/vss/_work/1/s/azure-storage-fuse/cmd/root.go:262 +0x8d
main.main()
        /mnt/vss/_work/1/s/azure-storage-fuse/main.go:45 +0x25

Have you found a mitigation/solution?

I looked at the code and it seems that the logic is expecting there to be 5 cli parms when there is only 4 in my command here...
https://github.com/Azure/azure-storage-fuse/blob/main/cmd/mount_all.go#L321

So I added an innocuous param (for me at least) --disable-version-check=true and it seems to work now.

I would expect an error message of some kind if it is missing a required parm rather than a runtime error

Please share logs if available.

@vibhansa-msft
Copy link
Member

"blobfuse2 mount all /mnt/azstorage"
Mount all needs a config file containing the azure storage account configuration. If it's not provided, then you shall have correct environment variables exported for the same.
Also, in your mount command you do not have tmp-path set. Kindly use correct command syntax. We will look into doing a graceful termination in case required inputs are not provided.

@souravgupta-msft
Copy link
Member

Thanks for reporting this issue. The fix has been merged and will be part of our next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants