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

Remove yara from sst_csi_client_tools #1331

Merged

Conversation

mhuth
Copy link
Contributor

@mhuth mhuth commented Nov 28, 2024

No description provided.

@mhuth
Copy link
Contributor Author

mhuth commented Nov 28, 2024

@ptoscano please do the needful.

@ptoscano
Copy link
Contributor

While removing it from sst_client_tools is correct, it will drop it entirely from the distro. yara must be in a workload to be shipped.

If yara belongs to sst_insights, then please move it to that workload.

@mhuth
Copy link
Contributor Author

mhuth commented Nov 28, 2024

@ptoscano It belongs to platmgmt-ins-security which isn't listed as a group in the config directory. But it is a rover group: https://rover.redhat.com/groups/group/platmgmt-ins-security

@ptoscano
Copy link
Contributor

It belongs to platmgmt-ins-security which isn't listed as a group in the config directory.

As I said, it must belong to a RHEL SST.

Copy link
Collaborator

@yselkowitz yselkowitz left a comment

Choose a reason for hiding this comment

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

It sounds like this is meant to be an ownership transfer rather than a removal, so it needs to be added to a new or existing config.

@mhuth mhuth force-pushed the remove_yara_sst_csi_client_tools branch from d415a05 to 79e04e3 Compare November 29, 2024 02:21
@mhuth mhuth requested a review from yselkowitz November 29, 2024 02:48
Comment on lines 9 to 14
package_placeholders:
- srpm_name: yara
rpms:
- rpm_name: yara
description: yara
dependencies: []
Copy link
Collaborator

Choose a reason for hiding this comment

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

@mhuth mhuth force-pushed the remove_yara_sst_csi_client_tools branch from 79e04e3 to 1a85db0 Compare November 29, 2024 04:02
@yselkowitz yselkowitz merged commit e35b6c3 into minimization:main Nov 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants