Commit 6ffa565
committed
RHOAIENG-58789: trust bots, lgtm/approved, scope xKS triggers
author_association misreports opendatahub-io members with private org
visibility, so maintainers were having to re-add run-xks-e2e after every
commit even for people who are actually trusted. Until a reliable org
membership check is in place, accept the label as long as it is present
on the PR instead of only on the labeled event.
Bot-authored PRs (odh-release-bot, jira-autofix, dependabot) hit the same
gap: author_association reports CONTRIBUTOR for them regardless of the
app's actual permissions, confirmed on opendatahub-io#3768, opendatahub-io#3760, and opendatahub-io#3756. Trust
those exact logins the same way trusted human associations are trusted.
lgtm and approved are trusted too, but only at the exact labeled event,
not whenever present. Prow strips lgtm from a PR on every new commit by
default in this repo (no sticky_lgtm_team or store_tree_hash configured),
so a malicious synchronize commit never carries the labeled action either
label needs, even if Prow hasn't removed the stale lgtm yet. approved
isn't auto-revoked on push, but that doesn't matter here since the check
never inspects current label state, only the labeling event itself.
Given that author_association gap, running the ODH xKS workflow on every
PR relied too much on a trust signal that fails for most of the org. This
workflow only exercises the kserve component on a Cloud Manager-
provisioned cluster (make e2e-test-xks sets E2E_TEST_COMPONENT=kserve),
so scope its trigger to kserve, cloudmanager, and config/rhaii paths
(the manifests make deploy-rhaii-local deploys for this test) instead of
running on every PR. The cloudmanager workflow already has its own path
filter and keeps the label-and-authorization gate as its primary check,
since that is where the actual secret risk is.1 parent 7cbf50d commit 6ffa565
2 files changed
Lines changed: 45 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| 45 | + | |
45 | 46 | | |
46 | 47 | | |
| 48 | + | |
47 | 49 | | |
| 50 | + | |
48 | 51 | | |
49 | 52 | | |
50 | 53 | | |
51 | 54 | | |
52 | | - | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
53 | 66 | | |
54 | 67 | | |
55 | 68 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
8 | 25 | | |
9 | 26 | | |
10 | 27 | | |
| |||
29 | 46 | | |
30 | 47 | | |
31 | 48 | | |
| 49 | + | |
32 | 50 | | |
33 | 51 | | |
| 52 | + | |
34 | 53 | | |
| 54 | + | |
35 | 55 | | |
36 | 56 | | |
37 | 57 | | |
38 | 58 | | |
39 | | - | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
40 | 70 | | |
41 | 71 | | |
42 | 72 | | |
| |||
0 commit comments