Add a hard timeout for all processes we spawn #460
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
juozasg
added a commit
to juozasg/vscode-gitops-tools
that referenced
this pull request
Aug 14, 2023
timeout kill all exec processes. default time is 10s
kingdonb
pushed a commit
that referenced
this pull request
Aug 21, 2023
timeout kill all exec processes. default time is 10s
kingdonb
pushed a commit
that referenced
this pull request
Sep 8, 2023
timeout kill all exec processes. default time is 10s
kingdonb
pushed a commit
that referenced
this pull request
Sep 8, 2023
timeout kill all exec processes. default time is 10s
kingdonb
pushed a commit
that referenced
this pull request
Sep 8, 2023
* Allow users to disable running `flux check` When I tested it locally it looked like we're calling `flux check` a bit more often than we should. This may have been fixed by now, but the flag remains so users who are experiencing issues from `flux check` can disable it entirely. * allow users to additionally suppress debugging messages Emits some debugging messages that are meant for the other authors of the extension to notice (and users won't be as interested in how many times we were apparently going to call flux check) Signed-off-by: Kingdon Barrett <kingdon@weave.works> * Add a hard timeout for all processes we spawn (#460) timeout kill all exec processes. default time is 10s * aync cluster treeview flux check with async cluster treeview proper async sources treeview correct expand all * responsive ui with oidc-login WIP * preserve collapsible states safer execs realtime namespace icons * 60 seconds exec kill * namespace UI better * namespace ui improvements * module tree-kill * node 20 missing npm packages npm audit fix fully qualify types my cluster has HelmReleases.cluster.loft.sh and it is wreaking all kinds of havoc here, let's fix this in the prerelease/edge channel ASAP Signed-off-by: Kingdon Barrett <kingdon@weave.works> simplify treeview states WIP api-resources loader WIP revert Kind.APIGroup gitops.editor.openKubeconfig api-resource loader and views Signed-off-by: Kingdon Barrett <kingdon@weave.works> * docs and templates treeview loads * update github actions workflows Signed-off-by: Kingdon Barrett <kingdon@weave.works> * First try: fully qualified Flux API names Signed-off-by: Kingdon Barrett <kingdon@weave.works> * More fully qualified kinds * fix type error Signed-off-by: Kingdon Barrett <kingdon@weave.works> * fix linting error Signed-off-by: Kingdon Barrett <kingdon@weave.works> * remove debugger stuff Signed-off-by: Kingdon Barrett <kingdon@weave.works> * fix expand all for workloads * fqkinds record lookup * add timeout configuration to after async Signed-off-by: Kingdon Barrett <kingdon@weave.works> * remove console.log and lint imports * treeview semantics cleanup * restore k8s proxy * refactor data providers to not break on context switches --------- Signed-off-by: Kingdon Barrett <kingdon@weave.works> Co-authored-by: Juozas Gaigalas <juozasgaigalas@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
User configurable SIGKILL timeout. 10 seconds by default.
Reorganize shell.exec imports