Skip to content

Revert "bring read mappings proc into c/storage"#1884

Merged
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
giuseppe:revert-bring-read-mappings-proc
Apr 11, 2024
Merged

Revert "bring read mappings proc into c/storage"#1884
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
giuseppe:revert-bring-read-mappings-proc

Conversation

@giuseppe
Copy link
Copy Markdown
Member

the package already offers the same functionality with GetHostIDMappings().

This reverts commit 2dd06ae.

the package already offers the same functionality with
GetHostIDMappings().

This reverts commit 2dd06ae.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@kannon92
Copy link
Copy Markdown
Contributor

thanks you your help on this so far!

/lgtm

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 10, 2024

@kannon92: changing LGTM is restricted to collaborators

Details

In response to this:

thanks you your help on this so far!

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 11, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: giuseppe, saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [giuseppe,saschagrunert]

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

@saschagrunert
Copy link
Copy Markdown
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Apr 11, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 1fd0dc1 into containers:main Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants