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

Upgrade BigchainDB Server to 0.9 and COALA IP-related dependencies #25

Merged
merged 9 commits into from
Feb 17, 2017

Conversation

sohkai
Copy link
Contributor

@sohkai sohkai commented Feb 10, 2017

Adds dependency links until new versions of pycoalaip and pycoalaip-bigchaindb are released.

TODO:

  • Remove dependency links and drop 8adaae9 once transfers and entity history look ups are implemented

BDB_NODE_HOST: "${BDB_NODE_HOST}"
BDB_NODE_PORT: "${BDB_NODE_PORT}"
BDB_NODE_HOST: bigchaindb
BDB_NODE_PORT: 9984
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missed this earlier when it was changed, but this shouldn't be breaking anything. bigchaindb is the service name of the container running the BigchainDB server so docker should know to hook up to it directly.

@sohkai sohkai mentioned this pull request Feb 10, 2017
@@ -1,6 +1,6 @@
FROM python:3.5

RUN apt-get update && apt-get install -y vim
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥇

setup.py Outdated
'bigchaindb-driver==0.1.2',
'coalaip==0.0.1.dev3',
'coalaip-bigchaindb==0.0.1.dev3',
'bigchaindb~=0.9.0',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, we're using ~, I'd still prefer it reading: 'bigchaindb~=0.9.1'

@sohkai sohkai force-pushed the upgrade-coalaip branch 2 times, most recently from 6d98f90 to 6951455 Compare February 17, 2017 20:37
@sohkai sohkai merged commit a7832ea into master Feb 17, 2017
@sohkai sohkai mentioned this pull request Feb 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants