)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"8cee813b0948ad9609fa035691443a821d56b6ba","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sreeram Vancheeswaran \u003csreeram.vancheeswaran@in.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-12 06:46:48 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding the specification for the blueprint https://blueprints.launchpad.net/nova/+spec/dpm-driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I2d873e4de50af362654fb272ef7b9235e1c887c7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_ba19d87b","line":7,"updated":"2016-10-12 13:59:44.000000000","message":"Just copy your text from the bp here to make it more readable.\n\n    Support IBM System z (S/390) PR/SM hypervisor in\n    DPM mode as a valid nova hypervisor platform\n\n    Add support for IBM PR/SM hypervisor in DPM mode as \n    a valid nova hypervisor platform. A new nova driver will\n    be implemented.\n\n    Additional OpenStack functionality beyond initial Nova\n    support is not part of this blueprint; we will have\n    specific additional blueprints for that, as needed.\n\n    A 3rd party Continuous Integration environment for\n    OpenStack for DPM on System z will be established and\n    maintained by IBM.\n\nMaybe you need to shorten the title to fit the max line length.","commit_id":"094e57661782060055ad9d0ddd67b696add6a906"}],"specs/ocata/approved/dpm-driver.rst":[{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"8cee813b0948ad9609fa035691443a821d56b6ba","unresolved":false,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Support IBM System z (S/390) PR/SM hypervisor in DPM mode as a valid nova hypervisor platform "},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/dpm-driver"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa6399be_f7d60926","line":8,"updated":"2016-10-12 13:59:44.000000000","message":"Please remove the space at the end of lines.","commit_id":"094e57661782060055ad9d0ddd67b696add6a906"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"8cee813b0948ad9609fa035691443a821d56b6ba","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Support IBM System z (S/390) PR/SM hypervisor in DPM mode as a valid nova hypervisor platform "},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/dpm-driver"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa6399be_57d25501","line":9,"updated":"2016-10-12 13:59:44.000000000","message":"nit: rst title markup (\u0027\u003d\u003d\u003d\u0027) should have the same length as the text.","commit_id":"094e57661782060055ad9d0ddd67b696add6a906"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"8cee813b0948ad9609fa035691443a821d56b6ba","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/dpm-driver"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Add support for IBM PR/SM hypervisor in DPM mode as a valid nova hypervisor platform.  A new nova driver will be implemented."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Additional OpenStack functionality beyond initial Nova support is not part of this blueprint; we will have specific additional blueprints for that, as needed."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa6399be_b7ed71f1","line":13,"updated":"2016-10-12 13:59:44.000000000","message":"You might want to introduce line breaks at around column 78. For all subsequent lines too.","commit_id":"094e57661782060055ad9d0ddd67b696add6a906"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"8cee813b0948ad9609fa035691443a821d56b6ba","unresolved":false,"context_lines":[{"line_number":76,"context_line":"Performance Impact"},{"line_number":77,"context_line":"------------------"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"None "},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Other deployer impact"},{"line_number":82,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa6399be_17ac5d9f","line":79,"updated":"2016-10-12 13:59:44.000000000","message":"Please remove the space","commit_id":"094e57661782060055ad9d0ddd67b696add6a906"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"c1f2b5ed23301049a3f98b6def1f72cfa67e7233","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Posting to get preliminary feedback on the scope of this spec."},{"line_number":110,"context_line":""},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Testing"},{"line_number":113,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa6399be_1aac849f","line":111,"updated":"2016-10-12 14:06:28.000000000","message":"You need to add:\n\n    Dependencies\n    \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\n    None\n\nhere to satisfy the template at https://raw.githubusercontent.com/openstack/nova-specs/master/specs/ocata-template.rst","commit_id":"094e57661782060055ad9d0ddd67b696add6a906"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bca173657f5331798e6d31d336cd9a0acacd71f2","unresolved":false,"context_lines":[{"line_number":38,"context_line":"This blueprint proposes to support the PR/SM hypervisor in DPM mode as a"},{"line_number":39,"context_line":"Nova hypervisor platform, by implementing a new Nova driver."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The other hypervisors on z Systems such as z/VM and KVM are already"},{"line_number":42,"context_line":"supported by OpenStack via Nova drivers. These drivers will continue to be"},{"line_number":43,"context_line":"supported. Adding support for PR/SM DPM allows addressing customers that"},{"line_number":44,"context_line":"need or want to run their workloads in logical partitions, and provides the"},{"line_number":45,"context_line":"advantage for them of using OpenStack as a uniform, standard, cloud"},{"line_number":46,"context_line":"management platform on z Systems."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Use Cases"},{"line_number":49,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa6399be_7e9f22b4","line":46,"range":{"start_line":41,"start_character":0,"end_line":46,"end_character":33},"updated":"2016-10-12 20:47:31.000000000","message":"So this is different from the zvm driver here?\n\nhttp://git.openstack.org/cgit/openstack/nova-zvm-virt-driver/\n\nIf so, then you\u0027d have:\n\n1. zVM\n2. zKVM\n3. zDPM\n\nIs that accurate?\n\nAnd the virt driver for DPM does not exist yet?","commit_id":"c0267acabd73071363023c255485ab2696ae9912"},{"author":{"_account_id":11351,"name":"Andreas Maier","email":"maiera@de.ibm.com","username":"maiera"},"change_message_id":"ec86ee86b158febc226ce44d75721e2c7a975671","unresolved":false,"context_lines":[{"line_number":38,"context_line":"This blueprint proposes to support the PR/SM hypervisor in DPM mode as a"},{"line_number":39,"context_line":"Nova hypervisor platform, by implementing a new Nova driver."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The other hypervisors on z Systems such as z/VM and KVM are already"},{"line_number":42,"context_line":"supported by OpenStack via Nova drivers. These drivers will continue to be"},{"line_number":43,"context_line":"supported. Adding support for PR/SM DPM allows addressing customers that"},{"line_number":44,"context_line":"need or want to run their workloads in logical partitions, and provides the"},{"line_number":45,"context_line":"advantage for them of using OpenStack as a uniform, standard, cloud"},{"line_number":46,"context_line":"management platform on z Systems."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Use Cases"},{"line_number":49,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa6399be_e2f3819c","line":46,"range":{"start_line":41,"start_character":0,"end_line":46,"end_character":33},"in_reply_to":"fa6399be_7e9f22b4","updated":"2016-10-13 08:53:05.000000000","message":"That is all accurate.","commit_id":"c0267acabd73071363023c255485ab2696ae9912"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bca173657f5331798e6d31d336cd9a0acacd71f2","unresolved":false,"context_lines":[{"line_number":60,"context_line":"worked upon. The nova driver will be implementd using the zhmcclient python"},{"line_number":61,"context_line":"library for interactions with the System z Hardware Management Console."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Posting to get preliminary feedback on the scope of this spec."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"Alternatives"},{"line_number":66,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa6399be_3ee4ca39","line":63,"updated":"2016-10-12 20:47:31.000000000","message":"I don\u0027t think it\u0027s even worth bringing up the idea of adding a new virt driver in nova when it doesn\u0027t even exist yet. At least the zVM driver exists.\n\nCompare this proposal to the powervm driver:\n\nhttps://review.openstack.org/#/c/381772/\n\nThat\u0027s basically been out of tree for ~2 years now, the team working on it has been attending the midcycles and summits, actively engaged, and they were told to (1) get third party CI working and (2) show usage before we\u0027d even consider inclusion within nova. This zDPM driver is nowhere near that point.","commit_id":"c0267acabd73071363023c255485ab2696ae9912"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bca173657f5331798e6d31d336cd9a0acacd71f2","unresolved":false,"context_lines":[{"line_number":144,"context_line":"Nova. Additional Nova unit tests for DPM specific behaviours will be"},{"line_number":145,"context_line":"written."},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"3rd party CI - 3rd party CI environment for DPM on Systemz will be set up"},{"line_number":148,"context_line":"and run by by IBM, to run full tempest tests."},{"line_number":149,"context_line":""},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa6399be_5e389ea0","line":148,"range":{"start_line":147,"start_character":0,"end_line":148,"end_character":45},"updated":"2016-10-12 20:47:31.000000000","message":"This would certainly be a requirement, but I very much doubt it\u0027s anything that would be completed in the Ocata release given the virt driver doesn\u0027t even exist. I have no idea if the zVM driver has third party CI, I know someone at one point in POK was working on it. And the zKVM support was supposed to have 3rd party CI since Kilo but that\u0027s still never happened.","commit_id":"c0267acabd73071363023c255485ab2696ae9912"}]}