Bug 602486

Summary:	org.rhq.core.util.updater.Deployer class uses Java6 API java.io.File.getUsableSpace()
Product:	[Other] RHQ Project	Reporter:	Ian Springer <ian.springer>
Component:	No Component	Assignee:	John Mazzitelli <mazz>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Corey Welton <cwelton>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	3.0.0	CC:	ccrouch
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	2.4	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-08-12 16:48:39 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	593121

Description Ian Springer 2010-06-09 23:24:56 UTC

/home/test_jon/release/rhq/modules/core/util/src/main/java/org/rhq/core/util/updater/Deployer.java:[249,36] cannot find symbol
symbol  : method getUsableSpace()
location: class java.io.File

Comment 1 John Mazzitelli 2010-06-10 01:11:04 UTC

dammit - can we pleeeeeeease move to JDK6?

should be easy to comment out that stuff and after the release we uncomment it since we are moving to JDK6 next release (RIGHT!?!?)

Comment 2 John Mazzitelli 2010-06-10 01:21:07 UTC

Here's the small patch that will fix this - after release, we undo this to bring it back (apply to org.rhq.core.util.updater.Deployer):

@@ -247,2 +247,4 @@ public class Deployer {
 
+        /*
+        OH TO BE ABLE TO USE A MODERN JDK - THIS NEEDS JDK6
         File partition = this.deploymentData.getDestinationDir();
@@ -257,2 +259,4 @@ public class Deployer {
         diskUsage.setMaxDiskUsable(usableSpace);
+        */
+        diskUsage.setMaxDiskUsable(Long.MAX_VALUE);

Comment 3 John Mazzitelli 2010-07-06 14:00:47 UTC

master commit 5a4cc93421258718ebc10736616968ede431b4d4

there is really nothing to QA, this is a code change. However, bundle deployment will no longer work if the agent is running on an older JDK - must use JDK 6 at least.

Comment 4 Corey Welton 2010-07-06 20:32:23 UTC

QA Closing.

Comment 5 Corey Welton 2010-08-12 16:48:39 UTC

Mass-closure of verified bugs against JON.