Skip to content

Commit

Permalink
Merge pull request #2471 from jpculp/admin-container-v0.9.2
Browse files Browse the repository at this point in the history
Migrations: migrate to admin container v0.9.2
  • Loading branch information
jpculp authored Oct 5, 2022
2 parents ea9634f + ca4e7c7 commit 29fc6f2
Show file tree
Hide file tree
Showing 9 changed files with 100 additions and 2 deletions.
2 changes: 2 additions & 0 deletions Release.toml
Original file line number Diff line number Diff line change
Expand Up @@ -147,4 +147,6 @@ version = "1.10.0"
"migrate_v1.10.0_dns-settings-metadata.lz4",
"migrate_v1.10.0_reboot-to-reconcile-setting.lz4",
"migrate_v1.10.0_kubelet-log-level.lz4",
"migrate_v1.10.0_aws-admin-container-v0-9-2.lz4",
"migrate_v1.10.0_public-admin-container-v0-9-2.lz4",
]
14 changes: 14 additions & 0 deletions sources/Cargo.lock

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

2 changes: 2 additions & 0 deletions sources/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ members = [
"api/migration/migrations/v1.10.0/dns-settings-metadata",
"api/migration/migrations/v1.10.0/reboot-to-reconcile-setting",
"api/migration/migrations/v1.10.0/kubelet-log-level",
"api/migration/migrations/v1.10.0/aws-admin-container-v0-9-2",
"api/migration/migrations/v1.10.0/public-admin-container-v0-9-2",

"bottlerocket-release",

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
[package]
name = "aws-admin-container-v0-9-2"
version = "0.1.0"
authors = ["Patrick J.P. Culp <[email protected]>"]
license = "Apache-2.0 OR MIT"
edition = "2018"
publish = false
# Don't rebuild crate just because of changes to README.
exclude = ["README.md"]

[dependencies]
migration-helpers = { path = "../../../migration-helpers", version = "0.1.0"}
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
#![deny(rust_2018_idioms)]

use migration_helpers::common_migrations::ReplaceTemplateMigration;
use migration_helpers::{migrate, Result};
use std::process;

const OLD_ADMIN_CTR_TEMPLATE: &str =
"{{ ecr-prefix settings.aws.region }}/bottlerocket-admin:v0.9.0";
const NEW_ADMIN_CTR_TEMPLATE: &str =
"{{ ecr-prefix settings.aws.region }}/bottlerocket-admin:v0.9.2";

/// We bumped the version of the default admin container
fn run() -> Result<()> {
migrate(ReplaceTemplateMigration {
setting: "settings.host-containers.admin.source",
old_template: OLD_ADMIN_CTR_TEMPLATE,
new_template: NEW_ADMIN_CTR_TEMPLATE,
})
}

// 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);
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
[package]
name = "public-admin-container-v0-9-2"
version = "0.1.0"
authors = ["Patrick J.P. Culp <[email protected]>"]
license = "Apache-2.0 OR MIT"
edition = "2018"
publish = false
# Don't rebuild crate just because of changes to README.
exclude = ["README.md"]

[dependencies]
migration-helpers = { path = "../../../migration-helpers", version = "0.1.0"}
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
#![deny(rust_2018_idioms)]

use migration_helpers::common_migrations::ReplaceStringMigration;
use migration_helpers::{migrate, Result};
use std::process;

const OLD_ADMIN_CTR_SOURCE_VAL: &str = "public.ecr.aws/bottlerocket/bottlerocket-admin:v0.9.0";
const NEW_ADMIN_CTR_SOURCE_VAL: &str = "public.ecr.aws/bottlerocket/bottlerocket-admin:v0.9.2";

/// We bumped the version of the default admin container
fn run() -> Result<()> {
migrate(ReplaceStringMigration {
setting: "settings.host-containers.admin.source",
old_val: OLD_ADMIN_CTR_SOURCE_VAL,
new_val: NEW_ADMIN_CTR_SOURCE_VAL,
})
}

// 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);
}
}
2 changes: 1 addition & 1 deletion sources/models/shared-defaults/aws-host-containers.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ superpowered = true

[metadata.settings.host-containers.admin.source]
setting-generator = "schnauzer settings.host-containers.admin.source"
template = "{{ ecr-prefix settings.aws.region }}/bottlerocket-admin:v0.9.0"
template = "{{ ecr-prefix settings.aws.region }}/bottlerocket-admin:v0.9.2"

[metadata.settings.host-containers.admin.user-data]
setting-generator = "shibaken generate-admin-userdata"
Expand Down
2 changes: 1 addition & 1 deletion sources/models/shared-defaults/public-host-containers.toml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
[settings.host-containers.admin]
enabled = false
superpowered = true
source = "public.ecr.aws/bottlerocket/bottlerocket-admin:v0.9.0"
source = "public.ecr.aws/bottlerocket/bottlerocket-admin:v0.9.2"

[settings.host-containers.control]
enabled = false
Expand Down

0 comments on commit 29fc6f2

Please sign in to comment.