Thanks for advices :)
We trigger post-review from Issue Tracker. From how we now what was
changed? This is simple because issue ticket contain all changeset
with number of versions commited to repository. There is no way to
commit code to repo without notifying changed revisions in issue
tracket. We always commit "into a ticket". So When issue ticket
change state to the "review" the trigger hook which send changeset and
few fields to wrapper (online website, normal POST request with some
additional data). This data is formatted to the repository available
on machine where wrapper working, and passed as --revision-range to
post-review which post new review in ReviewBoard. But all what we
store is ticket id, nothing more. So when changes are made (ticket
just contain more revisions numbers) wrapper get only new revision-
range, ticket id and that is all. But they don't know anything about
review id. The only way to find out is query ReviewBoard about that.
In our company this is simple - one ticket id one review and some kind
of patches for rb-tools will work well, but this change is not work in
overall ReviewBoard. I don't want make changes which have not chance
to be include to release. So in cases like that I'am looking ways to
solve the problem outside ReviewBoard ecosystem. This allow me to fast
update ReviewBoard and RBTools to newest version without loosing my
changes specific for company workflow. I'am impatient when my changes
made until now go into the release :) This allow me use ReviewBoard
without modifications because all needed modifications will be
included :]
On Apr 16, 3:48 am, Christian Hammond <
chip...@chipx86.com> wrote:
> You're right in that it's a bit tricky. Since we store bug IDs as part of a
> string, they're difficult to accurately query quickly. We can't quite index
> by bug ID, so queries won't be as fast as we'd prefer. Also, as you said,
> there may be multiple review requests returned.
>
> Personally, I think what you're doing is custom enough where I don't want
> the functionality to update by bug number in post-review. I don't mind
> adding APIs to query review requests based on bug ID, but that's only
> solving one part of this. Soonest we can add that is 1.5, but given the
> schedule, that may slip to 1.6. I'd want to wait until the new API work is
> done.
>
> If you want to do the wrapper, that'd probably work best. Shouldn't end up
> being too complicated. That is, once we have querying available. Down the
> road, I'd like for us to have a tool in RBTools for querying Review Board
> for review requests and displaying information. At that point, your wrapper
> script would be easy. It's probably a little ways off, though.
>
> I think it'd make the most sense to update all review requests with that bug
> number.
>
> What are you looking to use this functionality for? I'm curious about the
> workflow.
>
> Christian
>
> --
> Christian Hammond -
chip...@chipx86.com
> Review Board -
http://www.reviewboard.org
> VMware, Inc. -
http://www.vmware.com