Bug 51768 - Make UCRV needed for Bug #51724 available in Kelvin Container
Summary: Make UCRV needed for Bug #51724 available in Kelvin Container
Status: CLOSED FIXED
Alias: None
Product: UCS@school
Classification: Unclassified
Component: HTTP-API (Kelvin)
Version: UCS@school 4.4
Hardware: Other Linux
: P5 normal
Target Milestone: ---
Assignee: Ole Schwiegert
QA Contact: Daniel Tröder
URL:
Keywords:
Depends on:
Blocks: 51724
  Show dependency treegraph
 
Reported: 2020-08-04 12:53 CEST by Ole Schwiegert
Modified: 2020-08-13 08:43 CEST (History)
1 user (show)

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):
Customer ID:
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ole Schwiegert univentionstaff 2020-08-04 12:53:13 CEST
To make the new hook, implemented in Bug #51724 work, we need access to the following UCRV in the Kelvin Container:

        ucsschool/import/set/netlogon/script/path
        ucsschool/import/set/homedrive
        ucsschool/import/set/sambahome
        ucsschool/singlemaster
        ucsschool/import/set/serverprofile/path

To achieve this we decided to copy them into the container in the joinscript. Whenever the values change, the joinscript has to be run again, so the changes are copied into the Kelvin Container again.

To solve the general problem of accessing the hosts UCR in docker containers see Bug #51726
Comment 1 Ole Schwiegert univentionstaff 2020-08-04 12:54:51 CEST
Joinscript extended
README Update DE/EN extended
new version in testappcenter added


The update of the App to 1.1.2 currently fails. I will need some help to determine the cause of that. I probably made a mistake configuring the new app version.
Comment 2 Ole Schwiegert univentionstaff 2020-08-07 10:48:15 CEST
Apparently an App update does not work without supplying a new docker image.

I created a docker image for 1.1.2 which uses alpine 3.12 (updated) and thus uses python 3.8

Update works now and the UCR variables get synced into the container.

Please QA and reopen for Kelvin release
Comment 3 Daniel Tröder univentionstaff 2020-08-11 11:41:08 CEST
OK: in-container UCR has variables set like in host
OK: after changing UCR has variables in host and running the join script, in-container UCR variables are updated
OK: in-container tests:

/kelvin # . venv/bin/activate
(venv) /kelvin # cd ucs-school-lib/modules/ucsschool/lib/
(venv) /kelvin/ucs-school-lib/modules/ucsschool/lib # python3 -m pytest -l -v
[..]
40 passed, 2 xfailed, 518 warnings

(venv) /kelvin # cd /kelvin/kelvin-api
(venv) /kelvin/kelvin-api # python3 -m pytest -l -v
[..]
215 passed, 814 warnings

OK: README_UPDATE_*
OK: update of image base (alpine 3.12)
OK: update from app version 1.1.1 to 1.1.2
Comment 4 Daniel Tröder univentionstaff 2020-08-13 08:43:46 CEST
Kelvin App version 1.1.2 has been released.