Skip to content

feat(eda): make Event-Driven Ansible usable on AAP (sources, rulebooks, deps, docs)#12

Open
thomas-souliotis-lm wants to merge 6 commits into
mainfrom
eda/aap-cli-examples-and-docs
Open

feat(eda): make Event-Driven Ansible usable on AAP (sources, rulebooks, deps, docs)#12
thomas-souliotis-lm wants to merge 6 commits into
mainfrom
eda/aap-cli-examples-and-docs

Conversation

@thomas-souliotis-lm

Copy link
Copy Markdown

Summary

Follow-up to PR #11 to make the EDA content actually runnable on AAP and the ansible-rulebook CLI.

  • alerts (polling): fix broken endpoint — query POST /ui/query/records (not the 404 GET /api/v1/alerts); normalize real cf.*/meta.* keys; add min_severity/lookback.
  • webhook: accept standard X-Hub-Signature-256 HMAC header (keep legacy X-Edwin-Signature); guard the aiohttp import.
  • rulebooks/playbooks: add run_job_template variants under extensions/eda/rulebooks/aap/ (AAP doesn't support run_playbook) + a raw-payload event-stream variant; add playbooks/debug_event.yml smoke-test target.
  • deps: declare aiohttp/aiokafka in requirements.txt and meta/ee-requirements.txt.
  • docs: EDA sources, ingestion architectures, CLI/AAP testing, and the Decision Environment requirement; changelog fragment.

@stair-lm stair-lm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As mentioned in 1:1, you must update the semver:

version: 1.0.1

Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
Comment thread docs/getting-started.md Outdated
Comment thread extensions/eda/plugins/event_source/alerts.py Outdated
Comment thread extensions/eda/plugins/event_source/alerts.py Outdated
Comment thread extensions/eda/plugins/event_source/alerts.py Outdated
Comment thread extensions/eda/plugins/event_source/alerts.py
Comment thread extensions/eda/plugins/event_source/webhook.py Outdated
Comment thread extensions/eda/plugins/event_source/alerts.py Outdated
Comment thread README.md Outdated
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.

2 participants