)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"e373ce0d9200fe059d8a0e0c15d5bc14cfb79470","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dmitry Tantsur \u003cdivius.inside@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-11-08 17:57:59 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Clean up release notes from the upcoming release"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iacdc3fdcf1fc221995963f8900a7e8ee06c9900d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f4b6375_44e4e3e0","line":7,"updated":"2017-11-08 18:27:16.000000000","message":"nit s/from/for/ :)","commit_id":"01cd2f89ff4ca121eb96ce39eed214b2c9ab2d86"}],"releasenotes/notes/keystoneauth-adapter-opts-ca4f68f568e6cf6f.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    To facilitate automatic discovery of services from the keystone catalog,"},{"line_number":5,"context_line":"    the configuration file sections for service clients may include these"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_4e0c04ec","line":2,"updated":"2017-11-08 16:41:47.000000000","message":"Yes, features makes sense. Wondering if we want to also put it in upgrade. Although we only support service_catalog. I think the idea is we want folks to make these change when they upgrade.\n\nWell, I don\u0027t feel strongly about it so...","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    To facilitate automatic discovery of services from the keystone catalog,"},{"line_number":5,"context_line":"    the configuration file sections for service clients may include these"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_4ee404e1","line":2,"in_reply_to":"3f4b6375_4e0c04ec","updated":"2017-11-08 16:56:01.000000000","message":"Yeah, I understand the idea, but then we need to reword the whole thing from what can be done to what should be done..","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    The ``region_name`` option defaults to ``None`` and must be explicitly set"},{"line_number":21,"context_line":"    for multi-regional setup for endpoint discovery to succeed."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    Currently only the ``[service_catalog]`` section supports these options."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"deprecations:"},{"line_number":26,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_ce5114dc","line":23,"updated":"2017-11-08 16:41:47.000000000","message":"May be a question for Pavlo, but nothing here mentions the \u0027endpoint_override\u0027 option. Should we? It is mentioned below (L29).","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    The ``region_name`` option defaults to ``None`` and must be explicitly set"},{"line_number":21,"context_line":"    for multi-regional setup for endpoint discovery to succeed."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    Currently only the ``[service_catalog]`` section supports these options."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"deprecations:"},{"line_number":26,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_ce4594b6","line":23,"in_reply_to":"3f4b6375_ce5114dc","updated":"2017-11-08 16:56:01.000000000","message":"Good question. I think I\u0027ll add it.","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    Configuration option ``[conductor]api_url`` is deprecated"},{"line_number":28,"context_line":"    and will be removed in the Rocky release. Instead, use the"},{"line_number":29,"context_line":"    ``[service_catalog]endpoint_override`` configuration option to set"},{"line_number":30,"context_line":"    specific ironic API address if automatic discovery of the Bare Metal API"},{"line_number":31,"context_line":"    endpoint from keystone catalog is not desired."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    This new option defaults to ``None`` and must be set explicitly if needed."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_2ead489b","line":30,"updated":"2017-11-08 16:41:47.000000000","message":"do we want to also s/ironic API address/ironic-api service address/ ?","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    Configuration option ``[conductor]api_url`` is deprecated"},{"line_number":28,"context_line":"    and will be removed in the Rocky release. Instead, use the"},{"line_number":29,"context_line":"    ``[service_catalog]endpoint_override`` configuration option to set"},{"line_number":30,"context_line":"    specific ironic API address if automatic discovery of the Bare Metal API"},{"line_number":31,"context_line":"    endpoint from keystone catalog is not desired."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    This new option defaults to ``None`` and must be set explicitly if needed."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_0ed84cee","line":30,"in_reply_to":"3f4b6375_2ead489b","updated":"2017-11-08 16:56:01.000000000","message":"Done","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"70e75095f730deca4be53525bc079d0ba549a4d7","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    configuration options: ``service_type``, ``service_name``,"},{"line_number":7,"context_line":"    ``valid_interfaces``, ``region_name`` and other keystoneauth options."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    These options together must uniquely specify an endpoint for a service"},{"line_number":10,"context_line":"    registered in the service catalog. Alternatively, the ``endpoint_override``"},{"line_number":11,"context_line":"    option should be used to specify the endpoint."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1f485f77_8972fd10","line":9,"range":{"start_line":9,"start_character":32,"end_line":9,"end_character":48},"updated":"2017-11-09 09:23:12.000000000","message":"correspond to a unique?","commit_id":"01cd2f89ff4ca121eb96ce39eed214b2c9ab2d86"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"70e75095f730deca4be53525bc079d0ba549a4d7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    These options together must uniquely specify an endpoint for a service"},{"line_number":10,"context_line":"    registered in the service catalog. Alternatively, the ``endpoint_override``"},{"line_number":11,"context_line":"    option should be used to specify the endpoint."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"    Consult the `keystoneauth library documentation"},{"line_number":14,"context_line":"    \u003chttps://docs.openstack.org/keystoneauth/latest/\u003e`_ for a full list of"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1f485f77_09604d32","line":11,"range":{"start_line":11,"start_character":11,"end_line":11,"end_character":17},"updated":"2017-11-09 09:23:12.000000000","message":"can?","commit_id":"01cd2f89ff4ca121eb96ce39eed214b2c9ab2d86"}],"releasenotes/notes/new_capabilities-5241619c4b46a460.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for the following boolean capabilities keys to"},{"line_number":5,"context_line":"    the ``ilo`` inspect interface:"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    * ``sriov_enabled``"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_4e25245c","line":4,"updated":"2017-11-08 16:41:47.000000000","message":"it is \u0027Boolean\u0027, not \u0027boolean\u0027 (e.g. https://en.wikipedia.org/wiki/Boolean)","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for the following boolean capabilities keys to"},{"line_number":5,"context_line":"    the ``ilo`` inspect interface:"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    * ``sriov_enabled``"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_eefdf03b","line":4,"in_reply_to":"3f4b6375_4e25245c","updated":"2017-11-08 16:56:01.000000000","message":"oh, interesting to know. I\u0027m pretty sure we mostly use it lower-cased","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"}],"releasenotes/notes/oneview-timeout-power-88b02a9cd45876d7.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for ``timeout`` parameter when powering on/off an Ironic node"},{"line_number":5,"context_line":"    managed by the ``oneview`` hardware type."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_ee1bb019","line":4,"updated":"2017-11-08 16:41:47.000000000","message":"nit, maybe s/Ironic/bare metal/\n\nor s/Ironic/ironic/ :)","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds support for ``timeout`` parameter when powering on/off an Ironic node"},{"line_number":5,"context_line":"    managed by the ``oneview`` hardware type."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_4e9c040e","line":4,"in_reply_to":"3f4b6375_ee1bb019","updated":"2017-11-08 16:56:01.000000000","message":"Done","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"}],"releasenotes/notes/reboot-do-not-power-off-if-already-1452256167d40009.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes a problem when rebooting a node using the ``ipmitool`` power"},{"line_number":5,"context_line":"    interface could cause a deploy to fail. Now it no longer tries to power off"},{"line_number":6,"context_line":"    the nodes that are already off, because some BMCs will error in such case."},{"line_number":7,"context_line":"    See `bug 1718794 \u003chttps://bugs.launchpad.net/ironic/+bug/1718794\u003e`_"},{"line_number":8,"context_line":"    for details."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_2edf883b","line":6,"updated":"2017-11-08 16:41:47.000000000","message":"nit s/the//\n\nnit s/in such case/in these cases/.","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes a problem when rebooting a node using the ``ipmitool`` power"},{"line_number":5,"context_line":"    interface could cause a deploy to fail. Now it no longer tries to power off"},{"line_number":6,"context_line":"    the nodes that are already off, because some BMCs will error in such case."},{"line_number":7,"context_line":"    See `bug 1718794 \u003chttps://bugs.launchpad.net/ironic/+bug/1718794\u003e`_"},{"line_number":8,"context_line":"    for details."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_ae8c98d7","line":6,"in_reply_to":"3f4b6375_2edf883b","updated":"2017-11-08 16:56:01.000000000","message":"Done","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"}],"releasenotes/notes/rebuild-configdrive-f52479fd55b0f5ce.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Starting with the Bare Metal API version 1.35, it is possible to provide"},{"line_number":5,"context_line":"    a configdrive when rebuilding a node."},{"line_number":6,"context_line":"fixes:"},{"line_number":7,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_ceed94f1","line":4,"updated":"2017-11-08 16:41:47.000000000","message":"nit, I suspect we don\u0027t need s/the//\n\n(because I think \u0027with version 1.35\u0027 seems better than \u0027with the version 1.35\u0027. but a nit cuz I\u0027m not totally sure!)","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Starting with the Bare Metal API version 1.35, it is possible to provide"},{"line_number":5,"context_line":"    a configdrive when rebuilding a node."},{"line_number":6,"context_line":"fixes:"},{"line_number":7,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_2ebae86d","line":4,"in_reply_to":"3f4b6375_ceed94f1","updated":"2017-11-08 16:56:01.000000000","message":"not sure either.. I always use \"the\", but I\u0027m not the best source of truth on English :) Leave or remove?","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"}],"releasenotes/notes/remove-python-oneviewclient-f47e22322f7d151f.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"7140ec1a2882eff9d14d08d08f803b4541c4face","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``oneview`` drivers now use ``hpOneView`` and"},{"line_number":5,"context_line":"    ``python-ilorest-library`` libraries to communicate with OneView"},{"line_number":6,"context_line":"    appliances. The ``python-oneviewclient`` library is no longer needed."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_aec37857","line":6,"updated":"2017-11-08 16:41:47.000000000","message":"maybe s/needed/used/.\n\nor is it used still, just not needed?","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bab932bb9df03c11b3aef0cccaf412ac00a6a6c2","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``oneview`` drivers now use ``hpOneView`` and"},{"line_number":5,"context_line":"    ``python-ilorest-library`` libraries to communicate with OneView"},{"line_number":6,"context_line":"    appliances. The ``python-oneviewclient`` library is no longer needed."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_aea5b849","line":6,"in_reply_to":"3f4b6375_aec37857","updated":"2017-11-08 16:56:01.000000000","message":"mmm.. I wanted to express something like \"now you can delete it\". but thinking more of it, \"used\" is better.","commit_id":"7a1f4604fb7c2bd67e1cc330d603e95e162a199a"}]}