[GitHub] [cloudstack] svenvogel commented on issue #3824: clarify the usage of vlanipranges

2020-01-21 Thread GitBox
svenvogel commented on issue #3824: clarify the usage of vlanipranges
URL: https://github.com/apache/cloudstack/issues/3824#issuecomment-576580404
 
 
   @andrijapanicsb ok. hmm bad.
   
   @weizhouapache do you know about the problem, maybe in your environment?


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


With regards,
Apache Git Services


[GitHub] [cloudstack] bwsw commented on issue #3823: New Feature RFC: KVM domain XML transformer

2020-01-21 Thread GitBox
bwsw commented on issue #3823: New Feature RFC: KVM domain XML transformer
URL: https://github.com/apache/cloudstack/issues/3823#issuecomment-576581578
 
 
   @PaulAngus thanks for reply, but it's not what I mean. I really cry about 
this PR - it's huge pain for cloud admins of public cloud... It introduces 
user-driven VM alternations and very bad for troubleshooting. 
   
   I'm speaking about provider-driven XML alternation based on some kind of 
dynamic logic when VM is launched. E.g. when VM of SO_X is launched it searches 
for a spare GPU on host and it's ingested into XML pass-thru, or it cuts , say 
32GB of optane device LVM and passes it into VM as a cache device. It's 
provider driven and user cannot change it.
   
   Real-world example, for every domain/account we have VXLANID in MySQL table. 
When VM is launched, we create private VM VXLAN network and add it as an 
unmanaged NIC to VM. CloudStack knows nothing about it. User gets it as is. Now 
we use hotplug KVM feature for it, but better to change XML and launch VM with 
such NIC defined.
   


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


With regards,
Apache Git Services


[GitHub] [cloudstack] borisstoyanov commented on issue #3606: [WIP DO NOT MERGE] VM ingestion

2020-01-21 Thread GitBox
borisstoyanov commented on issue #3606: [WIP DO NOT MERGE] VM ingestion
URL: https://github.com/apache/cloudstack/pull/3606#issuecomment-576589268
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3606: [WIP DO NOT MERGE] VM ingestion

2020-01-21 Thread GitBox
blueorangutan commented on issue #3606: [WIP DO NOT MERGE] VM ingestion
URL: https://github.com/apache/cloudstack/pull/3606#issuecomment-576589634
 
 
   @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 to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[cloudstack-primate] branch master updated (3e2c584 -> 2a8e134)

2020-01-21 Thread rohit
This is an automated email from the ASF dual-hosted git repository.

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


from 3e2c584  network: Egress, PF, FW, VPN, LB tabs (#84)
 add 2a8e134  iam: Add LDAP Account Form (#116)

No new revisions were added by this update.

Summary of changes:
 src/components/view/ActionButton.vue |   4 +-
 src/components/widgets/OsLogo.vue|   2 +-
 src/config/section/iam.js|  11 +
 src/config/section/image.js  |   2 +-
 src/config/section/project.js|   2 +-
 src/store/getters.js |   3 +-
 src/store/modules/user.js|  13 +-
 src/views/iam/AddLdapAccount.vue | 412 +++
 8 files changed, 442 insertions(+), 7 deletions(-)
 create mode 100644 src/views/iam/AddLdapAccount.vue



[cloudstack-primate] branch master updated (2a8e134 -> fce6827)

2020-01-21 Thread rohit
This is an automated email from the ASF dual-hosted git repository.

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


from 2a8e134  iam: Add LDAP Account Form (#116)
 add fce6827  docs: update apis remaining list

No new revisions were added by this update.

Summary of changes:
 docs/api/apis.remaining | 3 ---
 1 file changed, 3 deletions(-)



[GitHub] [cloudstack] poussa commented on issue #3680: [WIP: DO NOT MERGE] CloudStack Kubernetes Service

2020-01-21 Thread GitBox
poussa commented on issue #3680: [WIP: DO NOT MERGE] CloudStack Kubernetes 
Service
URL: https://github.com/apache/cloudstack/pull/3680#issuecomment-576609242
 
 
   @shwstppr Are you planning of supporting other distros that coreos? Like 
ubuntu-18.04 and centos-7/8 ?
   
   Or is it already possible to use those distros with CKS? If so, can you 
point me to the docs? The cloud-init / Ignition in /conf/k8s-master.yml seems 
to be only for CoreOS.
   
   BTW, I switched to Advanced networking mode and I am able to create multiple 
clusters now.


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


With regards,
Apache Git Services


[cloudstack-primate] branch master updated (fce6827 -> 17f1778)

2020-01-21 Thread rohit
This is an automated email from the ASF dual-hosted git repository.

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


from fce6827  docs: update apis remaining list
 add 17f1778  config: update infra changes

No new revisions were added by this update.

Summary of changes:
 src/config/section/infra/hosts.js | 4 
 src/config/section/infra/zones.js | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)



[GitHub] [cloudstack] anuragaw commented on issue #3575: [WIP DO NOT MERGE] Health check feature for virtual router

2020-01-21 Thread GitBox
anuragaw commented on issue #3575: [WIP DO NOT MERGE] Health check feature for 
virtual router
URL: https://github.com/apache/cloudstack/pull/3575#issuecomment-576614865
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3575: [WIP DO NOT MERGE] Health check feature for virtual router

2020-01-21 Thread GitBox
blueorangutan commented on issue #3575: [WIP DO NOT MERGE] Health check feature 
for virtual router
URL: https://github.com/apache/cloudstack/pull/3575#issuecomment-576614962
 
 
   @anuragaw a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3575: [WIP DO NOT MERGE] Health check feature for virtual router

2020-01-21 Thread GitBox
blueorangutan commented on issue #3575: [WIP DO NOT MERGE] Health check feature 
for virtual router
URL: https://github.com/apache/cloudstack/pull/3575#issuecomment-576623588
 
 
   Packaging result: ✖centos6 ✔centos7 ✔debian. JID-635


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


With regards,
Apache Git Services


[GitHub] [cloudstack] shwstppr commented on issue #3680: [WIP: DO NOT MERGE] CloudStack Kubernetes Service

2020-01-21 Thread GitBox
shwstppr commented on issue #3680: [WIP: DO NOT MERGE] CloudStack Kubernetes 
Service
URL: https://github.com/apache/cloudstack/pull/3680#issuecomment-576626604
 
 
   @poussa Currently, the aim is to get a stable plugin in 4.14 with CoreOS 
support only. Therefore, much of the code is for CoreOS specific deployment.
   Support for a Debian based template is definitely in consideration for the 
next iteration.


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


With regards,
Apache Git Services


[cloudstack-primate] branch master updated: network: Static NAT enable form (#119)

2020-01-21 Thread rohit
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new 0cb23ed  network: Static NAT enable form (#119)
0cb23ed is described below

commit 0cb23ed5a8c1c9c517d2136d5bf12890c529fd8b
Author: Ritchie Vincent 
AuthorDate: Tue Jan 21 11:04:25 2020 +

network: Static NAT enable form (#119)

Add static NAT form for public IP
---
 src/components/view/ResourceView.vue  |   1 +
 src/config/section/network.js |   8 +-
 src/views/network/EnableStaticNat.vue | 277 ++
 3 files changed, 280 insertions(+), 6 deletions(-)

diff --git a/src/components/view/ResourceView.vue 
b/src/components/view/ResourceView.vue
index a1c4bf7..2dc572e 100644
--- a/src/components/view/ResourceView.vue
+++ b/src/components/view/ResourceView.vue
@@ -110,6 +110,7 @@ export default {
 },
 showHideTab (tab) {
   if ('networkServiceFilter' in tab) {
+if (this.resource.virtualmachineid && tab.name !== 'Firewall') return 
false
 return this.networkService && this.networkService.service &&
   tab.networkServiceFilter(this.networkService.service)
   } else if ('show' in tab) {
diff --git a/src/config/section/network.js b/src/config/section/network.js
index a2a429a..e1f57cc 100644
--- a/src/config/section/network.js
+++ b/src/config/section/network.js
@@ -260,12 +260,8 @@ export default {
   label: 'Enable Static NAT',
   dataView: true,
   show: (record) => { return !record.virtualmachineid && 
!record.issourcenat },
-  args: ['ipaddressid', 'virtualmachineid', 'vmguestip'],
-  mapping: {
-ipaddressid: {
-  value: (record) => { return record.id }
-}
-  }
+  popup: true,
+  component: () => import('@/views/network/EnableStaticNat.vue')
 },
 {
   api: 'disableStaticNat',
diff --git a/src/views/network/EnableStaticNat.vue 
b/src/views/network/EnableStaticNat.vue
new file mode 100644
index 000..30cbb62
--- /dev/null
+++ b/src/views/network/EnableStaticNat.vue
@@ -0,0 +1,277 @@
+// 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.
+
+
+  
+
+  
+
+
+
+
+  
+
+  
+{{ $t('name') }}
+{{ vm.name }}
+  
+  
+{{ $t('instancename') }}
+{{ vm.instancename }}
+  
+  
+{{ $t('displayname') }}
+{{ vm.displayname }}
+  
+  
+{{ $t('account') }}
+{{ vm.account }}
+  
+  
+{{ $t('zonenamelabel') }}
+{{ vm.zonename }}
+  
+  
+{{ $t('state') }}
+{{ vm.state }}
+  
+  
+
+
+
+  
+{{ item.ipaddress }}
+  
+
+  
+
+
+
+
+  {{ $t('cancel') }}
+  {{ $t('ok') }}
+
+
+  
+
+
+
+import { api } from '@/api'
+
+export default {
+  props: {
+resource: {
+  type: Object,
+  required: true
+}
+  },
+  inject: ['parentFetchData'],
+  data () {
+return {
+  loading: false,
+  vmsList: [],
+  selectedVm: null,
+  nicsList: [],
+  searchQuery: null,
+  selectedNic: null
+}
+  },
+  mounted () {
+this.fetchData()
+  },
+  methods: {
+fetchData () {
+  this.loading = true
+  api('listVirtualMachines', {
+page: 1,
+pageSize: 500,
+listAll: true,
+networkid: this.resource.associatednetworkid,
+account: this.resource.account,
+domainid: this.resource.domainid,
+keyword: this.searchQuery
+  }).then(response => {
+this.vmsList = response.listvirtualmachinesresponse.virtualmachine
+this.loading = false
+  }).catch(error => {
+this.$notification.error({
+  message: `Error ${error.response.status}`,
+  description: error.response.data.errorresponse.errortext
+})
+this.loading = false
+  })
+},
+

[cloudstack-primate] branch master updated: docs: update remaining apis list

2020-01-21 Thread rohit
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new 694c9c6  docs: update remaining apis list
694c9c6 is described below

commit 694c9c6651e578426b1d74e5cce3a5de905133a6
Author: Rohit Yadav 
AuthorDate: Tue Jan 21 16:39:35 2020 +0530

docs: update remaining apis list

Signed-off-by: Rohit Yadav 
---
 docs/api/apis.remaining | 10 --
 1 file changed, 10 deletions(-)

diff --git a/docs/api/apis.remaining b/docs/api/apis.remaining
index 55e6c1a..77a2f00 100644
--- a/docs/api/apis.remaining
+++ b/docs/api/apis.remaining
@@ -2,7 +2,6 @@ addNetworkServiceProvider
 addResourceDetail
 addTrafficType
 assignCertToLoadBalancer
-authorizeSamlSso
 configureInternalLoadBalancerElement
 configureVirtualRouterElement
 createLBHealthCheckPolicy
@@ -13,7 +12,6 @@ createNetworkACLList
 createPhysicalNetwork
 createPortableIpRange
 createPrivateGateway
-createSecondaryStagingStore
 createSnapshotFromVMSnapshot
 createStaticRoute
 createStorageNetworkIpRange
@@ -31,8 +29,6 @@ deleteNetworkServiceProvider
 deletePhysicalNetwork
 deletePortableIpRange
 deletePrivateGateway
-deleteProjectInvitation
-deleteSecondaryStagingStore
 deleteStaticRoute
 deleteStorageNetworkIpRange
 deleteVlanIpRange
@@ -40,16 +36,13 @@ deleteVpnConnection
 deleteVpnGateway
 findStoragePoolsForMigration
 listAffinityGroupTypes
-listAndSwitchSamlAccount
 listDedicatedClusters
 listDedicatedGuestVlanRanges
 listDedicatedHosts
 listDedicatedPods
 listDedicatedZones
 listDeploymentPlanners
-listHostHAProviders
 listHostTags
-listIdps
 listInternalLoadBalancerElements
 listInternalLoadBalancerVMs
 listLBHealthCheckPolicies
@@ -59,14 +52,11 @@ listNetworkServiceProviders
 listOsCategories
 listPortableIpRanges
 listRegisteredServicePackages
-listSamlAuthorization
-listSecondaryStagingStores
 listStaticRoutes
 listStorageNetworkIpRange
 listStorageProviders
 listStorageTags
 listSupportedNetworkServices
-listTemplateOvfProperties
 listTrafficTypes
 listVirtualRouterElements
 listVlanIpRanges



[GitHub] [cloudstack] Pearl1594 opened a new pull request #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
Pearl1594 opened a new pull request #3825: fixed inconsistency of IP on VR when 
VR is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825
 
 
   ## Description
   This address the issue mentioned in #3787  - Guest IP address of Shared 
Network VR is not persistent when VR is destroyed/recreated
   
   ## Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [x] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## How Has This Been Tested?
   - Created a shared network
   - Created a VM in the shared network
   - destroyed the Virtual router 
   - stopped and restarted the instance 
   - the VR created will have the same Guest IP as it had previously
   
   


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


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
rhtyd commented on issue #3825: fixed inconsistency of IP on VR when VR is 
destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576639349
 
 
   Regression testing and manual QA against VPC, shared and isolated network 
with/without redundant router. Also we need to check if L2 is affected (likely 
not, as there are no routers).


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3680: [WIP: DO NOT MERGE] CloudStack Kubernetes Service

2020-01-21 Thread GitBox
blueorangutan commented on issue #3680: [WIP: DO NOT MERGE] CloudStack 
Kubernetes Service
URL: https://github.com/apache/cloudstack/pull/3680#issuecomment-576643145
 
 
   Packaging result: ✖centos6 ✔centos7 ✔debian. JID-636


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


With regards,
Apache Git Services


[GitHub] [cloudstack] nvazquez commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance

2020-01-21 Thread GitBox
nvazquez commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance
URL: https://github.com/apache/cloudstack/pull/3610#issuecomment-576669693
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance

2020-01-21 Thread GitBox
blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling 
maintenance
URL: https://github.com/apache/cloudstack/pull/3610#issuecomment-576670025
 
 
   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance

2020-01-21 Thread GitBox
blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling 
maintenance
URL: https://github.com/apache/cloudstack/pull/3610#issuecomment-576677875
 
 
   Packaging result: ✖centos6 ✔centos7 ✔debian. JID-637


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


With regards,
Apache Git Services


[GitHub] [cloudstack] ravening commented on issue #3807: Enhancement: Allow creating network with duplicate name

2020-01-21 Thread GitBox
ravening commented on issue #3807: Enhancement: Allow creating network with 
duplicate name
URL: https://github.com/apache/cloudstack/pull/3807#issuecomment-576683272
 
 
   @DaanHoogland can this be merged?


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


With regards,
Apache Git Services


[GitHub] [cloudstack] svenvogel commented on issue #3780: Enhancement: Allow creating atmost 1 physical network with null tag

2020-01-21 Thread GitBox
svenvogel commented on issue #3780: Enhancement: Allow creating atmost 1 
physical network with null tag
URL: https://github.com/apache/cloudstack/pull/3780#issuecomment-576691676
 
 
   @weizhouapache @andrijapanicsb @ravening @DaanHoogland some news here? how 
we can proceed here?


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DennisKonrad commented on issue #3296: [master] noVNC Console Integration

2020-01-21 Thread GitBox
DennisKonrad commented on issue #3296: [master] noVNC Console Integration
URL: https://github.com/apache/cloudstack/pull/3296#issuecomment-576696540
 
 
   @ustcweizhou I'm tying to get this running in a test environment. Can you 
give me some indications on what maybe wrong. I'm using [master with this PR 
merged ](https://github.com/DennisKonrad/cloudstack/tree/novnc).
   
   It seems to me after or during the upgrade to a websocket connection 
something fails with the novnc console.
   
   
 Screenshots
   
   
![image](https://user-images.githubusercontent.com/33484272/72810765-84279380-3c5e-11ea-9e61-1d16d5fb4e32.png)

   
![image](https://user-images.githubusercontent.com/33484272/72810791-96a1cd00-3c5e-11ea-8924-57d55d68db01.png)
   
   The old cs console works fine when I set `novnc.console.default` to `false` 
and reboot the console system vm.
   
![image](https://user-images.githubusercontent.com/33484272/72810985-00ba7200-3c5f-11ea-8b2d-76ff0a820aaf.png)
   


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DennisKonrad edited a comment on issue #3296: [master] noVNC Console Integration

2020-01-21 Thread GitBox
DennisKonrad edited a comment on issue #3296: [master] noVNC Console Integration
URL: https://github.com/apache/cloudstack/pull/3296#issuecomment-576696540
 
 
   @ustcweizhou I'm tying to get this running in a test environment. Can you 
give me some indications on what maybe wrong. I'm using [master with this PR 
merged ](https://github.com/DennisKonrad/cloudstack/tree/novnc).
   
   It seems to me after or during the upgrade to a websocket connection 
something fails with the novnc console. 
   
   (After reading the older comments again: No certificate uploaded in this 
environment and guest.domain.suffix is `test.kvm` )
   
   
 Screenshots
   
   
![image](https://user-images.githubusercontent.com/33484272/72810765-84279380-3c5e-11ea-9e61-1d16d5fb4e32.png)

   
![image](https://user-images.githubusercontent.com/33484272/72810791-96a1cd00-3c5e-11ea-8924-57d55d68db01.png)
   
   The old cs console works fine when I set `novnc.console.default` to `false` 
and reboot the console system vm.
   
![image](https://user-images.githubusercontent.com/33484272/72810985-00ba7200-3c5f-11ea-8b2d-76ff0a820aaf.png)
   


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3179: Redundant VRouter guest network on wrong interface

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3179: Redundant VRouter guest network on wrong 
interface
URL: https://github.com/apache/cloudstack/issues/3179#issuecomment-576724793
 
 
   @DennisKonrad i read in your text above we can close this?


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan commented on issue #3758: server: Fix NPE while update displayvm 
on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-576731060
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570614494
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3759: server: fix resource count error when upgrade a vm

2020-01-21 Thread GitBox
blueorangutan commented on issue #3759: server: fix resource count error when 
upgrade a vm
URL: https://github.com/apache/cloudstack/pull/3759#issuecomment-576730571
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570598783
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3758: server: Fix NPE while update displayvm 
on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-576730673
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570919070
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR 
is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576729144
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570598725
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-576731910
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3759: server: fix resource count error when upgrade a vm

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3759: server: fix resource count error when 
upgrade a vm
URL: https://github.com/apache/cloudstack/pull/3759#issuecomment-576730213
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570814219
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] Pearl1594 commented on a change in pull request #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
Pearl1594 commented on a change in pull request #3825: fixed inconsistency of 
IP on VR when VR is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#discussion_r369064492
 
 

 ##
 File path: server/src/main/java/com/cloud/network/NetworkModelImpl.java
 ##
 @@ -2299,6 +2299,9 @@ public NicVO getPlaceholderNicForRouter(Network network, 
Long podId) {
 ipv6 = _ipv6Dao.findByNetworkIdAndIp(network.getId(), 
nic.getIPv6Address());
 }
 //return nic only when its ip address belong to the pod 
range (for the Basic zone case)
+if (vlans.isEmpty()) {
+return nic;
+}
 
 Review comment:
   @DaanHoogland I've moved the comment


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570814185
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3825: fixed inconsistency of IP on VR when VR 
is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576729104
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570614762
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570612248
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570609761
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-526


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
DaanHoogland commented on a change in pull request #3825: fixed inconsistency 
of IP on VR when VR is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#discussion_r369060822
 
 

 ##
 File path: server/src/main/java/com/cloud/network/NetworkModelImpl.java
 ##
 @@ -2299,6 +2299,9 @@ public NicVO getPlaceholderNicForRouter(Network network, 
Long podId) {
 ipv6 = _ipv6Dao.findByNetworkIdAndIp(network.getId(), 
nic.getIPv6Address());
 }
 //return nic only when its ip address belong to the pod 
range (for the Basic zone case)
+if (vlans.isEmpty()) {
+return nic;
+}
 
 Review comment:
   I don't think this comment still describves the line below. Can you clarify 
or remove it?


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3758: server: Fix NPE while update displayvm on vm with dynamic service offering

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3758: server: Fix NPE while update 
displayvm on vm with dynamic service offering
URL: https://github.com/apache/cloudstack/pull/3758#issuecomment-570612557
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-565185588
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
blueorangutan commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-576732053
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-565201402
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-460


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570612548
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570611093
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-527


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570612529
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570601106
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd removed a comment on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
rhtyd removed a comment on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-565185400
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
rhtyd removed a comment on issue #3726: vrouter: reload haproxy when cfg file 
is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-562915846
 
 
   @blueorangutan package 


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559979305
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559806567
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DennisKonrad commented on issue #3179: Redundant VRouter guest network on wrong interface

2020-01-21 Thread GitBox
DennisKonrad commented on issue #3179: Redundant VRouter guest network on wrong 
interface
URL: https://github.com/apache/cloudstack/issues/3179#issuecomment-576733004
 
 
   @DaanHoogland I would like to wait for the PR of wei to be tested and 
merged. If the community accepts it then we can close the issue.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR 
is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576735090
 
 
   Packaging result: ✖centos6 ✖centos7 ✖debian. JID-638


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3727: server: Capacity check should take vms 
in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-576733803
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3728: server: double check host capacity when 
start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-576733453
 
 
   @rhtyd @nvazquez @wido @GabrielBrascher is this alright?


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3477: RvR: Set up metadata/password/dhcp 
server on gateway IP instead of guest IP in RVR
URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-576736035
 
 
   ping @rhtyd @andrijapanicsb ??


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR 
is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576738007
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3726: vrouter: reload haproxy when cfg 
file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559979380
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3726: vrouter: reload haproxy when cfg 
file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559807859
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559985414
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-413


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559807909
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559824818
 
 
   Packaging result: ✖centos6 ✔centos7 ✖debian. JID-410


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3825: fixed inconsistency of IP on VR when VR 
is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576737846
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3726: vrouter: reload haproxy when cfg 
file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559996050
 
 
   @blueorangutan  test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559996124
 
 
   @blueorangutan  test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559806817
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3825: fixed inconsistency of IP on VR 
when VR is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576735090
 
 
   Packaging result: ✖centos6 ✖centos7 ✖debian. JID-638


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570814436
 
 
   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
DaanHoogland commented on issue #3726: vrouter: reload haproxy when cfg file is 
updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-576735321
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] rhtyd removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
rhtyd removed a comment on issue #3726: vrouter: reload haproxy when cfg file 
is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-562892405
 
 
   @blueorangutan package
   
   


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559979278
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3825: fixed inconsistency of IP on VR 
when VR is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576729144
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file 
is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-576735500
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-562917151
 
 
   Packaging result: ✖centos6 ✖centos7 ✔debian. JID-442


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559979461
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-562896707
 
 
   Packaging result: ✖centos6 ✖centos7 ✔debian. JID-439


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570814364
 
 
   @blueorangutan test


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3825: fixed inconsistency of IP on VR 
when VR is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576729104
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-562915864
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
blueorangutan commented on issue #3727: server: Capacity check should take vms 
in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-576734026
 
 
   @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3727: server: Capacity check should 
take vms in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559824817
 
 
   Packaging result: ✔centos6 ✔centos7 ✖debian. JID-409


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-562892444
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] DaanHoogland removed a comment on issue #3728: server: double check host capacity when start/migrate a vm

2020-01-21 Thread GitBox
DaanHoogland removed a comment on issue #3728: server: double check host 
capacity when start/migrate a vm
URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-570600863
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan removed a comment on issue #3726: vrouter: reload haproxy when 
cfg file is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559985413
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-414


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2020-01-21 Thread GitBox
blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file 
is updated
URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-576781262
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-642


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
blueorangutan commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-576781263
 
 
   Packaging result: ✖centos6 ✔centos7 ✔debian. JID-640


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3759: server: fix resource count error when upgrade a vm

2020-01-21 Thread GitBox
blueorangutan commented on issue #3759: server: fix resource count error when 
upgrade a vm
URL: https://github.com/apache/cloudstack/pull/3759#issuecomment-576781265
 
 
   Packaging result: ✖centos6 ✔centos7 ✔debian. JID-639


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2020-01-21 Thread GitBox
blueorangutan commented on issue #3727: server: Capacity check should take vms 
in Migrating state into calculation
URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-576781269
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-641


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


With regards,
Apache Git Services


[GitHub] [cloudstack] nvazquez commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance

2020-01-21 Thread GitBox
nvazquez commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance
URL: https://github.com/apache/cloudstack/pull/3610#issuecomment-576785216
 
 
   @blueorangutan package


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR is destroyed and recreated (shared network)

2020-01-21 Thread GitBox
blueorangutan commented on issue #3825: fixed inconsistency of IP on VR when VR 
is destroyed and recreated (shared network)
URL: https://github.com/apache/cloudstack/pull/3825#issuecomment-576784870
 
 
   Packaging result: ✖centos6 ✖centos7 ✖debian. JID-643


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance

2020-01-21 Thread GitBox
blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling 
maintenance
URL: https://github.com/apache/cloudstack/pull/3610#issuecomment-576785608
 
 
   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


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


With regards,
Apache Git Services


[GitHub] [cloudstack] onitake commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow takeVMSnapshot() exception

2020-01-21 Thread GitBox
onitake commented on issue #3761: [FIX] [BACKPORT] [4.13] Rethrow 
takeVMSnapshot() exception
URL: https://github.com/apache/cloudstack/pull/3761#issuecomment-576793458
 
 
   Are the build errors errors relevant?
   I see things like
   ```
   CloudstackAPIException: Execute cmd: listprojectaccounts failed, due to: 
errorCode: 431, errorText:Unable to find the project id=9
   ```
   which doesn't look related to me at all...


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


With regards,
Apache Git Services


[GitHub] [cloudstack] blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling maintenance

2020-01-21 Thread GitBox
blueorangutan commented on issue #3610: [WIP DO NOT MERGE] KVM: Rolling 
maintenance
URL: https://github.com/apache/cloudstack/pull/3610#issuecomment-576794007
 
 
   Packaging result: ✖centos6 ✔centos7 ✔debian. JID-644


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


With regards,
Apache Git Services


[GitHub] [cloudstack] Slair1 opened a new issue #3826: Unsafe Field Accessor with newer mysql-connector-java package

2020-01-21 Thread GitBox
Slair1 opened a new issue #3826: Unsafe Field Accessor with newer 
mysql-connector-java package
URL: https://github.com/apache/cloudstack/issues/3826
 
 
   
   
   # ISSUE TYPE
   
* Bug Report
   
   # COMPONENT NAME
   
   ~~~
   mysql-connector-java
   ~~~
   
   # CLOUDSTACK VERSION
   
   
   ~~~
   4.11.3
   all
   ~~~
   
   # CONFIGURATION
   
   
   
   # OS / ENVIRONMENT
   
   CentOS7
   
   # SUMMARY
   
   Looks like the new version (8.0.x) is stricter with conversions from Long to 
BigIntfor example.  Auto Increment fields in the database are of type BigInt, 
but in code we are using Long.  This generates the errors similar to the 
following at times:
   ~~~
   Can not set long field com.cloud.upgrade.dao.VersionVO.id to 
java.math.BigInteger
   ~~~
   # STEPS TO REPRODUCE
   
   Install any new version of mysql-connector-java
   
   
   ~~~
   We hit it while the database upgrade script was running 4.9.3.0 -> 4.10.0
   ~~~
   
   
   
   # EXPECTED RESULTS
   
   
   ~~~
   Looks like older versions of mysql-connector-java did not care that we use 
Long in code by BigInt in the database columns
   ~~~
   
   # ACTUAL RESULTS
   
   We get errors such as this:
   
   ~~~
   Caused by: java.lang.IllegalArgumentException: Can not set long field 
com.cloud.upgrade.dao.VersionVO.id to java.math.BigInteger
   at 
sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(UnsafeFieldAccessorImpl.java:167)
   at 
sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(UnsafeFieldAccessorImpl.java:171)
   at 
sun.reflect.UnsafeLongFieldAccessorImpl.set(UnsafeLongFieldAccessorImpl.java:102)
   ~~~
   


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


With regards,
Apache Git Services


[GitHub] [cloudstack] Slair1 commented on issue #3826: Unsafe Field Accessor with newer mysql-connector-java package

2020-01-21 Thread GitBox
Slair1 commented on issue #3826: Unsafe Field Accessor with newer 
mysql-connector-java package
URL: https://github.com/apache/cloudstack/issues/3826#issuecomment-576815858
 
 
   Not sure if the correct fix is to update the code to use something more 
compatible with BigInt or not...  But here is a band-aid, here is what our 
`mysql-community-connectors.repo` looks like now:
   
   ```
   [mysql-community-connectors]
   name=MySQL Community Connectors
   
baseurl=http://repo.mysql.com/yum/mysql-connectors-community/el/$releasever/$basearch/
   enabled=1
   gpgcheck=1
   exclude=mysql-connector-java*
   ```


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


With regards,
Apache Git Services


[GitHub] [cloudstack] andrijapanicsb commented on issue #3826: Unsafe Field Accessor with newer mysql-connector-java package

2020-01-21 Thread GitBox
andrijapanicsb commented on issue #3826: Unsafe Field Accessor with newer 
mysql-connector-java package
URL: https://github.com/apache/cloudstack/issues/3826#issuecomment-576820330
 
 
   The alternative (what we are using in Trillian) is to use 
mysql-connector-java from the Base CentOS 7 repo (still version 5.x) and 
mysql-connector-python from the Epel repo (an ancient version 1.1.6 verrsion) - 
i.e. forget/remove the repo.mysql.com repo.
   
   


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


With regards,
Apache Git Services


  1   2   >