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

slack-vitess-r15.0.5: use go1.21 #260

Merged
merged 7 commits into from
Mar 26, 2024

Conversation

timvaillancourt
Copy link
Member

@timvaillancourt timvaillancourt commented Mar 18, 2024

Description

This PR updates the v15 release to go1.21

This is the GA release of the builtin Go slog package. I plan to use this library instead of the Zap-logger library we used on v14 and I'll upstream the changes

This includes v18 PR vitessio#13853 to allow go1.21

Related Issue(s)

Checklist

  • "Backport to:" labels have been added if this change should be back-ported
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on the CI
  • Documentation was added or is not required

Deployment Notes

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
@github-actions github-actions bot added this to the v15.0.5 milestone Mar 18, 2024
github-actions bot and others added 3 commits March 18, 2024 23:41
Co-authored-by: frouioui <frouioui@users.noreply.github.com>
Co-authored-by: Florent Poinsard <florent.poinsard@outlook.fr>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
@timvaillancourt timvaillancourt marked this pull request as ready for review March 18, 2024 23:03
@timvaillancourt timvaillancourt requested a review from a team as a code owner March 18, 2024 23:03
@timvaillancourt
Copy link
Member Author

timvaillancourt commented Mar 19, 2024

docker_test_cluster_10 / Docker Test Cluster 10 CI failing on 🤔:

running tests for  vitess.io/vitess/go/test/endtoend/vtctldweb
go: downloading github.com/tebeka/selenium v0.9.9
go: downloading github.com/blang/semver v3.5.1+incompatible
unknown error - 33: session not created: This version of ChromeDriver only supports Chrome version 90
Current browser version is 116.0.5845.96 with binary path /usr/bin/chromium
  (Driver info: chromedriver=90.0.4430.24 (4c6d850f087da467d926e8eddb76550aed655991-refs/branch-heads/4430@{#429}),platform=Linux 5.15.0-1058-azure x86_64)
FAIL	vitess.io/vitess/go/test/endtoend/vtctldweb	1.213s
FAIL
2024/03/19 01:01:08 mysql57.web_test: saving test output to _test/20240319-004814.3795/mysql57.web_test-1.1.log
2024/03/19 01:01:08 mysql57.web_test: FAILED (try 1/1) in 1m33.5s: exit status 1
2024/03/19 01:01:08 mysql57.web_test: retry limit exceeded
2024/03/19 01:01:08 Removing temp dir /tmp/vt_85636810
2024/03/19 01:01:08 ============================================================
2024/03/19 01:01:08 mysql57.java                            	PASS
2024/03/19 01:01:08 mysql57.pitr                            	PASS
2024/03/19 01:01:08 mysql57.web_test                        	FAIL (1 tries)
2024/03/19 01:01:08 ============================================================
2024/03/19 01:01:08 2 PASSED, 0 FLAKY, 1 FAILED, 0 SKIPPED
2024/03/19 01:01:08 Total time: 12m54.2s
exit status 1
Error: Process completed with exit code 1.

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
@timvaillancourt timvaillancourt merged commit 1533744 into slack-vitess-r15.0.5 Mar 26, 2024
195 checks passed
@timvaillancourt timvaillancourt deleted the go1.21-slack-vitess-r15.0.5 branch March 26, 2024 17:40
timvaillancourt added a commit that referenced this pull request May 3, 2024
timvaillancourt added a commit that referenced this pull request May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants