Skip to content

Conversation

@aagbsn
Copy link
Contributor

@aagbsn aagbsn commented Oct 29, 2025

by using the clickhouse_server fixture, the tests wait until clickhouse has bootstrapped and no longer spam WARNING messages into the test output.

@aagbsn aagbsn force-pushed the fix_clickhouse_not_bootstrapped_warning branch from 80479dd to 13dcbf7 Compare October 31, 2025 12:58
@aagbsn
Copy link
Contributor Author

aagbsn commented Oct 31, 2025

I added a similar delay to is_clickhouse_running in oonimeasurements/tests/conftest.py

@LDiazN
Copy link
Contributor

LDiazN commented Nov 3, 2025

Nice, thanks for the good work!
Just requested a small change, everything else LGTM, let's merge after that!

@aagbsn
Copy link
Contributor Author

aagbsn commented Nov 5, 2025

build dep package is failing

Run ./debops-ci --show-commands ci --bucket-name ooni-internal-deb
debops-ci version 2023-11-22.2
Set AWS_ACCESS_KEY_ID / AWS_SECRET_ACCESS_KEY env vars
Error: Process completed with exit code 1.

is this because I opened the PR?

@LDiazN
Copy link
Contributor

LDiazN commented Nov 7, 2025

build dep package is failing

Run ./debops-ci --show-commands ci --bucket-name ooni-internal-deb
debops-ci version 2023-11-22.2
Set AWS_ACCESS_KEY_ID / AWS_SECRET_ACCESS_KEY env vars
Error: Process completed with exit code 1.

is this because I opened the PR?

Yeah I think so, not sure if relevant considering that we don't use the deb packages for microservices and we're not updating the monolith code anyways

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.

2 participants