Merge branch 'revert-mr-54089-incident-3632' into 'master'

Revert "Fixes GraphQL pipeline status notification with warnings"

See merge request gitlab-org/gitlab!54268

app/assets/javascripts/vue_merge_request_widget/queries/get_state.query.graphql

@@ -14,7 +14,6 @@ query getState($projectPath: ID!, $iid: String!) {
       pipelines(first: 1) {
         nodes {
           status
-          warnings
         }
       }
       shouldBeRebased

app/assets/javascripts/vue_merge_request_widget/stores/mr_widget_store.js

@@ -172,11 +172,6 @@ export default class MergeRequestStore {
     this.canBeMerged = mergeRequest.mergeStatus === 'can_be_merged';
     this.canMerge = mergeRequest.userPermissions.canMerge;
     this.ciStatus = pipeline?.status.toLowerCase();
-
-    if (pipeline?.warnings && this.ciStatus === 'success') {
-      this.ciStatus = `${this.ciStatus}-with-warnings`;
-    }
-
     this.commitsCount = mergeRequest.commitCount || 10;
     this.branchMissing = !mergeRequest.sourceBranchExists || !mergeRequest.targetBranchExists;
     this.hasConflicts = mergeRequest.conflicts;

app/graphql/types/ci/pipeline_type.rb

@@ -27,9 +27,6 @@ module Types
       field :status, PipelineStatusEnum, null: false,
             description: "Status of the pipeline (#{::Ci::Pipeline.all_state_names.compact.join(', ').upcase})"
 
-      field :warnings, GraphQL::BOOLEAN_TYPE, null: false, method: :has_warnings?,
-            description: "Indicates if a pipeline has warnings."
-
       field :detailed_status, Types::Ci::DetailedStatusType, null: false,
             description: 'Detailed status of the pipeline.'
 

changelogs/unreleased/ph-ph-fixWidgetGraphqlPipelineWarnings.yml

----
-title: Added warnings field to the pipelines GraphQL type
-merge_request: 54089
-author:
-type: added

doc/api/graphql/reference/gitlab_schema.graphql

@@ -18566,11 +18566,6 @@ type Pipeline {
   Permissions for the current user on the resource
   """
   userPermissions: PipelinePermissions!
-
-  """
-  Indicates if a pipeline has warnings.
-  """
-  warnings: Boolean!
 }
 
 type PipelineAnalytics {

doc/api/graphql/reference/gitlab_schema.json

@@ -54399,24 +54399,6 @@
               },
               "isDeprecated": false,
               "deprecationReason": null
-            },
-            {
-              "name": "warnings",
-              "description": "Indicates if a pipeline has warnings.",
-              "args": [
-
-              ],
-              "type": {
-                "kind": "NON_NULL",
-                "name": null,
-                "ofType": {
-                  "kind": "SCALAR",
-                  "name": "Boolean",
-                  "ofType": null
-                }
-              },
-              "isDeprecated": false,
-              "deprecationReason": null
             }
           ],
           "inputFields": null,

doc/api/graphql/reference/index.md

@@ -2807,7 +2807,6 @@ Information about pagination in a connection..
 | `upstream` | Pipeline | Pipeline that triggered the pipeline. |
 | `user` | User | Pipeline user. |
 | `userPermissions` | PipelinePermissions! | Permissions for the current user on the resource |
-| `warnings` | Boolean! | Indicates if a pipeline has warnings. |
 
 ### PipelineAnalytics
 

spec/graphql/types/ci/pipeline_type_spec.rb

@@ -12,7 +12,7 @@ RSpec.describe Types::Ci::PipelineType do
       id iid sha before_sha status detailed_status config_source duration
       coverage created_at updated_at started_at finished_at committed_at
       stages user retryable cancelable jobs source_job downstream
-      upstream path project active user_permissions warnings
+      upstream path project active user_permissions
     ]
 
     if Gitlab.ee?