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 event counting file #107

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mikemorency
Copy link
Collaborator

SUMMARY

This adds an event_counting.yml file to the repo. This file is still experimental, but counts and identifies the number of nodes indirectly managed by a module

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

event_counting

Copy link

codecov bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.60%. Comparing base (15b4a74) to head (c53bbe8).
Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   28.39%   27.60%   -0.79%     
==========================================
  Files          33       35       +2     
  Lines        2511     2840     +329     
  Branches      453      534      +81     
==========================================
+ Hits          713      784      +71     
- Misses       1798     2056     +258     
Flag Coverage Δ
sanity 27.60% <ø> (-0.79%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

meta/event_counting.yml Outdated Show resolved Hide resolved
@pb82
Copy link

pb82 commented Jan 28, 2025

@mikemorency Ansible Core asked us to move the file into extensions instead of meta.

@AlanCoding
Copy link

The extensions folder needs to have things in subfolders. We were talking about extensions/audit/event_query.yml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants