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

(v9) Bump Go to 1.17.9 #11932

Merged
merged 5 commits into from
Apr 14, 2022
Merged

(v9) Bump Go to 1.17.9 #11932

merged 5 commits into from
Apr 14, 2022

Conversation

r0mant
Copy link
Collaborator

@r0mant r0mant commented Apr 13, 2022

No description provided.

@@ -1,6 +1,6 @@
# The base image (buildbox:latest) is built by running `make -C build.assets`
# from the base repo directory $GOPATH/gravitational.com/teleport
FROM quay.io/gravitational/teleport-buildbox:go1.17.2
FROM quay.io/gravitational/teleport-buildbox:teleport9
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@tcsc I changed it to use the new buildbox tag here, is this ok?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure why it wasn't that already. Reasonably sure I put a PR in for it when I changed the rest.

Copy link
Contributor

@tcsc tcsc left a comment

Choose a reason for hiding this comment

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

Actually, the appropriate wat to change drone.yaml is to update dronegen

@r0mant
Copy link
Collaborator Author

r0mant commented Apr 13, 2022

Actually, the appropriate wat to change drone.yaml is to update dronegen

@tcsc I did use dronegen. I'll double-check again before merging just in case.

Copy link
Contributor

@espadolini espadolini left a comment

Choose a reason for hiding this comment

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

Reminder to update the e ref once gravitational/teleport.e#428 lands

@r0mant r0mant merged commit cd59194 into branch/v9 Apr 14, 2022
@r0mant r0mant deleted the roman/v9/go1179 branch April 14, 2022 23:36
@webvictim webvictim mentioned this pull request Apr 19, 2022
@webvictim webvictim mentioned this pull request Jun 8, 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.

None yet

4 participants