Skip to content

raise oonimeasurements clickhouse user max memory per query#417

Open
aagbsn wants to merge 1 commit intomainfrom
raise_oonimeasurements_max_memory
Open

raise oonimeasurements clickhouse user max memory per query#417
aagbsn wants to merge 1 commit intomainfrom
raise_oonimeasurements_max_memory

Conversation

@aagbsn
Copy link
Copy Markdown
Contributor

@aagbsn aagbsn commented Apr 27, 2026

queries to explorer fail when filtering by category_code:
https://explorer.ooni.org/search?since=2026-03-29&until=2026-04-29&failure=false&category_code=GOVT

Server Response

{
  "name": "AxiosError",
  "message": "Network Error",
  "baseURL": "https://api.ooni.org",
  "url": "/api/v1/measurements",
  "params": {
    "category_code": "GOVT",
    "since": "2026-03-29",
    "until": "2026-04-29",
    "failure": false,
    "limit": 50,
    "order": "desc"
  }
}

From the service logs:

	2026-04-27T14:17:01.787Z
	raise packet.exception
	Link
	2026-04-27T14:17:01.788Z
	clickhouse_driver.errors.ServerException: Code: 241.
	Link
	2026-04-27T14:17:01.788Z
	DB::Exception: Memory limit (for query) exceeded: would use 481.78 MiB (attempt to allocate chunk of 4207200 bytes), maximum: 477.79 MiB.: (while reading c
	```

@aagbsn aagbsn force-pushed the raise_oonimeasurements_max_memory branch from f36d394 to d442c90 Compare April 27, 2026 14:51
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

Ansible Run Output 🤖

Ansible Playbook Recap 🔍



Ansible playbook output 📖success

Show Execution

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
[ERROR]: the role 'geerlingguy.docker' was not found in /home/runner/work/devops/devops/ansible/roles:/home/runner/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/home/runner/work/devops/devops/ansible
Origin: /home/runner/work/devops/devops/ansible/deploy-testlists.yml:16:7

14         node_exporter_host: "0.0.0.0"
15         node_exporter_options: ""
16     - role: geerlingguy.docker
         ^ column 7

Pusher @aagbsn
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Tue, 28 Apr 2026 13:49:48 GMT

@aagbsn aagbsn changed the title raise oonimeasuremenets max memory raise oonimeasurements clickhouse user max memory per query Apr 28, 2026
@aagbsn
Copy link
Copy Markdown
Contributor Author

aagbsn commented Apr 28, 2026

Is there a query optimization that should be done instead of raising this limit?

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.

1 participant