)]}'
{"specs/kilo/approved/docker-hypervisor-plugin.rst":[{"author":{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},"change_message_id":"15c162ec95f1b7647510cc113d9a965760a5cc0c","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Docker (and containers in general) also have features which we may wish to"},{"line_number":38,"context_line":"consider supporting by creating a specialized container service.  However, this"},{"line_number":39,"context_line":"is not within the scope of this proposal."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Proposed change"},{"line_number":42,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a890539_66f3a29e","line":40,"updated":"2014-11-13 14:03:03.000000000","message":"Needs \"Use Cases\" and \"Project Priority\" sections here","commit_id":"fa0a39db108c4c7d2ff4ab532abc69b676e896dc"},{"author":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"change_message_id":"43b7939aaf3b213dc90d1a18764c4dfec9f5537b","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Several alternatives exist to work around not having native Docker container"},{"line_number":69,"context_line":"management in OpenStack. They are as follows:"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"* Get the nova-docker hypervisor plugin from stackforge and configure Nova to"},{"line_number":72,"context_line":"  use it."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"* Introduce a new container service in OpenStack that is completely independent"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_dcac4b8f","line":71,"updated":"2014-11-20 15:33:37.000000000","message":"Another alternative is to move nova-docker into the openstack namespace and have it \"blessed\". Consider options for possibly vendoring the code into Nova rather than bringing it into the review queue.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"5a28dc2c26b360f0719004a184d5b9490c4a9d6b","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Several alternatives exist to work around not having native Docker container"},{"line_number":69,"context_line":"management in OpenStack. They are as follows:"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"* Get the nova-docker hypervisor plugin from stackforge and configure Nova to"},{"line_number":72,"context_line":"  use it."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"* Introduce a new container service in OpenStack that is completely independent"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_2f5b4188","line":71,"in_reply_to":"5a890539_dcac4b8f","updated":"2014-11-20 22:02:51.000000000","message":"What would that accomplish?","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a4a030edd0179473825ea2a99b2737bd4c4ed571","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Several alternatives exist to work around not having native Docker container"},{"line_number":69,"context_line":"management in OpenStack. They are as follows:"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"* Get the nova-docker hypervisor plugin from stackforge and configure Nova to"},{"line_number":72,"context_line":"  use it."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"* Introduce a new container service in OpenStack that is completely independent"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a961159_9ecfac28","line":71,"in_reply_to":"5a890539_dcac4b8f","updated":"2014-12-12 12:33:08.000000000","message":"vendoring code means it\u0027s not actually maintained or fixed. I think that\u0027s completely counter productive.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"639a60af483c95385f6bd85ba28a92a847975d91","unresolved":false,"context_lines":[{"line_number":152,"context_line":"-----------"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"Primary assignee:"},{"line_number":155,"context_line":"  \u003cewindisch\u003e"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"Other contributors:"},{"line_number":158,"context_line":"  \u003ccalfonso\u003e"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_bec06e2b","line":155,"updated":"2014-11-20 00:29:39.000000000","message":"I would like to see a +1 from Eric on this patch","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"change_message_id":"43b7939aaf3b213dc90d1a18764c4dfec9f5537b","unresolved":false,"context_lines":[{"line_number":152,"context_line":"-----------"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"Primary assignee:"},{"line_number":155,"context_line":"  \u003cewindisch\u003e"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"Other contributors:"},{"line_number":158,"context_line":"  \u003ccalfonso\u003e"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_1ca173a9","line":155,"in_reply_to":"5a890539_bec06e2b","updated":"2014-11-20 15:33:37.000000000","message":"I\u0027d like clarity on how we\u0027ll be able to get code reviewed if it goes back into Nova\u0027s tree.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"639a60af483c95385f6bd85ba28a92a847975d91","unresolved":false,"context_lines":[{"line_number":177,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":178,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":179,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":180,"context_line":"  support is being actively worked on."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":183,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_9ec3b237","line":180,"updated":"2014-11-20 00:29:39.000000000","message":"I don\u0027t think we should accept this without volume support. That is a pretty basic feature. But not sure what the outcome of the minimum virt driver feature discussion was.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a4a030edd0179473825ea2a99b2737bd4c4ed571","unresolved":false,"context_lines":[{"line_number":177,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":178,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":179,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":180,"context_line":"  support is being actively worked on."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":183,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":7,"id":"3a961159_1e757cf9","line":180,"in_reply_to":"5a890539_0f54aa52","updated":"2014-12-12 12:33:08.000000000","message":"Cinder \u0026 Volumes aren\u0027t required for Nova, it\u0027s still considered an optional enhancement. Lots of things work fine without it.\n\nI honestly think this is a case where the administrator building their cloud can be considered smart enough to know if their users don\u0027t need that feature. The decision for docker is going to be based on other characteristics that they want.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"change_message_id":"43b7939aaf3b213dc90d1a18764c4dfec9f5537b","unresolved":false,"context_lines":[{"line_number":177,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":178,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":179,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":180,"context_line":"  support is being actively worked on."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":183,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_9cdac3e7","line":180,"in_reply_to":"5a890539_5ad30938","updated":"2014-11-20 15:33:37.000000000","message":"Boot from volume is something we are discussing as viable.\n\nWe will be able to support Cinder attach/detach soon, pending patchsets to Docker itself.\n\nThat doesn\u0027t address mounting, but an expectation of being able to mount filesystems is outside either of the Nova and Cinder APIs -- even speaking to Mike Perez, he agrees there is no reason to expect instances to mount filesystems as the contract is purely on the ability to provide the block device.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"b94770dc744e3ae14926b38949b68db7db80208a","unresolved":false,"context_lines":[{"line_number":177,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":178,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":179,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":180,"context_line":"  support is being actively worked on."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":183,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_0f54aa52","line":180,"in_reply_to":"5a890539_9cdac3e7","updated":"2014-11-21 15:07:23.000000000","message":"BTW, it should be noted that the Ironic driver doesn\u0027t support block volumes in any way. So we already have precedence of allowing drivers without this support \u0026 I don\u0027t think anyone would claim Ironic is useless without this feature. It would be unreasonable to hold Docker to a different standard than Ironic.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"33860741abe1ba55d37ff093252b9b100949da39","unresolved":false,"context_lines":[{"line_number":177,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":178,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":179,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":180,"context_line":"  support is being actively worked on."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":183,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_5ad30938","line":180,"in_reply_to":"5a890539_9e057216","updated":"2014-11-20 10:40:18.000000000","message":"There\u0027s no fundamental reason why Docker cannot do volumes. Certainly it is technically possible to have a volume for the root disk - it would just be another filesystem at the end of the day.  Doing hotplug/unplug of volumes is where any complexity / trouble would be, because Nova\u0027s (somewhat poorly designed) block hotplug mechansim relies on specifying the disk device node, whereas for docker you\u0027d really want to specify a target mount point. That\u0027s not an insurmountable problem - Nova could easily be extended to allow that in the block device mapping, but I don\u0027pt think that\u0027s something we need care about now.\n\nI don\u0027t consider volume support to be a mandatory requirement for virt drivers in Nova. There are plenty of ways in which you can make great use of Nova without any use of cinder volumes\n\n* Guest instance mounts a filesytem to get persistent storage from some network  filesystem (eg NFS, gluster, etc) provided by another VM instance they own\n* Guest instance is relying entirely on a network accessed database (oracle, postgresql, database as a service, etc) for persistent storage\n* Guest instance is just a transational processing system reading jobs off a message bus and putting results back on a message bus.\n\nThere are other examples I could come up with, but even those three clearly demonstrate there is value in using Nova without any support for block storage whatsoever being required. So calling cinder support mandatory is just putting an artificial obstacle in the way of our users wanting to make use of Docker","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"b888a54e0d83928fcb90e829b1976728392d2bba","unresolved":false,"context_lines":[{"line_number":177,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":178,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":179,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":180,"context_line":"  support is being actively worked on."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":183,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_9e057216","line":180,"in_reply_to":"5a890539_9ec3b237","updated":"2014-11-20 00:48:24.000000000","message":"I don\u0027t think that everyone felt that volume support was critical. Personally, I do, and I think it\u0027s a good benchmark for determining what belongs as a nova driver and what doesn\u0027t. Ironic is the oddball out here, but it seems to me like if you can\u0027t do volumes for some fundamental reason, maybe being a nova driver isn\u0027t the right level of abstraction.","commit_id":"7ad058ec3b08df383270637cfb71fb5f180f7391"}],"specs/kilo/docker-hypervisor-plugin.rst":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"bbe2fdebccc14f17fa77ce01a7094735726964d8","unresolved":false,"context_lines":[{"line_number":44,"context_line":"The proposal is that we move the stackforge/nova-docker directory, which"},{"line_number":45,"context_line":"contains all the code for the docker hypervisor plugin into the"},{"line_number":46,"context_line":"openstack/nova/nova/virt source directory along side the rest of the hypervisor"},{"line_number":47,"context_line":"plugins shipped with Nova."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Alternatives"},{"line_number":50,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_6a0028ab","line":47,"updated":"2014-10-20 12:17:40.000000000","message":"So basically returning to the way Nova supported docker in Icehouse.\n\nI think this is fine, as in retrospect I think we were mistaken in our desire to delete the Docker code from the project in the first place. It hasn\u0027t served our users well because it merely increased the pain in deploying \u0026 configuring it.\n\nThere is the ongoing discussion about the add-on containers APIs support, but I don\u0027t think we need to block docker inclusion in Nova for that. Even as Nova APIs stand today the docker driver I useful to people - as is the existing in tree LXC driver and proposed parallels container driver.","commit_id":"cda12560e72b6d873a036f1c03dab3d6346f0a18"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"bbe2fdebccc14f17fa77ce01a7094735726964d8","unresolved":false,"context_lines":[{"line_number":61,"context_line":"* Disregard the capabilities added to OpenStack via docker support with scheduler"},{"line_number":62,"context_line":"  support."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* Not include Docker container management in OpenStack at all."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Data model impact"},{"line_number":67,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_0ae79c13","line":64,"updated":"2014-10-20 12:17:40.000000000","message":"These are all worse than just having docker be part of Nova, but good list of alternatives none the less.","commit_id":"cda12560e72b6d873a036f1c03dab3d6346f0a18"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"bbe2fdebccc14f17fa77ce01a7094735726964d8","unresolved":false,"context_lines":[{"line_number":122,"context_line":"  the capability to attach and detach a host device to a running docker"},{"line_number":123,"context_line":"  container. Additionally, the implementation in the hypervisor plugin has"},{"line_number":124,"context_line":"  been added to call the Docker API to add/remove a device to/from a container."},{"line_number":125,"context_line":"  This will be merged once the Docker API has been merged."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Assignee(s)"},{"line_number":128,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_4a6c449d","line":125,"updated":"2014-10-20 12:17:40.000000000","message":"I don\u0027t think that volume attach/detach is a mandatory requirement for having a virt driver in tree, so while it would be nice to have I don\u0027t think we need consider it a blocker.","commit_id":"cda12560e72b6d873a036f1c03dab3d6346f0a18"},{"author":{"_account_id":7546,"name":"Blair Bethwaite","email":"blair.bethwaite@gmail.com","username":"blairo"},"change_message_id":"18ef1f922b563b1c09a3a3b0a28fa6190ed01e26","unresolved":false,"context_lines":[{"line_number":122,"context_line":"  the capability to attach and detach a host device to a running docker"},{"line_number":123,"context_line":"  container. Additionally, the implementation in the hypervisor plugin has"},{"line_number":124,"context_line":"  been added to call the Docker API to add/remove a device to/from a container."},{"line_number":125,"context_line":"  This will be merged once the Docker API has been merged."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Assignee(s)"},{"line_number":128,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_1a44a172","line":125,"in_reply_to":"9aa7fdbe_1d7fcde1","updated":"2014-10-27 01:43:29.000000000","message":"Easily deployable Docker would be of great value even without *any* volumes support. Isn\u0027t it better to have this land early in the cycle and track the attach/detach support as a separate/sub item? Then there\u0027s less chance of ending up in a situation of having a big patch set land late due to one feature holding everything else back.","commit_id":"cda12560e72b6d873a036f1c03dab3d6346f0a18"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e3f64e6d2f02a6e80d61d9e27ee7228a3d8ba962","unresolved":false,"context_lines":[{"line_number":122,"context_line":"  the capability to attach and detach a host device to a running docker"},{"line_number":123,"context_line":"  container. Additionally, the implementation in the hypervisor plugin has"},{"line_number":124,"context_line":"  been added to call the Docker API to add/remove a device to/from a container."},{"line_number":125,"context_line":"  This will be merged once the Docker API has been merged."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Assignee(s)"},{"line_number":128,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_1d7fcde1","line":125,"in_reply_to":"9aa7fdbe_4a6c449d","updated":"2014-10-21 01:01:57.000000000","message":"I think volume attach detach is required, so this should happen before the docker driver lands in nova.","commit_id":"cda12560e72b6d873a036f1c03dab3d6346f0a18"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e3f64e6d2f02a6e80d61d9e27ee7228a3d8ba962","unresolved":false,"context_lines":[{"line_number":128,"context_line":"-----------"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Primary assignee:"},{"line_number":131,"context_line":"  \u003cewindisch\u003e"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Other contributors:"},{"line_number":134,"context_line":"  \u003ccalfonso\u003e"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_9d6add18","line":131,"updated":"2014-10-21 01:01:57.000000000","message":"If Eric is the primary assignee, I would like to see his +1 on this","commit_id":"cda12560e72b6d873a036f1c03dab3d6346f0a18"}],"specs/liberty/approved/docker-hypervisor-plugin.rst":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"4a957fc0d5f0ef3b759c0f7c25ebd4f8cc985330","unresolved":false,"context_lines":[{"line_number":64,"context_line":"The proposal is that we move the stackforge/nova-docker directory, which"},{"line_number":65,"context_line":"contains all the code for the docker hypervisor plugin into the"},{"line_number":66,"context_line":"openstack/nova/nova/virt source directory along side the rest of the hypervisor"},{"line_number":67,"context_line":"plugins shipped with Nova."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Alternatives"},{"line_number":70,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3acd31a7_0d107ea7","line":67,"updated":"2015-05-01 15:04:13.000000000","message":"FYI I think it would be useful if you could submit a change to gerrit that updates this table with details of what APIs you intend to have coverage for in Docker once merged.\n\nhttp://docs.openstack.org/developer/nova/support-matrix.html","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"7cb67739742880a603c319cf02a32c8703b9334b","unresolved":false,"context_lines":[{"line_number":64,"context_line":"The proposal is that we move the stackforge/nova-docker directory, which"},{"line_number":65,"context_line":"contains all the code for the docker hypervisor plugin into the"},{"line_number":66,"context_line":"openstack/nova/nova/virt source directory along side the rest of the hypervisor"},{"line_number":67,"context_line":"plugins shipped with Nova."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Alternatives"},{"line_number":70,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3acd31a7_88b59c32","line":67,"in_reply_to":"3acd31a7_0d107ea7","updated":"2015-05-01 15:19:52.000000000","message":"Thanks Daniel. will do.","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":76,"context_line":"  use it."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Introduce a new container service in OpenStack that is completely independent"},{"line_number":79,"context_line":"  of Nova APIs and is soley focused on container capabilities."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"* Disregard the capabilities added to OpenStack via docker support with"},{"line_number":82,"context_line":"  scheduler support."}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_5c2f8ca1","line":79,"updated":"2015-05-06 07:54:39.000000000","message":"This now already exists:\nhttps://wiki.openstack.org/wiki/Magnum","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":79,"context_line":"  of Nova APIs and is soley focused on container capabilities."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"* Disregard the capabilities added to OpenStack via docker support with"},{"line_number":82,"context_line":"  scheduler support."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"* Not include Docker container management in OpenStack at all."},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_bc16a053","line":82,"updated":"2015-05-06 07:54:39.000000000","message":"What are these capabilities we get?","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":81,"context_line":"* Disregard the capabilities added to OpenStack via docker support with"},{"line_number":82,"context_line":"  scheduler support."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"* Not include Docker container management in OpenStack at all."},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"Data model impact"},{"line_number":87,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_7c092831","line":84,"updated":"2015-05-06 07:54:39.000000000","message":"Its already included here:\nhttps://wiki.openstack.org/wiki/Magnum","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"e35addeb359dcca64f11c6a5d10751d1271b7c19","unresolved":false,"context_lines":[{"line_number":91,"context_line":"REST API impact"},{"line_number":92,"context_line":"---------------"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"None"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Security impact"},{"line_number":97,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"9af37de9_21095096","line":94,"updated":"2015-05-11 08:23:07.000000000","message":"Is that true, I think we should discuss how there will be no container specific APIs added in Nova.","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":124,"context_line":"  installed with the rest of the hypervisor plugins."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"* The Docker plugin will be better maintained and integration testing can"},{"line_number":127,"context_line":"  be run on all nova changes ensuring ongoing functionality and development."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"* One could deploy nova compute instances within VMs allowing you to"},{"line_number":130,"context_line":"  distribute containers across a multitude of VMs and scale accordingly."}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_9f3ab6cc","line":127,"updated":"2015-05-06 07:54:39.000000000","message":"We should do this before we merge, ideally. Its not a requirement for this change right?","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":127,"context_line":"  be run on all nova changes ensuring ongoing functionality and development."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"* One could deploy nova compute instances within VMs allowing you to"},{"line_number":130,"context_line":"  distribute containers across a multitude of VMs and scale accordingly."},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"Developer impact"},{"line_number":133,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_1f4d8672","line":130,"updated":"2015-05-06 07:54:39.000000000","message":"Why? isn\u0027t magnum a better fit for that.","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":150,"context_line":"* Security considerations - Some operations within containers are considered"},{"line_number":151,"context_line":"  potentially threatening to the host machine.  Container technology itself is"},{"line_number":152,"context_line":"  not considered entirely secure and having multiple tenants operating on the"},{"line_number":153,"context_line":"  same hardware may be a potential security issue."},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Assignee(s)"},{"line_number":156,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_9faf56e3","line":153,"updated":"2015-05-06 07:54:39.000000000","message":"so this would only work for single tenant clouds I guess, should this not be in the Security Impact?","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"e4ab3b767cf463cbb77b262c0ef8acc7fcd66d27","unresolved":false,"context_lines":[{"line_number":182,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":183,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":184,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":185,"context_line":"  support is being actively worked on."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":188,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3acd31a7_e8141045","line":185,"updated":"2015-05-01 15:14:52.000000000","message":"the volumes patch was last updated 6 months ago: https://review.openstack.org/#/c/101243/\n\nI don\u0027t think that counts as actively worked on.","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"14bd118cc602dbc80d068acd61e21413994bde5e","unresolved":false,"context_lines":[{"line_number":182,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":183,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":184,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":185,"context_line":"  support is being actively worked on."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":188,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3acd31a7_a891789b","line":185,"in_reply_to":"3acd31a7_e8141045","updated":"2015-05-01 15:21:14.000000000","message":"Sorry one more update here - please see latest effort - https://review.openstack.org/#/c/178500/","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"a402ab9ecb03d29ac2d11855750a50edcf5801ad","unresolved":false,"context_lines":[{"line_number":182,"context_line":"  commit.  The Tempest configuration has a number of tests that don\u0027t apply to"},{"line_number":183,"context_line":"  containers that need to be turned off. The settings we are currently turning"},{"line_number":184,"context_line":"  off are: volume, resize, suspend, swift, rescue, cinder, and migrate.  Volume"},{"line_number":185,"context_line":"  support is being actively worked on."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"* A recent run of our Tempest coverage resulted in the following statistics:"},{"line_number":188,"context_line":"        Run: 1726 in 2750.246079 sec."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3acd31a7_689c80c0","line":185,"in_reply_to":"3acd31a7_e8141045","updated":"2015-05-01 15:19:02.000000000","message":"True, it\u0027s waiting on docker changes unfortunately\n\nhttps://github.com/docker/docker/issues/8829\nhttps://github.com/docker/docker/pulls?utf8\u003d%E2%9C%93\u0026q\u003dauthor%3Aimain","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fafe0929334e2b5cb99fae67a3e4d914d5cf4d5b","unresolved":false,"context_lines":[{"line_number":219,"context_line":"https://etherpad.openstack.org/p/container-block-storage"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"Openstack container meetings:"},{"line_number":222,"context_line":"https://wiki.openstack.org/wiki/Meetings/Containers"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1aca2d91_7f935aa4","line":222,"updated":"2015-05-06 07:54:39.000000000","message":"don\u0027t they now focus on Magnum, or is that separate?","commit_id":"3a36c87b1c0bb4be9d0f382d453a09ab137b1cca"}]}