)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":26936,"name":"Surya Seetharaman","email":"suryaseetharaman.9@gmail.com","username":"tssurya"},"change_message_id":"560fa714cb5e44b5c75fe3f9d47ce0a369334021","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This fixes the issue by excluding the specific fake marker"},{"line_number":17,"context_line":"instance when listing servers in the API."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"This admittedly isn\u0027t great but it\u0027s one of many not-so-great"},{"line_number":20,"context_line":"options (listed in the bug) and also something that we\u0027ll"},{"line_number":21,"context_line":"eventually remove when we drop the online data migration."},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_e366a4c1","line":19,"range":{"start_line":18,"start_character":0,"end_line":19,"end_character":27},"updated":"2019-05-14 10:01:58.000000000","message":"yeap :(","commit_id":"af07ddb734474051ed77c425bf8a50426c2d1a88"},{"author":{"_account_id":26936,"name":"Surya Seetharaman","email":"suryaseetharaman.9@gmail.com","username":"tssurya"},"change_message_id":"9b59a2e378ee220c05bc2fb420b14d5a9642bd14","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This fixes the issue by excluding the specific fake marker"},{"line_number":17,"context_line":"instance when listing servers in the API."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"This admittedly isn\u0027t great but it\u0027s one of many not-so-great"},{"line_number":20,"context_line":"options (listed in the bug) and also something that we\u0027ll"},{"line_number":21,"context_line":"eventually remove when we drop the online data migration."},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_e3a784bc","line":19,"range":{"start_line":18,"start_character":0,"end_line":19,"end_character":27},"in_reply_to":"dfbec78f_e366a4c1","updated":"2019-05-14 10:11:09.000000000","message":"Specially since we will have it in stein forever, but like you said no other great options since we don\u0027t have hard delete yet. One way would have been to just delete the marker as a follow up of the migration having completed successfully (as a part of the upgrade status check or something) but then there would still be a racing time when the marker exists.\n\nIrrespective of this case, I think the upgrade status checker should delete persistent markers once it deems the migration complete. We had a similar issues  where we had to use objects.RequestSpec.get_by_instance_uuid() for a script and this blew up for the request_spec marker that was left out since one of the migrations in newton.","commit_id":"af07ddb734474051ed77c425bf8a50426c2d1a88"}]}
