Home · About · Download · Documentation · Getting Help · Google+

Ticket #28 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

"node -s hwaddr=..." and "node set --hwaddr=..." behave differently

Reported by: yqin Owned by: gmk
Priority: minor Milestone: Beta release
Component: general Version: 0.1
Keywords: Cc:

Description

  1. Warewulf> node print n0001

#### n0001 ####################################################################

n0001: ID = 25
n0001: NAME = n0001
n0001: HWADDR = 52:54:00:52:34:61
n0001: TYPE = node
n0001: NETDEVS = netdev(NAME=eth0,IPADDR=10.0.0.101,HWADDR=52:54:00:52:34:61)

  1. Warewulf> node set n0001 -s hwaddr=aa:bb:cc:dd:ee:ff

Are you sure you want to make the following changes to 1 node(s):

SET: hwaddr = aa:bb:cc:dd:ee:ff

Yes/No> yes

  1. Warewulf> node print n0001

#### n0001 ####################################################################

n0001: ID = 25
n0001: NAME = n0001
n0001: TYPE = node
n0001: HWADDR = aa:bb:cc:dd:ee:ff
n0001: NETDEVS = netdev(NAME=eth0,IPADDR=10.0.0.101,HWADDR=52:54:00:52:34:61)

  1. Warewulf> node set n0001 --hwaddr=aa:bb:cc:dd:ee:ff

Are you sure you want to make the following changes to 1 node(s):

SET: HWADDR = aa:bb:cc:dd:ee:ff

Yes/No> yes

  1. Warewulf> node print n0001

#### n0001 ####################################################################

n0001: ID = 25
n0001: NAME = n0001
n0001: HWADDR = aa:bb:cc:dd:ee:ff
n0001: TYPE = node
n0001: NETDEVS = netdev(NAME=eth0,IPADDR=10.0.0.101,HWADDR=aa:bb:cc:dd:ee:ff)

  1. Warewulf> node set n0001 --hwaddr=01:02:03:04:05:06

Are you sure you want to make the following changes to 1 node(s):

SET: HWADDR = 01:02:03:04:05:06

Yes/No> yes

  1. Warewulf> node print n0001

#### n0001 ####################################################################

n0001: ID = 25
n0001: NAME = n0001
n0001: TYPE = node
n0001: HWADDR = 01:02:03:04:05:06,aa:bb:cc:dd:ee:ff
n0001: NETDEVS = netdev(NAME=eth0,IPADDR=10.0.0.101,HWADDR=01:02:03:04:05:06)

  1. Warewulf> node set n0001 -s hwaddr=aa:bb:cc:dd:ee:ff

Are you sure you want to make the following changes to 1 node(s):

SET: hwaddr = aa:bb:cc:dd:ee:ff

Yes/No> yes

  1. Warewulf> node print n0001

#### n0001 ####################################################################

n0001: ID = 25
n0001: NAME = n0001
n0001: HWADDR = aa:bb:cc:dd:ee:ff
n0001: TYPE = node
n0001: NETDEVS = netdev(NAME=eth0,IPADDR=10.0.0.101,HWADDR=01:02:03:04:05:06)

From above command sequence, #2 updates the value of "hwaddr" but does not update the value in "netdevs"; #4 corrects it. #6 adds one new value to "hwaddr" key instead of replaces it (since there's only one device in this context). #8 is the same as #2. It sounds to me that "-s hwaddr=..." needs a persist() call. We also need more sanity checkings for both methods.

Change History

comment:1 Changed 7 years ago by gmk

  • Status changed from new to closed
  • Resolution set to fixed

This should no longer be an issue with release [459].

Note: See TracTickets for help on using tickets.