[GitHub] cloudmonger commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring

[git]

cloudmonger commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring
URL: https://github.com/apache/cloudstack/pull/1593#issuecomment-283630504
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 412
Hypervisor xenserver
NetworkType Advanced
Passed=104
Failed=1
Skipped=7
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
   
   
   **Failed tests:**
   * test_non_contigiousvlan.py
   
* test_extendPhysicalNetworkVlan Failed
   
   
   **Skipped tests:**
   test_01_test_vm_volume_snapshot
   test_vm_nic_adapter_vmxnet3
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_routers_network_ops.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] cloudmonger commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring

[git]

cloudmonger commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring
URL: https://github.com/apache/cloudstack/pull/1593#issuecomment-318804911
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1023
Hypervisor xenserver
NetworkType Advanced
Passed=99
Failed=15
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_routers.py
   
* test_03_restart_network_cleanup Failing since 3 runs
   
* test_04_restart_network_wo_cleanup Failing since 3 runs
   
* test_05_router_basic Failing since 3 runs
   
* test_06_router_advanced Failing since 3 runs
   
* test_08_start_router Failing since 3 runs
   
* test_09_reboot_router Failing since 3 runs
   
   * test_network.py
   
* test_reboot_router Failing since 3 runs
   
   * test_router_dns.py
   
* test_router_dns_guestipquery Failed
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 78 runs
   
   * test_nic.py
   
* test_01_nic Failing since 3 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 78 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
111 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
111 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
109 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
109 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_non_contigiousvlan.py
   test_login.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318796032
 
 
   Trillian test result (tid-1272)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 38995 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2208-t1272-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_internal_lb.py
   Intermitten failure detected: /marvin/tests/smoke/test_network.py
   Intermitten failure detected: /marvin/tests/smoke/test_nic.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
   Test completed. 47 look ok, 5 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_VPC_nics_after_destroy | `Failure` | 302.93 | test_vpc_router_nics.py
   test_04_rvpc_privategw_static_routes | `Failure` | 396.90 | 
test_privategw_acl.py
   test_03_vpc_privategw_restart_vpc_cleanup | `Error` | 295.44 | 
test_privategw_acl.py
   test_01_nic | `Error` | 192.89 | test_nic.py
   test_reboot_router | `Error` | 272.83 | test_network.py
   test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Error` | 528.01 | 
test_internal_lb.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.02 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.06 | test_primary_storage.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch debian9-systemvmtemplate updated (fbe9d63 -> 5f3cb74)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard fbe9d63  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new 5f3cb74  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (fbe9d63)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (5f3cb74)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 tools/appliance/definitions/systemvmtemplate/configure_grub.sh  | 6 --
 tools/appliance/definitions/systemvmtemplate/configure_login.sh | 2 --
 2 files changed, 4 insertions(+), 4 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 5f3cb74987b2e30189cb8774025165f50890e1a6
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|  10 +-
 .../debian/config/etc/init.d/cloud-early-config|  23 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |  10 +-
 systemvm/patches/debian/config/etc/init.d/postinit |   6 +-
 systemvm/patches/debian/config/etc/rc.local|   2 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   8 +-
 .../systemvmtemplate/configure_login.sh|  15 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  94 +++-
 .../definitions/systemvmtemplate/definition.rb |  36 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  32 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  69 ++-
 .../definitions/systemvmtemplate/zerodisk.sh   |   9 +-
 20 files changed, 263 insertions(+), 810 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[git]

blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate 
systemvmtemplate to Debian9
URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-318790218
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-884
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch debian9-systemvmtemplate updated (f7d81f0 -> fbe9d63)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard f7d81f0  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new fbe9d63  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (f7d81f0)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (fbe9d63)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 tools/appliance/definitions/systemvmtemplate/configure_login.sh | 1 +
 tools/appliance/definitions/systemvmtemplate/zerodisk.sh| 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit fbe9d6349ae4526317b56500c0aef1eecd8b754f
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|  10 +-
 .../debian/config/etc/init.d/cloud-early-config|  23 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |  10 +-
 systemvm/patches/debian/config/etc/init.d/postinit |   6 +-
 systemvm/patches/debian/config/etc/rc.local|   2 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  15 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  94 +++-
 .../definitions/systemvmtemplate/definition.rb |  36 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  32 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  69 ++-
 .../definitions/systemvmtemplate/zerodisk.sh   |   9 +-
 20 files changed, 261 insertions(+), 808 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *

[cloudstack] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit f7d81f07e8ee902eb7d7a87b0b94c5fca3b51ca5
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|  10 +-
 .../debian/config/etc/init.d/cloud-early-config|  23 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |  10 +-
 systemvm/patches/debian/config/etc/init.d/postinit |   6 +-
 systemvm/patches/debian/config/etc/rc.local|   2 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  14 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  94 +++-
 .../definitions/systemvmtemplate/definition.rb |  36 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  32 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  69 ++-
 .../definitions/systemvmtemplate/zerodisk.sh   |   9 +-
 20 files changed, 260 insertions(+), 808 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *

[cloudstack] branch debian9-systemvmtemplate updated (338f893 -> f7d81f0)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard 338f893  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new f7d81f0  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (338f893)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (f7d81f0)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 systemvm/patches/debian/config/etc/init.d/cloud|  8 +--
 .../debian/config/etc/init.d/cloud-early-config|  4 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |  6 +-
 systemvm/patches/debian/config/etc/init.d/postinit |  6 +-
 .../configure_systemvm_services.sh | 67 +-
 .../definitions/systemvmtemplate/zerodisk.sh   |  7 ++-
 6 files changed, 78 insertions(+), 20 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[GitHub] cloudmonger commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

[git]

cloudmonger commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-297177135
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 583
Hypervisor xenserver
NetworkType Advanced
Passed=108
Failed=11
Skipped=6
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
   
   
   **Failed tests:**
   * test_vm_snapshots.py
   
* test_change_service_offering_for_vm_with_snapshots Failed
   
   * test_network.py
   
* test_delete_account Failed
   
   * test_router_dns.py
   
* test_router_dns_guestipquery Failed
   
   * test_deploy_virtio_scsi_vm.py
   
* test_01_verify_libvirt Failing since 4 runs
   
* test_02_verify_libvirt_after_restart Failing since 4 runs
   
* test_03_verify_libvirt_attach_disk Failing since 4 runs
   
* test_04_verify_guest_lspci Failing since 4 runs
   
* test_05_change_vm_ostype_restart Failing since 4 runs
   
* test_06_verify_guest_lspci_again Failing since 4 runs
   
* ContextSuite context=TestDeployVirtioSCSIVM>:teardown Failing since 4 runs
   
   * test_routers_network_ops.py
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failed
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] cloudmonger commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

[git]

cloudmonger commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-318783726
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1022
Hypervisor xenserver
NetworkType Advanced
Passed=96
Failed=15
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_routers.py
   
* test_03_restart_network_cleanup Failing since 2 runs
   
* test_04_restart_network_wo_cleanup Failing since 2 runs
   
* test_05_router_basic Failing since 2 runs
   
* test_06_router_advanced Failing since 2 runs
   
* test_08_start_router Failing since 2 runs
   
* test_09_reboot_router Failing since 2 runs
   
   * test_network.py
   
* test_reboot_router Failing since 2 runs
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 77 runs
   
   * test_nic.py
   
* test_01_nic Failing since 2 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 77 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
110 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
110 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
108 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
108 runs
   
* test_03_RVR_Network_check_router_state Failed
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_router_dns.py
   test_non_contigiousvlan.py
   test_login.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 
release effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318778913
 
 
   Trillian test result (tid-1267)
   Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6
   Total time taken: 49949 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2052-t1267-xenserver-65sp1.zip
   Intermitten failure detected: /marvin/tests/smoke/test_network.py
   Intermitten failure detected: /marvin/tests/smoke/test_nic.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
   Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermitten failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
   Test completed. 44 look ok, 5 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_VPC_nics_after_destroy | `Failure` | 574.56 | test_vpc_router_nics.py
   test_05_rvpc_multi_tiers | `Failure` | 555.22 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | `Failure` | 1428.25 | 
test_vpc_redundant.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 641.62 | 
test_vpc_redundant.py
   test_04_rvpc_privategw_static_routes | `Failure` | 706.78 | 
test_privategw_acl.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
`Error` | 500.86 | test_vpc_redundant.py
   test_03_vpc_privategw_restart_vpc_cleanup | `Error` | 670.44 | 
test_privategw_acl.py
   test_01_nic | `Error` | 344.94 | test_nic.py
   test_reboot_router | `Error` | 389.69 | test_network.py
   test_09_copy_delete_template | Skipped | 0.03 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.01 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch debian9-systemvmtemplate updated (36e6d58 -> 338f893)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard 36e6d58  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new 338f893  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (36e6d58)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (338f893)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 systemvm/patches/debian/config/etc/rc.local | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 338f8935a30f4e713b523ed95d971d6d269efabd
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|   2 +-
 .../debian/config/etc/init.d/cloud-early-config|  19 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |   8 +-
 systemvm/patches/debian/config/etc/rc.local|   2 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  14 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  29 +-
 .../definitions/systemvmtemplate/definition.rb |  36 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  32 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  69 ++-
 .../definitions/systemvmtemplate/zerodisk.sh   |   2 +-
 19 files changed, 185 insertions(+), 791 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *
-Pin: release o=Debian,a=stable
-Pin-Priority: 900
-EOF
-
-  

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[git]

blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate 
systemvmtemplate to Debian9
URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-318776863
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-883
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 36e6d58f59d953d66e9c98e35bd28deec695ee3a
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|   2 +-
 .../debian/config/etc/init.d/cloud-early-config|  19 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |   8 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  14 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  29 +-
 .../definitions/systemvmtemplate/definition.rb |  36 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  32 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  69 ++-
 .../definitions/systemvmtemplate/zerodisk.sh   |   2 +-
 18 files changed, 184 insertions(+), 790 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *
-Pin: release o=Debian,a=stable
-Pin-Priority: 900
-EOF
-
-  #apt-key exportall | chroot . apt-key add - &&
-  chroot . apt-get 

[cloudstack] branch debian9-systemvmtemplate updated (1be0b0b -> 36e6d58)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard 1be0b0b  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new 36e6d58  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (1be0b0b)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (36e6d58)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 tools/appliance/definitions/systemvmtemplate/definition.rb  |  4 ++--
 .../systemvmtemplate/install_systemvm_packages.sh   |  7 +--
 tools/appliance/definitions/systemvmtemplate/preseed.cfg| 13 -
 3 files changed, 11 insertions(+), 13 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[GitHub] blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 
release effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318766648
 
 
   Trillian test result (tid-1271)
   Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 44502 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2052-t1271-vmware-55u3.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
   Intermitten failure detected: /marvin/tests/smoke/test_templates.py
   Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
   Test completed. 47 look ok, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_rvpc_privategw_static_routes | `Failure` | 846.88 | 
test_privategw_acl.py
   test_CreateTemplateWithDuplicateName | `Error` | 5.10 | test_templates.py
   test_01_create_template | `Error` | 5.10 | test_templates.py
   ContextSuite context=TestTemplates>:setup | `Error` | 288.15 | 
test_templates.py
   test_08_resize_volume | Skipped | 5.09 | test_volumes.py
   test_07_resize_fail | Skipped | 10.19 | test_volumes.py
   test_09_copy_delete_template | Skipped | 0.01 | test_templates.py
   test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
   test_01_scale_vm | Skipped | 66.54 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.03 | test_primary_storage.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.01 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

[git]

rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to 
CIDRs for Admin API calls
URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-318761749
 
 
   I've tried several times now, the environments fails to come up; perhaps 
@borisstoyanov  can takeover testing...
   
   ```
   21:38:41 FAILED - RETRYING: TASK: get wait for state of system VMs to be 
Running (1 retries left).
   21:38:47 fatal: [pr2046-t1277-kvm-centos7-mgmt1]: FAILED! => {"attempts": 
200, "changed": true, "cmd": "cloudmonkey list systemvms | jq '.systemvm[]| 
select(.systemvmtype==\"consoleproxy\")|.state'", "delta": "0:00:00.186658", 
"end": "2017-07-28 20:38:19.617884", "failed": true, "rc": 0, "start": 
"2017-07-28 20:38:19.431226", "stderr": "", "stdout": "", "stdout_lines": [], 
"warnings": []}
   21:38:47 
   21:38:47 cmd: cloudmonkey list systemvms | jq '.systemvm[]| 
select(.systemvmtype=="consoleproxy")|.state'
   21:38:47 
   21:38:47 start: 2017-07-28 20:38:19.431226
   ```
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch debian9-systemvmtemplate updated (069739b -> 1be0b0b)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard 069739b  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new 1be0b0b  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (069739b)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (1be0b0b)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 tools/appliance/definitions/systemvmtemplate/preseed.cfg | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 1be0b0be4329edb1b67aa26daa81792c594c29f8
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|   2 +-
 .../debian/config/etc/init.d/cloud-early-config|  19 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |   8 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  14 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  29 +-
 .../definitions/systemvmtemplate/definition.rb |  34 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  29 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  62 +--
 .../definitions/systemvmtemplate/zerodisk.sh   |   2 +-
 18 files changed, 179 insertions(+), 783 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *
-Pin: release o=Debian,a=stable
-Pin-Priority: 900
-EOF
-
-  #apt-key exportall | chroot . apt-key add - &&
-  chroot . apt-get 

[GitHub] blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for 
Secondary IPv6 Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318745645
 
 
   @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 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] rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-31874
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 069739be38fb24801e64d66c2b2c5fe67d3c9f7a
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|   2 +-
 .../debian/config/etc/init.d/cloud-early-config|  19 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |   8 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  14 +-
 .../systemvmtemplate/configure_networking.sh   |  25 +-
 .../configure_systemvm_services.sh |  29 +-
 .../definitions/systemvmtemplate/definition.rb |  34 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  29 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  62 +--
 .../definitions/systemvmtemplate/zerodisk.sh   |   2 +-
 18 files changed, 179 insertions(+), 783 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *
-Pin: release o=Debian,a=stable
-Pin-Priority: 900
-EOF
-
-  #apt-key exportall | chroot . apt-key add - &&
-  chroot . apt-get 

[cloudstack] branch debian9-systemvmtemplate updated (319c457 -> 069739b)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


 discard 319c457  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9
 new 069739b  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (319c457)
\
 N -- N -- N   refs/heads/debian9-systemvmtemplate (069739b)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

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:
 tools/appliance/definitions/systemvmtemplate/configure_grub.sh| 2 +-
 .../definitions/systemvmtemplate/configure_networking.sh  | 1 +
 tools/appliance/definitions/systemvmtemplate/definition.rb| 2 +-
 tools/appliance/definitions/systemvmtemplate/preseed.cfg  | 8 
 4 files changed, 7 insertions(+), 6 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[GitHub] rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318741770
 
 
   Basic template building works now, closing this in favour of 
https://github.com/apache/cloudstack/pull/2211 which allows for 
commits/collaboration with other committers
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd closed pull request #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd closed pull request #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd opened a new pull request #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[git]

rhtyd opened a new pull request #2211: CLOUDSTACK-10013: Migrate 
systemvmtemplate to Debian9
URL: https://github.com/apache/cloudstack/pull/2211
 
 
 SystemVM changes to work on Debian 9
   - Migrate away from chkconfig to systemctl
   - Remove xenstore-utils override deb pkg
   - Fix runlevel in sysv scripts for systemd
   
   - [ ] Migrate sysv/init.d scripts to systemd
   - [ ] Migrate rc.local to systemd and fix `cmdline` consumption
   - [ ] Fix networking
   - [ ] Fix patching process (one time with systemd)
   
   Pinging for review and additional help with changes @wido @swill and others
   This branch is pushed on ASF remote and allows for all committers to 
contribute changes directly, feel free to push fixes as separate commits, 
thanks.
   
   Signed-off-by: Rohit Yadav 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch debian9-systemvmtemplate created (now 319c457)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


  at 319c457  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

This branch includes the following new commits:

 new 319c457  CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

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.


-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch debian9-systemvmtemplate
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 319c4570e4ccae83e592189cd2fdb34afc96191d
Author: Rohit Yadav 
AuthorDate: Sun Jul 23 18:01:35 2017 +0200

CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

  SystemVM changes to work on Debian 9
- Migrate away from chkconfig to systemctl
- Remove xenstore-utils override deb pkg
- Fix runlevel in sysv scripts for systemd

Signed-off-by: Rohit Yadav 
---
 systemvm/patches/debian/buildsystemvm.sh   | 577 -
 systemvm/patches/debian/config/etc/init.d/cloud|   2 +-
 .../debian/config/etc/init.d/cloud-early-config|  19 +-
 .../debian/config/etc/init.d/cloud-passwd-srvr |   8 +-
 .../debian/config/opt/cloud/bin/patchsystemvm.sh   | 118 ++---
 tools/appliance/build.sh   |   2 +-
 .../definitions/systemvmtemplate/apt_upgrade.sh|   9 +-
 .../systemvmtemplate/authorized_keys.sh|   1 +
 .../definitions/systemvmtemplate/build_time.sh |  26 -
 .../definitions/systemvmtemplate/cleanup.sh|   1 -
 .../definitions/systemvmtemplate/configure_grub.sh |   4 +-
 .../systemvmtemplate/configure_login.sh|  14 +-
 .../systemvmtemplate/configure_networking.sh   |  24 +-
 .../configure_systemvm_services.sh |  29 +-
 .../definitions/systemvmtemplate/definition.rb |  34 +-
 .../systemvmtemplate/install_systemvm_packages.sh  |  29 +-
 .../definitions/systemvmtemplate/preseed.cfg   |  62 +--
 .../definitions/systemvmtemplate/zerodisk.sh   |   2 +-
 18 files changed, 178 insertions(+), 783 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
deleted file mode 100755
index a34b1dd..000
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ /dev/null
@@ -1,577 +0,0 @@
-#!/bin/bash
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements.  See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership.  The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License.  You may obtain a copy of the License at
-# 
-#   http://www.apache.org/licenses/LICENSE-2.0
-# 
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied.  See the License for the
-# specific language governing permissions and limitations
-# under the License.
-
-echo ""
-echo " Note there is a new systemvm build script based on "
-echo " Veewee(Vagrant) under tools/appliance."
-echo ""
-
-set -e
-set -x
-
-IMAGENAME=systemvm
-LOCATION=/var/lib/images/systemvm
-PASSWORD=password
-#APT_PROXY=192.168.1.115:3142/
-APT_PROXY=
-HOSTNAME=systemvm
-SIZE=2000
-DEBIAN_MIRROR=ftp.us.debian.org/debian
-MINIMIZE=true
-CLOUDSTACK_RELEASE=4.0
-offset=4096
-baseimage() {
-  mkdir -p $LOCATION
-  #dd if=/dev/zero of=$IMAGELOC bs=1M  count=$SIZE
-  dd if=/dev/zero of=$IMAGELOC bs=1M seek=$((SIZE - 1)) count=1
-  loopdev=$(losetup -f)
-  losetup $loopdev $IMAGELOC
-  parted $loopdev -s 'mklabel msdos'
-  parted $loopdev -s 'mkpart primary ext3 4096B -1'
-  sleep 2 
-  losetup -d $loopdev
-  loopdev=$(losetup --show -o $offset -f $IMAGELOC )
-  mkfs.ext3  -L ROOT $loopdev
-  mkdir -p $MOUNTPOINT
-  tune2fs -c 100 -i 0 $loopdev
-  sleep 2 
-  losetup -d $loopdev
-  
-  mount -o loop,offset=$offset $IMAGELOC  $MOUNTPOINT
-  
-  #debootstrap --variant=minbase 
--keyring=/usr/share/keyrings/debian-archive-keyring.gpg wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-  debootstrap --variant=minbase --arch=i386 wheezy $MOUNTPOINT 
http://${APT_PROXY}${DEBIAN_MIRROR}
-}
-
-
-fixapt() {
-  if [ "$APT_PROXY" != "" ]; then
-  cat >> etc/apt/apt.conf.d/01proxy << EOF
-Acquire::http::Proxy "http://${APT_PROXY};;
-EOF
-  fi
-
-  cat > etc/apt/sources.list << EOF
-deb http://http.debian.net/debian/ wheezy main contrib non-free
-deb-src http://http.debian.net/debian/ wheezy main contrib non-free
-
-deb http://security.debian.org/ wheezy/updates main
-deb-src http://security.debian.org/ wheezy/updates main
-
-deb http://http.debian.net/debian/ wheezy-backports main
-deb-src http://http.debian.net/debian/ wheezy-backports main
-EOF
-
-  cat >> etc/apt/apt.conf << EOF
-APT::Default-Release "stable"; 
-EOF
-
-  cat >> etc/apt/preferences << EOF
-Package: *
-Pin: release o=Debian,a=stable
-Pin-Priority: 900
-EOF
-
-  #apt-key exportall | chroot . apt-key add - &&
-  chroot . apt-get 

[GitHub] blueorangutan commented on issue #2210: WIP - Speed up initial VR configuration

[git]

blueorangutan commented on issue #2210: WIP - Speed up initial VR configuration
URL: https://github.com/apache/cloudstack/pull/2210#issuecomment-318737698
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-882
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2210: WIP - Speed up initial VR configuration

[git]

blueorangutan commented on issue #2210: WIP - Speed up initial VR configuration
URL: https://github.com/apache/cloudstack/pull/2210#issuecomment-318733009
 
 
   @PaulAngus 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 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] PaulAngus commented on issue #2210: WIP - Speed up initial VR configuration

[git]

PaulAngus commented on issue #2210: WIP - Speed up initial VR configuration
URL: https://github.com/apache/cloudstack/pull/2210#issuecomment-318732931
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318728621
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-881
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for 
Secondary IPv6 Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318728330
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-880
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318675630
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-878
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318668616
 
 
   @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 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] rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318668435
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318682750
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-879
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

blueorangutan commented on issue #2028: CLOUDSTACK-9853: Add support for 
Secondary IPv6 Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318721291
 
 
   @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 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] rhtyd closed pull request #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account

[git]

rhtyd closed pull request #2171: CLOUDSTACK-9990 : Account name is giving null 
in event tab after successful creation of account
URL: https://github.com/apache/cloudstack/pull/2171
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch master updated: CLOUDSTACK-9990 : Fix for Account name is giving null in event tab after successful creation of account (#2171)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new 08a6c24  CLOUDSTACK-9990 : Fix for Account name is giving null in 
event tab after successful creation of account (#2171)
08a6c24 is described below

commit 08a6c24efc3f8b834541edc5b789b9966918c117
Author: pavanaravapalli 
AuthorDate: Fri Jul 28 23:25:00 2017 +0530

CLOUDSTACK-9990 : Fix for Account name is giving null in event tab after 
successful creation of account (#2171)
---
 .../apache/cloudstack/api/command/admin/account/CreateAccountCmd.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 
b/api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java
index 2e1fbd6..074df8d 100644
--- 
a/api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java
+++ 
b/api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java
@@ -184,7 +184,7 @@ public class CreateAccountCmd extends BaseCmd {
 @Override
 public void execute() {
 validateParams();
-CallContext.current().setEventDetails("Account Name: " + 
getAccountName() + ", Domain Id:" + getDomainId());
+CallContext.current().setEventDetails("Account Name: " + getUsername() 
+ ", Domain Id:" + getDomainId());
 UserAccount userAccount =
 _accountService.createUserAccount(getUsername(), getPassword(), 
getFirstName(), getLastName(), getEmail(), getTimeZone(), getAccountName(), 
getAccountType(), getRoleId(),
 getDomainId(), getNetworkDomain(), getDetails(), 
getAccountUUID(), getUserUUID());

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[GitHub] rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318721226
 
 
   @wido thanks, will re-run tests
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account

[git]

blueorangutan commented on issue #2171: CLOUDSTACK-9990 : Account name is 
giving null in event tab after successful creation of account
URL: https://github.com/apache/cloudstack/pull/2171#issuecomment-318714240
 
 
   Trillian test result (tid-1268)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 30462 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2171-t1268-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Test completed. 50 look ok, 1 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_rvpc_privategw_static_routes | `Failure` | 376.78 | 
test_privategw_acl.py
   test_change_service_offering_for_vm_with_snapshots | Skipped | 0.00 | 
test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.02 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
   test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.04 | test_primary_storage.py
   test_nested_virtualization_vmware | Skipped | 0.00 | 
test_nested_virtualization.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.03 | test_deploy_vgpu_enabled_vm.py
   test_3d_gpu_support | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 
release effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318714066
 
 
   Trillian test result (tid-1270)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 30089 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2052-t1270-kvm-centos7.zip
   Intermitten failure detected: /marvin/tests/smoke/test_nic.py
   Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Test completed. 47 look ok, 2 have error(s)
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_04_rvpc_privategw_static_routes | `Failure` | 381.67 | 
test_privategw_acl.py
   test_01_nic | `Error` | 218.02 | test_nic.py
   test_03_delete_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
   test_02_revert_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
   test_01_create_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
   test_09_copy_delete_template | Skipped | 0.02 | test_templates.py
   test_06_copy_template | Skipped | 0.00 | test_templates.py
   test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
   test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
   test_01_primary_storage_iscsi | Skipped | 0.05 | test_primary_storage.py
   test_06_copy_iso | Skipped | 0.00 | test_iso.py
   test_deploy_vgpu_enabled_vm | Skipped | 0.01 | test_deploy_vgpu_enabled_vm.py
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] cloudmonger commented on issue #1246: CLOUDSTACK-9165 unable to use reserved IP range in a network for exte?

[git]

cloudmonger commented on issue #1246: CLOUDSTACK-9165 unable to use reserved IP 
range in a network for exte?
URL: https://github.com/apache/cloudstack/pull/1246#issuecomment-318711637
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1021
Hypervisor xenserver
NetworkType Advanced
Passed=100
Failed=14
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_routers.py
   
* test_03_restart_network_cleanup Failed
   
* test_04_restart_network_wo_cleanup Failed
   
* test_05_router_basic Failed
   
* test_06_router_advanced Failed
   
* test_08_start_router Failed
   
* test_09_reboot_router Failed
   
   * test_network.py
   
* test_reboot_router Failed
   
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 76 runs
   
   * test_nic.py
   
* test_01_nic Failed
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 76 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
109 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
109 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
107 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
107 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_router_dnsservice.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_router_dns.py
   test_non_contigiousvlan.py
   test_login.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] cloudmonger commented on issue #1246: CLOUDSTACK-9165 unable to use reserved IP range in a network for exte?

[git]

cloudmonger commented on issue #1246: CLOUDSTACK-9165 unable to use reserved IP 
range in a network for exte?
URL: https://github.com/apache/cloudstack/pull/1246#issuecomment-284015235
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 419
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=2
Skipped=7
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
   
   
   **Failed tests:**
   * test_volumes.py
   
* test_06_download_detached_volume Failed
   
   * test_routers_network_ops.py
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failed
   
   
   **Skipped tests:**
   test_01_test_vm_volume_snapshot
   test_vm_nic_adapter_vmxnet3
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] PaulAngus opened a new pull request #2210: WIP - Speed up initial VR configuration

[git]

PaulAngus opened a new pull request #2210: WIP - Speed up initial VR 
configuration
URL: https://github.com/apache/cloudstack/pull/2210
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch cloudearlyinit created (now a2aabb8)

[paul_a]

This is an automated email from the ASF dual-hosted git repository.

paul_a pushed a change to branch cloudearlyinit
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


  at a2aabb8  first stab a adding a smaller 'start up helper' to VR patching

This branch includes the following new commits:

 new a2aabb8  first stab a adding a smaller 'start up helper' to VR patching

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.


-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: first stab a adding a smaller 'start up helper' to VR patching

[paul_a]

This is an automated email from the ASF dual-hosted git repository.

paul_a pushed a commit to branch cloudearlyinit
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit a2aabb8f126bb952ed78eb821e94d4b678ccbb22
Author: Paul Angus 
AuthorDate: Fri Jul 28 16:30:22 2017 +0100

first stab a adding a smaller 'start up helper' to VR patching
---
 systemvm/patches/debian/buildsystemvm.sh   |   4 +-
 .../debian/config/etc/init.d/cloud-early-config|   4 +-
 .../debian/config/etc/init.d/cloud-early-init  | 423 +
 .../configure_systemvm_services.sh |   4 +-
 4 files changed, 431 insertions(+), 4 deletions(-)

diff --git a/systemvm/patches/debian/buildsystemvm.sh 
b/systemvm/patches/debian/buildsystemvm.sh
index a34b1dd..f872643 100755
--- a/systemvm/patches/debian/buildsystemvm.sh
+++ b/systemvm/patches/debian/buildsystemvm.sh
@@ -228,7 +228,7 @@ cat > etc/init.d/iptables-persistent << EOF
 #!/bin/sh
 ### BEGIN INIT INFO
 # Provides:  iptables
-# Required-Start:mountkernfs $local_fs
+# Required-Start:mountkernfs $local_fs cloud-early-init
 # Required-Stop: $local_fs
 # Should-Start:  cloud-early-config
 # Default-Start: S
@@ -418,6 +418,8 @@ services() {
   
   /bin/cp -r ${scriptdir}/config/* ./
   chroot . chkconfig xl2tpd off
+  chroot . chkconfig --add cloud-early-init
+  chroot . chkconfig cloud-early-init on
   chroot . chkconfig --add cloud-early-config
   chroot . chkconfig cloud-early-config on
   chroot . chkconfig --add iptables-persistent
diff --git a/systemvm/patches/debian/config/etc/init.d/cloud-early-config 
b/systemvm/patches/debian/config/etc/init.d/cloud-early-config
index ac54ebf..ba306c7 100755
--- a/systemvm/patches/debian/config/etc/init.d/cloud-early-config
+++ b/systemvm/patches/debian/config/etc/init.d/cloud-early-config
@@ -1,8 +1,8 @@
 #!/bin/bash
 ### BEGIN INIT INFO
 # Provides:  cloud-early-config
-# Required-Start:mountkernfs $local_fs
-# Required-Stop: $local_fs
+# Required-Start:mountkernfs $local_fs cloud-early-init
+# Required-Stop: $local_fs cloud-early-init
 # Should-Start:  
 # Should-Stop:   
 # Default-Start: S
diff --git a/systemvm/patches/debian/config/etc/init.d/cloud-early-init 
b/systemvm/patches/debian/config/etc/init.d/cloud-early-init
new file mode 100644
index 000..77b8036
--- /dev/null
+++ b/systemvm/patches/debian/config/etc/init.d/cloud-early-init
@@ -0,0 +1,423 @@
+#!/bin/bash
+### BEGIN INIT INFO
+# Provides:  cloud-early-init
+# Required-Start:mountkernfs $local_fs
+# Required-Stop: $local_fs
+# Should-Start:  
+# Should-Stop:   
+# Default-Start: S
+# Default-Stop:  0 6
+# Short-Description: configure according to cmdline
+### END INIT INFO
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+PATH="/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin"
+
+[ -f /usr/sbin/hv_kvp_daemon ] && /usr/sbin/hv_kvp_daemon
+
+# Fix haproxy directory issue
+mkdir -p /var/lib/haproxy
+
+# Clear boot up flag, it would be created by rc.local after boot up done
+rm -f /var/cache/cloud/boot_up_done
+
+[ -x /sbin/ifup ] || exit 0
+
+. /lib/lsb/init-functions
+
+log_it() {
+  echo "$(date) $@" >> /var/log/cloud.log
+  log_action_msg "$@"
+}
+
+hypervisor() {
+  [ -d /proc/xen ] && mount -t xenfs none /proc/xen
+  [ -d /proc/xen ] && echo "xen-domU" && return 0
+
+  local try=$([ -x /usr/sbin/virt-what ] && virt-what | tail -1)
+  [ "$try" != "" ] && echo $try && return 0
+
+  vmware-checkvm &> /dev/null && echo "vmware" && return 0
+
+  grep -q QEMU /proc/cpuinfo  && echo "kvm" && return 0
+  grep -q QEMU /var/log/messages && echo "kvm" && return 0
+
+  echo "unknown" && return 1
+
+}
+
+get_boot_params() {
+  local EXTRA_MOUNT=/media/extra
+  local hyp=$(hypervisor)
+  [ $? -ne 0 ] && log_it "Failed to detect hypervisor type, bailing out of 
early init" && exit 10
+
+  case $hyp in
+ xen-domU|xen-hvm)
+  cat /proc/cmdline > /var/cache/cloud/cmdline
+  sed -i "s/%/ /g" /var/cache/cloud/cmdline
+  ;;
+ kvm)
+  VPORT=$(find /dev/virtio-ports -type l -name '*.vport' 
2>/dev/null|head -1)
+
+  

[GitHub] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318682750
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-879
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318679037
 
 
   @borisstoyanov 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 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] borisstoyanov commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

borisstoyanov commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318678889
 
 
   @blueorangutan test
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318675612
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-877
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318675630
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-878
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] DaanHoogland commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

DaanHoogland commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318674229
 
 
   sorry @borisroman , just missing you ;)
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

blueorangutan commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 
for systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318668616
 
 
   @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 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] rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318668435
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

blueorangutan commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318668355
 
 
   @borisstoyanov 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 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] borisstoyanov commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

borisstoyanov commented on issue #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-318668171
 
 
   Hey @DaanHoogland I think you pinged the wrong Boris, but don't worry 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318654433
 
 
   All checks have now passed @rhtyd 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318650268
 
 
   
![a](https://user-images.githubusercontent.com/95203/28718976-8147ab22-73a8-11e7-9dcc-4978b139.png)
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for systemvmtemplate

[git]

rhtyd commented on issue #2198: CLOUDSTACK-10013: Migrate to Debian9 for 
systemvmtemplate
URL: https://github.com/apache/cloudstack/pull/2198#issuecomment-318649146
 
 
   @wido I've got this to a stage where the VM is built and scripts run to 
install pkgs and configure it. However, given Debian9 is systemd based we'll 
need a lot of work to migrate existing scripts/code to be systemd aware. I've 
started some work in this regard but I think it may be a bigger refactoring 
effort than I previously thought.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] borisstoyanov opened a new pull request #2209: CLOUDSTACK-10017: Adding 'quick-test' attribute to list of smoketests?

[git]

borisstoyanov opened a new pull request #2209: CLOUDSTACK-10017: Adding 
'quick-test' attribute to list of smoketests?
URL: https://github.com/apache/cloudstack/pull/2209
 
 
   ? and adding 'vpc', 'templates', 'deploy-vm', 'roles', 'service-offerings', 
'storage', 'routers', 'networks', 'usage', 'accounts' tags
   
   quick-test label aims to provide quick feedback of recent changes to the 
developer, instead of waiting for the whole smoketest to execute(24h+). This 
execution lasts about 1.5h at Trillian environments.
   
   The categories aim to offer more efficient way to slice through tests in 
both smoke and component tests. Let's say as a developer you've made a change 
in the accounts. You can simply trigger marvin tests with tag=accounts pointing 
all the tests for example:
   ```
   
   nosetests --with-marvin --marvin-config=[config] --hypervisor=xenserver -a 
tags=accounts /tests/integration/*/*.*
   
   ```
   
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Test+Categories+and+Quick-test+in+Marvin
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] nvazquez commented on issue #2189: CLOUDSTACK-10006: Internal DRS-like load balancing implementation for Vmware

[git]

nvazquez commented on issue #2189: CLOUDSTACK-10006: Internal DRS-like load 
balancing implementation for Vmware
URL: https://github.com/apache/cloudstack/pull/2189#issuecomment-318634373
 
 
   Thanks @borisstoyanov, sorry for the delay in response. I'll work on a way 
to test this feature
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] nvazquez commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring

[git]

nvazquez commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring
URL: https://github.com/apache/cloudstack/pull/1593#issuecomment-318632304
 
 
   Sure, done @rhtyd 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] cloudmonger commented on issue #2144: CLOUDSTACK-9955 : Featured Templates/Iso's created by Root/admin user are not visible to Domain Admin users

[git]

cloudmonger commented on issue #2144: CLOUDSTACK-9955 : Featured 
Templates/Iso's created by Root/admin user are not visible to Domain Admin users
URL: https://github.com/apache/cloudstack/pull/2144#issuecomment-318630313
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 1020
Hypervisor xenserver
NetworkType Advanced
Passed=107
Failed=6
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_deploy_vm_iso.py
   
* test_deploy_vm_from_iso Failing since 75 runs
   
   * test_vm_life_cycle.py
   
* test_10_attachAndDetach_iso Failing since 75 runs
   
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 
108 runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
108 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 
106 runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
106 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_non_contigiousvlan.py
   test_login.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] cloudmonger commented on issue #2144: CLOUDSTACK-9955 : Featured Templates/Iso's created by Root/admin user are not visible to Domain Admin users

[git]

cloudmonger commented on issue #2144: CLOUDSTACK-9955 : Featured 
Templates/Iso's created by Root/admin user are not visible to Domain Admin users
URL: https://github.com/apache/cloudstack/pull/2144#issuecomment-308220131
 
 
   ### ACS CI BVT Run
**Sumarry:**
Build Number 836
Hypervisor xenserver
NetworkType Advanced
Passed=109
Failed=4
Skipped=12
   
   _Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
   
   **Failed tests:**
   * test_routers_network_ops.py
   
* test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 24 
runs
   
* test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 
24 runs
   
* test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 24 
runs
   
* test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
24 runs
   
   
   **Skipped tests:**
   test_vm_nic_adapter_vmxnet3
   test_01_verify_libvirt
   test_02_verify_libvirt_after_restart
   test_03_verify_libvirt_attach_disk
   test_04_verify_guest_lspci
   test_05_change_vm_ostype_restart
   test_06_verify_guest_lspci_again
   test_static_role_account_acls
   test_11_ss_nfs_version_on_ssvm
   test_nested_virtualization_vmware
   test_3d_gpu_support
   test_deploy_vgpu_enabled_vm
   
   **Passed test suits:**
   test_deploy_vm_with_userdata.py
   test_affinity_groups_projects.py
   test_portable_publicip.py
   test_vm_snapshots.py
   test_over_provisioning.py
   test_global_settings.py
   test_scale_vm.py
   test_service_offerings.py
   test_routers_iptables_default_policy.py
   test_loadbalance.py
   test_routers.py
   test_reset_vm_on_reboot.py
   test_deploy_vms_with_varied_deploymentplanners.py
   test_network.py
   test_router_dns.py
   test_non_contigiousvlan.py
   test_login.py
   test_deploy_vm_iso.py
   test_list_ids_parameter.py
   test_public_ip_range.py
   test_multipleips_per_nic.py
   test_metrics_api.py
   test_regions.py
   test_affinity_groups.py
   test_network_acl.py
   test_pvlan.py
   test_volumes.py
   test_nic.py
   test_deploy_vm_root_resize.py
   test_resource_detail.py
   test_secondary_storage.py
   test_vm_life_cycle.py
   test_disk_offerings.py
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch master updated: solidfire: Add NULL checks for various objects in SolidFire integration test API (#2205)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new a4cecd2  solidfire: Add NULL checks for various objects in SolidFire 
integration test API (#2205)
a4cecd2 is described below

commit a4cecd2366dbca3545b3fe0defb1ba458ff7f7bb
Author: Syed Mushtaq Ahmed 
AuthorDate: Fri Jul 28 05:58:05 2017 -0400

solidfire: Add NULL checks for various objects in SolidFire integration 
test API (#2205)
---
 .../SolidFireIntegrationTestManagerImpl.java   | 16 
 .../solidfire/SolidFireIntegrationTestUtil.java| 43 --
 2 files changed, 41 insertions(+), 18 deletions(-)

diff --git 
a/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/solidfire/SolidFireIntegrationTestManagerImpl.java
 
b/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/solidfire/SolidFireIntegrationTestManagerImpl.java
index ff6e72c..ba873aa 100644
--- 
a/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/solidfire/SolidFireIntegrationTestManagerImpl.java
+++ 
b/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/solidfire/SolidFireIntegrationTestManagerImpl.java
@@ -16,21 +16,20 @@
 // under the License.
 package org.apache.cloudstack.solidfire;
 
-import javax.inject.Inject;
-
-import org.apache.cloudstack.storage.datastore.util.SolidFireUtil;
-import org.apache.cloudstack.util.solidfire.SolidFireIntegrationTestUtil;
-import org.springframework.stereotype.Component;
-
 import com.cloud.dc.ClusterDetailsDao;
 import com.cloud.dc.ClusterDetailsVO;
 import com.cloud.storage.VolumeDetailVO;
 import com.cloud.storage.VolumeVO;
 import com.cloud.storage.dao.VolumeDao;
 import com.cloud.storage.dao.VolumeDetailsDao;
-import com.cloud.user.AccountDetailsDao;
 import com.cloud.user.AccountDetailVO;
+import com.cloud.user.AccountDetailsDao;
 import com.cloud.utils.exception.CloudRuntimeException;
+import org.apache.cloudstack.storage.datastore.util.SolidFireUtil;
+import org.apache.cloudstack.util.solidfire.SolidFireIntegrationTestUtil;
+import org.springframework.stereotype.Component;
+
+import javax.inject.Inject;
 
 @Component
 public class SolidFireIntegrationTestManagerImpl implements 
SolidFireIntegrationTestManager {
@@ -47,6 +46,9 @@ public class SolidFireIntegrationTestManagerImpl implements 
SolidFireIntegration
 long storagePoolId = 
util.getStoragePoolIdForStoragePoolUuid(storagePoolUuid);
 
 AccountDetailVO accountDetail = 
accountDetailsDao.findDetail(csAccountId, 
SolidFireUtil.getAccountKey(storagePoolId));
+if (accountDetail == null){
+throw new CloudRuntimeException("Unable to find SF account for 
storage " + storagePoolUuid + " for CS account " + csAccountUuid);
+}
 String sfAccountId = accountDetail.getValue();
 
 return Long.parseLong(sfAccountId);
diff --git 
a/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/util/solidfire/SolidFireIntegrationTestUtil.java
 
b/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/util/solidfire/SolidFireIntegrationTestUtil.java
index 307e8c5..427af11 100644
--- 
a/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/util/solidfire/SolidFireIntegrationTestUtil.java
+++ 
b/plugins/api/solidfire-intg-test/src/org/apache/cloudstack/util/solidfire/SolidFireIntegrationTestUtil.java
@@ -16,15 +16,6 @@
 // under the License.
 package org.apache.cloudstack.util.solidfire;
 
-import java.util.ArrayList;
-import java.util.List;
-
-import javax.inject.Inject;
-
-import 
org.apache.cloudstack.api.response.solidfire.ApiVolumeSnapshotDetailsResponse;
-import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
-import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
-
 import com.cloud.dc.ClusterVO;
 import com.cloud.dc.dao.ClusterDao;
 import com.cloud.storage.SnapshotVO;
@@ -35,6 +26,14 @@ import com.cloud.storage.dao.SnapshotDetailsVO;
 import com.cloud.storage.dao.VolumeDao;
 import com.cloud.user.Account;
 import com.cloud.user.dao.AccountDao;
+import com.cloud.utils.exception.CloudRuntimeException;
+import 
org.apache.cloudstack.api.response.solidfire.ApiVolumeSnapshotDetailsResponse;
+import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
+import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
+
+import javax.inject.Inject;
+import java.util.ArrayList;
+import java.util.List;
 
 public class SolidFireIntegrationTestUtil {
 @Inject private AccountDao accountDao;
@@ -48,48 +47,70 @@ public class SolidFireIntegrationTestUtil {
 
 public long getAccountIdForAccountUuid(String accountUuid) {
 Account account = accountDao.findByUuid(accountUuid);
-
+if (account == null){
+throw new CloudRuntimeException("Unable to find Account for ID: " 
+ accountUuid);
+}
 

[GitHub] rhtyd closed pull request #2205: Add NULL checks for various objects in SolidFire integration test API

[git]

rhtyd closed pull request #2205: Add NULL checks for various objects in 
SolidFire integration test API
URL: https://github.com/apache/cloudstack/pull/2205
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

[git]

SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create 
Storage_pool_host_ref entry?
URL: https://github.com/apache/cloudstack/pull/876#issuecomment-318608705
 
 
   @karuturi I had rebased the branch. 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API

[git]

blueorangutan commented on issue #2205: Add NULL checks for various objects in 
SolidFire integration test API
URL: https://github.com/apache/cloudstack/pull/2205#issuecomment-318607806
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-876
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2193: CLOUDSTACK-10007 Isolation methods

[git]

rhtyd commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-318607052
 
 
   @DaanHoogland the JIRA id in the PR is not enough and useful, if the JIRA id 
is part of the git commit (summary) it is easier to generate changelog 
(https://github.com/apache/cloudstack-docs-rn/tree/master/utils) and 
check/refer to PRs/issues later from git history. This has been an issue for 
me, and I found a workaround that is to use Github UI to squash/merge the PR 
and added the cloudstack ID (if specified in the PR summary).
   
   Please fix conflicts and rebase against latest master, and address issues 
raised by reviewers.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] DaanHoogland commented on a change in pull request #2193: CLOUDSTACK-10007 Isolation methods

[git]

DaanHoogland commented on a change in pull request #2193: CLOUDSTACK-10007 
Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#discussion_r130050101
 
 

 ##
 File path: 
plugins/network-elements/nicira-nvp/src/com/cloud/network/guru/NiciraNvpGuestNetworkGuru.java
 ##
 @@ -0,0 +1,254 @@
+//
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+//
+
+package com.cloud.network.guru;
+
+import com.cloud.agent.AgentManager;
+import com.cloud.agent.api.CreateLogicalSwitchAnswer;
+import com.cloud.agent.api.CreateLogicalSwitchCommand;
+import com.cloud.agent.api.DeleteLogicalSwitchAnswer;
+import com.cloud.agent.api.DeleteLogicalSwitchCommand;
+import com.cloud.dc.DataCenter;
+import com.cloud.dc.DataCenter.NetworkType;
+import com.cloud.dc.dao.DataCenterDao;
+import com.cloud.deploy.DeployDestination;
+import com.cloud.deploy.DeploymentPlan;
+import com.cloud.exception.InsufficientAddressCapacityException;
+import com.cloud.exception.InsufficientVirtualNetworkCapacityException;
+import com.cloud.host.HostVO;
+import com.cloud.host.dao.HostDao;
+import com.cloud.host.dao.HostDetailsDao;
+import com.cloud.network.Network;
+import com.cloud.network.Network.GuestType;
+import com.cloud.network.Network.Service;
+import com.cloud.network.Network.State;
+import com.cloud.network.NetworkModel;
+import com.cloud.network.NetworkProfile;
+import com.cloud.network.Networks.BroadcastDomainType;
+import com.cloud.network.NiciraNvpDeviceVO;
+import com.cloud.network.PhysicalNetwork;
+import com.cloud.network.PhysicalNetwork.IsolationMethod;
+import com.cloud.network.dao.NetworkDao;
+import com.cloud.network.dao.NetworkVO;
+import com.cloud.network.dao.NiciraNvpDao;
+import com.cloud.network.dao.PhysicalNetworkDao;
+import com.cloud.network.dao.PhysicalNetworkVO;
+import com.cloud.offering.NetworkOffering;
+import com.cloud.offerings.dao.NetworkOfferingServiceMapDao;
+import com.cloud.resource.ResourceManager;
+import com.cloud.user.Account;
+import com.cloud.user.dao.AccountDao;
+import com.cloud.vm.NicProfile;
+import com.cloud.vm.ReservationContext;
+import com.cloud.vm.VirtualMachineProfile;
+import org.apache.log4j.Logger;
+
+import javax.ejb.Local;
+import javax.inject.Inject;
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.util.List;
+
+@Local(value = NetworkGuru.class)
+public class NiciraNvpGuestNetworkGuru extends GuestNetworkGuru {
 
 Review comment:
   and another one
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] DaanHoogland commented on a change in pull request #2193: CLOUDSTACK-10007 Isolation methods

[git]

DaanHoogland commented on a change in pull request #2193: CLOUDSTACK-10007 
Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#discussion_r130050022
 
 

 ##
 File path: 
plugins/network-elements/bigswitch-vns/src/com/cloud/network/guru/BigSwitchVnsGuestNetworkGuru.java
 ##
 @@ -0,0 +1,238 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package com.cloud.network.guru;
+
+import com.cloud.agent.AgentManager;
+import com.cloud.agent.api.CreateVnsNetworkAnswer;
+import com.cloud.agent.api.CreateVnsNetworkCommand;
+import com.cloud.agent.api.DeleteVnsNetworkCommand;
+import com.cloud.dc.DataCenter;
+import com.cloud.dc.DataCenter.NetworkType;
+import com.cloud.dc.dao.DataCenterDao;
+import com.cloud.deploy.DeployDestination;
+import com.cloud.deploy.DeploymentPlan;
+import com.cloud.exception.InsufficientAddressCapacityException;
+import com.cloud.exception.InsufficientVirtualNetworkCapacityException;
+import com.cloud.host.HostVO;
+import com.cloud.host.dao.HostDao;
+import com.cloud.host.dao.HostDetailsDao;
+import com.cloud.network.BigSwitchVnsDeviceVO;
+import com.cloud.network.Network;
+import com.cloud.network.Network.GuestType;
+import com.cloud.network.Network.State;
+import com.cloud.network.NetworkProfile;
+import com.cloud.network.Networks.BroadcastDomainType;
+import com.cloud.network.PhysicalNetwork;
+import com.cloud.network.PhysicalNetwork.IsolationMethod;
+import com.cloud.network.dao.BigSwitchVnsDao;
+import com.cloud.network.dao.NetworkVO;
+import com.cloud.network.dao.PhysicalNetworkDao;
+import com.cloud.network.dao.PhysicalNetworkVO;
+import com.cloud.offering.NetworkOffering;
+import com.cloud.resource.ResourceManager;
+import com.cloud.user.Account;
+import com.cloud.user.dao.AccountDao;
+import com.cloud.vm.NicProfile;
+import com.cloud.vm.ReservationContext;
+import com.cloud.vm.VirtualMachineProfile;
+import org.apache.log4j.Logger;
+
+import javax.ejb.Local;
+import javax.inject.Inject;
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.util.List;
+
+@Local(value = NetworkGuru.class)
+public class BigSwitchVnsGuestNetworkGuru extends GuestNetworkGuru {
 
 Review comment:
   another rebase error, will remove it
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2193: CLOUDSTACK-10007 Isolation methods

[git]

rhtyd commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-318607052
 
 
   @DaanHoogland the JIRA id in the PR is not enough and useful, if the JIRA id 
is part of the git commit (summary) it is easier to generate changelog 
(https://github.com/apache/cloudstack-docs-rn/tree/master/utils) and 
check/refer to PRs/issues later from git history. This has been an issue for 
me, and I found a workaround that is to use Github UI to squash/merge the PR 
and added the cloudstack ID (if specified in the PR summary).
   Please fix conflicts and rebase against latest master.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] 01/01: Merge branch '4.10'

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

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

commit 619da913860b01f461abe5c0c4e8ecd183d040fa
Merge: b83aa2a e9f526e
Author: Rohit Yadav 
AuthorDate: Fri Jul 28 11:28:20 2017 +0200

Merge branch '4.10'

 agent/conf/agent.properties|  4 ++
 .../affinitygroup/UpdateVMAffinityGroupCmd.java| 12 +++--
 .../virtualnetwork/VirtualRoutingResource.java |  9 ++--
 .../network/guru/NuageVspGuestNetworkGuru.java | 36 +++---
 .../src/com/cloud/util/NuageVspEntityBuilder.java  | 56 +++---
 .../com/cloud/util/NuageVspEntityBuilderTest.java  | 33 ++---
 server/src/com/cloud/server/StatsCollector.java|  4 +-
 systemvm/patches/debian/config/etc/vpcdnsmasq.conf |  2 +-
 test/integration/component/test_affinity_groups.py | 27 ++-
 9 files changed, 121 insertions(+), 62 deletions(-)


-- 
To stop receiving notification emails like this one, please contact
"commits@cloudstack.apache.org" .

[cloudstack] branch master updated (b83aa2a -> 619da91)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

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


from b83aa2a  CLOUDSTACK-10021: Marvin test to check VR internal DNS 
Service (#1784)
 add 7ed3179  CLOUDSTACK-9569: add router.aggregation.command.each.timeout 
to agent.properties (#1933)
 add 471b686  CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from 
remote access vpn clients (#2201)
 add 28bc995  CLOUDSTACK-9631: API: affinitygroupids or affinitygroupnames 
must be given (#1798)
 add 0ead11e  CLOUDSTACK-9751: Fix public ip not applied, when added while 
VR is starting. (#1925)
 add e9f526e  Merge branch '4.9' into 4.10
 new 619da91  Merge branch '4.10'

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:
 agent/conf/agent.properties|  4 ++
 .../affinitygroup/UpdateVMAffinityGroupCmd.java| 12 +++--
 .../virtualnetwork/VirtualRoutingResource.java |  9 ++--
 .../network/guru/NuageVspGuestNetworkGuru.java | 36 +++---
 .../src/com/cloud/util/NuageVspEntityBuilder.java  | 56 +++---
 .../com/cloud/util/NuageVspEntityBuilderTest.java  | 33 ++---
 server/src/com/cloud/server/StatsCollector.java|  4 +-
 systemvm/patches/debian/config/etc/vpcdnsmasq.conf |  2 +-
 test/integration/component/test_affinity_groups.py | 27 ++-
 9 files changed, 121 insertions(+), 62 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[cloudstack] 01/01: Merge branch '4.9' into 4.10

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch 4.10
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit e9f526e221480ea783f704db947fa030c40a9578
Merge: c1f30d9 0ead11e
Author: Rohit Yadav 
AuthorDate: Fri Jul 28 10:47:47 2017 +0200

Merge branch '4.9' into 4.10

Signed-off-by: Rohit Yadav 

 agent/conf/agent.properties|  4 ++
 .../affinitygroup/UpdateVMAffinityGroupCmd.java| 12 +++--
 .../virtualnetwork/VirtualRoutingResource.java |  9 ++--
 .../network/guru/NuageVspGuestNetworkGuru.java | 36 +++---
 .../src/com/cloud/util/NuageVspEntityBuilder.java  | 56 +++---
 .../com/cloud/util/NuageVspEntityBuilderTest.java  | 33 ++---
 server/src/com/cloud/server/StatsCollector.java|  4 +-
 systemvm/patches/debian/config/etc/vpcdnsmasq.conf |  2 +-
 test/integration/component/test_affinity_groups.py | 27 ++-
 9 files changed, 121 insertions(+), 62 deletions(-)

diff --cc agent/conf/agent.properties
index c15b78e,8b99ee3..3ed382a
--- a/agent/conf/agent.properties
+++ b/agent/conf/agent.properties
@@@ -158,21 -153,6 +158,25 @@@ hypervisor.type=kv
  # Some newer linux kernels are incapable of reliably migrating vms with 
kvmclock
  # This is a workaround for the bug, admin can set this to true per-host
  #
 -#router.aggregation.command.each.timeout=600
 -#timeout value for aggregation commands send to virtual router
 +# vm.rng.enable=false
 +# This enabled the VirtIO Random Number Generator device for guests.
 +#
 +# vm.rng.model=random
 +# The model of VirtIO Random Number Generator (RNG) to present to the Guest.
 +# Currently only 'random' is supported.
 +#
 +# vm.rng.path=/dev/random
 +# Local Random Number Device Generator to use for VirtIO RNG for Guests.
 +# This is usually /dev/random, but per platform this might be different
 +#
 +# vm.rng.rate.bytes=2048
 +# The amount of bytes the Guest may request/obtain from the RNG in the period
 +# specified below.
 +#
 +# vm.rng.rate.period=1000
 +# The number of milliseconds in which the guest is allowed to obtain the bytes
 +# specified above.
++
++# router.aggregation.command.each.timeout=600
++# timeout value for aggregation commands send to virtual router
+ #
diff --cc 
core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
index 1ea007e,e424bac..75f55f9
--- 
a/core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
+++ 
b/core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
@@@ -387,11 -390,11 +390,11 @@@ public class VirtualRoutingResource 
  ScriptConfigItem scriptConfigItem = new 
ScriptConfigItem(VRScripts.VR_CFG, "-c " + VRScripts.CONFIG_CACHE_LOCATION + 
cfgFileName);
  // 120s is the minimal timeout
  Duration timeout = 
_eachTimeout.withDurationAdded(_eachTimeout.getStandardSeconds(), answerCounts);
- if (timeout.isShorterThan(VRScripts.VR_SCRIPT_EXEC_TIMEOUT)) {
- timeout = VRScripts.VR_SCRIPT_EXEC_TIMEOUT;
+ if (s_logger.isDebugEnabled()){
+ s_logger.debug("Aggregate action timeout in seconds is " 
+ timeout.getStandardSeconds());
  }
  
 -ExecutionResult result = 
applyConfigToVR(cmd.getRouterAccessIp(), fileConfigItem);
 +ExecutionResult result = 
applyConfigToVR(cmd.getRouterAccessIp(), fileConfigItem, timeout);
  if (!result.isSuccess()) {
  return new Answer(cmd, false, result.getDetails());
  }
diff --cc 
plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
index 80f6aab,d99c71d..735356a
--- 
a/plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
+++ 
b/plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
@@@ -25,25 -23,17 +25,9 @@@ import java.util.Set
  
  import javax.inject.Inject;
  
- import com.google.common.collect.FluentIterable;
- import net.nuage.vsp.acs.client.api.model.VspDhcpDomainOption;
- import net.nuage.vsp.acs.client.api.model.VspDhcpVMOption;
- import net.nuage.vsp.acs.client.api.model.VspDomain;
--import net.nuage.vsp.acs.client.api.model.VspNetwork;
--import net.nuage.vsp.acs.client.api.model.VspNic;
--import net.nuage.vsp.acs.client.api.model.VspStaticNat;
--import net.nuage.vsp.acs.client.api.model.VspVm;
--
--import org.apache.log4j.Logger;
--
--import com.google.common.base.Strings;
- import com.google.common.collect.Iterables;
- import com.google.common.collect.LinkedListMultimap;
- import com.google.common.collect.Lists;
- import com.google.common.collect.Maps;
--
  import org.apache.cloudstack.resourcedetail.VpcDetailVO;
  import org.apache.cloudstack.resourcedetail.dao.VpcDetailsDao;
++import 

[cloudstack] branch 4.10 updated (c1f30d9 -> e9f526e)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a change to branch 4.10
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


from c1f30d9  Merge branch '4.9' into 4.10
 add 7ed3179  CLOUDSTACK-9569: add router.aggregation.command.each.timeout 
to agent.properties (#1933)
 add 471b686  CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from 
remote access vpn clients (#2201)
 add 28bc995  CLOUDSTACK-9631: API: affinitygroupids or affinitygroupnames 
must be given (#1798)
 add 0ead11e  CLOUDSTACK-9751: Fix public ip not applied, when added while 
VR is starting. (#1925)
 new e9f526e  Merge branch '4.9' into 4.10

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:
 agent/conf/agent.properties|  4 ++
 .../affinitygroup/UpdateVMAffinityGroupCmd.java| 12 +++--
 .../virtualnetwork/VirtualRoutingResource.java |  9 ++--
 .../network/guru/NuageVspGuestNetworkGuru.java | 36 +++---
 .../src/com/cloud/util/NuageVspEntityBuilder.java  | 56 +++---
 .../com/cloud/util/NuageVspEntityBuilderTest.java  | 33 ++---
 server/src/com/cloud/server/StatsCollector.java|  4 +-
 systemvm/patches/debian/config/etc/vpcdnsmasq.conf |  2 +-
 test/integration/component/test_affinity_groups.py | 27 ++-
 9 files changed, 121 insertions(+), 62 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" '].

[GitHub] DaanHoogland commented on issue #2193: CLOUDSTACK-10007 Isolation methods

[git]

DaanHoogland commented on issue #2193: CLOUDSTACK-10007 Isolation methods
URL: https://github.com/apache/cloudstack/pull/2193#issuecomment-318605456
 
 
   @rhtyd there is already a jira id on the title of the PR
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] DaanHoogland opened a new pull request #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

[git]

DaanHoogland opened a new pull request #2208: CLOUDSTACK-9542 make listNics and 
ListUserVms return uniform NIC data
URL: https://github.com/apache/cloudstack/pull/2208
 
 
   @PaulAngus @borisroman please see if this is what is expected
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API

[git]

rhtyd commented on issue #2205: Add NULL checks for various objects in 
SolidFire integration test API
URL: https://github.com/apache/cloudstack/pull/2205#issuecomment-318603123
 
 
   LGTM, the changes are strictly in solidfire codebase. GIven Travis passed, 
additional Trillian run are not needed.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API

[git]

blueorangutan commented on issue #2205: Add NULL checks for various objects in 
SolidFire integration test API
URL: https://github.com/apache/cloudstack/pull/2205#issuecomment-318602984
 
 
   @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 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] rhtyd commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API

[git]

rhtyd commented on issue #2205: Add NULL checks for various objects in 
SolidFire integration test API
URL: https://github.com/apache/cloudstack/pull/2205#issuecomment-318602860
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318602694
 
 
   @wido okay re-ran failed travis run. You can go to a travis job and re-kick 
it, might be one of the intermittent/env failures.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

[git]

wido commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 
Addresses and Subnets
URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318600756
 
 
   I checked @rhtyd, might be that I forgot a local force push. Did that again. 
All tests go through locally.
   
   Travis now fails on a OOB/IPMI test:
   
   The command ./tools/travis/script.sh smoke/test_outofbandmanagement 
smoke/test_over_provisioning smoke/test_password_server 
smoke/test_portable_publicip smoke/test_primary_storage 
smoke/test_privategw_acl smoke/test_public_ip_range smoke/test_pvlan 
smoke/test_regions smoke/test_reset_vm_on_reboot smoke/test_resource_detail 
smoke/test_router_dhcphosts smoke/test_router_dns smoke/test_routers 
smoke/test_routers_iptables_default_policy smoke/test_routers_network_ops 
smoke/test_scale_vm smoke/test_secondary_storage smoke/test_service_offerings 
smoke/test_snapshots smoke/test_ssvm smoke/test_staticroles 
smoke/test_templates smoke/test_usage_events smoke/test_vm_life_cycle 
smoke/test_vm_snapshots smoke/test_volumes smoke/test_vpc_redundant 
smoke/test_vpc_router_nics smoke/test_vpc_vpn smoke/misc/test_deploy_vm 
smoke/misc/test_escalations_templates smoke/misc/test_vm_ha 
smoke/misc/test_vm_sync exited with 1.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2094: Agent logrotation

[git]

blueorangutan commented on issue #2094: Agent logrotation
URL: https://github.com/apache/cloudstack/pull/2094#issuecomment-318599080
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-875
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 
release effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318598775
 
 
   @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + 
vmware55u3, centos7 mgmt + kvmcentos7) 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 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] rhtyd commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

rhtyd commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release 
effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318598643
 
 
   @blueorangutan test matrix
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

[git]

rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a 
multiple of 4 characters
URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-318598573
 
 
   @marcaurele can you fix build failures:
   
/home/travis/build/apache/cloudstack/engine/schema/src/com/cloud/upgrade/dao/Upgrade41000to41100.java:29:1:
 error: Duplicate import to line 26 - java.sql.Connection.
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

[git]

blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data 
is a multiple of 4 characters
URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-318595335
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-874
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 release effort

[git]

blueorangutan commented on issue #2052: [4.9] Backported fixes for 4.9.3.0 
release effort
URL: https://github.com/apache/cloudstack/pull/2052#issuecomment-318595116
 
 
   Packaging result: ?centos6 ?centos7 ?debian. JID-873
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] blueorangutan commented on issue #2094: Agent logrotation

[git]

blueorangutan commented on issue #2094: Agent logrotation
URL: https://github.com/apache/cloudstack/pull/2094#issuecomment-318594702
 
 
   @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 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] rhtyd commented on issue #2094: Agent logrotation

[git]

rhtyd commented on issue #2094: Agent logrotation
URL: https://github.com/apache/cloudstack/pull/2094#issuecomment-318594619
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #2083: Iptables speedup

[git]

rhtyd commented on issue #2083: Iptables speedup
URL: https://github.com/apache/cloudstack/pull/2083#issuecomment-317170918
 
 
   @DaanHoogland is this still valid, should we consider? The branch has 
conflicts and need to be rebased.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] rhtyd commented on issue #1966: CLOUDSTACK-9801: IPSec VPN does not work after vRouter reboot or recreate

[git]

rhtyd commented on issue #1966: CLOUDSTACK-9801: IPSec VPN does not work after 
vRouter reboot or recreate
URL: https://github.com/apache/cloudstack/pull/1966#issuecomment-318594347
 
 
   I'm not in position to decide merging this one, needs an additional review 
from one of the VPN gurus @swill @JayapalUradi and others
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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] branch 4.9 updated: CLOUDSTACK-9751: Fix public ip not applied, when added while VR is starting. (#1925)

[bhaisaab]

This is an automated email from the ASF dual-hosted git repository.

bhaisaab pushed a commit to branch 4.9
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/4.9 by this push:
 new 0ead11e  CLOUDSTACK-9751: Fix public ip not applied, when added while 
VR is starting. (#1925)
0ead11e is described below

commit 0ead11e8f1417f691851dc27341cdaf8b0eb4a4d
Author: Frank Maximus 
AuthorDate: Fri Jul 28 10:30:43 2017 +0200

CLOUDSTACK-9751: Fix public ip not applied, when added while VR is 
starting. (#1925)

Public IP state wasn't passed to the vsp client, making it ignore the apply 
public ip.

Reported-By: Raf Smeets 
---
 .../network/guru/NuageVspGuestNetworkGuru.java | 31 ++
 .../src/com/cloud/util/NuageVspEntityBuilder.java  | 70 +++---
 .../com/cloud/util/NuageVspEntityBuilderTest.java  |  2 +
 3 files changed, 68 insertions(+), 35 deletions(-)

diff --git 
a/plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
 
b/plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
index a6c17e3..d99c71d 100644
--- 
a/plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
+++ 
b/plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java
@@ -19,6 +19,22 @@
 
 package com.cloud.network.guru;
 
+import java.util.List;
+
+import javax.inject.Inject;
+
+import net.nuage.vsp.acs.client.api.model.VspNetwork;
+import net.nuage.vsp.acs.client.api.model.VspNic;
+import net.nuage.vsp.acs.client.api.model.VspStaticNat;
+import net.nuage.vsp.acs.client.api.model.VspVm;
+
+import org.apache.log4j.Logger;
+
+import com.google.common.base.Strings;
+
+import org.apache.cloudstack.resourcedetail.VpcDetailVO;
+import org.apache.cloudstack.resourcedetail.dao.VpcDetailsDao;
+
 import com.cloud.agent.AgentManager;
 import com.cloud.agent.api.Answer;
 import com.cloud.agent.api.guru.DeallocateVmVspCommand;
@@ -59,6 +75,7 @@ import com.cloud.offerings.dao.NetworkOfferingServiceMapDao;
 import com.cloud.user.Account;
 import com.cloud.user.AccountVO;
 import com.cloud.user.dao.AccountDao;
+import com.cloud.util.NuageVspEntityBuilder;
 import com.cloud.utils.StringUtils;
 import com.cloud.utils.db.DB;
 import com.cloud.utils.exception.CloudRuntimeException;
@@ -66,18 +83,6 @@ import com.cloud.vm.NicProfile;
 import com.cloud.vm.NicVO;
 import com.cloud.vm.ReservationContext;
 import com.cloud.vm.VirtualMachineProfile;
-import com.google.common.base.Strings;
-import com.cloud.util.NuageVspEntityBuilder;
-import net.nuage.vsp.acs.client.api.model.VspNetwork;
-import net.nuage.vsp.acs.client.api.model.VspNic;
-import net.nuage.vsp.acs.client.api.model.VspStaticNat;
-import net.nuage.vsp.acs.client.api.model.VspVm;
-import org.apache.cloudstack.resourcedetail.VpcDetailVO;
-import org.apache.cloudstack.resourcedetail.dao.VpcDetailsDao;
-import org.apache.log4j.Logger;
-
-import javax.inject.Inject;
-import java.util.List;
 
 public class NuageVspGuestNetworkGuru extends GuestNetworkGuru {
 public static final Logger s_logger = 
Logger.getLogger(NuageVspGuestNetworkGuru.class);
@@ -256,7 +261,7 @@ public class NuageVspGuestNetworkGuru extends 
GuestNetworkGuru {
 VspStaticNat vspStaticNat = null;
 if (staticNatIp != null) {
 VlanVO staticNatVlan = 
_vlanDao.findById(staticNatIp.getVlanId());
-vspStaticNat = _nuageVspEntityBuilder.buildVspStaticNat(null, 
staticNatIp, staticNatVlan, null);
+vspStaticNat = _nuageVspEntityBuilder.buildVspStaticNat(null, 
staticNatIp, staticNatVlan, vspNic);
 }
 
 HostVO nuageVspHost = 
getNuageVspHost(network.getPhysicalNetworkId());
diff --git 
a/plugins/network-elements/nuage-vsp/src/com/cloud/util/NuageVspEntityBuilder.java
 
b/plugins/network-elements/nuage-vsp/src/com/cloud/util/NuageVspEntityBuilder.java
index 8757ad0..db24050 100644
--- 
a/plugins/network-elements/nuage-vsp/src/com/cloud/util/NuageVspEntityBuilder.java
+++ 
b/plugins/network-elements/nuage-vsp/src/com/cloud/util/NuageVspEntityBuilder.java
@@ -19,6 +19,30 @@
 
 package com.cloud.util;
 
+import java.util.HashSet;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Set;
+import java.util.TreeSet;
+
+import javax.inject.Inject;
+
+import net.nuage.vsp.acs.client.api.model.VspAclRule;
+import net.nuage.vsp.acs.client.api.model.VspDomain;
+import net.nuage.vsp.acs.client.api.model.VspNetwork;
+import net.nuage.vsp.acs.client.api.model.VspNic;
+import net.nuage.vsp.acs.client.api.model.VspStaticNat;
+import net.nuage.vsp.acs.client.api.model.VspVm;
+import net.nuage.vsp.acs.client.common.model.Pair;
+
+import org.apache.commons.lang.StringUtils;
+import org.apache.log4j.Logger;
+
+import com.google.common.collect.Iterables;

[GitHub] rhtyd closed pull request #1925: [4.9] CLOUDSTACK-9751: Fix public ip not applied, when added while VR is starting

[git]

rhtyd closed pull request #1925: [4.9] CLOUDSTACK-9751: Fix public ip not 
applied, when added while VR is starting
URL: https://github.com/apache/cloudstack/pull/1925
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on 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