Switch to serde_yaml_ng

serde_yml turns out to be maintained weirdly (issues disabled, weird refactors)
and is now archived anyway. _ng is what the community is now recommending.

Signed-off-by: Robert Detjens <[email protected]>

Author: detjensrobert

Cargo.lock

@@ -9,7 +9,7 @@ clap = { version = "4.5.4", features = ["unicode", "env", "derive"] }
 itertools = "0.12.1"
 glob = "0.3.1"
 serde = { version = "1.0", features = ["derive"] }
-serde_yml = "0.0.12"
+serde_yaml_ng = "0.10.0"
 serde_nested_with = "0.2.5"
 fully_pub = "0.1.4"
 void = "1"

Cargo.toml

@@ -303,7 +303,7 @@ pub async fn apply_manifest_yaml(
 
     // this manifest has multiple documents (crds, deployment)
     for yaml in multidoc_deserialize(manifest)? {
-        let obj: DynamicObject = serde_yml::from_value(yaml)?;
+        let obj: DynamicObject = serde_yaml_ng::from_value(yaml)?;
         trace!(
             "applying resource {} {}",
             obj.types.clone().unwrap_or_default().kind,
@@ -334,14 +334,14 @@ pub async fn apply_manifest_yaml(
 }
 
 /// Deserialize multi-document yaml string into a Vec of the documents
-fn multidoc_deserialize(data: &str) -> Result<Vec<serde_yml::Value>> {
+fn multidoc_deserialize(data: &str) -> Result<Vec<serde_yaml_ng::Value>> {
     use serde::Deserialize;
 
     let mut docs = vec![];
-    for de in serde_yml::Deserializer::from_str(data) {
-        match serde_yml::Value::deserialize(de)? {
+    for de in serde_yaml_ng::Deserializer::from_str(data) {
+        match serde_yaml_ng::Value::deserialize(de)? {
             // discard any empty documents (e.g. from trailing ---)
-            serde_yml::Value::Null => (),
+            serde_yaml_ng::Value::Null => (),
             not_null => docs.push(not_null),
         };
     }

src/clients.rs

@@ -16,7 +16,7 @@ use kube::{Api, ResourceExt};
 use minijinja;
 use owo_colors::OwoColorize;
 use serde;
-use serde_yml;
+use serde_yaml_ng;
 use tempfile;
 use tracing::{debug, error, info, trace, warn};
 
@@ -95,7 +95,7 @@ pub async fn install_extdns(profile: &config::ProfileConfig) -> Result<()> {
     let values = render_strict(
         VALUES_TEMPLATE,
         minijinja::context! {
-            provider_credentials => serde_yml::to_string(&profile.dns)?,
+            provider_credentials => serde_yaml_ng::to_string(&profile.dns)?,
             chal_domain => profile.challenges_domain
         },
     )?;

src/cluster_setup/mod.rs

@@ -142,7 +142,7 @@ struct ProfileConfig {
     kubeconfig: Option<String>,
     kubecontext: String,
     s3: S3Config,
-    dns: serde_yml::Value,
+    dns: serde_yaml_ng::Value,
 }
 
 #[derive(Debug, PartialEq, Clone, Serialize, Deserialize)]

src/configparser/config.rs

@@ -111,7 +111,7 @@ fn all_yaml() {
                         access_key: "accesskey".to_string(),
                         secret_key: "secretkey".to_string(),
                     },
-                    dns: serde_yml::to_value(HashMap::from([
+                    dns: serde_yaml_ng::to_value(HashMap::from([
                         ("provider", "somebody"),
                         ("thing", "whatever"),
                     ]))
@@ -231,7 +231,7 @@ fn registry_tag_format() {
                         access_key: "accesskey".to_string(),
                         secret_key: "secretkey".to_string(),
                     },
-                    dns: serde_yml::to_value(HashMap::from([
+                    dns: serde_yaml_ng::to_value(HashMap::from([
                         ("provider", "somebody"),
                         ("thing", "whatever"),
                     ]))