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

helm: re-generate quick-install.yaml after PR #10289 #10604

Merged
merged 1 commit into from Mar 17, 2020

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Mar 17, 2020

quick-install.yaml wasn't re-generated in PR #10289 which reduced the
default NAT table size. This lead to the following error when
installing/upgrading from quick-install.yaml:

level=fatal msg="Error while creating daemon" error="invalid daemon configuration: specified NAT tables size 841429 must not exceed maximum CT table size 786432" subsys=daemon

Fix it by re-generating quick-install.yaml so the new default NAT table
size is used.

Reported-by: Michi Mutsuzaki michi@isovalent.com
Reported-by: Martynas Pumputis m@lambda.lt
Signed-off-by: Tobias Klauser tklauser@distanz.ch


This change is Reviewable

quick-install.yaml wasn't re-generated in PR #10289 which reduced the
default NAT table size. This lead to the following error when
installing/upgrading from quick-install.yaml:

```
level=fatal msg="Error while creating daemon" error="invalid daemon configuration: specified NAT tables size 841429 must not exceed maximum CT table size 786432" subsys=daemon
```

Fix it by re-generating quick-install.yaml so the new default NAT table
size is used.

Reported-by: Michi Mutsuzaki <michi@isovalent.com>
Reported-by: Martynas Pumputis <m@lambda.lt>
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser added pending-review release-note/misc This PR makes changes that have no direct user impact. labels Mar 17, 2020
@tklauser tklauser requested a review from a team March 17, 2020 10:46
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Mar 17, 2020
@tklauser tklauser mentioned this pull request Mar 17, 2020
10 tasks
@raybejjani raybejjani merged commit a7ffd98 into master Mar 17, 2020
1.8.0 automation moved this from In progress to Merged Mar 17, 2020
@raybejjani raybejjani deleted the pr/tklauser/quick-install-fix-nat-table-size branch March 17, 2020 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.8.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

3 participants