)]}'
{"specs/kilo/irmc-power-driver.rst":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e3f11b70665062410aac0a28126beadd006e0ed7","unresolved":false,"context_lines":[{"line_number":11,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-pxe"},{"line_number":12,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-agent"},{"line_number":13,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-nfs"},{"line_number":14,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-cifs"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The proposal presents the work required to add support for power"},{"line_number":17,"context_line":"management features for FUJITSU PRIMERGY iRMC, integrated Remote"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_261bd581","line":14,"updated":"2014-11-18 14:58:25.000000000","message":"line 10 to 14 should be replaced by the following one line.\nhttps://blueprints.launchpad.net/ironic/+spec/irmc-power-driver","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4dcd534836a3a5b576a1fcddee86ec0a6a71175f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-pxe"},{"line_number":12,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-agent"},{"line_number":13,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-nfs"},{"line_number":14,"context_line":"https://blueprints.launchpad.net/ironic/+spec/irmc-driver-for-cifs"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The proposal presents the work required to add support for power"},{"line_number":17,"context_line":"management features for FUJITSU PRIMERGY iRMC, integrated Remote"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_382e72a6","line":14,"in_reply_to":"5a890539_261bd581","updated":"2014-11-19 10:38:13.000000000","message":"+1","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4dcd534836a3a5b576a1fcddee86ec0a6a71175f","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"},{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":"Adding new iRMC Drivers, pxe_irmc, agent_irmc, nfs_irmc and cifs_irmc,"},{"line_number":30,"context_line":"to the list of available drivers in Ironic and implementing the iRMC"},{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_185a9650","line":29,"updated":"2014-11-19 10:38:13.000000000","message":"\"pxe_irmc, agent_irmc\" are quite obvious. \"nfs_irmc and cifs_irmc\" are not and probably require more than just iRMC power driver, right?\n\nI suggest you drop them for now, leaving only pxe_irmc, agent_irmc, and come up with the follow-up spec on nfs_irmc and cifs_irmc, explaining what they are.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"},{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":"Adding new iRMC Drivers, pxe_irmc, agent_irmc, nfs_irmc and cifs_irmc,"},{"line_number":30,"context_line":"to the list of available drivers in Ironic and implementing the iRMC"},{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_b19fb302","line":29,"in_reply_to":"5a890539_185a9650","updated":"2014-11-20 12:15:37.000000000","message":"Dmitry,\n\nThere is a dilemma about \"nfs_irmc and cifs_irmc\" you suggested.\nThat\u0027s the reason I originally wrote 4 blue prints for pxe_irmc, agent_irmc, nfs_irmc and cifs_irmc respectively.\nActually agent_irmc, nfs_irmc and cifs_irmc use virtual media deployment which described in \"iRMC Virtual Media Deploy Driver\",\nhttps://blueprints.launchpad.net/ironic/+spec/irmc-virtualmedia-deploy-driver\nhttps://review.openstack.org/134865\n\nHow about putting the reference to the  \"iRMC Virtual Media Deploy Driver\" blue print or spec, or both?","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"0ebecab731e8b87488a7724c81ace258f361620d","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"},{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":"Adding new iRMC Drivers, pxe_irmc, agent_irmc, nfs_irmc and cifs_irmc,"},{"line_number":30,"context_line":"to the list of available drivers in Ironic and implementing the iRMC"},{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_d8e0dcfc","line":29,"in_reply_to":"5a890539_b19fb302","updated":"2014-11-20 14:01:03.000000000","message":"hmmmm... ideally it should be the following specs:\n\n1.iRMC Power driver introducing pxe_irmc (this one)\n\n2. iRMC Virtual media introducing agent_irmc, nfs_irmc and cifs_irmc, should depend on this one","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"46d8ccf34b311b8e3592c0c0041fdf78c21a9e17","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed change"},{"line_number":28,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":29,"context_line":"Adding new iRMC Drivers, pxe_irmc, agent_irmc, nfs_irmc and cifs_irmc,"},{"line_number":30,"context_line":"to the list of available drivers in Ironic and implementing the iRMC"},{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_5683dccb","line":29,"in_reply_to":"5a890539_d8e0dcfc","updated":"2014-11-20 15:19:07.000000000","message":"Okay, I got it!\nI\u0027ll divides that way.\nthanks.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4dcd534836a3a5b576a1fcddee86ec0a6a71175f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"ServerView Common Command Interface (SCCI) uses HTTP/HTTPS POST protocol."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_9303c90c","line":34,"updated":"2014-11-19 10:38:13.000000000","message":"Will you use some client library or will you issue HTTP[s] requests directly? Please state.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"46d8ccf34b311b8e3592c0c0041fdf78c21a9e17","unresolved":false,"context_lines":[{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"ServerView Common Command Interface (SCCI) uses HTTP/HTTPS POST protocol."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_56d11ccc","line":34,"in_reply_to":"5a890539_18c98479","updated":"2014-11-20 15:19:07.000000000","message":"Okay, I\u0027ll use \u0027requests\u0027 library.\nThanks!","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"ServerView Common Command Interface (SCCI) uses HTTP/HTTPS POST protocol."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_b1125360","line":34,"in_reply_to":"5a890539_9303c90c","updated":"2014-11-20 12:15:37.000000000","message":"I\u0027m thinking to use Httplib2.\nThere is no python library for iRMC right now.\nI\u0027ll add the statement.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"0ebecab731e8b87488a7724c81ace258f361620d","unresolved":false,"context_lines":[{"line_number":31,"context_line":"power module to interact with ServerView Common Command Interface"},{"line_number":32,"context_line":"(SCCI) described in `FUJITSU Software ServerView Suite, Remote Management, iRMC S4 -   integrated Remote Management Controller \u003chttp://manuals.ts.fujitsu.com/file/11470/irmc-s4-ug-en.pdf\u003e`_"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"ServerView Common Command Interface (SCCI) uses HTTP/HTTPS POST protocol."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_18c98479","line":34,"in_reply_to":"5a890539_b1125360","updated":"2014-11-20 14:01:03.000000000","message":"Note that \u0027requests\u0027 library seems much more popular in the OpenStack community.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"0dc80c91f9eb215b89e11ecca8ff411a62081000","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Other end user impact"},{"line_number":69,"context_line":"---------------------"},{"line_number":70,"context_line":"The following driver_info fields are required:"},{"line_number":71,"context_line":"  * irmc_host: hostname or IP of iRMC"},{"line_number":72,"context_line":"  * irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":73,"context_line":"  * irmc_username: username for iRMC with administrator privileges"},{"line_number":74,"context_line":"  * irmc_password: password for irmc_username"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_c873c013","line":71,"updated":"2014-11-19 23:30:05.000000000","message":"Please use \"irmc_address\" for consistency with other drivers","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Other end user impact"},{"line_number":69,"context_line":"---------------------"},{"line_number":70,"context_line":"The following driver_info fields are required:"},{"line_number":71,"context_line":"  * irmc_host: hostname or IP of iRMC"},{"line_number":72,"context_line":"  * irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":73,"context_line":"  * irmc_username: username for iRMC with administrator privileges"},{"line_number":74,"context_line":"  * irmc_password: password for irmc_username"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_51012f05","line":71,"in_reply_to":"5a890539_c873c013","updated":"2014-11-20 12:15:37.000000000","message":"Devananda,\n\nOkay.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"0dc80c91f9eb215b89e11ecca8ff411a62081000","unresolved":false,"context_lines":[{"line_number":72,"context_line":"  * irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":73,"context_line":"  * irmc_username: username for iRMC with administrator privileges"},{"line_number":74,"context_line":"  * irmc_password: password for irmc_username"},{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_e85f9c73","line":75,"updated":"2014-11-19 23:30:05.000000000","message":"Would \"irmc_auth_method\" be clearer, then?","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"0e66d4f9723ee271c5e058fe47c97ca16a6b95b8","unresolved":false,"context_lines":[{"line_number":72,"context_line":"  * irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":73,"context_line":"  * irmc_username: username for iRMC with administrator privileges"},{"line_number":74,"context_line":"  * irmc_password: password for irmc_username"},{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_43abf31d","line":75,"in_reply_to":"5a890539_442fab88","updated":"2014-11-21 13:39:15.000000000","message":"sorry, I\u0027ll change it to \"irmc_auth_method\".","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":72,"context_line":"  * irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":73,"context_line":"  * irmc_username: username for iRMC with administrator privileges"},{"line_number":74,"context_line":"  * irmc_password: password for irmc_username"},{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_442fab88","line":75,"in_reply_to":"5a890539_e85f9c73","updated":"2014-11-20 12:15:37.000000000","message":"I\u0027ll change it to \"irmc_authentication_method\".","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"0dc80c91f9eb215b89e11ecca8ff411a62081000","unresolved":false,"context_lines":[{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Scalability impact"},{"line_number":81,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_0865c844","line":78,"updated":"2014-11-19 23:30:05.000000000","message":"I agree with Dmitry\u0027s suggestion","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4dcd534836a3a5b576a1fcddee86ec0a6a71175f","unresolved":false,"context_lines":[{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Scalability impact"},{"line_number":81,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_38059214","line":78,"updated":"2014-11-19 10:38:13.000000000","message":"I think it should be in the configuration, rather than in driver_info. Also not sure about irmc_auth","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Scalability impact"},{"line_number":81,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_043cc3be","line":78,"in_reply_to":"5a890539_0865c844","updated":"2014-11-20 12:15:37.000000000","message":"Okay.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":75,"context_line":"  * irmc_auth: authentication method (basic or digest) for iRMC. The"},{"line_number":76,"context_line":"    default value is basic."},{"line_number":77,"context_line":"  * irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":78,"context_line":"    is 60 seconds."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Scalability impact"},{"line_number":81,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_a4576f01","line":78,"in_reply_to":"5a890539_38059214","updated":"2014-11-20 12:15:37.000000000","message":"Okay.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"0dc80c91f9eb215b89e11ecca8ff411a62081000","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Dependencies"},{"line_number":115,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":116,"context_line":"* This feature requires at least BX S4 or RX S7 generation of FUJITSU"},{"line_number":117,"context_line":"  PRIMERGY servers."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Testing"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_285a8483","line":117,"updated":"2014-11-19 23:30:05.000000000","message":"Will an external library be used to interface with iRMC devices? Where can a user get this library from and what license is that library available under?","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Dependencies"},{"line_number":115,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":116,"context_line":"* This feature requires at least BX S4 or RX S7 generation of FUJITSU"},{"line_number":117,"context_line":"  PRIMERGY servers."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Testing"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_44b40b35","line":117,"in_reply_to":"5a890539_285a8483","updated":"2014-11-20 12:15:37.000000000","message":"iRMC driver doesn\u0027t use python external library such as iLO and DRAC.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4dcd534836a3a5b576a1fcddee86ec0a6a71175f","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Testing"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":121,"context_line":"TBD"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":124,"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":3,"id":"5a890539_7830fa07","line":121,"updated":"2014-11-19 10:38:13.000000000","message":"At least you have to use unit tests :)","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"0dc80c91f9eb215b89e11ecca8ff411a62081000","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Testing"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":121,"context_line":"TBD"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":124,"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":3,"id":"5a890539_485fd072","line":121,"updated":"2014-11-19 23:30:05.000000000","message":"Unit tests are required.\n\nwhether or not you (fujitsu) plans to provide third-party CI should be stated here.","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Testing"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":121,"context_line":"TBD"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":124,"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":3,"id":"5a890539_842fd33d","line":121,"in_reply_to":"5a890539_485fd072","updated":"2014-11-20 12:15:37.000000000","message":"Okay. I\u0027ll add the following statements:\n\n* Unit Tests\n* Fujitsu plans Third-party CI Tests","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"b136f3d4f3b1082aba86ea499ed37988d6b21bb5","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Testing"},{"line_number":120,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":121,"context_line":"TBD"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":124,"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":3,"id":"5a890539_a47ccf73","line":121,"in_reply_to":"5a890539_7830fa07","updated":"2014-11-20 12:15:37.000000000","message":"Okay","commit_id":"118bd970351702be6e750eab01d614aa446fd4cb"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"b488a3df824e464e287aeadf0334a7bf0ab1801c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"protocol. This driver uses python `requests \u003chttps://pypi.python.org/pypi/requests/\u003e`_ library."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"iRMC doesn\u0027t use external vendor library such as `OpenWSMAN library \u003chttp://openwsman.github.io/\u003e`_"},{"line_number":34,"context_line":"of DRAC or `proliantutils library \u003chttps://github.com/hpproliant/proliantutils\u003e`_ of iLO."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_ec9ab012","line":34,"updated":"2014-11-25 16:32:11.000000000","message":"This is a nit. I think if the text was reorganized a bit, it would read/be easier to understand.\n\neg,  iRMC doesn\u0027t use external vendor libraries... Instead, it makes HTTP/HTTPS POST requests to the SCCI client. The python requests library will be used for sending these HTTP requests.","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"27211efd6d193f8e7bcbc44dafbfdf3aae7dc410","unresolved":false,"context_lines":[{"line_number":31,"context_line":"protocol. This driver uses python `requests \u003chttps://pypi.python.org/pypi/requests/\u003e`_ library."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"iRMC doesn\u0027t use external vendor library such as `OpenWSMAN library \u003chttp://openwsman.github.io/\u003e`_"},{"line_number":34,"context_line":"of DRAC or `proliantutils library \u003chttps://github.com/hpproliant/proliantutils\u003e`_ of iLO."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_16f53982","line":34,"in_reply_to":"5a890539_ec9ab012","updated":"2014-11-26 05:08:27.000000000","message":"Ruby, \n\nThanks for the comment. I\u0027ll fix that.","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"b488a3df824e464e287aeadf0334a7bf0ab1801c","unresolved":false,"context_lines":[{"line_number":70,"context_line":"The following driver_info fields are required:"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"* irmc_address: hostname or IP of iRMC"},{"line_number":73,"context_line":"* irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":74,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":75,"context_line":"* irmc_password: password for irmc_username"},{"line_number":76,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_4c22e452","line":73,"updated":"2014-11-25 16:32:11.000000000","message":"not sure if there is a typo here. 443 or 433 or both?","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":5805,"name":"Chris Krelle","email":"nobodycam@gmail.com","username":"nobodycam"},"change_message_id":"7ede9714869ceada4c8bfee38f9149835d9b936f","unresolved":false,"context_lines":[{"line_number":70,"context_line":"The following driver_info fields are required:"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"* irmc_address: hostname or IP of iRMC"},{"line_number":73,"context_line":"* irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":74,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":75,"context_line":"* irmc_password: password for irmc_username"},{"line_number":76,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_8f6afe7e","line":73,"in_reply_to":"5a890539_4c22e452","updated":"2014-11-25 16:40:34.000000000","message":"oh good catch Ruby +1 for fixing this.","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"27211efd6d193f8e7bcbc44dafbfdf3aae7dc410","unresolved":false,"context_lines":[{"line_number":70,"context_line":"The following driver_info fields are required:"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"* irmc_address: hostname or IP of iRMC"},{"line_number":73,"context_line":"* irmc_port: port of iRMC (80 or 443). The default value is 433."},{"line_number":74,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":75,"context_line":"* irmc_password: password for irmc_username"},{"line_number":76,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_364795bd","line":73,"in_reply_to":"5a890539_8f6afe7e","updated":"2014-11-26 05:08:27.000000000","message":"Yes, it\u0027s typo :-), I\u0027ll fix that.","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"b488a3df824e464e287aeadf0334a7bf0ab1801c","unresolved":false,"context_lines":[{"line_number":79,"context_line":"The following parameter is added into the ironic configuration file"},{"line_number":80,"context_line":"which is typically located at /etc/ironic/ironic.conf."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"* irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":83,"context_line":"  is 60 seconds."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Scalability impact"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_0c0b7c7d","line":82,"updated":"2014-11-25 16:32:11.000000000","message":"we have a \u0027client_timeout\u0027 in the [ilo] section of the config file, so I\u0027m guessing we\u0027ll want this in an [irmc] section, and named \u0027client_timeout\u0027 too (for consistency).","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"27211efd6d193f8e7bcbc44dafbfdf3aae7dc410","unresolved":false,"context_lines":[{"line_number":79,"context_line":"The following parameter is added into the ironic configuration file"},{"line_number":80,"context_line":"which is typically located at /etc/ironic/ironic.conf."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"* irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":83,"context_line":"  is 60 seconds."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Scalability impact"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_d6365150","line":82,"in_reply_to":"5a890539_0c0b7c7d","updated":"2014-11-26 05:08:27.000000000","message":"Okay, I\u0027ll fix that.","commit_id":"e73f919c3825e082ecf183a63998e24928c6b1ae"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"d14af12d0c6942d0f3045d9c030948dc9056fc99","unresolved":false,"context_lines":[{"line_number":84,"context_line":"the ironic configuration file which is typically located at"},{"line_number":85,"context_line":"/etc/ironic/ironic.conf."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"* irmc_client_timeout: timeout for SCCI operations. The default value"},{"line_number":88,"context_line":"  is 60 seconds."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Scalability impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a890539_baf516cc","line":87,"updated":"2014-11-26 14:57:28.000000000","message":"I think \u0027irmc_client_timeout\u0027 -\u003e \u0027client_timeout\u0027. The \u0027irmc\u0027 isn\u0027t needed since it will be in the [irmc] section.","commit_id":"60ee550a9d9351a08c6e5f820390e5a5d862a32e"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"353a7536c087c80e37045bd7104f2182c5eb3b0b","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Other end user impact"},{"line_number":77,"context_line":"---------------------"},{"line_number":78,"context_line":"The following driver_info fields are required:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* irmc_address: hostname or IP of iRMC"},{"line_number":81,"context_line":"* irmc_port: port of iRMC (80 or 443). The default value is 443."}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a890539_24c9f179","line":78,"updated":"2014-11-27 09:43:28.000000000","message":"nit: not all of these are required","commit_id":"bcfe1478daf165a1635c9aff54e74992ae82de59"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"353a7536c087c80e37045bd7104f2182c5eb3b0b","unresolved":false,"context_lines":[{"line_number":82,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":83,"context_line":"* irmc_password: password for irmc_username"},{"line_number":84,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"},{"line_number":85,"context_line":"  iRMC. The default value is basic."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":88,"context_line":"the ironic configuration file which is typically located at"}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a890539_a4924149","line":85,"updated":"2014-11-27 09:43:28.000000000","message":"the default value should be in the configuration file. Imagine the whole deployment uses \u0027digest\u0027. The deployer will have to specify it for every node, while it could be set in ironic.conf","commit_id":"bcfe1478daf165a1635c9aff54e74992ae82de59"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"f06ee349442e4bc95ce23a9c5c9697e8fcd1af50","unresolved":false,"context_lines":[{"line_number":83,"context_line":"* irmc_password: password for irmc_username"},{"line_number":84,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"},{"line_number":85,"context_line":"  iRMC. The default value is basic."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":88,"context_line":"the ironic configuration file which is typically located at"},{"line_number":89,"context_line":"/etc/ironic/ironic.conf."}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a890539_af0f7f87","line":86,"updated":"2014-11-27 07:13:50.000000000","message":"As Dmitry said before, is it necessary to put auth_method in driver_info?\nServers in one cloud should be consistent, right?\nI am thinking to have the same auth_method and even same port?\nThe only different is the address and password.\nI am not admin, just some thought.","commit_id":"bcfe1478daf165a1635c9aff54e74992ae82de59"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"fb820a90064ed3ad19a6d7b58afa4a000919e160","unresolved":false,"context_lines":[{"line_number":83,"context_line":"* irmc_password: password for irmc_username"},{"line_number":84,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"},{"line_number":85,"context_line":"  iRMC. The default value is basic."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":88,"context_line":"the ironic configuration file which is typically located at"},{"line_number":89,"context_line":"/etc/ironic/ironic.conf."}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a890539_87ce27b6","line":86,"in_reply_to":"5a890539_35ecc2f3","updated":"2014-11-27 09:59:45.000000000","message":"Discuss with Tamura on IRC and get agreement on this.","commit_id":"bcfe1478daf165a1635c9aff54e74992ae82de59"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"11ae5a4c383ef17b55b8c39051a947188635f925","unresolved":false,"context_lines":[{"line_number":83,"context_line":"* irmc_password: password for irmc_username"},{"line_number":84,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"},{"line_number":85,"context_line":"  iRMC. The default value is basic."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":88,"context_line":"the ironic configuration file which is typically located at"},{"line_number":89,"context_line":"/etc/ironic/ironic.conf."}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a890539_35ecc2f3","line":86,"in_reply_to":"5a890539_6a1bc580","updated":"2014-11-27 08:35:12.000000000","message":"What I mean is to put these options in ironic.conf.\nYes. We can discuss in IRC.","commit_id":"bcfe1478daf165a1635c9aff54e74992ae82de59"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"2f1768ce17487de819fcdea7b0740742e24e17a5","unresolved":false,"context_lines":[{"line_number":83,"context_line":"* irmc_password: password for irmc_username"},{"line_number":84,"context_line":"* irmc_auth_method: authentication method (basic or digest) for"},{"line_number":85,"context_line":"  iRMC. The default value is basic."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":88,"context_line":"the ironic configuration file which is typically located at"},{"line_number":89,"context_line":"/etc/ironic/ironic.conf."}],"source_content_type":"text/x-rst","patch_set":6,"id":"5a890539_6a1bc580","line":86,"in_reply_to":"5a890539_af0f7f87","updated":"2014-11-27 08:18:33.000000000","message":"Hi Tan,\n\nThank you for the comment.\nAt least, deployer doesn\u0027t have to care about authentication method, because the default is \"basic\" on https.\nAnd we can discuss in IRC futher if you like. I\u0027m looking forward to seeing you in IRC.\n\nBest regards,\nNaohiro","commit_id":"bcfe1478daf165a1635c9aff54e74992ae82de59"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"0f1fdfc4c2da4c7985a3eddb0eb4ad62cca529a0","unresolved":false,"context_lines":[{"line_number":81,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":82,"context_line":"* irmc_password: password for irmc_username"},{"line_number":83,"context_line":"* client_port: port number iRMC (optional)"},{"line_number":84,"context_line":"* client_auth_method: authentication method for iRMC (optional)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":87,"context_line":"the ironic configuration file which is typically located at"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_3a3d812d","line":84,"updated":"2014-11-27 15:27:07.000000000","message":"these should start with irmc_ for the sake of consistency.","commit_id":"ef3f3c505ce0f4c3faa136b255de5fede034fc3e"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e0b6dfef059c28d9c4a4f23e1d5c22f335f99cb4","unresolved":false,"context_lines":[{"line_number":81,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":82,"context_line":"* irmc_password: password for irmc_username"},{"line_number":83,"context_line":"* client_port: port number iRMC (optional)"},{"line_number":84,"context_line":"* client_auth_method: authentication method for iRMC (optional)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":87,"context_line":"the ironic configuration file which is typically located at"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_79f705d5","line":84,"in_reply_to":"5a890539_29529468","updated":"2014-11-28 13:00:57.000000000","message":"Tan,\n\nThank you for the comment.\nI\u0027ll take your proposal.\n\nBest regards,\nNaohiro","commit_id":"ef3f3c505ce0f4c3faa136b255de5fede034fc3e"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"d3ed5aabfe145311c1c3759ed7f02d76d82d453c","unresolved":false,"context_lines":[{"line_number":81,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":82,"context_line":"* irmc_password: password for irmc_username"},{"line_number":83,"context_line":"* client_port: port number iRMC (optional)"},{"line_number":84,"context_line":"* client_auth_method: authentication method for iRMC (optional)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":87,"context_line":"the ironic configuration file which is typically located at"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_b3b997cd","line":84,"in_reply_to":"5a890539_3a3d812d","updated":"2014-11-28 01:01:02.000000000","message":"Hi Dmitry,\n\nThank you for the comment.\nI understand your point. Here we need to think about not only consistency but also redundancy.  If we use  \"irmc_auth_method\",  [irmc] section has \"irmc_auth_method\" which causes \"irmc_\" redundancy.\nRuby commented on this redundancy issue in the Patch Set 5.\nDo you have any idea how we can satisfy both?\n\nBest regards,\n--\nNaohiro","commit_id":"ef3f3c505ce0f4c3faa136b255de5fede034fc3e"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"e0b6dfef059c28d9c4a4f23e1d5c22f335f99cb4","unresolved":false,"context_lines":[{"line_number":81,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":82,"context_line":"* irmc_password: password for irmc_username"},{"line_number":83,"context_line":"* client_port: port number iRMC (optional)"},{"line_number":84,"context_line":"* client_auth_method: authentication method for iRMC (optional)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":87,"context_line":"the ironic configuration file which is typically located at"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_4e3fddd6","line":84,"in_reply_to":"5a890539_b3b997cd","updated":"2014-11-28 13:00:57.000000000","message":"Dmitry,\n\nWe agreed in IRC as follows, and I\u0027ll take the second one which Tan proposed.\n\n\u003cdtantsur\u003e naohirot, oh well... actually it\u0027s better to have either [irmc]client_port and irmc_client_port OR [irmc]port and irmc_port for the sake of consistency\n\n\n1. driver_info\nchange \"client_port\" to \"irmc_port\"\nchange \"client_auth_method\" to \"irmc_auth_method\"\n\n\n2. [rimc] section in config file\nchange \"client_port\" to \"port\"\nchange \"client_auth_method\" to \"auth_method\"\n\n\nBest regards,\nNaohiro","commit_id":"ef3f3c505ce0f4c3faa136b255de5fede034fc3e"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"b9fd1fe3fe7a463405ebe512a3e655b2d53cf5e8","unresolved":false,"context_lines":[{"line_number":81,"context_line":"* irmc_username: username for iRMC with administrator privileges"},{"line_number":82,"context_line":"* irmc_password: password for irmc_username"},{"line_number":83,"context_line":"* client_port: port number iRMC (optional)"},{"line_number":84,"context_line":"* client_auth_method: authentication method for iRMC (optional)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"The following parameter is added into newly created [irmc] section in"},{"line_number":87,"context_line":"the ironic configuration file which is typically located at"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5a890539_29529468","line":84,"in_reply_to":"5a890539_b3b997cd","updated":"2014-11-28 05:15:21.000000000","message":"What about use irmc_port in driver_info\nand port in conf.\nyou can load the value in parse_driver_info like:\ndriver_info[\u0027irmc_port\u0027] \u003d info.get(\u0027irmc_port\u0027, CONF.irmc.port)","commit_id":"ef3f3c505ce0f4c3faa136b255de5fede034fc3e"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c89656cc84f8decb15f216a334e0931eeb22d359","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Other end user impact"},{"line_number":77,"context_line":"---------------------"},{"line_number":78,"context_line":"The following driver_info fields are required:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* irmc_address: hostname or IP of iRMC"},{"line_number":81,"context_line":"* irmc_username: username for iRMC with administrator privileges"}],"source_content_type":"text/x-rst","patch_set":9,"id":"5a890539_3c49ca8e","line":78,"updated":"2014-12-01 14:26:36.000000000","message":"nit: you say \"fields are required\" and then mention optional fields :)","commit_id":"ff24d84ccc0ea5be45d9ca9fde833aa0d49d9bf6"}]}