python-jenkins/eb7f2f07fca5649d40ff164a1c698f31d90706f3
Gerrit User 2475 e2e462f027 Update patch set 7
Patch Set 7:

(2 comments)

Patch-set: 7
Attention: {"person_ident":"Gerrit User 34829 \u003c34829@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_2475\u003e replied on the change"}
Attention: {"person_ident":"Gerrit User 2475 \u003c2475@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_2475\u003e replied on the change"}
2023-07-17 13:26:37 +00:00

144 lines
4.3 KiB
Plaintext

{
"comments": [
{
"unresolved": true,
"key": {
"uuid": "2bafa32f_08f4feb6",
"filename": ".zuul.yaml",
"patchSetId": 6
},
"lineNbr": 55,
"author": {
"id": 2475
},
"writtenOn": "2023-07-17T12:32:13Z",
"side": 1,
"message": "I think all those `nodeset: ubuntu-xenial` have been added by merged change https://review.opendev.org/c/jjb/python-jenkins/+/877699/",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "94ccb3b8_f02a5d69",
"filename": ".zuul.yaml",
"patchSetId": 6
},
"lineNbr": 55,
"author": {
"id": 34829
},
"writtenOn": "2023-07-17T12:59:48Z",
"side": 1,
"message": "Done",
"parentUuid": "2bafa32f_08f4feb6",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "a4c70a34_1bb23df3",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 6
},
"lineNbr": 0,
"author": {
"id": 2475
},
"writtenOn": "2023-07-17T12:32:13Z",
"side": 1,
"message": "I guess this change got amended with multiple modifications until it passed on CI. The fixes are available several small changes and as such should be rebased to only include the addition of new python versions :) Maybe once the other changes got merged if they are needed.",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "1020a471_2eb59fbf",
"filename": "test-requirements.txt",
"patchSetId": 6
},
"lineNbr": 16,
"author": {
"id": 2475
},
"writtenOn": "2023-07-17T12:32:13Z",
"side": 1,
"message": "multiprocess is proposed in another change: https://review.opendev.org/c/jjb/python-jenkins/+/865915",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "ffdcd751_4da536f1",
"filename": "test-requirements.txt",
"patchSetId": 6
},
"lineNbr": 16,
"author": {
"id": 34829
},
"writtenOn": "2023-07-17T12:59:48Z",
"side": 1,
"message": "right I removed it\nthough, I needed it to be able to test locally in my environment without errors.",
"parentUuid": "1020a471_2eb59fbf",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "6cc20ebd_785390f0",
"filename": "test-requirements.txt",
"patchSetId": 6
},
"lineNbr": 16,
"author": {
"id": 2475
},
"writtenOn": "2023-07-17T13:26:37Z",
"side": 1,
"message": "Ack :)",
"parentUuid": "ffdcd751_4da536f1",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "62ff7f65_a7f58bd6",
"filename": "tests/test_jenkins.py",
"patchSetId": 6
},
"lineNbr": 70,
"author": {
"id": 2475
},
"writtenOn": "2023-07-17T12:32:13Z",
"side": 1,
"message": "I have incorporated the use of `jenkins.DEFAULT_TIMEOUT` in https://review.opendev.org/c/jjb/python-jenkins/+/882757 :)",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "1553f931_0e3d3d18",
"filename": "tests/test_jenkins.py",
"patchSetId": 6
},
"lineNbr": 70,
"author": {
"id": 34829
},
"writtenOn": "2023-07-17T12:59:48Z",
"side": 1,
"message": "rebased to get it",
"parentUuid": "62ff7f65_a7f58bd6",
"revId": "eb7f2f07fca5649d40ff164a1c698f31d90706f3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}