Skip to content

Child entity customizer#310

Merged
asfgit merged 2 commits intoapache:masterfrom
grkvlt:child-entity-customizer
Aug 30, 2016
Merged

Child entity customizer#310
asfgit merged 2 commits intoapache:masterfrom
grkvlt:child-entity-customizer

Conversation

@grkvlt
Copy link
Copy Markdown
Member

@grkvlt grkvlt commented Aug 28, 2016

Extends the jclouds location customizer to be a BasicConfigurableObject allowing better configuration etc. when created using $brooklyn:object in the DSL, and add a new ChildEntityCustomizer that creates a child entity before the main entity has started up, allowing more flexible configuration.

@grkvlt grkvlt force-pushed the child-entity-customizer branch from 0fcaced to 1cdf00f Compare August 30, 2016 14:22
@asfgit asfgit merged commit 1cdf00f into apache:master Aug 30, 2016
asfgit pushed a commit that referenced this pull request Aug 30, 2016
* github/pr/310:
  Add new location customizer that creates child entities early
  Update BasicJcloudsLocationCustomizer to be configurable and remove deprecated methods
@grkvlt grkvlt deleted the child-entity-customizer branch August 30, 2016 18:47
*
* <p>See
* <a href='http://unix.stackexchange.com/questions/122616/why-do-i-need-a-tty-to-run-sudo-if-i-can-sudo-without-a-password'>http://unix.stackexchange.com/questions/122616/why-do-i-need-a-tty-to-run-sudo-if-i-can-sudo-without-a-password</a>
* @see <a href='http://unix.stackexchange.com/questions/122616/why-do-i-need-a-tty-to-run-sudo-if-i-can-sudo-without-a-password'>http://unix.stackexchange.com/questions/122616/why-do-i-need-a-tty-to-run-sudo-if-i-can-sudo-without-a-password</a>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw this is already part of JcloudsLocation. Should we deprecate this now? Any use-cases where the location config can't be used?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants