Skip to content

python: adapt for new Python module versions#1546

Merged
joergsteffens merged 7 commits intobareos:masterfrom
joergsteffens:dev/joergs/master/fix_restapi
Sep 28, 2023
Merged

python: adapt for new Python module versions#1546
joergsteffens merged 7 commits intobareos:masterfrom
joergsteffens:dev/joergs/master/fix_restapi

Conversation

@joergsteffens
Copy link
Member

@joergsteffens joergsteffens commented Sep 11, 2023

This PR makes some changes, so that python-bareos and bareos-restapi are able to be used with the current versions of the Python modules packaging (tested with 23.1) and pydantic (tested with 2.3.0).

The pydantic problem has been reported by #1544

Backport of PR #0000 to bareos-2x (remove this line if this is no backport; for backport use cherry-pick -x)

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

Make sure you check/merge the PR using devtools/pr-tool to have some simple automated checks run and a proper changelog record added.

General
  • Is the PR title usable as CHANGELOG entry?
  • Purpose of the PR is understood
  • Commit descriptions are understandable and well formatted
Source code quality
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
Tests
  • Decision taken that a test is required (if not, then remove this paragraph)
  • The choice of the type of test (unit test or systemtest) is reasonable
  • Testname matches exactly what is being tested
  • On a fail, output of the test leads quickly to the origin of the fault

@joergsteffens joergsteffens self-assigned this Sep 11, 2023
@joergsteffens joergsteffens force-pushed the dev/joergs/master/fix_restapi branch from f494b97 to 8f4c063 Compare September 13, 2023 08:26
@pstorz pstorz requested a review from sduehr September 14, 2023 10:16
@joergsteffens joergsteffens marked this pull request as ready for review September 14, 2023 10:30
@joergsteffens joergsteffens force-pushed the dev/joergs/master/fix_restapi branch 4 times, most recently from 82d8c25 to cccdf33 Compare September 27, 2023 07:07
Copy link
Member

@sduehr sduehr left a comment

Choose a reason for hiding this comment

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

Thanks, looks good. Only one small suggestion.

@joergsteffens joergsteffens force-pushed the dev/joergs/master/fix_restapi branch from 18adc85 to 5562733 Compare September 28, 2023 11:11
Copy link
Member

@sduehr sduehr left a comment

Choose a reason for hiding this comment

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

Thanks, looks good now.

@joergsteffens
Copy link
Member Author

@sduehr I integrated your suggestion. Please check again.

Copy link
Member

@sduehr sduehr left a comment

Choose a reason for hiding this comment

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

Thanks

joergsteffens and others added 7 commits September 28, 2023 16:56
Bareos version numbers don't comply to PEP 440
and needs to be translated.
Version translation has been adapted to the
more strict requirements of the
Python packaging modul >=23.
Don't require preinstalled python-bareos package,
as this modul is used from the local checkout
(and not available on the testing systems).
Co-authored-by: sduehr <stephan.duehr@bareos.com>
@joergsteffens joergsteffens force-pushed the dev/joergs/master/fix_restapi branch from 862353b to 13c3546 Compare September 28, 2023 14:56
@joergsteffens joergsteffens merged commit 9ecffc4 into bareos:master Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants