-
Notifications
You must be signed in to change notification settings - Fork 200
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
Override config metrics enable with cli #1658
Conversation
check also ovms --help output and make sure metrics params are in server settings section. |
Missing ticket ID and description |
"Flag enabling metrics endpoint on rest_port.", | ||
cxxopts::value<bool>()->default_value("false"), | ||
"METRICS") | ||
("metrics_list", |
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.
Should we include information that when used, config.json cannot contain metric configuration? @dtrawins
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.
I would keep it short but indeed documentation should be adjusted in https://github.com/openvinotoolkit/model_server/blob/develop/docs/parameters.md
A note in https://github.com/openvinotoolkit/model_server/blob/main/docs/metrics.md could be also useful that configuration can not be duplicated.
24ec16e
to
aaceb69
Compare
Allows using metrics_enable param in CLI even with config. CLI will overwrite config setting if defined. Added warning in the logs.
JIRA:CVS-99880