)]}'
{"placement-api-ref/source/parameters.yaml":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"c0cfaba1ad703b33e3d84ee9320c39110a0c1dd3","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    resource of a specified class that a provider must have the"},{"line_number":71,"context_line":"    capacity to serve::"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        resources\u003dVCPU:4,DISK_GB:64,MEMORY_MB:2048"},{"line_number":74,"context_line":"trait_associated:"},{"line_number":75,"context_line":"  type: string"},{"line_number":76,"context_line":"  in: query"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f2577a7_e20e2c31","line":73,"updated":"2017-09-08 20:07:17.000000000","message":"it ought to work to continue inheriting from the resources_query anchor and adding\n\nmin_version: 1.4\nrequired: true\n\nas everything else is the same","commit_id":"84360f693fbaa8c6b4993d30c1ddf6db547ec770"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"010a7685df4a1e39f0156fa090d8974da2e47b79","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    resource of a specified class that a provider must have the"},{"line_number":71,"context_line":"    capacity to serve::"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        resources\u003dVCPU:4,DISK_GB:64,MEMORY_MB:2048"},{"line_number":74,"context_line":"trait_associated:"},{"line_number":75,"context_line":"  type: string"},{"line_number":76,"context_line":"  in: query"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f2577a7_25a2f99b","line":73,"in_reply_to":"5f2577a7_763561f7","updated":"2017-09-09 02:22:49.000000000","message":"I went ahead and implemented my version and then while doing so realized that maybe what you\u0027re suggesting is that you don\u0027t want the requirement for \"resources\" in allocation_candidates to say anything about version since the whole thing is new with version. Here\u0027s what I did in case you want it:\n\n\t--- a/placement-api-ref/source/parameters.yaml\n\t+++ b/placement-api-ref/source/parameters.yaml\n\t@@ -33,6 +33,7 @@ member_of:\n\t\t A comma-separated list of strings representing aggregate uuids.\n\t\t The returned resource providers must be associated with at\n\t\t least one of the aggregates identified by uuid.\n\t+  min_version: 1.3\n\t project_id: \u0026project_id\n\t   type: string\n\t   in: query\n\t@@ -59,9 +60,11 @@ resources_query: \u0026resources_query\n\t\t capacity to serve::\n\t \n\t\t\t resources\u003dVCPU:4,DISK_GB:64,MEMORY_MB:2048\n\t+  min_version: 1.4\n\t resources_query_required:\n\t   \u003c\u003c: *resources_query\n\t   required: true\n\t+  min_version: 1.10\n\t trait_associated:\n\t   type: string\n\t   in: query","commit_id":"84360f693fbaa8c6b4993d30c1ddf6db547ec770"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"247cd37e1283fa4b956ae6d8663afb3c9c25a75e","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    resource of a specified class that a provider must have the"},{"line_number":71,"context_line":"    capacity to serve::"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        resources\u003dVCPU:4,DISK_GB:64,MEMORY_MB:2048"},{"line_number":74,"context_line":"trait_associated:"},{"line_number":75,"context_line":"  type: string"},{"line_number":76,"context_line":"  in: query"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f2577a7_f6d83180","line":73,"in_reply_to":"5f2577a7_e20e2c31","updated":"2017-09-08 21:00:43.000000000","message":"But then the min version of 1.4 is going to show up in the allocation_candidates reference and that API was added in 1.10, so that would be confusing and weird.","commit_id":"84360f693fbaa8c6b4993d30c1ddf6db547ec770"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"1241a72e62201fc65a19be3aacdb7ad554e6b601","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    resource of a specified class that a provider must have the"},{"line_number":71,"context_line":"    capacity to serve::"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        resources\u003dVCPU:4,DISK_GB:64,MEMORY_MB:2048"},{"line_number":74,"context_line":"trait_associated:"},{"line_number":75,"context_line":"  type: string"},{"line_number":76,"context_line":"  in: query"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f2577a7_763561f7","line":73,"in_reply_to":"5f2577a7_f6d83180","updated":"2017-09-08 21:31:32.000000000","message":"Sorry I meant:\n\nput min_version 1.4 on resources_query\n\ninherit resources_query in resources_query_requires and there set required:true and min_version: 1.10","commit_id":"84360f693fbaa8c6b4993d30c1ddf6db547ec770"}]}