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

add vendor-data EC2 style path (like user-data) #77

Closed
wants to merge 1 commit into from

Conversation

displague
Copy link
Member

Description

When "vendordata" is included in the hardware metadata, expose it
through the EC2 path structure.

Although vendor-data does not appear to be an EC2 exposed path, tools
like cloud-init make all of the paths listed in 2009-04-04 available to
scripts and cloud-init takes advantage of these paths, including
vendor-data.

Why is this needed

Fixes: #62

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

When "vendordata" is included in the hardware metadata, expose it
through the EC2 path structure.

Although vendor-data does not appear to be an EC2 exposed path, tools
like cloud-init make all of the paths listed in 2009-04-04 available to
scripts and cloud-init takes advantage of these paths, including
vendor-data.

Signed-off-by: Marques Johansson <[email protected]>
@displague
Copy link
Member Author

displague commented Jan 12, 2022

Reviewer note: I don't believe this has a hard dependency on tinkerbell/tink#579, but it may be worth looking at first.

@jacobweinstock
Copy link
Member

Anyone more familiar with this code base able to review? @mmlb , @nshalman ?

@chrisdoherty4
Copy link
Member

@displague Vendor data isn't strictly part of the Equinix metadata standard. I'm not totally sure we should add it.

Given we're developing the new Hegel metadata standard how would you feel about exposing vendor data in that instead?

@chrisdoherty4
Copy link
Member

Closing, see #62.

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.

Implement vendor-data paths
3 participants