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
ci: upgrade to otp 26 #12085
ci: upgrade to otp 26 #12085
Conversation
7afa0cb
to
c2269a6
Compare
c2269a6
to
6fdd338
Compare
f402211
to
893acd9
Compare
a93ce04
to
685dd8a
Compare
ed164d8
to
b234d7f
Compare
dialyzer fix and mix file fix
On OTP 26, cacerts or cacertfile must be provided when verify_peer
- upgraded from 3.7.0-emqx-2 - 3.7.0.3 is a dependency upgrade: - lhttpc upgraded from 1.6.2 to 1.6.3 - lhttpc 1.6.3 fixed otp 26 compatibility
otherwise it will raise an exit exception
2ce5ae2
to
e45ed3d
Compare
@@ -66,7 +66,7 @@ | |||
{plt_location, "."}, | |||
{plt_prefix, "emqx_dialyzer"}, | |||
{plt_apps, all_apps}, | |||
{plt_extra_apps, [hocon]}, | |||
{plt_extra_apps, [hocon,mnesia,bcrypt,os_mon,pbkdf2,emqx_http_lib, recon]}, |
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.
Strange that we now need to do that, some of them probably need to be added to the emqx.app.src
?
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.
mainly for two reasons
- apps (e.g. mnesia) need to be added to all apps which calls mnesia, this is a bit annoying
- apps (e.g. hocon, bcrpyt(when using nif), emqx_http_lib) they are static libs dummy root sup or no root sup at all.
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.
Understood.
they are static libs dummy root sup or no root sup at all.
Even then, a lot of stuff treats entries in app.src as dependencies that need to be taken into account, not just dialyzer (i.e., relx).
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.
Sorry. this is actually just for the standalone app.
Will fix it in the next PR.
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.
and maybe bcrypt is not needed to be dynamic dep since we dropped windows.
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.
Fixes EMQX-11510
Release version: v/e5.4.0
Summary
Build 5.4 on OTP 26.
PR Checklist
Please convert it to a draft if any of the following conditions are not met. Reviewers may skip over until all the items are checked:
changes/(ce|ee)/(feat|perf|fix|breaking)-<PR-id>.en.md
filesChecklist for CI (.github/workflows) changes
changes/
dir for user-facing artifacts update