[WIP] Testing downstream: Make dbus controller more efficient

[trozet]

The previous code was iterating a for loop every 50ms to try to detect if there was a state change to an internal cache provided by coreos dbus package. This consumes too much CPU. Instead, directly use dbus to receive systemd unit events. This drastically reduces CPU and still detects when Kubelet is restarted.

Signed-off-by: Tim Rozet [email protected]
(cherry picked from commit 84dee1f59bcff40ec296d01d61c17f0b5a7d1cb3)

📑 Description

Fixes #

Additional Information for reviewers

✅ Checks

• My code requires changes to the documentation
• if so, I have updated the documentation as required
• My code requires tests
• if so, I have added and/or updated the tests as required
• All the tests have passed in the CI

How to verify it

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: trozet

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [trozet]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[jtaleric]

/test qe-perfscale-payload-control-plane-6nodes

[openshift-ci]

@trozet: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-azure-ovn-techpreview	bd26029	link	false	/test e2e-azure-ovn-techpreview
ci/prow/security	bd26029	link	false	/test security
ci/prow/e2e-aws-ovn-local-gateway	bd26029	link	true	/test e2e-aws-ovn-local-gateway
ci/prow/e2e-metal-ipi-ovn-ipv4-bgp-techpreview	bd26029	link	false	/test e2e-metal-ipi-ovn-ipv4-bgp-techpreview
ci/prow/e2e-metal-ipi-ovn-ipv6-techpreview	bd26029	link	false	/test e2e-metal-ipi-ovn-ipv6-techpreview
ci/prow/e2e-vsphere-ovn-techpreview	bd26029	link	false	/test e2e-vsphere-ovn-techpreview
ci/prow/e2e-openstack-ovn	bd26029	link	false	/test e2e-openstack-ovn
ci/prow/e2e-azure-ovn	bd26029	link	false	/test e2e-azure-ovn
ci/prow/okd-scos-e2e-aws-ovn	bd26029	link	false	/test okd-scos-e2e-aws-ovn
ci/prow/e2e-metal-ipi-ovn-dualstack	bd26029	link	true	/test e2e-metal-ipi-ovn-dualstack
ci/prow/e2e-azure-ovn-upgrade	bd26029	link	true	/test e2e-azure-ovn-upgrade

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[openshift-merge-robot]

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.