Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore: upgrade Elixir #2244

Merged
merged 8 commits into from
Oct 30, 2024
Merged

chore: upgrade Elixir #2244

merged 8 commits into from
Oct 30, 2024

Conversation

chasers
Copy link
Contributor

@chasers chasers commented Oct 30, 2024 •
edited
Loading

  • upgrades Elixir
  • adds system monitor for long_message_queue
  • disables compilation warning action (for now)

@chasers chasers requested a review from Ziinc October 30, 2024 20:12
@chasers
Copy link
Contributor Author

chasers commented Oct 30, 2024

Not obvious how to get rid of these:

warning: using map.field notation (without parentheses) to invoke function Opentelemetry.Proto.Collector.Trace.V1.TraceService.Service.__rpc_calls__() is deprecated, you must add parentheses instead: remote.function()
Warning:   lib/logflare_grpc/trace/server.ex:10: (module)

@chasers
Copy link
Contributor Author

chasers commented Oct 30, 2024

Latest version of grpc fixes it but protobuf version conflicts:

surgeventures/grpc@3439ee2

@chasers chasers merged commit dddb5e3 into main Oct 30, 2024
5 checks passed
@chasers chasers deleted the chore/upgrade-elixir branch October 30, 2024 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filipecabaco filipecabaco filipecabaco approved these changes

@Ziinc Ziinc Awaiting requested review from Ziinc

@hauleth hauleth Awaiting requested review from hauleth

Assignees

@chasers chasers

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chasers @filipecabaco