Skip to content

Commit

Permalink
CI: replace yaml lib
Browse files Browse the repository at this point in the history
yaml was removed from core in some version between 3.8 and 3.10.
  • Loading branch information
wreis committed Nov 9, 2023
1 parent bedd054 commit 33e5996
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip
python -m pip install --upgrade black pytest sh
python -m pip install --upgrade black
- name: Check Python formatting
run: |
set -eux
Expand All @@ -37,7 +37,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip
python -m pip install --upgrade pytest sh
pip install --upgrade pytest sh ruamel.yaml
- name: Running addons tests
run: |
set -x
Expand Down
8 changes: 6 additions & 2 deletions tests/test-addons.py
Original file line number Diff line number Diff line change
@@ -1,13 +1,17 @@
import sh
import yaml

from ruamel.yaml import YAML

from utils import microk8s_enable, wait_for_pod_state, microk8s_disable


yaml = YAML(typ="safe")


class TestAddons(object):
def test_python_demo_nginx(self):
microk8s_enable("python-hello-k8s")
wait_for_pod_state("", "default", "running", label="app=python-demo-nginx")
status = yaml.safe_load(sh.microk8s.status(format="yaml").stdout)
status = yaml.load(sh.sudo.microk8s.status(format="yaml").stdout)
expected = {"python-hello-k8s": "enabled"}
microk8s_disable("python-hello-k8s")
8 changes: 6 additions & 2 deletions tests/utils.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
import datetime
import time
import yaml

from subprocess import check_output, CalledProcessError
from ruamel.yaml import YAML


yaml = YAML(typ="safe")


def run_until_success(cmd, timeout_insec=60, err_out=None):
Expand Down Expand Up @@ -58,7 +62,7 @@ def kubectl_get(target, timeout_insec=300):
"""
cmd = "get -o yaml " + target
output = kubectl(cmd, timeout_insec)
return yaml.safe_load(output)
return yaml.load(output)


def wait_for_pod_state(
Expand Down

0 comments on commit 33e5996

Please sign in to comment.