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

Update staging to use the test connection stream dams4-test-securetoken. #474

Merged
merged 2 commits into from
Jul 3, 2018

Conversation

lsitu
Copy link
Member

@lsitu lsitu commented Jul 3, 2018

Local Checklist

  • Tests written and passing locally?
  • Code style checked?
  • QA-ed locally?
  • Rebased with master branch?
  • Configuration updated (if needed)?
  • Documentation updated (if needed)?

What does this PR do?

Update staging to use the test connection stream dams4-test-securetoken to stream from staging environment.

Why are we doing this? Any context of related work?

@remerjohnson reported that the video object he ingested (https://librarytest.ucsd.edu/dc/object/bd9167310g) is streaming with "404 Not Found" error. And Ron created stream connection dams4-test-securetoken to stream from the staging localStore.

Deployment Instructions

Deploy to staging for test.

@ucsdlib/developers, @rstanonik - please review

@lsitu lsitu requested a review from rstanonik July 3, 2018 16:20
@rstanonik
Copy link
Contributor

QA, Staging, and Production have separate localStores (Isilon shares) for DAMS data. Wowza has separate connection URLs to stream data from the Staging and Production localStores (dams4-securetoken and dams4-test-securetoken), so the Staging and Production environments must use the matching streaming URL, especially for testing ingestion. Currently there is no Wowza connection URL for QA. Let me know if you need one.

@lsitu
Copy link
Member Author

lsitu commented Jul 3, 2018

@rstanonik Thanks. Could we have the connection streaming for QA as well?

@rstanonik
Copy link
Contributor

I created a similar Wowza connection url for QA dams4-qa-securetoken.

@rstanonik
Copy link
Contributor

Here are simple examples demonstrating each

https://aal.ucsd.edu/tchu/dams4-securetoken/

https://aal.ucsd.edu/tchu/dams4-test-securetoken/

https://aal.ucsd.edu/tchu/dams4-qa-securetoken/

They are in aal because I was working with Vivian on something at the time.

@rstanonik
Copy link
Contributor

If it is useful to keep the examples around, I'll find someplace more suitable.

@lsitu
Copy link
Member Author

lsitu commented Jul 3, 2018

That sounds nice. Thanks @rstanonik. I've updated the QA environment to use dams4-qa-securetoken as well (see commit 2677389).

@lsitu
Copy link
Member Author

lsitu commented Jul 3, 2018

@hweng Could you review and merge it if you get a chance. We need to fix it on staging so that it won't delay @remerjohnson's ingest process. Thanks.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 65.98% when pulling 2677389 on feature/test_connection_stream into fdb14d0 on develop.

@hweng hweng merged commit b683d4b into develop Jul 3, 2018
@hweng hweng deleted the feature/test_connection_stream branch July 3, 2018 22:20
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.

None yet

4 participants