Add deployment.keys.<name>.uploadAt

This mirrors the functionality recently added in morph and allows
for the uploading of keys after system profile activation.

Fixes #10.

README.md

@@ -258,6 +258,8 @@ For example, to deploy DNS-01 credentials for use with `security.acme`:
       user = "acme";               # Default: root
       group = "nginx";             # Default: root
       permissions = "0640";        # Default: 0600
+
+      uploadAt = "pre-activation"; # Default: pre-activation, Alternative: post-activation
     };
     # Rest of configuration...
   };

src/nix/deployment.rs

@@ -6,6 +6,7 @@ use futures::future::join_all;
 use tokio::sync::{Mutex, Semaphore};
 
 use super::{Hive, Host, CopyOptions, NodeConfig, Profile, StoreDerivation, ProfileMap, host};
+use super::key::{Key, UploadAt};
 use crate::progress::{Progress, TaskProgress, OutputStyle};
 
 /// Amount of RAM reserved for the system, in MB.
@@ -523,10 +524,20 @@ impl Deployment {
             }
         }
 
-        if self.options.upload_keys && !target.config.keys.is_empty() {
+        let pre_activation_keys = target.config.keys.iter()
+            .filter(|(_, v)| v.upload_at() == UploadAt::PreActivation)
+            .map(|(k, v)| (k.clone(), v.clone()))
+            .collect::<HashMap<String, Key>>();
+
+        let post_activation_keys = target.config.keys.iter()
+            .filter(|(_, v)| v.upload_at() == UploadAt::PostActivation)
+            .map(|(k, v)| (k.clone(), v.clone()))
+            .collect::<HashMap<String, Key>>();
+
+        if self.options.upload_keys && !pre_activation_keys.is_empty() {
             bar.log("Uploading keys...");
 
-            if let Err(e) = target.host.upload_keys(&target.config.keys).await {
+            if let Err(e) = target.host.upload_keys(&pre_activation_keys).await {
                 bar.failure_err(&e);
 
                 let mut results = self.results.lock().await;
@@ -546,6 +557,21 @@ impl Deployment {
 
         match target.host.deploy(&profile, self.goal, copy_options).await {
             Ok(_) => {
+                // FIXME: This is ugly
+                if self.options.upload_keys && !post_activation_keys.is_empty() {
+                    bar.log("Uploading keys (post-activation)...");
+
+                    if let Err(e) = target.host.upload_keys(&post_activation_keys).await {
+                        bar.failure_err(&e);
+
+                        let mut results = self.results.lock().await;
+                        let stage = Stage::Apply(name.to_string());
+                        let logs = target.host.dump_logs().await.map(|s| s.to_string());
+                        results.push(DeploymentResult::failure(stage, logs));
+                        return;
+                    }
+                }
+
                 bar.success(self.goal.success_str().unwrap());
 
                 let mut results = self.results.lock().await;

src/nix/eval.nix

@@ -241,6 +241,18 @@ let
         default = "0600";
         type = types.str;
       };
+      uploadAt = lib.mkOption {
+        description = ''
+          When to upload the keys.
+
+          - pre-activation (default): Upload the keys before activating the new system profile.
+          - post-activation: Upload the keys after successfully activating the new system profile.
+
+          For `colmena upload-keys`, all keys are uploaded at the same time regardless of the configuration here.
+        '';
+        default = "pre-activation";
+        type = types.enum [ "pre-activation" "post-activation" ];
+      };
     };
   };
 

src/nix/key.rs

@@ -75,6 +75,18 @@ struct KeySources {
     file: Option<PathBuf>,
 }
 
+/// When to upload a given key.
+#[derive(Debug, Copy, Clone, PartialEq, Serialize, Deserialize)]
+pub enum UploadAt {
+    /// Before activating the system profile.
+    #[serde(rename = "pre-activation")]
+    PreActivation,
+
+    /// After successfully activating the system profile.
+    #[serde(rename = "post-activation")]
+    PostActivation,
+}
+
 #[derive(Debug, Clone, Validate, Serialize, Deserialize)]
 pub struct Key {
     #[serde(flatten)]
@@ -91,6 +103,9 @@ pub struct Key {
     group: String,
 
     permissions: String,
+
+    #[serde(rename = "uploadAt")]
+    upload_at: UploadAt,
 }
 
 impl Key {
@@ -133,6 +148,7 @@ impl Key {
     pub fn user(&self) -> &str { &self.user }
     pub fn group(&self) -> &str { &self.group }
     pub fn permissions(&self) -> &str { &self.permissions }
+    pub fn upload_at(&self) -> UploadAt { self.upload_at }
 }
 
 fn validate_unix_name(name: &str) -> Result<(), ValidationError> {