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

Snap package #73

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Snap package #73

wants to merge 6 commits into from

Conversation

MarianRaphael
Copy link
Contributor

@MarianRaphael MarianRaphael commented Apr 16, 2023

Description

  • Added the Snap package build configuration (/snap), allowing the project to be built and published as a Snap package.
  • Created a README.md file specifically for the Snap package, detailing installation, configuration, and usage instructions.
  • Included a template configuration file (device.yml) and a start script (start.sh) to simplify the setup process.
  • Implemented the Snap build and publish pipeline in GitHub Actions, ensuring that new releases are automatically published to the Snap Store.

Open Tasks

  • Add the Snapcraft secret
  • Test new "publish" GitHub pipeline (Initial publishing of the snap)

Related Issue(s)

#71

Tests

I have tested the snap and the GitHub build pipeline. After configuring the snap, it successfully connected to the FlowForge Prod instance. The tests were carried out on a Linux (Ubuntu 20.04) virtual machine, and the results were positive, as illustrated in the screenshots below:

Screenshot 2023-04-16 at 15 23 36

Screenshot 2023-04-16 at 15 24 00

Snap package for additional tests

https://drive.google.com/file/d/18XebwXFBd2uyVFpeQ89Dbdiqjp4j_DQ3/view?usp=share_link

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on flowforge/helm to update ConfigMap Template
    • Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

@MarianRaphael MarianRaphael requested a review from knolleary April 16, 2023 15:57
@MarianRaphael MarianRaphael self-assigned this Apr 16, 2023
@MarianRaphael MarianRaphael linked an issue Apr 16, 2023 that may be closed by this pull request
@MarianRaphael
Copy link
Contributor Author

Needs to be changed so that new command line options for the UI will be handled correctly.

@MarianRaphael MarianRaphael marked this pull request as draft June 7, 2023 08:40
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.

Provide Snap package of the device agent
1 participant