ISSUE ID : 7729 skip uploading log files

[jonatluu]

Motivation

Packages location has logs being uploaded to it. Logs should be blocked from being uploaded there like how txt files are currently being blocked

therock-ci-artifacts.s3.amazonaws.com/33013518162-linux/packages/deb/index.html wrong because logs are being uploaded where packages should be

therock-ci-artifacts.s3.amazonaws.com/33013518162-linux/logs/packaging/index.html correct location for logs

Fixes: #7729

Technical Details

skip log files when uploading. Change upload_to_s3 function

Test Plan

Test Result

Submission Checklist

• Look over the contributing guidelines at https://github.com/ROCm/TheRock/blob/main/CONTRIBUTING.md.

[therock-pr-bot]

✅ All Checks Passed — Ready for Review

Check	Status	Details
📝 PR Description	✅ Pass	—
⛔ Forbidden Files	✅ Pass	—
🧪 Unit Test	⚠️ Warning	Error: Source/code files changed without an accompanying unit test. Expected: add at least one test file named like test_<name>.py / test_<name>.cpp (or <name>_test.*). Current: code file(s) changed: build_tools/packaging/linux/upload_package_repo.py; no test file found
🔎 pre-commit	✅ Pass	—
🚫 Draft PR	🔜 To Be Enabled	—
🚩 Feature Flag	🔜 To Be Enabled	—
📊 Code Coverage	🔜 To Be Enabled	—
🤖 therock-pr-bot	✅ Pass	—

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

[therock-pr-bot]

🎉 All checks passed! This PR is ready for review.

[raramakr]

@jonatluu Pls create a corresponing Github issue and fix the pr-bot errors

[ScottTodd]

You could also/instead change the log directory to not be nested under Path(config.dest_dir) and mixed with package outputs:

• TheRock/build_tools/packaging/linux/build_package.py

  Lines 675 to 676 in 1fb7810

  	logs_dir = Path(config.dest_dir) / "logs"
  	logs_dir.mkdir(parents=True, exist_ok=True)

• TheRock/build_tools/packaging/linux/upload_package_repo.py

  Lines 344 to 347 in 1fb7810

  	log_dir = package_dir / "logs"
  	if not log_dir.is_dir():
  	print(f"[INFO] Log directory {log_dir} not found. Skipping log upload.")
  	return None

Or exclude logs/ instead of just *.log if there are other log files in the directory today or some time in the future