Skip to content

Commit

Permalink
migrations: Change public control host-container source metadata to o…
Browse files Browse the repository at this point in the history
…bject

This migration will change the public control host-container source to
setting-generator metadata(that will generate source using sundog)
on forward migration.

In backward migration, it will reset the value of the source as string.
  • Loading branch information
vyaghras committed Dec 6, 2024
1 parent 3323f8e commit 7356b2c
Show file tree
Hide file tree
Showing 5 changed files with 54 additions and 0 deletions.
1 change: 1 addition & 0 deletions Release.toml
Original file line number Diff line number Diff line change
Expand Up @@ -381,6 +381,7 @@ version = "1.28.0"
]
"(1.28.0, 1.29.0)" = [
"migrate_v1.29.0_update-settings-generator-admin.lz4",
"migrate_v1.29.0_update-settings-generator-control.lz4",
"migrate_v1.29.0_change-public-admin-container-to-set-gen.lz4",
"migrate_v1.29.0_change-public-control-container-to-set-gen.lz4",
]
Expand Down
7 changes: 7 additions & 0 deletions sources/Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions sources/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ members = [
"settings-migrations/v1.28.0/aws-control-container-v0-7-18",
"settings-migrations/v1.28.0/public-control-container-v0-7-18",
"settings-migrations/v1.29.0/update-settings-generator-admin",
"settings-migrations/v1.29.0/update-settings-generator-control",
"settings-migrations/v1.29.0/change-public-admin-container-to-set-gen",
"settings-migrations/v1.29.0/change-public-control-container-to-set-gen",

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
[package]
name = "update-settings-generator-control"
version = "0.1.0"
authors = ["Shikha Vyaghra <[email protected]>"]
license = "Apache-2.0 OR MIT"
edition = "2021"
publish = false
# Don't rebuild crate just because of changes to README.
exclude = ["README.md"]


# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
migration-helpers.workspace = true
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
use migration_helpers::common_migrations::MetadataStringToStructMigration;
use migration_helpers::{migrate, Result};
use std::process;

const OLD_CONTROL_CTR_CMDLINE: &str =
"schnauzer-v2 render --requires 'aws@v1(helpers=[ecr-prefix])' --template '{{ ecr-prefix settings.aws.region }}/bottlerocket-control:v0.7.18'";
const NEW_CONTROL_CTR_CMDLINE: &str =
"schnauzer-v2 render --requires 'aws@v1(helpers=[ecr-prefix])' --template '{{ ecr-prefix settings.aws.region }}/bottlerocket-control:v0.7.18'";

/// We bumped the version of the default control container
fn run() -> Result<()> {
migrate(MetadataStringToStructMigration {
setting: "settings.host-containers.control.source",
old_cmdline: OLD_CONTROL_CTR_CMDLINE,
new_cmdline: NEW_CONTROL_CTR_CMDLINE,
metadata_type: "setting-generator",
binary_for_generator: "schnauzer-v2",
skip_if_populated: true,
})
}

// Returning a Result from main makes it print a Debug representation of the error, but with Snafu
// we have nice Display representations of the error, so we wrap "main" (run) and print any error.
// https://github.com/shepmaster/snafu/issues/110
fn main() {
if let Err(e) = run() {
eprintln!("{}", e);
process::exit(1);
}
}

0 comments on commit 7356b2c

Please sign in to comment.