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

Update default GOS to otherGuest64 from other-64 #1543

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Update default GOS to otherGuest64 from other-64 #1543

merged 1 commit into from
Jan 14, 2022

Conversation

tenthirtyam
Copy link
Collaborator

Description

  • Updates the default GOS to otherGuest64 from invalid other-64.
  • Updates guest_id and alternate_guest_name to correctly mention otherGuest and otherGuest64 - not other and other-64
  • Also updates the descriptions in virtual_machine_config_structure.go with more inclusive/accurate terminology, cleaner structure, and correct component proper name.

Release Note

Release note for CHANGELOG:

Updates the default GOS to `otherGuest64` from invalid `other-64`.

References

Closes: #1263

vSphere Web Services API: Enum - VirtualMachineGuestOsIdentifier

- Updates the default GOS to `otherGuest64` from `other-64` (invalid).
- Updates `guest_id` and `alternate_guest_name` to correctly mention `otherGuest` and `otherGuest64` - not `other` and `other-64`
- Also updates the descriptions in `virtual_machine_config_structure.go` with more inclusive terminology, cleaner structure, and correct component proper name.

Reference: #1263

Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
@github-actions github-actions bot added provider Type: Provider size/xs Relative Sizing: Extra-Small labels Dec 21, 2021
@appilon
Copy link
Contributor

appilon commented Jan 14, 2022

@tenthirtyam Was other/other-64 ever correct in previous versions of vsphere < 7? Particularly for version 6.5+ (which is technically still supported by the provider). If not I will merge, if yes, then we may have to make the descriptions more detailed and make do an if else in the code based on the detected version

@tenthirtyam
Copy link
Collaborator Author

The introduction timeline of other / other64 is not mentioned in vim.vm.GuestOsDescriptor.GuestOsIdentifier spec. I suspect that it's been present for at least vSphere 6.5 and later but I do not have a v6.5 testbed to validate this assumption.

other-64 does not exist in the spec.

Ryan

@appilon
Copy link
Contributor

appilon commented Jan 14, 2022

I get the sense from link you shared that other was never a valid identifier? so let's go with this change

@tenthirtyam
Copy link
Collaborator Author

Yes, that's correct @appilon.

@appilon appilon merged commit 04f9ec0 into hashicorp:master Jan 14, 2022
@tenthirtyam tenthirtyam deleted the gh-1263 branch January 19, 2022 22:59
@tenthirtyam tenthirtyam added this to the v2.1.0 milestone Feb 14, 2022
@github-actions
Copy link

github-actions bot commented Mar 2, 2022

This functionality has been released in v2.1.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

@github-actions
Copy link

github-actions bot commented Apr 2, 2022

I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
provider Type: Provider size/xs Relative Sizing: Extra-Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default guestid other-64 isn't working on vsphere_virtual_machine resource
2 participants