oci: keep LC_ env variables to conmon

[giuseppe]

it is necessary for conmon to deal with the correct locale, otherwise
it uses C as a fallback.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1893567
Requires: containers/conmon#215

Signed-off-by: Giuseppe Scrivano [email protected]

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: giuseppe

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:

• OWNERS [giuseppe]

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

[vrothberg]

LGTM

Any chance we can add a test?

[rhatdan]

LGTM

[haircommander]

I was wondering why my fix didn't work. thanks for picking this up @giuseppe :)

[mheon]

I'm going to go ahead and merge - if a test gets added, we can do so in a further PR.
/lgtm

[giuseppe]

I'm going to go ahead and merge - if a test gets added, we can do so in a further PR.

I think it would be better to have an updated conmon first so we can add an e2e test

[mheon]

/hold
Fair enough. Definitely need this in by Monday though.

[rhatdan]

I have merged the fix in conmon, now we need @haircommander to cut a new release and then @cevich to create a new VM to testing.

[rhatdan]

I have no problem with this being merged, so it will not effect anything, but it would be nice to make sure everything works correctly

[haircommander]

I have to deal with the backwards compat problem I mentioned last week, then I'll cut a release.