[GitHub] brooklyn-server issue #674: [WIP] [JCLOUDS 2.1.0] Remove Azure workarounds a...

2018-04-04 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/674 Looks like this has been addressed as well so closing. ---

[GitHub] brooklyn-server pull request #674: [WIP] [JCLOUDS 2.1.0] Remove Azure workar...

2018-04-04 Thread neykov
Github user neykov closed the pull request at: https://github.com/apache/brooklyn-server/pull/674 ---

[GitHub] brooklyn-server pull request #673: [WIP] [JCLOUDS 2.1.0] Upgrade to jclouds ...

2018-04-04 Thread neykov
Github user neykov closed the pull request at: https://github.com/apache/brooklyn-server/pull/673 ---

[GitHub] brooklyn-server issue #673: [WIP] [JCLOUDS 2.1.0] Upgrade to jclouds 2.1

2018-04-04 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/673 Thanks @andreaturli, @geomacy, closing. ---

[jira] [Commented] (BROOKLYN-548) Brooklyn node ID changes on restart

2017-11-08 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16244537#comment-16244537 ] Svetoslav Neykov commented on BROOKLYN-548: --- [~tbouron] I believe that&#

[GitHub] brooklyn-server issue #860: improve locking for sensor values

2017-10-13 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/860 Seems this fixes [BROOKLYN-525](https://issues.apache.org/jira/browse/BROOKLYN-525) as well. Worth checking by enabling the corresponding tests as done in https://github.com/apache

[GitHub] brooklyn-server issue #794: Test Framework TestNG runner

2017-08-07 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/794 :+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

[GitHub] brooklyn-server pull request #794: Test Framework TestNG runner

2017-08-05 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/794 Test Framework TestNG runner Useful for running the test framework tests as live tests on the project. From the test included in the PR, example usage

[GitHub] brooklyn-server pull request #392: [WIP] $brooklyn:self().template("freemark...

2017-08-04 Thread neykov
Github user neykov closed the pull request at: https://github.com/apache/brooklyn-server/pull/392 --- 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

[GitHub] brooklyn-server issue #392: [WIP] $brooklyn:self().template("freemarker")

2017-08-04 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/392 Closing in favour of https://github.com/apache/brooklyn-server/pull/786. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] brooklyn-server pull request #641: [WIP] Fix sensor changes event ordering

2017-08-04 Thread neykov
Github user neykov closed the pull request at: https://github.com/apache/brooklyn-server/pull/641 --- 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

[GitHub] brooklyn-server issue #641: [WIP] Fix sensor changes event ordering

2017-08-04 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/641 Fix failed to address the problem. Closing. Created an issue to track the bug. https://issues.apache.org/jira/browse/BROOKLYN-525 --- If your project is set up for it, you can reply to

[jira] [Created] (BROOKLYN-525) Sensor change events fired out of order

2017-08-04 Thread Svetoslav Neykov (JIRA)
Svetoslav Neykov created BROOKLYN-525: - Summary: Sensor change events fired out of order Key: BROOKLYN-525 URL: https://issues.apache.org/jira/browse/BROOKLYN-525 Project: Brooklyn Issue

[GitHub] brooklyn-server issue #786: Freemarker DSL

2017-08-03 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/786 Thanks @geomacy, fixed the typo. --- 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 issue #786: Freemarker DSL

2017-08-03 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/786 Added additional template variables (substitutions) as a second argument. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] brooklyn-server issue #786: Freemarker DSL

2017-08-03 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/786 Added more test coverage. --- 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

[GitHub] brooklyn-server pull request #786: Freemarker DSL

2017-08-02 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/786 Freemarker DSL Provides `$brooklyn:template(...)` evaluated against the context entity as a freemarker template. The main advantage of the new DSL is that it's now possible to do s

[GitHub] brooklyn-server issue #782: Adds `PUT /applications/{appid}`

2017-07-28 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/782 That's used for machine names which already include a bunch of other unique tags, including a jclouds generated random number. Of course we could use a similar strategy, it's just

[GitHub] brooklyn-server issue #782: Adds `PUT /applications/{appid}`

2017-07-28 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/782 Just a heads up that some downstream apps/entities embed the appId in the name of cloud resources and having an ID with a length of 1022 will lead to failures when trying to create them

[GitHub] brooklyn-server pull request #781: Support DSL in template options

2017-07-28 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/781#discussion_r130041426 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/templates/customize/TemplateOptionsOption.java --- @@ -39,8 +45,17

[GitHub] brooklyn-server pull request #781: Support DSL in template options

2017-07-27 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/781#discussion_r129914806 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/templates/customize/TemplateOptionsOption.java --- @@ -39,8 +45,17

Re: Proposal: Add appId optional paramater to deploy api

2017-07-26 Thread Svetoslav Neykov
;>> `org.apache.brooklyn.deploymentId` approach. >>> >>> For "/application's ID is not meant to be user-supplied/", that has >>> historically been the case but why should we stick to that? What matters is >>> that the appId is definitely u

[GitHub] brooklyn-server issue #767: Adds BrooklynViewerLauncher (for tests)

2017-07-21 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/767 +1, 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

[GitHub] brooklyn-dist issue #101: [WIP] Update logging config for log4j2 due to kara...

2017-07-20 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-dist/pull/101 If we are changing logging providers, perhaps it's the right time to look at using logback, as was the initial plan? Any preferences here for log4j2 vs logback? --- If your proje

[GitHub] brooklyn-server issue #771: [WIP] Updates karaf to 4.1.1

2017-07-20 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/771 :+1: glad to see so few changes needed. --- 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

[GitHub] brooklyn-server issue #770: Add null checks to Azure default network

2017-07-20 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/770 Tests are passing. 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

[GitHub] brooklyn-server issue #769: Fix feature install bundle version issue

2017-07-20 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/769 Tests pass, 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

[GitHub] brooklyn-server issue #769: Fix feature install bundle version issue

2017-07-20 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/769 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] brooklyn-server issue #767: Adds BrooklynViewerLauncher (for tests)

2017-07-19 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/767 Looks a lot like `BrooklynJavascriptGuiLauncher`, but otherwise LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] brooklyn-dist issue #100: Remove another reference to a deleted file.

2017-07-19 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-dist/pull/100 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

[GitHub] brooklyn-server issue #765: REST API returns plane-id

2017-07-17 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/765 :+1: somehow I've been viewing only the first commit. All good! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] brooklyn-server issue #765: REST API returns plane-id

2017-07-17 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/765 Is it worth adding it to `/ha/states` as well, along `nodeId`? LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

2017-07-15 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/745 @Graeme-Miller Brooklyn has upgraded to jclouds 2.0.2, which opens the door to supporting default region as well. --- If your project is set up for it, you can reply to this email and have

[GitHub] brooklyn-server pull request #764: Fixes tested response from /server/user

2017-07-15 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/764 Fixes tested response from /server/user Fixes the integration test `ServerResourceIntegrationTest.testGetUser()`. You can merge this pull request into a Git repository by running

[GitHub] brooklyn-server issue #753: Update jclouds to 2.0.2

2017-07-11 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/753 Build succeeds after rebasing on master with the `brooklyn-locations-container` pom.xml fix applied. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] brooklyn-server issue #753: Update jclouds to 2.0.2

2017-07-10 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/753 retest this please --- 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

[GitHub] brooklyn-server pull request #754: Fix REST API response of methods with Str...

2017-07-10 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/754#discussion_r126417847 --- Diff: rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/PolicyConfigResource.java --- @@ -66,7 +66,7

[GitHub] brooklyn-dist issue #98: Don't avoid importing jclouds google compute engine...

2017-07-10 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-dist/pull/98 Thanks @geomacy, merging. Lot's more entries need removing from here, can be addressed in another PR. --- If your project is set up for it, you can reply to this email and have

Re: Proposal: Add appId optional paramater to deploy api

2017-07-07 Thread Svetoslav Neykov
Hi Duncan, I've solved this problem before by adding a caller generated config key on the app (now it's also possible to tag them), then iterating over the deployed apps, looking for the key. An alternative which I'd like to mention is creating an async deploy operation which immediately retur

Re: BROOKLYN-519 suggestion to remove BasicSpecParameter from serializing

2017-07-07 Thread Svetoslav Neykov
Hi Valentin, Parameters set on the specs are later converted to config keys on the entity which are then used for type inference and applying restrictions (can mark parameters as required). That's why I think it would be useful to keep the parameters passed in from yaml. We had a similar probl

[GitHub] brooklyn-server issue #753: Update jclouds to 2.0.2

2017-07-06 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/753 retest this please --- 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

[GitHub] brooklyn-server issue #754: Fix REST API response of methods with String ret...

2017-07-05 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/754 Suggest we keep the changes for `PolicyConfigApi.get(...)`, making it work analogously to `EntityConfigApi.get(...)`. The value there can be any object, so it needs to be encoded as JSON

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

2017-07-04 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/745 Thanks @Graeme-Miller, 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

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

2017-07-04 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/745 LGTM with the "waiting to complete" changes. @Graeme-Miller have you retried your test, has the failure rate improved? If no failure this is good to merge. Otherwise we need t

[GitHub] brooklyn-server pull request #753: [WIP] Update jclouds to 2.0.2-rc1

2017-07-03 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/753 [WIP] Update jclouds to 2.0.2-rc1 `azurecompute-arm` has changed considerably since `2.0.1`. `jclouds-labs` providers usually backport all master changes to iterate quickly, free to break

[GitHub] brooklyn-server issue #752: Cache the default region in location config

2017-07-03 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/752 Closing because will not consistently set the property. When the location doesn't have a region set, but one is passed through `flags` (for example in `provisioning.properties`) we wil

[GitHub] brooklyn-server pull request #752: Cache the default region in location conf...

2017-07-03 Thread neykov
Github user neykov closed the pull request at: https://github.com/apache/brooklyn-server/pull/752 --- 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

[GitHub] brooklyn-server pull request #752: Cache the default region in location conf...

2017-07-03 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/752 Cache the default region in location config You can merge this pull request into a Git repository by running: $ git pull https://github.com/neykov/brooklyn-server cache-default-region

[jira] [Commented] (BROOKLYN-524) Windows paths treated as `bundle:resource` pairs resulting in file not found errors

2017-07-03 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16071992#comment-16071992 ] Svetoslav Neykov commented on BROOKLYN-524: --- Unconfirmed workaround - pr

[jira] [Created] (BROOKLYN-524) Windows paths treated as `bundle:resource` pairs resulting in file not found errors

2017-07-03 Thread Svetoslav Neykov (JIRA)
Svetoslav Neykov created BROOKLYN-524: - Summary: Windows paths treated as `bundle:resource` pairs resulting in file not found errors Key: BROOKLYN-524 URL: https://issues.apache.org/jira/browse/BROOKLYN-524

[jira] [Created] (BROOKLYN-523) Unhelpful warn message "Configuration being made to SshMachineLocation"

2017-06-30 Thread Svetoslav Neykov (JIRA)
Svetoslav Neykov created BROOKLYN-523: - Summary: Unhelpful warn message "Configuration being made to SshMachineLocation" Key: BROOKLYN-523 URL: https://issues.apache.org/jira/browse/BR

[jira] [Created] (BROOKLYN-521) Unhelpful message "Attempted release of unknown machine"

2017-06-29 Thread Svetoslav Neykov (JIRA)
Svetoslav Neykov created BROOKLYN-521: - Summary: Unhelpful message "Attempted release of unknown machine" Key: BROOKLYN-521 URL: https://issues.apache.org/jira/browse/BROOKLYN-521

[GitHub] brooklyn-server issue #748: Catalog internal references

2017-06-29 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/748 @alx I think the fix makes sense. --- 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 issue #748: Catalog internal references

2017-06-29 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/748 Sounds reasonable (on first read :) ). Still seems like a hacky workflow though. Separating the metadata and the content just so it's friendly to ad-hoc editing is not encouraging goo

[GitHub] brooklyn-server issue #731: BROOKLYN-440: ssh not use StreamGobbler for logg...

2017-06-29 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/731 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] brooklyn-server issue #748: Catalog internal references

2017-06-29 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/748 Support for the [`type: url`](https://github.com/apache/incubator-brooklyn/pull/71) was added around the same time we added support for [yaml catalog items](https://github.com/apache

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

2017-06-29 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/745 Fine by me to merge the PR as it already improves on the existing functionality. Can you create another PR which targets post 2.0.1 jclouds, handling the default region case and mark it `WIP

[GitHub] brooklyn-server issue #740: New versioning rules prep

2017-06-29 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/740 > for other things yes it will be surprising but someone making an OSGi bundle relying on that converter's obscure behaviour is asking for trouble That's everyone usin

[GitHub] brooklyn-server issue #737: Clean up inference around parsing `name:version`...

2017-06-27 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/737 @geomacy left my comments in https://github.com/apache/brooklyn-server/pull/740 which includes this PR. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] brooklyn-server pull request #745: Azure ARM default network fixes from PR s...

2017-06-26 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/745#discussion_r124042939 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -64,57

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

2017-06-26 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/745 * re azure jclouds 2.0.1 not having enough functionality - the changes here (and in previous PR) will create the default network, but will not assign it to the nodes because `ipOptions

[GitHub] brooklyn-server pull request #745: Azure ARM default network fixes from PR s...

2017-06-26 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/745#discussion_r124042243 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -64,57

[GitHub] brooklyn-server issue #743: Version new rules

2017-06-26 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/743 LGTM retest this please. --- 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 issue #740: New versioning rules prep

2017-06-25 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/740 Couldn't find where we sanitise the catalog items ID to be used for persistence file names. If changing the naming rules for the IDs then perhaps changing the sanitising rules wou

[GitHub] brooklyn-server issue #740: New versioning rules prep

2017-06-25 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/740 LGTM, but a bit nervous about not relying on `DefaultMaven2OsgiConverter` for the version conversion. The same version string will create a maven bundle with one OSGi version and a catalog

[GitHub] brooklyn-server issue #742: DefaultAzureArmNetworkCreator improvements

2017-06-22 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/742 LGTM, also see the comments I just added to https://github.com/apache/brooklyn-server/pull/739 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] brooklyn-server issue #739: A default network is now created when deploying ...

2017-06-22 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/739 The most important from above comments is that `location` (which is region in brooklyn speak) could not be set, using the jclouds' default. Would be useful to have that improved. An

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

2017-06-22 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/739#discussion_r123533239 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -0,0

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

2017-06-22 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/739#discussion_r123531214 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -0,0

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

2017-06-22 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/739#discussion_r123530475 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -0,0

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

2017-06-22 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/739#discussion_r123529723 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -0,0

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

2017-06-22 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/739#discussion_r123528419 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java --- @@ -0,0

[GitHub] brooklyn-dist issue #97: Update maven-compiler-plugin to 3.6.1

2017-06-22 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-dist/pull/97 Builds fine for me, 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

[GitHub] brooklyn-ui issue #46: Update maven-compiler-plugin to 3.6.1

2017-06-22 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-ui/pull/46 Builds fine for me, 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

[GitHub] brooklyn-server issue #741: Update maven-compiler-plugin to 3.6.1

2017-06-22 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/741 Builds fine for me, 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

[GitHub] brooklyn-server issue #741: Update maven-compiler-plugin to 3.6.1

2017-06-22 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/741 +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

Re: Versions in Brooklyn

2017-06-22 Thread Svetoslav Neykov
Makes sense. > On 22.06.2017 г., at 12:27, Alex Heneveld > wrote: > > > inline > > On 22/06/2017 10:10, Svetoslav Neykov wrote: >> +1 to the proposal. >> >> One thing I have reservations about is having a recommended version syntax >> with

Re: Versions in Brooklyn

2017-06-22 Thread Svetoslav Neykov
+1 to the proposal. One thing I have reservations about is having a recommended version syntax with other formats still supported but deprecated. As far as I understand the recommended syntax is there so we can guarantee a uniqueness of the OSGi versions (when the source version is unique). Inst

[jira] [Commented] (BROOKLYN-519) Persisting yaml blueprint with $brooklyn:entitySpec stores all configs

2017-06-21 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16057560#comment-16057560 ] Svetoslav Neykov commented on BROOKLYN-519: --- To restate the problem

[jira] [Resolved] (BROOKLYN-180) Provide a better implementation for WinRM

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-180?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-180. --- Resolution: Fixed The winrm4j project switched to a pure java implementation, based

[jira] [Comment Edited] (BROOKLYN-160) No-op entity for Windows VMs

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055934#comment-16055934 ] Svetoslav Neykov edited comment on BROOKLYN-160 at 6/20/17 3:2

[jira] [Commented] (BROOKLYN-160) No-op entity for Windows VMs

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055934#comment-16055934 ] Svetoslav Neykov commented on BROOKLYN-160: --- There's {{EmptyWindo

[jira] [Resolved] (BROOKLYN-160) No-op entity for Windows VMs

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-160?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-160. --- Resolution: Fixed Fix Version/s: 0.9.0 > No-op entity for Windows

[jira] [Resolved] (BROOKLYN-94) TOSCA Simple Profile YAML

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-94?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-94. -- Resolution: Won't Fix Available as an external add-on to Brooklyn at

[jira] [Commented] (BROOKLYN-139) policy which sets up /etc/hosts on a group of servers

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055926#comment-16055926 ] Svetoslav Neykov commented on BROOKLYN-139: --- Sounds like this is alr

[jira] [Resolved] (BROOKLYN-80) Update Swagger dependencies to version 1.3

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-80?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-80. -- Resolution: Fixed Fix Version/s: 0.10.0 Swagger is now at 1.5.6 > Upd

[jira] [Commented] (BROOKLYN-65) Brooklyn website needs list of supported entities

2017-06-20 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055908#comment-16055908 ] Svetoslav Neykov commented on BROOKLYN-65: -- It's almost there - just

[GitHub] brooklyn-server issue #731: BROOKLYN-440: ssh not use StreamGobbler for logg...

2017-06-19 Thread neykov
Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/731 @aledsage I'm fine with not updating the code to handle multi-threaded calls. Would be nice if you mark it explicitly as not supported it in the docs. It's been used just fine

[GitHub] brooklyn-server pull request #731: BROOKLYN-440: ssh not use StreamGobbler f...

2017-06-19 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/731#discussion_r122729269 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/stream/LoggingOutputStream.java --- @@ -0,0 +1,134 @@ +/* + * Licensed to the

[jira] [Resolved] (BROOKLYN-44) YAML text entry box - increase size and allow resizing

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-44?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-44. -- Resolution: Fixed Fix Version/s: 0.11.0 Fixed with the {{Composer}} tab, giving

[jira] [Resolved] (BROOKLYN-41) openIptables config appears to customise entries for requested ports, not assigned

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-41?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-41. -- Resolution: Fixed Fix Version/s: 0.12.0 Should've been fixed with

[jira] [Commented] (BROOKLYN-39) Support CAMP archives

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053712#comment-16053712 ] Svetoslav Neykov commented on BROOKLYN-39: -- This got an alterna

[jira] [Resolved] (BROOKLYN-13) OSGi for blueprint versioning, and catalog updates/revamp

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-13?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-13. -- Resolution: Fixed Fix Version/s: (was: 0.7.0) 0.11.0 We

[jira] [Resolved] (BROOKLYN-8) Some REST API methods are not JSON-compliant

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-8?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-8. - Resolution: Fixed Fix Version/s: 0.11.0 Response is proper JSON now. Not sure when

[jira] [Resolved] (BROOKLYN-7) Enable Brooklyn monitoring through Server REST API

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-7?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-7. - Resolution: Fixed It's possible to att

[jira] [Resolved] (BROOKLYN-9) Add support for Couchbase Sync Gateway

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-9?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-9. - Resolution: Fixed Support added in https://github.com/apache/incubator-brooklyn/pull/25

[jira] [Commented] (BROOKLYN-367) Upload catalogue boms and their dependencies to the server

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053693#comment-16053693 ] Svetoslav Neykov commented on BROOKLYN-367: --- Persisting of bun

[jira] [Resolved] (BROOKLYN-53) Define new locations in the web UI

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-53?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-53. -- Resolution: Fixed Adding locations is now possible through the UI. They are persisted

[jira] [Resolved] (BROOKLYN-17) Release 0.7.0-M2

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-17?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-17. -- Resolution: Fixed Release 0.7.0 is out. > Release 0.7.0

[jira] [Resolved] (BROOKLYN-27) IPMC vote to release

2017-06-19 Thread Svetoslav Neykov (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-27?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Svetoslav Neykov resolved BROOKLYN-27. -- Resolution: Fixed Release 0.7.0 is out. > IPMC vote to rele

  1   2   3   4   5   6   7   8   9   10   >