Fix ISO tagging tests: wait for ISO download before tagging operations

[awen11123]

🤖

Description

The create_iso() helper in test/integration/component/test_tags.py calls Iso.create() and immediately returns, but the ISO may still be downloading when tagging tests proceed. This causes tests 7, 16, and 17 to fail intermittently when the ISO upload hasn't completed.

This PR adds a call to iso.download() after creation. The download() method (on the Iso class in marvin) already has a built-in retry loop (300 retries × 5s interval) that waits for the ISO to reach Successfully Installed and isready state.

If the download fails after the retry period, the test is skipped with a clear message rather than failing with an opaque upload error.

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Fixes

Fixes #7041

Affected tests

• test_07_iso_tag
• test_16_query_tags_other_account
• test_17_query_tags_admin_account

[boring-cyborg]

Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
Here are some useful points:

• In case of a new feature add useful documentation (raise doc PR at https://github.com/apache/cloudstack-documentation)
• Be patient and persistent. It might take some time to get a review or get the final approval from the committers.
• Pay attention to the quality of your code, ensure tests are passing and your PR doesn't have conflicts.
• Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Issues, Mailing list and Slack.
• Be sure to read the CloudStack Coding Conventions.
  Apache CloudStack is a community-driven project and together we are making it better 🚀.
  In case of doubts contact the developers at:
  Mailing List: dev@cloudstack.apache.org (https://cloudstack.apache.org/mailing-lists.html)
  Slack: https://apachecloudstack.slack.com/

[DaanHoogland]

code looks good, just wonder if there is a chance for retry?
also would list not give an indication of FAIL?

[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 18264

[DaanHoogland]

@blueorangutan test

[blueorangutan]

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

[blueorangutan]

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

Test	Result	Time (s)	Test File