Skip to content

[skip-ci] RPM: own dirs#2030

Merged
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
lsm5:rpm-own-dirs
May 31, 2024
Merged

[skip-ci] RPM: own dirs#2030
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
lsm5:rpm-own-dirs

Conversation

@lsm5
Copy link
Member

@lsm5 lsm5 commented May 31, 2024

The rpm creates and installs to dirs /usr/share/rhel and /usr/lib/containers . So, it should own them as well.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2283290

The rpm creates and installs to dirs /usr/share/rhel and
/usr/lib/containers . So, it should own them as well.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2283290

Signed-off-by: Lokesh Mandvekar <lsm5@redhat.com>
@lsm5
Copy link
Member Author

lsm5 commented May 31, 2024

@jnovy @rhatdan @edsantiago @Luap99 PTAL

@lsm5
Copy link
Member Author

lsm5 commented May 31, 2024

@TomSweeneyRedHat if we can wait to get this into v0.59 release as well, that'd be great. Or else I could do manual updates.

@rhatdan
Copy link
Member

rhatdan commented May 31, 2024

/approve
LGTM

Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 31, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lsm5, Luap99, rhatdan

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:

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

@openshift-merge-bot openshift-merge-bot bot merged commit 42a8b62 into containers:main May 31, 2024
@lsm5 lsm5 deleted the rpm-own-dirs branch May 31, 2024 13:37
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