The proposal to merge ~raharper/cloud-init:fix-net-convert into 
cloud-init:master has been updated.

Description changed to:

tools/net-convert: fix argument order for render_network_state

We're calling Renderer.render_network_state() with incorrect args.

% 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'

The method signature requires passing <target dir>, <network_state>.
This patch fixes the call order.

Fixes: LP:1685944

For more details, see:
https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/323083
-- 
Your team cloud init development team is requested to review the proposed merge 
of ~raharper/cloud-init:fix-net-convert into cloud-init:master.

_______________________________________________
Mailing list: https://launchpad.net/~cloud-init-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to