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 custom image test WD-10108 #732

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

edlerd
Copy link
Collaborator

@edlerd edlerd commented Apr 9, 2024

Done

  • Add custom image test

Fixes WD-10108

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:
    • ensure test suite passes

@webteam-app
Copy link

@edlerd edlerd force-pushed the add-image-test branch 2 times, most recently from 8f5b2f6 to ea27976 Compare April 9, 2024 15:21
Signed-off-by: David Edler <david.edler@canonical.com>
@edlerd edlerd force-pushed the add-image-test branch 3 times, most recently from 606ce43 to 1c92de7 Compare April 9, 2024 16:32
Signed-off-by: David Edler <david.edler@canonical.com>
import { test } from "./fixtures/lxd-test";
import { deleteInstance, randomInstanceName } from "./helpers/instances";

test("search for custom image and create an instance from it", async ({
Copy link
Collaborator

Choose a reason for hiding this comment

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

this test would fail locally I think, maybe we should skip it for dev environment? Or we could setup a shell script to run before the e2e tests kicks off?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Right, this hinges on the availability of the custom image.
I was planning to extend this test to create the custom image from the UI in the next cycle (as we have the item to add support for that on our roadmap). Maybe we can ignore this incompatibility for local test runs for now and manually create the custom image on our dev machines?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Cool sounds good 👍

@mas-who
Copy link
Collaborator

mas-who commented Apr 10, 2024

I see the test run time came down a fair bit, nice! Just one query about running the custom image test locally

@edlerd edlerd merged commit 74f03ea into canonical:main Apr 10, 2024
10 checks passed
@edlerd edlerd deleted the add-image-test branch April 10, 2024 08:43
github-actions bot pushed a commit that referenced this pull request Apr 10, 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