Skip to content

Test: Hold all snaps to prevent unplanned refresh#652

Merged
roosterfish merged 1 commit intocanonical:mainfrom
roosterfish:fix_eof
Feb 20, 2025
Merged

Test: Hold all snaps to prevent unplanned refresh#652
roosterfish merged 1 commit intocanonical:mainfrom
roosterfish:fix_eof

Conversation

@roosterfish
Copy link
Contributor

@roosterfish roosterfish commented Feb 20, 2025

The test suite should hold the lxd, microceph, microovn and microcloud snaps to not cause unplanned refreshes during test execution.

This was observed in the pipeline every now and then and caused some of the daemons to be stopped out of nowhere which resulted in API endpoints returning EOF.
In the daemon logs this is clearly indicated by systemd stopping the respective service and starting it afterwards:

Feb 20 09:53:13 micro03 systemd[1]: Stopping snap.microcloud.daemon.service - Service for snap application microcloud.daemon...
Feb 20 09:53:13 micro03 systemd[1]: snap.microcloud.daemon.service: Deactivated successfully.
Feb 20 09:53:13 micro03 systemd[1]: Stopped snap.microcloud.daemon.service - Service for snap application microcloud.daemon.
Feb 20 09:53:16 micro03 systemd[1]: Started snap.microcloud.daemon.service - Service for snap application microcloud.daemon.

An example error visible from the users (MicroCloud) point of view is this. It happened at the same time as snapd started to refresh the MicroCloud snap:

Error: System "micro03" failed to join the cluster: Failed to update cluster status of services: Put "https://10.87.144.161:9443/1.0/services": EOF

The test suite should hold the lxd, microceph, microovn and microcloud snaps to not cause
unplanned refreshes during test execution.

This was observed in the pipeline every now and then and caused some of the daemons to be stopped out of nowhere
which resulted in API endpoints returning EOF.
In the daemon logs this is clearly indicated by systemd stopping the respective service and starting it afterwards.

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Copy link
Member

@simondeziel simondeziel left a comment

Choose a reason for hiding this comment

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

Thanks!

@roosterfish roosterfish merged commit e26a31e into canonical:main Feb 20, 2025
24 checks passed
@roosterfish roosterfish deleted the fix_eof branch February 20, 2025 14:17
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.

2 participants