mirror of
https://github.com/meta-llama/llama-stack.git
synced 2025-12-03 09:53:45 +00:00
ci: add pre-commit check ensuring FIPS compliance (#3899)
# What does this PR do? this commit adds a new pre-commit hook to scan for non-FIPS compliant function usage within llama-stack Closes #3427 ## Test Plan Ran locally Signed-off-by: Nathan Weinberg <nweinber@redhat.com>
This commit is contained in:
parent
e809d21357
commit
b6bb8fbf64
1 changed files with 17 additions and 2 deletions
|
|
@ -152,7 +152,6 @@ repos:
|
||||||
files: ^src/llama_stack/ui/.*\.(ts|tsx)$
|
files: ^src/llama_stack/ui/.*\.(ts|tsx)$
|
||||||
pass_filenames: false
|
pass_filenames: false
|
||||||
require_serial: true
|
require_serial: true
|
||||||
|
|
||||||
- id: check-log-usage
|
- id: check-log-usage
|
||||||
name: Ensure 'llama_stack.log' usage for logging
|
name: Ensure 'llama_stack.log' usage for logging
|
||||||
entry: bash
|
entry: bash
|
||||||
|
|
@ -171,7 +170,23 @@ repos:
|
||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
exit 0
|
exit 0
|
||||||
|
- id: fips-compliance
|
||||||
|
name: Ensure llama-stack remains FIPS compliant
|
||||||
|
entry: bash
|
||||||
|
language: system
|
||||||
|
types: [python]
|
||||||
|
pass_filenames: true
|
||||||
|
exclude: '^tests/.*$' # Exclude test dir as some safety tests used MD5
|
||||||
|
args:
|
||||||
|
- -c
|
||||||
|
- |
|
||||||
|
grep -EnH '^[^#]*\b(md5|sha1|uuid3|uuid5)\b' "$@" && {
|
||||||
|
echo;
|
||||||
|
echo "❌ Do not use any of the following functions: hashlib.md5, hashlib.sha1, uuid.uuid3, uuid.uuid5"
|
||||||
|
echo " These functions are not FIPS-compliant"
|
||||||
|
echo;
|
||||||
|
exit 1;
|
||||||
|
} || true
|
||||||
ci:
|
ci:
|
||||||
autofix_commit_msg: 🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
|
autofix_commit_msg: 🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
|
||||||
autoupdate_commit_msg: ⬆ [pre-commit.ci] pre-commit autoupdate
|
autoupdate_commit_msg: ⬆ [pre-commit.ci] pre-commit autoupdate
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue