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

Please add zypper from SUSE to the snippet.yml file #693

Closed
jloehel opened this issue May 20, 2020 · 7 comments · Fixed by #714
Closed

Please add zypper from SUSE to the snippet.yml file #693

jloehel opened this issue May 20, 2020 · 7 comments · Fixed by #714
Labels
feature new feature help wanted intermediate Intermediate level issue
Milestone

Comments

@jloehel
Copy link

jloehel commented May 20, 2020

Describe the Feature
I am missing zypper in snippet.yml to analyze openSUSE/SLES images correctly. Could you please add this functionality.

Use Cases
Generating compliance reports for openSUSE or SLES based Docker images.

Implementation Changes
I am not sure if something additionally is necessary but something like this might be enough:

zypper:
  install: 'in'
  remove: 'rm'
  ignore:
    - 'clean'
  packages: 'rpm'

Zypper allows you to shorten the subcommand. Is it possible to add more than one value for install and remove?

zypper:
  install: 
  - 'in'
  - 'install'
  remove: 
  - 'rm'
  - 'remove'
  ignore:
    - 'clean'
  packages: 'rpm'
@nishakm nishakm added the feature new feature label May 20, 2020
@nishakm
Copy link
Contributor

nishakm commented May 20, 2020

I am not familiar with zypper. Do you have a link to a openSUSE/SLES image on Dockerhub we can play around with?

@jloehel
Copy link
Author

jloehel commented May 20, 2020

I am not familiar with zypper. Do you have a link to a openSUSE/SLES image on Dockerhub we can play around with?

Hi Nisha,

thanks for the feedback. SUSE uses their own build service and registry to distribute the images. But all images are also synced to DockerHub: leap and tumbleweed. Zypper uses libzypp.

cheers
jl

@nishakm nishakm added this to the Release 2.2.0 milestone Jun 1, 2020
@nishakm nishakm added help wanted intermediate Intermediate level issue labels Jun 1, 2020
@mukultaneja
Copy link
Contributor

@nishakm can I work on it?

@rnjudge
Copy link
Contributor

rnjudge commented Jun 3, 2020

@jloehel there is now a way to list more than one directive to install/remove snippet packages. We should have a PR merged soon to add the zypper support that you requested. Thanks for opening this issue!

@jloehel
Copy link
Author

jloehel commented Jun 3, 2020

@jloehel there is now a way to list more than one directive to install/remove snippet packages. We should have a PR merged soon to add the zypper support that you requested. Thanks for opening this issue!

Thank you very much. You are doing great job here. Thanks for the support.

@00032118
Copy link

00032118 commented Jun 5, 2020

please Add microdnf by the way!thank you.
microdnf:
install: 'install'
remove: 'remove'
ignore:
- 'check-update'
- 'clean'
packages: 'rpm'

@rnjudge
Copy link
Contributor

rnjudge commented Jun 5, 2020

please Add microdnf by the way!thank you.
microdnf:
install: 'install'
remove: 'remove'
ignore:

  • 'check-update'
  • 'clean'
    packages: 'rpm'

@00032118 If you want to submit a PR for this, we would gladly welcome it! Otherwise, please file a new issue requesting this functionality so we can refer to it separately :)

Thanks you!

rnjudge pushed a commit that referenced this issue Jun 5, 2020
This commit adds zypper package manager
in snippet.yml to analyze openSUSE
images

Resolved: #693

Signed-off-by: mukultaneja <mtaneja@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature new feature help wanted intermediate Intermediate level issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants