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

Disable CGO usage in builds #486

Merged
merged 2 commits into from
Oct 9, 2023
Merged

Disable CGO usage in builds #486

merged 2 commits into from
Oct 9, 2023

Conversation

mattlord
Copy link
Collaborator

@mattlord mattlord commented Oct 9, 2023

Vitess has not used CGO since v15 where our usage of the native sqlite C library was replaced: vitessio/vitess#12214

We should not build the operator using CGO as it will not run on platforms that have an older glibc version than the one used on the golang container image used for the build.

This also moves the backup and restore test to using the default/mysql80 images in test/endtoend/operator/101_initial_cluster_backup.yaml. This was the only test still using the mysql57 images and this moves everything to 8.0 (MySQL 5.7 is EOL next month).

@mattlord mattlord added Backport to: release-2.8 Adding this label to a Pull Request will automatically create a backport onto the release-2.8 branch Backport to: release-2.9 Adding this label to a Pull Request will automatically create a backport onto the release-2.9 branc labels Oct 9, 2023
@mattlord mattlord removed Backport to: release-2.8 Adding this label to a Pull Request will automatically create a backport onto the release-2.8 branch Backport to: release-2.9 Adding this label to a Pull Request will automatically create a backport onto the release-2.9 branc labels Oct 9, 2023
@mattlord mattlord merged commit b1640f8 into release-2.10 Oct 9, 2023
4 checks passed
@mattlord mattlord deleted the disable_cgo branch October 9, 2023 18:48
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.

3 participants