104 lines
3.0 KiB
Plaintext
104 lines
3.0 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"key": {
|
|
"uuid": "9a89bdaa_4ef4565a",
|
|
"filename": "manifests/master.pp",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 129,
|
|
"author": {
|
|
"id": 7979
|
|
},
|
|
"writtenOn": "2016-09-12T16:52:54Z",
|
|
"side": 1,
|
|
"message": "Using /tmp means that on each restart the deb would get unnecessarily pulled in again. I am not sure what is canonical path to use in cases like these. Maybe /opt/ ?",
|
|
"range": {
|
|
"startLine": 129,
|
|
"startChar": 0,
|
|
"endLine": 129,
|
|
"endChar": 44
|
|
},
|
|
"revId": "6cfa126636fbe042fb212ca2d9b8c2285fface37",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": false
|
|
},
|
|
{
|
|
"key": {
|
|
"uuid": "9a89bdaa_677a18f3",
|
|
"filename": "manifests/master.pp",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 129,
|
|
"author": {
|
|
"id": 6494
|
|
},
|
|
"writtenOn": "2016-09-12T19:51:52Z",
|
|
"side": 1,
|
|
"message": "That\u0027s true, I don\u0027t think /opt is a good alternative though - maybe /var/lib/jenkins or /home/jenkins?",
|
|
"parentUuid": "9a89bdaa_4ef4565a",
|
|
"range": {
|
|
"startLine": 129,
|
|
"startChar": 0,
|
|
"endLine": 129,
|
|
"endChar": 44
|
|
},
|
|
"revId": "6cfa126636fbe042fb212ca2d9b8c2285fface37",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": false
|
|
},
|
|
{
|
|
"key": {
|
|
"uuid": "9a89bdaa_107500f9",
|
|
"filename": "manifests/master.pp",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 129,
|
|
"author": {
|
|
"id": 6039
|
|
},
|
|
"writtenOn": "2016-09-12T21:07:45Z",
|
|
"side": 1,
|
|
"message": "The two alternatives that make sense to me are to put it in /var/tmp so that it\u0027s still understood to be a temp file but will persist across reboots, or we could make a /var/lib/jenkins/debs directory for these files.",
|
|
"revId": "6cfa126636fbe042fb212ca2d9b8c2285fface37",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": false
|
|
},
|
|
{
|
|
"key": {
|
|
"uuid": "9a89bdaa_40741535",
|
|
"filename": "manifests/master.pp",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 129,
|
|
"author": {
|
|
"id": 7979
|
|
},
|
|
"writtenOn": "2016-09-12T23:17:50Z",
|
|
"side": 1,
|
|
"message": "I think /var/tmp/ is good enough.",
|
|
"parentUuid": "9a89bdaa_107500f9",
|
|
"revId": "6cfa126636fbe042fb212ca2d9b8c2285fface37",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": false
|
|
},
|
|
{
|
|
"key": {
|
|
"uuid": "7a8ec9b2_c40dcb38",
|
|
"filename": "manifests/master.pp",
|
|
"patchSetId": 1
|
|
},
|
|
"lineNbr": 129,
|
|
"author": {
|
|
"id": 6494
|
|
},
|
|
"writtenOn": "2016-09-15T09:59:23Z",
|
|
"side": 1,
|
|
"message": "Done",
|
|
"parentUuid": "9a89bdaa_40741535",
|
|
"revId": "6cfa126636fbe042fb212ca2d9b8c2285fface37",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
|
"unresolved": false
|
|
}
|
|
]
|
|
} |