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

fix: failed to resume /active vm schedule job #69

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

a110605
Copy link
Collaborator

@a110605 a110605 commented Jan 8, 2025

Summary

There is a sudden harvesterhci.io.schedulevmbackup websocket changes data object missing metadata.state.name.

See
Screenshot 2025-01-08 at 2 55 06 PM

PR Checklists

  • Do we need to backport this PR change to the Harvester Dashboard?
    • Yes, the relevant PR is at:
  • Are backend engineers aware of UI changes?

Related Issue

harvester/harvester#7174

Test screenshot/video

Screen.Recording.2025-01-08.at.4.00.47.PM.mov

Extra technical notes summary

After synced with @WebberHuang1118 , we can consume status.suspended = true to determine schedule job is suspended or not. There is no other status for schedule job so far.

We can enhance get state() if backend has other status in the future.

Screenshot 2025-01-08 at 3 49 15 PM

see HEP : https://github.com/harvester/harvester/blob/master/enhancements/20240611-vm-schedule-backup.md#crd

status.suspended presents if the schedule is suspended

@a110605 a110605 self-assigned this Jan 8, 2025
@a110605 a110605 requested a review from houhoucoop January 8, 2025 07:04
@a110605 a110605 changed the title fix: resume/active scheduling job failed fix: failed to resume /active vm schedule job Jan 8, 2025
Copy link
Member

@WebberHuang1118 WebberHuang1118 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.

Copy link
Collaborator

@torchiaf torchiaf left a comment

Choose a reason for hiding this comment

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

LGTM

@a110605 a110605 merged commit 4ca6cae into harvester:main Jan 10, 2025
2 checks passed
a110605 added a commit that referenced this pull request Jan 10, 2025
….5/pr-69

fix: failed to resume /active vm schedule job (backport #69)
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.

3 participants