I'm seeing the same thing with 1.0.5.1 and ClearCase using post-review.  The
error text is "Substring not found" with error 105.  I found this out by
dumping the fields in the response the server sends back.  I can't seem to
find the hash where the 1xx series of errors are kept.  The 2xx series is
readily available in the RB source code.

--Bryan

On Wed, Dec 9, 2009 at 4:35 PM, Kunjal <kunjal.par...@gmail.com> wrote:

> Chris,
>
> Do you have any update?
> I am really stuck now...
>
> Kunjal
>
>
>
> On Dec 8, 3:44 pm, Kunjal <kunjal.par...@gmail.com> wrote:
> > sorry,
> >
> > here is the missing error message...
> >
> > >>> Uploading diff
> > >>> HTTP POSTing tohttp://
> rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di
> >
> > ff/new/: {}
> > Error uploading diff: One or more fields had errors (105)
> > Your review request still exists, but the diff is not attached.
> >
> > On Dec 8, 12:19 pm, Christian Hammond <chip...@chipx86.com> wrote:
> >
> >
> >
> > > I don't see any error in here. That line indicates that we're doing an
> HTTP
> > > POST to that URL but aren't sending any fields, which is correct. The
> files
> > > are likely sent, but we don't log that on the console.
> >
> > > Does it just stop after that? Is there error information I'm just not
> > > seeing?
> >
> > > Christian
> >
> > > --
> > > Christian Hammond - chip...@chipx86.com
> > > Review Board -http://www.reviewboard.org
> > > VMware, Inc. -http://www.vmware.com
> >
> > > On Tue, Dec 8, 2009 at 10:47 AM, Kunjal <kunjal.par...@gmail.com>
> wrote:
> >
> > > > Chris,
> >
> > > > We have now set up production server which will released in few days.
> > > > During my final testing I found that when I post review, I can not
> > > > post the diff. Looks like RB-Perforce integration issue.
> > > > I read few blogs and you said that P4 should be in web server's path.
> > > > However, I check httpd.conf and I do not see any P4 specific
> > > > settings.
> > > > Where else I can debug?
> >
> > > > Here is the error...
> >
> > > > >>> Uploading diff
> > > > >>> HTTP POSTing to
> > > >http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di
> > > > ff/new/<
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di%0Aff/new/>:
> > > > {}
> >
> > > > -Kunjal
> >
> > > > kapar...@ltsdo-kaparikh ~
> > > > $ post-review.cygwin 250188 -d
> >
> > > > This tool looks at the given Pending-CL# & posts it for review.
> > > > [Version: 200911
> > > > 13_1141]
> >
> > > > >>> Repository info 'Path: pf-sj1-mob.sj.broadcom.com:1668, Base
> path:
> > > > None, Sup
> > > > ports changesets: True'
> > > > >>> Generating diff for changenum 250188
> > > > >>> P4CLIENT is LTSDO-KAPARIKH
> > > > >>> status is pending
> > > > >>> Lines are:
> > > > ['//depot/Sources/Integration/Automation_Scripts/1DDR_Test_Reques
> > > > t_Script/1DDR_TestRequest.pl#1 - edit change 250188 (text) by
> > > > kapar...@ltsdo-kap
> > > > ARIKH\r\n']
> > > > >>> Client LTSDO-KAPARIKH depot path
> > > > //depot/Sources/Integration/Automation_Scri
> > > > pts/1DDR_Test_Request_Script/1DDR_TestRequest.pl
> > > > >>> Writing
> > > > "//depot/Sources/Integration/Automation_Scripts/1DDR_Test_Request_Sc
> > > > ript/1DDR_TestRequest.pl#1" to "/cygdrive/c/Temp/tmpBiyySU"
> > > > >>> Old file: /cygdrive/c/Temp/tmpBiyySU New file
> > > > c:/Perforce/Sources/Integratio
> > > > n/Automation_Scripts/1DDR_Test_Request_Script/1DDR_TestRequest.pl
> > > > >>> Attempting to create review request for 250188
> > > > repository path pf-sj1-mob.sj.broadcom.com:1668
> > > > >>> HTTP POSTing to
> > > >http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/new/
> > > > : {'repository_path': 'pf-sj1-mob.sj.broadcom.com:1668',
> 'changenum':
> > > > '250188'}
> > > > >>> Review request already exists. Updating it...
> > > > >>> HTTP POSTing to
> > > >http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/up
> > > > date_from_changenum/<
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/up%0Adate_f...>:
> > > > {}
> > > > >>> Review request created
> > > > >>> ---
> > > >
> c:\Perforce\Sources\Integration\Automation_Scripts\1DDR_Test_Request_Scr
> > > > ipt\1DDR_TestRequest.pl
> //depot/Sources/Integration/Automation_Scripts/
> > > > 1DDR_Test
> > > > _Request_Script/1DDR_TestRequest.pl#1
> > > > +++ c:\Perforce\Sources\Integration\Automation_Scripts
> > > > \1DDR_Test_Request_Script\
> > > > 1DDR_TestRequest.pl     2009-12-08 10:23:30
> > > > @@ -12,6 +12,20 @@
> > > >  use MIME::Lite;
> > > >  use CGI qw/:standard/;
> > > >  use File::Basename;
> > > > +#!/usr/bin/perl
> >
> > > >
> +#-------------------------------------------------------------------------­­-----
> >
> > > > +#Script Name: 1DDR_TestRequest.pl
> > > > +#Usage:The script is used to get list of zip files from the latest
> CL
> > > > +#from the path "\\\\brcm-irv\\dfs\\projects\\mcsi_2ddr\\programname"
> > > > +#and send the mail notification for the 1DDR Test Request.
> > > > +#Arguments: The script takes one argument which is the
> > > > +#Branch/Program name/SDB name.
> >
> > > >
> +#-------------------------------------------------------------------------­­-----
> >
> > > > +#use warnings;
> > > > +#use strict;
> > > > +use MIME::Lite;
> > > > +use CGI qw/:standard/;
> > > > +use File::Basename;
> >
> > > >  use Data::Dumper;
> >
> > > >
> #--------------------------------------------------------------------------­­----
> >
> > > > >>> Uploading diff
> > > > >>> HTTP POSTing to
> > > >http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di
> > > > ff/new/<
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di%0Aff/new/>:
> > > > {}
> >
> > > > --
> > > > Want to help the Review Board project? Donate today at
> > > >http://www.reviewboard.org/donate/
> > > > Happy user? Let us know athttp://www.reviewboard.org/users/
> > > > -~----------~----~----~----~------~----~------~--~---
> > > > To unsubscribe from this group, send email to
> > > > reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> <reviewboard%2bunsubscr...@googlegr­­oups.com>
> > > > For more options, visit this group at
> > > >http://groups.google.com/group/reviewboard?hl=en-Hide quoted text -
> >
> > > - Show quoted text -- Hide quoted text -
> >
> > - Show quoted text -
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~----------~----~----~----~------~----~------~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en
>

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Reply via email to