Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

DRAFT: fcrepo_endpoint monitor support #743

Closed
wants to merge 3 commits into from

Conversation

aprilrieger
Copy link
Contributor

@aprilrieger aprilrieger commented Mar 29, 2024

  • Add more detailed response.success to target for fedora for s24x7 monitor support
  • Update rspec tests to account for new support.

Story

When Fedora goes down we have no way of knowing. In order to add a monitor for the service I needed to ensure that the response.success I was targeting wasn't just the word "Ok" since the other services use the same response, I was getting false positives upon testing the status, and after deliberately taking the fcrepo service down in docker to imitate the service being down.

Screenshot

Dev
Screenshot 2024-03-28 at 23 35 27

Production (To show what it looked like before the PR because I forgot to take a screenshot of dev before I deployed this)
Screenshot 2024-03-28 at 23 37 37

…and specifically for adding a monitor to the service
@aprilrieger aprilrieger changed the title fcrepo_endpoint monitor support Mar 29, 2024
@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Mar 29, 2024

To clarify, we'll get notified via site 24x7? Is that attached to a pager duty person?

Thanks for getting this done so quickly!

@aprilrieger
Copy link
Contributor Author

@ShanaLMoore Just so you are up to speed, as I was still in flight on this and should have had it as a draft but it was late and I forgot. Here is a link to my current update: https://assaydepot.slack.com/archives/C060R0LLYRX/p1711695746829609

Deployed to staging to test the addition of the Adventist’s fcrepo monitor and when I went to set it all up and test it in Site24x7, and it gave me authentication issues, it finally dawned on me that we need to expose the check outside of the dashboard due to the redirect to the signin page. 

@aprilrieger aprilrieger changed the title fcrepo_endpoint monitor support DRAFT: fcrepo_endpoint monitor support Mar 29, 2024
Making it even better by condensing it to a single monitor string set for error checking

Co-authored-by: Jeremy Friesen <jeremy.n.friesen@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants