)]}'
{"specs/rocky/approved/alloc-candidates-member-of.rst":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ab09649051d7a5214666ce80a3cd922137dce14f","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  \"foo\" (either \"tenant_foo_old_computes\" or"},{"line_number":50,"context_line":"  \"tenant_foo_new_computes\") and are also in AZ \"US Chicago\"."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"The desired nodes the resource providers that are in in the union of"},{"line_number":53,"context_line":"all the aggregates that define suitable computes assigned to the"},{"line_number":54,"context_line":"tenant by the operator, which intersect with the aggregate that"},{"line_number":55,"context_line":"defines the AZ requested by the user."}],"source_content_type":"text/x-rst","patch_set":1,"id":"df7087c5_8b5db448","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":21},"updated":"2018-03-22 18:48:10.000000000","message":"nodes are the ?","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"a0850fb6d34d3c78b4afe26155834bf46092de8a","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  \"foo\" (either \"tenant_foo_old_computes\" or"},{"line_number":50,"context_line":"  \"tenant_foo_new_computes\") and are also in AZ \"US Chicago\"."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"The desired nodes the resource providers that are in in the union of"},{"line_number":53,"context_line":"all the aggregates that define suitable computes assigned to the"},{"line_number":54,"context_line":"tenant by the operator, which intersect with the aggregate that"},{"line_number":55,"context_line":"defines the AZ requested by the user."}],"source_content_type":"text/x-rst","patch_set":1,"id":"df7087c5_a6e46f0e","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":21},"in_reply_to":"df7087c5_8b5db448","updated":"2018-03-22 18:56:09.000000000","message":"Yes.","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ab09649051d7a5214666ce80a3cd922137dce14f","unresolved":false,"context_lines":[{"line_number":52,"context_line":"The desired nodes the resource providers that are in in the union of"},{"line_number":53,"context_line":"all the aggregates that define suitable computes assigned to the"},{"line_number":54,"context_line":"tenant by the operator, which intersect with the aggregate that"},{"line_number":55,"context_line":"defines the AZ requested by the user."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Proposed change"},{"line_number":58,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"df7087c5_e60a671e","line":55,"updated":"2018-03-22 18:48:10.000000000","message":"Thanks, with this writeup I feel like I understand the use case. (I didn\u0027t before - I was just taking it on faith :)","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ab09649051d7a5214666ce80a3cd922137dce14f","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":" * Allow multiple `member_of` query parameters and AND the results from each"},{"line_number":79,"context_line":" * Use an additional puncutation mark to delineate multiple sets. Currently a"},{"line_number":80,"context_line":"   comma is used as the OR operator, so some other mark (such as a colon) must"},{"line_number":81,"context_line":"   be used to signify the boundary of the AND."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"df7087c5_0b20c4ab","line":80,"range":{"start_line":80,"start_character":67,"end_line":80,"end_character":72},"updated":"2018-03-22 18:48:10.000000000","message":"-1 for colon, which is used for \u0027in:\u0027.  (That wouldn\u0027t *stop* us from using it to delineate ANDed sets, but it would look confusing.)","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"a0850fb6d34d3c78b4afe26155834bf46092de8a","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":" * Allow multiple `member_of` query parameters and AND the results from each"},{"line_number":79,"context_line":" * Use an additional puncutation mark to delineate multiple sets. Currently a"},{"line_number":80,"context_line":"   comma is used as the OR operator, so some other mark (such as a colon) must"},{"line_number":81,"context_line":"   be used to signify the boundary of the AND."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"df7087c5_66ead7d8","line":80,"range":{"start_line":80,"start_character":67,"end_line":80,"end_character":72},"in_reply_to":"df7087c5_0b20c4ab","updated":"2018-03-22 18:56:09.000000000","message":"Oh right I forgot.","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ab09649051d7a5214666ce80a3cd922137dce14f","unresolved":false,"context_lines":[{"line_number":79,"context_line":" * Use an additional puncutation mark to delineate multiple sets. Currently a"},{"line_number":80,"context_line":"   comma is used as the OR operator, so some other mark (such as a colon) must"},{"line_number":81,"context_line":"   be used to signify the boundary of the AND."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Alternatives"},{"line_number":84,"context_line":"------------"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"df7087c5_c6ef231b","line":82,"updated":"2018-03-22 18:48:10.000000000","message":"Note to reviewers on why this isn\u0027t a trivial decision.\n\n- The concept of multiple \u0027member_of\u0027 query parameters is totally supported by HTTP and the tooling.  But all of the other paths in the placement API today disallow it, instead using comma to denote logical AND.  So this option would be a deviation from the \"norm\".\n- But using punctuation is problematic because comma is already taken for member_of; we would have to choose a different delimiter to denote logical AND.  So this option would also be a deviation from the \"norm\".","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"8affc5b9cb35f859523cdaac9265f29992b500f9","unresolved":false,"context_lines":[{"line_number":79,"context_line":" * Use an additional puncutation mark to delineate multiple sets. Currently a"},{"line_number":80,"context_line":"   comma is used as the OR operator, so some other mark (such as a colon) must"},{"line_number":81,"context_line":"   be used to signify the boundary of the AND."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Alternatives"},{"line_number":84,"context_line":"------------"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"bf659307_6fa20e9d","line":82,"in_reply_to":"df7087c5_c6ef231b","updated":"2018-03-26 12:24:08.000000000","message":"member_of\u003din:foo,bar,in:snap,bar\n\nis probably parseable. Not really suggesting it, just pointing it out.","commit_id":"caa4d750ecc26bb101b7e236184a8daf7658bf95"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"679cb1d895b74e84487d9fc962c99206f00c641f","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  \"foo\" (either \"tenant_foo_old_computes\" or"},{"line_number":50,"context_line":"  \"tenant_foo_new_computes\") and are also in AZ \"US Chicago\"."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"The desired nodes are the resource providers that are in in the union"},{"line_number":53,"context_line":"of all the aggregates that define suitable computes assigned to the"},{"line_number":54,"context_line":"tenant by the operator, which intersect with the aggregate that"},{"line_number":55,"context_line":"defines the AZ requested by the user."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_7b8aa91b","line":52,"range":{"start_line":52,"start_character":54,"end_line":52,"end_character":56},"updated":"2018-03-27 14:53:36.000000000","message":"strike","commit_id":"7f660b9dd318577757ce275a34e7ebbc03e4104b"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"679cb1d895b74e84487d9fc962c99206f00c641f","unresolved":false,"context_lines":[{"line_number":83,"context_line":""},{"line_number":84,"context_line":"  Candidate resource providers should be in either agg1 or agg2, but definitely"},{"line_number":85,"context_line":"  in agg3."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Alternatives"},{"line_number":88,"context_line":"------------"},{"line_number":89,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_5bea45ac","line":86,"updated":"2018-03-27 14:53:36.000000000","message":"Will this feature be ported back to GET /rps as well?","commit_id":"7f660b9dd318577757ce275a34e7ebbc03e4104b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b7c57ee3ac3c3c997a477a12ca83e3ab5f1d47de","unresolved":false,"context_lines":[{"line_number":77,"context_line":"be considered by the underlying implementation to be ANDed together. In other"},{"line_number":78,"context_line":"words, the following query string::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  \u0026member_of\u003din:agg1,agg2\u0026member_of\u003dagg3"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"would translate logically to:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf659307_2b2fdca1","line":80,"updated":"2018-03-28 15:35:35.000000000","message":"This is a new microversion, https://blueprints.launchpad.net/nova/+spec/alloc-candidates-member-of is already marked complete with version 1.21, so we can\u0027t just amend the spec after the fact and change the behavior of the API.","commit_id":"d1a3e8daa5c4974050f4654112b2b85680a4e021"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"0ce6b83697406483854eef07953b09be6293dce9","unresolved":false,"context_lines":[{"line_number":77,"context_line":"be considered by the underlying implementation to be ANDed together. In other"},{"line_number":78,"context_line":"words, the following query string::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  \u0026member_of\u003din:agg1,agg2\u0026member_of\u003dagg3"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"would translate logically to:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf659307_2e1daa65","line":80,"in_reply_to":"bf659307_2b2fdca1","updated":"2018-03-28 15:37:27.000000000","message":"Of course, that\u0027s implied no?","commit_id":"d1a3e8daa5c4974050f4654112b2b85680a4e021"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e1d52710f36ea0af0b19bc1cdf571759c54eeef7","unresolved":false,"context_lines":[{"line_number":77,"context_line":"be considered by the underlying implementation to be ANDed together. In other"},{"line_number":78,"context_line":"words, the following query string::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  \u0026member_of\u003din:agg1,agg2\u0026member_of\u003dagg3"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"would translate logically to:"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf659307_6e416239","line":80,"in_reply_to":"bf659307_2e1daa65","updated":"2018-03-28 15:38:57.000000000","message":"You\u0027d be surprised how often that is debated, so I never know.","commit_id":"d1a3e8daa5c4974050f4654112b2b85680a4e021"}]}