plugin system cleanup

Summary

• fix: make plugin operations properly reenterable

• fix: type in if_not_exists

• fix: remove auto-migration flags from plugin_install & plugin_remove

• chore: don't use name Error for such a useless thing

• refactor: cleanup CaS code in plugins

• fix: remove redundant DisablePlugin governor step

• fix: cleanup backtrace logging

• chore: document why I'm not removing redundant RemovePlugin raft op

• chore: fix compilation warning

• test: reduce test_sdk_log flakiness

• fix: remove redundant UpdateTopology raft op

• fix: remove redundant EnablePlugin raft op

• refactor: extract plugin raft operations into a Op::Plugin sub operation

• fix: remove long erroneous waits when leader changes during cas operation

• refactor: use a single pico property for all plugin operations

• fix: add CallbackResult to prelude, because it's needed for all services

• fix: remove redundant read_index operations

• chore: every PropertyName must have a doc-comment

• chore: minor tweaks

• fix: used to allow removing plugins with applied UP migrations

• fix: used to leave dead _pico_plugin_migration records around

• fix: update tarantool crate to include recent fiber api fixes

---

• Closes #833 (closed)
• Closes #819 (closed)
• Cherry-pick to: none
• Docs follow-up: not necessary

changed milestone to %24.6 - Plugins and Tiers

changed title from fix: multiline input to plugin system cleanup

resolved all threads

mentioned in issue #837 (closed)

added 20 commits

• 8ece989d - 1 commit from branch gmoshkin/plugin-only-one-registrar
• 8ece989d...0fa07110 - 9 earlier commits
• 2adf98c6 - test: reduce test_sdk_log flakiness
• 3e36155e - chore: fix compilation warning
• a20af2b0 - chore: document why I'm not removing redundant RemovePlugin raft op
• e1e7ddbd - fix: cleanup backtrace logging
• 1e2b95da - fix: remove redundant DisablePlugin governor step
• 252df91a - refactor: cleanup CaS code in plugins
• b99df133 - chore: don't use name `Error` for such a useless thing
• 9176aab9 - fix: remove auto-migration flags from plugin_install & plugin_remove
• 2a85ae28 - fix: type in if_not_exists
• 35ad907b - fix: make plugin operations properly reenterable

Compare with previous version

added 15 commits

• 35ad907b...c57fe782 - 5 earlier commits
• ff09ae8b - test: reduce test_sdk_log flakiness
• b445e3f1 - chore: fix compilation warning
• 43453987 - chore: document why I'm not removing redundant RemovePlugin raft op
• 6be2f6a8 - fix: cleanup backtrace logging
• ddc90191 - fix: remove redundant DisablePlugin governor step
• ffd42ad2 - refactor: cleanup CaS code in plugins
• 7d5048c4 - chore: don't use name `Error` for such a useless thing
• 35d9fd99 - fix: remove auto-migration flags from plugin_install & plugin_remove
• 6879392e - fix: type in if_not_exists
• 34c5c148 - fix: make plugin operations properly reenterable

Compare with previous version

changed the description

added 1 commit

• 7f773a88 - fix: make plugin operations properly reenterable

Compare with previous version

added 1 commit

• 1495e611 - fix: struct Clusterwide fields use plural naming

Compare with previous version

added 16 commits

• 1495e611...36db8f9c - 6 earlier commits
• c51ff566 - chore: fix compilation warning
• 5c47fd27 - chore: document why I'm not removing redundant RemovePlugin raft op
• f58de46d - fix: cleanup backtrace logging
• 95f55003 - fix: remove redundant DisablePlugin governor step
• 6f6c812d - refactor: cleanup CaS code in plugins
• 2c302ce6 - chore: don't use name `Error` for such a useless thing
• 243894d5 - fix: remove auto-migration flags from plugin_install & plugin_remove
• 03d8b087 - fix: type in if_not_exists
• 761e207a - fix: make plugin operations properly reenterable
• 9f131562 - fix: struct Clusterwide fields use plural naming

Compare with previous version

added 1 commit

• 3f74991b - fix: cleanup InstallPlugin governor step

Compare with previous version

added 1 commit

• 263f5e56 - fix: cleanup EnablePlugin governor step

Compare with previous version

added 4 commits

• bac4eb01 - fix: make plugin operations properly reenterable
• 6f23d52c - fix: struct Clusterwide fields use plural naming
• b977fb0d - fix: cleanup InstallPlugin governor step
• e2b2b920 - fix: cleanup EnablePlugin governor step

Compare with previous version

added 5 commits

• 7997174d - refactor: preparations for returning errors from governor
• 638feda2 - fix: used to return incorrect error in case plugin disappears during enable plugin operation
• 5cb6354e - fix: used to panic when calling RaftStorage::entries with high=u64::MAX
• 6e7fb65c - fixup cas error handling
• 7e2c6db7 - fix: return concrete error from EnablePlugin

Compare with previous version

added 17 commits

• 7e2c6db7...06481786 - 7 earlier commits
• a122e67f - fix: cleanup backtrace logging
• a6a164b1 - fix: remove redundant DisablePlugin governor step
• 51c1aa43 - refactor: cleanup CaS code in plugins
• bf021ab1 - chore: don't use name `Error` for such a useless thing
• 9e12f3c8 - fix: remove auto-migration flags from plugin_install & plugin_remove
• 6abe1e6a - fix: type in if_not_exists
• 719977fb - fix: make plugin operations properly reenterable
• 604fbbd6 - fix: struct Clusterwide fields use plural naming
• d948875f - fix: cleanup InstallPlugin governor step
• 81efbb22 - fix: cleanup EnablePlugin governor step

Compare with previous version

changed the description

added 6 commits

• ee31e8de - fix: remove auto-migration flags from plugin_install & plugin_remove
• 7d995de0 - fix: type in if_not_exists
• 4b5b351a - fix: make plugin operations properly reenterable
• 922d1be9 - fix: struct Clusterwide fields use plural naming
• e5fa4759 - fix: cleanup InstallPlugin governor step
• b39cbd00 - fix: cleanup EnablePlugin governor step

Compare with previous version

deleted the gmoshkin/plugin-only-one-registrar branch. This merge request now targets the master branch

requested review from @rosik

changed the description

approved this merge request

resolved all threads

added 52 commits

• b39cbd00...e08a3225 - 32 commits from branch master
• e08a3225...2fcb0ebd - 10 earlier commits
• 0674a830 - fix: cleanup backtrace logging
• dca8812f - fix: remove redundant DisablePlugin governor step
• d40e24f3 - refactor: cleanup CaS code in plugins
• ec05176c - chore: don't use name `Error` for such a useless thing
• 0b639d9d - fix: remove auto-migration flags from plugin_install & plugin_remove
• 4418ac70 - fix: type in if_not_exists
• 0c36f8f9 - fix: make plugin operations properly reenterable
• 4fcd42cb - fix: struct Clusterwide fields use plural naming
• 8dcc753a - fix: cleanup InstallPlugin governor step
• 3015c0eb - fix: cleanup EnablePlugin governor step

Compare with previous version

changed the description

added 3 commits

• 5fc82c76 - fix: struct Clusterwide fields use plural naming
• 4c605476 - fix: cleanup InstallPlugin governor step
• 9bedbfd2 - fix: cleanup EnablePlugin governor step

Compare with previous version

enabled an automatic merge when the pipeline for 9bedbfd2 succeeds

merged

changed milestone to %24.5 - Lowercase