-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[CI] update serverless test index #17137
base: main
Are you sure you want to change the base?
Conversation
|
💛 Build succeeded, but was flaky
Failed CI Steps |
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.
Change LGTM!
Some questions:
- One thing if we can validate: when upstream KB & ES changes get merged, serverless integration tests expectedly should fail, right? It would be great if we can validate first.
- Does serverless tests run on main only? I mean how are we covering 8.x?
Yes. good suggestion
Yes, main only. You are right. We should cover 8x. Let me create an issue to track it. |
follow-up for testing 8.x #17154 |
@kaisecheng the PRs above have been merged |
This pull request does not have a backport label. Could you fix it @kaisecheng? 🙏
|
|
update metricbeat
.monitoring-logstash-8-mb
to.monitoring-logstash-9-mb
wait until [1], [2] get merged