)]}'
{"id":"openstack%2Fnova-specs~313607","triplet_id":"openstack%2Fnova-specs~master~I3c78f70f2a174ea220d12a145927e80146c8bf70","project":"openstack/nova-specs","branch":"master","topic":"hypervisor-availability","hashtags":[],"change_id":"I3c78f70f2a174ea220d12a145927e80146c8bf70","subject":"nova-api can check hypervisor availability based on boot params","status":"ABANDONED","created":"2016-05-06 15:34:21.000000000","updated":"2016-10-28 13:52:56.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f07b5b40935525f34d701b41f65f321377114085","_number":313607,"virtual_id_number":313607,"owner":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2016-05-06 20:15:54.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2016-05-17 13:51:07.000000000","_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},{"value":-1,"date":"2016-05-17 02:17:06.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},{"date":"2016-05-16 19:03:34.000000000","_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"date":"2016-05-16 19:03:48.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-05-06 20:15:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-05-16 19:03:34.000000000","updated_by":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"reviewer":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"state":"REVIEWER"},{"updated":"2016-05-16 19:03:48.000000000","updated_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"reviewer":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"state":"REVIEWER"}],"messages":[{"id":"eefd096ace5e6b2a6b5d6b2a5f85af5df3c383a6","author":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"date":"2016-05-06 15:34:21.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"40d968b2375897a0f6321198a9070fb964dd2dab","author":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"date":"2016-05-06 15:35:52.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"45889e3f6ee9596693a3547aa39f836067a5a824","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-06 15:55:49.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-nova-specs-docs http://docs-draft.openstack.org/07/313607/2/check/gate-nova-specs-docs/fad521c//doc/build/html/ : SUCCESS in 8m 42s\n- gate-nova-specs-pep8 http://logs.openstack.org/07/313607/2/check/gate-nova-specs-pep8/641f26d/ : SUCCESS in 2m 24s\n- gate-nova-specs-python27 http://logs.openstack.org/07/313607/2/check/gate-nova-specs-python27/2acd54c/ : FAILURE in 2m 55s","accounts_in_message":[],"_revision_number":2},{"id":"51094623d5a1978d9b5df7785e6d240f938466d5","author":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"date":"2016-05-06 20:07:55.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"1ed7708dbeef1d33913d6ed85288625bdc67b7b9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-06 20:15:54.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-nova-specs-docs http://docs-draft.openstack.org/07/313607/3/check/gate-nova-specs-docs/ba9bbeb//doc/build/html/ : SUCCESS in 4m 17s\n- gate-nova-specs-pep8 http://logs.openstack.org/07/313607/3/check/gate-nova-specs-pep8/317b33d/ : SUCCESS in 1m 40s\n- gate-nova-specs-python27 http://logs.openstack.org/07/313607/3/check/gate-nova-specs-python27/efee468/ : FAILURE in 2m 25s","accounts_in_message":[],"_revision_number":3},{"id":"8a76c057528fa0ea35fb4a9f77d4cce31a698179","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2016-05-17 02:17:06.000000000","message":"Patch Set 3: Code-Review-1\n\nThis is not really a direction I would like to go. Basically, what you are asking for is a valid use case -- i.e. for request of X amount of Y resources, find out how many resource providers can satisfy said request.\n\nHowever, there are a number of problems with this proposal. Firstly, we are attempting to get away from using flavors as the giant coupled mess that they currently are. We want to decompose flavors into their respective quantitative and qualitative components as soon as possible, and adding any new API pieces that refer to the flavor is not ideal.\n\nSecondly, you are proposing a REST API that isn\u0027t RESTful :) The \"schedule-test\" is not a resource endpoint. It is an action of sorts that doesn\u0027t seem to belong where you put it.\n\nI think once we get the placement API split out you should be able to call something along the lines of:\n\n GET /resource-providers?resources\u003dVCPU:10;DISK_GB:200;MEMORY_MB\u003d32768\u0026capabilities\u003dhw:x86_cpu_inst_set_ext:avx2\n\nAs a privileged user and get a list of resource providers that (at that particular point in time) could have satisfied that request for resources and capabilities.\n\nBest,\n\n-jay","accounts_in_message":[],"_revision_number":3},{"id":"aa05f730e6166ef8abb6174e4f5b5a0cb7d09a45","author":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"date":"2016-05-17 13:51:07.000000000","message":"Patch Set 3:\n\n\u003e This is not really a direction I would like to go. Basically, what\n \u003e you are asking for is a valid use case -- i.e. for request of X\n \u003e amount of Y resources, find out how many resource providers can\n \u003e satisfy said request.\n \u003e \n \u003e However, there are a number of problems with this proposal.\n \u003e Firstly, we are attempting to get away from using flavors as the\n \u003e giant coupled mess that they currently are. We want to decompose\n \u003e flavors into their respective quantitative and qualitative\n \u003e components as soon as possible, and adding any new API pieces that\n \u003e refer to the flavor is not ideal.\n \u003e \n \u003e Secondly, you are proposing a REST API that isn\u0027t RESTful :) The\n \u003e \"schedule-test\" is not a resource endpoint. It is an action of\n \u003e sorts that doesn\u0027t seem to belong where you put it.\n \u003e \n \u003e I think once we get the placement API split out you should be able\n \u003e to call something along the lines of:\n \u003e \n \u003e GET /resource-providers?resources\u003dVCPU:10;DISK_GB:200;MEMORY_MB\u003d32768\u0026capabilities\u003dhw:x86_cpu_inst_set_ext:avx2\n \u003e \n \u003e As a privileged user and get a list of resource providers that (at\n \u003e that particular point in time) could have satisfied that request\n \u003e for resources and capabilities.\n \u003e \n \u003e Best,\n \u003e \n \u003e -jay\n\nHi Jay, thanks a lot for the review, if you don\u0027t mind i\u0027d like to discuss it:\n\nI was not aware of the new resources provider concept, so that is interesting, but regarding \"referencing a flavor\" in the API is not the intent of this blueprints, it\u0027s only my use case.   Since this would take the same parameters as nova create, if it gets decoupled from the flavor, this new endpoint will aswell.\n\nRegarding the REST Api, i tend to agree it was not ideal, and i should have included a sample of how the parameters should be passed.   A POST to /servers can contain quite the body and i\u0027m not sure how that will correctly render in the limited querystring of a GET.   So i don\u0027t really know how to fit this in a REST api :-/","accounts_in_message":[],"_revision_number":3},{"id":"3a214cef994a10bb7f452d499b8e01f60a095c85","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2016-10-28 13:52:56.000000000","message":"Abandoned\n\nPer discussion at the summit (https://etherpad.openstack.org/p/ocata-nova-summit-meetup), we are abandoning Newton specs that haven\u0027t merged. You may restore this patch and move it to Ocata if you\u0027d like.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"b5429bddf5f68ecaad40f933118e568aadededd7","revisions":{"b3b153345c214119101a40c971893cd7905da585":{"kind":"REWORK","_number":1,"created":"2016-05-06 15:34:21.000000000","uploader":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"ref":"refs/changes/07/313607/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/07/313607/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/1"}}},"commit":{"parents":[{"commit":"338915a2d70640e0181779b71bdf54b578ef4494","subject":"Merge \"Add spec for lower case metadata keys only\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/338915a2d70640e0181779b71bdf54b578ef4494"}]}],"author":{"name":"Martin Roy","email":"mroy@internap.com","date":"2016-05-06 15:34:04.000000000","tz":-240},"committer":{"name":"Martin Roy","email":"mroy@internap.com","date":"2016-05-06 15:34:04.000000000","tz":-240},"subject":"nova-api can check hypervisor availability based on boot parameters","message":"nova-api can check hypervisor availability based on boot parameters\n\nAdd an endpoint to determine which hypervisors fit a given set of parameters\nsimilar to the nova create.\n\nAPIImpact\nDocImpact\n\nChange-Id: I3c78f70f2a174ea220d12a145927e80146c8bf70\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/b3b153345c214119101a40c971893cd7905da585"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/b3b153345c214119101a40c971893cd7905da585"}]},"branch":"refs/heads/master"},"f588edaea618ce715afbb08ef448f2baa55dd04b":{"kind":"REWORK","_number":2,"created":"2016-05-06 15:35:52.000000000","uploader":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"ref":"refs/changes/07/313607/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/07/313607/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/2"}}},"commit":{"parents":[{"commit":"338915a2d70640e0181779b71bdf54b578ef4494","subject":"Merge \"Add spec for lower case metadata keys only\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/338915a2d70640e0181779b71bdf54b578ef4494"}]}],"author":{"name":"Martin Roy","email":"mroy@internap.com","date":"2016-05-06 15:34:04.000000000","tz":-240},"committer":{"name":"Martin Roy","email":"mroy@internap.com","date":"2016-05-06 15:35:29.000000000","tz":-240},"subject":"nova-api can check hypervisor availability based on boot params","message":"nova-api can check hypervisor availability based on boot params\n\nAdd an endpoint to determine which hypervisors fit a given set of parameters\nsimilar to the nova create.\n\nAPIImpact\nDocImpact\n\nChange-Id: I3c78f70f2a174ea220d12a145927e80146c8bf70\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/f588edaea618ce715afbb08ef448f2baa55dd04b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/f588edaea618ce715afbb08ef448f2baa55dd04b"}]},"branch":"refs/heads/master"},"b5429bddf5f68ecaad40f933118e568aadededd7":{"kind":"REWORK","_number":3,"created":"2016-05-06 20:07:55.000000000","uploader":{"_account_id":16934,"name":"Martin Roy","email":"lindycoder@gmail.com","username":"lindycoder"},"ref":"refs/changes/07/313607/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/07/313607/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/07/313607/3"}}},"commit":{"parents":[{"commit":"338915a2d70640e0181779b71bdf54b578ef4494","subject":"Merge \"Add spec for lower case metadata keys only\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/338915a2d70640e0181779b71bdf54b578ef4494"}]}],"author":{"name":"Martin Roy","email":"mroy@internap.com","date":"2016-05-06 20:07:33.000000000","tz":-240},"committer":{"name":"Martin Roy","email":"mroy@internap.com","date":"2016-05-06 20:07:33.000000000","tz":-240},"subject":"nova-api can check hypervisor availability based on boot params","message":"nova-api can check hypervisor availability based on boot params\n\nAdd an endpoint to determine which hypervisors fit a given set of parameters\nsimilar to the nova create.\n\nAPIImpact\nDocImpact\n\nChange-Id: I3c78f70f2a174ea220d12a145927e80146c8bf70\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/b5429bddf5f68ecaad40f933118e568aadededd7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/b5429bddf5f68ecaad40f933118e568aadededd7"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}