Skip to content

Conversation

@soulbird
Copy link
Contributor

@soulbird soulbird commented Oct 9, 2022

Description

Currently the t/plugin test case takes too long to run, causing the entire CI to take too long to run.
Split the t/plugin test case into two parts and execute them concurrently to reduce the overall running time of CI.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@soulbird soulbird marked this pull request as ready for review October 10, 2022 09:11
@tzssangglass
Copy link
Member

CI / build (ubuntu-20.04, linux_openresty, t/plugin/[a-o]*) (pull_request) Successful in 45m
CI / build (ubuntu-20.04, linux_openresty, t/plugin/[p-z]*) (pull_request) Successful in 33m

CI Centos7 / run ci on centos7 (t/plugin/[a-o]*) (pull_request) Successful in 43m
CI Centos7 / run ci on centos7 (t/plugin/[p-z]*) (pull_request) Successful in 33m

It looks like [a-o] is taking ten more minutes

tokers
tokers previously approved these changes Oct 12, 2022
@soulbird soulbird marked this pull request as draft October 12, 2022 06:25
@soulbird soulbird marked this pull request as ready for review October 13, 2022 01:34
@soulbird soulbird requested a review from tzssangglass October 13, 2022 01:34
tzssangglass
tzssangglass previously approved these changes Oct 13, 2022
@spacewander spacewander merged commit 4e25e5b into apache:master Oct 14, 2022
Liu-Junlin pushed a commit to Liu-Junlin/apisix that referenced this pull request Nov 4, 2022
Co-authored-by: soulbird <zhaothree@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants