What happens if you upload this same diff through the web UI? Do you see any
additional error information?

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Tue, Dec 8, 2009 at 3:44 PM, Kunjal <kunjal.par...@gmail.com> wrote:

> sorry,
>
> here is the missing error message...
>
>
> >>> 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/>:
> {}
> 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 -
>
> --
> 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