string/tuple unpacking bug in new rewrite
Bug #1517607 reported by
Simon Davy
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
postgresql (Juju Charms Collection) |
Fix Released
|
Undecided
|
Simon Davy |
Bug Description
Trying the new rewrite in a manually added lxd container, there's a traceback
Log here:
http://
I'm not sure as to the cause, I will try an investigate some more.
summary: |
- string/tuple unbpacking bug in new rewrite + string/tuple unpacking bug in new rewrite |
Changed in postgresql (Juju Charms Collection): | |
assignee: | nobody → Simon Davy (bloodearnest) |
status: | New → Fix Released |
To post a comment you must log in.
On line 440 of service.py, rules was thus:
(Pdb) pprint(rules)
[('### BEGIN JUJU SETTINGS ###',),
('local', 'all', 'postgres', 'peer', 'map=juju_charm'),
('local', 'all', 'all', 'peer'),
('host', 'all', 'all', '10.0.0.0/8', 'md5', '# admin_addresses config'),
('host', 'all', 'all', '127.0.0.1/32', 'md5', '# admin_addresses config'),
(('local all sso md5,local all ssoadmin md5', '# extra_pg_auth config'),),
('local', 'all', 'all', 'reject', '# Refuse by default'),
('host', 'all', 'all', 'all', 'reject', '# Refuse by default'),
('### END JUJU SETTINGS ###',)]
So, this is specific config that I've added to the charm to allow ssoadmin to use the local socket like so:
juju set sso-postgresql extra_pg_ conf="local all sso md5,local all ssoadmin md5"
Is it possible there's a regression here?
Line service.py:414 has the following:
# And anything-goes rules, if specified by the operator. 'extra_ pg_auth' ].splitlines( ):
for line in config[
add((line, '# extra_pg_auth config'))
Should that split on ',' rather than lines? Or is this is intended as a breaking change?