Skip to content

fix(lxc): do not reboot for cpu block changes #1932

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

Merged
merged 1 commit into from
Apr 27, 2025
Merged

Conversation

CCreek96
Copy link
Contributor

@CCreek96 CCreek96 commented Apr 27, 2025

Contributor's Note

Removed rebootRequired in containerUpdate function when changes are being applied to the cpu. Tested the build in my cluster and this fixes the bug reported in #1920

  • I have added / updated documentation in /docs for any user-facing features or additions.
  • I have added / updated acceptance tests in /fwprovider/tests for any new or updated resources / data sources.
  • I have ran make example to verify that the change works as expected.

Proof of Work

Screenshot from 2025-04-26 22-19-08

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Closes #1920

Signed-off-by: Connor Creek <creek.connor@yahoo.com>
Copy link
Owner

@bpg bpg left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this @CCreek96

LGTM! 🚀

@bpg bpg merged commit 5e49b98 into bpg:main Apr 27, 2025
7 checks passed
@bpg
Copy link
Owner

bpg commented Apr 27, 2025

@all-contributors please add @CCreek96 for code

Copy link
Contributor

@bpg

@CCreek96 already contributed before to code

@bpg bpg changed the title fix(container): do not reboot for cpu block changes fix(lxc): do not reboot for cpu block changes Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Changing an attribute in cpu block for a container causes a reboot
2 participants