Skip to content

Conversation

fidencio
Copy link
Member

Otherwise we just cannot start a container that requests more than 1 vcpu.

Otherwise we just cannot start a container that requests more than 1
vcpu.

Signed-off-by: Fabiano Fidêncio <fidencio@northflank.com>
@fidencio fidencio requested review from lifupan and Apokleos June 25, 2025 12:38
@fidencio
Copy link
Member Author

@BbolroC, any chance to help me figuring out why s390x is failing due to this change here?

Copy link
Contributor

@Apokleos Apokleos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx, it will make max vcpus default value align with other vmms.
LGTM.

@BbolroC
Copy link
Member

BbolroC commented Jun 26, 2025

@BbolroC, any chance to help me figuring out why s390x is failing due to this change here?

Resolved. Thanks for your patience!

Copy link
Contributor

@Champ-Goblem Champ-Goblem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@Champ-Goblem Champ-Goblem merged commit b6cd6e6 into kata-containers:main Jun 26, 2025
930 of 963 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants