kolla-kubernetes/0bdac3ef87f13120a90e87621a458246024cba8a

{
  "comments": [
    {
      "key": {
        "uuid": "1f013ff3_dce2f72f",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 98,
      "author": {
        "id": 23717
      },
      "writtenOn": "2017-05-15T06:28:30Z",
      "side": 1,
      "message": "I tried with freshllky installed ubuntu 16.04\n\nI think these are not for ubuntu please check.\nnot working for me.",
      "range": {
        "startLine": 97,
        "startChar": 0,
        "endLine": 98,
        "endChar": 62
      },
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "1f013ff3_e6a2a29a",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 98,
      "author": {
        "id": 22368
      },
      "writtenOn": "2017-05-16T07:59:53Z",
      "side": 1,
      "message": "These are not required for typical Ubuntu but AFAIK it could be relevant on some Ubuntu systems. I vote we just note that these commands might fail and thats ok.",
      "parentUuid": "1f013ff3_dce2f72f",
      "range": {
        "startLine": 97,
        "startChar": 0,
        "endLine": 98,
        "endChar": 62
      },
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "1f013ff3_dc50b7d7",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 103,
      "author": {
        "id": 23717
      },
      "writtenOn": "2017-05-15T06:28:30Z",
      "side": 1,
      "message": "ditto",
      "range": {
        "startLine": 102,
        "startChar": 4,
        "endLine": 103,
        "endChar": 36
      },
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "1f013ff3_c6520668",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 103,
      "author": {
        "id": 22368
      },
      "writtenOn": "2017-05-16T07:59:53Z",
      "side": 1,
      "message": "It is possible for Ubuntu to run firewalld although it might not be enabled by default. I vote we leave this non-platoform specific instruction with a note it might fail (and thats ok).",
      "parentUuid": "1f013ff3_dc50b7d7",
      "range": {
        "startLine": 102,
        "startChar": 4,
        "endLine": 103,
        "endChar": 36
      },
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "1f013ff3_fc6c939d",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 152,
      "author": {
        "id": 23717
      },
      "writtenOn": "2017-05-15T06:28:30Z",
      "side": 1,
      "message": "IMO we need to provide the all the dependecy should be installed at thr first place.\n\nIf above steps are working for you then please provide the your steps output.\n\nPlease follow http://paste.openstack.org/show/609517/ to find my experience of execution.",
      "range": {
        "startLine": 152,
        "startChar": 4,
        "endLine": 152,
        "endChar": 76
      },
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "1f013ff3_c6f68690",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 152,
      "author": {
        "id": 22368
      },
      "writtenOn": "2017-05-16T07:59:53Z",
      "side": 1,
      "message": "For ubuntu you need these packages \n\n- ca-certificates\n- apt-transport-https",
      "parentUuid": "1f013ff3_fc6c939d",
      "range": {
        "startLine": 152,
        "startChar": 4,
        "endLine": 152,
        "endChar": 76
      },
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    },
    {
      "key": {
        "uuid": "1f013ff3_a6ab7a1b",
        "filename": "doc/source/deployment-guide.rst",
        "patchSetId": 10
      },
      "lineNbr": 380,
      "author": {
        "id": 22368
      },
      "writtenOn": "2017-05-16T07:59:53Z",
      "side": 1,
      "message": "If these are different then they should probably be extracted into Centos and Ubuntu sections.",
      "revId": "0bdac3ef87f13120a90e87621a458246024cba8a",
      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
      "unresolved": false
    }
  ]
}