fix: data races

[juev]

Overview

Fixing data races in code and tests

What this PR does / why we need it

Some edits

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

[Copilot]

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

pkg/schedule_manager/schedule_manager.go:18

• Converting ScheduleManager from an interface to a concrete struct type may break abstraction boundaries and affect consumers expecting an interface. Confirm that this change is intentional and that all dependent code is updated accordingly.

type ScheduleManager struct {

[Copilot]

Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

test/hook/context/generator.go:132

• [nitpick] Consider using b.started.Store(true) instead of Swap(true) if the previous value is not required, as it more clearly conveys that the flag is simply being set.

b.started.Swap(true)

pkg/kube_events_manager/kube_events_manager.go:100

• If no monitor exists for the given monitorID, calling monitor.Start(mgr.ctx) may result in a nil pointer dereference. Consider adding a nil check and error handling before starting the monitor.

monitor := mgr.Monitors[monitorID]

[Copilot]

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

[miklezzzz]

a breath of fresh air :)