Skip to content

Conversation

@liudmylaru
Copy link
Member

@liudmylaru liudmylaru commented Nov 13, 2023

Description

Recently unit tests with optional dependencies started failing. Reason - system library install with apt-get on ubuntu failed with "E: Failed to fetch... 404 Not Found" errors. To fix we need to add apt update before using apt-get to install libkrb5-dev graphviz, which required for some of the unit tests with optional dependencies. This libraries can't be installed by pip or conda only.

Jira: no ticket

What was done

  • added apt update before apt-get install
  • rearranged step "Test env setup" into separate action, to avoid duplicate code
  • fixed expected value in test_opctl_conda.py::TestOpctlConda::test_conda_publish
  • removed failed test_get_service_packs_cust_tenancy() as redundant one - we already testing response, which is mocked for dev tenancy - test_get_service_packs() - link

Pre-commit

check python ast.........................................................Passed
check docstring is first.............................(no files to check)Skipped
check json...........................................(no files to check)Skipped
check for merge conflicts................................................Passed
check yaml...........................................(no files to check)Skipped
detect private key.......................................................Passed
fix end of files.........................................................Passed
pretty format json...................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
black....................................................................Passed
rst ``code`` is two backticks........................(no files to check)Skipped
rst ``inline code`` next to normal text..............(no files to check)Skipped
Detect hardcoded secrets.................................................Passed
check-copyright..........................................................Passed
[fix_failing_tests_setup 0ccd1c4] Fix failing system library setup for unit tests

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Nov 13, 2023
@liudmylaru liudmylaru changed the title Fix failing system library install in action for unit tests Fix failing apt-get install in action for unit tests Nov 13, 2023
@z7ye z7ye self-requested a review November 13, 2023 21:11
z7ye
z7ye previously approved these changes Nov 13, 2023
@github-actions
Copy link

📌 Cov diff with main:

No success to gather report. 😿

📌 Overall coverage:

No success to gather report. 😿

@github-actions
Copy link

📌 Cov diff with main:

No success to gather report. 😿

📌 Overall coverage:

No success to gather report. 😿

@lu-ohai lu-ohai self-requested a review November 13, 2023 23:57
Copy link
Member

@lu-ohai lu-ohai left a comment

Choose a reason for hiding this comment

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

Thanks!

@github-actions
Copy link

📌 Cov diff with main:

No lines with coverage information in this diff.

📌 Overall coverage:

Coverage-71.49%

@liudmylaru liudmylaru merged commit 6b75c58 into main Nov 14, 2023
@liudmylaru liudmylaru deleted the fix_failing_tests_setup branch November 14, 2023 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants