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.

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

build linux x86-64 with reasonable glibc #1733

Merged
merged 6 commits into from
Mar 24, 2023
Merged

Conversation

onur-ozkan
Copy link
Member

@onur-ozkan onur-ozkan commented Mar 24, 2023

Using ubuntu-latest builds mm2 with gcc 12, which breaks compatibility with older versions. We must used a version that is not too old/recent(which this pr provides already) to provide wide support.

Resolves #1732

@cipig
Copy link
Member

cipig commented Mar 24, 2023

thanks, works now on debian 11
on debian 10 it doesn't: /lib/x86_64-linux-gnu/libm.so.6: version GLIBC_2.29 not found (latest on debian 10 is GLIBC_2.28)
do we want/need it?

@onur-ozkan
Copy link
Member Author

Sadly this is expected.

I will try some other workarounds to make it work for deb10 as well.

@onur-ozkan onur-ozkan force-pushed the static-ubuntu-version branch 8 times, most recently from b64f219 to 78a7958 Compare March 24, 2023 12:24
@cipig
Copy link
Member

cipig commented Mar 24, 2023

http://54.170.62.22:8000/static-ubuntu-version/mm2_-linux-x86-64.zip (has no build id) works on debian 10, i also did a swap
saw this panick on console 15 minutes later

24 13:41:43, common:491] panicked at 'attempt to add with overflow', /github/home/.cargo/registry/src/github.com-1ecc6299db9ec823/memory-db-0.29.0/src/lib.rs:455:9
24 13:41:43, common:492] backtrace
  panicking.rs:575] rust_begin_unwind 0x562ed8d793d1
  panicking.rs:65] core::panicking::panic_fmt::h68463fb299862f1f 0x562ed4381a52
  panicking.rs:115] core::panicking::panic::hf3ad889fdb69a756 0x562ed4381b2c
  lib.rs:455] <memory_db::MemoryDB<H,KF,T,M> as parity_util_mem::malloc_size::MallocSizeOf>::size_of::hcd777819d5c698d9 0x562ed519ffc1
  lib.rs:92] parity_util_mem::malloc_size::h99ccbd58f4dc8cb8 0x562ed52b9416
  lp_ordermatch.rs:2438] mm2_main::mm2::lp_ordermatch::collect_orderbook_metrics::hb4bcb975172132dd 0x562ed4e8f29a
  lp_ordermatch.rs:3089] mm2_main::mm2::lp_ordermatch::lp_ordermatch_loop::{{closure}}::h0aa4ecc1e9fedd63 0x562ed4776061
  mod.rs:91] <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll::h255141eb8c73a1d5 0x562ed4b3a571
  abortable.rs:159] <futures_util::abortable::Abortable<Fut> as core::future::future::Future>::poll::{{closure}}::ha0608944163af8a3 0x562ed56a4f21
  abortable.rs:133] futures_util::abortable::Abortable<T>::try_poll::h6b66f494254a18e4 0x562ed56a0e7a
  abortable.rs:159] <futures_util::abortable::Abortable<Fut> as core::future::future::Future>::poll::h30837fbd26dfa70e 0x562ed56a4746
  future.rs:124] <core::pin::Pin<P> as core::future::future::Future>::poll::hc64d99bdcefadbc7 0x562ed8181ed8
  mod.rs:562] futures_util::future::future::FutureExt::poll_unpin::h77132375be0ad951 0x562ed8181847
  select.rs:103] <futures_util::future::select::Select<A,B> as core::future::future::Future>::poll::hd9973d2bea2803a8 0x562ed8181430
  abortable_queue.rs:99] common::executor::abortable_system::abortable_queue::WeakSpawner::spawn_with_settings_impl::{{closure}}::h53461effd44fffee 0x562ed4f68ee1
  mod.rs:91] <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll::hd087cf910c65cd6b 0x562ed4b54f91
  unwind_safe.rs:271] <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::h077ce300b912222b 0x562ed5728089
  ??:0] __rust_try 0x562ed5727faa
  task.rs:42] <tokio::runtime::blocking::task::BlockingTask<T> as core::future::future::Future>::poll::hee26c126377807ef 0x562ed74da9cc
  unwind_safe.rs:271] <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::hae7938a9c9c2feff 0x562ed75be7e9
  ??:0] __rust_try 0x562ed75bb9ba
  unwind_safe.rs:271] <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::h2093242db4b8d457 0x562ed8a2db21
  ??:0] __rust_try 0x562ed8a51c0a
  ??:0] start_thread 0x7f0cf005afa2
  ??:0] clone 0x7f0cefe0206e
Virtual memory addresses of /home/flow/atomicDEX-API/target/release/mm2  PHDR:0x562ed3d88040  INTERP:0x562ed3d882e0  LOAD:0x562ed3d88000  LOAD:0x562ed437e000  LOAD:0x562ed8da8000  LOAD:0x562eda093f90  DYNAMIC:0x562eda2534e0  NOTE:0x562ed3d882fc  TLS:0x562eda093f90  GNU_EH_FRAME:0x562ed9287798  GNU_STACK:0x562ed3d88000  GNU_RELRO:0x562eda093f90

mm2 did not crash, it still runs... related?

@onur-ozkan
Copy link
Member Author

http://54.170.62.22:8000/static-ubuntu-version/mm2_-linux-x86-64.zip (has no build id) works on debian 10, i also did a swap saw this panick on console 15 minutes later

24 13:41:43, common:491] panicked at 'attempt to add with overflow', /github/home/.cargo/registry/src/github.com-1ecc6299db9ec823/memory-db-0.29.0/src/lib.rs:455:9
24 13:41:43, common:492] backtrace
  panicking.rs:575] rust_begin_unwind 0x562ed8d793d1
  panicking.rs:65] core::panicking::panic_fmt::h68463fb299862f1f 0x562ed4381a52
  panicking.rs:115] core::panicking::panic::hf3ad889fdb69a756 0x562ed4381b2c
  lib.rs:455] <memory_db::MemoryDB<H,KF,T,M> as parity_util_mem::malloc_size::MallocSizeOf>::size_of::hcd777819d5c698d9 0x562ed519ffc1
  lib.rs:92] parity_util_mem::malloc_size::h99ccbd58f4dc8cb8 0x562ed52b9416
  lp_ordermatch.rs:2438] mm2_main::mm2::lp_ordermatch::collect_orderbook_metrics::hb4bcb975172132dd 0x562ed4e8f29a
  lp_ordermatch.rs:3089] mm2_main::mm2::lp_ordermatch::lp_ordermatch_loop::{{closure}}::h0aa4ecc1e9fedd63 0x562ed4776061
  mod.rs:91] <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll::h255141eb8c73a1d5 0x562ed4b3a571
  abortable.rs:159] <futures_util::abortable::Abortable<Fut> as core::future::future::Future>::poll::{{closure}}::ha0608944163af8a3 0x562ed56a4f21
  abortable.rs:133] futures_util::abortable::Abortable<T>::try_poll::h6b66f494254a18e4 0x562ed56a0e7a
  abortable.rs:159] <futures_util::abortable::Abortable<Fut> as core::future::future::Future>::poll::h30837fbd26dfa70e 0x562ed56a4746
  future.rs:124] <core::pin::Pin<P> as core::future::future::Future>::poll::hc64d99bdcefadbc7 0x562ed8181ed8
  mod.rs:562] futures_util::future::future::FutureExt::poll_unpin::h77132375be0ad951 0x562ed8181847
  select.rs:103] <futures_util::future::select::Select<A,B> as core::future::future::Future>::poll::hd9973d2bea2803a8 0x562ed8181430
  abortable_queue.rs:99] common::executor::abortable_system::abortable_queue::WeakSpawner::spawn_with_settings_impl::{{closure}}::h53461effd44fffee 0x562ed4f68ee1
  mod.rs:91] <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll::hd087cf910c65cd6b 0x562ed4b54f91
  unwind_safe.rs:271] <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::h077ce300b912222b 0x562ed5728089
  ??:0] __rust_try 0x562ed5727faa
  task.rs:42] <tokio::runtime::blocking::task::BlockingTask<T> as core::future::future::Future>::poll::hee26c126377807ef 0x562ed74da9cc
  unwind_safe.rs:271] <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::hae7938a9c9c2feff 0x562ed75be7e9
  ??:0] __rust_try 0x562ed75bb9ba
  unwind_safe.rs:271] <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::h2093242db4b8d457 0x562ed8a2db21
  ??:0] __rust_try 0x562ed8a51c0a
  ??:0] start_thread 0x7f0cf005afa2
  ??:0] clone 0x7f0cefe0206e
Virtual memory addresses of /home/flow/atomicDEX-API/target/release/mm2  PHDR:0x562ed3d88040  INTERP:0x562ed3d882e0  LOAD:0x562ed3d88000  LOAD:0x562ed437e000  LOAD:0x562ed8da8000  LOAD:0x562eda093f90  DYNAMIC:0x562eda2534e0  NOTE:0x562ed3d882fc  TLS:0x562eda093f90  GNU_EH_FRAME:0x562ed9287798  GNU_STACK:0x562ed3d88000  GNU_RELRO:0x562eda093f90

mm2 did not crash, it still runs... related?

I haven't finish the fix, but I don't think it's related

@onur-ozkan onur-ozkan changed the title update ubuntu image version for CI build linux x86-64 with reasonable glibc Mar 24, 2023
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the fast fix! I like the added cancel-in-progress CI config a lot :)

@shamardy shamardy merged commit 759d180 into dev Mar 24, 2023
@shamardy shamardy deleted the static-ubuntu-version branch March 24, 2023 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants