)]}'
{"doc/source/enforcer.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"f3f22cddf04c9587548d40addb4986fc025a626e","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    category \u003d \"Enforcer\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"def get_proxy_methods():"},{"line_number":28,"context_line":"    \"\"\"Return a set of public names on all proxies\"\"\""},{"line_number":29,"context_line":"    names \u003d [\"openstack.baremetal.v1._proxy\","},{"line_number":30,"context_line":"             \"openstack.clustering.v1._proxy\","}],"source_content_type":"text/x-python","patch_set":6,"id":"1f485f77_76239e57","line":27,"updated":"2017-11-15 13:26:44.000000000","message":"I know the following name changes weren\u0027t initiated here, but with them being all listed together in this particular change I think it makes the comments about them make more sense.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"7240ecb0c9bedfabe65b66012826e1e1c87c7270","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    category \u003d \"Enforcer\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"def get_proxy_methods():"},{"line_number":28,"context_line":"    \"\"\"Return a set of public names on all proxies\"\"\""},{"line_number":29,"context_line":"    names \u003d [\"openstack.baremetal.v1._proxy\","},{"line_number":30,"context_line":"             \"openstack.clustering.v1._proxy\","}],"source_content_type":"text/x-python","patch_set":6,"id":"1f485f77_19549b20","line":27,"in_reply_to":"1f485f77_76239e57","updated":"2017-11-15 14:11:02.000000000","message":"I hear you.\n\nIn general these name changes are driven not on their own merit but to align with the Service Types Authority https://service-types.openstack.org/. That block-storage and object-store are different is ... sad ... but those are the names of the services and that\u0027s manifesting in a bunch of ways now. For instance:\n\n  https://developer.openstack.org/api-ref/object-store/\n  https://developer.openstack.org/api-ref/block-storage/\n\nand the spec/guideline about consuming the catalog:\n\n  http://specs.openstack.org/openstack/api-wg/guidelines/consuming-catalog.html#user-request\n\nthat we\u0027ve pointing the SDK authors from all the languages at.\n\nI don\u0027t agree with all the names, and I certainly don\u0027t agree with the inconsistency (I agree with you on the specifics of each of the concerns) The suggestion was actually made to change swift\u0027s official type to object-storage - but ultimately the churn was deemed less important than the consistency.\n\nIt\u0027s also important because the config values are based on the official name. So if someone wants to set the version of the API to be used for cinder, they need to set OS_BLOCK_STORAGE_API_VERSION (or OS_VOLUME_STORAGE_API_VERSION) - or block_storage_api_version in their clouds.yaml file. Since clouds.yaml and OS_ variables should be recognized consistently across the language ecosystem, that config code also needs to be driven by the STA.\n\nI think it\u0027s important, if the general interface being presented is having an object per OpenStack service using service names and not code names, that we use the official names.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"f3f22cddf04c9587548d40addb4986fc025a626e","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"def get_proxy_methods():"},{"line_number":28,"context_line":"    \"\"\"Return a set of public names on all proxies\"\"\""},{"line_number":29,"context_line":"    names \u003d [\"openstack.baremetal.v1._proxy\","},{"line_number":30,"context_line":"             \"openstack.clustering.v1._proxy\","},{"line_number":31,"context_line":"             \"openstack.block_storage.v2._proxy\","},{"line_number":32,"context_line":"             \"openstack.compute.v2._proxy\","}],"source_content_type":"text/x-python","patch_set":6,"id":"1f485f77_b657d6ca","line":29,"range":{"start_line":29,"start_character":24,"end_line":29,"end_character":33},"updated":"2017-11-15 13:26:44.000000000","message":"This was originally bare_metal to stick with other two-word formats and the general convention around the codebase to split two-word names like that. It was also like that because I had mostly seen it as \"bare metal\" in a lot of places both out in the world and on various openstack.org sites, though it\u0027s also shown in a lot of places as the one-word version.\n\nKeep the change if you think it makes it better, but it was one of those small consistency changes to bring a consistent look.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"f3f22cddf04c9587548d40addb4986fc025a626e","unresolved":false,"context_lines":[{"line_number":27,"context_line":"def get_proxy_methods():"},{"line_number":28,"context_line":"    \"\"\"Return a set of public names on all proxies\"\"\""},{"line_number":29,"context_line":"    names \u003d [\"openstack.baremetal.v1._proxy\","},{"line_number":30,"context_line":"             \"openstack.clustering.v1._proxy\","},{"line_number":31,"context_line":"             \"openstack.block_storage.v2._proxy\","},{"line_number":32,"context_line":"             \"openstack.compute.v2._proxy\","},{"line_number":33,"context_line":"             \"openstack.database.v1._proxy\","}],"source_content_type":"text/x-python","patch_set":6,"id":"1f485f77_56c42291","line":30,"range":{"start_line":30,"start_character":24,"end_line":30,"end_character":34},"updated":"2017-11-15 13:26:44.000000000","message":"This was originally cluster in order to match things like message and network. We got the original list of names out of some description yaml file and named everything else that came afterward using the same convention.\n\nA bunch of other similar names don\u0027t really work as their -ing version. Compute and image don\u0027t work that way, though message and network would. Consistency won out there.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"f3f22cddf04c9587548d40addb4986fc025a626e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    \"\"\"Return a set of public names on all proxies\"\"\""},{"line_number":29,"context_line":"    names \u003d [\"openstack.baremetal.v1._proxy\","},{"line_number":30,"context_line":"             \"openstack.clustering.v1._proxy\","},{"line_number":31,"context_line":"             \"openstack.block_storage.v2._proxy\","},{"line_number":32,"context_line":"             \"openstack.compute.v2._proxy\","},{"line_number":33,"context_line":"             \"openstack.database.v1._proxy\","},{"line_number":34,"context_line":"             \"openstack.identity.v2._proxy\","}],"source_content_type":"text/x-python","patch_set":6,"id":"1f485f77_b33808ee","line":31,"range":{"start_line":31,"start_character":24,"end_line":31,"end_character":37},"updated":"2017-11-15 13:26:44.000000000","message":"This is one of those changes where I think consistency within the SDK matters more than consistency with an outside source, even if it\u0027s the project itself. This now makes for having object_store and block_storage.\n\nIf this needs to be block_storage then I\u0027d prefer object_store changes with it.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"f3f22cddf04c9587548d40addb4986fc025a626e","unresolved":false,"context_lines":[{"line_number":43,"context_line":"             \"openstack.network.v2._proxy\","},{"line_number":44,"context_line":"             \"openstack.object_store.v1._proxy\","},{"line_number":45,"context_line":"             \"openstack.orchestration.v1._proxy\","},{"line_number":46,"context_line":"             \"openstack.meter.v2._proxy\","},{"line_number":47,"context_line":"             \"openstack.meter.alarm.v2._proxy\","},{"line_number":48,"context_line":"             \"openstack.workflow.v2._proxy\"]"},{"line_number":49,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1f485f77_360086ab","line":46,"range":{"start_line":46,"start_character":24,"end_line":46,"end_character":29},"updated":"2017-11-15 13:26:44.000000000","message":"I never really liked the telemetry name, but it still seems correct, I think?","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"}],"doc/source/users/guides/connect.rst":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"f3f22cddf04c9587548d40addb4986fc025a626e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Create Connection"},{"line_number":12,"context_line":"-----------------"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The :class:`~openstack.connection.Connection` is a context for a connection"},{"line_number":15,"context_line":"to an OpenStack cloud."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":".. literalinclude:: ../examples/connect.py"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1f485f77_16434ab1","line":14,"range":{"start_line":14,"start_character":51,"end_line":14,"end_character":58},"updated":"2017-11-15 13:26:44.000000000","message":"What exactly is `context` here and will a developer trying to find a place to enter their cloud creds into something know what is is?\n\nI also think we should use `openstack.connect` here instead if we\u0027re going to have that.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"7240ecb0c9bedfabe65b66012826e1e1c87c7270","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Create Connection"},{"line_number":12,"context_line":"-----------------"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The :class:`~openstack.connection.Connection` is a context for a connection"},{"line_number":15,"context_line":"to an OpenStack cloud."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":".. literalinclude:: ../examples/connect.py"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1f485f77_392cf7aa","line":14,"range":{"start_line":14,"start_character":51,"end_line":14,"end_character":58},"in_reply_to":"1f485f77_16434ab1","updated":"2017-11-15 14:11:02.000000000","message":"That\u0027s a great question, and yah - I think it\u0027s a terrible word. Will fix.\n\nAlso - totally agree, great point on using openstack.connect. Will fix that too.","commit_id":"f6621499410c84a5cfc52b34c2e628c3b74ec528"}]}
