Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

influxdb: document which paramters may be used from triggerauthentication #472

Closed
wants to merge 1 commit into from

Conversation

acobaugh
Copy link

This updates the influxdb scaler docs for 2.4 to state which parameters can be specified from (cluster)triggerauthentication, as a followup to kedacore/keda#1904

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO)
  • A PR is opened to update the documentation on our docs repo

content/docs/2.4/scalers/influxdb.md Show resolved Hide resolved
content/docs/2.4/scalers/influxdb.md Outdated Show resolved Hide resolved
content/docs/2.4/scalers/influxdb.md Outdated Show resolved Hide resolved
@acobaugh
Copy link
Author

Is there an example scaler that's formatted the way we want? I'm having a hard time understanding what needs to be changed.

content/docs/2.4/scalers/influxdb.md Show resolved Hide resolved
content/docs/2.4/scalers/influxdb.md Outdated Show resolved Hide resolved
content/docs/2.4/scalers/influxdb.md Outdated Show resolved Hide resolved
content/docs/2.4/scalers/influxdb.md Outdated Show resolved Hide resolved
@tomkerkhove
Copy link
Member

I have added some more suggestions which hopefully help but https://github.com/kedacore/keda-docs/blob/master/content/docs/2.4/scalers/azure-service-bus.md is typically the one I refer to

@zroubalik
Copy link
Member

@acobaugh do you think you'll have a time to finish this PR?

@acobaugh
Copy link
Author

I think this is good to my eye, but could use a second look from someone since we've changed a bunch of formatting and wording since the PR was originally opened. Thanks!

@tomkerkhove
Copy link
Member

Thank you! Can you have a look at #472 (comment) please?

…tion and provide an example

Signed-off-by: Andy Cobaugh <[email protected]>
@acobaugh
Copy link
Author

I think this should be good now.

@JorTurFer
Copy link
Member

Hi @acobaugh ,
Could you add it also to version 2.6?

@tomkerkhove
Copy link
Member

Closing due to no response

@tomkerkhove tomkerkhove closed this Dec 9, 2022
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.

4 participants