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

Disable recursive option in Linux #3963

Merged
merged 1 commit into from Feb 16, 2023

Conversation

rak-phillip
Copy link
Contributor

This resolves an issue with Kubernetes contexts updating in the tray menu by disabling the recursive option in Linux.

I had originally attempted upgrading to Node.js 18 in #3958 but the issue still persisted. The recursive options should be supported in node 18, so we will have to circle back to understand why this approach failed.

closes #3765

pkg/rancher-desktop/main/tray.ts Outdated Show resolved Hide resolved
pkg/rancher-desktop/main/tray.ts Outdated Show resolved Hide resolved
Signed-off-by: Phillip Rak <rak.phillip@gmail.com>
@mook-as mook-as merged commit 6e5d6f1 into rancher-sandbox:main Feb 16, 2023
@rak-phillip rak-phillip deleted the bugfix/3765-contexts branch February 17, 2023 00:25
@jandubois
Copy link
Member

The recursive options should be supported in node 18, so we will have to circle back to understand why this approach failed.

Do we have to revisit this now that we are on node 18?

It is not clear to me if disabling the recursive option has any negative effects for us.

@mook-as
Copy link
Contributor

mook-as commented Dec 11, 2023

The documentation says:

The recursive option is only supported on macOS and Windows. An ERR_FEATURE_UNAVAILABLE_ON_PLATFORM exception will be thrown when the option is used on a platform that does not support it.

So it sounds like it's still not there.

That line isn't there for docs for NodeJS 20, so we can look again once we go there.

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

Successfully merging this pull request may close these issues.

Contexts aren't added/deleted from the Tray menu on Linux platforms
3 participants