Skip to content

Commit

Permalink
Add bundle test to ci (#8)
Browse files Browse the repository at this point in the history
* Add bundle test to ci
* Remove juju 2.4
* Add jujuna timeout and dump debug-log on failure
* Add permissions workaround
  * See actions/upload-artifact#38 for upstream issue
  • Loading branch information
Sandor Zeestraten authored Jan 9, 2020
1 parent 4c7c045 commit b6e984e
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
10 changes: 9 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ jobs:
fail-fast: false
matrix:
juju_channel:
- 2.4/stable
- 2.5/stable
- 2.6/stable
- 2.7/stable
Expand All @@ -55,6 +54,10 @@ jobs:
with:
name: charm
path: /tmp/charm-builds/megaraid-check
# Workarounds for https://github.com/actions/upload-artifact/issues/38
- run: chmod +x -R /tmp/charm-builds/megaraid-check/bin
- run: chmod +x -R /tmp/charm-builds/megaraid-check/hooks
- run: ls -la -R /tmp/charm-builds/megaraid-check/
- name: Set up LXD
run: |
sudo usermod --append --groups lxd $(whoami)
Expand All @@ -67,3 +70,8 @@ jobs:
run: |
sudo snap install juju --classic --channel=${{ matrix.juju_channel }}
sudo su $(whoami) -c 'juju bootstrap localhost --no-gui'
- name: Run tests
run: make test-${{ matrix.test_bundles }}
- name: Display juju debug log on failure
run: juju debug-log --lines 100
if: failure()
4 changes: 2 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@ commands = charm-build --report --no-local-layers --force --log-level INFO

[testenv:test-xenial]
deps = jujuna
commands = jujuna deploy --wait tests/bundles/xenial.yaml
commands = jujuna deploy --wait --timeout 600 tests/bundles/xenial.yaml

[testenv:test-bionic]
deps = jujuna
commands = jujuna deploy --wait tests/bundles/bionic.yaml
commands = jujuna deploy --wait --timeout 600 tests/bundles/bionic.yaml

[isort]
force_single_line = True

0 comments on commit b6e984e

Please sign in to comment.