Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make use of new pasta option from c/common #5724

Merged
merged 2 commits into from
Sep 9, 2024

Conversation

Luap99
Copy link
Member

@Luap99 Luap99 commented Sep 6, 2024

What type of PR is this?

/kind api-change
/kind bug
/kind cleanup
/kind deprecation
/kind design
/kind documentation
/kind failing-test

/kind feature

/kind flake
/kind other

What this PR does / why we need it:

Me always map 169.254.1.2 to the host now by default so ensure the
host.containers.internal entry is set correctly.

How to verify it

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Buildah now uses the pasta --map-guest-addr option by default which is used for the host.containers.internal entry to allow the container to reach the host by default.

Includes my pasta changes

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Copy link
Collaborator

@flouthoc flouthoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

openshift-ci bot commented Sep 6, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: flouthoc, Luap99

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:

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

@openshift-ci openshift-ci bot added the approved label Sep 6, 2024
We always map 169.254.1.2 with pasta to the host now so ensure the
host.containers.internal entry is set correctly.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@Luap99 Luap99 force-pushed the pasta-map-guest-addr branch from 0620035 to 5184bf1 Compare September 6, 2024 15:06
@rhatdan
Copy link
Member

rhatdan commented Sep 9, 2024

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Sep 9, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 599520f into containers:main Sep 9, 2024
31 checks passed
@Luap99 Luap99 deleted the pasta-map-guest-addr branch September 9, 2024 15:56
Luap99 added a commit to Luap99/common that referenced this pull request Oct 30, 2024
This was just added as alias to allow vendoring without having to fix
all callers at the same time.

These are the PRs that updat the callers:
containers/podman#24417
containers/buildah#5724

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Luap99 added a commit to Luap99/common that referenced this pull request Oct 30, 2024
This was just added as alias to allow vendoring without having to fix
all callers at the same time.

These are the PRs that update the callers:
containers/podman#24417
containers/buildah#5724

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Dec 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants