Skip to content

python-{dir,fd,sd} plugins: do not load python2 and 3 versions in parallel (backport of PR #730)#733

Merged
pstorz merged 4 commits intobareos-20from
dev/pstorz/bareos-20/pyplug-parallel-load
Feb 17, 2021
Merged

python-{dir,fd,sd} plugins: do not load python2 and 3 versions in parallel (backport of PR #730)#733
pstorz merged 4 commits intobareos-20from
dev/pstorz/bareos-20/pyplug-parallel-load

Conversation

@pstorz
Copy link
Member

@pstorz pstorz commented Feb 15, 2021

Please follow these few prerequisites before you hand in a PR

Consider the following chapters in the Bareos Developer Documentation

Add some information

  • Add a small description to the CHANGELOG.md file and refer to your PR using this syntax '[PR #xyz]'
  • Add your name to the AUTHORS file

Keep spirit!

  • Do not be afraid to hand in a PR!

@pstorz pstorz force-pushed the dev/pstorz/bareos-20/pyplug-parallel-load branch 2 times, most recently from 7585ae5 to f540440 Compare February 15, 2021 13:51
@pstorz pstorz added the is a backport to 19.2 This is a backport from master to bareos-19.2 label Feb 15, 2021
@pstorz pstorz requested a review from franku February 15, 2021 13:52
Copy link
Contributor

@franku franku left a comment

Choose a reason for hiding this comment

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

You may want to add a line to the Fixed section in the changelog as proposed.

### Fixed
- fix parallel python plugin jobs [PR #732]
- fix oVirt plugin problem with config file [PR #732]

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- fix crash when loading both python-fd and python3-fd plugins [PR #730]

pstorz and others added 4 commits February 17, 2021 10:16
- Check if Py_IsInitialized() before initializing the plugin and skip
  double initialization.
- unloadPlugin(): skip the Python shutdown so that a crash is avoided
This is the reason why the "plugin names" should always be specified.
@pstorz pstorz force-pushed the dev/pstorz/bareos-20/pyplug-parallel-load branch from 48f1427 to 4c54502 Compare February 17, 2021 09:16
@pstorz pstorz merged commit 1f53aaf into bareos-20 Feb 17, 2021
@pstorz pstorz deleted the dev/pstorz/bareos-20/pyplug-parallel-load branch February 17, 2021 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is a backport to 19.2 This is a backport from master to bareos-19.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants