Skip to content

Draft: 259631_Improve_Projects_ContainerRepository_Gitlab_DeleteTagsService_logging

Maksim Stankevic requested to merge (removed):master into master

What does this MR do?

This MR improves Container Registry tag deletion logging, issue #259631 (closed)

Sample log output after change:

2020-10-18T17:37:06.282Z: {:service_class=>"Projects::ContainerRepository::DeleteTagsService", :container_repository_id=>1, :message=>"timeout while deleting tags", :deleted_tags_count=>1}

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Maksim Stankevic

Merge request reports