[jira] [Commented] (CLOUDSTACK-8829) Consecutive cold migration fails

2016-02-20 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8829?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15155813#comment-15155813
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8829:


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

https://github.com/apache/cloudstack/pull/797#discussion_r53559299
  
--- Diff: 
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ---
@@ -1776,19 +1773,26 @@ private void orchestrateStorageMigration(final 
String vmUuid, final StoragePool
 // If VM was cold migrated between clusters belonging to 
two different VMware DCs,
 // unregister the VM from the source host and cleanup the 
associated VM files.
 if (vm.getHypervisorType().equals(HypervisorType.VMware)) {
+Long srcClusterId = null;
+Long srcHostId = vm.getHostId() != null ? 
vm.getHostId() : vm.getLastHostId();
+if (srcHostId != null) {
+HostVO srcHost = _hostDao.findById(srcHostId);
--- End diff --

@alexandrelimassantana I don't see any problem with that **if** (line 
1778). Both variables (**srcHostId** and **srcClusterId**) are used.

The conditional at the line 1784 uses **srcClusterId**.

The method at line 1793 sends the **UnregisterVMCommand** to a given host 
with the **srcHostId**.


> Consecutive cold migration fails
> 
>
> Key: CLOUDSTACK-8829
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8829
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Maneesha
>Assignee: Maneesha
>
> The following scenario is broken:
> 1.Deploy VM and stop it
> 2.Migrate stopped VM to a different primary storage pool
> 3.Again migrate the same VM to another/same storage pool. Fails with NPE.
> java.lang.NullPointerException
> at 
> com.cloud.vm.VirtualMachineManagerImpl.orchestrateStorageMigration(VirtualMachineManagerImpl.java:1745)
> at 
> com.cloud.vm.VirtualMachineManagerImpl.orchestrateStorageMigration(VirtualMachineManagerImpl.java:4716)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:606)
> at 
> com.cloud.vm.VmWorkJobHandlerProxy.handleVmWorkJob(VmWorkJobHandlerProxy.java:107)
> at 
> com.cloud.vm.VirtualMachineManagerImpl.handleVmWorkJob(VirtualMachineManagerImpl.java:4723)
> at 
> com.cloud.vm.VmWorkJobDispatcher.runJob(VmWorkJobDispatcher.java:103)



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


[jira] [Commented] (CLOUDSTACK-6928) IOPS throttling setting isn't applied to a dinamically attached volume

2016-02-20 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-6928?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15155741#comment-15155741
 ] 

ASF GitHub Bot commented on CLOUDSTACK-6928:


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

https://github.com/apache/cloudstack/pull/1410#discussion_r53556330
  
--- Diff: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java
 ---
@@ -1006,6 +1006,19 @@ protected synchronized String 
attachOrDetachDisk(final Connect conn, final boole
 } else if (attachingDisk.getFormat() == 
PhysicalDiskFormat.RAW) {
 diskdef.defBlockBasedDisk(attachingDisk.getPath(), 
devId, DiskDef.DiskBus.VIRTIO);
 }
+
+if ((bytesReadRate != null) && (bytesReadRate > 0)) {
+diskdef.setBytesReadRate(bytesReadRate);
+}
+if ((bytesWriteRate != null) && (bytesWriteRate > 0)) {
+diskdef.setBytesWriteRate(bytesWriteRate);
+}
+if ((iopsReadRate != null) && (iopsReadRate > 0)) {
+diskdef.setIopsReadRate(iopsReadRate);
+}
+if ((iopsWriteRate != null) && (iopsWriteRate > 0)) {
--- End diff --

@ustcweizhou Those conditions coud be done by a method, where the I/O 
source (bytesReadRate , bytesWriteRate, iopsReadRate  or iopsWriteRate) is 
passed as parameter. For example:

`public boolean isIORateNotNullAndBiggerThanZero(Long io) {`
`retun (io != null) && (io > 0);`
`}`

This sugestion could allow a test case, also it makes the code more 
readable (in my opinion).

Thanks.


> IOPS throttling setting isn't applied to a dinamically attached volume
> --
>
> Key: CLOUDSTACK-6928
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-6928
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: KVM
>Affects Versions: Future, 4.4.0
> Environment: CloudStack 4.4-forward w/ KVM deployment
> Ubuntu Server 14.04
>Reporter: Yoshikazu Nojima
>Assignee: Wei Zhou
>Priority: Critical
>  Labels: KVM
>
> IOPS throttling setting is NOT applied to a volume attached while VM is 
> running.
> I confirmed the setting is applied to a volume attached while VM is stopped.
> attach volume agent log:
> 2014-06-17 19:07:22,356 DEBUG [cloud.agent.Agent] 
> (agentRequest-Handler-3:null) Processing command: 
> org.apache.cloudstack.storage.command.AttachCommand
> 2014-06-17 19:07:22,401 DEBUG [kvm.storage.KVMStorageProcessor] 
> (agentRequest-Handler-3:null) Attaching device:  type='file'>
> 
>  file='/mnt/ec7a4ea0-a11f-3ab6-89f5-6c2702e3fcf8/e01df6a7-f832-4616-a151-8ad8bcd4cf64'/>
> 
> 
> start instance agent log:
> 2014-06-17 19:10:47,984 DEBUG [kvm.resource.LibvirtComputingResource] 
> (agentRequest-Handler-1:null) starting i-2-3-VM: 
> i-2-3-VM
> 8fad689b-d63d-4802-81d9-d11acf91b879
> CentOS 5.5 (64-bit)
> 
> 
> 
> 
> 
> 
> 
> 
> /usr/bin/kvm-spice
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
>  passwd='987a62e28e4358d8'/>
> 
> 
>  file='/mnt/e70b1e09-b7e8-3f14-8779-cfb75b651119/b1967fe4-6100-4777-8804-26a349ef06ea'/>
> 
> 
> 
> 
>  file='/mnt/ec7a4ea0-a11f-3ab6-89f5-6c2702e3fcf8/92920d85-1190-43ce-af03-5d2f6a6e1004'/>
> 
> 
> 200
> 200
> 
> 
> 
> 
>  file='/mnt/ec7a4ea0-a11f-3ab6-89f5-6c2702e3fcf8/e01df6a7-f832-4616-a151-8ad8bcd4cf64'/>
> 
> 
> 200
> 200
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 1048576
> 
> 
> 
> 1
> 
> hvm
> 
> 
> 
> 
> 1000
> 
> restart
> destroy
> destroy
> 



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


[jira] [Commented] (CLOUDSTACK-9104) VM naming convention in case vmware is used

2016-02-20 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9104?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15155715#comment-15155715
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9104:


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

https://github.com/apache/cloudstack/pull/1302#discussion_r53554647
  
--- Diff: 
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
 ---
@@ -2030,12 +2030,29 @@ int getReservedCpuMHZ(VirtualMachineTO vmSpec) {
 return new String[] {datastoreDiskPath};
 }
 
-// Pair
-private Pair composeVmNames(VirtualMachineTO vmSpec) {
-String vmInternalCSName = vmSpec.getName();
-String vmNameOnVcenter = vmSpec.getName();
-if (_instanceNameFlag && vmSpec.getHostName() != null) {
-vmNameOnVcenter = vmSpec.getHostName();
+
+/**
+ * This method gemerate VM name for Vcenter and Cloudstack( when 
Hypervisor is VMware).
--- End diff --

Just a little typo in line 2035, "gemerate".

Thanks for taking the time to write the javadoc.


> VM naming convention in case vmware is used
> ---
>
> Key: CLOUDSTACK-9104
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9104
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Priyank Parihar
>
> ISSUE
> ==
> VM naming convention in case vmware is used.
> Description
> ==
> User with different account cannot create VMs with the same name, which was 
> possible earlier (I am not sure in which CCP version). That time naming 
> convention used was like this “I--”
> Currently if vm.instancename.flag is set to true the VM name will be exactly 
> as display name given. 



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


[jira] [Created] (CLOUDSTACK-9293) CloudStack 4.8 on CentOS 6: Unknown API command: addVmwareDc

2016-02-20 Thread Arash Shams (JIRA)
Arash Shams created CLOUDSTACK-9293:
---

 Summary: CloudStack 4.8 on CentOS 6: Unknown API command: 
addVmwareDc
 Key: CLOUDSTACK-9293
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9293
 Project: CloudStack
  Issue Type: Bug
  Components: VMware
Affects Versions: 4.8.0
 Environment: CentOS 6 - Official 4.8 Mirror : 
https://cloudstack.apache.org/downloads.html

Reporter: Arash Shams


Hello 

I installed CloudStack 4.8 from official rpm repository on centos 6 in this url 
: https://cloudstack.apache.org/downloads.html 

in adding zone with vmware datacenter creating error appears : 
Unknown API command: addVmwareDc

I searched for this error on jira but cant get a proper answer some of them 
compile from source some of them not centos and so on . 

CentOS release 6.7 (Final) 
vCenter 5.5.0 Server Build 3252642



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