Skip to content

Fix scheduler and controllermanager k8s tests#30870

Merged
ChrsMark merged 2 commits into7.17from
fix_k8s_tests
Mar 16, 2022
Merged

Fix scheduler and controllermanager k8s tests#30870
ChrsMark merged 2 commits into7.17from
fix_k8s_tests

Conversation

@ChrsMark
Copy link
Copy Markdown
Member

@ChrsMark ChrsMark commented Mar 16, 2022

What does this PR do?

Align testing with #26729 after default k8s version for integration test was updated at #30747.

Actually this is due to https://github.com/elastic/infra/pull/35111, because for goInteg tests kind is a requirement and is actually installed by default on the workers by https://github.com/elastic/infra/pull/35111.

@ChrsMark ChrsMark self-assigned this Mar 16, 2022
@ChrsMark ChrsMark added the Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team label Mar 16, 2022
Copy link
Copy Markdown
Contributor

@tetianakravchenko tetianakravchenko left a comment

Choose a reason for hiding this comment

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

LGTM

@botelastic botelastic Bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Mar 16, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 16, 2022

💚 Build Succeeded

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-03-16T17:10:52.066+0000

  • Duration: 83 min 15 sec

Test stats 🧪

Test Results
Failed 0
Passed 9331
Skipped 2374
Total 11705

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Signed-off-by: chrismark <chrismarkou92@gmail.com>
@ph
Copy link
Copy Markdown
Contributor

ph commented Mar 16, 2022

@ChrsMark This should probably go in all our branch? 7.17, 8, 8.1 ? I've hit this problem in other PR.

@ChrsMark
Copy link
Copy Markdown
Member Author

@ChrsMark This should probably go in all our branch? 7.17, 8, 8.1 ? I've hit this problem in other PR.

I have one PR for 8.0 too: #30872. I think for 8.1 we should be fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants