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

feat: Add option to run smoke tests with fake IMDS servers. #3380

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

jirihnidek
Copy link
Contributor

  • This change allow to run smoke tests of cloud provider with custom facts. Examples of custom facts are described in comments for each cloud provider

* This change allow to run smoke tests of cloud provider
  with custom facts. Examples of custom facts are described
  in comments for each cloud provider
@cnsnyder cnsnyder requested review from a team and m-horky and removed request for a team February 2, 2024 11:05
Copy link

github-actions bot commented Feb 2, 2024

Coverage

Coverage (computed on Fedora latest) •
FileStmtsMissCoverMissing
cloud_what/providers
   aws.py1854874%88, 103, 111, 148–149, 151–154, 156, 165, 168–169, 199, 203–204, 213–214, 253, 265, 324, 385–387, 389–390, 392–396, 398–399, 408–412, 414–418, 420–423, 429
   azure.py1404468%108, 134, 138, 151–152, 179, 235–237, 261–262, 264, 266–267, 269–273, 275–276, 283–287, 289–295, 298–299, 302, 305–311, 317
   gcp.py1503477%131, 146, 150, 225–226, 241, 247, 254, 262, 272, 351–352, 354, 356–357, 359–363, 365–366, 370–375, 377–378, 380–382, 388
TOTAL18227461974% 

Tests Skipped Failures Errors Time
2640 14 💤 0 ❌ 0 🔥 40.776s ⏱️

Copy link
Contributor

@m-horky m-horky left a comment

Choose a reason for hiding this comment

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

LGTM

@m-horky m-horky merged commit b6fad11 into main Jun 5, 2024
17 checks passed
@m-horky m-horky deleted the jhnidek/fix_cloud_what_smoke_tests branch June 5, 2024 08:41
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