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

Stop all extensions before quitting app #7450

Merged
merged 1 commit into from
Mar 31, 2023
Merged

Conversation

Nokel81
Copy link
Collaborator

@Nokel81 Nokel81 commented Mar 31, 2023

Motivation: Perhaps some extensions are holding onto resources which are causing the application to crash loudly. This should help stop that.

Signed-off-by: Sebastian Malton <sebastian@malton.name>
@Nokel81 Nokel81 added the bug Something isn't working label Mar 31, 2023
@Nokel81 Nokel81 added this to the 6.4.13 milestone Mar 31, 2023
@Nokel81 Nokel81 requested a review from a team as a code owner March 31, 2023 16:59
@Nokel81 Nokel81 requested review from Iku-turso and jim-docker and removed request for a team March 31, 2023 16:59
@Nokel81 Nokel81 merged commit a0fea20 into master Mar 31, 2023
@Nokel81 Nokel81 deleted the stop-extensions-on-quit branch March 31, 2023 17:37
Nokel81 added a commit that referenced this pull request Mar 31, 2023
Signed-off-by: Sebastian Malton <sebastian@malton.name>
@Nokel81 Nokel81 mentioned this pull request Mar 31, 2023
Nokel81 added a commit that referenced this pull request Mar 31, 2023
* Stop all extensions before quitting app (#7450)

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Close Lens Proxy on quit of backend (#7453)

- Extract global shared state of shell sessions

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Fix crash due to non base64 encoded secrets (#7448)

* Fix crash due to non base64 encoded secrets

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Update snapshot

Signed-off-by: Sebastian Malton <sebastian@malton.name>

---------

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Add temporal dependency to initializeExtensions (#7454)

- So that extensions can ensure that the shell env is up to date

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Release 6.4.13

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Fixup cherry pick

- 6.5 has @k8slens/utilities but 6.4 does not

Signed-off-by: Sebastian Malton <sebastian@malton.name>

* Fix cherry-pick due to files being moved between 6.4 and 6.5

Signed-off-by: Sebastian Malton <sebastian@malton.name>

---------

Signed-off-by: Sebastian Malton <sebastian@malton.name>
jweak pushed a commit that referenced this pull request Apr 3, 2023
Signed-off-by: Sebastian Malton <sebastian@malton.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants