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

Support for fully removing relations #21

Merged
merged 3 commits into from
Oct 8, 2017

Conversation

tbaumann
Copy link
Contributor

@tbaumann
Copy link
Contributor Author

Sorry for not noticing 6f48e5c before. Since the hook runs every time, it writes the conf file every time and then restarts the service.
It should only restart the service if the file changed.

@lazypower
Copy link

@tbaumann This looks like it has overlap with #25. Could you rebase this commit with the work that landed so we can shimmy this fix in? (also are you interested in taking over maintainership of the elastic layers? :) )

Copy link

@lazypower lazypower left a comment

Choose a reason for hiding this comment

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

Needs a rebase. Please rebase and resubmit

@tbaumann
Copy link
Contributor Author

tbaumann commented Oct 4, 2017

Sorry @chuckbutler I was super busy recently. I will have a look at this. Will get back to you soon.

About maintaining. Better not, I don't know much at all about Elastic. I just added the features I needed.
And quite frankly, the project that was planned with filebeat, which I was involved with might not even work. I fear my priorities will shift away soon...

But I'm glad to help with anything. I don't mind at all to play around with charms if I have the time. :)

…o issue/16

Solved  Conflicts:
	reactive/filebeat.py
@tbaumann
Copy link
Contributor Author

tbaumann commented Oct 4, 2017

Done. Was basically a whitespace fix.

@tbaumann
Copy link
Contributor Author

tbaumann commented Oct 4, 2017

Ack. This should solve #25

@lazypower lazypower merged commit dbd9e8a into canonical:master Oct 8, 2017
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