)]}'
{"nova/scheduler/filters/type_filter.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3df3481c6e38549d0c2203ddf7a3949870f00865","unresolved":false,"context_lines":[{"line_number":36,"context_line":"                    \u002717.0.0 Queens release. There is no replacement planned \u0027"},{"line_number":37,"context_line":"                    \u0027for this filter. It is fundamentally flawed in that it \u0027"},{"line_number":38,"context_line":"                    \u0027relies on the flavors.id primary key and if a flavor \u0027"},{"line_number":39,"context_line":"                    \u0027\\\"changed\\\" (deleted and re-created with new values) \u0027"},{"line_number":40,"context_line":"                    \u0027it will result in this filter thinking it is a \u0027"},{"line_number":41,"context_line":"                    \u0027different flavor, thus breaking the usefulness of this \u0027"},{"line_number":42,"context_line":"                    \u0027filter.\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_a8a630aa","line":39,"range":{"start_line":39,"start_character":21,"end_line":39,"end_character":22},"updated":"2017-04-20 16:50:40.000000000","message":"These shouldn\u0027t be needed?","commit_id":"b3eded44b60ffdc2d14a5f26a5bc0ede70e64459"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c92a51bd703ece0e09259ed825757108545928c2","unresolved":false,"context_lines":[{"line_number":36,"context_line":"                    \u002717.0.0 Queens release. There is no replacement planned \u0027"},{"line_number":37,"context_line":"                    \u0027for this filter. It is fundamentally flawed in that it \u0027"},{"line_number":38,"context_line":"                    \u0027relies on the flavors.id primary key and if a flavor \u0027"},{"line_number":39,"context_line":"                    \u0027\\\"changed\\\" (deleted and re-created with new values) \u0027"},{"line_number":40,"context_line":"                    \u0027it will result in this filter thinking it is a \u0027"},{"line_number":41,"context_line":"                    \u0027different flavor, thus breaking the usefulness of this \u0027"},{"line_number":42,"context_line":"                    \u0027filter.\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_be18aa1d","line":39,"range":{"start_line":39,"start_character":21,"end_line":39,"end_character":22},"in_reply_to":"7ffa3b31_a8a630aa","updated":"2017-04-20 17:52:02.000000000","message":"Yeah I suppose not. It doesn\u0027t seem to hurt anything though:\n\n\u003e\u003e\u003e msg \u003d (\u0027relies on the flavors.id primary key and if a flavor \u0027\n...                     \u0027\\\"changed\\\" (deleted and re-created with new values) \u0027)\n\u003e\u003e\u003e msg\n\u0027relies on the flavors.id primary key and if a flavor \"changed\" (deleted and re-created with new values) \u0027\n\u003e\u003e\u003e","commit_id":"b3eded44b60ffdc2d14a5f26a5bc0ede70e64459"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3df3481c6e38549d0c2203ddf7a3949870f00865","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def __init__(self):"},{"line_number":34,"context_line":"        super(TypeAffinityFilter, self).__init__()"},{"line_number":35,"context_line":"        LOG.warning(\u0027TypeAffinityFilter is deprecated for removal in the \u0027"},{"line_number":36,"context_line":"                    \u002717.0.0 Queens release. There is no replacement planned \u0027"},{"line_number":37,"context_line":"                    \u0027for this filter. It is fundamentally flawed in that it \u0027"},{"line_number":38,"context_line":"                    \u0027relies on the flavors.id primary key and if a flavor \u0027"},{"line_number":39,"context_line":"                    \u0027\\\"changed\\\" (deleted and re-created with new values) \u0027"},{"line_number":40,"context_line":"                    \u0027it will result in this filter thinking it is a \u0027"},{"line_number":41,"context_line":"                    \u0027different flavor, thus breaking the usefulness of this \u0027"},{"line_number":42,"context_line":"                    \u0027filter.\u0027)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def host_passes(self, host_state, spec_obj):"},{"line_number":45,"context_line":"        \"\"\"Dynamically limits hosts to one instance type"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_e8c9d858","line":42,"range":{"start_line":35,"start_character":0,"end_line":42,"end_character":30},"updated":"2017-04-20 16:50:40.000000000","message":"This is an exceptionally long message for a warning. Perhaps we should cut this down to a warning and maybe a debug?","commit_id":"b3eded44b60ffdc2d14a5f26a5bc0ede70e64459"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c92a51bd703ece0e09259ed825757108545928c2","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def __init__(self):"},{"line_number":34,"context_line":"        super(TypeAffinityFilter, self).__init__()"},{"line_number":35,"context_line":"        LOG.warning(\u0027TypeAffinityFilter is deprecated for removal in the \u0027"},{"line_number":36,"context_line":"                    \u002717.0.0 Queens release. There is no replacement planned \u0027"},{"line_number":37,"context_line":"                    \u0027for this filter. It is fundamentally flawed in that it \u0027"},{"line_number":38,"context_line":"                    \u0027relies on the flavors.id primary key and if a flavor \u0027"},{"line_number":39,"context_line":"                    \u0027\\\"changed\\\" (deleted and re-created with new values) \u0027"},{"line_number":40,"context_line":"                    \u0027it will result in this filter thinking it is a \u0027"},{"line_number":41,"context_line":"                    \u0027different flavor, thus breaking the usefulness of this \u0027"},{"line_number":42,"context_line":"                    \u0027filter.\u0027)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def host_passes(self, host_state, spec_obj):"},{"line_number":45,"context_line":"        \"\"\"Dynamically limits hosts to one instance type"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_1eb8bee2","line":42,"range":{"start_line":35,"start_character":0,"end_line":42,"end_character":30},"in_reply_to":"7ffa3b31_e8c9d858","updated":"2017-04-20 17:52:02.000000000","message":"The filters are loaded up once when the HostManager is created and then cached, so you\u0027d only see this once when starting the nova-scheduler process, assuming you have this filter enabled at all.","commit_id":"b3eded44b60ffdc2d14a5f26a5bc0ede70e64459"}]}