Skip to content

Commit

Permalink
Make Anitya work with OpenShift s2i builders
Browse files Browse the repository at this point in the history
A couple changes are required to make the s2i builder for OpenShift
work:

  * Move the requirements into the repository root
  * Depend on psycopg and gunicorn
  * Add a configuration file for the builder in .s2i

Additionally, this drops the EL7 test requirements since we'll be
deploying on a Fedora-based image.

Signed-off-by: Jeremy Cline <[email protected]>
  • Loading branch information
jeremycline committed Jan 25, 2018
1 parent c257229 commit 2490c9a
Show file tree
Hide file tree
Showing 10 changed files with 13 additions and 40 deletions.
2 changes: 2 additions & 0 deletions .s2i/environment
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
APP_MODULE=anitya.wsgi:application
UPGRADE_PIP_TO_LATEST=true
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ env:
- TOXENV=lint
- TOXENV=docs
- TOXENV=py27
- TOXENV=el7
matrix:
include:
- python: 3.5
Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
include LICENSE README.rst CHANGELOG.rst
include alembic.ini
include runtests.sh createdb.py nosetests migrate_wiki.py
include requirements.txt readthedocs.txt test_requirements.txt
recursive-include fedmsg.d *
recursive-include files *
recursive-include requirements *
recursive-include anitya/templates *
recursive-include anitya/static *
4 changes: 2 additions & 2 deletions ansible/roles/anitya-dev/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
- name: Install Anitya requirements into the virtualenv
become_user: "{{ ansible_env.SUDO_USER }}"
pip:
requirements: "requirements/requirements.txt"
requirements: "requirements.txt"
virtualenv: /home/{{ ansible_env.SUDO_USER }}/.virtualenvs/anitya/
virtualenv_python: python2
args:
Expand All @@ -61,7 +61,7 @@
- name: Install Anitya test requirements into the virtualenv
become_user: "{{ ansible_env.SUDO_USER }}"
pip:
requirements: "requirements/test_requirements.txt"
requirements: "test_requirements.txt"
virtualenv: /home/{{ ansible_env.SUDO_USER }}/.virtualenvs/anitya/
virtualenv_python: python2
args:
Expand Down
File renamed without changes.
2 changes: 2 additions & 0 deletions requirements/requirements.txt → requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ python-dateutil
dateutils
fedmsg[commands,consumers]
flask-login
gunicorn
social-auth-app-flask
social-auth-app-flask-sqlalchemy
flask-restful
Expand All @@ -12,5 +13,6 @@ jinja2 >= 2.4
sqlalchemy >= 0.9
# https://github.com/ironfroggy/straight.plugin/issues/17#issuecomment-41466275
straight.plugin==1.4.0-post-1
psycopg2
pytoml
wtforms
16 changes: 0 additions & 16 deletions requirements/el7-requirements.txt

This file was deleted.

2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ def get_project_version():
return match.groups()[0].decode("utf-8")


def get_requirements(requirements_file='requirements/requirements.txt'):
def get_requirements(requirements_file='requirements.txt'):
"""Get the contents of a file listing the requirements.
:arg requirements_file: path to a requirements file
Expand Down
File renamed without changes.
24 changes: 5 additions & 19 deletions tox.ini
Original file line number Diff line number Diff line change
@@ -1,25 +1,11 @@
[tox]
envlist = el7,py27,py35,py36,lint,docs
envlist = py27,py35,py36,lint,docs

[testenv]
passenv = TRAVIS TRAVIS_*
deps =
-rrequirements/requirements.txt
-rrequirements/test_requirements.txt
whitelist_externals =
rm
commands =
rm -rf htmlcov coverage.xml
py.test --cov-config .coveragerc --cov=anitya --cov-report term \
--cov-report xml --cov-report html


[testenv:el7]
passenv = TRAVIS TRAVIS_*
basepython = python2
deps =
-rrequirements/el7-requirements.txt
-rrequirements/test_requirements.txt
-rrequirements.txt
-rtest_requirements.txt
whitelist_externals =
rm
commands =
Expand All @@ -31,8 +17,8 @@ commands =
[testenv:docs]
changedir = docs
deps =
-rrequirements/readthedocs.txt
-rrequirements/requirements.txt
-rreadthedocs.txt
-rrequirements.txt
whitelist_externals =
mkdir
rm
Expand Down

0 comments on commit 2490c9a

Please sign in to comment.