Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Thanks for pointing this out. I can incorporate those changes in the morning and see if it does the trick. > On Feb 15, 2017, at 12:50 AM, Rene Moser wrote: > > Probably related to https://github.com/apache/cloudstack/pull/1927 > > > >> On 02/14/2017 10:44 PM, Tutkowski,

Re: Can't create a zone using master

2017-02-14 Thread Rene Moser
Probably related to https://github.com/apache/cloudstack/pull/1927 On 02/14/2017 10:44 PM, Tutkowski, Mike wrote: > Hi, > > I’m getting a NullPointerException when trying to create a zone using master. > > Below is the relevant code in ConfigurationManagerImpl. > > In the else block,

[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/672 @rhtyd @karuturi Should I close this PR then? --- 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 issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/672 @rhtyd @karuturi @syed Raised #1941 against 4.9. --- 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 #1941: CLOUDSTACK-8663: Fixed various issues to allo...

2017-02-14 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/1941 CLOUDSTACK-8663: Fixed various issues to allow VM snapshots and volume snapshots to exist together Reverting VM to disk only snapshot in Xenserver corrupts VM Stale NFS

[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/672 Thanks @anshul1886 kindly also squash your changes and fix the commit message/jira id etc --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/672 Resolving conflicts and rebasing the PR against 4.9 --- 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

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 @rhtyd 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 reply

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1836 @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 wishes

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-486 --- 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 #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/672 @anshul1886 I would suggest, if your PR already has @syed 's fix along with other useful fixes -- please rebase against 4.9, fix the conflicts and we can help review/test your PR if you've

[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/672 @karuturi @rhtyd @syed PR #1829 is subset of this PR i.e. that contains the same fix as one commit of this PR. How to proceed with that? --- If your project is set up for it, you can reply

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-14 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1741 thank you sir. :) --- 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

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 @rhtyd 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 reply

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1836 @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 and

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1928 @syed simply moving this to a file is not enough. Please see Upgrade4910to4920.java and the class usage to see how upgrade paths are implemented. We'll need a Upgrade4920to4930 upgrade path,

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1741 Thanks @swill @jayapalu for the work. I'll build a new systemvmtemplate to use with upcoming Trilian tests on master/4.10+. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/672 @anshul1886 can you retarget this PR for 4.9? Also rebase against 4.9, fix conflicts thanks. /cc @karuturi @syed @koushik-das --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1829 LGTM. @karuturi this can be merged. --- 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

[GitHub] cloudstack issue #865: CLOUDSTACK-8856 Primary Storage Used(type tag with va...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/865 ### ACS CI BVT Run **Sumarry:** Build Number 340 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack pull request #1939: CLOUDSTACK-8886: Limitations is listUsageReco...

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1939 --- 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 pull request #858: CLOUDSTACK-8886: Limitations is listUsageRecor...

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/858 --- 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 is

[GitHub] cloudstack issue #1939: CLOUDSTACK-8886: Limitations is listUsageRecords out...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1939 merging --- 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

[GitHub] cloudstack issue #1838: CLOUDSTACK-9682: Block VM migration to a storage whi...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1838 ### ACS CI BVT Run **Sumarry:** Build Number 336 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=0 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1246: CLOUDSTACK-9165 unable to use reserved IP range in a...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1246 ### ACS CI BVT Run **Sumarry:** Build Number 328 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manu...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1252 ### ACS CI BVT Run **Sumarry:** Build Number 327 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #872: Strongswan vpn feature

2017-02-14 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/872 PR#1741 is closed. So closed 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 does not have this feature

[GitHub] cloudstack pull request #872: Strongswan vpn feature

2017-02-14 Thread jayapalu
Github user jayapalu closed the pull request at: https://github.com/apache/cloudstack/pull/872 --- 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 #1900: CLOUDSTACK-8862: Introduced new state attaching for ...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1900 ### ACS CI BVT Run **Sumarry:** Build Number 324 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #843: Security group ingress/egress issues with xenserver 6...

2017-02-14 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/843 Rebased against master. This PR changes are for basic zone. CI runs test cases for advanced zone. So tested manually. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1868: CLOUDSTACK-9707: While using hostid parameter, vm ge...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1868 ### ACS CI BVT Run **Sumarry:** Build Number 323 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=0 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1867: CLOUDSTACK-9706: Added snapshots cleanup in start an...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1867 ### ACS CI BVT Run **Sumarry:** Build Number 322 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1865: CLOUDSTACK-9705: Unauthenticated API allows Admin pa...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1865 ### ACS CI BVT Run **Sumarry:** Build Number 321 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=0 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-14 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1918 @rafaelweingartner please ommit my last comment, service offering didn't use reserve CPU, so it didn't limit CPU speed to 500Mhz, it used all the available frecuency of the CPU instead. ---

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-14 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1918 @rafaelweingartner sorry for the last example, actually I think we've found a bug on change service offering, as I've been using that instance for changing its service offering when it had vm

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
If it helps in debugging, I typically assign 10.117.40.20 - 10.117.40.25 for Management IPs and 10.117.40.26 - 10.117.40.99 for Guest IPs. On the network in my lab, I am assigned the following range (by our IT people): 10.117.40.0 /25 (10.117.40.0 - 10.117.40.127) Subnet Mask: 255.255.255.128

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Also, for any CloudStack networking person out there who is curious about this, I see the following in cloud.log for my SSVM: Wed Feb 15 01:07:50 UTC 2017 Incompleted parameters STORAGE_IP:, STORAGE_NETMASK:, STORAGE_CIDR:. Cannot setup storage network Wed Feb 15 01:07:50 UTC 2017 Not setting

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
I’m not very familiar with the networking code that’s failing, but I worked around it with these changes: https://github.com/mike-tutkowski/cloudstack/commit/834b7ea7df18f278cbc9eb1bfaa9578fad2debeb I’m not sure if those are viable changes in this case, but perhaps someone who works in the

[GitHub] cloudstack-www pull request #25: Updated list with makewholist.py

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-www/pull/25 --- 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 #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

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

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
By the way, this is for a Basic Zone. On 2/14/17, 2:44 PM, "Tutkowski, Mike" wrote: Hi, I’m getting a NullPointerException when trying to create a zone using master. Below is the relevant code in ConfigurationManagerImpl. In the else

Re: PRs for 4.10

2017-02-14 Thread Syed Ahmed
I'd like to include https://github.com/apache/cloudstack/pull/1928 to 4.10 as well. This is a simple fix that adds hypervisor capabilities for XenServer 7 Thanks, -Syed On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens wrote: > Not sure, I will see if I can find some time

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-14 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1928 Thanks @rhtyd for the review. I've moved it to the schema4920to4930.sql file. @karuturi Can we also push this for 4.10? --- If your project is set up for it, you can reply to this email and

Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Hi, I’m getting a NullPointerException when trying to create a zone using master. Below is the relevant code in ConfigurationManagerImpl. In the else block, network.getCidr() returns null and NetUtil.getCidrNetmask then throws a NullPointerException. I noticed that network.getGateway() also

Re: Annotations Exception

2017-02-14 Thread Syed Ahmed
It's been happening to me recently as well Mike. I think there was some commit after which the management server takes way too long to start spending time scanning annotations. On Tue, Feb 14, 2017 at 4:07 PM, Tutkowski, Mike wrote: > In case this happens to anyone

Re: Annotations Exception

2017-02-14 Thread Tutkowski, Mike
In case this happens to anyone else, here is how I solved it (maxWait parameter): mvn –pl client jetty:run –Dnoredist –Dorg.eclipse.jetty.annotations.maxWait=240 On 2/14/17, 1:20 PM, "Tutkowski, Mike" wrote: Hi, Anyone know why I might be getting this

Annotations Exception

2017-02-14 Thread Tutkowski, Mike
Hi, Anyone know why I might be getting this exception starting up the management server (with master)? java.lang.Exception: Timeout scanning annotations at org.eclipse.jetty.annotations.AnnotationConfiguration.scanForAnnotations(AnnotationConfiguration.java:578) at

[GitHub] cloudstack issue #1792: CLOUDSTACK-9623: Deploying virtual machine fails due...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1792 ping @sateesh-chodapuneedi @karuturi --- 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 #1786: CLOUDSTACK-9618: Load Balancer configuration page do...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1786 ping @sateesh-chodapuneedi @rajesh-battala @karuturi --- 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 #1771: CLOUDSTACK-9611: Dedicating a Guest VLAN range to Pr...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1771 ping @karuturi @sateesh-chodapuneedi --- 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 #1758: CLOUDSTACK-9588: Add Load Balancer functionality in ...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1758 ping @rajesh-battala @karuturi --- 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

[GitHub] cloudstack issue #1939: CLOUDSTACK-8886: Limitations is listUsageRecords out...

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

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-14 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1918 @nvazquez, this helped. I checked the “VmwareResource.java” at line 5113 is obtained the CPU usage of VM. From @jayakarteek explanations I understood that the usage does

[GitHub] cloudstack pull request #1907: Fix public IPs not being removed from the VR ...

2017-02-14 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1907#discussion_r101062037 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -659,7 +663,10 @@ def is_guest_gateway(self, bag, ip): gw =

[GitHub] cloudstack issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-14 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1919 Thanks for the explanation @serbaut. That is exactly why I asked; so, it is not a reboot/restart per se. It is a re-deploy; the old virtual machine is destroyed and a new one is

[GitHub] cloudstack issue #1884: CLOUDSTACK-9699: Add global setting for enable/disab...

2017-02-14 Thread rashmidixit
Github user rashmidixit commented on the issue: https://github.com/apache/cloudstack/pull/1884 @rhtyd Thanks for the update. --- 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

[GitHub] cloudstack issue #1855: CLOUDSTACK-9697: Better error message if user tries ...

2017-02-14 Thread rashmidixit
Github user rashmidixit commented on the issue: https://github.com/apache/cloudstack/pull/1855 @sadhugit I have updated the bug description based on your comments. --- 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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 @blueorangutan test centos7 xenserver-65sp1 --- 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1829 @borisstoyanov unsupported parameters provided. Supported mgmt server os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, kvm-centos7, kvm-ubuntu,

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 @blueorangutan test centos7 xenserver-65sp2 --- 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 #1741: Updated StrongSwan VPN Implementation

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1741 --- 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 #1741: Updated StrongSwan VPN Implementation

2017-02-14 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1741 It has required LGTMs and tests. I am merging this. --- 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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1829 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-485 --- 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 #1741: Updated StrongSwan VPN Implementation

2017-02-14 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1741 @deepthimachiraju remember that you need a new system vm built from this PR for this functionality to work. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1829 @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 #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 @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

Re: Handling of DB migrations on forks

2017-02-14 Thread Wei ZHOU
Then you have to create your own branch forked from 4.10.0 In our branch, I moved some table changes (eg ALTER TABLE, CREATE TABLE) from schema-.sql to engine/schema/src/com/cloud/upgrade/dao/UpgradeXXXtoYYY.java. If SQLException is throwed, then show a warning message instead upgrade

Handling of DB migrations on forks

2017-02-14 Thread Jeff Hair
Hi all, Many people in the CS community maintain forks of CloudStack, and might have implemented features or bug fixes long before they get into mainline. I'm curious as to how people handle database migrations with their forks. To make a DB migration, the CS version must be updated. If a

[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1940 @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 #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1940 @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 #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1940 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-484 --- 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 #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-14 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1919 I agree with @serbaut , different from password which is applied to VR only once, the public keys should be set in VR each time when we recreate a VR. this LGTM+1 --- If your

[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1940 @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 #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1940 @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 #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-14 Thread serbaut
Github user serbaut commented on the issue: https://github.com/apache/cloudstack/pull/1919 The VPC VR maintains metadata (http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/virtual_machines/user-data.html) as static files in /var/www/html/metadata. When a VR

[GitHub] cloudstack pull request #1940: CLOUDSTACK-9781:ACS records ID in events tabl...

2017-02-14 Thread jayantpatil1234
GitHub user jayantpatil1234 opened a pull request: https://github.com/apache/cloudstack/pull/1940 CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. ISSUE = Wrong presentation of volume id in ASC events. While creating a snapshot, only volume ID is

[GitHub] cloudstack issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-14 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1818 @karuturi in my understanding, the zoneid is not necessary if template is not cross-zone. If template is cross-zone, the zoneid should be passed if template is deleted from the 'Zones'

[GitHub] cloudstack issue #1881: CLOUDSTACK-9721: Remove deprecated/unused global con...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1881 configuration cleanup. needs BVT --- 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

[GitHub] cloudstack issue #1882: CLOUDSTACK-8737: Removed the missed out-of-band VR r...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1882 unused class is removed. needs BVT --- 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

[GitHub] cloudstack pull request #873: CLOUDSTACK-8896: allocated percentage of stora...

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/873 --- 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 is

[GitHub] cloudstack issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1818 @ustcweizhou in the above snippet, its adding the zoneid if its not cross-zone. if no zoneid is provided, it will default to -1 which means cross zone. So, this check is required. This

[GitHub] cloudstack pull request #1885: CLOUDSTACK-9724: Fixed missing additional pub...

2017-02-14 Thread jayapalu
Github user jayapalu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1885#discussion_r100985099 --- Diff: server/src/com/cloud/network/IpAddressManagerImpl.java --- @@ -460,6 +460,12 @@ boolean checkIfIpAssocRequired(Network network, boolean

[GitHub] cloudstack pull request #1794: added more guest os

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1794 --- 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 pull request #1244: Bug-ID: CLOUDSTACK-8654: Added guest_os mappi...

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1244 --- 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 #1939: CLOUDSTACK-8886: Limitations is listUsageRecords out...

2017-02-14 Thread kishankavala
Github user kishankavala commented on the issue: https://github.com/apache/cloudstack/pull/1939 LGTM API response: admin 8c91c04e-f282-11e6-8a09-d4ae52cb9a54 8c91ab6a-f282-11e6-8a09-d4ae52cb9a54 ROOT a6401f5b-b090-4a64-9d73-c04369d15ca8

[GitHub] cloudstack issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request t...

2017-02-14 Thread marcaurele
Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1934 @remibergsma Good point, I was aware of that difference, which I think doesn't help to make systems reliable. Another improvement would be to remove this function and refactor the code

[GitHub] cloudstack issue #1939: CLOUDSTACK-8886: Limitations is listUsageRecords out...

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1939 @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 #1939: CLOUDSTACK-8886: Limitations is listUsageRecords out...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1939 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-483 --- 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