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

govc: Attach Disk to SCSI Controller #2117

Open
gothicx opened this issue Sep 14, 2020 · 11 comments
Open

govc: Attach Disk to SCSI Controller #2117

gothicx opened this issue Sep 14, 2020 · 11 comments

Comments

@gothicx
Copy link

gothicx commented Sep 14, 2020

Hi,

It would be useful to have a device.scsi.attach to attach a disk to a scsi controller.

govc device.scsi.attach -vm $vm pvscsi-1001

Thank you.

@dougm
Copy link
Member

dougm commented Sep 18, 2020

I think we could add a -controller flag to vm.disk.change to support this. What do you think?

@gothicx
Copy link
Author

gothicx commented Sep 19, 2020

That's ok for me. Thank you !

@github-actions
Copy link
Contributor

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@gothicx
Copy link
Author

gothicx commented Mar 24, 2021

/remove-lifecycle stale

@github-actions
Copy link
Contributor

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Mark as fresh by adding the comment /remove-lifecycle stale.

@gothicx
Copy link
Author

gothicx commented Jun 23, 2021

/remove-lifecycle stale

@jbertozzi
Copy link

jbertozzi commented Jun 28, 2021

Hello,

It seems the code has been modifed to add this functionnality.

However, it seems not to be working (using a VSAN datastore):

&types.LocalizedMethodFault{
    Fault: &types.SystemError{
        RuntimeFault: types.RuntimeFault{
            MethodFault: types.MethodFault{
                FaultCause:   (*types.LocalizedMethodFault)(nil),
                FaultMessage: []types.LocalizableMessage{
                    {
                        Key:     "vob.fssvec.CreateFile.file.failed",
                        Arg:     nil,
                        Message: "File system specific implementation of CreateFile[file] failed",
                    },
                    {
                        Key:     "vob.fssvec.CreateFile.file.failed",
                        Arg:     nil,
                        Message: "File system specific implementation of CreateFile[file] failed",
                    },
                    {
                        Key:     "vob.fssvec.CreateFile.file.failed",
                        Arg:     nil,
                        Message: "File system specific implementation of CreateFile[file] failed",
                    },
                },
            },
        },
        Reason: "Error creating disk Function not implemented",
    },
    LocalizedMessage: "A general system error occurred: Error creating disk Function not implemented",
}

Should I open a new issue for this?

@embano1
Copy link
Contributor

embano1 commented Jun 28, 2021

Yes, if this issue is not correctly reflecting the bug you're seeing, please open a new one.

@gothicx
Copy link
Author

gothicx commented Jun 28, 2021

@jbertozzi This bug was reported to add a controller to an existing disk and not while creating a new VM, so you probably want to report a new bug.

@jbertozzi
Copy link

New issue opened:

#2568

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2021

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Mark as fresh by adding the comment /remove-lifecycle stale.

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

No branches or pull requests

4 participants