Skip to content

Remove report portal dependency#2834

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
JoaoFula:remove-report-portal-dependency
Mar 25, 2026
Merged

Remove report portal dependency#2834
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
JoaoFula:remove-report-portal-dependency

Conversation

@JoaoFula
Copy link
Copy Markdown
Contributor

Description

Removed ReportPortal code from tests/e2e/conftest.py:

  • Removed imports: reportportal_client, logging, tarfile, tempfile, uuid
  • Removed --rp_name pytest option
  • Removed rp_logger fixture and all ReportPortal helper functions
  • Simplified pytest_sessionfinish() to only gather OLS artifacts

Removed ReportPortal dependencies from pyproject.toml:

  • Removed reportportal-client>=5.5.6
  • Removed pytest-reportportal>=5.4.1

Updated uv.lock:

  • Ran uv sync which automatically uninstalled both packages

Deleted scripts/upload_artifact_s3.py:

  • This file was only used for ReportPortal artifact uploads

Updated README.md:

  • Removed the entire "Uploading artifact to S3 bucket" sectio

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

@openshift-ci openshift-ci bot requested review from raptorsun and xrajesh March 20, 2026 08:44
Removing report portal dependency
@JoaoFula JoaoFula force-pushed the remove-report-portal-dependency branch from d969606 to 749be85 Compare March 20, 2026 09:40
@JoaoFula
Copy link
Copy Markdown
Contributor Author

/retest

1 similar comment
@JoaoFula
Copy link
Copy Markdown
Contributor Author

/retest

@raptorsun
Copy link
Copy Markdown
Contributor

/test e2e-ols-cluster

Comment thread uv.lock
@@ -1,5 +1,5 @@
version = 1
revision = 3
revision = 2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Have you run uv sync with an older version of uv?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yeah, it's possible

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

well, just updated uv and ran uv sync and still the revision remained the same

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ok, I guess it will be changed again in some other PR later

@onmete
Copy link
Copy Markdown
Contributor

onmete commented Mar 25, 2026

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 25, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 25, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: onmete

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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 25, 2026
@onmete
Copy link
Copy Markdown
Contributor

onmete commented Mar 25, 2026

openai.test_query_endpoint: test_valid_question failure
/test e2e-ols-cluster

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 25, 2026

@JoaoFula: 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.

@openshift-merge-bot openshift-merge-bot bot merged commit 0abc7ae into openshift:main Mar 25, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

3 participants