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

[feature] implement crash detection for qemu vms #4332

Open
wants to merge 152 commits into
base: master
Choose a base branch
from

Conversation

philfry
Copy link

@philfry philfry commented Mar 10, 2020

Hi all,

right now afaik OpenNebula is unable to tell whether a qemu-VM has crashed.
This PR adds the ability to detect VM crashes/panics and to define what action should be taken.

If set, an isa-panic-device is added to the vm as well as the on_crash-action itself:

<on_crash>{destroy|restart|preserve|coredump-destroy|coredump-restart}</on_crash>
<devices>
  <panic model='isa'/>
</devices>

@philfry philfry force-pushed the feature_crash_detection branch 3 times, most recently from 83c2f72 to 54521c9 Compare March 10, 2020 13:49
@philfry
Copy link
Author

philfry commented Mar 11, 2020

Since DCO is complaining: What's the correct way to handle long email addresses that cause the signed-off-by-line to exceed 78 chars? Is a line break okay?

Angel Luis Moya Gonzalez and others added 25 commits June 16, 2020 11:28
Avoid reusing global `vm` var in threads.
Sergio Betanzos and others added 3 commits August 19, 2020 11:49
…penNebula#160)

Note: Branch 5.12 didn't really suffer from the similar problem in 5.10,
as here "saving" reason is considered as UNKNOWN state for which we
don't collect the I/O statistics. This change only aligns with previous
behaviour to consider "saving" as RUNNING state, and explicitly lists
few RUNNING reasons (including new "saving") to skip monitoring for.

It's more a polishment, than a real bugfix.

(cherry picked from commit 5b25744)
@philfry philfry force-pushed the feature_crash_detection branch 2 times, most recently from 09c560c to 0937a12 Compare August 25, 2020 11:15
Sergio Betanzos and others added 19 commits August 26, 2020 12:15
…#159)

* Fix actions and commands
* Add effective group login and switcher
* Fix sidebar component
* Fix header component and test api
* Add pool actions and system section
* Remove package lock json
* POST oneflow fireedge
* schemas oneflow fireedge
* http code accepted

Signed-off-by: Jorge Lobo <[email protected]>
* Fix sidebar component
* Add responsive sidebar
* Update internal grid layout
* F OpenNebula#3951: Update app layouts and router

* F OpenNebula#3951: Fix footer component

* F OpenNebula#3951: Add loading screen & update sidebar

* F OpenNebula#3951: Add responsiveness to sidebar
Signed-off-by: Jorge Lobo <[email protected]>

Co-authored-by: Jorge Lobo <[email protected]>
(cherry picked from commit 3c3b237)
* Add upgradeable app styles
* Add routing transition effect
(cherry picked from commit 7a00d16631e8a3611c1cfdb60a7131e4bf1273fe)
* Fix switcher component
* Fix header styles
* Add popover component in header
(cherry picked from commit c4c98e2)
@philfry philfry force-pushed the feature_crash_detection branch from 0937a12 to 6dc7388 Compare September 8, 2020 13:30
@rsmontero rsmontero force-pushed the master branch 2 times, most recently from bdce88d to 14635f2 Compare December 14, 2021 11:20
@rsmontero rsmontero force-pushed the master branch 2 times, most recently from 339f9c9 to 1cf07c3 Compare January 16, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants