Skip to content
This repository was archived by the owner on Mar 2, 2022. It is now read-only.

Add Azure building instructions#50

Open
liztio wants to merge 1 commit into
vmware-archive:masterfrom
liztio:azure
Open

Add Azure building instructions#50
liztio wants to merge 1 commit into
vmware-archive:masterfrom
liztio:azure

Conversation

@liztio

@liztio liztio commented Mar 15, 2018

Copy link
Copy Markdown
Contributor

Signed-off-by: liz liz@heptio.com

Signed-off-by: liz <liz@heptio.com>
@liztio liztio requested a review from craigtracey March 15, 2018 21:48
Comment thread packer/README.md

```
/path/to/packer build -var-file <YOUR REGION>.json -var kubernetes_version=<YOUR K8S VERSION> -var kubernetes_cni_version=<YOUR K8S CNI VERSION> -var build_version=`git rev-parse HEAD` packer.json
/path/to/packer build -var-file <YOUR REGION>.json -var kubernetes_version=<YOUR K8S VERSION> -var kubernetes_cni_version=<YOUR K8S CNI VERSION> -var build_version=`git rev-parse HEAD` -only ami-ubuntu-16.04 packer.json

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

centos is also an available target.

Comment thread packer/README.md

[script]: https://www.packer.io/docs/builders/azure-setup.html#guided-setup

Creating the image ------------------ The script will spit out a bunch of URLs,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like formatting errors here.

Comment thread packer/README.md

Creating the image ------------------ The script will spit out a bunch of URLs,
which can be very confusing. If you go into the azure console, however, you will
not see your image in [VM Images][images]. That is because this packer crosses a

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

s/crosses/creates/ ?

Comment thread packer/packer.json
"kubernetes_cni_version": "{{user `kubernetes_cni_version`}}"
}
},
{

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would like for us to maintain parity with other clouds with regard to baseline support (ie. Ubuntu and CentOS). Happy to take CentOS in a follow-up PR, but we should Issue it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm looking through CentOS images and I don't see any "official" CentOS images:

$ az vm image list -f CentOS --all | jq -r '.[].publisher' | sort | uniq
MicrosoftRServer
OpenLogic
RightScaleLinux
batch
center-for-internet-security-inc
cloudera
cloudhub-technologies
cognosys
ingrammicro
saltstack
scalegrid
tokyosystemhouse
tunnelbiz
ubercloud
wmspanel

any idea what we should do there?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like the Azure supported version is from OpenLogic.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants