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

T&A: add metric to collect inconsistencies for #37759 #7426

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

klees
Copy link
Member

@klees klees commented Apr 25, 2024

Hi everyone,

in todays meeting of the T&A tech squad we decided to collect known inconsistency metrics for the database that arise during investigations of mantis tickets. This is the code to do so for Mantis 37759 (i hope...).

@mbecker-databay (or someone else who feels knowledgeable enough): Please look into my query, I hope I've understand the issue and the database correctly.

Kind regards!

@mbecker-databay
Copy link
Contributor

I am not entirely sure if this one query catches them all.
For our purposes, this seems to bring the issue to light, we have an active and object data is gone.
There may be other effects, for example in object_reference and even more side-dishes in integrity, but I am fine with this one.

I will see to begin writing the "tutorialette" and leave this PR open for the time being, so we can review my text and this code side by side. Due to the regionality of the changes, I do not think we have to fear conflicts here.

In the mantis issue, as discussed, I will bring in an object-existence check and pull it up, the more general approach to fixing metered anomalies as per our agreement from the TechSquad in a more holistic approach.

@dsstrassner
Copy link
Contributor

We discussed the status of this PR at today's TechSquad Meeting, and Max stated that he is on it.
He looked at the available documentation and thinks that he could link to most of the information there.

He will push the work on this task in the next weeks.

@dsstrassner
Copy link
Contributor

Any updates on this @mbecker-databay?

@mbecker-databay
Copy link
Contributor

I did in fact look into this, but I seriously have no meaningful addition because the base documentation is WAY better than I thought from reading other such pieces. I will merge this now.

@mbecker-databay mbecker-databay merged commit 0acaa75 into ILIAS-eLearning:release_9 Sep 26, 2024
3 checks passed
mbecker-databay added a commit that referenced this pull request Sep 26, 2024
oliversamoila pushed a commit to oliversamoila/ILIAS that referenced this pull request Sep 30, 2024
oliversamoila pushed a commit to oliversamoila/ILIAS that referenced this pull request Sep 30, 2024
oliversamoila pushed a commit to oliversamoila/ILIAS that referenced this pull request Sep 30, 2024
dsstrassner pushed a commit to dsstrassner/ILIAS that referenced this pull request Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants