Skip to content

[DO NOT MERGE] Test PR for integration test trigger #2160

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[DO NOT MERGE] Test PR for integration test trigger #2160

wants to merge 1 commit into from

Conversation

mlassak
Copy link
Member

@mlassak mlassak commented Jul 14, 2025

Description

DO NOT MERGE

This PR serves just to test proper interaction to trigger Jenkins test suite

JIRA ref: RHOAIENG-27973

@coderabbitai coderabbitai
Copy link

coderabbitai bot commented Jul 14, 2025 •
edited
Loading

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@openshift-ci openshift-ci bot requested review from CFSNM and robotmaxtron July 14, 2025 11:58
@mlassak mlassak marked this pull request as draft July 14, 2025 11:59
@mlassak
Copy link
Member Author

mlassak commented Jul 14, 2025

/label run-integration-tests

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from 376a2ed to 08c3736 Compare July 14, 2025 12:15
@codecov Codecov
Copy link

codecov bot commented Jul 14, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.34%. Comparing base (5140c52) to head (171f77e).

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2160      +/-   ##
==========================================
- Coverage   40.40%   40.34%   -0.07%     
==========================================
  Files         149      149              
  Lines       12079    12079              
==========================================
- Hits         4881     4873       -8     
- Misses       6794     6802       +8     
  Partials      404      404

☔ 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.

@github-actions GitHub Actions
Copy link
Contributor

/test-integration

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch 2 times, most recently from 3369788 to 279c3c3 Compare July 18, 2025 14:05
@github-actions GitHub Actions
Copy link
Contributor

/test-integration

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from 279c3c3 to b237a06 Compare July 22, 2025 14:35
@github-actions GitHub Actions
Copy link
Contributor

/test-integration

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from b237a06 to 6464c5b Compare August 1, 2025 06:32
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 1, 2025

/test-integration

@zdtsw
Copy link
Member

zdtsw commented Aug 5, 2025

is this still needed or we can close it?

@mlassak
Copy link
Member Author

mlassak commented Aug 5, 2025

Still relevant for testing purposes

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from 6464c5b to 1975144 Compare August 6, 2025 09:28
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Aug 6, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 6, 2025

/test-integration

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from 1975144 to 0915e01 Compare August 6, 2025 10:58
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 6, 2025

/test-integration

@rhods-ci-bot
Copy link
Collaborator

E2E Tests #6 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

E2E Tests #1 triggered by #2160 (comment)

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch 2 times, most recently from 4d1a8f7 to b4e47dc Compare August 7, 2025 06:10
@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #7 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

1 similar comment
@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Tests #1 completed, but test results file could not be found.

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

@CFSNM
Copy link
Member

CFSNM commented Aug 11, 2025

@mlassak why are the jobs being triggered in the custom local instance running on 10.0.188.37?

@mlassak
Copy link
Member Author

mlassak commented Aug 11, 2025 •
edited
Loading

@CFSNM You can ignore it, it's a side-effect from my testing. To explain, I cannot emulate the UMB trigger flow on my test local Jenkins instance, so I'm using a pre-defined, hardcoded mock CI MESSAGE that references this PR -> that explains these messages.

I'm currently already implementing/testing the functionality to post Jenkins test results as a comment

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Aug 11, 2025

@mlassak: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/opendatahub-operator-e2e e1e1a59 link true /test opendatahub-operator-e2e

Full PR test history. Your PR dashboard.

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.

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #2 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

❌ ODH Operator Integration Tests Results

Build: #1 → rhoai-test-flow #1
Result: FAILURE (78% pass rate)

Test Summary:

  • 📊 Total Tests: 53
  • ✅ Passed: 39
  • ❌ Failed: 11
  • ⚠️ Errors: 0
  • ⏭️ Skipped: 3

Details: View Full Test Report

@mlassak
Copy link
Member Author

mlassak commented Aug 13, 2025

/remove-label run-integration-tests

@mlassak
Copy link
Member Author

mlassak commented Aug 13, 2025

/label run-integration-tests

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

1 similar comment
@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #1 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Tests #1 completed, but test results file could not be found.

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from e1e1a59 to 929bf45 Compare August 25, 2025 06:19
@github-actions GitHub Actions
Copy link
Contributor

/test-integration

@mlassak mlassak force-pushed the test-branch-rhoaieng-27973 branch from 929bf45 to 171f77e Compare August 25, 2025 09:42
@github-actions GitHub Actions
Copy link
Contributor

/test-integration

@rhods-ci-bot
Copy link
Collaborator

ODH Operator Integration Tests #21 triggered by #2160 (comment)

@rhods-ci-bot
Copy link
Collaborator

⚠️ ODH Operator Integration Tests Results

Build: #21 → rhoai-test-flow #6397
Result: UNSTABLE (96% pass rate)

Test Summary:

  • 📊 Total Tests: 56
  • ✅ Passed: 51
  • ❌ Failed: 2
  • ⚠️ Errors: 0
  • ⏭️ Skipped: 3

Details: View Full Test Report

@mlassak
Copy link
Member Author

mlassak commented Aug 25, 2025

Integration test pipeline is working as intended, closing this testing PR

@mlassak mlassak closed this Aug 25, 2025
@github-project-automation github-project-automation bot moved this from Todo to Done in ODH Platform Planning Aug 25, 2025
@mlassak mlassak mentioned this pull request Aug 25, 2025
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CFSNM CFSNM Awaiting requested review from CFSNM

@robotmaxtron robotmaxtron Awaiting requested review from robotmaxtron

Assignees
No one assigned
Labels
do-not-merge/work-in-progress run-integration-tests
Projects
ODH Platform Planning
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mlassak @zdtsw @rhods-ci-bot @CFSNM