Skip to content

fix(ci): Do not copy extended attributes for darwin build - #2282

Merged
jefchien merged 2 commits into
mainfrom
fix-darwin-cp
Sep 9, 2026
Merged

jefchien merged 2 commits into
mainfrom
fix-darwin-cp

Conversation

@jefchien

@jefchien jefchien commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Description of the issue

The MakeMacPkg job in the Build Test Artifacts workflow has started failing at the "Copy binary" step:
https://github.com/aws/amazon-cloudwatch-agent/actions/runs/33896579998/job/101133623806

cp: build/bin/darwin/amd64/.: unable to copy extended attributes to /tmp/.: Operation not permitted

On macOS, cp attempts to replicate the source directory's extended attributes onto the target directory (/tmp). The only difference between the previously passing run and this failure is that the GitHub macOS runner image was updated from 20260727.0256 to 20260829.0321 and now seems to deny the copy, which breaks the darwin package build.

Description of changes

Added the -X flag to the cp commands that copy the build output and pkg dependencies into /tmp.

$ man cp
...
     -X    Do not copy Extended Attributes (EAs) or resource forks.

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

Ran the workflow against the branch
https://github.com/aws/amazon-cloudwatch-agent/actions/runs/34255505664/job/102166185058

Requirements

Before commiting your code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make lint

Integration Tests

To run integration tests against this PR, add the ready for testing label.

@jefchien
jefchien requested a review from a team as a code owner September 8, 2026 18:17
@jefchien
jefchien merged commit 2771e19 into main Sep 9, 2026
26 checks passed
@jefchien
jefchien deleted the fix-darwin-cp branch September 9, 2026 15:14
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.

3 participants