Skip to content

DLPX-93823 cloud-init configuration contains module names that are no longer correct #519

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sebroy
Copy link
Contributor

@sebroy sebroy commented Apr 1, 2025

A few years ago, a cloud-init developer decided to rename all of the cloud-init
modules that had embedded hyphens in their name to replace those hyphens with
underscores. Our custom cloud-init configuration was never adjusted to account
for this.

This is also a good opportunity to verify what all of these modules do and validate
the list. If you're curious to know what each of these is supposed to do, they're
all documented at https://github.com/delphix/cloud-init/tree/develop/doc/module-docs (or https://docs.cloud-init.io/en/latest/reference/modules.html if you want a more user-friendly way to consume that).

Note that because we already configure cloud-init with allow_userdata: false, none of these
modules have access to any externally provided user data, which eliminates the
possibility of a cloud or hypervisor admin tampering with the Delphix configuration.

ab-pre-push: https://selfservice-jenkins.eng-tools-prd.aws.delphixcloud.com/job/appliance-build-orchestrator-pre-push/10695/

@sebroy sebroy force-pushed the dlpx/pr/sebroy/7b3f9616-29e3-4adc-896f-30b831fe81e4 branch from ed3a1a0 to af5abee Compare April 1, 2025 00:01
@sebroy sebroy marked this pull request as ready for review April 1, 2025 00:34
@sebroy sebroy force-pushed the dlpx/pr/sebroy/7b3f9616-29e3-4adc-896f-30b831fe81e4 branch from af5abee to ba69444 Compare April 1, 2025 01:49
@sebroy sebroy force-pushed the dlpx/pr/sebroy/7b3f9616-29e3-4adc-896f-30b831fe81e4 branch from ba69444 to bf74b15 Compare April 1, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants