Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Support kernel >=5.1 #735

Closed
zimmski opened this issue Sep 27, 2019 · 4 comments · Fixed by gravitational/satellite#124
Closed

Support kernel >=5.1 #735

zimmski opened this issue Sep 27, 2019 · 4 comments · Fixed by gravitational/satellite#124
Assignees

Comments

@zimmski
Copy link

zimmski commented Sep 27, 2019

Describe the bug

Some kernel modules that do not exist with kernel >=5.1 anymore, e.g. https://cateee.net/lkddb/web-lkddb/NF_NAT_IPV4.html, are required by the gravity preflight checks. Hence, Gravity cannot be installed e.g. on openSUSE's MicroOS.

To Reproduce
Install a VM e.g. with the latest openSUSE MicroOS which is using kernel 5.2 (at the moment) and try to install gravity.

Expected behavior
The expectation is that Gravity is supported on every modern distribution. Hence, also on distributions with a newer kernel.

Logs

...
[×] required kernel boot config parameter CONFIG_NF_NAT_IPV4 missing
[×] required kernel boot config parameter CONFIG_NF_NAT_NEEDED missing

Environment (please complete the following information):

  • OS openSUSE MicroOS
  • Gravity 6.2.0
@klizhentas
Copy link
Contributor

@knisbet @r0mant can you please take a look? seems important

@knisbet
Copy link
Contributor

knisbet commented Oct 1, 2019

@zimmski Thank you for the report, we'll take a look at ensuring these checks only run on the relevant kernels.

As a workaround, you can use GRAVITY_CHECKS_OFF=true as an environment variable to the install and join commands, that will skip any pre-flight check failures.

@zimmski
Copy link
Author

zimmski commented Oct 15, 2019

Thanks for the changes. We try to run Gravity on openSUSE MicroOS and #745 is basically the next issue we ran into. However, this changes works as intended.

@zimmski zimmski closed this as completed Oct 15, 2019
@zimmski
Copy link
Author

zimmski commented Oct 15, 2019

Sorry for closing the issue, I just saw that the change is still not upstreamed.

@zimmski zimmski reopened this Oct 15, 2019
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 22, 2019
)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 22, 2019
)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 22, 2019
)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 22, 2019
)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 22, 2019
)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 25, 2019
) (#130)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 25, 2019
) (#129)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
knisbet pushed a commit to gravitational/satellite that referenced this issue Oct 25, 2019
) (#128)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
bernardjkim pushed a commit to gravitational/satellite that referenced this issue Mar 6, 2020
) (#129)

* Filter CONFIG_NF_NAT_IPV4 / CONFIG_NF_NAT_NEEDED on recent kernels

Fixes gravitational/gravity#735

* fix gofmt
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants