-
Notifications
You must be signed in to change notification settings - Fork 3k
Revert "Merge pull request #8788 from kaidokert/master" #8915
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
Conversation
|
CI started. |
|
|
Exporters failure I could not reproduce this yesterday, trying again now. will check if target contain these definitions. |
|
@timurh01 ^^ I reported this yesterday, seems like after few exporters run, we have it again |
Test run: SUCCESSSummary: 4 of 4 test jobs passed |
|
I restarted exporters, and created an issue for the failure #8917 |
|
Exporters restarted, 🤞 |
|
Restarted client test to confirm this fixes the issue |
|
No version, as the PR is only on master |
|
Actually this PR needs to go to whatever version the original offending PR is targetting . Ah should be ok as the original was just this one commit by the looks of it and not targeting a release. |
|
IT was just on master, I removed the version label (it was 5.11.1 but we havent yet released it), so just master. All should be good. Or both for 5.11.1 (does not make sense to me). |
Description
This is a continuation of #8914. As of right now, this apperas to be a suspect commit/PR, since nothing has passed since this PR came in.
Putting this in CI in parallel with the baseline test to get more information.
Note:
The last CI run on #8788 was three days ago.
It's possible that something within the CI tests have been updated such that they're now causing problems, since #8788 PR originally passed all tests.
Pull request type