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

Put the release file in a folder inside the .tar.gz #173

Closed
grantstephens opened this issue Jun 27, 2018 · 7 comments
Closed

Put the release file in a folder inside the .tar.gz #173

grantstephens opened this issue Jun 27, 2018 · 7 comments
Labels

Comments

@grantstephens
Copy link

So this might be a weird one, but essentially all the other exporters that I've come across and use put the binary inside another folder in the release .tar.gz- this might be annoying or whatever, but at least it was consistent, except for this exporter.
Would it be possible to adjust the release process to make this possible?

@oliver006
Copy link
Owner

Hey there,

my instinct is to not change it at this point because probably people rely on it to not be in a folder and their puppet, ansible, whatever scripts would break if out of a sudden future releases are in a folder.

Is this currently causing serious issues for you or is it more of a convenience thing?

@grantstephens
Copy link
Author

Thanks for the reply and sorry for my delayed reply.
So we currently have a generic exporter deploy script which covers all the exporters we have encountered so far (around 8 off the top of my head) where the binary is another folder deep.
So I guess it is a convenience thing, but it is also a conformity thing.
Don't know if it is worth leaving the issue open to see if anybody else comes across it and can give a thumbs up if they agree?

@dkanchev
Copy link
Contributor

@oliver006 @grantstephens Can't we use a symlink and this way work around this problem?

@zoonage
Copy link

zoonage commented Oct 2, 2018

+1 for this, I would like to remove the customisations in https://github.com/lest/prometheus-rpm
for this exporter.

@grantstephens
Copy link
Author

@dkanchev it looks like that would be possible. Something like ln -s ../redis_exporter redis_exporter-vxxxx.linux-arch/redis_exporter should do it.

@oliver006 oliver006 mentioned this issue May 2, 2019
6 tasks
@oliver006
Copy link
Owner

Heads up, PR #256 will fix this and put the binary in a folder. I'm expecting to merge it some time next week.

@oliver006
Copy link
Owner

As of v1.0.0 the binary is now in a folder named e.g. redis_exporter-v1.0.0.linux-amd64.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants