mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Re: Review Request 70000: Fixed to not abort review verification prematurely.
Date Tue, 19 Feb 2019 17:10:02 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 19, 2019, 6:10 p.m.)

Review request for mesos, Till Toenshoff and Vinod Kone.


Really fix issue raised by Vinod

Bugs: MESOS-9582

Repository: mesos


Previously `support/` would abort prematurely
whenever a review a patch could not be applied. This was due to
the `shell` function used to call `support/` invoking
`exit(1)` on the first error which stopped iteration over all
outstanding reviews.

In this patch that explicit call to `exit` is removed, and instead we
let a possible `subprocess.CalledProcessError` propagate up for it to
be handled at a higher level. Currently this will post a note on the
review in question to (1) notify the submitter, and (2) prevent the
review from being checked again.

With the changes here the script will not stop verify reviews in such

Diffs (updated)

  support/ a88a91f8ecb3794846d2d5eddee27ada770d55b9 




Ran the script (with a dummy user and password) on today's reviewboard state. The script attempted
to post a review on the last patch in the chain instead of aborting (see the `TODO` in the
code on why we weren't able to diagnose the issue in the faulty patch with the current implementation).


Benjamin Bannier

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message