Skip to content

Commit

Permalink
test: fix jaas integration tests for microk8s
Browse files Browse the repository at this point in the history
The jaas integration tests were missing a change for how the
microk8s.config is provided to the acceptance tests. Causing
the add k8s cloud resource test to fail.
  • Loading branch information
hmlanigan committed Dec 3, 2024
1 parent c271cfd commit 53a36dc
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/test_integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ jobs:
echo "TEST_MANAGEMENT_BR=10.150.40.0/24" >> $GITHUB_ENV
echo "TEST_PUBLIC_BR=10.170.80.0/24" >> $GITHUB_ENV
- name: "Set additional environment for LXD"
# setup the microk8s config to be used by the tests.
if: ${{ matrix.action-operator.cloud == 'lxd' }}
# language=bash
run: |
Expand Down
8 changes: 6 additions & 2 deletions .github/workflows/test_integration_jaas.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,6 @@ jobs:
sudo microk8s.enable dns storage
sudo microk8s.enable dns local-storage
sudo -g snap_microk8s -E microk8s status --wait-ready --timeout=600
echo "MICROK8S_CONFIG<<EOF" >> $GITHUB_ENV
sudo microk8s.config view >> $GITHUB_ENV
echo "EOF" >> $GITHUB_ENV
- name: Create additional networks when testing with LXD
run: |
Expand All @@ -93,6 +91,12 @@ jobs:
echo "EOF" >> $GITHUB_ENV
echo "TEST_MANAGEMENT_BR=10.150.40.0/24" >> $GITHUB_ENV
echo "TEST_PUBLIC_BR=10.170.80.0/24" >> $GITHUB_ENV
- name: "Set additional environment for LXD"
# setup the microk8s config to be used by the tests.
if: ${{ matrix.action-operator.cloud == 'lxd' }}
# language=bash
run: |
sudo microk8s.config > /home/$USER/microk8s-config.yaml
- run: go mod download
- env:
TF_ACC: "1"
Expand Down

0 comments on commit 53a36dc

Please sign in to comment.