Skip to content

Test changes to mandatory field attributes#3239

Closed
andrewkroh wants to merge 4 commits intoelastic:mainfrom
andrewkroh:refactor/remove-fields-yml-keys
Closed

Test changes to mandatory field attributes#3239
andrewkroh wants to merge 4 commits intoelastic:mainfrom
andrewkroh:refactor/remove-fields-yml-keys

Conversation

@andrewkroh
Copy link
Member

DO NOT MERGE

Testing package-spec changes to strictly validate the fields.yml contents.

@elasticmachine
Copy link

elasticmachine commented Apr 29, 2022

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-05-03T22:48:42.682+0000

  • Duration: 35 min 7 sec

Test stats 🧪

Test Results
Failed 0
Passed 185
Skipped 1
Total 186

Steps errors 129

Expand to view the steps failures

Show only the first 10 steps failures

Check integration: tomcat
  • Took 0 min 3 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: vsphere
  • Took 0 min 3 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: windows
  • Took 0 min 3 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zeek
  • Took 0 min 9 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zerofox
  • Took 0 min 3 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zookeeper
  • Took 0 min 1 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zoom
  • Took 0 min 2 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zscaler
  • Took 0 min 2 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zscaler_zia
  • Took 0 min 1 sec . View more details here
  • Description: ../../build/elastic-package check -v
Check integration: zscaler_zpa
  • Took 0 min 3 sec . View more details here
  • Description: ../../build/elastic-package check -v

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

elasticmachine commented Apr 29, 2022

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (12/12) 💚
Files 100.0% (17/17) 💚 3.415
Classes 100.0% (17/17) 💚 3.415
Methods 87.815% (209/238) 👎 -0.42
Lines 97.223% (3256/3349) 👍 8.334
Conditionals 100.0% (0/0) 💚

andrewkroh added 4 commits May 3, 2022 18:46
- default_field
- deprecated
- example
- footnote
- group
- level
- overwrite
- possible_values
- prefix
- release
- required
- title

[git-generate]
go install github.com/andrewkroh/go-examples/yaml-remove-key@main
yaml-remove-key -w -key "default_field,deprecated,example,footnote,group,level,overwrite,possible_values,prefix,release,required,title" packages/*/data_stream/*/fields/*.yml
- descripion
- dimensions
- dimensiont
- field

[git-generate]
gsed -i 's/descripion:/description:/g' packages/*/data_stream/*/fields/*.yml
gsed -i 's/dimensions:/dimension:/g' packages/*/data_stream/*/fields/*.yml
gsed -i 's/dimensiont:/dimension:/g' packages/*/data_stream/*/fields/*.yml
gsed -i 's/field:/fields:/g' packages/azure/data_stream/auditlogs/fields/fields.yml
[git-generate]
for i in $(git diff --name-only elastic/main packages | cut -d '/' -f 2 | sort | uniq); do
  echo $i
  pushd packages/$i;
  elastic-package build;
  elastic-package-changelog add-next --type bugfix --description "Remove unused field properties and fix typos." --pr 3239;
  popd;
done
@andrewkroh andrewkroh closed this May 13, 2022
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