Fix handling of entrypoint

[rhatdan]

If a user specifies an entrypoint of "" then we should not use the images
entrypoint.

Signed-off-by: Daniel J Walsh [email protected]

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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

[rhatdan]

Fixes: #6935

[mheon]

LGTM

…

On Sat, Jul 11, 2020, 08:27 Daniel J Walsh ***@***.***> wrote: Fixes: #6935 <#6935> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#6939 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB3AOCGMGXGNXB7VYLNTORTR3BLEBANCNFSM4OXJC2EA> .

[vincent-olivert-riera]

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rhatdan

That looks weird. How can a PR be approved by the same person who created it?

[mheon]

Repo maintainers (anyone in the OWNERS file in repo root) are auto-approved. We mostly use approved to indicate that a PR is OK to proceed to review. Merge still requires an LGTM from two separate maintainers, who may not be the original submitter.

[vincent-olivert-riera]

@rhatdan , your commit for fixing the absence of systemd in the Fedora image clashes with this other PR: #6952

The one that gets merged first will win and the other one will need to rebase, of course. But perhaps you want to coordinate with the creator of the other PR.

[rhatdan]

@vincent-olivert-riera I agree, but for now, I just want to get one of them in. I will talk to @baude about it.

[rhatdan]

@baude @jwhonce @vrothberg @QiWang19 @ashley-cui @TomSweeneyRedHat PTAL

[QiWang19]

LGTM

[mheon]

/lgtm

[vincent-olivert-riera]

Thanks everyone!