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

vmware_host_acceptance: Remove acceptance_level #1877

Merged

Conversation

mariolenz
Copy link
Collaborator

SUMMARY

Fixes #1872

Removing acceptance_level and move its options to state so that

acceptance_level: 'community'
state: present

will become

state: 'community'

This also means there won't be a state list anymore. In order to get information about the current acceptance level, I will introduce a new module vmware_host_acceptance_info.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

vmware_host_acceptance
vmware_host_acceptance_info

ADDITIONAL INFORMATION

@mariolenz mariolenz added bug This issue/PR relates to a bug WIP Work in progress new_module New module tests tests labels Oct 13, 2023
@mariolenz mariolenz force-pushed the issue1872 branch 7 times, most recently from d4ea0cc to ccec2e2 Compare October 13, 2023 17:32
@mariolenz
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/613f3b063d9142149a31150a68133dc3

⚠️ ansible-tox-linters CANCELED
build-ansible-collection FAILURE in 7m 56s
⚠️ ansible-test-cloud-integration-vcenter7_only-stable215 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_2esxi-stable215 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_1esxi-stable215_1_of_2 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_1esxi-stable215_2_of_2 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-galaxy-importer SKIPPED Skipped due to failed job build-ansible-collection

@mariolenz
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/f8e49e483fce4bbdb4fae16b09c829fb

⚠️ ansible-tox-linters CANCELED
build-ansible-collection FAILURE in 7m 52s
⚠️ ansible-test-cloud-integration-vcenter7_only-stable215 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_2esxi-stable215 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_1esxi-stable215_1_of_2 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_1esxi-stable215_2_of_2 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-galaxy-importer SKIPPED Skipped due to failed job build-ansible-collection

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz mariolenz closed this Oct 14, 2023
@mariolenz mariolenz reopened this Oct 14, 2023
@softwarefactory-project-zuul
Copy link

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/1599fa8e1a18461d9a5e9091b8a71489

⚠️ ansible-tox-linters CANCELED
build-ansible-collection FAILURE in 8m 06s
⚠️ ansible-test-cloud-integration-vcenter7_only-stable215 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_2esxi-stable215 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_1esxi-stable215_1_of_2 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-test-cloud-integration-vcenter7_1esxi-stable215_2_of_2 SKIPPED Skipped due to failed job build-ansible-collection
⚠️ ansible-galaxy-importer SKIPPED Skipped due to failed job build-ansible-collection

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz
Copy link
Collaborator Author

recheck

@mariolenz mariolenz changed the title [WIP] vmware_host_acceptance: Remove acceptance_level vmware_host_acceptance: Remove acceptance_level Oct 15, 2023
@mariolenz mariolenz removed the WIP Work in progress label Oct 15, 2023
@mariolenz mariolenz requested a review from ihumster October 15, 2023 12:53
@mariolenz
Copy link
Collaborator Author

@ihumster Could you please review? I think it should be OK now, but you might see something that I've overlooked.

Copy link
Collaborator

@ihumster ihumster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice for both modules.

@mariolenz
Copy link
Collaborator Author

Thanks for reviewing @ihumster!

@softwarefactory-project-zuul
Copy link

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/0d96df0b33c94cffaa570fe92ba3cb1c

✔️ ansible-tox-linters SUCCESS in 8m 32s
✔️ build-ansible-collection SUCCESS in 7m 45s
✔️ ansible-galaxy-importer SUCCESS in 3m 36s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 71b7db7 into ansible-collections:main Oct 15, 2023
12 checks passed
@mariolenz mariolenz deleted the issue1872 branch October 15, 2023 14:05
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Oct 24, 2023
vmware_host_acceptance_info: Fix module doc

SUMMARY
Fix documentation of new module vmware_host_acceptance_info.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
vmware_host_acceptance_info
ADDITIONAL INFORMATION
#1877

Reviewed-by: Alexander Nikitin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug mergeit new_module New module tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vmware_host_acceptance: Remove state list
2 participants