Skip to content

Feature/deprecate lookups #608

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

Merged

Conversation

mikemorency
Copy link
Collaborator

SUMMARY

This deprecates the lookup plugins in favor of vmware.vmware.moid_from_path, since the lookup plugins in this repo will not work with ansible-core 2.19 and above

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

vmware.vmware_rest.*_moid

@mariolenz
Copy link
Collaborator

It looks like the docs test fails because the moid_from_path lookup plugin isn't part of a release yet. But I'm not 100% sure there.

I don't understand why the integration tests fail... could this be something temporarily?

@mikemorency
Copy link
Collaborator Author

I think it is a temporary thing. I feel like Ive seen the error before. Im going to wait until the vmware.vmware release is available before looking much harder

@mikemorency
Copy link
Collaborator Author

Looks like the test failures were temporary. I had to add a module link to the doc builder GHA, but now that is happy too

@mikemorency mikemorency marked this pull request as ready for review June 12, 2025 13:38
@mikemorency mikemorency requested review from mariolenz and p3ck June 12, 2025 13:38
Copy link
Collaborator

@p3ck p3ck left a comment

Choose a reason for hiding this comment

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

other than the type of changelog fragment it looks good to me

@mikemorency mikemorency merged commit b369d09 into ansible-collections:main Jun 12, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants