Skip to content

Commit

Permalink
[fix] Updated #274
Browse files Browse the repository at this point in the history
Fixes 274
  • Loading branch information
praptisharma28 committed Jun 1, 2024
1 parent 46ea00c commit 83694c5
Showing 1 changed file with 4 additions and 30 deletions.
34 changes: 4 additions & 30 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
---
name: OpenWISP Monitoring CI Build

on:
Expand All @@ -17,26 +18,9 @@ jobs:

services:
redis:
image: redis:5.0-alpine
image: redis
ports:
- 6379:6379
influxdb:
image: influxdb:1.8-alpine
ports:
- 8086:8086
influxdb2:
image: influxdb:2.0-alpine
volumes:
- influxdb2-data:/var/lib/influxdb2
ports:
- 8087:8087
options: >-
--env DOCKER_INFLUXDB_INIT_MODE=setup
--env DOCKER_INFLUXDB_INIT_USERNAME=openwisp
--env DOCKER_INFLUXDB_INIT_PASSWORD=openwisp
--env DOCKER_INFLUXDB_INIT_ORG=openwisp
--env DOCKER_INFLUXDB_INIT_BUCKET=openwisp2
--env DOCKER_INFLUXDB_INIT_RETENTION=1w

strategy:
fail-fast: false
Expand All @@ -49,9 +33,6 @@ jobs:
- django~=3.2.0
- django~=4.1.0
- django~=4.2.0
influxdb-version:
- influxdb
- influxdb2

steps:
- name: Install system packages
Expand Down Expand Up @@ -83,8 +64,8 @@ jobs:
- name: Install npm dependencies
run: sudo npm install -g jshint stylelint

- name: Start InfluxDB ${{ matrix.influxdb-version }} container
run: docker-compose up -d ${{ matrix.influxdb-version }}
- name: Start InfluxDB container
run: docker-compose up -d influxdb

- name: Install test dependencies
run: |
Expand All @@ -96,13 +77,6 @@ jobs:
pip uninstall -y django
pip install ${{ matrix.django-version }}
- name: Set InfluxDB environment variables for InfluxDB 2.x
if: matrix.influxdb-version == 'influxdb2'
run: |
export INFLUXDB_HOST=localhost
export INFLUXDB_PORT=8088
export USE_INFLUXDB2=True
- name: QA checks
run: |
./run-qa-checks
Expand Down

0 comments on commit 83694c5

Please sign in to comment.