fix: Place internal secrets in mutable Kubernetes Secrets

[sbernauer]

Description

Part of stackabletech/issues#843

This PR also

• Improves the code logic, as it is a bit silly to always generate random Secret contains and throw them away most of the time
• Thus improves CPU usage
• Adds tracing messages
• Adds documentation

Testing

Tested from-scratch install:

INFO reconciling object{object.ref=TrinoCluster.v1alpha1.trino.stackable.tech/simple-trino.default object.reason=object updated}: stackable_trino_operator::controller: Random Secret missing, creating it k8s.secret.name="simple-trino-internal-secret" k8s.secret.namespace="default"
INFO reconciling object{object.ref=TrinoCluster.v1alpha1.trino.stackable.tech/simple-trino.default object.reason=object updated}: stackable_trino_operator::controller: Random Secret missing, creating it k8s.secret.name="simple-trino-spooling-secret" k8s.secret.namespace="default"

As well as and update from 0.0.0-dev:

INFO reconciling object{object.ref=TrinoCluster.v1alpha1.trino.stackable.tech/simple-trino.default object.reason=object updated}: stackable_trino_operator::controller: Old (immutable) Secret detected, re-creating it to be able to make it mutable. The contents will stay the same. k8s.secret.name="simple-trino-internal-secret" k8s.secret.namespace="default"
INFO reconciling object{object.ref=TrinoCluster.v1alpha1.trino.stackable.tech/simple-trino.default object.reason=object updated}: stackable_trino_operator::controller: Old (immutable) Secret detected, re-creating it to be able to make it mutable. The contents will stay the same. k8s.secret.name="simple-trino-spooling-secret" k8s.secret.namespace="default"

Follow-up

It seems a bit silly to-copy paste this to airflow, IMHO we should put something like this into operator-rs

Definition of Done Checklist

• 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

Author

• Changes are OpenShift compatible
• CRD changes approved
• CRD documentation for all fields, following the style guide.
• Helm chart can be installed and deployed operator works
• Integration tests passed (for non trivial changes)
• Changes need to be "offline" compatible
• Links to generated (nightly) docs added
• Release note snippet added

Reviewer

• Code contains useful comments
• Code contains useful logging statements
• (Integration-)Test cases added
• Documentation added or updated. Follows the style guide.
• Changelog updated
• Cargo.toml only contains references to git tags (not specific commits or branches)

Acceptance

• Feature Tracker has been updated
• Proper release label has been added
• Links to generated (nightly) docs added
• Release note snippet added
• Add type/deprecation label & add to the deprecation schedule
• Add type/experimental label & add to the experimental features tracker

[razvan]

small remark, otherwise it lgtm

[razvan]

I would prefer to make immutable: false explicit here too.