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.

Sign up for GitHub

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

Jump to bottom

feat: add `updates-jenkins-io` service to `publick8s` cluster #4211

Merged

lemeurherve merged 22 commits into jenkins-infra:main from lemeurherve:helpdesk2649-uc-r2

Aug 11, 2023

Member

lemeurherve commented Aug 1, 2023 •

edited

Loading

This PR adds an experimental updates-jenkins-io service on publick8s cluster

Ref: jenkins-infra/helpdesk#2649 (comment)


          feat: experimental uc-r2 service

c262e7a

lemeurherve force-pushed the helpdesk2649-uc-r2 branch from 1f8b04e to c262e7a Compare

August 1, 2023 14:31

lemeurherve added 3 commits

August 1, 2023 16:32


          update PV name

85b2da6


          reduce PV size from 1000Gi to 1Gi

7c6997c


          comment out mirrorbits ingress path

4b7d715

dduportal reviewed

View reviewed changes

clusters/publick8s.yaml Outdated Show resolved Hide resolved

dduportal reviewed

View reviewed changes

clusters/publick8s.yaml Outdated Show resolved Hide resolved

dduportal reviewed

View reviewed changes

config/uc-r2.yaml Outdated Show resolved Hide resolved

dduportal reviewed

View reviewed changes

config/uc-r2.yaml Outdated Show resolved Hide resolved

dduportal reviewed

View reviewed changes

config/uc-r2.yaml Outdated Show resolved Hide resolved

dduportal reviewed

View reviewed changes

config/uc-r2.yaml Outdated Show resolved Hide resolved


          apply review suggestions

7f9cd53

lemeurherve changed the title ~~feat: experimental uc-r2 service~~ feat: experimental updates-jenkins-io service

lemeurherve and others added 4 commits

August 4, 2023 15:52


          remove specific release parameters from mirrorbits, will re-add them …

f35f408

…only if needed


          set PV name so we can reattach to an existing PV without redefining i…

d571428

…t here


          update values (follow up of jenkins-infra/azure#445) and use an azure…

990e71d

… disk SC


          Merge branch 'main' into helpdesk2649-uc-r2

d644238

Member Author

lemeurherve commented Aug 7, 2023

Needs jenkins-infra/helm-charts#580

lemeurherve mentioned this pull request

[INFRA-3100] Migrate updates.jenkins.io to another Cloud jenkins-infra/helpdesk#2649

Open


          bump mirrorbits chart version

754fd86

lemeurherve changed the title ~~feat: experimental updates-jenkins-io service~~ feat: add updates-jenkins-io service to publick8s cluster

lemeurherve added 4 commits

August 8, 2023 11:11


          fix secret name ref

a378b8d


          full name override to get rid of 'mirrorbits' in objet names

a6873e8


          don't override image chart values

6496b67


          adapt file share size to the one declared in jenkins-infra/azure

4daf373

lemeurherve mentioned this pull request

fix(updates.jenkins.io): allow non SSL connections to Redis jenkins-infra/azure#447

Merged

lemeurherve added a commit to jenkins-infra/azure that referenced this pull request


          fix(updates.jenkins.io): allow non SSL connections to Redis (#447)

7026eb3

Fix the following error when trying to deploy
jenkins-infra/kubernetes-management#4211:

> UTC Database is down: dial tcp 52.184.161.24:6379: i/o timeout

Manually tested via Azure Portal, current settings:
<img width="162" alt="image"
src="https://github.com/jenkins-infra/azure/assets/91831478/0b92bb88-3321-4d74-b6a9-d57c4190045b">

Settings with this change:
<img width="159" alt="image"
src="https://github.com/jenkins-infra/azure/assets/91831478/6ac15d3a-190c-4ce9-9d47-bc4e177be6ec">

Connection OK.


          Merge branch 'main' into helpdesk2649-uc-r2

eef2f8c

lemeurherve marked this pull request as ready for review

August 8, 2023 13:03

lemeurherve requested a review from a team

August 8, 2023 13:03

Member Author

lemeurherve commented Aug 8, 2023 •

edited

Loading

We'll have to wait a bit 😅

Failed to wait for order resource "updates-jenkins-io-tls-8k49h-1994758939" to become ready: order is in "errored" state: Failed to create Order: 429 urn:ietf:params:acme:error:rateLimited: Error creating new order :: too many certificates (5) already issued for this exact set of domains in the last 168 hours: azure.updates.jenkins.io, retry after 2023-08-09T00:55:28Z: see https://letsencrypt.org/docs/duplicate-certificate-limit/


          mount PV as readonly

15799e0

Member Author

lemeurherve commented Aug 8, 2023

Blocked by jenkins-infra/helm-charts#583

This was referenced Aug 9, 2023

chore(mirrorbits): add unit testing jenkins-infra/helm-charts#586

Merged

The files service selector app.kubernetes.io/name label of mirrorbits isn't the same as the deployment one jenkins-infra/helm-charts#589

Closed


          nameOverride instead of fullnameOverride

f2baaaa

lemeurherve mentioned this pull request

feat(mirrorbits): allow setting ingress pathType jenkins-infra/helm-charts#583

Merged

lemeurherve and others added 4 commits

August 10, 2023 17:18


          Merge branch 'main' into helpdesk2649-uc-r2

8fccbbe


          use last mirrorbits chart version with service & ingress fixes

5bdc2d5


          changes from last versions of mirrorbits chart

41fa978


          bump the actual chart 🤦

7dda2fb

dduportal approved these changes

View reviewed changes

Contributor

dduportal left a comment

LGTM, both code and helmfil diff looks good!

IIUC you already tried the installation manually (and cleaned it up) so go go go


          Merge branch 'main' into helpdesk2649-uc-r2

dad2cfb

lemeurherve merged commit 3d02585 into jenkins-infra:main

6 checks passed

lemeurherve deleted the helpdesk2649-uc-r2 branch

August 11, 2023 08:52

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

Labels

None yet