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

Che server technical debt #18159

Closed
2 of 13 tasks
skabashnyuk opened this issue Oct 21, 2020 · 1 comment
Closed
2 of 13 tasks

Che server technical debt #18159

skabashnyuk opened this issue Oct 21, 2020 · 1 comment
Labels
area/che-server kind/epic A long-lived, PM-driven feature request. Must include a checklist of items that must be completed. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.

Comments

@skabashnyuk
Copy link
Contributor

skabashnyuk commented Oct 21, 2020

Is your enhancement related to a problem? Please describe.

As part of this issue, I would like to summarise existed technical debt for the "Che server".
The current "Che server" architecture has a long history that comes from Che4/Che5. Since that time
it was changed from mainly Docker strimmed technology to mainly K8s based. Quite a few of the internal process
is no longer needed or has no sense at all. Devfile introduction in Che7 was made in a very
short period that doesn't allow us to make a lot of pieces in an efficient manner. For example, during workspace startup, we convert Devfile to WorkspaceConfig and still use Che6 models.
All of that brings some complication which accumulates over time. That means that introduction
of new features such as #17829 can be significantly slower next time.

Workspaces startup process.

As I mention during the workspace startup we convert Devfile to WorkspaceConfig and still use Che6 models.
We would like to revise our workspaces startup process and make it more up to date. However, before we start doing any changes would be reasonable to document(internal tech document) existed process to better understand what is happening and what
we can improve.
Potential improvement after that

Legacy concepts/code/objects/methods

  • We uses WorkspaceConfig(che6) internally during workspace startup
  • We still have notion of Installers(che6)
  • We still have notion of Agents(che6)
  • Legacy k8s namespace strategy introduce additional complexity

General source code maintenance

Single user vs multi-user

TBD

Single host vs multi-host

TBD

Describe the solution you'd like

action items

Describe alternatives you've considered

n/a

Additional context

n/a

@skabashnyuk skabashnyuk added kind/epic A long-lived, PM-driven feature request. Must include a checklist of items that must be completed. area/che-server labels Oct 21, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Oct 21, 2020
@skabashnyuk skabashnyuk removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Oct 21, 2020
@che-bot
Copy link
Contributor

che-bot commented May 4, 2021

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 4, 2021
@skabashnyuk skabashnyuk added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-server kind/epic A long-lived, PM-driven feature request. Must include a checklist of items that must be completed. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.
Projects
None yet
Development

No branches or pull requests

2 participants