Fix mvn warnings

[vishesh92]

Description

This PR fixes the below warnings which appear while building cloudstack.

[WARNING] The following plugins are not marked as thread-safe in Apache CloudStack Client UI:                                                                                                 
[WARNING]   ru.concerteza.buildnumber:maven-jgit-buildnumber-plugin:1.2.6                                                                                                                     
[WARNING]                                                                                                                                                                                     
[WARNING] Enable debug to see precisely which goals are not marked as thread-safe. 

----

[WARNING] The following plugins are not marked @threadSafe in Apache CloudStack Developer Mode:
[WARNING] org.codehaus.mojo:properties-maven-plugin:1.0-alpha-2
[WARNING] Enable debug to see more precisely which goals are not marked @threadSafe.

----

[WARNING] While downloading mysql:mysql-connector-java:8.0.33
  This artifact has been relocated to com.mysql:mysql-connector-j:8.0.33.
  MySQL Connector/J artifacts moved to reverse-DNS compliant Maven 2+ coordinates.

---

[INFO] --- exec:3.2.0:exec (package) @ cloud-marvin ---                                                                                                                                       
[WARNING] [stderr] /home/vishesh/lab/cloudstack/tools/marvin/mvn-setup.py:36: SyntaxWarning: invalid escape sequence '\s'                                                                     
[WARNING] [stderr]   needle = '\nVERSION\s*=\s*[\'"][^\'"]*[\'"]'                                                                                                                             
[WARNING] [stderr] Warning: 'platforms' should be a list, got type 'tuple'                                                                                                                    
[INFO] [stdout] running sdist                                                                                                                                                                 
[INFO] [stdout] running egg_info                                                                                                                                                              
[INFO] [stdout] writing Marvin.egg-info/PKG-INFO                                                                                                                                              
[INFO] [stdout] writing dependency_links to Marvin.egg-info/dependency_links.txt                                                                                                              
[INFO] [stdout] writing entry points to Marvin.egg-info/entry_points.txt                                                                                                                      
[INFO] [stdout] writing requirements to Marvin.egg-info/requires.txt
[INFO] [stdout] writing top-level names to Marvin.egg-info/top_level.txt
[INFO] [stdout] reading manifest template 'MANIFEST.in'
[WARNING] [stderr] warning: no files found matching '*.txt' under directory 'docs'
[INFO] [stdout] adding license file 'LICENSE.txt'
[INFO] [stdout] writing manifest file 'Marvin.egg-info/SOURCES.txt'
[INFO] [stdout] running check
[WARNING] [stderr] warning: sdist: standard file not found: should have one of README, README.rst, README.txt, README.md
[WARNING] [stderr] 
[INFO] [stdout] creating marvin-

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?

[vishesh92]

@blueorangutan package

[blueorangutan]

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

[Copilot]

Pull Request Overview

This PR resolves various build-time warnings by updating plugin versions, correcting Python setup scripts, and aligning Maven coordinates for MySQL Connector/J. Key changes include:

• Adjusted Python packaging scripts (setup.py, mvn-setup.py) to fix type and regex warnings.
• Added a minimal README.md for the Marvin client.
• Updated Maven plugin versions (properties-maven-plugin, buildnumber-maven-plugin) and migrated MySQL Connector to com.mysql:mysql-connector-j.

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tools/marvin/setup.py	Switched platforms from tuple to list to satisfy setuptools.
tools/marvin/mvn-setup.py	Made regex literal raw string to avoid invalid escape warning.
tools/marvin/README.md	Added basic project README.
tools/devcloud4/pom.xml	Bumped properties-maven-plugin to version 1.2.0.
tools/devcloud-kvm/pom.xml	Bumped properties-maven-plugin to version 1.2.0.
pom.xml	Rebased MySQL connector to com.mysql:mysql-connector-j and removed duplicate.
plugins/network-elements/tungsten/pom.xml	Updated MySQL connector coordinates for test scope.
plugins/network-elements/globodns/pom.xml	Updated MySQL connector coordinates for test scope.
framework/db/pom.xml	Updated MySQL connector coordinates.
engine/storage/snapshot/pom.xml	Updated MySQL connector coordinates.
developer/pom.xml	Bumped properties-maven-plugin to version 1.2.0.
client/pom.xml	Switched to buildnumber-maven-plugin, updated goals and manifest entries.

Comments suppressed due to low confidence (2)

client/pom.xml:681

• After switching to buildnumber-maven-plugin and using its create goal, verify that ${buildNumber} and ${scmBranch} are actually populated. The plugin may require explicit <propertyName> or <branchName> configuration to match these manifest entries.

<goal>create</goal>

tools/marvin/README.md:1

• [nitpick] Consider expanding this README with basic installation or quick-start instructions (e.g., dependencies, setup commands) to help new users onboard more quickly.

# Marvin

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 17.88%. Comparing base (9d523cb) to head (ba00498).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #10909      +/-   ##
============================================
- Coverage     17.88%   17.88%   -0.01%     
+ Complexity    16075    16074       -1     
============================================
  Files          5936     5936              
  Lines        532610   532610              
  Branches      65147    65147              
============================================
- Hits          95265    95242      -23     
- Misses       426675   426701      +26     
+ Partials      10670    10667       -3     

Flag	Coverage Δ
uitests	3.67% <ø> (ø)
unittests	18.98% <ø> (-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]

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

[vishesh92]

@blueorangutan test

[blueorangutan]

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

[blueorangutan]

[SF] Trillian test result (tid-13376)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 61684 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10909-t13376-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_create_pvlan_network	Error	0.17	test_pvlan.py

[DaanHoogland]

clgtm, but the tests should all pass. Is this applicable to older branches as well?

[vishesh92]

clgtm, but the tests should all pass. Is this applicable to older branches as well?

I doubt the test failure is due to my changes. I think this will be applicable to older branches as well, but I haven't verified.

[weizhouapache]

clgtm, but the tests should all pass. Is this applicable to older branches as well?

@DaanHoogland @vishesh92
I think it would be better to keep the target as main/4.21

[DaanHoogland]

@blueorangutan package

[blueorangutan]

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

[blueorangutan]

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

[weizhouapache]

code lgtm

[DaanHoogland]

this should have no run-time impact, but running regression tests to be sure anyway.

[DaanHoogland]

@blueorangutan test

[blueorangutan]

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

[DaanHoogland]

clgtm

[blueorangutan]

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

Test	Result	Time (s)	Test File
test_03_deploy_and_scale_kubernetes_cluster	Failure	26.82	test_kubernetes_clusters.py
test_01_create_template	Error	22.69	test_templates.py
test_CreateTemplateWithDuplicateName	Error	30.94	test_templates.py
test_02_create_template_with_checksum_sha1	Error	65.58	test_templates.py
test_03_create_template_with_checksum_sha256	Error	65.56	test_templates.py
test_04_create_template_with_checksum_md5	Error	65.56	test_templates.py
test_01_template_usage	Error	176.47	test_usage.py

[github-actions]

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

[vishesh92]

@blueorangutan package

[blueorangutan]

@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

[blueorangutan]

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

[harikrishna-patnala]

LGTM. Did not observed any warnings during build