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

Better support for ACSF environments #80

Open
japerry opened this issue Jan 18, 2024 · 0 comments
Open

Better support for ACSF environments #80

japerry opened this issue Jan 18, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@japerry
Copy link

japerry commented Jan 18, 2024

Is your feature request related to a problem? Please describe.
Currently the ACSF detector looks for the multisite config json file. While that is fine for detecting the presence of ACSF, we need more information to know what ACSF environment we are on.

Specifically -- ACSF has an update process that will move production environments to an update environment while the sites are being updated. For quick, small number of sites this is fine; but for larger platforms, or when an error occurs and the production environment is stuck in the update environment, we need to know the correct variable name.

For instance, isAhProdEnv would return false for sites undergoing update, which is not correct. But instead of hard coding update01 via regex, we should be looking for the presence of gardens_site_settings and use those values instead. For example:

    // ACSF Sites should use the pre-configured env and db roles instead.
    if (isset($GLOBALS['gardens_site_settings'])) {
      $ahEnv = $GLOBALS['gardens_site_settings']['env'];
      $ahDbRole = $GLOBALS['gardens_site_settings']['conf']['acsf_db_name'];
    }

We're looking at integrating the environment detector into Acquia Connector and Search so we aren't duplicating logic that this library has, but we will need to make changes here (and have a release) before we can rely on this library in this module. Making an issue here so we can track it for a PR later down the road.

@japerry japerry added the enhancement New feature or request label Jan 18, 2024
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