)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c920952d16484246c400b6cbd7801c7f5de39cca","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Init placement before nova"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With change I7e1e89cd66397883453935dcf7172d977bf82e84 the placement"},{"line_number":10,"context_line":"service may optionally use its own database. In order for this to"},{"line_number":11,"context_line":"work, however, the ordering of how both nova and placement are"},{"line_number":12,"context_line":"configured and initialized in stack.sh requires careful control."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f6a8fd7_e300b9ad","line":9,"updated":"2018-04-25 15:41:13.000000000","message":"Why doesn\u0027t this devstack change depend on this nova change?","commit_id":"e8bad5cd6a65821c54d66bbc5f7ba17091439d34"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"cb129606d85227025352971dc3d826856d4373b7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Init placement before nova"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With change I7e1e89cd66397883453935dcf7172d977bf82e84 the placement"},{"line_number":10,"context_line":"service may optionally use its own database. In order for this to"},{"line_number":11,"context_line":"work, however, the ordering of how both nova and placement are"},{"line_number":12,"context_line":"configured and initialized in stack.sh requires careful control."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_2021f54d","line":9,"in_reply_to":"9f6a8fd7_83b9bd79","updated":"2018-06-12 05:18:30.000000000","message":"You have mentioned it but a Needed-By: often calls it out explicitly","commit_id":"e8bad5cd6a65821c54d66bbc5f7ba17091439d34"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"ceb44e4a939d64af6f1941d233544a8c586ce153","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Init placement before nova"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With change I7e1e89cd66397883453935dcf7172d977bf82e84 the placement"},{"line_number":10,"context_line":"service may optionally use its own database. In order for this to"},{"line_number":11,"context_line":"work, however, the ordering of how both nova and placement are"},{"line_number":12,"context_line":"configured and initialized in stack.sh requires careful control."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f6a8fd7_83b9bd79","line":9,"in_reply_to":"9f6a8fd7_e300b9ad","updated":"2018-04-25 15:59:57.000000000","message":"Basically, because this change can merge independently of the nova changes. The reorderings that happen here are required for the nova change that depends on this one ( https://review.openstack.org/#/c/564067/ ) but have no impact otherwise. So that change actually does depend on this one, whereas this one has no impact for the common case.\n\nThe PLACEMENT_DB_ENABLED stuff has been present in lib/placement for a long time, but until the above wasn\u0027t exercised. That exercised pointed out these ordering problems.","commit_id":"e8bad5cd6a65821c54d66bbc5f7ba17091439d34"}],"stack.sh":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"77835935efcd50c53505c03f78ce27d83b0a1ffc","unresolved":false,"context_lines":[{"line_number":1186,"context_line":"    init_cinder"},{"line_number":1187,"context_line":"fi"},{"line_number":1188,"context_line":""},{"line_number":1189,"context_line":"# Placement Service"},{"line_number":1190,"context_line":"# ---------------"},{"line_number":1191,"context_line":""},{"line_number":1192,"context_line":"if is_service_enabled placement; then"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5f7c97a3_129498e7","line":1189,"updated":"2018-06-07 22:12:42.000000000","message":"Perhaps a similar note explaining why this has to go before Nova here (but not above, which is a weird chickeny/eggy thing).","commit_id":"e8bad5cd6a65821c54d66bbc5f7ba17091439d34"}]}