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

chore(updates.jenkins.io): use new mirrorbits-parent helm chart #4320

Conversation

lemeurherve
Copy link
Member

@lemeurherve lemeurherve commented Sep 7, 2023

This PR use the new mirrorbits-parent helm chart instead of the mirrorbits one to deploy updates.jenkins.io

Content of the secret:

mirrorbits-lite:
    geoipupdate:
        account_id: THE_ID
        license_key: THE_KEY
    repository:
        secrets:
            enabled: true
            data:
                azurestorageaccountkey: THE_KEY
                azurestorageaccountname: THE_NAME
    conf: |
        Repository: /srv/repo
        Templates: /usr/share/mirrorbits/templates
        GeoipDatabasePath: /usr/share/GeoIP/
        Gzip: true
        ListenAddress: :8080
        #RPCListenAddress: localhost:3390
        RedisAddress: updates-jenkins-io.redis.cache.windows.net:6379
        # [...snip...]

Ref: jenkins-infra/helpdesk#2649

@lemeurherve lemeurherve force-pushed the helpdesk2649-test-mirrorbits-parent-chart branch from bb0e9d8 to b5b5511 Compare September 8, 2023 06:49
@lemeurherve
Copy link
Member Author

Unrelated updatecli error 🤔

image

@lemeurherve lemeurherve marked this pull request as ready for review September 8, 2023 13:11
@lemeurherve lemeurherve requested a review from a team September 8, 2023 13:11
@dduportal
Copy link
Contributor

Unrelated updatecli error 🤔
image

If you look at the error associated with checkPodTemplate :

WARNING: current yaml key is "spec.containers[0].name" and should be updated to "$.spec.containers[0].name"
 ✗ key "$.spec.containers[0].name", in YAML file "PodTemplates.yaml", is incorrectly set to "\"jnlp\"" and should be "jnlp"

I believe if you solve the warning, then updatecli will behave (unrelated to your PR)

@lemeurherve
Copy link
Member Author

Updatecli now fails only on mirrorbits-parent target (expected as not yet on main branch), merging.

@lemeurherve lemeurherve merged commit 0b660f2 into jenkins-infra:main Sep 11, 2023
6 checks passed
@lemeurherve lemeurherve deleted the helpdesk2649-test-mirrorbits-parent-chart branch September 11, 2023 08:28
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