57 lines
2.0 KiB
Plaintext
57 lines
2.0 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "5f795786_e165fe05",
|
|
"filename": "distributedcloud/dcmanager/api/controllers/v1/phased_subcloud_deploy.py",
|
|
"patchSetId": 12
|
|
},
|
|
"lineNbr": 540,
|
|
"author": {
|
|
"id": 36205
|
|
},
|
|
"writtenOn": "2024-05-06T20:44:25Z",
|
|
"side": 1,
|
|
"message": "Do we ensure that install-values are provided part of enrollment? I think we need to ensure this. I think bmc_address / bmc_username are requirement for the enrollment.",
|
|
"revId": "b56f039f855d37c27aeb1be34f6236a862827acd",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "2e9843ec_ad6001fd",
|
|
"filename": "distributedcloud/dcmanager/api/controllers/v1/phased_subcloud_deploy.py",
|
|
"patchSetId": 12
|
|
},
|
|
"lineNbr": 540,
|
|
"author": {
|
|
"id": 36205
|
|
},
|
|
"writtenOn": "2024-05-06T20:45:40Z",
|
|
"side": 1,
|
|
"message": "@Yuxing Jiang could you please clarify, thanks",
|
|
"parentUuid": "5f795786_e165fe05",
|
|
"revId": "b56f039f855d37c27aeb1be34f6236a862827acd",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "bdd36031_5dfacca8",
|
|
"filename": "distributedcloud/dcmanager/api/controllers/v1/phased_subcloud_deploy.py",
|
|
"patchSetId": 12
|
|
},
|
|
"lineNbr": 540,
|
|
"author": {
|
|
"id": 31954
|
|
},
|
|
"writtenOn": "2024-05-07T01:40:40Z",
|
|
"side": 1,
|
|
"message": "Yes, bmc_address and bmc username/password should be required for the following operation. We can split the check of the install values in the next story which using the bmc address and user/password for the seed iso though.",
|
|
"parentUuid": "2e9843ec_ad6001fd",
|
|
"revId": "b56f039f855d37c27aeb1be34f6236a862827acd",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |