Skip to content

Commit

Permalink
chore: rename safenode-version arg to version
Browse files Browse the repository at this point in the history
With the faucet gone, there is no need to qualify the version name.
  • Loading branch information
jacderida committed Nov 3, 2024
1 parent e0cdc47 commit 9519990
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ pub struct UpgradeOptions {
pub interval: Duration,
pub name: String,
pub provider: CloudProvider,
pub safenode_version: Option<String>,
pub version: Option<String>,
}

impl UpgradeOptions {
Expand All @@ -356,7 +356,7 @@ impl UpgradeOptions {
if self.force {
extra_vars.add_variable("force", &self.force.to_string());
}
if let Some(version) = &self.safenode_version {
if let Some(version) = &self.version {
extra_vars.add_variable("safenode_version", version);
}
extra_vars.build()
Expand Down
6 changes: 3 additions & 3 deletions src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -672,7 +672,7 @@ enum Commands {
/// specified to downgrade to a known good version.
///
/// There should be no 'v' prefix.
safenode_version: Option<String>,
version: Option<String>,
},
/// Upgrade the safenode-manager binaries to a particular version.
///
Expand Down Expand Up @@ -2138,7 +2138,7 @@ async fn main() -> Result<()> {
interval,
name,
provider,
safenode_version,
version,
} => {
// The upgrade intentionally uses a small value for `forks`, but this is far too slow
// for retrieving the inventory from a large deployment. Therefore, we will use 50
Expand Down Expand Up @@ -2179,7 +2179,7 @@ async fn main() -> Result<()> {
interval,
name: name.clone(),
provider,
safenode_version,
version,
})?;

// Recreate the deployer with an increased number of forks for retrieving the status.
Expand Down

0 comments on commit 9519990

Please sign in to comment.