Skip to content

smoke test: fix test_vm_deployment_planner - #7183

Merged
DaanHoogland merged 2 commits into
apache:mainfrom
weizhouapache:4.18-fix-test_vm_deployment_planner
Feb 9, 2023
Merged

smoke test: fix test_vm_deployment_planner#7183
DaanHoogland merged 2 commits into
apache:mainfrom
weizhouapache:4.18-fix-test_vm_deployment_planner

Conversation

@weizhouapache

@weizhouapacheweizhouapache commented Feb 9, 2023

Copy link
Copy Markdown
Member

Description

This PR fixes test failure with test_vm_deployment_planner

2023-02-09T08:48:38.5713780Z ==== Marvin Init Successful ====
2023-02-09T08:48:38.5714093Z === TestName: test_01_deploy_vm_on_specific_host | Status : SUCCESS ===
2023-02-09T08:48:38.5714287Z 2023-02-09T08:48:38.5714459Z === TestName: test_02_deploy_vm_on_specific_cluster | Status : EXCEPTION ===
2023-02-09T08:48:38.5714653Z 2023-02-09T08:48:38.5714813Z === TestName: test_03_deploy_vm_on_specific_pod | Status : EXCEPTION ===
2023-02-09T08:48:38.5715003Z 2023-02-09T08:48:38.5715179Z === TestName: test_04_deploy_vm_on_host_override_pod_and_cluster | Status : EXCEPTION ===
2023-02-09T08:48:38.5715383Z 2023-02-09T08:48:38.5715550Z === TestName: test_05_deploy_vm_on_cluster_override_pod | Status : EXCEPTION ===
2023-02-09T08:48:38.5715742Z 2023-02-09T08:48:38.5715931Z === Final results are now copied to: /tmp/MarvinLogs/test_vm_deployment_planner_5A094E ===

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@boring-cyborgboring-cyborgBot added component:integration-test Python Warning... Python code Ahead! labels Feb 9, 2023
Comment threadtest/integration/smoke/test_vm_deployment_planner.py Outdated
@weizhouapache

Copy link
Copy Markdown
MemberAuthor

simulator CI looks good now

2023-02-09T12:16:56.8399362Z ==== Marvin Init Started ====
2023-02-09T12:16:56.8399653Z 2023-02-09T12:16:56.8399865Z === Marvin Parse Config Successful ===
2023-02-09T12:16:56.8400126Z 2023-02-09T12:16:56.8400326Z === Marvin Setting TestData Successful===
2023-02-09T12:16:56.8400571Z 2023-02-09T12:16:56.8400857Z ==== Log Folder Path: /tmp/MarvinLogs/Feb_09_2023_12_16_03_TP0NMX All logs will be available here ====
2023-02-09T12:16:56.8401167Z 2023-02-09T12:16:56.8401355Z === Marvin Init Logging Successful===
2023-02-09T12:16:56.8401607Z 2023-02-09T12:16:56.8401791Z ==== Marvin Init Successful ====
2023-02-09T12:16:56.8402171Z === TestName: test_01_deploy_vm_on_specific_host | Status : SUCCESS ===
2023-02-09T12:16:56.8402445Z 2023-02-09T12:16:56.8402686Z === TestName: test_02_deploy_vm_on_specific_cluster | Status : SUCCESS ===
2023-02-09T12:16:56.8402961Z 2023-02-09T12:16:56.8403192Z === TestName: test_03_deploy_vm_on_specific_pod | Status : SUCCESS ===
2023-02-09T12:16:56.8403459Z 2023-02-09T12:16:56.8403716Z === TestName: test_04_deploy_vm_on_host_override_pod_and_cluster | Status : SUCCESS ===
2023-02-09T12:16:56.8404003Z 2023-02-09T12:16:56.8404427Z === TestName: test_05_deploy_vm_on_cluster_override_pod | Status : SUCCESS ===
2023-02-09T12:16:56.8404740Z 2023-02-09T12:16:56.8405008Z === Final results are now copied to: /tmp/MarvinLogs/test_vm_deployment_planner_UU7867 ===
2023-02-09T12:16:56.9048923Z 2023-02-09T12:16:56.9049346Z real	0m54.252s
2023-02-09T12:16:56.9049691Z user	0m0.997s
2023-02-09T12:16:56.9050447Z sys	0m0.068s

@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed! Quality Gate passed

BugA0 Bugs
VulnerabilityA0 Vulnerabilities
Security HotspotA0 Security Hotspots
Code SmellA0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov

codecovBot commented Feb 9, 2023

Copy link
Copy Markdown

Codecov Report

Merging #7183 (8f27dbf) into main (1b89a22) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## main #7183 +/- ##
=========================================
Coverage 12.63% 12.63% Complexity 8613 8613 =========================================
Files 2711 2711 Lines 255752 255752 Branches 39873 39873 =========================================
Hits 32325 32325 Misses 219320 219320 Partials 4107 4107 

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@DaanHoogland

Copy link
Copy Markdown
Contributor

test code only proven by ci code looks good; merging.

@DaanHooglandDaanHoogland added this to the 4.18.0.0 milestone Feb 9, 2023
@DaanHoogland
DaanHoogland merged commit f3e8f39 into apache:mainFeb 9, 2023
@weizhouapache
weizhouapache deleted the 4.18-fix-test_vm_deployment_planner branch February 13, 2023 09:53
@DaanHooglandDaanHoogland mentioned this pull request Nov 14, 2023
37 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:integration-testPythonWarning... Python code Ahead!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@weizhouapache@DaanHoogland