-
Notifications
You must be signed in to change notification settings - Fork 358
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
feat: [M3-8017] - Support Linodes in Distributed Compute Regions on Image Create #10544
feat: [M3-8017] - Support Linodes in Distributed Compute Regions on Image Create #10544
Conversation
… Linode in a distributed compute region
await findByText('Image scheduled for creation.'); | ||
}); | ||
|
||
it('should render a notice if the user selects a Linode in a distributed compute region', async () => { |
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.
Unit test for this PR's addition
); | ||
}); | ||
|
||
it('should render an encryption notice if disk encryption is enabled and the Linode is not in a distributed compute region', async () => { |
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.
Unit test for @dwiley-akamai 's change in #10521
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.
Oops, I just realized you covered this in a cypress test already. I'll keep it here anyway unless there are any objections.
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.
Verified the notice appears when selecting a Linode in a distributed region.
This Linode is in a distributed compute region. Images captured | ||
from this Linode will be stored in the closest core site. | ||
</Notice> | ||
)} |
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.
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 increased overall spacing in ba513a7, let me know if that looks okay to you! 👁️
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.
Looks good, thanks!
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 for adding unit tests for this component! 🙌🏽
Coverage Report: ✅ |
Description 📝
This Linode is in a distributed compute region. Images captured from this Linode will be stored in the closest core site.
if you select a Linode that is in a distributed region on the Image Create page ℹ️Image will be stored in the closest core site to (us-den-edge-1)
below the Linode select if the selected Linode is in a distributed region ℹ️These changes are based on the Image Service Gen2 Figma wireframes 🎨 - Ask me for a link if you want to reference it.
Preview 📷
How to test 🧪
Prerequisites
Verification steps
As an Author I have considered 🤔