diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-05-17 19:05:49 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-05-17 19:05:49 +0300 |
commit | 43a25d93ebdabea52f99b05e15b06250cd8f07d7 (patch) | |
tree | dceebdc68925362117480a5d672bcff122fb625b /lib/api/clusters | |
parent | 20c84b99005abd1c82101dfeff264ac50d2df211 (diff) |
Add latest changes from gitlab-org/gitlab@16-0-stable-eev16.0.0-rc42
Diffstat (limited to 'lib/api/clusters')
-rw-r--r-- | lib/api/clusters/agent_tokens.rb | 15 | ||||
-rw-r--r-- | lib/api/clusters/agents.rb | 4 |
2 files changed, 11 insertions, 8 deletions
diff --git a/lib/api/clusters/agent_tokens.rb b/lib/api/clusters/agent_tokens.rb index 68eef21903d..9e23756db33 100644 --- a/lib/api/clusters/agent_tokens.rb +++ b/lib/api/clusters/agent_tokens.rb @@ -7,7 +7,7 @@ module API before { authenticate! } - feature_category :kubernetes_management + feature_category :deployment_management params do requires :id, types: [String, Integer], desc: 'The ID or URL-encoded path of the project' @@ -28,7 +28,7 @@ module API end get do agent = ::Clusters::AgentsFinder.new(user_project, current_user).find(params[:agent_id]) - agent_tokens = ::Clusters::AgentTokensFinder.new(agent, current_user).execute + agent_tokens = ::Clusters::AgentTokensFinder.new(agent, current_user, status: :active).execute present paginate(agent_tokens), with: Entities::Clusters::AgentTokenBasic end @@ -43,7 +43,7 @@ module API end get ':token_id' do agent = ::Clusters::AgentsFinder.new(user_project, current_user).find(params[:agent_id]) - token = ::Clusters::AgentTokensFinder.new(agent, current_user).find(params[:token_id]) + token = ::Clusters::AgentTokensFinder.new(agent, current_user, status: :active).find(params[:token_id]) present token, with: Entities::Clusters::AgentToken end @@ -65,7 +65,9 @@ module API agent = ::Clusters::AgentsFinder.new(user_project, current_user).find(params[:agent_id]) result = ::Clusters::AgentTokens::CreateService.new( - container: agent.project, current_user: current_user, params: token_params.merge(agent_id: agent.id) + agent: agent, + current_user: current_user, + params: token_params ).execute bad_request!(result[:message]) if result[:status] == :error @@ -86,8 +88,9 @@ module API agent = ::Clusters::AgentsFinder.new(user_project, current_user).find(params[:agent_id]) token = ::Clusters::AgentTokensFinder.new(agent, current_user).find(params[:token_id]) - # Skipping explicit error handling and relying on exceptions - token.revoked! + result = ::Clusters::AgentTokens::RevokeService.new(token: token, current_user: current_user).execute + + bad_request!(result[:message]) if result[:status] == :error status :no_content end diff --git a/lib/api/clusters/agents.rb b/lib/api/clusters/agents.rb index 62d4fb009c6..02469fbad21 100644 --- a/lib/api/clusters/agents.rb +++ b/lib/api/clusters/agents.rb @@ -7,7 +7,7 @@ module API before { authenticate! } - feature_category :kubernetes_management + feature_category :deployment_management urgency :low params do @@ -23,7 +23,7 @@ module API use :pagination end get ':id/cluster_agents' do - not_found!('ClusterAgents') unless can?(current_user, :read_cluster, user_project) + not_found!('ClusterAgents') unless can?(current_user, :read_cluster_agent, user_project) agents = ::Clusters::AgentsFinder.new(user_project, current_user).execute |