Merge ~kzapalowicz/snappy-hwe-snaps/+git/tpm:fix/tests-grep-regexp into ~snappy-hwe-team/snappy-hwe-snaps/+git/tpm:master
Proposed by
Konrad Zapałowicz
Status: | Merged |
---|---|
Approved by: | Jim Hodapp |
Approved revision: | 59e15c515e3728a202c950bb8df6e0d13512d3d3 |
Merged at revision: | 870d9b8406869cef80853c9ffa70b189a21dafa7 |
Proposed branch: | ~kzapalowicz/snappy-hwe-snaps/+git/tpm:fix/tests-grep-regexp |
Merge into: | ~snappy-hwe-team/snappy-hwe-snaps/+git/tpm:master |
Diff against target: |
15 lines (+2/-2) 1 file modified
tests/main/installation/task.yaml (+2/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jim Hodapp (community) | Approve | ||
System Enablement Bot | continuous-integration | Approve | |
Review via email:
|
Description of the change
tests: fix regexp for main/installation test
The updated regex should work for a situation where we check for a plug/slot connection of a snap that is not first on the connection
list - it is listed alphabetically.
E.g.:
:network-bind core,tpm2
The old version would return a failure as it started matching with spaces at the beginning. The new version understands that there might be a snap name too.
To post a comment you must log in.
PASSED: Continuous integration, rev:59e15c515e3 728a202c950bb8d f6e0d13512d3d3 /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 1500/ /jenkins. canonical. com/system- enablement/ job/generic- build-snap- worker/ 1269 /jenkins. canonical. com/system- enablement/ job/generic- update- snap-mp/ 1406/console /jenkins. canonical. com/system- enablement/ job/generic- test-snap/ 1695 /jenkins. canonical. com/system- enablement/ job/generic- cleanup- snap/881/ console
https:/
Executed test runs:
SUCCESS: https:/
None: https:/
SUCCESS: https:/
None: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 1500/rebuild
https:/