designate-tempest-plugin/5d6aaea210832b5d67a4495a4c5...

1712 lines
52 KiB
Plaintext

{
"comments": [
{
"unresolved": false,
"key": {
"uuid": "fb8b392e_e11060ad",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 15
},
"lineNbr": 0,
"author": {
"id": 28609
},
"writtenOn": "2023-03-22T11:20:08Z",
"side": 1,
"message": "recheck",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "d0defa66_364cf359",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 15
},
"lineNbr": 0,
"author": {
"id": 28609
},
"writtenOn": "2023-03-22T15:55:33Z",
"side": 1,
"message": "recheck\nRecheck one more time, failed gates aren\u0027t related to the changes in this particular patch.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "501e3512_cb845442",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 15
},
"lineNbr": 0,
"author": {
"id": 11628
},
"writtenOn": "2023-03-25T00:03:45Z",
"side": 1,
"message": "recheck\ngrenade gate issue is resolved",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "b08a8468_8d826b50",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 15
},
"lineNbr": 0,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We have a few things to fix here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "cc14956a_92cf6e03",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 15
},
"lineNbr": 0,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Thanks for review Michael 👍\nI\u0027m about to upload a new patchset.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "2f35838c_04d84051",
"filename": "designate_tempest_plugin/data_utils.py",
"patchSetId": 15
},
"lineNbr": 22,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "This should be up under the \"from string\" section as uuid is a python builtin.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "966a841c_bd582f4d",
"filename": "designate_tempest_plugin/data_utils.py",
"patchSetId": 15
},
"lineNbr": 22,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Yep, I had to check Tempest\u0027s data utils first, will be removed as well as the method itself added in L298.",
"parentUuid": "2f35838c_04d84051",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "0261718a_3acd5a3e",
"filename": "designate_tempest_plugin/data_utils.py",
"patchSetId": 15
},
"lineNbr": 298,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Consider using the data_utils from tempest for this:\nhttps://github.com/openstack/tempest/blob/master/tempest/lib/common/utils/data_utils.py#L24",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "ee79637d_a46afcba",
"filename": "designate_tempest_plugin/data_utils.py",
"patchSetId": 15
},
"lineNbr": 298,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "0261718a_3acd5a3e",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "96cdf0b7_80856f32",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 288,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "The fix for this bug has merged, we should be able to remove this skip.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "f2280074_60386142",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 288,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Yep, test is now PASS, \"skip\" will be removed.",
"parentUuid": "96cdf0b7_80856f32",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "bbb3650b_06b7afb6",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 311,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "\"is True\" is not needed here, you can just use \"config.CONF.dns.nameservers\"",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "905b455b_ee1db630",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 311,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "I\u0027m curious why I had to include this at all 😕\nThere are no API tests based on nameservers check, I\u0027ve probably lost my concentration while coding all these new tests 😊\nThis and similar will be removed.",
"parentUuid": "bbb3650b_06b7afb6",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "22de8713_a0c12a83",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 312,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "This comment doesn\u0027t seem to go with the skip condition.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "ba91aeaf_c44c943f",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 312,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "22de8713_a0c12a83",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "bd4c724f_304603ff",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 317,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "The sudo header is different than the all_projects_header.\nhttps://docs.openstack.org/api-ref/dns/dns-api-v2-index.html?expanded\u003dcreate-shared-zone-detail#id52",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "23407bfc_18454c82",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 317,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "OK, I\u0027ll rename \"sudo_header\" to \"all_projects_header\"",
"parentUuid": "bd4c724f_304603ff",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "6e50f490_cb510bc5",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 360,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Same two comments as above.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "43ac19f3_733dda82",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 360,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "6e50f490_cb510bc5",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "56d41b30_2b748bce",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 365,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Same here, sudo header is not the all projects header.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "2280ebcc_b751f0b7",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 365,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "56d41b30_2b748bce",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "5976b525_c7b9585c",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 403,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Same two comments as above.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "6457f257_9e7e803f",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 403,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "5976b525_c7b9585c",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "76cbf900_f1ef0585",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 408,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Sudo header is not the same as the all projects header",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "6a296088_fee640b1",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 408,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "76cbf900_f1ef0585",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "bba85db9_25116491",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 411,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "This header really isn\u0027t needed here right?",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "8f951187_58caab22",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 411,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "OK, it\u0027s needed here as it\u0027s actually shares Primary zone with Alt.\nin base.py zone client is created for a Primary tenant:\nhttps://github.com/openstack/designate-tempest-plugin/blob/master/designate_tempest_plugin/tests/base.py#L108\nWithout using this header we\u0027ll get:\nDetails: {\u0027code\u0027: 404, \u0027type\u0027: \u0027zone_not_found\u0027, \u0027message\u0027: \u0027Could not find Zone\u0027, \u0027request_id\u0027: \u0027req-259cf1fd-046e-4dd5-be9a-2f84ae357e92\u0027}\nas expected.\nI\u0027ll edit the comment to make it more clear, by specifying that the source zone is owned by Primary.",
"parentUuid": "bba85db9_25116491",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "a3d7e909_c5a2991b",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 414,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "This header really isn\u0027t needed here right?",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "6f85da06_1679f5df",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 414,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "a3d7e909_c5a2991b",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "3f90bb9c_8ff365a3",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 453,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "3e885225_4bf6a37d",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 453,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "3f90bb9c_8ff365a3",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "19b599bb_f8b7c279",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 455,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here. Use:\n(\u003cexception\u003e, \u003cmethod name\u003e, \u003cparam\u003e, \u003cparam\u003e, etc.)",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "0f0901f5_beb40271",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 455,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "19b599bb_f8b7c279",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "ef07ea66_6272e73b",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 467,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "a9b7cfde_a997b058",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 467,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "ef07ea66_6272e73b",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "87470db2_abd8f355",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 468,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "78bed8b6_2adcea23",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 468,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "87470db2_abd8f355",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "ae629cff_afb9866d",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 478,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "ba65d15a_a6d6fa18",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 478,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "ae629cff_afb9866d",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "902abcb9_c86b9f97",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 479,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "90101363_816bd657",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 479,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "902abcb9_c86b9f97",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "1b9b9d47_bcab724f",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 495,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "65f52b69_797af6ed",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 495,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "1b9b9d47_bcab724f",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "366e4715_70dd1ea6",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 497,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "a9fcd44c_a83fa177",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 497,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "366e4715_70dd1ea6",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "8980b10f_aad18579",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 509,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "88dda307_6bea5b20",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 509,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "8980b10f_aad18579",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "3bcc586f_1b63b9a2",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 510,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "4a698f28_0d4be77f",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 510,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "3bcc586f_1b63b9a2",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "68755f4a_924ce2c5",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 518,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "c171e569_02e31477",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 518,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "68755f4a_924ce2c5",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "de78769d_d3a9c768",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 520,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "7b3841af_a5d0f772",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 520,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "de78769d_d3a9c768",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "69dc396e_866fc9c7",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 524,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "test_delete_zone_share_invalid_project_id",
"range": {
"startLine": 524,
"startChar": 8,
"endLine": 524,
"endChar": 43
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "7317cd54_dffc8837",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 524,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "69dc396e_866fc9c7",
"range": {
"startLine": 524,
"startChar": 8,
"endLine": 524,
"endChar": 43
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "cac9aef9_d4281cc6",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 538,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "a36de4d9_4d033d96",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 538,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "cac9aef9_d4281cc6",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "21aeac7c_4f6c5a20",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 540,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "14db28b1_7615e867",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 540,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "21aeac7c_4f6c5a20",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "55035cda_8d901da0",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 545,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "test_delete_zone_share_invalide_shared_zone_id",
"range": {
"startLine": 545,
"startChar": 8,
"endLine": 545,
"endChar": 47
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "14a719b9_94554a38",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 545,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "55035cda_8d901da0",
"range": {
"startLine": 545,
"startChar": 8,
"endLine": 545,
"endChar": 47
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "57392035_e1c5ec20",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 559,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "48a0272e_d1fb5a80",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 559,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "57392035_e1c5ec20",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "77fe7769_d6e31b51",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 560,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "e7c878c0_5123543e",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 560,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "77fe7769_d6e31b51",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "4d583a06_b0e64939",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 564,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "test_delete_zone_share_invalid_zone_id",
"range": {
"startLine": 564,
"startChar": 8,
"endLine": 564,
"endChar": 41
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "821713a4_8cec1806",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 564,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "4d583a06_b0e64939",
"range": {
"startLine": 564,
"startChar": 8,
"endLine": 564,
"endChar": 41
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "cb946ff0_92ccf601",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 578,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "lambda is not needed here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "1788f1d5_9524356e",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 578,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "cb946ff0_92ccf601",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "287a1be3_520d3322",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 579,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "dd425bb9_e19123d7",
"filename": "designate_tempest_plugin/tests/api/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 579,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "287a1be3_520d3322",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "338d8027_7d9fb4f3",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 255,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Since you updated the delete_zone client to have a delete_shares parameter, we should update this test to use it.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "4dd40ebe_68cdd960",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 255,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "338d8027_7d9fb4f3",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "9f4a5640_fe10008d",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 256,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Since you updated the delete_zone client to have a delete_shares parameter, we should update this test to use it.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "3fdab72d_7c037c0e",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 256,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "9f4a5640_fe10008d",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "a7d71e04_f358d307",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 688,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "7d2cb8ec_19787ebc",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 688,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "a7d71e04_f358d307",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "f7fa36f0_8318cd54",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 700,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "7f96ff1a_5ab0e85b",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 700,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "f7fa36f0_8318cd54",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "66763df9_ff6ba3fd",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 707,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably us data_utils from tempest for this.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "2bf83fd8_db4e96e6",
"filename": "designate_tempest_plugin/tests/api/v2/test_zones.py",
"patchSetId": 15
},
"lineNbr": 707,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "66763df9_ff6ba3fd",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "5fae2271_98273468",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_quotas.py",
"patchSetId": 15
},
"lineNbr": 459,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Only a project with admin can set a quota, so this sudo should not be used here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "3edf1d94_cd12eece",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_quotas.py",
"patchSetId": 15
},
"lineNbr": 459,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "It doesn\u0027t seem to be ignored, without this header API fails with:\ntempest.lib.exceptions.Forbidden: Forbidden\nDetails: {\u0027code\u0027: 403, \u0027type\u0027: \u0027forbidden\u0027, \u0027request_id\u0027: \u0027req-23259806-f28f-47de-b1cd-147c99df538c\u0027}",
"parentUuid": "5fae2271_98273468",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "88a2cae1_f1530f81",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_quotas.py",
"patchSetId": 15
},
"lineNbr": 498,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Only a project with admin can set a quota, so this sudo should not be used here. It will be ignored.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "97f1bc1f_a4cbd294",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_quotas.py",
"patchSetId": 15
},
"lineNbr": 498,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "It doesn\u0027t seem to be ignored, without this header API fails with:\ntempest.lib.exceptions.Forbidden: Forbidden\nDetails: {\u0027code\u0027: 403, \u0027type\u0027: \u0027forbidden\u0027, \u0027request_id\u0027: \u0027req-23259806-f28f-47de-b1cd-147c99df538c\u0027}",
"parentUuid": "88a2cae1_f1530f81",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "84c2b1c0_9b0001fb",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_quotas.py",
"patchSetId": 15
},
"lineNbr": 539,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "Only a project with admin can set a quota, so this sudo should not be used here. It will be ignored.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "e0719ab2_3b63e26d",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_quotas.py",
"patchSetId": 15
},
"lineNbr": 539,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "It doesn\u0027t seem to be ignored, without this header API fails with:\ntempest.lib.exceptions.Forbidden: Forbidden\nDetails: {\u0027code\u0027: 403, \u0027type\u0027: \u0027forbidden\u0027, \u0027request_id\u0027: \u0027req-23259806-f28f-47de-b1cd-147c99df538c\u0027}",
"parentUuid": "84c2b1c0_9b0001fb",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "fe0a6d19_6bf1e506",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 217,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "to",
"range": {
"startLine": 217,
"startChar": 34,
"endLine": 217,
"endChar": 37
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "27531b21_2a19ecf7",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 217,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "fe0a6d19_6bf1e506",
"range": {
"startLine": 217,
"startChar": 34,
"endLine": 217,
"endChar": 37
},
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "7605088f_620e7935",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 228,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "We should probably use:\nself.zones_client.project_id here as that is the project the zone was created with. Even though both zones_client and share_zone_client are using the \"primary\" project, it\u0027s confusing to mix the clients here.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "0a49ab87_89fb5750",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 228,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Done",
"parentUuid": "7605088f_620e7935",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "da4fb014_92181bda",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 392,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "This says \"delete\", but the test tries to create a zone export (which is duplicate to the test above).",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "04900404_7eebcb13",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 392,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "Yep, it had to be \"delete\" here.",
"parentUuid": "da4fb014_92181bda",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "514946a5_7becfb3c",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 408,
"author": {
"id": 11628
},
"writtenOn": "2023-04-12T22:59:37Z",
"side": 1,
"message": "This test never creates a zone export, so the test is invalid.",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "b91d8a26_0336d0e9",
"filename": "designate_tempest_plugin/tests/scenario/v2/test_shared_zones.py",
"patchSetId": 15
},
"lineNbr": 408,
"author": {
"id": 28609
},
"writtenOn": "2023-04-13T12:13:12Z",
"side": 1,
"message": "That\u0027s correct, the point here was to ensure that perhaps it does create zonefile for some reason, in spite of the fact that such an API is supposed to be forbidden for Alt as this tenant is not zone\u0027s owner.\nI\u0027ll rename the test name accordingly.",
"parentUuid": "514946a5_7becfb3c",
"revId": "5d6aaea210832b5d67a4495a4c5a1ff5d498e031",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}