Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

design: add azure section to cloud agents #102

Merged
merged 1 commit into from
Dec 20, 2018
Merged

Conversation

arithx
Copy link
Contributor

@arithx arithx commented Dec 19, 2018

Adds design decisions from #65 & #97.

@arithx
Copy link
Contributor Author

arithx commented Dec 19, 2018

cc @dustymabe

Wasn't sure if this ended up being the best place to put it as it's a sub-piece of the Cloud Agents but it seems to kind of jumble things together. Open to ideas.

@dustymabe
Copy link
Member

Wasn't sure if this ended up being the best place to put it as it's a sub-piece of the Cloud Agents

seems appropriate to me

Design.md Outdated Show resolved Hide resolved
Design.md Outdated Show resolved Hide resolved
Design.md Outdated Show resolved Hide resolved
@arithx
Copy link
Contributor Author

arithx commented Dec 19, 2018

Updates pushed ⬆️


- We've identified one major gap with not shipping the [Microsoft Azure Linux Agent](https://github.com/Azure/WALinuxAgent): the machine will not check-in and will eventually be culled by Azure for being stuck in the creation process.
- This gap will be covered by work done in [coreos-metadata](https://github.com/coreos/coreos-metadata/issues/120).
- One additional gap which will __not__ be covered is a lack of ephemeral disk support. We plan to ship udev rules but will not have a service which formats the disk unless we receive feature requests in the future. This was discussed in [#97](https://github.com/coreos/fedora-coreos-tracker/issues/97).
Copy link
Member

Choose a reason for hiding this comment

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

i don't know if there is any link we can provide to users who don't know what ephemeral disk support is. If we can that would be nice to have here. If not then it's fine too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some quick googling led to https://blogs.msdn.microsoft.com/mast/2013/12/06/understanding-the-temporary-drive-on-windows-azure-virtual-machines/ but the concept of ephemeral storage is pretty consistent among cloud providers (for instance: AWS info here https://heitorlessa.com/working-with-amazon-aws-ec2-ephemeral-disks-fcefd46e4ec)

Copy link
Member

Choose a reason for hiding this comment

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

right. I think users will know what ephemeral disks are, but they might not know what ephemeral disk support means. Never having used Azure much, and mostly working with AWS in the past, I didn't know what was meant by that phrase. I think we're good here for now though since later in the paragraph we mention formatting the disk(s).

@dustymabe
Copy link
Member

some final comments. in general LGTM

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.

None yet

2 participants