Skip to content

Revert "Update Go version to 1.20.5 (#408)" #409

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

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

Kaderinho
Copy link
Contributor

Go upgrade is causing issue on the Agent side (go tests are taking way longer) and the version mismatch between this repo and the Agent repo is blocking 7.47 preparation

@Kaderinho Kaderinho requested review from a team as code owners June 27, 2023 14:10
@KSerrania
Copy link
Collaborator

From what I'm seeing on main of datadog-agent (here), the go 1.19.10 upgrade (from #402) was only done in 7.46.x of datadog-agent, and not in main (DataDog/datadog-agent#17619), therefore even with this revert, datadog-agent-buildimages and datadog-agent are in incompatible states (which prevents using new images in datadog-agent). I see two options:

@Kaderinho
Copy link
Contributor Author

I think 1.19.10 is desirable in any case so my guess is that we should port DataDog/datadog-agent#17619 to main imho

@KSerrania
Copy link
Collaborator

KSerrania commented Jun 27, 2023

I think 1.19.10 is desirable in any case so my guess is that we should port DataDog/datadog-agent#17619 to main imho

👍 In that case, let's merge this PR, and update main of datadog-agent with the image from this PR's merge + the port of DataDog/datadog-agent#17619. Let's do this ASAP to unblock other teams that need to do their own changes to the buildimages.

@Kaderinho Kaderinho merged commit 3f177db into main Jun 28, 2023
@Kaderinho Kaderinho deleted the nicolas.guerguadj/revert-go-1.20 branch June 28, 2023 09:30
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