Skip to content

enable write for id-token on permissions#71

Merged
angeloskyratzakos merged 9 commits intomainfrom
CLD-9264-Move-Plugin-ci-to-role
Jun 11, 2025
Merged

enable write for id-token on permissions#71
angeloskyratzakos merged 9 commits intomainfrom
CLD-9264-Move-Plugin-ci-to-role

Conversation

@angeloskyratzakos
Copy link
Copy Markdown
Contributor

Summary

enable write for id-token on permissions

Ticket Link

Modified the AWS configuration in both plugin-cd.yml and plugin-ci.yml to use a specific region (us-east-1) and assume a role for credentials, enabling role chaining. This change enhances security and aligns with best practices for AWS credential management.
Updated the AWS region configuration in both plugin-cd.yml and plugin-ci.yml to use a dynamic input variable instead of a hardcoded value. This change improves flexibility and allows for easier adjustments to the AWS region in future deployments.
@angeloskyratzakos angeloskyratzakos requested a review from stafot June 11, 2025 11:26
@angeloskyratzakos angeloskyratzakos merged commit cf22e0f into main Jun 11, 2025
@angeloskyratzakos angeloskyratzakos deleted the CLD-9264-Move-Plugin-ci-to-role branch June 11, 2025 11:30
@hanzei
Copy link
Copy Markdown
Contributor

hanzei commented Jun 11, 2025

The commit seems to break the existing plugins pipeline. The starter template is failing with: https://github.com/mattermost/mattermost-plugin-starter-template/actions/runs/15584352568/workflow

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants