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

feat: Add fail2ban_exporter role #294

Merged
merged 4 commits into from Feb 22, 2024

Conversation

anviar
Copy link
Contributor

@anviar anviar commented Feb 17, 2024

Add new role: fail2ban_exporter

Signed-off-by: anviar <oleg.kluchkin+github@gmail.com>
@github-actions github-actions bot added the enhancement New feature or request label Feb 17, 2024
Copy link
Contributor

github-actions bot commented Feb 17, 2024

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and the docs are now incorporated into main:
https://prometheus-community.github.io/ansible/branch/main

Signed-off-by: anviar <oleg.kluchkin+github@gmail.com>
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 17, 2024
Signed-off-by: anviar <oleg.kluchkin+github@gmail.com>
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 17, 2024
Signed-off-by: anviar <oleg.kluchkin+github@gmail.com>
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 17, 2024
armv6l: 'armv6'

go_arch: "{{ go_arch_map[ansible_architecture] | default(ansible_architecture) }}"
_gitlab_projectid: 24199687
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we need to update .github/scripts/version_updater.sh to handle checking for new versions from GitLab projects.

Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Not going to block this on having auto-updates. But it would be good to followup with some refactoring to handle github vs gitlab.

@SuperQ SuperQ requested a review from gardar February 21, 2024 16:51
@anviar
Copy link
Contributor Author

anviar commented Feb 21, 2024

This project have github mirror, but I prefer to use original source.
I will check .github/scripts/version_updater.sh, may be I can help with that.
Should I open another PR for that update or use this one?

@SuperQ
Copy link
Contributor

SuperQ commented Feb 21, 2024

A new PR is fine.

@SuperQ SuperQ merged commit 945f393 into prometheus-community:main Feb 22, 2024
43 checks passed
@anviar anviar deleted the fail2ban_exporter branch March 6, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants