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

HELM: Updated docker compose for hagrid #8196

Merged
merged 5 commits into from
Nov 10, 2023

Conversation

madhavajay
Copy link
Collaborator

  • Added docker clone existing container
  • Improvements in seaweedfs container / build
  • Turned off tracing and added try except for telemetry import

Description

Please include a summary of the change, the motivation, and any additional context that will help others understand your PR. If it closes one or more open issues, please tag them as described here.

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

- Added docker clone existing container
- Improvements in seaweedfs container / build
- Turned off tracing and added try except for telemetry import
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@koenvanderveen
Copy link
Collaborator

it seems like the workers are not correctly connecting to the mongo store


  File "/usr/local/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "uvloop/loop.pyx", line 1517, in uvloop.loop.Loop.run_until_complete
  File "/root/.local/lib/python3.11/site-packages/uvicorn/server.py", line 68, in serve
    config.load()
  File "/root/.local/lib/python3.11/site-packages/uvicorn/config.py", line 473, in load
    self.loaded_app = import_from_string(self.app)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/uvicorn/importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/app/grid/main.py", line 13, in <module>
    from grid.api.router import api_router
  File "/app/grid/api/router.py", line 12, in <module>
    from grid.api.new.new import router as new_router
  File "/app/grid/api/new/new.py", line 5, in <module>
    from grid.core.node import worker
  File "/app/grid/core/node.py", line 71, in <module>
    worker = Domain(
             ^^^^^^^
  File "/app/syft/src/syft/node/node.py", line 318, in __init__
    self.init_stores(
  File "/app/syft/src/syft/node/node.py", line 693, in init_stores
    self.action_store = MongoActionStore(
                        ^^^^^^^^^^^^^^^^^
TypeError: __init__() should return None, not 'Err'

@koenvanderveen koenvanderveen merged commit a69a0ef into OpenMined:helm Nov 10, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants