-
Notifications
You must be signed in to change notification settings - Fork 711
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
Wrong default value for RUNTIME in package-webapi.sh #544
Comments
Hi @Rsimk, until recently the deployed version of the API didn't have a start page (other than /healthz). Now that the react app is no longer deployed as a static web app it, the start page should render (if deployed). Can you confirm that you have deployed the webapp from a recent version of the repo? |
webapp was deployed from Oct 23, commit e139555. |
|
I assume you where asking about "healthz", not "heathz". /healthz return 404. I change parameter, packaged, deployed and all started working. Including /healthz. |
Thank you for the follow-up. (Yes: healthz, apologies) |
Describe the bug
webapi package script has default value for RUNTIME set to "linux-x86", but webapi is Windows application. App packaging and app deployment scripts work without error, but webapp would fail with no indications about wrong runtime. Only changing RUNTIME to "win-x64" solves the problem.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
webapi URL should open copilot start page.
Screenshots
![image](https://private-user-images.githubusercontent.com/56922442/278257011-a9f4f88b-cea0-4e1f-a62d-5ff7888d7e87.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk0MDI4NjgsIm5iZiI6MTczOTQwMjU2OCwicGF0aCI6Ii81NjkyMjQ0Mi8yNzgyNTcwMTEtYTlmNGY4OGItY2VhMC00ZTFmLWE2MmQtNWZmNzg4OGQ3ZTg3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjEyVDIzMjI0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWE2ZDZmMjU5NDJkOTVmNjBlNGNmN2ExYTc4ZDlmNDE3ZjhmZDI3OWI5NWIyZmQyNjA0ODgyNzlmMDdiODQxNWUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.DLnS2m-6irAM9VeemVyMF-tQYK2E2FlcZluxRN_ZhbM)
Platform
The text was updated successfully, but these errors were encountered: