keda-docs
keda-docs copied to clipboard
Remove unused variables and correct queueDepth default in IBM MQ scaler
- The current documentation states that the default
queueDepthis5, while the code default is20. The documentation will be updated to reflect the correct default value of20, aligning it with the code. - The
queueManagerparameter is currently checked to see if it's filled, but it's not used elsewhere in the code. This unused code is removed. The documentation is updated accordingly. - The parameter
tlsDisabledis removed. All other scalers use the parameterunsafeSsl. To maintain consistency across the other scalers, the code related totlsDisabledis removed, asunsafeSslalready handles this functionality. - Finally, some minor adjustments to the documentation to keep the formatting the same.
Checklist
- [X] Commits are signed with Developer Certificate of Origin (DCO)
Fixes #6033
Thank you for your contribution! 🙏 We will review your PR as soon as possible.
While you are waiting, make sure to:
- Add your contribution to all applicable KEDA versions
- GitHub checks are passing
- Is the DCO check failing? Here is how you can fix DCO issues
Learn more about:
- How to become a listed end-user or a listed commerical offering
- Our contribution guide
Deploy Preview for keda ready!
| Name | Link |
|---|---|
| Latest commit | 1b5436999a2f71950d30c1aecc06096c41e7aedf |
| Latest deploy log | https://app.netlify.com/sites/keda/deploys/66c4571e32a8bc000839c595 |
| Deploy Preview | https://deploy-preview-1445--keda.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Are all deprecated?
I thought that only the tls one is deprecated
Are all deprecated?
I thought that only the tls one is deprecated
Yes, sorry. My bad. I misunderstood something. I have changed it back.
