Github user nakomis commented on the issue:
https://github.com/apache/brooklyn-server/pull/150
A couple of minor comments, but other than that, LGTM. We should probably
refactor the away the `winrm.computerName` config key, and instead set the
`hostname` sensor correctly during (or immediately after) machine provisioning,
but that can happen in a future PR
Have successfully tested a deployment of Active Directory Services to a
Windows VM on AWS Oregon
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
