-
Notifications
You must be signed in to change notification settings - Fork 379
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
upcoming: [M3-9499] - Maintenance policy types and queries #11990
Conversation
…maintenance-types-and-queries
…maintenance-types-and-queries
There was a problem hiding this 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
packages/api-v4/src/account/types.ts
Outdated
'linode_migration', | ||
'linode_mutate_create', | ||
'linode_mutate', | ||
'linode_power_on_off', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'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?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Hi @hkhalil-akamai, can you please resolve the merge conflicts before approving ! |
0b36319
to
cfd3008
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @hkhalil-akamai 🚢
…maintenance-types-and-queries
cfd3008
to
2ff2b09
Compare
Waiting for additional changes to the API spec before merging. |
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 |
…maintenance-types-and-queries
…maintenance-types-and-queries
Cloud Manager UI test results🔺 1 failing test on test run #9 ↗︎
Details
TroubleshootingUse this command to re-run the failing tests: pnpm cy:run -s "cypress/e2e/core/stackscripts/smoke-community-stackscripts.spec.ts" |
Test failure appears unrelated. |
Description 📝
Adds types, endpoints and query hooks for the Host & VM Maintenance Policy project.
Changes 🔄
API-v4
MaintenancePolicy
typegetMaintenancePolicies
endpointEvent
,Maintenance
andLinode
types with new fields'Maintenance Policy'
region capabilityManager
Preview 📷
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
As an Author, before moving this PR from Draft to Open, I confirmed ✅