Skip to content

feat(cli): add plugin shutdown timeout flag#569

Merged
jakobmoellerdev merged 4 commits into
open-component-model:mainfrom
jakobmoellerdev:cli-plugin-shutdown
Aug 18, 2025
Merged

feat(cli): add plugin shutdown timeout flag#569
jakobmoellerdev merged 4 commits into
open-component-model:mainfrom
jakobmoellerdev:cli-plugin-shutdown

Conversation

@jakobmoellerdev

Copy link
Copy Markdown
Member

What this PR does / why we need it

  • Introduced --plugin-shutdown-timeout flag to manage plugin termination.
  • Default timeout is set to 10 seconds. Forces plugins to shut down if not completed within the specified time.
  • Updated CLI commands and documentation to include the new flag.

Which issue(s) this PR fixes

Rationale: This ensures plugins do not hang during shutdown, improving the CLI's reliability and user experience.

part of the helm input binary test did with @Skarlso
part of open-component-model/ocm-project#490

- Introduced `--plugin-shutdown-timeout` flag to manage plugin termination.
- Default timeout is set to 10 seconds. Forces plugins to shut down if not completed within the specified time.
- Updated CLI commands and documentation to include the new flag.

Rationale: This ensures plugins do not hang during shutdown, improving the CLI's reliability and user experience.
Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
@jakobmoellerdev jakobmoellerdev requested a review from a team as a code owner August 14, 2025 17:38
@github-actions github-actions Bot added kind/feature new feature, enhancement, improvement, extension size/l Large labels Aug 14, 2025
Skarlso
Skarlso previously approved these changes Aug 14, 2025
Signed-off-by: Jakob Möller <jakob.moeller@sap.com>

# Conflicts:
#	cli/docs/reference/ocm.md
#	cli/docs/reference/ocm_add.md
#	cli/docs/reference/ocm_add_component-version.md
#	cli/docs/reference/ocm_completion.md
#	cli/docs/reference/ocm_completion_bash.md
#	cli/docs/reference/ocm_completion_fish.md
#	cli/docs/reference/ocm_completion_powershell.md
#	cli/docs/reference/ocm_completion_zsh.md
#	cli/docs/reference/ocm_download.md
#	cli/docs/reference/ocm_download_plugin.md
#	cli/docs/reference/ocm_download_resource.md
#	cli/docs/reference/ocm_generate.md
#	cli/docs/reference/ocm_generate_docs.md
#	cli/docs/reference/ocm_get.md
#	cli/docs/reference/ocm_get_component-version.md
#	cli/docs/reference/ocm_version.md
@jakobmoellerdev jakobmoellerdev enabled auto-merge (squash) August 18, 2025 07:48
@jakobmoellerdev jakobmoellerdev merged commit 798be1b into open-component-model:main Aug 18, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature new feature, enhancement, improvement, extension size/l Large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants