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 mysqld_exporter role #45

Merged
merged 1 commit into from
Mar 14, 2023
Merged

feat: Add mysqld_exporter role #45

merged 1 commit into from
Mar 14, 2023

Conversation

SuperQ
Copy link
Contributor

@SuperQ SuperQ commented Mar 5, 2023

Add a role for deploying the mysqld_exporter.

Copy link
Member

@gardar gardar left a comment

Choose a reason for hiding this comment

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

Also needs:

  • Argument specs for the role (meta/argument_specs.yml.
  • Integration tests to be able to run molecule with ansible-test.
    Basically just create
    tests/integration/targets/molecule-<role name>-<molecule scenario>/runme.sh
    such as:
    tests/integration/targets/molecule-mysqld_exporter-default/runme.sh
    And have the following in the runme script:
#!/usr/bin/env bash

collection_root=$(pwd | grep -oP ".+\/ansible_collections\/\w+?\/\w+")
source "$collection_root/tests/integration/molecule.sh"

.config/yamllint/config.yml Outdated Show resolved Hide resolved
roles/mysqld_exporter/README.md Outdated Show resolved Hide resolved
roles/mysqld_exporter/README.md Outdated Show resolved Hide resolved
roles/mysqld_exporter/README.md Outdated Show resolved Hide resolved
@SuperQ
Copy link
Contributor Author

SuperQ commented Mar 5, 2023

Yup, I just copied the node_exporter and am still working on this.

@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from 99f04e0 to d360b26 Compare March 9, 2023 10:31
@SuperQ SuperQ changed the title Add mysqld_exporter role feat: Add mysqld_exporter role Mar 9, 2023
@github-actions github-actions bot added the enhancement New feature or request label Mar 9, 2023
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from d360b26 to a36ac10 Compare March 9, 2023 10:36
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 9, 2023
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from a36ac10 to 1286c7e Compare March 9, 2023 15:12
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 9, 2023
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from 1286c7e to 29f4228 Compare March 9, 2023 16:08
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 9, 2023
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from 29f4228 to 993d9f8 Compare March 9, 2023 17:54
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 9, 2023
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from 993d9f8 to dab41e8 Compare March 9, 2023 18:32
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 9, 2023
@SuperQ SuperQ marked this pull request as ready for review March 9, 2023 18:59
@SuperQ SuperQ requested a review from gardar March 9, 2023 18:59
roles/mysqld_exporter/tasks/selinux.yml Outdated Show resolved Hide resolved
roles/mysqld_exporter/defaults/main.yml Outdated Show resolved Hide resolved
roles/mysqld_exporter/tasks/preflight.yml Outdated Show resolved Hide resolved
roles/mysqld_exporter/templates/mysqld_exporter.service.j2 Outdated Show resolved Hide resolved
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from dab41e8 to 871b6fe Compare March 10, 2023 04:08
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 10, 2023
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from 871b6fe to 9a44d1b Compare March 10, 2023 04:12
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 10, 2023
@SuperQ SuperQ requested a review from gardar March 10, 2023 07:55
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from 9a44d1b to f8e655e Compare March 13, 2023 14:05
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 13, 2023
@SuperQ
Copy link
Contributor Author

SuperQ commented Mar 13, 2023

Ok, I updated based on #65

roles/mysqld_exporter/meta/argument_specs.yml Show resolved Hide resolved
roles/mysqld_exporter/vars/main.yml Show resolved Hide resolved
roles/mysqld_exporter/defaults/main.yml Outdated Show resolved Hide resolved
roles/mysqld_exporter/defaults/main.yml Outdated Show resolved Hide resolved
Add a role for deploying the mysqld_exporter.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ SuperQ force-pushed the superq/mysqld_exporter branch from f8e655e to af8fc8f Compare March 13, 2023 20:08
@SuperQ SuperQ requested a review from gardar March 13, 2023 20:09
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 13, 2023
@SuperQ SuperQ merged commit 31f35f3 into main Mar 14, 2023
@SuperQ SuperQ deleted the superq/mysqld_exporter branch March 14, 2023 06:33
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.

2 participants