Skip to content

[release] v0.8.1 [Falkor]#600

Merged
ca333 merged 47 commits intomasterfrom
dev
Nov 14, 2023
Merged

[release] v0.8.1 [Falkor]#600
ca333 merged 47 commits intomasterfrom
dev

Conversation

@ca333
Copy link
Copy Markdown

@ca333 ca333 commented Nov 3, 2023

No description provided.

smk762 and others added 30 commits July 1, 2023 10:14
the cross-compilation process in the rust komodo recipe
differs slightly from the current native_rust recipe used
in Zcash. therefore, we have commented out all invocations
of the "update_stdlib_hash" function until the komodo rust
recipe is updated.
todo: we still need rework rust integration, crates build, etc.
as the bootstrap_url (https://sirseven.me/share/bootstrap.tar.gz) is
no more accessible we will temp disable these jobs, until revise/rethink
needed steps (mean, CC is kind of EOL, do we still need these tests
here?) and re-create chains, if needed.
temp disable other tests to make debugging simpler.

TODO: don't forget to enable them again.
TODO: add needed checks, also add komodoevents file check in all tests
__BYTE_ORDER != __LITTLE_ENDIAN
move extrabuf from condition block scope to the main komodo_args
scope to avoid "dangling" pointer and garbage in buffer after exit
from condition block. add proper init of variables.
For example, the group of test_block tests changes the value
of IsInitialBlockDownload() to false after the first call
to generateBlock. As a result, when komodo_connectblock is called,
komodo_validate_chain should return 1 for the LegacyEvents tests.
Additionally, if we are operating with various data directories
between different tests, the static file pointer fp in komodo.cpp
should be cleared between these tests to avoid writing to the same
file in different tests.
@smk762
Copy link
Copy Markdown

smk762 commented Nov 8, 2023

The Actions workflows output for linux fail to launch in Ubuntu 20.04

For the CI workflows:

./komodod: /lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by ./komodod)
./komodod: /lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by ./komodod)
./komodod: /lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.13' not found (required by ./komodod)
./komodod: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by ./komodod)
./komodod: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by ./komodod)
./komodod: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by ./komodod)

For the CD workflow

./komodod: /lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by ./komodod)

This (CD ouput) is pushed to https://hub.docker.com/r/komodoofficial/komodo and needed for use in the notary node docker setups. I'll see if I can find a resolution.

Note: Building locally on ubuntu20.04 following the readme resulted in usable binaries.

fix komodo_args, add ParseArgumentsTests, fix LegacyEvents test
@DeckerSU
Copy link
Copy Markdown

DeckerSU commented Nov 8, 2023

The Actions workflows output for linux fail to launch in Ubuntu 20.04

I believe that the komodo_linux_ci.yml workflow is failing because it's running on ubuntu-latest, which is currently ubuntu-22.04. The komodod_cd.yml workflow, however, is using ubuntu-20.04. To resolve this issue, we should either use Ubuntu 20.04 in both workflows or build for multiple GLIBC versions, as I have done in the KomodoOcean repository. The KomodoOcean builds successfully with Xenial (and works on Xenial and later releases) as well as Focal. The release binaries include both builds.

Copy link
Copy Markdown

@DeckerSU DeckerSU left a comment

Choose a reason for hiding this comment

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

LGTM. We might want to consider changing runs-on: ubuntu-latest to runs-on: ubuntu-20.04 in the komodo_linux_ci.yml file to align it with komodod_cd.yml.

@smk762
Copy link
Copy Markdown

smk762 commented Nov 10, 2023

https://github.com/KomodoPlatform/komodo/actions/runs/6799081197 output fails to launch for Mac with error below :
image

maybe we need to use macos-11 for builds instead of macos-latest. I'll test this and PR CI/CD updates accordingly.

Device specs:
image

@DeckerSU
Copy link
Copy Markdown

DeckerSU commented Nov 10, 2023

It's because komodod file content is not even Mach-O 64-bit executable x86_64:

image

I remember that this issue exists due to incompatibility or a mismatch between the Rust and XCode versions. Switching to macos-11 should resolve the problem.

@ca333
Copy link
Copy Markdown
Author

ca333 commented Nov 14, 2023

maybe we need to use macos-11 for builds instead of macos-latest. I'll test this and PR CI/CD updates accordingly.

yes best is to build with "older" (sec patched / supported) version for wider (backwards) compatibility

@ca333 ca333 merged commit d9144d0 into master Nov 14, 2023
@DeckerSU
Copy link
Copy Markdown

maybe we need to use macos-11 for builds instead of macos-latest. I'll test this and PR CI/CD updates accordingly.

Issue resolved. The root cause was identified as bsdtar on MacOS, which was causing file corruption during archive creation.

who-biz pushed a commit to who-biz/komodo that referenced this pull request Jun 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants