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

feat(verify.go): Add pod fail reason and message to output #9589

Merged
merged 2 commits into from
Jan 14, 2025

Conversation

idsulik
Copy link
Contributor

@idsulik idsulik commented Nov 28, 2024

Fixes: #9587

Description
Add pod fail details Into the watchJob method of the verifier

User facing changes
before: no details - %q running job %q errored during run
after: fail details - %q running job %q errored during run: reason=%q, message=%q

pkg/skaffold/verify/k8sjob/verify.go Outdated Show resolved Hide resolved
@louisjimenez louisjimenez requested review from louisjimenez and removed request for nathanperkins December 9, 2024 20:42
@louisjimenez
Copy link

@idsulik Can you pull HEAD into this PR, I believe that will address the failing checks. This commit should help https://github.com/GoogleContainerTools/skaffold/pull/9604/commits

@idsulik idsulik requested a review from a team as a code owner December 18, 2024 22:13
@idsulik idsulik requested a review from ChrisGe4 December 18, 2024 22:13
@idsulik
Copy link
Contributor Author

idsulik commented Dec 18, 2024

@louisjimenez thank you, rebased main

@louisjimenez
Copy link

@idsulik We recently merged some changes to fix dependency issues. Are you able to rebase this one more time? My apologies for the inconvenience.

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
@idsulik
Copy link
Contributor Author

idsulik commented Jan 13, 2025

@louisjimenez rebased this PR and this #9624

@louisjimenez louisjimenez merged commit 59c01b7 into GoogleContainerTools:main Jan 14, 2025
11 checks passed
@idsulik idsulik deleted the issue-9587 branch January 14, 2025 14:28
alphanota pushed a commit to alphanota/skaffold that referenced this pull request Jan 14, 2025
…tainerTools#9589)

* feat(verify.go): Add pod fail reason and message to output

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>

* fix verify.go

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>

---------

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

skaffold verify fails immediately without any useful logs
4 participants