Skip to content

🌱 OCPBUGS-60693, OCPBUGS-60958: Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios#2612

Merged
openshift-merge-bot[bot] merged 3 commits intooperator-framework:mainfrom
camilamacedo86:bump-crddiff
Mar 31, 2026
Merged

🌱 OCPBUGS-60693, OCPBUGS-60958: Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios#2612
openshift-merge-bot[bot] merged 3 commits intooperator-framework:mainfrom
camilamacedo86:bump-crddiff

Conversation

@camilamacedo86
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 commented Mar 31, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 31, 2026 07:53
@openshift-ci openshift-ci bot requested review from joelanford and oceanc80 March 31, 2026 07:53
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 31, 2026 •
edited
Loading

✅ Deploy Preview for olmv1 ready!

Name Link
🔨 Latest commit f5dedc1
🔍 Latest deploy log https://app.netlify.com/projects/olmv1/deploys/69cbf523a09808000781c183
😎 Deploy Preview https://deploy-preview-2612--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 project configuration.

Copilot AI reviewed Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the sigs.k8s.io/crdify module dependency used by the operator-controller codebase to a newer pseudo-version, reflecting an upstream change without modifying any internal logic.

Changes:

  • Bump sigs.k8s.io/crdify from v0.5.0 to v0.5.1-0.20260309184313-54162f2e3097 in go.mod.
  • Update go.sum checksums to match the new crdify version.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
go.mod Updates the required sigs.k8s.io/crdify version to the newer pseudo-version.
go.sum Refreshes module checksums for the updated sigs.k8s.io/crdify dependency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camilamacedo86 camilamacedo86 changed the title 🌱 Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 🌱 Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios Mar 31, 2026
@camilamacedo86 camilamacedo86 changed the title 🌱 Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios 🌱 OCPBUGS-60693: Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios Mar 31, 2026
Copilot AI review requested due to automatic review settings March 31, 2026 08:16
Copilot AI reviewed Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camilamacedo86 camilamacedo86 changed the title 🌱 OCPBUGS-60693: Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios 🌱 OCPBUGS-60693, OCPBUGS-60958: Upgrade sigs.k8s.io/crdify v0.5.0 => v0.5.1-0.20260309184313-54162f2e3097 and add tests to ensure bug fix scenarios Mar 31, 2026
Copilot AI review requested due to automatic review settings March 31, 2026 08:33
Copilot AI reviewed Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.83%. Comparing base (435e9c5) to head (f5dedc1).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2612      +/-   ##
==========================================
- Coverage   68.86%   68.83%   -0.03%     
==========================================
  Files         139      139              
  Lines        9872     9930      +58     
==========================================
+ Hits         6798     6835      +37     
- Misses       2557     2579      +22     
+ Partials      517      516       -1
Flag Coverage Δ
e2e 37.51% <0.00%> (+0.03%) ⬆️
experimental-e2e 52.33% <0.00%> (-0.25%) ⬇️
unit 53.59% <100.00%> (+0.08%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI review requested due to automatic review settings March 31, 2026 16:24
Copilot AI reviewed Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tmshort
Copy link
Copy Markdown
Contributor

tmshort commented Mar 31, 2026

/approve

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 31, 2026
rashmigottipati
rashmigottipati approved these changes Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@rashmigottipati rashmigottipati 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 Mar 31, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 31, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rashmigottipati, tmshort

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit ba1e4cb into operator-framework:main Mar 31, 2026
26 checks passed
@camilamacedo86 camilamacedo86 deleted the bump-crddiff branch April 1, 2026 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@rashmigottipati rashmigottipati rashmigottipati approved these changes

@joelanford joelanford Awaiting requested review from joelanford

@oceanc80 oceanc80 Awaiting requested review from oceanc80

Assignees

@rashmigottipati rashmigottipati

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@camilamacedo86 @tmshort @rashmigottipati