)]}'
{"specs/approved/L3-based-deployment.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":"270d72b91c63c101254c1028cf80a85dbe874ac7","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Example use cases:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Multi Rack deployment: Inter-Rack L2 switch configuration is a challenge"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"* Remote site targets: When Target nodes are located in a remote routed network"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_97233aa7","line":27,"updated":"2018-02-13 16:34:48.000000000","message":"Can we get a little more detail on what is meant by a challenge? Perhaps you mean L2 switching beyond the Top of Rack switch?","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6ca403896124f78e7b6751fe6c78b17c0ea39498","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Example use cases:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Multi Rack deployment: Inter-Rack L2 switch configuration is a challenge"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"* Remote site targets: When Target nodes are located in a remote routed network"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f9dbf25_f9c04053","line":27,"in_reply_to":"3fa0c359_97233aa7","updated":"2018-02-28 14:38:52.000000000","message":"Done","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"270d72b91c63c101254c1028cf80a85dbe874ac7","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":35,"context_line":"and booting. As on date even these servers supporting booting with Virtual"},{"line_number":36,"context_line":"media rely on dhcp to get the IP configurations. On the Ironic Conductor side"},{"line_number":37,"context_line":"this IP config data of target node can be bundled as a configdrive and attached"},{"line_number":38,"context_line":"to the target node either to the end of ironic deploy ISO or Virtual Floppy"},{"line_number":39,"context_line":"image. On the Ironic-Python-Agent Ramdisk side a new service needs to be added"},{"line_number":40,"context_line":"to load the configdrive data to configure network before IPA service starts"},{"line_number":41,"context_line":"initiating REST APIs towards ironic conductor. os-net-config can be used for"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_1c019106","line":38,"range":{"start_line":36,"start_character":48,"end_line":38,"end_character":74},"updated":"2018-02-13 16:34:48.000000000","message":"How will this service tell the difference between a virtual config drive that is attached, and a config drive on the disk of the server IPA is booting up on to clean?","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6ca403896124f78e7b6751fe6c78b17c0ea39498","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":35,"context_line":"and booting. As on date even these servers supporting booting with Virtual"},{"line_number":36,"context_line":"media rely on dhcp to get the IP configurations. On the Ironic Conductor side"},{"line_number":37,"context_line":"this IP config data of target node can be bundled as a configdrive and attached"},{"line_number":38,"context_line":"to the target node either to the end of ironic deploy ISO or Virtual Floppy"},{"line_number":39,"context_line":"image. On the Ironic-Python-Agent Ramdisk side a new service needs to be added"},{"line_number":40,"context_line":"to load the configdrive data to configure network before IPA service starts"},{"line_number":41,"context_line":"initiating REST APIs towards ironic conductor. os-net-config can be used for"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f9dbf25_d989c426","line":38,"range":{"start_line":36,"start_character":48,"end_line":38,"end_character":74},"in_reply_to":"3fa0c359_1c019106","updated":"2018-02-28 14:38:52.000000000","message":"My bad! this is not configdrive but it is new boot parameter added to parameters.txt inside the virtual floppy drive. I fixed this terminology in my next revision.","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"270d72b91c63c101254c1028cf80a85dbe874ac7","unresolved":false,"context_lines":[{"line_number":36,"context_line":"media rely on dhcp to get the IP configurations. On the Ironic Conductor side"},{"line_number":37,"context_line":"this IP config data of target node can be bundled as a configdrive and attached"},{"line_number":38,"context_line":"to the target node either to the end of ironic deploy ISO or Virtual Floppy"},{"line_number":39,"context_line":"image. On the Ironic-Python-Agent Ramdisk side a new service needs to be added"},{"line_number":40,"context_line":"to load the configdrive data to configure network before IPA service starts"},{"line_number":41,"context_line":"initiating REST APIs towards ironic conductor. os-net-config can be used for"},{"line_number":42,"context_line":"converting/configuring configdrive IP data to realtime environment. This solves"},{"line_number":43,"context_line":"L2 dependency."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_a8393fa4","line":43,"range":{"start_line":39,"start_character":7,"end_line":43,"end_character":14},"updated":"2018-02-13 16:34:48.000000000","message":"The CoreOS IPA ramdisk already does this, however the problem with this in general is that coreos would confuse the configuration drive of a system it was booting on for cleaning or rebuilding, and then attempt to act upon it.","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6ca403896124f78e7b6751fe6c78b17c0ea39498","unresolved":false,"context_lines":[{"line_number":36,"context_line":"media rely on dhcp to get the IP configurations. On the Ironic Conductor side"},{"line_number":37,"context_line":"this IP config data of target node can be bundled as a configdrive and attached"},{"line_number":38,"context_line":"to the target node either to the end of ironic deploy ISO or Virtual Floppy"},{"line_number":39,"context_line":"image. On the Ironic-Python-Agent Ramdisk side a new service needs to be added"},{"line_number":40,"context_line":"to load the configdrive data to configure network before IPA service starts"},{"line_number":41,"context_line":"initiating REST APIs towards ironic conductor. os-net-config can be used for"},{"line_number":42,"context_line":"converting/configuring configdrive IP data to realtime environment. This solves"},{"line_number":43,"context_line":"L2 dependency."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Alternatives"},{"line_number":46,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f9dbf25_593e142a","line":43,"range":{"start_line":39,"start_character":7,"end_line":43,"end_character":14},"in_reply_to":"3fa0c359_a8393fa4","updated":"2018-02-28 14:38:52.000000000","message":"My bad again in terminology of using config drive in place of boot parameters in virtmedia. Fixed it!","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"270d72b91c63c101254c1028cf80a85dbe874ac7","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* Addition of new service and os-net-config tool in IPA Ramdisk."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"* Creating a reference Ironic Driver for one reference hardware with L3"},{"line_number":150,"context_line":"  deployment support."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Update documentation"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_28244f5d","line":149,"range":{"start_line":149,"start_character":30,"end_line":149,"end_character":36},"updated":"2018-02-13 16:34:48.000000000","message":"We don\u0027t use \"driver\" anymore, we use \"Hardware type\"","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6ca403896124f78e7b6751fe6c78b17c0ea39498","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* Addition of new service and os-net-config tool in IPA Ramdisk."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"* Creating a reference Ironic Driver for one reference hardware with L3"},{"line_number":150,"context_line":"  deployment support."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Update documentation"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f9dbf25_f91840d4","line":149,"range":{"start_line":149,"start_character":30,"end_line":149,"end_character":36},"in_reply_to":"3fa0c359_28244f5d","updated":"2018-02-28 14:38:52.000000000","message":"Done","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"270d72b91c63c101254c1028cf80a85dbe874ac7","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* Addition of new service and os-net-config tool in IPA Ramdisk."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"* Creating a reference Ironic Driver for one reference hardware with L3"},{"line_number":150,"context_line":"  deployment support."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Update documentation"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_e8c74763","line":150,"range":{"start_line":149,"start_character":69,"end_line":150,"end_character":21},"updated":"2018-02-13 16:34:48.000000000","message":"This seems misleading since this can be done today, but requires knowledge of and additional DHCP configuration. There is also our Routed Network support which realistically will achieve a portion of this.","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6ca403896124f78e7b6751fe6c78b17c0ea39498","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* Addition of new service and os-net-config tool in IPA Ramdisk."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"* Creating a reference Ironic Driver for one reference hardware with L3"},{"line_number":150,"context_line":"  deployment support."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Update documentation"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f9dbf25_d978e4f9","line":150,"range":{"start_line":149,"start_character":69,"end_line":150,"end_character":21},"in_reply_to":"3fa0c359_e8c74763","updated":"2018-02-28 14:38:52.000000000","message":"Dream is to make deployment DHCP free! I have updated the Spec with these statements. I hope it would be clear what we are aiming for!","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"270d72b91c63c101254c1028cf80a85dbe874ac7","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* os-net-config tool in IPA ramdisk"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"* Hardware specific APIs to attach detach Virtual Media"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"Testing"},{"line_number":162,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_97581a31","line":159,"updated":"2018-02-13 16:34:48.000000000","message":"Some of these already exist in ironic or in libraries ironic interacts with.\n\nThis would have to be classified as a specific style of boot interface most likely.","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6ca403896124f78e7b6751fe6c78b17c0ea39498","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* os-net-config tool in IPA ramdisk"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"* Hardware specific APIs to attach detach Virtual Media"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"Testing"},{"line_number":162,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f9dbf25_79699023","line":159,"in_reply_to":"3fa0c359_97581a31","updated":"2018-02-28 14:38:52.000000000","message":"Done","commit_id":"e8f3e466b73460cc90fa202a7b2d2366cabf0b6e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property fields eg: \"internal_info\" or \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_768102fb","line":54,"range":{"start_line":54,"start_character":54,"end_line":54,"end_character":62},"updated":"2018-03-30 15:10:58.000000000","message":"Extra is user writable, and seems to be the common place where people do this sort of thing, +1","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property fields eg: \"internal_info\" or \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_36962a4e","line":54,"range":{"start_line":54,"start_character":35,"end_line":54,"end_character":49},"updated":"2018-03-30 15:10:58.000000000","message":"Note: internal_info is not user writable.","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"970e5f6c5be25f56e073a4bd2cfe9d9535920d2c","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property fields eg: \"internal_info\" or \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_b3fbe444","line":54,"range":{"start_line":54,"start_character":35,"end_line":54,"end_character":49},"in_reply_to":"bf659307_36962a4e","updated":"2018-04-04 08:38:57.000000000","message":"Done","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"970e5f6c5be25f56e073a4bd2cfe9d9535920d2c","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property fields eg: \"internal_info\" or \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_73978ced","line":54,"range":{"start_line":54,"start_character":54,"end_line":54,"end_character":62},"in_reply_to":"bf659307_768102fb","updated":"2018-04-04 08:38:57.000000000","message":"Done","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property fields eg: \"internal_info\" or \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_b6d47a21","line":56,"range":{"start_line":56,"start_character":3,"end_line":56,"end_character":16},"updated":"2018-03-30 15:10:58.000000000","message":"So this is not exclusive to TripleO. Ironic merely powers tripleo, but there is a whole Bare Metal as a Service story. This is an integrated deployment.","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"970e5f6c5be25f56e073a4bd2cfe9d9535920d2c","unresolved":false,"context_lines":[{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property fields eg: \"internal_info\" or \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_339d9409","line":56,"range":{"start_line":56,"start_character":3,"end_line":56,"end_character":16},"in_reply_to":"bf659307_b6d47a21","updated":"2018-04-04 08:38:57.000000000","message":"Done","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_76da624b","line":59,"range":{"start_line":59,"start_character":11,"end_line":59,"end_character":15},"updated":"2018-03-30 15:10:58.000000000","message":"s/meda/media/","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"970e5f6c5be25f56e073a4bd2cfe9d9535920d2c","unresolved":false,"context_lines":[{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_53a04842","line":59,"range":{"start_line":59,"start_character":11,"end_line":59,"end_character":15},"in_reply_to":"bf659307_76da624b","updated":"2018-04-04 08:38:57.000000000","message":"Done","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_36d14a24","line":60,"range":{"start_line":59,"start_character":67,"end_line":60,"end_character":28},"updated":"2018-03-30 15:10:58.000000000","message":"I\u0027m unsure this is really needed. Such as with ironic-python-agent on coreos. What if we take the arguments/parameters and teach IPA to set interface configuration?","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"970e5f6c5be25f56e073a4bd2cfe9d9535920d2c","unresolved":false,"context_lines":[{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_9336e002","line":60,"range":{"start_line":59,"start_character":67,"end_line":60,"end_character":28},"in_reply_to":"bf659307_36d14a24","updated":"2018-04-04 08:38:57.000000000","message":"We are talking about complex network structures like IP on VLAN on PHY (or) IP on Bond on PHY interfaces (or) IP on VLAN on Bond on PHY. Easiest is IP on PHY. All of these are nicely handled by os-net-config tool. If we start adding these cases to IPA, it would be re-inventing wheel. In my opinion it easier to re-use stuff from os-net-config.","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"6835c278c2612697535c24051193ae155ef7fc3d","unresolved":false,"context_lines":[{"line_number":56,"context_line":"2. TripleO Mode:- Neutron\u0027s network info is derivable from ironic node-port or"},{"line_number":57,"context_line":"   portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmeda hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_bcf82dc4","line":60,"range":{"start_line":59,"start_character":67,"end_line":60,"end_character":28},"in_reply_to":"bf659307_9336e002","updated":"2018-04-09 20:56:51.000000000","message":"That is fair, thanks!","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"1. A new systemd service needs to be added to read the virtual media parameters"},{"line_number":66,"context_line":"   data. Look for os-net-config JSON structure. If found create a config file"},{"line_number":67,"context_line":"   /etc/os-net-config/config.json. (OR) Adding this logic in IPA itself."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"},{"line_number":70,"context_line":"   accordence to /etc/os-net-config/config.json, before IPA service starts"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_d6a7aebc","line":67,"range":{"start_line":67,"start_character":40,"end_line":67,"end_character":72},"updated":"2018-03-30 15:10:58.000000000","message":"+1 to adding to ipa, since new services might be problematic across possible build targets.","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"970e5f6c5be25f56e073a4bd2cfe9d9535920d2c","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"1. A new systemd service needs to be added to read the virtual media parameters"},{"line_number":66,"context_line":"   data. Look for os-net-config JSON structure. If found create a config file"},{"line_number":67,"context_line":"   /etc/os-net-config/config.json. (OR) Adding this logic in IPA itself."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"},{"line_number":70,"context_line":"   accordence to /etc/os-net-config/config.json, before IPA service starts"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_138650ba","line":67,"range":{"start_line":67,"start_character":40,"end_line":67,"end_character":72},"in_reply_to":"bf659307_d6a7aebc","updated":"2018-04-04 08:38:57.000000000","message":"Done","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1d801a6a39944df8c0fc753ddb019fbb55257f3e","unresolved":false,"context_lines":[{"line_number":191,"context_line":"Testing"},{"line_number":192,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"Test the ironic deployment without enabling L2 switch connectivity between"},{"line_number":195,"context_line":"Ironic-conductor and Target nodes."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":198,"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"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_567cfe1d","line":195,"range":{"start_line":194,"start_character":0,"end_line":195,"end_character":34},"updated":"2018-03-30 15:10:58.000000000","message":"This seems a little difficult to test without a specific third party CI doing this. Thoughts?","commit_id":"12a289f7d4b2e32340f8267c7a6377d8fe6861d0"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b13896da85260168a68e4ea715f0768b33288e1f","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_b57d896f","line":54,"updated":"2018-04-10 16:35:20.000000000","message":"\"extra\" should not be used by ironic, that is purely for operators. Is there a better field (or a new field?) that can work for this?","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"fce640900170d27da90763a8dd6e9523cf06a44b","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_e030b458","line":54,"in_reply_to":"9f6a8fd7_629b6317","updated":"2018-04-25 11:56:14.000000000","message":"Done","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"8f77f599c67f3b97b2c89cb6099c6f06bfcb6821","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_dd2242a3","line":54,"in_reply_to":"bf659307_b57d896f","updated":"2018-04-13 16:02:51.000000000","message":"Isn\u0027t the operators, who should configure these network values about the target node. If so what is the issue using this with a new sub-key eg:-\"network-config\".\n\n(If not)\n\nCan you suggest a better way?\n\"internal_info\" is good for case-2 (Integrated Deployment Mode) but can\u0027t be user writable in Stand-alone mode. Should we then go for a new property which can be used in both case-1 and case-2?","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"9a3c4c571cbd7915962f9560255e850fa6d0d3b4","unresolved":false,"context_lines":[{"line_number":51,"context_line":"as follows:"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_629b6317","line":54,"in_reply_to":"bf659307_dd2242a3","updated":"2018-04-24 13:21:23.000000000","message":"Well, ironic should not read the extra field either, IMO.\n\nPortgroup has a \"properties\" field, maybe we can add this to Port as well.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b13896da85260168a68e4ea715f0768b33288e1f","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_75dcd173","line":57,"updated":"2018-04-10 16:35:20.000000000","message":"Is this worth caching? For example, we might boot the ramdisk many times while cleaning.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"fce640900170d27da90763a8dd6e9523cf06a44b","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_8033f861","line":57,"in_reply_to":"9f6a8fd7_22ec8b8d","updated":"2018-04-25 11:56:14.000000000","message":"Done","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"9a3c4c571cbd7915962f9560255e850fa6d0d3b4","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_22ec8b8d","line":57,"in_reply_to":"bf659307_02dff90c","updated":"2018-04-24 13:21:23.000000000","message":"Sure, a quick mention would be good. A spec is about implementation details :)","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"8f77f599c67f3b97b2c89cb6099c6f06bfcb6821","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   portgroup\u0027s property field \"extra\"."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":57,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_02dff90c","line":57,"in_reply_to":"bf659307_75dcd173","updated":"2018-04-13 16:02:51.000000000","message":"Yes it would be worth. But isn\u0027t it an implementation detail!","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b13896da85260168a68e4ea715f0768b33288e1f","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_35f099c9","line":61,"updated":"2018-04-10 16:35:20.000000000","message":"Could you describe this structure, or link to a document on it? As I understand it, parameters.txt is simply a space-separated key\u003dvalue structure just like a kernel command line. Will the JSON be stuffed into this, and will it be remotely readable? I\u0027m imagining `os-net-config\u003d{\"some\":{\"keys\":[\"and\",\"values\"]}}`. Will spaces be escaped? How? Lots of questions here :)\n\nAlso, is there a better resource to link to than a (probably outdated) Juno spec?","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"fce640900170d27da90763a8dd6e9523cf06a44b","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_20422cac","line":61,"in_reply_to":"9f6a8fd7_dd39eed6","updated":"2018-04-25 11:56:14.000000000","message":"Done","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"8f77f599c67f3b97b2c89cb6099c6f06bfcb6821","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_ac0ac622","line":61,"in_reply_to":"bf659307_35f099c9","updated":"2018-04-13 16:02:51.000000000","message":"Done!\n\nI have updated with an example structure, and provided a link.\n\nYes they are key value pairs, can be read by https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/utils.py#L91","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"02465783831c66dcae26e9910e120ac2c73ee776","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":60,"context_line":"os-net-config JSON structure. This can be added to parameters.txt when running"},{"line_number":61,"context_line":"create_vfat_image:[1] and attached to the target node\u0027s Virtual Floppy."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_dd39eed6","line":61,"in_reply_to":"bf659307_ac0ac622","updated":"2018-04-24 13:34:07.000000000","message":"Thank you. We\u0027ll need to make sure the JSON does not contain spaces, else the parser will not read it correctly. Or find some way to escape them.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b13896da85260168a68e4ea715f0768b33288e1f","unresolved":false,"context_lines":[{"line_number":69,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"},{"line_number":70,"context_line":"   accordence to /etc/os-net-config/config.json, before initiating REST APIs"},{"line_number":71,"context_line":"   towards ironic conductor."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_9578852b","line":72,"updated":"2018-04-10 16:35:20.000000000","message":"How will a deployer configure a node to boot this way? I don\u0027t see any way for Ironic to discover that a node can (or should) boot this way.\n\nWhat changes are necessary to cause Ironic to not configure DHCP for an instance booting this way?\n\nMaybe this could/should be a separate boot interface, I guess? Though, if we go that way we\u0027d need both an IloVirtualMediaWithoutDHCPBoot and IRMCVirtualMediaWithoutDHCPBoot, which seems odd to me.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"8f77f599c67f3b97b2c89cb6099c6f06bfcb6821","unresolved":false,"context_lines":[{"line_number":69,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"},{"line_number":70,"context_line":"   accordence to /etc/os-net-config/config.json, before initiating REST APIs"},{"line_number":71,"context_line":"   towards ironic conductor."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_e2dd2541","line":72,"in_reply_to":"bf659307_9578852b","updated":"2018-04-13 16:02:51.000000000","message":"Done.\n\nAdded couple of lines stating \"All vendors supporting Virtual media connection and booting needs to create a \nnew virtmedia boot_interface\"","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"b13896da85260168a68e4ea715f0768b33288e1f","unresolved":false,"context_lines":[{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"None"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Data model impact"},{"line_number":79,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_555e6d12","line":76,"updated":"2018-04-10 16:35:20.000000000","message":"Are there alternatives to os-net-config? cloud-init? glean? others?","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"fce640900170d27da90763a8dd6e9523cf06a44b","unresolved":false,"context_lines":[{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"None"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Data model impact"},{"line_number":79,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_406f202a","line":76,"in_reply_to":"9f6a8fd7_62f60366","updated":"2018-04-25 11:56:14.000000000","message":"Sorry, my bad. I wanted to refer to Tripleo\u0027s os-collect-config and family using os-net-config for initial network configurations.\n\nAnyways now I added glean as an alternative to os-net-config.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"8f77f599c67f3b97b2c89cb6099c6f06bfcb6821","unresolved":false,"context_lines":[{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"None"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Data model impact"},{"line_number":79,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf659307_62703541","line":76,"in_reply_to":"bf659307_555e6d12","updated":"2018-04-13 16:02:51.000000000","message":"os-net-config is light weight compared to cloud-init and glean. Cloud-init internally uses os-net-config to perform network configurations.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"9a3c4c571cbd7915962f9560255e850fa6d0d3b4","unresolved":false,"context_lines":[{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"None"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Data model impact"},{"line_number":79,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9f6a8fd7_62f60366","line":76,"in_reply_to":"bf659307_62703541","updated":"2018-04-24 13:21:23.000000000","message":"I don\u0027t see os-net-config in the cloud-init source. Can you point me to where it uses it?\n\nAlso, glean is incredibly lightweight :)\n\nI still think it\u0027s worth mentioning these in this section.","commit_id":"6e19e221fac3a4025572cb83921b5603cedb736e"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4d2a40d5cc4c7827e00ba44d62cd66aa23b7b3ea","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Ironic L3 based deployment"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://bugs.launchpad.net/ironic/+bug/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration and (optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_b0254e0b","line":11,"updated":"2018-05-14 12:50:18.000000000","message":"This should be updated to be the URL into storyboard at this point.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"7d5b7afad9857cfb58d4557dc48515733a49eca9","unresolved":false,"context_lines":[{"line_number":68,"context_line":"\"192.168.1.21/24\"}], \"device\": \"ens3f0\", \"use_dhcp\": false, \"type\": \"vlan\","},{"line_number":69,"context_line":"\"vlan_id\": 20}, {\"use_dhcp\": false, \"type\": \"interface\", \"name\": \"ens3f0\"}]}"},{"line_number":70,"context_line":"Note:- This JSON structure should be validated by ironic-conductor to avoid"},{"line_number":71,"context_line":"parser erros on ironic-python-agent side."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_f13b3ace","line":71,"updated":"2018-05-08 12:56:29.000000000","message":"nit: s/erros/errors/","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"ac9d4a64533d4c31e90c40e728034af015992923","unresolved":false,"context_lines":[{"line_number":68,"context_line":"\"192.168.1.21/24\"}], \"device\": \"ens3f0\", \"use_dhcp\": false, \"type\": \"vlan\","},{"line_number":69,"context_line":"\"vlan_id\": 20}, {\"use_dhcp\": false, \"type\": \"interface\", \"name\": \"ens3f0\"}]}"},{"line_number":70,"context_line":"Note:- This JSON structure should be validated by ironic-conductor to avoid"},{"line_number":71,"context_line":"parser erros on ironic-python-agent side."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_30313e5b","line":71,"in_reply_to":"5f7c97a3_f13b3ace","updated":"2018-05-14 19:24:16.000000000","message":"Done","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"7d5b7afad9857cfb58d4557dc48515733a49eca9","unresolved":false,"context_lines":[{"line_number":73,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"1. Adding logic in IPA to read the virtual media parameters data. Look for"},{"line_number":76,"context_line":"   \"network_info\" JSON structure. If found create a config file"},{"line_number":77,"context_line":"   /etc/os-net-config/config.json."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_312212a6","line":76,"updated":"2018-05-08 12:56:29.000000000","message":"Maybe call this ironic_network_info or openstack_network_info? I\u0027m worried about a future kernel adding a network_info parameter.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"ac9d4a64533d4c31e90c40e728034af015992923","unresolved":false,"context_lines":[{"line_number":73,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"1. Adding logic in IPA to read the virtual media parameters data. Look for"},{"line_number":76,"context_line":"   \"network_info\" JSON structure. If found create a config file"},{"line_number":77,"context_line":"   /etc/os-net-config/config.json."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_902a92d8","line":76,"in_reply_to":"5f7c97a3_312212a6","updated":"2018-05-14 19:24:16.000000000","message":"Done","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4d2a40d5cc4c7827e00ba44d62cd66aa23b7b3ea","unresolved":false,"context_lines":[{"line_number":73,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"1. Adding logic in IPA to read the virtual media parameters data. Look for"},{"line_number":76,"context_line":"   \"network_info\" JSON structure. If found create a config file"},{"line_number":77,"context_line":"   /etc/os-net-config/config.json."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"2. os-net-config tool needs to be added to configure realtime network in"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_70d496e9","line":76,"in_reply_to":"5f7c97a3_312212a6","updated":"2018-05-14 12:50:18.000000000","message":"I\u0027m not sure it really matters since IPA opens the file after the kernel has deployed in vmedia scenarios.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"7d5b7afad9857cfb58d4557dc48515733a49eca9","unresolved":false,"context_lines":[{"line_number":83,"context_line":"Alternatives"},{"line_number":84,"context_line":"------------"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_714dea70","line":86,"updated":"2018-05-08 12:56:29.000000000","message":"I\u0027d love to see us use glean instead of os-net-config, if it has the features we need. But I will wait to see what others think.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4d2a40d5cc4c7827e00ba44d62cd66aa23b7b3ea","unresolved":false,"context_lines":[{"line_number":83,"context_line":"Alternatives"},{"line_number":84,"context_line":"------------"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_f001866b","line":86,"in_reply_to":"5f7c97a3_714dea70","updated":"2018-05-14 12:50:18.000000000","message":"+1, but glean is one of the hardest things to get patches landed into because it is a low priority infra item. Glean would surely be lower weight/cost though.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"7d5b7afad9857cfb58d4557dc48515733a49eca9","unresolved":false,"context_lines":[{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"None"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"State Machine Impact"},{"line_number":94,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_0e09f12f","line":91,"updated":"2018-05-08 12:56:29.000000000","message":"Need to add a bit about adding the properties field to the port data model.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"ac9d4a64533d4c31e90c40e728034af015992923","unresolved":false,"context_lines":[{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"None"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"State Machine Impact"},{"line_number":94,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_0f1094a7","line":91,"in_reply_to":"5f7c97a3_0e09f12f","updated":"2018-05-14 19:24:16.000000000","message":"Done","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4d2a40d5cc4c7827e00ba44d62cd66aa23b7b3ea","unresolved":false,"context_lines":[{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"None"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"State Machine Impact"},{"line_number":94,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_704256ad","line":91,"in_reply_to":"5f7c97a3_0e09f12f","updated":"2018-05-14 12:50:18.000000000","message":"Great catch, properties would be needed on the individual port level which it presently is not there. It is there for portgroups though.","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"ac9d4a64533d4c31e90c40e728034af015992923","unresolved":false,"context_lines":[{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"None"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"State Machine Impact"},{"line_number":94,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f7c97a3_af2ac0d7","line":91,"in_reply_to":"5f7c97a3_704256ad","updated":"2018-05-14 19:24:16.000000000","message":"Done","commit_id":"d782a9cfc23bd1d3375144175e76c800ade09b3a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"00855d953afc236a0ceb9f9274c65235c8b2ff9b","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Ironic L3 based deployment"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://bugs.launchpad.net/ironic/+bug/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration and (optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"}],"source_content_type":"text/x-rst","patch_set":8,"id":"5f7c97a3_5e64d249","line":11,"updated":"2018-05-16 14:29:37.000000000","message":"nit: We should change this out to the storyboard URL, easy quick fix after approval is fine with me.","commit_id":"f510a7ab9fb3289755ac455181e6b5dcc39b5dc5"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"1ff186da2264a1fc63110b62febb2c63e81faf69","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Ironic L3 based deployment"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://bugs.launchpad.net/ironic/+bug/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration and (optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"}],"source_content_type":"text/x-rst","patch_set":8,"id":"5f7c97a3_4158732f","line":11,"in_reply_to":"5f7c97a3_5e64d249","updated":"2018-05-16 14:40:21.000000000","message":"Done","commit_id":"f510a7ab9fb3289755ac455181e6b5dcc39b5dc5"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"4c8fbb143ebc485b2560dabac578496fcb933777","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"5f7c97a3_a866f628","updated":"2018-05-17 10:24:46.000000000","message":"Digression: On initial look this looks like a usecase that would fit with the changes I am proposing in Heat. (If Heat is used to deploy the instances.) I imagined getting the network config data onto a node by putting it in config_drive and thus remove the need for DHCP on the network.","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"4c8fbb143ebc485b2560dabac578496fcb933777","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  relay DHCP requests."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"* Remote site targets: It would be a great idea to deploy Servers in different"},{"line_number":37,"context_line":"  remote sites from a cetral location. With the current setup of L2 dependency"},{"line_number":38,"context_line":"  we need to provide VPN L2 tunnel between Ironic Conductor and remote target"},{"line_number":39,"context_line":"  server."},{"line_number":40,"context_line":"  example: Edge cloud servers or Distributed VNF Flexi Zone Controllers."}],"source_content_type":"text/x-rst","patch_set":9,"id":"5f7c97a3_08028a3f","line":37,"range":{"start_line":37,"start_character":22,"end_line":37,"end_character":28},"updated":"2018-05-17 10:24:46.000000000","message":"nit: typo","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"b37afd791f3fc8431f72d5dc663cca4cba10bb8d","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  relay DHCP requests."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"* Remote site targets: It would be a great idea to deploy Servers in different"},{"line_number":37,"context_line":"  remote sites from a cetral location. With the current setup of L2 dependency"},{"line_number":38,"context_line":"  we need to provide VPN L2 tunnel between Ironic Conductor and remote target"},{"line_number":39,"context_line":"  server."},{"line_number":40,"context_line":"  example: Edge cloud servers or Distributed VNF Flexi Zone Controllers."}],"source_content_type":"text/x-rst","patch_set":9,"id":"5f7c97a3_05f46c86","line":37,"range":{"start_line":37,"start_character":22,"end_line":37,"end_character":28},"in_reply_to":"5f7c97a3_08028a3f","updated":"2018-05-22 06:59:37.000000000","message":"Done","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"4c8fbb143ebc485b2560dabac578496fcb933777","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection and booting needs to create a"},{"line_number":52,"context_line":"new virtmedia boot_interface with IP config data of target nodes which can be"},{"line_number":53,"context_line":"derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":9,"id":"5f7c97a3_683e9edb","line":52,"range":{"start_line":52,"start_character":4,"end_line":52,"end_character":13},"updated":"2018-05-17 10:24:46.000000000","message":"Could we put config drive on virtual media?\n\nIf we could it would be possible to deploy instances on say libvirt compute node without having a dhcp agent on each segment of a network as well. Edge computing with L3 neutron routed provider networks?\n\nSo instead of having Ironic do all this writing network things to a drive, we just instruct the user to put the data into config_drive. (Config drive has has support all over nova, heat etc so potentially benefit more users?)\n\nIPA would mount config drive and configure the network.","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"b37afd791f3fc8431f72d5dc663cca4cba10bb8d","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection and booting needs to create a"},{"line_number":52,"context_line":"new virtmedia boot_interface with IP config data of target nodes which can be"},{"line_number":53,"context_line":"derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":9,"id":"5f7c97a3_c5ed14b8","line":52,"range":{"start_line":52,"start_character":4,"end_line":52,"end_character":13},"in_reply_to":"5f7c97a3_683e9edb","updated":"2018-05-22 06:59:37.000000000","message":"Dropped!\nAt this stage it is focused at configuring IPA so that it can deploy the image to the disk, Not configuring the instance.\n\nInstance configuration would anyway be done by configdrive written to the disk by IPA during provisioning.","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"4c8fbb143ebc485b2560dabac578496fcb933777","unresolved":false,"context_lines":[{"line_number":83,"context_line":"Alternatives"},{"line_number":84,"context_line":"------------"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"5f7c97a3_8303ddbe","line":86,"range":{"start_line":86,"start_character":0,"end_line":86,"end_character":55},"updated":"2018-05-17 10:24:46.000000000","message":"Could cloud-init do it?\nhttp://cloudinit.readthedocs.io/en/latest/topics/network-config-format-v1.html#subnet-ip\n\nThe Subnet Static Example?","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"b37afd791f3fc8431f72d5dc663cca4cba10bb8d","unresolved":false,"context_lines":[{"line_number":83,"context_line":"Alternatives"},{"line_number":84,"context_line":"------------"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Data model impact"},{"line_number":89,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"5f7c97a3_c542f4ab","line":86,"range":{"start_line":86,"start_character":0,"end_line":86,"end_character":55},"in_reply_to":"5f7c97a3_8303ddbe","updated":"2018-05-22 06:59:37.000000000","message":"Dropped!","commit_id":"7a1a0f4458535ee173517b9a63a728c7c06a6126"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://storyboard.openstack.org/#!/story/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration and (optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_8c6e4094","line":13,"range":{"start_line":13,"start_character":53,"end_line":13,"end_character":77},"updated":"2018-05-24 16:17:35.000000000","message":"guess this needs to go before and?","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://storyboard.openstack.org/#!/story/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration and (optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_714b732a","line":13,"range":{"start_line":13,"start_character":53,"end_line":13,"end_character":77},"in_reply_to":"5f7c97a3_8c6e4094","updated":"2018-05-25 11:15:10.000000000","message":"Done","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection and booting needs to create a"},{"line_number":52,"context_line":"new virtmedia boot_interface with IP config data of target nodes which can be"},{"line_number":53,"context_line":"derived as follows:"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_8cd5e0b9","line":51,"updated":"2018-05-24 16:17:35.000000000","message":"they also need to ensure that conductor can access node controllers over l3 with management/power/etc. interfaces right?","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection and booting needs to create a"},{"line_number":52,"context_line":"new virtmedia boot_interface with IP config data of target nodes which can be"},{"line_number":53,"context_line":"derived as follows:"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_b13d8b8f","line":51,"in_reply_to":"5f7c97a3_8cd5e0b9","updated":"2018-05-25 11:15:10.000000000","message":"Done","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection and booting needs to create a"},{"line_number":52,"context_line":"new virtmedia boot_interface with IP config data of target nodes which can be"},{"line_number":53,"context_line":"derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_4c4348b0","line":52,"range":{"start_line":52,"start_character":29,"end_line":52,"end_character":33},"updated":"2018-05-24 16:17:35.000000000","message":"what do you mean here by \u0027boot interface with data\u0027? I\u0027d say create a virtual media boot interface that will use the IP config data...","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection and booting needs to create a"},{"line_number":52,"context_line":"new virtmedia boot_interface with IP config data of target nodes which can be"},{"line_number":53,"context_line":"derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_113d5f8e","line":52,"range":{"start_line":52,"start_character":29,"end_line":52,"end_character":33},"in_reply_to":"5f7c97a3_4c4348b0","updated":"2018-05-25 11:15:10.000000000","message":"Done","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":59,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_0c1790a5","line":59,"range":{"start_line":59,"start_character":26,"end_line":59,"end_character":28},"updated":"2018-05-24 16:17:35.000000000","message":"not sure what do these arrows mean. that data will be collected from all of these?","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":59,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_3a16354b","line":59,"range":{"start_line":59,"start_character":26,"end_line":59,"end_character":28},"in_reply_to":"5f7c97a3_0c1790a5","updated":"2018-05-25 11:15:10.000000000","message":"It means that we have to derive this data using this hierarchy of datastructures. This data is mainly collected from neutron subnet (IP info) and neutron network (Vlan segment if any).","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":109,"context_line":"REST API impact"},{"line_number":110,"context_line":"---------------"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"None"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"Client (CLI) impact"},{"line_number":115,"context_line":"-------------------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_4c71080c","line":112,"updated":"2018-05-24 16:17:35.000000000","message":"need to add, as new field should be exposed in api","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":109,"context_line":"REST API impact"},{"line_number":110,"context_line":"---------------"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"None"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"Client (CLI) impact"},{"line_number":115,"context_line":"-------------------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_d1cda7b7","line":112,"in_reply_to":"5f7c97a3_4c71080c","updated":"2018-05-25 11:15:10.000000000","message":"Done","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Client (CLI) impact"},{"line_number":115,"context_line":"-------------------"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"None"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"\"ironic\" CLI"},{"line_number":120,"context_line":"~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_ec61f43a","line":117,"updated":"2018-05-24 16:17:35.000000000","message":"ditto","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Client (CLI) impact"},{"line_number":115,"context_line":"-------------------"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"None"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"\"ironic\" CLI"},{"line_number":120,"context_line":"~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_91c3af89","line":117,"in_reply_to":"5f7c97a3_ec61f43a","updated":"2018-05-25 11:15:10.000000000","message":"Done","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9093396bc84f2a457e51909ebf939f616feb75f4","unresolved":false,"context_lines":[{"line_number":124,"context_line":"\"openstack baremetal\" CLI"},{"line_number":125,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"None"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"RPC API impact"},{"line_number":130,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_ac5bfc84","line":127,"updated":"2018-05-24 16:17:35.000000000","message":"ditto","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"4e9d2c5554edac1ffeee38ce9bf96504f797cc72","unresolved":false,"context_lines":[{"line_number":124,"context_line":"\"openstack baremetal\" CLI"},{"line_number":125,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"None"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"RPC API impact"},{"line_number":130,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5f7c97a3_31bc1b05","line":127,"in_reply_to":"5f7c97a3_ac5bfc84","updated":"2018-05-25 11:15:10.000000000","message":"Done","commit_id":"d537336444ff82314e5676654420e4171122e1c0"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":29,"context_line":"Example use cases:"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Multi Rack deployment: Inter-Rack L2 switching beyond ToR is a challenge with"},{"line_number":32,"context_line":"  DHCP-relaying, as this adds requirement of either having ToR with"},{"line_number":33,"context_line":"  DHCP-relaying, or having a system or VM listening on the remote subnet to"},{"line_number":34,"context_line":"  relay DHCP requests."},{"line_number":35,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_65923ba2","line":32,"range":{"start_line":32,"start_character":2,"end_line":32,"end_character":15},"updated":"2018-06-05 02:00:09.000000000","message":"Nit: I would change this to just DHCP-relay in both places.","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":29,"context_line":"Example use cases:"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Multi Rack deployment: Inter-Rack L2 switching beyond ToR is a challenge with"},{"line_number":32,"context_line":"  DHCP-relaying, as this adds requirement of either having ToR with"},{"line_number":33,"context_line":"  DHCP-relaying, or having a system or VM listening on the remote subnet to"},{"line_number":34,"context_line":"  relay DHCP requests."},{"line_number":35,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_998fbf2d","line":32,"range":{"start_line":32,"start_character":2,"end_line":32,"end_character":15},"in_reply_to":"5f7c97a3_65923ba2","updated":"2018-06-05 13:31:12.000000000","message":"Done","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"* Multi Rack deployment: Inter-Rack L2 switching beyond ToR is a challenge with"},{"line_number":32,"context_line":"  DHCP-relaying, as this adds requirement of either having ToR with"},{"line_number":33,"context_line":"  DHCP-relaying, or having a system or VM listening on the remote subnet to"},{"line_number":34,"context_line":"  relay DHCP requests."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"* Remote site targets: It would be a great idea to deploy Servers in different"},{"line_number":37,"context_line":"  remote sites from a central location. With the current setup of L2 dependency"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_45da9f79","line":34,"range":{"start_line":34,"start_character":2,"end_line":34,"end_character":12},"updated":"2018-06-05 02:00:09.000000000","message":"Nit: Change to \"relay or proxy DHCP requests\"","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":31,"context_line":"* Multi Rack deployment: Inter-Rack L2 switching beyond ToR is a challenge with"},{"line_number":32,"context_line":"  DHCP-relaying, as this adds requirement of either having ToR with"},{"line_number":33,"context_line":"  DHCP-relaying, or having a system or VM listening on the remote subnet to"},{"line_number":34,"context_line":"  relay DHCP requests."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"* Remote site targets: It would be a great idea to deploy Servers in different"},{"line_number":37,"context_line":"  remote sites from a central location. With the current setup of L2 dependency"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_399c6b69","line":34,"range":{"start_line":34,"start_character":2,"end_line":34,"end_character":12},"in_reply_to":"5f7c97a3_45da9f79","updated":"2018-06-05 13:31:12.000000000","message":"Done","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":43,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. As on date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_c5cf8fb1","line":46,"range":{"start_line":46,"start_character":13,"end_line":46,"end_character":23},"updated":"2018-06-05 02:00:09.000000000","message":"Change from \"As on date\" to \"To date\".","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":43,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. As on date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_59992777","line":46,"range":{"start_line":46,"start_character":13,"end_line":46,"end_character":23},"in_reply_to":"5f7c97a3_c5cf8fb1","updated":"2018-06-05 13:31:12.000000000","message":"Done","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"eg:-"},{"line_number":68,"context_line":"openstack_network_info\u003d{\"network_config\": [{\"addresses\": [{\"ip_netmask\":"},{"line_number":69,"context_line":"\"192.168.1.21/24\"}], \"device\": \"ens3f0\", \"use_dhcp\": false, \"type\": \"vlan\","},{"line_number":70,"context_line":"\"vlan_id\": 20}, {\"use_dhcp\": false, \"type\": \"interface\", \"name\": \"ens3f0\"}]}"},{"line_number":71,"context_line":"Note:- This JSON structure should be validated by ironic-conductor to avoid"},{"line_number":72,"context_line":"parser errors on ironic-python-agent side."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_a5113307","line":70,"range":{"start_line":68,"start_character":0,"end_line":70,"end_character":76},"updated":"2018-06-05 02:00:09.000000000","message":"We can also pass MTU (derived from Neutron network).  Eventually perhaps also bonding configuration for port-groups. It might be useful to pass NIC aliases that are understood by os-net-config (\"nic1\").","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"eg:-"},{"line_number":68,"context_line":"openstack_network_info\u003d{\"network_config\": [{\"addresses\": [{\"ip_netmask\":"},{"line_number":69,"context_line":"\"192.168.1.21/24\"}], \"device\": \"ens3f0\", \"use_dhcp\": false, \"type\": \"vlan\","},{"line_number":70,"context_line":"\"vlan_id\": 20}, {\"use_dhcp\": false, \"type\": \"interface\", \"name\": \"ens3f0\"}]}"},{"line_number":71,"context_line":"Note:- This JSON structure should be validated by ironic-conductor to avoid"},{"line_number":72,"context_line":"parser errors on ironic-python-agent side."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_9c39ed40","line":70,"range":{"start_line":68,"start_character":0,"end_line":70,"end_character":76},"in_reply_to":"5f7c97a3_a5113307","updated":"2018-06-05 13:31:12.000000000","message":"Done.\n\nBridge support is not needed at this stage.\nAlso ovs-services will not be running at this point of time.\n\nOnly linux bonds shall be supported.","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":90,"context_line":"   \"openstack_network_info\" JSON structure. If found create a config file"},{"line_number":91,"context_line":"   /etc/os-net-config/config.json."},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"3. os-net-config tool needs to be added to configure realtime network in"},{"line_number":94,"context_line":"   accordence to /etc/os-net-config/config.json, before initiating REST APIs"},{"line_number":95,"context_line":"   towards ironic conductor."},{"line_number":96,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_251683f3","line":93,"range":{"start_line":93,"start_character":3,"end_line":93,"end_character":39},"updated":"2018-06-05 02:00:09.000000000","message":"os-net-config also has a list of required python modules, so these will need to be included as well. The current requirements.txt lists: pbr, anyjson, six, eventlet, iso8601, netaddr, PyYAML, jsonschema, oslo.concurrency, and oslo.utils.","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":90,"context_line":"   \"openstack_network_info\" JSON structure. If found create a config file"},{"line_number":91,"context_line":"   /etc/os-net-config/config.json."},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"3. os-net-config tool needs to be added to configure realtime network in"},{"line_number":94,"context_line":"   accordence to /etc/os-net-config/config.json, before initiating REST APIs"},{"line_number":95,"context_line":"   towards ironic conductor."},{"line_number":96,"context_line":""}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_39746bfe","line":93,"range":{"start_line":93,"start_character":3,"end_line":93,"end_character":39},"in_reply_to":"5f7c97a3_251683f3","updated":"2018-06-05 13:31:12.000000000","message":"Done","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"  New request/response field:"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"  * ``properties`` Key/value properties related to the port\u0027s configuration."},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"Client (CLI) impact"},{"line_number":135,"context_line":"-------------------"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_8532f75d","line":132,"range":{"start_line":132,"start_character":2,"end_line":132,"end_character":76},"updated":"2018-06-05 02:00:09.000000000","message":"Will this be flexible enough to support bonds and bridges in addition to interfaces? I wonder if we should pass one JSON with the entire config?","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"  New request/response field:"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"  * ``properties`` Key/value properties related to the port\u0027s configuration."},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"Client (CLI) impact"},{"line_number":135,"context_line":"-------------------"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_62cd4385","line":132,"range":{"start_line":132,"start_character":2,"end_line":132,"end_character":76},"in_reply_to":"5f7c97a3_8532f75d","updated":"2018-06-05 13:31:12.000000000","message":"Yes, We need to pass an entire JSON with the Key \"openstack_network_info\" and value \"\u003cos-net-config network config json\u003e\".\n\nBond should be supported but not the Bridge.","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"00764ca9ed01c5e0d4aa3f1c68f1ad1a313db47c","unresolved":false,"context_lines":[{"line_number":137,"context_line":"\"ironic\" CLI"},{"line_number":138,"context_line":"~~~~~~~~~~~~"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Update ``ironic port-create`` and ``ironic port-update`` commands to accept"},{"line_number":141,"context_line":"  one more argument ``--property \u003ckey\u003dvalue\u003e``, with help text Key/value"},{"line_number":142,"context_line":"  property related to port\u0027s configuration. Can be specified multiple times."},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"* Extend the output of the ``ironic port-show`` command with the ``properties``"},{"line_number":145,"context_line":"  column."}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_05d8271c","line":142,"range":{"start_line":140,"start_character":0,"end_line":142,"end_character":76},"updated":"2018-06-05 02:00:09.000000000","message":"I\u0027m not sure this is a good idea. What if we want to pass both an interface and a bridge config, with the interface being a member of the bridge? Perhaps instead of the property being called multiple times, we should include all ports defined in one JSON? This would be the equivalent of the \"network_config\" in os-net-config. The os-net-config binary currently uses a JSON with { \"network_config\": \u003cconfig_json\u003e }, and may be extended to support other top-level objects such as \"route_table\" in addition to \"network_config\". This would allow one pass of os-net-config, instead of calling it multiple times or having to reassemble a JSON with all configs before calling os-net-config.","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"94e1c6de789e7e4edca9235d02b14f145af30ad8","unresolved":false,"context_lines":[{"line_number":137,"context_line":"\"ironic\" CLI"},{"line_number":138,"context_line":"~~~~~~~~~~~~"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Update ``ironic port-create`` and ``ironic port-update`` commands to accept"},{"line_number":141,"context_line":"  one more argument ``--property \u003ckey\u003dvalue\u003e``, with help text Key/value"},{"line_number":142,"context_line":"  property related to port\u0027s configuration. Can be specified multiple times."},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"* Extend the output of the ``ironic port-show`` command with the ``properties``"},{"line_number":145,"context_line":"  column."}],"source_content_type":"text/x-rst","patch_set":12,"id":"5f7c97a3_a296fb3d","line":142,"range":{"start_line":140,"start_character":0,"end_line":142,"end_character":76},"in_reply_to":"5f7c97a3_05d8271c","updated":"2018-06-05 13:31:12.000000000","message":"Yes that is the idea, we will have single JSON structure for network configurations. But --property should allow multiple key-value pairs for any future properties to be added.","commit_id":"bb4f3f0a780fa51733fb401de05ac03030b3e89b"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_4c2243a5","line":16,"range":{"start_line":16,"start_character":54,"end_line":16,"end_character":63},"updated":"2018-06-11 11:20:53.000000000","message":"`hardware` does not have plural form [1]\n\n1. https://www.quora.com/What-is-the-plural-form-for-hardware","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_f6e8a7df","line":16,"range":{"start_line":16,"start_character":54,"end_line":16,"end_character":63},"in_reply_to":"5f7c97a3_4c2243a5","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_d3a7b4bb","line":16,"range":{"start_line":16,"start_character":54,"end_line":16,"end_character":63},"in_reply_to":"5f7c97a3_4c2243a5","updated":"2018-06-12 13:51:11.000000000","message":"Great catch, this is a good follow-up.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_27880b6f","line":16,"range":{"start_line":16,"start_character":54,"end_line":16,"end_character":63},"in_reply_to":"5f7c97a3_d3a7b4bb","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_6c8e1f28","line":17,"range":{"start_line":16,"start_character":64,"end_line":17,"end_character":18},"updated":"2018-06-11 11:20:53.000000000","message":"Could this spec be used with UEFI boot (e.g. fetching image over HTTP) as well? Is there a technical requirement why virtual media is explicitly linked to this spec?","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ac30b7ea","line":17,"range":{"start_line":17,"start_character":48,"end_line":17,"end_character":58},"updated":"2018-06-11 11:20:53.000000000","message":"rephrase perhaps: to run on purely routed L3","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_134d2c99","line":17,"range":{"start_line":16,"start_character":64,"end_line":17,"end_character":18},"in_reply_to":"5f7c97a3_6c8e1f28","updated":"2018-06-20 10:33:07.000000000","message":"Fetching image (IPA ISO) is specific to the vendor. For eg:- HP iLO supports fetching virtual boot image over HTTP. At the moment Nokia HW supports fetching Virtual media over NFS.\n\nBut once the target boots with this Virutal media image along with the IP configuration it can communicate with Ironic conductor over HTTP REST and fetch the rootfs instance image.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_7396e04b","line":17,"range":{"start_line":16,"start_character":64,"end_line":17,"end_character":18},"in_reply_to":"5f7c97a3_6c8e1f28","updated":"2018-06-12 13:51:11.000000000","message":"Not exactly. The virtual media boot is being used as an out of band mechanism to inform the physical machine of the information needed for networking as opposed to there being any DHCP operations for the machine being deployed.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_36913f53","line":17,"range":{"start_line":17,"start_character":48,"end_line":17,"end_character":58},"in_reply_to":"5f7c97a3_ac30b7ea","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ec3f8fb2","line":18,"range":{"start_line":18,"start_character":26,"end_line":18,"end_character":30},"updated":"2018-06-11 11:20:53.000000000","message":"...and PXE?","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardwares supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_d3957433","line":18,"range":{"start_line":18,"start_character":26,"end_line":18,"end_character":30},"in_reply_to":"5f7c97a3_ec3f8fb2","updated":"2018-06-12 13:51:11.000000000","message":"Given that this driver is scoped to support purely out of band communication, there would be no dhcp interactions. This will result in some bugs/issues needing to be fixed in any deployment that uses neutron, but those are things that can be worked through later. Otherwise, I think I\u0027m good with just leaving it at removing the DHCP dependency since PXE requires DHCP.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_f3b190e1","line":20,"range":{"start_line":20,"start_character":37,"end_line":20,"end_character":52},"updated":"2018-06-12 13:51:11.000000000","message":"I\u0027m assuming this is meant to enable a boot interface, and we should revise this afterwards.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":17,"context_line":"virtual media boot, which enables deployment to purely run on routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_70ad602f","line":20,"range":{"start_line":20,"start_character":37,"end_line":20,"end_character":52},"in_reply_to":"5f7c97a3_f3b190e1","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ecad4fea","line":21,"range":{"start_line":21,"start_character":39,"end_line":21,"end_character":43},"updated":"2018-06-11 11:20:53.000000000","message":"Is it pxe hardware boot interface?","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ec68efba","line":21,"range":{"start_line":21,"start_character":39,"end_line":21,"end_character":43},"updated":"2018-06-11 11:20:53.000000000","message":"nit: DHCP","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_f0665029","line":21,"range":{"start_line":21,"start_character":39,"end_line":21,"end_character":43},"in_reply_to":"5f7c97a3_ec68efba","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests enabling extra interface for deployment with"},{"line_number":21,"context_line":"backward compatibility to the existing dhcp based interface."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Problem description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_13076c84","line":21,"range":{"start_line":21,"start_character":39,"end_line":21,"end_character":43},"in_reply_to":"5f7c97a3_ecad4fea","updated":"2018-06-12 13:51:11.000000000","message":"I don\u0027t believe it is, but it would be a boot interface if I grok the desire correctly.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":43,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ac1c37c2","line":47,"range":{"start_line":46,"start_character":48,"end_line":47,"end_character":14},"updated":"2018-06-11 11:20:53.000000000","message":"May be it\u0027s just me, but I\u0027d appreciate some more details on the proposed workflow. The big picture, so to say. ;-)\n\nOne thing which is not obvious is that this spec is focusing on IPA boot rather than on instance boot, if I get it correctly.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ccdc13e3","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":14},"updated":"2018-06-11 11:20:53.000000000","message":"nit: probably should be singular","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":43,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_933ffc23","line":47,"range":{"start_line":46,"start_character":48,"end_line":47,"end_character":14},"in_reply_to":"5f7c97a3_ac1c37c2","updated":"2018-06-12 13:51:11.000000000","message":"In theory, we could support both... as long as we detach the virtual media later. I think irmc needs to be explicit, I think ilo times out as time goes on.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":43,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_b309d831","line":47,"range":{"start_line":46,"start_character":48,"end_line":47,"end_character":14},"in_reply_to":"5f7c97a3_ac1c37c2","updated":"2018-06-20 10:33:07.000000000","message":"Yes this spec pertains only on IPA booting. It suggests implementing a new \"boot_interface\" (per hardware, which support Virtual media).","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_107d0cb1","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":14},"in_reply_to":"5f7c97a3_b351d864","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_b351d864","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":14},"in_reply_to":"5f7c97a3_ccdc13e3","updated":"2018-06-12 13:51:11.000000000","message":"+1 to singular, good as a follow-up.\n\nI personally struggle with the use of the word \"bypassed\"... it feels like \"supported\" might be the better option.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Most of the Hardware offerings in the market support Virtual media connection"},{"line_number":46,"context_line":"and booting. To date even these servers rely on DHCP to get the IP"},{"line_number":47,"context_line":"configurations. This dependency on DHCP can be bypassed with below proposed"},{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ad93b5a2","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":14},"in_reply_to":"5f7c97a3_ccdc13e3","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power managment on"},{"line_number":52,"context_line":"L3, needs to create a new virtmedia boot_interface that will use IP config data"},{"line_number":53,"context_line":"of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_ec546f56","line":51,"range":{"start_line":51,"start_character":67,"end_line":51,"end_character":76},"updated":"2018-06-11 11:20:53.000000000","message":"typo: management","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power managment on"},{"line_number":52,"context_line":"L3, needs to create a new virtmedia boot_interface that will use IP config data"},{"line_number":53,"context_line":"of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_854b3385","line":51,"range":{"start_line":51,"start_character":67,"end_line":51,"end_character":76},"in_reply_to":"5f7c97a3_ec546f56","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power managment on"},{"line_number":52,"context_line":"L3, needs to create a new virtmedia boot_interface that will use IP config data"},{"line_number":53,"context_line":"of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_13968cf1","line":52,"range":{"start_line":52,"start_character":22,"end_line":52,"end_character":56},"updated":"2018-06-12 13:51:11.000000000","message":"I get that it makes sense to provide a flag, but it seems like we would want to standardize the naming, or maybe provide a global flat for vmedia based drivers to look at so we minimize the number of new interfaces.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power managment on"},{"line_number":52,"context_line":"L3, needs to create a new virtmedia boot_interface that will use IP config data"},{"line_number":53,"context_line":"of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_2ec8c9a5","line":52,"range":{"start_line":52,"start_character":22,"end_line":52,"end_character":56},"in_reply_to":"5f7c97a3_13968cf1","updated":"2018-06-20 10:33:07.000000000","message":"Are you indicating that instead of having a new boot_interface we have a Flag which decides weather to use IP configurations from Virtual Media or from DHCP. eg:- For HP IloVirtualMediaBoot and IloVirtualMediaIPBoot(new interface TBD).\n\nThis is an architecture decision. So it would be great if you propose. I will update the Spec accordingly.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a5c58bf570b1659fc165e749944e3f0935fda504","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power managment on"},{"line_number":52,"context_line":"L3, needs to create a new virtmedia boot_interface that will use IP config data"},{"line_number":53,"context_line":"of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_d0726fe3","line":52,"range":{"start_line":52,"start_character":22,"end_line":52,"end_character":56},"in_reply_to":"5f7c97a3_2ec8c9a5","updated":"2018-06-22 01:42:29.000000000","message":"Mentally, there is a conundrum there with however the IPA image operates. It may try and block on DHCP operation. I don\u0027t _think_ we need a separate boot interface besides a general virtual media interface and the ability to have IPA read optional IP address information out.\n\nPerhaps the vmedia driver could have a dhcp flag so that it does not explicitly fail? I don\u0027t know your use case well enough to say, but I think we ought to work to avoid multiple vmedia drivers for vendor specific drivers.\n\nI hope that answers your question.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power managment on"},{"line_number":52,"context_line":"L3, needs to create a new virtmedia boot_interface that will use IP config data"},{"line_number":53,"context_line":"of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_fcc7ab97","line":52,"range":{"start_line":52,"start_character":22,"end_line":52,"end_character":56},"in_reply_to":"5f7c97a3_d0726fe3","updated":"2018-06-29 14:06:23.000000000","message":"Done.\nUpdated the Spec, so that the existing virtmedia based drivers should have an extra flag to enable DHCP based IP config fetch, else it would follow this L3 based network config method.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":59,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_0cf52bcb","line":58,"range":{"start_line":58,"start_character":3,"end_line":58,"end_character":29},"updated":"2018-06-11 11:20:53.000000000","message":"Is this an established term for a non-standalone operation?","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":59,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_cd7ab175","line":58,"range":{"start_line":58,"start_character":3,"end_line":58,"end_character":29},"in_reply_to":"5f7c97a3_0cf52bcb","updated":"2018-06-20 10:33:07.000000000","message":"Please refer Line 56: https://review.openstack.org/#/c/543936/2/specs/approved/L3-based-deployment.rst","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":58,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":59,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This can be added to parameters.txt when"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_5356442c","line":61,"updated":"2018-06-12 13:51:11.000000000","message":"What would perform the caching operation? Would this entirely skip the concept of a provisioning network? I\u0027m guessing it would since the machine and network information would be statically set.\n\nPerhaps we need a \"persistent vif\" concept...","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":58,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":59,"context_line":"   node-port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This can be added to parameters.txt when"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_35773ea2","line":61,"in_reply_to":"5f7c97a3_5356442c","updated":"2018-06-20 10:33:07.000000000","message":"In Standalone mode we set the machine and network information statically. But in case of Integrated deployment, I guess we still need the provisioning network in order to derive information about the network/subnet and target IP. I guess the driver should perform the caching.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This can be added to parameters.txt when"},{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"eg:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_d337b4fd","line":64,"range":{"start_line":64,"start_character":39,"end_line":64,"end_character":42},"updated":"2018-06-12 13:51:11.000000000","message":"nit: s/can/would/","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This can be added to parameters.txt when"},{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"eg:-"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_c7959733","line":64,"range":{"start_line":64,"start_character":39,"end_line":64,"end_character":42},"in_reply_to":"5f7c97a3_d337b4fd","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"04f901086313800f171c5e60100c2cde280c4e38","unresolved":false,"context_lines":[{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"eg:-"},{"line_number":68,"context_line":"openstack_network_info\u003d{\"network_config\": [{\"addresses\": [{\"ip_netmask\":"},{"line_number":69,"context_line":"\"192.168.1.21/24\"}], \"device\": \"nic1\", \"use_dhcp\": false, \"type\": \"vlan\","},{"line_number":70,"context_line":"\"mtu\": 1500, \"vlan_id\": 20}, {\"use_dhcp\": false, \"type\": \"interface\", \"name\":"},{"line_number":71,"context_line":"\"nic1\", \"mtu\": 1500}], \"interface_mapping\": {\"nic1\": \"em3\"}}"},{"line_number":72,"context_line":"Note:- This JSON structure should be validated by ironic-conductor to avoid"},{"line_number":73,"context_line":"parser errors on ironic-python-agent side."},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_0cd9ab81","line":71,"range":{"start_line":68,"start_character":0,"end_line":71,"end_character":60},"updated":"2018-06-11 11:20:53.000000000","message":"Could you please indent this JSON for readability?","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"eg:-"},{"line_number":68,"context_line":"openstack_network_info\u003d{\"network_config\": [{\"addresses\": [{\"ip_netmask\":"},{"line_number":69,"context_line":"\"192.168.1.21/24\"}], \"device\": \"nic1\", \"use_dhcp\": false, \"type\": \"vlan\","},{"line_number":70,"context_line":"\"mtu\": 1500, \"vlan_id\": 20}, {\"use_dhcp\": false, \"type\": \"interface\", \"name\":"},{"line_number":71,"context_line":"\"nic1\", \"mtu\": 1500}], \"interface_mapping\": {\"nic1\": \"em3\"}}"},{"line_number":72,"context_line":"Note:- This JSON structure should be validated by ironic-conductor to avoid"},{"line_number":73,"context_line":"parser errors on ironic-python-agent side."},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_052403c5","line":71,"range":{"start_line":68,"start_character":0,"end_line":71,"end_character":60},"in_reply_to":"5f7c97a3_0cd9ab81","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":77,"context_line":"* Option-1: This vfat image can be attached to the target node\u0027s Virtual Floppy"},{"line_number":78,"context_line":"  if it is supported by the hardware."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* Option-2: Some hardware do not support attaching more than one Virtual"},{"line_number":81,"context_line":"  device. In such case the vfat image can be zipped and append to the end of"},{"line_number":82,"context_line":"  IPA boot ISO with an extra 64K block."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_76148e41","line":82,"range":{"start_line":80,"start_character":0,"end_line":82,"end_character":39},"updated":"2018-06-12 13:51:11.000000000","message":"I\u0027m a little concerned by this option.\n\nDo we know what hardware actually doesn\u0027t support a virtual floppy and virtual cd at the same time?\n\nIt feels like this would only be valid for use if ironic replaced the ramdisk in swift or hosted it via a local service. I guess the fact that we have varying forms of access from private library to just pass an http url doesn\u0027t help this. :\\","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":77,"context_line":"* Option-1: This vfat image can be attached to the target node\u0027s Virtual Floppy"},{"line_number":78,"context_line":"  if it is supported by the hardware."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* Option-2: Some hardware do not support attaching more than one Virtual"},{"line_number":81,"context_line":"  device. In such case the vfat image can be zipped and append to the end of"},{"line_number":82,"context_line":"  IPA boot ISO with an extra 64K block."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_f509c62a","line":82,"range":{"start_line":80,"start_character":0,"end_line":82,"end_character":39},"in_reply_to":"5f7c97a3_76148e41","updated":"2018-06-20 10:33:07.000000000","message":"At the moment atleast Nokia HW does not support attaching Virtual CD and Virtual Floppy at the same time.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a5c58bf570b1659fc165e749944e3f0935fda504","unresolved":false,"context_lines":[{"line_number":77,"context_line":"* Option-1: This vfat image can be attached to the target node\u0027s Virtual Floppy"},{"line_number":78,"context_line":"  if it is supported by the hardware."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* Option-2: Some hardware do not support attaching more than one Virtual"},{"line_number":81,"context_line":"  device. In such case the vfat image can be zipped and append to the end of"},{"line_number":82,"context_line":"  IPA boot ISO with an extra 64K block."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_50509f32","line":82,"range":{"start_line":80,"start_character":0,"end_line":82,"end_character":39},"in_reply_to":"5f7c97a3_f509c62a","updated":"2018-06-22 01:42:29.000000000","message":"Oh :( I\u0027m going to add this to the meeting agenda for discussion.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"\"ironic\" CLI"},{"line_number":139,"context_line":"~~~~~~~~~~~~"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"* Update ``ironic port-create`` and ``ironic port-update`` commands to accept"},{"line_number":142,"context_line":"  one more argument ``--property \u003ckey\u003dvalue\u003e``, with help text Key/value"},{"line_number":143,"context_line":"  property related to port\u0027s configuration. Can be specified multiple times."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* Extend the output of the ``ironic port-show`` command with the ``properties``"},{"line_number":146,"context_line":"  column."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"\"openstack baremetal\" CLI"},{"line_number":149,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_96afeae1","line":146,"range":{"start_line":140,"start_character":0,"end_line":146,"end_character":9},"updated":"2018-06-12 13:51:11.000000000","message":"At this point in time, I don\u0027t see changes to the original ironic cli landing since we are slated to remove it next cycle.\n\nWe should just put \"None\" or \"N/A\" here. I\u0027m otherwise +2 with the changes described as long as we follow-up and fix this section shortly afterwards.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"\"ironic\" CLI"},{"line_number":139,"context_line":"~~~~~~~~~~~~"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"* Update ``ironic port-create`` and ``ironic port-update`` commands to accept"},{"line_number":142,"context_line":"  one more argument ``--property \u003ckey\u003dvalue\u003e``, with help text Key/value"},{"line_number":143,"context_line":"  property related to port\u0027s configuration. Can be specified multiple times."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* Extend the output of the ``ironic port-show`` command with the ``properties``"},{"line_number":146,"context_line":"  column."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"\"openstack baremetal\" CLI"},{"line_number":149,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_07614fa9","line":146,"range":{"start_line":140,"start_character":0,"end_line":146,"end_character":9},"in_reply_to":"5f7c97a3_96afeae1","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":241,"context_line":"Testing"},{"line_number":242,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"Test the ironic deployment without enabling L2 switch connectivity between"},{"line_number":245,"context_line":"Ironic-conductor and Target nodes."},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"Upgrades and Backwards Compatibility"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_76dece6a","line":244,"updated":"2018-06-12 13:51:11.000000000","message":"We should likely detail that there would be no dhcp in this case.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":241,"context_line":"Testing"},{"line_number":242,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"Test the ironic deployment without enabling L2 switch connectivity between"},{"line_number":245,"context_line":"Ironic-conductor and Target nodes."},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"Upgrades and Backwards Compatibility"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_84047d4b","line":244,"in_reply_to":"5f7c97a3_76dece6a","updated":"2018-06-20 10:33:07.000000000","message":"Done","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"efae6638766196c2ad8a71780f4781fe5d31a8a9","unresolved":false,"context_lines":[{"line_number":242,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"Test the ironic deployment without enabling L2 switch connectivity between"},{"line_number":245,"context_line":"Ironic-conductor and Target nodes."},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":248,"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"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_b6c1c682","line":245,"updated":"2018-06-12 13:51:11.000000000","message":"I guess the question is how will we test this in upstream CI without dhcp. I guess we might be able to add support to something like our redfish or ipmi bmc emulators.  Or would we see this with a nokia specific management/power/boot interfaces and third-party CI?","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"6b34d8ae7e792445e4fcde4724bcd1f60a6c2270","unresolved":false,"context_lines":[{"line_number":242,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"Test the ironic deployment without enabling L2 switch connectivity between"},{"line_number":245,"context_line":"Ironic-conductor and Target nodes."},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":248,"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"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_c43d958e","line":245,"in_reply_to":"5f7c97a3_b6c1c682","updated":"2018-06-20 10:33:07.000000000","message":"This should be tested on 3rd Party CI with Nokia or any other HW vendor specific. We can also make an implementation for Virtual HW (ssh based hw_types) for testing.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a5c58bf570b1659fc165e749944e3f0935fda504","unresolved":false,"context_lines":[{"line_number":242,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"Test the ironic deployment without enabling L2 switch connectivity between"},{"line_number":245,"context_line":"Ironic-conductor and Target nodes."},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":248,"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"}],"source_content_type":"text/x-rst","patch_set":13,"id":"5f7c97a3_907877c1","line":245,"in_reply_to":"5f7c97a3_c43d958e","updated":"2018-06-22 01:42:29.000000000","message":"So it might be good to circle back with the redfish contirbutors, but we can do that in one of the irc meetings.","commit_id":"e8664056117a92221fe9487087256eb2faae2801"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1d32844ae4184c3ddee82465c212485a36d1ccdd","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power management"},{"line_number":52,"context_line":"on L3, needs to create a new virtmedia boot_interface that will use IP config"},{"line_number":53,"context_line":"data of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_19b1c6e0","line":53,"range":{"start_line":51,"start_character":0,"end_line":53,"end_character":53},"updated":"2018-06-25 15:25:56.000000000","message":"I think they already have, things like ilo-virtual-media, irmc-virtual-media, etc.. or do you mean another set of virtual media boot interfaces? this is going to be quite confusing, and I\u0027m not sure why we cannot put these changes to existing interfaces.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"97924320186f46cd2a731b9da774413bbcd0c730","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power management"},{"line_number":52,"context_line":"on L3, needs to create a new virtmedia boot_interface that will use IP config"},{"line_number":53,"context_line":"data of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_7855424f","line":53,"range":{"start_line":51,"start_character":0,"end_line":53,"end_character":53},"in_reply_to":"5f7c97a3_19b1c6e0","updated":"2018-06-26 09:36:28.000000000","message":"+1","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":48,"context_line":"changes."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"On the Ironic Conductor side:-"},{"line_number":51,"context_line":"All vendors supporting Virtual media connection, booting and power management"},{"line_number":52,"context_line":"on L3, needs to create a new virtmedia boot_interface that will use IP config"},{"line_number":53,"context_line":"data of target nodes which can be derived as follows:"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"1. Stand-alone mode:- IP network information can be fed to ironic node-port or"},{"line_number":56,"context_line":"   portgroup\u0027s property field \"properties/openstack_network_info\"."}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_4d914ff7","line":53,"range":{"start_line":51,"start_character":0,"end_line":53,"end_character":53},"in_reply_to":"5f7c97a3_19b1c6e0","updated":"2018-06-29 14:06:23.000000000","message":"Done!","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1d32844ae4184c3ddee82465c212485a36d1ccdd","unresolved":false,"context_lines":[{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This would be added to parameters.txt when"},{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_b9cbf250","line":63,"range":{"start_line":63,"start_character":7,"end_line":63,"end_character":31},"updated":"2018-06-25 15:25:56.000000000","message":"hardware types? I think you meant boot interfaces?","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":60,"context_line":"   This info can be cached in ironic node-port or portgroup\u0027s property field"},{"line_number":61,"context_line":"   \"properties/openstack_network_info\"."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This would be added to parameters.txt when"},{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_fa5e2111","line":63,"range":{"start_line":63,"start_character":7,"end_line":63,"end_character":31},"in_reply_to":"5f7c97a3_b9cbf250","updated":"2018-06-29 14:06:23.000000000","message":"Done","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"97924320186f46cd2a731b9da774413bbcd0c730","unresolved":false,"context_lines":[{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This would be added to parameters.txt when"},{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Example:- Content for *openstack_network_info*"},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_38558a1e","line":65,"updated":"2018-06-26 09:36:28.000000000","message":"Are these kernel parameters? If so, is there a length limit?","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Ironic virtmedia hardware types can use this derived information to construct"},{"line_number":64,"context_line":"os-net-config JSON structure:[2]. This would be added to parameters.txt when"},{"line_number":65,"context_line":"running create_vfat_image:[1]."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Example:- Content for *openstack_network_info*"},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_7a6bf1b7","line":65,"in_reply_to":"5f7c97a3_38558a1e","updated":"2018-06-29 14:06:23.000000000","message":"No. These are not kernel parameters.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"97924320186f46cd2a731b9da774413bbcd0c730","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"   {"},{"line_number":72,"context_line":"       \"interface_mapping\": {"},{"line_number":73,"context_line":"           \"nic1\": \"em3\""},{"line_number":74,"context_line":"       },"},{"line_number":75,"context_line":"       \"network_config\": ["},{"line_number":76,"context_line":"           {"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_f8bdd267","line":73,"updated":"2018-06-26 09:36:28.000000000","message":"How will we know the names of the interfaces?\n\nThe network_data.json format used by OpenStack metadata supports referencing interfaces by their MAC address rather than their name, which maps better to the information we know in ironic.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"   {"},{"line_number":72,"context_line":"       \"interface_mapping\": {"},{"line_number":73,"context_line":"           \"nic1\": \"em3\""},{"line_number":74,"context_line":"       },"},{"line_number":75,"context_line":"       \"network_config\": ["},{"line_number":76,"context_line":"           {"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_9a55adfb","line":73,"in_reply_to":"5f7c97a3_1118a8ca","updated":"2018-06-29 14:06:23.000000000","message":"Done","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"   {"},{"line_number":72,"context_line":"       \"interface_mapping\": {"},{"line_number":73,"context_line":"           \"nic1\": \"em3\""},{"line_number":74,"context_line":"       },"},{"line_number":75,"context_line":"       \"network_config\": ["},{"line_number":76,"context_line":"           {"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_57acd033","line":73,"in_reply_to":"5f7c97a3_9126f866","updated":"2018-06-29 14:06:23.000000000","message":"Done","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"67ab9dee0f0a84cd3922f3ac4edde31411fa9f70","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"   {"},{"line_number":72,"context_line":"       \"interface_mapping\": {"},{"line_number":73,"context_line":"           \"nic1\": \"em3\""},{"line_number":74,"context_line":"       },"},{"line_number":75,"context_line":"       \"network_config\": ["},{"line_number":76,"context_line":"           {"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_1118a8ca","line":73,"in_reply_to":"5f7c97a3_9126f866","updated":"2018-06-26 12:21:50.000000000","message":"Thanks Harald, I\u0027ll remove my -1.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"   {"},{"line_number":72,"context_line":"       \"interface_mapping\": {"},{"line_number":73,"context_line":"           \"nic1\": \"em3\""},{"line_number":74,"context_line":"       },"},{"line_number":75,"context_line":"       \"network_config\": ["},{"line_number":76,"context_line":"           {"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_3a401935","line":73,"in_reply_to":"5f7c97a3_f8bdd267","updated":"2018-06-29 14:06:23.000000000","message":"Done","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"5aa8e336d92d8cb26970ee4ad2ca4096fb67958f","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"   {"},{"line_number":72,"context_line":"       \"interface_mapping\": {"},{"line_number":73,"context_line":"           \"nic1\": \"em3\""},{"line_number":74,"context_line":"       },"},{"line_number":75,"context_line":"       \"network_config\": ["},{"line_number":76,"context_line":"           {"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_9126f866","line":73,"in_reply_to":"5f7c97a3_f8bdd267","updated":"2018-06-26 12:10:48.000000000","message":"os-net-config supports mac address mapping as well.\n\nie we can do: \n\u0027interface_mapping\u0027: {\n  \u0027nic1\u0027: \u0027aa:bb:cc:d0:e0:f0\u0027,\n  \u0027nic2\u0027: \u0027aa:bb:cc:d0:e0:f1\u0027,\n}\n\nref: https://github.com/openstack/os-net-config/blob/master/etc/os-net-config/samples/mapping.yaml","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"97924320186f46cd2a731b9da774413bbcd0c730","unresolved":false,"context_lines":[{"line_number":83,"context_line":"               \"mtu\": 1500,"},{"line_number":84,"context_line":"               \"type\": \"vlan\","},{"line_number":85,"context_line":"               \"use_dhcp\": false,"},{"line_number":86,"context_line":"               \"vlan_id\": 20"},{"line_number":87,"context_line":"           },"},{"line_number":88,"context_line":"           {"},{"line_number":89,"context_line":"               \"mtu\": 1500,"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_78da22e9","line":86,"updated":"2018-06-26 09:36:28.000000000","message":"This example suggests that tagged VLANs might be used. Is that the case, or would we assume nodes are connected to access mode switch ports?","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":83,"context_line":"               \"mtu\": 1500,"},{"line_number":84,"context_line":"               \"type\": \"vlan\","},{"line_number":85,"context_line":"               \"use_dhcp\": false,"},{"line_number":86,"context_line":"               \"vlan_id\": 20"},{"line_number":87,"context_line":"           },"},{"line_number":88,"context_line":"           {"},{"line_number":89,"context_line":"               \"mtu\": 1500,"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_f71cfcc2","line":86,"in_reply_to":"5f7c97a3_78da22e9","updated":"2018-06-29 14:06:23.000000000","message":"Yes. This example was indicating, that with this solution we do not have to anymore rely on un-tagged L2 networks. This example illustrates a Multi-rack case where the target node can be in next rack to the ironic-conductor.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"97924320186f46cd2a731b9da774413bbcd0c730","unresolved":false,"context_lines":[{"line_number":92,"context_line":"               \"use_dhcp\": false"},{"line_number":93,"context_line":"           }"},{"line_number":94,"context_line":"       ]"},{"line_number":95,"context_line":"   }"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":".. end"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_b80f5a37","line":95,"updated":"2018-06-26 09:36:28.000000000","message":"Is there any versioning for this schema? Would it be more future proof to define it as:\n\n{\n    \"type\": \"os-net-config\",\n    \"version\": 1,\n    \"config\": \u003cabove config\u003e\n}","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"f83c2057fdcce4ee130f35ac28ffd411614ba2e3","unresolved":false,"context_lines":[{"line_number":92,"context_line":"               \"use_dhcp\": false"},{"line_number":93,"context_line":"           }"},{"line_number":94,"context_line":"       ]"},{"line_number":95,"context_line":"   }"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":".. end"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_c54674a8","line":95,"in_reply_to":"5f7c97a3_3a3b39d1","updated":"2018-07-02 08:55:06.000000000","message":"I wasn\u0027t suggesting that, but that is an interesting idea, if os-net-config provides schema validation support.\n\nI was more thinking about future proofing, with versioning in case the os-net-config schema changes, and a type field in case we need to support other tools than os-net-config. These could both be over-engineering, but thought I\u0027d bring it up.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":92,"context_line":"               \"use_dhcp\": false"},{"line_number":93,"context_line":"           }"},{"line_number":94,"context_line":"       ]"},{"line_number":95,"context_line":"   }"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":".. end"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_3a3b39d1","line":95,"in_reply_to":"5f7c97a3_b80f5a37","updated":"2018-06-29 14:06:23.000000000","message":"Are you suggesting that we make a validation with the os-net-config tool installed in on the IPA side before loading the configuration, with the version info provided in this Json?","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"97924320186f46cd2a731b9da774413bbcd0c730","unresolved":false,"context_lines":[{"line_number":125,"context_line":"Alternatives"},{"line_number":126,"context_line":"------------"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Data model impact"},{"line_number":131,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_d8e576dc","line":128,"updated":"2018-06-26 09:36:28.000000000","message":"Or cloud-init with a config-drive.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"adc88319ce79965567b7c4799284eb64edcd1d86","unresolved":false,"context_lines":[{"line_number":125,"context_line":"Alternatives"},{"line_number":126,"context_line":"------------"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Data model impact"},{"line_number":131,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5f7c97a3_d791c047","line":128,"in_reply_to":"5f7c97a3_d8e576dc","updated":"2018-06-29 14:06:23.000000000","message":"This was already discussed in Harald Jensås\u0027s comments in Patch Set 9 and Patch Set 10. Please check the review history for more details.","commit_id":"133f15b8f4a3d70fc80017f79c8359436805000d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"f83c2057fdcce4ee130f35ac28ffd411614ba2e3","unresolved":false,"context_lines":[{"line_number":60,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\"."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":66,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":15,"id":"5f7c97a3_256968f7","line":63,"updated":"2018-07-02 08:55:06.000000000","message":"I\u0027d like to see a little more information on the life cycle of this field in both cases.\n\nStandalone: does ironic ever modify/clear this field?\n\nIntegrated: will the logic be roughly equivalent to the existing code? All pxe_enabled ports are added to host_network_info during deployment/cleaning/rescue after being bound? Will they all be access mode (untagged)?","commit_id":"de6ddd1cc346530073626bdf06f00626cb53e7a6"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"15dfce01c23f250f1e03f84859ed9f974a81005e","unresolved":false,"context_lines":[{"line_number":60,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\"."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":66,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":15,"id":"5f7c97a3_98c8c24d","line":63,"in_reply_to":"5f7c97a3_256968f7","updated":"2018-07-03 13:58:24.000000000","message":"Done","commit_id":"de6ddd1cc346530073626bdf06f00626cb53e7a6"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c95bb7af709c1a2d25fff5a7953403d333480edb","unresolved":false,"context_lines":[{"line_number":55,"context_line":"nodes which can be derived as follows:"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"1. Stand-alone mode:- IP network information can be fed/removed directly by the"},{"line_number":58,"context_line":"   end user to ironic node port or portgroup\u0027s field \"host_network_info\"."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_673bbf38","line":58,"updated":"2018-07-03 17:54:59.000000000","message":"Should we call this something like deploy_network_info, to make it clear that it relates to the deployment phase rather than the instance?","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"ac1008624ec4be40fda32a615620f40fe05778db","unresolved":false,"context_lines":[{"line_number":55,"context_line":"nodes which can be derived as follows:"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"1. Stand-alone mode:- IP network information can be fed/removed directly by the"},{"line_number":58,"context_line":"   end user to ironic node port or portgroup\u0027s field \"host_network_info\"."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_e281efc3","line":58,"in_reply_to":"5f7c97a3_673bbf38","updated":"2018-07-04 11:28:53.000000000","message":"True. But would it then cater to other operations like cleaning, introspection etc?","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"778af9d78a001df26dca3cb232a8c28e150f6a07","unresolved":false,"context_lines":[{"line_number":55,"context_line":"nodes which can be derived as follows:"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"1. Stand-alone mode:- IP network information can be fed/removed directly by the"},{"line_number":58,"context_line":"   end user to ironic node port or portgroup\u0027s field \"host_network_info\"."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_a2452e17","line":58,"in_reply_to":"5f7c97a3_e281efc3","updated":"2018-07-04 17:38:07.000000000","message":"Fair point.","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c95bb7af709c1a2d25fff5a7953403d333480edb","unresolved":false,"context_lines":[{"line_number":60,"context_line":"2. Integrated Deployment Mode:- Neutron\u0027s network info is derivable from ironic"},{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_67241f0f","line":63,"updated":"2018-07-03 17:54:59.000000000","message":"Ironic will clear the field automatically, right?\n\nAlso, nit: baremetal","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c95bb7af709c1a2d25fff5a7953403d333480edb","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":67,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_a70a3776","line":64,"updated":"2018-07-03 17:54:59.000000000","message":"Would it not be cleared when the node is unplugged from the provisioning network? And then reconfigured when the node goes through cleaning or is rescued, possibly with a different IP?","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"778af9d78a001df26dca3cb232a8c28e150f6a07","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":67,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_22635eb3","line":64,"in_reply_to":"5f7c97a3_58a3f49b","updated":"2018-07-04 17:38:07.000000000","message":"+1","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"be25a7f59f4a28814685bf2f473de94f0962d112","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":67,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_93cdee5f","line":64,"in_reply_to":"5f7c97a3_58a3f49b","updated":"2018-07-17 13:04:28.000000000","message":"Done","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"359103aecb52f005532ff13fd7f0cd81dd061264","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":67,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_58a3f49b","line":64,"in_reply_to":"5f7c97a3_58b69427","updated":"2018-07-04 11:48:08.000000000","message":"There is still a requirement for a provisioning network, the Ironic python agent needs to be able to communicate to the ironic API and the conductor\u0027s need to be able to communicate with the IPA API, this isn\u0027t possible with isolated tenant networks as they can have overlapping IPs and therefore aren\u0027t necessarily uniquely addressed. So we have a provisioning network (which can be an L3 network itself, that the conductor is directly plumbed into providing the conductor --\u003e baremetal(IPA api) connectivity","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"be25a7f59f4a28814685bf2f473de94f0962d112","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":67,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_b3ca6a43","line":64,"in_reply_to":"5f7c97a3_a70a3776","updated":"2018-07-17 13:04:28.000000000","message":"Done","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"ac1008624ec4be40fda32a615620f40fe05778db","unresolved":false,"context_lines":[{"line_number":61,"context_line":"   node port or portgroup -\u003e neutron port -\u003e neutron subnet -\u003e neutron network."},{"line_number":62,"context_line":"   This info can be cached in ironic node port or portgroup\u0027s field"},{"line_number":63,"context_line":"   \"host_network_info\". This field should be cleared when the baremeatl node is"},{"line_number":64,"context_line":"   disassociated with nova node."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Ironic virtmedia boot interfaces can use this derived information to construct"},{"line_number":67,"context_line":"os-net-config JSON structure:[2]. This would be added as \"host_network_info\" in"}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_58b69427","line":64,"in_reply_to":"5f7c97a3_a70a3776","updated":"2018-07-04 11:28:53.000000000","message":"In my opinion we don\u0027t need a separate provisioning network to support multi-tenancy anymore with this solution. Tenant network can be used straight for provisioning as well, as there is no dependency on L2 anymore.","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"359103aecb52f005532ff13fd7f0cd81dd061264","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"3. os-net-config tool and its dependencies need to be added to configure"},{"line_number":125,"context_line":"   realtime network in accordence to /etc/os-net-config/config.json, before"},{"line_number":126,"context_line":"   initiating REST APIs towards ironic conductor."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"This method can be used by all ironic in-band operations, for avoiding L2"},{"line_number":129,"context_line":"dependency."}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_58bc54b4","line":126,"updated":"2018-07-04 11:48:08.000000000","message":"We already have added support in the ironic python agent ramdisk for config-drive based configuration for rescue which supports config of the network interfaces, lets use that form of network interface configuration and not add another dependency.\n\nSee: https://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/metadata-service-network-info.html#rest-api-impact\n\nThis is supported by cloud-init and openstack\u0027s own glean tool.","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"778af9d78a001df26dca3cb232a8c28e150f6a07","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"3. os-net-config tool and its dependencies need to be added to configure"},{"line_number":125,"context_line":"   realtime network in accordence to /etc/os-net-config/config.json, before"},{"line_number":126,"context_line":"   initiating REST APIs towards ironic conductor."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"This method can be used by all ironic in-band operations, for avoiding L2"},{"line_number":129,"context_line":"dependency."}],"source_content_type":"text/x-rst","patch_set":16,"id":"5f7c97a3_62e71611","line":126,"in_reply_to":"5f7c97a3_58bc54b4","updated":"2018-07-04 17:38:07.000000000","message":"Interesting. I suppose that is another application of the instance config-drive, which is opaque to ironic. This requires ironic to be able to generate the network configuration.","commit_id":"19d26a7fb777e11a87f66af7b5f236275a50cc1a"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"e2ddb5a2491b5a2a713333a856ca26fe772a9dbf","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://storyboard.openstack.org/#!/story/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardware supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to run on purely routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests creating extra option to virtual media based boot"},{"line_number":21,"context_line":"interfaces for deployment with backward compatibility to the existing DHCP"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_b7fd2409","line":18,"range":{"start_line":13,"start_character":0,"end_line":18,"end_character":31},"updated":"2018-07-09 20:13:22.000000000","message":"Since we have recently extended Ironic to support booting remote subnets over DHCP using DHCP relay, I wonder if this paragraph should be edited to reflect that. Or perhaps a second paragraph could be added specifying the following:\n\nIronic now supports DHCP (and optionally PXE booting) of nodes on remote subnets via DHCP relay. When the node boots, its DHCP requests are intercepted by a DHCP relay listening on the subnet attached to the node, and forwarded to the DHCP server. However, the DHCP relay is typically placed on a network router, so configuration requires administrative access to the underlying network, or configuration by a network administrator.\n\nThis is touched on in the Problem Description, but without context.","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"31d6990dea70f17706bd996817c97431946042c5","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://storyboard.openstack.org/#!/story/1749193"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ironic relies on L2 network for IP configuration (and optionally PXE booting)"},{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardware supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to run on purely routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests creating extra option to virtual media based boot"},{"line_number":21,"context_line":"interfaces for deployment with backward compatibility to the existing DHCP"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_5a01e36c","line":18,"range":{"start_line":13,"start_character":0,"end_line":18,"end_character":31},"in_reply_to":"5f7c97a3_b7fd2409","updated":"2018-07-13 20:47:10.000000000","message":"I think mentioning so would detract from the context attempting to be set, which is that there are situations and cases where DHCP is... either by policy, practice, or technical limitations, not feasible or is impossible.","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"e2ddb5a2491b5a2a713333a856ca26fe772a9dbf","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Example use cases:"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Multi Rack deployment: Inter-Rack L2 switching beyond ToR is a challenge with"},{"line_number":34,"context_line":"  DHCP-relay, as this adds requirement of either having ToR with DHCP-relay, or"},{"line_number":35,"context_line":"  having a system or VM listening on the remote subnet to relay or proxy DHCP"},{"line_number":36,"context_line":"  requests."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"* Remote site targets: It would be a great idea to deploy Servers in different"},{"line_number":39,"context_line":"  remote sites from a central location. With the current setup of L2 dependency"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_f7163c4a","line":36,"range":{"start_line":33,"start_character":0,"end_line":36,"end_character":11},"updated":"2018-07-09 20:13:22.000000000","message":"Above you state that DHCP requires L2 connectivity, but this bullet point contradicts that statement.","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"e2ddb5a2491b5a2a713333a856ca26fe772a9dbf","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  having a system or VM listening on the remote subnet to relay or proxy DHCP"},{"line_number":36,"context_line":"  requests."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"* Remote site targets: It would be a great idea to deploy Servers in different"},{"line_number":39,"context_line":"  remote sites from a central location. With the current setup of L2 dependency"},{"line_number":40,"context_line":"  we need to provide VPN L2 tunnel between Ironic Conductor and remote target"},{"line_number":41,"context_line":"  server."},{"line_number":42,"context_line":"  example: Edge cloud servers or Distributed VNF Flexi Zone Controllers."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_9754a8fe","line":41,"range":{"start_line":38,"start_character":0,"end_line":41,"end_character":9},"updated":"2018-07-09 20:13:22.000000000","message":"This bullet point seems to contradict the above bullet point. Clearly it is possible to handle remote site targets using DHCP relay, which doesn\u0027t require an L2 VPN. The issue is when DHCP relay is not an option when the operator can\u0027t modify the network infrastructure. This is a common scenario, either in hosted environments or due to security policy within an organization.","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"e2ddb5a2491b5a2a713333a856ca26fe772a9dbf","unresolved":false,"context_lines":[{"line_number":131,"context_line":"Alternatives"},{"line_number":132,"context_line":"------------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Data model impact"},{"line_number":137,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_17941840","line":134,"range":{"start_line":134,"start_character":0,"end_line":134,"end_character":55},"updated":"2018-07-09 20:13:22.000000000","message":"Another option when using CoreOS is ignition https://coreos.com/ignition/docs/latest/network-configuration.html","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"be25a7f59f4a28814685bf2f473de94f0962d112","unresolved":false,"context_lines":[{"line_number":131,"context_line":"Alternatives"},{"line_number":132,"context_line":"------------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Data model impact"},{"line_number":137,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_132ebeaa","line":134,"range":{"start_line":134,"start_character":0,"end_line":134,"end_character":55},"in_reply_to":"5f7c97a3_17941840","updated":"2018-07-17 13:04:28.000000000","message":"Done","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"31d6990dea70f17706bd996817c97431946042c5","unresolved":false,"context_lines":[{"line_number":131,"context_line":"Alternatives"},{"line_number":132,"context_line":"------------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"glean:[3] can also be used instead of os-net-config:[2]"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Data model impact"},{"line_number":137,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_7a937f00","line":134,"range":{"start_line":134,"start_character":0,"end_line":134,"end_character":55},"in_reply_to":"5f7c97a3_17941840","updated":"2018-07-13 20:47:10.000000000","message":"Great suggestion!\n\nMight be something worth considering as a follow-up to this spec. I would be hesitant about focusing on it too much though, because of the size of CoreOS, we\u0027ve had to restrict its testing to the IPA gate.","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":12398,"name":"Dan Sneddon","email":"dsneddon@redhat.com","username":"dsneddon"},"change_message_id":"e2ddb5a2491b5a2a713333a856ca26fe772a9dbf","unresolved":false,"context_lines":[{"line_number":248,"context_line":"Work Items"},{"line_number":249,"context_line":"----------"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"* Changes to IPA and addition of os-net-config tool in IPA Ramdisk."},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"* Add support to build os-net-config JSON structure using create_vfat_image:[1]"},{"line_number":254,"context_line":"  on hardware which support virtual media boot."}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_f742bcab","line":251,"range":{"start_line":251,"start_character":0,"end_line":251,"end_character":67},"updated":"2018-07-09 20:13:22.000000000","message":"Perhaps mention that we will need to include the python dependencies listed in requirements.txt in the os-net-config package.\n\nThe latest list of requirements is in https://review.openstack.org/#/c/580290\n\npbr\u003e\u003d2.0.0 # Apache-2.0\nanyjson\u003e\u003d0.3.3 # BSD\nsix\u003e\u003d1.9.0 # MIT\neventlet!\u003d0.18.3,\u003e\u003d0.18.2 # MIT\niso8601\u003e\u003d0.1.11 # MIT\nnetaddr\u003e\u003d0.7.13 #BSD\noslo.concurrency\u003e\u003d3.8.0 # Apache-2.0\noslo.utils\u003e\u003d3.20.0 # Apache-2.0\nPyYAML\u003e\u003d3.10.0 # MIT\njsonschema\u003e\u003d2.0.0,\u003c3.0.0  # MIT","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"31d6990dea70f17706bd996817c97431946042c5","unresolved":false,"context_lines":[{"line_number":248,"context_line":"Work Items"},{"line_number":249,"context_line":"----------"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"* Changes to IPA and addition of os-net-config tool in IPA Ramdisk."},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"* Add support to build os-net-config JSON structure using create_vfat_image:[1]"},{"line_number":254,"context_line":"  on hardware which support virtual media boot."}],"source_content_type":"text/x-rst","patch_set":17,"id":"5f7c97a3_9a71bb10","line":251,"range":{"start_line":251,"start_character":0,"end_line":251,"end_character":67},"in_reply_to":"5f7c97a3_f742bcab","updated":"2018-07-13 20:47:10.000000000","message":"We attempt to keep these work items more as high level tasks or objectives. tl;dr this is acceptable.\n\nBesides, anyjson, oslo.concurrency, pyyaml, and jsonschema would be the only additional requirements above and beyond what are installed with IPA today.","commit_id":"6277732760b32e08f7d99351e3556cb47aa6a490"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"b857b6b0fcfd86f62e4c787af243ef15bc46e329","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"},{"line_number":8,"context_line":"Ironic L3 based deployment"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://storyboard.openstack.org/#!/story/1749193"}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_8e4e2ac0","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":26},"updated":"2018-07-27 13:48:34.000000000","message":"this is a poor name for this blueprint. what you are describing isn\u0027t \"An L3 based deployment\". What you are describing is a deployment that uses static networking configuration instead of DHCP.\n\nPerhaps consider renaming the spec \"Inject static network configuration instead of using DHCP\"?","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"55fa4069974755b03c79de00bc0cf3edad0f9b01","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"},{"line_number":8,"context_line":"Ironic L3 based deployment"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://storyboard.openstack.org/#!/story/1749193"}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_3ec8491e","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":26},"in_reply_to":"5f7c97a3_8e4e2ac0","updated":"2018-07-30 10:55:15.000000000","message":"I guess this name change means that we need to rename this file, storyboard:- https://storyboard.openstack.org/#!/story/1749193\nLaunchpad:- https://bugs.launchpad.net/ironic/+bug/1749193. Any place else where?","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"b857b6b0fcfd86f62e4c787af243ef15bc46e329","unresolved":false,"context_lines":[{"line_number":14,"context_line":"during provisioning baremetal nodes. It puts limitations on Multi-rack"},{"line_number":15,"context_line":"deployments and remote site deployments like Edge Cloud etc. This specification"},{"line_number":16,"context_line":"proposes passing IP network info via virtual media on hardware supporting"},{"line_number":17,"context_line":"virtual media boot, which enables deployment to run on purely routed L3 layers"},{"line_number":18,"context_line":"without any dependency on DHCP."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This specification suggests creating extra option to virtual media based boot"},{"line_number":21,"context_line":"interfaces for deployment with backward compatibility to the existing DHCP"}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_aee1a699","line":18,"range":{"start_line":17,"start_character":20,"end_line":18,"end_character":30},"updated":"2018-07-27 13:48:34.000000000","message":"I would change this to \"which enables deployment of statically-configured networking instead of relying on DHCP to assign IP addresses\".","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"b857b6b0fcfd86f62e4c787af243ef15bc46e329","unresolved":false,"context_lines":[{"line_number":20,"context_line":"This specification suggests creating extra option to virtual media based boot"},{"line_number":21,"context_line":"interfaces for deployment with backward compatibility to the existing DHCP"},{"line_number":22,"context_line":"based booting. This extra option should be selectable with a flag at boot"},{"line_number":23,"context_line":"interface level."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Problem description"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_0e969af0","line":23,"updated":"2018-07-27 13:48:34.000000000","message":"This is probably just my lack of understanding of Ironic internals, but I don\u0027t understand why the boot interface (virtual media or otherwise) makes a difference to whether an Ironic node\u0027s network information can be statically-configured instead of dynamically controlled via DHCP. Could someone clue me in here?","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"55fa4069974755b03c79de00bc0cf3edad0f9b01","unresolved":false,"context_lines":[{"line_number":20,"context_line":"This specification suggests creating extra option to virtual media based boot"},{"line_number":21,"context_line":"interfaces for deployment with backward compatibility to the existing DHCP"},{"line_number":22,"context_line":"based booting. This extra option should be selectable with a flag at boot"},{"line_number":23,"context_line":"interface level."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Problem description"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_f2bd2503","line":23,"in_reply_to":"5f7c97a3_0e969af0","updated":"2018-07-30 10:55:15.000000000","message":"Done.\n\nThis is an optional option to be added to node port/portgroup info. If this exists, IPA configures node IP address with the available static info, else by DHCP.","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7dbf753f929a5a73f2c4d0d7adbff6301547a7d2","unresolved":false,"context_lines":[{"line_number":39,"context_line":"  remote sites from a central location. With the current setup of L2 dependency"},{"line_number":40,"context_line":"  we need to provide VPN L2 tunnel between Ironic Conductor and remote target"},{"line_number":41,"context_line":"  server."},{"line_number":42,"context_line":"  example: Edge cloud servers or Distributed VNF Flexi Zone Controllers."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Proposed change"},{"line_number":45,"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":18,"id":"5f7c97a3_85287ced","line":42,"range":{"start_line":42,"start_character":33,"end_line":42,"end_character":71},"updated":"2018-07-26 22:13:39.000000000","message":"what is this? A link would be nice, especially since it seems to be a Nokia-specific product line:\n\nhttps://networks.nokia.com/products/flexi-zone","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"55fa4069974755b03c79de00bc0cf3edad0f9b01","unresolved":false,"context_lines":[{"line_number":39,"context_line":"  remote sites from a central location. With the current setup of L2 dependency"},{"line_number":40,"context_line":"  we need to provide VPN L2 tunnel between Ironic Conductor and remote target"},{"line_number":41,"context_line":"  server."},{"line_number":42,"context_line":"  example: Edge cloud servers or Distributed VNF Flexi Zone Controllers."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Proposed change"},{"line_number":45,"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":18,"id":"5f7c97a3_5ecd052d","line":42,"range":{"start_line":42,"start_character":33,"end_line":42,"end_character":71},"in_reply_to":"5f7c97a3_85287ced","updated":"2018-07-30 10:55:15.000000000","message":"Done","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"b857b6b0fcfd86f62e4c787af243ef15bc46e329","unresolved":false,"context_lines":[{"line_number":108,"context_line":"* Option-1: This vfat image can be attached to the target node\u0027s Virtual Floppy"},{"line_number":109,"context_line":"  if it is supported by the hardware."},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"* Option-2: Some hardware do not support attaching more than one Virtual"},{"line_number":112,"context_line":"  device. In such case the vfat image can be zipped and append to the end of"},{"line_number":113,"context_line":"  IPA boot ISO with an extra 64K block."},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_2e4036a3","line":113,"range":{"start_line":111,"start_character":2,"end_line":113,"end_character":39},"updated":"2018-07-27 13:48:34.000000000","message":"this seems incredibly fragile to me... why 64K? why append to the (readonly) IPA boot ISO? Why not read the virtual media from some other source (Glance? Swift?)","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"55fa4069974755b03c79de00bc0cf3edad0f9b01","unresolved":false,"context_lines":[{"line_number":108,"context_line":"* Option-1: This vfat image can be attached to the target node\u0027s Virtual Floppy"},{"line_number":109,"context_line":"  if it is supported by the hardware."},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"* Option-2: Some hardware do not support attaching more than one Virtual"},{"line_number":112,"context_line":"  device. In such case the vfat image can be zipped and append to the end of"},{"line_number":113,"context_line":"  IPA boot ISO with an extra 64K block."},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"On the Ironic-Python-Agent Ramdisk side below changes are needed:"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-rst","patch_set":18,"id":"5f7c97a3_b2100d4e","line":113,"range":{"start_line":111,"start_character":2,"end_line":113,"end_character":39},"in_reply_to":"5f7c97a3_2e4036a3","updated":"2018-07-30 10:55:15.000000000","message":"Yes, the IPA boot ISO base should be stored in Glance, and the node specific patched ISO should be stored in Swift. During the node provisioning time the Node ISO\u0027s Swift URI will be attached to the Target node BMC for IPA booting.\n\n\u003e\u003e Why not read the virtual media from some other source (Glance? Swift?)\nTarget node\u0027s static IP info is embedded in this chunk, Unless we configure this to the node, it cannot access Glance/Swift.\n\n\u003e\u003e Why 64K?\nI guess it should be multiple of 32K for ISOs. We can then make it to 32KB, if we are sure that our network config and other static info (like Ironic conductor URI) does not exceed 32KB.","commit_id":"0932a331634d9a1ce5eb9b3a146a1e194460f1da"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"2814933ebf60ec97ba7ab295f2d52b12831647f2","unresolved":false,"context_lines":[{"line_number":174,"context_line":"* Update ``openstack baremetal port create``, ``openstack baremetal port set``,"},{"line_number":175,"context_line":"  ``openstack baremetal port group create`` and"},{"line_number":176,"context_line":"  ``openstack baremetal port group set`` commands to accept one more argument"},{"line_number":177,"context_line":"  ``--host_network_info \u003cJSON\u003e``, with help text describing JSON structure of"},{"line_number":178,"context_line":"  the host network configuration."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"* Extend the output of the ``openstack baremetal port show`` and"}],"source_content_type":"text/x-rst","patch_set":20,"id":"3f79a3b5_c2c5ba5d","line":177,"updated":"2018-08-23 14:02:57.000000000","message":"--host-network-info (dashed, not underscores here)","commit_id":"ec52e732af9090c78dc7d6652d852b5c54c07953"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"78df12eb9a9c4e4c814bb118c6bd47a1dfc622be","unresolved":false,"context_lines":[{"line_number":174,"context_line":"* Update ``openstack baremetal port create``, ``openstack baremetal port set``,"},{"line_number":175,"context_line":"  ``openstack baremetal port group create`` and"},{"line_number":176,"context_line":"  ``openstack baremetal port group set`` commands to accept one more argument"},{"line_number":177,"context_line":"  ``--host_network_info \u003cJSON\u003e``, with help text describing JSON structure of"},{"line_number":178,"context_line":"  the host network configuration."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"* Extend the output of the ``openstack baremetal port show`` and"}],"source_content_type":"text/x-rst","patch_set":20,"id":"3f79a3b5_f743c13d","line":177,"in_reply_to":"3f79a3b5_c2c5ba5d","updated":"2018-08-24 09:14:09.000000000","message":"Done","commit_id":"ec52e732af9090c78dc7d6652d852b5c54c07953"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"2814933ebf60ec97ba7ab295f2d52b12831647f2","unresolved":false,"context_lines":[{"line_number":202,"context_line":"* Changes to IPA service, which would check availability of \"host_network_info\""},{"line_number":203,"context_line":"  parameter in Virtual Floppy."},{"line_number":204,"context_line":""},{"line_number":205,"context_line":"* os-net-config package needs to be added to IPA deploy ramdisk."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"Security impact"},{"line_number":208,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":20,"id":"3f79a3b5_e2c23653","line":205,"updated":"2018-08-23 14:02:57.000000000","message":"Only if this feature is used, right?","commit_id":"ec52e732af9090c78dc7d6652d852b5c54c07953"},{"author":{"_account_id":22194,"name":"chandra shekar","email":"chandra.s.rangavajjula@nokia.com","username":"shekar"},"change_message_id":"78df12eb9a9c4e4c814bb118c6bd47a1dfc622be","unresolved":false,"context_lines":[{"line_number":202,"context_line":"* Changes to IPA service, which would check availability of \"host_network_info\""},{"line_number":203,"context_line":"  parameter in Virtual Floppy."},{"line_number":204,"context_line":""},{"line_number":205,"context_line":"* os-net-config package needs to be added to IPA deploy ramdisk."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"Security impact"},{"line_number":208,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":20,"id":"3f79a3b5_576cf5ce","line":205,"in_reply_to":"3f79a3b5_e2c23653","updated":"2018-08-24 09:14:09.000000000","message":"Yes.\nBut there is no harm (except for some size increase) in adding this package to ramdisk even if it is not used right!","commit_id":"ec52e732af9090c78dc7d6652d852b5c54c07953"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"8200467a5f58ab844ef20245bb279f52e3feb386","unresolved":false,"context_lines":[{"line_number":159,"context_line":""},{"line_number":160,"context_line":"  New request/response field:"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"  * ``host-network-info`` JSON structure of the host network configuration."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"Client (CLI) impact"},{"line_number":165,"context_line":"-------------------"}],"source_content_type":"text/x-rst","patch_set":21,"id":"3f79a3b5_d1007c05","line":162,"updated":"2018-08-31 10:32:54.000000000","message":"I was only talking about the CLI. You cannot have a field with dashes in Python..","commit_id":"4c5866d3788c08a4973cfe1f670d7ce13f7a107c"},{"author":{"_account_id":28950,"name":"Nokia Airframe CI","email":"ci.openstack-ironic@nokia.com","username":"cinokiaironic"},"change_message_id":"43ec2ba829a1a3333e2cc66eedd44233005d72f3","unresolved":false,"context_lines":[{"line_number":159,"context_line":""},{"line_number":160,"context_line":"  New request/response field:"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"  * ``host-network-info`` JSON structure of the host network configuration."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"Client (CLI) impact"},{"line_number":165,"context_line":"-------------------"}],"source_content_type":"text/x-rst","patch_set":21,"id":"3f79a3b5_f4cd6686","line":162,"in_reply_to":"3f79a3b5_d1007c05","updated":"2018-08-31 11:01:46.000000000","message":"Done","commit_id":"4c5866d3788c08a4973cfe1f670d7ce13f7a107c"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"8200467a5f58ab844ef20245bb279f52e3feb386","unresolved":false,"context_lines":[{"line_number":175,"context_line":"* Update ``openstack baremetal port create``, ``openstack baremetal port set``,"},{"line_number":176,"context_line":"  ``openstack baremetal port group create`` and"},{"line_number":177,"context_line":"  ``openstack baremetal port group set`` commands to accept one more argument"},{"line_number":178,"context_line":"  ``--host-network-info \u003cJSON\u003e``, with help text describing JSON structure of"},{"line_number":179,"context_line":"  the host network configuration."},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"* Extend the output of the ``openstack baremetal port show`` and"}],"source_content_type":"text/x-rst","patch_set":21,"id":"3f79a3b5_74fdd606","line":178,"updated":"2018-08-31 10:32:54.000000000","message":"This was the only place that required changing to dashes.","commit_id":"4c5866d3788c08a4973cfe1f670d7ce13f7a107c"},{"author":{"_account_id":28950,"name":"Nokia Airframe CI","email":"ci.openstack-ironic@nokia.com","username":"cinokiaironic"},"change_message_id":"43ec2ba829a1a3333e2cc66eedd44233005d72f3","unresolved":false,"context_lines":[{"line_number":175,"context_line":"* Update ``openstack baremetal port create``, ``openstack baremetal port set``,"},{"line_number":176,"context_line":"  ``openstack baremetal port group create`` and"},{"line_number":177,"context_line":"  ``openstack baremetal port group set`` commands to accept one more argument"},{"line_number":178,"context_line":"  ``--host-network-info \u003cJSON\u003e``, with help text describing JSON structure of"},{"line_number":179,"context_line":"  the host network configuration."},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"* Extend the output of the ``openstack baremetal port show`` and"}],"source_content_type":"text/x-rst","patch_set":21,"id":"3f79a3b5_14c90278","line":178,"in_reply_to":"3f79a3b5_74fdd606","updated":"2018-08-31 11:01:46.000000000","message":"Done","commit_id":"4c5866d3788c08a4973cfe1f670d7ce13f7a107c"}]}