Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Pin jsonpickle. See whole-tale/girder_wholetale#447 #137

Merged
merged 1 commit into from
Mar 22, 2021
Merged

Conversation

Xarthisius
Copy link
Contributor

"Fixes" issue seen on stage deployment:

[2021-03-21 19:09:01,870: INFO/ForkPoolWorker-4] Successfully built image registry.stage.wholetale.org/605792f48b171335abe6d3d2/1616352419@sha256:25ce82fdb6a492f91c6f08e682b30b5ad9df0015ffcd3d9d79bd1f5ed45d123d
[2021-03-21 19:09:02,036: WARNING/ForkPoolWorker-4] /usr/local/lib/python3.8/dist-packages/celery/app/trace.py:622: RuntimeWarning: Exception raised outside body: AttributeError("'dict' object has no attribute 'apply_async'"):
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/kombu/utils/objects.py", line 42, in __get__
    return obj.__dict__[self.__name__]
KeyError: '_apply_async'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/celery/app/trace.py", line 474, in trace_task
    _chsig.apply_async(
  File "/usr/local/lib/python3.8/dist-packages/celery/canvas.py", line 224, in apply_async
    _apply = self._apply_async
  File "/usr/local/lib/python3.8/dist-packages/kombu/utils/objects.py", line 44, in __get__
    value = obj.__dict__[self.__name__] = self.__get(obj)
  File "/usr/local/lib/python3.8/dist-packages/celery/canvas.py", line 516, in _apply_async
    return self.type.apply_async
AttributeError: 'dict' object has no attribute 'apply_async'

@Xarthisius Xarthisius requested a review from craig-willis March 22, 2021 15:11
@codecov
Copy link

codecov bot commented Mar 22, 2021

Codecov Report

Merging #137 (310b562) into master (36ddf9e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   62.36%   62.36%           
=======================================
  Files           9        9           
  Lines        1225     1225           
=======================================
  Hits          764      764           
  Misses        461      461           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36ddf9e...310b562. Read the comment docs.

@Xarthisius
Copy link
Contributor Author

@craig-willis Since it's a trivial change, I'm gonna merge it so that a new image is built and waiting for you when you get back to this.

@Xarthisius Xarthisius merged commit 13df457 into master Mar 22, 2021
@Xarthisius Xarthisius deleted the jsonpickle branch March 22, 2021 15:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant