Skip to content

T9164: accept NOS-/VD- Jira keys in task-id check; pin NOS in coderabbit - #1260

Merged
andamasov merged 1 commit into
rollingfrom
nos-task-id
Aug 5, 2026
Merged

T9164: accept NOS-/VD- Jira keys in task-id check; pin NOS in coderabbit#1260
andamasov merged 1 commit into
rollingfrom
nos-task-id

Conversation

@andamasov

Copy link
Copy Markdown
Member

Fleet replication of the adversarially-reviewed canary vyos/vyos-1x#5379 for the VD→NOS Jira project rename (T9164). .github/mergify.yml — the invalid-task-id rule's two regexes and its description now accept T<digits>:, NOS-<digits>: and legacy VD-<digits>: keys; the rule name is byte-identical, so this stays additive rather than an extends: override. .coderabbit.yamlknowledge_base.jira.project_keys pins NOS ahead of the retained legacy VD. Byte-identical template application — skip-adversarial per the documented escape hatch (2 findings were found and fixed on the canary; regexes ship tightened: ASCII digits + literal-space separators).

Phase 0 (local CodeRabbit CLI, --base origin/rolling): clean — 0 findings.

Advances: IS-640

🤖 Generated by robots

VD tracker project renamed to NOS (2026-07) - PR titles and commit headlines
using the new NOS- key must stop being flagged invalid-task-id; legacy VD-
keys stay accepted, and CodeRabbit now resolves NOS alongside VD. Fleet
replication of canary vyos/vyos-1x#5379, whose adversarial review tightened
the regexes to ASCII [0-9] + literal-space separators.

🤖 Generated by [robots](https://vyos.io)
@andamasov andamasov added the skip-adversarial adversarial gate escape hatch label Aug 4, 2026
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 7c272b31-64e6-4502-b67c-6035c335b1fd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@mergify mergify Bot added the rolling label Aug 4, 2026
@mergify

mergify Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 📃 Configuration Change Requirements

Mergify configuration change

  • check-success = Configuration changed

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

CI integration 👍 passed!

Details

CI logs

  • Config tests 👍 passed

@andamasov
andamasov marked this pull request as ready for review August 5, 2026 09:29

@vyos-bot vyos-bot Bot 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.

Approved by vyos-bot on operator instruction (bot-approve flow, IS-619) — NOS fleet sweep batch (T9164).

@andamasov
andamasov merged commit b27468d into rolling Aug 5, 2026
16 checks passed
@andamasov
andamasov deleted the nos-task-id branch August 5, 2026 09:29
@vyos-bot vyos-bot Bot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant