Skip to content

Support XenServer 8.4 / XCP 8.3 - make scripts python3 compatible - #10684

Merged
DaanHoogland merged 36 commits into
4.20from
support-xen84-py3
May 13, 2025
Merged

Support XenServer 8.4 / XCP 8.3 - make scripts python3 compatible#10684
DaanHoogland merged 36 commits into
4.20from
support-xen84-py3

Conversation

@Pearl1594

@Pearl1594Pearl1594 commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

Description

This PR is just to test #10483

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)
  • build/CI
  • test (unit or integration test code)

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?

How did you try to break this feature and the system with this change?

yadvrand others added 6 commits February 28, 2025 11:07
This aims to add support for XenServer 8.4 and XCP-ng 8.3.
Signed-off-by: Rohit Yadav <rohit@yadav.cloud>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@boring-cyborgboring-cyborgBot added component:XenServer Python Warning... Python code Ahead! labels Apr 9, 2025
@Pearl1594

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@Pearl1594 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@codecov

codecovBot commented Apr 9, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 203 lines in your changes missing coverage. Please review.

Project coverage is 16.12%. Comparing base (c1923a2) to head (0c8a738).
Report is 23 commits behind head on 4.20.

Files with missing linesPatch %Lines
.../main/java/com/xensource/xenapi/ConnectionNew.java0.00%156 Missing ⚠️
...rce/wrapper/xenbase/CitrixStartCommandWrapper.java0.00%18 Missing ⚠️
...cloudstack/storage/template/UploadManagerImpl.java0.00%10 Missing ⚠️
...isor/xenserver/discoverer/XcpServerDiscoverer.java0.00%4 Missing ⚠️
...ervisor/xenserver/resource/CitrixResourceBase.java0.00%4 Missing ⚠️
...rvisor/xenserver/resource/XcpServer83Resource.java0.00%4 Missing ⚠️
...rvisor/xenserver/resource/Xenserver84Resource.java0.00%4 Missing ⚠️
...or/xenserver/resource/XenServerConnectionPool.java0.00%3 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## 4.20 #10684 +/- ##
============================================
- Coverage 16.13% 16.12% -0.01% - Complexity 13219 13220 +1 
============================================
Files 5651 5654 +3 Lines 496740 496988 +248 Branches 60183 60238 +55 ============================================
+ Hits 80136 80154 +18 - Misses 407680 407910 +230 
Partials 8924 8924 
FlagCoverage Δ
uitests4.00% <ø> (-0.01%)⬇️
unittests16.97% <0.00%> (-0.01%)⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@blueorangutan

Copy link
Copy Markdown

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 12999

@weizhouapache

Copy link
Copy Markdown
Member

@blueorangutan test ol8 xenserver-84 keepEnv

@blueorangutan

Copy link
Copy Markdown

@weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + xenserver-84) has been kicked to run smoke tests

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian Build Failed (tid-12931)

@DaanHooglandDaanHoogland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lint issues

Comment threadscripts/vm/hypervisor/xenserver/xenserver84/cloud-plugin-storage Outdated
Comment threadscripts/vm/hypervisor/xenserver/xenserver84/vmops Outdated
Comment threadscripts/vm/hypervisor/xenserver/xenserver84/vmops Outdated
Comment threadscripts/vm/hypervisor/xenserver/xenserver84/vmops Outdated
@yadvr

Copy link
Copy Markdown
Member

Good to see this, @Pearl1594 can we also add support for XCP-ng 8.3 (& related files as needed)? cc @weizhouapache@harikrishna-patnala

@Pearl1594

Copy link
Copy Markdown
ContributorAuthor

yes @rohityadavcloud . Working with @harikrishna-patnala towards that.

Comment threadscripts/vm/hypervisor/xenserver/xenserver84/vmops Outdated
@Pearl1594

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@Pearl1594 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

@Pearl1594 a [SL] Trillian-Jenkins test job (ol8 mgmt + xcpng83) has been kicked to run smoke tests

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13206)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 56594 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13206-kvm-ol8.zip
Smoke tests completed. 141 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13207)
Environment: kvm-ubuntu22 (x2), Advanced Networking with Mgmt server u22
Total time taken: 61798 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13207-kvm-ubuntu22.zip
Smoke tests completed. 139 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
test_oobm_multiple_mgmt_server_ownershipFailure30.88test_outofbandmanagement.py
test_02_list_cpvm_vmFailure0.04test_ssvm.py
test_04_cpvm_internalsFailure0.05test_ssvm.py

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13208)
Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 67120 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13208-vmware-70u3.zip
Smoke tests completed. 138 look OK, 3 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
test_01_deployVMInSharedNetworkError181.12test_network.py
test_02_restore_vm_with_disk_offeringError63.66test_restore_vm.py
test_03_restore_vm_with_disk_offering_custom_sizeError59.47test_restore_vm.py
test_02_restore_vm_strict_tags_failureError55.85test_vm_strict_host_tags.py

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13210)
Environment: xenserver-84 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 67815 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13210-xenserver-84.zip
Smoke tests completed. 140 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
ContextSuite context=TestSharedNetworkWithConfigDrive>:setupError7.50test_network.py

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13211)
Environment: xcpng83 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 74220 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13211-xcpng83.zip
Smoke tests completed. 140 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
ContextSuite context=TestSharedNetworkWithConfigDrive>:setupError7.55test_network.py

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13209)
Environment: xcpng82 (x2), Advanced Networking with Mgmt server ol9
Total time taken: 82436 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13209-xcpng82.zip
Smoke tests completed. 139 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
ContextSuite context=TestSharedNetworkWithConfigDrive>:setupError10.60test_network.py
test_10_list_volumesFailure371.05test_volumes.py

@Pearl1594

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@Pearl1594 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13297

@yadvryadvr closed this May 9, 2025
@github-project-automationgithub-project-automationBot moved this from In Progress to Done in ACS 4.20.1May 9, 2025
@yadvryadvr reopened this May 9, 2025
@yadvr

yadvr commented May 9, 2025

Copy link
Copy Markdown
Member

@blueorangutan test ol8 xcpng83

@blueorangutan

Copy link
Copy Markdown

@rohityadavcloud a [SL] Trillian-Jenkins test job (ol8 mgmt + xcpng83) has been kicked to run smoke tests

@harikrishna-patnalaharikrishna-patnala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

changes LGTM

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate FailedQuality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 40%)
C Maintainability Rating on New Code (required ≥ B)
C Reliability Rating on New Code (required ≥ B)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Pearl1594Pearl1594 moved this from Done to In Progress in ACS 4.20.1May 9, 2025
@blueorangutan

Copy link
Copy Markdown

[SF] Trillian test result (tid-13242)
Environment: xcpng83 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 75161 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10684-t13242-xcpng83.zip
Smoke tests completed. 140 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

TestResultTime (s)Test File
ContextSuite context=TestSharedNetworkWithConfigDrive>:setupError1532.02test_network.py

@kiranchavalakiranchavala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, Tested with Xenserver 8.4 and XCPng 8.3

Test Case ExecutionResult
Start a VM from a templatePass
Remove/add an xenserver 8.4 and xcpng 8.3 hostPass
Volume snapshotsPass
Move a running VM to another hostPass
Create / restore VM snapshotPass
Register ISO image and create a VM from it.Pass
Standard VM operations – start/stop/rebootPass
Remove / add primary storagePass
check the console of vm'sPass
Check the volume operations on a vmPass
Check the guest os catagories and mappingsPass

@blueorangutan

Copy link
Copy Markdown

[SF] Trillian Build Failed (tid-13286)

@DaanHoogland
DaanHoogland merged commit 0648d00 into 4.20May 13, 2025
@github-project-automationgithub-project-automationBot moved this from In Progress to Done in ACS 4.20.1May 13, 2025
@DaanHoogland
DaanHoogland deleted the support-xen84-py3 branch May 13, 2025 10:35
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

10 participants

@Pearl1594@blueorangutan@weizhouapache@yadvr@olivierlambert@bleader@kiranchavala@DaanHoogland@harikrishna-patnala@borisstoyanov