Remove apt.llvm.org dependency in verify job; use distro clang-format#3126
Remove apt.llvm.org dependency in verify job; use distro clang-format#3126sasmita016 wants to merge 2 commits intokubernetes-sigs:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sasmita016 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @sasmita016! |
|
Hi @sasmita016. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3126 +/- ##
===========================================
- Coverage 45.50% 24.64% -20.86%
===========================================
Files 79 126 +47
Lines 7782 17992 +10210
===========================================
+ Hits 3541 4435 +894
- Misses 4099 13265 +9166
- Partials 142 292 +150 🚀 New features to boost your workflow:
|
|
The build / bpf error is unrelated. |
|
/release-note-none |
|
@sasmita016: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
pull-security-profiles-operator-verify fails. |
|
I guess this is needed, feel free to reopen if I'm missing anything. |
Remove apt.llvm.org dependency in verify job; use distro clang-format
The verify script installs LLVM/Clang 21 via apt.llvm.org, which is currently failing on Debian Trixie due to apt signature (SHA1) restrictions.
Since the script only requires
clang-formatbefore runningmake verify, replace the external LLVM install with the distro-providedclang-format.This removes the failing dependency while preserving the intended behavior.
Fixes: kubernetes/kubernetes#137097