Skip to content

Add missing system resources - #16

Merged
mtojek merged 1 commit into
elastic:masterfrom
mtojek:447-add-system-resources
May 12, 2020
Merged

mtojek merged 1 commit into
elastic:masterfrom
mtojek:447-add-system-resources

Conversation

@mtojek

@mtojek mtojek commented May 12, 2020

Copy link
Copy Markdown
Contributor

Issue: elastic/package-registry#447

Pushing everything at once will cause a huge mess and make all PRs non-reviewable. I will solve this issue in small steps.

First one: add missing system resources

@mtojek
mtojek requested a review from ruflin May 12, 2020 10:24
@mtojek mtojek self-assigned this May 12, 2020
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

@mtojek
mtojek merged commit df1b0ae into elastic:master May 12, 2020
zmoog added a commit that referenced this pull request Apr 14, 2026
Transit gateway flow logs can have both tgw-src-eni and tgw-dst-eni
set to "-", leaving no eni- in the record. Since tgw- is specific
enough on its own (CloudTrail is matched before this check), promote
it to a standalone primary path:

  tgw- || ((eni- || nat-) && (accept || reject || nodata || skipdata))

Adds test case #16: transit gateway with OK status and no ENI fields,
detected solely via tgw-.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
zmoog added a commit that referenced this pull request Apr 14, 2026
Transit gateway flow logs can have both tgw-src-eni and tgw-dst-eni
set to "-", leaving no eni- in the record. Since tgw- is specific
enough on its own (CloudTrail is matched before this check), promote
it to a standalone primary path:

  tgw- || ((eni- || nat-) && (accept || reject || nodata || skipdata))

Adds test case #16: transit gateway with OK status and no ENI fields,
detected solely via tgw-.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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