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

WIP: Add finding of unused network resources #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

queria
Copy link
Contributor

@queria queria commented May 12, 2016

This adds new --unused-netres selection behaviour,
which selects (for possible cleanup) resources as:

  • floating IPs which are not assigned to VM
  • networks which do not have any VM (port for compute)
  • routers which do not have any port

While excluding shared/external resources, and
doing intersect of 3 searches with 1 minute delay
(both configurable) to rule out possibly freshly created
resources (e.g. script creating network but not yet reached adding vm).

Possibly solution to #2.

This adds new --unused-netres selection behaviour,
which selects (for possible cleanup) resources as:
- floating IPs which are not assigned to VM
- networks which do not have any VM (port for compute)
- routers which do not have any port

While excluding shared/external resources, and
doing intersect of 3 searches with 1 minute delay
(both configurable) to rule out possibly freshly created
resources (e.g. script creating network but not yet reached adding vm).
@queria
Copy link
Contributor Author

queria commented May 12, 2016

I don't expect this to be merged as is,
opening this PR for commenting.

Still want to test it a bit more, possibly expose more opts on cli,
and add at least basic tests, though that maybe in separate commit later.

@queria queria changed the title Add finding of unused network resources WIP: Add finding of unused network resources May 12, 2016
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.

1 participant