Skip to content

Change exporter-toolkit module name(https -> web)#1931

Merged
roidelapluie merged 3 commits intoprometheus:masterfrom
changmink:https-to-web
Jan 18, 2021
Merged

Change exporter-toolkit module name(https -> web)#1931
roidelapluie merged 3 commits intoprometheus:masterfrom
changmink:https-to-web

Conversation

@changmink
Copy link
Contributor

I cannot install this exporter. Because module name changed in exporter-toolkit. So I changed module name that.

Signed-off-by: changmink <changmin043@gmail.com>
Signed-off-by: changmink <changmin043@gmail.com>
Copy link
Member

@roidelapluie roidelapluie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this. CI is not happy, can you please address my comments?

@roidelapluie
Copy link
Member

roidelapluie commented Jan 15, 2021

Note that you should not have hit any issues at first. I suspect a go mod issue.

Signed-off-by: changmink <changmin043@gmail.com>
@roidelapluie
Copy link
Member

Thanks!

@roidelapluie roidelapluie merged commit cfdd9dd into prometheus:master Jan 18, 2021
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Change deprecated Listen to ListenAndServe
* Change exporter-toolkit module name(https -> web)

Signed-off-by: changmink <changmin043@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Change deprecated Listen to ListenAndServe
* Change exporter-toolkit module name(https -> web)

Signed-off-by: changmink <changmin043@gmail.com>
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 this pull request may close these issues.

2 participants