Skip to content

feat: Dummy test DO NOT MERGE run-int-test#911

Closed
BATMAN-JD wants to merge 1 commit intoopenshift:masterfrom
BATMAN-JD:dummy_test_do_not_merge
Closed

feat: Dummy test DO NOT MERGE run-int-test#911
BATMAN-JD wants to merge 1 commit intoopenshift:masterfrom
BATMAN-JD:dummy_test_do_not_merge

Conversation

@BATMAN-JD
Copy link
Copy Markdown
Contributor

What is being added?

Not doing anything other than testing the integration tests will actually run with run-int-test in PR title.

Checklist before requesting review

  • I have tested this locally
  • I have included unit tests
  • I have updated any corresponding documentation

Steps To Manually Test

Please provide us steps to reproduce the scenarios needed to test this. If an integration test is provided, let us know how to run it. If not, enumerate the steps to validate this does what it's supposed to.

  1. Start the operator
  2. Run the thing
  3. Observe X in the spec for the thing
  4. Clean up the thing

Ref OSD-0000

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 6, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 6, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 6, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BATMAN-JD
Once this PR has been reviewed and has the lgtm label, please assign dakotalongrh for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 6, 2026

Codecov Report

❌ Patch coverage is 34.19689% with 127 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.38%. Comparing base (02b0c73) to head (ea1b9ce).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
controllers/accountclaim/reuse.go 43.18% 69 Missing and 6 partials ⚠️
controllers/account/account_finalizer.go 0.00% 31 Missing ⚠️
controllers/accountclaim/accountclaim_finalizer.go 30.00% 20 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #911      +/-   ##
==========================================
- Coverage   47.22%   46.38%   -0.85%     
==========================================
  Files          46       46              
  Lines        6621     6778     +157     
==========================================
+ Hits         3127     3144      +17     
- Misses       3175     3305     +130     
- Partials      319      329      +10     
Files with missing lines Coverage Δ
controllers/accountclaim/accountclaim_finalizer.go 63.38% <30.00%> (-25.99%) ⬇️
controllers/account/account_finalizer.go 14.63% <0.00%> (-25.37%) ⬇️
controllers/accountclaim/reuse.go 42.49% <43.18%> (-11.01%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch 12 times, most recently from 3be742c to 5750a20 Compare January 8, 2026 21:57
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch from 5750a20 to 1b13b78 Compare January 8, 2026 22:19
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch from 1b13b78 to 8c18e90 Compare January 8, 2026 22:46
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

2 similar comments
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch from 8c18e90 to 5c9818c Compare January 9, 2026 16:39
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch from 5c9818c to d6b8df3 Compare January 9, 2026 19:14
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch from d6b8df3 to 79c38ea Compare January 9, 2026 22:02
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@BATMAN-JD BATMAN-JD force-pushed the dummy_test_do_not_merge branch from 79c38ea to ea1b9ce Compare January 9, 2026 23:04
@BATMAN-JD
Copy link
Copy Markdown
Contributor Author

/retest

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 9, 2026

@BATMAN-JD: 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

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants