CrunchyData / crunchy-containers

Containers for Managing PostgreSQL on Kubernetes by Crunchy Data
https://www.crunchydata.com/
Apache License 2.0
1.01k stars 328 forks source link

Updates for go1.17 #1395

Closed jmckulk closed 2 years ago

jmckulk commented 2 years ago

Checklist:

Type of Changes:

What is the current behavior? (link to any open issues here)

What is the new behavior (if this is a feature change)? Go binaries are built using go1.17 (excluding pgBadger that uses a golang:1.15.5 docker image)

Other information: [sc-12565]

jmckulk commented 2 years ago

LGTM. (Separate issue: do we need an "Upgrade" option in our PR template for things like this?)

Oh yeah good idea. Maybe something like

andrewlecuyer commented 2 years ago

@benjaminjb @jmckulk lets be sure to bring up the template during our retrospective for the current sprint.

Seems like a good opportunity to review these templates in general to see what makes sense per our current processes.