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

[Bug] can't open controller metricsExporter #7697

Closed
3 tasks done
baijun44 opened this issue Dec 26, 2023 · 4 comments · Fixed by #7705
Closed
3 tasks done

[Bug] can't open controller metricsExporter #7697

baijun44 opened this issue Dec 26, 2023 · 4 comments · Fixed by #7705

Comments

@baijun44
Copy link
Contributor

Before Creating the Bug Report

  • I found a bug, not just asking a question, which should be created in GitHub Discussions.

  • I have searched the GitHub Issues and GitHub Discussions of this repository and believe that this is not a duplicate.

  • I have confirmed that this bug belongs to the current repository, not other repositories of RocketMQ.

Runtime platform environment

OS: ubuntu 22.04

RocketMQ version

rocketmq 5.1.4

JDK Version

openjdk 1.8

Describe the Bug

add a config "metricsExporterType = PROM" for controller, but it doesn't work.I read source file(ControllerConfig) and found this field 'metricsExporterType' has no 'set method' by passing String type parameter.

Steps to Reproduce

1.add a config "metricsExporterType = PROM"
2.start controller
3.open localhost:5557

What Did You Expect to See?

can open localhost:5557

What Did You See Instead?

can't open localhost:5557

Additional Context

No response

@RongtongJin
Copy link
Contributor

@baijun44 Could you submit a PR to fix the issue?

@baijun44
Copy link
Contributor Author

@baijun44 Could you submit a PR to fix the issue?

Yes! I will.

@baijun44
Copy link
Contributor Author

@RongtongJin Please help me review.Thanks!

@iamgd67
Copy link
Contributor

iamgd67 commented Mar 20, 2024

dumplicate to #7491

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 a pull request may close this issue.

3 participants