Skip to content

run conformance tests on three Kubernetes versions#681

Merged
arkodg merged 3 commits intoenvoyproxy:mainfrom
skriss:pr-k8s-version-matrix
Nov 3, 2022
Merged

run conformance tests on three Kubernetes versions#681
arkodg merged 3 commits intoenvoyproxy:mainfrom
skriss:pr-k8s-version-matrix

Conversation

@skriss
Copy link
Copy Markdown
Contributor

@skriss skriss commented Nov 2, 2022

Closes #493.

Signed-off-by: Steve Kriss krisss@vmware.com

@skriss skriss requested a review from a team as a code owner November 2, 2022 22:51
@skriss skriss mentioned this pull request Nov 2, 2022
4 tasks
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 2, 2022

Codecov Report

Merging #681 (05f0ff1) into main (0646f94) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #681      +/-   ##
==========================================
- Coverage   63.63%   63.61%   -0.03%     
==========================================
  Files          47       47              
  Lines        5766     5749      -17     
==========================================
- Hits         3669     3657      -12     
+ Misses       1873     1866       -7     
- Partials      224      226       +2     
Impacted Files Coverage Δ
internal/provider/kubernetes/httproute.go 59.25% <0.00%> (-8.24%) ⬇️
internal/provider/kubernetes/gatewayclass.go 72.46% <0.00%> (-1.45%) ⬇️
internal/cmd/root.go 100.00% <0.00%> (ø)
internal/provider/kubernetes/gateway.go 51.59% <0.00%> (+1.91%) ⬆️
internal/cmd/versions.go 50.00% <0.00%> (+21.42%) ⬆️

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

Closes envoyproxy#493.

Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss skriss force-pushed the pr-k8s-version-matrix branch from b976e81 to 2accddd Compare November 2, 2022 23:16
verbose: true

conformance-tests:
runs-on: ubuntu-latest
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we ideally want unit-tests, conformance and push to be on the same runner

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

AFAIK there's no way to run steps within a job in parallel, hence splitting conformance out into its own matrix job so it's not too slow. We can use upload/download artifact steps to share the binaries between jobs (this is the gist of #683). Are there other reasons for wanting everything on the same runner?

Copy link
Copy Markdown
Contributor

@arkodg arkodg Nov 3, 2022

Choose a reason for hiding this comment

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

the only one is to make sure the same binary is used between all steps, so recommend

Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss skriss force-pushed the pr-k8s-version-matrix branch from 02e2042 to fa7dd3a Compare November 3, 2022 21:09
Signed-off-by: Steve Kriss <krisss@vmware.com>
@arkodg arkodg merged commit 80c78a5 into envoyproxy:main Nov 3, 2022
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.

test EG against multiple Kubernetes versions

3 participants