)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"5aa137cfbe3743893d3a79919f69094c9182cdd8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f3242b55_6a219e98","updated":"2022-12-16 14:24:49.000000000","message":"In the document mentioned in the commit message, i see that the current syntax that is being used is \"name @ url\" replacing \"egg\u003d\" format.\n\nSo in the document change shouldn\u0027t it be like the following ? -\nos-brick@URL\ncinder@URL","commit_id":"c0ecd0cf5fdfc2291ffbc6ab760603f225fd6cd7"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"8dfe3c743a6301c35bd8fc17b9cd010b5713a6eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d38d3739_4ab4995a","in_reply_to":"91e3b251_ce061a74","updated":"2022-12-16 15:40:04.000000000","message":"Awesome. Thanks for the clarification. +1","commit_id":"c0ecd0cf5fdfc2291ffbc6ab760603f225fd6cd7"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f9123d73f2f700bc81c4d886618932ca9b2fb250","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"91e3b251_ce061a74","in_reply_to":"f3242b55_6a219e98","updated":"2022-12-16 15:35:11.000000000","message":"Good question.  The reason for using url#egg\u003dname or name@url is when the package name isn\u0027t included in the url.  So for example, \n\n  git+https://github.com/some-org/some-repo#egg\u003dfoo\n\nthe package name we want to reference is \u0027foo\u0027, but the repo name is \u0027some-repo\u0027.  So in that case, we would need to use the preferred PEP 509 syntax of\n\n  foo@git+https://github.com/some-org/some-repo\n  \nBut in our case, we have the package name and the repo name the same:\n\n  git+https://opendev.org/openstack/cinder#egg\u003dcinder\n  \nSo we can just use the normal git url without specifying an extra name.","commit_id":"c0ecd0cf5fdfc2291ffbc6ab760603f225fd6cd7"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ec95585ccea8a9c39636af70b42ac83a01092acd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ce625c7c_e4a1ad35","updated":"2022-12-20 16:56:24.000000000","message":"Not an expert on this but we are clearly not using egg as it\u0027s intended to be used so we shouldn\u0027t be affected by removing it. LGTM.","commit_id":"642fbea5e409e8aaa91d4c911df34498b8e6fe4d"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"8dfe3c743a6301c35bd8fc17b9cd010b5713a6eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"84f45f3a_bc769296","updated":"2022-12-16 15:40:04.000000000","message":"Thanks for the clarification on the usage. +1","commit_id":"642fbea5e409e8aaa91d4c911df34498b8e6fe4d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"472794f4f92ddaacadd64a246ebfd8b1d6a9cb86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bc8e3fd0_ddb6bfc3","updated":"2023-01-20 15:58:06.000000000","message":"Useful!","commit_id":"642fbea5e409e8aaa91d4c911df34498b8e6fe4d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2bf69464ae9f742571d61c8dc2e40a2571047042","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bd831669_261baf51","updated":"2022-12-19 16:38:43.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican - got a 500 from identity service","commit_id":"642fbea5e409e8aaa91d4c911df34498b8e6fe4d"}]}