Skip to content

Conversation

@emiliecowen
Copy link
Contributor

  • handle deleted mm users; skip document creation
  • end-to-end and pytests passing
  • requires db re-init, but existing environment is okay

@emiliecowen emiliecowen self-assigned this Oct 15, 2025
@emiliecowen
Copy link
Contributor Author

@dygwon review this one second

@emiliecowen
Copy link
Contributor Author

@dygwon, if you see the follow error during init:
ERROR: | 2026-01-21 17:02:37 | https://chat.il4.dso.mil/api/v4/users/9ok6ijnyntnbmckde1xqy5e1ky request failed: 404 https://chat.il4.dso.mil/api/v4/users/9ok6ijnyntnbmckde1xqy5e1ky request failed: 404 Traceback (most recent call last): File "<frozen runpy>", line 198, in _run_module_as_main File "<frozen runpy>", line 88, in _run_code File "/Users/em23761/repos/caitt/c3po-model-server_wip/app/initial_data.py", line 565, in <module> main() File "/Users/em23761/repos/caitt/c3po-model-server_wip/app/initial_data.py", line 551, in main init_large_objects(db) File "/Users/em23761/repos/caitt/c3po-model-server_wip/app/initial_data.py", line 461, in init_large_objects doc_objs = init_mattermost_documents(db, bot_obj) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Users/em23761/repos/caitt/c3po-model-server_wip/app/initial_data.py", line 434, in init_mattermost_documents user_obj = crud_mattermost.get_or_create_mm_user_object(db, user_id=uid) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/Users/em23761/repos/caitt/c3po-model-server_wip/app/mattermost/crud/crud_mattermost.py", line 379, in get_or_create_mm_user_object raise HTTPException( fastapi.exceptions.HTTPException: 422: Mattermost user not found init script finished
or these test failures:
tests/aimodels/bertopic/integration/test_train.py FFF [ 9%]

changes in this PR should resolve it. so getting this merged into main is a priority, and then I'll rebase the remaining PRs for you.

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