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

Introduce kairos.pull_datasources cmdline option to force pulling them #1196

Merged

Conversation

jimmykarily
Copy link
Contributor

@jimmykarily jimmykarily commented Jan 3, 2025

Fixes kairos-io/kairos#3032

If we merge this, it needs docs.

@jimmykarily jimmykarily requested a review from a team January 3, 2025 07:35
@jimmykarily jimmykarily self-assigned this Jan 3, 2025
Copy link
Member

@Itxaka Itxaka left a comment

Choose a reason for hiding this comment

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

Umm I'm not sure about adding it on cmdline, I hate to rely on cmdline stuff for runtime stuff as it's prone to forgetting or in the case of UKI, hardcode

@jimmykarily
Copy link
Contributor Author

Umm I'm not sure about adding it on cmdline, I hate to rely on cmdline stuff for runtime stuff as it's prone to forgetting or in the case of UKI, hardcode

I would also prefer it to be in the user config instead but the overlay files don't have access to the user config, do they? The only option would be to move the condition outside this overlay file and implement this whole stage in immucore itself (where user config is available). Is there any other option I'm missing?

@jimmykarily jimmykarily merged commit 6d46020 into main Jan 3, 2025
11 checks passed
@jimmykarily jimmykarily deleted the 3032_allow_parsing_datasources_using_cmdline_option branch January 3, 2025 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Allow runtime parsing of datasources using a config option
2 participants