Skip to content

New Solace Direct Messaging Scaler#6546

Merged
wozniakjan merged 73 commits into
kedacore:mainfrom
alfonso-chacon:changelog-merge
Aug 8, 2025
Merged

New Solace Direct Messaging Scaler#6546
wozniakjan merged 73 commits into
kedacore:mainfrom
alfonso-chacon:changelog-merge

Conversation

@alfonso-chacon

@alfonso-chacon alfonso-chacon commented Feb 12, 2025

Copy link
Copy Markdown
Contributor

New Solace Direct Messaging Scaler Added

Checklist

Fixes #

Fixes #6545
[#1537] (kedacore/keda-docs#1537) - Documentation

Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: Alfonso Chacón <[email protected]>
…scarded messages while scaling up

Signed-off-by: Alfonso Chacón <[email protected]>
- Added Configuration validations
- Variables Refactor
- Multiple URLs
- Remove commented blocks

Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: Alfonso Chacón <[email protected]>
Comment thread pkg/scalers/solace_dm_scaler.go Fixed
* Code Cleanup - CodeQL/StaticChecks
Signed-off-by: Alfonso Chacón <[email protected]>
* Code cleanup /golangci-lint
Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: Alfonso Chacón <[email protected]>
@alfonso-chacon
alfonso-chacon marked this pull request as ready for review February 13, 2025 04:27
@alfonso-chacon
alfonso-chacon requested a review from a team as a code owner February 13, 2025 04:27
Comment thread pkg/scalers/solace_dm_scaler.go Outdated
Comment thread pkg/scalers/solace_dm_scaler.go
Comment thread pkg/scalers/solace_dm_scaler.go Outdated
Comment thread pkg/scalers/solace_dm_scaler.go Outdated
Comment thread pkg/scalers/solace_dm_scaler.go Outdated
@JorTurFer

JorTurFer commented Feb 14, 2025

Copy link
Copy Markdown
Member

/run-e2e solace_dm
Update: You can check the progress here

alfonso-chacon and others added 6 commits February 15, 2025 00:37
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Signed-off-by: Alfonso Chacón <[email protected]>
@JorTurFer

JorTurFer commented Feb 15, 2025

Copy link
Copy Markdown
Member

/run-e2e solace_dm
Update: You can check the progress here

@alfonso-chacon

Copy link
Copy Markdown
Contributor Author

@JorTurFer Is there anything I can do for the failed check? its complaining about a timeout in a API

=== FAIL: pkg/scalers TestPredictKubeGetMetricSpecForScaling (3.00s)
2025/02/17 08:01:26 🚀 starting mock grpc server. On host 0.0.0.0, with port: 44809
predictkube_scaler_test.go:187:
Error Trace: /__w/keda/keda/pkg/scalers/predictkube_scaler_test.go:187
Error: Received unexpected error:
error create Prometheus client and API objects: Get "http://demo.robustperception.io:9090/api/v1/status/runtimeinfo": dialing to the given TCP address timed out
Test: TestPredictKubeGetMetricSpecForScaling
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x277cad8]

@alfonso-chacon

Copy link
Copy Markdown
Contributor Author

@wozniakjan @zroubalik @JorTurFer Hi Guys! @rickbrouwer approved the pull request, can you help me with your approval?

@zroubalik

zroubalik commented Jul 21, 2025

Copy link
Copy Markdown
Member

/run-e2e solace_dm
Update: You can check the progress here

@zroubalik
zroubalik requested a review from Copilot July 21, 2025 15:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request introduces a new Solace Direct Messaging scaler for KEDA, which enables scaling based on client transmission metrics from Solace message brokers. The scaler monitors aggregated client transmission rates (both message and byte rates) and queue metrics to make scaling decisions.

Key Changes

  • Adds a new Solace Direct Messaging scaler with support for four different transmission rate metrics
  • Implements comprehensive test coverage including unit tests and end-to-end integration tests
  • Integrates the new scaler into KEDA's scaling infrastructure

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
pkg/scalers/solace_dm_scaler.go Core implementation of the Solace Direct Messaging scaler with SEMP API integration
pkg/scalers/solace_dm_scaler_test.go Comprehensive unit tests covering configuration parsing and validation scenarios
pkg/scaling/scalers_builder.go Registration of the new scaler in KEDA's scaler factory
tests/scalers/solace/direct-messaging/solace_dm_test.go End-to-end integration tests with Solace broker deployment
CHANGELOG.md Documentation of the new feature addition

Comment thread pkg/scalers/solace_dm_scaler.go Outdated
Comment thread pkg/scalers/solace_dm_scaler.go Outdated
Comment thread tests/scalers/solace/direct-messaging/solace_dm_test.go Outdated
Comment thread tests/scalers/solace/direct-messaging/solace_dm_test.go Outdated
Comment thread tests/scalers/solace/direct-messaging/solace_dm_test.go Outdated
Comment thread tests/scalers/solace/direct-messaging/solace_dm_test.go Outdated

@JorTurFer JorTurFer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

great job!
Just a nitpick inline, but I already keep the approval to merge the PR after comments are ready

Comment thread tests/scalers/solace/direct-messaging/solace_dm_test.go Outdated
@rickbrouwer rickbrouwer added the waiting-author-response All PR's or Issues where we are waiting for a response from the author label Jul 29, 2025
@rickbrouwer rickbrouwer removed the waiting-author-response All PR's or Issues where we are waiting for a response from the author label Jul 29, 2025
@rickbrouwer

Copy link
Copy Markdown
Member

Hi @alfonso-chacon

Can you do a git commit --amend --no-edit? then the tests are triggered again

@alfonso-chacon

Copy link
Copy Markdown
Contributor Author

done, smoke and e2e tests pending

@rickbrouwer

rickbrouwer commented Jul 31, 2025

Copy link
Copy Markdown
Member

/run-e2e solace_dm
Update: You can check the progress here

@rickbrouwer rickbrouwer added the ok-to-merge This PR can be merged label Jul 31, 2025
@rickbrouwer

rickbrouwer commented Jul 31, 2025

Copy link
Copy Markdown
Member

/run-e2e solace_dm
Update: You can check the progress here

@alfonso-chacon

Copy link
Copy Markdown
Contributor Author

@zroubalik @wozniakjan Hi Guys, Can you please approve?

@wozniakjan
wozniakjan enabled auto-merge (squash) August 8, 2025 09:13

@wozniakjan wozniakjan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@wozniakjan
wozniakjan merged commit 56a023d into kedacore:main Aug 8, 2025
24 checks passed
@alfonso-chacon
alfonso-chacon deleted the changelog-merge branch September 6, 2025 00:11
dpochopsky pushed a commit to dpochopsky/keda that referenced this pull request Sep 12, 2025
* * Solace Direct Messaging Scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* Fix D-1 queue calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor variables

Signed-off-by: Alfonso Chacón <[email protected]>

* Inlcude D-1 messages in calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Modify url to be able to use https protocol

Signed-off-by: Alfonso Chacón <[email protected]>

* Added UnsafeSSL parameter to avoid SSL validations

Signed-off-by: Alfonso Chacón <[email protected]>

* Added a factor to multiply the weight of queues messages to reduce discarded messages while scaling up

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor and enhancements

- Added Configuration validations
- Variables Refactor
- Multiple URLs
- Remove commented blocks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Scaler E2E testing

Signed-off-by: Alfonso Chacón <[email protected]>

* Adding New Solace Direct Messaging Scaler in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code Cleanup - CodeQL/StaticChecks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup /golangci-lint

Signed-off-by: Alfonso Chacón <[email protected]>

* * Correct order in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove trailing white spaces

Signed-off-by: Alfonso Chacón <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - change log level

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * E2E tests - reduce time in scale down cfg

Signed-off-by: Alfonso Chacón <[email protected]>

* * E2E fix typo

Signed-off-by: Alfonso Chacón <[email protected]>

* * Avoid read secrets from metadata

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once on logger init

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup recommendations

- Move regex to pkg level
- Remove unnecessary struct and interface

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Updated to sync with other sol scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary trailing newline

Signed-off-by: Alfonso Chacón <[email protected]>

* Remove optional keyword for optional fields

Co-authored-by: Jan Wozniak <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code fixes

-move unused variables to test file
-return json unmarshall error

Signed-off-by: Alfonso Chacón <[email protected]>

* * Removing redundant validation

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary new line

Signed-off-by: Alfonso Chacón <[email protected]>

* fix triggerMetadata syntax

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update pkg/scalers/solace_dm_scaler.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

---------

Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Co-authored-by: Zbynek Roubalik <[email protected]>
Co-authored-by: Jan Wozniak <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: David Pochopsky <[email protected]>
alt-dima pushed a commit to alt-dima/keda that referenced this pull request Dec 13, 2025
* * Solace Direct Messaging Scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* Fix D-1 queue calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor variables

Signed-off-by: Alfonso Chacón <[email protected]>

* Inlcude D-1 messages in calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Modify url to be able to use https protocol

Signed-off-by: Alfonso Chacón <[email protected]>

* Added UnsafeSSL parameter to avoid SSL validations

Signed-off-by: Alfonso Chacón <[email protected]>

* Added a factor to multiply the weight of queues messages to reduce discarded messages while scaling up

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor and enhancements

- Added Configuration validations
- Variables Refactor
- Multiple URLs
- Remove commented blocks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Scaler E2E testing

Signed-off-by: Alfonso Chacón <[email protected]>

* Adding New Solace Direct Messaging Scaler in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code Cleanup - CodeQL/StaticChecks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup /golangci-lint

Signed-off-by: Alfonso Chacón <[email protected]>

* * Correct order in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove trailing white spaces

Signed-off-by: Alfonso Chacón <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - change log level

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * E2E tests - reduce time in scale down cfg

Signed-off-by: Alfonso Chacón <[email protected]>

* * E2E fix typo

Signed-off-by: Alfonso Chacón <[email protected]>

* * Avoid read secrets from metadata

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once on logger init

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup recommendations

- Move regex to pkg level
- Remove unnecessary struct and interface

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Updated to sync with other sol scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary trailing newline

Signed-off-by: Alfonso Chacón <[email protected]>

* Remove optional keyword for optional fields

Co-authored-by: Jan Wozniak <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code fixes

-move unused variables to test file
-return json unmarshall error

Signed-off-by: Alfonso Chacón <[email protected]>

* * Removing redundant validation

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary new line

Signed-off-by: Alfonso Chacón <[email protected]>

* fix triggerMetadata syntax

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update pkg/scalers/solace_dm_scaler.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

---------

Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Co-authored-by: Zbynek Roubalik <[email protected]>
Co-authored-by: Jan Wozniak <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Dmitriy Altuhov <[email protected]>
tangobango5 pushed a commit to tangobango5/keda that referenced this pull request Dec 22, 2025
* * Solace Direct Messaging Scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* Fix D-1 queue calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor variables

Signed-off-by: Alfonso Chacón <[email protected]>

* Inlcude D-1 messages in calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Modify url to be able to use https protocol

Signed-off-by: Alfonso Chacón <[email protected]>

* Added UnsafeSSL parameter to avoid SSL validations

Signed-off-by: Alfonso Chacón <[email protected]>

* Added a factor to multiply the weight of queues messages to reduce discarded messages while scaling up

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor and enhancements

- Added Configuration validations
- Variables Refactor
- Multiple URLs
- Remove commented blocks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Scaler E2E testing

Signed-off-by: Alfonso Chacón <[email protected]>

* Adding New Solace Direct Messaging Scaler in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code Cleanup - CodeQL/StaticChecks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup /golangci-lint

Signed-off-by: Alfonso Chacón <[email protected]>

* * Correct order in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove trailing white spaces

Signed-off-by: Alfonso Chacón <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - change log level

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * E2E tests - reduce time in scale down cfg

Signed-off-by: Alfonso Chacón <[email protected]>

* * E2E fix typo

Signed-off-by: Alfonso Chacón <[email protected]>

* * Avoid read secrets from metadata

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once on logger init

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup recommendations

- Move regex to pkg level
- Remove unnecessary struct and interface

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Updated to sync with other sol scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary trailing newline

Signed-off-by: Alfonso Chacón <[email protected]>

* Remove optional keyword for optional fields

Co-authored-by: Jan Wozniak <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code fixes

-move unused variables to test file
-return json unmarshall error

Signed-off-by: Alfonso Chacón <[email protected]>

* * Removing redundant validation

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary new line

Signed-off-by: Alfonso Chacón <[email protected]>

* fix triggerMetadata syntax

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update pkg/scalers/solace_dm_scaler.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

---------

Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Co-authored-by: Zbynek Roubalik <[email protected]>
Co-authored-by: Jan Wozniak <[email protected]>
Co-authored-by: Copilot <[email protected]>
tangobango5 pushed a commit to tangobango5/keda that referenced this pull request Feb 13, 2026
* * Solace Direct Messaging Scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* Fix D-1 queue calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor variables

Signed-off-by: Alfonso Chacón <[email protected]>

* Inlcude D-1 messages in calculation

Signed-off-by: Alfonso Chacón <[email protected]>

* Modify url to be able to use https protocol

Signed-off-by: Alfonso Chacón <[email protected]>

* Added UnsafeSSL parameter to avoid SSL validations

Signed-off-by: Alfonso Chacón <[email protected]>

* Added a factor to multiply the weight of queues messages to reduce discarded messages while scaling up

Signed-off-by: Alfonso Chacón <[email protected]>

* Refactor and enhancements

- Added Configuration validations
- Variables Refactor
- Multiple URLs
- Remove commented blocks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Scaler E2E testing

Signed-off-by: Alfonso Chacón <[email protected]>

* Adding New Solace Direct Messaging Scaler in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code Cleanup - CodeQL/StaticChecks

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup /golangci-lint

Signed-off-by: Alfonso Chacón <[email protected]>

* * Correct order in CHANGELOG.md

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove trailing white spaces

Signed-off-by: Alfonso Chacón <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - remove comment

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Cleanup code - change log level

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * E2E tests - reduce time in scale down cfg

Signed-off-by: Alfonso Chacón <[email protected]>

* * E2E fix typo

Signed-off-by: Alfonso Chacón <[email protected]>

* * Avoid read secrets from metadata

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Remove constant used once on logger init

Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup recommendations

- Move regex to pkg level
- Remove unnecessary struct and interface

Signed-off-by: Alfonso Chacón <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code cleanup

Co-authored-by: Zbynek Roubalik <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Updated to sync with other sol scaler

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary trailing newline

Signed-off-by: Alfonso Chacón <[email protected]>

* Remove optional keyword for optional fields

Co-authored-by: Jan Wozniak <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * Code fixes

-move unused variables to test file
-return json unmarshall error

Signed-off-by: Alfonso Chacón <[email protected]>

* * Removing redundant validation

Signed-off-by: Alfonso Chacón <[email protected]>

* * Remove unnecessary new line

Signed-off-by: Alfonso Chacón <[email protected]>

* fix triggerMetadata syntax

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update pkg/scalers/solace_dm_scaler.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* Update tests/scalers/solace/direct-messaging/solace_dm_test.go

Co-authored-by: Copilot <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

* * fix a typo and change the ScaleUp/Down to ScaleIn/Out

Signed-off-by: Alfonso Chacón <[email protected]>

---------

Signed-off-by: Alfonso Chacón <[email protected]>
Signed-off-by: alfonso-chacon <[email protected]>
Co-authored-by: Jorge Turrado Ferrero <[email protected]>
Co-authored-by: Zbynek Roubalik <[email protected]>
Co-authored-by: Jan Wozniak <[email protected]>
Co-authored-by: Copilot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-merge This PR can be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New Solace scaler for Direct Messaging

7 participants