verification-tests icon indicating copy to clipboard operation
verification-tests copied to clipboard

add debug for deployment roll out failure

Open zhouying7780 opened this issue 1 year ago • 5 comments

to debug https://issues.redhat.com/browse/OCPQE-3204 ; @kasturinarra Please help review , thanks

zhouying7780 avatar Jun 21 '24 09:06 zhouying7780

/lgtm

kasturinarra avatar Jun 24 '24 16:06 kasturinarra

@liangxia could you please help review and approve

kasturinarra avatar Jun 24 '24 16:06 kasturinarra

The change will add a lot of logs in our CI, both in Prow CI and Jenkins CI. Perhaps we can add those debug steps in the test case/scenario level ?

liangxia avatar Jun 25 '24 00:06 liangxia

@zhouying7780: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

openshift-ci[bot] avatar Aug 26 '24 06:08 openshift-ci[bot]

@zhouying7780 could you help me understand where you are adding the debug here ?

kasturinarra avatar Aug 26 '24 07:08 kasturinarra

@kasturinarra The case always failed with pod not running , actually we don't need to check the pod status , just check the probe correct is okay . so removed some steps.

zhouying7780 avatar Aug 26 '24 11:08 zhouying7780

@kasturinarra At beginning I added the logs step, but liang said this will add a lot of logging , so remove it

zhouying7780 avatar Aug 26 '24 11:08 zhouying7780

@zhouying7780 got it, thanks !!

kasturinarra avatar Aug 26 '24 14:08 kasturinarra

/lgtm

kasturinarra avatar Aug 26 '24 14:08 kasturinarra

@liangxia could you please help review and approve ? thanks !!

kasturinarra avatar Aug 26 '24 14:08 kasturinarra

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kasturinarra

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Aug 26 '24 14:08 openshift-ci[bot]