Skip to content

fix(proxy): create IAM role for helix-deploy-proxy if necessary - #935

Merged
dominique-pfister merged 3 commits into
mainfrom
fix/create-proxy-role
Jul 20, 2026
Merged

fix(proxy): create IAM role for helix-deploy-proxy if necessary#935
dominique-pfister merged 3 commits into
mainfrom
fix/create-proxy-role

Conversation

@dominique-pfister

Copy link
Copy Markdown
Contributor

No description provided.

Restricts the CloudWatch Logs permissions on the helix-deploy-proxy
role to its own log group instead of granting access to all log
groups in the account.
@github-actions

Copy link
Copy Markdown

This PR will trigger a patch release when merged.

Code: { ZipFile: zipFile },
}));
FunctionArn = res.FunctionArn;
functionArn = await this.createFunctionWithRetry(roleArn, zipFile);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is not immediately obvious: creating a role takes some time before it can be used, which is why we retry creating the function with that new role

@dominique-pfister
dominique-pfister merged commit 3c007fa into main Jul 20, 2026
5 checks passed
@dominique-pfister
dominique-pfister deleted the fix/create-proxy-role branch July 20, 2026 06:17
adobe-bot pushed a commit that referenced this pull request Jul 20, 2026
## [14.1.1](v14.1.0...v14.1.1) (2026-07-20)

### Bug Fixes

* **proxy:** create IAM role for `helix-deploy-proxy` if necessary ([#935](#935)) ([3c007fa](3c007fa))
@adobe-bot

Copy link
Copy Markdown

🎉 This PR is included in version 14.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants