)]}'
{"spec/unit/type/glance_rootwrap_config_spec.rb":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"80d23443fab211376be15f9765b316adfa4a9486","unresolved":true,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"  it \u0027should not accept a value with whitespace\u0027 do"},{"line_number":37,"context_line":"    @glance_rootwrap_config[:value] \u003d \u0027b ar\u0027"},{"line_number":38,"context_line":"    expect(@glance_rootwrap_config[:value]).to eq(\u0027b ar\u0027)"},{"line_number":39,"context_line":"  end"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  it \u0027should accept valid ensure values\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":12,"id":"065f65a0_937d7c6e","line":38,"updated":"2022-07-11 13:49:17.000000000","message":"Should this raise an error? Or, perhaps, values with a whitespace are acceptable?","commit_id":"1794500d4bd6ed7b8e1467ee29985f27857bf484"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4cc7b7b0d2c008245bb7d8a66d8add0d629f8817","unresolved":true,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"  it \u0027should not accept a value with whitespace\u0027 do"},{"line_number":37,"context_line":"    @glance_rootwrap_config[:value] \u003d \u0027b ar\u0027"},{"line_number":38,"context_line":"    expect(@glance_rootwrap_config[:value]).to eq(\u0027b ar\u0027)"},{"line_number":39,"context_line":"  end"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  it \u0027should accept valid ensure values\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":12,"id":"5aeeac89_961a0dde","line":38,"in_reply_to":"065f65a0_937d7c6e","updated":"2022-07-11 14:22:40.000000000","message":"Good catch. I see we have this same problem in multiple modules and multiple files and I copied that problem here :-(\n\nIMO we should not reject a whitespace here. For example ListOpt in oslo.config can accept something like\n\n[foo]\nbar \u003d value1, value2\n\nand rejecting a whitespace breaks that usage .\n\nI\u0027ll fix the case title so that it properly describe the expected behavior. (and will fix the same in the other places...)","commit_id":"1794500d4bd6ed7b8e1467ee29985f27857bf484"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"04a7334aa80f6ef3d5a31934841f8c5fc827513f","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"  it \u0027should not accept a value with whitespace\u0027 do"},{"line_number":37,"context_line":"    @glance_rootwrap_config[:value] \u003d \u0027b ar\u0027"},{"line_number":38,"context_line":"    expect(@glance_rootwrap_config[:value]).to eq(\u0027b ar\u0027)"},{"line_number":39,"context_line":"  end"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  it \u0027should accept valid ensure values\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":12,"id":"02215e90_6aea1345","line":38,"in_reply_to":"5aeeac89_961a0dde","updated":"2022-07-11 14:39:47.000000000","message":"Done\n\nI\u0027ve started fixing this in the other places. Once a few pilot patches are merged I\u0027ll cover the remaining modules(or if anybody is interested then he/she can take over a few modules)\n\nhttps://review.opendev.org/q/topic:fix-test-desc","commit_id":"1794500d4bd6ed7b8e1467ee29985f27857bf484"}]}