Skip to content

SGC: pass opts to permissions Command in writeFileDefinition#47950

Open
s-alad wants to merge 2 commits intomainfrom
saad/fix-secret-permissions-race
Open

SGC: pass opts to permissions Command in writeFileDefinition#47950
s-alad wants to merge 2 commits intomainfrom
saad/fix-secret-permissions-race

Conversation

@s-alad
Copy link
Member

@s-alad s-alad commented Mar 17, 2026

What does this PR do?

adds opts to the permissions Command, making it a HostAgent child so the restart is guaranteed to wait for chmod to finish

(now like other resources that inherit opts)

Motivation

Should help alleviate this failed agent ci test: https://datadoghq.atlassian.net/jira/software/c/projects/AGENTCFG/boards/8668?selectedIssue=AGENTCFG-660

Describe how you validated your changes

CI

Additional Notes

@s-alad s-alad marked this pull request as ready for review March 17, 2026 18:34
@s-alad s-alad requested a review from a team as a code owner March 17, 2026 18:34
@agent-platform-auto-pr
Copy link
Contributor

agent-platform-auto-pr bot commented Mar 17, 2026

Files inventory check summary

File checks results against ancestor 0b457eee:

Results for datadog-agent_7.78.0~devel.git.684.cc9ca99.pipeline.103286662-1_amd64.deb:

No change detected

@github-actions github-actions bot added the short review PR is simple enough to be reviewed quickly label Mar 17, 2026
@dd-octo-sts dd-octo-sts bot added internal Identify a non-fork PR team/agent-devx labels Mar 17, 2026
@s-alad s-alad added qa/done QA done before merge and regressions are covered by tests changelog/no-changelog No changelog entry needed and removed qa/done QA done before merge and regressions are covered by tests labels Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog No changelog entry needed internal Identify a non-fork PR short review PR is simple enough to be reviewed quickly team/agent-devx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants