touch-packages team mailing list archive

[Steve Langasek <steve.langasek@xxxxxxxxxxxxx>]

This bug is important to fix before phone images go to factory, since it
impacts our ability to do phased over-the-air updates.

** Changed in: system-image (Ubuntu)
   Importance: Undecided => High

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to system-image in Ubuntu.
https://bugs.launchpad.net/bugs/1383539

Title:
  phased update support does not give idempotent answer for each
  (machine,update)

Status in “system-image” package in Ubuntu:
  New

Bug description:
  The phased update implementation in ./systemimage/helpers.py does:

  def phased_percentage(*, reset=False):
      global _pp_cache
      if _pp_cache is None:
          with open(UNIQUE_MACHINE_ID_FILE, 'rb') as fp:
              data = fp.read()
          now = str(time.time()).encode('us-ascii')
          r = random.Random()
          r.seed(data + now)
          _pp_cache = r.randint(0, 100)
      try:
          return _pp_cache
      finally:
          if reset:
              _pp_cache = None

  
  Since the current time is used as a seed, this function will return a different value each time it's called.

  The phased update model requires that, for each given machine and each
  given update, the calculated percentage persists.  With this
  implementation, since the seed is different for each invocation and
  the result is not cached, phasing will happen much more quickly than
  intended.

  system-image should avoid using the time as a seed, and instead use a
  predictable (but well-distributed) seed of the machine ID plus an
  image identifier (perhaps full channel name + image ID).

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/system-image/+bug/1383539/+subscriptions