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

Redis #95

Merged
merged 23 commits into from
Apr 22, 2016
Merged

Redis #95

merged 23 commits into from
Apr 22, 2016

Conversation

atarkowska
Copy link
Member

@atarkowska atarkowska commented Mar 25, 2016

This PR test redis installation on Unix, see doc ome/omero-documentation#1431

To test:

$ ENV=(centos7_nginx|centos7_apache24)
$ WEBSESSION=true ./docker-build.sh $ENV
OSX: $ WEBSESSION=true DMNAME=dev ./test_services.sh # docker machine can be obtained from docker-machine ls
UNIX: $ WEBSESSION=true ./test_services.sh
$ WEBSESSION=true ./test_services.sh

# test REDIS manually
OSX: $ docker run -d --privileged -p 8888:80 --name omeroinstall omero_install_test_$ENV
UNIX: $ docker run -d --name omeroinstall -v /sys/fs/cgroup:/sys/fs/cgroup:ro -v /run omero_install_test_$ENV
# wait 1 min until omero accept connections
$ docker exec -it omeroinstall /bin/bash -c "service redis status"
$ docker exec -it omeroinstall bash
[root@42c0852b59ef /]# redis-cli
127.0.0.1:6379> PING
PONG
127.0.0.1:6379> MONITOR
OK
(here you will see the content)

open another console, (if you use docker-machine eval $(docker-machine env dev))

$ WEB_HOST=$(docker-machine ip dev):8888 ./test_login_to_web.sh

while script is logging in, on redis monitor you will see:

1460330344.663488 [0 127.0.0.1:55185] "SELECT" "1"
1460330344.663733 [1 127.0.0.1:55185] "EXISTS" ":1:django.contrib.sessions.cachehd4570rlkho9e6ngpy3qr8cvakpzpk1b"
1460330344.664056 [1 127.0.0.1:55185] "SETNX" ":1:django.contrib.sessions.cachehd4570rlkho9e6ngpy3qr8cvakpzpk1b" "\x80\x02}q\x01U\tconnectorq\x02comeroweb.connector\nConnector\nq\x03)\x81q\x04}q\x05(U\tis_secureq\x06\x89U\tserver_idq\aX\x01\x00\x00\x001U\auser_idq\b\x8a\x00U\x11omero_session_keyq\tU$162d57e5-34db-47b1-93b7-4262258d2b23q\nU\tis_publicq\x0b\x89ubs."
1460330344.664377 [1 127.0.0.1:55185] "EXPIRE" ":1:django.contrib.sessions.cachehd4570rlkho9e6ngpy3qr8cvakpzpk1b" "86400"
1460330344.681574 [1 127.0.0.1:55164] "GET" ":1:django.contrib.sessions.cachehd4570rlkho9e6ngpy3qr8cvakpzpk1b"
1460330344.760156 [1 127.0.0.1:55164] "DEL" ":1:django.contrib.sessions.cachehd4570rlkho9e6ngpy3qr8cvakpzpk1b"

TESTED via travis https://travis-ci.org/ome/omero-install/builds/123777478

@snoopycrimecop
Copy link
Member

Conflicting PR. Removed from build OMERO-DEV-merge-homebrew/ICE=3.5,OSX=snapper#137. See the console output for more details.
Possible conflicts:

  • PR Parameters #94 jburel 'Parameters'
    • linux/install_centos7_nginx.sh
    • linux/install_centos7_apache24.sh
    • linux/install_centos6_py27_apache24.sh

@joshmoore
Copy link
Member

NB: I assume testing of this occurs according to ome/devspace#20 (comment)

@snoopycrimecop
Copy link
Member

Conflicting PR. Removed from build OMERO-DEV-merge-homebrew/ICE=3.5,OSX=snapper#138. See the console output for more details.
Possible conflicts:

  • PR Parameters #94 jburel 'Parameters'
    • linux/install_centos7_nginx.sh
    • linux/install_centos7_apache24.sh
    • linux/install_centos6_py27_apache24.sh

@jburel
Copy link
Member

jburel commented Mar 29, 2016

in step01_centos6_py27_ius_deps_experimental.sh

+set +u
 +source /opt/rh/python27/enable
 +set -u

is not correct. This command is for scl python
for ius you will have to use virtualenv

@jburel
Copy link
Member

jburel commented Mar 29, 2016

I will also break it into 2 PRs: one for linux, one for OSX

@snoopycrimecop
Copy link
Member

Conflicting PR. Removed from build OMERO-DEV-merge-homebrew/ICE=3.5,OSX=snapper#139. See the console output for more details.
Possible conflicts:

  • PR Parameters #94 jburel 'Parameters'
    • linux/install_centos7_nginx.sh
    • linux/install_centos7_apache24.sh
    • linux/install_centos6_py27_apache24.sh

@snoopycrimecop
Copy link
Member

Conflicting PR. Removed from build OMERO-DEV-merge-homebrew/ICE=3.5,OSX=snapper#140. See the console output for more details.
Possible conflicts:

  • PR Parameters #94 jburel 'Parameters'
    • linux/install_centos7_nginx.sh
    • linux/install_centos7_apache24.sh
    • linux/install_centos6_py27_apache24.sh

@atarkowska
Copy link
Member Author

Excluding for now as we need first #99

@atarkowska atarkowska force-pushed the dev_redis branch 9 times, most recently from 6573b06 to 2e0c87a Compare April 11, 2016 19:23
@atarkowska
Copy link
Member Author

atarkowska commented Apr 18, 2016

you will have to adjust the run.sh files

@jburel redis is now started in run.sh

@atarkowska atarkowska closed this Apr 18, 2016
@atarkowska atarkowska reopened this Apr 18, 2016
@jburel
Copy link
Member

jburel commented Apr 19, 2016

I have opened a PR against this branch so redis server is not started by default.
Could you also add a section on the README about web session ?

@manics
Copy link
Member

manics commented Apr 19, 2016

Is Redis supported on all platforms? Or would it be easier to support just the latest (centos 7)?

@atarkowska
Copy link
Member Author

redis is supported for python 2.7 only

@jburel
Copy link
Member

jburel commented Apr 19, 2016

very minor: CentOS7 => CentOS 7

@atarkowska
Copy link
Member Author

done

@atarkowska atarkowska closed this Apr 19, 2016
@atarkowska atarkowska reopened this Apr 19, 2016
@atarkowska atarkowska closed this Apr 19, 2016
@atarkowska atarkowska reopened this Apr 19, 2016
@atarkowska atarkowska closed this Apr 19, 2016
@atarkowska atarkowska reopened this Apr 19, 2016
@atarkowska atarkowska closed this Apr 20, 2016
@atarkowska atarkowska reopened this Apr 20, 2016
@atarkowska atarkowska closed this Apr 20, 2016
@atarkowska atarkowska reopened this Apr 20, 2016
@atarkowska atarkowska closed this Apr 20, 2016
@atarkowska atarkowska reopened this Apr 20, 2016
@jburel
Copy link
Member

jburel commented Apr 22, 2016

Thanks @aleksandra-tarkowska merging

@jburel jburel merged commit 6c0587b into ome:develop Apr 22, 2016
@atarkowska atarkowska deleted the dev_redis branch June 30, 2016 11:21
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.

5 participants