python-dracclient/87b709a40dbf2aa5fa4c63c4031...

658 lines
18 KiB
Plaintext

{
"comments": [
{
"key": {
"uuid": "9f560f44_71863816",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 7,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "Looking at the code, it looks more like that a title \"Add method for Last system inventory time\" describes better what is changed",
"range": {
"startLine": 7,
"startChar": 0,
"endLine": 7,
"endChar": 21
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_a0da3b48",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 7,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "updated",
"parentUuid": "9f560f44_71863816",
"range": {
"startLine": 7,
"startChar": 0,
"endLine": 7,
"endChar": 21
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_518cd42d",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 9,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "Returns This property -\u003e Returns property that ?",
"range": {
"startLine": 9,
"startChar": 0,
"endLine": 9,
"endChar": 21
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_c0c92f7a",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 9,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "updated",
"parentUuid": "9f560f44_518cd42d",
"range": {
"startLine": 9,
"startChar": 0,
"endLine": 9,
"endChar": 21
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_51b5f4cf",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 10,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "Looking at the code I see that `LastSystemInventoryTime` is used. From where does \"System Inventory collection On Reboot(CSIOR)\" come? How are they related?",
"range": {
"startLine": 9,
"startChar": 45,
"endLine": 10,
"endChar": 27
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_206bcb85",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 10,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "https://downloads.dell.com/solutions/general-solution-resources/White%20Papers/Dell_SystemInfo_Profile.pdf\n\nfield DCIM_SystemView.LastSystemInventoryTime its the description of this field",
"parentUuid": "9f560f44_51b5f4cf",
"range": {
"startLine": 9,
"startChar": 45,
"endLine": 10,
"endChar": 27
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_11753c33",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 14,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "I\u0027d say this is irrelevant to this patch where this is going to be used. It can be used in more scenarios than listed here and planned now. Can remove this.",
"range": {
"startLine": 13,
"startChar": 0,
"endLine": 14,
"endChar": 38
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_00c4c75e",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 14,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "updated",
"parentUuid": "9f560f44_11753c33",
"range": {
"startLine": 13,
"startChar": 0,
"endLine": 14,
"endChar": 38
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_d1c8e457",
"filename": "/COMMIT_MSG",
"patchSetId": 3
},
"lineNbr": 18,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "To be clarified, if this is necessary at all for python-dracclient project or need to be replaced with smth else.",
"range": {
"startLine": 17,
"startChar": 0,
"endLine": 18,
"endChar": 10
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_91fdcc70",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 99,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "same as in commit message - need to clarify this text.",
"range": {
"startLine": 98,
"startChar": 11,
"endLine": 99,
"endChar": 50
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_9132ec5d",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 100,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "Don\u0027t need to repeat this if it is already in :returns: section.",
"range": {
"startLine": 99,
"startChar": 51,
"endLine": 100,
"endChar": 26
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_2c8c6734",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 100,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "updated",
"parentUuid": "9f560f44_9132ec5d",
"range": {
"startLine": 99,
"startChar": 51,
"endLine": 100,
"endChar": 26
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_f1ed28b8",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 101,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "better say what is returned, \"value is\" does not read well + \"value\" is too generic, for example, could use - Last system inventory time",
"range": {
"startLine": 101,
"startChar": 18,
"endLine": 101,
"endChar": 56
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_86f0f4ff",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 106,
"author": {
"id": 22348
},
"writtenOn": "2020-08-28T07:28:01Z",
"side": 1,
"message": "pep8: H405 multi line docstring summary not separated with an empty line",
"tag": "autogenerated:zuul:check",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_86dab448",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 106,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "Done",
"parentUuid": "9f560f44_86f0f4ff",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_910b0c7c",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 108,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "parse this to python datetime here instead of returning string in format? Otherwise in most cases consumers of this method will have to parse it anyway and/or format in different date format.",
"range": {
"startLine": 108,
"startChar": 15,
"endLine": 108,
"endChar": 60
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_0c052315",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 108,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "consumer of this method we need to read current time, there code is written i have reused the same method instead of converting here also",
"parentUuid": "9f560f44_910b0c7c",
"range": {
"startLine": 108,
"startChar": 15,
"endLine": 108,
"endChar": 60
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_31084090",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 907,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "here and below need to revert this change as not related to the patch",
"range": {
"startLine": 906,
"startChar": 0,
"endLine": 907,
"endChar": 13
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_c00c8f3b",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 907,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "Done",
"parentUuid": "9f560f44_31084090",
"range": {
"startLine": 906,
"startChar": 0,
"endLine": 907,
"endChar": 13
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_66f5e0ed",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 925,
"author": {
"id": 22348
},
"writtenOn": "2020-08-28T07:28:01Z",
"side": 1,
"message": "pep8: H405 multi line docstring summary not separated with an empty line",
"tag": "autogenerated:zuul:check",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_c6d94c85",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 926,
"author": {
"id": 22348
},
"writtenOn": "2020-08-28T07:28:01Z",
"side": 1,
"message": "pep8: W293 blank line contains whitespace",
"tag": "autogenerated:zuul:check",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_6025c3ae",
"filename": "dracclient/client.py",
"patchSetId": 3
},
"lineNbr": 926,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "Done",
"parentUuid": "9f560f44_c6d94c85",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_111ffcb7",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 64,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "same comments as elsewhere",
"range": {
"startLine": 62,
"startChar": 12,
"endLine": 64,
"endChar": 23
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_408cff34",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 64,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "done",
"parentUuid": "9f560f44_111ffcb7",
"range": {
"startLine": 62,
"startChar": 12,
"endLine": 64,
"endChar": 23
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_b121f0ff",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 66,
"author": {
"id": 27909
},
"writtenOn": "2020-08-28T09:17:42Z",
"side": 1,
"message": "is it the time when machine was last rebooted? Or Last system inventory time?",
"range": {
"startLine": 66,
"startChar": 18,
"endLine": 66,
"endChar": 60
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_80f577f0",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 66,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "last system inventory time , its updated",
"parentUuid": "9f560f44_b121f0ff",
"range": {
"startLine": 66,
"startChar": 18,
"endLine": 66,
"endChar": 60
},
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_a6d6d852",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 71,
"author": {
"id": 22348
},
"writtenOn": "2020-08-28T07:28:01Z",
"side": 1,
"message": "pep8: H405 multi line docstring summary not separated with an empty line",
"tag": "autogenerated:zuul:check",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_e0d9d384",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 71,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "Done",
"parentUuid": "9f560f44_a6d6d852",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_06e4e43b",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 72,
"author": {
"id": 22348
},
"writtenOn": "2020-08-28T07:28:01Z",
"side": 1,
"message": "pep8: W293 blank line contains whitespace",
"tag": "autogenerated:zuul:check",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "9f560f44_c0d64f54",
"filename": "dracclient/resources/system.py",
"patchSetId": 3
},
"lineNbr": 72,
"author": {
"id": 32108
},
"writtenOn": "2020-09-04T12:20:12Z",
"side": 1,
"message": "Done",
"parentUuid": "9f560f44_06e4e43b",
"revId": "87b709a40dbf2aa5fa4c63c4031d82b680286c76",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}