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

Make e2e test not often fail #732

Merged
merged 11 commits into from
Aug 11, 2020
Merged

Make e2e test not often fail #732

merged 11 commits into from
Aug 11, 2020

Conversation

baurine
Copy link
Collaborator

@baurine baurine commented Aug 10, 2020

  • Add an extra step to debug TiUP

  • Search logs by keyword "welcome" to reduce the result size

    After this change, the "Test / UI" job has success 5 times continuously. (Although I don't know why the big result size causes the job not stable)

    image

@baurine baurine changed the title Check why e2e test still fails [WIP] Check why e2e test still fails Aug 10, 2020
@baurine baurine added the status/DNM Do not merge label Aug 10, 2020
@baurine baurine changed the title [WIP] Check why e2e test still fails Make e2e test not often fail Aug 10, 2020
@baurine
Copy link
Collaborator Author

baurine commented Aug 10, 2020

@breeswish , PTAL, thanks!

Copy link
Member

@breezewish breezewish left a comment

Choose a reason for hiding this comment

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

Cool

@breezewish breezewish merged commit d4e9edb into pingcap:master Aug 11, 2020
@baurine baurine deleted the check-ci branch August 11, 2020 06:46
breezewish pushed a commit that referenced this pull request Sep 8, 2020
(cherry picked from commit d4e9edb)
breezewish added a commit that referenced this pull request Sep 8, 2020
* feature: Query Editor (#713)
* Backend: replace UNIX_TIMESTAMP with FROM_UNIXTIME in statement query (#731)
* test: Stablize e2e tests (#732)
* ui: Add store location tree (#728)
* log_search: Display instance port (#722)
* ui: show store location topology (#719)
* ui: Online Config (#733)
* ui: Support sharing session (#741)
* doc: Update to sig-diagnosis (#742)
* statement: Display number of plans in the list (#746)
* log search: return zip instead of tar or gzip (#724)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/DNM Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants