feat: Add support for human-readable duration formats (#647)

* Update author constant

* Add humantime Duration

* Update changelog

* Add `from_secs` and `from_millis` helper functions

* Trim trailing whitespace using pre-commit hook

* Fix markdownlint errors using pre-commit hook

* Implement first iteration of custom parsing

* Remove humantime dependency

* Add `Display` impl, add various `ops` impls

* Add negative tests

* Rework `Duration` implementation based on @nightkr gist

This reworks/simplifies the `Duration` implementation. Supported
units are now clearly defned by an enum and the parsing/display
mechanism is now way more explicit and defined in one place.

Co-authored-by: Natalie Klestrup Röijezon <teo.roijezon@stackable.de>

* Add documentation

Co-authored-by: Natalie Klestrup Röijezon <teo.roijezon@stackable.de>

* Fix module documentation links

Co-authored-by: Natalie Klestrup Röijezon <teo.roijezon@stackable.de>

* Fix bug

We need to check if the iterator progressed based on the start index of
the unit / the end index of the numeric value.

* Update parser, add context to errors, add more tests

This commit now parses durations in the Go format. This format
doesn't use whitespaces to separate each fragment. Instead, all
fragments are concated without any space between. The parser needs
to be a little cleverer when trying to keep it streamlined.

Additionally, the error variants now contain some more context to
make it easier for the user to spot the error in the provided value.

Co-authored-by: Natalie Klestrup Röijezon <teo.roijezon@stackable.de>

* Update src/duration/mod.rs

Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>

* Re-add millisecond support, switch to Snafu

* Fix markdown link references

* duration::serde_impl exports nothing

* Split duration parsing context selectors into a submodule

* typo: "expected character" -> "unexpected character"

* Fail if duration buffer is non-empty at the end of parsing

* Use Debug impls to quote strings in errors

* Non-ASCII input will be caught anyway, because it's never a valid character

* DurationUnit doesn't need to be public

* Add check for invalid unit order and duplicate units

* Revert "Non-ASCII input will be caught anyway, because it's never a valid character"

This reverts commit fe1911e.

* Fix doc comment

* Impl Mul and Div

* Impl Mul and Div 2

* impl JsonSchema for Duration

* Re-order `SubAssign` impl

* Revert "Revert "Non-ASCII input will be caught anyway, because it's never a valid character""

This reverts commit 2d3d32c.

* Remove `impl Div<Duration> for u32`

* Fix scanning of multibyte chars

---------

Co-authored-by: Natalie Klestrup Röijezon <teo.roijezon@stackable.de>
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>

Author: 3 people

.github/pull_request_template.md

@@ -1,4 +1,4 @@
-## Description
+# Description
 
 *Please add a description here. This will become the commit message of the merge request later.*
 
@@ -7,7 +7,6 @@
 - Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
 - Please make sure all these things are done and tick the boxes
 
-
 ```[tasklist]
 # Author
 - [ ] Changes are OpenShift compatible

CHANGELOG.md

@@ -10,6 +10,11 @@ All notable changes to this project will be documented in this file.
 
 - `PodListeners` CRD ([#644]).
 - Add support for tls pkcs12 password to secret operator volume builder ([#645]).
+- Add `Duration` capable of parsing human-readable duration formats ([#647])
+
+[#644]: https://github.com/stackabletech/operator-rs/pull/644
+[#645]: https://github.com/stackabletech/operator-rs/pull/645
+[#647]: https://github.com/stackabletech/operator-rs/pull/647
 
 ### Changed
 

src/cli.rs

@@ -116,7 +116,7 @@ use std::{
     path::{Path, PathBuf},
 };
 
-pub const AUTHOR: &str = "Stackable GmbH - info@stackable.de";
+pub const AUTHOR: &str = "Stackable GmbH - info@stackable.tech";
 
 /// Framework-standardized commands
 ///

src/commons/authentication/static_.rs

@@ -17,7 +17,7 @@
 //!   The update time depends on the kubelet syncing period.
 //!   This would need additional functionality in restart controller to white- or blacklist certain volumes. Additionally, we would need a sidecar container that
 //!   periodically converts the secret contents to the required product format.
-//!  
+//!
 //! See <https://github.com/stackabletech/operator-rs/issues/494>
 use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};

src/commons/opa.rs

@@ -22,7 +22,7 @@
 //! )]
 //! #[serde(rename_all = "camelCase")]
 //! pub struct TestClusterSpec {
-//!     opa: Option<OpaConfig>    
+//!     opa: Option<OpaConfig>
 //! }
 //!
 //! let cluster: TestCluster = serde_yaml::from_str(