Issue Details (XML | Word | Printable)

Key: BAM-407
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Ben Kuo
Reporter: Randall DeFauw
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Bamboo

Unable to get change logs for Perforce

Created: 07/Nov/06 07:54 PM   Updated: 16/Nov/06 09:10 PM
Component/s: Integration (JIRA)
Affects Version/s: 0.6.1
Fix Version/s: 0.7.1

Time Tracking:
Not Specified

File Attachments: 1. Text File 1.log (2.12 MB)
2. Text File atlassian-bamboo.log (303 kB)
3. Text File atlassian-bamboo.log (198 kB)

Environment: Bamboo 0.61 running on Windows XP (stand-alone installer, installed over 0.5x alpha series)
Issue Links:
Duplicate
 

Participants: Ben Kuo, Edwin Wong [Atlassian], Mark Chaimungkalanont [Atlassian] and Randall DeFauw
Since last comment: 2 years, 8 weeks, 1 day ago
Number of comments: 12
Labels:


 Description  « Hide
I tried to configure a JIRA server to test the new integration feature. I entered the details of a test JIRA instance running locally, and gave it a sample issue ID. When I used the 'test' feature, it spit out an XML error. I'm attaching the log file to this run.

I assume that this error is preventing the 'Changes' feature from working properly; I didn't see my check-in comments from Perforce there.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Edwin Wong [Atlassian] added a comment - 07/Nov/06 09:27 PM
Hi Randall,

Thanks for trying out Bamboo 0.6.

We have had another report of this issue coming in. (see BAM-399).

We are looking at it and hope to get to the bottom of this shortly.

Regards,
Edwin


Ben Kuo added a comment - 08/Nov/06 07:57 PM
Hi Randall,

As mentioned in BAM-399, this seems to be a problem with one of Bamboo's dependencies which is loading an old version of the xml parser.

For a temporary workaround while we fix this, could you please try removing the xerces-1.2.3.jar from the webapp/WEB-INF/lib folder where you have installed Bamboo.

I don't think the reason why the check in comments are not appearing. I think that's a separate issue we'll look into.

Thanks for reporting,

cheers

Ben


Ben Kuo added a comment - 08/Nov/06 11:11 PM
Hi Randall,

You can try the workaround above or download 0.6.2 shortly on the Bamboo page at http://confluence.atlassian.com/display/BAMBOO/Download

Could you please let me know if your still having problems in regards to the status of the check-in comments from Perforce.

Cheers

Ben


Randall DeFauw added a comment - 09/Nov/06 10:49 AM
Log file from 0.6.2 on 9 Nov

Randall DeFauw added a comment - 09/Nov/06 10:55 AM
Hi Ben,

I installed the 0.6.2 release and tried it again. I was able to successfully configure and test my JIRA server. However, I'm still not seeing the Perforce check-in comments, so I can't see the JIRA issue I referenced.

I agree that this is a different problem. I attached my log file again. If you look at line 1932, you'll see this:
2006-11-09 09:40:51,046 INFO [output stream pumper] [StreamToListPumper] [PROJA] //depot/main/src/My.java#5 - updating c:\p4local.ws\depot\main\src\My.java
2006-11-09 09:40:51,156 INFO [BAM::BuildChangeDetector] [Command] err>Unknown command. Try 'p4 help' for info.
2006-11-09 09:40:51,156 INFO [BAM::BuildChangeDetector] [AbstractBuildStrategy] project A has 1 changes.

It looks like the p4 client did not understand one of the commands that Bamboo issues. Its odd, since obviously the rest of the P4 interaction (to sync the workspace) is working. I assume you're just using 'p4 changes' to get the list of files that have changed. I'm using the 2006.1 version of P4 (Rev. P4/NTX86/2006.1.BETA/101179 (2006/06/07)), but a basic command like 'changes' doesn't change very often.

Also, on line 1957, you'll see a large stack trace that happened when I clicked on the 'code changes' link for my latest build. I'm not sure if this is related to the P4 error.

I'm not sure if you want me to open another issue for this.

Cheers,
Randy


Randall DeFauw added a comment - 09/Nov/06 11:09 AM
I turned on DEBUG logging, and grabbed a snapshot of the log. Look at the section starting at line 12293:

2006-11-09 10:00:51,140 DEBUG [BAM::BuildChangeDetector] [Command] C:\Program Files\Perforce\p4.EXE -u rdefauw -p 1666 -c local.ws sync //local.ws/depot/main/...
2006-11-09 10:00:51,203 INFO [output stream pumper] [StreamToListPumper] [PROJA] //depot/main/src/My.java#6 - updating c:\p4\local.ws\depot\main\src\My.java
2006-11-09 10:00:51,203 DEBUG [BAM::BuildChangeDetector] [Command] working directory = null
2006-11-09 10:00:51,203 DEBUG [BAM::BuildChangeDetector] [Command] C:\Program Files\Perforce\p4.EXE -p 1666 -c local.ws changes -s submitted //local.ws/depot/main/...@2006/11/09:09:40:51,2006/11/09:10:00:51
2006-11-09 10:00:51,250 DEBUG [BAM::BuildChangeDetector] [Command] working directory = null
2006-11-09 10:00:51,250 DEBUG [BAM::BuildChangeDetector] [Command] C:\Program Files\Perforce\p4.EXE -p 1666 -c local.ws getPrettyPrint -s 10
2006-11-09 10:00:51,281 INFO [BAM::BuildChangeDetector] [Command] err>Unknown command. Try 'p4 help' for info.

So after the sync picks up a new file, it asks for the submitted changes, which is fine. I think it detected that change list #10 was the one that caused the build. But the next p4 command is kind of strange:
p4 getPrettyPrint -s 10
There's no p4 command called 'getPrettyPrint'.

Cheers,
Randy


Mark Chaimungkalanont [Atlassian] added a comment - 09/Nov/06 04:26 PM
Randall,

I'll look into this a bit further. I've updated the issue to reflect your actual issue.

Cheers

Mark C


Mark Chaimungkalanont [Atlassian] added a comment - 09/Nov/06 04:53 PM
Randy,

Looking through the code, it looks like we were trying to get the change list details through the use of the getPrettyPrint command... From the P4 reference, it looks as though the command should really be describe.

Could your try running on the command line:

C:\Program Files\Perforce\p4.EXE -p 1666 -c local.ws describe -s 10

and pasting the exact output of it back into the issue? This might be the command we're looking for.

Cheers,

Mark C


Randall DeFauw added a comment - 09/Nov/06 05:12 PM
Hi Mark,

Yes, the 'describe' command is probably what you're looking for. Here's the output:
C:\p4\local.ws2\depot\main>p4 describe -s 10
Change 10 by rdefauw@local.ws2 on 2006/11/09 09:59:22

another test

Affected files ...

... //depot/main/src/My.java#6 edit

If you're just looking for check-in comments, the 'changes -l' command will work, although it doesn't have the file list.


Mark Chaimungkalanont [Atlassian] added a comment - 09/Nov/06 05:43 PM
Randy,

Thanks for this info. We'll try to get up a patch for you to try.

Cheers,

Mark C


Ben Kuo added a comment - 10/Nov/06 06:05 AM
Hi Randy

Sorry but unfortunately I was unable to look into getting this fixed today. Hopefully we'll have something early next week.

Regards

Ben


Ben Kuo added a comment - 14/Nov/06 02:00 AM
Hi Randy,

Thanks for the feedback. The bug has been fixed for 0.7 but unfortunately we won't be issuing a patch for this for the 0.6.x releases as there are a few flow on effects that needs to be addressed.

Cheers

Ben