-
Notifications
You must be signed in to change notification settings - Fork 517
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
[Hotfix]use production build profile for the archive binary #8091
Conversation
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.
yeah that works for me
Makefile
Outdated
@@ -83,7 +83,7 @@ build: git_hooks reformat-diff libp2p_helper | |||
|
|||
build_archive: git_hooks reformat-diff | |||
$(info Starting Build) | |||
ulimit -s 65532 && (ulimit -n 10240 || true) && dune build src/app/archive/archive.exe --profile=$(DUNE_PROFILE) | |||
ulimit -s 65532 && (ulimit -n 10240 || true) && dune build src/app/archive/archive.exe --profile=testnet_postake_medium_curves |
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.
Can you change the profiles in the CI jobs instead? This will clobber the build artifacts when using this during development.
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.
I think the change in 4d26dac should do
…ina into fix/archive-build-profile
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.
Looks good to me, thanks!
Tested on one of our archive nodes (archive1) and blocks (with snarks in them) are getting added. Also verified that coinbase amount in the database is 720 |
!approved-for-mainnet |
DUNE_PROFILE is not being set when building the archive.exe causing versioning mismatch between daemon and the archive process.
Fixes #8082
Fixes #8080