Skip to content

Refactor InfluxDB scaler config#6240

Merged
wozniakjan merged 2 commits intokedacore:mainfrom
dovics:influxdb
Oct 21, 2024
Merged

Refactor InfluxDB scaler config#6240
wozniakjan merged 2 commits intokedacore:mainfrom
dovics:influxdb

Conversation

@dovics
Copy link
Contributor

@dovics dovics commented Oct 15, 2024

Refactor the scaler to match the new metadata guidelines.

Checklist

Fixes #

Related: #5797

@dovics dovics requested a review from a team as a code owner October 15, 2024 08:21
Copy link
Member

@JorTurFer JorTurFer left a comment

Choose a reason for hiding this comment

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

Looking good, just a single type inline. PTAL @wozniakjan

Copy link
Member

@wozniakjan wozniakjan left a comment

Choose a reason for hiding this comment

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

lgtm, thank you!

fwiw, the unit test failure seems to be related to #6246

@wozniakjan
Copy link
Member

wozniakjan commented Oct 21, 2024

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

Signed-off-by: wangrushen <wrs369@88.com>
Signed-off-by: wangrushen <wrs369@88.com>
@wozniakjan
Copy link
Member

wozniakjan commented Oct 21, 2024

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

@wozniakjan wozniakjan enabled auto-merge (squash) October 21, 2024 07:16
@wozniakjan wozniakjan merged commit 0a4aef6 into kedacore:main Oct 21, 2024
rickbrouwer pushed a commit to rickbrouwer/keda that referenced this pull request Oct 21, 2024
* Refactor InfluxDB scaler config

Signed-off-by: wangrushen <wrs369@88.com>

* fix: typo in influxdv unsafessl

Signed-off-by: wangrushen <wrs369@88.com>

---------

Signed-off-by: wangrushen <wrs369@88.com>
mpechner-akasa pushed a commit to nrichardson-akasa/keda that referenced this pull request Nov 29, 2024
* Refactor InfluxDB scaler config

Signed-off-by: wangrushen <wrs369@88.com>

* fix: typo in influxdv unsafessl

Signed-off-by: wangrushen <wrs369@88.com>

---------

Signed-off-by: wangrushen <wrs369@88.com>
Signed-off-by: michael pechner <mike.pechner@akasa.com>
chiragbhatia8 pushed a commit to chiragbhatia8/keda that referenced this pull request Jan 9, 2025
* Refactor InfluxDB scaler config

Signed-off-by: wangrushen <wrs369@88.com>

* fix: typo in influxdv unsafessl

Signed-off-by: wangrushen <wrs369@88.com>

---------

Signed-off-by: wangrushen <wrs369@88.com>
Signed-off-by: Chirag Bhatia <chiragbhatia8@gmail.com>
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