Revert "tide: use org-level query for openstack-k8s-operators" - #82821
Merged
openshift-merge-bot[bot] merged 1 commit intoAug 3, 2026
Merged
Conversation
…ift#82798)" This reverts commit 745c993.
Contributor
WalkthroughThe PR moves Prow settings from the organization level to repository-level configurations. Most repositories receive approval, LGTM, ChangesRepository Prow policy configuration
Estimated code review effort: 3 (Moderate) | ~25 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
openshift-ci
Bot
added
the
approved
Contributor
|
@Prucek: all tests passed! Full PR test history. Your PR dashboard. 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. |
openshift-merge-bot
Bot
added
the
rehearsals-ack
Contributor
|
[REHEARSALNOTIFIER] Note: If this PR includes changes to step registry files ( |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@core-services/prow/02_config/openstack-k8s-operators/glance-operator/_prowconfig.yaml`:
- Around line 1-14: Replace the per-repository tide.queries entries in all
listed _prowconfig.yaml files with the organization-level query using orgs:
[openstack-k8s-operators], preserving the required organization-level plugin
configuration and avoiding individual repos qualifiers in each query.
In
`@core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml`:
- Around line 45-47: Add review_acts_as_lgtm: true under the lgtm configuration
in
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml
lines 45-47, lightspeed-operator/_pluginconfig.yaml lines 45-47, and
lightspeed-rag-content/_pluginconfig.yaml lines 45-47, preserving the existing
repository entries.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 1842f07b-526e-47ab-8272-4a28ebd6a732
📒 Files selected for processing (73)
core-services/prow/02_config/openstack-k8s-operators/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/architecture/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/barbican-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/barbican-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ci-framework/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ci-framework/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ci-playground/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/cinder-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/cinder-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/data-plane-adoption/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/data-plane-adoption/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/designate-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/designate-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/edpm-ansible/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/edpm-ansible/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/edpm-image-builder/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/edpm-image-builder/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/glance-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/glance-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/heat-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/heat-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/horizon-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/horizon-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/infra-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/infra-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/install_yamls/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/install_yamls/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ironic-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ironic-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/keystone-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/keystone-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/manila-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/manila-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/mariadb-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/mariadb-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/neutron-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/neutron-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/nova-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/nova-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/octavia-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/octavia-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-baremetal-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-baremetal-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-k8s-operators-ci/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-k8s-operators-ci/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-must-gather/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-must-gather/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/openstack-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/osp-director-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/osp-director-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ovn-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/ovn-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/repo-setup/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/repo-setup/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/sg-core/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/sg-core/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/swift-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/swift-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/tcib/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/tcib/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/telemetry-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/telemetry-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/test-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/test-operator/_prowconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/watcher-operator/_pluginconfig.yamlcore-services/prow/02_config/openstack-k8s-operators/watcher-operator/_prowconfig.yaml
💤 Files with no reviewable changes (1)
- core-services/prow/02_config/openstack-k8s-operators/_pluginconfig.yaml
Comment on lines
+1
to
+14
| tide: | ||
| queries: | ||
| - labels: | ||
| - approved | ||
| - lgtm | ||
| missingLabels: | ||
| - backports/unvalidated-commits | ||
| - do-not-merge/hold | ||
| - do-not-merge/invalid-owners-file | ||
| - do-not-merge/work-in-progress | ||
| - jira/invalid-bug | ||
| - needs-rebase | ||
| repos: | ||
| - openstack-k8s-operators/glance-operator |
Contributor
There was a problem hiding this comment.
🩺 Stability & Availability | 🟠 Major | 🏗️ Heavy lift
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -e
printf '%s\n' '--- target files ---'
for f in \
core-services/prow/02_config/openstack-k8s-operators/glance-operator/_prowconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/heat-operator/_prowconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/horizon-operator/_prowconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/infra-operator/_prowconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/install_yamls/_prowconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/ironic-operator/_prowconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/keystone-operator/_prowconfig.yaml
do
printf '\n### %s\n' "$f"
cat -n "$f"
done
printf '%s\n' '--- related Tide configuration ---'
rg -n -C 3 'openstack-k8s-operators|backports/unvalidated-commits|tide:|queries:' core-services/prow | head -n 500
printf '%s\n' '--- repository-wide matching prow configs ---'
git ls-files 'core-services/prow/**/_prowconfig.yaml' | wc -l
git ls-files 'core-services/prow/**/_prowconfig.yaml' | rg 'openstack-k8s-operators|prowconfig' | head -n 200Repository: openshift/release
Length of output: 50373
🌐 Web query:
openshift/release pull request 82798 Tide resource limit 35 repo qualifiers
💡 Result:
There is no public record of a pull request numbered 82798 in the openshift/release repository. Search results indicate that pull request numbers in the openshift/release repository have recently reached the 81xxx range (e.g., #81370 [1], #81744 [2]). It is possible that the PR number mentioned is either incorrect, from a different repository, or refers to a future or private pull request. Regarding the technical context of the query, Tide is a Prow component used in OpenShift CI to manage pull request merges [3][4]. Tide uses GitHub search queries to identify mergeable pull requests based on criteria such as labels, authors, and repository qualifications [5][3]. Configuration for Tide is managed in _prowconfig.yaml files [4]. While there are various configuration limits in Prow and Tide (such as API rate limits [3][6] and query management [6]), there is no widely documented "35 repo qualifiers" resource limit associated with Tide in public OpenShift or Kubernetes documentation. It is likely that the query refers to a specific configuration adjustment or troubleshooting effort within an internal OpenShift release engineering context that is not publicly accessible.
Citations:
- 1: 9910921
- 2: c266a73
- 3: https://docs.prow.k8s.io/docs/components/core/tide/
- 4: https://docs.ci.openshift.org/how-tos/onboarding-a-new-component/
- 5: https://docs.prow.k8s.io/docs/components/core/tide/config/
- 6: https://docs.prow.k8s.io/docs/components/core/tide/maintainers/
🏁 Script executed:
#!/bin/bash
set -e
printf '%s\n' '--- OpenStack K8s Operators configuration files ---'
git ls-files 'core-services/prow/02_config/openstack-k8s-operators/**' | sed -n '1,200p'
printf '%s\n' '--- organization-scoped Tide queries ---'
rg -n -C 5 'openstack-k8s-operators:' core-services/prow/02_config core-services/prow/01_config 2>/dev/null | head -n 200
printf '%s\n' '--- Tide query merge and generation documentation/code references ---'
rg -n -i -C 3 'prowconfig|tide.*quer|queries.*tide|github.*search|repo qualifier|resource limit' \
README.md docs core-services prow 2>/dev/null | head -n 400
printf '%s\n' '--- recent history for the affected directory ---'
git log --oneline -20 -- core-services/prow/02_config/openstack-k8s-operators
printf '%s\n' '--- repository state and candidate comparison refs ---'
git status --short
git branch --all --no-color | head -n 50Repository: openshift/release
Length of output: 30043
🌐 Web query:
site:github.com/openshift/release "openstack-k8s-operators" "tide" "queries"
💡 Result:
In the context of the openstack-k8s-operators project and the OpenShift release engineering infrastructure, Tide is a Prow component responsible for managing automated PR merges [1][2]. Tide uses a configuration that defines which pull requests are eligible to be automatically merged based on specific criteria, such as passing CI status, required labels, and review approvals [1][3]. There is no single public query explicitly titled "openstack-k8s-operators tide queries." Instead, Tide configuration is managed globally within the openshift/release repository [1][2]. Individual repositories, such as those under the openstack-k8s-operators organization, are subject to these centralized rules, which are defined in Prow configuration files (often located under core-services/prow/02_config/) [1][4][5][6]. Key points regarding how these queries and configurations are managed: 1. Centralized Management: Tide configurations for various repositories, including those for the OpenStack operators, are defined within the openshift/release repository [1][2]. Changes to these configurations, such as removing or updating merge rules for a specific repository, are performed via pull requests to the openshift/release repo [1]. 2. Functionality: Tide queries effectively function by selecting sets of PRs that meet specific criteria (e.g., matching certain labels) and verifying their status against required presubmit jobs [1][3]. These jobs are defined in the ci-operator configuration for each operator [7][6]. 3. Repository Interaction: The openstack-k8s-operators repositories (e.g., nova-operator, cinder-operator) maintain their own CI definitions and OWNERS files, which are frequently synchronized with the central release engineering configuration [4][5][6]. If you are looking to troubleshoot why a PR is not merging via Tide, you should check the Prow configuration files in the openshift/release repository for the specific repository path or verify the status of the required CI jobs for that repository [1][7].
Citations:
- 1: 9910921
- 2: https://github.com/openshift/release/blob/master/Makefile
- 3: https://github.com/openshift/release/blob/main/ci-operator/jobs/infra-periodics.yaml
- 4: b2821cd
- 5: 196f079
- 6: 6043438
- 7: Move openstack-operator jobs to OCP 4.20 #80790
🏁 Script executed:
#!/bin/bash
set -e
printf '%s\n' '--- commit metadata ---'
git show --no-patch --format=fuller 745c993cef
git show --no-patch --format=fuller eec99f1f31
printf '%s\n' '--- changed paths in PR `#82798` ---'
git diff --name-status 745c993cef^ 745c993cef
printf 'changed path count: '
git diff --name-only 745c993cef^ 745c993cef | wc -l
printf '%s\n' '--- changed paths in the revert ---'
git diff --name-status eec99f1f31^ eec99f1f31
printf 'reverted path count: '
git diff --name-only eec99f1f31^ eec99f1f31 | wc -l
printf '%s\n' '--- exact Tide configuration change for affected files ---'
git diff --find-renames --unified=4 745c993cef^ 745c993cef -- \
'core-services/prow/02_config/openstack-k8s-operators/**/_prowconfig.yaml' \
| sed -n '1,500p'
printf '%s\n' '--- current and pre-PR organization-level configuration ---'
for ref in 745c993cef^ 745c993cef eec99f1f31; do
printf '\n### %s\n' "$ref"
git ls-tree -r --name-only "$ref" core-services/prow/02_config/openstack-k8s-operators \
| while read -r f; do
git show "$ref:$f" 2>/dev/null | grep -q '^tide:' && {
echo "--- $f"
git show "$ref:$f"
}
done
doneRepository: openshift/release
Length of output: 50373
🏁 Script executed:
#!/bin/bash
set -e
printf '%s\n' '--- configuration fragment merge rules ---'
cat -n core-services/prow/02_config/README.md | sed -n '1,120p'
printf '%s\n' '--- Tide query configuration documentation references ---'
rg -n -i -C 4 'merge.*query|query.*merge|tide|github search|repos:' \
core-services/prow/02_config/README.md core-services/prow/02_config/_config.yaml \
core-services/prow/02_config 2>/dev/null \
| rg -i 'merge|tide|query|github|openstack-k8s-operators' \
| head -n 300Repository: openshift/release
Length of output: 34008
🏁 Script executed:
#!/bin/bash
set -e
python3 - <<'PY'
from pathlib import Path
import re
root = Path("core-services/prow/02_config/openstack-k8s-operators")
pattern = re.compile(
r"^tide:\n"
r" queries:\n"
r" - labels:\n"
r"(?: - .*\n)+"
r" missingLabels:\n"
r"(?: - .*\n)+"
r" repos:\n"
r" - ([^\n]+)\n",
re.MULTILINE,
)
matches = []
for path in sorted(root.glob("*/_prowconfig.yaml")):
text = path.read_text()
match = pattern.search(text)
if match:
matches.append((path, match.group(1), match.group(0)))
print("matching per-repository Tide blocks:", len(matches))
print("unique repositories:", len({repo for _, repo, _ in matches}))
print("unique query bodies:", len({block for _, _, block in matches}))
print("generated repo qualifier length:",
sum(len(f'repo:{repo} ') for _, repo, _ in matches))
for path, repo, _ in matches:
print(f"{repo}\t{path}")
PY
printf '%s\n' '--- organization-level replacement from PR `#82798` ---'
git show 745c993cef:core-services/prow/02_config/openstack-k8s-operators/_prowconfig.yaml
printf '%s\n' '--- current restored organization-level path ---'
git ls-tree -r --name-only HEAD core-services/prow/02_config/openstack-k8s-operators \
| grep '/_prowconfig.yaml$' \
| grep -E '/openstack-k8s-operators/_prowconfig.yaml$|/tcib/_prowconfig.yaml$' || trueRepository: openshift/release
Length of output: 4848
Do not restore the per-repository Tide queries.
These files restore 35 identical per-repository queries. Prow combines them into one GitHub search with 35 repo: qualifiers. PR #82798 records that this exceeds GitHub’s resource limit and causes Tide sync to fail. Restore the organization-level query with orgs: [openstack-k8s-operators] and preserve the required organization-level plugin configuration.
📍 Affects 7 files
core-services/prow/02_config/openstack-k8s-operators/glance-operator/_prowconfig.yaml#L1-L14(this comment)core-services/prow/02_config/openstack-k8s-operators/heat-operator/_prowconfig.yaml#L1-L14core-services/prow/02_config/openstack-k8s-operators/horizon-operator/_prowconfig.yaml#L1-L14core-services/prow/02_config/openstack-k8s-operators/infra-operator/_prowconfig.yaml#L1-L14core-services/prow/02_config/openstack-k8s-operators/install_yamls/_prowconfig.yaml#L1-L14core-services/prow/02_config/openstack-k8s-operators/ironic-operator/_prowconfig.yaml#L1-L14core-services/prow/02_config/openstack-k8s-operators/keystone-operator/_prowconfig.yaml#L1-L14
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In
`@core-services/prow/02_config/openstack-k8s-operators/glance-operator/_prowconfig.yaml`
around lines 1 - 14, Replace the per-repository tide.queries entries in all
listed _prowconfig.yaml files with the organization-level query using orgs:
[openstack-k8s-operators], preserving the required organization-level plugin
configuration and avoiding individual repos qualifiers in each query.
Source: Path instructions
Comment on lines
+45
to
+47
| lgtm: | ||
| - repos: | ||
| - openstack-k8s-operators/lightspeed-mcps |
Contributor
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
rg -n -C 2 'review_acts_as_lgtm|openstack-k8s-operators/lightspeed-(mcps|operator|rag-content)' \
core-services/prow/02_config/openstack-k8s-operatorsRepository: openshift/release
Length of output: 29212
🏁 Script executed:
#!/bin/bash
set -euo pipefail
for f in \
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/ironic-operator/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/keystone-operator/_pluginconfig.yaml
do
echo "===== $f ====="
cat -n "$f" | sed -n '1,55p'
done
echo "===== all review_acts_as_lgtm entries in Prow configuration ====="
rg -n -B 3 -A 2 '^ review_acts_as_lgtm:' core-services/prowRepository: openshift/release
Length of output: 50374
🏁 Script executed:
#!/bin/bash
set -euo pipefail
files=(
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/ironic-operator/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/keystone-operator/_pluginconfig.yaml
)
for f in "${files[@]}"; do
echo "===== $f ====="
awk '
/^lgtm:/ { in_lgtm=1; print NR ":" $0; next }
in_lgtm && /^ [^ -][^:]*:/ { in_lgtm=0 }
in_lgtm && (/- repos:/ || /review_acts_as_lgtm:/ || /- openstack-k8s-operators\/lightspeed/ || /- openstack-k8s-operators\/ironic/ || /- openstack-k8s-operators\/keystone/) {
print NR ":" $0
}
' "$f"
done
echo "===== Prow references to review_acts_as_lgtm ====="
rg -l 'review_acts_as_lgtm' --glob '*.go' --glob '*.md' --glob '*.yaml' --glob '*.yml' . \
| head -n 80Repository: openshift/release
Length of output: 7827
🏁 Script executed:
#!/bin/bash
set -euo pipefail
files=(
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/ironic-operator/_pluginconfig.yaml
core-services/prow/02_config/openstack-k8s-operators/keystone-operator/_pluginconfig.yaml
)
for f in "${files[@]}"; do
echo "===== $f ====="
sed -n '30,45p' "$f"
done
echo "===== exact lightspeed configuration occurrences ====="
rg -n -B 8 -A 3 'review_acts_as_lgtm|^lgtm:' \
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/ironic-operator/_pluginconfig.yaml \
core-services/prow/02_config/openstack-k8s-operators/keystone-operator/_pluginconfig.yamlRepository: openshift/release
Length of output: 10253
Restore automatic LGTM handling.
Add review_acts_as_lgtm: true under lgtm in all three files:
lightspeed-mcps/_pluginconfig.yamllightspeed-operator/_pluginconfig.yamllightspeed-rag-content/_pluginconfig.yaml
Without this setting, approving reviews do not satisfy the lgtm label requirement.
📍 Affects 3 files
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml#L45-L47(this comment)core-services/prow/02_config/openstack-k8s-operators/lightspeed-operator/_pluginconfig.yaml#L45-L47core-services/prow/02_config/openstack-k8s-operators/lightspeed-rag-content/_pluginconfig.yaml#L45-L47
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In
`@core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml`
around lines 45 - 47, Add review_acts_as_lgtm: true under the lgtm configuration
in
core-services/prow/02_config/openstack-k8s-operators/lightspeed-mcps/_pluginconfig.yaml
lines 45-47, lightspeed-operator/_pluginconfig.yaml lines 45-47, and
lightspeed-rag-content/_pluginconfig.yaml lines 45-47, preserving the existing
repository entries.
deepsm007
approved these changes
Aug 3, 2026
Contributor
Contributor
|
@Prucek: Updated the following 2 configmaps:
DetailsIn response to this:
Instructions 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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reverts #82798
Summary by CodeRabbit
This PR reverts the organization-level Tide query for
openstack-k8s-operators.It restores repository-specific Prow configuration for the affected component repositories. These configurations define repository-specific Tide requirements, including
approvedandlgtmlabels and blocking labels.It also restores repository-level plugins such as
approve,assign,hold,lgtm,lifecycle,verify-owners,wip, andneeds-rebase.