190 lines
5.9 KiB
Plaintext
190 lines
5.9 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "f0db8fe1_65b4be0d",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 7,
|
|
"author": {
|
|
"id": 35148
|
|
},
|
|
"writtenOn": "2024-04-26T21:02:22Z",
|
|
"side": 1,
|
|
"message": "Title is confusing",
|
|
"range": {
|
|
"startLine": 7,
|
|
"startChar": 0,
|
|
"endLine": 7,
|
|
"endChar": 56
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "a3c0dac6_26994e21",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 7,
|
|
"author": {
|
|
"id": 9926
|
|
},
|
|
"writtenOn": "2024-04-27T01:09:05Z",
|
|
"side": 1,
|
|
"message": "title should summarize the fix",
|
|
"parentUuid": "f0db8fe1_65b4be0d",
|
|
"range": {
|
|
"startLine": 7,
|
|
"startChar": 0,
|
|
"endLine": 7,
|
|
"endChar": 56
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "8476d55a_d7b4b913",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 10,
|
|
"author": {
|
|
"id": 35148
|
|
},
|
|
"writtenOn": "2024-04-26T21:02:22Z",
|
|
"side": 1,
|
|
"message": "Please explain why the rejection was happening.",
|
|
"range": {
|
|
"startLine": 9,
|
|
"startChar": 0,
|
|
"endLine": 10,
|
|
"endChar": 53
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "7a9ab605_59b98086",
|
|
"filename": "software/software/software_controller.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 2739,
|
|
"author": {
|
|
"id": 9926
|
|
},
|
|
"writtenOn": "2024-04-26T19:09:57Z",
|
|
"side": 1,
|
|
"message": "this variable isnt really required, since not referenced elsewhere",
|
|
"range": {
|
|
"startLine": 2739,
|
|
"startChar": 8,
|
|
"endLine": 2739,
|
|
"endChar": 28
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "36b04eef_e0738a11",
|
|
"filename": "software/software/software_controller.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 3047,
|
|
"author": {
|
|
"id": 9926
|
|
},
|
|
"writtenOn": "2024-04-26T19:09:57Z",
|
|
"side": 1,
|
|
"message": "this should resuse self._get_software_upgrade() to obtain the from_release, to_release",
|
|
"range": {
|
|
"startLine": 3044,
|
|
"startChar": 0,
|
|
"endLine": 3047,
|
|
"endChar": 60
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "ca89c4ae_f35883ad",
|
|
"filename": "software/software/software_controller.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 3050,
|
|
"author": {
|
|
"id": 9926
|
|
},
|
|
"writtenOn": "2024-04-26T19:09:57Z",
|
|
"side": 1,
|
|
"message": "This may not be needed if all we need to do is check the return value of from_release and to_release from _get_software_upgrade() ... \n\notherwise, could this check use some of the Version methods?\n\n\nfrom_release \u003d version.Version(from_release)\nto_release \u003d version.Version(to_release)\n\nif from_release.major !\u003d to_release.major or from_release.minor !\u003d to_release.minor: \n\n....",
|
|
"range": {
|
|
"startLine": 3048,
|
|
"startChar": 1,
|
|
"endLine": 3050,
|
|
"endChar": 82
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "16bf0aa7_74ec4bef",
|
|
"filename": "software/software/software_controller.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 3052,
|
|
"author": {
|
|
"id": 33390
|
|
},
|
|
"writtenOn": "2024-04-26T18:56:17Z",
|
|
"side": 1,
|
|
"message": "This function is more complex than it needs, please see https://opendev.org/starlingx/update/src/commit/1d6add41a22ebed6a99517915147a5244cf5c6a5/software/software/software_controller.py#L3038-L3058\n\n\nIt could be like:\n\n```\ndeploy \u003d self.get_software_upgrade()\nif deploy:\n from_major_release \u003d \".\".join(deploy.get(\"from_release\").rsplit(\".\")[:-1])\n to_major_release \u003d \".\".join(deploy.get(\"to_release\").rsplit(\".\")[:-1]\n return from_major_release !\u003d to_major_release\nreturn None\n```\n\nBy the way, the function `get_software_upgrade` should be renamed, as it seems to get any deployment in progress, regardless if it is patch or major release.",
|
|
"range": {
|
|
"startLine": 3039,
|
|
"startChar": 4,
|
|
"endLine": 3052,
|
|
"endChar": 35
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "40fc4387_9097678e",
|
|
"filename": "software/software/software_controller.py",
|
|
"patchSetId": 4
|
|
},
|
|
"lineNbr": 3052,
|
|
"author": {
|
|
"id": 33390
|
|
},
|
|
"writtenOn": "2024-04-26T19:10:31Z",
|
|
"side": 1,
|
|
"message": "Update, rsplit can be simplified with:\n\n`from_major_release \u003d deploy.get(\"from_release\").rsplit(\".\", 1)[0]`",
|
|
"parentUuid": "16bf0aa7_74ec4bef",
|
|
"range": {
|
|
"startLine": 3039,
|
|
"startChar": 4,
|
|
"endLine": 3052,
|
|
"endChar": 35
|
|
},
|
|
"revId": "0c7cdcc94a07efdb8c323179322e46a6a7ce99a8",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |