[GitHub] cloudstack pull request: fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
GitHub user lttmtins opened a pull request:

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

fix mariadb related listCapacity bug (CLOUDSTACK-8966)  a bigint(20) …

…with varchar type does not work well on MariaDB
so forcing it to type decimal
Tested on ACS4.7 with MariaDB10.0.22 (galera)

To test (this is the output with the pull request code underneath):
(nl2) > list capacity sortby=usage type=0
count = 1
capacity:
capacitytotal = 0
capacityused = 3501740523520
percentused = 0
type = 0
zoneid = 76251030-aca1-44c6-b47d-8010ee17e0ad
zonename = NL2
(nl2) > 

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

$ git pull https://github.com/lttmtins/cloudstack master

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

https://github.com/apache/cloudstack/pull/1313.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 #1313


commit 957e37df012f0b3be84735a5bde70eb004d18493
Author: Anton Opgenoort 
Date:   2016-01-06T16:15:02Z

fix mariadb related listCapacity bug (CLOUDSTACK-8966)  a bigint(20) with 
varchar type does not work well on MariaDB
so forcing it to type decimal




---
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: Fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
GitHub user lttmtins opened a pull request:

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

Fix mariadb related listCapacity bug (CLOUDSTACK-8966)

type bigint(20) with type  varchar does not work well on MariaDB
So forcing it to type decimal

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

$ git pull https://github.com/lttmtins/cloudstack 4.7-CLOUDSTACK-8966

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

https://github.com/apache/cloudstack/pull/1314.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 #1314


commit b90019e0541b18580d654870d5cce2430072dbd7
Author: Anton Opgenoort 
Date:   2016-01-06T16:30:55Z

Fix mariadb related listCapacity bug (CLOUDSTACK-8966)

type bigint(20) with type  varchar does not work well on MariaDB
So forcing it to type decimal




---
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: fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
Github user lttmtins commented on the pull request:

https://github.com/apache/cloudstack/pull/1313#issuecomment-169381926
  
On advice of release master this is now a pull request against 4.7


---
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: fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
Github user lttmtins closed the pull request at:

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


---
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: Fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
Github user lttmtins commented on the pull request:

https://github.com/apache/cloudstack/pull/1314#issuecomment-169384312
  
To test with cloudmonkey (this is the output with the pull request code 
underneath) you need 2 API calls (e.g. via cloudmonkey):
(nl2) > list capacity sortby=usage type=0
count = 1
capacity:
capacitytotal = 0
capacityused = 3501740523520
percentused = 0
type = 0
zoneid = 76251030-aca1-44c6-b47d-8010ee17e0ad
zonename = NL2
(nl2) >

Suggested tests to see if the 'capacitytotal != 0':
(nl2) > list capacity type=0 clusterid=
(nl2) > list capacity type=1 clusterid=
(nl2) > list capacity type=3 clusterid=
(nl2) > list capacity type=0 sortby=usage
(nl2) > list capacity type=1 sortby=usage
(nl2) > list capacity type=3 sortby=usage


---
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: Fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
GitHub user lttmtins reopened a pull request:

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

Fix mariadb related listCapacity bug (CLOUDSTACK-8966)

type bigint(20) with type  varchar does not work well on MariaDB
So forcing it to type decimal

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

$ git pull https://github.com/lttmtins/cloudstack 4.7-CLOUDSTACK-8966

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

https://github.com/apache/cloudstack/pull/1314.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 #1314


commit b90019e0541b18580d654870d5cce2430072dbd7
Author: Anton Opgenoort 
Date:   2016-01-06T16:30:55Z

Fix mariadb related listCapacity bug (CLOUDSTACK-8966)

type bigint(20) with type  varchar does not work well on MariaDB
So forcing it to type decimal




---
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: Fix mariadb related listCapacity bug (CLO...

2016-01-06 Thread lttmtins
Github user lttmtins closed the pull request at:

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


---
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: Remove template ulimit from createtmplt.s...

2016-01-13 Thread lttmtins
Github user lttmtins commented on the pull request:

https://github.com/apache/cloudstack/pull/1223#issuecomment-171251743
  
Actually the createtmplt.sh script starts with the hardcoded ulimit -f, but 
there is an '-S" flag within the script which would set the ulimit -f value to 
the Size+10. 
however the 
services/secondary-storage/server/src/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java
 code does not implement the -S flag; see line 314. 


---
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: Remove template ulimit from createtmplt.s...

2016-01-13 Thread lttmtins
Github user lttmtins commented on the pull request:

https://github.com/apache/cloudstack/pull/1223#issuecomment-171262924
  
However just removing the ulimit as per suggestion also works for me.


---
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.
---