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

[fix]: validate decoded bootstrap data does not exceed the user_data byte limit according to the Linode API #94

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

AshleyDumaine
Copy link
Contributor

Fixes #89

@AshleyDumaine AshleyDumaine force-pushed the validate-bootstrap-data-size branch from fd5c96f to 2a0c4d7 Compare February 8, 2024 17:47
@cbzzz
Copy link
Contributor

cbzzz commented Feb 8, 2024

logger.Error("Failed to get bootstrap data", "error", err.Error())

on
https://github.com/linode/cluster-api-provider-linode/blob/2a0c4d72ec59190e203698adf264ed59121d986b/controller/linodemachine_controller_helpers.go?

LTGM! 👍

@AshleyDumaine AshleyDumaine force-pushed the validate-bootstrap-data-size branch from 2a0c4d7 to 5545b85 Compare February 8, 2024 21:15
@AshleyDumaine AshleyDumaine force-pushed the validate-bootstrap-data-size branch from 5545b85 to 5eee2fb Compare February 8, 2024 21:15
@AshleyDumaine AshleyDumaine merged commit 660e466 into main Feb 8, 2024
6 checks passed
@AshleyDumaine AshleyDumaine deleted the validate-bootstrap-data-size branch February 8, 2024 21:47
amold1 pushed a commit that referenced this pull request May 17, 2024
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.

Validate bootstrap data size of Linode instance
3 participants