Bug 53172 - DHCP build broken
DHCP build broken
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: DHCP
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0
Assigned To: Philipp Hahn
Jürn Brodersen
: interim-1
Depends on: 51340
Blocks:
  Show dependency treegraph
 
Reported: 2021-04-27 14:06 CEST by Philipp Hahn
Modified: 2021-05-25 15:58 CEST (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 3: A User would likely not purchase the product
User Pain: 0.257
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 Philipp Hahn univentionstaff 2021-04-27 14:06:22 CEST
patches/20_restart-univention-dhcp.patch is no longer needed

manual pages are not included

...
Comment 1 Philipp Hahn univentionstaff 2021-04-27 18:15:04 CEST
r19363 | Bug #53172: isc-dhcp

Merge 21 and 22

Drop useless "export": All those environment variables are exported by dhclient
as part of the API between it and the dhclient-script. As the script does not
change any of those variables they need no re-export.

Drop broken check for hook scripts reporting != 0: The old code "exit $?"
always returned "0" as "!" inverted the failed result from "run_hook".
Aborting the script on any failed hook is dangerous as any hook doing a
"if cond; then ...; fi" would abort when "cond" is not met.
If any script wants to abort further processing it simply could use "exit" as
the hooks are sources by "dhclient-script", e.g. not executed in a sub-shell.

Refactor code to output "option:timeout" for /etc/resolv.conf

Package: isc-dhcp
Version: 4.4.1-2A~5.0.0.202104271757

OK: for d in /var/univention/buildsystem2/apt/ucs_5.0-0/amd64/isc-dhcp-*.deb; do dpkg -c "$d"; done | grep -F /usr/share/man/

[phahn/dhcp] a21846aaf0 Bug #53175: univention-dhcp
 doc/changelog/changelog-5.0-0.xml | 4 ++++
 1 file changed, 4 insertions(+)
Comment 2 Jürn Brodersen univentionstaff 2021-05-06 11:53:51 CEST
What I tested:
Variables are exported (tested with dhclient debug hook) -> OK
Hook scripts are executed, "exit 1" stops processing -> OK
Refactor code to output "option:timeout" (moved resolvconf hook away to test this) -> OK
Changelog -> OK

-> Verified
Comment 3 Florian Best univentionstaff 2021-05-25 15:58:44 CEST
UCS 5.0 has been released:
 https://docs.software-univention.de/release-notes-5.0-0-en.html
 https://docs.software-univention.de/release-notes-5.0-0-de.html

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