Merge ~morphis/snappy-hwe-snaps/+git/wifi-ap:feature/wait-for-snap-setup into ~snappy-hwe-team/snappy-hwe-snaps/+git/wifi-ap:master
Proposed by
Simon Fels
Status: | Merged |
---|---|
Approved by: | Matteo Croce |
Approved revision: | e577ecf5589141ae3fdcfec9ce9515d96955876c |
Merged at revision: | 3334d49fa005b243e544b8448585746462b57e94 |
Proposed branch: | ~morphis/snappy-hwe-snaps/+git/wifi-ap:feature/wait-for-snap-setup |
Merge into: | ~snappy-hwe-team/snappy-hwe-snaps/+git/wifi-ap:master |
Diff against target: |
86 lines (+35/-1) 4 files modified
bin/automatic-setup.sh (+7/-0) cmd/service/main.go (+15/-0) hooks/configure (+7/-0) tests/main/conf-wizard-disabled-from-gadget/task.yaml (+6/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Matteo Croce (community) | Approve | ||
System Enablement Bot | continuous-integration | Approve | |
Review via email: mp+316544@code.launchpad.net |
Description of the change
Wait for initial snap setup to be done before starting service
In the case that the gadget snap provides configuration items for us those are set after our services are already started which leads to the problem that we don't respect any preset configuration options on first startup.
This fixes the problem by telling our services to wait for a file $SNAP_COMMON/
To post a comment you must log in.
FAILED: Continuous integration, rev:760b3bc0674 894c2104f2ca136 4ea2c64fcf7b59 /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 866/ /jenkins. canonical. com/system- enablement/ job/generic- update- snap-mp/ 774/console
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 866/rebuild
https:/