Re: F5 compile issue with 4.5.x

2017-04-04 Thread Rafael Weingärtner
I feel that I have seen this before. What is your JDK version? On Tue, Apr 4, 2017 at 4:06 PM, Tutkowski, Mike wrote: > Hi, > > I see the following error now when trying to compile 4.5.x code: > > [INFO] Compiling 12 source files to /Users/solidfire/Documents/ >

Re: F5 compile issue with 4.5.x

2017-04-04 Thread Tutkowski, Mike
I am running with Java 8 because I typically have been working with 4.10. I just needed to go back to 4.5.x for another project I’m working on. Thanks On 4/4/17, 2:09 PM, "Rafael Weingärtner" wrote: I feel that I have seen this before. What is your JDK

[GitHub] cloudstack issue #1834: CLOUDSTACK-9679:Allow master user to manage subordin...

2017-04-04 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1834 @pdumbre @karuturi @syed I thinks this introduce an issue where a domain admin cannot get a template by id. a domain admin can list feature template and get a template via `ids=` but

[GitHub] cloudstack issue #1944: CLOUDSTACK-9783: Improve metrics view performance

2017-04-04 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1944 Please ommit my last comment, we were using commons-beanutils version 1.8.3 instead of 1.9.2 LGTM [root@ussarlabcsmgt41 cloudstack]# cat

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

2017-04-04 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1918 LGTM for testing. Tested on Vmware 6.0, I attach some screenshots for test scenarios, in which vm was deployed using service offering, infinite loop was executed to increase CPU utilization

Re: F5 compile issue with 4.5.x

2017-04-04 Thread Tutkowski, Mike
To try to get around it, I removed the F5 project. It almost compiled then. ( I got this in the client project: [ERROR] Failed to execute goal on project cloud-client-ui: Could not resolve dependencies for project org.apache.cloudstack:cloud-client-ui:war:4.5.1: Could not find artifact

F5 compile issue with 4.5.x

2017-04-04 Thread Tutkowski, Mike
Hi, I see the following error now when trying to compile 4.5.x code: [INFO] Compiling 12 source files to /Users/solidfire/Documents/CloudStack/CloudStack/plugins/network-elements/f5/target/classes [INFO] - [ERROR] COMPILATION ERROR :

Re: F5 compile issue with 4.5.x

2017-04-04 Thread Rafael Weingärtner
What version of Java 8 (e.g. jdk1.8.0_112)? I had a similar problem quite a long time ago, the compilation would work with jdk1.8.0_11 (or JDK 7), but would not work with anything higher. Did you try using Java JDK 7? On Tue, Apr 4, 2017 at 4:14 PM, Tutkowski, Mike

[GitHub] cloudstack issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not clea...

2017-04-04 Thread mrunalinikankariya
Github user mrunalinikankariya commented on the issue: https://github.com/apache/cloudstack/pull/1740 In ACS, all the snapshots are deleted from primary storage as soon as it is successfully backed up and copied on secondary storage, hence this issue is not valid for ACS.

Re: F5 compile issue with 4.5.x

2017-04-04 Thread Tutkowski, Mike
javac 1.8.0_121 By removing F5, though, I just got a successful compile. :) On 4/4/17, 2:19 PM, "Rafael Weingärtner" wrote: What version of Java 8 (e.g. jdk1.8.0_112)? I had a similar problem quite a long time ago, the compilation would work with

[GitHub] cloudstack pull request #2026: CLOUDSTACK-9861: Expire VM snapshots after co...

2017-04-04 Thread abhinandanprateek
GitHub user abhinandanprateek opened a pull request: https://github.com/apache/cloudstack/pull/2026 CLOUDSTACK-9861: Expire VM snapshots after configured duration Default value of the account level global config vmsnapshot.expire.interval is -1 that conforms to legacy behaviour

[GitHub] cloudstack issue #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2014 @abhinandanprateek 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

[GitHub] cloudstack issue #2026: CLOUDSTACK-9861: Expire VM snapshots after configure...

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2026 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-616 --- 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 #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/2014 @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

[GitHub] cloudstack issue #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2014 @abhinandanprateek 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

[GitHub] cloudstack pull request #1996: CLOUDSTACK-9099: SecretKey is returned from t...

2017-04-04 Thread kishankavala
Github user kishankavala commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1996#discussion_r109835294 --- Diff: server/src/com/cloud/user/AccountManager.java --- @@ -198,4 +200,11 @@ void buildACLViewSearchCriteria(SearchCriteria s public

[GitHub] cloudstack issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2017-04-04 Thread jayantpatil1234
Github user jayantpatil1234 commented on the issue: https://github.com/apache/cloudstack/pull/1849 @sudhansu7 There is one file "Upgrade4910to4920.java", which does not have any changes, still that get added in PR. If possible could you remove that file? Otherwise code

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

2017-04-04 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1935 @borisstoyanov I've rebased master branch, can we re-run tests 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

[GitHub] cloudstack pull request #2020: WIP: Expire VM snapshots

2017-04-04 Thread abhinandanprateek
Github user abhinandanprateek closed the pull request at: https://github.com/apache/cloudstack/pull/2020 --- 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 #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-04-04 Thread borisstoyanov
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 #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-04-04 Thread blueorangutan
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 pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

2017-04-04 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r102538324 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -109,6 +112,20 @@ @Inject MessageBus _messageBus;

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

2017-04-04 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r103529869 --- Diff: server/test/com/cloud/user/DomainManagerImplTest.java --- @@ -134,4 +164,69 @@ public void testFindDomainByIdOrPathValidId() {

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

2017-04-04 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r103529571 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,82 +284,145 @@ public boolean deleteDomain(long domainId, Boolean

[GitHub] cloudstack issue #1997: CLOUDSTACK-9208: Assertion Error in VM_POWER_STATE h...

2017-04-04 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1997 @jayapalu your check is ok, what is not ok is that it is needed. Somehow a stop can be send to a VM for which no host is known? I am worried about the reason of your fix and that we

[GitHub] cloudstack issue #2012: CLOUDSTACK-9842: Unable to map root volume usage to ...

2017-04-04 Thread yvsubhash
Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/2012 Tested this change. LGTM for 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

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

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-614 --- 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 #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 @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...

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

Re: :[VOTE] Apache Cloudstack 4.10.0.0

2017-04-04 Thread Wei ZHOU
tomcat7 on ubuntu 16.04 is not working fine in my testing. Anyone has same issue ? -Wei 2017-04-03 15:11 GMT+02:00 Pierre-Luc Dion : > Look like we need a new systemvm named "systemvm-xenserver-4.10". t also > seams that older VR ex: 4.7.x are still usable if the

[GitHub] cloudstack issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-04-04 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1996 @rhtyd Once this PR got the LGTMs, I can rebase it on 4.9. Can you please review this PR --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #921: CLOUDSTACK-8944 Template download possible from new n...

2017-04-04 Thread yvsubhash
Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/921 Closing this PR, as this is no longer needed --- 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 pull request #921: CLOUDSTACK-8944 Template download possible fro...

2017-04-04 Thread yvsubhash
Github user yvsubhash closed the pull request at: https://github.com/apache/cloudstack/pull/921 --- 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 #2018: CLOUDSTACK-9848: Added exit status checking for the ...

2017-04-04 Thread bvbharatk
Github user bvbharatk commented on the issue: https://github.com/apache/cloudstack/pull/2018 code changes LGTM. --- 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 #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2014 @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 #2020: WIP: Expire VM snapshots

2017-04-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2020 @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: :[VOTE] Apache Cloudstack 4.10.0.0

2017-04-04 Thread Wei ZHOU
Guys, I created a PR for erros in restartnetwork in 4.10.0.0-RC, please review. https://github.com/apache/cloudstack/pull/2025 -Wei 2017-04-04 9:27 GMT+02:00 Wei ZHOU : > tomcat7 on ubuntu 16.04 is not working fine in my testing. > Anyone has same issue ? > > -Wei > >

[GitHub] cloudstack issue #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2014 Packaging result: ✖centos6 ✖centos7 ✖debian. JID-613 --- 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 #914: BUG-ID CLOUDSTACK-8939 VM Snapshot size with memory i...

2017-04-04 Thread niteshsarda
Github user niteshsarda commented on the issue: https://github.com/apache/cloudstack/pull/914 I have tested this and **LGTM** for test. Following are the test results : **Before applying fix :** [root@xenserver-jay ~]# xe vbd-list vm-name-label=i-2-120-VM

[GitHub] cloudstack pull request #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubu...

2017-04-04 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1950#discussion_r109633875 --- Diff: debian/control --- @@ -16,14 +16,15 @@ Description: A common package which contains files which are shared by several C Package:

[GitHub] cloudstack issue #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/2014 @borisstoyanov checking --- 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 #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2017-04-04 Thread jayakarteek
Github user jayakarteek commented on the issue: https://github.com/apache/cloudstack/pull/1849 @sudhansu7 Tested the fix, LGTM. I am able to change from small computer offering to medium computer offering for centos-7. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2017-04-04 Thread yvsubhash
Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/1726 tag:mergeready --- 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 #1997: CLOUDSTACK-9208: Assertion Error in VM_POWER_STATE h...

2017-04-04 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1997 @ramkatru I am not sure about Daan comments. The tests results are positive. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request #2025: Fix error in restart network in 4.10.0.0 RC

2017-04-04 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/2025 Fix error in restart network in 4.10.0.0 RC The PR fixes the error in restart network: 2017-04-04 10:27:39,217 DEBUG [c.c.n.r.NetworkHelperImpl] (API-Job-Executor-2:ctx-08904854

[GitHub] cloudstack issue #2020: WIP: Expire VM snapshots

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2020 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-612 --- 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 #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubuntu 14....

2017-04-04 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1950 it would be nice if this can be merged into 4.10.0.0 --- 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 #2020: WIP: Expire VM snapshots

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2020 @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 #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2014 @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 #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/2014 @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

[GitHub] cloudstack issue #2026: CLOUDSTACK-9861: Expire VM snapshots after configure...

2017-04-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2026 @abhinandanprateek 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

[GitHub] cloudstack issue #2026: CLOUDSTACK-9861: Expire VM snapshots after configure...

2017-04-04 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/2026 @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

[GitHub] cloudstack pull request #2014: WIP: Fix primary storage usage for vm snaphot

2017-04-04 Thread abhinandanprateek
Github user abhinandanprateek closed the pull request at: https://github.com/apache/cloudstack/pull/2014 --- 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