GraphQL: Use query field instead of mutation

What does this MR do and why?

This MR is a follow-up to !145598 (comment 1789427546) to replace the provisionGoogleCloudRunner mutation with the provisioningSteps query field. It also renames runnerCloudProvisioningOptions to runnerCloudProvisioning, since this is no longer only about options.

EE: true

Closes #438316 (closed)

Review status

Reviews	Reviewer	Status
backend (initial)	@10io
backend (maintainer)	@dmeshcharakou
frontend (maintainer)	@mrincon
devopsverify	@vshushlin
authorization	@jarka
groupimport and integrate	@bmarjanovic
documentation	@fneill

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before	After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

1. Run the following query in http://gdk.test:3000/-/graphql-explorer:

   {
     project(fullPath: "gitlab-org/playground") {
       id
       runnerCloudProvisioning(
         provider: GOOGLE_CLOUD
         cloudProjectId: "dev-gcp-s3c-integrati-9abafed1"
       ) {
         ... on CiRunnerGoogleCloudProvisioning {
           provisioningSteps(
             region: "us-central1"
             zone: "us-central1-a"
             ephemeralMachineType: "n2d-standard-2"
             runnerToken: "glrt-Ei-AAaaaaAAAaaAA"
           ) {
             title
             languageIdentifier
             instructions
           }
         }
       }
     }
   }

changed milestone to %16.10

added devopsverify grouprunner sectionci + 1 deleted label

assigned to @pedropombeiro

mentioned in merge request !145598 (merged)

added Architecture Evolution Blueprint backend database databasereview pending documentation feature flag feature flagexists frontend labels

	7 Warnings
	This merge request is quite big (612 lines changed), please consider splitting it into multiple merge requests.
	e616c5d3: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	fb7346e5: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
	395d3b37: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
	d4f83932: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
	d4f83932: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	70cd5a04: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.

	2 Messages
	CHANGELOG missing: If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog. If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
	This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge.

Documentation review

The following files require a review from a technical writer:

• doc/api/graphql/reference/index.md (Link to current live version)

The review does not need to block merging this merge request. See the:

• Metadata for the *.md files that you've changed. The first few lines of each *.md file identify the stage and group most closely associated with your docs change.
• The Technical Writer assigned for that stage and group.
• Documentation workflows for information on when to assign a merge request for review.

Reviewer roulette

Category	Reviewer	Maintainer
backend	@rcobb (UTC-8, 9 hours behind author)	@patrickbajao (UTC+8, 7 hours ahead of author)
frontend	@andr3 (UTC+0, 1 hour behind author)	@himkp (UTC+1, same timezone as author)
groupimport and integrate (backend)	@carlad-gl (UTC+8, 7 hours ahead of author)	Maintainer review is optional for groupimport and integrate (backend)
Verify	Reviewer review is optional for Verify	@stanhu (UTC-8, 9 hours behind author)
~"Authorization"	Reviewer review is optional for ~"Authorization"	@alexbuijs (UTC+1, same timezone as author)

Please check reviewer's status!

• Reviewer is available!
• Reviewer is unavailable!

Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

added Data WarehouseImpact Check label

Proper labels assigned to this merge request. Please ignore me.

@pedropombeiro - please see the following guidance and update this merge request.

	1 Error
	Please add typebug typefeature, or typemaintenance label to this merge request.

added 74 commits

• aa7bffda...3db2be1d - 71 commits from branch pedropombeiro/438316/add-script
• 56d3e9be - GraphQL: Use query field instead of mutation
• 1aae930f - Use scalar types
• 7c23a542 - GraphQL: Remove Mutation.provisionGoogleCloudRunner

Compare with previous version

marked this merge request as draft

changed title from Add temporary index for locating orphaned notes to Draft: GraphQL: Use query field instead of mutation

removed Architecture Evolution Blueprint Data WarehouseImpact Check database databasereview pending feature flag frontend + 1 deleted label

added Category:Fleet Visibility GraphQL featureconsolidation gcp+gitlab integration typefeature labels

changed the description

added 3 commits

• 0070f241 - GraphQL: Use query field instead of mutation
• 44492c9f - Use scalar types
• 66dbf54f - GraphQL: Remove Mutation.provisionGoogleCloudRunner

Compare with previous version

marked this merge request as ready

Hey @10io , mind doing the backend initial review? This is probably easier to review commit-by-commit.

requested review from @10io

mentioned in epic &12111 (closed)

added 1 commit

• 0993c3ba - GraphQL: Rename `Project.runnerCloudProvisioningOptions`

Compare with previous version

added 1 commit

• 0b67231f - GraphQL: Rename `Project.runnerCloudProvisioningOptions`

Compare with previous version

E2E Test Result Summary

allure-report-publisher generated test report!

e2e-package-and-test: test report for e616c5d3

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Create      | 149    | 4      | 19      | 4     | 172   | ❌     |
| Verify      | 147    | 0      | 30      | 3     | 177   | ✅     |
| Plan        | 8      | 0      | 0       | 0     | 8     | ✅     |
| Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
| Package     | 0      | 0      | 2       | 0     | 2     | ➖     |
| Govern      | 6      | 0      | 0       | 0     | 6     | ✅     |
| Data Stores | 4      | 0      | 0       | 0     | 4     | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 322    | 4      | 51      | 7     | 377   | ❌     |
+-------------+--------+--------+---------+-------+-------+--------+

e2e-test-on-gdk: test report for e616c5d3

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Verify      | 35     | 0      | 1       | 0     | 36    | ✅     |
| Govern      | 3      | 0      | 0       | 0     | 3     | ✅     |
| Plan        | 4      | 0      | 0       | 0     | 4     | ✅     |
| Create      | 8      | 0      | 3       | 0     | 11    | ✅     |
| Data Stores | 2      | 0      | 0       | 0     | 2     | ✅     |
| Monitor     | 4      | 0      | 0       | 0     | 4     | ✅     |
| Package     | 0      | 0      | 1       | 0     | 1     | ➖     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 56     | 0      | 5       | 0     | 61    | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+

added 1 commit

• ec3d83af - GraphQL: Rename `Project.runnerCloudProvisioningOptions`

Compare with previous version

added 1 commit

• cc96799a - GraphQL: Rename `Project.runnerCloudProvisioningOptions`

Compare with previous version

added frontend label

removed review request for @10io

changed the description

deleted the pedropombeiro/438316/add-script branch. This merge request now targets the master branch

added 241 commits

• cc96799a...d76a1c11 - 235 commits from branch master
• 89b6621e - GraphQL: Use query field instead of mutation
• e78854aa - Use scalar types
• d5ff6d5d - GraphQL: Remove Mutation.provisionGoogleCloudRunner
• 6ef79f71 - GraphQL: Rename `Project.runnerCloudProvisioningOptions`
• 17b4abe3 - Minor improvements
• de29b47c - Add missing default_value for runner_token

Compare with previous version

requested review from @10io

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits 5115da23 and e616c5d3

Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
average	4.21 MB	4.21 MB	-	-0.0 %
mainChunk	3.21 MB	3.21 MB	-	-0.0 %

---

Note: We do not have exact data for 5115da23. So we have used data from: 512e1564.
The intended commit has no webpack pipeline, so we chose the last commit with one before it.

Please look at the full report for more details

---

Read more about how this report works.

Generated by Danger

added 138 commits

• de29b47c...110f977a - 132 commits from branch master
• 3199d44c - GraphQL: Use query field instead of mutation
• 45263c52 - Use scalar types
• 3a337c6f - GraphQL: Remove Mutation.provisionGoogleCloudRunner
• ac97645f - GraphQL: Rename `Project.runnerCloudProvisioningOptions`
• d6da7f19 - Minor improvements
• 4016786a - Add missing default_value for runner_token

Compare with previous version

changed the description

Hey @vshushlin , mind doing the maintainer Verify review?

requested review from @vshushlin

Hey @mrincon , mind doing the frontend maintainer review? I believe it is because of the rename that happend in app/assets/javascripts/graphql_shared/possible_types.json

requested review from @mrincon

Hey @jarka , mind doing the authorization maintainer review?

requested review from @jarka

Hey @bmarjanovic , mind doing the groupimport and integrate review?

requested review from @bmarjanovic

requested review from @fneill

changed the description

added RunnerP1 priority1 labels

added 2 commits

• 03f02d3a - Minor improvements
• bdb07e82 - Add missing default_value for runner_token

Compare with previous version

approved this merge request

added pipeline:mr-approved label

@vshushlin, thanks for approving this merge request.

This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break master, a new pipeline will be started shortly.

Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.

changed the description

removed pipeline:mr-approved label

Thanks, @pedropombeiro! I did only surface-level review, but it looks good Great refactoring, and thanks for not postponing it

I left one question just out of curiosity =)

removed review request for @vshushlin

added pipeline:mr-approved label

added 2 commits

• a1d2c23d - Fix spec
• 0651f948 - Add alias

Compare with previous version

added 1 commit

• 4d6615ff - Add alias

Compare with previous version

reset approvals from @vshushlin by pushing to the branch

added 1 commit

• 1f9b5021 - Add alias

Compare with previous version

approved this merge request

approved this merge request

removed review request for @10io

approved this merge request

requested review from @dmeshcharakou

changed the description

approved this merge request

removed review request for @fneill

Hi @fneill ,

GitLab Bot has added the Technical Writing label because a Technical Writer has approved or merged this MR.

This message was generated automatically. You're welcome to improve it.

added Technical Writing label

changed the description

added 1 commit

• 2c32e9cb - Fix coverage issue

Compare with previous version

reset approvals from @mrincon, @vshushlin, and @10io by pushing to the branch

changed the description

removed review request for @dmeshcharakou

approved this merge request

removed review request for @mrincon

requested review from @bmarjanovic

approved this merge request

removed review request for @bmarjanovic

added 1 commit

• 4fa35910 - Address MR review comments

Compare with previous version

reset approvals from @bmarjanovic and @dmeshcharakou by pushing to the branch

changed the description

@vshushlin do you mind reapproving, as the approval was lost with the recent changes?

resolved all threads

added 78 commits

• 4fa35910...f379c859 - 68 commits from branch master
• b71d5c48 - GraphQL: Use query field instead of mutation
• 70cd5a04 - Use scalar types
• 04dfc257 - GraphQL: Remove Mutation.provisionGoogleCloudRunner
• f939d293 - GraphQL: Rename `Project.runnerCloudProvisioningOptions`
• d4f83932 - Minor improvements
• 991d0910 - Add missing default_value for runner_token
• 395d3b37 - Fix spec
• fb7346e5 - Add alias
• b94c5656 - Fix coverage issue
• 77f695c4 - Address MR review comments

Compare with previous version

reset approvals from @fneill by pushing to the branch

approved this merge request

changed the description

added 1 commit

• e616c5d3 - Address MR review comments

Compare with previous version

reset approvals from @vshushlin by pushing to the branch

approved this merge request

approved this merge request

removed review request for @jarka

changed the description

requested review from @jarka

resolved all threads

enabled an automatic merge when the pipeline for 9090f91f succeeds

merged

mentioned in commit 650f801f

mentioned in merge request !146107 (merged)

added workflowstaging-canary label

added workflowcanary label and removed workflowstaging-canary label

added workflowstaging label and removed workflowcanary label

added workflowproduction label and removed workflowstaging label

added releasedcandidate label

added workflowpost-deploy-db-staging label and removed workflowproduction label

added workflowpost-deploy-db-production label and removed workflowpost-deploy-db-staging label

added releasedpublished label and removed releasedcandidate label

added pipelinetier-3 label