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

Update to Matomo 5.1.1 #19

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Update to Matomo 5.1.1 #19

merged 1 commit into from
Aug 28, 2024

Conversation

cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Aug 27, 2024

How this addresses that need:

Side effects of this change:
None.

Relevant ticket(s):

What does this PR do?

  • Update the Dockerfile to pull the 5.1.1 version of the Matomo container
  • Update the config.ini.php file to reflect the fact that we are using Maria DB (not MySQL) for the database
  • Update the config.ini.php file to fix the issue that the Matomo System Check page did not recognize that the database had already enabled the utf8mb4 character set.

Helpful background context

UXWS requested an upgrade to our instance of Matomo. In the process of testing the upgrade in Dev1, I discovered that there was an issue with both the database schema configuration and the database character set configuration. Both of those issues were minor and are addressed with the changes to the config.ini.php file.

How can a reviewer manually see the effects of these changes?

This updated container is deployed to Dev1 so the reviewer can log in to the Dev1 instance of Matomo (https://matomo-ecs-dev.dev1.mitlibrary.net:8443) and verify that the reported version is correct and that the System Check page only has the Geolocation warning (that's a warning that we don't care about).

Includes new or updated dependencies?

NO

What are the relevant tickets?

Developer

  • [n/a] All new ENV is documented in README (or there is none)
  • Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

  • The commit message is clear and follows our guidelines
    (not just this pull request message)
  • There are appropriate tests covering any new functionality
  • The documentation has been updated or is unnecessary
  • The changes have been verified
  • New dependencies are appropriate or there were no changes

Why these changes are being introduced:
UXWS requested an upgrade from 5.1.0 to 5.1.1 for Matomo.

How this addresses that need:
* Update the Dockerfile to pull the 5.1.1 version of the Matomo
container
* Update the config.ini.php file to reflect the fact that we are using
Maria DB (not MySQL) for the database
* Update the config.ini.php file to fix the issue that the Matomo
System Check page did not recognize that the database had already
enabled the utf8mb4 character set.

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/INFRA-471
@cabutlermit cabutlermit requested a review from dhrutibc August 27, 2024 20:40
Copy link

@dhrutibc dhrutibc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dev looks good

@cabutlermit cabutlermit merged commit 640c444 into main Aug 28, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants