Skip to content

Conversation

@florentinl
Copy link
Contributor

What does this PR do?

Fix the build_layer.yml github action by adding a missing symlink for the c++ compiler

Motivation

Failing pipelines on main for AL2 images: https://github.com/DataDog/datadog-lambda-python/actions/runs/19683478639

Testing Guidelines

Triggered a github action from this branch to verify it works correctly: https://github.com/DataDog/datadog-lambda-python/actions/runs/19827184647

Additional Notes

Types of Changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)

@florentinlflorentinl marked this pull request as ready for review December 1, 2025 15:48
@florentinlflorentinl requested review from a team as code ownersDecember 1, 2025 15:48
@cbeauchesne
Copy link
Collaborator

Could you add this job to pull requests ? it will prevent a futur PR to break it

@florentinl
Copy link
ContributorAuthor

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codexbot commented Dec 1, 2025

View all feedbacks in Devflow UI.

2025-12-01 16:39:19 UTC ℹ️ Start processing command /merge


2025-12-01 16:39:26 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2025-12-01 16:46:18 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 0s (p90).


2025-12-01 16:54:10 UTC ℹ️ MergeQueue: This merge request was already merged

This pull request was merged directly.

@florentinlflorentinl merged commit c678c09 into mainDec 1, 2025
93 of 94 checks passed
@florentinlflorentinl deleted the florentin.labelle/fix-gcc-version-error branch December 1, 2025 16:54
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@florentinl@cbeauchesne@lucaspimentel@taegyunkim