)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"73ae62af9e5a3f80dd33a2926e942721cd7b75ce","unresolved":false,"context_lines":[{"line_number":29,"context_line":"   - ctl1-F-new"},{"line_number":30,"context_line":"   - ctl2-M-new"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"During the third state we have a cut in the control plane as ctl2 is"},{"line_number":33,"context_line":"the master and there is no slave to fall back to. Then we end up"},{"line_number":34,"context_line":"loosing HA as only one node is active.  The error persists after"},{"line_number":35,"context_line":"reboot.  Only a pcs resource cleanup will bring the cluster online."},{"line_number":36,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_6be4b53c","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":48},"updated":"2019-10-14 13:12:37.000000000","message":"Is this a control plane cut or \"just\" that we lose HA?","commit_id":"751b3fc09632cf184bf947be8f74d771554a23d0"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"ee0d99c12d6c9d27da72573d7f47139eae7367df","unresolved":false,"context_lines":[{"line_number":29,"context_line":"   - ctl1-F-new"},{"line_number":30,"context_line":"   - ctl2-M-new"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"During the third state we have a cut in the control plane as ctl2 is"},{"line_number":33,"context_line":"the master and there is no slave to fall back to. Then we end up"},{"line_number":34,"context_line":"loosing HA as only one node is active.  The error persists after"},{"line_number":35,"context_line":"reboot.  Only a pcs resource cleanup will bring the cluster online."},{"line_number":36,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_8b4d713c","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":48},"in_reply_to":"3fa7e38b_6be4b53c","updated":"2019-10-14 13:17:25.000000000","message":"We have lost HA: ctl0-Failed-new, ctl1-Failed, hence the cut in the ctl plane when we update the ctl2 container.\n\nBy the way do you have a way to continually test the ctl plane ?  I would really need number here.","commit_id":"751b3fc09632cf184bf947be8f74d771554a23d0"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"ec5784cace0e4506414c1a5d36ca5364ca39ba80","unresolved":false,"context_lines":[{"line_number":29,"context_line":"   - ctl1-F-new"},{"line_number":30,"context_line":"   - ctl2-M-new"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"During the third state we have a cut in the control plane as ctl2 is"},{"line_number":33,"context_line":"the master and there is no slave to fall back to. Then we end up"},{"line_number":34,"context_line":"loosing HA as only one node is active.  The error persists after"},{"line_number":35,"context_line":"reboot.  Only a pcs resource cleanup will bring the cluster online."},{"line_number":36,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_76eed8ed","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":48},"in_reply_to":"3fa7e38b_8b4d713c","updated":"2019-10-14 14:11:41.000000000","message":"No that I know of. I guess there\u0027s nothing specific for OVN, whatever checks the controlplane for Neutron (if there\u0027s such a thing) should work for this purpose as well","commit_id":"751b3fc09632cf184bf947be8f74d771554a23d0"}],"deployment/ovn/ovn-dbs-pacemaker-puppet.yaml":[{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"6edab5ce25a1e3f7a50bfce8551919e9bdcc4da2","unresolved":false,"context_lines":[{"line_number":271,"context_line":"        # one of the updated Stopped ovn.  They are in error, that why"},{"line_number":272,"context_line":"        # we need the cleanup just before."},{"line_number":273,"context_line":"        - name: Ban ovndb resource on the current node."},{"line_number":274,"context_line":"          shell: \"pcs resource ban ovn-dbs-bundle $(hostname | cut -d. -f1)\""},{"line_number":275,"context_line":"          when:"},{"line_number":276,"context_line":"            - step|int \u003d\u003d 1"},{"line_number":277,"context_line":"        - name: Get docker ovn-dbs image"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_69fb48bd","line":274,"updated":"2019-10-15 15:06:40.000000000","message":"could you just use $(hostname -s) here instead of the cut?","commit_id":"751b3fc09632cf184bf947be8f74d771554a23d0"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"2dd5da56b2708d43ecbfae361d59f5b3a334969c","unresolved":false,"context_lines":[{"line_number":271,"context_line":"        # one of the updated Stopped ovn.  They are in error, that why"},{"line_number":272,"context_line":"        # we need the cleanup just before."},{"line_number":273,"context_line":"        - name: Ban ovndb resource on the current node."},{"line_number":274,"context_line":"          shell: \"pcs resource ban ovn-dbs-bundle $(hostname | cut -d. -f1)\""},{"line_number":275,"context_line":"          when:"},{"line_number":276,"context_line":"            - step|int \u003d\u003d 1"},{"line_number":277,"context_line":"        - name: Get docker ovn-dbs image"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_7f753663","line":274,"in_reply_to":"3fa7e38b_69fb48bd","updated":"2019-10-15 16:34:57.000000000","message":"I didn\u0027t want a too \"clever\" tool here so that I get short all the time.  But hostname -s seems to have the \"dumb\" behavior than the cut -d. -f1.\n\nNevertheless, I have tested that version twice now and unless it\u0027s cosmetic I would rather push that code than spin a new review. Is that ok ?","commit_id":"751b3fc09632cf184bf947be8f74d771554a23d0"}]}