Skip to content

Conversation

@dczupryn-dt
Copy link
Contributor

@dczupryn-dt dczupryn-dt commented Jan 27, 2026

Enable possibility to change settings HIGH_CPU_USAGE_BROKER_SERVICE_NAME and HIGH_CPU_USAGE_FLAG_NAME. That fixes problem with bad name for broker-service deployment.

Before:

2026-01-28T07:33:08.957Z        DEBUG   Feature flag    featureflag/serviceconnector.go:78      Fetching a flag {"flag": "high_cpu_usage"}
2026-01-28T07:33:08.971Z        ERROR   Operator        operator/operator.go:125        An error occurred while updating state for the "high_cpu_usage" flag - failed to fetch the resource for the "high_cpu_usage" flag: failed to get the deployment: deployments.apps "broker-service" not found
2026-01-28T07:33:08.971Z        ERROR   Operator        operator/operator.go:83 An error occurred while updating the state (failed to fetch the resource for the "high_cpu_usage" flag: failed to get the deployment: deployments.apps "broker-service" not found)

After:

2026-01-28T07:30:48.755Z        INFO    Operator        operator/operator.go:100        Updating state...       {"namespace": "easytrade-helm"}
2026-01-28T07:30:48.755Z        DEBUG   Feature flag    featureflag/serviceconnector.go:78      Fetching a flag {"flag": "high_cpu_usage"}
2026-01-28T07:30:48.851Z        INFO    High CPU Usage | Broker Service highcpuusage/brokerservice.go:141       Successfully got the easytrade-broker-service deployment
2026-01-28T07:30:48.851Z        INFO    Operator        operator/operator.go:169        State of the easytrade-broker-service is synchronized {"flag": "high_cpu_usage", "enabled": false}
2026-01-28T07:30:48.861Z        INFO    Operator        operator/operator.go:76 Successfully updated state

@dczupryn-dt dczupryn-dt requested a review from a team as a code owner January 27, 2026 14:19
@dczupryn-dt dczupryn-dt merged commit 80aa2f7 into main Jan 28, 2026
8 checks passed
@dczupryn-dt dczupryn-dt deleted the problem-operator-configuration branch January 29, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants