-
Notifications
You must be signed in to change notification settings - Fork 66
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
[release-4.11] WINC-983: [e2e] Ensure required log files are non-empty #1463
[release-4.11] WINC-983: [e2e] Ensure required log files are non-empty #1463
Conversation
This commit adds a check to the logs tests to check that log files have non-zero size, ensuring they are actually being written to by the services.
@openshift-cherrypick-robot: An error was encountered cloning bug for cherrypick for bug WINC-983 on the Jira server at https://issues.redhat.com/. No known errors were detected, please see the full error message for details. Full error message.
request failed. Please analyze the request body for more details. Status code: 400: {"errorMessages":["Number value expected as the Sprint id."],"errors":{}}
Please contact an administrator to resolve this issue, then request a bug refresh with In response to this:
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/test-infra repository. |
/lgtm |
/cherry-pick release-4.10 |
@mansikulkarni96: once the present PR merges, I will cherry-pick it on top of release-4.10 in a new PR and assign it to you. In response to this:
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/test-infra repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aravindhp, openshift-cherrypick-robot 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 |
@mansikulkarni96 Please cherry pick from master for non bug PRs |
/retest-required azure-e2e-operator is failing because the VM is taking a long time to be configured. This is a flake we have seen in the past on Azure. platform-none-vsphere-e2e-operator failed with:
which looks like a flake. |
Cluster install failure |
@openshift-cherrypick-robot: 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/test-infra repository. I understand the commands that are listed here. |
@mansikulkarni96: #1463 failed to apply on top of branch "release-4.10":
In response to this:
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/test-infra repository. |
…openshift-cherrypick-robot/cherry-pick-1421-to-release-4.11 Updated upstream source commit. Commit details follow: Signed-off-by: OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> Project: https://github.com/openshift/windows-machine-config-operator.git Branch: release-4.11 Commit: 4520bf0 Author: OpenShift Merge Robot Date: Wed, 8 Mar 2023 17:17:14 +0100 Merge pull request openshift#1463 from openshift-cherrypick-robot/cherry-pick-1421-to-release-4.11 [release-4.11] WINC-983: [e2e] Ensure required log files are non-empty Additional included commits: 55bfac3, 2da90d3, 0838689, 9729b36 Referenced RedHat JIRA issues: OCPBUGS-8055, WINC-983 automerge: yes x-md5: c69f9bf5c04d390019477488e87a1632 Change-Id: Ib12ff5739e997853cdbb96525f9ff5c27a6e45ed
…7a1632' into 'rhaos-4.11-rhel-8' Updated US source to: 4520bf0 Merge pull request openshift#1463 from openshift-cherrypick-robot/cherry-pick-1421-to-release-4.11 See merge request openshift-winc-midstream/openshift-winc-midstream!167
This is an automated cherry-pick of #1421
/assign mansikulkarni96