Bug 13559 - [systemd] only start smb when network interfaces are up
Summary: [systemd] only start smb when network interfaces are up
Status: RESOLVED FIXED
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: Other (show other bugs)
Version: 4.9.0rc1
Hardware: All Linux
: P5 minor (vote)
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-03 12:33 UTC by Oleksandr Natalenko
Modified: 2018-08-14 11:17 UTC (History)
2 users (show)

See Also:

Attachments
patch for 4.9 (1.36 KB, patch)
2018-08-07 09:08 UTC, Andreas Schneider 		ab: review+
Details
patch for 4.8 (1.36 KB, patch)
2018-08-07 09:09 UTC, Andreas Schneider 		ab: review+
Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Oleksandr Natalenko 2018-08-03 12:33:43 UTC 
For smb, if the smb.conf contains explicit bindings to the network interfaces, the service must wait till network interfaces are up, otherwise the service won't be operational.

The 0e571054a61e commit and the BZ 13184 have fixed this for nmb and samba, so do exactly the same here, for smb.
Comment 1 Oleksandr Natalenko 2018-08-03 12:36:45 UTC 
Submission: https://lists.samba.org/archive/samba-technical/2018-August/129509.html
Comment 2 Andreas Schneider 2018-08-07 09:08:58 UTC 
Created attachment 14396 [details]
patch for 4.9
Comment 3 Andreas Schneider 2018-08-07 09:09:57 UTC 
Created attachment 14397 [details]
patch for 4.8
Comment 4 Alexander Bokovoy 2018-08-07 13:06:02 UTC 
Comment on attachment 14397 [details]
patch for 4.8

LGTM
Comment 5 Alexander Bokovoy 2018-08-07 13:06:35 UTC 
Comment on attachment 14396 [details]
patch for 4.9

LGTM
Comment 6 Andreas Schneider 2018-08-07 13:47:50 UTC 
Karolin, please add the patches to the relevant branches. Thanks!
Comment 7 Karolin Seeger 2018-08-13 10:50:28 UTC 
(In reply to Andreas Schneider from comment #6)
Pushed to autobuild-v4-{9,8}-test.
Comment 8 Karolin Seeger 2018-08-14 11:17:53 UTC 
(In reply to Karolin Seeger from comment #7)
Pushed to both branches.
Closing out bug report.

Thanks!