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.

Sign up for GitHub

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

Jump to bottom

Provide better and more complete cleaning options with stats #989

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Provide better and more complete cleaning options with stats #989

wants to merge 2 commits into from

Conversation

DevonianTeuchter
Copy link
Collaborator

What type of PR is this? (check all applicable)

  • Refactor
  • [x ] Feature
  • [x ] Bug Fix
  • Optimization
  • Content Update

Category

  • zopen build framework
  • [x ] zopen package manager
  • Documentation
  • CI/CD
  • Tools

Description

This PR:

  • updates zopen-clean to clean metadata associated with a package when the --clean option is specified.
  • adds statistics as to space freed on the system after an operation, with --nostats option to hide the stats
  • provides a --deep option that will perform all cleanup operations [--all is used for a different purpose aready]
  • allows for more than one cleanup operation; eg. zopen clean --cache --meta

Related Issues

[optional] Are there any post-deployment tasks or follow-up actions required?

IgorTodorovskiIBM
IgorTodorovskiIBM approved these changes Mar 20, 2025
View reviewed changes
Copy link
Collaborator

@IgorTodorovskiIBM IgorTodorovskiIBM left a comment

Choose a reason for hiding this comment

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

LGTM

@IgorTodorovskiIBM
Copy link
Collaborator

Looks like the metaport tests failed as a result of this change

@DevonianTeuchter
Copy link
Collaborator Author

Looks like the metaport tests failed as a result of this change

Update to zopen_check_clean in metaport should resolve this once merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IgorTodorovskiIBM IgorTodorovskiIBM IgorTodorovskiIBM approved these changes

@MikeFultonDev MikeFultonDev Awaiting requested review from MikeFultonDev MikeFultonDev is a code owner

@HarithaIBM HarithaIBM Awaiting requested review from HarithaIBM HarithaIBM is a code owner

@v1gnesh v1gnesh Awaiting requested review from v1gnesh v1gnesh is a code owner

@sachintu47 sachintu47 Awaiting requested review from sachintu47 sachintu47 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

zopen clean does not clean
2 participants
@DevonianTeuchter @IgorTodorovskiIBM