Commit Graph

6 Commits

Author SHA1 Message Date
Ian Wienand 76037c434e linter: add name to blocks
A later version of ansible-lint enforces names on blocks.  This is
generally a good rule; fix a few missing blocks here.

Change-Id: Ia87a0c21ec0ed1662e37cbc9e17a0df344b54e57
2022-07-28 12:03:23 +10:00
Ian Wienand 0326363e48 Update ansible-lint and add fake zuul_return
- Use zuul_return fake module to make ansible lint happy, this allows
  to remove Zuul import.
- While ansible-lint 4.2.0 is now able to detect playbooks/roles, this
  is broken, so don't use it
- move its config out of tox.ini so it can be used by any tools, or
  without tox

Change-Id: Ie8935f47db855647e19ae091044e5ac1871f1551
Co-Authored-By: Sorin Sbarnea <ssbarnea@redhat.com>
Co-Authored-By: Andreas Jaeger <aj@suse.com>
2020-03-18 22:28:10 +01:00
Monty Taylor d45f691599
Just use chmod instead of file for log permissions
The file module with recurse goes to the bad place when the content has
a symlink to '.'. Rather than fighting it - just use chmod -R, which
does what we want.

Change-Id: Ied4cc2a7e252db555ffdc3f1fd47edaf93b666da
2018-06-18 08:15:45 -05:00
Monty Taylor 8f8c6774e1
Don't follow symlinks when setting log permissions
Ansible 2.5 changed the default value of follow to true.
We don't want to follow symlinks here, so set follow to false directly.

Needed-By: https://review.openstack.org/551989/
Change-Id: I858f328118a301d648bef485cc606a7fd6364ada
2018-06-14 09:06:45 -05:00
James E. Blair 28a3f190c8 Test change to upload-logs
This tests the change in https://review.openstack.org/556877

Change-Id: I2dfdb858b4ea1ecceabcb6064b3515ad90dbba72
2018-03-27 09:16:43 -07:00
Monty Taylor 068a754bad
Add modified upload-logs as test-upload-logs to base-test
Change-Id: I645727fb30de6abe325c694d04886711c205e620
2018-01-25 15:04:13 -06:00