Skip to content

feat(keda): Update kafka keda triggers#1717

Merged
wcmitchell merged 2 commits intoRedHatInsights:masterfrom
wcmitchell:update_kafka_keda_config
Mar 24, 2026
Merged

feat(keda): Update kafka keda triggers#1717
wcmitchell merged 2 commits intoRedHatInsights:masterfrom
wcmitchell:update_kafka_keda_config

Conversation

@wcmitchell
Copy link
Copy Markdown
Contributor

Full broker list, tls and sasl configs

Comment thread controllers/cloud.redhat.com/providers/autoscaler/keda.go Outdated
@maknop
Copy link
Copy Markdown
Contributor

maknop commented Mar 18, 2026

/test-e2e

1 similar comment
@maknop
Copy link
Copy Markdown
Contributor

maknop commented Mar 18, 2026

/test-e2e

@maknop
Copy link
Copy Markdown
Contributor

maknop commented Mar 18, 2026

Tests seem to be failing. I ran it twice.

wcmitchell and others added 2 commits March 20, 2026 10:04
Full broker list, tls and sasl configs

Signed-off-by: Chris Mitchell <cmitchel@redhat.com>
Add some defensive guards around what we're parsing through

Co-authored-by: Jacopo Andrea Giola <jacopo@giola.org>
@wcmitchell wcmitchell force-pushed the update_kafka_keda_config branch from cbe6d0b to a77b4d9 Compare March 20, 2026 14:04
@wcmitchell
Copy link
Copy Markdown
Contributor Author

/test-e2e

@wcmitchell
Copy link
Copy Markdown
Contributor Author

Ok! Everything passing but Grype, that'll need broader clowder dependency updates

Copy link
Copy Markdown
Contributor

@maknop maknop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@wcmitchell wcmitchell merged commit f5e62e3 into RedHatInsights:master Mar 24, 2026
8 of 9 checks passed
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