From 965ea8337b8b06ab2ba3bd21949727a458833764 Mon Sep 17 00:00:00 2001 From: Gerrit User 31954 <31954@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Date: Mon, 22 Apr 2024 14:46:19 +0000 Subject: [PATCH] Update patch set 2 Patch Set 2: (2 comments) Patch-set: 2 Reviewer: Gerrit User 36205 <36205@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Reviewer: Gerrit User 19319 <19319@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Attention: {"person_ident":"Gerrit User 36205 \u003c36205@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_31954\u003e replied on the change"} Attention: {"person_ident":"Gerrit User 19319 \u003c19319@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_31954\u003e replied on the change"} --- f83fe08336349267b7b2d09638cf7ab9eb90a55f | 35 ++++++++++++++++++++++++ 1 file changed, 35 insertions(+) diff --git a/f83fe08336349267b7b2d09638cf7ab9eb90a55f b/f83fe08336349267b7b2d09638cf7ab9eb90a55f index cf78b99..4756383 100644 --- a/f83fe08336349267b7b2d09638cf7ab9eb90a55f +++ b/f83fe08336349267b7b2d09638cf7ab9eb90a55f @@ -35,6 +35,23 @@ "revId": "f83fe08336349267b7b2d09638cf7ab9eb90a55f", "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" }, + { + "unresolved": false, + "key": { + "uuid": "ae0b2d03_6aa6532a", + "filename": "/PATCHSET_LEVEL", + "patchSetId": 2 + }, + "lineNbr": 0, + "author": { + "id": 31954 + }, + "writtenOn": "2024-04-22T14:46:19Z", + "side": 1, + "message": "I would expect the enroll in phase operation also need the", + "revId": "f83fe08336349267b7b2d09638cf7ab9eb90a55f", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" + }, { "unresolved": true, "key": { @@ -98,6 +115,24 @@ "revId": "f83fe08336349267b7b2d09638cf7ab9eb90a55f", "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" }, + { + "unresolved": true, + "key": { + "uuid": "e3664711_23cbd33f", + "filename": "distributedcloud-client/dcmanagerclient/commands/v1/phased_subcloud_deploy_manager.py", + "patchSetId": 2 + }, + "lineNbr": 561, + "author": { + "id": 31954 + }, + "writtenOn": "2024-04-22T14:46:19Z", + "side": 1, + "message": "Sorry for my mistake, the deploy config should not be necessary, but the install values, BMC password should be expected to have for the factory install the subclouds, I would @kyle and @salman to comment on this topic", + "parentUuid": "036cd9dd_286611f9", + "revId": "f83fe08336349267b7b2d09638cf7ab9eb90a55f", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" + }, { "unresolved": true, "key": {