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

System tests misc improvements #290

Merged
merged 45 commits into from
Apr 24, 2024

Conversation

simondeziel
Copy link
Member

No description provided.

microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
microcloud/test/includes/microcloud.sh Fixed Show fixed Hide fixed
@simondeziel simondeziel force-pushed the system-tests-misc-improvements branch 3 times, most recently from 71506c8 to 305c796 Compare April 20, 2024 02:54
@simondeziel simondeziel force-pushed the system-tests-misc-improvements branch 4 times, most recently from 3ea1371 to a7e9f59 Compare April 23, 2024 17:37
Since various snap edge channels are used, it makes sense to test them more
often than on PRs against MicroCloud itself.

Signed-off-by: Simon Deziel <[email protected]>
This reverts commit a827156.

As pointed out by Max, `rm -rf /foo/*/*` is just emptying all the
subdirectories so it is not equivalent to `rm -rf /foo/*`.

Reported-by: Max Asnaashari <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
This was pointless because `reset_systems` is always called at the begining of
each test.

Reported-by: Max Asnaashari <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
We initially intend to test against LXD latest/edge so don't warn about it.

Signed-off-by: Simon Deziel <[email protected]>
latest/edge uses core24 and other latest/* are not a valid channels.

Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
With output comparison, `set -x` output is easier to understand as we know both
sides of the comparison.

Signed-off-by: Simon Deziel <[email protected]>
@simondeziel simondeziel force-pushed the system-tests-misc-improvements branch from a7e9f59 to 5c536bd Compare April 24, 2024 18:31
@simondeziel simondeziel marked this pull request as ready for review April 24, 2024 22:55
@simondeziel
Copy link
Member Author

@masnax I know we agreed one splitting the interactive one even more but since I got a full green run, is that OK if we merge it as-is?

Copy link
Contributor

@masnax masnax left a comment

Choose a reason for hiding this comment

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

Yay! All green :)

I'll have another PR coming up in a sec that will reorder the runners and log lxc info --resources plus a few other workarounds for issues I faced locally.

@masnax masnax merged commit 5864751 into canonical:main Apr 24, 2024
15 checks passed
@simondeziel simondeziel deleted the system-tests-misc-improvements branch April 25, 2024 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants