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

Add instructions for installing required software packages #261

Merged
merged 2 commits into from
Jun 24, 2016

Conversation

asaunier
Copy link
Member

@asaunier asaunier commented Apr 26, 2016

Similar to c2corg/v6_ui#324


* Python 3.4
* PostgreSQL 9.4 / PostGIS 2.1
* virtualenv (1.7 or higher)
Copy link
Member

Choose a reason for hiding this comment

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

  • ElasticSearch 2.3.2
  • Redis 2.8

Copy link
Member Author

Choose a reason for hiding this comment

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

OK thanks.
I have added those components to the list at 727756d
Not sure what packages should be installed though.

Copy link
Member

Choose a reason for hiding this comment

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

For redis: redis-server.
And for ElasticSearch the DEB can be downloaded from here: https://www.elastic.co/downloads/past-releases/elasticsearch-2-3-2

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, updated!

Copy link
Member Author

Choose a reason for hiding this comment

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

I see that on the demo server ES 2.1.0 is still used. Should we upgrade to 2.3.2 on this server?

Copy link
Member

Choose a reason for hiding this comment

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

That would make sense, yes.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK I will open a ticket for the sysadmins.
I don't know if it is related but when I try to run the indexing on my instance on the dev server, the load skyrockets (I have succeeded to index the main DB though...).

Copy link
Member

Choose a reason for hiding this comment

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

I don't know if it is related but when I try to run the indexing on my instance on the dev server, the load skyrockets (I have succeeded to index the main DB though...).

Thanks for noticing (my work station is too powerful...). See #281

@asaunier asaunier mentioned this pull request May 23, 2016
@asaunier
Copy link
Member Author

I have just added GEOS to the requirements, see 27430fa See also c2corg/v6_ui#446

I just wonder if this was necessary since GEOS is required by PostGIS I think and should be pulled when installing PostGIS, shouldn't it?

@tsauerwein
Copy link
Member

I just wonder if this was necessary since GEOS is required by PostGIS I think and should be pulled when installing PostGIS, shouldn't it?

PostGIS does not need the package with the header files (libgeos-dev) which seems to be required by Shapely.

@asaunier
Copy link
Member Author

OK, then my recent change makes sense. Thanks for the tip!

@asaunier asaunier changed the title [WIP] Add instructions for installing required software packages Add instructions for installing required software packages Jun 24, 2016
@@ -23,6 +23,7 @@ phpserialize==1.3.0
kombu==3.0.35
redis==2.10.5
apscheduler==3.1.0
requests==2.9.1
Copy link
Member

Choose a reason for hiding this comment

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

You can then remove it from dev-requirements.txt.

And yes, it is correct. It should be listed under requirements.txt.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, you answer my question below :P I will remove it from dev-requirements.txt.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK I have updated the PR. Looks good?

@asaunier
Copy link
Member Author

@tsauerwein Your link http://stackoverflow.com/a/31234304 was the good one to fix the "make install" crash with bcrypt. With the packages now listed in the README make -f config/alex install succeeds on my laptop.

@gberaudo I had to add requests=2.9.1 (version that I have found in my instance on the dev server) with 3a0e4d9 else make -f config/alex serve fails because:

File "/home/alex/Desktop/v6_api/c2corg_api/security/discourse_client.py", line 1, in
from pydiscourse.client import DiscourseClient
File "/home/alex/Desktop/v6_api/.build/venv/lib/python3.4/site-packages/pydiscourse/client.py", line 4, in
import requests
ImportError: No module named 'requests'

Do you think it's OK? Should requests have been install in some other way?

The PR is ready for reviewing.

@tsauerwein
Copy link
Member

Looks good, thanks!

@asaunier
Copy link
Member Author

Thanks for reviewing @tsauerwein !

@asaunier asaunier merged commit f1fb9e6 into master Jun 24, 2016
@asaunier asaunier deleted the install-doc branch June 24, 2016 12:38
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