Fixed the containerfile not found during remote build

[tnk4on]

What this PR does / why we need it:

Fixed the containerfile not found during remote build when the context directory is symlink.

How to verify it

• Prepare the podman remote environment for the client and server.
• Put the context directory and Containerfile on the client side only.
• Create a symlink for the context directory. (e.g. ln -s target link)
• Execute a remote build by specifying symlink for the context directory.(e.g. podman -r build -t remote-build link)

Which issue(s) this PR fixes:

Fixes #12409

Special notes for your reviewer:

#11755 is insufficient to fix Issue #11732.

#11755 only works in the following cases.
(e.g. The client and server have the same context directory and Containerfile.)

Signed-off-by: Shion Tanaka [email protected]

[mheon]

CI is complaining that a new test is required

[mheon]

Change LGTM overall

[mheon]

/approve

@rhatdan @jwhonce PTAL

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mheon, tnk4on

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

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

[rhatdan]

Please squash your commits.

[rhatdan]

LGTM
@Luap99 @giuseppe PTAL

[umohnani8]

LGTM

[giuseppe]

could you add a test?

[tnk4on]

The test for this PR is the same as #11755. Do I need to add anything? Please give me some ideas.

https://github.com/containers/podman/pull/11755/files#diff-c5e29dbfa8d0225bea760a6935c60db51f2d9ad9535073a7f9e254682b60b487

The cause of this PR is that the client and server were tested by the same host, so the problem was overlooked and the #11755 fix was incomplete.
If we want to do a complete remote test, we need to change the UNIX Socket specification (I don't think it's that necessary).

[Luap99]

I do not think we can add a test where client and server are on separate systems. One reason more for podman machine testing.

[giuseppe]

ok fair. Could you please add [NO NEW TESTS NEEDED] to your commit message when you squash the two commits?

[TomSweeneyRedHat]

Symlinks always give me the heebie-jeebies, but this LGTM
I've restarted the failing test, looks to be a flake.

[Luap99]

/lgtm

[Luap99]

@mheon Another possible backport for 3.4.