Skip to content

fix: Case issue with pod State (#386)#387

Merged
enocom merged 1 commit intoGoogleCloudPlatform:mainfrom
CoryHagerman:fix-386
Jul 6, 2023
Merged

fix: Case issue with pod State (#386)#387
enocom merged 1 commit intoGoogleCloudPlatform:mainfrom
CoryHagerman:fix-386

Conversation

@CoryHagerman
Copy link
Copy Markdown
Contributor

The case of "off" in "CrashLoopBackoff" needs to be capitalized in order for the comparison check to match in podspec_updates.go

Fixes #386

The case of "off" in "CrashLoopBackoff" needs to be capitalized in order for the comparison check to match in podspec_updates.go
@CoryHagerman CoryHagerman requested a review from a team as a code owner July 6, 2023 13:41
Copy link
Copy Markdown
Member

@enocom enocom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@enocom
Copy link
Copy Markdown
Member

enocom commented Jul 6, 2023

@hessjcg Will you take a look and merge this assuming we're good?

Copy link
Copy Markdown
Collaborator

@hessjcg hessjcg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Thank you for the fix.

@enocom enocom merged commit 0a45c32 into GoogleCloudPlatform:main Jul 6, 2023
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.

Pods in a CrashLoopBackOff state are not deleted by the controller

3 participants