Matomo 5.2.2 Upgrade #21
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Why these changes are being introduced: UXWS requested an upgrade to the latest version of Matomo. As part of the initial testing of the upgrade in Dev1, it was noted that an additional PlugIn appeared in the config.ini.php file. How this addresses that need: * Upgrade the Docker source from 5.1.2 to 5.2.2 * Update the config.ini.php file to include the newly listed FeatureFlags PlugIn * Update the config.ini.php to include the collation detail Side effects of this change: None. Relevant ticket(s): * https://mitlibraries.atlassian.net/browse/INFRA-506 * https://mitlibraries.atlassian.net/browse/INFRA-499 * https://mitlibraries.atlassian.net/browse/INFRA-495
Why these changes are being introduced: During the testing and verification of the new 5.2.2 version of Matomo, it was discovered that there were newer versions of the two extra PlugIns that we use. At the same time, we cleared out some old copies of plugins from this repository. How this addresses that need: * Add new CustomVariables 5.0.4 plugin folder * Add new EnvironmentVariables 5.0.3 plugin folder * Remove extra older plugin folders for both CustomVariables and EnvironmentVariables plugins Side effects of this change: None. Relevant ticket(s): * https://mitlibraries.atlassian.net/browse/INFRA-506
|
Actually --- hold off on reviewing! Just remembered that I need to update the README. I'll push updates first thing tomorrow morning. |
Why these changes are being introduced: All Infra-related repositories need a CODEOWNERS file and additional sections in the README to ensure that the links between repositories are clearly documented. How this addresses that need: * Create a CODEOWNERS file * Add a Related Assets section and a Maintainers section to the README Side effects of this change: None.
|
@MITLibraries/infraeng-terraform-reviewers Documentation updates are pushed. Now it's ready for review. |
tim-rix
approved these changes
Feb 5, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Upgrades our Matomo ECS service in AWS to version 5.2.2.
Helpful background context
UXWS requested an upgrade to our Matomo instance. As part of the initial testing in Dev1, not only are we upgrading the version of Matomo, we are also upgrading the versions of the two plugins and making a minor change to the config file.
How can a reviewer manually see the effects of these changes?
The new version is currently running in Dev1 at Dev1-Matomo.
Includes new or updated dependencies?
NO
What are the relevant tickets?
Developer
Code Reviewer
(not just this pull request message)