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

Fix Code becoming integer. #1

Open
wants to merge 12 commits into
base: issues/545
Choose a base branch
from
20 changes: 6 additions & 14 deletions .github/workflows/tox.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ['3.6', '3.7', '3.8', '3.9', '3.10']
python-version: ['3.8', '3.9', '3.10']

steps:
- uses: actions/checkout@v1
Expand All @@ -35,21 +35,11 @@ jobs:
fail-fast: false
matrix:
juju_channel:
- latest/stable
- 2.9/stable
- 2.8/stable
- 3.1/stable
bundle:
- first
- second
- third
exclude:
# disable 'first' and 'second' bundles for juju 2.8 since 'magpie'
# is not a promulgated charm in the charmstore, only on charmhub
# which 2.8 can't talk to.
- juju_channel: 2.8/stable
bundle: first
- juju_channel: 2.8/stable
bundle: second
env:
TEST_ZAZA_BUG_LP1987332: "on" # http://pad.lv/1987332
needs: build
Expand All @@ -60,19 +50,21 @@ jobs:
set -euxo pipefail
python -m pip install --upgrade pip
pip install tox tox-gh-actions
sudo snap install --channel ${{ matrix.juju_channel }} --classic juju
sudo snap install --channel ${{ matrix.juju_channel }} juju
sudo snap install --classic juju-crashdump
sudo lxd init --auto
# This is a throw-away CI environment, do not do this at home
sudo chmod 666 /var/snap/lxd/common/lxd/unix.socket
# until Juju provides stable IPv6-support we unfortunately need this
lxc network set lxdbr0 ipv6.address none
sudo iptables -F FORWARD
sudo iptables -P FORWARD ACCEPT
# pull images
lxc image copy --alias juju/bionic/amd64 --copy-aliases ubuntu-daily:bionic local:
lxc image copy --alias juju/focal/amd64 --copy-aliases ubuntu-daily:focal local:
lxc image copy --alias juju/jammy/amd64 --copy-aliases ubuntu-daily:jammy local:
lxc image list
juju bootstrap --no-gui localhost
juju bootstrap localhost
- name: Functional test
run: |
set -euxo pipefail
Expand Down
3 changes: 1 addition & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
pyparsing<3.0.0 # pin for aodhclient which is held for py35
async_generator
kubernetes<18.0.0; python_version < '3.6' # pined, as juju uses kubernetes
# pinned until 3.0 regressions are handled: https://github.com/openstack-charmers/zaza/issues/545
juju<3.0
juju<3.2
juju_wait
PyYAML>=3.0
pbr==5.6.0
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@

'hvac<0.7.0',
'jinja2',
'juju<3.0',
'juju<3.2',
'juju-wait',
'PyYAML',
'tenacity',
Expand Down
4 changes: 2 additions & 2 deletions test-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,5 +17,5 @@ keystoneauth1
oslo.config
python-novaclient
tenacity
# pinned until 3.0 regressions are handled: https://github.com/openstack-charmers/zaza/issues/545
juju<3.0
# Fix upper version to ensure compatibility with Juju 3.1
juju<3.2
Loading