Sort operator versions descending (#102)

## Description

For better readable output

Author: sbernauer

CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - Print Nifi adminuser credentials within services command ([#80](https://github.com/stackabletech/stackablectl/pull/80))
 
+### Changed
+
+- Sort operator versions descending ([#102](https://github.com/stackabletech/stackablectl/pull/102))
+
 ### Removed
 - Remove Spark from supported operators ([#100](https://github.com/stackabletech/stackablectl/pull/100))
 

Cargo.lock

@@ -20,6 +20,7 @@ lazy_static = "1.4"
 log = "0.4"
 openssl = { version = "0.10.36", features = ["vendored"] } # Must match version from kube
 which = "4.2"
+semver = "1.0"
 serde = { version = "1.0", features = ["derive"]}
 serde_json = "1.0"
 serde_yaml = "0.9"

Cargo.toml

@@ -2,6 +2,7 @@ use crate::{arguments::OutputType, helm, helm::HELM_REPOS, kind, AVAILABLE_OPERA
 use clap::{Parser, ValueHint};
 use indexmap::IndexMap;
 use log::{info, warn};
+use semver::Version;
 use serde::Serialize;
 use std::{error::Error, str::FromStr};
 
@@ -167,6 +168,8 @@ async fn describe_operator(operator: &str, output_type: &OutputType) -> Result<(
     Ok(())
 }
 
+/// Returns list of available versions for a specific operator within a helm repo.
+/// Uses the semver crate to parse and sort the versions descending.
 async fn get_versions_from_repo(
     operator: &str,
     helm_repo_name: &str,
@@ -188,11 +191,18 @@ async fn get_versions_from_repo(
             warn!("Could not find {operator} operator (chart name {chart_name}) in helm repo {helm_repo_name}");
             Ok(vec![])
         }
-        Some(versions) => Ok(versions
-            .iter()
-            .map(|entry| entry.version.clone())
-            .rev()
-            .collect()),
+        Some(repo_entries) => {
+            let mut versions = repo_entries
+                .iter()
+                .map(|entry| Version::parse(&entry.version))
+                .collect::<Result<Vec<Version>, _>>()?;
+            versions.sort();
+            Ok(versions
+                .iter()
+                .rev()
+                .map(|version| version.to_string())
+                .collect())
+        }
     }
 }