Merge lp://staging/~ttx/nova/bexar-2011.1.1 into lp://staging/nova/bexar

Proposed by Thierry Carrez
Status: Merged
Approved by: Soren Hansen
Approved revision: 648
Merged at revision: 650
Proposed branch: lp://staging/~ttx/nova/bexar-2011.1.1
Merge into: lp://staging/nova/bexar
Diff against target: 21 lines (+3/-3)
1 file modified
nova/version.py (+3/-3)
To merge this branch: bzr merge lp://staging/~ttx/nova/bexar-2011.1.1
Reviewer Review Type Date Requested Status
Soren Hansen (community) Approve
Vish Ishaya (community) Approve
Devin Carlen (community) Approve
Review via email: mp+50297@code.staging.launchpad.net

Commit message

Add support for micro-versions, and bump version number to 2011.1.1.

Description of the change

Add support for micro-versions, and bump version number to 2011.1.1.
Under the new simplified form it should probably not be forward-ported to Cactus.

To post a comment you must log in.
Revision history for this message
Soren Hansen (soren) wrote :

I think this is needlessly paranoid. I don't believe we intend to ever not have a micro version again, and the safe guards (the try/except blocks) are there to protect against changes to code in the very same file, which seems a bit pointless to me?

review: Needs Information
Revision history for this message
Thierry Carrez (ttx) wrote :

Simplified. You should know by now that I'm a bit paranoid.

648. By Thierry Carrez

Simplify patch to only support version number with microversions, since this is a one-off

Revision history for this message
Devin Carlen (devcamcar) wrote :

lgtm

review: Approve
Revision history for this message
Vish Ishaya (vishvananda) wrote :

lgtm. Waiting for soren on marking approved

review: Approve
Revision history for this message
Soren Hansen (soren) wrote :

Let's do it.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches