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

[ASCII-1786][Deb ARM64] Update GCC from 5.4 to 9 to support Go 1.22 #719

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

chouetz
Copy link
Member

@chouetz chouetz commented Nov 15, 2024

The tests_deb-arm64-py3 is currently failing on the 6.53.x branch of datadog-agent with:

/go/pkg/mod/golang.org/[email protected]/pkg/tool/linux_arm64/link: running gcc failed: exit status 1
/tmp/go-link-279306332/000025.o: In function `__sanitizer::DecreaseTotalMmap(unsigned long)':
gotsan.cpp:(.text+0x12fc): undefined reference to `__aarch64_ldadd8_acq_rel'
/tmp/go-link-279306332/000025.o: In function `__sanitizer_acquire_crash_state':

The job is successful with a recent image. I cherry-pick this commit as it's most probably what is missing in the current 6.53.x image

@chouetz chouetz requested review from a team as code owners November 15, 2024 17:06
Comment on lines +50 to +54
RUN apt install -y software-properties-common \
&& add-apt-repository -y ppa:ubuntu-toolchain-r/test \
&& apt-get update \
&& apt-get install -y gcc-9 g++-9 \
&& update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-9 0

Choose a reason for hiding this comment

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

Code Quality Violation

package g++-9 should have version pinned (...read more)

When using apt-get install, pin the version to avoid unwanted upgrades and undefined behavior.

View in Datadog  Leave us feedback  Documentation

Copy link
Contributor

@amenasria amenasria left a comment

Choose a reason for hiding this comment

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

❓ question: ‏Shouldn't we pin gcc-9 and g++-9? Other than that LGTM !

@chouetz chouetz merged commit da48ad7 into 6.53.x Nov 15, 2024
21 of 23 checks passed
@chouetz chouetz deleted the nschweitzer/debarm branch November 15, 2024 19:37
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