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

quarry on k8s #31

Merged
merged 16 commits into from
Apr 1, 2024
Merged

quarry on k8s #31

merged 16 commits into from
Apr 1, 2024

Conversation

vivian-rook
Copy link
Contributor

https://quarry-test.wmcloud.org/ offers a running, and seemingly working, quarry on k8s. Which is running off of this PR.

I'm not sure if we want to include ansible or not. The original intention of it was to deploy volumes, though it would seem that we're not going to be able to deploy a ReadWriteMany volume that will function across nodes. So the utility of ansible is reduced in this case. Opinions on if it should be removed outright?

Bug: T349032
@siddharthvp
Copy link
Collaborator

I don't have any experience with Ansible or Terraform, so wouldn't be of much help in reviewing this.

@vivian-rook
Copy link
Contributor Author

I don't have any experience with Ansible or Terraform, so wouldn't be of much help in reviewing this.

No problem, I can deal with those bits fine. Though I could use some help in testing that everything seems to be working. In particular the error in https://phabricator.wikimedia.org/T349032 I propped up the web pod with a livenessProbe, but ideally it wouldn't be restarting a few times an hour.

@vivian-rook
Copy link
Contributor Author

Increasing the number of workers seemed to remove the restarting issue. Perhaps something required there be multiple threads? That seems odd, but I guess fits with the symptoms.

NAME                         READY   STATUS    RESTARTS   AGE
pod/redis-5bb5646ff8-pnwqf   1/1     Running   0          20d
pod/web-5b9f9bbd5c-8wkbb     1/1     Running   0          22h
pod/web-5b9f9bbd5c-c245k     1/1     Running   0          22h
pod/web-5b9f9bbd5c-mtcsj     1/1     Running   0          22h
pod/web-5b9f9bbd5c-pql7m     1/1     Running   0          22h
pod/web-5b9f9bbd5c-sfs25     1/1     Running   0          22h
pod/web-5b9f9bbd5c-wkcm9     1/1     Running   0          22h
pod/web-5b9f9bbd5c-xlwgz     1/1     Running   0          22h
pod/web-5b9f9bbd5c-xvm8p     1/1     Running   0          22h
pod/worker-8fc758576-62dhh   1/1     Running   0          20d
pod/worker-8fc758576-67f5g   1/1     Running   0          22h

@vivian-rook vivian-rook merged commit 5b2ff73 into main Apr 1, 2024
2 checks passed
@vivian-rook vivian-rook deleted the T349032 branch April 1, 2024 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants