Skip to content
This repository has been archived by the owner on Apr 25, 2019. It is now read-only.

[HOPSWORKS-634] Show proper warning when mysql is down #953

Closed
wants to merge 17 commits into from
Closed

[HOPSWORKS-634] Show proper warning when mysql is down #953

wants to merge 17 commits into from

Conversation

jimdowling
Copy link
Contributor

@jimdowling jimdowling commented Aug 3, 2018

Make sure there is no duplicate PR for this issue

  • Please check if the PR meets the following requirements
  • Adds tests for the submitted changes (for bug fixes & features)
  • Passes the tests
  • HOPSWORKS JIRA issue has been opened for this PR
  • All commits have been squashed down to a single commit
  • Post a link to the associated JIRA issue
    https://hopshadoop.atlassian.net/browse/HOPSWORKS-634

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Feature.

  • What is the new behavior (if this is a feature change)?
    Shows an "Annoucement" on the login page if the mysql server or NDB is not working.

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:
    This works by catching an Exception (runtime exceptions like EclipsePersistenceException are thrown when the DB is not working) at users.findUser() and also at settings.updateVariable(). They now throw an AppException with 503 (service_unavailable) status.
    At the javascript controller, we catch a 503 (service_unavailable) to indicate that we couldn't access the DB.

@jimdowling jimdowling requested a review from SirOibaf August 3, 2018 12:08
@jimdowling jimdowling closed this Aug 14, 2018
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