Add latest changes from gitlab-org/gitlab@master

Author: GitLab Bot

app/assets/javascripts/vue_shared/components/crud_component.vue

@@ -1,13 +1,20 @@
 <script>
-import { GlButton, GlIcon, GlBadge, GlLoadingIcon, GlLink, GlTooltipDirective } from '@gitlab/ui';
+import {
+  GlButton,
+  GlIcon,
+  GlBadge,
+  GlSkeletonLoader,
+  GlLink,
+  GlTooltipDirective,
+} from '@gitlab/ui';
 import { __ } from '~/locale';
 
 export default {
   components: {
     GlButton,
     GlIcon,
     GlBadge,
-    GlLoadingIcon,
+    GlSkeletonLoader,
     GlLink,
   },
   directives: {
@@ -121,7 +128,7 @@ export default {
     },
     displayedCount() {
       if (this.isLoading) {
-        return '...';
+        return null;
       }
 
       return this.icon && !this.count ? '0' : this.count;
@@ -276,7 +283,8 @@ export default {
       :class="[bodyClass, { 'gl-rounded-b-base': !$scopedSlots.footer }]"
       data-testid="crud-body"
     >
-      <gl-loading-icon v-if="isLoading" size="sm" data-testid="crud-loading" />
+      <gl-skeleton-loader v-if="isLoading" :width="400" :lines="3" data-testid="crud-loading" />
+
       <span v-else-if="$scopedSlots.empty" class="gl-text-subtle" data-testid="crud-empty">
         <slot name="empty"></slot>
       </span>

app/assets/javascripts/webhooks/components/webhook_form_app.vue

@@ -1,9 +1,14 @@
 <script>
+import { GlFormGroup, GlFormInput, GlFormTextarea } from '@gitlab/ui';
+
 import FormUrlApp from './form_url_app.vue';
 import FormCustomHeaders from './form_custom_headers.vue';
 
 export default {
   components: {
+    GlFormGroup,
+    GlFormInput,
+    GlFormTextarea,
     FormUrlApp,
     FormCustomHeaders,
   },
@@ -16,19 +21,57 @@ export default {
     initialUrlVariables: {
       type: Array,
       required: false,
-      default: null,
+      default: () => [],
     },
     initialCustomHeaders: {
       type: Array,
       required: false,
-      default: null,
+      default: () => [],
+    },
+    initialName: {
+      type: String,
+      required: false,
+      default: '',
+    },
+    initialDescription: {
+      type: String,
+      required: false,
+      default: '',
     },
   },
+  data() {
+    return {
+      name: this.initialName,
+      description: this.initialDescription,
+    };
+  },
 };
 </script>
 
 <template>
   <div>
+    <gl-form-group :label="s__('Webhooks|Name (optional)')" label-for="webhook-name">
+      <gl-form-input
+        id="webhook-name"
+        v-model="name"
+        name="hook[name]"
+        class="gl-form-input-xl"
+        data-testid="webhook-name"
+      />
+    </gl-form-group>
+
+    <gl-form-group :label="s__('Webhooks|Description (optional)')" label-for="webhook-description">
+      <gl-form-textarea
+        id="webhook-description"
+        v-model="description"
+        name="hook[description]"
+        class="gl-form-input-xl"
+        rows="4"
+        maxlength="2048"
+        data-testid="webhook-description"
+      />
+    </gl-form-group>
+
     <form-url-app :initial-url="initialUrl" :initial-url-variables="initialUrlVariables" />
     <form-custom-headers :initial-custom-headers="initialCustomHeaders" />
   </div>

app/assets/javascripts/webhooks/index.js

@@ -9,14 +9,22 @@ export default () => {
     return null;
   }
 
-  const { url: initialUrl, urlVariables, customHeaders } = el.dataset;
+  const {
+    name: initialName,
+    description: initialDescription,
+    url: initialUrl,
+    urlVariables,
+    customHeaders,
+  } = el.dataset;
 
   return new Vue({
     el,
     name: 'WebhookFormRoot',
     render(createElement) {
       return createElement(WebhookFormApp, {
         props: {
+          initialName,
+          initialDescription,
           initialUrl,
           initialUrlVariables: JSON.parse(urlVariables),
           initialCustomHeaders: JSON.parse(customHeaders),

app/helpers/hooks_helper.rb

@@ -3,6 +3,8 @@
 module HooksHelper
   def webhook_form_data(hook)
     {
+      name: hook.name,
+      description: hook.description,
       url: hook.url,
       url_variables: Gitlab::Json.dump(hook.url_variables.keys.map { { key: _1 } }),
       custom_headers: Gitlab::Json.dump(hook.custom_headers.keys.map { { key: _1, value: WebHook::SECRET_MASK } })

app/models/packages/terraform_module/metadatum.rb

@@ -15,7 +15,7 @@ class Metadatum < ApplicationRecord
 
       attribute :fields, default: -> { {} }
 
-      ignore_column :semver_patch_convert_to_bigint, remove_with: '18.1', remove_after: '2025-05-20'
+      ignore_column :semver_patch_convert_to_bigint, remove_with: '18.2', remove_after: '2025-06-20'
 
       validates :package, :project, presence: true
       validates :fields, json_schema: { filename: 'terraform_module_metadata', detail_errors: true },

app/views/admin/hooks/_form.html.haml

@@ -1,15 +1,15 @@
 = form_errors(hook)
 
-.form-group
-  = form.label :url, _('URL'), class: 'label-bold'
-  = form.text_field :url, class: 'form-control gl-form-input'
-  %p.form-text.gl-text-subtle= _('URL must be percent-encoded if necessary.')
 .form-group
   = form.label :name, s_('Webhooks|Name (optional)'), class: 'label-bold'
   = form.text_field :name, value: hook.name, class: 'form-control gl-form-input gl-form-input-xl'
 .form-group
   = form.label :description, s_('Webhooks|Description (optional)'), class: 'label-bold'
   = form.text_area :description, value: hook.description, class: 'form-control gl-form-input gl-form-input-xl', rows: 4, maxlength: 2048
+.form-group
+  = form.label :url, _('URL'), class: 'label-bold'
+  = form.text_field :url, class: 'form-control gl-form-input'
+  %p.form-text.gl-text-subtle= _('URL must be percent-encoded if necessary.')
 .form-group
   = form.label :token, _('Secret token'), class: 'label-bold'
   = form.password_field :token, value: hook.masked_token, autocomplete: 'new-password', class: 'form-control gl-form-input gl-max-w-48'

app/views/shared/web_hooks/_form.html.haml

@@ -1,12 +1,6 @@
 = form_errors(hook)
 
 .js-vue-webhook-form{ data: webhook_form_data(hook) }
-.form-group
-  = form.label :name, s_('Webhooks|Name (optional)'), class: 'label-bold'
-  = form.text_field :name, value: hook.name, class: 'form-control gl-form-input gl-form-input-xl'
-.form-group
-  = form.label :description, s_('Webhooks|Description (optional)'), class: 'label-bold'
-  = form.text_area :description, value: hook.description, class: 'form-control gl-form-input gl-form-input-xl', rows: 4, maxlength: 2048
 .form-group
   = form.label :token, s_('Webhooks|Secret token'), class: 'label-bold'
   = form.password_field :token, value: hook.masked_token, autocomplete: 'new-password', class: 'form-control gl-form-input gl-form-input-xl'

config/routes/dashboard.rb

@@ -13,7 +13,11 @@
     resources :milestones, only: [:index]
     resources :labels, only: [:index]
 
-    resources :groups, only: [:index]
+    resources :groups, only: [:index] do
+      collection do
+        get :member, :inactive, to: 'groups#index'
+      end
+    end
     resources :snippets, only: [:index]
 
     resources :todos, only: [:index, :destroy]

db/docs/batched_background_migrations/backfill_organization_id_on_fork_networks.yml

@@ -5,4 +5,4 @@ feature_category: source_code_management
 introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/184051
 milestone: '17.11'
 queued_migration_version: 20250310143706
-finalized_by: # version of the migration that finalized this BBM
+finalized_by: 20250415100300

db/post_migrate/20250415100300_finalize_backfill_organization_id_on_fork_networks.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+class FinalizeBackfillOrganizationIdOnForkNetworks < Gitlab::Database::Migration[2.2]
+  milestone '18.0'
+  disable_ddl_transaction!
+
+  restrict_gitlab_migration gitlab_schema: :gitlab_main
+
+  def up
+    ensure_batched_background_migration_is_finished(
+      job_class_name: 'BackfillOrganizationIdOnForkNetworks',
+      table_name: :fork_networks,
+      column_name: :id,
+      job_arguments: [],
+      finalize: true
+    )
+  end
+
+  def down; end
+end

db/post_migrate/20250415130342_finalize_terraform_module_metadata_semver_patch_bigint_conversion.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+class FinalizeTerraformModuleMetadataSemverPatchBigintConversion < Gitlab::Database::Migration[2.2]
+  disable_ddl_transaction!
+  milestone '18.0'
+
+  restrict_gitlab_migration gitlab_schema: :gitlab_main
+
+  def up
+    ensure_backfill_conversion_of_integer_to_bigint_is_finished(
+      :packages_terraform_module_metadata,
+      %i[semver_patch],
+      primary_key: :package_id
+    )
+  end
+
+  def down; end
+end

db/post_migrate/20250415155913_swap_terraform_module_metadata_semver_patch_bigint_conversion.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+class SwapTerraformModuleMetadataSemverPatchBigintConversion < Gitlab::Database::Migration[2.2]
+  include Gitlab::Database::MigrationHelpers::Swapping
+
+  disable_ddl_transaction!
+  milestone '18.0'
+
+  TABLE_NAME = :packages_terraform_module_metadata
+  COLUMN_NAME = :semver_patch
+  BIGINT_COLUMN_NAME = :semver_patch_convert_to_bigint
+  TRIGGER_NAME = :trigger_dd7cb7bd6c9e
+
+  def up
+    swap
+  end
+
+  def down
+    swap
+  end
+
+  private
+
+  def swap
+    with_lock_retries(raise_on_exhaustion: true) do
+      lock_tables(TABLE_NAME)
+      swap_columns(TABLE_NAME, COLUMN_NAME, BIGINT_COLUMN_NAME)
+      reset_trigger_function(TRIGGER_NAME)
+    end
+  end
+end

db/schema_migrations/20250415100300

@@ -0,0 +1 @@
+d0c3f38c39552dae2b9948174e29b47c6dd3202902924e2533a4553f35833c84

db/schema_migrations/20250415130342

@@ -0,0 +1 @@
+0e5e1a95cfb7992fb5fb3d51a12953b92889c4448e071a121f4f4a4cc7369178

db/schema_migrations/20250415155913

@@ -0,0 +1 @@
+26450b49009494fc34950ea6a75758a8eecbc262448154e75ebaacfca538020d

db/structure.sql

@@ -19770,9 +19770,9 @@ CREATE TABLE packages_terraform_module_metadata (
     fields jsonb NOT NULL,
     semver_major integer,
     semver_minor integer,
-    semver_patch integer,
+    semver_patch_convert_to_bigint integer,
     semver_prerelease text,
-    semver_patch_convert_to_bigint bigint,
+    semver_patch bigint,
     CONSTRAINT check_46aa6c883a CHECK ((char_length(semver_prerelease) <= 255)),
     CONSTRAINT chk_rails_49f7b485ae CHECK ((char_length((fields)::text) <= 10485760))
 );

doc/api/openapi/openapi_v2.yaml

@@ -50208,7 +50208,16 @@ definitions:
         example: '2017-05-19T13:40:17.727Z'
       owner:
         "$ref": "#/definitions/API_Entities_UserBasic"
+      inputs:
+        "$ref": "#/definitions/API_Entities_Ci_Input"
     description: API_Entities_Ci_PipelineSchedule model
+  API_Entities_Ci_Input:
+    type: object
+    properties:
+      name:
+        type: string
+      value:
+        type: string
   API_Entities_Ci_PipelineScheduleDetails:
     type: object
     properties:
@@ -50245,6 +50254,8 @@ definitions:
         example: '2017-05-19T13:40:17.727Z'
       owner:
         "$ref": "#/definitions/API_Entities_UserBasic"
+      inputs:
+        "$ref": "#/definitions/API_Entities_Ci_Input"
       last_pipeline:
         "$ref": "#/definitions/API_Entities_Ci_PipelineBasic"
       variables:
@@ -50275,6 +50286,30 @@ definitions:
         description: The activation of pipeline schedule
         default: true
         example: true
+      inputs:
+        type: array
+        description: Inputs for the pipeline schedule
+        example:
+        - name: array_input
+          value:
+          - 1
+          - 2
+        - name: boolean_input
+          value: true
+        items:
+          type: object
+          properties:
+            name:
+              type: string
+              description: The name of the input
+              example: deploy_strategy
+            value:
+              type: string
+              description: The value of the input
+              example: blue-green
+          required:
+          - name
+          - value
     required:
     - description
     - ref
@@ -50303,6 +50338,30 @@ definitions:
         type: boolean
         description: The activation of pipeline schedule
         example: true
+      inputs:
+        type: array
+        description: Inputs for the pipeline schedule
+        example:
+        - name: deploy_strategy
+          value: blue-green
+        items:
+          type: object
+          properties:
+            name:
+              type: string
+              description: The name of the input
+              example: deploy_strategy
+            destroy:
+              type: boolean
+              description: Whether to delete the input
+              default: false
+            value:
+              type: string
+              description: The value of the input
+              example: blue-green
+          required:
+          - name
+          - value
     description: Edit a pipeline schedule
   postApiV4ProjectsIdPipelineSchedulesPipelineScheduleIdVariables:
     type: object