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

migrate storage from premium to standard for jenkins-infra, jenkins-weekly and jenkins-release #4044

Open
smerle33 opened this issue Apr 16, 2024 · 5 comments

Comments

@smerle33
Copy link
Contributor

smerle33 commented Apr 16, 2024

Service(s)

infra.ci.jenkins.io, release.ci.jenkins.io, weekly.ci.jenkins.io

Summary

a checked with the metrics, standard ZRS hdd will be enough to handle the workload for those 3 controllers.
lets try to save some money.

this will be the occasion to handle the volumes/disk with terraform and remove the Datasource annotation from the helmchart values for controllers.

We will need to create a new Storage Class (on publick8s and privatek8s).

Sidenote: we will have to handle the boostrap permissions from the terraform managed volumes.

Reproduction steps

No response

@smerle33
Copy link
Contributor Author

smerle33 commented Apr 16, 2024

WIP (infra and release)

Capture d’écran 2024-04-11 à 10 56 00 Capture d’écran 2024-04-11 à 10 58 07 Capture d’écran 2024-04-15 à 15 43 19 Capture d’écran 2024-04-15 à 15 44 39 Capture d’écran 2024-04-11 à 10 55 22 Capture d’écran 2024-04-11 à 10 55 32 Capture d’écran 2024-04-11 à 10 55 48

@dduportal dduportal added this to the infra-team-sync-2024-04-23 milestone Apr 16, 2024
@dduportal dduportal removed the triage Incoming issues that need review label Apr 16, 2024
@smerle33
Copy link
Contributor Author

smerle33 commented Apr 19, 2024

WEEKLY.CI first

when time ok:

  • take a new snapshot for security (remove the one done above)
  • disable kubernetes-management builds
  • remove the weekly statefulset
  • on the temporary pod re-run the rsync (should be fast)
  • change the helm values for weekly to use the new PVC : persistence.existingClaim to PVC_NAME
  • enable kubernetes-management builds
  • merge the PR (changing the chart)
  • check that the pod start with the correct PV/PVC
  • remove the OLD PV/PVC/DISK (keep the snapshot for a few days)
  • remove the snapshot

if all went well, redo for infra.ci/release.ci

smerle33 added a commit to jenkins-infra/azure that referenced this issue Apr 23, 2024
as per jenkins-infra/helpdesk#4044

add a new storage class for ZRS in standard mode to be used for
weekly.ci on the `publick8s` and infra.ci/release.ci on the `privatek8s`

---------

Co-authored-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal
Copy link
Contributor

Update:

@dduportal
Copy link
Contributor

Update: on hold until after the 15th of May 2024

@smerle33
Copy link
Contributor Author

The aim is to be able to change the disk type without recreating everything next time. We choose to create PV/PVC/Disk from terraform instead of just the PVC for that (not following only creating the PVC should be enough: kubernetes.io/docs/concepts/storage/persistent-volumes/#dynamic)
The disk size can be changed on both scenarii.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants