[network] There is no way to override add-patch action

Bug #1590735 reported by Michael Polenchuk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Sergey Vasilenko
Mitaka
Fix Released
High
Sergey Vasilenko

Bug Description

Detailed bug description:
  There is no way to override "add-patch" action
  since "override" action matches item by name,
  which is absent for "add-patch" action (generated later).

Description of the environment:
  latest F9.0 with ODL plugin

Actual result:
  Please see description above

Expected result:
  "add-patch" action is properly handled by "override" action

Steps to reproduce:
  Try to add rule for override "add-patch" action

Changed in fuel:
importance: Undecided → Critical
milestone: none → 9.0
tags: added: l23network
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
no longer affects: fuel/newton
description: updated
tags: removed: need-info
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → l23network (l23network)
status: New → Confirmed
Changed in fuel:
assignee: l23network (l23network) → Sergey Vasilenko (xenolog)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/327653

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
milestone: 9.0 → 10.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/327672

tags: added: customer-found
Revision history for this message
Sergey Shevorakov (sshevorakov) wrote :

Why this case is Crit?
Why it is customer-found (when it is 9.0 and 10.0 related)?

Anton Matveev (amatveev)
tags: added: move-to-mu
Changed in fuel:
importance: Critical → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/327653
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=705b9ae14944d20af764345a8f931f8a90107c9a
Submitter: Jenkins
Branch: master

commit 705b9ae14944d20af764345a8f931f8a90107c9a
Author: Sergey Vasilenko <email address hidden>
Date: Thu Jun 9 16:13:41 2016 +0300

    Add ability for override patches in transformations.

    for override patches a following syntax should be used:

      - action: override
        override: 'patch-br1:br2'
        bridges: ['new-br1', 'new-br2']

    For addition information see Change-Id == Ic4c9c6c34d9e3a57392446b534d997666578d110
    Closes-bug: #1590735

    Change-Id: I89ef5630ab2dfd373b8cd4b7db481278c659db75

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/327672
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=dd3bf916c1c235770e0ba3cc3cb2303b04496181
Submitter: Jenkins
Branch: stable/mitaka

commit dd3bf916c1c235770e0ba3cc3cb2303b04496181
Author: Sergey Vasilenko <email address hidden>
Date: Thu Jun 9 16:13:41 2016 +0300

    Add ability for override patches in transformations.

    for override patches a following syntax should be used:

      - action: override
        override: 'patch-br1:br2'
        bridges: ['new-br1', 'new-br2']

    For addition information see Change-Id == Ic4c9c6c34d9e3a57392446b534d997666578d110
    Closes-bug: #1590735

    Change-Id: I89ef5630ab2dfd373b8cd4b7db481278c659db75

Changed in fuel:
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.