[GitHub] brooklyn-server issue #677: Adds EBS Location Customizers

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

https://github.com/apache/brooklyn-server/pull/677
  
@neykov Done


---
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 #677: Adds EBS Location Customizers

2017-05-18 Thread neykov
Github user neykov commented on the issue:

https://github.com/apache/brooklyn-server/pull/677
  
I mean if you deploy to another cloud it just doesn't do anything, not even 
populating the template options.


---
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 #677: Adds EBS Location Customizers

2017-05-18 Thread neykov
Github user neykov commented on the issue:

https://github.com/apache/brooklyn-server/pull/677
  
@nakomis Do you think it's worth making the customizers skip unknown 
locations (i.e. do their work only for amazon)?
This would let blueprints customize their environment through 
`provisioning.properties` but still be cloud agnostic.



---
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 #677: Adds EBS Location Customizers

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

https://github.com/apache/brooklyn-server/pull/677
  
@drigodwin description an example added
@sjcorbett the classes sit adjacent to existing cloud-specific classes such 
as `SoftLayerSameVlanLocationCustomizer` and `AwsAvailabilityZoneExtension` so 
it's not like this introduces the idea of cloud-specific classes. The 
alternative would be to move them out of Brooklyn and into a 3rd party (i.e. 
non-Apache) project


---
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 #677: Adds EBS Location Customizers

2017-05-12 Thread sjcorbett
Github user sjcorbett commented on the issue:

https://github.com/apache/brooklyn-server/pull/677
  
Those projects are both licensed to Cloudsoft so are perhaps not the best 
examples but my question stands: should we encourage cloud-specific features in 
core Brooklyn? 


---
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 #677: Adds EBS Location Customizers

2017-05-12 Thread sjcorbett
Github user sjcorbett commented on the issue:

https://github.com/apache/brooklyn-server/pull/677
  
Is an AWS-specific feature something we want in Brooklyn or should it be 
provided in a self-contained repository? There's some precedent for the latter. 
- https://github.com/cloudsoft/brooklyn-aws-elb
- https://github.com/cloudsoft/brooklyn-aws-route53



---
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 #677: Adds EBS Location Customizers

2017-05-11 Thread drigodwin
Github user drigodwin commented on the issue:

https://github.com/apache/brooklyn-server/pull/677
  
Would you be able to put some more information on what this PR adds please 
@nakomis? Possibly with an example. Thankyou!


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