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

reword(rabbitmq): make condition match error string #3725

Merged

Conversation

tbobm
Copy link
Contributor

@tbobm tbobm commented Oct 5, 2022

Reword a conditional statement in the RabbitMQ Scaler to match the error message that is returned when evaluating to true.

Translated in a sentence, the conditional statement would match "if excludeUnacknowledged is set and the protocol used is equal to amqp" instead of "if excludeUnacknowledged is set and the protocol used is different then http"

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)

@tbobm tbobm requested a review from a team as a code owner October 5, 2022 15:53
Translated in a sentence, the conditional statement would match "if
excludeUnacknowledged is set and the protocol used is equal to amqp"
instead of "if excludeUnacknowledged is set and the protocol used is
different then http"

Signed-off-by: Theo Bob Massard <[email protected]>
@tbobm tbobm force-pushed the refactor/rabbitmq-scaler-conditional-wording branch from 664ed46 to 160bdc4 Compare October 5, 2022 15:55
@JorTurFer JorTurFer closed this Oct 10, 2022
@JorTurFer JorTurFer reopened this Oct 10, 2022
@JorTurFer
Copy link
Member

I have closed/reopened the PR because the CI checks were stuck

@JorTurFer
Copy link
Member

JorTurFer commented Oct 10, 2022

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

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.

LGTM!
Thanks for the improvement

@JorTurFer JorTurFer merged commit b7f8ebf into kedacore:main Oct 10, 2022
@tbobm tbobm deleted the refactor/rabbitmq-scaler-conditional-wording branch October 10, 2022 16:34
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