openstack
/
ansible-role-container-registry
Code Issues Proposed changes

Update patch set 1 

Patch Set 1:

(2 comments)

Patch-set: 1
This commit is contained in:
Gerrit User 12393 2022-04-29 08:03:07 +00:00 committed by Gerrit Code Review
parent 3488b00021
commit 470ee83670
1 changed files with 41 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
d70b04ec566bed0b43053832f8741ec0949216a0
View File

@ -23,6 +23,30 @@
"revId": "d70b04ec566bed0b43053832f8741ec0949216a0",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "3ed64559_fceae809",
"filename": "/COMMIT_MSG",
"patchSetId": 1
},
"lineNbr": 13,
"author": {
"id": 12393
},
"writtenOn": "2022-04-29T08:03:07Z",
"side": 1,
"message": "Done",
"parentUuid": "585e6df9_f5b06f4a",
"range": {
"startLine": 13,
"startChar": 21,
"endLine": 13,
"endChar": 48
},
"revId": "d70b04ec566bed0b43053832f8741ec0949216a0",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
@ -39,6 +63,23 @@
"message": "change is OK but commit message is confusing i don\u0027t think this blocks anything now \n\nc7 removal topic is there https://review.opendev.org/q/topic:ooo_c7_teardown i think you can use the same?\n\n",
"revId": "d70b04ec566bed0b43053832f8741ec0949216a0",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "35f8b0ef_0f03a2b9",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 1
},
"lineNbr": 0,
"author": {
"id": 12393
},
"writtenOn": "2022-04-29T08:03:07Z",
"side": 1,
"message": "Thank you Marios for reviewing it, Commit message is fixed now.",
"revId": "d70b04ec566bed0b43053832f8741ec0949216a0",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}