Skip to content

Fix ansible-doc log for automation hub#55

Closed
cloudguruab wants to merge 3 commits intoos-migrate:mainfrom
cloudguruab:ansible-doc-errors
Closed

Fix ansible-doc log for automation hub#55
cloudguruab wants to merge 3 commits intoos-migrate:mainfrom
cloudguruab:ansible-doc-errors

Conversation

@cloudguruab
Copy link
Copy Markdown
Contributor

@cloudguruab cloudguruab mentioned this pull request Jun 3, 2025
@cloudguruab
Copy link
Copy Markdown
Contributor Author

Do we care about validating go modules here?

@rebtoor
Copy link
Copy Markdown
Member

rebtoor commented Jun 4, 2025

Hey @cloudguruab , the doc validation is already done during the ansible sanity tests [1]. Here's [2] the log part where that test is performed (you don't see any output because i chose do not show any logs if the validation is fine).

Also, to reply to your question, here [3] you can see where i disabled the validation for golang modules.

Thanks for the PR but i think we can close it :)

[1] https://docs.ansible.com/ansible/latest/dev_guide/testing/sanity/index.html#all-sanity-tests
[2] https://github.com/os-migrate/vmware-migration-kit/actions/runs/15444971423/job/43471972662?pr=57#step:4:562
[3]

ansible-test sanity --python $(PYTHON_VERSION) --requirements \
--exclude aee/ \
--exclude scripts/ \
--exclude plugins/modules/create_server/ \
--exclude plugins/modules/import_image/ \
--exclude plugins/modules/migrate/ \
--exclude plugins/modules/volume_metadata_info/ && \

@cloudguruab cloudguruab closed this Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants