Skip to content

Commit 12883c3

Browse files
committed
Don't double send enable_modules and debug_mode in kwargs..
This broke non-disti-mode actor tree spawn / runtime, seemingly because the cli entrypoint for a `piker chart` also sends these values down through the call stack independently? Pretty sure we don't need to send the `enable_modules` from the chart actor anyway.
1 parent 8ceaa27 commit 12883c3

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

piker/service/_actor_runtime.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,6 @@ async def open_pikerd(
174174
# having the root have more then permissions to
175175
# spawn other specialized daemons I think?
176176
enable_modules=_root_modules,
177-
178177
loglevel=loglevel,
179178
debug_mode=debug_mode,
180179
registry_addr=registry_addr,
@@ -330,7 +329,6 @@ async def maybe_open_pikerd(
330329
# configured address
331330
async with open_pikerd(
332331
loglevel=loglevel,
333-
debug_mode=kwargs.get('debug_mode', False),
334332
registry_addr=registry_addr,
335333

336334
# ahabd (docker super) specific controls

piker/ui/cli.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -181,9 +181,6 @@ def chart(
181181
'debug_mode': pdb,
182182
'loglevel': tractorloglevel,
183183
'name': 'chart',
184-
'enable_modules': [
185-
'piker.clearing._client'
186-
],
187184
'registry_addr': config.get('registry_addr'),
188185
},
189186
)

0 commit comments

Comments
 (0)