Skip to content

feat: support service module system vars and auto module deletion#4771

Open
Cynthia-0203 wants to merge 27 commits into
koderover:mainfrom
Cynthia-0203:feat/service-module-management
Open

feat: support service module system vars and auto module deletion#4771
Cynthia-0203 wants to merge 27 commits into
koderover:mainfrom
Cynthia-0203:feat/service-module-management

Conversation

@Cynthia-0203

@Cynthia-0203 Cynthia-0203 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Note: this PR depends on the custom resource support PR #4747.

What this PR does / Why we need it:

This PR improves service module management based on the custom resource support work.

It adds system variables for service module images, so each detected service module can expose an image variable like $<module-name>-image$.

It also adds support for deleting auto-discovered service modules by ID. This is useful when automatically parsed modules are no longer needed or need to be cleaned up manually.

What is changed and how it works?

  • For each valid service module, a $<module-name>-image$ variable is generated from the module image.
  • Adds delete support for auto-discovered service modules.

Does this PR introduce a user-facing change?

  • API change
  • database schema change
  • upgrade assistant change
  • change in non-functional attributes such as efficiency or availability
  • fix of a previous issue

This change is Reviewable

@Cynthia-0203
Cynthia-0203 force-pushed the feat/service-module-management branch 3 times, most recently from e7e46c1 to a9ea55a Compare June 30, 2026 02:17
@Cynthia-0203
Cynthia-0203 force-pushed the feat/service-module-management branch from 506d438 to fb9e450 Compare July 6, 2026 03:58
jamsman94 and others added 25 commits July 7, 2026 09:16
Signed-off-by: Min Min <jamsman94@gmail.com>
Signed-off-by: Min Min <minmin@Mins-MacBook-Pro.local>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
…ontainers

Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
…llers

Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
@Cynthia-0203
Cynthia-0203 force-pushed the feat/service-module-management branch from d6e1b0e to fb71146 Compare July 7, 2026 01:21
@lilianzhu
lilianzhu requested a review from PetrusZ July 8, 2026 02:56
Signed-off-by: Cynthia <chenchen@koderover.com>
Signed-off-by: Cynthia <chenchen@koderover.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants