-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add metric to Solace Scaler #4667
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lookin good, please fix the e2e test - see Static Checks failure, you can ignore the "Welcome PR" failure.
/run-e2e |
@zroubalik - there were problems with the helm chart references (the chart url context changed) - which I didn't detect because I was skipping that part of the test. I updated the code and ran the test successfully on my machine. Should be good to go. |
/run-e2e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dennis-brinley it is not possible to merge this PR due to some conflicts, could you please rebase your branch? Thanks!
merge conflict should be resolved |
/run-e2e solace |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dennis-brinley I can see that your rebase unfortunately included additional commits in your PR. You can see that there are some unrelevant changes. Could you please fix that?
3c4b83c
to
0e7662d
Compare
Signed-off-by: Dennis Brinley <[email protected]>
chopped and re-committed |
Tests failed on predictkube scaler. I don't think there is anything I can do about that directly.
|
/run-e2e solace |
Signed-off-by: Dennis Brinley <[email protected]>
Added new metric: messageReceiveRateTarget to Solace Scaler
Checklist
Fixes #4665