Mir

Merge lp:~alan-griffiths/mir/bump-protocol-version into lp:mir

Proposed by Alan Griffiths
Status: Merged
Approved by: Gerry Boland
Approved revision: no longer in the source branch.
Merged at revision: 4203
Proposed branch: lp:~alan-griffiths/mir/bump-protocol-version
Merge into: lp:mir
Diff against target: 18 lines (+2/-2)
1 file modified
src/include/common/mir/protobuf/protocol_version.h (+2/-2)
To merge this branch: bzr merge lp:~alan-griffiths/mir/bump-protocol-version
Reviewer Review Type Date Requested Status
Mir CI Bot continuous-integration Approve
Gerry Boland (community) Approve
Alan Griffiths Pending
Review via email: mp+326583@code.launchpad.net

This proposal supersedes a proposal from 2017-06-29.

Commit message

The protocol version needs to change we dropped message RaiseRequest (and subsumed it into RequestOperation)

To post a comment you must log in.
Revision history for this message
Alan Griffiths (alan-griffiths) wrote : Posted in a previous version of this proposal

While this is technically correct, it might be better to retain compatibility with 0.26 clients. For that to happen:

1. the server needs to support RaiseRequest (deprecated)
2. the protocol needs bumping to 4.1

The advantage being that (for example) a 0.27 based server snap could interwork with a 0.26 based client snap. (Of course, if the client snap relied on mir-libs this wouldn't be necessary.)

Opinions?

review: Needs Information
Revision history for this message
Gerry Boland (gerboland) wrote : Posted in a previous version of this proposal

Changing the protocol version only impacts the Snappy world. We should ask them if a protocol change impacts any project of theirs (I'm not aware of any Mir-based work happening on their side though)

Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

On second thoughts, I think we should be able to rely on client snaps following the "use mir-libs" guideline.

Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

> On second thoughts, I think we should be able to rely on client snaps
> following the "use mir-libs" guideline.

On third thoughts - we won't be shipping 0.27 on UbuntuCore anyway

Revision history for this message
Gerry Boland (gerboland) wrote :

All good so

review: Approve
Revision history for this message
Mir CI Bot (mir-ci-bot) wrote :

PASSED: Continuous integration, rev:4202
https://mir-jenkins.ubuntu.com/job/mir-ci/3470/
Executed test runs:
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-mir/4741
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/4899
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=artful/4888
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/4888
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=zesty/4888
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=artful/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=artful/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=zesty/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=zesty/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=artful/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=artful/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=zesty/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=zesty/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=mesa,release=artful/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=mesa,release=artful/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=mesa,release=zesty/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=mesa,release=zesty/4778/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/4778
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/4778/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://mir-jenkins.ubuntu.com/job/mir-ci/3470/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/include/common/mir/protobuf/protocol_version.h'
--- src/include/common/mir/protobuf/protocol_version.h 2017-05-08 03:04:26 +0000
+++ src/include/common/mir/protobuf/protocol_version.h 2017-06-30 10:57:33 +0000
@@ -27,12 +27,12 @@
27{27{
28inline constexpr int current_protocol_version()28inline constexpr int current_protocol_version()
29{ 29{
30 return MIR_VERSION_NUMBER(0,4,0);30 return MIR_VERSION_NUMBER(0,5,0);
31}31}
3232
33inline constexpr int oldest_compatible_protocol_version()33inline constexpr int oldest_compatible_protocol_version()
34{34{
35 return MIR_VERSION_NUMBER(0,4,0);35 return MIR_VERSION_NUMBER(0,5,0);
36}36}
3737
38inline constexpr int next_incompatible_protocol_version()38inline constexpr int next_incompatible_protocol_version()

Subscribers

People subscribed via source and target branches