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

[#135] Add cache consistency check #154

Merged
merged 4 commits into from
Oct 26, 2023
Merged

Conversation

matthewhilton
Copy link
Contributor

@matthewhilton matthewhilton commented Oct 23, 2023

Closes #135 , Based on Brendans PR #149

Changes

  • Adds cache check, which checks the get_config (cached) and DB values (not cached) in cron and web environments.
  • If a difference is reported, it means there is a cache mismatch / config issue somewhere.
  • (Unrelated to issue) Updated README and moved branches to top, and added php versions (these are the versions listed by Moodle docs for each version)

Example output:

Screenshot 2023-10-24 at 08-35-29 mheartbeat-cache Administration Reports System status

@matthewhilton matthewhilton force-pushed the 135-cache-consistency-check branch 6 times, most recently from e223155 to 207bbca Compare October 24, 2023 04:24
@matthewhilton matthewhilton marked this pull request as ready for review October 24, 2023 04:38
@matthewhilton matthewhilton force-pushed the 135-cache-consistency-check branch from 207bbca to 866fee8 Compare October 25, 2023 22:00
@matthewhilton
Copy link
Contributor Author

TODO after merging:

  • Cherry pick to MOODLE_39_STABLE branch

@matthewhilton matthewhilton force-pushed the 135-cache-consistency-check branch from a17a285 to 9570d84 Compare October 26, 2023 03:22
README.md Outdated Show resolved Hide resolved
@matthewhilton matthewhilton merged commit af01778 into master Oct 26, 2023
26 checks passed
This was referenced Nov 2, 2023
@brendanheywood brendanheywood deleted the 135-cache-consistency-check branch November 12, 2023 23:28
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.

Add a check for cache consistency
2 participants