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

gather/aws-console: Make log gathering non-fatal #17735

Merged
merged 1 commit into from
Apr 14, 2021

Conversation

cgwalters
Copy link
Member

Something in our console content is either UTF-8 and this is
expecting ASCII, or is invalid UTF-8. To unblock RHCOS
promotion jobs, make gathering the console non-fatal.

Something in our console content is either UTF-8 and this is
expecting ASCII, or is invalid UTF-8.  To unblock RHCOS
promotion jobs, make gathering the console non-fatal.
@smarterclayton
Copy link
Contributor

/lgtm

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, smarterclayton

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-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 14, 2021
@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 14, 2021
@openshift-merge-robot openshift-merge-robot merged commit 3e6838b into openshift:master Apr 14, 2021
@openshift-ci-robot
Copy link
Contributor

@cgwalters: Updated the step-registry configmap in namespace ci at cluster app.ci using the following files:

  • key gather-aws-console-commands.sh using file ci-operator/step-registry/gather/aws-console/gather-aws-console-commands.sh

In response to this:

Something in our console content is either UTF-8 and this is
expecting ASCII, or is invalid UTF-8. To unblock RHCOS
promotion jobs, make gathering the console non-fatal.

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.

wking added a commit to wking/openshift-release that referenced this pull request Jun 29, 2021
In recent CI, we have failed console gathers like [1]:

  Gathering console logs for i-026ba0637214cd027
  'ascii' codec can't encode character '\u2026' in position 14093: ordinal not in range(128)
  Failed to gather console logs

This is descended from d060927 (gather/aws-console: Make log
gathering non-fatal, 2021-04-14, openshift#17735).  But we want to gather these
logs, not silently fail to gather the logs.  This commit restores our
old "fail the step when we fail to gather" behavior, and sets
PYTHONIOENCODING [2] to ask for UTF-8 instead of ASCII output.

[1]: https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/periodic-ci-openshift-[…]e2e-aws-serial/gather-aws-console/build-log.txt
[2]: https://docs.python.org/3/using/cmdline.html#envvar-PYTHONIOENCODING
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
4 participants