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

[tasks] stop the build if GO111MODULE is set to on. #4995

Merged
merged 9 commits into from
Mar 10, 2020

Conversation

remeh
Copy link
Contributor

@remeh remeh commented Feb 26, 2020

What does this PR do?

Add a step in all builds task to stop the build if GO111MODULE is set to on.

Motivation

The datadog-agent is not using the Go modules yet. The default behavior of the Go modules while compiling the Agent in a GOPATH is fine (it's not using the Go modules because we're in a GOPATH and there is no go.mod file available), however, if GO111MODULE has been forced to on in the user environment, the build would fail.

@remeh remeh added [deprecated] team/agent-core Deprecated. Use metrics-logs / shared-components labels instead.. changelog/no-changelog labels Feb 26, 2020
@remeh remeh added this to the 7.19.0 milestone Feb 26, 2020
@remeh remeh requested review from a team as code owners February 26, 2020 11:11
@remeh remeh force-pushed the remeh/invoke-go111module branch from ac2e968 to c266667 Compare February 26, 2020 11:12
Copy link
Member

@olivielpeau olivielpeau left a comment

Choose a reason for hiding this comment

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

LGTM overal!

Let's add this check to inv deps as well? + I made a few inline nits

tasks/utils.py Outdated Show resolved Hide resolved
tasks/utils.py Outdated Show resolved Hide resolved
tasks/utils.py Outdated Show resolved Hide resolved
Copy link
Contributor

@albertvaka albertvaka left a comment

Choose a reason for hiding this comment

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

Small nit but apart from that LGTM.

tasks/utils.py Outdated Show resolved Hide resolved
Copy link
Member

@olivielpeau olivielpeau left a comment

Choose a reason for hiding this comment

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

+1 on Albert's comment + a few nits

tasks/utils.py Outdated Show resolved Hide resolved
tasks/utils.py Outdated Show resolved Hide resolved
tasks/agent.py Outdated Show resolved Hide resolved
remeh and others added 4 commits March 9, 2020 11:55
Co-Authored-By: Olivier Vielpeau <olivielpeau@users.noreply.github.com>
Co-Authored-By: Olivier Vielpeau <olivielpeau@users.noreply.github.com>
Copy link
Member

@olivielpeau olivielpeau left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@remeh remeh merged commit 6804080 into master Mar 10, 2020
@remeh remeh deleted the remeh/invoke-go111module branch March 10, 2020 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog [deprecated] team/agent-core Deprecated. Use metrics-logs / shared-components labels instead..
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants