Updated epoch yaml syntax

Bug #1780981 reported by Matias Bordese
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Click Reviewers tools (obsolete)
Fix Released
Undecided
Jamie Strandboge

Bug Description

Epochs (stepped upgrades) support is being reworked, and its syntax in the YAML will change respect to the original implementation (which wasn't really used).

Right now the reviewer tools are rejecting any revision with an epoch value that is not an integer:

The store was unable to accept this snap.
  - malformed 'epoch': 3* (not an integer)

However, the epoch field should start allowing (with the right yaml syntax):
  - an integer or a string, all digits, with an optional * suffix (eg. 1, 3 or 5*)
  - a dict with optional 'read' and/or 'write' keys, each holding a list of integers, non-empty and sorted (from lower to higher) (eg. {'read': [1, 2]}, {'write': [3, 4]} or {'read': [1, 2], 'write': [2]})
  - field is optional

This validation is also being implemented as part of the revision scan task in the store.

Changed in click-reviewers-tools:
status: New → Triaged
Changed in click-reviewers-tools:
status: Triaged → In Progress
assignee: nobody → Jamie Strandboge (jdstrand)
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

This is now in trunk.

Changed in click-reviewers-tools:
status: In Progress → Fix Committed
Changed in click-reviewers-tools:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.