Skip to content

[Tooling] Add --exclude flag to Generator to support field removal testing - #1411

Merged
djptek merged 45 commits into
elastic:masterfrom
djptek:exclude_set
May 24, 2021
Merged

[Tooling] Add --exclude flag to Generator to support field removal testing#1411
djptek merged 45 commits into
elastic:masterfrom
djptek:exclude_set

Conversation

@djptek

@djptek djptek commented May 12, 2021

Copy link
Copy Markdown
Contributor
@djptek
djptek requested review from ebeahan and kgeller May 12, 2021 11:58
@djptek
djptek marked this pull request as draft May 12, 2021 12:00
@djptek

djptek commented May 12, 2021

Copy link
Copy Markdown
Contributor Author

Note: Moved to Draft while fixing regression
@djptek djptek changed the title [Tooling] Add --exclude flag to Generetor to support field removal testing May 12, 2021
@djptek
djptek marked this pull request as ready for review May 12, 2021 12:24
@djptek

djptek commented May 12, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Regression fixed: the files in folder ./rfcs/text/0017 have been included temporarily as I used those for testing, these will not be included in the final commit
@djptek
djptek marked this pull request as draft May 12, 2021 12:27
@djptek
djptek marked this pull request as ready for review May 12, 2021 12:36
ebeahan
ebeahan reviewed May 12, 2021
View reviewed changes
Comment thread scripts/generator.py Outdated
Comment thread rfcs/text/0017/exclude-set.yml Outdated
Comment thread scripts/schema/subset_filter.py Outdated
@djptek
djptek marked this pull request as draft May 14, 2021 09:18
@djptek

djptek commented May 14, 2021

Copy link
Copy Markdown
Contributor Author

moved to draft while updating unit tests
kgeller
kgeller reviewed May 19, 2021
View reviewed changes
Comment thread scripts/schema/loader.py Outdated
ebeahan
ebeahan reviewed May 19, 2021
View reviewed changes

@ebeahan ebeahan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for refactoring a bit and adding the unit tests!
Comment thread rfcs/text/0017/exclude-set.yml Outdated
Comment thread scripts/generator.py
Comment thread scripts/tests/unit/test_schema_exclude_filter.py Outdated
Comment thread scripts/schema/exclude_filter.py Outdated
@djptek djptek mentioned this pull request May 20, 2021
Closed
@djptek
djptek merged commit 581b12e into elastic:master May 24, 2021
This was referenced May 26, 2021
Merged
Merged
Merged
Merged
Closed
Closed
djptek added a commit that referenced this pull request May 26, 2021
@djptek
[Tooling] Add --exclude flag to Generator to support field removal te…
a00eb89 
…sting (#1411) (#1431)

* add --exclude flag to Generator
djptek added a commit that referenced this pull request May 26, 2021
@djptek
[Tooling] Add --exclude flag to Generator to support field removal te…
8e7f91f 
…sting (#1411) (#1432)

* add --exclude flag to Generator
djptek pushed a commit that referenced this pull request May 26, 2021
djptek
Revert "[Tooling] Add --exclude flag to Generator to support field re…
b29706a 
…moval testing (#1411) (#1432)"

This reverts commit 8e7f91f.
rylnd added a commit to rylnd/ecs that referenced this pull request May 28, 2021
@rylnd
Merge branch 'master' into threat-enrichment-stage-1
caa8983 
* master:
  Stage 2 changes for RFC 0018 - extending the `threat.*` field set (elastic#1438)
  Remove deprecated `host.user.*` fields (elastic#1439)
  Explicitly include user identifiers in `related.user` field description (elastic#1420)
  Set the merge date on RFC 0018 stage 2 (elastic#1429)
  [RFC] Extend Threat Fieldset - Stage 2 Proposal (elastic#1395)
  [Tooling] Add --exclude flag to Generator to support field removal testing (elastic#1411)
  Add `host.user.*` deprecation notice in field reuse description (elastic#1422)
  Stage 2 changes for RFC 0015 - `elf` header (elastic#1410)
  Stage 3 changes for RFC 0012 - `orchestrator` field set (elastic#1417)
  Support `match_only_text` in Go code generator (elastic#1418)
  Stage 3 Orchestrator RFC (elastic#1343)
  moving into folder (elastic#1416)
  removing use-cases (elastic#1405)
  removing --oss (elastic#1404)
  Set the merge date on RFC 0015 stage 2 (elastic#1409)
  Consolidate `Breaking changes` sections in `CHANGELOG.next` (elastic#1408)
  RFC-Stage-0: Proposal to add a "ticket" schema / field definition to ECS (elastic#1383)
  [RFC] `match_only_text` type migration - Stage 0 (elastic#1396)
  Client port is wrongly documented (elastic#1402) (elastic#1406)
@djptek djptek mentioned this pull request Jun 2, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@djptek @ebeahan @kgeller