diff --git a/README.md b/README.md index fdf036e..2d025c4 100644 --- a/README.md +++ b/README.md @@ -51,10 +51,10 @@ _While installation is possible using non-Linux clients, it's not a well-establi ### Provision with Terraform -Set the `GOOGLE_PROJECT`, `ISIDRO_DOMAIN`, `MATTERMOST_DOMAIN`, and `DNS_ZONE_NAME` environment variables, with something like: +Set the `GOOGLE_PROJECT`, `API_DOMAIN`, `MATTERMOST_DOMAIN`, and `DNS_ZONE_NAME` environment variables, with something like: ```bash export GOOGLE_PROJECT=example -export ISIDRO_DOMAIN=isidro.example.com +export API_DOMAIN=api.example.com export MATTERMOST_DOMAIN=mattermost.example.com export DNS_ZONE_NAME="example-com" ``` @@ -141,7 +141,7 @@ Hydrate configurations: cp skaffold.dev.yaml skaffold.yaml sed -i "s/GOOGLE_PROJECT/$GOOGLE_PROJECT/g" skaffold.yaml sed -i "s/MATTERMOST_DOMAIN/$MATTERMOST_DOMAIN/g" skaffold.yaml -sed -i "s/ISIDRO_DOMAIN/$ISIDRO_DOMAIN/g" skaffold.yaml +sed -i "s/API_DOMAIN/$API_DOMAIN/g" skaffold.yaml sed -i "s/DNS_ZONE_NAME/$DNS_ZONE_NAME/g" skaffold.yaml ``` @@ -157,7 +157,7 @@ Hydrate confiigurations: cp skaffold.prod.yaml skaffold.yaml sed -i "s/GOOGLE_PROJECT/$GOOGLE_PROJECT/g" skaffold.yaml sed -i "s/MATTERMOST_DOMAIN/$MATTERMOST_DOMAIN/g" skaffold.yaml -sed -i "s/ISIDRO_DOMAIN/$ISIDRO_DOMAIN/g" skaffold.yaml +sed -i "s/API_DOMAIN/$API_DOMAIN/g" skaffold.yaml sed -i "s/DNS_ZONE_NAME/$DNS_ZONE_NAME/g" skaffold.yaml ``` @@ -203,7 +203,7 @@ _Relevant if you are using Mattermost as your chat tool_ 1. Recommended description is "Push notification enabling the Isidro chatbot to respond to @mentions" 1. Application type is "application/json" 1. Trigger word is "@isidro" - 1. Callback URL is https://isidro.example.com/isidro/api/v1/submit (replace isidro.example.com with your Isidro domain) + 1. Callback URL is https://api.example.com/v1/submit (replace api.example.com with your Isidro API domain) 1. Leave the remaining values as the defaults 1. Copy the verification token to the Helm values (or Skaffold overrides) as `mattermost.verificationToken` 1. Upgrade the Helm installation @@ -219,7 +219,7 @@ Mention @isidro in Slack messages, and get a response. Use separate message thr ### Test payload ```bash -curl -X POST https://isidro.example.com/api/v1/submit \ +curl -X POST https://api.example.com/v1/submit \ -H "Content-Type: application/json" \ -d '{"token": "1234567890", "event": {"channel": "quality", "ts": "1234567890", "user": "me", "text": "Hello"}}' ``` \ No newline at end of file diff --git a/auxiliary/multicluster/mci.yaml b/auxiliary/multicluster/mci.yaml index 29a6147..01fd30b 100644 --- a/auxiliary/multicluster/mci.yaml +++ b/auxiliary/multicluster/mci.yaml @@ -5,7 +5,7 @@ metadata: name: isidro-ingress namespace: isidro annotations: - networking.gke.io/static-ip: global/addresses/isidro + networking.gke.io/static-ip: global/addresses/isidro-api spec: template: spec: @@ -23,7 +23,7 @@ metadata: name: mattermost-ingress namespace: mattermost annotations: - networking.gke.io/static-ip: global/addresses/mattermost + networking.gke.io/static-ip: global/addresses/isidro-mattermost spec: template: spec: diff --git a/auxiliary/networking/templates/certs.yaml b/auxiliary/networking/templates/certs.yaml index df43b8e..9256d81 100644 --- a/auxiliary/networking/templates/certs.yaml +++ b/auxiliary/networking/templates/certs.yaml @@ -16,7 +16,7 @@ spec: apiVersion: cert-manager.io/v1 kind: Certificate metadata: - name: isidro + name: api namespace: isidro spec: secretName: ingress-tls @@ -24,7 +24,7 @@ spec: kind: ClusterIssuer name: cloud-dns-issuer dnsNames: - - {{ .Values.isidro.domain }} + - {{ .Values.api.domain }} --- apiVersion: cert-manager.io/v1 kind: Certificate diff --git a/auxiliary/networking/templates/dns.yaml b/auxiliary/networking/templates/dns.yaml index cb7b532..9cef6d3 100644 --- a/auxiliary/networking/templates/dns.yaml +++ b/auxiliary/networking/templates/dns.yaml @@ -1,26 +1,26 @@ apiVersion: dns.cnrm.cloud.google.com/v1beta1 kind: DNSRecordSet metadata: - name: isidro + name: api namespace: configconnector-resources spec: - name: "{{ .Values.isidro.domain }}." + name: "{{ .Values.api.domain }}." type: A ttl: 300 managedZoneRef: external: "{{ .Values.dns.zone }}" rrdatasRefs: - - name: isidro + - name: api kind: ComputeAddress --- apiVersion: compute.cnrm.cloud.google.com/v1beta1 kind: ComputeAddress metadata: - name: isidro + name: api namespace: configconnector-resources spec: location: global - resourceID: isidro + resourceID: isidro-api --- apiVersion: dns.cnrm.cloud.google.com/v1beta1 kind: DNSRecordSet @@ -44,4 +44,4 @@ metadata: namespace: configconnector-resources spec: location: global - resourceID: mattermost \ No newline at end of file + resourceID: isidro-mattermost \ No newline at end of file diff --git a/auxiliary/networking/values.yaml b/auxiliary/networking/values.yaml index 823f827..8610332 100644 --- a/auxiliary/networking/values.yaml +++ b/auxiliary/networking/values.yaml @@ -1,7 +1,7 @@ project: PROJECT -isidro: - domain: isidro.example.com +api: + domain: api.example.com mattermost: domain: mattermost.example.com diff --git a/gatekeeper/main.py b/gatekeeper/main.py index 1f5a33d..18d6e64 100644 --- a/gatekeeper/main.py +++ b/gatekeeper/main.py @@ -139,7 +139,7 @@ def general_response(self): return "" -@app.route("/isidro/api/v1/submit", methods=["POST"]) +@app.route("/v1/submit", methods=["POST"]) def submission(): gatekeeper = Gatekeeper(request) gatekeeper.validate_verification_token_correct() diff --git a/skaffold.dev.yaml b/skaffold.dev.yaml index ded8df9..c14a6e2 100644 --- a/skaffold.dev.yaml +++ b/skaffold.dev.yaml @@ -217,7 +217,7 @@ deploy: - name: isidro-configconnector chartPath: auxiliary/configconnector setValues: - configconnector.project: GOOGLE_PROJECT + project: GOOGLE_PROJECT hooks: after: # Ensure required CRDs are available @@ -241,7 +241,7 @@ deploy: skipBuildDependencies: true setValues: project: GOOGLE_PROJECT - isidro.domain: ISIDRO_DOMAIN + api.domain: API_DOMAIN mattermost.domain: MATTERMOST_DOMAIN dns.zone: DNS_ZONE_NAME --- diff --git a/skaffold.prod.yaml b/skaffold.prod.yaml index e71dff7..271f90d 100644 --- a/skaffold.prod.yaml +++ b/skaffold.prod.yaml @@ -361,7 +361,7 @@ deploy: - name: isidro-configconnector chartPath: auxiliary/configconnector setValues: - configconnector.project: GOOGLE_PROJECT + project: GOOGLE_PROJECT hooks: after: # Ensure required CRDs are available @@ -385,7 +385,7 @@ deploy: skipBuildDependencies: true setValues: project: GOOGLE_PROJECT - isidro.domain: ISIDRO_DOMAIN + api.domain: API_DOMAIN mattermost.domain: MATTERMOST_DOMAIN dns.zone: DNS_ZONE_NAME --- diff --git a/slack/manifest.yaml b/slack/manifest.yaml index 769a7a4..af4d311 100644 --- a/slack/manifest.yaml +++ b/slack/manifest.yaml @@ -17,7 +17,7 @@ oauth_config: - im:history settings: event_subscriptions: - request_url: https://isidro.example.com/isidro/api/v1/submit + request_url: https://api.example.com/v1/submit bot_events: - app_mention org_deploy_enabled: false