linuxmuster / linuxmuster-linuxclient7

The new approach on connecting linuxclients to the Linuxmuster.net v7 Active Directory server.
3 stars 5 forks source link

Consolidate config file #32

Open dorianim opened 2 years ago

dorianim commented 2 years ago

There currently is one config file: /etc/linumuster-linuxclient7/network.conf. This is not ideal, as the name limits it to network stuff. I think, we should change the name of it to something more generic and also change the file format to yaml while we're at it. What this needs:

kiarn commented 2 years ago

Hello Dorian,

We are using the following files : /etc/ajenti/config.yml and /etc/linuxmuster/webui/config.yml. So config.yml would be a good candidate.

Arnaud

dorianim commented 2 years ago

Yes, I agree

kiarn commented 2 years ago

Hi,

For the scalable structure, this will evolve with the needs, but I can suggest :

general:
  options ...
shares:
  home:
    options ...
  other:
    options ...
printers:
  options ...
scripts:
  options ...

What options do we need ? That's the big question :

That's only a suggestion, maybe you know more awaited features.

For the conversion and cleanup, just see what it is done here : https://github.com/ajenti/ajenti/blob/398e314ddd3486ed054aea4079d1c06f0c6c4f84/ajenti-core/aj/config.py#L36

( both functions ensure_structure --> for consistent structure and migrate_users_to_own_configfile --> was needed to convert to a new structure )

Arnaud

dorianim commented 2 years ago

Hi Arnaud,

Thanks for your ideas. I like your proposal and I think we should realize it like this :+1:

Unfortunately I don't have any time to do this at the moment. But I hope that I will have some more time during the Semesterferien.

Regards, Dorian