chore: Make progress bar template reusable through a LazyLock

Author: NickLarsenNZ

rust/stackable-cockpit/src/lib.rs

@@ -1,3 +1,7 @@
+use std::sync::LazyLock;
+
+use indicatif::ProgressStyle;
+
 pub mod common;
 pub mod constants;
 pub mod engine;
@@ -6,3 +10,10 @@ pub mod oci;
 pub mod platform;
 pub mod utils;
 pub mod xfer;
+
+pub(crate) static PROGRESS_BAR_STYLE: LazyLock<ProgressStyle> = LazyLock::new(|| {
+    ProgressStyle::with_template(
+        "{span_child_prefix} Progress {msg}: {wide_bar:.magenta/cyan} {pos}/{len}",
+    )
+    .expect("valid progress template")
+});

rust/stackable-cockpit/src/platform/manifests.rs

@@ -7,6 +7,7 @@ use tracing::{Instrument as _, Span, debug, info, info_span, instrument};
 use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 
 use crate::{
+    PROGRESS_BAR_STYLE,
     common::manifest::ManifestSpec,
     helm,
     utils::{
@@ -76,10 +77,7 @@ pub trait InstallManifestsExt {
     ) -> Result<(), Error> {
         debug!("Installing manifests");
 
-        Span::current().pb_set_style(
-            &ProgressStyle::with_template("Progress: {wide_bar} {pos}/{len}")
-                .expect("valid progress template"),
-        );
+        Span::current().pb_set_style(&PROGRESS_BAR_STYLE);
         Span::current().pb_set_length(manifests.len() as u64);
 
         let mut parameters = parameters.clone();

rust/stackable-cockpit/src/platform/release/spec.rs

@@ -1,6 +1,5 @@
 use futures::{StreamExt as _, TryStreamExt};
 use indexmap::IndexMap;
-use indicatif::ProgressStyle;
 use serde::{Deserialize, Serialize};
 use snafu::{ResultExt, Snafu};
 use tokio::task::JoinError;
@@ -10,7 +9,7 @@ use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 use utoipa::ToSchema;
 
 use crate::{
-    helm,
+    PROGRESS_BAR_STYLE, helm,
     platform::{
         operator::{self, ChartSourceType, OperatorSpec},
         product,
@@ -64,10 +63,7 @@ impl ReleaseSpec {
         chart_source: &ChartSourceType,
     ) -> Result<()> {
         info!("Installing release");
-        Span::current().pb_set_style(
-            &ProgressStyle::with_template("Progress: {wide_bar} {pos}/{len}")
-                .expect("valid progress template"),
-        );
+        Span::current().pb_set_style(&PROGRESS_BAR_STYLE);
 
         include_products.iter().for_each(|product| {
             Span::current().record("product.included", product);
@@ -125,10 +121,7 @@ impl ReleaseSpec {
     pub fn uninstall(&self, namespace: &str) -> Result<()> {
         info!("Uninstalling release");
 
-        Span::current().pb_set_style(
-            &ProgressStyle::with_template("Progress: {wide_bar} {pos}/{len}")
-                .expect("valid progress template"),
-        );
+        Span::current().pb_set_style(&PROGRESS_BAR_STYLE);
         Span::current().pb_set_length(self.products.len() as u64);
 
         for (product_name, product_spec) in &self.products {