Skip to content

ci(rest): re-enable main publish after tag scheme fix#1924

Open
lachen-nv wants to merge 4 commits into
NVIDIA:mainfrom
lachen-nv:chore/rest-reenable-main-publish
Open

ci(rest): re-enable main publish after tag scheme fix#1924
lachen-nv wants to merge 4 commits into
NVIDIA:mainfrom
lachen-nv:chore/rest-reenable-main-publish

Conversation

@lachen-nv
Copy link
Copy Markdown
Contributor

Summary

Re-enables REST publish (docker + helm) on main / release / tags. Follow-up to #1922.

Net diff vs main

Just the tag scheme fix from #1922 (`rest-build-push-service.yml`). The disable toggles introduced in #1922 are reverted back to the original conditional gates.

Merge order

Either order is safe. Closing #1922 in favor of this PR is also fine.

Expected after merge

Main pushes will produce clean tags:
```
nvcr.io/0837451325059433/carbide-dev/nico-rest-api:0.11.0-pr-3-g72855901
nvcr.io/0837451325059433/carbide-dev/nico-rest-api:latest
```

And helm chart:
```
nico-rest:0.11.0-pr-3.g72855901
```

@lachen-nv lachen-nv requested a review from a team as a code owner May 25, 2026 05:41
@github-actions
Copy link
Copy Markdown

🔐 TruffleHog Secret Scan

No secrets or credentials found!

Your code has been scanned for 700+ types of secrets and credentials. All clear! 🎉

🔗 View scan details

🕐 Last updated: 2026-05-25 05:43:20 UTC | Commit: 07c0e4b

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 25, 2026

🔍 Container Scan Summary

Service Total Critical High Medium Low Other
nico-flow 66 4 34 18 2 8
nico-nsm 82 2 28 43 9 0
nico-psm 67 4 35 18 2 8
nico-rest-api 100 6 53 30 3 8
nico-rest-cert-manager 65 4 34 18 1 8
nico-rest-db 66 4 34 18 2 8
nico-rest-site-agent 65 4 34 18 1 8
nico-rest-site-manager 65 4 34 18 1 8
nico-rest-workflow 67 4 35 18 2 8
TOTAL 643 36 321 199 23 64

Per-CVE detail lives in the per-service grype-* artifacts (JSON + SARIF). Severity counts only — no CVE IDs published here.

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