Skip to content
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

[AUD-452] - Fix ipld blocks table init #1335

Merged
merged 1 commit into from
Mar 22, 2021

Conversation

vicky-g
Copy link
Contributor

@vicky-g vicky-g commented Mar 22, 2021

Description

What is the purpose of this PR? What is the current behavior? New behavior? Relevant links (e.g. Trello) and/or information pertaining to PR?

Similar to #1191

Tests

List the manual tests and repro instructions to verify that this PR works as anticipated. Include log analysis if possible.
❗ If this change impacts clients, make sure that you have tested the clients ❗

Before:

Error msg:

{"levelno": 40, "level": "ERROR", "msg": "IPLDBLACKLIST | Fatal error in main loop\nTraceback (most recent call last):\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 211, in update_ipld_blacklist_task\n    initialize_blacklist_blocks_table_if_necessary(db)\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 28, in initialize_blacklist_blocks_table_if_necessary\n    assert (\nAssertionError: Corrupted DB State - Expect single row marked as current", "timestamp": "2021-03-22 18:28:10,809"}
{"levelno": 40, "level": "ERROR", "msg": "Task update_ipld_blacklist[23f7e973-a49d-4617-b319-ad571a8a7ac4] raised unexpected: AssertionError('Corrupted DB State - Expect single row marked as current')\nTraceback (most recent call last):\n  File \"/usr/local/lib/python3.8/site-packages/celery/app/trace.py\", line 382, in trace_task\n    R = retval = fun(*args, **kwargs)\n  File \"/usr/local/lib/python3.8/site-packages/celery/app/trace.py\", line 641, in __protected_call__\n    return self.run(*args, **kwargs)\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 325, in update_ipld_blacklist_task\n    raise e\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 211, in update_ipld_blacklist_task\n    initialize_blacklist_blocks_table_if_necessary(db)\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 28, in initialize_blacklist_blocks_table_if_necessary\n    assert (\nAssertionError: Corrupted DB State - Expect single row marked as current", "timestamp": "2021-03-22 18:28:10,810", "data": {"hostname": "celery@b01bb43a3d83", "id": "23f7e973-a49d-4617-b319-ad571a8a7ac4", "name": "update_ipld_blacklist", "exc": "AssertionError('Corrupted DB State - Expect single row marked as current')", "traceback": "Traceback (most recent call last):\n  File \"/usr/local/lib/python3.8/site-packages/celery/app/trace.py\", line 382, in trace_task\n    R = retval = fun(*args, **kwargs)\n  File \"/usr/local/lib/python3.8/site-packages/celery/app/trace.py\", line 641, in __protected_call__\n    return self.run(*args, **kwargs)\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 325, in update_ipld_blacklist_task\n    raise e\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 211, in update_ipld_blacklist_task\n    initialize_blacklist_blocks_table_if_necessary(db)\n  File \"/audius-discovery-provider/src/tasks/index_blacklist.py\", line 28, in initialize_blacklist_blocks_table_if_necessary\n    assert (\nAssertionError: Corrupted DB State - Expect single row marked as current\n", "args": "[]", "kwargs": "{}", "description": "raised unexpected", "internal": false}}

Screen Shot 2021-03-22 at 11 33 44 AM

Now:
Screen Shot 2021-03-22 at 11 33 52 AM

Added a user and confirmed the indexing worked as expected.

@vicky-g vicky-g merged commit 55d5802 into master Mar 22, 2021
@vicky-g vicky-g deleted the vg-fix-ipld-blocks-table-init-2 branch March 22, 2021 19:20
@AudiusProject AudiusProject deleted a comment from linear bot Sep 11, 2023
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.

None yet

2 participants