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

FOREIGN profiles are not cleaned up #195

Open
gombasg opened this issue Dec 5, 2018 · 0 comments
Open

FOREIGN profiles are not cleaned up #195

gombasg opened this issue Dec 5, 2018 · 0 comments
Labels

Comments

@gombasg
Copy link
Contributor

gombasg commented Dec 5, 2018

The documentation of fetchProfile() says:

Only a single (most recent) copy of the foreign copy will be stored: previous versions will be removed.

However, that functionality was apparently broken by commit 7bc02dd by removing this fragment:

      if ($foreign_profile) {
          $latest = 0;
          system "rm -rf $cache_root/profile.$latest";
      }

... and I did not found any replacement of this code fragment. Since we've recently done an upgrade from an ancient version of CCM on our AII servers, we've just noticed the space not getting cleaned up.

Since the code was rewritten substantially, I don't immediately see where could this old logic be re-implemented.

@jrha jrha added the bug label Dec 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants