-
Notifications
You must be signed in to change notification settings - Fork 6
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
request pending from IPR client to API in dev docker setup #56
Comments
Nvm, I realized that I separated the branches for the PR but I need to test with the merge of both PRs still |
Looks like this is still an issue even when I merge both branches in |
@creisle could you post the container logs? |
I couldn't find much in the logs. Here's the IPR API logs.
|
@creisle yes maybe let's take a look at the client logs just to rule out problems there |
client logs
|
This may be an intermittent network problem. Today with no changes I was able to set up and run this fine. I will try again later to be sure |
On dockerdev with a new install of the develop branch as is, I was able, a moment ago, to populate the report's list in the ipr client, so it fixes issue https://www.bcgsc.ca/jira/browse/KBDEV-971 and did not encountered current issue. |
@creisle ran into situations like this when the API was down for a bit due to maintenance or something, but it is intermittent. afaik there was no CORS changes |
When I updated to the latest images for some reason the requests seem to hang and not complete (see GIF below)
https://github.com/bcgsc/pori/blob/master/.github/workflows/compose-test.yml#L15
I am testing this off dev branch here currently: https://github.com/bcgsc/pori/tree/feature/re-install-instructions
@kttkjl or @brayden-bcgsc or @mathieulemieux have any of you run into this? Did the CORS permissions change at all in the latest releases?
The text was updated successfully, but these errors were encountered: