Skip to content
This repository was archived by the owner on Dec 1, 2025. It is now read-only.

chore: fix integration ci - #72

Merged
IronCore864 merged 2 commits into
canonical:mainfrom
IronCore864:fix-integration-test-ci
Mar 24, 2025
Merged

chore: fix integration ci#72
IronCore864 merged 2 commits into
canonical:mainfrom
IronCore864:fix-integration-test-ci

Conversation

@IronCore864

@IronCore864 IronCore864 commented Mar 24, 2025

Copy link
Copy Markdown
Contributor

Add a missing step for integration CI and use a specific channel for microk8s.

More information: when using the charmed-kubernetes/actions-operator action to setup an environment for integration test, with Juju 3.3+, microk8s needs a strict channel version. Without this setting, there will be an error like juju "3.x.x" can only work with strictly confined microk8s. See an existing issue here and an unmerged PR here.

Related: canonical/operator#1644.

Also there is a change to not run unit/integration tests for other branches (like main).

@IronCore864
IronCore864 marked this pull request as ready for review March 24, 2025 02:24
@IronCore864
IronCore864 merged commit 3ba5050 into canonical:main Mar 24, 2025
IronCore864 added a commit to canonical/operator that referenced this pull request Mar 24, 2025
This PR adds a channel for microk8s for the test explanation doc in the
CI section.

More information: when using the [charmed-kubernetes/actions-operator
action](https://github.com/charmed-kubernetes/actions-operator) to setup
an environment for integration test, with Juju 3.3+, microk8s needs a
strict channel version. Without this setting, there will be an error
like `juju "3.x.x" can only work with strictly confined microk8s`. See
an [existing issue
here](charmed-kubernetes/actions-operator#70)
and [an unmerged PR
here](charmed-kubernetes/actions-operator#57).

Related: canonical/juju-sdk-tutorial-k8s#72.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants