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

github: Extended testing #11846

Merged
merged 7 commits into from
Jun 19, 2023
Merged

Conversation

tomponline
Copy link
Member

@tomponline tomponline commented Jun 19, 2023

  • Adds latest Go version test for dir driver.
  • Adds unit testing as part of "Code tests" (formerly Static analysis) workflow.
  • Renames "Unit tests (client)" to "Client tests".
  • Adds support for opportunistic Go dependency build caching (seems to work only for later Go versions).
  • Attempts fix of intermittent BTRFS test failure.

@tomponline tomponline marked this pull request as draft June 19, 2023 09:33
@tomponline tomponline self-assigned this Jun 19, 2023
@tomponline tomponline force-pushed the tp-gh-setup-go branch 2 times, most recently from baf63d5 to dcd01c8 Compare June 19, 2023 09:48
@tomponline tomponline changed the title github: Switch to setup-go v4 github: Switch to setup-go v4 and add latest stable Go version system tests Jun 19, 2023
@tomponline tomponline marked this pull request as ready for review June 19, 2023 09:49
@tomponline tomponline force-pushed the tp-gh-setup-go branch 6 times, most recently from b1d3de4 to 5f02e25 Compare June 19, 2023 11:21
@tomponline tomponline changed the title github: Switch to setup-go v4 and add latest stable Go version system tests github: Improvements to tests Jun 19, 2023
@tomponline tomponline changed the title github: Improvements to tests github: Extended testing Jun 19, 2023
Allows opportunistic usage of dependency caching.

Signed-off-by: Thomas Parrott <[email protected]>
To allow caching to work if available.

Signed-off-by: Thomas Parrott <[email protected]>
Try and fix intermittent log matching issue.

Signed-off-by: Thomas Parrott <[email protected]>
Copy link
Member

@simondeziel simondeziel left a comment

Choose a reason for hiding this comment

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

LGTM. Looking at previous test runs, btrfs usually is faster in either standalone, or cluster or sometimes both and by ~2-3 minutes. When btrfs isn't faster than dir, it usually is within a minute of dir.

That's the "CI minutes" aspect of the storage driver, maybe you picked dir for a specific reason.

@tomponline
Copy link
Member Author

Yes there isn't much in it. I picked dir because its what we do now, and is the least "exotic" of the storage drivers (also its the default one used if not specified).

@tomponline tomponline merged commit 9350731 into canonical:master Jun 19, 2023
@tomponline tomponline deleted the tp-gh-setup-go branch June 19, 2023 13:54
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.

3 participants