fix(containers): Docker update fails on M Macs due to platform / (#1360)
Suppresses the `--platform` parameter to docker pull if platform is only '/'. fixes #1154
This commit is contained in:
@@ -142,12 +142,12 @@ pub fn run_containers(ctx: &ExecutionContext) -> Result<()> {
|
|||||||
|
|
||||||
for container in &containers {
|
for container in &containers {
|
||||||
debug!("Pulling container '{}'", container);
|
debug!("Pulling container '{}'", container);
|
||||||
let args = vec![
|
let mut args = vec!["pull", container.repo_tag.as_str()];
|
||||||
"pull",
|
if container.platform.as_str() != "/" {
|
||||||
container.repo_tag.as_str(),
|
args.push("--platform");
|
||||||
"--platform",
|
args.push(container.platform.as_str());
|
||||||
container.platform.as_str(),
|
}
|
||||||
];
|
|
||||||
let mut exec = ctx.execute(&crt);
|
let mut exec = ctx.execute(&crt);
|
||||||
|
|
||||||
if let Err(e) = exec.args(&args).status_checked() {
|
if let Err(e) = exec.args(&args).status_checked() {
|
||||||
|
|||||||
Reference in New Issue
Block a user