Skip to content
This repository was archived by the owner on May 31, 2023. It is now read-only.

Add option to disable alert execution #139

Merged
merged 4 commits into from
Feb 20, 2019

Conversation

Duologic
Copy link
Contributor

No description provided.

@Duologic Duologic force-pushed the disable_alert_execution branch 2 times, most recently from 21ba04f to 53c45fd Compare February 19, 2019 18:53
@Duologic Duologic force-pushed the disable_alert_execution branch 2 times, most recently from 1ce35b7 to 12f94a4 Compare February 19, 2019 19:02
@Duologic Duologic force-pushed the disable_alert_execution branch from 12f94a4 to c2943b2 Compare February 19, 2019 19:36
@Duologic
Copy link
Contributor Author

I'm trying something with installing an older version of Ansible-lint as it is giving issues due to a stricter rule set. See articles linked in the commit.

@Duologic Duologic force-pushed the disable_alert_execution branch from c2943b2 to 739d3f3 Compare February 19, 2019 19:42
@Duologic
Copy link
Contributor Author

Okay, that didn't work. Any ideas? 😄

@paulfantom
Copy link
Member

paulfantom commented Feb 19, 2019

You can add .ansible-lint with following content:

---
skip_list:
- '602'

This is acceptable workaround for now.

We are doing similar thing for node_exporter role: https://github.com/cloudalchemy/ansible-node-exporter/blob/master/.ansible-lint

@Duologic Duologic force-pushed the disable_alert_execution branch from 739d3f3 to facfe16 Compare February 20, 2019 07:58
@Duologic
Copy link
Contributor Author

We got white smoke. I believe this should cover the new feature.

@paulfantom paulfantom merged commit 32f1a59 into cloudalchemy:master Feb 20, 2019
@lock
Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants