bottlerocket
bottlerocket copied to clipboard
thar-be-updates: migrate to use configuration file
Issue number: #3625
Closes #3625
Description of changes:
This change migrates the thar-be-updates tool to utilize a configuration file at /etc/thar-be-updates.toml instead of calling back to the settings api.
log:
- migration and template
- migrate thar-be-updates tool to read settings from a config file
Testing done: Tested update from 1.18.0 -> 1.18.1 and ensured settings get populated appropriately
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.
fix build issue where i left unused imports
- Rebased off latest
- Tested upgrade -> downgrade ensured everything is clean
- Removed tokio dependency
- Adjusted config_path use
error: unused import: `PathBuf`
--> api/thar-be-updates/src/status.rs:13:23
|
13 | use std::path::{Path, PathBuf};
| ^^^^^^^
|
= note: `-D unused-imports` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_imports)]`
updated on latest
Fix the if condition around the boolean value causing it to not be rendered when false
Updated per ben's comments, will begin testing now