Update patch set 1

Patch Set 1: Code-Review-1 Workflow-1

(1 comment)

This change is ready for review.

Patch-set: 1
Reviewer: Gerrit User 34221 <34221@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=-1
Label: Workflow=-1
Work-in-progress: false
Attention: {"person_ident":"Gerrit User 15435 \u003c15435@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Bo Yuan Chang replied on the change"}
This commit is contained in:
Gerrit User 34221 2021-11-25 15:05:40 +00:00 committed by Gerrit Code Review
parent 991a71ebde
commit 36860ff002
1 changed files with 24 additions and 0 deletions

View File

@ -102,6 +102,30 @@
},
"revId": "d7a64e12e5296e89ec6dda734ef95cbec8db3624",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "14580400_c9f65e55",
"filename": "puppet-manifests/src/modules/platform/manifests/params.pp",
"patchSetId": 1
},
"lineNbr": 67,
"author": {
"id": 34221
},
"writtenOn": "2021-11-25T15:05:40Z",
"side": 1,
"message": "Tee suggested that sysinv should not exceed 6 workers which is controlled by eng_workers_by_5\nand no more than 10 workers for the rest. \n\nThe example shown in the puppet document did not specify the upper limit of how many numbers we can include. I will make a note of this and attempt to avoid nesting for next commit.",
"parentUuid": "1d348603_b9f54fd7",
"range": {
"startLine": 67,
"startChar": 99,
"endLine": 67,
"endChar": 100
},
"revId": "d7a64e12e5296e89ec6dda734ef95cbec8db3624",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}