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

Workaround ceilometer-agent-compute not running #499

Merged
merged 1 commit into from
Jul 23, 2024
Merged

Conversation

samuelallan72
Copy link
Contributor

@samuelallan72 samuelallan72 commented Jul 18, 2024

This is a bug in nova-compute and/or ceilometer-agent
( https://bugs.launchpad.net/charm-ceilometer-agent/+bug/1947585 ),
where the nova-compute resume action can sometimes fail.
The workaround added here is to restart ceilometer-agent-compute
if it fails for this reason.

Fixes: #427

@samuelallan72 samuelallan72 force-pushed the fix-nova-resume branch 2 times, most recently from 33c1992 to c0784ae Compare July 18, 2024 09:09
@samuelallan72 samuelallan72 changed the title WIP: workaround ceilometer-agent-compute not running in nova-compute … Workaround ceilometer-agent-compute not running in nova-compute-resume Jul 19, 2024
@samuelallan72 samuelallan72 changed the title Workaround ceilometer-agent-compute not running in nova-compute-resume Workaround ceilometer-agent-compute not running Jul 19, 2024
@samuelallan72 samuelallan72 force-pushed the fix-nova-resume branch 3 times, most recently from 7f12e09 to 329dc8e Compare July 22, 2024 05:14
This is a bug in nova-compute and/or ceilometer-agent
( https://bugs.launchpad.net/charm-ceilometer-agent/+bug/1947585 ),
where the nova-compute `resume` action can sometimes fail.
The workaround added here is to restart ceilometer-agent-compute
if it fails for this reason.

Fixes: #427
@samuelallan72 samuelallan72 marked this pull request as ready for review July 22, 2024 06:21
@samuelallan72 samuelallan72 requested a review from a team as a code owner July 22, 2024 06:21
cou/apps/core.py Show resolved Hide resolved
Copy link
Contributor

@chanchiwai-ray chanchiwai-ray left a comment

Choose a reason for hiding this comment

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

Looks reasonable, let's see if it fixes the issue

@samuelallan72 samuelallan72 merged commit 2c2e36c into main Jul 23, 2024
5 checks passed
@samuelallan72 samuelallan72 deleted the fix-nova-resume branch July 23, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants