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

Add jail.conf.epp template for bionic #89

Merged
merged 4 commits into from
Jan 18, 2019

Conversation

mnencia
Copy link

@mnencia mnencia commented Sep 28, 2018

Support for Ubuntu Bionic (18.04)

@bastelfreak bastelfreak changed the title Added jail.conf.epp template for bionic Add jail.conf.epp template for bionic Sep 29, 2018
@bastelfreak
Copy link
Member

Hi @mnencia, thanks for this PR. Can you please add ubuntu 18.04 to our metadata.json? Also you can enable acceptance tests by copying this block (and adjusting it to 18.04). Last but not least please add it to our .sync.yml

@bastelfreak bastelfreak added enhancement New feature or request needs-work not ready to merge just yet labels Sep 29, 2018
@jg-development
Copy link

@mnencia
+1 for metadata.json and merge

Signed-off-by: Francesco Canovai <francesco.canovai@2ndquadrant.it>
Signed-off-by: Francesco Canovai <francesco.canovai@2ndquadrant.it>
Signed-off-by: Francesco Canovai <francesco.canovai@2ndquadrant.it>
@fcanovai
Copy link
Contributor

fcanovai commented Jan 8, 2019

I've made the requested changes and fixed the jail template because a test was failing. It seems a couple of test still fail but for unrelated reasons, as restarting them without changing anything.

@bastelfreak bastelfreak added needs-work not ready to merge just yet and removed needs-work not ready to merge just yet labels Jan 16, 2019
@bastelfreak
Copy link
Member

thanks!

@bastelfreak bastelfreak merged commit 0c12580 into voxpupuli:master Jan 18, 2019
cegeka-jenkins pushed a commit to cegeka/puppet-fail2ban that referenced this pull request Mar 30, 2020
Add jail.conf.epp template for bionic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants