forked from lightningnetwork/lnd
-
Notifications
You must be signed in to change notification settings - Fork 0
Test threadpool #2
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
Open
pinheadmz
wants to merge
1,264
commits into
master
Choose a base branch
from
test-threadpool
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
graph/db: expand TestPopulateViaMigration for easy testing
remove expired key
…se-wait-wallet-sync-timeout lnwallet/test: increase wait for wallet to sync to `30s` to match with the returned error in case of timeout
This commit ensures that we start with the alias, node color, addresses, and features as advertised in the node's previous runtime. This approach maintains consistency in the node's advertised information across restarts.
This commit adds an itest that verify the behaviour of correctly reusing persisted node ann configs across restarts. It also ensures that the node ann configs are applied using the correct hierarchy.
…nnouncement lnd: use persisted node announcement settings across restarts
…eck-to-daily move pgp check to daily builds
This commit starts reusing test cases which are not dependant on the kv db backend. So they can be later used with the native db implementation as well.
we make the index assertion db independant so it is a noop for a future native sql backend. This allows us to reuse even more tests for the different db architectures.
This matches the same naming as used in the graph package.
…yments-code-05 Refactor Payment PR 5
…sInHorizon Catch bad gossip peer and fix `UpdatesInHorizon`
If CGO is enabled, Go tries to use the gold linker (-fuse-ld=gold), which is not installed. CGO was disabled, because it is not needed for golangci-lint.
…-height-hint fix height hint Zero issue in utxonursery
In this commit, we significantly expand the cleanup-space GitHub Actions workflow to free up substantially more disk space on GitHub runners. The previous cleanup only removed three large toolsets (dotnet, android, ghc), which should free ~14GB. This enhancement adds removal of several additional large packages and caches, bringing the total freed space to approximately 20-25GB. The specific additions include removing Swift and Julia language runtimes, the hosted toolcache directory, all Docker images, numerous large apt packages (aspnetcore, llvm, php, mongodb, mysql, azure-cli, browsers, and development tools), and various cache directories. We also add disk space reporting before and after cleanup to provide visibility into how much space is actually being freed during workflow runs. This enhancement was motivated by release builds running out of disk space when building for all 15 supported platforms (darwin, freebsd, linux, netbsd, openbsd, windows across multiple architectures). The sequential builds with verbose output were consuming more space than the basic cleanup could provide.
In this commit, we replace the basic inline cleanup command in the release workflow with the comprehensive cleanup-space action that was previously only used in the main CI workflow. The previous release workflow cleanup simply removed the hostedtoolcache directory, which freed only a few gigabytes and proved insufficient for multi-platform release builds. By switching to the cleanup-space action (now enhanced to free 20-25GB), the release workflow will have substantially more disk space available before beginning the build process. This should resolve the disk space exhaustion issues that were occurring during the Windows ARM build phase, which is one of the final platforms in the 15-platform build sequence.
In this commit, we add a call to "go clean -cache" after each platform build in the release script to prevent the Go build cache from accumulating unbounded disk space during the sequential 15-platform build process. When building for multiple platforms in sequence with "go build -v", Go creates intermediate build artifacts and caches compiled packages for each target platform. While this caching improves build performance within a single platform build, it causes the cache to grow substantially when building for many platforms sequentially. With 15 different platform/ architecture combinations, each with their own cached artifacts, this accumulation was contributing to the disk space exhaustion. By clearing the build cache after each platform completes, we prevent this unbounded growth while still allowing each individual platform build to benefit from caching during its own compilation. The module cache is preserved (we only clear the build cache), so dependencies don't need to be re-downloaded between platforms.
…space build: fix disk space exhaustion in release builds
build: bump version to v0.20.0 rc2
We also update the go.mod files to pin to Go 1.24.9.
Signed-off-by: saubyk <[email protected]>
My key recently expired, in this commit, we update the keys to the new refreshed version. These are the same keys, but with an expiry further out. Here's a clear sign of the latest Bitcoin block hash: -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 000000000000000000013215ef7c32bc0427f388fc83623affe712f388 -----BEGIN PGP SIGNATURE----- iHUEARYKAB0WIQQpYhJoGq3wVlaize6QUl997uCthgUCaPi6xwAKCRCQUl997uCt hpqNAQC5VnnbO6h/PjywGhU4LLRvH8SdgdDEMSc7xrtWd1vgPgD+IDrHqiAb+h38 ORBnUVJCVuZrPebtdnYXVQhII91eaw4= =WRbl -----END PGP SIGNATURE-----
`-multi_file` should be `--multi_file`
…expiry-update scripts/keys: update roasbeef keys with new expiry
…pstructure build: update Go 1.25.3 / 1.24.9
…re rescan This commit addresses a regression where Neutrino rescanning starts from an outdated height (~100k blocks behind) instead of using the current synced height. Root Cause: In commit 16a8b62, the initialization order was changed so that Chain Notifier starts before wallet syncing completes. This means the rescan begins using the stale height from BuildChainControl rather than the fully synced height. Old behavior (commit 1dfb5a0): 1. RPC server starts 2. Headers sync as part of daemon server 3. Chain Notifier starts after sync completes 4. Rescan begins from current (synced) height Current behavior (regression): 1. Chain Notifier starts in newServer (before RPC) 2. Wallet sync happens after RPC server starts 3. Rescan uses outdated height from BuildChainControl Solution: - Ensure headers are fully synced before starting the chain notifier, and after starting the RPC server. - Move chain notifier startup to its correct location after headers are fully synced. - Make sure the starting beat is lazily called after chain notifier started and before that starting beat result is used.
…ze-neutrino-rescan rpcserver: resolve root cause of premature wallet rescanning
…-doc-url-fix docs: fix build instructions doc url
release.sh: make sure GNU gzip is used
[docs] Typo recovery.md
docs: added a user guide for db migration
8ee52f2 to
c655b5d
Compare
25758d8 to
105add7
Compare
d0db0c4 to
105add7
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.