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

fix(deployment): suspend loculus-revoke-and-regroup-cronjob #2862

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

anna-parker
Copy link
Contributor

@anna-parker anna-parker commented Sep 23, 2024

resolves #

preview URL: https://stop-revoke-cronjob.loculus.org/

Summary

Although loculus-revoke-and-regroup-cronjob was scheduled to a non-existent time it was still being started by argocd, this caused concurrent ingest and duplication of ingested sequences.

Edit from Theo:
I suspect it was being started by kubernetes, not argocd

Screenshot

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by an appropriate test.

@anna-parker anna-parker added the preview Triggers a deployment to argocd label Sep 23, 2024
@theosanderson
Copy link
Member

lgtm

kubectl get pods -n prev-stop-revoke-cronjob
NAME                                                              READY   STATUS    RESTARTS        AGE
loculus-database-689c7969cc-25dcd                                 1/1     Running   0               2m42s
loculus-keycloak-database-8646dcddcc-7lhz2                        1/1     Running   0               2m42s
loculus-docs-b7877c6d7-qxhsm                                      1/1     Running   0               2m42s
loculus-ena-submission-79ff7d7f6f-z7f7d                           1/1     Running   0               2m42s
loculus-backend-6597557654-ksx9k                                  1/1     Running   0               2m41s
loculus-keycloak-5bfd94466d-thxfb                                 1/1     Running   0               2m41s
loculus-website-76bc8d84dc-qxz26                                  1/1     Running   0               2m40s
loculus-preprocessing-west-nile-v1-0-99b556b6d-2wxht              1/1     Running   3 (2m11s ago)   2m42s
loculus-lapis-silo-not-aligned-organism-58cc89bcb-hrzmh           3/3     Running   0               2m42s
loculus-lapis-silo-cchf-6c5dcd69d9-w7wzh                          3/3     Running   0               2m42s
loculus-lapis-silo-west-nile-6b464c485-r2lvn                      3/3     Running   0               2m42s
loculus-lapis-silo-dummy-organism-74788d7d8d-cr46l                3/3     Running   0               2m42s
loculus-lapis-silo-ebola-zaire-78b4cdb6b-txgw9                    3/3     Running   0               2m41s
loculus-preprocessing-not-aligned-organism-v1-0-689db99fd-pqgvd   1/1     Running   3 (2m6s ago)    2m42s
loculus-preprocessing-cchf-v1-0-7b46f79d58-wrcjq                  1/1     Running   3 (2m8s ago)    2m42s
loculus-preprocessing-ebola-zaire-v1-0-55fc6c9694-crp6x           1/1     Running   3 (2m8s ago)    2m40s
loculus-preprocessing-dummy-organism-v1-0-659f5947cb-bdbmw        1/1     Running   3 (114s ago)    2m41s
loculus-ingest-deployment-west-nile-644b78bdb6-52ssv              1/1     Running   1 (79s ago)     2m42s
loculus-ingest-deployment-ebola-zaire-5997cc75fb-27ft7            1/1     Running   2 (62s ago)     2m42s
loculus-ingest-deployment-cchf-787ddfdf5d-8s4gw                   1/1     Running   1 (37s ago)     2m41s

@anna-parker anna-parker marked this pull request as ready for review September 23, 2024 15:09
@anna-parker anna-parker changed the title suspend loculus-revoke-and-regroup-cronjob fix(ci): suspend loculus-revoke-and-regroup-cronjob Sep 23, 2024
@theosanderson theosanderson changed the title fix(ci): suspend loculus-revoke-and-regroup-cronjob fix(deployment): suspend loculus-revoke-and-regroup-cronjob Sep 23, 2024
@theosanderson
Copy link
Member

(re name change: I had understood that we typically use CI to mean GitHub actions stuff and deployment to mean kubernetes)

@anna-parker anna-parker merged commit 20bf761 into main Sep 23, 2024
15 checks passed
@anna-parker anna-parker deleted the stop_revoke_cronjob branch September 23, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Triggers a deployment to argocd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants