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)

@florentinl florentinl marked this pull request as ready for review December 1, 2025 15:48
@florentinl florentinl requested review from a team as code owners December 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
Contributor Author

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot 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.

@florentinl florentinl merged commit c678c09 into main Dec 1, 2025
93 of 94 checks passed
@florentinl florentinl deleted the florentin.labelle/fix-gcc-version-error branch December 1, 2025 16:54
Sign up for free to 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