Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Octant should not crash on startup due to plugin errors #2796

Open
xtreme-vikram-yadav opened this issue Aug 24, 2021 · 0 comments
Open

Octant should not crash on startup due to plugin errors #2796

xtreme-vikram-yadav opened this issue Aug 24, 2021 · 0 comments
Labels

Comments

@xtreme-vikram-yadav
Copy link
Contributor

xtreme-vikram-yadav commented Aug 24, 2021

Describe the problem/challenge you have
Octant currently fails to start up when a plugin throws an error. This is different behavior from the plugin watch functionality where the new plugin errors are logged but octant keeps functioning correctly.

Steps to reproduce:

  1. Add a panic("") here: https://github.com/vmware-tanzu/octant/blob/71e75e12b05764588d3f0aede455d143e35a3355/cmd/octant-sample-plugin/main.go#L31.

  2. Restart octant.

  3. Octant fails to start with error:
    `unable to start runner: failed to start service api: start plugin manager: get rpc client for "octant-sample-plugin.exe": Unrecognized remote plugin message:

This usually means that the plugin is either invalid or simply
needs to be recompiled to support the latest protocol.
exit status 1
`

Describe the solution you'd like
Octant should log plugin runtime errors but continue to work correctly.

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

Environment:

  • Octant version (use octant version): 0.23
  • OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): Windows 10
xtreme-vikram-yadav pushed a commit to xtreme-vikram-yadav/octant that referenced this issue Aug 24, 2021
- Do not error out from octant when plugin startup fails

vmware-archive#2796
xtreme-vikram-yadav pushed a commit to xtreme-vikram-yadav/octant that referenced this issue Aug 24, 2021
- Do not error out from octant when plugin startup fails

Signed-off-by: Vikram Yadav <yvikram@vmware.com>

vmware-archive#2796
GuessWhoSamFoo pushed a commit that referenced this issue Aug 27, 2021
* Show error for JS plugin watcher

Signed-off-by: Vikram Yadav <yvikram@vmware.com>

#1273

* log plugin startup errors and resume octant startup

- Do not error out from octant when plugin startup fails

Signed-off-by: Vikram Yadav <yvikram@vmware.com>

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

No branches or pull requests

1 participant