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

Add image creation from instance snapshot WD-10293 #733

Merged

Conversation

edlerd
Copy link
Collaborator

@edlerd edlerd commented Apr 10, 2024

Done

  • Add image creation from instance snapshot
  • Avoid overflowing notification content in toast list
  • Improve copy of all action buttons on snapshots to say "Edit snapshot", "Delete snapshot" or "Save changes", so they contain a verb and a noun.

Fixes WD-10293

QA

  1. Run the LXD-UI:
    • On the demo server via the link posted by @webteam-app below. This is only available for PRs created by collaborators of the repo. Ask @mas-who or @edlerd for access.
    • With a local copy of this branch, run as described here.
  2. Perform the following QA steps:
    • open instance detail page > snapshots
    • create a snapshot
    • click publish snapshot
    • confirm modal
    • check under images the new entry and use it to launch a new instance.

@webteam-app
Copy link

@edlerd edlerd force-pushed the image-publishing-from-instance-snapshot branch 8 times, most recently from 169fe6e to 857d890 Compare April 11, 2024 09:08
@edlerd edlerd changed the title Add image creation from instance snapshot Add image creation from instance snapshot WD-10293 Apr 11, 2024
@mas-who
Copy link
Collaborator

mas-who commented Apr 15, 2024

This looks good on the happy paths already. See below a few issues observed from QA:

  1. When publishing a image from snapshot, we get a notification that the process has started but no further notifications for success or failure.
  2. When trying to publish an image from the same snapshot, the below error occurs
    Screenshot from 2024-04-15 16-27-14
    Is it possible to prevent the user from publishing an image in this scenario somehow? Not sure if this is possible but we could also rely on the error message which relates back to point 1.

@edlerd edlerd force-pushed the image-publishing-from-instance-snapshot branch from 857d890 to dbef728 Compare April 15, 2024 14:54
@mas-who
Copy link
Collaborator

mas-who commented Apr 15, 2024

Code looks good with just one small nit. I'm not sure why when testing on the demo server the success / failure messages are not coming through, since I can see clearly it's implemented in code and testing it locally the notifications does show.

Edit: I think I was just a little impatient and refreshed the browser before the events came though from the demo server. All good here.

@edlerd edlerd force-pushed the image-publishing-from-instance-snapshot branch from dbef728 to 4638bae Compare April 15, 2024 15:12
@edlerd edlerd requested a review from mas-who April 15, 2024 15:14
@edlerd edlerd force-pushed the image-publishing-from-instance-snapshot branch from 4638bae to 7158d6c Compare April 15, 2024 15:49
@edlerd edlerd force-pushed the image-publishing-from-instance-snapshot branch from 7158d6c to aeaa188 Compare April 15, 2024 16:11
@edlerd edlerd merged commit 4460636 into canonical:main Apr 16, 2024
10 checks passed
@edlerd edlerd deleted the image-publishing-from-instance-snapshot branch April 16, 2024 06:59
github-actions bot pushed a commit that referenced this pull request Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants