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

[videoreserves] Add monitoring in CheckMK for videoreserves #5466

Open
1 of 5 tasks
maxkadel opened this issue Oct 24, 2024 · 0 comments
Open
1 of 5 tasks

[videoreserves] Add monitoring in CheckMK for videoreserves #5466

maxkadel opened this issue Oct 24, 2024 · 0 comments
Labels
maintenance post-incident created from a post-incident meeting

Comments

@maxkadel
Copy link
Contributor

What maintenance needs to be done?

So that we know when videoreserves goes down before users report issues, we need to add monitoring.

The last incident we had, the page was returning as a 200, but was actually showing the literal code, rather than the video player (see #5457).

Level of urgency

  • High
  • Moderate
  • Low

Why is this maintenance needed?

  • A clear description of why this is needed.

Acceptance criteria

  • We get a push notification when video reserves is not rendering / is serving up the PHP script text
  • We get a push notification when video reserves is not sending people to CAS (this is a stretch goal and can be split into its own ticket)
  • [ ]

Implementation notes, if any

  • For rendering, we might be able to check for some of the text on the page, e.g. "Assigned audio and video materials are made available only to the students...", rather than trying to check for the video player.
  • We might also be able to check that the PHP script text is not present, e.g. include('../../vendor/autoload.php');
  • For checking CAS, we might check that https://videoreserves-prod.princeton.edu/hrc/vod/clip.php returns a 302 (redirect to CAS), we would just want to be sure that we do not follow the redirect, so we don't have issues with the campus IDP service
@maxkadel maxkadel added maintenance post-incident created from a post-incident meeting labels Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance post-incident created from a post-incident meeting
Projects
None yet
Development

No branches or pull requests

1 participant