Failed to include file with russian characters

Bug #1361090 reported by Dmitry Moskowski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Core Infrastructure
Fix Committed
Medium
Clark Boylan

Bug Description

Raw include can't parse file with Russian characters. Output in attachment.

If i remove all Russian characters from file it includes successfuly.

Tags: jjb
Revision history for this message
Dmitry Moskowski (i-am-corpix) wrote :
Jeremy Stanley (fungi)
Changed in openstack-ci:
status: New → Triaged
importance: Undecided → Medium
tags: added: jjb
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to jenkins-job-builder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/117041

Changed in openstack-ci:
assignee: nobody → Clark Boylan (cboylan)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to jenkins-job-builder (master)

Reviewed: https://review.openstack.org/117041
Committed: https://git.openstack.org/cgit/openstack-infra/jenkins-job-builder/commit/?id=8b982ed886d5caf085d9028b2c627362f7df90c9
Submitter: Jenkins
Branch: master

commit 8b982ed886d5caf085d9028b2c627362f7df90c9
Author: Clark Boylan <email address hidden>
Date: Tue Aug 26 15:25:42 2014 -0700

    Handle utf8 in JJB

    JJB didn't actually handle unicode data very well for a couple reasons.
    First the local yaml loader was loading files into yaml as strings
    instead of unicode which we should just go ahead and do because yaml's
    built int loader loads utf-8 by default (and we don't override the
    default). Second we need to do parameter substitution on unicode and
    regular strings so change the substitution typecheck to use basestring
    instead of str. Finally we need to use UTF-8 as the encoding when
    emitting XML so do that.

    Add tests to actually test this in the yamlparser tests. The addition of
    these new tests comes with a little bit of cleanup in the test classes
    to make sure we load unicode files as utf8 more consistently.

    Change-Id: I2169e19aae2cdc7ddbd1e7217ef7584c786a039a
    Fixes-bug: 1361090

Changed in openstack-ci:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.