{"action":"closed","issue":{"url":"https://api.github.com/repos/xcat2/xcat-core/issues/641","repository_url":"https://api.github.com/repos/xcat2/xcat-core","labels_url":"https://api.github.com/repos/xcat2/xcat-core/issues/641/labels{/name}","comments_url":"https://api.github.com/repos/xcat2/xcat-core/issues/641/comments","events_url":"https://api.github.com/repos/xcat2/xcat-core/issues/641/events","html_url":"https://github.com/xcat2/xcat-core/issues/641","id":127898396,"node_id":"MDU6SXNzdWUxMjc4OTgzOTY=","number":641,"title":"[FVT] Wrong link members of the link aggregation interface created by postscript confignetwork, when node definition changed","user":{"login":"neo954","id":12424088,"node_id":"MDQ6VXNlcjEyNDI0MDg4","avatar_url":"https://avatars0.githubusercontent.com/u/12424088?v=4","gravatar_id":"","url":"https://api.github.com/users/neo954","html_url":"https://github.com/neo954","followers_url":"https://api.github.com/users/neo954/followers","following_url":"https://api.github.com/users/neo954/following{/other_user}","gists_url":"https://api.github.com/users/neo954/gists{/gist_id}","starred_url":"https://api.github.com/users/neo954/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/neo954/subscriptions","organizations_url":"https://api.github.com/users/neo954/orgs","repos_url":"https://api.github.com/users/neo954/repos","events_url":"https://api.github.com/users/neo954/events{/privacy}","received_events_url":"https://api.github.com/users/neo954/received_events","type":"User","site_admin":false},"labels":[{"id":310576846,"node_id":"MDU6TGFiZWwzMTA1NzY4NDY=","url":"https://api.github.com/repos/xcat2/xcat-core/labels/component:postscripts","name":"component:postscripts","color":"fbca04","default":false},{"id":243855099,"node_id":"MDU6TGFiZWwyNDM4NTUwOTk=","url":"https://api.github.com/repos/xcat2/xcat-core/labels/priority:normal","name":"priority:normal","color":"009800","default":false},{"id":400624883,"node_id":"MDU6TGFiZWw0MDA2MjQ4ODM=","url":"https://api.github.com/repos/xcat2/xcat-core/labels/status:won't%20fix","name":"status:won't fix","color":"006b75","default":false},{"id":243855749,"node_id":"MDU6TGFiZWwyNDM4NTU3NDk=","url":"https://api.github.com/repos/xcat2/xcat-core/labels/type:feature","name":"type:feature","color":"c7def8","default":false}],"state":"closed","locked":false,"assignee":{"login":"bybai","id":6743951,"node_id":"MDQ6VXNlcjY3NDM5NTE=","avatar_url":"https://avatars2.githubusercontent.com/u/6743951?v=4","gravatar_id":"","url":"https://api.github.com/users/bybai","html_url":"https://github.com/bybai","followers_url":"https://api.github.com/users/bybai/followers","following_url":"https://api.github.com/users/bybai/following{/other_user}","gists_url":"https://api.github.com/users/bybai/gists{/gist_id}","starred_url":"https://api.github.com/users/bybai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bybai/subscriptions","organizations_url":"https://api.github.com/users/bybai/orgs","repos_url":"https://api.github.com/users/bybai/repos","events_url":"https://api.github.com/users/bybai/events{/privacy}","received_events_url":"https://api.github.com/users/bybai/received_events","type":"User","site_admin":false},"assignees":[{"login":"bybai","id":6743951,"node_id":"MDQ6VXNlcjY3NDM5NTE=","avatar_url":"https://avatars2.githubusercontent.com/u/6743951?v=4","gravatar_id":"","url":"https://api.github.com/users/bybai","html_url":"https://github.com/bybai","followers_url":"https://api.github.com/users/bybai/followers","following_url":"https://api.github.com/users/bybai/following{/other_user}","gists_url":"https://api.github.com/users/bybai/gists{/gist_id}","starred_url":"https://api.github.com/users/bybai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bybai/subscriptions","organizations_url":"https://api.github.com/users/bybai/orgs","repos_url":"https://api.github.com/users/bybai/repos","events_url":"https://api.github.com/users/bybai/events{/privacy}","received_events_url":"https://api.github.com/users/bybai/received_events","type":"User","site_admin":false}],"milestone":null,"comments":9,"created_at":"2016-01-21T10:55:59Z","updated_at":"2018-06-19T08:40:52Z","closed_at":"2018-06-19T08:40:52Z","author_association":"CONTRIBUTOR","body":"Below were the `nicdevices` and `nictypes` attributes of node `c910f02c03p23` at beginning.\n\n```\n# lsdef c910f02c03p23 -i nicdevices,nictypes\nObject name: c910f02c03p23\n    nicdevices.bond0=eth2|eth1\n    nictypes.eth2=ethernet\n    nictypes.bond0=bond\n    nictypes.eth1=ethernet\n    nictypes.eth0=ethernet\n```\n\nAfter created the definition, command `updatenode c910f02c03p23 confignetwork` was run. And it run successfully. A link aggregation interface named `bond0` was created, with `eth2` and `eth1` as its link members.\n\nAfter that, the following command were run, and changed the node definition a little.\n\n```\n# chdef c910f02c03p23 nicdevices.bond0=\"eth0|eth1\"\n1 object definitions have been created or modified.\n# lsdef c910f02c03p23 -i nicdevices,nictypes\nObject name: c910f02c03p23\n    nicdevices.bond0=eth0|eth1\n    nictypes.eth2=ethernet\n    nictypes.bond0=bond\n    nictypes.eth1=ethernet\n    nictypes.eth0=ethernet\n```\n\nAnd then, command `updatenode c910f02c03p23 confignetwork` was run again.\n\nHere is what happened after all. A link aggregation interface `bond0` is created with all `eth0`, `eth1` and `eth2` as its link members, instead of the correct behavior, just have `eth0` and `eth1` as its link members.\n\nThis behavior is incorrect, and should be fixed definitely.\n\n```\n[root@c910f02c03p23 ~]# cat /proc/net/bonding/bond0\nEthernet Channel Bonding Driver: v3.7.1 (April 27, 2011)\n\nBonding Mode: IEEE 802.3ad Dynamic link aggregation\nTransmit Hash Policy: layer2 (0)\nMII Status: up\nMII Polling Interval (ms): 100\nUp Delay (ms): 0\nDown Delay (ms): 0\n\n802.3ad info\nLACP rate: slow\nMin links: 0\nAggregator selection policy (ad_select): stable\nActive Aggregator Info:\n    Aggregator ID: 2\n    Number of ports: 1\n    Actor Key: 9\n    Partner Key: 9\n    Partner Mac Address: 16:3f:42:91:ed:04\n\nSlave Interface: eth0\nMII Status: up\nSpeed: 1000 Mbps\nDuplex: full\nLink Failure Count: 0\nPermanent HW addr: 16:3f:4d:b4:fe:03\nSlave queue ID: 0\nAggregator ID: 1\nActor Churn State: monitoring\nPartner Churn State: monitoring\nActor Churned Count: 0\nPartner Churned Count: 0\ndetails actor lacp pdu:\n    system priority: 65535\n    port key: 9\n    port priority: 255\n    port number: 1\n    port state: 77\ndetails partner lacp pdu:\n    system priority: 65535\n    oper key: 1\n    port priority: 255\n    port number: 1\n    port state: 1\n\nSlave Interface: eth1\nMII Status: up\nSpeed: 1000 Mbps\nDuplex: full\nLink Failure Count: 0\nPermanent HW addr: 16:3f:4d:b4:fe:04\nSlave queue ID: 0\nAggregator ID: 2\nActor Churn State: monitoring\nPartner Churn State: monitoring\nActor Churned Count: 0\nPartner Churned Count: 0\ndetails actor lacp pdu:\n    system priority: 65535\n    port key: 9\n    port priority: 255\n    port number: 2\n    port state: 13\ndetails partner lacp pdu:\n    system priority: 65535\n    oper key: 9\n    port priority: 255\n    port number: 1\n    port state: 5\n\nSlave Interface: eth2\nMII Status: up\nSpeed: Unknown\nDuplex: Unknown\nLink Failure Count: 0\nPermanent HW addr: 00:21:5e:a9:54:56\nSlave queue ID: 0\nAggregator ID: 3\nActor Churn State: monitoring\nPartner Churn State: monitoring\nActor Churned Count: 0\nPartner Churned Count: 0\ndetails actor lacp pdu:\n    system priority: 65535\n    port key: 0\n    port priority: 255\n    port number: 3\n    port state: 69\ndetails partner lacp pdu:\n    system priority: 65535\n    oper key: 1\n    port priority: 255\n    port number: 1\n    port state: 1\n```\n"}}
{"actor":{"display_login":"zet809"}}