)]}'
{"nova/conf/placement.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"be09b39636288fdbb5404e14fa4895d764b08cbb","unresolved":false,"context_lines":[{"line_number":29,"context_line":"* Any string representing region name"},{"line_number":30,"context_line":"\"\"\"),"},{"line_number":31,"context_line":"    cfg.StrOpt(\u0027os_interface\u0027,"},{"line_number":32,"context_line":"               default\u003d\"public\","},{"line_number":33,"context_line":"               help\u003d\"\"\""},{"line_number":34,"context_line":"Endpoint interface for this node. This is used when picking the URL in the"},{"line_number":35,"context_line":"service catalog."}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_cf08162b","line":32,"range":{"start_line":32,"start_character":24,"end_line":32,"end_character":30},"updated":"2017-01-31 10:35:27.000000000","message":"I wonder if we should actually default to admin or private? It should be an admin only API.","commit_id":"58d053abbef3354c40ee36565fb57f0a94878140"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9406b9167eb26ef903480c218485b931e6b1a901","unresolved":false,"context_lines":[{"line_number":30,"context_line":"\"\"\"),"},{"line_number":31,"context_line":"    cfg.StrOpt(\u0027os_interface\u0027,"},{"line_number":32,"context_line":"               default\u003d\"public\","},{"line_number":33,"context_line":"               help\u003d\"\"\""},{"line_number":34,"context_line":"Endpoint interface for this node. This is used when picking the URL in the"},{"line_number":35,"context_line":"service catalog."},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_0f24de5b","line":33,"updated":"2017-01-31 10:42:25.000000000","message":"Ideally we would restrict this to only allow public, admin or internal, using:\n\n    choices\u003d[\"public\",\"admin\",\"internal\"]","commit_id":"58d053abbef3354c40ee36565fb57f0a94878140"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"07e8b52b7945273b0a03c44f0c2729fe39b909a9","unresolved":false,"context_lines":[{"line_number":33,"context_line":"               choices\u003d[\"public\", \"admin\", \"internal\"],"},{"line_number":34,"context_line":"               help\u003d\"\"\""},{"line_number":35,"context_line":"Endpoint interface for this node. This is used when picking the URL in the"},{"line_number":36,"context_line":"service catalog."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Possible values:"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3a461143_daf0cbcb","line":36,"updated":"2017-01-31 13:35:24.000000000","message":"Though the placement API is currently restricted by built in policy to be admin only, the long term intent is that at least some information within it could be access from dashboard like interfaces (such as horizon), presumably restricted in various ways.\n\nSo I\u0027m not sure if this is the right precedent to set.\n\nIf we\u0027ve already been using public as the default, can we introduce a new default?","commit_id":"aa090ad5f29a619677bdbbcbdb2a08ce8a1fca70"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"f50809ca33a90aabeeb28f9cba6bb39d642aa43e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"Endpoint interface for this node. This is used when picking the URL in the"},{"line_number":36,"context_line":"service catalog."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Possible values:"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* admin"},{"line_number":41,"context_line":"* internal"},{"line_number":42,"context_line":"* public"},{"line_number":43,"context_line":"\"\"\")"},{"line_number":44,"context_line":"]"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a461143_73198f22","line":42,"range":{"start_line":38,"start_character":0,"end_line":42,"end_character":8},"updated":"2017-02-01 15:43:59.000000000","message":"Nit: there is no need for this, because its covered in choices already.","commit_id":"870f5b70019f9fc9b81596260bab339823b375bf"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"aef8c46733359a100590a18e03cd01dc091ed85d","unresolved":false,"context_lines":[{"line_number":30,"context_line":"\"\"\"),"},{"line_number":31,"context_line":"    cfg.StrOpt(\u0027os_interface\u0027,"},{"line_number":32,"context_line":"               default\u003d\"public\","},{"line_number":33,"context_line":"               choices\u003d[\"public\", \"admin\", \"internal\"],"},{"line_number":34,"context_line":"               help\u003d\"\"\""},{"line_number":35,"context_line":"Endpoint interface for this node. This is used when picking the URL in the"},{"line_number":36,"context_line":"service catalog."}],"source_content_type":"text/x-python","patch_set":6,"id":"fa31d9ce_badc4174","line":33,"updated":"2017-02-13 19:37:07.000000000","message":"This choices list should be removed, this is actually a completely free form field from keystone.","commit_id":"1257d338db480ba237f6878cc478d072b28f7695"}],"releasenotes/notes/placement-api-endpoint-interface-set-29af8b9400ce7775.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4a43641470487a7351610097026a32d49f5dfb32","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - The placement API can be set to connect to a specific"},{"line_number":4,"context_line":"    keystone endpoint interface using the ``os_interface``"},{"line_number":5,"context_line":"    option in the ``[placement]`` section inside ``nova.conf``."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fa31d9ce_5d109f59","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"updated":"2017-02-13 20:28:50.000000000","message":"This shouldn\u0027t be \u0027features\u0027 probably, it\u0027s not really a new feature. I\u0027d just make it \u0027other\u0027.","commit_id":"1257d338db480ba237f6878cc478d072b28f7695"}]}