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

Update GitHub actions CI workflow #124

Merged

Conversation

austinvazquez
Copy link
Member

@austinvazquez austinvazquez commented Oct 25, 2022

Update GitHub actions runner OS from Ubuntu 18.04 to Ubuntu 22.04.
Update Go compiler from Go 1.17 to Go 1.20. Update actions/setup-go from
v2 to v3. Update golangci-lint to v1.51.x for Go 1.20 support. Remove
deprecated -i flag from make coverage target.

Related issue: containerd/project#95

Signed-off-by: Austin Vazquez [email protected]

@austinvazquez austinvazquez marked this pull request as ready for review October 25, 2022 22:27
@estesp
Copy link
Member

estesp commented Oct 28, 2022

This seems to be consistently failing even after 2 re-runs:

--- FAIL: TestServerShutdown (0.00s)
    server_test.go:275: expected connections to be empty: map[0xc000097d10:{}]

only on Windows platform. Not sure what's up there

@austinvazquez
Copy link
Member Author

This seems to be consistently failing even after 2 re-runs:

--- FAIL: TestServerShutdown (0.00s)
    server_test.go:275: expected connections to be empty: map[0xc000097d10:{}]

only on Windows platform. Not sure what's up there

I'm looking into this more. It looks like a flaky test on Windows as it is now failing in mainline.

@austinvazquez austinvazquez force-pushed the update-github-actions-workflow branch 6 times, most recently from 8caeecb to 86d0a4c Compare February 24, 2023 16:17
@austinvazquez austinvazquez marked this pull request as draft February 24, 2023 16:25
@austinvazquez austinvazquez force-pushed the update-github-actions-workflow branch 12 times, most recently from f7b1c88 to 81130e7 Compare February 25, 2023 00:38
@austinvazquez austinvazquez marked this pull request as ready for review February 25, 2023 00:58
@austinvazquez
Copy link
Member Author

I split troubleshooting of flaky tests into #129 and #131

@estesp
Copy link
Member

estesp commented Mar 8, 2023

Rebase now and this should be mergable

@austinvazquez austinvazquez force-pushed the update-github-actions-workflow branch from 81130e7 to 7e1dc73 Compare March 8, 2023 21:54
Update GitHub Actions runner OS from Ubuntu 18.04 to Ubuntu 22.04.
Update actions/setup-go from v2 to v3. Update Go compiler from Go 1.17
to Go 1.20. Update golangci-lint from v1.45.0 to v1.51.2 for Go 1.20
support. Remove deprecated `-i` flag from make coverage target.

Signed-off-by: Austin Vazquez <[email protected]>
@austinvazquez austinvazquez force-pushed the update-github-actions-workflow branch from 7e1dc73 to c7b5a32 Compare March 8, 2023 23:06
@austinvazquez
Copy link
Member Author

Rebase now and this should be mergable

Rebased. Will continue to troubleshoot Windows CI flakiness further but this should in a good enough state to keep the lights on for CI / unblock the telemetry work by @swagatbora90 .

@estesp estesp merged commit 98b5f64 into containerd:main Mar 9, 2023
@austinvazquez austinvazquez deleted the update-github-actions-workflow branch March 9, 2023 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants