Hi Christian,
Thanks for answering.
This is not a public repo, but I can try to reproduce against one; I'll see
later today. I have not used ReviewBoard before so I do not know if this is
"new" behavior (I did try RB before but at that time bazaar support did not
work for me at all).
I have some more info: I strace with some bigger strings, and it appears
the server is trying to get every and all revision.. Example lines from the
trace are:
sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0es\0\0\0hl31:Repository.get_rev_id_for_revno1:.i1021eli3434e48:puzzler@itris"...,
166, 0, NULL, 0) = 166
recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0eoSs\0\0\0Bl2:ok57:wouter.van.vl...@itris.nl-20100525092412-w8pngwzvjbof1h32"...,
65536, 0, NULL, NULL) = 130
sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0es\0\0\0!l25:Repository.iter_revisions1:.eb\0\0\0009wouter.van.vl...@itris.nl-201"...,
157, 0, NULL, 0) = 157
recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0eoSs\0\0\0\nl2:ok2:10eb\0\0\1dx\234uQA\222\244
\20\274\357+\370\200.`)JLL\314\304\376a\367\214RjM#\30@k\364\274~\260'vo{+2\310\254\312L\347:="...,
65536, 0, NULL, NULL) = 434
recvfrom(23, "e", 65536, 0, NULL, NULL) = 1
sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0es\0\0\0gl31:Repository.get_rev_id_for_revno1:.i994eli3434e48:puzzler@itris."...,
165, 0, NULL, 0) = 165
recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0eoSs\0\0\0@l2
:ok55:rob.gerstel...@itris.nl-20100519131703-bpf39z43o203n3k6ee", 65536, 0,
NULL, NULL) = 128
sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0es\0\0\0!l25:Repository.iter_revisions1:.eb\0\0\0007rob.gerstel...@itris.nl-20100"...,
155, 0, NULL, 0) = 155
recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0eoSs\0\0\0\nl2:ok2:10eb\0\0\1Ax\234\215\220\315N\2050\20F_\245\17
7\323\37\2404\306\270s\357\302}\v\3Lni\261\255\350\345\351\5M\214\33\0237\223\314L\346\314\311\347}c"...,
65536, 0, NULL, NULL) = 399
recvfrom(23, "e", 65536, 0, NULL, NULL) = 1
sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0es\0\0\0hl31:Repository.get_rev_id_for_revno1:.i1008eli3434e48:puzzler@itris"...,
166, 0, NULL, 0) = 166
recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0eoSs\0\0\0:l2:ok49:marc@itris.nl-20100520153251-zs98p50oravikd6bee",
65536, 0, NULL, NULL) = 122
sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0es\0\0\0!l25:Repository.iter_revisions1:.eb\0\0\0001marc@itris.nl-20100520153"...,
149, 0, NULL, 0) = 149
recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:Software
version5:2.5.0eoSs\0\0\0\nl2:ok2:10eb\0\0\1\27x\234\205\220Mn\303
\20F\257\222\v\0201\30\214AUU\251\353\34\2\343qB\315\217\v\330\222}\372\222v\331E4\322h6\337{\243\317\373^\317)"...,
65536, 0, NULL, NULL) = 357
The log file only states:
2012-12-11 07:57:39,243 - DEBUG - DiffParser.parse: Beginning parse of
diff, size = 959639
2012-12-11 07:57:39,365 - DEBUG - DiffParser.parse: Finished parsing diff.
I'll try to see if I can find some public bzr repo to check against.
Frits
On Mon, Dec 10, 2012 at 8:08 PM, Christian Hammond wrote:
> Wow, that looks pretty bad.
>
> Is this by any chance a public repository we can test against?
>
> Is there anything in reviewboard.log relevant to this?
>
> Did you have this problem with other versions?
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
>
> On Mon, Dec 10, 2012 at 8:11 AM, Frits Jalvingh wrote:
>
>> Hi all,
>>
>> I'm trying ReviewBoard 1.7rc1 with a Bazaar repository. I cannot add new
>> review requests using a Bazaar diff: the upload process works, but no
>> response comes (the hourglass never disappears). At the same time, each
>> upload that is attempted causes one apache server process to loop
>> apparently accessing the Bazaar server...
>>
>> The bazaar server at this time uses a lot of CPU, 100...150% and more.
>>
>> I traced one of the Apache processes and got:
>>
>> Process 24703 attached - interrupt to quit
>> recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:"..., 65536, 0, NULL,
>> NULL) = 130
>> sendto(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:"..., 157, 0, NULL, 0)
>> = 157
>> recvfrom(23, "bzr message 3 (bzr 1.6)\n\0\0\0\34d16:"..., 65536, 0, NULL,
>> NULL) = 380
>> recvfrom(23, "e", 65536, 0, NULL, NULL) = 1
>> sendto(23, "bzr message 3 (bzr
