)]}'
{"specs/liberty/approved/robustify_evacuate.rst":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"6e98b8a9e9d17ced534d4c92e33ae3b2abfe455c","unresolved":false,"context_lines":[{"line_number":132,"context_line":"which can be used to hide migrations from the user. Since things like"},{"line_number":133,"context_line":"live-migrations were not previously exposed in the os-migrations API,"},{"line_number":134,"context_line":"admins may have a need to prevent administrative migrations from being"},{"line_number":135,"context_line":"shown to the user."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"REST API impact"},{"line_number":138,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3afb71cf_64af6a2c","line":135,"updated":"2015-06-11 13:56:44.000000000","message":"Emm...this should be handled by policy enforcement.\n\nAs live-migration is admin-api default in policy.json, but is is configurable. When operator make live-migration available for normal user, then this migration isn\u0027t administrative migrations anymore.\n\nSo we can add \u0027project_id\u0027 field in the database, we can get project_id from request\u0027s context. Then we can now this migration is requested by admin or normal user. and we need update os-migrations api\u0027s policy rule to rule:admin_or_owner.","commit_id":"58c8b2c1e25d45d3419c922dd0a019d7b7f3c476"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"489e472b6524fbd935e84bb164654f1d5f9dc83f","unresolved":false,"context_lines":[{"line_number":132,"context_line":"which can be used to hide migrations from the user. Since things like"},{"line_number":133,"context_line":"live-migrations were not previously exposed in the os-migrations API,"},{"line_number":134,"context_line":"admins may have a need to prevent administrative migrations from being"},{"line_number":135,"context_line":"shown to the user."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"REST API impact"},{"line_number":138,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa32b979_ab8e2ece","line":135,"in_reply_to":"3afb71cf_64af6a2c","updated":"2015-06-22 14:41:05.000000000","message":"No, the desire is for most people to show them to the user, independent of who has the permission to actually initiate live migrations. The field is present so that operators that don\u0027t wish to show a specific migration to the user can hide it.\n\nThis part of the spec is already merged in the database, FWIW:\n\nhttps://review.openstack.org/#/c/181109/\n\nThis spec amendment is making sure the spec matches what we actually landed.","commit_id":"58c8b2c1e25d45d3419c922dd0a019d7b7f3c476"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"8ce62121465689b689f2e0a1e34e8ddefb1920bb","unresolved":false,"context_lines":[{"line_number":132,"context_line":"which can be used to hide migrations from the user. Since things like"},{"line_number":133,"context_line":"live-migrations were not previously exposed in the os-migrations API,"},{"line_number":134,"context_line":"admins may have a need to prevent administrative migrations from being"},{"line_number":135,"context_line":"shown to the user."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"REST API impact"},{"line_number":138,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"ba3cc151_d0721349","line":135,"in_reply_to":"fa32b979_ab8e2ece","updated":"2015-07-01 08:25:00.000000000","message":"I must miss something.  I didn\u0027t find any code set hidden https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/robustify-evacuate,n,z","commit_id":"58c8b2c1e25d45d3419c922dd0a019d7b7f3c476"}]}