[GitHub] brooklyn-server pull request #906: CreatePassworkSensor: support "character ...

GitHub user rdowner opened a pull request: https://github.com/apache/brooklyn-server/pull/906 CreatePassworkSensor: support "character groups" Character groups are groups like "capital letters", "numbers", etc. This option allows a blueprint to require that the generated password

[GitHub] brooklyn-server issue #906: CreatePassworkSensor: support "character groups"

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/906 LGTM. Is it worth a YAML file illustrating it and a java test case? Probably needs to be in `brooklyn-camp`, could base on `ElectPrimaryTest.testSimpleRebind` -- maybe worth

[GitHub] brooklyn-server issue #906: CreatePassworkSensor: support "character groups"

Github user rdowner commented on the issue: https://github.com/apache/brooklyn-server/pull/906 Thanks for the review @ahgittin. There's already a couple of `CreatePasswordSensorTest`s, so I've added a test for this specific behaviour addition. ---

[GitHub] brooklyn-server issue #906: CreatePassworkSensor: support "character groups"

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/906 i am finding it really useful where we have yaml examples i can search for, that's why i suggested that, but that's not a requirement so i can't object to this being merged!

[GitHub] brooklyn-server issue #906: CreatePassworkSensor: support "character groups"

Github user rdowner commented on the issue: https://github.com/apache/brooklyn-server/pull/906 I'd be in favour of defining code/doc patterns where we can do this kind of stuff. Initializers are a bit of an odd case - catalog items such as entities, policies etc. are all

Build failed in Jenkins: brooklyn-master-build-docker #47

See -- [...truncated 6.78 MB...] at org.apache.brooklyn.core.mgmt.internal.LocalManagementContext.(LocalManagementContext.java:159) at