-
Notifications
You must be signed in to change notification settings - Fork 426
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cleaning up some issues with multitenancy support #3158
Conversation
f2004dc
to
6ccae3f
Compare
Codecov Report
@@ Coverage Diff @@
## master #3158 +/- ##
==========================================
+ Coverage 79.82% 79.83% +0.01%
==========================================
Files 396 396
Lines 32255 32266 +11
==========================================
+ Hits 25748 25761 +13
+ Misses 6507 6505 -2
Continue to review full report at Codecov.
|
to get the list of all the modules that miss declaration of dynamic_domains feature, run these commands: test-runner.sh --preset pgsql_mnesia --skip-cover --skip-small-tests --spec dynamic_domains.spec grep -r "dynamic_domains_feature_is_not_supported:" _build/*/rel/mongooseim/log | sed -E "s/.*dynamic_domains_feature_is_not_supported: ([^ ]*) .*/\1/" | sort -u
…ype if module doesn't explicitly declare support for this feature
5ab429a
to
5877191
Compare
The @~2 commit is cherry-picked from beekeeper's repo. it addes declaration of dynamic_domains feature support to some modules. To see the full list of the changed modules use this command: git show --name-only --pretty="format:" @~2 | sed -r "s_.*/(.*)\.erl_\1_p;d" | sort The conversion of 'mod_commands' and 'mod_muc_light_commands' modules seems to be done, however there's no integration testing enabled for that modules in dynamic_domains.spec Other modules (except mod_mam_meta) are dynamically initialised during the execution of the tests currently configured in dynamic_domains.spec, however there's no waranty that all of them are fully converted and tested. So ideally we should recheck all of them, here is the list of that modules: * mod_mam * mod_mam_cache_user * mod_mam_meta * mod_mam_mnesia_prefs * mod_mam_muc * mod_mam_muc_cache_user * mod_mam_muc_rdbms_arch * mod_mam_muc_rdbms_async_pool_writer * mod_mam_rdbms_arch * mod_mam_rdbms_async_pool_writer * mod_mam_rdbms_prefs * mod_mam_rdbms_user * mod_muc * mod_muc_light * mod_muc_log
5877191
to
5baa21c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
this PR introduces the following changes:
dynamic_domains
feature support at gen_moddynamic_domains
support for 'mod_commands' and 'mod_muc_light_commands' modules. Quick check shows that these modules are converted, however there is no proper testing done for them. these 2 modules are changed mostly for alignment with bkpr's codebase.