search

kata-containers / packaging

Kata Containers version 1.x packaging (for version 2.x see https://github.com/kata-containers/kata-containers).
https://katacontainers.io/
Apache License 2.0
119 stars 92 forks source link

kernel: virtiofs: remove already applied patch #1096

Closed jcvenegas closed 4 years ago

jcvenegas commented 4 years ago

The virtios-dev tag changed recently to a new version:

Linux 5.8-rc4

This already include some patches, remove not needed patches.

Jose Carlos Venegas Munoz jose.carlos.venegas.munoz@intel.com

Fixes: #1095

jodh-intel commented 4 years ago

/test

jcvenegas commented 4 years ago

Hey Folks, I still having some build issue locally now are config issues, I am looking to it. The first error I get is that the final config does not have CONFIG_MEMCG_SWAP_ENABLED=y. Seems that this was removed or moved in this version. I am looking where it is now or if only changed the name.

GabyCT commented 4 years ago

thanks @jcvenegas the issue is gone http://jenkins.katacontainers.io/job/kata-metrics-runtime-ubuntu-virtiofs-18-04-PR/6/console

GabyCT commented 4 years ago

/test

GabyCT commented 4 years ago

/AzurePipelines run

azure-pipelines[bot] commented 4 years ago
Azure Pipelines successfully started running 2 pipeline(s).
GabyCT commented 4 years ago

@jcvenegas you need to bump the kernel config version please

10:40:16 ERROR: Please bump version in kernel/kata_config_version
jcvenegas commented 4 years ago

/test

GabyCT commented 4 years ago

/AzurePipelines run

azure-pipelines[bot] commented 4 years ago
Azure Pipelines successfully started running 2 pipeline(s).
jcvenegas commented 4 years ago

@amshinde so, the plan is to wait for a proper tag, should I close this PR ?

amshinde commented 4 years ago

I dont think this is required any longer. Closing this @jcvenegas.