livebook-dev/livebook
1
1
Fork 0
mirror of https://github.com/livebook-dev/livebook.git synced 2025-01-28 01:39:15 +08:00
Code Issues Projects Releases Packages Wiki Activity

Fix node resolution in CI (#173)

Browse source 
* Limit the number of Elixir nodes spawned at the same time in tests

* Increase timeout

* Update CI to Elixir 1.12

* Terminate unused runtimes

* Increase timeout on the spawning node side

* Cleanup

* Run test with --trace

* Up

* Up

* Run tests with long node names

* Just longnames CI

* Move CI dependent config to config/test.exs
This commit is contained in:
Jonatan Kłosko 2021-04-14 14:28:00 +02:00 committed by GitHub
parent 539cb8e2b1
commit f31428739f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
config/runtime.exs
View file

@ -3,7 +3,7 @@ import Config
# Configure the type of names used for distribution and the node name.
# By default a random short name is used.
# config :livebook, :node, {:shortnames, "livebook"}
# config :livebook, :node, {:longnames, "livebook@127.0.0.1"}
# config :livebook, :node, {:longnames, :"livebook@127.0.0.1"}
if config_env() == :prod do
# We don't need persistent session, so it's fine to just

6
config/test.exs
View file

@ -10,3 +10,9 @@ config :livebook, LivebookWeb.Endpoint,
config :logger, level: :warn
config :livebook, :token_authentication, false
# Use longnames when running tests in CI, so that no host resolution is required,
# see https://github.com/elixir-nx/livebook/pull/173#issuecomment-819468549
if System.get_env("CI") == "true" do
config :livebook, :node, {:longnames, :"livebook@127.0.0.1"}
end