Bug 36623 - local networks only parsed from "interfaces/eth"
local networks only parsed from "interfaces/eth"
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Squid
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 4.0-1-errata
Assigned To: Philipp Hahn
Janek Walkenhorst
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-13 10:02 CET by Janis Meybohm
Modified: 2015-03-25 16:42 CET (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Janis Meybohm univentionstaff 2014-11-13 10:02:57 CET
2014111221000249

squid.conf template does iterate over "interfaces/eth" to figure out the "local neworks". In times of bug26058 this is insufficient :-)
Comment 1 Florian Best univentionstaff 2014-11-13 11:15:43 CET
from univention.management.console.modules.setup.network import Interfaces
i = Interfaces()
print i.items()

→ we could move the lib from system-setup into univention.lib. It is capable of everything which have to do with bonding, bridging, vlan, ethernet. It is generic so that it does not depend on the rest of system setup.
Comment 2 Philipp Hahn univentionstaff 2015-03-15 16:54:30 CET
r59012 | Bug #36623 squid: Support interfaces other then ethX
 Use univention.config_registry.interfaces#Interfaces

Package: univention-squid
Version: 8.0.2-3.225.201503151642
Branch: ucs_4.0-0
Scope: errata4.0-1

r59014 | Bug #36623 squid: Support interfaces other then ethX YAML
 2015-03-13-univention-squid.yaml

QA:
ucr set interfaces/restart/auto=false interfaces/br0/{address=1.2.3.4,netmask=255.0.0.0,network=1.0.0.0,broadcast=1.255.255.255}
ucr filter < squid.conf | grep localnet
Comment 3 Janek Walkenhorst univentionstaff 2015-03-24 09:47:55 CET
Code review: OK
Tests: OK
Advisory: OK
Comment 4 Janek Walkenhorst univentionstaff 2015-03-25 16:42:30 CET
<http://errata.univention.de/ucs/4.0/126.html>