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

chore: use charmcraft latest/candidate for check libs #95

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

NohaIhab
Copy link
Contributor

This PR temporarily changes the channel of charmcraft used in check-libs action to latest/candidate, by default it uses latest/stable which currently does not support platforms syntax. It is needed to work on the charmcraft.yaml ST124 changes, for example canonical/kubeflow-tensorboards-operator#154.

Copy link
Contributor

@DnPlas DnPlas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @NohaIhab !

@NohaIhab NohaIhab merged commit fbfb58d into main Jan 23, 2025
@NohaIhab NohaIhab deleted the chore-use-latest-candidate-check-libs branch January 23, 2025 16:39
NohaIhab added a commit to canonical/kfp-operators that referenced this pull request Jan 24, 2025
NohaIhab pushed a commit to canonical/kfp-operators that referenced this pull request Jan 24, 2025
* ci, tests: pass charm artefacts to deploy and test charms

This commit enables the "--charm-path" option to pass .charm artefacts
to be deployed and tested at an individual level.
This change enables the option to pass pre-built charms to the tests
to avoid building them on the same test. It also ensures compatibility
with the build_charm.py reusable workflow (from canonical/data-platform-workflows).

Fixes: #639

Use single (cached) build for integration tests & release

Fix hardcoded path for `platforms` syntax

skip: remove duplicated download-charms

remove workflow dispatch from release.yaml not needed

add description for outputs

rename get charm paths job and channel output

skip: rename job in needs

pin quality checks back to main due to merging canonical/charmed-kubeflow-workflows#95

Use stage instead of prime in charmcraft files part (#658)
NohaIhab added a commit to canonical/kfp-operators that referenced this pull request Jan 24, 2025
Use single (cached) build for integration tests & release

Fix hardcoded path for `platforms` syntax

remove workflow dispatch from release.yaml not needed

add description for outputs

rename get charm paths job and channel output

pin quality checks back to main due to merging canonical/charmed-kubeflow-workflows#95

Use stage instead of prime in charmcraft files part (#658)

Co-authored-by: Daniela Plascencia <daniela.plascencia@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants