utils/034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f

{
  "comments": [
    {
      "key": {
        "uuid": "3f79a3b5_23e82d12",
        "filename": "/COMMIT_MSG",
        "patchSetId": 8
      },
      "lineNbr": 11,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "This is part of a change sequence. Please add:\n\n Depends-On: https://review.openstack.org/613702\n Depends-On: https://review.openstack.org/614043",
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_e3d55549",
        "filename": "Makefile",
        "patchSetId": 8
      },
      "lineNbr": 56,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "I wouldn\u0027t necessarily expect a test target to install the Helm chart. What is the purpose of this target?",
      "range": {
        "startLine": 56,
        "startChar": 1,
        "endLine": 56,
        "endChar": 57
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_c7504562",
        "filename": "Makefile",
        "patchSetId": 8
      },
      "lineNbr": 56,
      "author": {
        "id": 21813
      },
      "writtenOn": "2018-11-26T23:58:47Z",
      "side": 1,
      "message": "I think it\u0027s more close to real-world usage.\nMaybe need to add clean-up after the test.",
      "parentUuid": "3f79a3b5_e3d55549",
      "range": {
        "startLine": 56,
        "startChar": 1,
        "endLine": 56,
        "endChar": 57
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_a3dbdd78",
        "filename": "charts/mini-mirror/requirements.yaml",
        "patchSetId": 8
      },
      "lineNbr": 2,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "nit: we do not include this character in any other repository header files.",
      "range": {
        "startLine": 2,
        "startChar": 12,
        "endLine": 2,
        "endChar": 15
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_25227554",
        "filename": "charts/mini-mirror/templates/deployment.yaml",
        "patchSetId": 8
      },
      "lineNbr": 2,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "nit: we do not include this character in any other repository header files.",
      "range": {
        "startLine": 2,
        "startChar": 12,
        "endLine": 2,
        "endChar": 15
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_63cb85a6",
        "filename": "charts/mini-mirror/templates/ingress.yaml",
        "patchSetId": 8
      },
      "lineNbr": 3,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "nit: we do not include this character in any other repository header files.",
      "range": {
        "startLine": 3,
        "startChar": 10,
        "endLine": 3,
        "endChar": 13
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_23c10d84",
        "filename": "charts/mini-mirror/templates/service.yaml",
        "patchSetId": 8
      },
      "lineNbr": 3,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "nit: we do not include this character in any other repository header files.",
      "range": {
        "startLine": 3,
        "startChar": 10,
        "endLine": 3,
        "endChar": 13
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_e3431511",
        "filename": "tools/gate/playbooks/zuul-check.yaml",
        "patchSetId": 8
      },
      "lineNbr": 18,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "Instead of using:\n\n sudo\n\nuse:\n\n become: yes\n\n[0] https://docs.ansible.com/ansible/2.6/user_guide/become.html#id1",
      "range": {
        "startLine": 18,
        "startChar": 15,
        "endLine": 18,
        "endChar": 19
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_a3499d30",
        "filename": "tools/gate/playbooks/zuul-check.yaml",
        "patchSetId": 8
      },
      "lineNbr": 18,
      "author": {
        "id": 28618
      },
      "writtenOn": "2018-11-23T18:00:22Z",
      "side": 1,
      "message": "Is there a reason there is a Makefile target for this? We could perform the installation directly from the playbook.",
      "range": {
        "startLine": 18,
        "startChar": 20,
        "endLine": 18,
        "endChar": 42
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "3f79a3b5_87db2de9",
        "filename": "tools/gate/playbooks/zuul-check.yaml",
        "patchSetId": 8
      },
      "lineNbr": 18,
      "author": {
        "id": 21813
      },
      "writtenOn": "2018-11-26T23:58:47Z",
      "side": 1,
      "message": "Yeah, what\u0027s the difference?",
      "parentUuid": "3f79a3b5_a3499d30",
      "range": {
        "startLine": 18,
        "startChar": 20,
        "endLine": 18,
        "endChar": 42
      },
      "revId": "034dae9e0f6a9e9e0c6bc581ca6d8e3ee1065b7f",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    }
  ]
}