-
Notifications
You must be signed in to change notification settings - Fork 45
cmsweb/exporters - build a new version #1613
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
docker/exporters/Dockerfile
Outdated
|
|
||
| # build mongo exporter | ||
| # TODO: consider using an actively developed exporter, such as | ||
| # https://github.com/percona/mongodb_exporter |
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.
For this I suggest you open CMSMONIT issue and ask Monitoring group to install and test it with existing dashboard. If it is compatible then we can easily switch otherwise we need a new dashboard which will use this exporter metrics.
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 see, but since we are not using mongodb_expoerter in our wmcore images, can we merge this PR, build a new exporter image and tag it with -stable, use it for #1611 , then build a new tag with the new percona/mongodb_exporter after cmsmonit team finish their investigations?
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.
yes we can since you dont' change version of mongodb exporter anyway. But please be aware that exporters are used across all services and if you claim we (WM) group not using particular exporter it does not mean it is not in use. The mongodb exporter is used by DAS, and since DAS can be treated as stand-alone entity or part of WM, we (broader scope) is using it :)
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.
as I suggested please open up new CMSMON ticket and link it here for cross references, then we can proceed with merge and stable images.
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 opened the ticket [1] and added it to the comments in the dockerfile.
if you claim we (WM) group not using particular exporter it does not mean it is not in use.
I know, and this is why i proposed a change instead of just removing removing or changing it :)
[1] https://its.cern.ch/jira/projects/CMSMONIT/issues/CMSMONIT-650?filter=allissues
41ab376 to
08437e7
Compare
08437e7 to
0537a20
Compare
|
@mapellidario , one additional comment I want to clarify with you. You deleted few scripts and I want you to double check that they are no longer in use anywhere. Please add this to the description of the issue that we can refer to this change if necessary. These scripts I used/developed while we port CMSWEB services from RPM based deployment to k8s but it would be useful to grep and see if they are no longer in use. Thanks. |
|
I removed the scripts because they have been commented out from the dockerfile ( The only way to use them is to use docker build from a wider context, let's say from I am in favor of cleanups even if i understand that it can be risky. if you prefer, i can restore those files [1] CMSKubernetes/docker/exporters/Dockerfile Lines 47 to 48 in 3c5a1e0
|
|
thank you Dario for cross-checking, I'm ok with deleting those scripts I just want to have a references to easily find them if we'll need them to recover something. That's why I suggested to include explicitly what we delete in a description of this PR. |
|
I updated the PR description |
vkuznet
left a comment
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.
thank you Dario, changes look good to me.
we noticed that the
latesttag of cmsweb/exporters is a few years old. we should build a new one.changes
suggestion
fyi: @amaltaro
[1]
The following scripts have been removed from all the dockerfiles in
docker/exporters/and are not imported by any other docker image (or so it seems from a few greps)