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

Error message doesn't match getting started #34

Open
bmann opened this issue Jan 25, 2024 · 5 comments
Open

Error message doesn't match getting started #34

bmann opened this issue Jan 25, 2024 · 5 comments
Labels
design documentation Improvements or additions to documentation

Comments

@bmann
Copy link
Contributor

bmann commented Jan 25, 2024

We describe using the getting started repo which contains a settings.toml. If Homestar is not running when we visit the control panel, we get:

"Failed to connect to Homestar. Please check the WebSocket endpoint in your .env and ensure Homestar is running."

We should probably reference settings.toml here rather than .env

@bmann bmann added documentation Improvements or additions to documentation design labels Jan 25, 2024
@avivash
Copy link
Member

avivash commented Jan 25, 2024

This seems potentially error-prone, since there are a few different ways to install Homestar. Maybe a good alternative is updating the getting started steps to ensure users set the correct value in their .env file?

@bmann
Copy link
Contributor Author

bmann commented Jan 25, 2024

There is no env file in our getting started instructions or workspace. Is this maybe for running control locally?

The install method doesn’t matter AFAICT — everyone runs a Homestar start that points at a settings.toml.

@avivash
Copy link
Member

avivash commented Jan 25, 2024

The .env is used when running the control panel both locally and in the cloud. It contains the CID for the uploaded workflow wasm and the ws address of the homestar node to connect to. also the fission server uri once we merge the auth branch.

Ryan and I have chatted before about having a way for users to update the address of the homestar node from within the control panel, so maybe that's something we need to look at adding soon. Otherwise I'm not sure of the best way for users to point the CP to the correct resources if they're running it from control.everywhere.computer 🤔

@avivash
Copy link
Member

avivash commented Jan 25, 2024

to simplify things we could also look at running the CP locally using a .toml file instead of a .env file. Though a .toml file may be foreign to a lot of Node users 🤔

@bmann
Copy link
Contributor Author

bmann commented Jan 25, 2024

Thanks for the background @avivash. Just logged it while it was fresh in mind, I think we can let this cook / bake for a while and see what feedback we get. And, "hosted Homestar" will be in the mix here too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants