Skip to content

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

Open
hanwen-cluster wants to merge 2 commits intoaws:integ-tests-3.15.0from
hanwen-cluster:integ-tests-3.15.0
Open

[integ-tests-framework] Use Lazy import of dependencies in reports_generator#7313
hanwen-cluster wants to merge 2 commits intoaws:integ-tests-3.15.0from
hanwen-cluster:integ-tests-3.15.0

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:34
@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.
⚠️ Please upload report for BASE (integ-tests-3.15.0@b6b4e59). Learn more about missing BASE report.

Additional details and impacted files
@@                  Coverage Diff                  @@
##             integ-tests-3.15.0    #7313   +/-   ##
=====================================================
  Coverage                      ?   90.08%           
=====================================================
  Files                         ?      182           
  Lines                         ?    16730           
  Branches                      ?        0           
=====================================================
  Hits                          ?    15071           
  Misses                        ?     1659           
  Partials                      ?        0           
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.

@hanwen-cluster hanwen-cluster enabled auto-merge (rebase) March 31, 2026 20:11
@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.

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