[GitHub] brooklyn-server pull request #624: Adds authSudo support (for sudo’ing wit...

2017-04-07 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/624 Adds authSudo support (for sudo’ing with password) Required for some clouds (e.g. azure-arm), when using `useJcloudsSshInit: false` Tested on `azure-arm` using a location like t

[GitHub] brooklyn-client pull request #44: Added the ability to upload a zip via the ...

2017-04-07 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/44#discussion_r110366235 --- Diff: cli/api/catalog/catalog.go --- @@ -170,15 +178,102 @@ func Locations(network *net.Network) ([]models.CatalogItemSummary, error) { r

[GitHub] brooklyn-server pull request #623: BundleMaker copy fix

2017-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/623 --- 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 featur

[GitHub] brooklyn-server issue #617: Persist management plane ID

2017-04-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/617 Thanks @geomacy. Addressed comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] brooklyn-server pull request #617: Persist management plane ID

2017-04-07 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/617#discussion_r110387016 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/rebind/PeriodicDeltaChangeListener.java --- @@ -382,6 +401,8 @@ protected void persistNow

[GitHub] brooklyn-server issue #623: BundleMaker copy fix

2017-04-07 Thread Graeme-Miller
Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/623 @drigodwin no, it doesn't. If we were creating a zip from scratch we would do what I have done here, and create a zip entry with only the name. The zip code would then figure out the r

[GitHub] brooklyn-server issue #623: BundleMaker copy fix

2017-04-07 Thread Graeme-Miller
Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/623 Can you provide steps to re-produce @m4rkmckenna quickest way to replicate is to use the brooklyn cli from here: https://github.com/apache/brooklyn-client/pull/44. Using that CLI, z

[GitHub] brooklyn-server issue #623: BundleMaker copy fix

2017-04-07 Thread drigodwin
Github user drigodwin commented on the issue: https://github.com/apache/brooklyn-server/pull/623 Does it matter that we will lose all the information from ZipEntry apart from the file name? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] brooklyn-server issue #623: BundleMaker copy fix

2017-04-07 Thread m4rkmckenna
Github user m4rkmckenna commented on the issue: https://github.com/apache/brooklyn-server/pull/623 @Graeme-Miller Can you provide steps to re-produce ... does this happen for every uploaded zip? --- If your project is set up for it, you can reply to this email and have your reply app

[jira] [Commented] (BROOKLYN-460) Brooklyn Camp syntax for adding tags to an entity spec

2017-04-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15960625#comment-15960625 ] ASF GitHub Bot commented on BROOKLYN-460: - Github user geomacy commented on a di

[GitHub] brooklyn-docs pull request #162: BROOKLYN-460: Brooklyn Camp syntax guidance

2017-04-07 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/162#discussion_r110369079 --- Diff: guide/yaml/yaml-reference.md --- @@ -79,38 +79,44 @@ the entity being defined, with these being the most common: * `default`: a defau

[GitHub] brooklyn-server pull request #623: BundleMaker copy fix

2017-04-07 Thread Graeme-Miller
GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/623 BundleMaker copy fix When cloning a zip, we were using the ZipEntry from the input zip. This causes issues, as the info in the input ZipEntry is not necessarily correct for the ou

[jira] [Commented] (BROOKLYN-460) Brooklyn Camp syntax for adding tags to an entity spec

2017-04-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15960619#comment-15960619 ] ASF GitHub Bot commented on BROOKLYN-460: - Github user drigodwin commented on a

[jira] [Commented] (BROOKLYN-460) Brooklyn Camp syntax for adding tags to an entity spec

2017-04-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15960618#comment-15960618 ] ASF GitHub Bot commented on BROOKLYN-460: - Github user drigodwin commented on a

[jira] [Commented] (BROOKLYN-460) Brooklyn Camp syntax for adding tags to an entity spec

2017-04-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15960617#comment-15960617 ] ASF GitHub Bot commented on BROOKLYN-460: - Github user drigodwin commented on a

[jira] [Commented] (BROOKLYN-460) Brooklyn Camp syntax for adding tags to an entity spec

2017-04-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15960616#comment-15960616 ] ASF GitHub Bot commented on BROOKLYN-460: - Github user drigodwin commented on a

[GitHub] brooklyn-docs pull request #162: BROOKLYN-460: Brooklyn Camp syntax guidance

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/162#discussion_r110367803 --- Diff: guide/yaml/yaml-reference.md --- @@ -79,38 +79,44 @@ the entity being defined, with these being the most common: * `default`: a def

[GitHub] brooklyn-docs pull request #162: BROOKLYN-460: Brooklyn Camp syntax guidance

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/162#discussion_r110367351 --- Diff: guide/yaml/yaml-reference.md --- @@ -79,38 +79,44 @@ the entity being defined, with these being the most common: * `default`: a def

[GitHub] brooklyn-docs pull request #162: BROOKLYN-460: Brooklyn Camp syntax guidance

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/162#discussion_r110367663 --- Diff: guide/yaml/yaml-reference.md --- @@ -79,38 +79,44 @@ the entity being defined, with these being the most common: * `default`: a def

[GitHub] brooklyn-docs pull request #162: BROOKLYN-460: Brooklyn Camp syntax guidance

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/162#discussion_r110367262 --- Diff: guide/yaml/yaml-reference.md --- @@ -79,38 +79,44 @@ the entity being defined, with these being the most common: * `default`: a def

[GitHub] brooklyn-server pull request #617: Persist management plane ID

2017-04-07 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/617#discussion_r110366800 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/rebind/PeriodicDeltaChangeListener.java --- @@ -382,6 +401,8 @@ protected void persistNo

[GitHub] brooklyn-client pull request #44: Added the ability to upload a zip via the ...

2017-04-07 Thread Graeme-Miller
GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-client/pull/44 Added the ability to upload a zip via the CLI @geomacy would you please review? You can merge this pull request into a Git repository by running: $ git pull https://github.com/Gra

[GitHub] brooklyn-server pull request #617: Persist management plane ID

2017-04-07 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/617#discussion_r110364489 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/rebind/PeriodicDeltaChangeListener.java --- @@ -382,6 +401,8 @@ protected void persistNow

[GitHub] brooklyn-server pull request #617: Persist management plane ID

2017-04-07 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/617#discussion_r110364101 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/persist/BrooklynMementoPersisterToObjectStore.java --- @@ -674,6 +685,22 @@ private void

[GitHub] brooklyn-server pull request #617: Persist management plane ID

2017-04-07 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/617#discussion_r110363846 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/ha/HighAvailabilityManagerImpl.java --- @@ -456,6 +460,12 @@ else if (startMode==HighAvai

[GitHub] brooklyn-docs pull request #164: Adds documentation for bundling catalog res...

2017-04-07 Thread tbouron
Github user tbouron commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/164#discussion_r110357761 --- Diff: guide/ops/catalog/index.md --- @@ -371,6 +372,141 @@ increment an internal version number for the catalog item. When referencing a blue

[GitHub] brooklyn-docs pull request #164: Adds documentation for bundling catalog res...

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/164#discussion_r110355127 --- Diff: guide/ops/catalog/index.md --- @@ -371,6 +372,121 @@ increment an internal version number for the catalog item. When referencing a bl

[GitHub] brooklyn-docs pull request #164: Adds documentation for bundling catalog res...

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/164#discussion_r110355247 --- Diff: guide/ops/catalog/index.md --- @@ -371,6 +372,121 @@ increment an internal version number for the catalog item. When referencing a bl

[GitHub] brooklyn-docs pull request #164: Adds documentation for bundling catalog res...

2017-04-07 Thread drigodwin
Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/164#discussion_r110355931 --- Diff: guide/ops/catalog/index.md --- @@ -371,6 +372,121 @@ increment an internal version number for the catalog item. When referencing a bl

[GitHub] brooklyn-docs pull request #164: Adds documentation for bundling catalog res...

2017-04-07 Thread nakomis
GitHub user nakomis opened a pull request: https://github.com/apache/brooklyn-docs/pull/164 Adds documentation for bundling catalog resources in a zip file You can merge this pull request into a Git repository by running: $ git pull https://github.com/nakomis/brooklyn-docs doc

[jira] [Created] (BROOKLYN-470) Brooklyn can not use certain CentOS 7 images on Azure ARM

2017-04-07 Thread Yavor Yanchev (JIRA)
Yavor Yanchev created BROOKLYN-470: -- Summary: Brooklyn can not use certain CentOS 7 images on Azure ARM Key: BROOKLYN-470 URL: https://issues.apache.org/jira/browse/BROOKLYN-470 Project: Brooklyn

[GitHub] brooklyn-library pull request #95: Fix executeScript effector for MariaDB an...

2017-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/95 --- 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 featur

[GitHub] brooklyn-library issue #95: Fix executeScript effector for MariaDB and Postr...

2017-04-07 Thread geomacy
Github user geomacy commented on the issue: https://github.com/apache/brooklyn-library/pull/95 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 s

[GitHub] brooklyn-library pull request #95: Fix executeScript effector for MariaDB an...

2017-04-07 Thread iyovcheva
GitHub user iyovcheva opened a pull request: https://github.com/apache/brooklyn-library/pull/95 Fix executeScript effector for MariaDB and PostreSQL Fixes executeScript effector to fail on wrong command. For postresql the script was returning exit code 0 and required command