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.

Sign up for GitHub

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

Jump to bottom

nginx and nginx_exporter roles were added #62

Merged
merged 2 commits into from
Nov 22, 2023
Merged

nginx and nginx_exporter roles were added #62

merged 2 commits into from
Nov 22, 2023

Conversation

kogeler
Copy link
Contributor

@kogeler kogeler commented Nov 22, 2023 •
edited
Loading

changes:

  • nginx and nginx_exporter roles were added
  • lxd molecule driver was disabled. This driver was archived and it is not maintained anymore. It has dependencies conflicts with other components
  • the lint stage was removed from molecule. lint running was added as a separate step. the latest version of ansible-lint has new requirements, it will be enabled as a separate PR
  • Ansible version was upgraded to the latest version

kogeler added 2 commits November 22, 2023 14:11
nginx and nginx_exporter roles were added
da3687e 
Signed-off-by: kogeler <roman.gavrilov@parity.io>
fix
ee1ac02 
Signed-off-by: kogeler <roman.gavrilov@parity.io>
@kogeler kogeler marked this pull request as ready for review November 22, 2023 12:20
@kogeler kogeler merged commit 1b2565d into main Nov 22, 2023
6 checks passed
@kogeler kogeler deleted the add-nginx branch November 22, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PierreBesson PierreBesson PierreBesson 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.
2 participants
@kogeler @PierreBesson