Skip to content

Add pod condition documentation#219

Merged
s-vitaliy merged 3 commits intomainfrom
fix-failed-job-condition
Feb 5, 2026
Merged

Add pod condition documentation#219
s-vitaliy merged 3 commits intomainfrom
fix-failed-job-condition

Conversation

@s-vitaliy
Copy link
Contributor

@s-vitaliy s-vitaliy commented Feb 5, 2026

Fixes #218

Scope

Implemented:

  • This PR adds the documentation that describes the behavior of the operator that can confuse users. If the pod is deleted and the streaming application does not handle termination gracefully, the job can fail because of incorrect exit code of the container.

Additional changes:

Checklist

  • GitHub issue exists for this change.
  • Unit tests added and they pass.
  • Line Coverage is at least 80%.
  • Review requested on latest commit.

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

go-test-coverage report:

Package coverage threshold (70%) satisfied:	PASS
Total coverage threshold (75%) satisfied:	PASS
Total test coverage: 85.3% (405/475)

@s-vitaliy s-vitaliy changed the title Fix failed job condition Add pod condition documentation Feb 5, 2026
@s-vitaliy s-vitaliy marked this pull request as ready for review February 5, 2026 11:05
@s-vitaliy s-vitaliy requested a review from a team as a code owner February 5, 2026 11:05
@s-vitaliy s-vitaliy merged commit cf2a3a4 into main Feb 5, 2026
1 check passed
@s-vitaliy s-vitaliy deleted the fix-failed-job-condition branch February 5, 2026 11:56
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.

[BUG] Manual deletion of the pod fails the stream

2 participants