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

Mm feature/add lookup tests #529

Conversation

mikemorency
Copy link
Collaborator

@mikemorency mikemorency commented Sep 10, 2024

SUMMARY

Including a playbook that can be used to test the lookup plugins. Its designed to be run manually and needs inputs from a specific environment.

Once automated testing is set up, this test can be modified to require less inputs from the user

Also included a bugfix which was found while testing. Lookng up the contents of a datacenter would raise an exception instead of returning the correct results

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

Lookup plugins

Copy link
Contributor

Copy link
Contributor

Copy link
Collaborator

@anna-savina anna-savina left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

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

✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 14m 37s
✔️ build-ansible-collection SUCCESS in 7m 42s
✔️ ansible-galaxy-importer SUCCESS in 4m 21s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit fab0aff into ansible-collections:main Sep 12, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants