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.

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

SPEC: clean up ldb-cache files on uninstall #7150

Closed
wants to merge 1 commit into from

Conversation

alexey-tikhonov
Copy link
Member

No description provided.

@alexey-tikhonov alexey-tikhonov added the no-backport This should go to target branch only. label Jan 29, 2024
@alexey-tikhonov alexey-tikhonov changed the title Spec clean ldb SPEC: clean up ldb-cache files on uninstall Jan 29, 2024
@sumit-bose
Copy link
Contributor

Hi,

in #7140 you used the %__rm macro instead of just calling rm.

About Pavel's comment in #7140 about overrides. I would consider the overrides as configuration data and in retrospect I would prefer to have them e.g. in a config snippet and add them during startup instead of directly adding them into a somewhat volatile database and require explicit backup. However, if there are override stored in the cache we can assume that sss_override is installed in the system and could call sss_override user-export and sss_override group-export, the remaining question then would be where this data should be written?

bye,
Sumit

@alexey-tikhonov
Copy link
Member Author

in #7140 you used the %__rm macro instead of just calling rm.

This part is fixed now (updated was postponed until #7140 was merged).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport This should go to target branch only.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants