ci: use snaps output from snapcraft pack action
Metadata
Current evaluation
Merged #438 to use the snapcraft pack action's plural `snaps` output, preventing empty artifact paths and restoring snap upload and publishing.
Resolution: closed resolved
Reason: PR #438 was merged after approval with all CI checks passing. It replaced the obsolete singular `snap` output with `snaps`, fixing the empty-path upload-artifact failure.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes an issue where the snap workflow failed during the upload artifact step with `Input required and not supplied: path` (see [run 34387324540](https://github.com/canonical/imagecraft/actions/runs/34387324540/job/102587117930)).
The `canonical/craft-actions/snapcraft/pack` action exposes artifact outputs using plural naming (`snaps`), whereas the previous action used singular `snap`. This caused `${{ steps.build-imagecraft.outputs.snap }}` to evaluate to an empty string, causing `actions/upload-artifact` and `canonical/action-publish` to receive empty paths.
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/imagecraft/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #438 to use the snapcraft pack action's plural `snaps` output, preventing empty artifact paths and restoring snap upload and publishing. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged CI fix switched the snap workflow to the plural snaps output from snapcraft pack, preventing empty artifact paths during upload. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #438 as commit 41503a6, updating the snap workflow to use the pack action’s plural `snaps` output so artifact publishing receives the correct path. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Resolved CI artifact upload failure by updating the snapcraft pack action output reference from singular snap to plural snaps. All CI checks passed. | |
| qwen/qwen3.6-35b-a3b |
Impact:
50
Quick Win:
48
Staleness:
0
Complexity:
5
Confidence:
85
|
needs review | Fixes snap workflow artifact upload by changing snapcraft/pack output reference from 'snap' to 'snaps'. All CI checks passing, pending review. |
Update history
| Date | Change |
|---|---|
| updated | |
| updated | |
| updated | |
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #307 ci: pack snap using craft-actions | imagecraft | merged | Merged in commit 3f26a09 via #307, replacing action-build with canonical/craft-actions' Snapcraft pack action. | |
| #45 feat: create new snapcraft-pack action | craft-actions | merged | Merged as commit 8756708, adding a snapcraft/pack action modeled on charmcraft/pack, with shared common/ workflows, working `path` input, and snap/ dir added to the cache key. All CI checks passed; docs followed in #46. |