This is preparation for a later version of ansbile-lint, which finds
missing names on blocks. This seems a reasonable rule, and the
Ansible manual says [1]
Names for blocks have been available since Ansible 2.3. We recommend
using names in all tasks, within blocks or elsewhere, for better
visibility into the tasks being executed when you run the playbook.
This simply adds a name tag for blocks that are missing it. This
should have no operational change, but allows us to update the linter
in a follow-on change.
[1] https://docs.ansible.com/ansible/latest/user_guide/playbooks_blocks.html
Change-Id: I92ed4616775650aced352bc9088a07e919f1a25f
After https://review.opendev.org/#/c/742971 the report is generated on
the host. For system-config, this means the report ends up copied to
"bridge.openstack.org/ara-report" in the logs directory; but the
artifact points to the non-existant top-directory.
Add a variable to allow a top-level dir to be set.
Needed-By: https://review.opendev.org/747101
Change-Id: I61a7d7328b5a45fc8dee09909f9411c397dd31b6
https://review.opendev.org/#/c/742971 changed
defaults for ara_report_path to be an absolute path but
missed changes in zuul artifacts. The url here
needs to be relative to log dir to be browsable.
Change-Id: I53511709227ea890eb417778be89e49e03771ff2
This role had some assumptions that it would run only on the
zuul-executor. If we remove those assumptions, it can run on any
worker node, which allows it to be used in untrusted playbooks.
Note that this is backwards compatible only in the case that the
user did not provide a non-default value for ara_report_path. If
a user overrode that (eg, to put the report in a directory other
than "ara" under the executor log dir), this will break.
Change-Id: I3a28368a049fa583398ece8a66f90783f53098f4
ansible-2.9 does not allow "|skipped" anymore, it needs "is skipped"
instead, fix the only usage.
See also Ic0c93f2ea896ead76922694e6150882d8a280daf and
Id6c62da8f8a94c270d8a5d5981683a9d8c4e7ad6.
Change-Id: I597c92d2f6dce31dab1ac4bf52bb534c8146b33b
This seems to introduce the following error with Ansible 2.9
"The conditional check 'not ara_generated | skipped' failed.
The error was: template error while templating string: no filter named 'skipped'.
This reverts commit 262c16b30583f634588f5a52bd7aab944687a8a4.
Change-Id: Id6c62da8f8a94c270d8a5d5981683a9d8c4e7ad6
Default to 'ara' for the executable but also allow for a user to use the
full path if they want.
Change-Id: I26aac0d25f071b65b20f97622836abf56e65cebc
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This fix ara-report usage when the user manually set the ara-report role variable.
Otherwise the set_fact doesn't override the variable name and the role fails with:
There was an issue creating ara-report as requested: [Errno 30] Read-only file system: b'ara-report'
Change-Id: I8a521e7573d1d1870088efd58be6e1cef2ad06c2
This change uses the ara_report_run variable to toggle the database copy.
Otherwise, when ara_report_run is set to 'failure', then the report is
still created if the job succeed
Change-Id: I722a2bff82af2a37fbdec49b2bfd1ad95059ecbe
We are going to provide support the ARA sqlite middleware which
provides the ability for an ARA web application to dynamically load
databases, eliminating the need for HTML generation.
HTML generation does not scale very well: it's orders of magnitude
larger than the database file containing the data and it also takes
a significant amount of time to generate the report.
It would be awkward to add dynamic database things into a role called
"emit-ara-html" and so I took the opportunity to refactor things a
little bit and make the role less prone to failures.
Note that this role was developed with the nested Ansible use case in
mind -- a job running it's own copy of Ansible with ARA should be able
to use this role to generate the report or save the database.
This role will be tested with 'base-test' first and it is expected to
make the 'emit-ara-html' role obsolete if everything works well.
Change-Id: Idedc0bfa1f0f89356b795fb9e2a16f9421a2dc18
Depends-On: I3b10c93b4902a9b45e23c227863e472697f662ef