FTR - optimize service initialization (#212421)

## Summary

This PR optimizes the FTR service initialization by not loading UI
service for API tests and by removing retries during test user setup

## Changes

- Remove loading of common UI services from common services (UI services
should not be loaded for API tests)
- Move `security` service from `@kbn/ftr-common-functional-ui-services`
to `@kbn/ftr-common-functional-services` as it should be available to
API tests as well
- Only try once to delete `testUser` during init (this user usually does
not exist on a fresh deployment - and if it does, a single delete
request is enough to get rid of it)

## Benchmark results

**These changes will reduce FTR CI runtime overall by ~100 minutes**
:rocket:
Due to parallel workers in CI, the effective runtime of the whole CI job
will be less than that.

- The removal of UI service loading (which includes starting a browser
instance) for API tests reduces init time by ~0.5 seconds. With 313 API
configs that are started on CI, this reduces the runtime overall by ~156
seconds / ~2.6 minutes.
- The removal of test user delete retries reduces init time by ~10
seconds. With 589 FTR configs that are started on CI, this reduces the
runtime overall by ~5890 seconds / ~98 minutes.
- These numbers have been taken on a local machine and since CI workers
are usually slower, we should see at least this amount of improvement if
not more in CI.

Author: pheyos

src/platform/packages/shared/kbn-ftr-common-functional-services/index.ts

@@ -46,3 +46,4 @@ export { IndexPatternsService } from './services/index_patterns';
 export { RandomnessService } from './services/randomness';
 export { KibanaSupertestProvider, ElasticsearchSupertestProvider } from './services/supertest';
 export { retryForSuccess } from './services/retry/retry_for_success';
+export { SecurityService } from './services/security';

src/platform/packages/shared/kbn-ftr-common-functional-services/services/all.ts

@@ -21,6 +21,7 @@ import { RandomnessService } from './randomness';
 import { SupertestWithoutAuthProvider } from './supertest_without_auth';
 import { SamlAuthProvider } from './saml_auth';
 import { KibanaSupertestProvider, ElasticsearchSupertestProvider } from './supertest';
+import { SecurityServiceProvider } from './security';
 
 export const services = {
   es: EsProvider,
@@ -38,4 +39,5 @@ export const services = {
   supertest: KibanaSupertestProvider,
   esSupertest: ElasticsearchSupertestProvider,
   supertestWithoutAuth: SupertestWithoutAuthProvider,
+  security: SecurityServiceProvider,
 };

src/platform/packages/shared/kbn-ftr-common-functional-ui-services/services/security/index.ts renamed to src/platform/packages/shared/kbn-ftr-common-functional-services/services/security/index.ts

@@ -10,8 +10,6 @@
 import { format as formatUrl } from 'url';
 import supertest from 'supertest';
 
-import type { Browser } from '../browser';
-import type { TestSubjects } from '../test_subjects';
 import { Role } from './role';
 import { User } from './user';
 import { FtrService, FtrProviderContext } from '../ftr_provider_context';
@@ -23,15 +21,13 @@ export class TestUser extends FtrService {
   private readonly config = this.ctx.getService('config');
   private readonly log = this.ctx.getService('log');
 
-  private readonly browser: Browser | void =
+  private readonly browser =
     // browser service is not normally available in common.
-    this.ctx.hasService('browser') ? (this.ctx.getService('browser' as any) as Browser) : undefined;
+    this.ctx.hasService('browser') ? this.ctx.getService('browser' as any) : undefined;
 
-  private readonly testSubjects: TestSubjects | undefined =
+  private readonly testSubjects =
     // testSubject service is not normally available in common.
-    this.ctx.hasService('testSubjects')
-      ? (this.ctx.getService('testSubjects' as any) as TestSubjects)
-      : undefined;
+    this.ctx.hasService('testSubjects') ? this.ctx.getService('testSubjects' as any) : undefined;
 
   constructor(
     ctx: FtrProviderContext,
@@ -129,7 +125,7 @@ export async function createTestUserService(ctx: FtrProviderContext, role: Role,
 
     // delete the test_user if present (will it error if the user doesn't exist?)
     try {
-      await user.delete(TEST_USER_NAME);
+      await user.delete(TEST_USER_NAME, { retries: 1 });
     } catch (exception) {
       log.debug('no test user to delete');
     }

src/platform/packages/shared/kbn-ftr-common-functional-ui-services/services/security/role.ts renamed to src/platform/packages/shared/kbn-ftr-common-functional-services/services/security/role.ts

@@ -32,11 +32,12 @@ export class User {
     this.log.debug(`created user ${username}`);
   }
 
-  public async delete(username: string) {
+  public async delete(username: string, options?: { retries?: number }) {
     this.log.debug(`deleting user ${username}`);
     const { data, status, statusText } = await this.kbnClient.request({
       path: `/internal/security/users/${username}`,
       method: 'DELETE',
+      retries: options?.retries,
     });
     if (status !== 204) {
       throw new Error(

src/platform/packages/shared/kbn-ftr-common-functional-ui-services/services/security/role_mappings.ts renamed to src/platform/packages/shared/kbn-ftr-common-functional-services/services/security/role_mappings.ts

@@ -23,4 +23,3 @@ export {
 } from './services/remote/network_profiles';
 export type { TimeoutOpt } from './types';
 export { TestSubjects } from './services/test_subjects';
-export { SecurityService } from './services/security';