Skip to content

Conversation

joelanford
Copy link
Member

Description

This PR renames the internal operator-controller service from "operator-controller-controller-manager-metrics-service" to "operator-controller-service".

It also moves the service kustomize manifest from the rbac base to the manager base.

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

@joelanford joelanford requested a review from a team as a code owner January 6, 2025 15:51
Copy link

netlify bot commented Jan 6, 2025

Deploy Preview for olmv1 ready!

Name Link
🔨 Latest commit 6702698
🔍 Latest deploy log https://app.netlify.com/sites/olmv1/deploys/677bfd2c848b480008dee98e
😎 Deploy Preview https://deploy-preview-1544--olmv1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@joelanford joelanford force-pushed the move-rename-service branch from 24be992 to c4a6850 Compare January 6, 2025 15:52
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
@joelanford joelanford force-pushed the move-rename-service branch from c4a6850 to 6702698 Compare January 6, 2025 15:56
Copy link
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 6, 2025
Copy link

codecov bot commented Jan 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.25%. Comparing base (de6a813) to head (6702698).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1544   +/-   ##
=======================================
  Coverage   74.25%   74.25%           
=======================================
  Files          42       42           
  Lines        3329     3329           
=======================================
  Hits         2472     2472           
  Misses        676      676           
  Partials      181      181           
Flag Coverage Δ
e2e 52.11% <ø> (ø)
unit 56.98% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@joelanford joelanford added this pull request to the merge queue Jan 6, 2025
Merged via the queue into operator-framework:main with commit 2c812aa Jan 6, 2025
19 checks passed
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 6, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 6, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 6, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 6, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 6, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 6, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 8, 2025
camilamacedo86 pushed a commit to camilamacedo86/operator-controller that referenced this pull request Jan 8, 2025
@joelanford joelanford deleted the move-rename-service branch February 14, 2025 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants