Skip to content

ci: adjust runner and permissions for cloudbeds/composite-actions/argocd/trigger-update-app@v2#2

Merged
potyl merged 1 commit into
mainfrom
emo/fix-argocd-trigger-update-app-permissions
May 15, 2026
Merged

ci: adjust runner and permissions for cloudbeds/composite-actions/argocd/trigger-update-app@v2#2
potyl merged 1 commit into
mainfrom
emo/fix-argocd-trigger-update-app-permissions

Conversation

@potyl
Copy link
Copy Markdown
Contributor

@potyl potyl commented May 15, 2026

ci: adjust runner and permissions for cloudbeds/composite-actions/argocd/trigger-update-app@v2

The runner x1-core is good enough for this purpose. We also need to ensure
that we have the permissions:

permissions:
id-token: write
contents: read

Refs: PEN-194

@potyl potyl added the infra Infrastructure changes label May 15, 2026
@potyl potyl self-assigned this May 15, 2026
Copy link
Copy Markdown

@unblocked unblocked Bot left a comment

Choose a reason for hiding this comment

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

✅ No issues found

About Unblocked

Unblocked has been set up to automatically review your team's pull requests to identify genuine bugs and issues.

📖 Documentation — Learn more in our docs.

💬 Ask questions — Mention @unblocked to request a review or summary, or ask follow-up questions.

👍 Give feedback — React to comments with 👍 or 👎 to help us improve.

⚙️ Customize — Adjust settings in your preferences.

…ocd/trigger-update-app@v2

The runner x1-core is good enough for this purpose. We also need to ensure
that we have the permissions:

  permissions:
    id-token: write
    contents: read

Refs: PEN-194
@potyl potyl force-pushed the emo/fix-argocd-trigger-update-app-permissions branch from be45b4c to 5401a30 Compare May 15, 2026 15:21
@potyl potyl merged commit 34b78ac into main May 15, 2026
0 of 3 checks passed
@potyl potyl deleted the emo/fix-argocd-trigger-update-app-permissions branch May 15, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infrastructure changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants