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

[DRAFT][DO NOT MERGE]: Feature indirect host counting #15791

Closed
wants to merge 8 commits into from

Conversation

pb82
Copy link
Contributor

@pb82 pb82 commented Jan 29, 2025

SUMMARY

Check to see if tests pass

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
AWX VERSION
devel
ADDITIONAL INFORMATION

AlanCoding and others added 6 commits January 29, 2025 17:16

Verified

This commit was signed with the committer’s verified signature.
alexsnaps Alex Snaps
…on (#15774)

* Add jq dependency

* Add file in progress

* Add license for jq

* Write test and get it passing
* Callback plugin method from cmeyers adapted to global collection list

Get tests passing

Mild rebranding

Put behind feature flag, flip true in dev

Add noqa flag

* Add missing wait_for_events
* Minor import changes to collection processing in callback plugin

* Move agreed location of event_query file
…#15784)

* Rename the indirect host counting test file

* Combine artifacts saving logic
@github-actions github-actions bot added component:api dependencies Pull requests that update a dependency file community labels Jan 29, 2025
Copy link

codecov bot commented Jan 29, 2025

Codecov Report

Attention: Patch coverage is 39.30348% with 122 lines in your changes missing coverage. Please review.

Project coverage is 53.16%. Comparing base (c6930bd) to head (6b24590).

✅ All tests successful. No failed tests found.

❌ Your project status has failed because the head coverage (53.16%) is below the target coverage (100.00%). You can increase the head coverage or adjust the target coverage.

❗ There is a different number of reports uploaded between BASE (c6930bd) and HEAD (6b24590). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (c6930bd) HEAD (6b24590)
CI-GHA 8 7
pytest 5 4
OS-Linux 8 7

Copy link

@AlanCoding AlanCoding closed this Jan 29, 2025
@AlanCoding AlanCoding deleted the feature-indirect-host-counting branch January 29, 2025 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community component:api dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants