-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
feat: Update the Aliyun Metrics Exporter functionality #15238
feat: Update the Aliyun Metrics Exporter functionality #15238
Conversation
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 📦 Click here to get additional PR build artifactsArtifact URLs |
@ZPascal - Is this a PR you plan to work on further? Looks like you were working on adding support for |
@powersj Yes, that's correct and I'll add support for it. I'm currently testing/ debugging it. |
Hello! I am closing this issue due to inactivity. I hope you were able to resolve your problem, if not please try posting this question in our Community Slack or Community Forums or provide additional details in this issue and reqeust that it be re-opened. Thank you! |
Happy to see a PR with updates! We do try to keep the open PR list to things ready for review. Let me know if you push updates and we can reopen as needed. Thanks! |
@powersj I've adapted the feature branch and adjusted the scope. I've added now rds performance support! Should I open a new PR or should we handle it here? |
We generally prefer smaller PRs with a narrow focus, so if it is easily done could you split them out? |
Sure, I'll open a new PR with a dedicated focus on the RDS performance metrics. |
Summary
Checklist
Related issues
resolves #