Bug 50685 - Make ImportFramework configurable in Kelvin Container
Summary: Make ImportFramework configurable 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: 50637
  Show dependency treegraph
 
Reported: 2020-01-06 10:20 CET by Ole Schwiegert
Modified: 2021-02-23 09:29 CET (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-01-06 10:20:20 CET
USER STORY
As a domain administrator, I would like to be able to configure the import framework in the Kelvin container as usual.

ACCEPTANCE CRITERION
The Config is created in the host UCS and mounted into the container
The existing configurations are loaded, as in the normal import, but there is an additional Kelvin-config (comparable to HTTP import config)
Overloading of classes is not supported
implementation notes
===
Json configfiles in the Docker Container
Configurability of the Import Framework => concept regarding compatibility
Tests (unit tests, not integration tests)
Configuration_check that "classes" to be overwritten are empty
Comment 1 Ole Schwiegert univentionstaff 2020-01-08 09:08:50 CET
ed3870579e00fa220492a07f87b20cbd070e51a7: Bug #50685: Add default kelvin config
dcf3fdd73b666dae2daff0aec3b7c231a1ec1baf: Bug #50685: Implement default kelvin config and optional kelvin config in /var/lib
Comment 2 Ole Schwiegert univentionstaff 2020-01-08 11:52:16 CET
The directory /var/lib/ucs-school-import/configs on the host will be mounted into the container as /var/lib/ucs-school-import/configs. This allows for direct configuration of the Kelvin APIs Import Framework via the hosts config files
Comment 3 Daniel Tröder univentionstaff 2020-01-09 13:08:20 CET
[feature/kelvin] 22c8d329f Bug #50685: Add class_overwrites check
[feature/kelvin] 6ad8b23b5 Bug #50685: Add config tests plus format
[feature/kelvin] 33aeb564c Bug #50685: Create empty import config for kelvin if non is present
[feature/kelvin] eb78a5b3f Bug #50685: Fix error in check

I added a fixture that allows to get rid of the Configuration singleton to completely reset the import configuration. I moved the paths to the constants module.

[feature/kelvin] 99bc3c373 Bug #50685: fix import config tests
[feature/kelvin] a9934077c Bug #50685: move paths to constants module
[feature/kelvin] 0580b8d26 Bug #50685: PS: move paths to constants module
[feature/kelvin] ce6f80a25 Bug #50685: fix format
Comment 4 Daniel Tröder univentionstaff 2020-01-22 12:49:53 CET
OK: settings in /var/lib/ucs-school-import/configs/kelvin.py are used in the "users" resource.
Comment 5 Daniel Tröder univentionstaff 2021-02-23 09:29:55 CET
A Kelvin API app with the changes in this bug has been published.