Multiple networks support and a few readability tweaks #1

Merged
merged 6 commits into from Jan 30, 2017

Conversation

Projects
None yet
2 participants
Contributor

dimitern commented Oct 18, 2016

Order of arguments to kvm-maas-add-node has been altered to put
networks (one or more) at the end. For kvm-maas-remove-node, a
list of networks is accepted for compatibility, but it's ignored.

Contributor

dimitern commented Oct 19, 2016

@frobware So how about this PR ? I already have a couple of ideas for follow-ups :)

Owner

frobware commented Jan 30, 2017

There's an issue with the order of networks - the first should be the PXE network. I'll merge and fixup afterwards.

@frobware frobware closed this Jan 30, 2017

@frobware frobware reopened this Jan 30, 2017

Owner

frobware commented Jan 30, 2017

Thanks and apologies for the delay.

@frobware frobware merged commit 67defc6 into frobware:master Jan 30, 2017

@dimitern dimitern deleted the dimitern:multiple-networks branch Jan 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment