Kubernetes Config

[khalford]

Description:

Changes so we can also deploy with Kubernetes:

Kubernetes can't mount a configmap and secrets to the same directory. separating them so we can deploy with kubernetes.
Adds a deployment.yml to deploy onto a cluster

As the secrets location is changing this is a major change. All existing ChatOps instances will break and require re-deployment

---

Submitter:

Have you done the following?:

• Labeled the pull request from the following? major | minor | patch or documentation | workflow
• Updated the documentation and template config files?
• Added unit tests for new / untested code?
• Verified the relevant staging Docker image runs on dev-cloud-chatops.nubes.rl.ac.uk?

New / Existing features:

• Written integration tests?
• Configured the Slack Application with new commands or permission scope changes?
• Updated the version.txt and docker-compose.yml files?

---

Reviewer:

Have you checked the following?:

• Version change is appropriate to the code changes? Semantic versioning documentation here
• Unit test code coverage is acceptable?
• Do the CI jobs pass?

[codecov]

Codecov Report

Attention: Patch coverage is 96.77419% with 3 lines in your changes missing coverage. Please review.

Project coverage is 96.49%. Comparing base (54c30b3) to head (2a5a4d1).
Report is 12 commits behind head on master.

Files with missing lines	Patch %	Lines
cloud-chatops/src/read_data.py	88.88%	2 Missing ⚠️
cloud-chatops/src/features/pr_reminder.py	90.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
- Coverage   96.52%   96.49%   -0.04%     
==========================================
  Files          13       13              
  Lines         634      627       -7     
==========================================
- Hits          612      605       -7     
  Misses         22       22              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[khalford]

Ignore codecov failing as those lines don't have tests in the first place. Therefore the coverage diff is really 0%