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

Support EL10 as Management Server and KVM host #10496

Open
wants to merge 3 commits into
base: 4.20
Choose a base branch
from

Conversation

rohityadavcloud
Copy link
Member

This adds support for Fedora 40 and (upcoming) EL10 distro to be used as mgmt/usage server, mysql/nfs & KVM host. Python3 version has changed to 3.12.9 which isn't automatically determining the python-path.

Fixes #10494

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

How Has This Been Tested?

See #10494 (comment)

This adds support for Fedora 40 and (upcoming) EL10 distro to be used
as mgmt/usage server, mysql/nfs & KVM host. Python3 version has changed
to 3.12.9 which isn't automatically determining the python-path.

Signed-off-by: Rohit Yadav <[email protected]>
Need to discuss/check if we can skip this code. Where/how is cgroup
setup used with KVM agent.

Signed-off-by: Rohit Yadav <[email protected]>
@rohityadavcloud rohityadavcloud added this to the 4.20.1 milestone Mar 4, 2025
@boring-cyborg boring-cyborg bot added component:agent Python Warning... Python code Ahead! labels Mar 4, 2025
@rohityadavcloud
Copy link
Member Author

@blueorangutan package

@blueorangutan
Copy link

@rohityadavcloud 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.

Copy link

codecov bot commented Mar 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.00%. Comparing base (b92fd17) to head (4ca7058).
Report is 13 commits behind head on 4.20.

Additional details and impacted files
@@             Coverage Diff              @@
##               4.20   #10496      +/-   ##
============================================
+ Coverage     15.98%   16.00%   +0.02%     
- Complexity    13086    13107      +21     
============================================
  Files          5650     5651       +1     
  Lines        495756   495841      +85     
  Branches      60018    60044      +26     
============================================
+ Hits          79261    79379     +118     
+ Misses       407641   407599      -42     
- Partials       8854     8863       +9     
Flag Coverage Δ
uitests 4.00% <ø> (-0.01%) ⬇️
unittests 16.85% <ø> (+0.02%) ⬆️

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.

@blueorangutan
Copy link

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

@rohityadavcloud rohityadavcloud linked an issue Mar 4, 2025 that may be closed by this pull request
6 tasks
@rohityadavcloud rohityadavcloud marked this pull request as draft March 4, 2025 10:52
@rohityadavcloud
Copy link
Member Author

@blueorangutan test

@blueorangutan
Copy link

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

@blueorangutan
Copy link

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

Test Result Time (s) Test File
test_06_purge_expunged_vm_background_task Failure 419.45 test_purge_expunged_vms.py

Fixes issue for Fedora, it was running old EL6 hooks which isn't
applicable for modern Fedora version that are closer to EL8/9/10

Signed-off-by: Rohit Yadav <[email protected]>
@rohityadavcloud rohityadavcloud marked this pull request as ready for review March 5, 2025 10:12
@rohityadavcloud
Copy link
Member Author

This is ready for review now - cc @DaanHoogland @weizhouapache @Pearl1594 @shwstppr
@blueorangutan package

@blueorangutan
Copy link

@rohityadavcloud 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

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

@rohityadavcloud
Copy link
Member Author

@blueorangutan test

@blueorangutan
Copy link

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

@blueorangutan
Copy link

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

Test Result Time (s) Test File
ContextSuite context=TestSharedNetworkWithConfigDrive>:setup Error 1519.56 test_network.py
test_06_purge_expunged_vm_background_task Failure 394.20 test_purge_expunged_vms.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:agent Python Warning... Python code Ahead!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support EL10
3 participants