1. set "POST" for server actions

jxiaobin · jxiaobin · commit 017c1e10dd0c · 2013-05-27T11:02:02.000+08:00
2. RebootAction does not need return type
diff --git a/nova-client/src/main/java/org/openstack/nova/api/ServersResource.java b/nova-client/src/main/java/org/openstack/nova/api/ServersResource.java
@@ -126,7 +126,7 @@ public UpdateServer(String id, Server server) {
 	public abstract class Action<T> extends OpenStackRequest<T> {
 
 		public Action(String id, Entity<?> entity, Class<T> returnType) {
-			super(CLIENT, HttpMethod.GET, new StringBuilder("/servers/").append(id).append("/action").toString(), entity, returnType);
+			super(CLIENT, HttpMethod.POST, new StringBuilder("/servers/").append(id).append("/action").toString(), entity, returnType);
 		}
 
 	}
@@ -141,12 +141,12 @@ public ChangePasswordAction(String id, ChangePassword action) {
 
 	}
 
-	public class RebootAction extends Action<Server> {
+	public class RebootAction extends Action<Void> {
 
 		private Reboot action;
 
 		public RebootAction(String id, Reboot action) {
-			super(id, Entity.json(action), Server.class);
+			super(id, Entity.json(action), Void.class);
 		}
 
 	}
@@ -290,7 +290,7 @@ public static GetConsoleOutputServer getConsoleOutput(String id, int length) {
 		return new GetConsoleOutputServer(id, action);
 	}
 	
-public class PauseServer extends OpenStackRequest {
+	public class PauseServer extends OpenStackRequest {
 		
 		private Pause action;
 
@@ -441,7 +441,7 @@ public CreateBackupServer createBackup(String serverId, CreateBackup action) {
 		return new CreateBackupServer(serverId, action);
 	}
 	
-public class RescueServer extends OpenStackRequest {
+	public class RescueServer extends OpenStackRequest {
 		
 		private Rescue action;
 

Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,7 @@ public UpdateServer(String id, Server server) {`
`126`	`126`	`public abstract class Action<T> extends OpenStackRequest<T> {`
`127`	`127`
`128`	`128`	`public Action(String id, Entity<?> entity, Class<T> returnType) {`
`129`		`- super(CLIENT, HttpMethod.GET, new StringBuilder("/servers/").append(id).append("/action").toString(), entity, returnType);`
	`129`	`+ super(CLIENT, HttpMethod.POST, new StringBuilder("/servers/").append(id).append("/action").toString(), entity, returnType);`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`}`
`@@ -141,12 +141,12 @@ public ChangePasswordAction(String id, ChangePassword action) {`
`141`	`141`
`142`	`142`	`}`
`143`	`143`
`144`		`- public class RebootAction extends Action<Server> {`
	`144`	`+ public class RebootAction extends Action<Void> {`
`145`	`145`
`146`	`146`	`private Reboot action;`
`147`	`147`
`148`	`148`	`public RebootAction(String id, Reboot action) {`
`149`		`- super(id, Entity.json(action), Server.class);`
	`149`	`+ super(id, Entity.json(action), Void.class);`
`150`	`150`	`}`
`151`	`151`
`152`	`152`	`}`
`@@ -290,7 +290,7 @@ public static GetConsoleOutputServer getConsoleOutput(String id, int length) {`
`290`	`290`	`return new GetConsoleOutputServer(id, action);`
`291`	`291`	`}`
`292`	`292`
`293`		`-public class PauseServer extends OpenStackRequest {`
	`293`	`+ public class PauseServer extends OpenStackRequest {`
`294`	`294`
`295`	`295`	`private Pause action;`
`296`	`296`
`@@ -441,7 +441,7 @@ public CreateBackupServer createBackup(String serverId, CreateBackup action) {`
`441`	`441`	`return new CreateBackupServer(serverId, action);`
`442`	`442`	`}`
`443`	`443`
`444`		`-public class RescueServer extends OpenStackRequest {`
	`444`	`+ public class RescueServer extends OpenStackRequest {`
`445`	`445`
`446`	`446`	`private Rescue action;`
`447`	`447`