Add latest changes from gitlab-org/gitlab@master

GITLAB_KAS_VERSION

@@ -1 +1 @@
-13.6.1
+13.7.0

app/models/user.rb

@@ -591,13 +591,7 @@ class User < ApplicationRecord
 
       sanitized_order_sql = Arel.sql(sanitize_sql_array([order, query: query]))
 
-      search_query = if Feature.enabled?(:user_search_secondary_email)
-                       search_with_secondary_emails(query)
-                     else
-                       search_without_secondary_emails(query)
-                     end
-
-      search_query.reorder(sanitized_order_sql, :name)
+      search_with_secondary_emails(query).reorder(sanitized_order_sql, :name)
     end
 
     # Limits the result set to users _not_ in the given query/list of IDs.

changelogs/unreleased/ash2k-kas-13-7-0.yml

@@ -0,0 +1,5 @@
+---
+title: Update gitlab-kas to v13.7.0
+merge_request: 49318
+author:
+type: changed

changelogs/unreleased/cat-remove-user-secondary-email-ff.yml

@@ -0,0 +1,5 @@
+---
+title: Remove user_search_secondary_email feature flag
+merge_request: 49312
+author:
+type: changed

config/feature_flags/development/user_search_secondary_email.yml

@@ -1,8 +0,0 @@
----
-name: user_search_secondary_email
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/47587
-rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/282137
-milestone: '13.7'
-type: development
-group: group::access
-default_enabled: false

qa/qa/specs/features/browser_ui/1_manage/project/create_project_spec.rb

@@ -3,7 +3,9 @@
 module QA
   RSpec.describe 'Manage', :smoke do
     describe 'Project creation' do
-      it 'user creates a new project', testcase: 'https://gitlab.com/gitlab-org/quality/testcases/-/issues/429' do
+      it 'user creates a new project',
+         testcase: 'https://gitlab.com/gitlab-org/quality/testcases/-/issues/429',
+         quarantine: { issue: 'https://gitlab.com/gitlab-org/gitlab/-/issues/283925', type: :investigating } do
         Flow::Login.sign_in
 
         created_project = Resource::Project.fabricate_via_browser_ui! do |project|

spec/models/user_spec.rb

@@ -2078,32 +2078,16 @@ RSpec.describe User do
     end
 
     describe 'secondary email matching' do
-      context 'feature flag :user_search_secondary_email is enabled' do
-        it 'returns users with a matching secondary email' do
-          expect(described_class.search(email.email)).to include(email.user)
-        end
-
-        it 'does not return users with a matching part of secondary email' do
-          expect(described_class.search(email.email[1...-1])).to be_empty
-        end
-
-        it 'returns users with a matching secondary email regardless of the casing' do
-          expect(described_class.search(email.email.upcase)).to include(email.user)
-        end
+      it 'returns users with a matching secondary email' do
+        expect(described_class.search(email.email)).to include(email.user)
       end
 
-      context 'feature flag :user_search_secondary_email is disabled' do
-        before do
-          stub_feature_flags(user_search_secondary_email: false)
-        end
+      it 'does not return users with a matching part of secondary email' do
+        expect(described_class.search(email.email[1...-1])).to be_empty
+      end
 
-        it 'does not return users with a matching secondary email' do
-          expect(described_class.search(email.email)).not_to include(email.user)
-        end
-
-        it 'does not return users with a matching part of secondary email' do
-          expect(described_class.search(email.email[1...-1])).to be_empty
-        end
+      it 'returns users with a matching secondary email regardless of the casing' do
+        expect(described_class.search(email.email.upcase)).to include(email.user)
       end
     end