)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8652,"name":"Attila Darazs","email":"adarazs@redhat.com","username":"adarazs"},"change_message_id":"e766a6028c7014c25d41a29580f608a5160e7827","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a variable for the dir where the repo is built"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Previously, the repo was built in the ansible_user_dir."},{"line_number":10,"context_line":"If multiple jobs are run with this role ocncurrently, there can"},{"line_number":11,"context_line":"be a clash."},{"line_number":12,"context_line":"This review add a variable (and defaults to the original value) so"},{"line_number":13,"context_line":"that users can move the build dir to a job-specific workspace."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f044301_2e763e08","line":10,"range":{"start_line":10,"start_character":40,"end_line":10,"end_character":52},"updated":"2017-05-04 14:07:08.000000000","message":"concurrently","commit_id":"3ca9976f1349781d099ff2cb63a9408b6a475309"},{"author":{"_account_id":8652,"name":"Attila Darazs","email":"adarazs@redhat.com","username":"adarazs"},"change_message_id":"e766a6028c7014c25d41a29580f608a5160e7827","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Previously, the repo was built in the ansible_user_dir."},{"line_number":10,"context_line":"If multiple jobs are run with this role ocncurrently, there can"},{"line_number":11,"context_line":"be a clash."},{"line_number":12,"context_line":"This review add a variable (and defaults to the original value) so"},{"line_number":13,"context_line":"that users can move the build dir to a job-specific workspace."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Idb08b559f2d98c8084a05fcdfe0edb0e5feca441"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f044301_0e738217","line":12,"range":{"start_line":12,"start_character":12,"end_line":12,"end_character":15},"updated":"2017-05-04 14:07:08.000000000","message":"adds","commit_id":"3ca9976f1349781d099ff2cb63a9408b6a475309"}],"roles/build-test-packages/defaults/main.yml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"ee7804149960ac214cb3c2f61f9f1649042fff76","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"build_repo_dir: \"{{ ansible_user_dir }}\""},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"artg_dlrn_repo_url: \"https://github.com/openstack-packages/DLRN.git\""},{"line_number":6,"context_line":"artg_rdoinfo_repo_url: \"https://github.com/redhat-openstack/rdoinfo\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5ff73747_7491d746","line":3,"updated":"2017-05-02 08:44:09.000000000","message":"could we use the work_dir instead of adding another var?","commit_id":"1c1fc0ce38193ed0db9d51545458a18d9bc23486"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"031bc8aafb6b27489c02b4ac867bc6ad63b5210d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"build_repo_dir: \"{{ ansible_user_dir }}\""},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"artg_dlrn_repo_url: \"https://github.com/openstack-packages/DLRN.git\""},{"line_number":6,"context_line":"artg_rdoinfo_repo_url: \"https://github.com/redhat-openstack/rdoinfo\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5ff73747_541b4972","line":3,"in_reply_to":"5ff73747_7491d746","updated":"2017-05-03 14:59:11.000000000","message":"@Bogdan, I think this role is mostly used on virthost where we can not to have the same user as in undercloud (or non root user at all), so it will be overhead (or constraint) to create user/working_dir on virthost as it was on undercloud just for building.","commit_id":"1c1fc0ce38193ed0db9d51545458a18d9bc23486"},{"author":{"_account_id":8652,"name":"Attila Darazs","email":"adarazs@redhat.com","username":"adarazs"},"change_message_id":"e766a6028c7014c25d41a29580f608a5160e7827","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"build_repo_dir: \"{{ ansible_user_dir }}\""},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"artg_dlrn_repo_url: \"https://github.com/openstack-packages/DLRN.git\""},{"line_number":6,"context_line":"artg_rdoinfo_repo_url: \"https://github.com/redhat-openstack/rdoinfo\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f044301_2991989a","line":3,"range":{"start_line":3,"start_character":0,"end_line":3,"end_character":14},"updated":"2017-05-04 14:07:08.000000000","message":"Please add the new variable to README.md","commit_id":"3ca9976f1349781d099ff2cb63a9408b6a475309"}]}