)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"7332083527399cf2cd0159966e9ace704ab60ca2","unresolved":false,"context_lines":[{"line_number":9,"context_line":"ComputeCapabilitiesFilter will fail to pick a host if used"},{"line_number":10,"context_line":"along with AggregateInstanceExtraSpecsFilter. This fix will at least"},{"line_number":11,"context_line":"warn the users that they shouldn\u0027t use the two filters together."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I98b1eef6484bffc4305ff84e8badbde7992132ed"},{"line_number":14,"context_line":"Fixes-Bug: #1279719"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3acd31a7_25a151aa","line":12,"updated":"2015-04-29 18:59:14.000000000","message":"Needs DocImpact\n\n- I can update the commit msg by myself but I prefer to leave a comment here for leaving the discussion with mriedem on the last patchset","commit_id":"193ca209e79383942858a30aed704370375951fe"}],"doc/source/devref/filter_scheduler.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"92a42a0b79aff94434b0bd5b0697a315252abebb","unresolved":false,"context_lines":[{"line_number":69,"context_line":"  * \u003cor\u003e (find one of these)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"  Examples are: \"\u003e\u003d 5\", \"s\u003d\u003d 2.1.0\", \"\u003cin\u003e gcc\", \"\u003call-in\u003e aes mmx\", and \"\u003cor\u003e fpu \u003cor\u003e gpu\""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"* |AggregateInstanceExtraSpecsFilter| - checks that the aggregate metadata"},{"line_number":74,"context_line":"  satisfies any extra specifications associated with the instance type (that"},{"line_number":75,"context_line":"  have no scope or are scoped with ``aggregate_instance_extra_specs``)."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3acd31a7_ea57a8c8","line":72,"updated":"2015-04-29 18:30:02.000000000","message":"The docs have a similar warning against using this with the ComputeCapabilitiesFilter:\n\nhttp://docs.openstack.org/juno/config-reference/content/section_compute-scheduler.html#aggregate-instanceextraspecsfilter\n\nSo shouldn\u0027t you also update this?\n\nBut really, it\u0027s kind of annoying that we have duplicate documentation like this.  I\u0027m assuming the nova devref came first and now the config reference is getting more activity.  Wouldn\u0027t it be better to sync them up and make sure the config reference doc is accurate with the devref and then deprecate the devref and make it point to the config reference docs - or vice versa?","commit_id":"193ca209e79383942858a30aed704370375951fe"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"7332083527399cf2cd0159966e9ace704ab60ca2","unresolved":false,"context_lines":[{"line_number":69,"context_line":"  * \u003cor\u003e (find one of these)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"  Examples are: \"\u003e\u003d 5\", \"s\u003d\u003d 2.1.0\", \"\u003cin\u003e gcc\", \"\u003call-in\u003e aes mmx\", and \"\u003cor\u003e fpu \u003cor\u003e gpu\""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"* |AggregateInstanceExtraSpecsFilter| - checks that the aggregate metadata"},{"line_number":74,"context_line":"  satisfies any extra specifications associated with the instance type (that"},{"line_number":75,"context_line":"  have no scope or are scoped with ``aggregate_instance_extra_specs``)."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3acd31a7_45ee8ded","line":72,"in_reply_to":"3acd31a7_ea57a8c8","updated":"2015-04-29 18:59:14.000000000","message":"That\u0027s an excellent point : having duplicate sections seems useless and even errorprone because docs people can\u0027t usually be up-to-date like the developers changing how the filters are working.\n\nSince we need to have a filters documentation up-to-date, we need to keep it in devref IMHO and make sure that any change to it is having a DocImpact tag.\nThen, we could ask the docs people if it would maybe good to have an iframe calling the Nova devref section to get the information for the docs.openstack.org section IMHO.","commit_id":"193ca209e79383942858a30aed704370375951fe"}]}