[GitHub] [cloudstack] svenvogel commented on issue #3294: Set default service offering over api and ui

2019-05-21 Thread GitBox
svenvogel commented on issue #3294: Set default service offering over api and ui
URL: https://github.com/apache/cloudstack/pull/3294#issuecomment-494569336
 
 
   @PaulAngus thanks for info i will discuss this with @pbankonier and come 
back 


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 commented on issue #3346: Fix template size for managed storage / refactor cloud-install-sys-tmplt and createtmplt.sh

2019-05-21 Thread GitBox
svenvogel commented on issue #3346: Fix template size for managed storage / 
refactor cloud-install-sys-tmplt and createtmplt.sh
URL: https://github.com/apache/cloudstack/pull/3346#issuecomment-494568021
 
 
   @syed can you check this?


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 #3346: Fix template size for managed storage / refactor cloud-install-sys-tmplt and createtmplt.sh

2019-05-21 Thread GitBox
svenvogel opened a new pull request #3346: Fix template size for managed 
storage / refactor cloud-install-sys-tmplt and createtmplt.sh
URL: https://github.com/apache/cloudstack/pull/3346
 
 
   ## Description
   
   * remove unused old vars and functions e.g. today, chksum...
   * rename createtmplt.sh to createtemplt to unify the files in overall context
   * add function to calculate the virtual size of template for managed storage 
(netapp solidfire) so that the copy process is working from NFS to iSCSI 
correctly
   * refactor cloud-install-sys-tmplt creates all settings of 
template.properties
   * refactor createtmplt shall create only the templates e.g. uncompress and 
install ...
   
   
   
   
   
   
   
   
   ## 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)
   - [X] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## Screenshots (if appropriate):
   
   ## How Has This Been Tested?
   
   tested manually in our production environment with different files like vhd, 
qcow2 and ova
   
   
   
   
   
   


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] nathanejohnson commented on issue #3345: Fix iops values when creating a compute offering

2019-05-21 Thread GitBox
nathanejohnson commented on issue #3345: Fix iops values when creating a 
compute offering
URL: https://github.com/apache/cloudstack/pull/3345#issuecomment-494557717
 
 
   @svenvogel I will test this locally.  thanks for the 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] ustcweizhou commented on issue #3331: api/server: Add option 'details' to listProjects and listAccounts

2019-05-21 Thread GitBox
ustcweizhou commented on issue #3331: api/server: Add option 'details' to 
listProjects and listAccounts
URL: https://github.com/apache/cloudstack/pull/3331#issuecomment-494540543
 
 
   @anuragaw added integration test as you suggested.
   


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] ustcweizhou commented on issue #3343: listVirtualMachines does not show instances belonging to a project

2019-05-21 Thread GitBox
ustcweizhou commented on issue #3343: listVirtualMachines does not show 
instances belonging to a project
URL: https://github.com/apache/cloudstack/issues/3343#issuecomment-494504125
 
 
   @PaulAngus We have some changes in 4.7.1 to list all project resources in 
default view. That's the default behavior in our fork. As you can see, it 
breaks the backwards compatibility.
   We need to make more changes before creating a pull request to mainstream. I 
will look into 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] blueorangutan commented on issue #3333: server: ssh-keygen in PEM format and reduce main systemvm patching script

2019-05-21 Thread GitBox
blueorangutan commented on issue #: server: ssh-keygen in PEM format and 
reduce main systemvm patching script
URL: https://github.com/apache/cloudstack/pull/#issuecomment-494503124
 
 
   Trillian test result (tid-3565)
   Environment: kvm-centos6 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 33066 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr-t3565-kvm-centos6.zip
   Intermittent failure detected: /marvin/tests/smoke/test_service_offerings.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Smoke tests completed. 67 look OK, 1 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_service_offering_cpu_limit_use | `Error` | 1.35 | 
test_service_offerings.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] PaulAngus commented on issue #3294: Set default service offering over api and ui

2019-05-21 Thread GitBox
PaulAngus commented on issue #3294: Set default service offering over api and ui
URL: https://github.com/apache/cloudstack/pull/3294#issuecomment-494469362
 
 
   @svenvogel @pbankonier - I can see that you've put a lot of work into this, 
but unless I've missed something, couldn't this be done a bit simpler by 
populating the service_offering_id for all of the default network offerings as: 
9 or 10 (shared/local storage offerings for domainrouter).
   
   Then the network offering would be linked to an actual service offering 
rather than the weird global settings business.
   
   Migrating from the global settings if people have updated them wouldn't be 
difficult, the service offerings 9 and 10 would just have to be updated with 
the values in the global settings.
   
   Editing the material properties of a offering has always been a no-no (with 
very good reasons) but maybe we can make an exception for the system vm 
offerings to allow change CPU and RAM values.
   // @ustcweizhou 
   
   
 


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] PaulAngus commented on issue #3343: listVirtualMachines does not show instances belonging to a project

2019-05-21 Thread GitBox
PaulAngus commented on issue #3343: listVirtualMachines does not show instances 
belonging to a project
URL: https://github.com/apache/cloudstack/issues/3343#issuecomment-494456541
 
 
   @ustcweizhou did you say that you guys are planning on changing this 
behaviour or just that you would also like to see it changed?


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] pbankonier opened a new pull request #3345: Fix iops values when creating a compute offering

2019-05-21 Thread GitBox
pbankonier opened a new pull request #3345: Fix iops values when creating a 
compute offering
URL: https://github.com/apache/cloudstack/pull/3345
 
 
   ## Description
   When creating a new compute offering over the ui, the isCustomIops parameter 
is null.
   In this case the minIops and maxIops values of the offering gets set to 0 
instead of null and when trying to change an instance to this offering an error 
occurs.
   This was introduced in this PR 
https://github.com/apache/cloudstack/pull/3133/files.
   I already opened an issue regarding this. 
(https://github.com/apache/cloudstack/issues/3332)
   
   ## 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:
   ![Bildschirmfoto 2019-05-21 um 13 41 
50](https://user-images.githubusercontent.com/6997263/58094720-acdc2280-7bd1-11e9-8bf8-e62f4ff5c33d.png)
   
   ![Bildschirmfoto 2019-05-21 um 13 42 
18](https://user-images.githubusercontent.com/6997263/58094731-af3e7c80-7bd1-11e9-9e32-0b83ccbe12a2.png)
   
   ![Bildschirmfoto 2019-05-21 um 14 06 
02](https://user-images.githubusercontent.com/6997263/58094818-d006d200-7bd1-11e9-860e-78a212f26482.png)
   
   https://user-images.githubusercontent.com/6997263/58095320-0b55d080-7bd3-11e9-8be1-f110b00222d1.png";>
   
   https://user-images.githubusercontent.com/6997263/58095301-042ec280-7bd3-11e9-8033-7696a77a664a.png";>
   
   
   ## How Has This Been Tested?
   Deployed to local environment and tested functionality.
   


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 #3344: [WIP] server: return usage description with resource names and UUIDs

2019-05-21 Thread GitBox
blueorangutan commented on issue #3344: [WIP] server: return usage description 
with resource names and UUIDs
URL: https://github.com/apache/cloudstack/pull/3344#issuecomment-494362674
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2745


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 #3333: server: ssh-keygen in PEM format and reduce main systemvm patching script

2019-05-21 Thread GitBox
blueorangutan commented on issue #: server: ssh-keygen in PEM format and 
reduce main systemvm patching script
URL: https://github.com/apache/cloudstack/pull/#issuecomment-494358412
 
 
   Trillian test result (tid-3564)
   Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 6
   Total time taken: 31352 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr-t3564-xenserver-71.zip
   Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
   Intermittent failure detected: /marvin/tests/smoke/test_templates.py
   Intermittent failure detected: /marvin/tests/smoke/test_usage.py
   Smoke tests completed. 56 look OK, 2 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_scale_vm | `Error` | 16.37 | test_scale_vm.py
   test_01_ISO_usage | `Error` | 1.06 | test_usage.py
   test_01_lb_usage | `Error` | 8.28 | test_usage.py
   test_01_nat_usage | `Error` | 15.43 | test_usage.py
   test_01_public_ip_usage | `Error` | 1.08 | test_usage.py
   test_01_snapshot_usage | `Error` | 57.15 | test_usage.py
   test_01_template_usage | `Error` | 50.26 | test_usage.py
   test_01_vm_usage | `Error` | 27.69 | test_usage.py
   test_01_volume_usage | `Error` | 28.00 | test_usage.py
   test_01_vpn_usage | `Error` | 19.65 | 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] blueorangutan commented on issue #3344: [WIP] server: return usage description with resource names and UUIDs

2019-05-21 Thread GitBox
blueorangutan commented on issue #3344: [WIP] server: return usage description 
with resource names and UUIDs
URL: https://github.com/apache/cloudstack/pull/3344#issuecomment-494356047
 
 
   @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 #3344: [WIP] server: return usage description with resource names and UUIDs

2019-05-21 Thread GitBox
rhtyd commented on issue #3344: [WIP] server: return usage description with 
resource names and UUIDs
URL: https://github.com/apache/cloudstack/pull/3344#issuecomment-494355713
 
 
   @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] rhtyd opened a new pull request #3344: [WIP] server: return usage description with resource names and UUIDs

2019-05-21 Thread GitBox
rhtyd opened a new pull request #3344: [WIP] server: return usage description 
with resource names and UUIDs
URL: https://github.com/apache/cloudstack/pull/3344
 
 
   The usage record descriptions have CloudStack's internal integer IDs
   which makes it difficult for users to read their usages. This PRs
   introduces a new API boolean flag `oldformat` which when set to true
   would return the older description format, otherwise by default
   listUsageRecords will process and return description with names and
   UUIDs of resources.
   
   ## Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [x] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)


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 #3333: server: ssh-keygen in PEM format and reduce main systemvm patching script

2019-05-21 Thread GitBox
rhtyd commented on a change in pull request #: server: ssh-keygen in PEM 
format and reduce main systemvm patching script
URL: https://github.com/apache/cloudstack/pull/#discussion_r285968792
 
 

 ##
 File path: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtStartCommandWrapper.java
 ##
 @@ -125,9 +125,14 @@ public Answer execute(final StartCommand command, final 
LibvirtComputingResource
 // try to patch and SSH into the systemvm for up to 5 minutes
 for (int count = 0; count < 10; count++) {
 // wait and try passCmdLine for 30 seconds at most for 
CLOUDSTACK-2823
-libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs());
-// check router is up?
-final VirtualRoutingResource virtRouterResource = 
libvirtComputingResource.getVirtRouterResource();
+if (libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs())) {
+break;
+}
 
 Review comment:
   @ustcweizhou I can reduce that to 3 maybe.
   I could consistently reproduce the issue with latest Ubuntu 19.10 with 
latest Centos7.6 used as kvm host.


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 #3333: server: ssh-keygen in PEM format and reduce main systemvm patching script

2019-05-21 Thread GitBox
rhtyd commented on a change in pull request #: server: ssh-keygen in PEM 
format and reduce main systemvm patching script
URL: https://github.com/apache/cloudstack/pull/#discussion_r285968792
 
 

 ##
 File path: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtStartCommandWrapper.java
 ##
 @@ -125,9 +125,14 @@ public Answer execute(final StartCommand command, final 
LibvirtComputingResource
 // try to patch and SSH into the systemvm for up to 5 minutes
 for (int count = 0; count < 10; count++) {
 // wait and try passCmdLine for 30 seconds at most for 
CLOUDSTACK-2823
-libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs());
-// check router is up?
-final VirtualRoutingResource virtRouterResource = 
libvirtComputingResource.getVirtRouterResource();
+if (libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs())) {
+break;
+}
 
 Review comment:
   @ustcweizhou I can reduce that to 3 maybe.
   I could consistently reproduce the issue with latest Ubuntu 19.04 with 
latest Centos7.6 used as kvm host.


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] ustcweizhou commented on a change in pull request #3333: server: ssh-keygen in PEM format and reduce main systemvm patching script

2019-05-21 Thread GitBox
ustcweizhou commented on a change in pull request #: server: ssh-keygen in 
PEM format and reduce main systemvm patching script
URL: https://github.com/apache/cloudstack/pull/#discussion_r285903428
 
 

 ##
 File path: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtStartCommandWrapper.java
 ##
 @@ -125,9 +125,14 @@ public Answer execute(final StartCommand command, final 
LibvirtComputingResource
 // try to patch and SSH into the systemvm for up to 5 minutes
 for (int count = 0; count < 10; count++) {
 // wait and try passCmdLine for 30 seconds at most for 
CLOUDSTACK-2823
-libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs());
-// check router is up?
-final VirtualRoutingResource virtRouterResource = 
libvirtComputingResource.getVirtRouterResource();
+if (libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs())) {
+break;
+}
 
 Review comment:
   @rhtyd ok, sorry I did not notice this change.
   in this case, do we still need to pass cmd line for max 10 times ?
   
   which ssh-keygen version is impacted ? I would like to test 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] NuxRo commented on issue #3343: listVirtualMachines does not show instances belonging to a project

2019-05-21 Thread GitBox
NuxRo commented on issue #3343: listVirtualMachines does not show instances 
belonging to a project
URL: https://github.com/apache/cloudstack/issues/3343#issuecomment-494316301
 
 
   ping @ustcweizhou @PaulAngus 


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] NuxRo opened a new issue #3343: listVirtualMachines does not show instances belonging to a project

2019-05-21 Thread GitBox
NuxRo opened a new issue #3343: listVirtualMachines does not show instances 
belonging to a project
URL: https://github.com/apache/cloudstack/issues/3343
 
 
   Relevant email thread 
https://lists.apache.org/thread.html/6aca3e7b99816eef127ce516e5ef93bddb7a60a53e72762d5af3c050@%3Cdev.cloudstack.apache.org%3E
   
   As a user I have created a project and in it I created a few VMs. These VMs 
will not show up in "listVirtualMachines" unless the project is specified.
   
   e.g.
   
   (localcloud) 🐱 > list virtualmachines
   (localcloud) 🐱 > 
   (localcloud) 🐱 > list virtualmachines 
projectid=b9a75ec4-82fc-4e88-8733-022df7956143 filter=id,
   count = 2
   virtualmachine:
   id = 7b7259ac-ac89-43e7-9772-5f7866762be3
   

   id = 0a1bb820-2ad6-4a71-bda0-29bb0ead6781
   
   


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] ustcweizhou opened a new pull request #3342: VPC: Fail to restart VPC with cleanup if there are multiple public IPs in different subnets

2019-05-21 Thread GitBox
ustcweizhou opened a new pull request #3342: VPC: Fail to restart VPC with 
cleanup if there are multiple public IPs in different subnets
URL: https://github.com/apache/cloudstack/pull/3342
 
 
   ## Description
   
   If there are multiple IPs in different subnet assigned to a VPC, after 
restarting VPC with cleanup, the VRs will be FAULT state.
   
   Step to reproduce:
   (1) create vpc, source nat IP is 10.11.118.X
   (2) assign two public IPs in other subnet to this VPC. 10.11.119.X and 
10.11.119.Y
   (3) deploy two vms in the vpc, and enable static nat 10.11.119.X and 
10.11.119.Y to these two vms
   (4) restart vpc with cleanup. There are more than 1 nic allocated for 
10.11.119 to new VRs
   
   Logs as below:
   2019-05-10 14:12:24,652 DEBUG [o.a.c.e.o.NetworkOrchestrator] 
(API-Job-Executor-36:ctx-839f6522 job-652 ctx-35fb4667) (logid:1ab7aa37) 
Allocating nic for vm VM[DomainRouter|r-85-VM] in network Ntwk[200|Public|1] 
with requested profile NicProfile[0-0-null-10.11.118.157-vlan://untagged
   2019-05-10 14:12:24,676 DEBUG [o.a.c.e.o.NetworkOrchestrator] 
(API-Job-Executor-36:ctx-839f6522 job-652 ctx-35fb4667) (logid:1ab7aa37) 
Allocating nic for vm VM[DomainRouter|r-85-VM] in network Ntwk[200|Public|1] 
with requested profile NicProfile[0-0-null-10.11.119.110-vlan://119
   2019-05-10 14:12:24,699 DEBUG [o.a.c.e.o.NetworkOrchestrator] 
(API-Job-Executor-36:ctx-839f6522 job-652 ctx-35fb4667) (logid:1ab7aa37) 
Allocating nic for vm VM[DomainRouter|r-85-VM] in network Ntwk[200|Public|1] 
with requested profile NicProfile[0-0-null-10.11.119.110-vlan://119
   2019-05-10 14:12:24,723 DEBUG [o.a.c.e.o.NetworkOrchestrator] 
(API-Job-Executor-36:ctx-839f6522 job-652 ctx-35fb4667) (logid:1ab7aa37) 
Allocating nic for vm VM[DomainRouter|r-85-VM] in network Ntwk[200|Public|1] 
with requested profile NicProfile[0-0-null-10.11.119.110-vlan://119
   
   This is a regression issue caused by commit 
1d382e0cb48fc428bbe753dcfe4116bc71231206
   
   
   
   
   
   
   
   
   
   
   ## 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)
   
   ## How Has This Been Tested?
   
   
   
   
   see Description
   Everything works fine with this patch.
   
   
   


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 #3333: server: ssh-keygen in PEM format and reduce main systemvm patching script

2019-05-21 Thread GitBox
rhtyd commented on a change in pull request #: server: ssh-keygen in PEM 
format and reduce main systemvm patching script
URL: https://github.com/apache/cloudstack/pull/#discussion_r285889257
 
 

 ##
 File path: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtStartCommandWrapper.java
 ##
 @@ -125,9 +125,14 @@ public Answer execute(final StartCommand command, final 
LibvirtComputingResource
 // try to patch and SSH into the systemvm for up to 5 minutes
 for (int count = 0; count < 10; count++) {
 // wait and try passCmdLine for 30 seconds at most for 
CLOUDSTACK-2823
-libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs());
-// check router is up?
-final VirtualRoutingResource virtRouterResource = 
libvirtComputingResource.getVirtRouterResource();
+if (libvirtComputingResource.passCmdLine(vmName, 
vmSpec.getBootArgs())) {
+break;
+}
 
 Review comment:
   @ustcweizhou the blind waiting and passing cmdline is not necessary now, in 
the new patch.sh script it waits until it's able to communicate with the 
qemu-guest-agent running inside the systemvm appliance. See: 
https://github.com/apache/cloudstack/blob/master/scripts/vm/hypervisor/kvm/patch.sh#L61


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] ustcweizhou commented on a change in pull request #3338: ui: fix enable static nat only towards first nic and not on any other interface

2019-05-21 Thread GitBox
ustcweizhou commented on a change in pull request #3338: ui: fix enable static 
nat only towards first nic and not on any other interface
URL: https://github.com/apache/cloudstack/pull/3338#discussion_r285883007
 
 

 ##
 File path: ui/scripts/network.js
 ##
 @@ -87,12 +87,21 @@
 
 } else { //non-portable IP which has only one NIC
 // Get NIC IPs
+var data = {
+virtualmachineid: instance.id,
+};
+if (args.context.networkid != undefined) {
+$.extend(data, {
+networkid: args.context.networkid
+});
+} else {
+$.extend(data, {
+networkid: network.id
+});
+}
 
 Review comment:
   @anuragaw 
   Changed the code as you suggested, tested ok.
   thanks for good suggestion.
   


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] DennisKonrad commented on issue #3296: [4.13] noVNC console

2019-05-21 Thread GitBox
DennisKonrad commented on issue #3296: [4.13] noVNC console
URL: https://github.com/apache/cloudstack/pull/3296#issuecomment-494265390
 
 
   @ustcweizhou No error, just not loading anything. I will try this again 
tomorror and report back.
   
   I'm not sure anymore if I correctly set the global variable. But the 
variable is present. If you have any questions I should check on please feel 
free to ask.


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