scylla-operator-bot[bot]
commented
3 weeks ago @rzetelskik: GitHub didn't allow me to request PR reviews from the following users: rzetelskik.
Note that only scylladb members and repo collaborators can review this PR, and authors cannot review their own PRs.
In response to [this](https://github.com/scylladb/scylla-operator/pull/2165):
>
>
>**Description of your changes:** wip
>
>**Which issue is resolved by this Pull Request:**
>Resolves #1920
>
>/kind feature
>/priority important-longterm
>/cc
Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
rzetelskik
tnozicka
Description of your changes: Node tune controller does not currently set status conditions on NodeConfig's reconciliation. Instead it only set status' NodeStatuses. While this is enough for checking the status programmatically, setting and aggregating status conditions allows for a better UX, by giving the users an option to query NodeConfig conditions. This PR extends the node tune controller with per-node status conditions and further aggregates them with the existing ones in NodeConfig controller.
It also adds handlers for NodeConfig events to node tune controller and aligns the existing e2e tests with the changes.
Which issue is resolved by this Pull Request: Resolves #1920
/kind feature /priority important-longterm /cc