[GitHub] [cloudstack] rhtyd commented on issue #3470: Datera storage plugin

2019-07-25 Thread GitBox
rhtyd commented on issue #3470: Datera storage plugin
URL: https://github.com/apache/cloudstack/pull/3470#issuecomment-514923090
 
 
   @swill are you lgtm on it? We can merge, the regression tests show the 
plugin is isolated enough and doesn't bring in any regression. cc 
@borisstoyanov @shwstppr @anuragaw


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd closed issue #3041: Decision making logic of which Secondary storage to be used

2019-07-25 Thread GitBox
rhtyd closed issue #3041: Decision making logic of which Secondary storage to 
be used
URL: https://github.com/apache/cloudstack/issues/3041
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3041: Decision making logic of which Secondary storage to be used

2019-07-25 Thread GitBox
rhtyd commented on issue #3041: Decision making logic of which Secondary 
storage to be used
URL: https://github.com/apache/cloudstack/issues/3041#issuecomment-514923234
 
 
   Related/dupes https://github.com/apache/cloudstack/issues/3478 closing this 
one.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange

2019-07-25 Thread GitBox
kioie commented on a change in pull request #3522: New API endpoint: 
UpdateVlanIpRange
URL: https://github.com/apache/cloudstack/pull/3522#discussion_r307143962
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/vlan/UpdateVlanIpRangeCmd.java
 ##
 @@ -0,0 +1,115 @@
+package org.apache.cloudstack.api.command.admin.vlan;
+
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.ApiErrorCode;
+import org.apache.cloudstack.api.BaseAsyncCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.ServerApiException;
+import org.apache.cloudstack.api.response.VlanIpRangeResponse;
+import org.apache.log4j.Logger;
+
+import com.cloud.dc.Vlan;
+import com.cloud.event.EventTypes;
+import com.cloud.exception.ConcurrentOperationException;
+import com.cloud.exception.ResourceAllocationException;
+import com.cloud.exception.ResourceUnavailableException;
+import com.cloud.user.Account;
+
+
+@APICommand(name = "updateVlanIpRange", description = "Updates a VLAN IP 
range.", responseObject =
+VlanIpRangeResponse.class, since = "4.9.0",
 
 Review comment:
   based on #3111 I thought we can push it back to 4.9, should I switch it to 
4.13?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
blueorangutan commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-514924601
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-187


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] wido commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange

2019-07-25 Thread GitBox
wido commented on a change in pull request #3522: New API endpoint: 
UpdateVlanIpRange
URL: https://github.com/apache/cloudstack/pull/3522#discussion_r307144321
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/vlan/UpdateVlanIpRangeCmd.java
 ##
 @@ -0,0 +1,115 @@
+package org.apache.cloudstack.api.command.admin.vlan;
+
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.ApiErrorCode;
+import org.apache.cloudstack.api.BaseAsyncCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.ServerApiException;
+import org.apache.cloudstack.api.response.VlanIpRangeResponse;
+import org.apache.log4j.Logger;
+
+import com.cloud.dc.Vlan;
+import com.cloud.event.EventTypes;
+import com.cloud.exception.ConcurrentOperationException;
+import com.cloud.exception.ResourceAllocationException;
+import com.cloud.exception.ResourceUnavailableException;
+import com.cloud.user.Account;
+
+
+@APICommand(name = "updateVlanIpRange", description = "Updates a VLAN IP 
range.", responseObject =
+VlanIpRangeResponse.class, since = "4.9.0",
 
 Review comment:
   4.9 is no longer supported nor maintained and there will be no new release.
   
   As this is a new feature it should go into 4.13


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] kioie commented on issue #3522: New API endpoint: UpdateVlanIpRange

2019-07-25 Thread GitBox
kioie commented on issue #3522: New API endpoint: UpdateVlanIpRange
URL: https://github.com/apache/cloudstack/pull/3522#issuecomment-514926699
 
 
   > There is no IPv6 support it seems in this change nor has the UI been 
updated. Correct?
   
   Wanted to separate UI changes from api changes, easier for review and 
approval


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
rhtyd commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-514928213
 
 
   @blueorangutan test


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
blueorangutan commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-514928397
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] digitalgust opened a new issue #3523: 4.11.3.0 create ceph primary storage error

2019-07-25 Thread GitBox
digitalgust opened a new issue #3523: 4.11.3.0 create ceph primary storage error
URL: https://github.com/apache/cloudstack/issues/3523
 
 
   
   
   # ISSUE TYPE
   
* Bug Report
   
   
   # COMPONENT NAME
   
   ~~~
   Core
   ~~~
   
   # CLOUDSTACK VERSION
   
   
   ~~~
   4.11.3.0
   ~~~
   
   # CONFIGURATION
   
   no more
   
   # OS / ENVIRONMENT
   
   centos 7.6
   
   # SUMMARY
   
   Add ceph as primary storage error when create zone 
   
   
   ```
   2019-07-25 15:33:23,326 DEBUG [c.c.a.ApiServlet] 
(qtp504527234-11:ctx-9f0958ff) (logid:a1b00ebd) ===START===  10.0.1.11 -- GET  
command=createStoragePool&scope=zone&zoneid=33e90deb-64c0-44ac-9e05-88d1d71347b9&hypervisor=KVM&name=ZONE1_CEPH1&provider=DefaultPrimary&url=rbd%3A%2F%2Fcloudstack%3AAQDllC5dgh72MhAA56Lz2RxC9r4SzL4qplXCyg%3D%3D%40cephmon.csdomain%2Fcloudstack&response=json&_=1564039019055
   2019-07-25 15:33:23,330 DEBUG [c.c.a.ApiServer] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) CIDRs from which 
account 'Acct[738fea37-ae83-11e9-9730-000c29a29fa5-admin]' is allowed to 
perform API calls: 0.0.0.0/0,::/0
   2019-07-25 15:33:23,343 DEBUG 
[o.a.c.s.d.l.CloudStackPrimaryDataStoreLifeCycleImpl] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) createPool Params 
@ scheme - rbd storageHost - cephmon.csdomain hostPath - /cloudstack port - -1
   2019-07-25 15:33:23,351 DEBUG 
[o.a.c.s.d.l.CloudStackPrimaryDataStoreLifeCycleImpl] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) In createPool. 
Attaching the pool to each of the hosts.
   2019-07-25 15:33:23,355 DEBUG [c.c.s.StorageManagerImpl] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) Adding pool null 
to  host 5
   2019-07-25 15:33:23,358 DEBUG [c.c.a.t.Request] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) Seq 
5-200691658394698070: Sending  { Cmd , MgmtId: 52238131109, via: 5(host2), Ver: 
v1, Flags: 100011, 
[{"com.cloud.agent.api.ModifyStoragePoolCommand":{"add":true,"pool":{"id":3,"uuid":"a8ed569f-960c-3943-92fc-8f9a389f9c55","host":"cephmon.csdomain","path":"cloudstack","userInfo":"cloudstack:AQDllC5dgh72MhAA56Lz2RxC9r4SzL4qplXCyg==","port":6789,"type":"RBD"},"localPath":"/mnt//22eb6805-e0fb-3c2e-8124-437272e2d4e8","wait":0}}]
 }
   2019-07-25 15:33:23,455 DEBUG [c.c.a.t.Request] 
(AgentManager-Handler-12:null) (logid:) Seq 5-200691658394698070: Processing:  
{ Ans: , MgmtId: 52238131109, via: 5, Ver: v1, Flags: 10, 
[{"com.cloud.agent.api.Answer":{"result":false,"details":"com.cloud.utils.exception.CloudRuntimeException:
 Failed to create storage pool: a8ed569f-960c-3943-92fc-8f9a389f9c55\n\tat 
com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePool(LibvirtStorageAdaptor.java:550)\n\tat
 
com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:322)\n\tat
 
com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:316)\n\tat
 
com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolCommandWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:42)\n\tat
 
com.cloud.hypervisor.kvm.resource.wrapper.LibvirtModifyStoragePoolCommandWrapper.execute(LibvirtModifyStoragePoolCommandWrapper.java:35)\n\tat
 
com.cloud.hypervisor.kvm.resource.wrapper.LibvirtRequestWrapper.execute(LibvirtRequestWrapper.java:78)\n\tat
 
com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(LibvirtComputingResource.java:1450)\n\tat
 com.cloud.agent.Agent.processRequest(Agent.java:640)\n\tat 
com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:1053)\n\tat 
com.cloud.utils.nio.Task.call(Task.java:83)\n\tat 
com.cloud.utils.nio.Task.call(Task.java:29)\n\tat 
java.util.concurrent.FutureTask.run(FutureTask.java:266)\n\tat 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)\n\tat
 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)\n\tat
 java.lang.Thread.run(Thread.java:748)\n","wait":0}}] }
   2019-07-25 15:33:23,455 DEBUG [c.c.a.t.Request] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) Seq 
5-200691658394698070: Received:  { Ans: , MgmtId: 52238131109, via: 5(host2), 
Ver: v1, Flags: 10, { Answer } }
   2019-07-25 15:33:23,456 DEBUG [c.c.a.m.AgentManagerImpl] 
(qtp504527234-11:ctx-9f0958ff ctx-27596fcf) (logid:a1b00ebd) Details from 
executing class com.cloud.agent.api.ModifyStoragePoolCommand: 
com.cloud.utils.exception.CloudRuntimeException: Failed to create storage pool: 
a8ed569f-960c-3943-92fc-8f9a389f9c55
at 
com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePool(LibvirtStorageAdaptor.java:550)
at 
com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:322)
at 
com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:316)
at 
com.cloud.hypervisor.kvm.resour

[GitHub] [cloudstack] kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange

2019-07-25 Thread GitBox
kioie commented on a change in pull request #3522: New API endpoint: 
UpdateVlanIpRange
URL: https://github.com/apache/cloudstack/pull/3522#discussion_r307155030
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/vlan/UpdateVlanIpRangeCmd.java
 ##
 @@ -0,0 +1,115 @@
+package org.apache.cloudstack.api.command.admin.vlan;
+
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.ApiErrorCode;
+import org.apache.cloudstack.api.BaseAsyncCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.ServerApiException;
+import org.apache.cloudstack.api.response.VlanIpRangeResponse;
+import org.apache.log4j.Logger;
+
+import com.cloud.dc.Vlan;
+import com.cloud.event.EventTypes;
+import com.cloud.exception.ConcurrentOperationException;
+import com.cloud.exception.ResourceAllocationException;
+import com.cloud.exception.ResourceUnavailableException;
+import com.cloud.user.Account;
+
+
+@APICommand(name = "updateVlanIpRange", description = "Updates a VLAN IP 
range.", responseObject =
+VlanIpRangeResponse.class, since = "4.9.0",
+requestHasSensitiveInfo = false, responseHasSensitiveInfo = false)
+public class UpdateVlanIpRangeCmd extends BaseAsyncCmd {
+public static final Logger s_logger = 
Logger.getLogger(UpdateVlanIpRangeCmd.class.getName());
+
+private static final String s_name = "updatevlaniprangeresponse";
+
+/
+ API parameters /
+/
+
+
+@Parameter(name = ApiConstants.ID, type = CommandType.UUID, entityType = 
VlanIpRangeResponse.class, required = true,
+description = "the UUID of the VLAN IP range")
+private Long id;
+
+@Parameter(name = ApiConstants.END_IP, type = CommandType.STRING, 
description = "the ending IP address in the VLAN IP range")
+private String endIp;
+
+@Parameter(name = ApiConstants.GATEWAY, type = CommandType.STRING, 
description = "the gateway of the VLAN IP range")
+private String gateway;
+
+@Parameter(name = ApiConstants.NETMASK, type = CommandType.STRING, 
description = "the netmask of the VLAN IP range")
+private String netmask;
+
+@Parameter(name = ApiConstants.START_IP, type = CommandType.STRING, 
description = "the beginning IP address in the VLAN IP range")
+private String startIp;
+
+@Parameter(name = ApiConstants.VLAN, type = CommandType.INTEGER, 
description = "Optional. the vlan the ip range sits on")
+private Integer vlan;
+
 
 Review comment:
   Added ipv6 support


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] kioie commented on a change in pull request #3522: New API endpoint: UpdateVlanIpRange

2019-07-25 Thread GitBox
kioie commented on a change in pull request #3522: New API endpoint: 
UpdateVlanIpRange
URL: https://github.com/apache/cloudstack/pull/3522#discussion_r307155230
 
 

 ##
 File path: 
api/src/org/apache/cloudstack/api/command/admin/vlan/UpdateVlanIpRangeCmd.java
 ##
 @@ -0,0 +1,115 @@
+package org.apache.cloudstack.api.command.admin.vlan;
+
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.ApiErrorCode;
+import org.apache.cloudstack.api.BaseAsyncCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.ServerApiException;
+import org.apache.cloudstack.api.response.VlanIpRangeResponse;
+import org.apache.log4j.Logger;
+
+import com.cloud.dc.Vlan;
+import com.cloud.event.EventTypes;
+import com.cloud.exception.ConcurrentOperationException;
+import com.cloud.exception.ResourceAllocationException;
+import com.cloud.exception.ResourceUnavailableException;
+import com.cloud.user.Account;
+
+
+@APICommand(name = "updateVlanIpRange", description = "Updates a VLAN IP 
range.", responseObject =
+VlanIpRangeResponse.class, since = "4.9.0",
 
 Review comment:
   Alright, changed to 4.13


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] kioie edited a comment on issue #3522: New API endpoint: UpdateVlanIpRange

2019-07-25 Thread GitBox
kioie edited a comment on issue #3522: New API endpoint: UpdateVlanIpRange
URL: https://github.com/apache/cloudstack/pull/3522#issuecomment-514926699
 
 
   > There is no IPv6 support it seems in this change nor has the UI been 
updated. Correct?
   
   Wanted to separate UI changes from api changes, easier for review and 
approval, I'll add UI on a separate PR


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] digitalgust commented on issue #3509: 4.12 add host fail , using tls connection

2019-07-25 Thread GitBox
digitalgust commented on issue #3509: 4.12 add host fail ,using tls connection
URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-514946131
 
 
   test 4.11.3.0,   the problem same as 4.12
   hypervisor add unreliable ,
   i have 2 host for hypervisor in my test environment.
   add fisrst host , it always shown "alert“, then remove the first host,  mgmt 
note success message,  then add the second host, second show "alert" ,!!! the 
first host turn "UP", but the first host is success deleted a momont ago.
   or keep it for serial hours in "alert" state,  the hosts convert "UP".
   when host is in "alert" state, it cant't connect to mgmt, but in the host 
can ping mgmt, and can mgmt 
port 8250 is normal,  the same time , there is another host in "UP" state .
   i  can abstractly confirm the host's network is connectable . 
   Puzzling real .
   
   this is agent log  ,the host in "alert" state 
   ```
   2019-07-25 16:03:40,152 INFO  [cloud.agent.Agent] (Agent Timer:null) 
(logid:) Reconnecting to host:192.168.100.1
   2019-07-25 16:03:40,153 INFO  [utils.nio.NioClient] (Agent Timer:null) 
(logid:) Connecting to 192.168.100.1:8250
   2019-07-25 16:03:40,154 INFO  [utils.nio.Link] (Agent Timer:null) (logid:) 
Conf file found: /etc/cloudstack/agent/agent.properties
   2019-07-25 16:03:40,289 INFO  [utils.nio.NioClient] (Agent Timer:null) 
(logid:) SSL: Handshake done
   2019-07-25 16:03:40,290 INFO  [utils.nio.NioClient] (Agent Timer:null) 
(logid:) Connected to 192.168.100.1:8250
   2019-07-25 16:03:40,290 DEBUG [kvm.resource.LibvirtConnection] 
(Agent-Handler-1:null) (logid:) Looking for libvirtd connection at: 
qemu:///system
   2019-07-25 16:03:40,296 DEBUG [kvm.resource.LibvirtCapXMLParser] 
(Agent-Handler-1:null) (logid:) Found /usr/libexec/qemu-kvm as a suiteable 
emulator
   2019-07-25 16:03:40,296 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing: /bin/bash -c qemu-img --help|grep 
convert 
   2019-07-25 16:03:40,297 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing while with timeout : 180
   2019-07-25 16:03:40,314 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Execution is successful.
   2019-07-25 16:03:40,315 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:)   convert [--object objectdef] [--image-opts] 
[-c] [-p] [-q] [-n] [-f fmt] [-t cache] [-T src_cache] [-O output_fmt] [-o 
options] [-s snapshot_id_or_name] [-l snapshot_param] [-S sparse_size] [-m 
num_coroutines] [-W] filename [filename2 [...]] output_filename
   options are: 'none', 'writeback' (default, except for convert), 
'writethrough',
   'directsync' and 'unsafe' (default for convert)
   Parameters to convert subcommand:
 '-m' specifies how many coroutines work in parallel during the convert
   
   2019-07-25 16:03:40,315 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) cpus=8, speed=4500, ram=7378771968, 
_dom0MinMem=1073741824, _dom0OvercommitMem=0, cpu sockets=1
   2019-07-25 16:03:40,315 DEBUG [cloud.resource.ServerResourceBase] 
(Agent-Handler-1:null) (logid:) Parameters for private nic: 192.168.100.50 - 
1c:69:7a:00:e2:76-255.255.255.0
   2019-07-25 16:03:40,315 DEBUG [cloud.resource.ServerResourceBase] 
(Agent-Handler-1:null) (logid:) Parameters for storage nic: 192.168.100.50 - 
1c:69:7a:00:e2:76-255.255.255.0
   2019-07-25 16:03:40,315 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing: 
/usr/share/cloudstack-common/scripts/vm/hypervisor/versions.sh 
   2019-07-25 16:03:40,316 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing while with timeout : 180
   2019-07-25 16:03:40,324 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Execution is successful.
   2019-07-25 16:03:40,330 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing: sudo grep InitiatorName= 
/etc/iscsi/initiatorname.iscsi 
   2019-07-25 16:03:40,331 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing while with timeout : 360
   2019-07-25 16:03:40,346 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Execution is successful.
   2019-07-25 16:03:40,346 INFO  [kvm.storage.LibvirtStorageAdaptor] 
(Agent-Handler-1:null) (logid:) Attempting to create storage pool 
43a0fdb9-9faf-406b-ba0b-388aa1bbc7d4 (Filesystem) in libvirt
   2019-07-25 16:03:40,346 DEBUG [kvm.resource.LibvirtConnection] 
(Agent-Handler-1:null) (logid:) Looking for libvirtd connection at: 
qemu:///system
   2019-07-25 16:03:40,347 INFO  [kvm.storage.LibvirtStorageAdaptor] 
(Agent-Handler-1:null) (logid:) Found existing defined storage pool 
43a0fdb9-9faf-406b-ba0b-388aa1bbc7d4, using it.
   2019-07-25 16:03:40,347 INFO  [kvm.storage.LibvirtStorageAdaptor] 
(Agent-Handler-1:null) (logi

[GitHub] [cloudstack] digitalgust edited a comment on issue #3509: 4.12 add host fail , using tls connection

2019-07-25 Thread GitBox
digitalgust edited a comment on issue #3509: 4.12 add host fail ,using tls 
connection
URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-514946131
 
 
   test 4.11.3.0,   the problem same as 4.12
   hypervisor add unreliable ,
   i have 2 hosts for hypervisor in my test environment.
   add fisrst host , it always shown "alert“, then remove the first host,  mgmt 
note success message,  then add the second host, second show "alert" ,!!! the 
first host turn "UP", but the first host is success deleted a momont ago.
   or keep it for serial hours in "alert" state,  the host turn "UP".
   when host in "alert" state,  agent cant't connect to mgmt, but in the host 
ping mgmt normal, and  mgmt 
port 8250 is working,  the same time , there is another host in "UP" state .
   i  can abstractly confirm the host's network is connectable . 
   Puzzling .
   
   this is agent log  ,the host in "alert" state 
   ```
   2019-07-25 16:03:40,152 INFO  [cloud.agent.Agent] (Agent Timer:null) 
(logid:) Reconnecting to host:192.168.100.1
   2019-07-25 16:03:40,153 INFO  [utils.nio.NioClient] (Agent Timer:null) 
(logid:) Connecting to 192.168.100.1:8250
   2019-07-25 16:03:40,154 INFO  [utils.nio.Link] (Agent Timer:null) (logid:) 
Conf file found: /etc/cloudstack/agent/agent.properties
   2019-07-25 16:03:40,289 INFO  [utils.nio.NioClient] (Agent Timer:null) 
(logid:) SSL: Handshake done
   2019-07-25 16:03:40,290 INFO  [utils.nio.NioClient] (Agent Timer:null) 
(logid:) Connected to 192.168.100.1:8250
   2019-07-25 16:03:40,290 DEBUG [kvm.resource.LibvirtConnection] 
(Agent-Handler-1:null) (logid:) Looking for libvirtd connection at: 
qemu:///system
   2019-07-25 16:03:40,296 DEBUG [kvm.resource.LibvirtCapXMLParser] 
(Agent-Handler-1:null) (logid:) Found /usr/libexec/qemu-kvm as a suiteable 
emulator
   2019-07-25 16:03:40,296 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing: /bin/bash -c qemu-img --help|grep 
convert 
   2019-07-25 16:03:40,297 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing while with timeout : 180
   2019-07-25 16:03:40,314 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Execution is successful.
   2019-07-25 16:03:40,315 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:)   convert [--object objectdef] [--image-opts] 
[-c] [-p] [-q] [-n] [-f fmt] [-t cache] [-T src_cache] [-O output_fmt] [-o 
options] [-s snapshot_id_or_name] [-l snapshot_param] [-S sparse_size] [-m 
num_coroutines] [-W] filename [filename2 [...]] output_filename
   options are: 'none', 'writeback' (default, except for convert), 
'writethrough',
   'directsync' and 'unsafe' (default for convert)
   Parameters to convert subcommand:
 '-m' specifies how many coroutines work in parallel during the convert
   
   2019-07-25 16:03:40,315 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) cpus=8, speed=4500, ram=7378771968, 
_dom0MinMem=1073741824, _dom0OvercommitMem=0, cpu sockets=1
   2019-07-25 16:03:40,315 DEBUG [cloud.resource.ServerResourceBase] 
(Agent-Handler-1:null) (logid:) Parameters for private nic: 192.168.100.50 - 
1c:69:7a:00:e2:76-255.255.255.0
   2019-07-25 16:03:40,315 DEBUG [cloud.resource.ServerResourceBase] 
(Agent-Handler-1:null) (logid:) Parameters for storage nic: 192.168.100.50 - 
1c:69:7a:00:e2:76-255.255.255.0
   2019-07-25 16:03:40,315 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing: 
/usr/share/cloudstack-common/scripts/vm/hypervisor/versions.sh 
   2019-07-25 16:03:40,316 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing while with timeout : 180
   2019-07-25 16:03:40,324 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Execution is successful.
   2019-07-25 16:03:40,330 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing: sudo grep InitiatorName= 
/etc/iscsi/initiatorname.iscsi 
   2019-07-25 16:03:40,331 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Executing while with timeout : 360
   2019-07-25 16:03:40,346 DEBUG [kvm.resource.LibvirtComputingResource] 
(Agent-Handler-1:null) (logid:) Execution is successful.
   2019-07-25 16:03:40,346 INFO  [kvm.storage.LibvirtStorageAdaptor] 
(Agent-Handler-1:null) (logid:) Attempting to create storage pool 
43a0fdb9-9faf-406b-ba0b-388aa1bbc7d4 (Filesystem) in libvirt
   2019-07-25 16:03:40,346 DEBUG [kvm.resource.LibvirtConnection] 
(Agent-Handler-1:null) (logid:) Looking for libvirtd connection at: 
qemu:///system
   2019-07-25 16:03:40,347 INFO  [kvm.storage.LibvirtStorageAdaptor] 
(Agent-Handler-1:null) (logid:) Found existing defined storage pool 
43a0fdb9-9faf-406b-ba0b-388aa1bbc7d4, using it.
   2019-07-25 16:03:40,347 INFO  [kvm.storage.LibvirtStorageAdaptor] 
(Agent-Handler-1:null) (lo

[GitHub] [cloudstack] rhtyd merged pull request #3470: Datera storage plugin

2019-07-25 Thread GitBox
rhtyd merged pull request #3470: Datera storage plugin
URL: https://github.com/apache/cloudstack/pull/3470
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-documentation] branch L2-networks-doc-update created (now f975a0c)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch L2-networks-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


  at f975a0c  Merge pull request #56 from apache/doc-update-iso-from-local

No new revisions were added by this update.



[cloudstack-documentation] branch L2-networks-doc-update updated: L2-networks-gui image added

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch L2-networks-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/L2-networks-doc-update by this 
push:
 new d8800de  L2-networks-gui image added
d8800de is described below

commit d8800dea2f1419f9c473ce0a17ade07f4fb993fb
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 10:49:14 2019 +0200

L2-networks-gui image added
---
 source/_static/images/L2-networks-gui.JPG | Bin 0 -> 38506 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/source/_static/images/L2-networks-gui.JPG 
b/source/_static/images/L2-networks-gui.JPG
new file mode 100644
index 000..239e6bc
Binary files /dev/null and b/source/_static/images/L2-networks-gui.JPG differ



[cloudstack-documentation] branch L2-networks-doc-update updated: example GUI dialog box added

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch L2-networks-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/L2-networks-doc-update by this 
push:
 new 1e7d6b9  example GUI dialog box added
1e7d6b9 is described below

commit 1e7d6b94e6573090b7accc0f81902bffb03f54fc
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 10:57:38 2019 +0200

example GUI dialog box added
---
 source/adminguide/networking.rst | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/source/adminguide/networking.rst b/source/adminguide/networking.rst
index ec8d2d3..b8dd66d 100644
--- a/source/adminguide/networking.rst
+++ b/source/adminguide/networking.rst
@@ -105,6 +105,10 @@ IP addresses.
 -  Userdata and metadata can be passed to the VM using a config drive
(which must be enabled in the network service offering)
 
+Example GUI dialog box (for a regular user account) is shown below:
+
+|L2-networks-gui.JPG|
+
 
 Runtime Allocation of Virtual Network Resources
 ~~~
@@ -438,3 +442,5 @@ To create a network offering:
 
 #. Click Add.
 
+.. |L2-networks-gui.JPG| image:: /_static/images/L2-networks-gui.JPG
+   :alt: Creating L2 network from GUI



[GitHub] [cloudstack] rhtyd commented on issue #3509: 4.12 add host fail , using tls connection

2019-07-25 Thread GitBox
rhtyd commented on issue #3509: 4.12 add host fail ,using tls connection
URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-514965919
 
 
   Are you running multiple management server(s) are starting them concurrently 
@digitalgust ? I'll test against 4.11.2.0 and 4.11.3.0.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd edited a comment on issue #3509: 4.12 add host fail , using tls connection

2019-07-25 Thread GitBox
rhtyd edited a comment on issue #3509: 4.12 add host fail ,using tls connection
URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-514965919
 
 
   Are you running multiple management server(s) are starting them concurrently 
@digitalgust ? Can you also share details about your env, like number of 
secondary and primary storages, networking etc?
   
   I'll test against 4.11.2.0 and 4.11.3.0.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-documentation] branch L2-networks-doc-update updated (1e7d6b9 -> b9dddf2)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch L2-networks-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


from 1e7d6b9  example GUI dialog box added
 add b9dddf2  Add files via upload

No new revisions were added by this update.

Summary of changes:
 source/_static/images/L2-networks-gui.JPG | Bin 38506 -> 24398 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)



[cloudstack-documentation] branch master updated (f975a0c -> b0bd641)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


from f975a0c  Merge pull request #56 from apache/doc-update-iso-from-local
 add d8800de  L2-networks-gui image added
 add 1e7d6b9  example GUI dialog box added
 add b9dddf2  Add files via upload
 new b0bd641  Merge pull request #57 from apache/L2-networks-doc-update

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 source/_static/images/L2-networks-gui.JPG | Bin 0 -> 24398 bytes
 source/adminguide/networking.rst  |   6 ++
 2 files changed, 6 insertions(+)
 create mode 100644 source/_static/images/L2-networks-gui.JPG



[cloudstack-documentation] 01/01: Merge pull request #57 from apache/L2-networks-doc-update

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git

commit b0bd641f8eee33bc52dd7fee6865b00261931f1c
Merge: f975a0c b9dddf2
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 11:31:35 2019 +0200

Merge pull request #57 from apache/L2-networks-doc-update

Minor L2 networks doc update

 source/_static/images/L2-networks-gui.JPG | Bin 0 -> 24398 bytes
 source/adminguide/networking.rst  |   6 ++
 2 files changed, 6 insertions(+)



[cloudstack-documentation] branch L2-networks-doc-update deleted (was b9dddf2)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch L2-networks-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


 was b9dddf2  Add files via upload

The revisions that were on this branch are still contained in
other references; therefore, this change does not discard any commits
from the repository.



[GitHub] [cloudstack] digitalgust commented on issue #3509: 4.12 add host fail , using tls connection

2019-07-25 Thread GitBox
digitalgust commented on issue #3509: 4.12 add host fail ,using tls connection
URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-514987620
 
 
   1 mgmt  : centos 7.6 /intel7500U/2G  192.168.100.1 /2nic
   1 secondary : centos 7.6 / intel7500U/2G  192.168.100.20 /1nic
   2 Hypervisor : centos 7.6/ intel8559U8259U/8G 192.168.100.50,192.168.100.51 
/libvirtd 4.5/qemu2.9
   2 ceph: centos 7.6 /intel8259U/8G 192.168.100.10,192.168.100.11 ceph 14
   
   cloudstack 4.12 / 4.11.3.0
   advance network
   KVM
   guest CIDR 10.1.1.0/16
   manage_network & storage_network on cloudbr0  connect to hardware switch 0
   public_newwork & guest_network on cloudbr1 connect to hardware switch 1 
   public traffic: gateway 10.0.99.254,  255.255.0.0  start ip:10.0.99.100 end 
ip :10.0.99.199
   pod: gateway 192.168.100.254, mask 255.255.255.0  reserve ip : 
192.168.100.100-192.168.100.199
   vlan : 1000-1009
   storage traffic: gateway192.168.100.254, start ip: 192.168.100.10- 
192.168.100.19
   host : 192.168.100.50
   primary storage : nfs or ceph both work
   secondary storage: nfs work
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] digitalgust edited a comment on issue #3509: 4.12 add host fail , using tls connection

2019-07-25 Thread GitBox
digitalgust edited a comment on issue #3509: 4.12 add host fail ,using tls 
connection
URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-514987620
 
 
   1 mgmt  : centos 7.6 /intel7500U/2G  192.168.100.1 /2nic
   1 secondary : centos 7.6 / intel7500U/2G  192.168.100.20 /1nic
   2 Hypervisor : centos 7.6/ intel8559U8259U/8G 192.168.100.50,192.168.100.51 
/libvirtd 4.5/qemu2.9
   2 ceph: centos 7.6 /intel8259U/8G 192.168.100.10,192.168.100.11 ceph 14
   
   cloudstack 4.12 / 4.11.3.0
   advance network
   KVM
   guest CIDR 10.1.1.0/16
   manage_network & storage_network on cloudbr0  connect to hardware switch 0
   public_newwork & guest_network on cloudbr1 connect to hardware switch 1 
   public traffic: gateway 10.0.99.254,  255.255.0.0  start ip:10.0.99.100 end 
ip :10.0.99.199
   pod: gateway 192.168.100.254, mask 255.255.255.0  reserve ip : 
192.168.100.100-192.168.100.199
   vlan : 1000-1009
   storage traffic: gateway192.168.100.254, start ip: 192.168.100.10- 
192.168.100.19
   host : 192.168.100.50
   primary storage : nfs or ceph both work 4.12,  ceph dont work 4.11.3
   secondary storage: nfs work
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-documentation] branch hiding-columns-and-keyboard-labeling-doc-update created (now 64e0667)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch 
hiding-columns-and-keyboard-labeling-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


  at 64e0667  hiding columns and keyboard labeling doc update

This branch includes the following new commits:

 new 64e0667  hiding columns and keyboard labeling doc update

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[cloudstack-documentation] 01/01: hiding columns and keyboard labeling doc update

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch 
hiding-columns-and-keyboard-labeling-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git

commit 64e06671684695c36b0d1e0c9b7f109f7039a16b
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 15:21:13 2019 +0200

hiding columns and keyboard labeling doc update
---
 source/adminguide/ui.rst | 88 
 1 file changed, 88 insertions(+)

diff --git a/source/adminguide/ui.rst b/source/adminguide/ui.rst
index 7c7fd81..a4566cb 100644
--- a/source/adminguide/ui.rst
+++ b/source/adminguide/ui.rst
@@ -169,5 +169,93 @@ new, unique value.
 #. Type the new password, and click OK.
 
 
+Modifying the UI
+
+
+Root Administrators can modify some aspect of the UI, like:
+
+   -  Changing the keyboard names/labels (in the "Add Instance" wizard only)
+   -  Changing the text and the title in the "About" dialog box
+   -  Changing the Help link
+   -  Changing the application title on the browser tab
+   -  Hiding some columns in the Instance Metrics and Volume Metrics 
tables/views
+ 
+This can be done by editing the file 
"/usr/share/cloudstack-management/webapp/config.js" on your management 
server(s).
+After the file change, you do need to clear your browser cache. Hiding columns 
in Instance Metrics and Volume Metrics tables/views is only applicable to end 
users - i.e. those are always visible to Root Administrators.
+
+This is the default config.js content (comments ommitted for brevity):
+
+.. code:: javascript
+
+   cloudStackOptions = {
+  aboutText: "label.app.name", // This is the text shown in the 'About' box
+  aboutTitle: "label.about.app", // This is the Application 'Title' shown 
in the 'About' box
+  docTitle: "label.app.name", // This is the Application 'Title' shown on 
browser tab.
+  helpURL: "http://docs.cloudstack.apache.org/";, // This is the URL that 
opens when users click Help
+  keyboardOptions: {
+ "us": "label.standard.us.keyboard",
+ "uk": "label.uk.keyboard",
+ "fr": "label.french.azerty.keyboard",
+ "jp": "label.japanese.keyboard",
+ "sc": "label.simplified.chinese.keyboard"
+  },
+  hiddenFields: { // Fields to be hidden only for users in the tables below
+ "metrics.instances": [], // Options - "name", "state", "ipaddress", 
"zonename", "cpuused", "memused", "network", "disk"
+ "metrics.volumes": [] // Options - "name", "state", "vmname", 
"sizegb", "physicalsize", "utilization", "storagetype", "storage"
+  }
+   };
+
+Example of a changed confi.js file, is given bellow, as well as the apropriate 
screenshots which reflect (most of) those changes. Make sure that you are 
logged in as end user account.
+
+.. code:: javascript
+
+   cloudStackOptions = {
+  aboutText: "Custom About text!", // This is the text shown in the 
'About' box
+  aboutTitle: "Custom About Title!", // This is the Application 'Title' 
shown in the 'About' box
+  docTitle: "MyCloud", // This is the Application 'Title' shown on browser 
tab.
+  helpURL: "http://help.mycloud.com/";, // This is the URL that opens when 
users click Help
+  keyboardOptions: {
+ "us": "US",
+ "uk": "UK",
+ "fr": "FR",
+ "jp": "JP",
+ "sc": "CN"
+  },
+  hiddenFields: { // Fields to be hidden only for users in the tables below
+ "metrics.instances": ["zonename"], // Options - "name", "state", 
"ipaddress", "zonename", "cpuused", "memused", "network", "disk"
+ "metrics.volumes": ["utilization"] // Options - "name", "state", 
"vmname", "sizegb", "physicalsize", "utilization", "storagetype", "storage"
+  }
+   };
+
+|about-cloudstack-customized.JPG|
+
+Notice custom title and text
+
+
+|add-instance-keyboards-customized.JPG|
+
+Notice short keyboard names (based on country code)
+
+
+|instance-metrics-customized-view.JPG|
+
+Notice missing "Zone" column
+
+
+|volume-metrics-customized-view.JPG|
+
+Notice missing "Utilisation" column
+
+
+
+
 .. |change-password.png| image:: /_static/images/change-password.png
:alt: button to change a user's password
+.. |volume-metrics-customized-view.JPG| image:: 
/_static/images/volume-metrics-customized-view.JPG
+   :alt: Notice missing "Utilisation" column
+.. |instance-metrics-customized-view.JPG| image:: 
/_static/images/instance-metrics-customized-view.JPG
+   :alt: Notice missing "Zone" column
+.. |about-cloudstack-customized.JPG| image:: 
/_static/images/about-cloudstack-customized.JPG
+   :alt: Notice custom title and text   
+.. |add-instance-keyboards-customized.JPG| image:: 
/_static/images/add-instance-keyboards-customized.JPG
+   :alt: Notice short keyboard names (based on country code)   



[cloudstack-documentation] branch hiding-columns-and-keyboard-labeling-doc-update updated: images

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch 
hiding-columns-and-keyboard-labeling-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to 
refs/heads/hiding-columns-and-keyboard-labeling-doc-update by this push:
 new 6d3d819  images
6d3d819 is described below

commit 6d3d8196db76e183cbb1cf9d16dab7ccda66f82d
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 15:22:36 2019 +0200

images
---
 source/_static/images/about-cloudstack-customized.JPG   | Bin 0 -> 14176 bytes
 .../images/add-instance-keyboards-customized.JPG| Bin 0 -> 42977 bytes
 .../_static/images/instance-metrics-customized-view.JPG | Bin 0 -> 35848 bytes
 .../_static/images/volume-metrics-customized-view.JPG   | Bin 0 -> 34893 bytes
 4 files changed, 0 insertions(+), 0 deletions(-)

diff --git a/source/_static/images/about-cloudstack-customized.JPG 
b/source/_static/images/about-cloudstack-customized.JPG
new file mode 100644
index 000..d314cd4
Binary files /dev/null and 
b/source/_static/images/about-cloudstack-customized.JPG differ
diff --git a/source/_static/images/add-instance-keyboards-customized.JPG 
b/source/_static/images/add-instance-keyboards-customized.JPG
new file mode 100644
index 000..35b5b8d
Binary files /dev/null and 
b/source/_static/images/add-instance-keyboards-customized.JPG differ
diff --git a/source/_static/images/instance-metrics-customized-view.JPG 
b/source/_static/images/instance-metrics-customized-view.JPG
new file mode 100644
index 000..bc0f829
Binary files /dev/null and 
b/source/_static/images/instance-metrics-customized-view.JPG differ
diff --git a/source/_static/images/volume-metrics-customized-view.JPG 
b/source/_static/images/volume-metrics-customized-view.JPG
new file mode 100644
index 000..fff6935
Binary files /dev/null and 
b/source/_static/images/volume-metrics-customized-view.JPG differ



[GitHub] [cloudstack] blueorangutan commented on issue #3517: 4.13.0.0/master: health checks smoketests

2019-07-25 Thread GitBox
blueorangutan commented on issue #3517: 4.13.0.0/master: health checks 
smoketests
URL: https://github.com/apache/cloudstack/pull/3517#issuecomment-515045260
 
 
   Trillian test result (tid-218)
   Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 31481 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3517-t218-xenserver-71.zip
   Intermittent failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
   Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
   Smoke tests completed. 75 look OK, 2 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_03_list_snapshots | `Error` | 0.05 | test_list_ids_parameter.py
   test_01_scale_vm | `Failure` | 16.59 | test_scale_vm.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] Doni7722 commented on issue #3520: VM's ROOT disk created from ISO, honors Service offering instead of Disk offering

2019-07-25 Thread GitBox
Doni7722 commented on issue #3520: VM's ROOT disk created from ISO, honors 
Service offering instead of Disk offering
URL: https://github.com/apache/cloudstack/issues/3520#issuecomment-515050706
 
 
   Did test this with 4.11.3 and there it's reproducible.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] andrijapanicsb commented on issue #3415: Fix interval descrption

2019-07-25 Thread GitBox
andrijapanicsb commented on issue #3415: Fix interval descrption
URL: https://github.com/apache/cloudstack/pull/3415#issuecomment-515055263
 
 
   This somehow didn't get to master - i.e. a fresh master install (as of 
22.07) still has the bad description ?
   @anuragaw @shwstppr can you please check ?
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-documentation] branch volumes-from-snaps-doc-update created (now 37c9f0e)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch volumes-from-snaps-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


  at 37c9f0e  offering required for volumes created from snaps

This branch includes the following new commits:

 new 37c9f0e  offering required for volumes created from snaps

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[cloudstack-documentation] 01/01: offering required for volumes created from snaps

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch volumes-from-snaps-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git

commit 37c9f0e4bb7d015f5a801657cc52c1e0ff271b76
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 17:24:35 2019 +0200

offering required for volumes created from snaps
---
 source/adminguide/storage.rst | 12 +++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/source/adminguide/storage.rst b/source/adminguide/storage.rst
index 44f5fcb..29153c1 100644
--- a/source/adminguide/storage.rst
+++ b/source/adminguide/storage.rst
@@ -230,6 +230,15 @@ are not allocated on the physical storage device until you 
attach the
 volume. This optimization allows the CloudStack to provision the volume
 nearest to the guest that will use it when the first attachment is made.
 
+When creating a new volume from an existing ROOT volume snapshot,
+it is required to explicitely define a Disk offering (UI will offer only Disk
+offerings whose disk size is equal or bigger than the size of the snapshot).
+
+|volume-from-snap.PNG|
+
+When creating a new volume from an existing DATA volume snapshot, Disk offering
+associated with the snapshots (inherited from the original volume), is assigned
+to the new volume.
 
 Using Local Storage for Data Volumes
 
@@ -657,7 +666,6 @@ to a VM.
 A completed snapshot is copied from primary storage to secondary
 storage, where it is stored until deleted or purged by newer snapshot.
 
-
 How to Snapshot a Volume
 
 
@@ -802,3 +810,5 @@ snapshot data.
:alt: Detach Disk Button.
 .. |Migrateinstance.png| image:: /_static/images/migrate-instance.png
:alt: button to migrate a volume.
+.. |volume-from-snap.PNG| image:: /_static/images/volume-from-snap.PNG
+   :alt: Offering is needed when creating a volume from the ROOT volume 
snapshot.   



[cloudstack-documentation] branch volumes-from-snaps-doc-update updated: image added

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch volumes-from-snaps-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/volumes-from-snaps-doc-update 
by this push:
 new 39a107e  image added
39a107e is described below

commit 39a10791607631ac707fcdd31e65d9e5d838
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 17:25:16 2019 +0200

image added
---
 source/_static/images/volume-from-snap.PNG | Bin 0 -> 8404 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/source/_static/images/volume-from-snap.PNG 
b/source/_static/images/volume-from-snap.PNG
new file mode 100644
index 000..5eb9a5e
Binary files /dev/null and b/source/_static/images/volume-from-snap.PNG differ



[GitHub] [cloudstack] blueorangutan commented on issue #3517: 4.13.0.0/master: health checks smoketests

2019-07-25 Thread GitBox
blueorangutan commented on issue #3517: 4.13.0.0/master: health checks 
smoketests
URL: https://github.com/apache/cloudstack/pull/3517#issuecomment-515090655
 
 
   Trillian test result (tid-217)
   Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 38147 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3517-t217-vmware-65u2.zip
   Intermittent failure detected: /marvin/tests/smoke/test_network.py
   Intermittent failure detected: /marvin/tests/smoke/test_routers.py
   Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
   Intermittent failure detected: /marvin/tests/smoke/test_usage.py
   Smoke tests completed. 73 look OK, 4 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_reboot_router | `Error` | 351.35 | test_network.py
   test_08_start_router | `Error` | 73.28 | test_routers.py
   test_09_reboot_router | `Error` | 1.14 | test_routers.py
   test_02_list_snapshots_with_removed_data_store | `Error` | 1.22 | 
test_snapshots.py
   test_01_volume_usage | `Error` | 8.49 | test_usage.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-documentation] branch volumes-from-snaps-doc-update updated: spell check

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch volumes-from-snaps-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/volumes-from-snaps-doc-update 
by this push:
 new 4c5fdd6  spell check
4c5fdd6 is described below

commit 4c5fdd6af1b7f7063bcd5ee82308b8c2668b27b8
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 17:26:40 2019 +0200

spell check
---
 source/adminguide/storage.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source/adminguide/storage.rst b/source/adminguide/storage.rst
index 29153c1..6f04066 100644
--- a/source/adminguide/storage.rst
+++ b/source/adminguide/storage.rst
@@ -237,7 +237,7 @@ offerings whose disk size is equal or bigger than the size 
of the snapshot).
 |volume-from-snap.PNG|
 
 When creating a new volume from an existing DATA volume snapshot, Disk offering
-associated with the snapshots (inherited from the original volume), is assigned
+associated with the snapshots (inherited from the original volume) is assigned
 to the new volume.
 
 Using Local Storage for Data Volumes



[GitHub] [cloudstack] blueorangutan commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
blueorangutan commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-515102301
 
 
   Trillian test result (tid-223)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 29550 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3152-t223-kvm-centos7.zip
   Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
   Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 77 look OK, 0 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-documentation] branch volumes-metrics-doc-update created (now 753c9b1)

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a change to branch volumes-metrics-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


  at 753c9b1  Description on the volume metrics

This branch includes the following new commits:

 new 753c9b1  Description on the volume metrics

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[cloudstack-documentation] 01/01: Description on the volume metrics

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch volumes-metrics-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git

commit 753c9b159967c4abb2fba57846d662368ca2e4e9
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 18:02:51 2019 +0200

Description on the volume metrics
---
 source/adminguide/storage.rst | 31 +++
 1 file changed, 31 insertions(+)

diff --git a/source/adminguide/storage.rst b/source/adminguide/storage.rst
index 44f5fcb..63b2ae7 100644
--- a/source/adminguide/storage.rst
+++ b/source/adminguide/storage.rst
@@ -789,6 +789,35 @@ snapshot data.
been taken and stored in OVA format will continue to exist in that 
format, and will continue to work as expected.
 
+Volume Metrics
+~~
+
+Volume statistics are collected on a regular interval (defined by global
+setting volume.stats.interval with a default of 600 seconds). 
+This feature is currently only available for VMware and KVM. 
+Volume stats include include bytes/s and IO/s statistics as shown in the
+API output bellow.
+
+.. code:: bash
+
+   [root@mgmt]# cmk list volumesmetrics id=272c3d8b-ef2c-499e-abfb-736b54d3d6b1
+   {
+   "count": 1,
+   "volume": [
+  {
+ ...
+ "diskiopstotal": 30245,
+ "diskioread": 22443,
+ "diskiowrite": 7802,
+ "diskkbsread": 343124,
+ "diskkbswrite": 217619,
+ ...
+Bytes read/write statistics are not exposed via UI, as shown in the image 
below.
+
+|volume-metrics.png|
+
+These statistics are obtained from the hypervisor directly and they represent
+"current" bytes/s and IO/s values at the time of collection.
 
 .. |AttachDiskButton.png| image:: /_static/images/attach-disk-icon.png
:alt: Attach Disk Button.
@@ -802,3 +831,5 @@ snapshot data.
:alt: Detach Disk Button.
 .. |Migrateinstance.png| image:: /_static/images/migrate-instance.png
:alt: button to migrate a volume.
+.. |volume-metrics.png| image:: /_static/images/volume-metrics.png
+   :alt: Volume metrics   



[cloudstack-documentation] branch volumes-metrics-doc-update updated: image added

2019-07-25 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository.

andrijapanic pushed a commit to branch volumes-metrics-doc-update
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/volumes-metrics-doc-update by 
this push:
 new 15dcc82  image added
15dcc82 is described below

commit 15dcc82c744f9c9ad5e1de2ddccfeaf953ff0749
Author: Andrija Panic <45762285+andrijapani...@users.noreply.github.com>
AuthorDate: Thu Jul 25 18:03:55 2019 +0200

image added
---
 source/_static/images/volume-metrics.png | Bin 0 -> 34416 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/source/_static/images/volume-metrics.png 
b/source/_static/images/volume-metrics.png
new file mode 100644
index 000..221171a
Binary files /dev/null and b/source/_static/images/volume-metrics.png differ



[GitHub] [cloudstack] blueorangutan commented on issue #3480: engine, server, services: fix for respecting secondary storage threshold limit

2019-07-25 Thread GitBox
blueorangutan commented on issue #3480: engine, server, services: fix for 
respecting secondary storage threshold limit
URL: https://github.com/apache/cloudstack/pull/3480#issuecomment-515116662
 
 
   Trillian test result (tid-220)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 33134 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3480-t220-kvm-centos7.zip
   Smoke tests completed. 77 look OK, 0 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3480: engine, server, services: fix for respecting secondary storage threshold limit

2019-07-25 Thread GitBox
rhtyd commented on issue #3480: engine, server, services: fix for respecting 
secondary storage threshold limit
URL: https://github.com/apache/cloudstack/pull/3480#issuecomment-515120653
 
 
   @blueorangutan test centos7 vmware-55u3
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3480: engine, server, services: fix for respecting secondary storage threshold limit

2019-07-25 Thread GitBox
blueorangutan commented on issue #3480: engine, server, services: fix for 
respecting secondary storage threshold limit
URL: https://github.com/apache/cloudstack/pull/3480#issuecomment-515120990
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been 
kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3480: engine, server, services: fix for respecting secondary storage threshold limit

2019-07-25 Thread GitBox
blueorangutan commented on issue #3480: engine, server, services: fix for 
respecting secondary storage threshold limit
URL: https://github.com/apache/cloudstack/pull/3480#issuecomment-515146500
 
 
   Trillian test result (tid-219)
   Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 37437 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3480-t219-xenserver-71.zip
   Intermittent failure detected: /marvin/tests/smoke/test_list_ids_parameter.py
   Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
   Smoke tests completed. 75 look OK, 2 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_03_list_snapshots | `Error` | 0.05 | test_list_ids_parameter.py
   test_01_scale_vm | `Failure` | 28.99 | test_scale_vm.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3480: engine, server, services: fix for respecting secondary storage threshold limit

2019-07-25 Thread GitBox
blueorangutan commented on issue #3480: engine, server, services: fix for 
respecting secondary storage threshold limit
URL: https://github.com/apache/cloudstack/pull/3480#issuecomment-515188851
 
 
   Trillian test result (tid-222)
   Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 43815 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3480-t222-vmware-65u2.zip
   Intermittent failure detected: 
/marvin/tests/smoke/test_deploy_vm_root_resize.py
   Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
   Intermittent failure detected: /marvin/tests/smoke/test_usage.py
   Smoke tests completed. 75 look OK, 2 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_02_list_snapshots_with_removed_data_store | `Error` | 1.20 | 
test_snapshots.py
   test_01_volume_usage | `Error` | 10.48 | test_usage.py
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] svenvogel opened a new pull request #3524: [WIP DO NOT MERGE] Fix Virtual Router bootstrapping/connection state in KVM

2019-07-25 Thread GitBox
svenvogel opened a new pull request #3524: [WIP DO NOT MERGE] Fix Virtual 
Router bootstrapping/connection state in KVM
URL: https://github.com/apache/cloudstack/pull/3524
 
 
   ## Description
   
   
   * After PR #3278 there was some changes on the channels. this does not seem 
like the problem. if we look to the VR XML file we see the following. the qemu 
guest agent must be the problem because the state of the VR is "disconnected".
   
   ```xml
  




  
   ```
   after investigation of the bootstrap process i think i found this problem. 
there is no startup from the qemu-guest-agent. we need them in connected state 
for the patch.sh script.
   
   
![image](https://user-images.githubusercontent.com/5433844/61913361-2d563100-af3d-11e9-9908-d4c2181fee66.png)
   
   the problems seems to be the startup of the "qemu-guest-agent". since this 
is not an really systemd service more than an sysvinit. the service will be 
started with "systemctl enable --now qemu-guest agent". this will be redirected 
to systemd-sysv-install. normally "--now" should enable and start the service. 
i think with this redirection to systemd-sysv-install there is no option for 
the startup.
   
   
![image](https://user-images.githubusercontent.com/5433844/61913305-fed85600-af3c-11e9-8d89-cdb570b7a046.png)
   
![image](https://user-images.githubusercontent.com/5433844/61913349-1f081500-af3d-11e9-90b7-6a6d5c719d52.png)
   
   * last one is to capture the error from the module load kvm pci hotplug
   
   
   
   Use actually master, build a SystemVM and try to deploy them
   
   
   
   
   
   ## Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [x] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## Screenshots (if appropriate):
   
   ## How Has This Been Tested?
   
   
   
   
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3480: engine, server, services: fix for respecting secondary storage threshold limit

2019-07-25 Thread GitBox
blueorangutan commented on issue #3480: engine, server, services: fix for 
respecting secondary storage threshold limit
URL: https://github.com/apache/cloudstack/pull/3480#issuecomment-515256242
 
 
   Trillian test result (tid-225)
   Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 23697 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3480-t225-vmware-55u3.zip
   Intermittent failure detected: /marvin/tests/smoke/test_network.py
   Intermittent failure detected: /marvin/tests/smoke/test_public_ip_range.py
   Intermittent failure detected: /marvin/tests/smoke/test_reset_vm_on_reboot.py
   Intermittent failure detected: 
/marvin/tests/smoke/test_resource_accounting.py
   Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
   Intermittent failure detected: /marvin/tests/smoke/test_router_dns.py
   Intermittent failure detected: /marvin/tests/smoke/test_router_dnsservice.py
   Intermittent failure detected: 
/marvin/tests/smoke/test_routers_iptables_default_policy.py
   Intermittent failure detected: 
/marvin/tests/smoke/test_routers_network_ops.py
   Intermittent failure detected: /marvin/tests/smoke/test_routers.py
   Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
   Intermittent failure detected: /marvin/tests/smoke/test_secondary_storage.py
   Intermittent failure detected: /marvin/tests/smoke/test_service_offerings.py
   Intermittent failure detected: /marvin/tests/smoke/test_snapshots.py
   Intermittent failure detected: /marvin/tests/smoke/test_ssvm.py
   Intermittent failure detected: /marvin/tests/smoke/test_templates.py
   Intermittent failure detected: /marvin/tests/smoke/test_usage.py
   Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
   Intermittent failure detected: /marvin/tests/smoke/test_vm_snapshots.py
   Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Smoke tests completed. 55 look OK, 22 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_reboot_router | `Error` | 370.84 | test_network.py
   ContextSuite context=TestResetVmOnReboot>:setup | `Error` | 0.00 | 
test_reset_vm_on_reboot.py
   test_01_so_removal_resource_update | `Error` | 1.46 | 
test_resource_accounting.py
   ContextSuite context=TestRouterDHCPHosts>:setup | `Error` | 0.00 | 
test_router_dhcphosts.py
   ContextSuite context=TestRouterDHCPOpts>:setup | `Error` | 0.00 | 
test_router_dhcphosts.py
   test_01_sys_vm_start | `Failure` | 0.12 | test_secondary_storage.py
   ContextSuite context=TestRouterDns>:setup | `Error` | 0.00 | 
test_router_dns.py
   ContextSuite context=TestRouterDnsService>:setup | `Error` | 0.00 | 
test_router_dnsservice.py
   test_02_routervm_iptables_policies | `Error` | 1.34 | 
test_routers_iptables_default_policy.py
   test_01_single_VPC_iptables_policies | `Error` | 4.40 | 
test_routers_iptables_default_policy.py
   test_01_isolate_network_FW_PF_default_routes_egress_true | `Error` | 1.48 | 
test_routers_network_ops.py
   test_02_isolate_network_FW_PF_default_routes_egress_false | `Error` | 1.48 | 
test_routers_network_ops.py
   test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | `Error` | 4.87 | 
test_routers_network_ops.py
   test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | `Error` | 6.00 | 
test_routers_network_ops.py
   test_03_RVR_Network_check_router_state | `Error` | 5.99 | 
test_routers_network_ops.py
   ContextSuite context=TestSnapshotRootDisk>:setup | `Error` | 0.00 | 
test_snapshots.py
   ContextSuite context=TestRouterServices>:setup | `Error` | 0.00 | 
test_routers.py
   ContextSuite context=TestScaleVm>:setup | `Error` | 0.00 | test_scale_vm.py
   ContextSuite context=TestServiceOfferings>:setup | `Error` | 2.33 | 
test_service_offerings.py
   test_01_list_sec_storage_vm | `Failure` | 0.04 | test_ssvm.py
   test_02_list_cpvm_vm | `Failure` | 0.04 | test_ssvm.py
   test_03_ssvm_internals | `Failure` | 0.04 | test_ssvm.py
   test_04_cpvm_internals | `Failure` | 0.04 | test_ssvm.py
   test_05_stop_ssvm | `Failure` | 0.04 | test_ssvm.py
   test_06_stop_cpvm | `Failure` | 0.04 | test_ssvm.py
   test_07_reboot_ssvm | `Failure` | 0.04 | test_ssvm.py
   test_08_reboot_cpvm | `Failure` | 0.04 | test_ssvm.py
   test_09_destroy_ssvm | `Failure` | 0.04 | test_ssvm.py
   test_10_destroy_cpvm | `Failure` | 0.04 | test_ssvm.py
   test_02_create_template_with_checksum_sha1 | `Error` | 65.59 | 
test_templates.py
   test_03_create_template_with_checksum_sha256 | `Error` | 65.52 | 
test_templates.py
   test_04_create_template_with_checksum_md5 | `Error` | 65.52 | 
test_templates.py
   test_05_create_template_

[GitHub] [cloudstack] digitalgust commented on issue #3523: 4.11.3.0 create ceph primary storage error

2019-07-25 Thread GitBox
digitalgust commented on issue #3523: 4.11.3.0 create ceph primary storage error
URL: https://github.com/apache/cloudstack/issues/3523#issuecomment-515283164
 
 
   since 4.9.2 cloudstack 
   add primary storage error:
   
   ```
   2019-07-26 10:16:49,763 DEBUG [c.c.a.ApiServlet] 
(catalina-exec-2:ctx-8368781e) (logid:151a79c4) ===START===  10.0.1.11 -- GET  
command=createStoragePool&zoneid=e744901f-745a-4bb0-b8e3-4b1cf4ca9b60&podId=964682b7-367b-4d1a-8a48-c55b4e986fe7&clusterid=9dc1e904-57d7-4678-b4e8-cb2380abc93e&name=Z1_PRI1_CEPH&scope=zone&hypervisor=KVM&url=rbd%3A%2F%2Fcloudstack%3AAQATDjld4LBvBxAABbwFU5C_m%2FaTVkI95xrSzw%3D%3D%40192.168.100.10%2Fcloudstack&response=json&_=1564107409711
   2019-07-26 10:16:49,776 DEBUG 
[o.a.c.s.d.l.CloudStackPrimaryDataStoreLifeCycleImpl] 
(catalina-exec-2:ctx-8368781e ctx-616f5ed3) (logid:151a79c4) createPool Params 
@ scheme - rbd storageHost - null hostPath - 
/aTVkI95xrSzw==@192.168.100.10/cloudstack port - -1
   2019-07-26 10:16:49,793 DEBUG [c.c.u.d.T.Transaction] 
(catalina-exec-2:ctx-8368781e ctx-616f5ed3) (logid:151a79c4) Rolling back the 
transaction: Time = 1 Name =  catalina-exec-2; called by 
-TransactionLegacy.rollback:889-TransactionLegacy.removeUpTo:832-TransactionLegacy.close:656-TransactionContextInterceptor.invoke:36-ReflectiveMethodInvocation.proceed:161-ExposeInvocationInterceptor.invoke:91-ReflectiveMethodInvocation.proceed:172-JdkDynamicAopProxy.invoke:204-$Proxy67.persist:-1-PrimaryDataStoreHelper.createPrimaryDataStore:132-CloudStackPrimaryDataStoreLifeCycleImpl.initialize:353-StorageManagerImpl.createPool:706
   2019-07-26 10:16:49,794 DEBUG [c.c.s.StorageManagerImpl] 
(catalina-exec-2:ctx-8368781e ctx-616f5ed3) (logid:151a79c4) Failed to add data 
store: DB Exception on: com.mysql.jdbc.JDBC4PreparedStatement@38670f4f: INSERT 
INTO storage_pool (storage_pool.id, storage_pool.name, storage_pool.uuid, 
storage_pool.pool_type, storage_pool.created, storage_pool.update_time, 
storage_pool.data_center_id, storage_pool.pod_id, storage_pool.used_bytes, 
storage_pool.capacity_bytes, storage_pool.status, 
storage_pool.storage_provider_name, storage_pool.host_address, 
storage_pool.path, storage_pool.port, storage_pool.user_info, 
storage_pool.cluster_id, storage_pool.scope, storage_pool.managed, 
storage_pool.capacity_iops, storage_pool.hypervisor) VALUES (0, 
_binary'Z1_PRI1_CEPH', _binary'ca5cbccb-fb46-3fdd-881a-aa857dcdac00', 'RBD', 
'2019-07-26 02:16:49', null, 1, null, 0, 0, 'Initialized', 
_binary'DefaultPrimary', null, 
_binary'aTVkI95xrSzw==@192.168.100.10/cloudstack', 6789, null, null, null, 0, 
null, null)
   com.cloud.utils.exception.CloudRuntimeException: DB Exception on: 
com.mysql.jdbc.JDBC4PreparedStatement@38670f4f: INSERT INTO storage_pool 
(storage_pool.id, storage_pool.name, storage_pool.uuid, storage_pool.pool_type, 
storage_pool.created, storage_pool.update_time, storage_pool.data_center_id, 
storage_pool.pod_id, storage_pool.used_bytes, storage_pool.capacity_bytes, 
storage_pool.status, storage_pool.storage_provider_name, 
storage_pool.host_address, storage_pool.path, storage_pool.port, 
storage_pool.user_info, storage_pool.cluster_id, storage_pool.scope, 
storage_pool.managed, storage_pool.capacity_iops, storage_pool.hypervisor) 
VALUES (0, _binary'Z1_PRI1_CEPH', 
_binary'ca5cbccb-fb46-3fdd-881a-aa857dcdac00', 'RBD', '2019-07-26 02:16:49', 
null, 1, null, 0, 0, 'Initialized', _binary'DefaultPrimary', null, 
_binary'aTVkI95xrSzw==@192.168.100.10/cloudstack', 6789, null, null, null, 0, 
null, null)
at com.cloud.utils.db.GenericDaoBase.persist(GenericDaoBase.java:1427)
at 
org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDaoImpl.persist(PrimaryDataStoreDaoImpl.java:262)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
com.cloud.utils.db.TransactionContextInterceptor.invoke(TransactionContextInterceptor.java:34)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:161)
at 
org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:172)
at 
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at com.sun.proxy.$P

[GitHub] [cloudstack] blueorangutan commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
blueorangutan commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-515330994
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-71) has been 
kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
rhtyd commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-515331061
 
 
   @blueorangutan test centos7 vmware-65u2
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
rhtyd commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-515330883
 
 
   @blueorangutan test centos7 xenserver-71


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3152: Refactoring to remove duplicate code.

2019-07-25 Thread GitBox
blueorangutan commented on issue #3152: Refactoring to remove duplicate code.
URL: https://github.com/apache/cloudstack/pull/3152#issuecomment-515331257
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-65u2) has been 
kicked to run smoke tests


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on a change in pull request #3524: [WIP DO NOT MERGE] Fix VR bootstrapping/connection state in KVM

2019-07-25 Thread GitBox
rhtyd commented on a change in pull request #3524: [WIP DO NOT MERGE] Fix VR 
bootstrapping/connection state in KVM
URL: https://github.com/apache/cloudstack/pull/3524#discussion_r307608158
 
 

 ##
 File path: systemvm/debian/opt/cloud/bin/setup/bootstrap.sh
 ##
 @@ -85,12 +85,18 @@ config_guest() {
   sed -i "s/%/ /g" $CMDLINE
   ;;
  kvm)
-  # Configure hot-plug
-  modprobe acpiphp || true
-  modprobe pci_hotplug || true
+  # Configure kvm hotplug support
+  if grep -E 'CONFIG_HOTPLUG_PCI=y|CONFIG_HOTPLUG_PCI_ACPI=y' 
/boot/config-`uname -r`; then
+log_it "acpiphp and pci_hotplug module already compiled in"
+  else
+modprobe acpiphp 2> /dev/null && log_it "acpiphp module loaded" || 
true
+modprobe pci_hotplug 2> /dev/null && log_it "pci_hotplug module 
loaded" || true
+  fi
+
   sed -i -e "/^s0:2345:respawn.*/d" /etc/inittab
   sed -i -e "/6:23:respawn/a\s0:2345:respawn:/sbin/getty -L 115200 
ttyS0 vt102" /etc/inittab
-  systemctl enable --now qemu-guest-agent
 
 Review comment:
   Minor nit: The `--now` does the same, enables and starts it.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3524: [WIP DO NOT MERGE] Fix VR bootstrapping/connection state in KVM

2019-07-25 Thread GitBox
rhtyd commented on issue #3524: [WIP DO NOT MERGE] Fix VR 
bootstrapping/connection state in KVM
URL: https://github.com/apache/cloudstack/pull/3524#issuecomment-515332059
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3524: [WIP DO NOT MERGE] Fix VR bootstrapping/connection state in KVM

2019-07-25 Thread GitBox
blueorangutan commented on issue #3524: [WIP DO NOT MERGE] Fix VR 
bootstrapping/connection state in KVM
URL: https://github.com/apache/cloudstack/pull/3524#issuecomment-515332317
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3524: [WIP DO NOT MERGE] Fix VR bootstrapping/connection state in KVM

2019-07-25 Thread GitBox
rhtyd commented on issue #3524: [WIP DO NOT MERGE] Fix VR 
bootstrapping/connection state in KVM
URL: https://github.com/apache/cloudstack/pull/3524#issuecomment-515332682
 
 
   @svenvogel is this work in progress or ready for review?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3519: kvm/cloudstack-guest-tool: Tool to query Qemu Guest Agent

2019-07-25 Thread GitBox
rhtyd commented on issue #3519: kvm/cloudstack-guest-tool: Tool to query Qemu 
Guest Agent
URL: https://github.com/apache/cloudstack/pull/3519#issuecomment-515333036
 
 
   @wido rpm packaging is failing, can you check? The new file needs 
declaration in the spec file.
   ```
   error: Installed (but unpackaged) file(s) found:
  /usr/bin/cloudstack-guest-tool
   
   
   RPM build errors:
   Binaries arch (1) not matching the package arch (2).
   Installed (but unpackaged) file(s) found:
  /usr/bin/cloudstack-guest-tool
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services