-
Notifications
You must be signed in to change notification settings - Fork 553
Remove duplicated fields #3316
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
Closed
Closed
Remove duplicated fields #3316
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
e22eec9
Activemq
jsoriano 8f242da
Akamai
jsoriano 05e4ca7
Apache
jsoriano c2c2f96
Auditd
jsoriano 6eed806
Auth0
jsoriano 64d2aa9
AWS
jsoriano 54797ac
Merge remote-tracking branch 'origin/main' into remove-duplicated-fields
jsoriano 2de2406
Recover missing docs
jsoriano f0d81fc
Remove duplicates in ecs files
jsoriano 6a2ab61
Remove fields and sub-fields defined in ECS
jsoriano fccd3de
More duplicates
jsoriano e4bd8c6
More duplicates in ECS files
jsoriano 43361a4
Remove empty files
jsoriano e98bde5
Remove duplicated from agent.yml
jsoriano c270995
More fields
jsoriano 51686b3
Merge remote-tracking branch 'origin/main' into remove-duplicated-fields
jsoriano fa356ff
Merge remote-tracking branch 'origin/main' into remove-duplicated-fields
jsoriano 166a710
Cassandra
jsoriano 89b4298
More duplicated fields
jsoriano d254c06
Build readmes
jsoriano 107d7a1
Merge remote-tracking branch 'origin/main' into remove-duplicated-fields
jsoriano cc1aa5d
Revert some removals
jsoriano 2b2f05d
Use elastic-package from master for testing
jsoriano 876186e
Adjustments in AWS
jsoriano 96a9841
Recover host.containerized
jsoriano b4c3119
Fixes for azure
jsoriano 00dc316
Cisco
jsoriano 6482a5a
Revert more deletions
jsoriano badb360
Fix netflow
jsoriano da3a906
Revert changes in network_traffic
jsoriano 25b9989
Revert removal of dns.answers
jsoriano 84a335a
Fix format
jsoriano 71c9f55
Merge remote-tracking branch 'origin/main' into remove-duplicated-fields
jsoriano File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't expect any adds or removes in the READMEs, only modifications to descriptions or types due to the duplicate removal.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adds are going to happen, because of elastic/elastic-package#814, we were installing fields that were not documented.
But yes, removals shouldn't happen. There are still some other things I have to polish here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrewkroh at the end having adds and removes was a a good indicator that something was wrong 🙂
elastic/package-spec#331
#3392