)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b394e575e75335ed93b42efea4b4c9f93ed5b489","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"776b4c39_a308873c","updated":"2026-03-11 19:12:12.000000000","message":"so not really a problem with this chnage but the failreu are because of 2.103\n\n\n+ lib/nova:create_flavors:1263             :   openstack --os-region-name\u003dRegionOne flavor create --id 5 --ram 16384 --disk 160 --vcpus 8 --property hw_rng:allowed\u003dTrue m1.xlarge\nBadRequestException: 400: Client Error for url: https://213.32.79.158/compute/v2.1/flavors, Invalid input for field/attribute flavor. Value: {\u0027name\u0027: \u0027m1.xlarge\u0027, \u0027id\u0027: \u00275\u0027, \u0027ram\u0027: 16384, \u0027OS-FLV-EXT-DATA:ephemeral\u0027: 0, \u0027vcpus\u0027: 8, \u0027os-flavor-access:is_public\u0027: True, \u0027disk\u0027: 160, \u0027rxtx_factor\u0027: 1.0, \u0027swap\u0027: 0}. Additional properties are not allowed (\u0027rxtx_factor\u0027 was unexpected)","commit_id":"d4fe61e547d2d4f2b83320a253da1273d1385794"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f1138c23c355f635e42478317d82a8f16c710f5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bbfbb8ab_f9ab0625","updated":"2026-05-06 09:05:20.000000000","message":"recheck","commit_id":"1089fd76372381d469d3f5fa4471291705f1e867"}],"openstack/compute/v2/flavor.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd22ac5432022865ce3d0c107e4a23fe0fd2d971","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        min_ram\u003d\"minRam\","},{"line_number":42,"context_line":"    )"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    # name filter introduced in 2.61"},{"line_number":45,"context_line":"    _max_microversion \u003d \u00272.102\u0027"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    # Properties"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ccb942c_5bbbdd38","line":44,"range":{"start_line":44,"start_character":34,"end_line":44,"end_character":36},"updated":"2026-03-11 15:33:49.000000000","message":"102\nim not sure you want to update his line or add a new one but it looks liek you started doiing one then ten finished doing the other","commit_id":"5f08a2569b170e2e19a3863b0b733c56e715a3bd"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"063bf2630c02ee45ac217d9ca1c5558120ba929e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        min_ram\u003d\"minRam\","},{"line_number":42,"context_line":"    )"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    # name filter introduced in 2.61"},{"line_number":45,"context_line":"    _max_microversion \u003d \u00272.102\u0027"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    # Properties"}],"source_content_type":"text/x-python","patch_set":1,"id":"cc6ea9cd_17abff8c","line":44,"range":{"start_line":44,"start_character":34,"end_line":44,"end_character":36},"in_reply_to":"7ccb942c_5bbbdd38","updated":"2026-03-11 17:41:29.000000000","message":"Done","commit_id":"5f08a2569b170e2e19a3863b0b733c56e715a3bd"}],"openstack/tests/unit/cloud/test_flavors.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd22ac5432022865ce3d0c107e4a23fe0fd2d971","unresolved":true,"context_lines":[{"line_number":90,"context_line":"                    json\u003d{\u0027flavors\u0027: fakes.FAKE_FLAVOR_LIST},"},{"line_number":91,"context_line":"                ),"},{"line_number":92,"context_line":"            ]"},{"line_number":93,"context_line":"        )"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        self.assertFalse(self.cloud.delete_flavor(\u0027invalid\u0027))"},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"304b24e5_f0ab3f6b","line":93,"updated":"2026-03-11 15:33:49.000000000","message":"i guess this does technically work to test it\n\nwoudl it be better to have a more explicy test that is sperate form delete","commit_id":"5f08a2569b170e2e19a3863b0b733c56e715a3bd"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"063bf2630c02ee45ac217d9ca1c5558120ba929e","unresolved":false,"context_lines":[{"line_number":90,"context_line":"                    json\u003d{\u0027flavors\u0027: fakes.FAKE_FLAVOR_LIST},"},{"line_number":91,"context_line":"                ),"},{"line_number":92,"context_line":"            ]"},{"line_number":93,"context_line":"        )"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        self.assertFalse(self.cloud.delete_flavor(\u0027invalid\u0027))"},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d4c907f9_12d4ae48","line":93,"in_reply_to":"304b24e5_f0ab3f6b","updated":"2026-03-11 17:41:29.000000000","message":"Ah, this is just a side effect of how SDK does looks under the hood. If there\u0027s a `name` query string parameter it will use it.","commit_id":"5f08a2569b170e2e19a3863b0b733c56e715a3bd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd22ac5432022865ce3d0c107e4a23fe0fd2d971","unresolved":true,"context_lines":[{"line_number":124,"context_line":"            \u0027vanilla\u0027,"},{"line_number":125,"context_line":"        )"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"    def test_list_flavors(self):"},{"line_number":128,"context_line":"        self.use_compute_discovery()"},{"line_number":129,"context_line":"        uris_to_mock \u003d ["},{"line_number":130,"context_line":"            dict("}],"source_content_type":"text/x-python","patch_set":1,"id":"b49ec13a_7a7691e9","line":127,"updated":"2026-03-11 15:33:49.000000000","message":"i.e. shoudl we add a sepeate list test by name with vanilla as the search key and the releven microverson","commit_id":"5f08a2569b170e2e19a3863b0b733c56e715a3bd"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"063bf2630c02ee45ac217d9ca1c5558120ba929e","unresolved":false,"context_lines":[{"line_number":124,"context_line":"            \u0027vanilla\u0027,"},{"line_number":125,"context_line":"        )"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"    def test_list_flavors(self):"},{"line_number":128,"context_line":"        self.use_compute_discovery()"},{"line_number":129,"context_line":"        uris_to_mock \u003d ["},{"line_number":130,"context_line":"            dict("}],"source_content_type":"text/x-python","patch_set":1,"id":"426601c4_eecab741","line":127,"in_reply_to":"b49ec13a_7a7691e9","updated":"2026-03-11 17:41:29.000000000","message":"As above.","commit_id":"5f08a2569b170e2e19a3863b0b733c56e715a3bd"}]}