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

Disabling an EPM removes database tables and fields (and counters) it added to an archive #422

Open
drn05r opened this issue Jan 15, 2025 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@drn05r
Copy link
Contributor

drn05r commented Jan 15, 2025

Using the Bazaar or tools/epm to disable an EPM will not just disable it in the specific repository archive but it will also remove any database tables / fields and counters defined by the EPM. This may be what the sysadmin wants but bad things from not removing these are limited (DB row length too long / too many indexes) are far outweighed by the bad things that do (deleting all the data in the tables/fields deleted).

It should be possible to build into an EPM that on disable it should retain database tables / fields and counters. It should also be possible to override this when using tools/epm at the command line, so you can really delete tables you don't want or not delete tables if you are just temporarily disabling an EPM on an archive.

@drn05r drn05r added the enhancement New feature or request label Jan 15, 2025
@drn05r drn05r added this to the 3.4.7 milestone Jan 15, 2025
@drn05r drn05r self-assigned this Jan 15, 2025
drn05r added a commit that referenced this issue Jan 15, 2025
@drn05r drn05r changed the title Disabling an EPM remove database tables and fields (and counters) it added to an archive Disabling an EPM removes database tables and fields (and counters) it added to an archive Jan 16, 2025
drn05r added a commit that referenced this issue Jan 21, 2025
Moves EPM phrases to Bazaar ingredient
Removes duplicate Storage::EPM plugin under perl_lib
Adds check_symlinks action
Allow --force with enable to use new rather than current/saved version
of config file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant