Skip to content

Add verbosity into tests for GolemBase #3434

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 35 commits into
base: golem-base/master
Choose a base branch
from
Open

Add verbosity into tests for GolemBase #3434

wants to merge 35 commits into from

Conversation

@krzysiekfonal
Copy link

@krzysiekfonal krzysiekfonal commented Jul 28, 2025

No description provided.

@nieznanysprawiciel
Enable more logs around publishing offer
c0e4052
@nieznanysprawiciel
Add logs in golem base sdk
4403b58
@nieznanysprawiciel
Log GolemBase containers logs
e5fd732
@nieznanysprawiciel
CLI get-offers returns expanded properties
52b5339
@nieznanysprawiciel
Update golem base sdk to solve problems with transactions that are no…
c39e43d 
…t delivered to sequencer
@nieznanysprawiciel
Increased timeout for sequencer transaction processing
ca70c91
@nieznanysprawiciel
Change timeout in market config to 120s
f156c7f
@nieznanysprawiciel
Display humantime properly(?)
bd5dd5f
@nieznanysprawiciel
Fix tests DiscoveryConfig initialization
f3ab25f
@nieznanysprawiciel
Discover if transaction was received by sequencer properly
4b6a720
@nieznanysprawiciel
GB sdk allows to configure entry publishing timeout and max retries; …
72e9992 
…Reconfigure
@nieznanysprawiciel
Configuration calculates timeout based on max retries and config timeout
16ccdba
@nieznanysprawiciel
Log transaction inclusion in block
8969dad
@nieznanysprawiciel
Include fix to race condition when checking block inclusion
2637557
@nieznanysprawiciel
Try waiting 300s with no retries
7be2416
@nieznanysprawiciel
Attempt transaction retry with different gas price
54f43c3
@nieznanysprawiciel
Expired Offers return 410 Gone; Silence warnings in case of expiration
dbea6a9
@nieznanysprawiciel
Minimum transaction price bump
c26d0d5
@nieznanysprawiciel
Fix bumping both gas fees
11c3683
@nieznanysprawiciel
More radical gas price bump
3cb04bf
@nieznanysprawiciel
Reduce tests to signle one
e5af623
@nieznanysprawiciel
Retry transaction when we get error sending request
f2f5f6d
@nieznanysprawiciel
Fix problems with RPC switching
9958f25
@nieznanysprawiciel
List previous transactions in case we timouted
2f8b29a
@nieznanysprawiciel
Merge branch 'golem-base/master' of github.com:golemfactory/yagna int…
798c50d 
…o golem-base/fix-market-tests-timeouts
@nieznanysprawiciel
Increase timeout waiting for offer subscribed in goth
e2a28ed
@nieznanysprawiciel
Attempt to publish another offer if one attempt timed out
e530761
@nieznanysprawiciel
Check on CI if we are able to publish offer from different id
facc089
@nieznanysprawiciel
More clear error message when sending second test transaction
9c83900
@krzysiekfonal krzysiekfonal requested a review from a team as a code owner July 28, 2025 15:40
@CLAassistant
Copy link

CLAassistant commented Jul 28, 2025 •
edited
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ nieznanysprawiciel
❌ krzysiekfonal
You have signed the CLA already but the status is still pending? Let us recheck it.

@krzysiekfonal krzysiekfonal force-pushed the golem-base/test-ci-bug-analysis branch from e7b6d0e to ce2b880 Compare July 29, 2025 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@krzysiekfonal @CLAassistant @nieznanysprawiciel