)]}'
{".zuul.yaml":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"cff19041448be9b41fe7044b0044a0d389522f70","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"# We want to run cinderlib with Cinder and OS-Brick\u0027s source code"},{"line_number":86,"context_line":"- job:"},{"line_number":87,"context_line":"    name: brick_source-cinder-tempest-plugin-lvm-lio-barbican"},{"line_number":88,"context_line":"    parent: cinder-tempest-plugin-lvm-lio-barbican"},{"line_number":89,"context_line":"    required-projects:"},{"line_number":90,"context_line":"      - openstack/os-brick"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_23e663e9","line":87,"updated":"2020-08-07 12:03:53.000000000","message":"For the record, os-brick defines this job already (os-brick-src-tempest-lvm-lio-barbican)\n\nhttps://opendev.org/openstack/os-brick/src/commit/32115be11ce100852b2ba8a0141d470af97b7d97/.zuul.yaml#L34","commit_id":"b3ba2518d5a741e7b2d0ac2eb668a6c742e441b6"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"34becee2b4d2ae566afbb89e2fa98b8ce85d6771","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"# We want to run cinderlib with Cinder and OS-Brick\u0027s source code"},{"line_number":86,"context_line":"- job:"},{"line_number":87,"context_line":"    name: brick_source-cinder-tempest-plugin-lvm-lio-barbican"},{"line_number":88,"context_line":"    parent: cinder-tempest-plugin-lvm-lio-barbican"},{"line_number":89,"context_line":"    required-projects:"},{"line_number":90,"context_line":"      - openstack/os-brick"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_7eb90e13","line":87,"in_reply_to":"9f560f44_23e663e9","updated":"2020-08-07 12:45:47.000000000","message":"Done","commit_id":"b3ba2518d5a741e7b2d0ac2eb668a6c742e441b6"}],"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"b52119a22c9503ba4a809c3e1276865185bdb9cc","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The reason why the CI didn\u0027t fail on the patch is because devstack"},{"line_number":14,"context_line":"installs the bindeps of the projects installed from source, but Zuul"},{"line_number":15,"context_line":"doesn\u0027t do the same for the \"required-projects\"."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"To solve this issue an prevent future gate breaks this patch uses the"},{"line_number":18,"context_line":"bindep zuul role to install os-brick missing packages before running any"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_96408ba7","line":15,"updated":"2020-08-04 12:14:47.000000000","message":"Personally, I think this is a devstack bug.","commit_id":"e62b4434933e97ef23442ceceebe510c134eed46"}],"playbooks/required-projects-bindeps.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0ff27dd426ac2d8c5e8d2c5fef25ad2d44a98d52","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    - name: Stat bindep.txt"},{"line_number":35,"context_line":"      stat:"},{"line_number":36,"context_line":"        path: \"{{ bindep_dir }}/bindep.txt\""},{"line_number":37,"context_line":"        register: bindep_file_stat"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    - debug:"},{"line_number":40,"context_line":"        msg: Exists {{ bindep_file_stat.stat.exists }} path {{ bindep_file_stat.stat.path }}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9f560f44_097c35ea","line":37,"updated":"2020-08-06 13:04:58.000000000","message":"\"Unsupported parameters for (stat) module: register Supported parameters include: checksum_algorithm, follow, get_attributes, get_checksum, get_md5, get_mime, path\"","commit_id":"a178652863734ee9aa0b3523a9737f42b7fe3acc"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ab734f159d0e6be0c74708df107403cf2391270b","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    - name: Stat bindep.txt"},{"line_number":35,"context_line":"      stat:"},{"line_number":36,"context_line":"        path: \"{{ bindep_dir }}/bindep.txt\""},{"line_number":37,"context_line":"        register: bindep_file_stat"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    - debug:"},{"line_number":40,"context_line":"        msg: Exists {{ bindep_file_stat.stat.exists }} path {{ bindep_file_stat.stat.path }}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9f560f44_7895b886","line":37,"in_reply_to":"9f560f44_097c35ea","updated":"2020-08-06 14:16:05.000000000","message":"Thanks, wrong indentation, as it\u0027s not a parameter to the stat, but the task.","commit_id":"a178652863734ee9aa0b3523a9737f42b7fe3acc"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"bc1e9ca51dc81216c5eb4f030c52a3b1c2085baf","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  roles:"},{"line_number":27,"context_line":"    - role: bindep"},{"line_number":28,"context_line":"      vars:"},{"line_number":29,"context_line":"        bindep_dir: \"{{ devstack_base_dir | default(\u0027/opt/stack\u0027) }}/os-brick\""},{"line_number":30,"context_line":"  tasks:"},{"line_number":31,"context_line":"    - debug:"},{"line_number":32,"context_line":"        msg: bindep_dir is {{ bindep_dir }}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9f560f44_3137db4f","line":29,"range":{"start_line":29,"start_character":24,"end_line":29,"end_character":41},"updated":"2020-08-06 16:43:31.000000000","message":"OK, I\u0027m dumb.\nWe are not running devstack here (that\u0027s the whole reason why this patch is necessary), so of course this variable is empty... And /opt/stack is not the right value for this job...","commit_id":"7bc1ed034e6b52d2c70a1d0f0050b2122009af37"}]}