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

Fix: sysctl integration test - virtualization type #226

Merged

Conversation

mandar242
Copy link
Contributor

SUMMARY

sysctl integration tests inside docker are being skipped as the check ansible_facts.virtualization_type == 'docker' fails.
On Debugging, ansible_facts.virtualization_type is being returned as "container".

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

sysctl

ADDITIONAL INFORMATION

Ansible Version

  • ansible [core 2.11.2]

Docker Version

  • Docker version 20.10.7, build f0df350

OS

  • Fedora 34

Actual Results

PLAY RECAP *********************************************************************
testhost                   : ok=1    changed=0    unreachable=0    failed=0    skipped=44   rescued=0    ignored=0   

Likely this code is related: https://github.com/ansible/ansible/blob/devel/lib/ansible/module_utils/facts/virtual/linux.py#L113

@gravesm
Copy link
Member

gravesm commented Jul 20, 2021

From what I can tell the tests are being run. The latest build on main: https://dev.azure.com/ansible/ansible.posix/_build/results?buildId=20594&view=logs&j=a9e0bcaa-f9fa-54f9-1e7a-69855170566b&t=a3d86cda-daaa-54cd-38ff-e247a2c0bdcd&l=1268. Are you seeing otherwise?

@mandar242
Copy link
Contributor Author

From what I can tell the tests are being run. The latest build on main: https://dev.azure.com/ansible/ansible.posix/_build/results?buildId=20594&view=logs&j=a9e0bcaa-f9fa-54f9-1e7a-69855170566b&t=a3d86cda-daaa-54cd-38ff-e247a2c0bdcd&l=1268. Are you seeing otherwise?

Yes, I'm seeing skipped as virtualization_type is being shown "container" while running the tests with --docker

@Akasurde Akasurde force-pushed the fix-integration-test-sysctl branch from 76de604 to 244c9b2 Compare August 4, 2021 16:31
@Akasurde Akasurde added the gate Gate PR in Zuul CI (Obsolete: Please set "mergeit" instead of "gate") label Aug 5, 2021
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gate Gate PR in Zuul CI (Obsolete: Please set "mergeit" instead of "gate")
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants