)]}'
{"specs/approved/intel-rsd-driver.rst":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"abd7ae560a191c828536225a9c8dacb901494397","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Intel RSD is an architecture that implements an interface based on"},{"line_number":20,"context_line":"DMTF Redfish. This proposal adds power and management capabilities"},{"line_number":21,"context_line":"for RSD into Ironic."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Proposed change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_ca620a17","line":21,"updated":"2017-09-22 15:37:51.000000000","message":"I\u0027d say it\u0027s mandatory, otherwise it\u0027s not clear why not use the existing driver.","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"a1255c91f6f77303b9527f696789146298cfc7de","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Intel RSD is an architecture that implements an interface based on"},{"line_number":20,"context_line":"DMTF Redfish. This proposal adds power and management capabilities"},{"line_number":21,"context_line":"for RSD into Ironic."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Proposed change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_7a089327","line":21,"updated":"2017-09-22 08:17:19.000000000","message":"It is worth adding the detail about why the current redfish driver in Ironic is not sufficient of Intel RSD.","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"de3470961cbfcae3511b71d863d736a02524a33e","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Intel RSD is an architecture that implements an interface based on"},{"line_number":20,"context_line":"DMTF Redfish. This proposal adds power and management capabilities"},{"line_number":21,"context_line":"for RSD into Ironic."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Proposed change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_cdf22107","line":21,"in_reply_to":"7f515b1d_ca620a17","updated":"2017-09-22 17:28:20.000000000","message":"I added further explanation, let me know if it\u0027s sufficient.","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"abd7ae560a191c828536225a9c8dacb901494397","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Proposed change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"This spec proposes adding a new driver, intel-rsd, that inherits from the"},{"line_number":27,"context_line":"redfish hardware type."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"The new interfaces will use the `rsd-lib`_ library to handle calls made"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_aa67ce25","line":26,"updated":"2017-09-22 15:37:51.000000000","message":"s/driver/hardware type/","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"de3470961cbfcae3511b71d863d736a02524a33e","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Proposed change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"This spec proposes adding a new driver, intel-rsd, that inherits from the"},{"line_number":27,"context_line":"redfish hardware type."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"The new interfaces will use the `rsd-lib`_ library to handle calls made"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_6d944d8c","line":26,"in_reply_to":"7f515b1d_aa67ce25","updated":"2017-09-22 17:28:20.000000000","message":"Done","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"abd7ae560a191c828536225a9c8dacb901494397","unresolved":false,"context_lines":[{"line_number":86,"context_line":"Other deployer impact"},{"line_number":87,"context_line":"---------------------"},{"line_number":88,"context_line":"The following driver_info fields are required while enrolling nodes"},{"line_number":89,"context_line":"into ironic:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    * ``podm_address``: The URL address to the RSD Pod Manager. It"},{"line_number":92,"context_line":"      should include schema and authority portion of the URL. For example:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_8a6c1209","line":89,"updated":"2017-09-22 15:37:51.000000000","message":"please prefix all field with your driver name or short name (like rsd_***)","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"de3470961cbfcae3511b71d863d736a02524a33e","unresolved":false,"context_lines":[{"line_number":86,"context_line":"Other deployer impact"},{"line_number":87,"context_line":"---------------------"},{"line_number":88,"context_line":"The following driver_info fields are required while enrolling nodes"},{"line_number":89,"context_line":"into ironic:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    * ``podm_address``: The URL address to the RSD Pod Manager. It"},{"line_number":92,"context_line":"      should include schema and authority portion of the URL. For example:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_8d978991","line":89,"in_reply_to":"7f515b1d_8a6c1209","updated":"2017-09-22 17:28:20.000000000","message":"Done","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"abd7ae560a191c828536225a9c8dacb901494397","unresolved":false,"context_lines":[{"line_number":137,"context_line":"Work Items"},{"line_number":138,"context_line":"----------"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Add a new ``intel-rsd`` driver based on the ``redfish`` hardware type,"},{"line_number":141,"context_line":"  implementing power and management interfaces."},{"line_number":142,"context_line":"* Write unit-tests for the new code."},{"line_number":143,"context_line":"* Set up a working CI system for hardware validation."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_6a71d6e0","line":140,"updated":"2017-09-22 15:37:51.000000000","message":"s/driver/hardware type/","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"de3470961cbfcae3511b71d863d736a02524a33e","unresolved":false,"context_lines":[{"line_number":137,"context_line":"Work Items"},{"line_number":138,"context_line":"----------"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"* Add a new ``intel-rsd`` driver based on the ``redfish`` hardware type,"},{"line_number":141,"context_line":"  implementing power and management interfaces."},{"line_number":142,"context_line":"* Write unit-tests for the new code."},{"line_number":143,"context_line":"* Set up a working CI system for hardware validation."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_cd7d816f","line":140,"in_reply_to":"7f515b1d_6a71d6e0","updated":"2017-09-22 17:28:20.000000000","message":"Done","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"abd7ae560a191c828536225a9c8dacb901494397","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"* Updating ironic install-guide documentation section"},{"line_number":167,"context_line":"  `Setup the drivers for the Bare Metal service"},{"line_number":168,"context_line":"  \u003chttp://docs.openstack.org/project-install-guide/baremetal/draft/setup-drivers.html\u003e`_."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"References"},{"line_number":171,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_4a761adb","line":168,"updated":"2017-09-22 15:37:51.000000000","message":"I\u0027m not sure we have anything driver-specific here. Also the link is to the old guide.","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"de3470961cbfcae3511b71d863d736a02524a33e","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"* Updating ironic install-guide documentation section"},{"line_number":167,"context_line":"  `Setup the drivers for the Bare Metal service"},{"line_number":168,"context_line":"  \u003chttp://docs.openstack.org/project-install-guide/baremetal/draft/setup-drivers.html\u003e`_."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"References"},{"line_number":171,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7f515b1d_ed883d72","line":168,"in_reply_to":"7f515b1d_4a761adb","updated":"2017-09-22 17:28:20.000000000","message":"Removed","commit_id":"ae13d7033a51a84d315cbf1974f9655257be99ae"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"99efef5a379fae1a253cc7357ee8aae6f408574e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"This spec proposes adding a new hardware type, intel-rsd, that"},{"line_number":39,"context_line":"inherits from the redfish hardware type."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The new interfaces will use the `rsd-lib`_ library to handle calls made"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f515b1d_fef7b7f4","line":38,"range":{"start_line":38,"start_character":47,"end_line":38,"end_character":56},"updated":"2017-09-24 12:46:08.000000000","message":"s/ intel-rsd / ``intel-rsd`` /","commit_id":"afa85a2de89612741f1ff4f5b710a2f1db878c91"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"99efef5a379fae1a253cc7357ee8aae6f408574e","unresolved":false,"context_lines":[{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"This spec proposes adding a new hardware type, intel-rsd, that"},{"line_number":39,"context_line":"inherits from the redfish hardware type."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"The new interfaces will use the `rsd-lib`_ library to handle calls made"},{"line_number":42,"context_line":"from Ironic to the RSD Pod Manager interface. RSD Lib is an extension of"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f515b1d_9ee68344","line":39,"range":{"start_line":39,"start_character":18,"end_line":39,"end_character":25},"updated":"2017-09-24 12:46:08.000000000","message":"s/ redfish / ``redfish`` /","commit_id":"afa85a2de89612741f1ff4f5b710a2f1db878c91"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"99efef5a379fae1a253cc7357ee8aae6f408574e","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Other deployer impact"},{"line_number":99,"context_line":"---------------------"},{"line_number":100,"context_line":"The following driver_info fields are required while enrolling nodes"},{"line_number":101,"context_line":"into ironic:"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    * ``rsd_podm_address``: The URL address to the RSD Pod Manager. It"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f515b1d_7ebce704","line":100,"range":{"start_line":100,"start_character":52,"end_line":100,"end_character":67},"updated":"2017-09-24 12:46:08.000000000","message":"s/ enrolling nodes / enrolling nodes with ``intel-rsd`` hardware type /","commit_id":"afa85a2de89612741f1ff4f5b710a2f1db878c91"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"00aba6c529d7903870bd667a2b13562bc37edddd","unresolved":false,"context_lines":[{"line_number":149,"context_line":"Work Items"},{"line_number":150,"context_line":"----------"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Add a new ``intel-rsd`` hardware tupe based on the ``redfish`` hardware"},{"line_number":153,"context_line":"  type, implementing power and management interfaces."},{"line_number":154,"context_line":"* Write unit-tests for the new code."},{"line_number":155,"context_line":"* Set up a working CI system for hardware validation."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f515b1d_6d043718","line":152,"range":{"start_line":152,"start_character":35,"end_line":152,"end_character":39},"updated":"2017-09-25 06:47:17.000000000","message":"s/tupe/type/","commit_id":"afa85a2de89612741f1ff4f5b710a2f1db878c91"},{"author":{"_account_id":7229,"name":"Ramon Acedo","email":"ramon@linux-labs.net","username":"racedo"},"change_message_id":"d8e5e044d183d2bffdb0d4ac5fb7fc7d693fb054","unresolved":false,"context_lines":[{"line_number":19,"context_line":"Intel RSD is an architecture that implements an interface based on"},{"line_number":20,"context_line":"DMTF Redfish. This proposal adds power and management capabilities"},{"line_number":21,"context_line":"for RSD into Ironic. The expansion upon standard redfish is needed"},{"line_number":22,"context_line":"because Intel RSD includes API endpoints and functionalities that"},{"line_number":23,"context_line":"don\u0027t exist in standard redfish, such as the /redfish/v1/Nodes"},{"line_number":24,"context_line":"endpoint and the ability to post requests to it in order to"},{"line_number":25,"context_line":"compose nodes using compute system resources, remote storage, and"},{"line_number":26,"context_line":"network resources. The difference in redfish and RSD is expressed"},{"line_number":27,"context_line":"in the `rsd-lib`_ extension for the `sushy`_ library, which"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3f4b6375_80b28ee5","line":24,"range":{"start_line":22,"start_character":9,"end_line":24,"end_character":7},"updated":"2017-10-20 08:15:56.000000000","message":"Could these API differences be coded directly in the Redfish driver rather than writing a whole new driver that includes the Redfish power management calls plus additional ones, as described here?","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":7229,"name":"Ramon Acedo","email":"ramon@linux-labs.net","username":"racedo"},"change_message_id":"d8e5e044d183d2bffdb0d4ac5fb7fc7d693fb054","unresolved":false,"context_lines":[{"line_number":21,"context_line":"for RSD into Ironic. The expansion upon standard redfish is needed"},{"line_number":22,"context_line":"because Intel RSD includes API endpoints and functionalities that"},{"line_number":23,"context_line":"don\u0027t exist in standard redfish, such as the /redfish/v1/Nodes"},{"line_number":24,"context_line":"endpoint and the ability to post requests to it in order to"},{"line_number":25,"context_line":"compose nodes using compute system resources, remote storage, and"},{"line_number":26,"context_line":"network resources. The difference in redfish and RSD is expressed"},{"line_number":27,"context_line":"in the `rsd-lib`_ extension for the `sushy`_ library, which"},{"line_number":28,"context_line":"implements these additional functionalities. Ironic won\u0027t be"},{"line_number":29,"context_line":"doing any node composition as that isn\u0027t in its scope, but it\u0027s"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3f4b6375_20799a2e","line":26,"range":{"start_line":24,"start_character":9,"end_line":26,"end_character":17},"updated":"2017-10-20 08:15:56.000000000","message":"Shouldn\u0027t RSD nodes composition be done by the python-rsdclient? I wouldn\u0027t mix this tasks with the power management tasks that Ironic drivers are meant for.","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e6e9303a68f682b64e8e26065973d336730f7261","unresolved":false,"context_lines":[{"line_number":30,"context_line":"important that it be able to interface with Nodes at the RSD"},{"line_number":31,"context_line":"endpoint, because they contain information that the underlying"},{"line_number":32,"context_line":"system does not, such as what attached PCIe devices are available"},{"line_number":33,"context_line":"and what remote storage has been attached to the system."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"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":6,"id":"7f515b1d_a8ad4832","line":33,"updated":"2017-09-26 14:48:41.000000000","message":"this still does not quite explain why sushy is not enough for ironic case specifically. looking at rsd-lib, Node object is the same as System from our perspective. Why cannot you use the redfish driver with redfish_system_id\u003d/redfish/v1/Nodes/\u003cnode ID\u003e? I think the only mandatory field in System is its ID, maybe \"Boot\" as well.","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"e5a7c639e9c5d4167515450f472d1af1959916f6","unresolved":false,"context_lines":[{"line_number":30,"context_line":"important that it be able to interface with Nodes at the RSD"},{"line_number":31,"context_line":"endpoint, because they contain information that the underlying"},{"line_number":32,"context_line":"system does not, such as what attached PCIe devices are available"},{"line_number":33,"context_line":"and what remote storage has been attached to the system."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"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":6,"id":"7f515b1d_afb547cf","line":33,"in_reply_to":"7f515b1d_068e882d","updated":"2017-09-26 18:12:58.000000000","message":"Also, it\u0027s a small thing, but in Sushy actions like \u0027Reset\u0027 are targeted at the #ComputerSystem.Reset endpoint, whereas a Node\u0027s endpoint is #ComposedNode.Reset, so some changes would need to be made there. The objects aren\u0027t actually exactly the same. https://github.com/openstack/sushy/blob/master/sushy/resources/system/system.py#L30","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"b3a48cee3e49199ca31fd4634827a7c29b218743","unresolved":false,"context_lines":[{"line_number":30,"context_line":"important that it be able to interface with Nodes at the RSD"},{"line_number":31,"context_line":"endpoint, because they contain information that the underlying"},{"line_number":32,"context_line":"system does not, such as what attached PCIe devices are available"},{"line_number":33,"context_line":"and what remote storage has been attached to the system."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"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":6,"id":"7f515b1d_068e882d","line":33,"in_reply_to":"7f515b1d_a8ad4832","updated":"2017-09-26 16:33:08.000000000","message":"One reason would be that a Node object can boot from remote storage that a system object doesn\u0027t know about. Also, I\u0027m not totally clear on this, but does Ironic report up to nova with information like PCIe devices that exist on a bare metal node, or anything relating to storage? That is more information that exists in a Node object that is out of the System\u0027s scope. If the sushy driver will work for all our needs that would be great and save us quite a bit of work, I just want to make sure all the bases are covered, since there is some information in a Node that isn\u0027t covered by System. If all that info is out of Ironic\u0027s scope though, I guess it likely wouldn\u0027t matter here.","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":12597,"name":"Lin Yang","email":"lin.a.yang@intel.com","username":"lin.yang"},"change_message_id":"a1c25ed237d9dbda8565ac52b55bedaa7898b25b","unresolved":false,"context_lines":[{"line_number":30,"context_line":"important that it be able to interface with Nodes at the RSD"},{"line_number":31,"context_line":"endpoint, because they contain information that the underlying"},{"line_number":32,"context_line":"system does not, such as what attached PCIe devices are available"},{"line_number":33,"context_line":"and what remote storage has been attached to the system."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"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":6,"id":"7f515b1d_c607ec17","line":33,"in_reply_to":"7f515b1d_afb547cf","updated":"2017-09-29 22:52:29.000000000","message":"@Dmitry, as Nate mentioned above, a RSD node may already contain an iSCSI remote storage, so the RSD driver will provide an interface that can be consumed by ironic to know which iSCSI should be connected to this node. From my current understanding, user have to call ironic API https://developer.openstack.org/api-ref/baremetal/#create-volume-connector and https://developer.openstack.org/api-ref/baremetal/#create-volume-target to manual connect remote volume to node, ironic cannot auto detect this kind of info. So this spec might need cover how ironic consume the remote storage info return be RSD driver.","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"3bb974c7b6b25b6c45c788ffab55b141bc7fad0e","unresolved":false,"context_lines":[{"line_number":30,"context_line":"important that it be able to interface with Nodes at the RSD"},{"line_number":31,"context_line":"endpoint, because they contain information that the underlying"},{"line_number":32,"context_line":"system does not, such as what attached PCIe devices are available"},{"line_number":33,"context_line":"and what remote storage has been attached to the system."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"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":6,"id":"7f515b1d_41955531","line":33,"in_reply_to":"7f515b1d_c607ec17","updated":"2017-10-09 18:00:39.000000000","message":"If I\u0027m reading the spec right, it only proposes power and management implementations. What you describe is a cool feature, but 1. this spec does not propose it, 2. it will require a spec of its own. This concerns both iSCSI discovery and PCI discovery.\n\nNow, the Reset situation is quite unfortunate. What doesn\u0027t RSD expose System-alike interface for nodes?","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e6e9303a68f682b64e8e26065973d336730f7261","unresolved":false,"context_lines":[{"line_number":108,"context_line":"      Possible types in RSD include System and Node. A system is a base"},{"line_number":109,"context_line":"      level resource in RSD while a node is a composition of disaggregated"},{"line_number":110,"context_line":"      resources, including a system, possible remote storage, and additional"},{"line_number":111,"context_line":"      network ports."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    * ``rsd_resource_id``: The canonical path to the resource that the"},{"line_number":114,"context_line":"      driver will interact with. It should include the root service, version"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7f515b1d_88accc34","line":111,"updated":"2017-09-26 14:48:41.000000000","message":"Why cannot this be extracted from the resource itself? And why do you need this difference for ironic specifically?","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"3bb974c7b6b25b6c45c788ffab55b141bc7fad0e","unresolved":false,"context_lines":[{"line_number":108,"context_line":"      Possible types in RSD include System and Node. A system is a base"},{"line_number":109,"context_line":"      level resource in RSD while a node is a composition of disaggregated"},{"line_number":110,"context_line":"      resources, including a system, possible remote storage, and additional"},{"line_number":111,"context_line":"      network ports."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    * ``rsd_resource_id``: The canonical path to the resource that the"},{"line_number":114,"context_line":"      driver will interact with. It should include the root service, version"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7f515b1d_81b8edc7","line":111,"in_reply_to":"7f515b1d_66ad3c87","updated":"2017-10-09 18:00:39.000000000","message":"Maybe we assume that the RSD driver has to be used with composable nodes, while the existing redfish driver has to be used with \"normal\" systems?","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"b3a48cee3e49199ca31fd4634827a7c29b218743","unresolved":false,"context_lines":[{"line_number":108,"context_line":"      Possible types in RSD include System and Node. A system is a base"},{"line_number":109,"context_line":"      level resource in RSD while a node is a composition of disaggregated"},{"line_number":110,"context_line":"      resources, including a system, possible remote storage, and additional"},{"line_number":111,"context_line":"      network ports."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    * ``rsd_resource_id``: The canonical path to the resource that the"},{"line_number":114,"context_line":"      driver will interact with. It should include the root service, version"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7f515b1d_66ad3c87","line":111,"in_reply_to":"7f515b1d_88accc34","updated":"2017-09-26 16:33:08.000000000","message":"Maybe it could be if you did some string extraction on the address to see if it were a /Nodes or a /Systems address, but the driver needs to know whether to do a get_system() or a get_node() call.","commit_id":"1bdf5ec585b2c9df2447e60a72c4fdbb90c98adb"}]}