Skip to content

Commit

Permalink
migrations: add migrations for kubernete device ownership
Browse files Browse the repository at this point in the history
  • Loading branch information
vigh-m committed Dec 24, 2024
1 parent d35e48a commit 543dbf8
Show file tree
Hide file tree
Showing 5 changed files with 48 additions and 1 deletion.
4 changes: 3 additions & 1 deletion Release.toml
Original file line number Diff line number Diff line change
Expand Up @@ -379,4 +379,6 @@ version = "1.29.0"
"migrate_v1.28.0_aws-control-container-v0-7-18.lz4",
"migrate_v1.28.0_public-control-container-v0-7-18.lz4",
]
"(1.28.0, 1.29.0)" = []
"(1.28.0, 1.29.0)" = [
"migrate_v1.29.0_kubernete-device-ownership.lz4"
]
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 @@ -71,6 +71,7 @@ members = [
"settings-migrations/v1.28.0/public-admin-container-v0-11-14",
"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.1/kubernetes-device-ownership",

"settings-plugins/aws-dev",
"settings-plugins/aws-ecs-1",
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
[package]
name = "kubernetes-device-ownership"
version = "0.1.0"
authors = ["Vighnesh Maheshwari <[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,22 @@
use migration_helpers::common_migrations::{AddMetadataMigration, SettingMetadata};
use migration_helpers::{migrate, Result};
use std::process;

/// We added a new setting, `settings.kubernetes.device-ownership-from-security-context` to allow containers to gain
/// ownership of the requested device
fn run() -> Result<()> {
migrate(AddMetadataMigration(&[SettingMetadata {
metadata: &["affected-services"],
setting: "settings.kubernetes.device-ownership-from-security-context",
}]))
}

// 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 543dbf8

Please sign in to comment.