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

Find more performant way to enable required modules #105

Open
tobybellwood opened this issue Nov 7, 2019 · 4 comments
Open

Find more performant way to enable required modules #105

tobybellwood opened this issue Nov 7, 2019 · 4 comments

Comments

@tobybellwood
Copy link

In
https://github.com/govCMS/govcms8lagoon/blob/master/.docker/images/govcms8/scripts/govcms-deploy#L66
we perform an enable/uninstall of the govcms_lagoon module, which is only used to ensure that the dependencies listed in
https://github.com/govCMS/govcms8lagoon/blob/master/modules/lagoon/govcms_lagoon/govcms_lagoon.info.yml#L8
Surely there's a better way (surely?)

@simesy
Copy link
Contributor

simesy commented Nov 7, 2019

My opinion is we have a module that is production friendly. Leave it enabled. Disable features locally, if possible through settings.php and friends.

@simesy
Copy link
Contributor

simesy commented Nov 7, 2019

imo, It's more reliable to vet this state than to alter the application during deplyment and hope it matches a state.

@tobybellwood
Copy link
Author

how can we ensure that the module is enabled in prod (or undisable-able)?

@simesy
Copy link
Contributor

simesy commented Nov 7, 2019

I can think of a few ways (particularly in light of the new paas scaffold/ci). vetting job, form validation/alteration for the UI modules, some code in the scaffold settings.php that just ensures the modules are there, etc, etc. I mean it's going to be a bit custom but if the gains are significant then i think it's worth it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants