[PR #134] [MERGED] Add Files tools #190

New issue

Closed

opened 2026-02-27 15:46:50 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 15:46:50 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/proxmoxer/proxmoxer/pull/134
Author: @jhollowe
Created: 2/20/2023
Status: ✅ Merged
Merged: 4/1/2023
Merged by: @jhollowe

Base: develop ← Head: tools-files

📝 Commits (7)

9c69d8b Add start of Files tools (checksum gather)
d6af694 Adjust Tasks polling_interval
3bd428d Complete Files file and fix pre-commit
631564a Add tests for Files tools
e9c09c4 Set devcontainer to oldest supported py version (3.7)
958e63f Add upload function to Files tools
d89c5ca Add tests for upload Files tools

📊 Changes

14 files changed (+900 additions, -17 deletions)

View changed files

📝 .devcontainer/devcontainer.json (+1 -1)
📝 .pre-commit-config.yaml (+1 -1)
📝 proxmoxer/backends/https.py (+5 -1)
📝 proxmoxer/tools/__init__.py (+1 -0)
➕ proxmoxer/tools/files.py (+276 -0)
📝 proxmoxer/tools/tasks.py (+2 -2)
📝 tests/api_mock.py (+97 -3)
➕ tests/files_mock.py (+127 -0)
📝 tests/test_command_base.py (+0 -1)
📝 tests/test_core.py (+0 -1)
📝 tests/test_https.py (+3 -5)
📝 tests/test_imports.py (+17 -0)
➕ tests/tools/test_files.py (+368 -0)
📝 tests/tools/test_tasks.py (+2 -2)

📄 Description

Add tooling to simplify uploading/downloading files to PVE.

Added Files tools module
Added download_file_to_storage to download a ISO/template from a URL
Added auto-discovery of checksum for file for common checksum file locations
Added upload_local_file_to_storage to calculate a checksum and upload a local ISO/template
Added testing for added code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/proxmoxer/proxmoxer/pull/134 **Author:** [@jhollowe](https://github.com/jhollowe) **Created:** 2/20/2023 **Status:** ✅ Merged **Merged:** 4/1/2023 **Merged by:** [@jhollowe](https://github.com/jhollowe) **Base:** `develop` ← **Head:** `tools-files` --- ### 📝 Commits (7) - [`9c69d8b`](https://github.com/proxmoxer/proxmoxer/commit/9c69d8bd293cff8976582095a40e21b65ad09e81) Add start of Files tools (checksum gather) - [`d6af694`](https://github.com/proxmoxer/proxmoxer/commit/d6af694b835a3e786aca9fec2a29d8bf76242707) Adjust Tasks polling_interval - [`3bd428d`](https://github.com/proxmoxer/proxmoxer/commit/3bd428d80ac1c67598a9391f9a9a63131b45a632) Complete Files file and fix pre-commit - [`631564a`](https://github.com/proxmoxer/proxmoxer/commit/631564a4170e6038f6a8453557438da9ca67c047) Add tests for Files tools - [`e9c09c4`](https://github.com/proxmoxer/proxmoxer/commit/e9c09c464cae910fe3c6e683d24a3db8b57d1f49) Set devcontainer to oldest supported py version (3.7) - [`958e63f`](https://github.com/proxmoxer/proxmoxer/commit/958e63f5f34fa84f048e2a425af6248ebf6c79e3) Add upload function to Files tools - [`d89c5ca`](https://github.com/proxmoxer/proxmoxer/commit/d89c5cafe14aee0c8dab9e481d7750d6a4cb9926) Add tests for upload Files tools ### 📊 Changes **14 files changed** (+900 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `.devcontainer/devcontainer.json` (+1 -1) 📝 `.pre-commit-config.yaml` (+1 -1) 📝 `proxmoxer/backends/https.py` (+5 -1) 📝 `proxmoxer/tools/__init__.py` (+1 -0) ➕ `proxmoxer/tools/files.py` (+276 -0) 📝 `proxmoxer/tools/tasks.py` (+2 -2) 📝 `tests/api_mock.py` (+97 -3) ➕ `tests/files_mock.py` (+127 -0) 📝 `tests/test_command_base.py` (+0 -1) 📝 `tests/test_core.py` (+0 -1) 📝 `tests/test_https.py` (+3 -5) 📝 `tests/test_imports.py` (+17 -0) ➕ `tests/tools/test_files.py` (+368 -0) 📝 `tests/tools/test_tasks.py` (+2 -2) </details> ### 📄 Description Add tooling to simplify uploading/downloading files to PVE. * Added Files tools module * Added `download_file_to_storage` to download a ISO/template from a URL * Added auto-discovery of checksum for file for common checksum file locations * Added `upload_local_file_to_storage` to calculate a checksum and upload a local ISO/template * Added testing for added code --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-27 15:46:50 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-02-27 15:46:53 +03:00

[PR #190] [MERGED] Fix commandline backends prefixing with host #205

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/proxmoxer#190

No description provided.

Rows
Columns