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

vsphere-iso: resource rool ignored when resource pool parent is another resource pool #304

Closed
adeturner opened this issue Sep 5, 2023 · 1 comment · Fixed by #342
Closed
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@adeturner
Copy link

adeturner commented Sep 5, 2023

Overview of the Issue

When there is a resource pool hierarchy such as

Cluster-1 > RP-TEST > RP-TEST-packer

  1. if you specify resource_pool = "RP-TEST" it works
  2. if you specify resource_pool = "RP-TEST-packer" it is ignored and the Cluster level resource pool is used
  3. if you specify resource_pool = "RP-TEST/RP-TEST-packer" it works but is not documented

This Line limits the path

A documentation fix might be sufficient

Reproduction Steps

  1. Create resource pool hierarchy and build the VM

Packer Version

  • 1.9.4

Plugin Version and Builders

Please provide the plugin version.

Latest

packer {
required_version = ">= 1.9.1"
vsphere = {
version = ">= v1.2.0"
source = "github.com/hashicorp/vsphere"
}
}

Please select the builder.

  • [] vsphere-iso

VMware vSphere Version

7..0.3

Guest Operating System

  • Ubuntu 22.04 LTS x64

Simplified Packer Buildfile

source "vsphere-iso" "linux-ubuntu" {

  // vCenter Server Endpoint Settings and Credentials
  vcenter_server      = var.vsphere_endpoint
  username            = var.vsphere_username
  password            = var.vsphere_password
  insecure_connection = var.vsphere_insecure_connection

  // vSphere Settings
  datacenter    = var.vsphere_datacenter
  cluster       = var.vsphere_cluster
  datastore     = var.vsphere_datastore
  folder        = var.vsphere_folder
  resource_pool = var.vsphere_resource_pool

  // Virtual Machine Settings
  vm_name              = local.vm_name
  guest_os_type        = local.vm_guest_os_type

Operating System and Environment Details

  • github runner

Log Fragments and crash.log Files

Nothing in log

@adeturner adeturner added the bug label Sep 5, 2023
@tenthirtyam tenthirtyam added documentation Improvements or additions to documentation and removed bug labels Oct 13, 2023
@tenthirtyam tenthirtyam self-assigned this Oct 13, 2023
tenthirtyam added a commit that referenced this issue Dec 5, 2023
- Updates the documentation for location items.
- Updates the `resource_pool` note the need for a full path.

Ref: #304

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
tenthirtyam added a commit that referenced this issue Dec 5, 2023
- Updates the documentation for location items.
- Updates the `resource_pool` to note the need for a full path to a nested resource pool.

Ref: #304

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
tenthirtyam added a commit that referenced this issue Dec 6, 2023
- Updates the documentation for location items.
- Updates the `resource_pool` to note the need for a full path to a nested resource pool.

Ref: #304

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
nywilken pushed a commit that referenced this issue Dec 7, 2023
- Updates the documentation for location items.
- Updates the `resource_pool` to note the need for a full path to a nested resource pool.

Ref: #304

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam
Copy link
Collaborator

Resolved in #342 for the next release.

@tenthirtyam tenthirtyam changed the title vsphere-iso: ResourcePool ignored when resource pool parent is another resource pool vsphere-iso: resource rool ignored when resource pool parent is another resource pool Dec 20, 2023
@hashicorp hashicorp locked and limited conversation to collaborators Jun 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants