Skip to content

Conversation

@motjuste
Copy link
Contributor

@motjuste motjuste commented May 28, 2025

Description

This adds re-usable workflow for building the Snap with the Checkbox provider for testing DSS. While this PR only adds this workflow, a subsequent PR will call it to build the Snap in GH before using it to test DSS.

The workflow to build the checkbox-ce-oem Snaps was used as a reference, except that the checkbox-dss Snap produced by this workflow is currently not published in the store. This is why the workflow produces the built Snap's artifact URI when it is triggered by a workflow_call event.

Finally, since the provider is validated as part of the Snap's build process, this is also a good check to be triggered on PRs and changes related to the DSS provider. However, the build will be triggered only on Approval of the PR and not on every commit to the PR.

Resolved issues

Part of CHECKBOX-1905

Documentation

No changes to documentation.

Tests

No changes to source code, but see run of workflow in the checks below!

  • Verify that the workflow is triggered on approval of this PR.

@motjuste motjuste marked this pull request as ready for review May 28, 2025 09:40
This change should not effect other ways of triggering the workflow
Copy link
Collaborator

@fernando79513 fernando79513 left a comment

Choose a reason for hiding this comment

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

Thanks for the change.
LGTM+1!

@motjuste motjuste merged commit 70cc75c into main May 28, 2025
20 of 41 checks passed
@motjuste motjuste deleted the CHECKBOX-1905-dss-provider-snap-build branch May 28, 2025 11:39
mreed8855 pushed a commit that referenced this pull request Jul 30, 2025
* Add reusable workflow to build checkbox-dss snap

* Add missing newline at end of file

* Restrict triggering on PRs only on approval

This change should not effect other ways of triggering the workflow
mreed8855 pushed a commit that referenced this pull request Jul 31, 2025
* Add reusable workflow to build checkbox-dss snap

* Add missing newline at end of file

* Restrict triggering on PRs only on approval

This change should not effect other ways of triggering the workflow
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.

3 participants