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

Lower Go version to 1.20 for plugin #23

Merged
merged 3 commits into from
Jun 5, 2024
Merged

Conversation

jsing-canva
Copy link

The build system for the kinesis plugin is rather complex - in short, we
pull this code into an AWS AL2 based container where Go is installed,
compile the plugin and then copy it out for inclusion in our own image.
This means that the minimum Go version specified in this code must be
no higher than the Go version available in the AWS AL2 image (which is
currently Go 1.20). Unfortunately this version is no longer receiving
reliability or security updates, but we'll have to deal with this issue
in the future.

The build system for the kinesis plugin is rather complex - in short, we
pull this code into an AWS AL2 based container where Go is installed,
compile the plugin and then copy it out for inclusion in our own image.
This means that the minimum Go version specified in this code must be
no higher than the Go version available in the AWS AL2 image (which is
currently Go 1.20). Unfortunately this version is no longer receiving
reliability or security updates, but we'll have to deal with this issue
in the future.
Copy link

@chengchengmu chengchengmu left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@jsing-canva jsing-canva merged commit 09458d2 into mainline Jun 5, 2024
1 check passed
@jsing-canva jsing-canva deleted the jsing-lower-go-version branch June 5, 2024 00:46
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.

2 participants