HEAD files are needed for conflict management

Bug #1505701 reported by Ann Taraday
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Ann Taraday

Bug Description

Some time ago we merged change https://review.openstack.org/#/c/227319/ that removes HEADS file. Validation of migration revisions using HEADS file was replaced with pep8. This allows us to avoid merge conflicts that appeared every time a new migration was merged.

The problem is that the original idea of HEAD file was not only to validate revisions, so as not to allow outdated changes go into merge queue, that could be very important for the end of the cycle when a lot of patches get approved.

Tags: db
Changed in neutron:
importance: Undecided → Low
status: New → Confirmed
Changed in neutron:
status: Confirmed → In Progress
Changed in neutron:
assignee: Ann Kamyshnikova (akamyshnikova) → Henry Gessau (gessau)
Henry Gessau (gessau)
Changed in neutron:
assignee: Henry Gessau (gessau) → Ann Kamyshnikova (akamyshnikova)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/232607
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=f144a283ea4f79714b8df288e9eb652e1af7f48d
Submitter: Jenkins
Branch: master

commit f144a283ea4f79714b8df288e9eb652e1af7f48d
Author: AKamyshnikova <email address hidden>
Date: Thu Oct 8 18:18:21 2015 +0300

    Per-branch HEAD files for conflict management

    We have pep8 check for validation revisions, but it allows
    outdated changes go into merge queue. To prevent this added
    CONTRACT_HEAD, EXPAND_HEAD files.

    Closes-bug: #1505701

    Change-Id: Ie4b727e55a0b1ecb12e915a0037094a928d8f975

Changed in neutron:
status: In Progress → Fix Committed
Changed in neutron:
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.