Skip to content
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

Fix scheduler and controllermanager k8s tests #30870

Merged
merged 2 commits into from
Mar 16, 2022
Merged

Fix scheduler and controllermanager k8s tests #30870

merged 2 commits into from
Mar 16, 2022

Conversation

ChrsMark
Copy link
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
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
Collaborator

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
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
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