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

[TECH DEBT] Deprecate ESXi execution modules in favor of VMware Salt extensions #62754

Closed
dmurphy18 opened this issue Sep 26, 2022 · 2 comments · Fixed by #63002
Closed

[TECH DEBT] Deprecate ESXi execution modules in favor of VMware Salt extensions #62754

dmurphy18 opened this issue Sep 26, 2022 · 2 comments · Fixed by #63002
Assignees

Comments

@dmurphy18
Copy link
Contributor

dmurphy18 commented Sep 26, 2022

Description of the tech debt to be addressed, include links and screenshots

Given these execution modules currently do not get much attention, esp. since 2016, and all VMware support is currently occurring on the VMware extensions. Any functionality currently provided by these modules and not support by VMware extensions, should be added to VMware extensions.

Should have a single point of development for VMware support, which is currently VMware extensions.
Removes chance of confusion as to which execution module to use.

Also note that the ESXi execution modules do not fully function with currently released VMware software, since they have not been maintained sufficiently to kleep up to date with VMware API changes.

Execution Modules to deprecate:

  • esxcluster.py
  • esxdatacenter.py
  • esxi.py
  • esxvm.py

Note associated state files:

  • esxcluster.py
  • esxdatacenter.py
  • esxi.py
  • esxvm.py

These state file will need to be checked to ensure they function with VMware Salt Extensions.

@dmurphy18 dmurphy18 added this to the Sulphur v3006.0 milestone Sep 26, 2022
@dmurphy18 dmurphy18 self-assigned this Sep 26, 2022
@dmurphy18
Copy link
Contributor Author

Note: these execution modules were the basis for the start of the VMware Salt Extensions, hence expect no issues with compatibility, but have to check.

@dmurphy18
Copy link
Contributor Author

dmurphy18 commented Sep 26, 2022

From initial review of the state files provided in Salt (salt/states/esxcluster.py), the same named state function is not provided in VMware Salt Extensions, hence expect need to port or alias similar functionality in VMware Salt Extensions.

Also what to do about the execution module vsphere.py

These files also expect to be operating on a proxy-minion, various appropriately named esxvm, esxdatacenter, etc.

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