Skip to content

feat(nextcloud): allow imagePullSecrets for cronjob - #781

Merged
wrenix merged 2 commits into
nextcloud:mainfrom
mazinsfr:feature/issue777-add-pullSecrets-to-cronjob
Oct 4, 2025
Merged

feat(nextcloud): allow imagePullSecrets for cronjob#781
wrenix merged 2 commits into
nextcloud:mainfrom
mazinsfr:feature/issue777-add-pullSecrets-to-cronjob

Conversation

@mazinsfr

@mazinsfr mazinsfr commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Description of the change

Adds imagePullSecrets for cronjob pod.

Benefits

Allows to pull nextcloud image from private registry for cronjob pod.

Possible drawbacks

N/A

Applicable issues

Additional information

Cronjob pod and nextcloud pod should use the same image. So I assume the user uses the same registry for nextcloud pod and cronjob pod.

Checklist

Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr>

@wrenix wrenix left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, do you like to rebase/merge when #784 is merged automatically?

Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com>
@wrenix wrenix changed the title feat: allow imagePullSecrets for cronjob feat(nextcloud): allow imagePullSecrets for cronjob Oct 4, 2025
@wrenix
wrenix enabled auto-merge (squash) October 4, 2025 07:15
@wrenix
wrenix merged commit 8440e74 into nextcloud:main Oct 4, 2025
10 checks passed
ojsef39 pushed a commit to ojsef39/helm that referenced this pull request Oct 7, 2025
Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr>
Signed-off-by: Josef Hofer <me@jhofer.de>
MichaelSp pushed a commit to MichaelSp/nextcloud-helm that referenced this pull request Nov 8, 2025
* fix(nextcloud): lower outdated apps prometheus alertrule to info nextcloud#784

Signed-off-by: Josef Hofer <me@jhofer.de>

* feat(nextcloud): allow imagePullSecrets for cronjob nextcloud#781

Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr>

* feat(nextcloud): add topologySpreadConstraints support nextcloud#782

Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr>

* fix(nextcloud): cronjob not working with no $.Values.podLabels set nextcloud#785

wrong type for value; expected map[string]interface {}; got interface {}

i think this issue got introduced with nextcloud#779

Signed-off-by: Josef Hofer <me@jhofer.de>

* feat(pvc): add extra labels on pvcs nextcloud#793

Signed-off-by: DrummyFloyd <jonathan.monnet28@gmail.com>

* fix(nextcloud): update AppVersion to v32.0.1 nextcloud#795

Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com>

* chore(deps): bump helm/kind-action from 1.12.0 to 1.13.0

Bumps [helm/kind-action](https://github.com/helm/kind-action) from 1.12.0 to 1.13.0.
- [Release notes](https://github.com/helm/kind-action/releases)
- [Commits](helm/kind-action@v1.12.0...v1.13.0)

---
updated-dependencies:
- dependency-name: helm/kind-action
  dependency-version: 1.13.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix(docs): typo in README.md for ipFamilyPolicy nextcloud#796

Signed-off-by: Roger Roger <me@rogerrogert.de>

---------

Signed-off-by: Josef Hofer <me@jhofer.de>
Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr>
Signed-off-by: DrummyFloyd <jonathan.monnet28@gmail.com>
Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Roger Roger <me@rogerrogert.de>
Co-authored-by: Josef Hofer <me@jhofer.de>
Co-authored-by: Mazins <133565773+mazinsfr@users.noreply.github.com>
Co-authored-by: DrummyFloyd <jonathan.monnet28@gmail.com>
Co-authored-by: WrenIX <133280015+wrenix@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: V Aretakis <vassilis@aretakis.eu>
Co-authored-by: Roger Roger <me@rogerrogert.de>
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.

Feature: Apply image.pullSecrets to cronjob

2 participants