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

Deprecated config default values in helmchart #12205

Closed
ftqualifio opened this issue Dec 13, 2024 · 3 comments
Closed

Deprecated config default values in helmchart #12205

ftqualifio opened this issue Dec 13, 2024 · 3 comments
Labels
in linear Issue or PR has been created in Linear for internal review

Comments

@ftqualifio
Copy link

Bug Description

The following default in the helm chart prevents n8n from starting as it is not in the schema anymore. The config shouldn't have any default value and be optional.

config:
  executions:
    # prune executions by default
    pruneData: "true"
    # Per default, we store 1 year of history
    pruneDataMaxAge: 3760

To Reproduce

Deploy the helm chart

Expected behavior

Being able to deploy without the default config.

Operating System

Debian 12

n8n Version

1.67.1

Node.js Version

20.18.0

Database

PostgreSQL

Execution mode

main (default)

@Joffcom
Copy link
Member

Joffcom commented Dec 13, 2024

Hey @ftqualifio,

We have created an internal ticket to look into this which we will be tracking as "N8N-7956"

@Joffcom Joffcom added the in linear Issue or PR has been created in Linear for internal review label Dec 13, 2024
@netroy
Copy link
Member

netroy commented Dec 13, 2024

That makes sense. but, which helm chart are you referring to?

@ftqualifio
Copy link
Author

I missed the good project on the links in artifacthub. The issue already exists: 8gears/n8n-helm-chart#125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in linear Issue or PR has been created in Linear for internal review
Projects
None yet
Development

No branches or pull requests

3 participants