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

Link the amtool only if it is installed via direct download. #328

Merged
merged 1 commit into from
Jul 11, 2019

Conversation

sezuan
Copy link

@sezuan sezuan commented Jun 27, 2019

Pull Request (PR) description

Only link /usr/bin/amtool if install_method is 'url'

This Pull Request (PR) fixes the following issues

Fixes #327

@sezuan sezuan force-pushed the fix_alertmanager_installation branch from 88abeaf to 4836259 Compare June 27, 2019 14:24
@bastelfreak
Copy link
Member

Thanks for the fix!

@bastelfreak bastelfreak added the bug Something isn't working label Jul 11, 2019
@bastelfreak bastelfreak merged commit 587c876 into voxpupuli:master Jul 11, 2019
@sezuan sezuan deleted the fix_alertmanager_installation branch August 13, 2019 13:00
cegeka-jenkins pushed a commit to cegeka/puppet-prometheus that referenced this pull request Aug 28, 2019
…ation

Link the amtool only if it is installed via direct download.
Rovanion pushed a commit to Rovanion/puppet-prometheus that referenced this pull request May 5, 2021
…ation

Link the amtool only if it is installed via direct download.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

amtool is unconditionally linked from /opt/, even if it is installed differently.
2 participants