Allow Persistent Data Volume to be Created Outside of Package

This changes the unique container IDs to be based off the toolchain ID, the host target name, the hash of the current working directory, the system time. This means it will be fully unique at the millisecond resolution. This also separated package and toolchain directories. Toolchain directories are the only ones required when creating persistent data volumes, while running cross requires both. There are no backwards incompatible changes, since the persistent data volumes stay the same.

Author: Alexhuszagh

.changes/1054.json

@@ -0,0 +1,4 @@
+{
+    "type": "internal",
+    "description": "change the unique container ID to be unique based off the toolchain and system time."
+}

src/bin/commands/containers.rs

@@ -395,10 +395,10 @@ pub fn create_persistent_volume(
     if let Some(channel) = channel {
         toolchain.channel = channel.channel.clone();
     };
-    let (dirs, metadata) = docker::get_package_info(engine, toolchain.clone(), msg_info)?;
-    let container =
-        docker::remote::unique_container_identifier(&toolchain.host().target, &metadata, &dirs)?;
-    let volume = dirs.toolchain.unique_toolchain_identifier()?;
+    let mount_finder = docker::MountFinder::create(engine)?;
+    let dirs = docker::ToolchainDirectories::assemble(&mount_finder, toolchain.clone())?;
+    let container = dirs.unique_container_identifier(&toolchain.host().target)?;
+    let volume = dirs.unique_toolchain_identifier()?;
 
     if docker::remote::volume_exists(engine, &volume, msg_info)? {
         eyre::bail!("Error: volume {volume} already exists.");
@@ -482,8 +482,9 @@ pub fn remove_persistent_volume(
     if let Some(channel) = channel {
         toolchain.channel = channel.channel.clone();
     };
-    let (dirs, _) = docker::get_package_info(engine, toolchain, msg_info)?;
-    let volume = dirs.toolchain.unique_toolchain_identifier()?;
+    let mount_finder = docker::MountFinder::create(engine)?;
+    let dirs = docker::ToolchainDirectories::assemble(&mount_finder, toolchain)?;
+    let volume = dirs.unique_toolchain_identifier()?;
 
     if !docker::remote::volume_exists(engine, &volume, msg_info)? {
         eyre::bail!("Error: volume {volume} does not exist.");

src/docker/local.rs

@@ -57,17 +57,17 @@ pub(crate) fn run(
     docker
         .args([
             "-v",
-            &format!("{}:{}:z", dirs.xargo.to_utf8()?, dirs.xargo_mount_path()),
+            &format!("{}:{}:z", dirs.xargo_host_path()?, dirs.xargo_mount_path()),
         ])
         .args([
             "-v",
-            &format!("{}:{}:z", dirs.cargo.to_utf8()?, dirs.cargo_mount_path()),
+            &format!("{}:{}:z", dirs.cargo_host_path()?, dirs.cargo_mount_path()),
         ])
         // Prevent `bin` from being mounted inside the Docker container.
         .args(["-v", &format!("{}/bin", dirs.cargo_mount_path())]);
     docker.args([
         "-v",
-        &format!("{}:{}:z", dirs.host_root.to_utf8()?, dirs.mount_root),
+        &format!("{}:{}:z", dirs.host_root().to_utf8()?, dirs.mount_root()),
     ]);
     docker
         .args([
@@ -78,12 +78,12 @@ pub(crate) fn run(
                 dirs.sysroot_mount_path()
             ),
         ])
-        .args(["-v", &format!("{}:/target:z", dirs.target.to_utf8()?)]);
+        .args(["-v", &format!("{}:/target:z", dirs.target().to_utf8()?)]);
     docker_cwd(&mut docker, &paths)?;
 
     // When running inside NixOS or using Nix packaging we need to add the Nix
     // Store to the running container so it can load the needed binaries.
-    if let Some(ref nix_store) = dirs.nix_store {
+    if let Some(nix_store) = dirs.nix_store() {
         docker.args([
             "-v",
             &format!(