Skip to content

Conversation

@weizhouapache
Copy link
Member

@weizhouapache weizhouapache commented Feb 9, 2023

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-cyborg boring-cyborg bot added component:integration-test Python Warning... Python code Ahead! labels Feb 9, 2023
@weizhouapache
Copy link
Member Author

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

sonarqubecloud bot commented Feb 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

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

@codecov
Copy link

codecov bot commented Feb 9, 2023

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
Contributor

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

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

Labels

component:integration-test Python Warning... Python code Ahead!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants