Re: Handling of DB migrations on forks

Something like Liquibase would be nice. Hibernate might be even better. Even idempotent migrations would be a step in the right direction. Perhaps reject any migration changes that aren't INSERT IGNORE, DROP IF EXISTS, etc? I'm not sure why the DAO was originally hand-rolled. Perhaps the original

[GitHub] cloudstack pull request #1771: CLOUDSTACK-9611: Dedicating a Guest VLAN rang...

Github user nitin-maharana commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1771#discussion_r102173991 --- Diff: server/src/com/cloud/network/NetworkServiceImpl.java --- @@ -3085,9 +3085,10 @@ public GuestVlan

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user HrWiggles commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102187534 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm) {

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user HrWiggles commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102177792 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm) {

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user HrWiggles commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102181285 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm) {

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user HrWiggles commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102179249 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm) {

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user HrWiggles commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102178630 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm) {

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user HrWiggles commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102184780 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm) {

Re: Handling of DB migrations on forks

Good strategy and I would make that not a warning but a fatal, as the resulting ACS version will probably not work. On Tue, Feb 14, 2017 at 12:16 PM, Wei ZHOU wrote: > Then you have to create your own branch forked from 4.10.0 > > In our branch, I moved some table changes

RE: [PROPOSAL] add native vm-cluster orchestration service (was: [PROPOSAL] add native container orchestration service)

Sure Daan. I'll publish the design on cwiki and share the link. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@shapeblue.com] Sent: Monday, February 20, 2017 7:27 PM To: dev@cloudstack.apache.org Subject: [PROPOSAL] add native vm-cluster orchestration service (was:

Re: [DISCUSS][FS] Host HA for CloudStack

See inline. Thanks, Koushik On 21/02/17, 11:47 AM, "Rohit Yadav" wrote: Hi Koushik, Thanks for sharing your comments and questions. 1. Yes, the FS is divided into two parts - a general HA framework which makes no assumption about the type

[GitHub] cloudstack issue #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM...

Github user ramkatru commented on the issue: https://github.com/apache/cloudstack/pull/1897 @sureshanaparti, please look into these failures. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request #1881: CLOUDSTACK-9721: Remove deprecated/unused glo...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1881 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

RE: Handling of DB migrations on forks

Marc-Aurele, you are totally right and people agree with you but no one seems to give this priority daan.hoogl...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, Utrecht Utrecht 3531 VENetherlands @shapeblue -Original Message- From: Marc-Aurèle Brothier

[GitHub] cloudstack issue #859: Bug-ID: CLOUDSTACK-8882: calculate network offering u...

Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/859 ### ACS CI BVT Run **Sumarry:** Build Number 369 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack pull request #1824: CLOUDSTACK-9657: Fixed security group ipset i...

Github user jayapalu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1824#discussion_r102192868 --- Diff: scripts/vm/hypervisor/xenserver/vmops --- @@ -232,28 +233,50 @@ def deleteFile(session, args): return txt +#using

[GitHub] cloudstack issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1861 @borisstoyanov, thanks for running tets. I see 1 test error in the above results, this test has been failing in many other PRs as well, and doesn't seem related to changes

Re: Handling of DB migrations on forks

IMO the database changes should be idempotent as much as possible with "CREATE OR REPLACE VIEW..." "DROP IF EXISTS". For other things like altering a table, it's more complicated to achieve that in pure SQL. A good call would be to integrate http://www.liquibase.org/ to manage the schema and

[GitHub] cloudstack pull request #1882: CLOUDSTACK-8737: Removed the missed out-of-ba...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1882 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack issue #859: Bug-ID: CLOUDSTACK-8882: calculate network offering u...

Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/859 @kishankavala i forgot about this one, guess we won't make 4.7 ;) can you appease cloudmonger? --- If your project is set up for it, you can reply to this email and have your reply

Re: [QUESTION] Upgrade path to JDK8

http://stackoverflow.com/questions/10895969/can-newer-jre-versions-run-java-programs-compiled-with-older-jdk-versions You can run code compiled by the java 1.7 or 1.6 or earlier SDKs on a Java 8 JVM. This gets you the improved speed of the Java 8 JVM even if you do not rebuild the code. If

[GitHub] cloudstack issue #1768: CLOUDSTACK 9601: Upgrade: change logic for update pa...

Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1768 @rhtyd Changing the sequence is only idempotent if you have been upgrading to each new versions step by step. If you jumped other versions, then the path is different for each original

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1955 We are deploying this to our QA environment right now and hope to have it tested in a few days. Great work @kiwiflyer and @nathanejohnson. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request #1957: CLOUDSTACK-9748:VPN Users search functionalit...

Github user Ashadeepa commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1957#discussion_r10411 --- Diff: server/src/com/cloud/network/vpn/RemoteAccessVpnManagerImpl.java --- @@ -621,6 +627,10 @@ public void

[GitHub] cloudstack pull request #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatchi...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1878#discussion_r102226833 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -2071,6 +2120,14 @@ protected

[GitHub] cloudstack pull request #1957: CLOUDSTACK-9748:VPN Users search functionalit...

Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1957#discussion_r102209429 --- Diff: server/src/com/cloud/network/vpn/RemoteAccessVpnManagerImpl.java --- @@ -621,6 +627,10 @@ public void

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1773 @priyankparihar I agree with @ustcweizhou regarding the default value of `forced` in terms of backwards compatibility. Also, why we permit deletion of a template when it is associated

[GitHub] cloudstack pull request #1957: CLOUDSTACK-9748:VPN Users search functionalit...

GitHub user Ashadeepa opened a pull request: https://github.com/apache/cloudstack/pull/1957 CLOUDSTACK-9748:VPN Users search functionality broken VPN Users search functionality broken If you try to search VPN users with it’s user name, you will not be able to search.

[GitHub] cloudstack issue #1910: CLOUDSTACK-9748:VPN Users search functionality broke...

Github user Ashadeepa commented on the issue: https://github.com/apache/cloudstack/pull/1910 @ustcweizhou : Thanks. I have made the changes. New PR : https://github.com/apache/cloudstack/pull/1957. --- If your project is set up for it, you can reply to this email and have

Re: apidocs build failure

Is there any chance we can fix the 'roles' issue with the API doc so we can get the docs split into the 'Admin', 'Domain Admin' and 'User' again? The introduction of the dynamic roles broke the generation of the API docs with the different roles and I was not able to figure out how to fix it.

Re: Handling of DB migrations on forks

I'm also in favor of Liquibase. Hibernate might be smart enough to add new columns, but to be able to for example change the type of a column, more complex data migration might be necessary. Liquibase preconditions, combined with good identifiers make up that upgrades will be more granular.

Re: Attaching more than 14 data volumes to an instance

On 18 February 2017 at 20:51:42, Suresh Anaparti ( suresh.anapa...@accelerite.com) wrote: I checked the limits set for VMware hypervisor and observed some discrepancies. These can be either updated from the updateHypervisorCapabilities API (max_data_volumes_limit, max_hosts_per_cluster after

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut I agree with @ustcweizhou. Please remove delaycompress and up to 10. I'd like to get this PR in as it is the second part of the problem resolution for my issue. After that LGTM.

Re: Adding VirtIO SCSI to KVM hypervisors

Actually, minor correction. When adding to VM/templates the name of the detail is rootDiskController for Root controller and dataDiskController for additional disks. Also, if you want to make changes on a global scale the changes need to go to vm_template_details and user_vm_details tables

Re: Adding VirtIO SCSI to KVM hypervisors

Sergey Levitskiy wrote: > Here it is the logic. > 1. Default value is taken from a global configuration > vmware.root.disk.controller > 2. To override add the same config to template or VM (starting from 4.10 > UI allows adding advanced settings to templates

Re: Adding VirtIO SCSI to KVM hypervisors

Wido den Hollander wrote: > >> Op 25 januari 2017 om 4:44 schreef Simon Weller : >> >> >> Maybe this is a good opportunity to discuss modernizing the OS >> selections so that drivers (and other features) could be selectable per >> OS. > > That seems like a

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1873 There are not failed test cases on CI run. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

Re: Adding VirtIO SCSI to KVM hypervisors

Sergey Levitskiy wrote: > On vmware rootdiskcontroller is passed over to the hypervisor in VM start > command. I know for the fact that the following rootdiskcontroller option > specified in template/vm details work fine: > ide > scsi > lsilogic > lsilogic1068

Re: Adding VirtIO SCSI to KVM hypervisors

Here it is the logic. 1. Default value is taken from a global configuration vmware.root.disk.controller 2. To override add the same config to template or VM (starting from 4.10 UI allows adding advanced settings to templates and/or VMs). If added to a template all VMs deployed from it

Re: Adding VirtIO SCSI to KVM hypervisors

Sergey Levitskiy wrote: > Actually, minor correction. When adding to VM/templates the name of the > detail is rootDiskController for Root controller and dataDiskController > for additional disks. > Also, if you want to make changes on a global scale the changes

[GitHub] cloudstack issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1953 Trillian test result (tid-877) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 26001 seconds Marvin logs:

Re: Adding VirtIO SCSI to KVM hypervisors

On vmware rootdiskcontroller is passed over to the hypervisor in VM start command. I know for the fact that the following rootdiskcontroller option specified in template/vm details work fine: ide scsi lsilogic lsilogic1068 In general, any scsi controller option that vmware recognizes should

[GitHub] cloudstack issue #1768: CLOUDSTACK 9601: Upgrade: change logic for update pa...

Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1768 @serg38 Not sure to understand what you mean with: > If I get it right with your proposed changes, upgrade scripts become obsolete since all the changes can be done in upgrade scripts.

[GitHub] cloudstack pull request #1959: CLOUDSTACK-9786:API reference guide entry for...

GitHub user Ashadeepa opened a pull request: https://github.com/apache/cloudstack/pull/1959 CLOUDSTACK-9786:API reference guide entry for associateIpAddress needs additional information Going through the code & implementation, it seems like either of the parameters are not

[GitHub] cloudstack issue #1957: CLOUDSTACK-9748:VPN Users search functionality broke...

Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1957 tested. LGTM btw, you can use "git push --force" to overwrite the code --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: apidocs build failure

Will, any specific use-case? For a live running cloudstack an apidocs plugin could also be written that displays apidocs for a role/user-account consuming from listApis. Regards. From: williamstev...@gmail.com on behalf of Will

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1935 @rafaelweingartner I think I got your point, I tried to keep code as similar as it was before, by declaring `rollBackState` as static class variable on line 114. This way inner `finally` block

[GitHub] cloudstack pull request #1910: CLOUDSTACK-9748:VPN Users search functionalit...

Github user Ashadeepa closed the pull request at: https://github.com/apache/cloudstack/pull/1910 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack issue #1957: CLOUDSTACK-9748:VPN Users search functionality broke...

Github user Ashadeepa commented on the issue: https://github.com/apache/cloudstack/pull/1957 @rafaelweingartner : This is due to the change in my remote urls. Closing the old PR https://github.com/apache/cloudstack/issues/1910. --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 Trillian test result (tid-876) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 35807 seconds Marvin logs:

[GitHub] cloudstack issue #1768: CLOUDSTACK 9601: Upgrade: change logic for update pa...

Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1768 I'll try to make my point clearer with a better use case. Let say you were running version ACS 4.4.2 and wish to upgrade to 4.7.1. After installing the 4.7.1, when ACS starts for the first

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user sureshanaparti commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102301645 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm)

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

Github user serbaut commented on the issue: https://github.com/apache/cloudstack/pull/1915 Is it safe to remove delaycompress across the board, I assume it is there for a reason? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

Github user serbaut commented on the issue: https://github.com/apache/cloudstack/pull/1915 Ok. I removed it from rsyslog since it should be safe there. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-520 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1773 Trillian test result (tid-875) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27492 seconds Marvin logs:

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user nathanejohnson commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102309146 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean checkWorkItems(final

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102309517 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user nathanejohnson commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102310032 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean checkWorkItems(final

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user sureshanaparti commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102309898 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO vm)

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user nathanejohnson commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102305779 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean checkWorkItems(final

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102308166 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean

[GitHub] cloudstack issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1879 Trillian test result (tid-869) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46839 seconds Marvin logs:

[GitHub] cloudstack issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1953 @remibergsma @borisstoyanov Updated the KVM code to generate the valid device name above id 25. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user sureshanaparti commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102311697 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -716,11 +734,6 @@ public DiskFmtType

[GitHub] cloudstack issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1953 @borisstoyanov Can you kick off Jenkins job on this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102310842 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean

[GitHub] cloudstack issue #1875: CLOUDSTACK-8608: [VMware] System VMs failed to start...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1875 Trillian test result (tid-867) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 48400 seconds Marvin logs:

[GitHub] cloudstack pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r102265150 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,79 +274,97 @@ public boolean deleteDomain(long domainId, Boolean cleanup) {

[GitHub] cloudstack pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r102265306 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,79 +274,97 @@ public boolean deleteDomain(long domainId, Boolean cleanup) {

[GitHub] cloudstack pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r102264841 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,79 +274,97 @@ public boolean deleteDomain(long domainId, Boolean cleanup) {

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 Trillian test result (tid-873) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33766 seconds Marvin logs:

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1878 @sureshanaparti why do the mac addresses need to be the same on both routers? We're also executing arpings to update our neighbours. Networking wise there is no need for them to be the

[GitHub] cloudstack issue #1957: CLOUDSTACK-9748:VPN Users search functionality broke...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1957 @Ashadeepa why do we have 2 PRs for the same problem? It seems that one of them can be closed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102289931 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user nathanejohnson commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102290456 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean checkWorkItems(final

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102291066 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102292787 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean

[GitHub] cloudstack pull request #1953: CLOUDSTACK-9794: Unable to attach more than 1...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1953#discussion_r102294407 --- Diff: server/src/com/cloud/storage/VolumeApiServiceImpl.java --- @@ -2639,22 +2639,23 @@ private int getMaxDataVolumesSupported(UserVmVO

[GitHub] cloudstack pull request #1956: CLOUDSTACK-9796 - Fix NPE in VirtualMachineMa...

Github user nathanejohnson commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1956#discussion_r102295771 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -744,14 +744,17 @@ protected boolean checkWorkItems(final

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1935 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1861: CLOUDSTACK-9698 [VMware] Make hardcorded wait timeou...

Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1861 yes @sateesh-chodapuneedi this failure has been a pain for a while... it'll be good to invest some time in fixing it.. --- If your project is set up for it, you can reply to this email

RE: apidocs build failure

@Rajani @Rohit I missed this mail and fixed the apidoc on build.a.o yesterday. I can disable it or throw it away might we so wish daan.hoogl...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, Utrecht Utrecht 3531 VENetherlands @shapeblue -Original Message- From:

Re: [QUESTION] Upgrade path to JDK8

No there isn't any issue except having the bugs & fixes of the JDK you're using. You can compile it with a JDK 1.8 as long as you don't change the target bytecode version for 1.7 On Tue, Feb 21, 2017 at 8:15 AM, Wei ZHOU wrote: > Marco, > > Good point. Is there any issue

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 Trillian test result (tid-864) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 43562 seconds Marvin logs:

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 Very nice indeed! I will take a look asap. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

Github user priyankparihar commented on the issue: https://github.com/apache/cloudstack/pull/1773 Hi @koushik-das @rajesh-battala , >The default value of forced is false, might cause issue on backwards compatibility. Should i make changes according to @serg38 and

[GitHub] cloudstack issue #1872: CLOUDSTACK-3223 Exception observed while creating CP...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1872 Trillian test result (tid-860) Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45192 seconds Marvin logs:

[GitHub] cloudstack issue #1779: CLOUDSTACK-9610: Disabled Host Keeps Being up status...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1779 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-517 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack issue #669: Made the adding new keyboard language support easier

Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/669 Added the missing license on one js file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-519 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack issue #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1949 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-518 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack issue #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9...

Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1949 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1878 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1878 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9...

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1949 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1935 @nvazquez great work. However, there is a catch there that I think you might have overlooked. This problem is caused by the method extraction I suggested. If you take a

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut Can you do a force push to kick off jenkins again. I'm guessing Jenkins just had an issue and not the PR. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1878 @remibergsma Same MAC for RVR has been re-introducted as part of [CLOUDSTACK-985](https://issues.apache.org/jira/browse/CLOUDSTACK-985). It confirms that peer NICs of RVRs should have

  1   2   >