You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by bostko <gi...@git.apache.org> on 2017/04/28 14:20:36 UTC
[GitHub] brooklyn-docs pull request #176: imageId example for Azure Compute ARM
GitHub user bostko opened a pull request:
https://github.com/apache/brooklyn-docs/pull/176
imageId example for Azure Compute ARM
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bostko/brooklyn-docs azure-arm/imageId-hint
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/brooklyn-docs/pull/176.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #176
----
commit 7d3d294d4fc83bc39956ded30f2c8e4c2c43e067
Author: Valentin Aitken <bo...@gmail.com>
Date: 2017-04-28T14:10:49Z
imageId example for Azure Compute ARM
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-docs issue #176: imageId example for Azure Compute ARM
Posted by bostko <gi...@git.apache.org>.
Github user bostko commented on the issue:
https://github.com/apache/brooklyn-docs/pull/176
@tbouron I agree we should look why `osVersionRegex: 7\..*` does not choose the latest and probably fire a bug instead of adding such note.
Closing PR. I will create a jira issue at some point.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-docs issue #176: imageId example for Azure Compute ARM
Posted by tbouron <gi...@git.apache.org>.
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-docs/pull/176
@bostko Can you reply and update your PR, based on @drigodwin's comments?
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-docs pull request #176: imageId example for Azure Compute ARM
Posted by drigodwin <gi...@git.apache.org>.
Github user drigodwin commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/176#discussion_r113957513
--- Diff: guide/locations/_azure-ARM.md ---
@@ -95,6 +95,17 @@ The loginUser can be anything, as long as it's specified.
The `overrideAuthenticateSudo: true` key tells Apache Brooklyn that default on Azure images do not have passwordless sudo
configured by default.
+ imageId: westeurope/OpenLogic/CentOS/7.3
+
+Listing images in Azure CLI [https://docs.microsoft.com/en-us/cli/azure/vm/image#list-skus](https://docs.microsoft.com/en-us/cli/azure/vm/image#list-skus).
+A VM Image can be specified using Brooklyn location config `imageId`.
+For Azure ARM, Apache Brooklyn expects `imageId` in one of the following formats:
+
+- `location/group/storage/offer/name`
+- `location/publisher/offer/sku`
+
+Consider using imageId sku instead of `osVersion` or `osVersionRegex`.
--- End diff --
Why should they consider this? I would remove this line.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-docs pull request #176: imageId example for Azure Compute ARM
Posted by drigodwin <gi...@git.apache.org>.
Github user drigodwin commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/176#discussion_r113957425
--- Diff: guide/locations/_azure-ARM.md ---
@@ -95,6 +95,17 @@ The loginUser can be anything, as long as it's specified.
The `overrideAuthenticateSudo: true` key tells Apache Brooklyn that default on Azure images do not have passwordless sudo
configured by default.
+ imageId: westeurope/OpenLogic/CentOS/7.3
+
+Listing images in Azure CLI [https://docs.microsoft.com/en-us/cli/azure/vm/image#list-skus](https://docs.microsoft.com/en-us/cli/azure/vm/image#list-skus).
--- End diff --
Why should users use this over [cloud:list-images](http://docs.cloudsoftdev.io/operations/tools/index.html#cloud-compute-tools)
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-docs pull request #176: imageId example for Azure Compute ARM
Posted by bostko <gi...@git.apache.org>.
Github user bostko closed the pull request at:
https://github.com/apache/brooklyn-docs/pull/176
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---