)]}'
{"roles/ceph-loop-device/tasks/main.yml":[{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"5155f6ec102a7dd1b5904f0a480cfb16b3f9f045","unresolved":true,"context_lines":[{"line_number":33,"context_line":"  shell: |"},{"line_number":34,"context_line":"    pvcreate {{ ceph_loop_device }}"},{"line_number":35,"context_line":"    vgcreate {{ ceph_logical_volume_group }} {{ ceph_loop_device }}"},{"line_number":36,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l +100%FREE {{ ceph_logical_volume_group }}"},{"line_number":37,"context_line":"    lvs"},{"line_number":38,"context_line":"  changed_when: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"88141dfe_2e2a6004","line":36,"updated":"2021-08-06 10:45:00.000000000","message":"We cannot merge this as is because it will break jobs using ceph-ansible (unless we also update those jobs to not use wal/db). Alternatively, we can add a condition here to make the wal/db LVs if version \u003c wallaby.","commit_id":"c395f504029180d5b8ca018476ef3a221e61f29c"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"0549318c1810c386100305222be69070284274a7","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  shell: |"},{"line_number":35,"context_line":"    pvcreate {{ ceph_loop_device }}"},{"line_number":36,"context_line":"    vgcreate {{ ceph_logical_volume_group }} {{ ceph_loop_device }}"},{"line_number":37,"context_line":"    {% if ci_branch in  [\u0027pike\u0027, \u0027queens\u0027, \u0027rocky\u0027, \u0027ussuri\u0027, \u0027victoria\u0027]%}"},{"line_number":38,"context_line":"    lvcreate -n {{ ceph_logical_volume_wal }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":39,"context_line":"    lvcreate -n {{ ceph_logical_volume_db }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":40,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l 2041 {{ ceph_logical_volume_group }}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c50dfb5b_a0835e13","line":37,"range":{"start_line":37,"start_character":24,"end_line":37,"end_character":73},"updated":"2021-08-06 12:56:23.000000000","message":"[\u0027pike\u0027, \u0027queens\u0027, \u0027rocky\u0027, \u0027ussuri\u0027, \u0027victoria\u0027] ?","commit_id":"02efee2784ea83379b92df0a5191dec5500e8a39"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"0549318c1810c386100305222be69070284274a7","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  shell: |"},{"line_number":35,"context_line":"    pvcreate {{ ceph_loop_device }}"},{"line_number":36,"context_line":"    vgcreate {{ ceph_logical_volume_group }} {{ ceph_loop_device }}"},{"line_number":37,"context_line":"    {% if ci_branch in  [\u0027pike\u0027, \u0027queens\u0027, \u0027rocky\u0027, \u0027ussuri\u0027, \u0027victoria\u0027]%}"},{"line_number":38,"context_line":"    lvcreate -n {{ ceph_logical_volume_wal }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":39,"context_line":"    lvcreate -n {{ ceph_logical_volume_db }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":40,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l 2041 {{ ceph_logical_volume_group }}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"b9d8a35c_2a05e08c","line":37,"range":{"start_line":37,"start_character":10,"end_line":37,"end_character":19},"updated":"2021-08-06 12:56:23.000000000","message":"|default(master)","commit_id":"02efee2784ea83379b92df0a5191dec5500e8a39"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"6313e954ad88b1b1d7142fd1de6e11dfe9ade772","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  shell: |"},{"line_number":35,"context_line":"    pvcreate {{ ceph_loop_device }}"},{"line_number":36,"context_line":"    vgcreate {{ ceph_logical_volume_group }} {{ ceph_loop_device }}"},{"line_number":37,"context_line":"    {% if ci_branch | default(\u0027master\u0027) in  [\u0027train\u0027, \u0027ussuri\u0027, \u0027victoria\u0027]%}"},{"line_number":38,"context_line":"    lvcreate -n {{ ceph_logical_volume_wal }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":39,"context_line":"    lvcreate -n {{ ceph_logical_volume_db }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":40,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l 2041 {{ ceph_logical_volume_group }}"},{"line_number":41,"context_line":"    {% else %}"},{"line_number":42,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l +100%FREE {{ ceph_logical_volume_group }}"},{"line_number":43,"context_line":"    {% endif %}"},{"line_number":44,"context_line":"    lvs"},{"line_number":45,"context_line":"  changed_when: true"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"d6802579_6cc8eb2c","line":43,"range":{"start_line":37,"start_character":4,"end_line":43,"end_character":15},"updated":"2021-08-06 13:01:41.000000000","message":"maybe we can split this in two tasks (it would help for maintenance purposes)","commit_id":"4ef52241001e21356f3ca63ad85ec9f429cc975a"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"9fed7b112fc3343da0aa5a6fa5af8e8190333dc6","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  shell: |"},{"line_number":35,"context_line":"    pvcreate {{ ceph_loop_device }}"},{"line_number":36,"context_line":"    vgcreate {{ ceph_logical_volume_group }} {{ ceph_loop_device }}"},{"line_number":37,"context_line":"    {% if ci_branch | default(\u0027master\u0027) in  [\u0027train\u0027, \u0027ussuri\u0027, \u0027victoria\u0027]%}"},{"line_number":38,"context_line":"    lvcreate -n {{ ceph_logical_volume_wal }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":39,"context_line":"    lvcreate -n {{ ceph_logical_volume_db }} -l 375 {{ ceph_logical_volume_group }}"},{"line_number":40,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l 2041 {{ ceph_logical_volume_group }}"},{"line_number":41,"context_line":"    {% else %}"},{"line_number":42,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l +100%FREE {{ ceph_logical_volume_group }}"},{"line_number":43,"context_line":"    {% endif %}"},{"line_number":44,"context_line":"    lvs"},{"line_number":45,"context_line":"  changed_when: true"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"2eadc584_dc6737b1","line":43,"range":{"start_line":37,"start_character":4,"end_line":43,"end_character":15},"in_reply_to":"d6802579_6cc8eb2c","updated":"2021-08-06 17:53:41.000000000","message":"Done","commit_id":"4ef52241001e21356f3ca63ad85ec9f429cc975a"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"8337a2cc8ba48d65d9ec7dc610bd42e7d16d4302","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    lvs"},{"line_number":44,"context_line":"  changed_when: true"},{"line_number":45,"context_line":"  when:"},{"line_number":46,"context_line":"    - ci_branch is not defined or"},{"line_number":47,"context_line":"      (ci_branch is defined and"},{"line_number":48,"context_line":"       ci_branch not in [\u0027train\u0027, \u0027ussuri\u0027, \u0027victoria\u0027])"},{"line_number":49,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"17e3be7f_678a2ace","line":46,"range":{"start_line":46,"start_character":6,"end_line":46,"end_character":16},"updated":"2021-08-09 10:07:00.000000000","message":"At this stage ci_branch is probably not defined yet, that\u0027s why we always see only ceph_lv_data instead of the required db/wal for the old branches.\nWe should include the code to detect (and define as a fact) the ci_branch before calling lvcreate.","commit_id":"23198809c8cbc75f478484b5611f87092011e12f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a41fa50d37044de400a78292513ae98805e5a894","unresolved":true,"context_lines":[{"line_number":33,"context_line":"  shell: |"},{"line_number":34,"context_line":"    pvcreate {{ ceph_loop_device }}"},{"line_number":35,"context_line":"    vgcreate {{ ceph_logical_volume_group }} {{ ceph_loop_device }}"},{"line_number":36,"context_line":"    lvcreate -n {{ ceph_logical_volume_data }} -l +100%FREE {{ ceph_logical_volume_group }}"},{"line_number":37,"context_line":"    lvs"},{"line_number":38,"context_line":"  changed_when: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"573b8c70_25b4da89","line":36,"updated":"2021-08-11 06:48:29.000000000","message":"latest run looks good \n\n\nhttps://03b09b2c4c61a51fac54-e21f3284a69ab3bb1c3a75be83aa4c39.ssl.cf2.rackcdn.com/772896/8/check/tripleo-ci-centos-8-scenario001-standalone/1156fc5/job-output.txt\n\n\n2021-08-10 23:54:17.355711 | TASK [ceph-loop-device : Use {pv,vg,lv}create to create logical volume on one loop device]\n2021-08-10 23:54:17.799312 | primary |   Physical volume \"/dev/loop4\" successfully created.\n2021-08-10 23:54:17.914081 | primary |   Volume group \"ceph_vg\" successfully created\n2021-08-10 23:54:18.036424 | primary |   Logical volume \"ceph_lv_data\" created.\n2021-08-10 23:54:18.096263 | primary |   LV           VG      Attr       LSize   Pool Origin Data%  Meta%  Move Log Cpy%Sync Convert\n2021-08-10 23:54:18.096335 | primary |   ceph_lv_data ceph_vg -wi-a----- \u003c14.00g\n2021-08-10 23:54:18.399835 | primary | ok: Runtime: 0:00:00.386534\n2021-08-10 23:54:18.434187 |","commit_id":"bfa6d5aec0bbb3fae00c93e0a99ec7991a8c0698"}]}