Browse Source

Update patch set 10

Patch Set 10:

(2 comments)

Patch-set: 10
Attention: {"person_ident":"Gerrit User 8367 \u003c8367@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Soniya Murlidhar Vyas replied on the change"}
Attention: {"person_ident":"Gerrit User 30742 \u003c30742@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"Soniya Murlidhar Vyas replied on the change"}
Attention: {"person_ident":"Gerrit User 9303 \u003c9303@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Soniya Murlidhar Vyas replied on the change"}
changes/69/820169/meta
Gerrit User 30742 4 months ago committed by Gerrit Code Review
parent
commit
fd4f336ad9
  1. 24
      1bd900a36c502787b3499a7f0ab5672962dfc97b
  2. 17
      3246f65d9de427b2e146d99d4044aea9bf0a0dff

24
1bd900a36c502787b3499a7f0ab5672962dfc97b

@ -70,6 +70,30 @@
},
"revId": "1bd900a36c502787b3499a7f0ab5672962dfc97b",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "572a928d_f6e91a8d",
"filename": "config_tempest/services/image.py",
"patchSetId": 7
},
"lineNbr": 89,
"author": {
"id": 30742
},
"writtenOn": "2022-01-19T10:18:52Z",
"side": 1,
"message": "okay",
"parentUuid": "2e8d6b1e_e74906fb",
"range": {
"startLine": 85,
"startChar": 8,
"endLine": 89,
"endChar": 71
},
"revId": "1bd900a36c502787b3499a7f0ab5672962dfc97b",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}

17
3246f65d9de427b2e146d99d4044aea9bf0a0dff

@ -16,6 +16,23 @@
"message": "@Soniya, do we have a testproject testing both configs?",
"revId": "3246f65d9de427b2e146d99d4044aea9bf0a0dff",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "05ee5178_0ec0e062",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 10
},
"lineNbr": 0,
"author": {
"id": 30742
},
"writtenOn": "2022-01-19T10:18:52Z",
"side": 1,
"message": "\u003e Patch Set 10: Code-Review-1\n\u003e \n\u003e Need the testproject to test if tempestconf will set properly the multistore, and also add a unittest\n\n@arxcruz, In the commit message I have mentioned the two testprojects exercising both the configs, please do have a look!!\nI will do a fresh review for the unittest since this patch is pending for too long.",
"revId": "3246f65d9de427b2e146d99d4044aea9bf0a0dff",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}
Loading…
Cancel
Save