)]}'
{"specs/approved/ilo-inspect-ports.rst":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7789d156d3fcd6be88aa6e57e97db8c6fbd117f8","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Add \u0027inspect_ports\u0027 as a \u0027driver_info\u0027 parameter to iLO drivers"},{"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\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://bugs.launchpad.net/ironic/+bug/1588177"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This specification proposes the change that allows the iLO drivers"},{"line_number":14,"context_line":"to inspect and create specific ironic ports passed by user."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f79a3b5_43481461","line":11,"updated":"2018-10-22 13:23:37.000000000","message":"Ironic does not use launchpad anymore. This should be reference the story in storyboard. https://storyboard.openstack.org.","commit_id":"c2cc7f334be64ea1ebd5e7c98f80a78cd574bcd2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7789d156d3fcd6be88aa6e57e97db8c6fbd117f8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"If some bare metals having two NICs are cabled such a way that one is"},{"line_number":20,"context_line":"cabled with fibre channel and other one is connected to ethernet. During"},{"line_number":21,"context_line":"provisioning only second NIC should be used as fibre channel NIC serves"},{"line_number":22,"context_line":"as the storage NIC and cannot be used for deploy."},{"line_number":23,"context_line":"In the current scenario there is no mechanism available to the user"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f79a3b5_604ae682","line":20,"range":{"start_line":19,"start_character":0,"end_line":20,"end_character":64},"updated":"2018-10-22 13:23:37.000000000","message":"It would likely be easier to just say they are both ethernet, but different classes or uses of networks.","commit_id":"c2cc7f334be64ea1ebd5e7c98f80a78cd574bcd2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7789d156d3fcd6be88aa6e57e97db8c6fbd117f8","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"If some bare metals having two NICs are cabled such a way that one is"},{"line_number":20,"context_line":"cabled with fibre channel and other one is connected to ethernet. During"},{"line_number":21,"context_line":"provisioning only second NIC should be used as fibre channel NIC serves"},{"line_number":22,"context_line":"as the storage NIC and cannot be used for deploy."},{"line_number":23,"context_line":"In the current scenario there is no mechanism available to the user"},{"line_number":24,"context_line":"to specify the NIC port that is to be used for provisioning. The Ironic"},{"line_number":25,"context_line":"\u0027inspection\u0027 creates ports for all the NICs being discovered."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f79a3b5_00b3f29b","line":22,"range":{"start_line":21,"start_character":47,"end_line":22,"end_character":48},"updated":"2018-10-22 13:23:37.000000000","message":"An FC adapter is not a ethernet nic, nor do we presently track them as NICs. They _should_ be tracked as storage volume connectors.","commit_id":"c2cc7f334be64ea1ebd5e7c98f80a78cd574bcd2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7789d156d3fcd6be88aa6e57e97db8c6fbd117f8","unresolved":false,"context_lines":[{"line_number":20,"context_line":"cabled with fibre channel and other one is connected to ethernet. During"},{"line_number":21,"context_line":"provisioning only second NIC should be used as fibre channel NIC serves"},{"line_number":22,"context_line":"as the storage NIC and cannot be used for deploy."},{"line_number":23,"context_line":"In the current scenario there is no mechanism available to the user"},{"line_number":24,"context_line":"to specify the NIC port that is to be used for provisioning. The Ironic"},{"line_number":25,"context_line":"\u0027inspection\u0027 creates ports for all the NICs being discovered."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f79a3b5_4047aa7b","line":24,"range":{"start_line":23,"start_character":36,"end_line":24,"end_character":60},"updated":"2018-10-22 13:23:37.000000000","message":"This is incorrect, each port has a pxe_enabled flag.","commit_id":"c2cc7f334be64ea1ebd5e7c98f80a78cd574bcd2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7789d156d3fcd6be88aa6e57e97db8c6fbd117f8","unresolved":false,"context_lines":[{"line_number":21,"context_line":"provisioning only second NIC should be used as fibre channel NIC serves"},{"line_number":22,"context_line":"as the storage NIC and cannot be used for deploy."},{"line_number":23,"context_line":"In the current scenario there is no mechanism available to the user"},{"line_number":24,"context_line":"to specify the NIC port that is to be used for provisioning. The Ironic"},{"line_number":25,"context_line":"\u0027inspection\u0027 creates ports for all the NICs being discovered."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"},{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f79a3b5_c0717ae1","line":25,"range":{"start_line":24,"start_character":61,"end_line":25,"end_character":61},"updated":"2018-10-22 13:23:37.000000000","message":"Why exactly is this a problem that we create the files on disk?","commit_id":"c2cc7f334be64ea1ebd5e7c98f80a78cd574bcd2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7789d156d3fcd6be88aa6e57e97db8c6fbd117f8","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Proposed change"},{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This spec proposes a mechanism for out-of-band inspection to"},{"line_number":31,"context_line":"specify which ports to create."},{"line_number":32,"context_line":"It will require implementation of \u0027inspect_ports\u0027 as an optional"},{"line_number":33,"context_line":"parameter to \u0027driver_info\u0027 for iLO drivers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This will be needed when user will do out-of-band inspection"},{"line_number":36,"context_line":"and wants to create ironic ports for specific NICs."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"This consists of following steps:"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* If inspect_ports parameter is present in driver_info,"},{"line_number":41,"context_line":"  first it will be validated whether a valid value has been passed or not."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* There will be three different values can be passed"},{"line_number":44,"context_line":"  \u0027list of port numbers\u0027, \u0027all\u0027 or \u0027none\u0027."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"  - if \u0027list of port numbers\u0027 will be passed, the specific"},{"line_number":47,"context_line":"    ironic ports will be created."},{"line_number":48,"context_line":"  - In case of \u0027all\u0027, ironic ports will be created for all detected NICs."},{"line_number":49,"context_line":"  - In case of \u0027none\u0027, no ironic ports will be created. User can create ports"},{"line_number":50,"context_line":"    after inspection."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"* If this parameter is not specified, Ironic would create ports for"},{"line_number":53,"context_line":"  all the discovered NIC ports."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"* To support this, the ironic.drivers.modules.ilo.inspect.IloInspect"},{"line_number":56,"context_line":"  will be modified."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"},{"line_number":59,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f79a3b5_c30d0437","line":56,"range":{"start_line":30,"start_character":0,"end_line":56,"end_character":19},"updated":"2018-10-22 13:23:37.000000000","message":"This is a different problem than the problem trying to be solved expressed in the problem statement. I\u0027m generally +1 for the ilo drivers to just go ahead and do this in their drivers without need of a spec to be approved. I understand the intent is to provide a similar mechanism the option in inspector and I believe that makes some sense for your IloInpsect interface.","commit_id":"c2cc7f334be64ea1ebd5e7c98f80a78cd574bcd2"}]}