-
Notifications
You must be signed in to change notification settings - Fork 428
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
Remove legacy CLI commands #4160
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
04162de
to
1938868
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
caf6c9f
to
909a5c4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
909a5c4
to
a416630
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
0f3f3a9
to
22eba40
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #4160 +/- ##
==========================================
+ Coverage 84.03% 84.29% +0.26%
==========================================
Files 564 550 -14
Lines 34064 33413 -651
==========================================
- Hits 28625 28165 -460
+ Misses 5439 5248 -191 ☔ View full report in Codecov by Sentry. |
fcb3d76
to
bcb0e77
Compare
This comment was marked as outdated.
This comment was marked as outdated.
bcb0e77
to
759a9d9
Compare
This comment was marked as outdated.
This comment was marked as outdated.
759a9d9
to
5b5b11a
Compare
This comment was marked as outdated.
This comment was marked as outdated.
5b5b11a
to
53f777b
Compare
This comment was marked as outdated.
This comment was marked as outdated.
53f777b
to
a6dd81c
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
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.
Nice to see the all the cleanup. The main issue I see is that some commands were partially removed, e.g. mongooseimctl
still has a lot of scripting around commands like join_cluster
. Furthermore, removal of such commands would break our Docker and Helm installations. Finally, these commands were not clearly marked as deprecated. Details in comments.
This comment was marked as outdated.
This comment was marked as outdated.
3b3d515
to
5f265e4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
5f265e4
to
21547ff
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
4d75d36
to
d249add
Compare
This comment was marked as outdated.
This comment was marked as outdated.
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.
Looks really good 👍
This comment was marked as outdated.
This comment was marked as outdated.
877040e
to
fb50d9f
Compare
This comment was marked as outdated.
This comment was marked as outdated.
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.
A few minor comments from me
elasticsearch_and_cassandra_26 / elasticsearch_and_cassandra_mnesia / d51f4fc small_tests_25 / small_tests / d51f4fc small_tests_26 / small_tests / d51f4fc small_tests_26_arm64 / small_tests / d51f4fc ldap_mnesia_25 / ldap_mnesia / d51f4fc dynamic_domains_mysql_redis_26 / mysql_redis / d51f4fc ldap_mnesia_26 / ldap_mnesia / d51f4fc dynamic_domains_pgsql_mnesia_26 / pgsql_mnesia / d51f4fc pgsql_cets_26 / pgsql_cets / d51f4fc internal_mnesia_26 / internal_mnesia / d51f4fc dynamic_domains_mssql_mnesia_26 / odbc_mssql_mnesia / d51f4fc muc_SUITE:register:user_submits_registration_form_twice{error,
{{assertion_failed,assert,is_iq_result,
[{xmlel,<<"iq">>,
[{<<"type">>,<<"set">>},
{<<"id">>,<<"4fd3318a89013a6160c6e688035cfb19">>},
{<<"to">>,<<"groupchats.domain.example.com">>}],
[{xmlel,<<"query">>,
[{<<"xmlns">>,<<"jabber:iq:register">>}],
[{xmlel,<<"x">>,
[{<<"xmlns">>,<<"jabber:x:data">>},
{<<"type">>,<<"submit">>}],
[{xmlel,<<"field">>,
[{<<"type">>,<<"hidden">>},
{<<"var">>,<<"FORM_TYPE">>}],
[{xmlel,<<"value">>,[],
[{xmlcdata,<<"jabber:iq:register">>}]}]},
{xmlel,<<"field">>,
[{<<"type">>,<<"text-single">>},
{<<"var">>,<<"nick">>}],
[{xmlel,<<"value">>,[],
[{xmlcdata,
<<"thirdwitchroom-a56e1ea50d">>}]}]}]}]}]}],
{xmlel,<<"iq">>,
[{<<"from">>,<<"groupchats.domain.example.com">>},
{<<"to">>,
<<"alice_user_submits_registration_form_twice_2611@domain.example.com/res1">>},
{<<"type">>,<<"error">>},
{<<"id">>,<<"4fd3318a89013a6160c6e688035cfb19">>}],
[{xmlel,<<"query">>,
[{<<"xmlns">>,<<"jabber:iq:register">>}],
[{xmlel,<<"x">>,
[{<<"xmlns">>,<<"jabber:x:data">>},
{<<"type">>,<<"submit">>}],
[{xmlel,<<"field">>,
[{<<"type">>,<<"hidden">>},
{<<"var">>,<<"FORM_TYPE">>}],
... pgsql_mnesia_25 / pgsql_mnesia / d51f4fc pgsql_mnesia_26 / pgsql_mnesia / d51f4fc mysql_redis_26 / mysql_redis / d51f4fc mssql_mnesia_26 / odbc_mssql_mnesia / d51f4fc graphql_muc_light_SUITE:admin_cli:admin_muc_light:end_per_group{error,
{{unregistering_failed,
{amount,1},
{unregistered_items,
[{{<<"_admin_delete_room_1126">>,
[{escalus_event_mgr,<0.24524.0>},
{tc_name,admin_delete_room},
{escalus_cleaner,<0.24523.0>},
{watchdog,<0.24522.0>},
{muc_light_host,<<"muclight.localhost">>},
{secondary_muc_light_host,<<"muclight.localhost.bis">>},
{protocol,cli},
{schema_endpoint,admin},
{{ejabberd_cwd,mongooseim@localhost},
"/home/circleci/project/_build/mim1/rel/mongooseim"},
{preset,"odbc_mssql_mnesia"},
{mim_data_dir,
"/home/circleci/project/big_tests/tests/graphql_muc_light_SUITE_data"},
{tc_logfile,
"https://circleci-mim-results.s3.eu-central-1.amazonaws.com/PR/4160/200780/odbc_mssql_mnesia.26.1.2-amd64/big/ct_run.test%40a2158090244d.2023-11-24_08.05.14/big_tests.tests.graphql_muc_light_SUITE.logs/run.2023-11-24_08.09.50/graphql_muc_light_suite.admin_delete_room.43651.html"},
{tc_group_properties,[{name,admin_muc_light}]},
{tc_group_path,[[{name,admin_cli}]]},
{data_dir,
"/home/circleci/project/big_tests/_build/default/lib/mongoose_tests/ebin/graphql_muc_light_SUITE_data/"},
{priv_dir,
"https://circleci-mim-results.s3.eu-central-1.amazonaws.com/PR/4160/200780/odbc_mssql_mnesia.26.1.2-amd64/big/ct_run.test%40a2158090244d.2023-11-24_08.05.14/big_tests.tests.graphql_muc_light_SUITE.logs/run.2023-11-24_08.09.50/log_private/"},
{{saved_modules,mongooseim@localhost,<<"localhost">>},
#{mod_vcard =>
#{matches => 30,
host => {prefix,<<"vjud.">>},
search => true,backend => rdbms,iqdisc => parallel},
mod_roster ... dynamic_domains_pgsql_mnesia_25 / pgsql_mnesia / d51f4fc dynamic_domains_mssql_mnesia_26 / odbc_mssql_mnesia / d51f4fc |
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.
👌
This pull request removes outdated command-line interface commands, along with their associated configuration options and tests. Additionally, it removes the
mnesia
command when executed without arguments, as its functionality is already covered by thesystemInfo
command.