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

Warn when fs is set to remote and file is read locally #2008

Open
eyalb181 opened this issue Oct 16, 2023 · 2 comments
Open

Warn when fs is set to remote and file is read locally #2008

eyalb181 opened this issue Oct 16, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@eyalb181
Copy link
Member

Users are sometimes caught by surprise when fs is set to remote but a file is read locally. We should show a warning the first time the app reads a local file from the override list, explaining the behavior and linking the full list of the overridden paths. Don't forget "Don't show again".

@eyalb181 eyalb181 added the enhancement New feature or request label Oct 16, 2023
@meowjesty meowjesty self-assigned this Oct 17, 2023
@meowjesty
Copy link
Member

Before we start this, we need some mechanism to pass messages from our hooks to the user. Something that would allow us communicate back from mirrord to the IDEs.

We could have a temporary file (maybe even in-memory only) that we write only important things to it, and the IDE reads those (tl;dr: our own stdout).

@Razz4780
Copy link
Contributor

Before we start this, we need some mechanism to pass messages from our hooks to the user. Something that would allow us communicate back from mirrord to the IDEs.

We could have a temporary file (maybe even in-memory only) that we write only important things to it, and the IDE reads those (tl;dr: our own stdout).

We could expose logs via websockets and have the IDE open a new terminal tab for display (would require either merging mirrord-console to CLI or moving the logs to the new intproxy)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
3 participants