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.

Sign up for GitHub

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

Jump to bottom

Debug system tests #288

Closed
wants to merge 55 commits into from
Closed

Debug system tests #288

wants to merge 55 commits into from

Conversation

masnax
Copy link
Contributor

@masnax masnax commented Apr 18, 2024

No description provided.

@simondeziel
test/includes/microcloud: allow using a dedicated storage source device
f3fcf41 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: only install zfsutils-linux and htop if CLO…
460a715 
…UD_INSPECT is set

Also, stop trying to install snapd and curl as they are included in `ubuntu-minimal:` images.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: stop adding ubuntu-minimal:
894df61 
All LXD releases we care about have that remote builtin now.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/check: require instance_ready_state API on host LXD
a3db46f 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: use ubuntu-minimal for c1/v1
29d4fd4 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: poll less frequently when waiting on c1/v1
5782411 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: report when c1/v1 have booted successfully
6703cab 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/main: fix indentation
84af383 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: fix check for compat with Go 1.22.0
d9c0bb7 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: align tests job with LXD's
63aef4e 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: add initial (snap-based) system tests
65027ba 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: require system tests to pass before triggering LP build
e37f1cb 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: nicer output when wiping/deleting disks
312586a 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suite/basic: nicer output when starting c1/v1
1685e7d 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: use lxd default channel (5.21/stable as of …
a4e80b0 
…today)

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: fix restore_system arg passing
8a82f7b 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: create smaller c1/v1 instances
841546a 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: rename test_instances to test_instances_launch
05d2021 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/main: rename test_instances to test_instances_launch
6647c2f 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: set suitable PATH in the profile
6044ed4 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: call new_system() in new_systems()
c26b1e6 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: use wait even if CONCURRENT_SETUP!=1
1fc2abc 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: use less subshells in test_instances_launch()
210ba10 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: add test_instances_config() test
cc1f5f6 
This test verifies storage pool and network default selection based on
available ones.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/preseed: use less subshells
28f140d 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/main: add test_instances_config
a21f3a6 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: optionally skip launching VMs on microcloud
552cc7d 
On GHA runners, the microcloud hosts are VMs so we are already 2 VM levels
deep. If a VM is started on top of that, we get pathologically slow VM boot. In
previous tests, the final VM stage didn't even get its LXD agent to dial in
inside of 2 hours.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: skip launching VMs on top of microcloud VMs
603022d 
On GHA runners, the microcloud hosts are VMs so we are already 2 VM levels
deep. If a VM is started on top of that, we get pathologically slow VM boot. In
previous tests, the final VM stage didn't even get its LXD agent to dial in
inside of 2 hours.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: fix TEST_CONSOLE=0
5718473 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/preseed: fix TEST_CONSOLE=0
51bb646 
Signed-off-by: Simon Deziel <[email protected]>
simondeziel and others added 25 commits April 18, 2024 09:14
@simondeziel
test/suites/add: reduce subshell use
5913911 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: safer grep
bb2da36 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/add: less subshells
b357b8d 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: safer grep
8bf8980 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: less subshells
05237c6 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/main: running single test case also require the new_systems call
05edfae 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: s/test_case/_test_case/ as it's not an actual test…
acb3363 
…ing function

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/main: rework how individual/group test(s) are run
ea238de 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: split system tests into suites
4bc8ba5 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: only take a snapshot if SNAPSHOT_RESTORE=1
f488f18 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: use cohorts for micro{ceph,ovn,cloud} too
2d2333d 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: workaround for LXD shutdown bug (debug)
1655238 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: add some echos to set_debug_binaries()
867fd0a 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: poll less aggressively in lxd_wait_vm()
4badb3f 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: less subshells
faf9cf1 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: use LXD latest/edge (canonical/lxd-pkg-snap…
f6cd47a 
…#426)

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: safer grep
a4d84df 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: safer grep for matching numeric args
d4cd4c3 
We don't want to match on any digit present anywhere like that:

```
$ echo "disk2" | grep -P '\d+'
2
```

Instead, require the searched string to be made of digits only.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: use quiet file pushes
c72c720 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
github: sideload microcloud{,d} binaries for system-tests
a74a6d9 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: add test OVN connectivity
78f92bc 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/suites/basic: use smaller rootfs
c1eef92 
Signed-off-by: Simon Deziel <[email protected]>
@simondeziel
test/includes/microcloud: simplify rm -rf globbing
a827156 
Signed-off-by: Simon Deziel <[email protected]>
@masnax
microcloud/test: Only run mismatch
a565f16 
Signed-off-by: Max Asnaashari <[email protected]>
@masnax
microcloud/test/suites: Run interactive first
3067243 
Signed-off-by: Max Asnaashari <[email protected]>
@masnax masnax closed this Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@masnax @simondeziel