)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9979ebcbe57e65b78d7df6082a46ba47582958ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"72d4d63e_b749ab9e","updated":"2022-06-14 17:24:11.000000000","message":"this looks promising\n\nmost other moduels i have seen have not supported multiopts properly.\n\nnova support multistropts where the values are strings conatining json\nand when parsed they form a list of string each of which has json embeded.\n\nthe data within a multiopt sting is and should be opaque to the cofnig module and preserved as is as\nquoting may be important as it is with the embeded json cases.","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7e4b92663d525da9c5843b9e3d78321811c0f7d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"db7344d7_b9928a04","updated":"2022-06-20 12:03:33.000000000","message":"+1 for content -1 for fomating issue that zuul found","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"94829c232f48a0c4ab2e6158be2401bec5fd20bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cab45311_0ad3ee50","updated":"2022-06-20 11:51:06.000000000","message":"lgtm","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6444fda6c1807d84d8aedce80c22bb02cf43c215","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5a2ebdaa_a5564a80","updated":"2022-06-20 12:50:33.000000000","message":"zuul is happy and this looks pretty good on the preview site\nso +1 https://1ecf3f270edb97c7c2b4-ecb639e7fbdbc73f937f4f0594fa940b.ssl.cf2.rackcdn.com/845768/8/check/openstack-tox-docs/aeb04c2/docs/#example-for-conf","commit_id":"17330e7558dc234bf1e485d0c515167bcaf10d40"}],"doc/source/index.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9979ebcbe57e65b78d7df6082a46ba47582958ad","unresolved":true,"context_lines":[{"line_number":45,"context_line":"          dest: \"test_dst.conf\""},{"line_number":46,"context_line":"          config_type: \"ini\""},{"line_number":47,"context_line":"          config_overrides:"},{"line_number":48,"context_line":"            hello:"},{"line_number":49,"context_line":"              cruel: world"},{"line_number":50,"context_line":"            section1:"},{"line_number":51,"context_line":"              option1: value2"},{"line_number":52,"context_line":"            listSection:"},{"line_number":53,"context_line":"              listOpt:"},{"line_number":54,"context_line":"                - listItem1"}],"source_content_type":"text/x-rst","patch_set":4,"id":"908bb711_cace8b79","line":51,"range":{"start_line":48,"start_character":1,"end_line":51,"end_character":29},"updated":"2022-06-14 17:24:11.000000000","message":"these are both just setting simple value but syalistically i think you shoudl be quoting the values as you are implictly relying on them being converted to strings\n\nif you used true for example that shoudl be interpreated as a boolean not a string.\nwhere as \"bool\" ture would shoudl preserve the quotes.","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"75333ee5061990ae516db013454cd762d1cb6144","unresolved":false,"context_lines":[{"line_number":45,"context_line":"          dest: \"test_dst.conf\""},{"line_number":46,"context_line":"          config_type: \"ini\""},{"line_number":47,"context_line":"          config_overrides:"},{"line_number":48,"context_line":"            hello:"},{"line_number":49,"context_line":"              cruel: world"},{"line_number":50,"context_line":"            section1:"},{"line_number":51,"context_line":"              option1: value2"},{"line_number":52,"context_line":"            listSection:"},{"line_number":53,"context_line":"              listOpt:"},{"line_number":54,"context_line":"                - listItem1"}],"source_content_type":"text/x-rst","patch_set":4,"id":"d7267fe9_19d0ff6b","line":51,"range":{"start_line":48,"start_character":1,"end_line":51,"end_character":29},"in_reply_to":"908bb711_cace8b79","updated":"2022-06-20 11:49:35.000000000","message":"True, but this is standard for any yaml inputs. Having to quote everything in the examples would create unnecessary noise, IMO.","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9979ebcbe57e65b78d7df6082a46ba47582958ad","unresolved":true,"context_lines":[{"line_number":50,"context_line":"            section1:"},{"line_number":51,"context_line":"              option1: value2"},{"line_number":52,"context_line":"            listSection:"},{"line_number":53,"context_line":"              listOpt:"},{"line_number":54,"context_line":"                - listItem1"},{"line_number":55,"context_line":"                - listItem2"},{"line_number":56,"context_line":"            multiStrOpSection:"},{"line_number":57,"context_line":"              multiStrOpOption:"},{"line_number":58,"context_line":"                ? multiStrOp1"}],"source_content_type":"text/x-rst","patch_set":4,"id":"6ae268c5_52d5fbea","line":55,"range":{"start_line":53,"start_character":0,"end_line":55,"end_character":27},"updated":"2022-06-14 17:24:11.000000000","message":"this is a simple ordered list\nand shoudl be used when ordring of the items is important such as schduler filters.","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"75333ee5061990ae516db013454cd762d1cb6144","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            section1:"},{"line_number":51,"context_line":"              option1: value2"},{"line_number":52,"context_line":"            listSection:"},{"line_number":53,"context_line":"              listOpt:"},{"line_number":54,"context_line":"                - listItem1"},{"line_number":55,"context_line":"                - listItem2"},{"line_number":56,"context_line":"            multiStrOpSection:"},{"line_number":57,"context_line":"              multiStrOpOption:"},{"line_number":58,"context_line":"                ? multiStrOp1"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1fd8afbf_0ec60f56","line":55,"range":{"start_line":53,"start_character":0,"end_line":55,"end_character":27},"in_reply_to":"6ae268c5_52d5fbea","updated":"2022-06-20 11:49:35.000000000","message":"Ack","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9979ebcbe57e65b78d7df6082a46ba47582958ad","unresolved":true,"context_lines":[{"line_number":54,"context_line":"                - listItem1"},{"line_number":55,"context_line":"                - listItem2"},{"line_number":56,"context_line":"            multiStrOpSection:"},{"line_number":57,"context_line":"              multiStrOpOption:"},{"line_number":58,"context_line":"                ? multiStrOp1"},{"line_number":59,"context_line":"                ? multiStrOp2"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"Resulting file on the remote host:"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"0b2ccd88_16412b26","line":59,"range":{"start_line":57,"start_character":0,"end_line":59,"end_character":29},"updated":"2022-06-14 17:24:11.000000000","message":"ack \nthis is technially an unsorted set in yaml terms\n\nhttps://yaml.org/spec/1.2.2/#example-unordered-sets\n\n\ni think order ist technially undefined for multi ops but in pratces its preserved in oslo.config i belive when the values fo the multi opt are merged into a list of values.\n\nnone of the usages im aware of for multiOpts in openstack depend on order and i would suggest listOpt is a much better choice when order is requried.\n\nso this syntax i think makes logical sense.","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"75333ee5061990ae516db013454cd762d1cb6144","unresolved":false,"context_lines":[{"line_number":54,"context_line":"                - listItem1"},{"line_number":55,"context_line":"                - listItem2"},{"line_number":56,"context_line":"            multiStrOpSection:"},{"line_number":57,"context_line":"              multiStrOpOption:"},{"line_number":58,"context_line":"                ? multiStrOp1"},{"line_number":59,"context_line":"                ? multiStrOp2"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"Resulting file on the remote host:"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"6b5e29e6_abf3b125","line":59,"range":{"start_line":57,"start_character":0,"end_line":59,"end_character":29},"in_reply_to":"0b2ccd88_16412b26","updated":"2022-06-20 11:49:35.000000000","message":"Ack","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"24169f67d2b112df6f52577225e6f0f8e25c261e","unresolved":true,"context_lines":[{"line_number":74,"context_line":"  [listSection]"},{"line_number":75,"context_line":"  listOpt \u003d listItem1,listItem2"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"  [multiStrOpSection]"},{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Example for .ini with remote_src"},{"line_number":82,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":4,"id":"efca2dbf_30b657cb","line":79,"range":{"start_line":77,"start_character":2,"end_line":79,"end_character":32},"updated":"2022-06-14 17:34:59.000000000","message":"this is a real world example of a multistropt supported by nova\n\nhttps://github.com/openstack/nova/blob/master/nova/pci/devspec.py#L201-L211\u003d\n[pci]\npassthrough_whitelist \u003d {\"address\":\"*:0a:00.*\",\n                        \"physical_network\":\"physnet1\"}\npassthrough_whitelist \u003d {\"address\": {\"domain\": \".*\",\n                                     \"bus\": \"02\",\n                                     \"slot\": \"01\",\n                                    \"function\": \"[0-2]\"},\n                        \"physical_network\":\"net1\"}\npassthrough_whitelist \u003d {\"vendor_id\":\"1137\",\"product_id\":\"0071\"}\n\nthe vlaues are json stings\nthe adress filed can be a simple string in which case it optionally supprot bash style wildcard * glob syntax\n\nthe embeded adress filed can also be a dictionary\nin which case it suppot python regex format in the sub fileds\n\nin all of the above can be used together\n\nand when the above config section is parsed it is formed into a python list contraing 3 strings that contain json so this is parsed in 2 steps\n\nfirst oslo.cofig constructs the list of opaque strings form the .conf file\nthen nova does a json loads one each item in the list of strings and marshaels the contented into python object that model the  whitelist and adress semantics.\n\nhttps://github.com/openstack/nova/blob/master/nova/pci/whitelist.py#L55-L83\u003d\n\n\n\nby the way i think \n\n[pci]\npassthrough_whitelist \u003d {\"address\":\"*:0a:00.*\",\n                        \"physical_network\":\"physnet1\"}\npassthrough_whitelist \u003d {\"address\": {\"domain\": \".*\",\n                                     \"bus\": \"02\",\n                                     \"slot\": \"01\",\n                                    \"function\": \"[0-2]\"},\n                        \"physical_network\":\"net1\"}\npassthrough_whitelist \u003d {\"vendor_id\":\"1137\",\"product_id\":\"0071\"}\n\nwoudl normally be witten like this\n\n[pci]\npassthrough_whitelist \u003d \u0027{\"address\":\"*:0a:00.*\", \"physical_network\":\"physnet1\"}\u0027\npassthrough_whitelist \u003d \u0027{\"address\": {\"domain\": \".*\", \"bus\": \"02\", \"slot\": \"01\", \"function\": \"[0-2]\"}, \"physical_network\":\"net1\"}\u0027\npassthrough_whitelist \u003d \u0027{\"vendor_id\":\"1137\",\"product_id\":\"0071\"}\u0027\n\ne.g. all on one line an in singel quotes but i dont recall if that is strictly required but i belive it is.\n\nthe expanded version in our comments is more for readblity","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"518a95e830f878eb71b5463cbb5fcabd936c8866","unresolved":true,"context_lines":[{"line_number":74,"context_line":"  [listSection]"},{"line_number":75,"context_line":"  listOpt \u003d listItem1,listItem2"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"  [multiStrOpSection]"},{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Example for .ini with remote_src"},{"line_number":82,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7e496f85_489813f1","line":79,"range":{"start_line":77,"start_character":2,"end_line":79,"end_character":32},"in_reply_to":"06406a86_bd77c185","updated":"2022-06-20 12:11:56.000000000","message":"Is there an additional suggestion for more content - or is that a final remark with no actions?","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"75333ee5061990ae516db013454cd762d1cb6144","unresolved":false,"context_lines":[{"line_number":74,"context_line":"  [listSection]"},{"line_number":75,"context_line":"  listOpt \u003d listItem1,listItem2"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"  [multiStrOpSection]"},{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Example for .ini with remote_src"},{"line_number":82,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":4,"id":"34f0b17d_c85a78dd","line":79,"range":{"start_line":77,"start_character":2,"end_line":79,"end_character":32},"in_reply_to":"49cf29ba_0fb1e32b","updated":"2022-06-20 11:49:35.000000000","message":"Examples added. Thanks!","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"408905f597453141e188c01153c6807c00cb495d","unresolved":true,"context_lines":[{"line_number":74,"context_line":"  [listSection]"},{"line_number":75,"context_line":"  listOpt \u003d listItem1,listItem2"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"  [multiStrOpSection]"},{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Example for .ini with remote_src"},{"line_number":82,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":4,"id":"06406a86_bd77c185","line":79,"range":{"start_line":77,"start_character":2,"end_line":79,"end_character":32},"in_reply_to":"49cf29ba_0fb1e32b","updated":"2022-06-20 11:56:23.000000000","message":"sorry the addtional context is as part of evolving triplo to be less complex there is ongoing work to to move to static ansibel roles instead of generatign them form heat.so as part of  implementing standalone ansible roles https://review.opendev.org/q/topic:standalone-roles we have also been looking to see if we could get rid of heat running puppet to generate config files and could instead could reuse the config_template module to replace puppet.\n\nwe also looked at the config_merger implmation that kolla-ansible had but one of its limitation is that it does not supprot multiopt parmaters.\n\nthe internals of there module uses config parser to load the configs into dicts and then merge the dict before renderign the templates using jina again.\n\nso jesse noted that one we aready package the config_template module and 2 it supported multioptstr so they wanted to imporve the docs examples and i provdied some actully examples form nova of how more fomplex multiopts are used in a project as a reference.","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"10324294f56aa583371ade2350835bd025937bfc","unresolved":true,"context_lines":[{"line_number":74,"context_line":"  [listSection]"},{"line_number":75,"context_line":"  listOpt \u003d listItem1,listItem2"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"  [multiStrOpSection]"},{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Example for .ini with remote_src"},{"line_number":82,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":4,"id":"49cf29ba_0fb1e32b","line":79,"range":{"start_line":77,"start_character":2,"end_line":79,"end_character":32},"in_reply_to":"4d56a50b_5ed47c41","updated":"2022-06-20 10:52:50.000000000","message":"sorry, wrong overwrite provided. correct one would be:\n\n  nova_nova_conf_overrides:\n    pci:\n      passthrough_whitelist: \u0027{ \"physical_network\": \"\u003cml2 network name\u003e\", \"devname\": \"\u003cphysical nic\u003e\" }\u0027","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"8fb91c98fe0cabf8fe44ce4f5bea5935d46d43bc","unresolved":true,"context_lines":[{"line_number":74,"context_line":"  [listSection]"},{"line_number":75,"context_line":"  listOpt \u003d listItem1,listItem2"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"  [multiStrOpSection]"},{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Example for .ini with remote_src"},{"line_number":82,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":4,"id":"4d56a50b_5ed47c41","line":79,"range":{"start_line":77,"start_character":2,"end_line":79,"end_character":32},"in_reply_to":"efca2dbf_30b657cb","updated":"2022-06-20 10:51:43.000000000","message":"I think I\u0027m missing some scope of the discussion here as feels that there\u0027s intended to be some extra use-case for config_tempalte ansible module then it is right now.\n\nBut to have that said, with this specific exemple, we in OSA would indeed provide just a string to the module, like:\n\n  nova_nova_conf_overrides:\n    pci: \u0027{ \"physical_network\": \"\u003cml2 network name\u003e\", \"devname\": \"\u003cphysical nic\u003e\" }\u0027\n    \nBut we also have an independent variable for that inside nova.conf.j2 template (that still can be overwritten with this module).","commit_id":"aeefe7d92a9917c6daebd419be1f6b731fc764d8"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"46d4c98b4e33ea94503d05d9210caf831e4f7465","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    connection: local"},{"line_number":36,"context_line":"    gather_facts: no"},{"line_number":37,"context_line":"    tasks:"},{"line_number":38,"context_line":"      - config_template:"},{"line_number":39,"context_line":"          content: |"},{"line_number":40,"context_line":"            [foo]"},{"line_number":41,"context_line":"            bar \u003d baz"}],"source_content_type":"text/x-rst","patch_set":5,"id":"c792304b_fc98443b","line":38,"range":{"start_line":38,"start_character":8,"end_line":38,"end_character":23},"updated":"2022-06-20 11:40:49.000000000","message":"maybe worth refferencing as collection? Ie:\n\n openstack.config_template.config_template","commit_id":"d65793ccbf5278e896622adf7a3fb973dccd7f29"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"75333ee5061990ae516db013454cd762d1cb6144","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    connection: local"},{"line_number":36,"context_line":"    gather_facts: no"},{"line_number":37,"context_line":"    tasks:"},{"line_number":38,"context_line":"      - config_template:"},{"line_number":39,"context_line":"          content: |"},{"line_number":40,"context_line":"            [foo]"},{"line_number":41,"context_line":"            bar \u003d baz"}],"source_content_type":"text/x-rst","patch_set":5,"id":"a3f7102a_76315475","line":38,"range":{"start_line":38,"start_character":8,"end_line":38,"end_character":23},"in_reply_to":"c792304b_fc98443b","updated":"2022-06-20 11:49:35.000000000","message":"Done","commit_id":"d65793ccbf5278e896622adf7a3fb973dccd7f29"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7e4b92663d525da9c5843b9e3d78321811c0f7d7","unresolved":true,"context_lines":[{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"A practical example would be for something like OpenStack\u0027s nova.conf where the input of:"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. code-block :: yaml"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"14669b18_e01ee856","line":81,"updated":"2022-06-20 12:03:33.000000000","message":"and this is also just slight too long","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"45b0251d17ac42a0aeaa15cd0bd8cf92a0d862e1","unresolved":false,"context_lines":[{"line_number":78,"context_line":"  multiStrOpOption \u003d multiStrOp1"},{"line_number":79,"context_line":"  multiStrOpOption \u003d multiStrOp2"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"A practical example would be for something like OpenStack\u0027s nova.conf where the input of:"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. code-block :: yaml"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"e7b045a1_3d91d1c9","line":81,"in_reply_to":"14669b18_e01ee856","updated":"2022-06-20 12:32:52.000000000","message":"Done","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7e4b92663d525da9c5843b9e3d78321811c0f7d7","unresolved":true,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. code-block :: yaml"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"nova_conf_override:"},{"line_number":86,"context_line":"  filter_scheduler:"},{"line_number":87,"context_line":"    enabled_filters:"},{"line_number":88,"context_line":"      - ComputeFilter"}],"source_content_type":"text/x-rst","patch_set":7,"id":"d3ccd388_605ad4da","line":85,"updated":"2022-06-20 12:03:33.000000000","message":"its unhappy with this code block becsue its not indented\n\nso it actully things the code block start on line 83 is empty","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"45b0251d17ac42a0aeaa15cd0bd8cf92a0d862e1","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. code-block :: yaml"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"nova_conf_override:"},{"line_number":86,"context_line":"  filter_scheduler:"},{"line_number":87,"context_line":"    enabled_filters:"},{"line_number":88,"context_line":"      - ComputeFilter"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3945a3e3_66e42504","line":85,"in_reply_to":"d3ccd388_605ad4da","updated":"2022-06-20 12:32:52.000000000","message":"Done","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7e4b92663d525da9c5843b9e3d78321811c0f7d7","unresolved":true,"context_lines":[{"line_number":90,"context_line":"      - ServerGroupAffinityFilter"},{"line_number":91,"context_line":"      - NUMATopologyFilter"},{"line_number":92,"context_line":"  pci:"},{"line_number":93,"context_line":"    ? passthrough_whitelist: \u0027{\"address\":\"*:0a:00.*\", \"physical_network\":\"physnet1\"}\u0027"},{"line_number":94,"context_line":"    ? passthrough_whitelist: \u0027{\"address\": {\"domain\": \".*\", \"bus\": \"02\", \"slot\": \"01\", \"function\": \"[0-2]\"}, \"physical_network\":\"net1\"}\u0027"},{"line_number":95,"context_line":"    ? passthrough_whitelist: \u0027{\"vendor_id\":\"1137\",\"product_id\":\"0071\"}\u0027"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Would produce:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"0004207d_326ab98c","line":94,"range":{"start_line":93,"start_character":2,"end_line":94,"end_character":135},"updated":"2022-06-20 12:03:33.000000000","message":"and becasue it is not considering this to be part of the code block these are raisign line lenght errors.","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"45b0251d17ac42a0aeaa15cd0bd8cf92a0d862e1","unresolved":false,"context_lines":[{"line_number":90,"context_line":"      - ServerGroupAffinityFilter"},{"line_number":91,"context_line":"      - NUMATopologyFilter"},{"line_number":92,"context_line":"  pci:"},{"line_number":93,"context_line":"    ? passthrough_whitelist: \u0027{\"address\":\"*:0a:00.*\", \"physical_network\":\"physnet1\"}\u0027"},{"line_number":94,"context_line":"    ? passthrough_whitelist: \u0027{\"address\": {\"domain\": \".*\", \"bus\": \"02\", \"slot\": \"01\", \"function\": \"[0-2]\"}, \"physical_network\":\"net1\"}\u0027"},{"line_number":95,"context_line":"    ? passthrough_whitelist: \u0027{\"vendor_id\":\"1137\",\"product_id\":\"0071\"}\u0027"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Would produce:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"e0f74bc4_fa6705dd","line":94,"range":{"start_line":93,"start_character":2,"end_line":94,"end_character":135},"in_reply_to":"0004207d_326ab98c","updated":"2022-06-20 12:32:52.000000000","message":"Done","commit_id":"2018e371b7515f9dded189bec8cbd5d0eb16fb33"}]}