Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CCAP-304: Add support for background jobs #9

Merged
merged 15 commits into from
Aug 28, 2024
Merged

Conversation

jamesiarmes
Copy link
Member

@jamesiarmes jamesiarmes commented Aug 20, 2024

🔗 Jira ticket

CCAP-304

✍️ Description

Implements delayed_job with a single recurring job that deactivates expired authentication keys.

📷 Design reference

Tech spec

🧪 Testing instructions

  • Build the docker image by running docker compose build
  • Launch (or update) the service by running docker compose up -d
  • Run docker compose logs worker to verify that the worker is running
  • Run docker compose exec worker bundle exec rake jobs:queue and verify it returns stats from the worker

✅ Completion tasks

  • Added relevant tests
  • Meets acceptance criteria

@jamesiarmes jamesiarmes requested review from a team as code owners August 20, 2024 20:33
Copy link

github-actions bot commented Aug 20, 2024

🔍 Vulnerabilities of pr-9:latest

📦 Image Reference pr-9:latest
digestsha256:df9160edc78bfa2e39f3a1cbb9ada2b8c99c15f022c3268144997034028bd047
vulnerabilitiescritical: 0 high: 0 medium: 0 low: 0
size516 MB
packages736
📦 Base Image ruby:3
also known as
  • 3-bookworm
  • 3.3
  • 3.3-bookworm
  • 3.3.4
  • 3.3.4-bookworm
  • bookworm
  • latest
digestsha256:6b49252a37a7664623dee0a132ea0b3a936c100ed6d9520318f4823302c396af
vulnerabilitiescritical: 2 high: 3 medium: 7 low: 106 unspecified: 3

Copy link

@cram-cfa cram-cfa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

from what i understand of this, it looks good to me!

@jamesiarmes jamesiarmes merged commit c7cb3ff into main Aug 28, 2024
6 checks passed
@jamesiarmes jamesiarmes deleted the background-jobs branch August 28, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants