Fall back to string for dockerfile parameter

[mwhahaha]

a9cb824 changed the expectations of the
dockerfile parameter to be json data however it's a string. In order to
support both, let's attempt json and fall back to a string if the json
parsing fails.

Closes #10660

Signed-off-by: Alex Schultz aschultz@redhat.com

[mheon]

Can you added a test for this? test/apiv2/10-images.at seems like a natural place.

[mwhahaha]

Doesn't look like you can post a file with the apiv2 stuff? I'm looking into it today

[mheon]

Ahhh. Yeah, the APIv2 tests might be too limited for this.

[mwhahaha]

I made it work with apiv2 tests. It's probably easier than trying to implemented it in the python tests but i can move it over there if needed

[rhatdan]

Awesome work @mwhahaha LGTM

[mheon]

/lgtm
/hold

[rhatdan]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mwhahaha, rhatdan

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 [rhatdan]

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

[TomSweeneyRedHat]

LGTM
/hold cancel