Skip to content

[integ-tests-framework] Use Lazy import of dependencies in reports_generator#7311

Open
hanwen-cluster wants to merge 2 commits intoaws:developfrom
hanwen-cluster:developmar25
Open

[integ-tests-framework] Use Lazy import of dependencies in reports_generator#7311
hanwen-cluster wants to merge 2 commits intoaws:developfrom
hanwen-cluster:developmar25

Conversation

@hanwen-cluster
Copy link
Copy Markdown
Contributor

Description of changes

Sometimes, only a few functions in reports_generator.py are called by a environment, which doesn't have to have all Python dependencies installed

Tests

  • test will run after the PR is merge

Checklist

  • Make sure you are pointing to the right branch.
  • If you're creating a patch for a branch other than develop add the branch name as prefix in the PR title (e.g. [release-3.6]).
  • Check all commits' messages are clear, describing what and why vs how.
  • Make sure to have added unit tests or integration tests to cover the new/modified code.
  • Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

…nerator

Sometimes, only a few functions in reports_generator.py are called by a environment, which doesn't have to have all Python dependencies installed
@hanwen-cluster hanwen-cluster requested review from a team as code owners March 25, 2026 15:32
@hanwen-cluster hanwen-cluster added the skip-changelog-update Disables the check that enforces changelog updates in PRs label Mar 25, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.08%. Comparing base (0a83ff2) to head (fbaabaa).
⚠️ Report is 7 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #7311   +/-   ##
========================================
  Coverage    90.08%   90.08%           
========================================
  Files          182      182           
  Lines        16730    16730           
========================================
  Hits         15071    15071           
  Misses        1659     1659           
Flag Coverage Δ
unittests 90.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.

@gmarciani
Copy link
Copy Markdown
Contributor

This is a short term solution to prevent alarms.
The long term solution is to remove the dependency on SUPPORTED_OS constant and install pandas and other dependencies in Jenkins.

@hanwen-cluster hanwen-cluster enabled auto-merge (rebase) March 31, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog-update Disables the check that enforces changelog updates in PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants