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/: {} 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> > > 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 For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en