diff options
| author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-03-31 11:54:09 +0200 |
|---|---|---|
| committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-03-31 11:54:09 +0200 |
| commit | a7466af3a6f31311d64654631a2ea2740c42b88e (patch) | |
| tree | 7eb1d3f63a73cc2d4cca13ba3640d16c7e654b98 /app/models/container_repository.rb | |
| parent | 600bbe15a103b63e14daa295abaffdf1aeafaef3 (diff) | |
| download | gitlab-ce-a7466af3a6f31311d64654631a2ea2740c42b88e.tar.gz | |
Improve code related to removing container image tags
Diffstat (limited to 'app/models/container_repository.rb')
| -rw-r--r-- | app/models/container_repository.rb | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/app/models/container_repository.rb b/app/models/container_repository.rb index 5663b3db92f..052d93c3bdc 100644 --- a/app/models/container_repository.rb +++ b/app/models/container_repository.rb @@ -45,15 +45,14 @@ class ContainerRepository < ActiveRecord::Base # TODO, specs needed # def has_tags? - tags.any? + tags.to_a.any? end - # TODO, add bang to this method - # - def delete_tags - return unless tags + def delete_tags! + return unless has_tags? + + digests = tags.map { |tag| tag.digest }.to_set - digests = tags.map {|tag| tag.digest }.to_set digests.all? do |digest| client.delete_repository_tag(self.path, digest) end |