Bug 55925 - Do not run join scripts within pre-join phase
Do not run join scripts within pre-join phase
Status: NEW
Product: UCS
Classification: Unclassified
Component: Join (univention-join)
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS maintainers
UCS maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2023-03-31 09:52 CEST by Jürn Brodersen
Modified: 2023-03-31 09:53 CEST (History)
0 users

See Also:
What kind of report is it?: Feature Request
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 Jürn Brodersen univentionstaff 2023-03-31 09:52:00 CEST
Do not run join scripts within pre-join phase

We are installing ucs@school packages within the pre-join phase, our intention is to have all school join scripts already installed before the initial join starts. This works nicely on a replica, because no join scripts are executed during a package install.

On backups however the join scripts are executed directly during the package installation. This results in timeouts and errors on backups during the ucs@school package installation because the system isn't joined yet.

My idea is that we do not run join scripts in the pre-join phase.

Maybe add a check here:
https://git.knut.univention.de/univention/ucs/-/blob/5.0-3/management/univention-join/join.sh#L80

This should fix bug 54716, but I keep it open for now in case we want to fix that in another way.

Acceptance criteria:
- [ ] join scripts are not run within pre-join phase
- [ ] documentation is updated
- [ ] Maybe a test, but the ucs@school tests might be considered sufficient
Comment 1 Jürn Brodersen univentionstaff 2023-03-31 09:53:30 CEST
^ Wrong bug number 54716 -> 54715