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

Matomo configured with wrong site id #314

Open
rosiel opened this issue Nov 25, 2022 · 7 comments
Open

Matomo configured with wrong site id #314

rosiel opened this issue Nov 25, 2022 · 7 comments

Comments

@rosiel
Copy link
Contributor

rosiel commented Nov 25, 2022

On TAG=1.0.10, ENVIRONMENT=starter_dev, Matomo is getting configured with site id = 2, but Drupal thinks site id = 1.

When I load Matomo's UI, it displays a "no data has been recorded yet, set up below" page. It specifies that the site ID expected is 2. However Drupal's Matomo configuration has it at 1.

Matomo's 'no data has been recorded' page

I'm currently spinning up make local to see if this happens there too.

@jefferya
Copy link
Contributor

I can reproduce on non isle-dc site using isle-buildkit TAG 1.0.7.

Is the proper value 2 or 1? I suspect the 2 in Matomo is from isle-buildkit during the matomo database creation, more specifically,

https://github.com/Islandora-Devops/isle-buildkit/blob/main/matomo/rootfs/etc/matomo/create-matomo-database.sql#L878

) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=utf8;

I've not found the time to dig deeper and test but the reasoning behind AUTO_INCREMENT=2 instead of 1 is not clear to me.

@rosiel
Copy link
Contributor Author

rosiel commented Nov 25, 2022

Confirmed, it's happening on ISLE-DC with the local configuration as well.

@noahwsmith
Copy link
Contributor

@nigelgbanks do you recall if that AUTO_INCREMENT=2 was intentional? Seems maybe like a bug/typo to me...

@nigelgbanks
Copy link
Contributor

nigelgbanks commented Dec 15, 2022

I can't remember, looks like @dannylamb added it here: Islandora-Devops/isle-buildkit@544b045#r93147610. Do you recall @dannylamb?

@dannylamb
Copy link
Member

For the life of me I can't remember why it was bumped from 1 to 2. I think it was just not working and for whatever reason, bumping it to two made it work. But now the shoe appears to be on the foot so I dunno 😅

@rosiel
Copy link
Contributor Author

rosiel commented Dec 21, 2022

OK, so. In the Tech call today Seth also mentioned doing something about this - like needing to set autoincrement=2. We'll broaden this issue out to:

Install Profile: Use site_id = 2 in configs.
Starter Site: same.
Playbook: Get Matomo set up with autoincrement 2.

@nigelgbanks
Copy link
Contributor

I should mention in upgrading Matomo for the 2.z release I've changed the way it installs such that we no longer need to create tables in this way, so hopefully the issue will not be present in the 2.X release.

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

5 participants