Skip to content
This repository has been archived by the owner on Mar 6, 2023. It is now read-only.

fedora support + issue 18 #24

Merged
merged 3 commits into from
Feb 18, 2018
Merged

fedora support + issue 18 #24

merged 3 commits into from
Feb 18, 2018

Conversation

paulfantom
Copy link
Member

@paulfantom paulfantom commented Feb 16, 2018 •
edited
Loading

Fixes:

  • installation on fedora 27 (replacing module yum with package)
  • binary in wrong directory (/opt) is now correctly placed in /usr/local/bin
  • resolves Adding option to configure collectors #18
  • removed enabled by default collectors from default/main.yml

@paulfantom paulfantom changed the title cleanup; fix fedora support fix fedora support + cleaner systemd file Feb 16, 2018
@paulfantom paulfantom changed the title fix fedora support + cleaner systemd file fedora support + issue 18 Feb 17, 2018
rdemachkovych
rdemachkovych suggested changes Feb 17, 2018
View reviewed changes
Copy link
Member

@rdemachkovych rdemachkovych left a comment

Choose a reason for hiding this comment

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

The Travis CI build failed

rdemachkovych
rdemachkovych approved these changes Feb 18, 2018
View reviewed changes
Copy link
Member

@rdemachkovych rdemachkovych left a comment

Choose a reason for hiding this comment

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

LGTM

@paulfantom paulfantom merged commit e498061 into master Feb 18, 2018
@paulfantom paulfantom deleted the fedora_support branch February 18, 2018 09:04
@lock
Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rdemachkovych rdemachkovych rdemachkovych approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Adding option to configure collectors
2 participants
@paulfantom @rdemachkovych