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

Update install script to check for newrelic.ini in "mods-available" #399

Closed
ZNeumann opened this issue Mar 30, 2022 · 1 comment
Closed

Update install script to check for newrelic.ini in "mods-available" #399

ZNeumann opened this issue Mar 30, 2022 · 1 comment
Labels
bug Something isn't working enhancement New feature or request

Comments

@ZNeumann
Copy link
Contributor

Currently, the use of a "mods-available" dir to house the newrelic.ini is not checked (even though it is a perfectly valid system configuration). This causes issues during agent upgrades, as the install script is unable to find any newrelic.ini files and creates a new/additional ini.
@ZNeumann ZNeumann added the enhancement New feature or request label Mar 30, 2022
@ZNeumann
Copy link
Contributor Author

ZNeumann commented Apr 7, 2022

Installing from tar using the newrelic-install.sh script will not properly find a mods-available directory. Installing from apt WILL find the mods-available directory IF it is located in /etc/php/VERSION/. A custom PHP installation location will not have its mods-available directory found.

@bduranleau-nr bduranleau-nr added the bug Something isn't working label Jan 20, 2023
@bduranleau-nr bduranleau-nr mentioned this issue Jan 20, 2023
Closed
@AdrienPoupa AdrienPoupa mentioned this issue Dec 6, 2023
Open
@ZNeumann ZNeumann mentioned this issue Oct 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ZNeumann @bduranleau-nr