Chris, I have not tried that. Will try that as well. But the issue is it works ok on RB 1.0.5 on test server and does not work on RB 1.0.5 on production server. So, something is missing between settings of these two servers and not sure what is missing... Any clue?
On Dec 9, 4:32 pm, Christian Hammond <chip...@chipx86.com> wrote: > 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...@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 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
