Skip to content

fix(ci): allow SLA tracker to label pull requests - #2066

Merged
ritaneves merged 1 commit into
mainfrom
rfneves/fix-pr-sla-permissions
Jul 17, 2026
Merged

fix(ci): allow SLA tracker to label pull requests#2066
ritaneves merged 1 commit into
mainfrom
rfneves/fix-pr-sla-permissions

Conversation

@ritaneves

Copy link
Copy Markdown
Contributor

Summary

  • Grant the PR SLA tracker pull-requests: write.
  • Update the workflow permission regression test.

Root cause

The first tracker run received 403 Resource not accessible by integration while adding an SLA label to PR #1788. The workflow granted only pull-requests: read, so it exited before creating or updating the tracker issue.

Impact

The workflow can now reconcile SLA labels on pull requests and continue to create or update [Tracker] Pull request handoff SLA.

Validation

  • .venv/bin/pytest -q tests/unit_tests/test_pr_sla_tracker.py
  • 24 passed

Failed run: https://github.com/NVIDIA-NeMo/Gym/actions/runs/29570272742/job/87852135838

@copy-pr-bot

copy-pr-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ritaneves
ritaneves marked this pull request as ready for review July 17, 2026 09:37
@ritaneves
ritaneves requested a review from a team as a code owner July 17, 2026 09:37
Signed-off-by: Rita Fernandes Neves <rfernandesne@nvidia.com>
@ritaneves
ritaneves force-pushed the rfneves/fix-pr-sla-permissions branch from 8112402 to ac4d69b Compare July 17, 2026 09:48
@ritaneves
ritaneves merged commit 938f967 into main Jul 17, 2026
29 checks passed
@ritaneves
ritaneves deleted the rfneves/fix-pr-sla-permissions branch July 17, 2026 09:55
OlegSudakov pushed a commit to OlegSudakov/Gym that referenced this pull request Aug 7, 2026
## Summary

- Grant the PR SLA tracker pull-requests: write.
- Update the workflow permission regression test.

## Root cause

The first tracker run received 403 Resource not accessible by
integration while adding an SLA label to PR NVIDIA-NeMo#1788. The workflow granted
only pull-requests: read, so it exited before creating or updating the
tracker issue.

## Impact

The workflow can now reconcile SLA labels on pull requests and continue
to create or update [Tracker] Pull request handoff SLA.

## Validation

- .venv/bin/pytest -q tests/unit_tests/test_pr_sla_tracker.py
- 24 passed

Failed run:
https://github.com/NVIDIA-NeMo/Gym/actions/runs/29570272742/job/87852135838

Signed-off-by: Rita Fernandes Neves <rfernandesne@nvidia.com>
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.

1 participant