Skip to content

fix: manifest generation fixes for initContainers, fix config type deploy status #68

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.

Sign up for GitHub

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

Jump to bottom
Merged
vigneshrajsb merged 3 commits into from
Dec 2, 2025
Merged

fix: manifest generation fixes for initContainers, fix config type deploy status #68

vigneshrajsb merged 3 commits into from
Dec 2, 2025

Conversation

@vigneshrajsb
Copy link
Contributor

@vigneshrajsb vigneshrajsb commented Dec 2, 2025 •
edited
Loading

What

  • fix: configuration type deploys stuck at queued status, so exclude from DeploymentManager to manage config type deploys
  • fix: deploy level manifest generation not the same as existing manifest generation logic. update to use initContainers instead of containers to push in manifest. also fixes missing attributes like datadog labels, volume mounts, ports and resources on init container
  • fix: rollout strategy when workloads have pvc attached

@vigneshrajsb vigneshrajsb marked this pull request as ready for review December 2, 2025 17:39
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner December 2, 2025 17:39
@vigneshrajsb vigneshrajsb merged commit e7277ea into main Dec 2, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vmelikyan vmelikyan vmelikyan approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vigneshrajsb @vmelikyan