[RESULT][VOTE] Apache CloudStack 4.8.0

2016-01-25 Thread Remi Bergsma
Hi all, After 72+ hours, the vote for CloudStack 4.8.0 [1] *passes* with 5 PMC + 1 non-PMC votes. +1 (PMC / binding) * Daan * Milamber * Remi * Boris * Nux +1 (non binding) * Glenn 0 Suresh Sadhu -1 none Thanks to everyone participating. I will now prepare the release announcement to go out

[RESULT][VOTE] Apache CloudStack 4.7.1

2016-01-25 Thread Remi Bergsma
Hi all, After 72+ hours, the vote for CloudStack 4.7.1 [1] *passes* with 4 PMC + 0 non-PMC votes. +1 (PMC / binding) * Boris * Remi * Daan * Milamber +1 (non binding) none 0 none -1 Tomasz Thanks to everyone participating. I will now prepare the release announcement to go out after 24 hours

[GitHub] cloudstack pull request: CLOUDSTACK-9256 add unique key for static...

2016-01-25 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1364#issuecomment-174863942 `cat /etc/cloudstack/staticroutes.json` before and after change: Before always had one entry: ``` { "10.135.255.10": {

[GitHub] cloudstack pull request: CLOUDSTACK-9245 - Cannot delete non-attac...

2016-01-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1358 --- 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 pull request: CLOUDSTACK-9256 add unique key for static...

2016-01-25 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1364#issuecomment-174861340 LGTM :+1: Already running in production at SBP! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

Build failed in Jenkins: build-master-slowbuild #3077

2016-01-25 Thread jenkins
See -- [...truncated 28713 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Fix issue in scale VM to dynamic service ...

2016-01-25 Thread glennwagner
Github user glennwagner commented on the pull request: https://github.com/apache/cloudstack/pull/1363#issuecomment-174850592 This looks good , Will test it in the LAB today --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

Re: Issue with VR after upgrade to 4.6.2

2016-01-25 Thread Remi Bergsma
No, just the same template so it's just a matter of upgrading package. Will ping you when the release is ready. Regards, Remi Sent from my iPhone > On 25 Jan 2016, at 21:41, Daniel Mezentsev wrote: > > Hi Remi, > > Absolutely. I don't see any issue to do upgrade to 4.7.1. Will it be with >

RE: [VOTE] Apache CloudStack 4.8.0

2016-01-25 Thread Suresh Sadhu
Hi, +0 CentOS+xenserver6.5+NFS as primary and secondary storage. Still testing in progress as of now not seen any major issue. Regards Sadhu -Original Message- From: Remi Bergsma [mailto:rberg...@schubergphilis.com] Sent: Thursday, 21 January 2016 1:02 AM To: dev@cloudstack.apac

Build failed in Jenkins: build-master-slowbuild #3076

2016-01-25 Thread jenkins
See -- [...truncated 28713 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-01-25 Thread serg38
Github user serg38 commented on the pull request: https://github.com/apache/cloudstack/pull/1361#issuecomment-174746870 I believe the main reason is to provide the most of backward compatibility. image_store_details doesn't have NFS version in existing installations so after the upgra

Build failed in Jenkins: build-master-slowbuild #3075

2016-01-25 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-01-25 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1361#issuecomment-174734613 @serg38, I did not know that, giving your explanation I rest my arguments ;) @nvazquez, you changed the method “getMountPoint” to receive a NFS

Build failed in Jenkins: build-master-slowbuild #3074

2016-01-25 Thread jenkins
See -- [...truncated 28713 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Re: Issue with VR after upgrade to 4.6.2

2016-01-25 Thread Daniel Mezentsev
Hi Remi, Absolutely. I don't see any issue to do upgrade to 4.7.1. Will it be with new templates for system VMs ? Hi Daniel, Is upgrading to 4.7.1 (once released tomorrow) an option? It has many improvements over 4.6.x and I'm quite sure it fixes the problem you experience now. Regards, Remi

RE: [VOTE] Apache CloudStack 4.8.0

2016-01-25 Thread Glenn Wagner
Hi, +1 Centos 6.7 Management Server (Advanced Networking) Two Xenserver 6.5 Hyper NFS Storage Basic Tests Done (Stop,start,migrate, create Snapshot, delete Snapshot , Create Template , Create ISO, Create VPC , Add Data Drive, Add Shared Network) Thanks Glenn [ShapeBlue]

Re: Issue with VR after upgrade to 4.6.2

2016-01-25 Thread Remi Bergsma
Hi Daniel, Is upgrading to 4.7.1 (once released tomorrow) an option? It has many improvements over 4.6.x and I'm quite sure it fixes the problem you experience now. Regards, Remi Sent from my iPhone > On 25 Jan 2016, at 16:29, Daniel Mezentsev wrote: > > in my configuration VR is acting a

Re: New Defects reported by Coverity Scan for cloudstack

2016-01-25 Thread Erik Weber
Slighly off topic, but could coverity be integrated with our PR workflow? Erik Den mandag 25. januar 2016 skrev Daan Hoogland følgende: > Did a quick scan of the reported new issues. All four are older then the > last report. Anybody feels like taking ownership? > > On Mon, Jan 25, 2016 at 4:13

Build failed in Jenkins: build-master-slowbuild #3073

2016-01-25 Thread jenkins
See -- [...truncated 28713 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9245 - Cannot delete non-attac...

2016-01-25 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1358#issuecomment-174600362 Thanks @borisroman and @remibergsma . Cheers, Wilder --- If your project is set up for it, you can reply to this email and have your reply appe

Re: [VOTE] Apache CloudStack 4.8.0

2016-01-25 Thread Nux!
Thanks, that was a bit unintuitive, I expected the version number to match ACS. :-) +1 from me based on 441-480 upgrade and various basic operations. The fact I can no longer change service offerings with "custom" ones is a bit disappointing as we use that pretty often, but I wouldn't categoris

[GitHub] cloudstack pull request: [4.5] vmware: improve support for disk co...

2016-01-25 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1366 [4.5] vmware: improve support for disk controllers - Improve disk chain usage while attaching, migrating disks - Gets root disk controller based diskDeviceBusName from volume's chain info

[GitHub] cloudstack pull request: vmware: improve support for disk controll...

2016-01-25 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1365 vmware: improve support for disk controllers - Improve disk chain usage while attaching, migrating disks - Gets root disk controller based diskDeviceBusName from volume's chain info You ca

[GitHub] cloudstack pull request: CLOUDSTACK-9228: Network update with mist...

2016-01-25 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1333#issuecomment-174578490 @rafaelweingartner : Yes, it will be forward merged to master. --- If your project is set up for it, you can reply to this email and have your reply appear

Re: [VOTE] Apache CloudStack 4.8.0

2016-01-25 Thread Remi Bergsma
It should be called systemvm-kvm-4.6 indeed. On 25/01/16 16:29, "Nux!" wrote: >I've added the 4.6.0 SysVM as systemvm-kvm-4.8 and attempted the upgrade (from >4.4.1). > >The upgrade dies with: > >http://fpaste.org/314484/14537357/raw/ > >I'll try to revert and also add systemvm-kvm-4.6, see if

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-25 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-174571489 There isn't a test for this class if you want a suggestion, create a new folder named test in cloud-controller-secondary-storage project and create a new pack

[GitHub] cloudstack pull request: CLOUDSTACK-9256 add unique key for static...

2016-01-25 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1364 CLOUDSTACK-9256 add unique key for static routes in json Static routes that are being set do not show up in the static_routes.json file. The reason for this is that the index that is used,

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-25 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-174561850 Also, where is the best place to put tests for the secondary storage manager? Is there already a test for it? --- If your project is set up for it, you can rep

[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-01-25 Thread serg38
Github user serg38 commented on the pull request: https://github.com/apache/cloudstack/pull/1361#issuecomment-174554271 Unfortunately NFS version negotiation doesn't work properly with all storage vendors. Some vendors e.g. Tintri require that vers=3 is supplied in mount command likel

Re: New Defects reported by Coverity Scan for cloudstack

2016-01-25 Thread Daan Hoogland
Did a quick scan of the reported new issues. All four are older then the last report. Anybody feels like taking ownership? On Mon, Jan 25, 2016 at 4:13 PM, wrote: > > Hi, > > Please find the latest report on new defect(s) introduced to cloudstack > found with Coverity Scan. > > 4 new defect(s) i

Re: [VOTE] Apache CloudStack 4.8.0

2016-01-25 Thread Nux!
I've added the 4.6.0 SysVM as systemvm-kvm-4.8 and attempted the upgrade (from 4.4.1). The upgrade dies with: http://fpaste.org/314484/14537357/raw/ I'll try to revert and also add systemvm-kvm-4.6, see if having that fixes it. -- Sent from the Delta quadrant using Borg technology! Nux! www.n

Re: Issue with VR after upgrade to 4.6.2

2016-01-25 Thread Daniel Mezentsev
in my configuration VR is acting as DHCP/DNS server, so it's not passing any traffic, so i can exclude public IP addresses. Definetely there is one of the VR script is acting badly.   If I disassociated 10 public IPs and keep 10 public IPs (but keep 20 vms associated with the network), the netw

Build failed in Jenkins: build-master-slowbuild #3072

2016-01-25 Thread jenkins
See -- [...truncated 28683 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

New Defects reported by Coverity Scan for cloudstack

2016-01-25 Thread scan-admin
Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 4 new defect(s) introduced to cloudstack found with Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 4 of 4 defect(s) ** CID 1147055: Null pointer dereferences (REVERSE_INU

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-25 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-174539020 Yes, your import is correct, I've done a mistake, the collections4 (new collection version) isn't used by ACS, sry for that ^_^' --- If your project is set u

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-25 Thread ProjectMoon
Github user ProjectMoon commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1360#discussion_r50702050 --- Diff: services/secondary-storage/controller/src/org/apache/cloudstack/secondarystorage/SecondaryStorageManagerImpl.java --- @@ -516,6 +517,79 @@

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-25 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1360#discussion_r50699965 --- Diff: services/secondary-storage/controller/src/org/apache/cloudstack/secondarystorage/SecondaryStorageManagerImpl.java --- @@ -516,6 +517,7

[GitHub] cloudstack pull request: Fix issue in scale VM to dynamic service ...

2016-01-25 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1363 Fix issue in scale VM to dynamic service offering This reverts commit 9c4162ac7f451fc3e2155418dcfff224c8c08a4a and 16baa1289b7de383e98d0070717b3f1873fa2db3 Before change: exception

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-25 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-174517194 I've added the changes requested, minus the tests (still working on those). I am pushing the changes now for review. @pedro-martins The CollectionUtils library

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2016-01-25 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-174510528 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 enab

Re: VM deployment from new template fails

2016-01-25 Thread Simon Weller
Have you done any debugging on the Secondary Storage VM yet? That's probably where I would focus. Here's a guide: https://cwiki.apache.org/confluence/display/CLOUDSTACK/SSVM,+templates,+Secondary+storage+troubleshooting - Si From: Indra Pramana Sent: S

[GitHub] cloudstack pull request: CLOUDSTACK-9245 - Cannot delete non-attac...

2016-01-25 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1358#issuecomment-174492298 LGTM :+1: Based on integration tests and code-review. --- If your project is set up for it, you can reply to this email and have your reply appear on G

Build failed in Jenkins: build-master-slowbuild #3071

2016-01-25 Thread jenkins
See -- [...truncated 824 lines...] at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155) at org.apache.maven.surefire.booter.ForkedBooter

Build failed in Jenkins: build-master-slowbuild #3070

2016-01-25 Thread jenkins
See -- [...truncated 28713 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9228: Network update with mist...

2016-01-25 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1333#issuecomment-174442821 @nitin-maharana, it is much better now (did you see the difference?). I would just recommend you declaring a function as “var functionName = function