after excute check-vlan-interface , the vlan interface was deleted.

Bug #1552506 reported by bailin.zhang on 2016-03-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
steth
Medium
changzhi

Bug Description

if the vlan interface is not exist,the command can not excute.

if I creat a vlan interface, the vlan interface was deleted after the command was excuted.

for example:
(steth) check-vlan-interface server-164 server-110 er1 20
Agent: server-164 has no interface named er1.20!
(steth) check-vlan-interface server-164 server-110 efl 5
AgentA and agentB has already added the interface efl.5
AgentA and agentB has already setup the IP 192.168.100.164/24 and IP 192.168.100.110/24
AgentA and agentB has already deleted thevlan 5 in efl

Changed in steth:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → yaowei (yaowei)
tags: added: steth

In check-vlan-interface process, we should :
1. check server-A eth.vlan_id exists, A_exists = True/False
   1.1 if A_exists ==False, then add vlan interface in server-A
2. check server-B eth.vlan_id exists, B_exists = True/False
 2.1 if B_exists ==False, then add vlan interface in server-B

3. do some check else such as ping server-B form server-A via eth.vlan_id

4. teardown vlan_interface:
 4.1 if if A_exists ==False, then teardown vlan interface in server-A
 4.2 if if B_exists ==False, then teardown vlan interface in server-B

@bailin.zhang, do you agree with this?

Changed in steth:
assignee: yaowei (yaowei) → changzhi (changzhi)
bailin.zhang (bailin-zhang) wrote :

OK

Reviewed: https://review.openstack.org/291518
Committed: https://git.openstack.org/cgit/openstack/steth/commit/?id=c8b511fe1ddf3b644716b9581e8b738dd68372a5
Submitter: Jenkins
Branch: master

commit c8b511fe1ddf3b644716b9581e8b738dd68372a5
Author: changzhi1990 <email address hidden>
Date: Fri Mar 11 10:31:29 2016 +0800

    Fix bug when checking vlan interface

    Change-Id: I91fde669874c12c49b325c58073372ea6632dd2b
    Close-Bug: 1552506

Changed in steth:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers