Skip to content

[code sync] Merge code from sonic-net/sonic-utilities:202605 to 202608#407

Merged
mssonicbld merged 2 commits into
Azure:202608from
mssonicbld:sonicbld/202608-merge
Jul 15, 2026
Merged

[code sync] Merge code from sonic-net/sonic-utilities:202605 to 202608#407
mssonicbld merged 2 commits into
Azure:202608from
mssonicbld:sonicbld/202608-merge

Conversation

@mssonicbld

Copy link
Copy Markdown
Collaborator
* b03f9bab - (origin/202605) [ci]sonic-ubuntu-1c agent pool: migrate pre-commit from pip to uv to address S360 vulnerability (#4689) (2026-07-15) [mssonicbld]<br>```

mssonicbld and others added 2 commits July 15, 2026 11:13
…address S360 vulnerability (#4689)

### Description of PR

Summary: Migrate the pre-commit static-analysis step off python3-pip to uv.

The CI agent pool **sonic-ubuntu-1c** (VMSS sonic-ubuntu-2ca, Ubuntu 22.04) is flagged by S360 for pip vulnerabilities **USN-8344-1** and **USN-8344-3** (`python3-pip`). The agent is being redeployed without python3-pip. `pre-commit-check.yml` runs directly on that host (no container), so `sudo pip install pre-commit` would break. uv is already installed on the agent and is a drop-in replacement.

### Type of change
- [x] Refactor / cleanup

### Approach
#### What is the motivation for this PR?
Remove python3-pip dependency from sonic-ubuntu-1c to clear S360 USN-8344-1/-3.
##### Work item tracking
- Microsoft ADO **(number only)**: 6035751, 6035790
#### How did you do it?
Replaced `sudo pip install pre-commit` with `sudo uv pip install --system pre-commit` in .azure-pipelines/pre-commit-check.yml (and the local-instructions text).
#### How did you verify/test it?
Static Analysis stage runs pre-commit successfully via uv.

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@mssonicbld mssonicbld merged commit baa7114 into Azure:202608 Jul 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant