)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f29082a841566ca31eb29f804fa70c99554b28bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ca4cd454_8be79133","updated":"2022-01-17 15:54:27.000000000","message":"Looks okay to me...","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2d0254e6b60df8465ddcd721f05fb1bf30497733","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f01d6c00_484e3150","updated":"2024-11-26 15:42:44.000000000","message":"OK reading this again it looks good, I\u0027ll push this.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"38c054a3c165988d1271abfac99a8db5f2004e1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1b4ca697_6f947d37","updated":"2022-01-17 19:51:58.000000000","message":"Thanks Dan for clarifying a few things. I would still like to see a more detailed explanation regarding the way to get the proper config.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"e46464e2f699c5d6ff09ee3f9a12e95c7f94090f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ebab07b4_52ea5715","updated":"2022-01-13 01:38:04.000000000","message":"Thanks for the additional doc, a couple of issues inlined.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"}],"README.rst":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"51c7b2e06249618b5318903dbe27b693bcd5e7fd","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    $ tempest list-plugins"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"4. Create new file tempest.conf inside etc directory from the tempest"},{"line_number":24,"context_line":"   directory run with below content, including rbac enablement"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"::"},{"line_number":27,"context_line":"     [identity]"},{"line_number":28,"context_line":"     auth_version \u003d v3"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f958f83a_1b28220a","line":25,"range":{"start_line":23,"start_character":3,"end_line":25,"end_character":0},"updated":"2021-07-23 18:00:01.000000000","message":"is this tempest.conf file required to create inside tempest project directory or at /etc/tempest directory?\n\nAlso what is the use of config.py inside glance-tempest-plugin repo, somehow can we utilize that to load default config for us?","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"578b562c328651b3baae02a755dd3839f85c8561","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    $ tempest list-plugins"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"4. Create new file tempest.conf inside etc directory from the tempest"},{"line_number":24,"context_line":"   directory run with below content, including rbac enablement"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"::"},{"line_number":27,"context_line":"     [identity]"},{"line_number":28,"context_line":"     auth_version \u003d v3"}],"source_content_type":"text/x-rst","patch_set":1,"id":"bf8a1d61_9c04eb7f","line":25,"range":{"start_line":23,"start_character":3,"end_line":25,"end_character":0},"in_reply_to":"f958f83a_1b28220a","updated":"2021-10-19 11:30:05.000000000","message":"yeah, sorry\nInside /etc/tempest\n\n\nWell, there is no config.py ATM in the repo but if it\u0027s added in future we can check that how we can utilize that to load default config.","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"51c7b2e06249618b5318903dbe27b693bcd5e7fd","unresolved":true,"context_lines":[{"line_number":55,"context_line":"above for all test runs. Developers can also use this locally to perform more"},{"line_number":56,"context_line":"extensive testing."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Any typical devstack instance should be able to run all Cinder plugin tests."},{"line_number":59,"context_line":"For completeness, here is an example of a devstack local.conf that should"},{"line_number":60,"context_line":"work. Update backend information to fit your environment."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"93da5442_1493c73f","line":58,"range":{"start_line":58,"start_character":56,"end_line":58,"end_character":76},"updated":"2021-07-23 18:00:01.000000000","message":"?","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"578b562c328651b3baae02a755dd3839f85c8561","unresolved":false,"context_lines":[{"line_number":55,"context_line":"above for all test runs. Developers can also use this locally to perform more"},{"line_number":56,"context_line":"extensive testing."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Any typical devstack instance should be able to run all Cinder plugin tests."},{"line_number":59,"context_line":"For completeness, here is an example of a devstack local.conf that should"},{"line_number":60,"context_line":"work. Update backend information to fit your environment."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"4ef340a5_b03b55cb","line":58,"range":{"start_line":58,"start_character":56,"end_line":58,"end_character":76},"in_reply_to":"93da5442_1493c73f","updated":"2021-10-19 11:30:05.000000000","message":"Done","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"51c7b2e06249618b5318903dbe27b693bcd5e7fd","unresolved":true,"context_lines":[{"line_number":72,"context_line":"    SYSLOG\u003dFalse"},{"line_number":73,"context_line":"    LOG_COLOR\u003dFalse"},{"line_number":74,"context_line":"    RECLONE\u003dyes"},{"line_number":75,"context_line":"    ENABLED_SERVICES\u003dg-api,c-api,c-bak,c-sch,c-vol,cinder,dstat,key"},{"line_number":76,"context_line":"    ENABLED_SERVICES+\u003d,mysql,n-api,n-cond,n-cpu,n-crt,n-sch,rabbit,tempest"},{"line_number":77,"context_line":"    GLANCE_ENABLE_MULTIPLE_STORES\u003dTrue"},{"line_number":78,"context_line":"    USE_CINDER_FOR_GLANCE\u003dTrue"}],"source_content_type":"text/x-rst","patch_set":1,"id":"70e6c8ef_79a4694a","line":75,"range":{"start_line":75,"start_character":28,"end_line":75,"end_character":57},"updated":"2021-07-23 18:00:01.000000000","message":"I think we don\u0027t need these services to run our tempest plugin.","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"578b562c328651b3baae02a755dd3839f85c8561","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    SYSLOG\u003dFalse"},{"line_number":73,"context_line":"    LOG_COLOR\u003dFalse"},{"line_number":74,"context_line":"    RECLONE\u003dyes"},{"line_number":75,"context_line":"    ENABLED_SERVICES\u003dg-api,c-api,c-bak,c-sch,c-vol,cinder,dstat,key"},{"line_number":76,"context_line":"    ENABLED_SERVICES+\u003d,mysql,n-api,n-cond,n-cpu,n-crt,n-sch,rabbit,tempest"},{"line_number":77,"context_line":"    GLANCE_ENABLE_MULTIPLE_STORES\u003dTrue"},{"line_number":78,"context_line":"    USE_CINDER_FOR_GLANCE\u003dTrue"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9690937f_7357b4c0","line":75,"range":{"start_line":75,"start_character":28,"end_line":75,"end_character":57},"in_reply_to":"70e6c8ef_79a4694a","updated":"2021-10-19 11:30:05.000000000","message":"Ack","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"51c7b2e06249618b5318903dbe27b693bcd5e7fd","unresolved":true,"context_lines":[{"line_number":73,"context_line":"    LOG_COLOR\u003dFalse"},{"line_number":74,"context_line":"    RECLONE\u003dyes"},{"line_number":75,"context_line":"    ENABLED_SERVICES\u003dg-api,c-api,c-bak,c-sch,c-vol,cinder,dstat,key"},{"line_number":76,"context_line":"    ENABLED_SERVICES+\u003d,mysql,n-api,n-cond,n-cpu,n-crt,n-sch,rabbit,tempest"},{"line_number":77,"context_line":"    GLANCE_ENABLE_MULTIPLE_STORES\u003dTrue"},{"line_number":78,"context_line":"    USE_CINDER_FOR_GLANCE\u003dTrue"},{"line_number":79,"context_line":"    GLANCE_ENABLE_MULTIPLE_STORES\u003dTrue"}],"source_content_type":"text/x-rst","patch_set":1,"id":"36ebd29c_79b19942","line":76,"range":{"start_line":76,"start_character":29,"end_line":76,"end_character":66},"updated":"2021-07-23 18:00:01.000000000","message":"these as well","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"578b562c328651b3baae02a755dd3839f85c8561","unresolved":false,"context_lines":[{"line_number":73,"context_line":"    LOG_COLOR\u003dFalse"},{"line_number":74,"context_line":"    RECLONE\u003dyes"},{"line_number":75,"context_line":"    ENABLED_SERVICES\u003dg-api,c-api,c-bak,c-sch,c-vol,cinder,dstat,key"},{"line_number":76,"context_line":"    ENABLED_SERVICES+\u003d,mysql,n-api,n-cond,n-cpu,n-crt,n-sch,rabbit,tempest"},{"line_number":77,"context_line":"    GLANCE_ENABLE_MULTIPLE_STORES\u003dTrue"},{"line_number":78,"context_line":"    USE_CINDER_FOR_GLANCE\u003dTrue"},{"line_number":79,"context_line":"    GLANCE_ENABLE_MULTIPLE_STORES\u003dTrue"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f69be8eb_a3a9b15a","line":76,"range":{"start_line":76,"start_character":29,"end_line":76,"end_character":66},"in_reply_to":"36ebd29c_79b19942","updated":"2021-10-19 11:30:05.000000000","message":"Ack","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"51c7b2e06249618b5318903dbe27b693bcd5e7fd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70864c6e_8eea0df3","line":82,"range":{"start_line":77,"start_character":4,"end_line":82,"end_character":0},"updated":"2021-07-23 18:00:01.000000000","message":"neither this","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"578b562c328651b3baae02a755dd3839f85c8561","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"25a5c81e_6ab8d074","line":82,"range":{"start_line":77,"start_character":4,"end_line":82,"end_character":0},"in_reply_to":"70864c6e_8eea0df3","updated":"2021-10-19 11:30:05.000000000","message":"Ack","commit_id":"82144bc762deaeaa24e4d0d909f32096bae707aa"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"e46464e2f699c5d6ff09ee3f9a12e95c7f94090f","unresolved":true,"context_lines":[{"line_number":15,"context_line":"2. Install the plugin::"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    $ pip3 install -e glance-tempest-plugin"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"3. Confirm it\u0027s installed::"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    $ tempest list-plugins"}],"source_content_type":"text/x-rst","patch_set":2,"id":"f7a8cb6b_780f410a","line":18,"updated":"2022-01-13 01:38:04.000000000","message":"I don\u0027t understand why I need to clone the repo *and* install the same project from PyPI. Shouldn\u0027t I rather:\n\n- Install from PyPI if I just want to use glance-tempest-plugin\n- Clone the repo and install glance-tempest-plugin if I want to hack on it?","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"38c054a3c165988d1271abfac99a8db5f2004e1c","unresolved":false,"context_lines":[{"line_number":15,"context_line":"2. Install the plugin::"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    $ pip3 install -e glance-tempest-plugin"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"3. Confirm it\u0027s installed::"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    $ tempest list-plugins"}],"source_content_type":"text/x-rst","patch_set":2,"id":"34e4f1d1_f1d49ea1","line":18,"in_reply_to":"a076030c_ef37cf06","updated":"2022-01-17 19:51:58.000000000","message":"Of course, silly me.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f29082a841566ca31eb29f804fa70c99554b28bb","unresolved":true,"context_lines":[{"line_number":15,"context_line":"2. Install the plugin::"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    $ pip3 install -e glance-tempest-plugin"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"3. Confirm it\u0027s installed::"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    $ tempest list-plugins"}],"source_content_type":"text/x-rst","patch_set":2,"id":"a076030c_ef37cf06","line":18,"in_reply_to":"f7a8cb6b_780f410a","updated":"2022-01-17 15:54:27.000000000","message":"\"install -e\" takes a project url or path. So this is installing from the local copy just cloned.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"e46464e2f699c5d6ff09ee3f9a12e95c7f94090f","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    $ tempest list-plugins"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"4. Create new file tempest.conf inside /etc/tempest directory and run with below"},{"line_number":24,"context_line":"   content, including rbac enablement"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"::"},{"line_number":27,"context_line":"     [identity]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"2b72097c_e9581e0e","line":24,"updated":"2022-01-13 01:38:04.000000000","message":"It is unclear what part of the config file is related to rbac enablement. Could we clarify that here?","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f29082a841566ca31eb29f804fa70c99554b28bb","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    $ tempest list-plugins"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"4. Create new file tempest.conf inside /etc/tempest directory and run with below"},{"line_number":24,"context_line":"   content, including rbac enablement"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"::"},{"line_number":27,"context_line":"     [identity]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df09e5c0_93ad717e","line":24,"in_reply_to":"2b72097c_e9581e0e","updated":"2022-01-17 15:54:27.000000000","message":"See lines 27-45? That\u0027s tempest config material, and it\u0027s all in the sample config but commented-out. Should be pretty straightforward, IMHO.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"38c054a3c165988d1271abfac99a8db5f2004e1c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    $ tempest list-plugins"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"4. Create new file tempest.conf inside /etc/tempest directory and run with below"},{"line_number":24,"context_line":"   content, including rbac enablement"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"::"},{"line_number":27,"context_line":"     [identity]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"36d8dc33_d8942960","line":24,"in_reply_to":"df09e5c0_93ad717e","updated":"2022-01-17 19:51:58.000000000","message":"OK. The phrasing leads me to think the new file should only contain lines 27-45, when really what the user is meant to do is *edit* the basic config.","commit_id":"affba5ca79e54b55a811f30087659384dc202ffd"}]}