)]}'
{"specs/newton/linux-ficon-support.rst":[{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"a4159088de4274c5835468fe9d00fbacc0a9a95a","unresolved":false,"context_lines":[{"line_number":102,"context_line":"\r"},{"line_number":103,"context_line":"\r"},{"line_number":104,"context_line":"Other contributors:\r"},{"line_number":105,"context_line":"  mzoeller\r"},{"line_number":106,"context_line":"  maiera\r"},{"line_number":107,"context_line":"\r"},{"line_number":108,"context_line":"\r"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_0a4494b4","line":105,"updated":"2016-04-07 14:55:02.000000000","message":"Maybe arecknag is the better contact here.","commit_id":"09a8f12b83e764ba70333011d35e6815eb09f19a"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"a3ff8dad0db9fcddc86719c0f8495dbc7222918a","unresolved":false,"context_lines":[{"line_number":245,"context_line":"Unit test:\r"},{"line_number":246,"context_line":"\r"},{"line_number":247,"context_line":"* Unit tests will be added and performed on System z, as well as\r"},{"line_number":248,"context_line":"  Intel-based machines.\r"},{"line_number":249,"context_line":"\r"},{"line_number":250,"context_line":"\r"},{"line_number":251,"context_line":"Documentation Impact\r"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_c4f77f89","line":248,"range":{"start_line":248,"start_character":0,"end_line":248,"end_character":23},"updated":"2016-04-06 12:57:31.000000000","message":"CI too?","commit_id":"09a8f12b83e764ba70333011d35e6815eb09f19a"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"03be81c525ecc98ff27f437c3784674822f82f21","unresolved":false,"context_lines":[{"line_number":245,"context_line":"Unit test:\r"},{"line_number":246,"context_line":"\r"},{"line_number":247,"context_line":"* Unit tests will be added and performed on System z, as well as\r"},{"line_number":248,"context_line":"  Intel-based machines.\r"},{"line_number":249,"context_line":"\r"},{"line_number":250,"context_line":"\r"},{"line_number":251,"context_line":"Documentation Impact\r"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_222901e7","line":248,"range":{"start_line":248,"start_character":0,"end_line":248,"end_character":23},"in_reply_to":"9a061dce_8838f13e","updated":"2016-04-06 15:19:54.000000000","message":"I think it is more accurate to say that we probably wouldn\u0027t merge this without a well maintained third party CI running it - any code that isn\u0027t tested quickly rots and this isn\u0027t technology that most people have any exposure to at all","commit_id":"09a8f12b83e764ba70333011d35e6815eb09f19a"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"ebdb3da1f81578bf5cdfc0199e828c16bd67671c","unresolved":false,"context_lines":[{"line_number":245,"context_line":"Unit test:\r"},{"line_number":246,"context_line":"\r"},{"line_number":247,"context_line":"* Unit tests will be added and performed on System z, as well as\r"},{"line_number":248,"context_line":"  Intel-based machines.\r"},{"line_number":249,"context_line":"\r"},{"line_number":250,"context_line":"\r"},{"line_number":251,"context_line":"Documentation Impact\r"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_8838f13e","line":248,"range":{"start_line":248,"start_character":0,"end_line":248,"end_character":23},"in_reply_to":"9a061dce_c4f77f89","updated":"2016-04-06 14:52:29.000000000","message":"I hope the answer will be yes :-) Need to clarify within our team who is going to do it","commit_id":"09a8f12b83e764ba70333011d35e6815eb09f19a"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1a122d0e_af753233","updated":"2016-05-04 07:10:09.000000000","message":"I replied to all comments and will upload an update in the next minutes.","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"ba5c2563420325f2e0d182e6974793a4bd99c91d","unresolved":false,"context_lines":[{"line_number":50,"context_line":"Change code in os-brick to address these issues, dependent on a new"},{"line_number":51,"context_line":"protocol type of \u0027fibre_channel_eckd\u0027, provided by the driver"},{"line_number":52,"context_line":"For details, see section `Work Items`."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"9a061dce_ea4200ad","line":53,"updated":"2016-04-07 14:54:51.000000000","message":"Will there be a Cinder Driver added? Any other changes in Cinder needed?","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[{"line_number":50,"context_line":"Change code in os-brick to address these issues, dependent on a new"},{"line_number":51,"context_line":"protocol type of \u0027fibre_channel_eckd\u0027, provided by the driver"},{"line_number":52,"context_line":"For details, see section `Work Items`."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_5233de61","line":53,"in_reply_to":"1a122d0e_399fb004","updated":"2016-05-04 07:10:09.000000000","message":"I agree with Alon. Existing drivers may be extended to support this. I will add a statement, that we will extend the existing driver for IBMs DS8000 and other vendors (such as EMC, Hitachi, HP) also provide storage subsystems supporting ECKD. They are candidates, too","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"6fb5e787fe7c27a9694d84156a4ef2bdbabc6a73","unresolved":false,"context_lines":[{"line_number":50,"context_line":"Change code in os-brick to address these issues, dependent on a new"},{"line_number":51,"context_line":"protocol type of \u0027fibre_channel_eckd\u0027, provided by the driver"},{"line_number":52,"context_line":"For details, see section `Work Items`."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_399fb004","line":53,"in_reply_to":"1a122d0e_960338b8","updated":"2016-04-23 11:50:35.000000000","message":"There are several potential drivers available to make use of this: https://en.wikipedia.org/wiki/FICON#Devices","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"7c12a9891bdc5032c62d06ec75f25271ddccab67","unresolved":false,"context_lines":[{"line_number":50,"context_line":"Change code in os-brick to address these issues, dependent on a new"},{"line_number":51,"context_line":"protocol type of \u0027fibre_channel_eckd\u0027, provided by the driver"},{"line_number":52,"context_line":"For details, see section `Work Items`."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_960338b8","line":53,"in_reply_to":"9a061dce_ea4200ad","updated":"2016-04-23 00:19:26.000000000","message":"Maybe some of the existing drivers can use this connector? Agreed it would be nice to know what is actually going to use it. Maybe in the \u0027Use Cases\u0027 section?","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"6fb5e787fe7c27a9694d84156a4ef2bdbabc6a73","unresolved":false,"context_lines":[{"line_number":112,"context_line":"  maiera"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Work Items"},{"line_number":116,"context_line":"----------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"In ``os-brick/initiator/connector.py``:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_b9810092","line":115,"range":{"start_line":115,"start_character":0,"end_line":115,"end_character":10},"updated":"2016-04-23 11:50:35.000000000","message":"I think we should separate the actual implementation from the spec. It\u0027s good to see that we have a clear road ahead, but it will be better to agree on the spec and then agree on the implementation.","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[{"line_number":112,"context_line":"  maiera"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Work Items"},{"line_number":116,"context_line":"----------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"In ``os-brick/initiator/connector.py``:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_4f1a9e19","line":115,"range":{"start_line":115,"start_character":0,"end_line":115,"end_character":10},"in_reply_to":"1a122d0e_b9810092","updated":"2016-05-04 07:10:09.000000000","message":"I am proposing to leave the code segments here to show what we plan to do. But focus should be on the spec itself and comments to the code changes should be done when releasing the code changes. Is this ok?","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"ba5c2563420325f2e0d182e6974793a4bd99c91d","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        \"\"\"Add the eckd volume to the Linux configuration. \"\"\""},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        full_device_identifier \u003d \"0.0.%04x\" % device_number"},{"line_number":194,"context_line":"        eckd_device_command \u003d (\u0027cio_ignore\u0027, \u0027-r\u0027,"},{"line_number":195,"context_line":"            \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":196,"context_line":"        LOG.debug(\"issue cio_ignore for s390: %s\", eckd_device_command)"},{"line_number":197,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"9a061dce_eafcc0b9","line":194,"updated":"2016-04-07 14:54:51.000000000","message":"cio_ignore will have to be present, thus installed. This should perhaps be an os-brick dependency?","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"7458b0be76b24b661acf6f723ce660e9713feca4","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        \"\"\"Add the eckd volume to the Linux configuration. \"\"\""},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        full_device_identifier \u003d \"0.0.%04x\" % device_number"},{"line_number":194,"context_line":"        eckd_device_command \u003d (\u0027cio_ignore\u0027, \u0027-r\u0027,"},{"line_number":195,"context_line":"            \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":196,"context_line":"        LOG.debug(\"issue cio_ignore for s390: %s\", eckd_device_command)"},{"line_number":197,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_9b2d7daa","line":194,"in_reply_to":"1a122d0e_76677cb7","updated":"2016-04-29 16:23:04.000000000","message":"OK, this is not an issue. Carry on....","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        \"\"\"Add the eckd volume to the Linux configuration. \"\"\""},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        full_device_identifier \u003d \"0.0.%04x\" % device_number"},{"line_number":194,"context_line":"        eckd_device_command \u003d (\u0027cio_ignore\u0027, \u0027-r\u0027,"},{"line_number":195,"context_line":"            \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":196,"context_line":"        LOG.debug(\"issue cio_ignore for s390: %s\", eckd_device_command)"},{"line_number":197,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_efd1ea30","line":194,"in_reply_to":"1a122d0e_9b2d7daa","updated":"2016-05-04 07:10:09.000000000","message":"Thanks","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"7c12a9891bdc5032c62d06ec75f25271ddccab67","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        \"\"\"Add the eckd volume to the Linux configuration. \"\"\""},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        full_device_identifier \u003d \"0.0.%04x\" % device_number"},{"line_number":194,"context_line":"        eckd_device_command \u003d (\u0027cio_ignore\u0027, \u0027-r\u0027,"},{"line_number":195,"context_line":"            \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":196,"context_line":"        LOG.debug(\"issue cio_ignore for s390: %s\", eckd_device_command)"},{"line_number":197,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_76677cb7","line":194,"in_reply_to":"9a061dce_eafcc0b9","updated":"2016-04-23 00:19:26.000000000","message":"Probably just needs to be documented. Same kind of story for multipath tools or things like the scaleio cli tool where its just something an admin has to install first.","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"ba5c2563420325f2e0d182e6974793a4bd99c91d","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                            \" %(code)s, stderr %(stderr)s\"),"},{"line_number":207,"context_line":"                        {\u0027code\u0027: exc.exit_code, \u0027stderr\u0027: exc.stderr})"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        eckd_device_command \u003d (\u0027chccwdev\u0027, \u0027-e\u0027,"},{"line_number":210,"context_line":"                \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":211,"context_line":"        LOG.debug(\"add ECKD command for s390: %s\", eckd_device_command)"},{"line_number":212,"context_line":"        out, info \u003d None, None"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9a061dce_2af198cd","line":209,"updated":"2016-04-07 14:54:51.000000000","message":"We\u0027d need to address having chccwdev usable (installed).","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                            \" %(code)s, stderr %(stderr)s\"),"},{"line_number":207,"context_line":"                        {\u0027code\u0027: exc.exit_code, \u0027stderr\u0027: exc.stderr})"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        eckd_device_command \u003d (\u0027chccwdev\u0027, \u0027-e\u0027,"},{"line_number":210,"context_line":"                \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":211,"context_line":"        LOG.debug(\"add ECKD command for s390: %s\", eckd_device_command)"},{"line_number":212,"context_line":"        out, info \u003d None, None"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_0fdf063b","line":209,"in_reply_to":"1a122d0e_b603300c","updated":"2016-05-04 07:10:09.000000000","message":"Thanks","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"7458b0be76b24b661acf6f723ce660e9713feca4","unresolved":false,"context_lines":[{"line_number":206,"context_line":"                            \" %(code)s, stderr %(stderr)s\"),"},{"line_number":207,"context_line":"                        {\u0027code\u0027: exc.exit_code, \u0027stderr\u0027: exc.stderr})"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        eckd_device_command \u003d (\u0027chccwdev\u0027, \u0027-e\u0027,"},{"line_number":210,"context_line":"                \u0027%(dev_id)s\u0027 % {\"dev_id\": full_device_identifier})"},{"line_number":211,"context_line":"        LOG.debug(\"add ECKD command for s390: %s\", eckd_device_command)"},{"line_number":212,"context_line":"        out, info \u003d None, None"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_b603300c","line":209,"in_reply_to":"9a061dce_2af198cd","updated":"2016-04-29 16:23:04.000000000","message":"Also, this is not an issue.","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"ba5c2563420325f2e0d182e6974793a4bd99c91d","unresolved":false,"context_lines":[{"line_number":250,"context_line":"Dependencies"},{"line_number":251,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"Nova blueprint to add support for ECKD for Linux on System z"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Testing"},{"line_number":256,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9a061dce_8a9b8465","line":253,"updated":"2016-04-07 14:54:51.000000000","message":"Please put the link here: https://blueprints.launchpad.net/nova/+spec/linux-ficon-support","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[{"line_number":250,"context_line":"Dependencies"},{"line_number":251,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"Nova blueprint to add support for ECKD for Linux on System z"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Testing"},{"line_number":256,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_0fb6e6f8","line":253,"in_reply_to":"9a061dce_8a9b8465","updated":"2016-05-04 07:10:09.000000000","message":"will do","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"ba5c2563420325f2e0d182e6974793a4bd99c91d","unresolved":false,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Testing"},{"line_number":256,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"Unit test:"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"* Unit tests will be added and performed on System z, as well as"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9a061dce_ead4c0ec","line":257,"updated":"2016-04-07 14:54:51.000000000","message":"As Duncan had stated, please document that 3rd Party CI will be run.","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11433,"name":"Stefan Amann","email":"stefan.amann@de.ibm.com","username":"StefanAmann"},"change_message_id":"88b51039d671769d0d6d22c71d460d1f82315db9","unresolved":false,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Testing"},{"line_number":256,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"Unit test:"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"* Unit tests will be added and performed on System z, as well as"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_4f8f3e2e","line":257,"in_reply_to":"1a122d0e_564b602b","updated":"2016-05-04 07:10:09.000000000","message":"will do","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"7c12a9891bdc5032c62d06ec75f25271ddccab67","unresolved":false,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Testing"},{"line_number":256,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"Unit test:"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"* Unit tests will be added and performed on System z, as well as"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a122d0e_564b602b","line":257,"in_reply_to":"9a061dce_ead4c0ec","updated":"2016-04-23 00:19:26.000000000","message":"+1","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"85a2230844e52765c6b944db18036b3053af500b","unresolved":false,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Testing"},{"line_number":256,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"Unit test:"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"* Unit tests will be added and performed on System z, as well as"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a18252c_ae83a7cb","line":257,"in_reply_to":"9a061dce_ead4c0ec","updated":"2016-04-12 15:08:42.000000000","message":"I agree. 3rd party CI should run on supporting drivers.","commit_id":"7e9db155264553ef19c166f6ec0a49c11719430c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"59d63ab28d8653e0a9c5db0f8bd7f02348f16ac6","unresolved":false,"context_lines":[{"line_number":249,"context_line":""},{"line_number":250,"context_line":"Cinder drivers need to report the Control Unit address, and Unit Address"},{"line_number":251,"context_line":"of the ECKD volume and set the driver_volume_type to \u0027fibre_channel_eckd\u0027"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"Dependencies"},{"line_number":254,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":255,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"dab17558_113a9634","line":252,"updated":"2016-05-13 14:46:36.000000000","message":"Can we just get a note added here that any drivers wishing to support FICON will need to have CI reporting results from running against a FICON attached host?","commit_id":"c7976ba06ff690381e9bd845f60348831a2be3d2"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"26686b0b5d9abc6361bce9903f464280ad091086","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        except putils.ProcessExecutionError as exc:"},{"line_number":243,"context_line":"            LOG.warning(_LW(\"remove ECKD call for s390 failed exit\""},{"line_number":244,"context_line":"                            \" %(code)s, stderr %(stderr)s\"),"},{"line_number":245,"context_line":"                        {\u0027code\u0027: exc.exit_code, \u0027stderr\u0027: exc.stderr})"},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"Adaptations to volume.filters to allow the new commands to be sent"},{"line_number":248,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"dab17558_766f66fc","line":245,"updated":"2016-05-16 05:43:19.000000000","message":"Writing code in the spec?:)  I don\u0027t think you need to add code here.","commit_id":"21e68e55a4143a785d2928eb24f5898fa76b0309"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"26686b0b5d9abc6361bce9903f464280ad091086","unresolved":false,"context_lines":[{"line_number":270,"context_line":""},{"line_number":271,"context_line":"CI environment:"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"* We will provide a 3rd party CI environment for DS8000."},{"line_number":274,"context_line":""},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":8,"id":"dab17558_b6238ebc","line":273,"updated":"2016-05-16 05:43:19.000000000","message":"Are you going to provide CI in both Cinder and os-brick?  This is a special protocol so CI in os-brick is required as well.","commit_id":"21e68e55a4143a785d2928eb24f5898fa76b0309"},{"author":{"_account_id":13816,"name":"Marco Pavone","email":"marco.pavone@de.ibm.com","username":"pavone"},"change_message_id":"8ac8db1044eeb0350b9765e3221e038f42252af8","unresolved":false,"context_lines":[{"line_number":270,"context_line":""},{"line_number":271,"context_line":"CI environment:"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"* We will provide a 3rd party CI environment for DS8000."},{"line_number":274,"context_line":""},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":8,"id":"bab6814e_daa82bde","line":273,"in_reply_to":"dab17558_b6238ebc","updated":"2016-05-19 09:09:42.000000000","message":"yes, os-brick will be part of the CI","commit_id":"21e68e55a4143a785d2928eb24f5898fa76b0309"}]}