Commit 5ee58c1c authored by Kamil Trzcinski's avatar Kamil Trzcinski

Add short_revision and use total_size

parent 57d331d7
...@@ -38,7 +38,7 @@ export default { ...@@ -38,7 +38,7 @@ export default {
tag: element.name, tag: element.name,
revision: element.revision, revision: element.revision,
shortRevision: element.short_revision, shortRevision: element.short_revision,
size: element.size, size: element.total_size,
layers: element.layers, layers: element.layers,
location: element.location, location: element.location,
createdAt: element.created_at, createdAt: element.created_at,
......
class ContainerTagEntity < Grape::Entity class ContainerTagEntity < Grape::Entity
include RequestAwareEntity include RequestAwareEntity
expose :name, :location, :revision, :total_size, :created_at expose :name, :location, :revision, :short_revision, :total_size, :created_at
expose :destroy_path, if: -> (*) { can_destroy? } do |tag| expose :destroy_path, if: -> (*) { can_destroy? } do |tag|
project_registry_repository_tag_path(project, tag.repository, tag.name, format: :json) project_registry_repository_tag_path(project, tag.repository, tag.name, format: :json)
......
---
title: Fix revision and total size missing for Container Registry
merge_request:
author:
type: fixed
...@@ -14,6 +14,11 @@ ...@@ -14,6 +14,11 @@
"revision": { "revision": {
"type": "string" "type": "string"
}, },
"short_revision": {
"type": "string",
"minLength": 8,
"maxLength": 8
},
"total_size": { "total_size": {
"type": "integer" "type": "integer"
}, },
......
...@@ -26,7 +26,7 @@ export const registryServerResponse = [ ...@@ -26,7 +26,7 @@ export const registryServerResponse = [
name: 'centos7', name: 'centos7',
short_revision: 'b118ab5b0', short_revision: 'b118ab5b0',
revision: 'b118ab5b0e90b7cb5127db31d5321ac14961d097516a8e0e72084b6cdc783b43', revision: 'b118ab5b0e90b7cb5127db31d5321ac14961d097516a8e0e72084b6cdc783b43',
size: 679, total_size: 679,
layers: 19, layers: 19,
location: 'location', location: 'location',
created_at: 1505828744434, created_at: 1505828744434,
...@@ -36,7 +36,7 @@ export const registryServerResponse = [ ...@@ -36,7 +36,7 @@ export const registryServerResponse = [
name: 'centos6', name: 'centos6',
short_revision: 'b118ab5b0', short_revision: 'b118ab5b0',
revision: 'b118ab5b0e90b7cb5127db31d5321ac14961d097516a8e0e72084b6cdc783b43', revision: 'b118ab5b0e90b7cb5127db31d5321ac14961d097516a8e0e72084b6cdc783b43',
size: 679, total_size: 679,
layers: 19, layers: 19,
location: 'location', location: 'location',
created_at: 1505828744434, created_at: 1505828744434,
......
...@@ -22,7 +22,7 @@ describe ContainerTagEntity do ...@@ -22,7 +22,7 @@ describe ContainerTagEntity do
end end
it 'exposes required informations' do it 'exposes required informations' do
expect(subject).to include(:name, :location, :revision, :total_size, :created_at) expect(subject).to include(:name, :location, :revision, :short_revision, :total_size, :created_at)
end end
context 'when user can manage repositories' do context 'when user can manage repositories' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment