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

Upgrade Ulmo to 0.8.5 #3143

Merged
merged 1 commit into from
Aug 13, 2019
Merged

Upgrade Ulmo to 0.8.5 #3143

merged 1 commit into from
Aug 13, 2019

Conversation

rajadain
Copy link
Member

@rajadain rajadain commented Aug 12, 2019

Overview

This version supports the native suds timeout parameter, which obviates the need for an external timeout library that we'd been using so far.

With this upgrade, the get_values call is updated to supply the timeout to suds, and the now extraneous library is removed.

Connects #3083

Demo

image

Notes

There may still be some other errors you encounter. If so, use the same shape / test on staging, and ensure you see the same failures there. If you do, then the failures are not related to the changes in this PR, but instead come from the third-party services we're querying instead.

Testing Instructions

  • Check out this branch

  • Destroy and recreate the app VM

    $ vagrant destroy app && vagrant up app
    
  • Go to :8000/ and pick a shape

  • Go to the Monitor tab search for "water"

  • Switch to the CUAHSI tab and click on any of the results in the sidebar

  • Wait for the values to come in

    • Ensure they do
      • If they fail, ensure they also fail on staging
  • Pick a new result and open it

  • Wait for the first details call to succeed (this brings in the units)

  • Then turn off your local internet to simulate a timeout for the values endpoint (which brings in the numbers)

  • Ensure that the error you see for the timeouts are different from a plain "Error 500 during fetch", as shown in the screenshot above

@rajadain rajadain added NSF Funding Source: National Science Foundation Monitor labels Aug 12, 2019
This version supports the native suds timeout
parameter, which obviates the need for an external
timeout library that we'd been using so far.

With this upgrade, the get_values call is updated
to supply the timeout to suds, and the now extraneous
library is removed.
@rajadain rajadain marked this pull request as ready for review August 12, 2019 20:52
Copy link
Member

@caseycesari caseycesari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working as described!

@caseycesari caseycesari assigned rajadain and unassigned caseycesari Aug 13, 2019
@rajadain rajadain merged commit 1a9fe06 into develop Aug 13, 2019
@rajadain rajadain deleted the tt/upgrade-ulmo branch August 13, 2019 17:55
@rajadain
Copy link
Member Author

Thanks for taking a look!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monitor NSF Funding Source: National Science Foundation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants