Skip to content

Fix magic steps and gaps in MetalLB procedures#111432

Open
kquinn1204 wants to merge 1 commit intoopenshift:mainfrom
kquinn1204:demo-install-configure-metallb-ai-1
Open

Fix magic steps and gaps in MetalLB procedures#111432
kquinn1204 wants to merge 1 commit intoopenshift:mainfrom
kquinn1204:demo-install-configure-metallb-ai-1

Conversation

@kquinn1204
Copy link
Copy Markdown
Contributor

Summary

  • install-cli: Replace file-save + oc create -f with inline cat << EOF | oc apply -f - for the Subscription CR (eliminates magic step)
  • initial-config: Add NOTE about controller deployment startup delay in verification section
  • L2-advertisement: Replace file-save patterns with inline apply, add missing .Verification section, add missing prerequisite ("Install the MetalLB Operator and start MetalLB")
  • configure-svc: Add concrete Service YAML with callouts, replace <service_name> placeholders with example-metallb-svc, update deprecated metallb.io/address-pool annotation to metallb.io/ip-allocated-from-pool, fix typo in description list

Test plan

  • All 4 procedures verified on a live OCP 4.21 cluster
  • Each command executed end-to-end: namespace, OperatorGroup, Subscription, MetalLB CR, IPAddressPool, L2Advertisement, LoadBalancer Service
  • Verification steps confirmed: controller deployment, speaker daemonset, IP allocation, service describe output
  • AsciiDoc renders correctly in preview build

🤖 Generated with Claude Code

Replaces file-save-then-apply patterns with inline cat << EOF | oc apply
across install, L2 advertisement, and service configuration procedures.
Adds missing verification section, prerequisites, timing notes, concrete
YAML examples, and updates deprecated annotation in example output.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 8, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 8, 2026

@kquinn1204: all tests passed!

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants