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

Add intg for apt_repo's mode argument. #16690

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
7 participants
@alikins
Contributor

alikins commented Jul 12, 2016

ISSUE TYPE
  • Bugfix Pull Request
ANSIBLE VERSION
ansible 2.2.0 (add_apt_repository_intg_tests b50b800e0a) last updated 2016/07/12 16:15:29 (GMT -400)
  lib/ansible/modules/core: (detached HEAD db8af4c5af) last updated 2016/07/12 15:32:31 (GMT -400)
  lib/ansible/modules/extras: (detached HEAD 482b1a640e) last updated 2016/07/12 15:32:31 (GMT -400)
  config file = /home/adrian/.ansible.cfg
  configured module search path = Default w/o overrides

SUMMARY

Adding integration tests for testing the 'mode' arg of the apt_repository module

related to:
ansible/ansible-modules-core#4072
#16370
ansible/ansible-modules-core#3991

@alikins alikins added this to the 2.2.0 milestone Jul 12, 2016

@alikins alikins force-pushed the alikins:add_apt_repository_intg_tests branch from b50b800 Jul 13, 2016

Add intg tests for apt_repo's mode argument.
Tests the 'apt_repository' module, in particular the 'mode'
argument. The mode argument for apt_repository should work
the same as the common file arg for 'mode'. This tests verifies
that the value provided is intrepeted correctly and the correct
file mode is used.

This verifies the fix for
#16370

Add a 'test_apt_repository_mode' tag for the mode related tests.

@alikins alikins force-pushed the alikins:add_apt_repository_intg_tests branch to bc0a04a Nov 17, 2016

@mattclay

This comment has been minimized.

Member

mattclay commented Feb 1, 2017

CI failure due to:

2017-02-01 22:29:43 TASK [apt_repository : include] ************************************************
2017-02-01 22:29:43 task path: /root/ansible/test/integration/targets/apt_repository/tasks/main.yml:22
2017-02-01 22:29:43 fatal: [testhost]: FAILED! => {
2017-02-01 22:29:43     "failed": true, 
2017-02-01 22:29:43     "reason": "the file_name '/root/ansible/test/integration/mode.yml' does not exist, or is not readable"
2017-02-01 22:29:43 }

Full logs here: https://app.shippable.com/runs/58925caeace68f1000da6229/17/console

@mattclay mattclay added the ci_verified label Feb 1, 2017

@ansibot ansibot added the stale_ci label Apr 11, 2017

@gundalow gundalow modified the milestone: 2.2.0 Apr 13, 2017

@ansibot

This comment has been minimized.

Contributor

ansibot commented Sep 7, 2017

@ansibot ansibot added the test label Sep 7, 2017

@lathama

This comment has been minimized.

Contributor

lathama commented Sep 23, 2017

@alikins can you update this by moving to integration/targets/apt/tasks and validating?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment