Remove the delegate-to, use elevated privs, and add new task
to make sure the destination directory exists.
Change-Id: I258d39afd77dd090198456950d873c99f98dbda7
This also adds an exception to ansible-lint, ANSIBLE0006, because
we want to use "apt-get update" for updating the apt cache [1]
[1]: https://review.openstack.org/#/c/492716/
This reverts commit 5fe5b6ca1a2777989413d8375b8e98a649e03ab8.
Change-Id: Icd79198964e86c7a2c73102f3e4d845d161b924d
Moves the zuul-cloner shim from the zuul repo to this repo as a
role, and templates the script to allow specifying the Zuul repo
source directory.
Change-Id: Id1609ce069cce6018b53025b1bd148ad0c36ae13
Nodes (and ansible) don't know about floating ips because they are
external. However, they are valid interfaces over which nodes might talk
to each other- especially if we're re-using the inventory generated for
the jobs.
Start off by grabbing all of the addresses from nodepool, if they exist.
Then do the loop for finding the addresses we want. Then collapse
that list with set and do a pass to create the actual list of hosts and
keys needed.
Change-Id: I748fae1c86d7bb9333dc251991308282858811a4
If there is no bindep file at all in the repository (which is likely
because tox depends on bindep and won't always be useful) then
bindep_file is never set. Then when the set_face bindep_command comes
around it tries to reference the unset bindep_file and fails.
Change-Id: I7be6be88209810eed715daaa4bb41315e838272d
Signed-off-by: Jamie Lennox <jamielennox@gmail.com>
The result of creating a temporary directory for bindep is a dictionary
of data include mode, uuid, and other things. When installing bindep
into that new directory we only want to specify the path to that dir.
Change-Id: If98f87ff44356911b78984eb884f7cd65660cebb
Signed-off-by: Jamie Lennox <jamielennox@gmail.com>
If we use emit-ara-html in a base job post playbook, and it fails,
we won't upload the log files that we *do* have.
Ignore errors from ara generate and compress.
Also, let's call a space a space.
Change-Id: Ie6d41e829b0edb105d205997766c5d87022a16b5
These roles are failing yamllint:
roles/upload-logs/tasks/main.yaml
23:5 error wrong indentation: expected 6 but found 4 (indentation)
roles/bindep/tasks/install.yaml
7:3 error wrong indentation: expected 4 but found 2 (indentation)
Change-Id: I189dd08c4d1c091d1e53ca87d52dbaf3f89ba45f
In working on a different patch it became clear that flake8 wasn't
actually running such that it found errors. Updated the
test-requirements to match what's in zuul and all started working (and
failing) properly.
Change-Id: Icfdb1fedbd92ff49484b116a0879686581274a25
tox_command_line is documented but not actually used. Replace it with
tox_extra_args in the zuul.yaml docs.
Change-Id: I9773a0297cd29f4d35b27c4b136a26d6027a932a
NamedTemporaryFile mode is mode='w+b' by default. Make sure we encode
utf8 and write bytes to our temp file.
Change-Id: I0b31abcd52c049efc7d95d6d6e39dfb5d0860620
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
It is possible doc/build/html is not the output path for sphinx. So
default to it but allow a user to override the value.
Change-Id: Ifdb258a9342216d9a2ea0162d7d00dbf0ad7fdeb
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We need to use filter_file.name to get the actually path.
Change-Id: I130284fdaccd1ace139d71b8a3395e9e9fa0df76
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
When we're pushing to a non-bare repo, git has a sad if we push to the
branch that's checked out. But that's ok - we're going to update the
checked out state anyway as a follow up. Tell git it's ok.
Then, after we've pushed, we need to reset after having pushed branch HEAD
to a non bare repo. Clean up after ourselves by removing the config
setting we added to allow the push in the first place.
Change-Id: Id84e9af9d33519e515b8e6d330b375a8e17c14f5
Lets display the data in job-output.txt, it will help with debugging.
I belive we also want to filter this potentially in the webui
streaming.
Change-Id: If0d24fe135838a2f57aeaa4c3fe13b3436384d19
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
block allows adding a name - but it's not actually emitted anywhere in
the logging. So while this documents the source code, it's not reflected
in logging.
I just happened to learn this elsewhere.
Change-Id: I95362ed94418144e332f7289b1bfa5b2ade68de7
The gating configuration for this project is in project-config to
make this repo easier to consume.
Change-Id: Iacc9fc4d510d3740d2fc089d0bada192f17870f2
Depends-On: Iaa8a8eec7746b2d59be41390462b85735de80a7a
This reverts commit f8e746cb14646cc86885f9f7a0a817d9da864ac3.
We shouldn't add project-templates to this repo since they reference
site-specific pipelines.
Change-Id: I4392480ed97082e947a65d2a4ca3450072568227
This reverts commit c462e1a2dfbd6c5e98dfc115dff038c9d762a841.
This repo is meant to be universally applicable; it's not fair for
us to put comments about our own configuration in it.
Also remove the entire comment about OpenStack vs. tox-docs job.
Change-Id: I6369bf31548ca2e32c5135f6c4733bd06a4483d0
