HYPERFLEET-1097 - fix: update vendor label to "Red Hat, Inc."#143
HYPERFLEET-1097 - fix: update vendor label to "Red Hat, Inc."#143ciaranRoche wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Enterprise Run ID: 📒 Files selected for processing (2)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR updates the Docker image metadata vendor label from "Red Hat" to "Red Hat, Inc." in two places: Dockerfile (line 54) and test/Dockerfile.integration (line 51). No other Dockerfile content or build behavior is changed. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@Dockerfile`:
- Line 54: The Dockerfile change updates the image label vendor to "Red Hat,
Inc." but other images still use vendor="Red Hat", so create a coordinated
follow-up to align stack-wide labels: search for vendor labels in
openshift-hyperfleet/hyperfleet-api, openshift-hyperfleet/hyperfleet-sentinel
and the architecture-standards examples and standardize them to the same value
(e.g., vendor="Red Hat, Inc."), update the Dockerfile labels and any CI/manifest
templates that inject this label (look for occurrences of vendor= or LABEL
vendor= and functions that generate image metadata), and open PRs across those
repos so Enterprise Contract behavior is consistent across the stack.
🪄 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: Central YAML (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Enterprise
Run ID: fd75faab-13a3-4f4e-8ebe-1bd301d87151
📒 Files selected for processing (1)
Dockerfile
EC requires vendor="Red Hat, Inc." not "Red Hat". Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
6446c55 to
86b7410
Compare
Summary
"Red Hat"to"Red Hat, Inc."for Enterprise Contractlabels.required_labelscomplianceTest plan
labels.required_labelscheck passes on next release pipeline run🤖 Generated with Claude Code