Update patch set 1
Patch Set 1: (2 comments) Patch-set: 1 Reviewer: Gerrit User 5241 <5241@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
This commit is contained in:
parent
2df08a43a3
commit
4e59761a4c
|
@ -176,6 +176,46 @@
|
|||
"revId": "c70c9249249504b39cde5ea95caa86f2df921acb",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "70157574_837ae555",
|
||||
"filename": "specs/xena/healthcheck-new-way.rst",
|
||||
"patchSetId": 1
|
||||
},
|
||||
"lineNbr": 63,
|
||||
"author": {
|
||||
"id": 5241
|
||||
},
|
||||
"writtenOn": "2021-08-06T12:25:17Z",
|
||||
"side": 1,
|
||||
"message": "I know nothing about the helper, but I would like to see \"has to be\" instead of \"would be nice\" in this part. We need to have standardized output to have the status machine parsable.",
|
||||
"range": {
|
||||
"startLine": 61,
|
||||
"startChar": 33,
|
||||
"endLine": 63,
|
||||
"endChar": 14
|
||||
},
|
||||
"revId": "c70c9249249504b39cde5ea95caa86f2df921acb",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "6d55e49b_89c95b22",
|
||||
"filename": "specs/xena/healthcheck-new-way.rst",
|
||||
"patchSetId": 1
|
||||
},
|
||||
"lineNbr": 102,
|
||||
"author": {
|
||||
"id": 5241
|
||||
},
|
||||
"writtenOn": "2021-08-06T12:25:17Z",
|
||||
"side": 1,
|
||||
"message": "So IIUC the health check script checking each container status will be executed from container host and not within the containers?",
|
||||
"revId": "c70c9249249504b39cde5ea95caa86f2df921acb",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": true
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue