Skip to content

upcoming: [M3-9499] - Maintenance policy types and queries #11990

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

Conversation

hkhalil-akamai
Copy link
Contributor

@hkhalil-akamai hkhalil-akamai commented Apr 8, 2025

Description 📝

Adds types, endpoints and query hooks for the Host & VM Maintenance Policy project.

Changes 🔄

API-v4

  • Add new MaintenancePolicy type
  • Add new getMaintenancePolicies endpoint
  • Update Event, Maintenance and Linode types with new fields
  • Add 'Maintenance Policy' region capability

Manager

  • Update event message generator with tentative maintenance event messages
  • Add maintenance policies query key and hook

Preview 📷

  • No visual changes expected to Cloud Manager

How to test 🧪

Verify newly defined and updates types, keys, and queries follow our existing standards.

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@hkhalil-akamai hkhalil-akamai changed the title upcoming: [M3-9499] - VM Host Maintenance types and queries upcoming: [M3-9499] - Maintenance policy types and queries Apr 9, 2025
@hkhalil-akamai hkhalil-akamai marked this pull request as ready for review April 9, 2025 16:22
@hkhalil-akamai hkhalil-akamai requested a review from a team as a code owner April 9, 2025 16:22
@hkhalil-akamai hkhalil-akamai requested review from harsh-akamai and hasyed-akamai and removed request for a team April 9, 2025 16:22
Copy link
Contributor

@harsh-akamai harsh-akamai left a comment

Choose a reason for hiding this comment

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

Type additions/changes ✅
Query additions ✅

Just one nit regarding the event actions

Comment on lines 397 to 400
'linode_migration',
'linode_mutate_create',
'linode_mutate',
'linode_power_on_off',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'linode_migration',
'linode_mutate_create',
'linode_mutate',
'linode_power_on_off',
'linode_migrate',
'linode_mutate_create',
'linode_mutate',
'linode_poweroff_on',

Event actions as mentioned in the APISpec, or am I missing something?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think you're correct, I will verify with the API development team.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

linode_migrate is an existing action type, so I was a little confused. I left a comment on the API spec for further clarification.

@harsh-akamai harsh-akamai added the Add'tl Approval Needed Waiting on another approval! label Apr 10, 2025
@hasyed-akamai
Copy link
Contributor

Hi @hkhalil-akamai, can you please resolve the merge conflicts before approving !

@hkhalil-akamai hkhalil-akamai force-pushed the M3-9499-vm-host-maintenance-types-and-queries branch from 0b36319 to cfd3008 Compare April 15, 2025 04:01
@hkhalil-akamai hkhalil-akamai requested a review from a team as a code owner April 15, 2025 04:01
@hkhalil-akamai hkhalil-akamai requested review from jdamore-linode and removed request for a team April 15, 2025 04:01
Copy link
Contributor

@hasyed-akamai hasyed-akamai left a comment

Choose a reason for hiding this comment

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

Thanks @hkhalil-akamai 🚢

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Apr 15, 2025
@hasyed-akamai hasyed-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Apr 15, 2025
@hkhalil-akamai hkhalil-akamai force-pushed the M3-9499-vm-host-maintenance-types-and-queries branch from cfd3008 to 2ff2b09 Compare April 15, 2025 13:41
@hkhalil-akamai
Copy link
Contributor Author

Waiting for additional changes to the API spec before merging.

Copy link

github-actions bot commented May 2, 2025

This PR is stale because it has been open 15 days with no activity. Please attend to this PR or it will be closed in 5 days

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #9 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing576 Passing5 Skipped102m 24s

Details

Failing Tests
SpecTest
smoke-community-stackscripts.spec.tsCloud Manager Cypress Tests→Community Stackscripts integration tests » Community Stackscripts integration tests

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/stackscripts/smoke-community-stackscripts.spec.ts"

@hkhalil-akamai
Copy link
Contributor Author

Test failure appears unrelated.

@hkhalil-akamai hkhalil-akamai merged commit 2569d36 into linode:develop May 6, 2025
34 of 35 checks passed
@hkhalil-akamai hkhalil-akamai deleted the M3-9499-vm-host-maintenance-types-and-queries branch May 6, 2025 18:29
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Multiple approvals and ready to merge! Ready for Review
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

4 participants