Skip to content

Try to use version of elastic-package with check for allowed values#3017

Closed
jsoriano wants to merge 11 commits intoelastic:mainfrom
jsoriano:enable-allowed-values-validation
Closed

Try to use version of elastic-package with check for allowed values#3017
jsoriano wants to merge 11 commits intoelastic:mainfrom
jsoriano:enable-allowed-values-validation

Conversation

@jsoriano
Copy link
Member

@jsoriano jsoriano commented Apr 6, 2022

DON'T MERGE

Tests elastic/elastic-package#771 with integrations.

@jsoriano jsoriano requested a review from a team as a code owner April 6, 2022 11:44
@jsoriano jsoriano self-assigned this Apr 6, 2022
@jsoriano jsoriano marked this pull request as draft April 6, 2022 11:45
@jsoriano jsoriano mentioned this pull request Apr 6, 2022
Closed
17 tasks
@elasticmachine
Copy link

elasticmachine commented Apr 6, 2022 •
edited
Loading

💔 Tests 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-06-08T22:05:39.242+0000

  • Duration: 93 min 41 sec

Test stats 🧪

Test Results
Failed 1
Passed 5080
Skipped 5
Total 5086

Test errors 1

Expand to view the tests failures

Check integrations / mysql / mysql: check / system test: default (variant: mariadb_10_3_14) – mysql.slowlog
    Expand to view the error details

     null

    Expand to view the stacktrace

     one or more errors found in documents stored in logs-mysql.slowlog-ep data stream: [0] found error.message in event: Provided Grok expressions do not match field value: [e_id, Offset, Is_DST, Abbreviation) VALUES\n (@time_zone_id, 0, -14400, 1, 'EDT')\n,(@time_zone_id, 1, -18000, 0, 'EST')\n,(@time_zone_id, 2, -14400, 1, 'EWT')\n,(@time_zone_id, 3, -14400, 1, 'EPT');]

Steps errors 2

Expand to view the steps failures

Test integration: mysql
  • Took 18 min 55 sec . View more details here
  • Description: eval "$(../../build/elastic-package stack shellinit)" ../../build/elastic-package test -v --report-format xUnit --report-output file --test-coverage
Boot up the Elastic stack
  • Took 2 min 44 sec . View more details here
  • Description: ../../build/elastic-package stack up -d -v --version 7.14.0

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@andrewkroh
Copy link
Member

andrewkroh commented Apr 6, 2022

Nice. This will be a really helpful validation to have.

@elasticmachine
Copy link

elasticmachine commented May 11, 2022 •
edited
Loading

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (258/258) 💚
Files 96.544% (419/434) 👎 -0.024
Classes 96.544% (419/434) 👎 -0.024
Methods 88.833% (4383/4934) 👎 -0.056
Lines 89.739% (78834/87848) 👎 -0.018
Conditionals 100.0% (0/0) 💚

@jsoriano jsoriano mentioned this pull request May 23, 2022
Closed
4 tasks
@jsoriano
Copy link
Member Author

jsoriano commented Jun 9, 2022

All issues identified by this change has been solved, closing this.

@jsoriano jsoriano closed this Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jsoriano jsoriano

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jsoriano @elasticmachine @andrewkroh