Skip to content

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

Merged
wrenix merged 3 commits into
nextcloud:mainfrom
ojsef39:fix/podLabel_cronjob
Oct 10, 2025
Merged

fix(nextcloud): cronjob not working with no $.Values.podLabels set#785
wrenix merged 3 commits into
nextcloud:mainfrom
ojsef39:fix/podLabel_cronjob

Conversation

@ojsef39

@ojsef39 ojsef39 commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

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

i think this issue got introduced with #779

Description of the change

adds empty podLabels so templating cronjobwill work :)

Benefits

Possible drawbacks

Applicable issues

  • fixes #

Additional information

Checklist

@wrenix

wrenix commented Oct 7, 2025

Copy link
Copy Markdown
Collaborator

Good found please sign you commit and bump the chart.yaml with a patch level

@ojsef39
ojsef39 force-pushed the fix/podLabel_cronjob branch from f7659c8 to 08bfe8b Compare October 7, 2025 12:20
mazinsfr and others added 2 commits October 7, 2025 14:22
Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr>
Signed-off-by: Josef Hofer <me@jhofer.de>
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>
@ojsef39
ojsef39 force-pushed the fix/podLabel_cronjob branch from 08bfe8b to f9cecd4 Compare October 7, 2025 12:22
@ojsef39

ojsef39 commented Oct 7, 2025

Copy link
Copy Markdown
Contributor Author

now i got it xD

@wrenix
wrenix merged commit 6ef85d1 into nextcloud:main Oct 10, 2025
10 checks passed
@ojsef39
ojsef39 deleted the fix/podLabel_cronjob branch October 10, 2025 07:49
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.

3 participants