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

Can we bundle QoL "out of tree" extensions? #107

Closed
AndrewSav opened this issue Jan 18, 2023 · 11 comments
Closed

Can we bundle QoL "out of tree" extensions? #107

AndrewSav opened this issue Jan 18, 2023 · 11 comments

Comments

@AndrewSav
Copy link

Apparently viewing logs with OpenLens is now out of tree extension for some reason. As per link above people reported that it works if the extension is installed separately. Can we bundle the extension with the rest of OpenLens since it seems illogical to go backwards?

@alebcay
Copy link

alebcay commented Jan 18, 2023

See also lensapp/lens#6855 requesting upstream document how to included desired extensions as bundled (in-tree) extensions.

@AndrewSav
Copy link
Author

@MuhammedKalkan was this done? If so can you link PR/commit please?

@MuhammedKalkan
Copy link
Owner

@alebcay is following this on the linked issue at Lens repo. No need another issue here

@AndrewSav
Copy link
Author

@MuhammedKalkan as far as I understand the linked issue is about documentation. This issue is about actually implementing this here. Does this make sense?

@MuhammedKalkan
Copy link
Owner

Nope not only that, he wants to know how to. I have already asked him here if we could bundle extensions. #83 (comment)

If someone has an idea or can do it , PRs are always welcome.

@AndrewSav
Copy link
Author

Right, so what you are saying is that we do not know how to that yet. Can we may be keep the issue open until the time we do, since we obviously have not solved it yet? Thanks!

@jkroepke
Copy link
Collaborator

The main issue here is that we are not aware of how to bundle them. This repository has the strategy to run a build based on unmodified source (build scripts are part of the sources). A side from that, there is no offical source for the out of tree extensions yet and I would not depend agaist other 3rd party repositories.

@AndrewSav
Copy link
Author

The main issue here is that we are not aware of how to bundle them.

That is understood, I hope that one way or another someone figures that out.

I would not depend agaist other 3rd party repositories

Unfortunately, the reality is that Lens started removing functionality (viewing logs? really?) from the main source tree so they could charge for it. It is probably just the first removal but not last. If we do not figure out how to bundle third party plugins, this installation will become less and less useful as they remove more features. I've spoken with a few users and for most of them not being able to view logs in Lens reduces the usefulness of the tool severely. A few more removal like this and there will be no point using OpenLens anymore, that's my concern.

@ebbypeter
Copy link
Collaborator

With the removal of "core" capabilities, I am wondering if we need to pivot the discussion to fork or even split from OpenLens.

@jkroepke
Copy link
Collaborator

jkroepke commented Feb 1, 2023

This OpenLens repo will not the long term solution here, Something like clone and patch build scripts is cat and mouse situation. Since they convert OpenLens to an npm package, they may remove the scripts for the distributions builds anyways in future.

A few more removal like this and there will be no point using OpenLens anymore, that's my concern.

Yes, so we are back to k9s.

And we are not aware of javascript. If some changes on the extensions are needed, no one is able to maintain it. Thats a good reason for keeping the extensions out of tree.

Fork is the way to go, But the project feels huge. I would be hart to maintain it. LibreLens already died. Long term way would be a complete alternative to Lens OR buy one license, if you stay with it.

@marafa-sugarcrm
Copy link

readme was updated to inform users how to install the missing extensions https://github.com/MuhammedKalkan/OpenLens#extensions

this can be closed

@jkroepke jkroepke closed this as completed Mar 2, 2023
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

No branches or pull requests

6 participants