Skip to content

Commit

Permalink
Merge pull request #853 from kids-first/fix-study-sync-task
Browse files Browse the repository at this point in the history
🐛 Sync all studies in dataservice
  • Loading branch information
znatty22 committed Aug 15, 2023
2 parents c855857 + aa5415d commit ba62b36
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 2 deletions.
1 change: 1 addition & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
- run:
name: Install dependencies
command: |
pip install --upgrade pip
pip install -r requirements.txt
pip install -r dev-requirements.txt
- save_cache:
Expand Down
3 changes: 2 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,8 @@ WORKDIR /app
COPY requirements.txt /app/
RUN apt-get update && \
apt-get install -y --no-install-recommends git postgresql-client
RUN pip install awscli && \
RUN pip install --upgrade pip && \
pip install awscli && \
pip install --no-cache-dir -r requirements.txt

COPY . /app/
Expand Down
2 changes: 1 addition & 1 deletion creator/studies/dataservice.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ def sync_dataservice_studies():
api = settings.DATASERVICE_URL
logger.info(f"Getting studies from Dataservice at {api}")
try:
resp = requests.get(f"{api}/studies?limit=100")
resp = requests.get(f"{api}/studies?limit=1000")
except RequestException as err:
logger.error(
f"There was a problem getting studies from the Dataservice: {err}"
Expand Down
1 change: 1 addition & 0 deletions dev-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,3 +14,4 @@ pytest-pycodestyle==2.2.0
pytest-mock==3.3.0
pytest-cov==2.12.1
-e git+https://github.com/codecov/codecov-python.git@4ec70db255cc2dc332fbd01aab3069d9b2e699dc#egg=codecov
jinja2<3.1.0

0 comments on commit ba62b36

Please sign in to comment.