Skip to content

Cluster Agent delete mutation for GraphQl

Emily Ring requested to merge emilyring-agent-graphql-delete into master

What does this MR do?

Related to issue: gitlab-org/cluster-integration/gitlab-agent#6 (closed)

We recently created an Agent Model and Token Model. MR: !33228 (merged)

Now that models have been created, we need to make GraphQl calls to access/remove agents and tokens

Per another discussion, this feature will only be available for premium plans

This MR will:

  • Add GraphQl Clusters::Agent delete mutation
  • Add Clusters::Agents::DeleteService
  • Add associated tests
  • Updated associated GraphQl documents

Previous MRs:

Future MRs will:

Screenshots

mutation createAgent {
  createClusterAgent(input: { projectPath: <project-path>, name: <agent-name> }) {
    clusterAgent {
      id 
      name
    }
    errors
  } 
}

mutation deleteAgent {
  clusterAgentDelete(input: { id: <agent-global-id> }) {
    errors
  } 
}

deleteAgent

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 Nick Thomas

Merge request reports