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

deploy note editing branch onto staging system #412

Closed
9 tasks done
btbonval opened this issue Feb 27, 2015 · 10 comments
Closed
9 tasks done

deploy note editing branch onto staging system #412

btbonval opened this issue Feb 27, 2015 · 10 comments
Assignees
Labels

Comments

@btbonval
Copy link
Member

  • rebase master into note-editing branch
  • run note-editing branch on VM
  • test
  • push note-editing branch to Beta
  • test
  • email test users
  • give admin access to test users
  • they test
  • merge note-editing into master

tests for data on Beta can be found in this comment:
#396 (comment)

@btbonval btbonval self-assigned this Feb 27, 2015
@btbonval
Copy link
Member Author

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.

@btbonval
Copy link
Member Author

Running worker by itself is far more successful. Finally my uploaded notes are running again.

@btbonval
Copy link
Member Author

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.

@btbonval
Copy link
Member Author

This branch has been prepared with a ton of fixes and master rebased in, tested, and ready for push to staging:
https://github.com/FinalsClub/karmaworld/tree/note-editing-merge-more

@btbonval
Copy link
Member Author

New error on the Heroku console:

-----> Noticed cffi. Bootstrapping libffi.
curl: /app/.heroku/vendor/lib/libsasl2.so.2: no version information available (required by /usr/lib/libldap_r-2.4.so.2)

gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Exiting with failure status due to previous errors

 !     Push rejected, failed to compile Python app

To [email protected]:karmanotes-beta.git
 ! [remote rejected] note-editing-merge-more -> master (pre-receive hook declined)
error: failed to push some refs to '[email protected]:karmanotes-beta.git'

I remember seeing this comment here to make buildpack install libffi
https://github.com/FinalsClub/karmaworld/blob/note-editing-merge-more/requirements.txt#L25

It looks like that has been in place for ages.
2d14f74

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.
https://github.com/FinalsClub/karmaworld/blob/note-editing-merge-more/requirements.txt#L1

Nothing changed in buildpack or upstream version of buildpack (or in upstream upstream version of buildpack).

@btbonval
Copy link
Member Author

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?

@btbonval
Copy link
Member Author

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:
https://blog.heroku.com/archives/2015/2/24/Heroku%20Connect%20Demo%20Edition%20Now%20Available%20via%20Heroku%20Button

It's just a new add-on.

@btbonval
Copy link
Member Author

Spawning #417 to track libffi error

@btbonval
Copy link
Member Author

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

@btbonval
Copy link
Member Author

Deployed to master a few days ago. This issue is completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant