Skip to content

[SVLS-8492] Fix permission error for durable forwarder publish job#337

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
masterfrom
yiming.luo/fix-durable-forwarder-release-validate-permission
Jul 10, 2026
Merged

[SVLS-8492] Fix permission error for durable forwarder publish job#337
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
masterfrom
yiming.luo/fix-durable-forwarder-release-validate-permission

Conversation

@lym953

@lym953 lym953 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Context

After merging #333, running the manual publish durable function event forwarder job failed:

An error occurred (AccessDenied) when calling the ValidateTemplate operation: User:
arn:aws:sts::486234852809:assumed-role/dd.gitlabRunner_cloudformationTemplate/dd-cloudiam-1783632319
is not authorized to perform: cloudformation:ValidateTemplate because no identity-based
policy allows the cloudformation:ValidateTemplate action

Job: https://gitlab.ddbuild.io/DataDog/cloudformation-template/-/jobs/1847411821

Fix

Drop the aws cloudformation validate-template call from release.sh. The GitLab
runner role for this project isn't granted cloudformation:ValidateTemplate (that's
an IAM change outside this repo), and this call isn't actually needed:

  • None of the other release.sh scripts in this repo (aws, aws_organizations,
    aws_quickstart, aws_llm, etc.) call the CloudFormation API to validate.
  • Template correctness is already checked by cfn-lint on every push
    (.github/workflows/lint.yml).

Test plan

  • Merge and re-run the publish durable function event forwarder manual job on
    the resulting master pipeline; confirm it uploads v0.1.0.yaml and latest.yaml
    to s3://datadog-cloudformation-template/aws/lambda-durable-function-event-forwarder/.

…ase.sh

The GitLab runner role for this project has no cloudformation:ValidateTemplate
permission, so the publish job failed with AccessDenied. None of the other
release.sh scripts in this repo call the CloudFormation API to validate; template
correctness is already covered by cfn-lint on every push
(.github/workflows/lint.yml).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@lym953 lym953 changed the title Fix durable forwarder publish job: drop cloudformation:ValidateTemplate call [SVLS-8492] Fix permission error for durable forwarder publish job Jul 9, 2026
@lym953 lym953 marked this pull request as ready for review July 9, 2026 23:41
@lym953 lym953 requested a review from a team as a code owner July 9, 2026 23:41
@lym953 lym953 requested review from Copilot and shreyamalpani July 9, 2026 23:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes an AWS CloudFormation API validation step from the durable forwarder release script to prevent the GitLab release job from failing due to missing cloudformation:ValidateTemplate permissions.

Changes:

  • Removed aws cloudformation validate-template from aws_durable_function_event_forwarder/release.sh to avoid AccessDenied during the manual publish job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gh-worker-dd-mergequeue-cf854d gh-worker-dd-mergequeue-cf854d Bot merged commit c0b91f2 into master Jul 10, 2026
7 checks passed
@gh-worker-dd-mergequeue-cf854d gh-worker-dd-mergequeue-cf854d Bot deleted the yiming.luo/fix-durable-forwarder-release-validate-permission branch July 10, 2026 15:37
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