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

Extend CRD to support configuring container.spec.lifecycle.preStop.command in injected container #49

Merged
merged 14 commits into from
Feb 11, 2025

Conversation

MatteoMori8
Copy link
Contributor

@MatteoMori8 MatteoMori8 commented Feb 10, 2025

Scope

  • We want to be able to modify container.spec of the injected container so that we can support more flexibility
  • The first usecase that we want to support if having lifecycle.preStop.exec.commad available in the injected container
  • Ideally, as we are adding support for custom container.spec, we should do the same for initContainer too

@MatteoMori8 MatteoMori8 force-pushed the AIRSHIP-4287/expand_dcb branch from 979a0a2 to 01145b4 Compare February 10, 2025 11:59
@MatteoMori8 MatteoMori8 changed the title Extend CRD to support configuring container.spec and initcontainer.spec Extend CRD to support configuring container.spec.lifecycle.preStop.command Feb 11, 2025
@MatteoMori8 MatteoMori8 changed the title Extend CRD to support configuring container.spec.lifecycle.preStop.command Extend CRD to support configuring container.spec.lifecycle.preStop.command in injected container Feb 11, 2025
@MatteoMori8 MatteoMori8 marked this pull request as ready for review February 11, 2025 12:27
@MatteoMori8 MatteoMori8 force-pushed the AIRSHIP-4287/expand_dcb branch from 385ca86 to e9a5a92 Compare February 11, 2025 13:49
@MatteoMori8 MatteoMori8 merged commit f2c59fc into master Feb 11, 2025
3 checks passed
@MatteoMori8 MatteoMori8 deleted the AIRSHIP-4287/expand_dcb branch February 11, 2025 15:55
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

Successfully merging this pull request may close these issues.

3 participants