Skip to content

Add issues write permission to untriaged label workflow#4827

Merged
gaiksaya merged 2 commits into
opensearch-project:mainfrom
shreyah963:fix/add-untriaged-permissions
May 21, 2026
Merged

Add issues write permission to untriaged label workflow#4827
gaiksaya merged 2 commits into
opensearch-project:mainfrom
shreyah963:fix/add-untriaged-permissions

Conversation

@shreyah963
Copy link
Copy Markdown
Contributor

Description

Add permissions: issues: write to the add-untriaged.yml workflow to fix 403 "Resource not accessible by integration" error when applying the 'untriaged' label to issues.

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: shreyah963 <shreyab963@gmail.com>
@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

Signed-off-by: shreyah963 <shreyab963@gmail.com>
@github-actions
Copy link
Copy Markdown

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 08ef30f.

PathLineSeverityDescription
.github/workflows/add-untriaged.yml12highGitHub Action dependency changed from the floating tag 'actions/github-script@v6' to a specific commit hash '3a2844b7e9c422d3c10d287c895573f7108da1b3'. Per mandatory review policy, all dependency/package changes must be flagged. The commit hash cannot be independently verified here as belonging to the legitimate actions/github-script repository — maintainers should confirm this hash resolves to the expected upstream commit (e.g., via 'git log --oneline' on the official repo) before merging.

The table above displays the top 10 most important findings.

Total: 1 | Critical: 0 | High: 1 | Medium: 0 | Low: 0


Pull Requests Author(s): Please update your Pull Request according to the report above.

Repository Maintainer(s): You can bypass diff analyzer by adding label skip-diff-analyzer after reviewing the changes carefully, then re-run failed actions. To re-enable the analyzer, remove the label, then re-run all actions.


⚠️ Note: The Code-Diff-Analyzer helps protect against potentially harmful code patterns. Please ensure you have thoroughly reviewed the changes beforehand.

Thanks.

@gaiksaya gaiksaya merged commit 1d3eb9a into opensearch-project:main May 21, 2026
8 of 10 checks passed
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.

2 participants