[GitHub] brooklyn-server pull request #771: Updates karaf to 4.1.2

2017-08-18 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/brooklyn-server/pull/771


---
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] brooklyn-dist issue #101: Updates due to karaf update to 4.1.2

2017-08-18 Thread m4rkmckenna
Github user m4rkmckenna commented on the issue:

https://github.com/apache/brooklyn-dist/pull/101
  
@geomacy Ive amended the log format (back to old format) and ive set 
`jclouds.wire` to `OFF`

Both fixed in 50d3470



---
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] brooklyn-server issue #798: Fix template forward reference

2017-08-18 Thread Graeme-Miller
Github user Graeme-Miller commented on the issue:

https://github.com/apache/brooklyn-server/pull/798
  
I have tested locally and can confirm that this fixes the issue that I saw. 
This PR adds a test for that issue, but thought it was best to double check.

This code looks good as well, although I don't know this area of the code 
particularly well.


---
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] brooklyn-server pull request #801: Removes karaf itests in favour of itests ...

2017-08-18 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/brooklyn-server/pull/801


---
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] brooklyn-server issue #791: Sync file system on temp file before moving it.

2017-08-18 Thread geomacy
Github user geomacy commented on the issue:

https://github.com/apache/brooklyn-server/pull/791
  
rebased against master, still haven't got round to integration tests


---
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] brooklyn-server pull request #801: Removes karaf itests in favour of itests ...

2017-08-18 Thread m4rkmckenna
GitHub user m4rkmckenna opened a pull request:

https://github.com/apache/brooklyn-server/pull/801

Removes karaf itests in favour of itests within brooklyn-dist

* These tests only test feature install which is covered by 
feature-validation
* There are more representative tests within brooklyn-dist
  that actaully test the brookyn distribution vs standard karaf

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

$ git pull https://github.com/m4rkmckenna/brooklyn-server remove-karaf-itest

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

https://github.com/apache/brooklyn-server/pull/801.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 #801


commit cc31925472142dc8754646c2f8b289d4c9b5bec6
Author: Mark McKenna 
Date:   2017-08-18T13:32:17Z

Removes karaf itests in favour of itests withing brooklyn-dist

* These tests only test feature install which is covered by 
feature-validation
* There are more representative tests within brooklyn-dist
  that actaully test the brookyn distribution vs standard karaf




---
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] brooklyn-server issue #771: Updates karaf to 4.1.2

2017-08-18 Thread m4rkmckenna
Github user m4rkmckenna commented on the issue:

https://github.com/apache/brooklyn-server/pull/771
  
@geomacy PAX exam tests fixed with latest commit in 
https://github.com/apache/brooklyn-dist/pull/101


---
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] brooklyn-server issue #800: AutoScalerPolicy to resize to limits on expunge

2017-08-18 Thread nakomis
Github user nakomis commented on the issue:

https://github.com/apache/brooklyn-server/pull/800
  
Should now be good to go


---
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] brooklyn-server issue #800: AutoScalerPolicy to resize to limits on expunge

2017-08-18 Thread nakomis
Github user nakomis commented on the issue:

https://github.com/apache/brooklyn-server/pull/800
  
On second thoughts, these tests seem a little fragile and may have 
non-deterministic failures. Please DO NOT MERGE (yet)


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