1 files changed, 36 insertions, 8 deletions

diff --git a/lib/api/helm_packages.rb b/lib/api/helm_packages.rb
index fa2537bcfc4..8260d8a88f8 100644
--- a/lib/api/helm_packages.rb
+++ b/lib/api/helm_packages.rb
@@ -32,15 +32,21 @@ module API
     end
 
     params do
-      requires :id, type: String, desc: 'The ID or full path of a project'
+      requires :id, types: [Integer, String], desc: 'The ID or full path of a project'
     end
     resource :projects, requirements: API::NAMESPACE_OR_PROJECT_REQUIREMENTS do
       namespace ':id/packages/helm', requirements: HELM_REQUIREMENTS do
         desc 'Download a chart index' do
           detail 'This feature was introduced in GitLab 14.0'
+          success code: 200
+          failure [
+            { code: 401, message: 'Unauthorized' },
+            { code: 403, message: 'Forbidden' }
+          ]
+          tags %w[helm_packages]
         end
         params do
-          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex
+          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex, documentation: { example: 'stable' }
         end
 
         get ":channel/index.yaml" do
@@ -56,10 +62,17 @@ module API
 
         desc 'Download a chart' do
           detail 'This feature was introduced in GitLab 14.0'
+          success code: 200
+          failure [
+            { code: 401, message: 'Unauthorized' },
+            { code: 403, message: 'Forbidden' },
+            { code: 404, message: 'Not Found' }
+          ]
+          tags %w[helm_packages]
         end
         params do
-          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex
-          requires :file_name, type: String, desc: 'Helm package file name'
+          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex, documentation: { example: 'stable' }
+          requires :file_name, type: String, desc: 'Helm package file name', documentation: { example: 'mychart' }
         end
         get ":channel/charts/:file_name.tgz" do
           project = authorized_user_project(action: :read_package)
@@ -67,16 +80,23 @@ module API
 
           package_file = Packages::Helm::PackageFilesFinder.new(project, params[:channel], file_name: "#{params[:file_name]}.tgz").most_recent!
 
-          track_package_event('pull_package', :helm, project: project, namespace: project.namespace)
+          track_package_event('pull_package', :helm, project: project, namespace: project.namespace, property: 'i_package_helm_user')
 
           present_package_file!(package_file)
         end
 
         desc 'Authorize a chart upload from workhorse' do
           detail 'This feature was introduced in GitLab 14.0'
+          success code: 200
+          failure [
+            { code: 401, message: 'Unauthorized' },
+            { code: 403, message: 'Forbidden' },
+            { code: 404, message: 'Not Found' }
+          ]
+          tags %w[helm_packages]
         end
         params do
-          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex
+          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex, documentation: { example: 'stable' }
         end
         post "api/:channel/charts/authorize" do
           authorize_workhorse!(
@@ -88,9 +108,16 @@ module API
 
         desc 'Upload a chart' do
           detail 'This feature was introduced in GitLab 14.0'
+          success code: 201
+          failure [
+            { code: 401, message: 'Unauthorized' },
+            { code: 403, message: 'Forbidden' },
+            { code: 404, message: 'Not Found' }
+          ]
+          tags %w[helm_packages]
         end
         params do
-          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex
+          requires :channel, type: String, desc: 'Helm channel', regexp: Gitlab::Regex.helm_channel_regex, documentation: { example: 'stable' }
           requires :chart, type: ::API::Validations::Types::WorkhorseFile, desc: 'The chart file to be published (generated by Multipart middleware)', documentation: { type: 'file' }
         end
         post "api/:channel/charts" do
@@ -110,7 +137,8 @@ module API
             package, chart_params.merge(build: current_authenticated_job)
           ).execute
 
-          track_package_event('push_package', :helm, project: authorized_user_project, namespace: authorized_user_project.namespace)
+          track_package_event('push_package', :helm, project: authorized_user_project, namespace: authorized_user_project.namespace,
+                                                     property: 'i_package_helm_user')
 
           ::Packages::Helm::ExtractionWorker.perform_async(params[:channel], chart_package_file.id) # rubocop:disable CodeReuse/Worker