chore: removed drone (#844)

Co-authored-by: ElderMatt <[email protected]>
Co-authored-by: svcAPLBot <[email protected]>

Author: 3 people

src/api.authz.test.ts

@@ -79,7 +79,6 @@ describe('API authz tests', () => {
         .put('/v1/settings/alerts')
         .send({
           alerts: {
-            drone: ['msteams'],
             groupInterval: '5m',
             msteams: { highPrio: 'bla', lowPrio: 'bla' },
             receivers: ['slack'],

src/app.ts

@@ -24,7 +24,6 @@ import { extract, getPaths, getValuesSchema } from 'src/utils'
 import {
   CHECK_LATEST_COMMIT_INTERVAL,
   cleanEnv,
-  DRONE_WEBHOOK_SECRET,
   EXPRESS_PAYLOAD_LIMIT,
   GIT_PASSWORD,
   GIT_PUSH_RETRIES,
@@ -35,7 +34,6 @@ import giteaCheckLatest from './gitea/connect'
 import { getBuildStatus, getSealedSecretStatus, getServiceStatus, getWorkloadStatus } from './k8s_operations'
 
 const env = cleanEnv({
-  DRONE_WEBHOOK_SECRET,
   CHECK_LATEST_COMMIT_INTERVAL,
   GIT_USER,
   GIT_PASSWORD,

src/openapi/app.yaml

@@ -8,7 +8,6 @@ AppList:
     - argocd
     - cert-manager
     - cnpg
-    - drone
     - external-dns
     - gitea
     - grafana

src/openapi/definitions.yaml

@@ -533,19 +533,6 @@ domainSuffix:
   $ref: '#/domain'
   title: Domain suffix
   description: The domain suffix for the cluster.
-droneGit:
-  properties:
-    clientID:
-      $ref: '#/wordCharacterPattern'
-    clientSecretValue:
-      type: string
-      x-acl:
-        platformAdmin: [read-any, update-any]
-        teamAdmin: []
-        teamMember: []
-      x-secret: true
-  type: object
-  required: [clientID, clientSecretValue]
 duration:
   description: 'Prometheus duration (See: https://prometheus.io/docs/prometheus/latest/configuration/configuration/#configuration-file)'
   pattern: '((([0-9]+)y)?(([0-9]+)w)?(([0-9]+)d)?(([0-9]+)h)?(([0-9]+)m)?(([0-9]+)s)?(([0-9]+)ms)?|0)'

src/otomi-stack.test.ts

@@ -366,7 +366,7 @@ describe('Workload values', () => {
   })
 
   test('returns filtered apps if App array is submitted isPreinstalled flag is true', () => {
-    const apps: App[] = [{ id: 'external-dns' }, { id: 'drone' }, { id: 'cnpg' }, { id: 'loki' }]
+    const apps: App[] = [{ id: 'external-dns' }, { id: 'cnpg' }, { id: 'loki' }]
     jest.spyOn(otomiStack, 'getSettingsInfo').mockReturnValue({ otomi: { isPreInstalled: true } })
     const filteredApps = otomiStack.filterExcludedApp(apps)
     expect(filteredApps).toEqual([{ id: 'cnpg' }, { id: 'loki' }])

src/validators.ts

@@ -22,7 +22,6 @@ export const VERSIONS = json({
   },
 })
 export const CUSTOM_ROOT_CA = str({ desc: 'The root CA used for certs', default: undefined })
-export const DRONE_WEBHOOK_SECRET = str({ desc: 'The drone secret to validate incoming webhooks', default: undefined })
 export const EDITOR_INACTIVITY_TIMEOUT = num({
   desc: 'Inactivity timeout in days after which editor session is removed to clean mem',
   default: 1,
@@ -62,7 +61,7 @@ export const TOOLS_HOST = str({ desc: 'The host of the tools server', default: '
 export const PREINSTALLED_EXCLUDED_APPS = json({
   desc: 'Applications that are managed by Linode, so they should be excluded from the apps page',
   default: {
-    apps: ['cert-manager', 'minio', 'velero', 'drone', 'external-dns'],
+    apps: ['cert-manager', 'minio', 'velero', 'external-dns'],
   },
 })
 export const HIDDEN_APPS = json({

test/core.yaml

@@ -13,10 +13,6 @@ k8s:
     - name: default
       disableIstioInjection: true
       disablePolicyChecks: true
-    - name: drone
-    - name: drone-pipelines
-      disableIstioInjection: true
-      disablePolicyChecks: true
     - name: external-dns
       disableIstioInjection: true
     - name: harbor
@@ -33,13 +29,13 @@ k8s:
     - name: istio-system
       disableIstioInjection: true
       labels:
-        "apl.io/ingress-controller-scope": "true"
+        'apl.io/ingress-controller-scope': 'true'
     - name: ingress
       # disabling istio sidecar as it does not preserve client ip (yet)
       # TODO: enable once it does
       disableIstioInjection: true
       labels:
-        "apl.io/ingress-controller-scope": "true"
+        'apl.io/ingress-controller-scope': 'true'
     - name: jaeger
       app: jaeger
     - name: jaeger-operator
@@ -141,24 +137,6 @@ adminApps:
     tags: [ingress, security, tls]
   - name: cnpg
     tags: [database]
-  - name: drone
-    tags: [cicd, deployment, pipeline]
-    isShared: true
-    ownHost: true
-    ingress:
-      - svc: drone
-        namespace: drone
-        type: public
-        auth: true
-        removeRequestHeaders:
-          - authorization
-      - svc: drone
-        namespace: drone
-        type: public
-        paths: [/hook, /api/user, /api/repo]
-        forwardPath: true
-        removeRequestHeaders:
-          - authorization
   - name: tty
     tags: [tty]
     isShared: true

test/env/apps/drone.yaml

@@ -1,6 +1,4 @@
 alerts:
-    drone:
-        - slack
     groupInterval: 5m
     receivers:
         - slack