tripleo-quickstart doc generation is not writing out all the appropriate docs

[wes hayutin]

oooq is not writing docs appropriately.

https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-periodic-newton-delorean-full-deploy-minimal-122/docs/build/

https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-periodic-master-delorean-full-deploy-minimal_pacemaker-153/

[Harry Rybacki (hrybacki-h)]

Initial assessment shows that this started happening around 17-Dec-2016[1].

Reviewing output from the collect-logs role we can see the problem is relates to the path where bash templates exist:

failed: [localhost] (item=overcloud-validate) => {"changed": true, "cmd": "awk -f \"/home/<snip>/usr/local/share/ansible/roles/collect-logs/scripts/doc_extrapolation.awk\" \"/home/<snip>/collected_files/undercloud/home/rhos-ci/overcloud-validate.sh\" > \"/home/<snip>/usr/local/share/ansible/roles/collect-logs/docs/source/overcloud-validate.rst\"", "delta": "0:00:00.003720", "end": "2017-01-04 13:09:12.095890", "failed": true, "item": "overcloud-validate", "rc": 2, "start": "2017-01-04 13:09:12.092170", "stderr": "awk: fatal: cannot open file `/home/rhos-ci/<snip>/tripleo-quickstart/collected_files/undercloud/home/rhos-ci/overcloud-validate.sh' for reading (No such file or directory)", "stdout": "", "stdout_lines": [], "warnings": []}

Crosschecking the date quickly lead to the root review that is causing this[2]. We removed the hardcoded `stack` user in a attempt to make the role more flexible but missed this regression. Assigned myself to work on solution.

[1] - https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-periodic-newton-delorean-full-deploy-minimal-104/docs/build/
[2] - https://review.openstack.org/#/c/404800/

[wes hayutin (weshayutin)]

Can we close this?

[Harry Rybacki (hrybacki-h)]

This bug can be closed as the issue has been fixed. However there are several reviews[1-3] that need to land in order to help prevent/identify further doc generation breakages.

[1] - https://review.openstack.org/425936
[2] - https://review.openstack.org/425868
[3] - https://review.openstack.org/421463

[Harry Rybacki (hrybacki-h)]

One more review specific to minimal_pacemaker issues: https://review.openstack.org/#/c/431025/

[Ronelle Landy (rlandy)]

https://review.openstack.org/#/c/431025/ is merged