-
Notifications
You must be signed in to change notification settings - Fork 6
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
deploy note editing branch onto staging system #412
Comments
I think this shell wrapper does not close out celery properly. This is an argument for pursuing a single python for both celery worker and celery beat, as noted in this comment: #407 (comment) I cannot seem to get this celerywrapper script to 100% of the time process added notes. Most of the time, I find it just runs through beat stuff and the worker does not process new notes, though I see the worker has clearly run. |
Running worker by itself is far more successful. Finally my uploaded notes are running again. |
Okay, bunch of tiny fixes implemented to get this ready for staging. Ran through a bunch of tests by hand and they passed. Cannot run proper tests because #311 is still an issue in my life. Going to sit on this overnight and push to staging sometime on Friday. |
This branch has been prepared with a ton of fixes and master rebased in, tested, and ready for push to staging: |
New error on the Heroku console:
I remember seeing this comment It looks like that has been in place for ages. I had added cffi as an explicit requirement because I was version locking prereqs. Perhaps that threw something out of whack. I removed the line specifying cffi. It didn't help. Nothing changed in buildpack or upstream version of buildpack (or in upstream upstream version of buildpack). |
Ack, something changed out from under us. beta is currently running the master branch. I changed a file slightly (README.md) to trigger reloading the master branch on the beta. Same error using the code its already running. The error has nothing to do with code we have changed. The buildpack doesn't seem to have changed. So something is different with Heroku? |
Last successful deployment to beta appears to be 8 days ago. The only news on the blog in the past nine days is one entry: It's just a new add-on. |
Spawning #417 to track libffi error |
Deployed to beta and Brown students are informed for their testing purposes. Let's wait and see what they say. Branch deployed to beta is https://github.com/FinalsClub/karmaworld/tree/note-editing-merge-more |
Deployed to master a few days ago. This issue is completed. |
tests for data on Beta can be found in this comment:
#396 (comment)
The text was updated successfully, but these errors were encountered: