← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1685944] Re: tools/net-convert: fix argument order for render_network_state

 

** Also affects: cloud-init
   Importance: Undecided
       Status: New

** Changed in: cloud-init
       Status: New => Confirmed

** No longer affects: cloud-init (Ubuntu)

** Changed in: cloud-init
   Importance: Undecided => Low

** Changed in: cloud-init
       Status: Confirmed => Fix Committed

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to cloud-init.
https://bugs.launchpad.net/bugs/1685944

Title:
  tools/net-convert: fix argument order for render_network_state

Status in cloud-init:
  Fix Committed

Bug description:
  % cat simple-v2.yaml
  network:
    version: 2
    # comment above ethernets
    ethernets:
        ens0:
           dhcp4: true
           match:
             macaddress: 00:11:22:33:44:55
           set-name: ens0
        switchports:
          # all cards on second PCI bus; unconfigured by themselves, will be added
          # to br0 below
          match:
            name: enp2*
          mtu: 1280

  % PYTHONPATH=`pwd` ./tools/net-convert.py --network-data simple-v2.yaml \
                                            --kind yaml \
                                            --output-kind netplan \
                                            --directory ./target

  Traceback (most recent call last):
    File "./tools/net-convert.py", line 82, in <module>
      main()
    File "./tools/net-convert.py", line 78, in main
      r.render_network_state(ns, target=args.directory)
  TypeError: render_network_state() got multiple values for argument 'target'

  
  This is broken on master.

To manage notifications about this bug go to:
https://bugs.launchpad.net/cloud-init/+bug/1685944/+subscriptions