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

Install ec2_eni test requirements #398

Merged
merged 1 commit into from
Jul 7, 2021

Conversation

jillr
Copy link
Collaborator

@jillr jillr commented Jul 7, 2021

SUMMARY

ansible.netcommon needs the netaddr python package to be installed in the test env, add this to test-requirements.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_eni

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration needs_triage plugins plugin (any type) small_patch Hopefully easy to review tests tests labels Jul 7, 2021
@jillr jillr changed the title DNM: Test ec2_eni test requirements Install ec2_eni test requirements Jul 7, 2021
@jillr jillr added the gate label Jul 7, 2021
ansible.netcommon.ipaddr requires the netaddr package
@ansible-zuul ansible-zuul bot removed the gate label Jul 7, 2021
@jillr jillr added the gate label Jul 7, 2021
@jillr
Copy link
Collaborator Author

jillr commented Jul 7, 2021

I'm bad at git today but once this lands it should unblock #337

@ansible-zuul ansible-zuul bot merged commit ea4fd33 into ansible-collections:main Jul 7, 2021
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
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 community_review integration tests/integration needs_triage plugins plugin (any type) small_patch Hopefully easy to review tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants