| Message ID | 20180314150744.25060-1-russell@ovn.org |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [ovs-dev] python: Enable flake8 checking of ovs-pipegen.py. | expand |
On Wed, Mar 14, 2018 at 11:07:44AM -0400, Russell Bryant wrote: > Signed-off-by: Russell Bryant <russell@ovn.org> flake8-check passes for me, with this change. Acked-by: Ben Pfaff <blp@ovn.org>
On Wed, Mar 14, 2018 at 3:48 PM, Ben Pfaff <blp@ovn.org> wrote: > On Wed, Mar 14, 2018 at 11:07:44AM -0400, Russell Bryant wrote: > > Signed-off-by: Russell Bryant <russell@ovn.org> > > flake8-check passes for me, with this change. > > Acked-by: Ben Pfaff <blp@ovn.org> > ​Thanks! applied to master.
diff --git a/utilities/automake.mk b/utilities/automake.mk index 1636cb93e..60cf1c5ed 100644 --- a/utilities/automake.mk +++ b/utilities/automake.mk @@ -146,6 +146,7 @@ endif FLAKE8_PYFILES += utilities/ovs-pcap.in \ utilities/checkpatch.py utilities/ovs-dev.py \ - utilities/ovs-tcpdump.in + utilities/ovs-tcpdump.in \ + utilities/ovs-pipegen.py include utilities/bugtool/automake.mk diff --git a/utilities/ovs-pipegen.py b/utilities/ovs-pipegen.py index f040a7e1b..8a2a4266e 100755 --- a/utilities/ovs-pipegen.py +++ b/utilities/ovs-pipegen.py @@ -18,6 +18,7 @@ import random import sys import textwrap + def flow_str(stage, match, action, priority=32768): mtd_match = "metadata=%d" % stage if match: @@ -109,11 +110,10 @@ def main(): """ % sys.argv[0]) parser = argparse.ArgumentParser(description=description, epilog=epilog, - formatter_class=\ - argparse.RawDescriptionHelpFormatter) + formatter_class=argparse.RawDescriptionHelpFormatter) parser.add_argument("--size", dest="size", default=1000, help="Size (rules) of each OpenFlow table.") - args=parser.parse_args() + args = parser.parse_args() pipeline(int(args.size))
Signed-off-by: Russell Bryant <russell@ovn.org> --- utilities/automake.mk | 3 ++- utilities/ovs-pipegen.py | 6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-)