Merge lp:~jonas-drange/ubuntu-system-settings/update-cannot-be-ota into lp:ubuntu-system-settings

Proposed by Jonas G. Drange
Status: Merged
Approved by: Jonas G. Drange
Approved revision: 1698
Merged at revision: 1707
Proposed branch: lp:~jonas-drange/ubuntu-system-settings/update-cannot-be-ota
Merge into: lp:ubuntu-system-settings
Diff against target: 33 lines (+1/-11)
2 files modified
plugins/system-update/PageComponent.qml (+1/-2)
tests/plugins/system-update/tst_systemupdate_pagecomponent.qml (+0/-9)
To merge this branch: bzr merge lp:~jonas-drange/ubuntu-system-settings/update-cannot-be-ota
Reviewer Review Type Date Requested Status
system-apps-ci-bot continuous-integration Approve
Ken VanDine Approve
Review via email: mp+303672@code.launchpad.net

Commit message

versionTag refers to the currently installed OTA, not a pending image update, so remove it

To post a comment you must log in.
Revision history for this message
Ken VanDine (ken-vandine) wrote :

looks good

review: Approve
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

PASSED: Continuous integration, rev:1698
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-system-settings-ci/123/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/1269
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/279
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1269
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=vivid+overlay/1136
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=xenial+overlay/1136
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=yakkety/1136
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1123/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1123
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1123/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-system-settings-ci/123/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/system-update/PageComponent.qml'
2--- plugins/system-update/PageComponent.qml 2016-08-12 12:15:26 +0000
3+++ plugins/system-update/PageComponent.qml 2016-08-23 11:49:54 +0000
4@@ -221,8 +221,7 @@
5 width: imageUpdateCol.width
6 updateState: model.updateState
7 progress: model.progress
8- version: SystemImage.versionTag ?
9- SystemImage.versionTag : remoteVersion
10+ version: remoteVersion
11 size: model.size
12 changelog: model.changelog
13 error: model.error
14
15=== modified file 'tests/plugins/system-update/tst_systemupdate_pagecomponent.qml'
16--- tests/plugins/system-update/tst_systemupdate_pagecomponent.qml 2016-08-12 12:15:26 +0000
17+++ tests/plugins/system-update/tst_systemupdate_pagecomponent.qml 2016-08-23 11:49:54 +0000
18@@ -302,15 +302,6 @@
19 return !!findChild(testRoot, "installationFailed");
20 }, false);
21 }
22-
23- // OTA updates are branded as such.
24- function test_otaUpdate() {
25- UpdateManager.model.mockAddUpdate("ubuntu", 1, Update.KindImage);
26- SystemImage.mockVersionTag("OTA-1");
27- UpdateManager.cancel();
28- var delegate = findChild(instance, "imageUpdatesDelegate-0");
29- compare(delegate.version, "OTA-1");
30- }
31 }
32
33 UbuntuTestCase {

Subscribers

People subscribed via source and target branches