Jenkins build is back to normal : build-master-simulator #1151

[jenkins]

See 

Build failed in Jenkins: build-master-simulator #1152

[jenkins]

See 

Changes:

[rajesh.battala] CLOUDSTACK-7929: While creating network offering if one 
specifies negative value for network rate then we will convert that value to 0 
i.e. unlimited

--
[...truncated 1273 lines...]
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-engine-components-api ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-engine-components-api ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Server 4.6.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-server ---
[INFO] Deleting 
 
(includes = [**/*], excludes = [])
[INFO] Deleting 
 (includes 
= [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-server ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-server 
---
[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-server ---
[INFO] Executing tasks

main:
 [copy] Copying 3 files to 

 [copy] Copying 1 file to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-server ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 30 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-server 
---
[INFO] Compiling 405 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-server ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 33 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-server ---
[INFO] Compiling 98 source files to 

[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ cloud-server ---
[INFO] Surefire report directory: 


---
 T E S T S
---
Running com.cloud.event.EventControlsUnitTest
log4j:WARN No appenders could be found for logger 
(com.cloud.event.EventControlsUnitTest).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.734 sec
Running com.cloud.keystore.KeystoreTest
org.apache.cloudstack.api.response.UserVmResponse/null/{"id":"3","securitygroup":[],"nic":[],"tags":[],"affinitygroup":[]}
org.apache.cloudstack.api.response.AlertResponse/null/{"id":"100","description":"Hello"}
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.176 sec
Running com.cloud.alert.AlertControlsUnitTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.026 sec
Running com.cloud.capacity.CapacityManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.147 sec
Running com.cloud.servlet.StaticResourceServletTest
Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.145 sec
Running com.cloud.servlet.ConsoleProxyServletTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 sec
Running com.cloud.resourcelimit.ResourceLimitManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.cloud.network.firewall.FirewallManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.001 sec
Running com.cloud.network.ExternalLoadBalancerDeviceManagerImplTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.644 sec
Running com.cloud.ne

Jenkins build is back to normal : build-master-simulator #1153

[jenkins]

See 

[DISCUSS] 4.3 branch backporting bugfixes

[Rohit Yadav]

Hi,

ACS 4.3 branch and both 4.3.0 and 4.3.1 were rock solid releases. Before 4.5.0 
is out or a stable and well-tested 4.4.x release, should we go through issues 
reported for affects version 4.3.0/4.3.1 but a fix version that is 
4.4.x/4.5.x/4.6.x [1] and backport on 4.3 branch.

Should we do a 4.3.2 release? If that may add DB upgrade path issues, I can 
help build a 4.3.1 patch and host on ShapeBlue’s packages hosting [2],  the 
branch of which is public [3] and useful for everyone (i.e. we build ACS 4.3.1 
with commits since 4.3.1 release). The useful thing about our patch releases is 
that we do short release notes [3] and upgrades are as simple as just 
apt-get/yum update/upgrade.

[1] 
https://issues.apache.org/jira/browse/CLOUDSTACK-7595?jql=project%20%3D%20CLOUDSTACK%20AND%20status%20in%20(Resolved%2C%20Closed)%20AND%20affectedVersion%20in%20(4.3.0%2C%204.3.1)%20AND%20fixVersion%20in%20(4.4.0%2C%204.4.1%2C%204.5.0%2C%204.6.0)%20AND%20created%20%3C%3D%20-8w
[2] shapeblue.com/packages
[3] https://github.com/shapeblue/cloudstack/tree/shapeblue-4.3.1

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Software 
Engineering
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: [DISCUSS] 4.3 branch backporting bugfixes

[Daan Hoogland]

Two points of concern, Rohit;

1. Doing this is making a promise for the future. Can and will we keep this?
2. don't port anything to 4.3.2 that you don't apply to the  4.4 branch as well.

otherwise I'm all for it.

[GitHub] cloudstack pull request: CLOUDSTACK-7929: Unhandled exception when...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/40


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/39


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/41


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

[karuturi]

Github user karuturi commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/41#discussion_r20492616
  
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java 
---
@@ -725,6 +725,21 @@ private String validateConfigurationValue(String name, 
String value, String scop
 type = c.getType();
 }
 
+String errMsg = null;
+try {
+if (type.equals(Integer.class)) {
+errMsg = "There was error in trying to parse value: " + 
value + ". Please enter a valid integer value for parameter " + name;
+Integer.parseInt(value);
+} else if (type.equals(Float.class)) {
+errMsg = "There was error in trying to parse value: " + 
value + ". Please enter a valid float value for parameter " + name;
+Float.parseFloat(value);
--- End diff --

I understand that it will fail during parsing. But, we can string replace 
"," with ""


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Build failed in Jenkins: build-4.5 #141

[jenkins]

See 

Changes:

[rajesh.battala] CLOUDSTACK-7688, CLOUDSTACK-7747: restricted various 
operations for VM with VM snapshots which breaks VM snapshots.

[rajesh.battala] CLOUDSTACK-7767: fixed events are not generated for snapshot 
creation

[rajesh.battala] CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed deployment planner 
stuck in infinite loop.

[rajesh.battala] CLOUDSTACK-7620: Added SNMP MIB file for snmp-alerts plugin

[rajesh.battala] CLOUDSTACK-7758: Fixed although api calls are failing, event 
tab shows them as successful

[rajesh.battala] CLOUDSTACK-7541: Added restriction to not allow custom disk 
offering with disksize UI doesn't allow but with API we were able to create 
custom disk offering with disk size which was causing this issue

[rajesh.battala] CLOUDSTACK-7930, CLOUDSTACK-7931: Do not allow to set invalid 
values for global settings which are of type integer and float

[rajesh.battala] CLOUDSTACK-7932: Fixed wrong semantics for isVmAlive() method 
in HypervInvestigator

[rajesh.battala] CLOUDSTACK-7929: While creating network offering if one 
specifies negative value for network rate then we will convert that value to 0 
i.e. unlimited

--
[...truncated 1630 lines...]
[INFO] Deleting 
 
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-orchestration ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-engine-network 
---
[INFO] Deleting 
 (includes = 
[target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-network ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-usage ---
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-usage ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-server ---
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-server ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-network ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-usage ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-network ---
[INFO] Compiling 1 source file to 

[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-orchestration ---
[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-usage ---
[INFO] Executing tasks

main:
 [copy] Copying 2 files to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-usage ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-usage 
---
[INFO] Compiling 19 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-orchestration ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-orchestration ---
[INFO] Compiling 68 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-

Re: Multiple local storage pools?

[Nux!]

Thanks, Marcus.
I was eyeing shared mountpoint as well, but was hoping for something better. I 
guess it will have to do. 
Cheers :)

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Marcus" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 18 November, 2014 07:27:13
> Subject: Re: Multiple local storage pools?

> Not directly, but one way you could make this happen would be to create a
> 'sharedmountpoint' primary storage, and use tags to ensure that it doesn't
> try to use that storage on any host other than the desired one.
> Sharedmountpoint is basically telling CS that you have done the work of
> mounting a clustered filesystem in a specific place.
> 
> For example, set the host with tag 'host1', the storage with tag
> 'host1ssd', and create a service offering with both storage tag and host
> tag to select that combination. Kind of a hack, but it should work.
> 
> On Mon, Nov 17, 2014 at 11:03 PM, Andrija Panic 
> wrote:
> 
>> Ok, waiting for someone to answer this :)
>>
>> Sent from Google Nexus 4
>> On Nov 18, 2014 12:35 AM, "Nux!"  wrote:
>>
>> > Andrija, but I want a more straight forward way to use the SSDs. ZFS I'm
>> > sure it's awesome, but it would just complicate my setup. Thanks though.
>> >
>> > --
>> > Sent from the Delta quadrant using Borg technology!
>> >
>> > Nux!
>> > www.nux.ro
>> >
>> > - Original Message -
>> > > From: "Andrija Panic" 
>> > > To: dev@cloudstack.apache.org
>> > > Sent: Monday, 17 November, 2014 19:07:15
>> > > Subject: Re: Multiple local storage pools?
>> >
>> > > Lucian, don't relly have the answer - but did you maybe consider ZFS on
>> > > Linux, mounted on /var/lib/libvirt/images/ - there you can add SSD for
>> > > writings and perhaps read cache (SLOG for writes and L2ARC for
>> readings)
>> > ?
>> > >
>> > > On 17 November 2014 19:37, Nux!  wrote:
>> > >
>> > >> Hi guys,
>> > >>
>> > >> Do you know how it would be possible to add multiple local primary
>> > storage
>> > >> pools? I know that if local storage is enabled /var/lib/libvirt/images
>> > (in
>> > >> KVM's case) is added automatically.
>> > >> But assuming I want to add an SSD or SATA array to this host and want
>> to
>> > >> differentiate it within cloudstack, how should I go about it?
>> > >>
>> > >> Lucian
>> > >>
>> > >> --
>> > >> Sent from the Delta quadrant using Borg technology!
>> > >>
>> > >> Nux!
>> > >> www.nux.ro
>> > >>
>> > >
>> > >
>> > >
>> > > --
>> > >
>> > > Andrija Panić
>> > > --
>> > >  http://admintweets.com
>> > > --
>> >

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

[karuturi]

Github user karuturi commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/41#discussion_r20492860
  
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java 
---
@@ -725,6 +725,21 @@ private String validateConfigurationValue(String name, 
String value, String scop
 type = c.getType();
 }
 
+String errMsg = null;
+try {
+if (type.equals(Integer.class)) {
+errMsg = "There was error in trying to parse value: " + 
value + ". Please enter a valid integer value for parameter " + name;
+Integer.parseInt(value);
+} else if (type.equals(Float.class)) {
+errMsg = "There was error in trying to parse value: " + 
value + ". Please enter a valid float value for parameter " + name;
+Float.parseFloat(value);
+}
+} catch (Exception e) {
--- End diff --

catching top level exception and swallowing the stacktrace makes it really 
difficult during debugging. 
For end user, we anyway wont show the exception but only the message. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

[karuturi]

Github user karuturi commented on the pull request:

https://github.com/apache/cloudstack/pull/39#issuecomment-63441065
  
I dont see a case when it could return false 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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Build failed in Jenkins: build-4.5 #142

[jenkins]

See 

--
[...truncated 1655 lines...]
[INFO] Deleting 
 
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-orchestration ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-engine-network 
---
[INFO] Deleting 
 
(includes = [**/*], excludes = [])
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-server ---
[INFO] Deleting  
(includes = [**/*], excludes = [])
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-server ---
[INFO] Deleting 
 (includes = 
[target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-network ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-usage ---
[INFO] Deleting  
(includes = [**/*], excludes = [])
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-usage ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-network ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-usage ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-network ---
[INFO] Compiling 1 source file to 

[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-orchestration ---
[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-usage ---
[INFO] Executing tasks

main:
 [copy] Copying 2 files to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-usage ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-usage 
---
[INFO] Compiling 19 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-orchestration ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-orchestration ---
[INFO] Compiling 68 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-engine-network ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-engine-network ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] Starting audit...
:89:8:
 Unused import - 
org.apache.cloudstack.storage.datastore.db.TemplateDataStoreDao.
:90:8:
 Unused import - org.apa

Jenkins build is still unstable: simulator-singlerun #648

[jenkins]

See 

Review Request 28163: CLOUDSTACK-7934: Fixed cleanup issues test_escalations_volumes.py

[Ashutosh Kelkar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28163/
---

Review request for cloudstack, sanjeev n and SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-7934
https://issues.apache.org/jira/browse/CLOUDSTACK-7934


Repository: cloudstack-git


Description
---

Snapshots, volumes get cleaned up after each test case as part of account 
cleanup. No need to add them separately to cleanup list.


Diffs
-

  test/integration/component/test_escalations_volumes.py 7290325 

Diff: https://reviews.apache.org/r/28163/diff/


Testing
---

Yes.

Log:
@summary: Test List Volumes pagination ... SKIP: Skip
@summary: Test List Volumes with Id ... SKIP: Skip
@summary: Test to verify creation and resize of data volume ... SKIP: Skip
@summary: Test to verify creation and resize of custom volume ... SKIP: Skip
@summary: Test to verify creation of snapshot from volume ... SKIP: Skip
@summary: Test to verify creation of Hourly Snapshot policies ... SKIP: Skip
@summary: Test to verify creation of Daily Snapshot policies ... SKIP: Skip
@summary: Test to verify creation of Weekly Snapshot policies ... SKIP: Skip
@summary: Test to verify creation of Monthly Snapshot policies ... SKIP: Skip
@summary: Test to verify pagination of snapshots for Volume ... SKIP: Skip
@summary: Test to verify extract/download a Volume ... SKIP: Skip
@summary: Test to verify upload volume ... SKIP: Skip
@Desc:Create volume from custom disk offering does not work as expected ... === 
TestName: test_13_volume_custom_disk_size | Status : SUCCESS ===
ok

--
Ran 13 tests in 170.553s

OK (SKIP=12)

@summary: Test List Volumes pagination ... SKIP: Skip
@summary: Test List Volumes with Id ... SKIP: Skip
@summary: Test to verify creation and resize of data volume ... SKIP: Skip
@summary: Test to verify creation and resize of custom volume ... SKIP: Skip
@summary: Test to verify creation of snapshot from volume ... === TestName: 
test_05_volume_snapshot | Status : SUCCESS ===
ok
@summary: Test to verify creation of Hourly Snapshot policies ... SKIP: Skip
@summary: Test to verify creation of Daily Snapshot policies ... SKIP: Skip
@summary: Test to verify creation of Weekly Snapshot policies ... SKIP: Skip
@summary: Test to verify creation of Monthly Snapshot policies ... SKIP: Skip
@summary: Test to verify pagination of snapshots for Volume ... === TestName: 
test_10_volume_snapshots_pagination | Status : SUCCESS ===
ok


Thanks,

Ashutosh Kelkar

Coverity and CCC EU

[Hugo Trippaers]

Heya,

Who would be interested in working on Coverity, FindBugs and/or PMD fixes 
during CCCEU? Happy to introduce people to the Coverity interface and discuss 
option to fix the reported issues. 

It’s a great way to get involved with the CloudStack development processes 
without requiring an intimate knowledge of CloudStack internals.


Cheers,


Hugo 

[GitHub] cloudstack pull request: Use the new build env on Travis

[joshk]

GitHub user joshk opened a pull request:

https://github.com/apache/cloudstack/pull/43

Use the new build env on Travis

faster vm boot times
more cpu cores and ram
better network
caching is available

docs coming soon

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/joshk/cloudstack patch-1

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/43.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #43


commit b52bb788fe6c40059bd8f23e61f8672c367e711f
Author: Josh Kalderimis 
Date:   2014-11-18T10:21:07Z

Use the new build env on Travis

faster vm boot times
more cpu cores and ram
better network
caching is available

docs coming soon




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Jenkins build is still unstable: simulator-singlerun #649

[jenkins]

See 

Re: Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

[Rajani Karuturi]

> On Nov. 17, 2014, 9:44 p.m., Brian Federle wrote:
> > I am getting conflicts in the properties file applying to the latest master 
> > (my HEAD is @ eba7cc78da751cc25994a5a736a87ce2e83c394e). There were a lot 
> > of changes lately to the localization strings, which is probably why it is 
> > conflicting; could you try updating the patch?
> > 
> > Applying: CLOUDSTACK-7915: Remove hard-coded values for Load Balancer 
> > algorithms in UI
> > error: patch failed: 
> > client/WEB-INF/classes/resources/messages_fr_FR.properties:1160
> > error: client/WEB-INF/classes/resources/messages_fr_FR.properties: patch 
> > does not apply
> > error: patch failed: 
> > client/WEB-INF/classes/resources/messages_ja_JP.properties:740
> > error: client/WEB-INF/classes/resources/messages_ja_JP.properties: patch 
> > does not apply
> > error: patch failed: 
> > client/WEB-INF/classes/resources/messages_zh_CN.properties:740
> > error: client/WEB-INF/classes/resources/messages_zh_CN.properties: patch 
> > does not apply
> > Patch failed at 0001 CLOUDSTACK-7915: Remove hard-coded values for Load 
> > Balancer algorithms in UI

Brian, if you use git am -3, it will do a 3-way merge and can resolve the 
conflicts.


- Rajani


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28043/#review61817
---


On Nov. 17, 2014, 6:26 p.m., Daniel Vega Simoes wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28043/
> ---
> 
> (Updated Nov. 17, 2014, 6:26 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> JIRA: CLOUDSTACK 7915
> 
> Removed hard-coded values for Load Balancer algorithms in UI. 
> 
> Instead, now UI sets values loaded dynamically through load balancer provider 
> capabilities.
> 
> Also updated internationalization messages.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 86eb5c2 
>   client/WEB-INF/classes/resources/messages_ar.properties 4f65118 
>   client/WEB-INF/classes/resources/messages_es.properties f2d754e 
>   client/WEB-INF/classes/resources/messages_fr_FR.properties 004187f 
>   client/WEB-INF/classes/resources/messages_it_IT.properties e2f3f0b 
>   client/WEB-INF/classes/resources/messages_ja_JP.properties 7bc90b5 
>   client/WEB-INF/classes/resources/messages_ko_KR.properties ce79d2e 
>   client/WEB-INF/classes/resources/messages_nb_NO.properties c169112 
>   client/WEB-INF/classes/resources/messages_nl_NL.properties 89ef828 
>   client/WEB-INF/classes/resources/messages_pl.properties 06d5ec2 
>   client/WEB-INF/classes/resources/messages_pt_BR.properties 8ee08ba 
>   client/WEB-INF/classes/resources/messages_ru_RU.properties ff68668 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties ebba5e0 
>   ui/dictionary.jsp 671f48f 
>   ui/scripts/network.js c27b999 
>   ui/scripts/vpc.js af19d87 
> 
> Diff: https://reviews.apache.org/r/28043/diff/
> 
> 
> Testing
> ---
> 
> Tests performed:
> - create new isolated network with DefaultIsolatedNetworkOffering, acquire 
> new IP, list load balancer algorithms (UI)
> - create new shared network with DefaultSharedNetworkWithSourceNat, acquire 
> new IP, list load balancer algorithms (UI)
> 
> It should work correctly as long as network offering is configured with LB 
> provider/capabilities. Otherwise, list of algorithms is empty in UI.
> 
> 
> Thanks,
> 
> Daniel Vega Simoes
> 
>

Re: git commit: updated refs/heads/4.3 to e6d00bc

[Daan Hoogland]

4.3.1 is out. is this really what we want, Rohit? seems this revert
shouldn't happen.

On Tue, Nov 18, 2014 at 11:35 AM,   wrote:
> Repository: cloudstack
> Updated Branches:
>   refs/heads/4.3 e615f218a -> e6d00bcc6
>
>
> Revert "Updating pom.xml version numbers for release 4.3.2-SNAPSHOT"
>
> This reverts commit 063bb8f5b482bf2f5f05335c21c2415e779695b9.
>
>
> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
> Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/e6d00bcc
> Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/e6d00bcc
> Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/e6d00bcc
>
> Branch: refs/heads/4.3
> Commit: e6d00bcc6670d77144e6d423091808877eaa610f
> Parents: e615f21
> Author: Rohit Yadav 
> Authored: Tue Nov 18 16:04:33 2014 +0530
> Committer: Rohit Yadav 
> Committed: Tue Nov 18 16:04:33 2014 +0530
>
> --
>  agent/pom.xml | 2 +-
>  api/pom.xml   | 2 +-
>  awsapi/pom.xml| 2 +-
>  client/pom.xml| 2 +-
>  core/pom.xml  | 2 +-
>  debian/changelog  | 6 --
>  deps/XenServerJava/pom.xml| 2 +-
>  developer/pom.xml | 2 +-
>  engine/api/pom.xml| 2 +-
>  engine/components-api/pom.xml | 2 +-
>  engine/network/pom.xml| 2 +-
>  engine/orchestration/pom.xml  | 2 +-
>  engine/pom.xml| 2 +-
>  engine/schema/pom.xml | 2 +-
>  engine/service/pom.xml| 2 +-
>  engine/storage/cache/pom.xml  | 2 +-
>  engine/storage/datamotion/pom.xml | 2 +-
>  engine/storage/image/pom.xml  | 2 +-
>  engine/storage/integration-test/pom.xml   | 2 +-
>  engine/storage/pom.xml| 2 +-
>  engine/storage/snapshot/pom.xml   | 2 +-
>  engine/storage/volume/pom.xml | 2 +-
>  framework/cluster/pom.xml | 2 +-
>  framework/config/pom.xml  | 2 +-
>  framework/db/pom.xml  | 2 +-
>  framework/events/pom.xml  | 2 +-
>  framework/ipc/pom.xml | 2 +-
>  framework/jobs/pom.xml| 2 +-
>  framework/managed-context/pom.xml | 2 +-
>  framework/pom.xml | 2 +-
>  framework/rest/pom.xml| 2 +-
>  framework/spring/lifecycle/pom.xml| 2 +-
>  framework/spring/module/pom.xml   | 2 +-
>  maven-standard/pom.xml| 2 +-
>  parents/checkstyle/pom.xml| 2 +-
>  plugins/acl/static-role-based/pom.xml | 2 +-
>  plugins/affinity-group-processors/explicit-dedication/pom.xml | 2 +-
>  plugins/affinity-group-processors/host-anti-affinity/pom.xml  | 2 +-
>  plugins/alert-handlers/snmp-alerts/pom.xml| 2 +-
>  plugins/alert-handlers/syslog-alerts/pom.xml  | 2 +-
>  plugins/api/discovery/pom.xml | 2 +-
>  plugins/api/rate-limit/pom.xml| 2 +-
>  plugins/database/mysql-ha/pom.xml | 2 +-
>  plugins/dedicated-resources/pom.xml   | 2 +-
>  plugins/deployment-planners/implicit-dedication/pom.xml   | 2 +-
>  plugins/deployment-planners/user-concentrated-pod/pom.xml | 2 +-
>  plugins/deployment-planners/user-dispersing/pom.xml   | 2 +-
>  plugins/event-bus/rabbitmq/pom.xml| 2 +-
>  plugins/file-systems/netapp/pom.xml   | 2 +-
>  plugins/ha-planners/skip-heurestics/pom.xml   | 2 +-
>  plugins/host-allocators/random/pom.xml| 2 +-
>  plugins/hypervisors/baremetal/pom.xml | 2 +-
>  plugins/hypervisors/hyperv/pom.xml| 2 +-
>  plugins/hypervisors/kvm/pom.xml   | 2 +-
>  plugins/hypervisors/ovm/pom.xml   | 2 +-
>  plugins/hypervisors/simulator/pom.xml | 2 +-
>  plug

[GitHub] cloudstack pull request: Use the new build env on Travis

[runseb]

Github user runseb commented on the pull request:

https://github.com/apache/cloudstack/pull/43#issuecomment-63452207
  
Thanks for this @joshk we will merge once it's green. thanks to TravisCI 
for your help


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack-cloudmonkey pull request: keep colons in the request to...

[remibergsma]

GitHub user remibergsma opened a pull request:

https://github.com/apache/cloudstack-cloudmonkey/pull/1

keep colons in the request to ACS

ACS does not decode %3A to colon and thus some requests break

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/remibergsma/cloudstack-cloudmonkey master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack-cloudmonkey/pull/1.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1


commit 5d5c88082b41cca113e9dd640c2d702f824d4fca
Author: Remi Bergsma 
Date:   2014-11-18T11:00:12Z

keep colons in the request to ACS

ACS does not decode %3A to colon and thus some requests break




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: Multiple local storage pools?

[Andrija Panic]

Might be a stupid suggestion - but adding one more record inside database,
storage_pools table - would that work ?


On 18 November 2014 10:04, Nux!  wrote:

> Thanks, Marcus.
> I was eyeing shared mountpoint as well, but was hoping for something
> better. I guess it will have to do.
> Cheers :)
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
> > From: "Marcus" 
> > To: dev@cloudstack.apache.org
> > Sent: Tuesday, 18 November, 2014 07:27:13
> > Subject: Re: Multiple local storage pools?
>
> > Not directly, but one way you could make this happen would be to create a
> > 'sharedmountpoint' primary storage, and use tags to ensure that it
> doesn't
> > try to use that storage on any host other than the desired one.
> > Sharedmountpoint is basically telling CS that you have done the work of
> > mounting a clustered filesystem in a specific place.
> >
> > For example, set the host with tag 'host1', the storage with tag
> > 'host1ssd', and create a service offering with both storage tag and host
> > tag to select that combination. Kind of a hack, but it should work.
> >
> > On Mon, Nov 17, 2014 at 11:03 PM, Andrija Panic  >
> > wrote:
> >
> >> Ok, waiting for someone to answer this :)
> >>
> >> Sent from Google Nexus 4
> >> On Nov 18, 2014 12:35 AM, "Nux!"  wrote:
> >>
> >> > Andrija, but I want a more straight forward way to use the SSDs. ZFS
> I'm
> >> > sure it's awesome, but it would just complicate my setup. Thanks
> though.
> >> >
> >> > --
> >> > Sent from the Delta quadrant using Borg technology!
> >> >
> >> > Nux!
> >> > www.nux.ro
> >> >
> >> > - Original Message -
> >> > > From: "Andrija Panic" 
> >> > > To: dev@cloudstack.apache.org
> >> > > Sent: Monday, 17 November, 2014 19:07:15
> >> > > Subject: Re: Multiple local storage pools?
> >> >
> >> > > Lucian, don't relly have the answer - but did you maybe consider
> ZFS on
> >> > > Linux, mounted on /var/lib/libvirt/images/ - there you can add SSD
> for
> >> > > writings and perhaps read cache (SLOG for writes and L2ARC for
> >> readings)
> >> > ?
> >> > >
> >> > > On 17 November 2014 19:37, Nux!  wrote:
> >> > >
> >> > >> Hi guys,
> >> > >>
> >> > >> Do you know how it would be possible to add multiple local primary
> >> > storage
> >> > >> pools? I know that if local storage is enabled
> /var/lib/libvirt/images
> >> > (in
> >> > >> KVM's case) is added automatically.
> >> > >> But assuming I want to add an SSD or SATA array to this host and
> want
> >> to
> >> > >> differentiate it within cloudstack, how should I go about it?
> >> > >>
> >> > >> Lucian
> >> > >>
> >> > >> --
> >> > >> Sent from the Delta quadrant using Borg technology!
> >> > >>
> >> > >> Nux!
> >> > >> www.nux.ro
> >> > >>
> >> > >
> >> > >
> >> > >
> >> > > --
> >> > >
> >> > > Andrija Panić
> >> > > --
> >> > >  http://admintweets.com
> >> > > --
> >> >
>



-- 

Andrija Panić
--
  http://admintweets.com
--

[GitHub] cloudstack-cloudmonkey pull request: keep colons in the request to...

[runseb]

Github user runseb commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/1#issuecomment-63455032
  
is there a jira issue for it ? if yes we will need to close it.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Use the new build env on Travis

[imduffy15]

Github user imduffy15 commented on the pull request:

https://github.com/apache/cloudstack/pull/43#issuecomment-63457295
  
Hi @joshk this is going to cause issues as we use sudo to install packages 
within the before_install script.

Is it possible to have sudo access and the speed benefits? 

Thanks,
Ian


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack-cloudmonkey pull request: keep colons in the request to...

[remibergsma]

Github user remibergsma commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/1#issuecomment-63458765
  
I didn't create a ticket for it. Will do if you want me to :-)

Sent from my iPhone

> On 18 Nov 2014, at 12:13, runseb  wrote:
> 
> is there a jira issue for it ? if yes we will need to close it.
> 
> —
> Reply to this email directly or view it on GitHub.
> 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Advised to send to Dev list - Broken update from 4.4 to 4.4.1

[Matthew Midgett]

I tried again to update from 4.4.0 to 4.4.1 with centos management server and 
xenserver 6.2. Here is the logs from the management server and the catalina.out

http://pastebin.com/mS0wjug7

http://pastebin.com/gwVCitvb

I did fresh install with a single vm witg the db and management server on the 
same host just installed just fine. 

What should I do next ?

Re: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

[Nux!]

Looks like those pastes have been removed, please repost them.

(make sure to anonymise IPs, keys, usernames etc)

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Matthew Midgett" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 18 November, 2014 11:48:55
> Subject: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

> I tried again to update from 4.4.0 to 4.4.1 with centos management server and
> xenserver 6.2. Here is the logs from the management server and the 
> catalina.out
> 
> http://pastebin.com/mS0wjug7
> 
> http://pastebin.com/gwVCitvb
> 
> I did fresh install with a single vm witg the db and management server on the
> same host just installed just fine.
> 
> What should I do next ?

Re: Multiple local storage pools?

[Nux!]

I think there are more things tying into the DB for storage, merely adding one 
more entry would probably just break stuff. 
I wouldn't dare to try it, thanks for trying to help though. :-)

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Andrija Panic" 
> To: dev@cloudstack.apache.org
> Sent: Tuesday, 18 November, 2014 11:07:17
> Subject: Re: Multiple local storage pools?

> Might be a stupid suggestion - but adding one more record inside database,
> storage_pools table - would that work ?
> 
> 
> On 18 November 2014 10:04, Nux!  wrote:
> 
>> Thanks, Marcus.
>> I was eyeing shared mountpoint as well, but was hoping for something
>> better. I guess it will have to do.
>> Cheers :)
>>
>> --
>> Sent from the Delta quadrant using Borg technology!
>>
>> Nux!
>> www.nux.ro
>>
>> - Original Message -
>> > From: "Marcus" 
>> > To: dev@cloudstack.apache.org
>> > Sent: Tuesday, 18 November, 2014 07:27:13
>> > Subject: Re: Multiple local storage pools?
>>
>> > Not directly, but one way you could make this happen would be to create a
>> > 'sharedmountpoint' primary storage, and use tags to ensure that it
>> doesn't
>> > try to use that storage on any host other than the desired one.
>> > Sharedmountpoint is basically telling CS that you have done the work of
>> > mounting a clustered filesystem in a specific place.
>> >
>> > For example, set the host with tag 'host1', the storage with tag
>> > 'host1ssd', and create a service offering with both storage tag and host
>> > tag to select that combination. Kind of a hack, but it should work.
>> >
>> > On Mon, Nov 17, 2014 at 11:03 PM, Andrija Panic > >
>> > wrote:
>> >
>> >> Ok, waiting for someone to answer this :)
>> >>
>> >> Sent from Google Nexus 4
>> >> On Nov 18, 2014 12:35 AM, "Nux!"  wrote:
>> >>
>> >> > Andrija, but I want a more straight forward way to use the SSDs. ZFS
>> I'm
>> >> > sure it's awesome, but it would just complicate my setup. Thanks
>> though.
>> >> >
>> >> > --
>> >> > Sent from the Delta quadrant using Borg technology!
>> >> >
>> >> > Nux!
>> >> > www.nux.ro
>> >> >
>> >> > - Original Message -
>> >> > > From: "Andrija Panic" 
>> >> > > To: dev@cloudstack.apache.org
>> >> > > Sent: Monday, 17 November, 2014 19:07:15
>> >> > > Subject: Re: Multiple local storage pools?
>> >> >
>> >> > > Lucian, don't relly have the answer - but did you maybe consider
>> ZFS on
>> >> > > Linux, mounted on /var/lib/libvirt/images/ - there you can add SSD
>> for
>> >> > > writings and perhaps read cache (SLOG for writes and L2ARC for
>> >> readings)
>> >> > ?
>> >> > >
>> >> > > On 17 November 2014 19:37, Nux!  wrote:
>> >> > >
>> >> > >> Hi guys,
>> >> > >>
>> >> > >> Do you know how it would be possible to add multiple local primary
>> >> > storage
>> >> > >> pools? I know that if local storage is enabled
>> /var/lib/libvirt/images
>> >> > (in
>> >> > >> KVM's case) is added automatically.
>> >> > >> But assuming I want to add an SSD or SATA array to this host and
>> want
>> >> to
>> >> > >> differentiate it within cloudstack, how should I go about it?
>> >> > >>
>> >> > >> Lucian
>> >> > >>
>> >> > >> --
>> >> > >> Sent from the Delta quadrant using Borg technology!
>> >> > >>
>> >> > >> Nux!
>> >> > >> www.nux.ro
>> >> > >>
>> >> > >
>> >> > >
>> >> > >
>> >> > > --
>> >> > >
>> >> > > Andrija Panić
>> >> > > --
>> >> > >  http://admintweets.com
>> >> > > --
>> >> >
>>
> 
> 
> 
> --
> 
> Andrija Panić
> --
>  http://admintweets.com
> --

[GitHub] cloudstack-cloudmonkey pull request: keep colons in the request to...

[remibergsma]

Github user remibergsma commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/1#issuecomment-63460966
  
Created CLOUDSTACK-7935 for documentation purposes. See: 
https://issues.apache.org/jira/browse/CLOUDSTACK-7935

You can close this issue when you merge this change.

Thanks!
Remi


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

[Daniel Vega Simoes]

> On Nov. 17, 2014, 9:44 p.m., Brian Federle wrote:
> > I am getting conflicts in the properties file applying to the latest master 
> > (my HEAD is @ eba7cc78da751cc25994a5a736a87ce2e83c394e). There were a lot 
> > of changes lately to the localization strings, which is probably why it is 
> > conflicting; could you try updating the patch?
> > 
> > Applying: CLOUDSTACK-7915: Remove hard-coded values for Load Balancer 
> > algorithms in UI
> > error: patch failed: 
> > client/WEB-INF/classes/resources/messages_fr_FR.properties:1160
> > error: client/WEB-INF/classes/resources/messages_fr_FR.properties: patch 
> > does not apply
> > error: patch failed: 
> > client/WEB-INF/classes/resources/messages_ja_JP.properties:740
> > error: client/WEB-INF/classes/resources/messages_ja_JP.properties: patch 
> > does not apply
> > error: patch failed: 
> > client/WEB-INF/classes/resources/messages_zh_CN.properties:740
> > error: client/WEB-INF/classes/resources/messages_zh_CN.properties: patch 
> > does not apply
> > Patch failed at 0001 CLOUDSTACK-7915: Remove hard-coded values for Load 
> > Balancer algorithms in UI
> 
> Rajani Karuturi wrote:
> Brian, if you use git am -3, it will do a 3-way merge and can resolve the 
> conflicts.

I could produce a new diff for latest master, but it will be most likely 
outdated when you try to apply it.

Rajani's solution should work. Otherwise, it is possible to checkout the base 
commit for the patch (efe47b07044a863e5a34f48ef6e2468265925604), apply the 
patch and then merge with latest master.


- Daniel


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28043/#review61817
---


On Nov. 17, 2014, 6:26 p.m., Daniel Vega Simoes wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28043/
> ---
> 
> (Updated Nov. 17, 2014, 6:26 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> JIRA: CLOUDSTACK 7915
> 
> Removed hard-coded values for Load Balancer algorithms in UI. 
> 
> Instead, now UI sets values loaded dynamically through load balancer provider 
> capabilities.
> 
> Also updated internationalization messages.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 86eb5c2 
>   client/WEB-INF/classes/resources/messages_ar.properties 4f65118 
>   client/WEB-INF/classes/resources/messages_es.properties f2d754e 
>   client/WEB-INF/classes/resources/messages_fr_FR.properties 004187f 
>   client/WEB-INF/classes/resources/messages_it_IT.properties e2f3f0b 
>   client/WEB-INF/classes/resources/messages_ja_JP.properties 7bc90b5 
>   client/WEB-INF/classes/resources/messages_ko_KR.properties ce79d2e 
>   client/WEB-INF/classes/resources/messages_nb_NO.properties c169112 
>   client/WEB-INF/classes/resources/messages_nl_NL.properties 89ef828 
>   client/WEB-INF/classes/resources/messages_pl.properties 06d5ec2 
>   client/WEB-INF/classes/resources/messages_pt_BR.properties 8ee08ba 
>   client/WEB-INF/classes/resources/messages_ru_RU.properties ff68668 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties ebba5e0 
>   ui/dictionary.jsp 671f48f 
>   ui/scripts/network.js c27b999 
>   ui/scripts/vpc.js af19d87 
> 
> Diff: https://reviews.apache.org/r/28043/diff/
> 
> 
> Testing
> ---
> 
> Tests performed:
> - create new isolated network with DefaultIsolatedNetworkOffering, acquire 
> new IP, list load balancer algorithms (UI)
> - create new shared network with DefaultSharedNetworkWithSourceNat, acquire 
> new IP, list load balancer algorithms (UI)
> 
> It should work correctly as long as network offering is configured with LB 
> provider/capabilities. Otherwise, list of algorithms is empty in UI.
> 
> 
> Thanks,
> 
> Daniel Vega Simoes
> 
>

Review Request 28168: public_ip type resource count for an account is not decremented upon IP range deletion

[Damodar Reddy Talakanti]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28168/
---

Review request for cloudstack, bharat kumar and Kishan Kavala.


Repository: cloudstack-git


Description
---

When deleting the IP range which is associated to an account the resource count 
for public_ip is not decremented accordingly which is causing not to add any 
new ranges to that account further once we reach max limit.

Repro Steps.
-
1. Add an IP range and associate it to a particular account. This will 
increment your resource count for public_ip to that range count.
2. Now try to delete this range and check the resource count for public_ip of 
that account. it will not be decreased.


Diffs
-

  server/src/com/cloud/configuration/ConfigurationManagerImpl.java 2cb4203 

Diff: https://reviews.apache.org/r/28168/diff/


Testing
---

Tested on latest master with Xen Hypervisor.

Tested with acquiring atleast an IP from that account and with out acquiring 
any IPs from that account.


Thanks,

Damodar Reddy Talakanti

Build failed in Jenkins: cloudstack-4.3-maven-build #632

[jenkins]

See 

Changes:

[Rohit Yadav] Revert "Updating pom.xml version numbers for release 
4.3.2-SNAPSHOT"

--
Started by an SCM change
[EnvInject] - Loading node environment variables.
Building remotely on cloudstack-buildslave-centos6-87d 
(cloudstack-buildslave-centos6) in workspace 

 > /usr/bin/git rev-parse --is-inside-work-tree # timeout=400
Fetching changes from the remote Git repository
 > /usr/bin/git config remote.origin.url git://git.apache.org/cloudstack.git # 
 > timeout=400
Fetching upstream changes from git://git.apache.org/cloudstack.git
 > /usr/bin/git --version # timeout=400
 > /usr/bin/git fetch --tags --progress git://git.apache.org/cloudstack.git 
 > +refs/heads/*:refs/remotes/origin/*
 > /usr/bin/git rev-parse origin/4.3^{commit} # timeout=400
Checking out Revision e6d00bcc6670d77144e6d423091808877eaa610f (origin/4.3)
 > /usr/bin/git config core.sparsecheckout # timeout=400
 > /usr/bin/git checkout -f e6d00bcc6670d77144e6d423091808877eaa610f
 > /usr/bin/git rev-list e615f218a98a499a63c86aedf4cfb55dd3776473 # timeout=400
FATAL: Couldn’t find any executable in /opt/apache-maven-3.0.5
Build step 'Invoke top-level Maven targets' marked build as failure

Jenkins build is back to stable : simulator-singlerun #650

[jenkins]

See 

Re: Review Request 27979: CLOUDSTACK-7903: Decrease minimal usage aggregation range value

[Ilia Shakitko]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27979/
---

(Updated Nov. 18, 2014, 1:44 p.m.)


Review request for cloudstack, Kishan Kavala, Rohit Yadav, Simon Fiddaman, 
Stephen Hoogendijk, and Wei Zhou.


Changes
---

1. Maintained latest changed in master. 
2. Removed constant declaration duplication, and decreased the limit (as 
suggested in feedback) to 1.


Bugs: CLOUDSTACK-7903
https://issues.apache.org/jira/browse/CLOUDSTACK-7903


Repository: cloudstack-git


Description (updated)
---

When you want to build 95-percentile billing model for the network traffic 
usage, it is good to have 5-minutes (or 1-minute) usage samples instead of 10 
(which is minimum allowed in ACS at the moment). Let admin to control it.
This change decreases the lower limit for USAGE_AGGREGATION_RANGE_MIN 
parameter, making it minimum possible positive integer. It is also removes 
declaration of the same constant in 3 places.

Will be happy if patch can be:
- cherry-picked to the 4.4 and 4.5 branches
- included to the 4.4.2 tag.


Diffs (updated)
-

  server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
23ddfb1 
  server/src/com/cloud/server/StatsCollector.java 1fcaa62 
  usage/src/com/cloud/usage/UsageManagerImpl.java fe7ee1e 

Diff: https://reviews.apache.org/r/27979/diff/


Testing (updated)
---

Tested on master, 4.4 and 4.5 branches. But then during the review time I had 
to re-prepare the patch and it become compatible only for master branch.

If you accept the patch, could you also cherry-pick my commit to 4.4 and 4.5 
branches? (there will be 1 one-line conflict to resolve)


Thanks,

Ilia Shakitko

RE: Review Request 27979: CLOUDSTACK-7903: Decrease minimal usage aggregation range value

[Ilia Shakitko]

Hi Rohit,

It's a week almost w/o an answer from Kishan. Meanwhile, I've redone the patch, 
removing some duplications (found 2 more places with same constant declared). 
And decreased the limit to 1 (minimal logical range limit). So it was a good 
feedback from you and Simon to actually let admins to decide. Maybe I don't 
need less than 5, but someone will need 1 or 2.

Could you please review it? Would be also cool to apply it to 4.4 and 4.5 (I 
wrote a remark in the review).


Kind regards,

Ilia Shakitko
Innovation Engineer
LeaseWeb Technologies B.V.

T: +31 20 316 0235
M:
E: i.shaki...@tech.leaseweb.com
W: http://www.leaseweb.com

Luttenbergweg 8, 1101 EC Amsterdam, Netherlands


-Original Message-
From: Rohit Yadav [mailto:nore...@reviews.apache.org] On Behalf Of Rohit Yadav
Sent: donderdag, 13 november, 2014 11:11
To: Kishan Kavala; Wei Zhou; Rohit Yadav
Cc: Ilia Shakitko; cloudstack
Subject: Re: Review Request 27979: CLOUDSTACK-7903: Decrease minimal usage 
aggregation range value


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27979/#review61224
---


I've added Kishan to this as well. Though it looks good and we can surely 
accept it. I think it makes sense to either set it to 1 or have a global 
configuration added which makes it easy for sysadmins to set a value they want 
instead of hard coding it.

- Rohit Yadav


On Nov. 13, 2014, 10:09 a.m., Ilia Shakitko wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27979/
> ---
>
> (Updated Nov. 13, 2014, 10:09 a.m.)
>
>
> Review request for cloudstack, Kishan Kavala, Rohit Yadav, and Wei Zhou.
>
>
> Bugs: CLOUDSTACK-7903
> https://issues.apache.org/jira/browse/CLOUDSTACK-7903
>
>
> Repository: cloudstack-git
>
>
> Description
> ---
>
> When you want to build 95-percentile billing model for the network traffic 
> usage, it is good to have 5-minutes usage samples instead of 10 (which is 
> minimum allowed in ACS at the moment).
> This change decreases the lower limit for USAGE_AGGREGATION_RANGE_MIN 
> parameter.
>
> Will be happy if patch can be included to the 4.4.2 tag.
>
>
> Diffs
> -
>
>   usage/src/com/cloud/usage/UsageManagerImpl.java fe7ee1e
>
> Diff: https://reviews.apache.org/r/27979/diff/
>
>
> Testing
> ---
>
> Tested on both 4.4 and master branches.
>
>
> Thanks,
>
> Ilia Shakitko
>
>

No event publish can be wrapped within db transaction...why?

[Leo Simons]

Hi Min, hi Koushik,

Cloudstack is shouting at me:
NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!

(full stack trace below). I've learned this is happening on our 
systemvm-persistent-config feature branch because it has commit 
ffaabdc13fde0f0f7b2667a483006e2a4b805f63 but it does not have commit 
f585dd266188a134a9c8b911376b066b9d3806e8 yet.

I'm now trying to understand what's happening here -- the transaction / 
concurrency / messaging logic gave me significant headache with its triple 
negatives, nested transaction scoping and home-grown gates, but I think I got 
it now.

As I understand it, in the olde world, creating an account:

  * opens a database transaction
  * creates an account in the db
  * creates the first user in that account in the db
  * publishes an event
* which is listened to by 0 subscribers
  * commmits the database transaction
  * check the user is there
* opens a database transaction
* find the created user in the database
* (auto)closes transaction
* returns success if the user is in the db

this, err, works, but in some other cases, apparently, there are concerns that 
the db transaction is open too long while message handling happens. So that's 
why the warning was added, and follow up on, and so now, creating an account:

  * opens a database transaction
  * creates an account in the db
  * creates the first user in that account in the db
  * commmits the database transaction
  * publishes an event
* which is still listened to by on average 0 subscribers,
  but there could be an IAM subscriber
  * check the user is there
* opens a database transaction
* find the created user in the database
* (auto)closes transaction
* returns success if the user is in the db

The one possible subscriber for account creation is IAMApiServiceImpl, which 
when receiving the event

  * opens a database transaction
  * adds the account to acl_group_account_map
  * commits the database transaction
  * finds the domain for the account
* opens a database transaction
* finds the domain for the account
* (auto)closes transaction
  * finds the domain groups for the domain
* opens a database transaction
* finds the domain groups for the domain
* (auto)closes transaction
  * for each domain group
* opens a database transaction
* adds the account to acl_group_account_map
* commits the database transaction

in other words, if there's 1 domain group and an enabled IAM thingie, this 
spreads out "make an account" over 6 transactions. Without IAM thingie its 2 
two transactions with a no-op message bus thingie in the middle. Is that 
correct?

If so, I don't understand this at all. The pre-November code doesn't make that 
much sense to me (why query the database? If you don't trust your database its 
ACID guarantees...why use transactions? Why do we ever need a message bus 
between two java components in the same classloader?), but the new code scares 
me.

In the case of errors in between transactions, you can end up with accounts 
that are not in all the groups they should be in. I imagine I would much rather 
see the whole thing fail, and the complete api call fail, so that I can re-try 
it as a whole, than end up with a somehow half-initialized account. I.e. have 
everything account-management-y happen in one transaction which is rolled back 
on any failure.

Any thoughts?


Thanks!


Leo (who can't ask Hugo since Hugo is at apachecon/ccceu and he isn't :))


2014-11-18 13:36:33,145 ERROR [o.a.c.f.m.MessageBusBase] 
(qtp1734055321-25:ctx-05df2079 ctx-25ea4461 ctx-3aac3268)
  NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!
  com.cloud.utils.exception.CloudRuntimeException:
  NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!
at 
org.apache.cloudstack.framework.messagebus.MessageBusBase.publish(MessageBusBase.java:167)
at 
com.cloud.user.AccountManagerImpl$2.doInTransaction(AccountManagerImpl.java:1052)
at 
com.cloud.user.AccountManagerImpl$2.doInTransaction(AccountManagerImpl.java:1027)
at com.cloud.utils.db.Transaction$2.doInTransaction(Transaction.java:57)
at com.cloud.utils.db.Transaction.execute(Transaction.java:45)
at com.cloud.utils.db.Transaction.execute(Transaction.java:54)
at 
com.cloud.user.AccountManagerImpl.createUserAccount(AccountManagerImpl.java:1027)
at sun.reflect.GeneratedMethodAccessor181.invoke(Unknown Source)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
or

Re: Review Request 27979: CLOUDSTACK-7903: Decrease minimal usage aggregation range value

[Simon Fiddaman]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27979/#review61917
---

Ship it!


Ship It!

- Simon Fiddaman


On Nov. 18, 2014, 1:44 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27979/
> ---
> 
> (Updated Nov. 18, 2014, 1:44 p.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala, Rohit Yadav, Simon Fiddaman, 
> Stephen Hoogendijk, and Wei Zhou.
> 
> 
> Bugs: CLOUDSTACK-7903
> https://issues.apache.org/jira/browse/CLOUDSTACK-7903
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> When you want to build 95-percentile billing model for the network traffic 
> usage, it is good to have 5-minutes (or 1-minute) usage samples instead of 10 
> (which is minimum allowed in ACS at the moment). Let admin to control it.
> This change decreases the lower limit for USAGE_AGGREGATION_RANGE_MIN 
> parameter, making it minimum possible positive integer. It is also removes 
> declaration of the same constant in 3 places.
> 
> Will be happy if patch can be:
> - cherry-picked to the 4.4 and 4.5 branches
> - included to the 4.4.2 tag.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
> 23ddfb1 
>   server/src/com/cloud/server/StatsCollector.java 1fcaa62 
>   usage/src/com/cloud/usage/UsageManagerImpl.java fe7ee1e 
> 
> Diff: https://reviews.apache.org/r/27979/diff/
> 
> 
> Testing
> ---
> 
> Tested on master, 4.4 and 4.5 branches. But then during the review time I had 
> to re-prepare the patch and it become compatible only for master branch.
> 
> If you accept the patch, could you also cherry-pick my commit to 4.4 and 4.5 
> branches? (there will be 1 one-line conflict to resolve)
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>

[GitHub] cloudstack-cloudmonkey pull request: keep colons in the request to...

[bhaisaab]

Github user bhaisaab commented on the pull request:


https://github.com/apache/cloudstack-cloudmonkey/pull/1#issuecomment-63473873
  
LGTM, 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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

RE: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

[Matthew Midgett]

Thanks Nux I realized that after I posted but had to leave the office. 

As I side note I did the upgrade again yesterday and it failed at starting the 
bridge networking.  I dropped the DB and set it up like a fresh install and 
that worked. I uploaded the upgraded db and it still failed to start so I 
decided to diff them. I then ran win merge and merged a  lot of lines and it 
started again. My network was not  showing correctly and that made since, since 
it would fail at the part. I spent most of the day yesterday mucking around the 
db before I said screw it and rolled the upgrade back. I am 100% sure that the 
problem is in the db upgrade. What I really need it is to give someone my pre 
update db and my post update db and let them run compare the two.  I am sure 
that a developer can help me. There was another user that was posting on my 
original email on the user list and his management server was failing at the 
same spot so it might not be related to just me.

My Db's of course contain sensitive information and I don't want to just add a 
link for everyone to see. What is the best practice for this? 


CONFIDENTIAL: This electronic mail, including any attachment(s) may contain 
information that is privileged, confidential, and/or otherwise protected from 
disclosure to anyone other than its intended recipient(s).  Any dissemination 
or use of this electronic email or its contents (including any attachments) by 
persons other than the intended recipient(s) is strictly prohibited.  If you 
have received this message in error, please notify me immediately by reply 
email so that I may correct my internal records.  Please then delete the 
original message (including any attachments) in its entirety.  Thank you.

-Original Message-
From: Matthew Midgett [mailto:clouds...@trick-solutions.com.INVALID] 
Sent: Tuesday, November 18, 2014 6:49 AM
To: dev@cloudstack.apache.org
Subject: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

I tried again to update from 4.4.0 to 4.4.1 with centos management server and 
xenserver 6.2. Here is the logs from the management server and the catalina.out

http://pastebin.com/mS0wjug7

http://pastebin.com/gwVCitvb

I did fresh install with a single vm witg the db and management server on the 
same host just installed just fine. 

What should I do next ?

Re: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

[Daan Hoogland]

Matthew, How big is the diff?

On Tue, Nov 18, 2014 at 3:12 PM, Matthew Midgett
 wrote:
> Thanks Nux I realized that after I posted but had to leave the office.
>
> As I side note I did the upgrade again yesterday and it failed at starting 
> the bridge networking.  I dropped the DB and set it up like a fresh install 
> and that worked. I uploaded the upgraded db and it still failed to start so I 
> decided to diff them. I then ran win merge and merged a  lot of lines and it 
> started again. My network was not  showing correctly and that made since, 
> since it would fail at the part. I spent most of the day yesterday mucking 
> around the db before I said screw it and rolled the upgrade back. I am 100% 
> sure that the problem is in the db upgrade. What I really need it is to give 
> someone my pre update db and my post update db and let them run compare the 
> two.  I am sure that a developer can help me. There was another user that was 
> posting on my original email on the user list and his management server was 
> failing at the same spot so it might not be related to just me.
>
> My Db's of course contain sensitive information and I don't want to just add 
> a link for everyone to see. What is the best practice for this?
>
>
> CONFIDENTIAL: This electronic mail, including any attachment(s) may contain 
> information that is privileged, confidential, and/or otherwise protected from 
> disclosure to anyone other than its intended recipient(s).  Any dissemination 
> or use of this electronic email or its contents (including any attachments) 
> by persons other than the intended recipient(s) is strictly prohibited.  If 
> you have received this message in error, please notify me immediately by 
> reply email so that I may correct my internal records.  Please then delete 
> the original message (including any attachments) in its entirety.  Thank you.
>
> -Original Message-
> From: Matthew Midgett [mailto:clouds...@trick-solutions.com.INVALID]
> Sent: Tuesday, November 18, 2014 6:49 AM
> To: dev@cloudstack.apache.org
> Subject: Advised to send to Dev list - Broken update from 4.4 to 4.4.1
>
> I tried again to update from 4.4.0 to 4.4.1 with centos management server and 
> xenserver 6.2. Here is the logs from the management server and the 
> catalina.out
>
> http://pastebin.com/mS0wjug7
>
> http://pastebin.com/gwVCitvb
>
> I did fresh install with a single vm witg the db and management server on the 
> same host just installed just fine.
>
> What should I do next ?
>
>
>



-- 
Daan

[GitHub] cloudstack-cloudmonkey pull request: keep colons in the request to...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack-cloudmonkey/pull/1


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

RE: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

[Matthew Midgett]

It’s a few lines and Dan If you would like to view them I can send them to you. 
The problem lies in the network part.

CONFIDENTIAL: This electronic mail, including any attachment(s) may contain 
information that is privileged, confidential, and/or otherwise protected from 
disclosure to anyone other than its intended recipient(s).  Any dissemination 
or use of this electronic email or its contents (including any attachments) by 
persons other than the intended recipient(s) is strictly prohibited.  If you 
have received this message in error, please notify me immediately by reply 
email so that I may correct my internal records.  Please then delete the 
original message (including any attachments) in its entirety.  Thank you.

-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] 
Sent: Tuesday, November 18, 2014 9:20 AM
To: dev
Subject: Re: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

Matthew, How big is the diff?

On Tue, Nov 18, 2014 at 3:12 PM, Matthew Midgett 
 wrote:
> Thanks Nux I realized that after I posted but had to leave the office.
>
> As I side note I did the upgrade again yesterday and it failed at starting 
> the bridge networking.  I dropped the DB and set it up like a fresh install 
> and that worked. I uploaded the upgraded db and it still failed to start so I 
> decided to diff them. I then ran win merge and merged a  lot of lines and it 
> started again. My network was not  showing correctly and that made since, 
> since it would fail at the part. I spent most of the day yesterday mucking 
> around the db before I said screw it and rolled the upgrade back. I am 100% 
> sure that the problem is in the db upgrade. What I really need it is to give 
> someone my pre update db and my post update db and let them run compare the 
> two.  I am sure that a developer can help me. There was another user that was 
> posting on my original email on the user list and his management server was 
> failing at the same spot so it might not be related to just me.
>
> My Db's of course contain sensitive information and I don't want to just add 
> a link for everyone to see. What is the best practice for this?
>
>
> CONFIDENTIAL: This electronic mail, including any attachment(s) may contain 
> information that is privileged, confidential, and/or otherwise protected from 
> disclosure to anyone other than its intended recipient(s).  Any dissemination 
> or use of this electronic email or its contents (including any attachments) 
> by persons other than the intended recipient(s) is strictly prohibited.  If 
> you have received this message in error, please notify me immediately by 
> reply email so that I may correct my internal records.  Please then delete 
> the original message (including any attachments) in its entirety.  Thank you.
>
> -Original Message-
> From: Matthew Midgett [mailto:clouds...@trick-solutions.com.INVALID]
> Sent: Tuesday, November 18, 2014 6:49 AM
> To: dev@cloudstack.apache.org
> Subject: Advised to send to Dev list - Broken update from 4.4 to 4.4.1
>
> I tried again to update from 4.4.0 to 4.4.1 with centos management 
> server and xenserver 6.2. Here is the logs from the management server 
> and the catalina.out
>
> http://pastebin.com/mS0wjug7
>
> http://pastebin.com/gwVCitvb
>
> I did fresh install with a single vm witg the db and management server on the 
> same host just installed just fine.
>
> What should I do next ?
>
>
>



--
Daan

Re: Advised to send to Dev list - Broken update from 4.4 to 4.4.1

[Daan Hoogland]

please sand them to daan hoogland on gmail if it needs to be private

On Tue, Nov 18, 2014 at 4:10 PM, Matthew Midgett
 wrote:
> It’s a few lines and Dan If you would like to view them I can send them to 
> you. The problem lies in the network part.
>
> CONFIDENTIAL: This electronic mail, including any attachment(s) may contain 
> information that is privileged, confidential, and/or otherwise protected from 
> disclosure to anyone other than its intended recipient(s).  Any dissemination 
> or use of this electronic email or its contents (including any attachments) 
> by persons other than the intended recipient(s) is strictly prohibited.  If 
> you have received this message in error, please notify me immediately by 
> reply email so that I may correct my internal records.  Please then delete 
> the original message (including any attachments) in its entirety.  Thank you.
>
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Tuesday, November 18, 2014 9:20 AM
> To: dev
> Subject: Re: Advised to send to Dev list - Broken update from 4.4 to 4.4.1
>
> Matthew, How big is the diff?
>
> On Tue, Nov 18, 2014 at 3:12 PM, Matthew Midgett 
>  wrote:
>> Thanks Nux I realized that after I posted but had to leave the office.
>>
>> As I side note I did the upgrade again yesterday and it failed at starting 
>> the bridge networking.  I dropped the DB and set it up like a fresh install 
>> and that worked. I uploaded the upgraded db and it still failed to start so 
>> I decided to diff them. I then ran win merge and merged a  lot of lines and 
>> it started again. My network was not  showing correctly and that made since, 
>> since it would fail at the part. I spent most of the day yesterday mucking 
>> around the db before I said screw it and rolled the upgrade back. I am 100% 
>> sure that the problem is in the db upgrade. What I really need it is to give 
>> someone my pre update db and my post update db and let them run compare the 
>> two.  I am sure that a developer can help me. There was another user that 
>> was posting on my original email on the user list and his management server 
>> was failing at the same spot so it might not be related to just me.
>>
>> My Db's of course contain sensitive information and I don't want to just add 
>> a link for everyone to see. What is the best practice for this?
>>
>>
>> CONFIDENTIAL: This electronic mail, including any attachment(s) may contain 
>> information that is privileged, confidential, and/or otherwise protected 
>> from disclosure to anyone other than its intended recipient(s).  Any 
>> dissemination or use of this electronic email or its contents (including any 
>> attachments) by persons other than the intended recipient(s) is strictly 
>> prohibited.  If you have received this message in error, please notify me 
>> immediately by reply email so that I may correct my internal records.  
>> Please then delete the original message (including any attachments) in its 
>> entirety.  Thank you.
>>
>> -Original Message-
>> From: Matthew Midgett [mailto:clouds...@trick-solutions.com.INVALID]
>> Sent: Tuesday, November 18, 2014 6:49 AM
>> To: dev@cloudstack.apache.org
>> Subject: Advised to send to Dev list - Broken update from 4.4 to 4.4.1
>>
>> I tried again to update from 4.4.0 to 4.4.1 with centos management
>> server and xenserver 6.2. Here is the logs from the management server
>> and the catalina.out
>>
>> http://pastebin.com/mS0wjug7
>>
>> http://pastebin.com/gwVCitvb
>>
>> I did fresh install with a single vm witg the db and management server on 
>> the same host just installed just fine.
>>
>> What should I do next ?
>>
>>
>>
>
>
>
> --
> Daan
>



-- 
Daan

Re: git commit: updated refs/heads/4.3 to e6d00bc

[Rohit Yadav]

This broke deploydb and since we may not get time today/tomorrow I had to 
revert. Our 4.3 release manager is well informed about this.

I hope when we change the version we should make sure CloudStack builds, and 
normal operations such as deploydb should work. We should re-revert this once 
we’ve an upgrade path, the code builds and deploydb too.

> On 18-Nov-2014, at 4:15 pm, Daan Hoogland  wrote:
>
> 4.3.1 is out. is this really what we want, Rohit? seems this revert
> shouldn't happen.

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Software 
Engineering
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: git commit: updated refs/heads/4.3 to e6d00bc

[Daan Hoogland]

This is why I introduced the branch on rc policy. I discussed it with
David this morning and we concluded that it is missing a merge back.
That doesn't mean that on each upgrade an emtpy upgrade path is
needed.

On Tue, Nov 18, 2014 at 5:11 PM, Rohit Yadav  wrote:
> This broke deploydb and since we may not get time today/tomorrow I had to 
> revert. Our 4.3 release manager is well informed about this.
>
> I hope when we change the version we should make sure CloudStack builds, and 
> normal operations such as deploydb should work. We should re-revert this once 
> we’ve an upgrade path, the code builds and deploydb too.
>
>> On 18-Nov-2014, at 4:15 pm, Daan Hoogland  wrote:
>>
>> 4.3.1 is out. is this really what we want, Rohit? seems this revert
>> shouldn't happen.
>
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +91 88 262 30892 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
>
>
>
> Find out more about ShapeBlue and our range of CloudStack related services
>
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software 
> Engineering
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
>
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.



-- 
Daan

Jenkins build became unstable: simulator-singlerun #653

[jenkins]

See 

Review Request 28179: CLOUDSTACK-7928 : Fixed the script 'test_vpc_vm_life_cycle.py' - Removed the Invalid test cases for Stopped VPC VR Scenario

[Chandan Purushothama]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28179/
---

Review request for cloudstack and sangeetha hariharan.


Bugs: CLOUDSTACK-7928
https://issues.apache.org/jira/browse/CLOUDSTACK-7928


Repository: cloudstack-git


Description
---

Following Test Cases currently fail in TestVMLifeCycleStoppedVPCVR Test Suite:

test_07_migrate_instance_in_network
test_08_user_data
test_09_meta_data
test_10_expunge_instance_in_network

The test cases fail for the obvious reason since the VPC VR is stopped. The 
Stopped VPCVR doesnt allow the traffic to or from the Guest VMs. Hence the test 
cases are not valid to be tested in such a scenario and should be removed.

Removed the test cases that are failing due to invalid scenario


Diffs
-

  test/integration/component/test_vpc_vm_life_cycle.py 7a1fd8a 

Diff: https://reviews.apache.org/r/28179/diff/


Testing
---

Testing not done


Thanks,

Chandan Purushothama

Jenkins build is still unstable: simulator-singlerun #654

[jenkins]

See 

Re: No event publish can be wrapped within db transaction...why?

[Min Chen]

Hi Leo,

"NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!" is along the
same line as "NO AGENT COMMAND CAN BE WRAPPED WITHIN DB TRANSACTION!". The
rationale behind this is simple: event subscriber execution or agent
command handling at resource layer may take too long, and we don't want to
have that long transaction window to hold DB for too long.
As for your questions about why we bother to use message bus to
communicate between two java component, there is a reason for it: loose
coupling. IAMApiServiceImpl is a class in IAM plugin service, which can be
deployed as a totally different service from CloudStack management server
and ideally with future 3rd-party authentication/authorization
integration, they may use a totally different database from "cloud"
database we are currently using just for simplicity. In this deployment
architecture, we have to make sure that this IAM service and CloudStack MS
components are loosely coupled. Message bus provided us a very good
approach to achieve that.
As you said, ideally we would like to achieve a prefect transaction
related to account creation in both CloudStack main component and its
plugin services, but in reality, this may not work always and big
transaction will be error-prone for large scale distributed systems,
especially for this loosely coupled components that are crossing different
DBs. The plugin architecture in CloudStack is designed to easily
enable/disable each plugin component without impacting too much on main
CloudStack components. So in this case, I would personally prefer that we
should make sure of data integrity in the scope of CloudStack main
components first and handle potential message handling failure in plugin
module separately through application level logic.

Thanks
-min



On 11/18/14 5:52 AM, "Leo Simons"  wrote:

>Hi Min, hi Koushik,
>
>Cloudstack is shouting at me:
>NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!
>
>(full stack trace below). I've learned this is happening on our
>systemvm-persistent-config feature branch because it has commit
>ffaabdc13fde0f0f7b2667a483006e2a4b805f63 but it does not have commit
>f585dd266188a134a9c8b911376b066b9d3806e8 yet.
>
>I'm now trying to understand what's happening here -- the transaction /
>concurrency / messaging logic gave me significant headache with its
>triple negatives, nested transaction scoping and home-grown gates, but I
>think I got it now.
>
>As I understand it, in the olde world, creating an account:
>
>  * opens a database transaction
>  * creates an account in the db
>  * creates the first user in that account in the db
>  * publishes an event
>* which is listened to by 0 subscribers
>  * commmits the database transaction
>  * check the user is there
>* opens a database transaction
>* find the created user in the database
>* (auto)closes transaction
>* returns success if the user is in the db
>
>this, err, works, but in some other cases, apparently, there are concerns
>that the db transaction is open too long while message handling happens.
>So that's why the warning was added, and follow up on, and so now,
>creating an account:
>
>  * opens a database transaction
>  * creates an account in the db
>  * creates the first user in that account in the db
>  * commmits the database transaction
>  * publishes an event
>* which is still listened to by on average 0 subscribers,
>  but there could be an IAM subscriber
>  * check the user is there
>* opens a database transaction
>* find the created user in the database
>* (auto)closes transaction
>* returns success if the user is in the db
>
>The one possible subscriber for account creation is IAMApiServiceImpl,
>which when receiving the event
>
>  * opens a database transaction
>  * adds the account to acl_group_account_map
>  * commits the database transaction
>  * finds the domain for the account
>* opens a database transaction
>* finds the domain for the account
>* (auto)closes transaction
>  * finds the domain groups for the domain
>* opens a database transaction
>* finds the domain groups for the domain
>* (auto)closes transaction
>  * for each domain group
>* opens a database transaction
>* adds the account to acl_group_account_map
>* commits the database transaction
>
>in other words, if there's 1 domain group and an enabled IAM thingie,
>this spreads out "make an account" over 6 transactions. Without IAM
>thingie its 2 two transactions with a no-op message bus thingie in the
>middle. Is that correct?
>
>If so, I don't understand this at all. The pre-November code doesn't make
>that much sense to me (why query the database? If you don't trust your
>database its ACID guarantees...why use transactions? Why do we ever need
>a message bus between two java components in the same classloader?), but
>the new code scares me.
>
>In the case of errors in between transactions, you can end up with
>accounts

Re: No event publish can be wrapped within db transaction...why?

[Leo Simons]

Hi Min,

Thanks for a very clear answer!

However, I'm afraid I still don't get it :-). So...

...do you have any specific example or use case of an external IAM service to 
integrate with? Is there some kind of design document for me to understand the 
goals?

I ask because all the ones that I'm familiar with tend to assume that the owner 
of identity information (and grouping, and possibly other kinds of AAA 
assertions) is externalized from systems like cloudstack to the identity 
system, i.e. integration is "the other way around".

So i.e. you would have AD or other LDAP or an SSO server or a SAML 
implementation (or all of those...), where systems like cloudstack then 
delegate AAA questions/assertions to those systems, rather than propagating 
local identities to that central system.

I imagine if you have an external identity provider, you plug in a different 
implementation of AccountManager (LDAPAccountManager? etc.), and then a 
CreateAccountCmd would fail with an error saying the server is configured to 
use <> so account creation is unsupported.


cheers!


Leo


On Nov 18, 2014, at 7:50 PM, Min Chen  wrote:
> Hi Leo,
> 
>   "NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!" is along the
> same line as "NO AGENT COMMAND CAN BE WRAPPED WITHIN DB TRANSACTION!". The
> rationale behind this is simple: event subscriber execution or agent
> command handling at resource layer may take too long, and we don't want to
> have that long transaction window to hold DB for too long.
>   As for your questions about why we bother to use message bus to
> communicate between two java component, there is a reason for it: loose
> coupling. IAMApiServiceImpl is a class in IAM plugin service, which can be
> deployed as a totally different service from CloudStack management server
> and ideally with future 3rd-party authentication/authorization
> integration, they may use a totally different database from "cloud"
> database we are currently using just for simplicity. In this deployment
> architecture, we have to make sure that this IAM service and CloudStack MS
> components are loosely coupled. Message bus provided us a very good
> approach to achieve that.
>   As you said, ideally we would like to achieve a prefect transaction
> related to account creation in both CloudStack main component and its
> plugin services, but in reality, this may not work always and big
> transaction will be error-prone for large scale distributed systems,
> especially for this loosely coupled components that are crossing different
> DBs. The plugin architecture in CloudStack is designed to easily
> enable/disable each plugin component without impacting too much on main
> CloudStack components. So in this case, I would personally prefer that we
> should make sure of data integrity in the scope of CloudStack main
> components first and handle potential message handling failure in plugin
> module separately through application level logic.
> 
>   Thanks
>   -min
> 
> 
> 
> On 11/18/14 5:52 AM, "Leo Simons"  wrote:
> 
>> Hi Min, hi Koushik,
>> 
>> Cloudstack is shouting at me:
>>   NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!
>> 
>> (full stack trace below). I've learned this is happening on our
>> systemvm-persistent-config feature branch because it has commit
>> ffaabdc13fde0f0f7b2667a483006e2a4b805f63 but it does not have commit
>> f585dd266188a134a9c8b911376b066b9d3806e8 yet.
>> 
>> I'm now trying to understand what's happening here -- the transaction /
>> concurrency / messaging logic gave me significant headache with its
>> triple negatives, nested transaction scoping and home-grown gates, but I
>> think I got it now.
>> 
>> As I understand it, in the olde world, creating an account:
>> 
>> * opens a database transaction
>> * creates an account in the db
>> * creates the first user in that account in the db
>> * publishes an event
>>   * which is listened to by 0 subscribers
>> * commmits the database transaction
>> * check the user is there
>>   * opens a database transaction
>>   * find the created user in the database
>>   * (auto)closes transaction
>>   * returns success if the user is in the db
>> 
>> this, err, works, but in some other cases, apparently, there are concerns
>> that the db transaction is open too long while message handling happens.
>> So that's why the warning was added, and follow up on, and so now,
>> creating an account:
>> 
>> * opens a database transaction
>> * creates an account in the db
>> * creates the first user in that account in the db
>> * commmits the database transaction
>> * publishes an event
>>   * which is still listened to by on average 0 subscribers,
>> but there could be an IAM subscriber
>> * check the user is there
>>   * opens a database transaction
>>   * find the created user in the database
>>   * (auto)closes transaction
>>   * returns success if the user is in the db
>> 
>> The one possible subscriber for account creation is IAMApiServiceImpl,
>>

Jenkins build is still unstable: simulator-singlerun #655

[jenkins]

See 

Re: No event publish can be wrapped within db transaction...why?

[Min Chen]

For IAM feature design goal, you can take a look at our FS at
https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+Identity+
and+Access+Management+%28IAM%29+Plugin. Also Prachi and I presented this
work at Denver Apache CloudStack Collab conference, you can also view our
slideshare at 
http://events.linuxfoundation.org/sites/events/files/slides/ApachIAM.pdf
and view the recorded presentation at
https://www.youtube.com/watch?v=iUThjMl2yl8&list=PLU2OcwpQkYCyPx_cwJxyOK0YK
SM86Mj9n&index=24.

Hope that those pointers can provide some help.

-min

On 11/18/14 11:26 AM, "Leo Simons"  wrote:

>Hi Min,
>
>Thanks for a very clear answer!
>
>However, I'm afraid I still don't get it :-). So...
>
>...do you have any specific example or use case of an external IAM
>service to integrate with? Is there some kind of design document for me
>to understand the goals?
>
>I ask because all the ones that I'm familiar with tend to assume that the
>owner of identity information (and grouping, and possibly other kinds of
>AAA assertions) is externalized from systems like cloudstack to the
>identity system, i.e. integration is "the other way around".
>
>So i.e. you would have AD or other LDAP or an SSO server or a SAML
>implementation (or all of those...), where systems like cloudstack then
>delegate AAA questions/assertions to those systems, rather than
>propagating local identities to that central system.
>
>I imagine if you have an external identity provider, you plug in a
>different implementation of AccountManager (LDAPAccountManager? etc.),
>and then a CreateAccountCmd would fail with an error saying the server is
>configured to use <> so account creation is unsupported.
>
>
>cheers!
>
>
>Leo
>
>
>On Nov 18, 2014, at 7:50 PM, Min Chen  wrote:
>> Hi Leo,
>> 
>>  "NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!" is along the
>> same line as "NO AGENT COMMAND CAN BE WRAPPED WITHIN DB TRANSACTION!".
>>The
>> rationale behind this is simple: event subscriber execution or agent
>> command handling at resource layer may take too long, and we don't want
>>to
>> have that long transaction window to hold DB for too long.
>>  As for your questions about why we bother to use message bus to
>> communicate between two java component, there is a reason for it: loose
>> coupling. IAMApiServiceImpl is a class in IAM plugin service, which can
>>be
>> deployed as a totally different service from CloudStack management
>>server
>> and ideally with future 3rd-party authentication/authorization
>> integration, they may use a totally different database from "cloud"
>> database we are currently using just for simplicity. In this deployment
>> architecture, we have to make sure that this IAM service and CloudStack
>>MS
>> components are loosely coupled. Message bus provided us a very good
>> approach to achieve that.
>>  As you said, ideally we would like to achieve a prefect transaction
>> related to account creation in both CloudStack main component and its
>> plugin services, but in reality, this may not work always and big
>> transaction will be error-prone for large scale distributed systems,
>> especially for this loosely coupled components that are crossing
>>different
>> DBs. The plugin architecture in CloudStack is designed to easily
>> enable/disable each plugin component without impacting too much on main
>> CloudStack components. So in this case, I would personally prefer that
>>we
>> should make sure of data integrity in the scope of CloudStack main
>> components first and handle potential message handling failure in plugin
>> module separately through application level logic.
>> 
>>  Thanks
>>  -min
>> 
>> 
>> 
>> On 11/18/14 5:52 AM, "Leo Simons"  wrote:
>> 
>>> Hi Min, hi Koushik,
>>> 
>>> Cloudstack is shouting at me:
>>>   NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!
>>> 
>>> (full stack trace below). I've learned this is happening on our
>>> systemvm-persistent-config feature branch because it has commit
>>> ffaabdc13fde0f0f7b2667a483006e2a4b805f63 but it does not have commit
>>> f585dd266188a134a9c8b911376b066b9d3806e8 yet.
>>> 
>>> I'm now trying to understand what's happening here -- the transaction /
>>> concurrency / messaging logic gave me significant headache with its
>>> triple negatives, nested transaction scoping and home-grown gates, but
>>>I
>>> think I got it now.
>>> 
>>> As I understand it, in the olde world, creating an account:
>>> 
>>> * opens a database transaction
>>> * creates an account in the db
>>> * creates the first user in that account in the db
>>> * publishes an event
>>>   * which is listened to by 0 subscribers
>>> * commmits the database transaction
>>> * check the user is there
>>>   * opens a database transaction
>>>   * find the created user in the database
>>>   * (auto)closes transaction
>>>   * returns success if the user is in the db
>>> 
>>> this, err, works, but in some other cases, apparently, there are
>>>concerns
>>> that the db transaction

Build failed in Jenkins: build-master-noredist #3814

[jenkins]

See 

--
Started by upstream project "build-master" build number 1912
originally caused by:
 Started by an SCM change
[EnvInject] - Loading node environment variables.
Building remotely on cloudstack-buildslave-centos6-87d 
(cloudstack-buildslave-centos6) in workspace 

 > /usr/bin/git rev-parse --is-inside-work-tree # timeout=400
Fetching changes from the remote Git repository
 > /usr/bin/git config remote.origin.url 
 > https://git-wip-us.apache.org/repos/asf/cloudstack.git # timeout=400
Fetching upstream changes from 
https://git-wip-us.apache.org/repos/asf/cloudstack.git
 > /usr/bin/git --version # timeout=400
 > /usr/bin/git fetch --tags --progress 
 > https://git-wip-us.apache.org/repos/asf/cloudstack.git 
 > +refs/heads/*:refs/remotes/origin/*
ERROR: Timeout after 400 minutes
FATAL: Failed to fetch from 
https://git-wip-us.apache.org/repos/asf/cloudstack.git
hudson.plugins.git.GitException: Failed to fetch from 
https://git-wip-us.apache.org/repos/asf/cloudstack.git
at hudson.plugins.git.GitSCM.fetchFrom(GitSCM.java:647)
at hudson.plugins.git.GitSCM.retrieveChanges(GitSCM.java:889)
at hudson.plugins.git.GitSCM.checkout(GitSCM.java:914)
at hudson.model.AbstractProject.checkout(AbstractProject.java:1258)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:622)
at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:528)
at hudson.model.Run.execute(Run.java:1759)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
at hudson.model.ResourceController.execute(ResourceController.java:89)
at hudson.model.Executor.run(Executor.java:240)
Caused by: hudson.plugins.git.GitException: Command "/usr/bin/git fetch --tags 
--progress https://git-wip-us.apache.org/repos/asf/cloudstack.git 
+refs/heads/*:refs/remotes/origin/*" returned status code 143:
stdout: 
stderr: error: RPC failed; result=52, HTTP code = 100

at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1435)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandWithCredentials(CliGitAPIImpl.java:1223)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.access$300(CliGitAPIImpl.java:85)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl$1.execute(CliGitAPIImpl.java:280)
at 
org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler$1.call(RemoteGitImpl.java:153)
at 
org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler$1.call(RemoteGitImpl.java:146)
at hudson.remoting.UserRequest.perform(UserRequest.java:121)
at hudson.remoting.UserRequest.perform(UserRequest.java:49)
at hudson.remoting.Request$2.run(Request.java:324)
at 
hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:68)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
at java.util.concurrent.FutureTask.run(FutureTask.java:166)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1146)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:701)
at ..remote call to cloudstack-buildslave-centos6-87d(Native Method)
at hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1356)
at hudson.remoting.UserResponse.retrieve(UserRequest.java:221)
at hudson.remoting.Channel.call(Channel.java:752)
at 
org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler.execute(RemoteGitImpl.java:146)
at sun.reflect.GeneratedMethodAccessor462.invoke(Unknown Source)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:601)
at 
org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler.invoke(RemoteGitImpl.java:132)
at sun.proxy.$Proxy112.execute(Unknown Source)
at hudson.plugins.git.GitSCM.fetchFrom(GitSCM.java:645)
... 10 more

Build failed in Jenkins: build-4.5 #143

[jenkins]

See 

Changes:

[jessicawang] CLOUDSTACK-7498: UI > ISO > Register ISO action > a javascript 
error "osTypeObjs is not defined" comes and goes.

--
[...truncated 1658 lines...]
[INFO] Deleting 
 (includes = 
[target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-network ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-usage ---
[INFO] Deleting  
(includes = [**/*], excludes = [])
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-server ---
[INFO] Deleting  
(includes = [**/*], excludes = [])
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-usage ---
[INFO] Deleting 
 
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-orchestration ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-network ---
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-server ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-usage ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-network ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-usage ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-orchestration ---
[INFO] Executing tasks

main:
 [copy] Copying 2 files to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-usage ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-usage 
---
[INFO] Compiling 19 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-orchestration ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-orchestration ---
[INFO] Compiling 68 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-engine-network ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-engine-network ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-usage ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 8 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-usage ---
[INFO] Compiling 4 source files to 

Jenkins build is back to normal : build-master-noredist #3815

[jenkins]

See 

Jenkins build is still unstable: simulator-singlerun #656

[jenkins]

See 

Jenkins build is back to stable : simulator-singlerun #657

[jenkins]

See 

Build failed in Jenkins: build-4.5 #144

[jenkins]

See 

Changes:

[jessicawang] CLOUDSTACK-6624: UI > create network offering > cloudStack does 
NOT support specifyIpRanges for isolated network - fix a bug that wrongly sends 
"specifyIpRanges=true" to createNetworkOffering API.

--
[...truncated 1655 lines...]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-engine-orchestration ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-usage ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-server ---
[INFO] Deleting 
 
(includes = [**/*], excludes = [])
[INFO] Deleting 
 
(includes = [**/*], excludes = [])
[INFO] Deleting  
(includes = [**/*], excludes = [])
[INFO] Deleting 
 (includes = 
[target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-network ---
[INFO] Deleting  
(includes = [**/*], excludes = [])
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-usage ---
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-server ---
[INFO] Deleting 
 
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-orchestration ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-network ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-usage ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-network ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-usage ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-orchestration ---
[INFO] Executing tasks

main:
 [copy] Copying 2 files to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-usage ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-usage 
---
[INFO] Compiling 19 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-orchestration ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-engine-orchestration ---
[INFO] Compiling 68 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-engine-network ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-engine-network ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-engine-network ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, E

Jenkins build is back to normal : build-4.5 #145

[jenkins]

See 

Jenkins build became unstable: simulator-singlerun #658

[jenkins]

See 

Jenkins build is back to stable : simulator-singlerun #659

[jenkins]

See 

Re: Review Request 28179: CLOUDSTACK-7928 : Fixed the script 'test_vpc_vm_life_cycle.py' - Removed the Invalid test cases for Stopped VPC VR Scenario

[sangeetha hariharan]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28179/#review62071
---

Ship it!


Ship It!

- sangeetha hariharan


On Nov. 18, 2014, 5:47 p.m., Chandan Purushothama wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28179/
> ---
> 
> (Updated Nov. 18, 2014, 5:47 p.m.)
> 
> 
> Review request for cloudstack and sangeetha hariharan.
> 
> 
> Bugs: CLOUDSTACK-7928
> https://issues.apache.org/jira/browse/CLOUDSTACK-7928
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Following Test Cases currently fail in TestVMLifeCycleStoppedVPCVR Test Suite:
> 
> test_07_migrate_instance_in_network
> test_08_user_data
> test_09_meta_data
> test_10_expunge_instance_in_network
> 
> The test cases fail for the obvious reason since the VPC VR is stopped. The 
> Stopped VPCVR doesnt allow the traffic to or from the Guest VMs. Hence the 
> test cases are not valid to be tested in such a scenario and should be 
> removed.
> 
> Removed the test cases that are failing due to invalid scenario
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_vpc_vm_life_cycle.py 7a1fd8a 
> 
> Diff: https://reviews.apache.org/r/28179/diff/
> 
> 
> Testing
> ---
> 
> Testing not done
> 
> 
> Thanks,
> 
> Chandan Purushothama
> 
>

Review Request 28213: CLOUDSTACK-7752: Management Server goes in infinite loop while creating a vm with tagged local data disk when the pool is not tagged

[Anshul Gangwar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28213/
---

Review request for cloudstack.


Bugs: CLOUDSTACK-7752
https://issues.apache.org/jira/browse/CLOUDSTACK-7752


Repository: cloudstack-git


Description
---

Fixed deployment planner stuck in infinite loop. If we create VM with shared 
service offering and attach disk with local disk offering, and one of storage 
pool is full(cannot be allocated) and other is not full then we are not putting 
the cluster in avoid list which is causing this infinite loop.

Fixed by putting the cluster in avoid list even if one of the storage pool is 
full(cannot be allocated)


Diffs
-

  server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java 
d9e1ecc2291a72a04ee0c1b35a079d5ff166bf77 

Diff: https://reviews.apache.org/r/28213/diff/


Testing
---


Thanks,

Anshul Gangwar

Re: Review Request 28213: CLOUDSTACK-7752: Management Server goes in infinite loop while creating a vm with tagged local data disk when the pool is not tagged

[Anshul Gangwar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28213/
---

(Updated Nov. 19, 2014, 5:03 a.m.)


Review request for cloudstack and Rajesh Battala.


Bugs: CLOUDSTACK-7752
https://issues.apache.org/jira/browse/CLOUDSTACK-7752


Repository: cloudstack-git


Description
---

Fixed deployment planner stuck in infinite loop. If we create VM with shared 
service offering and attach disk with local disk offering, and one of storage 
pool is full(cannot be allocated) and other is not full then we are not putting 
the cluster in avoid list which is causing this infinite loop.

Fixed by putting the cluster in avoid list even if one of the storage pool is 
full(cannot be allocated)


Diffs
-

  server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java 
d9e1ecc2291a72a04ee0c1b35a079d5ff166bf77 

Diff: https://reviews.apache.org/r/28213/diff/


Testing
---


Thanks,

Anshul Gangwar

Re: Review Request 28213: CLOUDSTACK-7752: Management Server goes in infinite loop while creating a vm with tagged local data disk when the pool is not tagged

[Anshul Gangwar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28213/
---

(Updated Nov. 19, 2014, 5:17 a.m.)


Review request for cloudstack, daan Hoogland and Rajesh Battala.


Bugs: CLOUDSTACK-7752
https://issues.apache.org/jira/browse/CLOUDSTACK-7752


Repository: cloudstack-git


Description
---

Fixed deployment planner stuck in infinite loop. If we create VM with shared 
service offering and attach disk with local disk offering, and one of storage 
pool is full(cannot be allocated) and other is not full then we are not putting 
the cluster in avoid list which is causing this infinite loop.

Fixed by putting the cluster in avoid list even if one of the storage pool is 
full(cannot be allocated)


Diffs
-

  server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java 
d9e1ecc2291a72a04ee0c1b35a079d5ff166bf77 

Diff: https://reviews.apache.org/r/28213/diff/


Testing
---


Thanks,

Anshul Gangwar

[GitHub] cloudstack pull request: Merge 4.5 to master

[karuturi]

Github user karuturi closed the pull request at:

https://github.com/apache/cloudstack/pull/42


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Jenkins build became unstable: simulator-singlerun #661

[jenkins]

See 

Review Request 28217: when a template is deleted and then copied over again , it is still marked as Removed in template_zone_ref table

[bharat kumar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28217/
---

Review request for cloudstack and Kishan Kavala.


Repository: cloudstack-git


Description
---

https://issues.apache.org/jira/browse/CLOUDSTACK-7939


Diffs
-

  engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 401a4a2 
  engine/schema/src/com/cloud/storage/dao/VMTemplateZoneDao.java 67f7c3f 
  engine/schema/src/com/cloud/storage/dao/VMTemplateZoneDaoImpl.java 8cadf61 

Diff: https://reviews.apache.org/r/28217/diff/


Testing
---


Thanks,

bharat kumar

Jenkins build is still unstable: simulator-singlerun #662

[jenkins]

See