Skip to content

fix: remove EDA self-POST play from introduce_cve.yml#134

Merged
toharris-rh merged 1 commit into
mainfrom
fix/introduce-cve-remove-eda-play
Jun 16, 2026
Merged

fix: remove EDA self-POST play from introduce_cve.yml#134
toharris-rh merged 1 commit into
mainfrom
fix/introduce-cve-remove-eda-play

Conversation

@toharris-rh

Copy link
Copy Markdown
Owner

Summary

  • Removes the second play from introduce_cve.yml that self-POSTed a CVE notification to the EDA event stream.
  • The SNow CVE Demo workflow already sets skip_eda_trigger: true on the node, so the play was always being skipped — but when not set, it hangs on the INSIGHTS_EDA_EVENT_STREAM_URL assert and then polls Insights for up to 5 minutes before failing.
  • The ServiceNow Business Rule now handles EDA triggering automatically when the INC is created, making the self-POST path redundant.
  • Backup preserved at playbooks/introduce_cve.yml.bak (gitignored).

Closes #133

🤖 Generated with Claude Code

The second play polled Insights and self-POSTed a CVE notification to
the EDA event stream. The SNow CVE Demo workflow bypasses it via
skip_eda_trigger: true, and the ServiceNow Business Rule now handles
EDA triggering automatically — making the play redundant and a source
of demo failures when INSIGHTS_EDA_EVENT_STREAM_URL is not set.

Closes #133

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@toharris-rh toharris-rh requested a review from ericcames as a code owner June 16, 2026 22:36
@toharris-rh toharris-rh merged commit 2577bcf into main Jun 16, 2026
2 checks passed
@toharris-rh toharris-rh deleted the fix/introduce-cve-remove-eda-play branch June 16, 2026 22:37
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.

Remove EDA self-POST section from introduce_cve.yml

1 participant