New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gnng --csv crashes #250

Closed
jathanism opened this Issue Feb 25, 2016 · 1 comment

Comments

Projects
None yet
1 participant
@jathanism
Member

jathanism commented Feb 25, 2016

This:

$ gnng --csv arista-sw1
Traceback (most recent call last):
  File "/Users/jathan/sandbox/virtualenvs/trigger/bin/gnng", line 459, in <module>
    main()
  File "/Users/jathan/sandbox/virtualenvs/trigger/bin/gnng", line 450, in main
    handle_output(all_rows, opts)
  File "/Users/jathan/sandbox/virtualenvs/trigger/bin/gnng", line 366, in handle_output
    writer.writerow([dev] + row)
NameError: global name 'dev' is not defined

@jathanism jathanism added the bug label Feb 25, 2016

@jathanism

This comment has been minimized.

Member

jathanism commented Feb 25, 2016

So does --sqldb:

$ gnng --sqldb foo.db arista-sw1
DEVICE: arista-sw1
Traceback (most recent call last):
  File "/Users/jathan/sandbox/virtualenvs/trigger/bin/gnng", line 459, in <module>
    main()
  File "/Users/jathan/sandbox/virtualenvs/trigger/bin/gnng", line 450, in main
    handle_output(all_rows, opts)
  File "/Users/jathan/sandbox/virtualenvs/trigger/bin/gnng", line 370, in handle_output
    write_sqldb(opts.sqldb, dev, rows)
NameError: global name 'dev' is not defined

@johnfzc johnfzc closed this in 6ec4747 Feb 25, 2016

johnfzc added a commit that referenced this issue Feb 25, 2016

Merge pull request #251 from jathanism/gnng-fix
Fixes #250 - Bugfix to bin/gnng that would crash on -c or -s output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment