Skip to content

Commit

Permalink
compatible(integration_test_charm.yaml): Add support for arm64 on mic…
Browse files Browse the repository at this point in the history
…rok8s (#177)

arch constraint needs to be set on lxd and microk8s models, not just lxd
models
  • Loading branch information
carlcsaposs-canonical authored May 27, 2024
1 parent ba12595 commit e90eb63
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions .github/workflows/integration_test_charm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -301,17 +301,13 @@ jobs:
run: |
mkdir -p ~/.local/share/juju # Workaround for juju 3 strict snap
sg '${{ steps.parse-cloud.outputs.group }}' -c "juju bootstrap '${{ inputs.cloud }}' '${{ steps.parse-versions.outputs.agent_bootstrap_option }}'"
- name: Set up environment
run: |
juju model-defaults logging-config='<root>=INFO; unit=DEBUG'
juju add-model test
# Unable to set constraint on all models because of Juju bug:
# https://bugs.launchpad.net/juju/+bug/2065050
juju set-model-constraints arch='${{ inputs.architecture }}'
pipx install tox
pipx install poetry
- name: Add architecture model constraint
if: ${{ inputs.cloud == 'lxd' }}
# Unable to set constraint on all models because of Juju bug:
# https://bugs.launchpad.net/juju/+bug/2065050
run: juju set-model-constraints arch='${{ inputs.architecture }}'
- name: Update python-libjuju version
if: ${{ inputs.libjuju-version-constraint }}
run: poetry add --lock --group integration juju@'${{ inputs.libjuju-version-constraint }}'
Expand Down

0 comments on commit e90eb63

Please sign in to comment.