-
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
updating setup-beam GH action and OTP versions #4152
Conversation
elasticsearch_and_cassandra_26 / elasticsearch_and_cassandra_mnesia / a03f0dd small_tests_25 / small_tests / a03f0dd small_tests_26 / small_tests / a03f0dd small_tests_26_arm64 / small_tests / a03f0dd dynamic_domains_pgsql_mnesia_25 / pgsql_mnesia / a03f0dd ldap_mnesia_26 / ldap_mnesia / a03f0dd ldap_mnesia_25 / ldap_mnesia / a03f0dd dynamic_domains_mysql_redis_26 / mysql_redis / a03f0dd dynamic_domains_pgsql_mnesia_26 / pgsql_mnesia / a03f0dd pgsql_mnesia_25 / pgsql_mnesia / a03f0dd dynamic_domains_mssql_mnesia_26 / odbc_mssql_mnesia / a03f0dd mysql_redis_26 / mysql_redis / a03f0dd carboncopy_SUITE:one2one:dropped_client_doesnt_create_duplicate_carbons{error,
{{badmatch,
[{xmlel,<<"message">>,
[{<<"from">>,
<<"alice_dropped_client_doesnt_create_duplicate_carbons_589@localhost">>},
{<<"to">>,
<<"alice_dropped_client_doesnt_create_duplicate_carbons_589@localhost/res2">>},
{<<"xmlns">>,<<"jabber:client">>},
{<<"type">>,<<"chat">>}],
[{xmlel,<<"sent">>,
[{<<"xmlns">>,<<"urn:xmpp:carbons:2">>}],
[{xmlel,<<"forwarded">>,
[{<<"xmlns">>,<<"urn:xmpp:forward:0">>}],
[{xmlel,<<"message">>,
[{<<"from">>,
<<"alice_dropped_client_doesnt_create_duplicate_carbons_589@localhost/res1">>},
{<<"type">>,<<"chat">>},
{<<"to">>,
<<"bob_dropped_client_doesnt_create_duplicate_carbons_589@localhost/res1">>},
{<<"xmlns">>,<<"jabber:client">>}],
[{xmlel,<<"body">>,[],
[{xmlcdata,
<<"And pious action">>}]}]}]}]}]}]},
[{carboncopy_SUITE,
'-dropped_client_doesnt_create_duplicate_carbons/1-fun-0-',4,
[{file,
"/home/circleci/project/big_tests/tests/carboncopy_SUITE.erl"},
{line,189}]},
{escalus_story,story,4,
[{file,
"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_story.erl"},
{line,72}]},
{test_server,ts_tc,3,[{file,"test_server.erl"},{line,1793}]},
{test_server,run_test_case_eval1,6,
[{file,"test_serv... internal_mnesia_26 / internal_mnesia / a03f0dd pgsql_cets_26 / pgsql_cets / a03f0dd pgsql_mnesia_26 / pgsql_mnesia / a03f0dd mssql_mnesia_26 / odbc_mssql_mnesia / a03f0dd mysql_redis_26 / mysql_redis / a03f0dd |
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4152 +/- ##
==========================================
+ Coverage 84.00% 84.06% +0.05%
==========================================
Files 561 561
Lines 34020 34020
==========================================
+ Hits 28578 28598 +20
+ Misses 5442 5422 -20 ☔ View full report in Codecov by Sentry. |
elasticsearch_and_cassandra_26 / elasticsearch_and_cassandra_mnesia / d435e2a small_tests_25 / small_tests / d435e2a small_tests_26 / small_tests / d435e2a |
The latest `setup-beam` GH Action (`v1.16.0` as of 18/10/2023) states that only versions up to OTP 25 are supported for ubuntu-22.04 runners (see [this page](https://github.com/marketplace/actions/setup-erlang-otp-with-optional-elixir-and-mix-and-or-rebar3?version=v1.16.0#compatibility-between-operating-system-and-erlangotp) for more details). However, for all Linux runners, the `setup-beam` GH Action uses prebuilt OTP releases stored at [builds.hex.pm](https://builds.hex.pm). Those builds are prepared by [Bob the builder](https://github.com/hexpm/bob#erlang-builds), and the actual list of the prebuilt OTP versions for ubuntu-22.04 runners can be found [here](https://builds.hex.pm/builds/otp/ubuntu-22.04/builds.txt). For more details, check the implementation of [getOTPVersions()](https://github.com/erlef/setup-beam/blob/v1.16.0/dist/index.js#L10120) and [installOTP()](https://github.com/erlef/setup-beam/blob/v1.16.0/dist/index.js#L9721) functions of `setup-beam` GH Action.
d435e2a
to
f82666d
Compare
elasticsearch_and_cassandra_26 / elasticsearch_and_cassandra_mnesia / f82666d small_tests_25 / small_tests / f82666d small_tests_26 / small_tests / f82666d small_tests_26_arm64 / small_tests / f82666d ldap_mnesia_25 / ldap_mnesia / f82666d dynamic_domains_mysql_redis_26 / mysql_redis / f82666d ldap_mnesia_26 / ldap_mnesia / f82666d pubsub_SUITE:dag+basic:retract_when_user_goes_offline_test{error,{{badmatch,false},
[{pubsub_tools,check_response,2,
[{file,"/home/circleci/project/big_tests/tests/pubsub_tools.erl"},
{line,444}]},
{pubsub_tools,receive_response,3,
[{file,"/home/circleci/project/big_tests/tests/pubsub_tools.erl"},
{line,434}]},
{pubsub_tools,receive_and_check_response,4,
[{file,"/home/circleci/project/big_tests/tests/pubsub_tools.erl"},
{line,424}]},
{escalus_story,story,4,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_story.erl"},
{line,72}]},
{test_server,ts_tc,3,[{file,"test_server.erl"},{line,1793}]},
{test_server,run_test_case_eval1,6,
[{file,"test_server.erl"},{line,1302}]},
{test_server,run_test_case_eval,9,
[{file,"test_server.erl"},{line,1234}]}]}} dynamic_domains_pgsql_mnesia_25 / pgsql_mnesia / f82666d metrics_c2s_SUITE:single:iq_one{error,
{{xmppStanzaReceived,
{value,18625},
[{times,25,
{error,
{badmatch,{value,18626}},
[{metrics_helper,assert_counter,3,
[{file,
"/home/circleci/project/big_tests/tests/metrics_helper.erl"},
{line,36}]},
{mongoose_helper,do_wait_until,2,
[{file,
"/home/circleci/project/big_tests/tests/mongoose_helper.erl"},
{line,361}]},
{metrics_c2s_SUITE,'-iq_one/1-fun-0-',1,
[{file,
"/home/circleci/project/big_tests/tests/metrics_c2s_SUITE.erl"},
{line,162}]},
{escalus_story,story,4,
[{file,
"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_story.erl"},
{line,72}]},
{test_server,ts_tc,3,
[{file,"test_server.erl"},{line,1782}]},
{test_server,run_test_case_eval1,6,
[{file,"test_server.erl"},{line,1291}]},
{test_server,run_test_case_eval,9,
[{file,"test_server.erl"},{line,1223}]}]}}],
ok},
[{mongoose_helper,do_wait_until,2,
[{file,"/home/circleci/project/big_tests/tests/mongoose_helper.erl"},
{line,358}]},
{metrics_c2s_SUITE,'-iq_one/1-fun-0-',1,
[{file,
"/home/circleci/project/big_tests/tests/metrics_c2s_SUITE.erl"},
{line,162}]},
{escalus_story,story,4,
[{file,
"/home/circleci/project/big_tests/_build/d... dynamic_domains_pgsql_mnesia_26 / pgsql_mnesia / f82666d pgsql_cets_26 / pgsql_cets / f82666d internal_mnesia_26 / internal_mnesia / f82666d dynamic_domains_mssql_mnesia_26 / odbc_mssql_mnesia / f82666d pgsql_mnesia_25 / pgsql_mnesia / f82666d mysql_redis_26 / mysql_redis / f82666d pgsql_mnesia_26 / pgsql_mnesia / f82666d mssql_mnesia_26 / odbc_mssql_mnesia / f82666d ldap_mnesia_26 / ldap_mnesia / f82666d dynamic_domains_pgsql_mnesia_25 / pgsql_mnesia / f82666d |
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 good in general 👍 I'm wondering though - should we also run dialyzer, xref and edoc on OTP 26 as well?
I think it's not critical, we already do so on CircleCI. So running it on OTP 25 at GH Actions gives us a bit of diversity. it might be beneficial, especially for dialyzer. |
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.
You're right, such diversity is beneficial. Thanks for the changes 🚀
GH Actions build - https://github.com/esl/MongooseIM/actions/runs/6555138169
The latest
setup-beam
GH Action (v1.16.0
as of 18/10/2023) states that only versions up to OTP 25 are supported for ubuntu-22.04 runners (see this page for more details).However, for all Linux runners, the
setup-beam
GH Action uses prebuilt OTP releases stored at builds.hex.pm. Those builds are prepared by Bob the builder, and the actual list of the prebuilt OTP versions for ubuntu-22.04 runners can be found here.For more details, check the implementation of getOTPVersions() and installOTP() functions of
setup-beam
GH Action.