Skip to content

BUILD-11517 Update Jira Automation#111

Merged
julien-carsique-sonarsource merged 2 commits into
masterfrom
Pavel/Automation
Jun 19, 2026
Merged

BUILD-11517 Update Jira Automation#111
julien-carsique-sonarsource merged 2 commits into
masterfrom
Pavel/Automation

Conversation

@pavel-mikula-sonarsource

@pavel-mikula-sonarsource pavel-mikula-sonarsource commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

These files should be similar to https://github.com/SonarSource/re-terraform-aws-vault/tree/master/.github/workflows, team-review-component value needs to be validated for each PR

The failed precommit looks like a false positive, sonar-xs exists.

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Jun 17, 2026

Copy link
Copy Markdown

BUILD-11517

Comment thread .github/workflows/PullRequestClosed.yml
Comment thread .github/workflows/PullRequestClosed.yml
@pavel-mikula-sonarsource
pavel-mikula-sonarsource marked this pull request as ready for review June 17, 2026 11:26
@pavel-mikula-sonarsource
pavel-mikula-sonarsource requested a review from a team as a code owner June 17, 2026 11:26
Copilot AI review requested due to automatic review settings June 17, 2026 11:26

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

This PR updates the repository’s GitHub Actions workflows used for Jira/PR review automation to align with the referenced re-terraform-aws-vault workflow setup, including switching to the sonar-xs runner label and updating Vault integration.

Changes:

  • Switch workflow runners from github-ubuntu-latest-s to sonar-xs.
  • Bump SonarSource/vault-action-wrapper from 3.2.0 to 3.6.1 (pinned by commit SHA).
  • Add Rootly token wiring (Vault secret + action input) and set team-review-component for PR creation automation.

Reviewed changes

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

File Description
.github/workflows/SubmitReview.yml Uses sonar-xs runner and updates Vault action wrapper pin/version.
.github/workflows/RequestReview.yml Uses sonar-xs, updates Vault wrapper, and adds ROOTLY_TOKEN + rootly-token input.
.github/workflows/PullRequestCreated.yml Uses sonar-xs, updates Vault wrapper, adds Rootly token wiring, and sets team-review-component.
.github/workflows/PullRequestClosed.yml Uses sonar-xs runner and updates Vault action wrapper pin/version.

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

@julien-carsique-sonarsource

Copy link
Copy Markdown
Contributor

Gitar: add sonar-xs to actionlint

Co-authored-by: Pavel Mikula <57188685+pavel-mikula-sonarsource@users.noreply.github.com>
@gitar-bot

gitar-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown

Gitar: add sonar-xs to actionlint

Added sonar-xs to the self-hosted runner labels in .github/actionlint.yaml.

@sonarqubecloud

Copy link
Copy Markdown

@julien-carsique-sonarsource
julien-carsique-sonarsource enabled auto-merge (squash) June 19, 2026 13:33
@julien-carsique-sonarsource
julien-carsique-sonarsource merged commit 946be00 into master Jun 19, 2026
6 checks passed
@julien-carsique-sonarsource
julien-carsique-sonarsource deleted the Pavel/Automation branch June 19, 2026 13:35
@gitar-bot

gitar-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Synchronizes Jira automation workflows with standard repository patterns to ensure consistent validation of the team-review-component value. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

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.

4 participants