Project

General

Profile

Bug #3319

py-vici: get_pools() TypeError: get_pools() missing 1 required positional argument: 'options'

Added by Dries Michiels 3 months ago. Updated 10 days ago.

Status:
Closed
Priority:
Normal
Category:
vici
Target version:
Start date:
Due date:
Estimated time:
Affected version:
5.8.2
Resolution:
Fixed

Description

Python 3.7.6 (default, Dec 24 2019, 14:52:26)
[Clang 8.0.1 (tags/RELEASE_801/final 366581)] on freebsd12
Type "help", "copyright", "credits" or "license" for more information.
>>> import vici
>>> s = vici.Session()
>>> s.version()
OrderedDict([('daemon', b'charon'), ('version', b'5.8.2'), ('sysname', b'FreeBSD'), ('release', b'12.1-STABLE'), ('machine', b'amd64')])
>>> s.get_conns()
OrderedDict([('conns', [b'ikev2-eap-mschapv2'])])
>>> s.get_pools()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: get_pools() missing 1 required positional argument: 'options'

Python2.7 errors out on:

s.get_pools()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: get_pools() takes exactly 2 arguments (1 given)

Associated revisions

Revision f78dfb7e (diff)
Added by Tobias Brunner 2 months ago

vici: Options are optional in get_pools() of Python bindings

Fixes #3319.

History

#1 Updated by Noel Kuntze 3 months ago

  • Status changed from New to Feedback

Pass options=None for now, that should work.

#2 Updated by Tobias Brunner 3 months ago

  • Tracker changed from Issue to Bug
  • Description updated (diff)
  • Assignee set to Tobias Brunner
  • Target version set to 5.8.3
  • Resolution set to Fixed

Pushed a fix to the 3319-vici-pools branch.

#3 Updated by Dries Michiels 2 months ago

Workaround and patch work fine thanks!

#4 Updated by Tobias Brunner 2 months ago

  • Status changed from Feedback to Closed

#5 Updated by Dries Michiels 12 days ago

Hi, could you release 5.8.3 on pypi? That way I can pull it in on FreeBSD. Thanks!

#6 Updated by Tobias Brunner 10 days ago

Hi, could you release 5.8.3 on pypi?

5.8.4 is now on PyPI.

Also available in: Atom PDF