Skip to content

Modify the proxmox VM when using clone in the salt-cloud profile if settings are present#55060

Merged
dwoz merged 14 commits intosaltstack:masterfrom
Akm0d:master-proxmox-vm
Jan 9, 2020
Merged

Modify the proxmox VM when using clone in the salt-cloud profile if settings are present#55060
dwoz merged 14 commits intosaltstack:masterfrom
Akm0d:master-proxmox-vm

Conversation

@Akm0d
Copy link
Contributor

@Akm0d Akm0d commented Oct 18, 2019

What does this PR do?

Port #49256 to master

@Akm0d Akm0d force-pushed the master-proxmox-vm branch from e3d34cc to f010c57 Compare October 18, 2019 20:41
@dwoz dwoz requested a review from a team as a code owner December 3, 2019 15:31
@ghost ghost requested a review from Ch3LL December 3, 2019 15:32
@waynew
Copy link
Contributor

waynew commented Dec 3, 2019

Is Tidy up pep8 issues the wrong title for this? Looks like it also needs tests.

@waynew waynew added the needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label Dec 3, 2019
@Akm0d Akm0d changed the title Tidy up pep8 issues Modify the proxmox VM when using clone in the salt-cloud profile if settings are present Dec 11, 2019
@Akm0d
Copy link
Contributor Author

Akm0d commented Dec 12, 2019

re-run full all

@Akm0d
Copy link
Contributor Author

Akm0d commented Dec 19, 2019

re-run full m2crypto

@Akm0d Akm0d removed the needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label Jan 7, 2020
@Akm0d Akm0d requested a review from dwoz January 8, 2020 17:03
@dwoz dwoz merged commit 28236b8 into saltstack:master Jan 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants