)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"12b6e7e82203ceddfc5a8530837a544e84d3da2a","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"/resource_providers/{rp_uuid}/allocations has been available since"},{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_1849aa31","line":12,"range":{"start_line":11,"start_character":35,"end_line":12,"end_character":14},"updated":"2017-08-31 22:13:17.000000000","message":"You can\u0027t do that. If you are changing the content of a response, it needs a *new* microversion.\n\nAlso - double spaces after a period? Eww.","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"22238593dc9329aee3b63df89556cabe5d5737d0","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"/resource_providers/{rp_uuid}/allocations has been available since"},{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_b69f3a64","line":12,"range":{"start_line":11,"start_character":35,"end_line":12,"end_character":14},"in_reply_to":"7f287b81_1849aa31","updated":"2017-09-01 12:42:14.000000000","message":"Hm, even for an unbounded list of same-type elements?\n\nOkay, well, this hardly seems worth a whole new microversion; so mebbe we fold it into the next one.","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"e345fec7c6b468ff6512baaad911918454339f5b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"/resource_providers/{rp_uuid}/allocations has been available since"},{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f4b6375_6d2bfbea","line":12,"in_reply_to":"7f287b81_9108c8d9","updated":"2017-10-20 20:40:17.000000000","message":"Done","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"8189cfb8f7b17907079edc86244275de9ffaec84","unresolved":false,"context_lines":[{"line_number":9,"context_line":"/resource_providers/{rp_uuid}/allocations has been available since"},{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_9108c8d9","line":12,"in_reply_to":"7f287b81_b69f3a64","updated":"2017-09-01 13:16:01.000000000","message":"Yeah, the strict interpretation is: you make a change, you make a version. Them\u0027s the breaks.","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"72c9bfe7e34eb1c63cb7d098e3caf582e33590a2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I6a1d320ce914926791d5f45e89bf4c601a6b10a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_833e3bd8","line":13,"updated":"2017-09-01 14:05:32.000000000","message":"Were you also going to mention that we also did this without a microversion in the past?\n\nhttps://review.openstack.org/#/c/468923/","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"e345fec7c6b468ff6512baaad911918454339f5b","unresolved":false,"context_lines":[{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I6a1d320ce914926791d5f45e89bf4c601a6b10a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f4b6375_2d350347","line":13,"in_reply_to":"7f287b81_0673794f","updated":"2017-10-20 20:40:17.000000000","message":"Undone","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"42d558bb57213ca827691957afa2d8edc2b6c41c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"microversion 1.0 [1], but wasn\u0027t listed in the \"links\" section of the"},{"line_number":11,"context_line":"GET /resource_providers response.  This change adds the link for all"},{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I6a1d320ce914926791d5f45e89bf4c601a6b10a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_0673794f","line":13,"in_reply_to":"7f287b81_833e3bd8","updated":"2017-09-01 14:21:38.000000000","message":"Done","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"72c9bfe7e34eb1c63cb7d098e3caf582e33590a2","unresolved":false,"context_lines":[{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I6a1d320ce914926791d5f45e89bf4c601a6b10a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_435bb353","line":15,"updated":"2017-09-01 14:05:32.000000000","message":"Closes-Bug: #1714275 ?","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"42d558bb57213ca827691957afa2d8edc2b6c41c","unresolved":false,"context_lines":[{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I6a1d320ce914926791d5f45e89bf4c601a6b10a0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_c692114c","line":15,"in_reply_to":"7f287b81_435bb353","updated":"2017-09-01 14:21:38.000000000","message":"Done","commit_id":"1e7880e696633ef9ba7d45ae93655da11da060f1"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5429672b37f8292494aa5338399fdb808fc6b4ff","unresolved":false,"context_lines":[{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Strictly speaking, this is an API change worthy of a new microversion"},{"line_number":15,"context_line":"itself.  However, it\u0027s eminently compatible; and the change which"},{"line_number":16,"context_line":"split which links show up depending on the microversion [2] broke that"},{"line_number":17,"context_line":"rule similarly."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f287b81_126cd39a","line":15,"range":{"start_line":15,"start_character":18,"end_line":15,"end_character":43},"updated":"2017-09-01 16:48:35.000000000","message":"This cannot be a reason why we can avoid a new microversion bump because that is a basic part of microversions.\n\nhttps://docs.openstack.org/nova/latest/contributor/microversions.html#when-do-i-need-a-new-microversion\n\n * the Response\n   the list of attributes and data structures returned\n   Example: adding a new attribute ‘locked’: True/False to the output of servers/{ID}","commit_id":"b27abbe2c67d13ee959203f9742668f4f46dfe3e"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"b606691ae528b5b62826c1cce72be76312275407","unresolved":false,"context_lines":[{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Strictly speaking, this is an API change worthy of a new microversion"},{"line_number":15,"context_line":"itself.  However, it\u0027s eminently compatible; and the change which"},{"line_number":16,"context_line":"split which links show up depending on the microversion [2] broke that"},{"line_number":17,"context_line":"rule similarly."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f4e5783_1530e4e7","line":15,"range":{"start_line":15,"start_character":18,"end_line":15,"end_character":43},"in_reply_to":"5f4e5783_70229d02","updated":"2017-10-13 12:48:59.000000000","message":"More relevant, perhaps, is that nobody is doing hypertextuality with these links.","commit_id":"b27abbe2c67d13ee959203f9742668f4f46dfe3e"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"eb3b02776f79b267cf9d5b9c4a1d5584439e0a69","unresolved":false,"context_lines":[{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Strictly speaking, this is an API change worthy of a new microversion"},{"line_number":15,"context_line":"itself.  However, it\u0027s eminently compatible; and the change which"},{"line_number":16,"context_line":"split which links show up depending on the microversion [2] broke that"},{"line_number":17,"context_line":"rule similarly."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f287b81_75c5285b","line":15,"range":{"start_line":15,"start_character":18,"end_line":15,"end_character":43},"in_reply_to":"7f287b81_126cd39a","updated":"2017-09-04 05:13:23.000000000","message":"I\u0027m not sure if there is anyone using the placement outside the Nova now. \n\nI think when we fix this patch https://review.openstack.org/#/c/468923, it is ok since the Nova is the only user of placement yet, the placement is really in the early stage.","commit_id":"b27abbe2c67d13ee959203f9742668f4f46dfe3e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5c27fb99a60a7f3b438c6663df0c4dfb6bfcee40","unresolved":false,"context_lines":[{"line_number":12,"context_line":"microversions."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Strictly speaking, this is an API change worthy of a new microversion"},{"line_number":15,"context_line":"itself.  However, it\u0027s eminently compatible; and the change which"},{"line_number":16,"context_line":"split which links show up depending on the microversion [2] broke that"},{"line_number":17,"context_line":"rule similarly."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f4e5783_70229d02","line":15,"range":{"start_line":15,"start_character":18,"end_line":15,"end_character":43},"in_reply_to":"7f287b81_75c5285b","updated":"2017-10-12 21:26:35.000000000","message":"Turns out neutron has had placement client code since ocata, so there are other things outside of nova using placement.","commit_id":"b27abbe2c67d13ee959203f9742668f4f46dfe3e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5429672b37f8292494aa5338399fdb808fc6b4ff","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Strictly speaking, this is an API change worthy of a new microversion"},{"line_number":15,"context_line":"itself.  However, it\u0027s eminently compatible; and the change which"},{"line_number":16,"context_line":"split which links show up depending on the microversion [2] broke that"},{"line_number":17,"context_line":"rule similarly."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[1] https://review.openstack.org/#/c/366789/"},{"line_number":20,"context_line":"[2] https://review.openstack.org/#/c/468923/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f287b81_329eaf54","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":15},"updated":"2017-09-01 16:48:35.000000000","message":"I guess the reason of this exception is the placement API, right?","commit_id":"b27abbe2c67d13ee959203f9742668f4f46dfe3e"}],"nova/api/openstack/placement/handlers/resource_provider.py":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"8189cfb8f7b17907079edc86244275de9ffaec84","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    if want_version \u003e\u003d (1, 1):"},{"line_number":90,"context_line":"        rel_types.append(\u0027aggregates\u0027)"},{"line_number":91,"context_line":"    if want_version \u003e\u003d (1, 6):"},{"line_number":92,"context_line":"        rel_types.append(\u0027traits\u0027)"},{"line_number":93,"context_line":"    for rel in rel_types:"},{"line_number":94,"context_line":"        links.append({\u0027rel\u0027: rel, \u0027href\u0027: \u0027%s/%s\u0027 % (url, rel)})"},{"line_number":95,"context_line":"    return links"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_d1c41062","side":"PARENT","line":92,"updated":"2017-09-01 13:16:01.000000000","message":"So yeah, if/when we do this, the way to do it is another clause like this one.\n\nHowever, I suspect we were not this strict when adding the aggregates and traits links, given that I think these lines were added well after version 1.1.\n\nIf that\u0027s the case we might have a case for some wiggle room here, if we want it. We might be able to lawyer it if we really want to.","commit_id":"2a4ca8bd6aa40ccd26300feaef4267aa71f69abf"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"f904b62ae24846b7f6643c01c8685355bcf26e47","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    if want_version \u003e\u003d (1, 1):"},{"line_number":90,"context_line":"        rel_types.append(\u0027aggregates\u0027)"},{"line_number":91,"context_line":"    if want_version \u003e\u003d (1, 6):"},{"line_number":92,"context_line":"        rel_types.append(\u0027traits\u0027)"},{"line_number":93,"context_line":"    for rel in rel_types:"},{"line_number":94,"context_line":"        links.append({\u0027rel\u0027: rel, \u0027href\u0027: \u0027%s/%s\u0027 % (url, rel)})"},{"line_number":95,"context_line":"    return links"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_91f32874","side":"PARENT","line":92,"in_reply_to":"7f287b81_d1c41062","updated":"2017-09-01 13:22:29.000000000","message":"Yup, these clauses were added all at once via https://review.openstack.org/#/c/468923/ - clearly at or after microversion 1.6, thereby violating \"the rules\" in an arguably more egregious fashion than we\u0027re suggesting here, because they took something away.","commit_id":"2a4ca8bd6aa40ccd26300feaef4267aa71f69abf"}],"nova/api/openstack/placement/microversion.py":[{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"6114efaaccd15e2b14115d2f975040c7f5447923","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            # PUT /allocations"},{"line_number":49,"context_line":"    \u00271.9\u0027,  # Adds GET /usages"},{"line_number":50,"context_line":"    \u00271.10\u0027,  # Adds GET /allocation_candidates resource endpoint"},{"line_number":51,"context_line":"    \u00271,11\u0027,  # Adds \u0027allocations\u0027 link to the GET /resource_providers response"},{"line_number":52,"context_line":"]"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3f4b6375_621c0c12","line":51,"range":{"start_line":51,"start_character":5,"end_line":51,"end_character":9},"updated":"2017-10-23 09:46:51.000000000","message":"TypeError: invalid version string: 1,11; invalid literal for int() with base 10: \u00271,11\u0027\n\ns/1,11/1.11/\n\n:)","commit_id":"88e9c9390be6ea8798c7445f867c059653ba05e2"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"84538565f341dda54c59bd3af5e98650f8683716","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            # PUT /allocations"},{"line_number":49,"context_line":"    \u00271.9\u0027,  # Adds GET /usages"},{"line_number":50,"context_line":"    \u00271.10\u0027,  # Adds GET /allocation_candidates resource endpoint"},{"line_number":51,"context_line":"    \u00271,11\u0027,  # Adds \u0027allocations\u0027 link to the GET /resource_providers response"},{"line_number":52,"context_line":"]"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3f4b6375_8b5ba352","line":51,"range":{"start_line":51,"start_character":5,"end_line":51,"end_character":9},"in_reply_to":"3f4b6375_621c0c12","updated":"2017-10-23 11:56:02.000000000","message":"Nice, thank you!\n\nI admit I have trouble parsing the test results in this format, and I\u0027ve been assuming POST_FAILURE was always a Zuul thing, not related to code.","commit_id":"88e9c9390be6ea8798c7445f867c059653ba05e2"}],"nova/tests/functional/api/openstack/placement/gabbits/resource-provider-links.yaml":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":65,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":66,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":67,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":68,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"- name: get rp 1.6"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_932bcd2e","line":67,"updated":"2017-10-23 17:27:18.000000000","message":"won\u0027t be here until 1.11","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":65,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":66,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":67,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":68,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"- name: get rp 1.6"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_b7f0f2fe","line":67,"in_reply_to":"3f4b6375_932bcd2e","updated":"2017-10-23 19:03:59.000000000","message":"Done","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":68,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"- name: get rp 1.6"},{"line_number":71,"context_line":"  desc: traits added in 1.6"},{"line_number":72,"context_line":"  GET: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":73,"context_line":"  request_headers:"},{"line_number":74,"context_line":"      openstack-api-version: placement 1.6"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_53e2f502","line":71,"updated":"2017-10-23 17:27:18.000000000","message":"should probably add an explicit \"allocations added in 1.11\" test, even though the latest one above gets it.","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":68,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"- name: get rp 1.6"},{"line_number":71,"context_line":"  desc: traits added in 1.6"},{"line_number":72,"context_line":"  GET: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":73,"context_line":"  request_headers:"},{"line_number":74,"context_line":"      openstack-api-version: placement 1.6"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_97f5f6ef","line":71,"in_reply_to":"3f4b6375_53e2f502","updated":"2017-10-23 19:03:59.000000000","message":"Done","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":77,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":78,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":79,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":80,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":81,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":82,"context_line":"      $.links[?rel \u003d \"traits\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/traits"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_73229154","line":80,"updated":"2017-10-23 17:27:18.000000000","message":"won\u0027t be here until 1.11","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":77,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":78,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":79,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":80,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":81,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":82,"context_line":"      $.links[?rel \u003d \"traits\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/traits"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_77defa6d","line":80,"in_reply_to":"3f4b6375_73229154","updated":"2017-10-23 19:03:59.000000000","message":"Done","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":91,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":92,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":93,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":94,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":95,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":96,"context_line":"      $.links[?rel \u003d \"traits\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/traits"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_531d550d","line":94,"updated":"2017-10-23 17:27:18.000000000","message":"won\u0027t be here until 1.11","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":91,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":92,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":93,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":94,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":95,"context_line":"      $.links[?rel \u003d \"aggregates\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/aggregates"},{"line_number":96,"context_line":"      $.links[?rel \u003d \"traits\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/traits"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_57dbfe7a","line":94,"in_reply_to":"3f4b6375_531d550d","updated":"2017-10-23 19:03:59.000000000","message":"Done","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"}],"nova/tests/functional/api/openstack/placement/gabbits/resource-provider.yaml":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":84,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":85,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":86,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":87,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"- name: get resource provider works with no accept"},{"line_number":90,"context_line":"  GET: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_53cb1571","line":87,"updated":"2017-10-23 17:27:18.000000000","message":"this will fail unless you set the microversion explicitly. The entire file is defaulting to minimum microversion.","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":84,"context_line":"      $.links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":85,"context_line":"      $.links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":86,"context_line":"      $.links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":87,"context_line":"      $.links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"- name: get resource provider works with no accept"},{"line_number":90,"context_line":"  GET: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_57c41e91","line":87,"in_reply_to":"3f4b6375_53cb1571","updated":"2017-10-23 19:03:59.000000000","message":"Thanks, I can\u0027t get it through my head that \u0027unspecified\u0027 doesn\u0027t mean \u0027latest\u0027.","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":111,"context_line":"      $.resource_providers[0].links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":112,"context_line":"      $.resource_providers[0].links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":113,"context_line":"      $.resource_providers[0].links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":114,"context_line":"      $.resource_providers[0].links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"- name: filter out all resource providers by name"},{"line_number":117,"context_line":"  GET: /resource_providers?name\u003dflubblebubble"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_f3a1e1aa","line":114,"updated":"2017-10-23 17:27:18.000000000","message":"ditto","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":111,"context_line":"      $.resource_providers[0].links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":112,"context_line":"      $.resource_providers[0].links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":113,"context_line":"      $.resource_providers[0].links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":114,"context_line":"      $.resource_providers[0].links[?rel \u003d \"allocations\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/allocations"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"- name: filter out all resource providers by name"},{"line_number":117,"context_line":"  GET: /resource_providers?name\u003dflubblebubble"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_37bd2202","line":114,"in_reply_to":"3f4b6375_f3a1e1aa","updated":"2017-10-23 19:03:59.000000000","message":"Done","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a9c6030f15eb57375c2cc931af47ba8ee156eaf2","unresolved":false,"context_lines":[{"line_number":160,"context_line":"      $.resource_providers[0].name: $ENVIRON[\u0027RP_NAME\u0027]"},{"line_number":161,"context_line":"      $.resource_providers[0].links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":162,"context_line":"      $.resource_providers[0].links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":163,"context_line":"      $.resource_providers[0].links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"- name: update a resource provider"},{"line_number":166,"context_line":"  PUT: /resource_providers/$RESPONSE[\u0027$.resource_providers[0].uuid\u0027]"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_5379d52c","line":163,"updated":"2017-10-23 17:27:18.000000000","message":"you\u0027ve got no confirmation that there\u0027s no allocations link here.\n\nYou can fake it by doing a len on links:\n\n$.resource_providers[0].links.`len`\n\n(i think)","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"5fa6663329a24c4597d1e23948ef8108833ed7b9","unresolved":false,"context_lines":[{"line_number":160,"context_line":"      $.resource_providers[0].name: $ENVIRON[\u0027RP_NAME\u0027]"},{"line_number":161,"context_line":"      $.resource_providers[0].links[?rel \u003d \"self\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"},{"line_number":162,"context_line":"      $.resource_providers[0].links[?rel \u003d \"inventories\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/inventories"},{"line_number":163,"context_line":"      $.resource_providers[0].links[?rel \u003d \"usages\"].href: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]/usages"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"- name: update a resource provider"},{"line_number":166,"context_line":"  PUT: /resource_providers/$RESPONSE[\u0027$.resource_providers[0].uuid\u0027]"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f4b6375_17fd86b0","line":163,"in_reply_to":"3f4b6375_5379d52c","updated":"2017-10-23 19:03:59.000000000","message":"Done","commit_id":"d4b42c54872c53562a057f891f466d36aea8fd01"}]}
