)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"91d0299dd4e74a94a3f5e779b15affe207bc60aa","unresolved":false,"context_lines":[{"line_number":7,"context_line":"content-provider jobs do not need to vote"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are times when the content provider"},{"line_number":10,"context_line":"runs when it\u0027s not needed... master patches"},{"line_number":11,"context_line":"to tripleo git repos still run the train"},{"line_number":12,"context_line":"content provider.  Let\u0027s not fail on it."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1f621f24_e6d1cc59","line":10,"range":{"start_line":10,"start_character":5,"end_line":10,"end_character":25},"updated":"2020-10-30 07:52:49.000000000","message":"why to run when it\u0027s not needed? shouldn\u0027t be it fixed to not run unneeded jobs.\nalso if making those nonvoting why to run in gate pipeline.","commit_id":"f7884b8469beb03a5dbb43b1ccfb85b3d86e1ed8"}],"zuul.d/standalone-jobs.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2bec695905d32ad10f236b9047d0baaf1b0eac33","unresolved":false,"context_lines":[{"line_number":1058,"context_line":"          service-clients.http_timeout: \u0027120\u0027"},{"line_number":1059,"context_line":""},{"line_number":1060,"context_line":"- job:"},{"line_number":1061,"context_line":"    name: tripleo-ci-centos-8-content-provider"},{"line_number":1062,"context_line":"    parent: tripleo-ci-base-standalone-centos-8"},{"line_number":1063,"context_line":"    branches: ^(?!stable/(newton|ocata|pike|queens|rocky|stein)).*$"},{"line_number":1064,"context_line":"    run:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1f621f24_66693cbd","line":1061,"range":{"start_line":1061,"start_character":4,"end_line":1061,"end_character":8},"updated":"2020-10-30 07:31:20.000000000","message":"this one too for completeness then let\u0027s not leave a special case unless there is a reason to?","commit_id":"ce4425cece2ee0a2e86da7215306c70aa40255ea"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"f24e6ea7983b2fa06ec91620fe6728e3a419216c","unresolved":false,"context_lines":[{"line_number":1076,"context_line":"- job:"},{"line_number":1077,"context_line":"    name: tripleo-ci-centos-8-content-provider-ussuri"},{"line_number":1078,"context_line":"    parent: tripleo-ci-centos-8-content-provider"},{"line_number":1079,"context_line":"    voting: false"},{"line_number":1080,"context_line":"    override-checkout: stable/ussuri"},{"line_number":1081,"context_line":"    vars:"},{"line_number":1082,"context_line":"      branch_override: stable/ussuri"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1f621f24_b813760d","line":1079,"range":{"start_line":1079,"start_character":12,"end_line":1079,"end_character":17},"updated":"2020-10-29 19:02:44.000000000","message":"this has no effect, if a provider fails, consumers will be skipped (voting and non-voting).\n\nIn which situation this would help?","commit_id":"ce4425cece2ee0a2e86da7215306c70aa40255ea"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"1e05579858d28ee48875147e0e91f02cd4be0604","unresolved":false,"context_lines":[{"line_number":1076,"context_line":"- job:"},{"line_number":1077,"context_line":"    name: tripleo-ci-centos-8-content-provider-ussuri"},{"line_number":1078,"context_line":"    parent: tripleo-ci-centos-8-content-provider"},{"line_number":1079,"context_line":"    voting: false"},{"line_number":1080,"context_line":"    override-checkout: stable/ussuri"},{"line_number":1081,"context_line":"    vars:"},{"line_number":1082,"context_line":"      branch_override: stable/ussuri"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1f621f24_d30e15e4","line":1079,"range":{"start_line":1079,"start_character":12,"end_line":1079,"end_character":17},"in_reply_to":"1f621f24_58fd0239","updated":"2020-10-29 20:30:49.000000000","message":"here\u0027s why it should be non-voting https://review.opendev.org/#/c/758945/\n\nIf the job actually fails and it has a child that fails.. fine.. the child job blocks the patch..\nhttps://review.opendev.org/#/c/758945/","commit_id":"ce4425cece2ee0a2e86da7215306c70aa40255ea"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"752faf6d5821015aa3af777399629769d05ba1d1","unresolved":false,"context_lines":[{"line_number":1076,"context_line":"- job:"},{"line_number":1077,"context_line":"    name: tripleo-ci-centos-8-content-provider-ussuri"},{"line_number":1078,"context_line":"    parent: tripleo-ci-centos-8-content-provider"},{"line_number":1079,"context_line":"    voting: false"},{"line_number":1080,"context_line":"    override-checkout: stable/ussuri"},{"line_number":1081,"context_line":"    vars:"},{"line_number":1082,"context_line":"      branch_override: stable/ussuri"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1f621f24_58fd0239","line":1079,"range":{"start_line":1079,"start_character":12,"end_line":1079,"end_character":17},"in_reply_to":"1f621f24_b813760d","updated":"2020-10-29 19:46:22.000000000","message":"actually I found a good use case for making provider job non-voting:\nif a consumer job is non-voting, you don\u0027t have to mark the provider one as nv. All providers will be non-voting by default.\nExample: https://review.opendev.org/#/c/760420/1/.zuul.yaml\n\nMakes sense ?","commit_id":"ce4425cece2ee0a2e86da7215306c70aa40255ea"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2bec695905d32ad10f236b9047d0baaf1b0eac33","unresolved":false,"context_lines":[{"line_number":1076,"context_line":"- job:"},{"line_number":1077,"context_line":"    name: tripleo-ci-centos-8-content-provider-ussuri"},{"line_number":1078,"context_line":"    parent: tripleo-ci-centos-8-content-provider"},{"line_number":1079,"context_line":"    voting: false"},{"line_number":1080,"context_line":"    override-checkout: stable/ussuri"},{"line_number":1081,"context_line":"    vars:"},{"line_number":1082,"context_line":"      branch_override: stable/ussuri"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1f621f24_a64a3452","line":1079,"range":{"start_line":1079,"start_character":12,"end_line":1079,"end_character":17},"in_reply_to":"1f621f24_d30e15e4","updated":"2020-10-30 07:31:20.000000000","message":"i came here to say the say thing that folco did but the example you gave is very valid weshay.\nindeed we have redundant parents in some cases because of the upgrade jobs in the layout.","commit_id":"ce4425cece2ee0a2e86da7215306c70aa40255ea"}]}