I have the time, will look into it now
On Thu, Nov 14, 2013 at 2:15 PM, Thomas Thrainer <thoma...@google.com>wrote: > Has anybody some time to look into this? That's the last patch series > which has to be pushed before a 2.10 beta can happen... > > Cheers, > Thomas > > > On Wed, Nov 6, 2013 at 2:32 PM, Thomas Thrainer <thoma...@google.com>wrote: > >> This patch series fixes a couple of OpenVSwitch related bugs, >> most notably issue 584. The biggest change is to make all node >> parameters mandatory with sensible defaults (like it is done >> with all other parameters), which makes dealing with those >> parameters easier and less error-prone. >> >> Thomas Thrainer (5): >> Use constant instead of raw string >> Pass VLAN parameter correctly in moves >> Make NIC VLAN queryable >> Move OVS node parameters to the right place >> Don't allow optional node parameters >> >> lib/bootstrap.py | 2 +- >> lib/client/gnt_node.py | 33 +++++----------------------- >> lib/cmdlib/cluster.py | 8 ------- >> lib/cmdlib/instance.py | 4 ---- >> lib/cmdlib/node.py | 44 >> ++++++++++++++++--------------------- >> lib/query.py | 21 ++++++++++++++++++ >> lib/utils/__init__.py | 5 ----- >> src/Ganeti/HsConstants.hs | 11 ++++------ >> src/Ganeti/Objects.hs | 2 +- >> test/py/cmdlib/instance_unittest.py | 9 -------- >> test/py/cmdlib/node_unittest.py | 13 ++--------- >> test/py/ganeti.utils_unittest.py | 3 +-- >> tools/move-instance | 2 +- >> 13 files changed, 55 insertions(+), 102 deletions(-) >> >> -- >> 1.8.4.1 >> >> > > > -- > Thomas Thrainer | Software Engineer | thoma...@google.com | > > Google Germany GmbH > Dienerstr. 12 > 80331 München > > Registergericht und -nummer: Hamburg, HRB 86891 > Sitz der Gesellschaft: Hamburg > Geschäftsführer: Graham Law, Christine Elizabeth Flores > Hrvoje Ribicic Ganeti Engineering Google Germany GmbH Dienerstr. 12, 80331, München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores Steuernummer: 48/725/00206 Umsatzsteueridentifikationsnummer: DE813741370