New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 696045 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
please use my google.com address
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 696031



Sign in to add a comment

Mojo EDK: Make it easy for us to discover version skew issues

Project Member Reported by roc...@chromium.org, Feb 24 2017

Issue description

We currently have no test coverage to guard against version skew issues, i.e. two different versions of the EDK talking to each other. This is fairly tricky to solve, but strawman proposal:

1. Develop client and server test binaries which coordinate tests between each other

2. Develop a script which mirrors a build configuration across two output directories and:

  a. Builds local changes in one, upstream in the other
  b. Runs the tests with upstream client vs local server
  c. Runs the tests with upstream service vs local client
 

Comment 1 by roc...@chromium.org, Feb 24 2017

Chatted with jam@ offline and we agree that trying to get waterfall/CQ coverage for something like this is probably too much effort to be worthwhile.

To extend the original issue, we want to manually track *all* revisions (back to, say, the date of this bug report) where the protocol has changed, and ensure that any new local EDK protocol change is compatible with those previous revisions.

This unfortunately means local testing only, as infra supports one revision, one build, one binary; and the nature of these changes is such that conditional behavior or mock data is an intractable solution to the problem.

Shouldn't be too bad since only a small number of engineers are making EDK protocol changes, and such changes are still (and should remain) quite rare.

Comment 2 by roc...@chromium.org, Jun 27 2018

Status: WontFix (was: Assigned)
Marking obsolete in favor of mojo_core shipping

Sign in to add a comment