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

feat(ci): add a special label to run full test matrix #4770

Merged
merged 8 commits into from
Aug 8, 2022

Conversation

slonka
Copy link
Contributor

@slonka slonka commented Aug 5, 2022

This PR gives the contributor an ability to run the whole matrix test suite by adding "ci/run-full-matrix" label to the PR.

The master branch is broken now because of #4769 but you can see from the history that all targets were triggered.

Signed-off-by: slonka slonka@users.noreply.github.com

Checklist prior to review

  • Link to docs PR or issue -- CI improvement
  • Link to UI issue or PR -- not a UI issue
  • Is the issue worked on linked? -- not reported as a bug
  • Unit Tests -- not a kuma feature
  • E2E Tests -- not a kuma feature
  • Manual Universal Tests -- not a kuma feature, but tested manually on circleCI
  • Manual Kubernetes Tests -- not a kuma feature
  • Do you need to update UPGRADE.md? -- no
  • Does it need to be backported according to the backporting policy? -- no

Signed-off-by: slonka <slonka@users.noreply.github.com>
@slonka slonka added the ci/run-full-matrix PR: Runs all possible e2e test combination (expensive use carefully) label Aug 5, 2022
Signed-off-by: slonka <slonka@users.noreply.github.com>
Signed-off-by: slonka <slonka@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented Aug 5, 2022

Codecov Report

Merging #4770 (6f38d53) into master (c4809e1) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4770      +/-   ##
==========================================
+ Coverage   46.35%   46.36%   +0.01%     
==========================================
  Files         690      690              
  Lines       46942    46942              
==========================================
+ Hits        21762    21767       +5     
+ Misses      23254    23251       -3     
+ Partials     1926     1924       -2     
Impacted Files Coverage Δ
pkg/plugins/runtime/gateway/route/sorter.go 71.79% <0.00%> (+5.12%) ⬆️
pkg/plugins/leader/postgres/leader_elector.go 89.36% <0.00%> (+6.38%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

.circleci/config.yml Outdated Show resolved Hide resolved
Signed-off-by: slonka <slonka@users.noreply.github.com>
@slonka slonka force-pushed the feat/run-full-matrix-on-label branch from 49e4ea3 to 69d96fa Compare August 5, 2022 12:51
Signed-off-by: slonka <slonka@users.noreply.github.com>
Signed-off-by: slonka <slonka@users.noreply.github.com>
Signed-off-by: slonka <slonka@users.noreply.github.com>
@slonka slonka removed the ci/run-full-matrix PR: Runs all possible e2e test combination (expensive use carefully) label Aug 8, 2022
@slonka slonka marked this pull request as ready for review August 8, 2022 09:27
@slonka slonka requested a review from a team as a code owner August 8, 2022 09:27
Signed-off-by: slonka <slonka@users.noreply.github.com>
Copy link
Contributor

@jakubdyszkiewicz jakubdyszkiewicz left a comment

Choose a reason for hiding this comment

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

awesome!

@slonka slonka merged commit 8948314 into kumahq:master Aug 8, 2022
@slonka slonka deleted the feat/run-full-matrix-on-label branch August 8, 2022 14:56
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