[jira] [Commented] (CLOUDSTACK-8581) Make S3 TCP KeepAlive and ConnectionTtl configureable

2015-06-23 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8581?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597769#comment-14597769
 ] 

ASF subversion and git services commented on CLOUDSTACK-8581:
-

Commit ba18252c5f8f0a8ce44ba2d6586c587f6a964135 in cloudstack's branch 
refs/heads/CLOUDSTACK-8581 from [~widodh]
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=ba18252 ]

CLOUDSTACK-8581: S3, make connection TTL and TCP KeepAlive configureable


 Make S3 TCP KeepAlive and ConnectionTtl configureable
 -

 Key: CLOUDSTACK-8581
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8581
 Project: CloudStack
  Issue Type: Improvement
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: SystemVM
Reporter: Wido den Hollander
Assignee: Wido den Hollander
 Fix For: 4.6.0


 Currently we can not set the TCP KeepAlive and ConnectionTtl settings for the 
 Amazon S3 ClientConfiguration class: 
 http://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/ClientConfiguration.html
 Our S3Util doesn't set these values so it defaults to what Amazon configures.
 This doesn't work in all situations and we want to make this configurable. 
 For example the ConnectionTtl should be lowered in cases where the Object 
 Store closes a HTTP connection after a very short period.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (CLOUDSTACK-8580) Users should be able to expunge VMs

2015-06-23 Thread Lennert den Teuling (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8580?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lennert den Teuling updated CLOUDSTACK-8580:

Description: 
When automating deployments of CloudStack (with for example Terraform) there 
are situations where VMs get recreated with the same name (and hostname). When 
VMs are destroyed by a user, the name will be reserved on the network until the 
VM truly gets expunged (depending on expunge.delay). Because of this, some 
automation tools cannot work because a new deployment with the same name gives 
an error.  

Users do not have the ability to directly expunge VMs (Only admin and 
domain-admins can), but they can destroy them and the admin can configure the 
expunge.delay where VMs truly get removed (expunged). 

Working with the expunge delay is very safe in case users accidentally remove a 
VM, but in some cases (when users know what they are doing) there should also 
be a option to completely remove the VM when destroying it (expunge). 

Ideally the admin should be able to configure this behavior trough the global 
settings, cause i believe the admin deliberately needs to turn it on (off by 
default).

We have looked into making our clients domain-admin by default, but that gives 
them abilities we do not want to give, so we see no other way then just 
enabling expunge for the user. 



  was:
When automating deployments of CloudStack (with for example Terraform) there 
are situations where VMs get recreated with the same name (and hostname). When 
VMs are destroyed by a user, the name will be reserved on the network until the 
VM truly gets expunged (depending on expunge.delay). 

Users do not have the ability to directly expunge VMs (Only admin and 
domain-admins can), but they can destroy them and the admin can configure the 
expunge.delay where VMs truly get removed (expunged). 

Working with the expunge delay is very safe in case users accidentally remove a 
VM, but in some cases (when users know what they are doing) there should also 
be a option to completely remove the VM when destroying it (expunge). 

Ideally the admin should be able to configure this behavior trough the global 
settings, cause i believe the admin deliberately needs to turn it on (off by 
default).

We have looked into making our clients domain-admin by default, but that gives 
them abilities we do not want to give, so we see no other way then just 
enabling expunge for the user. 




 Users should be able to expunge VMs
 ---

 Key: CLOUDSTACK-8580
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8580
 Project: CloudStack
  Issue Type: Improvement
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
Reporter: Lennert den Teuling
Priority: Minor

 When automating deployments of CloudStack (with for example Terraform) there 
 are situations where VMs get recreated with the same name (and hostname). 
 When VMs are destroyed by a user, the name will be reserved on the network 
 until the VM truly gets expunged (depending on expunge.delay). Because of 
 this, some automation tools cannot work because a new deployment with the 
 same name gives an error.  
 Users do not have the ability to directly expunge VMs (Only admin and 
 domain-admins can), but they can destroy them and the admin can configure the 
 expunge.delay where VMs truly get removed (expunged). 
 Working with the expunge delay is very safe in case users accidentally remove 
 a VM, but in some cases (when users know what they are doing) there should 
 also be a option to completely remove the VM when destroying it (expunge). 
 Ideally the admin should be able to configure this behavior trough the global 
 settings, cause i believe the admin deliberately needs to turn it on (off by 
 default).
 We have looked into making our clients domain-admin by default, but that 
 gives them abilities we do not want to give, so we see no other way then just 
 enabling expunge for the user. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597628#comment-14597628
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/510#issuecomment-114495407
  
@resmo can you send a doc PR so at least we can document this in release 
notes?


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597626#comment-14597626
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/510#issuecomment-114495233
  
this can potentially break clients depending on the behaviour when zoneid 
is passed


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597606#comment-14597606
 ] 

ASF subversion and git services commented on CLOUDSTACK-8578:
-

Commit 82ecc1aa06e75bb15fe5d0bd8628f7acc6b61623 in cloudstack's branch 
refs/heads/master from Rene Moser
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=82ecc1a ]

CLOUDSTACK-8578: fix listVirtualMachines does not return deleted machines when 
zone is specified

Signed-off-by: Daan Hoogland daan.hoogl...@gmail.com

This closes #510


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597607#comment-14597607
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/510


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597685#comment-14597685
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


GitHub user resmo opened a pull request:

https://github.com/apache/cloudstack/pull/513

CLOUDSTACK-8578: implement old behaviour for listing only non-destroy…

…ed VMs using state=present

Also see 
https://github.com/apache/cloudstack/pull/510#issuecomment-114502611

RFC @bhaisaab 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/resmo/cloudstack fix/CLOUDSTACK-8578-extend

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/513.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #513


commit bf67382521e6df3e12b311fd11f21569564e3920
Author: Rene Moser re...@apache.org
Date:   2015-06-23T13:47:06Z

CLOUDSTACK-8578: implement old behaviour for listing only non-destroyed VMs 
using state=present




 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597640#comment-14597640
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/510#issuecomment-114502611
  
@bhaisaab Ok. I'll make a doc PR. We could probably think about extending 
the list API to a state e.g. `present` which returns all VMs in states 
`starting, running, stopped, stopping'. So the *old* behaviour could be used as 
well in a proper way. Any thoughts?


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (CLOUDSTACK-8581) Make S3 TCP KeepAlive and ConnectionTtl configureable

2015-06-23 Thread Wido den Hollander (JIRA)
Wido den Hollander created CLOUDSTACK-8581:
--

 Summary: Make S3 TCP KeepAlive and ConnectionTtl configureable
 Key: CLOUDSTACK-8581
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8581
 Project: CloudStack
  Issue Type: Improvement
  Security Level: Public (Anyone can view this level - this is the default.)
  Components: SystemVM
Reporter: Wido den Hollander
Assignee: Wido den Hollander
 Fix For: 4.6.0


Currently we can not set the TCP KeepAlive and ConnectionTtl settings for the 
Amazon S3 ClientConfiguration class: 
http://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/ClientConfiguration.html

Our S3Util doesn't set these values so it defaults to what Amazon configures.

This doesn't work in all situations and we want to make this configurable. For 
example the ConnectionTtl should be lowered in cases where the Object Store 
closes a HTTP connection after a very short period.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (CLOUDSTACK-8580) Users should be able to expunge VMs

2015-06-23 Thread Lennert den Teuling (JIRA)
Lennert den Teuling created CLOUDSTACK-8580:
---

 Summary: Users should be able to expunge VMs
 Key: CLOUDSTACK-8580
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8580
 Project: CloudStack
  Issue Type: Improvement
  Security Level: Public (Anyone can view this level - this is the default.)
Reporter: Lennert den Teuling
Priority: Minor


When automating deployments of CloudStack (with for example Terraform) there 
are situations where VMs get recreated with the same name (and hostname). When 
VMs are destroyed by a user, the name will be reserved on the network until the 
VM truly gets expunged (depending on expunge.delay). 

Users do not have the ability to directly expunge VMs (Only admin and 
domain-admins can), but they can destroy them and the admin can configure the 
expunge.delay where VMs truly get removed (expunged). 

Working with the expunge delay is very safe in case users accidentally remove a 
VM, but in some cases (when users know what they are doing) there should also 
be a option to completely remove the VM when destroying it (expunge). 

Ideally the admin should be able to configure this behavior trough the global 
settings, cause i believe the admin deliberately needs to turn it on (off by 
default).

We have looked into making our clients domain-admin by default, but that gives 
them abilities we do not want to give, so we see no other way then just 
enabling expunge for the user. 





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597605#comment-14597605
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/510#issuecomment-114486026
  
good point so backwards compatibility is a glass eye anyway;) will merge


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8559) Source address spoofing prevention in Basic Networking only done for DNS

2015-06-23 Thread Wido den Hollander (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597671#comment-14597671
 ] 

Wido den Hollander commented on CLOUDSTACK-8559:


I already backported this to our production cloud and it works fine there.

 Source address spoofing prevention in Basic Networking only done for DNS
 

 Key: CLOUDSTACK-8559
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8559
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: KVM
Reporter: Wido den Hollander
Assignee: Wido den Hollander
 Fix For: 4.6.0, 4.5.2


 Looking at the security group rules being programmed for Instances it seems 
 that we only drop spoofed traffic when it's for DNS:
 if vm_ip is not None:
   execute(iptables -A  + vmchain_default +  -m physdev 
 --physdev-is-bridged --physdev-in  + vif +  -m set --set  + vmipsetName + 
  src -p udp --dport 53  -j RETURN )
   execute(iptables -A  + vmchain_default +  -m physdev 
 --physdev-is-bridged --physdev-in  + vif +  -m set --set  + vmipsetName + 
  src -j  + vmchain_egress)
 I think that we can drop ALL packets which do not match any of the IPs in the 
 list. I don't see a valid reason why we only do this for DNS/UDP 53.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14598382#comment-14598382
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/513#discussion_r33093929
  
--- Diff: server/src/com/cloud/api/query/QueryManagerImpl.java ---
@@ -968,7 +968,11 @@
 }
 
 if (state != null) {
-sc.setParameters(stateEQ, state);
+if (state.equalsIgnoreCase(present)) {
--- End diff --

Not sure if this make sense but is Expunged not !Present as well?


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597525#comment-14597525
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/510#issuecomment-114465874
  
@DaanHoogland We already do :). User permissions API is **not affected** of 
this change however.

The API does already return VMs `listVirtualMacines` in `destroyed` state 
if you have *admin* permissions. 

But if you provide `zoneid` in `listVirtualMacines` with *admin* 
permissions, it does not return VMs in `destroyed` state. This makes no sense 
to me. The bug was already fixed in CLOUDSTACK-284 but then the bug was 
reimplemented by accident.



 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8579) Fix cleanup in few test cases

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597539#comment-14597539
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8579:


GitHub user gauravaradhye opened a pull request:

https://github.com/apache/cloudstack/pull/511

CLOUDSTACK-8579: Fix cleanup operation in few test cases

Cleanup list should be declared before returning from setUpClass() so that 
tearDownClass() works properly.

Currently the test cases are failing with Class does not have cleanup 
attribute.

Tested with passing LXC as hypervisor and test cases skipped successfully 
without giving any error.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gauravaradhye/cloudstack 8579

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/511.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #511


commit d7feb1a54027b73cba3e1ffc3d09d97b072335e2
Author: Gaurav Aradhye gaurav.arad...@clogeny.com
Date:   2015-06-23T11:54:05Z

CLOUDSTACK-8579: Fix cleanup operation in few test cases




 Fix cleanup in few test cases
 -

 Key: CLOUDSTACK-8579
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8579
 Project: CloudStack
  Issue Type: Test
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: Automation
Affects Versions: 4.6.0
Reporter: Gaurav Aradhye
Assignee: Gaurav Aradhye
  Labels: automation
 Fix For: 4.6.0


 cleanup list should be declared before returning from setUpClass() so that 
 tearDownClass() works properly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8579) Fix cleanup in few test cases

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597544#comment-14597544
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8579:


Github user sanju1010 commented on the pull request:

https://github.com/apache/cloudstack/pull/511#issuecomment-114467280
  
LGTM...Please commit


 Fix cleanup in few test cases
 -

 Key: CLOUDSTACK-8579
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8579
 Project: CloudStack
  Issue Type: Test
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: Automation
Affects Versions: 4.6.0
Reporter: Gaurav Aradhye
Assignee: Gaurav Aradhye
  Labels: automation
 Fix For: 4.6.0


 cleanup list should be declared before returning from setUpClass() so that 
 tearDownClass() works properly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8579) Fix cleanup in few test cases

2015-06-23 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597552#comment-14597552
 ] 

ASF subversion and git services commented on CLOUDSTACK-8579:
-

Commit c841d6850ec591d3545d922dbb469a7e9b9ec4de in cloudstack's branch 
refs/heads/master from [~gauravaradhye]
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c841d68 ]

CLOUDSTACK-8579: Fix cleanup operation in few test cases

Signed-off-by: Gaurav Aradhye gaurav.arad...@clogeny.com
This closes #511


 Fix cleanup in few test cases
 -

 Key: CLOUDSTACK-8579
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8579
 Project: CloudStack
  Issue Type: Test
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: Automation
Affects Versions: 4.6.0
Reporter: Gaurav Aradhye
Assignee: Gaurav Aradhye
  Labels: automation
 Fix For: 4.6.0


 cleanup list should be declared before returning from setUpClass() so that 
 tearDownClass() works properly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8579) Fix cleanup in few test cases

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597553#comment-14597553
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8579:


Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/511


 Fix cleanup in few test cases
 -

 Key: CLOUDSTACK-8579
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8579
 Project: CloudStack
  Issue Type: Test
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: Automation
Affects Versions: 4.6.0
Reporter: Gaurav Aradhye
Assignee: Gaurav Aradhye
  Labels: automation
 Fix For: 4.6.0


 cleanup list should be declared before returning from setUpClass() so that 
 tearDownClass() works properly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8577) [Automation] fixing script test/integration/component/maint/testpath_disable_enable_zone.py

2015-06-23 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14598883#comment-14598883
 ] 

ASF subversion and git services commented on CLOUDSTACK-8577:
-

Commit 49451750a965a0b37409e9661cf4e65090c3047c in cloudstack's branch 
refs/heads/master from nitt10prashant
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=4945175 ]

CLOUDSTACK-8577: fixing script testpath_disable_enable_zone


 [Automation] fixing script  
 test/integration/component/maint/testpath_disable_enable_zone.py
 

 Key: CLOUDSTACK-8577
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8577
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: marvin
Affects Versions: 4.5.1
Reporter: prashant kumar mishra
Assignee: prashant kumar mishra
 Fix For: 4.5.1


 All four  test cases was failling , so modified accordingly . 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8574) Skip testcases including data disk creation for LXC if storagePool type is not RBD

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597223#comment-14597223
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8574:


Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/488#discussion_r33011867
  
--- Diff: test/integration/component/test_stopped_vm.py ---
@@ -53,6 +54,13 @@ def setUpClass(cls):
 cls.domain = get_domain(cls.api_client)
 cls.zone = get_zone(cls.api_client, 
cls.testClient.getZoneForTests())
 
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+if cls.hypervisor.lower() == 'lxc':
+if not find_storage_pool_type(cls.apiclient, 
storagetype='rbd'):
+raise unittest.SkipTest(RBD storage type is required for 
data volumes for LXC)
--- End diff --

Done with the review changes.


 Skip testcases including data disk creation for LXC if storagePool type is 
 not RBD
 --

 Key: CLOUDSTACK-8574
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8574
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: Automation
Affects Versions: 4.5.1
Reporter: Priti Sarap
 Fix For: 4.5.1


 As LXC DATA disk is supported only on RBD StoragePool skip the testcases 
 where storagepool type is other than RBD



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread JIRA
René Moser created CLOUDSTACK-8578:
--

 Summary: listVirtualMachines does not return deleted machines when 
zone is specified
 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public (Anyone can view this level - this is the default.)
  Components: API
Affects Versions: 4.5.1, 4.4.3, 4.3.1, 4.2.1, 4.1.1
Reporter: René Moser
Priority: Minor
 Fix For: 4.6.0, 4.4.4, 4.5.2


A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
return destroyed virtual machines in the response.

There was a similar issue reported in #284 and fixed but in accidentally 
re-implemented in 
https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
  




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597326#comment-14597326
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


GitHub user resmo opened a pull request:

https://github.com/apache/cloudstack/pull/510

CLOUDSTACK-8578: fix listVirtualMachines does not return deleted mach…

…ines when zone is specified

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/resmo/cloudstack fix/CLOUDSTACK-8578

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/510.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #510


commit 477fdef6d6b07de8f0b8c4fd2a99315be054264b
Author: Rene Moser re...@apache.org
Date:   2015-06-23T08:17:25Z

CLOUDSTACK-8578: fix listVirtualMachines does not return deleted machines 
when zone is specified




 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor
 Fix For: 4.6.0, 4.4.4, 4.5.2


 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (CLOUDSTACK-8579) Fix cleanup in few test cases

2015-06-23 Thread Gaurav Aradhye (JIRA)
Gaurav Aradhye created CLOUDSTACK-8579:
--

 Summary: Fix cleanup in few test cases
 Key: CLOUDSTACK-8579
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8579
 Project: CloudStack
  Issue Type: Test
  Security Level: Public (Anyone can view this level - this is the default.)
  Components: Automation
Affects Versions: 4.6.0
Reporter: Gaurav Aradhye
Assignee: Gaurav Aradhye
 Fix For: 4.6.0


cleanup list should be declared before returning from setUpClass() so that 
tearDownClass() works properly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597447#comment-14597447
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8578:


Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/510#issuecomment-114432879
  
why should we return machines that no longer exist?


 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor
 Fix For: 4.6.0, 4.4.4, 4.5.2


 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread Daan Hoogland (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597455#comment-14597455
 ] 

Daan Hoogland commented on CLOUDSTACK-8578:
---

Rene, your proposed fix will result in the UI showing destroyed machines as 
well. You can explicitaly search for state=DESTROYED and get them. If you need 
them in one call you'll have to create a new call. This will break existing 
behavior.

 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (CLOUDSTACK-8578) listVirtualMachines does not return deleted machines when zone is specified

2015-06-23 Thread Daan Hoogland (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daan Hoogland updated CLOUDSTACK-8578:
--
Fix Version/s: (was: 4.5.2)
   (was: 4.4.4)
   (was: 4.6.0)

 listVirtualMachines does not return deleted machines when zone is specified
 ---

 Key: CLOUDSTACK-8578
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public(Anyone can view this level - this is the 
 default.) 
  Components: API
Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
Reporter: René Moser
Priority: Minor

 A REST API query command=listVirtualMachineszoneid=zone-uuid does not 
 return destroyed virtual machines in the response.
 There was a similar issue reported in #284 and fixed but in accidentally 
 re-implemented in 
 https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
   



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)