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

Does vSphere iso on Packer allow adding SCSI Controllers and assigning hard disks to SCSI Controllers? #9518

Closed
amitbhadra opened this issue Jul 1, 2020 · 15 comments · Fixed by #9519
Labels

Comments

@amitbhadra
Copy link

From the vSphere UI, I can add "New SCSI Controller". My use case is to create 4 SCSI Controllers. And when I create a new hard disk, I want to assign each hard disk to some SCSI Controller.

So essentially, suppose I have 8 hard disks and 4 SCSI Controllers, and I want to add Hard disks 1 and 2 to SCSI 0 as 0:0 and 0:1, Hard disks 3 and 4 to SCSI 1 as 1:0 and 1:1, etc.

But I see no such option in Packer. When I create 8 Hard disks, they all get assigned to SCSI 0 as 0:0, 0:1, 0:2, 0:3 .. 0:7.

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 1, 2020

The https://www.packer.io/docs/builders/vmware/vsphere-iso#disk_controller_type lets you define what kind of controller is used. Currently only one controller is supported.

@amitbhadra
Copy link
Author

@jhawk28 Do you know if this is possible on Terraform?

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 1, 2020

I don't know about Terraform, but my pull request (#9519) should be able to add the capability.

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 1, 2020

@amitbhadra
Copy link
Author

Hey so will the config now look something like this?

"disk_controller_type" : [ "scsi", "scsi"],
"storage": [
    {
      "disk_size": 15000,
      "disk_controller_index": 0
    },
    {
      "disk_size": 20000,
      "disk_controller_index": 0
    },
    {
      "disk_size": 20000,
      "disk_controller_index": 1
    },
    {
      "disk_size": 20000,
      "disk_controller_index": 1
    }
],

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 2, 2020

exactly

@amitbhadra
Copy link
Author

Hi, your code changes exactly fit my use case! Thanks so much.
But, I must mention his. I don't know if something messed up the stability of the packer binary. I have tried with almost similar parameters(instead of 4 scsi, I'm working with 4 pvscsi controllers). Out of 7 runs, it errored out 3 times.

The first error out was a "general server fault" but rest assured there was no such fault. This time, it was typing out the boot command and in between, it failed.
Second and third errors happened when it finished typing the boot command and it was stuck at a network unreachable for about 5 mins. This was solved when I stopped the run and started a new run.

It'd be great if there's a stable binary for this.

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 2, 2020

this build would include the new boot command stuff: #9406
@sylviamoss

@amitbhadra
Copy link
Author

Thanks, will you have a new Packer official release with these changes any time soon?

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 2, 2020

No idea, I just create pull requests. They seem to release about once a month.

@amitbhadra
Copy link
Author

Please keep this issue open, I'll report back on the stability of the Packer binary you gave me. Or if you deliver newer binaries, I can test them out too.

@jhawk28
Copy link
Contributor

jhawk28 commented Jul 2, 2020

I think they would prefer to have a new issue open for the boot stability issues. @SwampDragons would know.

@amitbhadra
Copy link
Author

I'll make new issues with possible screenshots as and when I get them.

@SwampDragons
Copy link
Contributor

Correct, I would like a new issue for any new stability issues. Sounds like this one can be closed once I merge 9519.

@ghost
Copy link

ghost commented Aug 10, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Aug 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants