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

[DPE-4977] Run tests against juju 3.6 on a nightly schedule #601

Merged
merged 13 commits into from
Sep 11, 2024

Conversation

shayancanonical
Copy link
Contributor

Issue

  1. We are not running our integration tests against juju 3.6
  2. We are not running our integration tests against the latest lxd version

Solution

  1. Run integration tests on juju 3.6 on a nightly schedule only
  2. Do not specify lxd-snap-channel to always run tests against the latest lxd version

Copy link

codecov bot commented Sep 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.81%. Comparing base (f9032b8) to head (55fe81c).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #601   +/-   ##
=======================================
  Coverage   70.81%   70.81%           
=======================================
  Files          12       12           
  Lines        3039     3039           
  Branches      537      537           
=======================================
  Hits         2152     2152           
  Misses        771      771           
  Partials      116      116           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@taurus-forever taurus-forever left a comment

Choose a reason for hiding this comment

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

👍

@shayancanonical shayancanonical merged commit 01cfd3e into main Sep 11, 2024
91 checks passed
@shayancanonical shayancanonical deleted the feature/juju_3_6_tests branch September 11, 2024 16:07
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.

5 participants