]> git.apps.os.sepia.ceph.com Git - ceph-build.git/commitdiff
ceph-pr-commits: remove non-ascii char 1779/head
authorKefu Chai <kchai@redhat.com>
Thu, 25 Mar 2021 10:55:45 +0000 (18:55 +0800)
committerKefu Chai <kchai@redhat.com>
Thu, 25 Mar 2021 11:01:43 +0000 (19:01 +0800)
otherwise `jenkins-jobs test` fails when reading it in non-utf-8 locale,

Traceback (most recent call last):
  File "/tmp/venv.KVnqGXyZ3a/bin/jenkins-jobs", line 8, in <module>
    sys.exit(main())
  File "/tmp/venv.KVnqGXyZ3a/lib/python3.6/site-packages/jenkins_jobs/cli/entry.py", line 165, in main
    jjb.execute()
  File "/tmp/venv.KVnqGXyZ3a/lib/python3.6/site-packages/jenkins_jobs/cli/entry.py", line 146, in execute
    ext.obj.execute(self.options, self.jjb_config)
  File "/tmp/venv.KVnqGXyZ3a/lib/python3.6/site-packages/jenkins_jobs/cli/subcommand/test.py", line 66, in execute
    config_xml=options.config_xml)
  File "/tmp/venv.KVnqGXyZ3a/lib/python3.6/site-packages/jenkins_jobs/builder.py", line 279, in update_jobs
    output.write(job.output())
  File "/tmp/venv.KVnqGXyZ3a/lib/python3.6/codecs.py", line 836, in write
    return self.writer.write(data)
  File "/tmp/venv.KVnqGXyZ3a/lib/python3.6/codecs.py", line 376, in write
    data, consumed = self.encode(object, self.errors)
UnicodeEncodeError: 'latin-1' codec can't encode character '\u2014' in position 57823: ordinal not in range(256)

before this change:

$ iconv -f ascii -t ascii < ./ceph-pr-commits/build/build
...
input sequence at position 197

after this change,

$ iconv -f ascii -t ascii < ./ceph-pr-commits/build/build

finishes without issues.

Signed-off-by: Kefu Chai <kchai@redhat.com>
ceph-pr-commits/build/build

index 3264fa1d70c0746bb64eb346c52ccb5e845181e0..4c134b4d40216e299f62c78664956189fdea5557 100644 (file)
@@ -2,7 +2,7 @@
 
 # Don't require signed commits if only docs changed.
 # I tried using the excluded-regions parameter for the ghprb plugin but since
-# this job/check is required, it hung with 'Expected  Waiting for status to be reported'
+# this job/check is required, it hung with 'Expected - Waiting for status to be reported'
 docs_pr_only
 if [ "$DOCS_ONLY" = false ]; then
     echo "Not a docs only change.  Will proceed with signed commit check."