-
Notifications
You must be signed in to change notification settings - Fork 485
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
fix(snap): Install Unit of Measure config file on upgrade #4125
Conversation
Signed-off-by: Farshid Tavakolizadeh <farshid.tavakolizadeh@canonical.com>
Codecov Report
@@ Coverage Diff @@
## main #4125 +/- ##
=======================================
Coverage 43.61% 43.61%
=======================================
Files 120 120
Lines 10630 10630
=======================================
Hits 4636 4636
Misses 5607 5607
Partials 387 387 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Signed-off-by: Farshid Tavakolizadeh <farshid.tavakolizadeh@canonical.com>
Signed-off-by: Farshid Tavakolizadeh <farshid.tavakolizadeh@canonical.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@farshidtz Thank you, looks good!
@cloudxxx8 could you please approve this? It's been reviewed. |
PR #4119 added the following environment variable:
edgex-go/snap/snapcraft.yaml
Line 278 in 166620d
This makes the edge releases of snap backward incompatible as of v2.3.0-dev.45 because this file doesn't exist when upgrading from an older version. The latest snap beta release which isn't broken is 2.3.0-dev.44 (snap rev 5898/5899). This fix works when upgrading from the latest beta or older.
This PR adds the logic to add the configuration file when upgrading (i.e. using
snap refresh
).Signed-off-by: Farshid Tavakolizadeh farshid.tavakolizadeh@canonical.com
If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:
BREAKING CHANGE:
describing the break)Testing Instructions
Reproduce the error:
Test the changes using the snap built from this branch:
New Dependency Instructions (If applicable)