Skip to content

ci(release): allow entire SDK team to trigger workflows [SDK-4972]#3055

Merged
vgupta98 merged 2 commits into
developfrom
chore/sdk-4972-allow-sdk-team-to-trigger-release-workflows
Jun 9, 2026
Merged

ci(release): allow entire SDK team to trigger workflows [SDK-4972]#3055
vgupta98 merged 2 commits into
developfrom
chore/sdk-4972-allow-sdk-team-to-trigger-release-workflows

Conversation

@vgupta98

@vgupta98 vgupta98 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

PR Description

The manual-dispatch (workflow_dispatch) release workflows gate the triggering user through validate-actor.yml, which checks GitHub org team membership. They were configured to allow only the js-sdk team (3 members), so other SDK-team members who own a release could not trigger them — during the v3.111.0 release (SDK-4971) the release owner was blocked from running Draft a new release and had to ask a js-sdk member to trigger it.

This change replaces the js-sdk team with the broader sdk_team (the entire SDK team, a superset of js-sdk) in the actor allow-list, so any SDK-team member can trigger the release-related workflows. The integrations team continues to be allowed.

Updated workflows:

  • draft-new-release.ymlteam_names: 'sdk_team,integrations'
  • create-hotfix-branch.ymlteam_names: 'sdk_team,integrations'
  • deploy-beta.ymlteam_names: 'sdk_team,integrations'
  • rollback.ymlteam_names: 'sdk_team,integrations'
  • validate-actor.yml — default team_names updated to 'sdk_team'

No change to the validation logic itself, the branch restrictions, or any release behaviour — only the set of teams permitted to trigger.

Note: sdk_team is a superset of js-sdk, so no existing trigger access is removed. For a previously-excluded member to actually trigger a workflow they must be a member of the sdk_team GitHub org team (an org-level membership concern, outside this PR).

Linear task (optional)

SDK-4972

Cross Browser Tests

Please confirm you have tested for the following browsers:

  • Chrome
  • Firefox
  • IE11

N/A — CI/workflow configuration change only, no SDK runtime code affected.

Sanity Suite

  • All sanity suite test cases pass locally

N/A — no SDK code changes.

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

The actor validation mechanism is unchanged; access is broadened from one SDK sub-team to the parent SDK team only. Branch/tag restrictions on each workflow remain intact.

Summary by CodeRabbit

  • Chores
    • Updated CI workflow team configuration: replaced an internal team identifier with a new one and clarified that team names should use GitHub team slugs (URL slug, not display name).
    • Applied these updates across deployment, release drafting, rollback, beta, and validation workflows.

- Replace 'js-sdk' GitHub team with the broader 'sdk_team' in the
  validate-actor gate so all SDK-team members (not just js-sdk) can
  manually trigger release-related workflows
- Update draft-new-release, create-hotfix-branch, deploy-beta and
  rollback to gate on 'sdk_team,integrations'
- Update the default team_names in validate-actor to 'sdk_team'

Ref: SDK-4972
Copilot AI review requested due to automatic review settings June 9, 2026 12:27
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 14dd3ec9-2fa8-4054-bef1-14faee7a8576

📥 Commits

Reviewing files that changed from the base of the PR and between 729f160 and 725585f.

📒 Files selected for processing (1)
  • .github/workflows/validate-actor.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/validate-actor.yml

📝 Walkthrough

Walkthrough

This PR updates GitHub workflow actor validation to use the sdk_team slug instead of js-sdk. The reusable validate-actor.yml default and description are changed, and four workflows that call it now pass sdk_team,integrations.

Changes

Team name migration from js-sdk to sdk_team

Layer / File(s) Summary
Reusable workflow team name default
.github/workflows/validate-actor.yml
The validate-actor.yml reusable workflow input default and description for team_names are updated to use the sdk_team slug.
Caller workflows team name updates
.github/workflows/create-hotfix-branch.yml, .github/workflows/deploy-beta.yml, .github/workflows/draft-new-release.yml, .github/workflows/rollback.yml
Each workflow that invokes validate-actor now passes sdk_team,integrations instead of js-sdk,integrations.

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: broadening release workflow triggering access from the js-sdk team to the entire SDK team.
Description check ✅ Passed The description is comprehensive and well-structured, covering the problem, solution, affected workflows, security implications, and notes. It addresses the PR description template with context-appropriate explanations for N/A items.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the release/deploy workflow actor validation to allow a broader SDK team to trigger release-related GitHub Actions workflows.

Changes:

  • Changed the default allowed team in validate-actor.yml from js-sdk to sdk_team.
  • Updated multiple release/deploy workflows to pass team_names: 'sdk_team,integrations' when calling the actor validation workflow.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/validate-actor.yml Updates the default allowed team for the reusable actor-validation workflow.
.github/workflows/rollback.yml Updates allowed team list used to authorize rollback workflow triggers.
.github/workflows/draft-new-release.yml Updates allowed team list used to authorize drafting new releases.
.github/workflows/deploy-beta.yml Updates allowed team list used to authorize beta deploy triggers.
.github/workflows/create-hotfix-branch.yml Updates allowed team list used to authorize hotfix branch creation.

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

Comment thread .github/workflows/validate-actor.yml
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jun 9, 2026
@vgupta98 vgupta98 self-assigned this Jun 9, 2026
The value is used as the team_slug in the GitHub API path
(/orgs/{org}/teams/{slug}/memberships/...), so a display name
would always fail the membership check. Clarify the input
description to avoid that pitfall.

Ref: SDK-4972
@vgupta98 vgupta98 marked this pull request as ready for review June 9, 2026 12:42
@vgupta98 vgupta98 requested a review from a team as a code owner June 9, 2026 12:42
Copilot AI review requested due to automatic review settings June 9, 2026 12:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@vgupta98 vgupta98 merged commit a09c36e into develop Jun 9, 2026
17 checks passed
@vgupta98 vgupta98 deleted the chore/sdk-4972-allow-sdk-team-to-trigger-release-workflows branch June 9, 2026 12:51
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