Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

KCM should handle its own configuration itself #6926

Closed
aplopez opened this issue Sep 13, 2023 · 1 comment
Closed

KCM should handle its own configuration itself #6926

aplopez opened this issue Sep 13, 2023 · 1 comment
Assignees
@aplopez
Labels
Closed: Fixed Issue was closed as fixed. Future work

Comments

@aplopez
Copy link
Contributor

aplopez commented Sep 13, 2023 •
edited
Loading

KCM is currently using the shared config.ldb file to store its configuration. It should use a separate file so that it can be used independently of SSSD.

Replacing the ldb file is the simplest solution. It is also possible to keep the configuration in memory instead of an ldb file. Considering the file will not be shared with other daemon, there is no possibility of configuration mismatch between daemons.
@alexey-tikhonov alexey-tikhonov changed the title KCM should use its own separate configuration storage (config_kcm.ldb) KCM should handle its own configuration itself Sep 13, 2023
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Sep 13, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
b152d98 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
Instead 'sssd_kcm' should take care of parsing/preparing configuration
on its own (to be taken care of in separate PR).
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Sep 13, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
d016385 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
Instead 'sssd_kcm' should take care of parsing/preparing configuration
on its own (to be taken care of in separate PR).
aplopez added a commit to aplopez/sssd that referenced this issue Sep 21, 2023
@aplopez
KCM: Handle its own configuration
4d0d978 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
@aplopez aplopez mentioned this issue Sep 21, 2023
Closed
aplopez added a commit to aplopez/sssd that referenced this issue Sep 21, 2023
@aplopez
KCM: Handle its own configuration
7b02332 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Sep 22, 2023
@aplopez
KCM: Handle its own configuration
82b0512 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Sep 27, 2023
@aplopez
KCM: Handle its own configuration
dbb1232 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Sep 27, 2023
@aplopez
KCM: Handle its own configuration
600b72f 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Sep 27, 2023
@aplopez
KCM: Handle its own configuration
f67c605 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Sep 27, 2023
@aplopez
KCM: Handle its own configuration
2d4f5f3 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Sep 28, 2023
@aplopez
KCM: Handle its own configuration
4acf76d 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Sep 29, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
559bd65 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
aplopez added a commit to aplopez/sssd that referenced this issue Sep 29, 2023
@aplopez
KCM: Handle its own configuration
fa38674 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 2, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
c4d50c7 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 6, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
c3bb387 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
aplopez added a commit to aplopez/sssd that referenced this issue Oct 6, 2023
@aplopez
KCM: Handle its own configuration
8e403c6 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Oct 6, 2023
@aplopez
KCM: Handle its own configuration
e21a6be 
KCM now uses the /var/lib/sss/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
aplopez added a commit to aplopez/sssd that referenced this issue Oct 9, 2023
@aplopez
KCM: Handle its own configuration
68440c6 
KCM now uses the ${SSS_STATEDIR}/db/config_kcm.ldb database to store its
configuration. config.ldb is no longer used by KCM.

The configuration text file remains the same.

Resolves: SSSD#6926
@alexey-tikhonov
Copy link
Member

Pushed PR: #6948

  • master
    • 0485342 - KCM: Handle its own configuration
    • e6c1d3a - CONFDB: Fixed some missing dependencies in a header file
    • 7cc28f3 - CONFDB: Allow loading an empty configuration
    • 049edef - UTILS: Add the db file name to server_setup()'s parameters
    • 22f8eee - UTILS: Create a macro for the --config option

@alexey-tikhonov alexey-tikhonov added the Closed: Fixed Issue was closed as fixed. label Oct 10, 2023
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 11, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
0aecb76 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 12, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
287aac5 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 12, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
9555989 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 13, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
da9d713 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 16, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
c44d876 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Oct 16, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
2f7673f 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Nov 10, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
7d1395d 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Nov 14, 2023
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
78c7850 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 3, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
697da93 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 3, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
5aa2b16 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 4, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
40d48f1 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 7, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
80a5942 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 17, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
b50853c 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 19, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
f62284d 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 29, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
8a26bac 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 30, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
ece74c1 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 30, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
e35e3e4 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 30, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
a7fe3a7 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 31, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
2aa113c 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Jan 31, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
c32136e 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 1, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
d2d4d17 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 1, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
da60cfd 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 9, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
14598a0 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 12, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
00f51a7 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 12, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
b3aaa7d 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 13, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
3b8bd30 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 14, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
e2d4d44 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 19, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
3d67a73 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 19, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
f8b9f10 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 21, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
6eadc83 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 26, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
39782f7 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Feb 29, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
7fee258 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Mar 7, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
fdc8fbd 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit to alexey-tikhonov/sssd that referenced this issue Mar 13, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
8e05216 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see SSSD#6926)
alexey-tikhonov added a commit that referenced this issue Mar 18, 2024
@alexey-tikhonov
Get rid of --genconf and --genconf-section monitor options.
4a44cca 
The only usage was 'sssd-kcm.service', but it was wrong since 'sssd_kcm'
should be usable without other SSSD packages being installed (see #6926)

Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Pavel Březina <[email protected]>
Reviewed-by: Sumit Bose <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aplopez aplopez

Labels
Closed: Fixed Issue was closed as fixed. Future work
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

KCM uses its own configuration aplopez/sssd
2 participants
@aplopez @alexey-tikhonov