Bug 27829 - univention-ifplugd verwendet immer noch /etc/dhcp3/ statt /etc/dhcp/
univention-ifplugd verwendet immer noch /etc/dhcp3/ statt /etc/dhcp/
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Network
UCS 3.0
Other Linux
: P5 normal (vote)
: UCS 3.1
Assigned To: Philipp Hahn
Janek Walkenhorst
: interim-1
Depends on: 24282
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-04 20:29 CEST by Philipp Hahn
Modified: 2012-12-12 21:10 CET (History)
1 user (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): Troubleshooting
Max CVSS v3 score:
hahn: Patch_Available+


Attachments
univention-network-manager cleanup (23.55 KB, patch)
2012-07-04 20:59 CEST, Philipp Hahn
Details | Diff
univention-network-manager cleanup (38.22 KB, patch)
2012-07-05 07:55 CEST, Philipp Hahn
Details | Diff
univention-network-manager cleanup v3 (38.14 KB, patch)
2012-07-05 08:00 CEST, Philipp Hahn
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2012-07-04 20:29:56 CEST
Mit der Umstellung auf isc-dhcp-4.1 (Bug #22112, svn26810) liegen die Konfigurationsdateien für den dhclient jetzt unter /etc/dhcp/ statt früher /etc/dhcp3/. base/univention-network-manager scheint aber immer noch die alten Verzeichnisse zu benutzen:


$ git grep etc/dhcp3
conffiles/etc/dhcp/dhclient.conf:#script "/etc/dhcp3/dhclient-script";
debian/changelog:  * Renamed path etc/dhcp3 in original patch to etc/dhcp to make it fit.
debian/rules:   chmod 755 debian/univention-ifplugd/etc/dhcp3/dhclient-enter-hooks.d/univention-ifplugd-enter-hook
debian/univention-ifplugd.dirs:etc/dhcp3/dhclient-enter-hooks.d
debian/univention-ifplugd.dirs:etc/dhcp3/dhclient-enter-hooks.univention.d
debian/univention-ifplugd.install:univention-ifplugd-enter-hook etc/dhcp3/dhclient-enter-hooks.d/
univention-ifplugd-enter-hook:HOOKS_DIR=/etc/dhcp3/dhclient-enter-hooks.univention.d/
Comment 1 Stefan Gohmann univentionstaff 2012-07-04 20:43:38 CEST
Oder ist dann ein direktes Problem? Wenn ja, dann sollten wir das noch zur 3.0-2 beheben.
Comment 2 Philipp Hahn univentionstaff 2012-07-04 20:59:02 CEST
Created attachment 4500 [details]
univention-network-manager cleanup

Die UCR-Variablen waren nicht in beiden Dateien debian/univention-{ifplugd,network-manager}.univention-config-registry{,-variables} definiert bzw. registriert. Das ganze wird in das bereits vorhandene -common-Paket verschoben.

Im Template für /etc/dhcp/dhclient.conf haben die letzten Anpassungen aus Debian gefehlt: domain-search, rfc3442-classless-static-routes, ntp-servers

00_fallback.py wurde in (Bug #26416) zwar aus dem Paket entfernt, da es sicher aber im ein sog. conffile handelt, bleibt die Datei bestehen. Das wird per dpkg-maintscript-helper aufgeräumt.

/etc/dhcp3/dhclient-enter-hooks.d/univention-ifplugd-enter-hook wird nach /etc/dhcp/ verschoben.

Einige hook-Skripte loggen hart kodiert in /tmp/-Dateien, was einen Symlink-Angriff erlaubt. Das wurde nach /var/run/ verschoben, was nicht für alle Schreibbar ist. (Statt "set" wird "env" verwendet, weil ersteres auch die ganzen Bash-Funktionen anzeigt).

Einige Whitespace-Korrekturen wie space-before-tab, white-space-at-end-of-line/file
Comment 3 Philipp Hahn univentionstaff 2012-07-05 07:55:32 CEST
Created attachment 4501 [details]
univention-network-manager cleanup

git rebase
git split
+ Bug #25377
+ Bug #17802
Comment 4 Philipp Hahn univentionstaff 2012-07-05 08:00:29 CEST
Created attachment 4502 [details]
univention-network-manager cleanup v3

Fix services/univention-dhcp/debian/changelog
Comment 5 Stefan Gohmann univentionstaff 2012-07-17 17:09:31 CEST
UCS 3.1 will be the next release.
Comment 6 Philipp Hahn univentionstaff 2012-08-08 11:17:24 CEST
Patch wurde eingespielt.

svn34708, univention-network-manager_4.0.0-1.88.201208081114

Changelog: svn14243
\item Now all DHCP configuration are relocated from \ucsFile{/etc/dhcp3/} to \ucsFile{/etc/dhcp/} in \ucsUCSShort{-3.0} (\ucsBug{27829}).
Comment 7 Janek Walkenhorst univentionstaff 2012-09-12 18:31:23 CEST
Nach Neuinstallation sind Dateien nur unter /etc/dhcp/: OK
Comment 8 Janek Walkenhorst univentionstaff 2012-09-13 13:33:12 CEST
Nach Update sind Dateien nur noch unter /etc/dhcp/: OK
(Kleine Ausnahme ist eine veraltete /etc/dhcp3/dhclient.conf, die jedoch keine Auswirkung hat)

Code review: OK

Changelog: OK
Comment 9 Stefan Gohmann univentionstaff 2012-12-12 21:10:35 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

If this error occurs again, please use "Clone This Bug".