From 33363e4ef0134c25b7b4e9583b3e2ce9532b007e Mon Sep 17 00:00:00 2001 From: RZFeeser Date: Tue, 12 Dec 2023 16:56:08 -0500 Subject: [PATCH 1/3] Create requirements.yml best practice is to have a requirements.yml describing the collection(s) to be installed, in this case, just paloaltonetworks.panos --- requirements.yml | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 requirements.yml diff --git a/requirements.yml b/requirements.yml new file mode 100644 index 000000000..3c69986fd --- /dev/null +++ b/requirements.yml @@ -0,0 +1,3 @@ +--- +collections: + - paloaltonetworks.panos From c7717fc7c8226e9fc4e3c0749327395352922b44 Mon Sep 17 00:00:00 2001 From: RZFeeser Date: Sat, 16 Dec 2023 19:54:37 -0500 Subject: [PATCH 2/3] Update requirements.txt attempting to use the `requirements.txt` would fail as it did not have entries for `cryptography` and `pyopenssl`. pip is clear that in hash mode all installs must include hashes. As `panos-upgrade-assurance` has dependencies on these two files, they should be included. --- requirements.txt | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/requirements.txt b/requirements.txt index bb86da67f..3f1d222f8 100644 --- a/requirements.txt +++ b/requirements.txt @@ -9,4 +9,8 @@ xmltodict==0.12.0 ; python_version >= "3.8" and python_version < "4.0" \ --hash=sha256:8bbcb45cc982f48b2ca8fe7e7827c5d792f217ecf1792626f808bf41c3b86051 panos-upgrade-assurance==0.3.0 ; python_version >= "3.8" and python_version < "4.0" \ --hash=sha256:0e5bb1a1f5bf317086fe118fb80f5d03c1aa1426c8ecdcda657983f392accc63 \ - --hash=sha256:88bfc470f9bfa09ed22c919f8328782868c649d44e97a06c8d29c32e0211c257 \ No newline at end of file + --hash=sha256:88bfc470f9bfa09ed22c919f8328782868c649d44e97a06c8d29c32e0211c257 +pyopenssl==23.3.0 ; python_version > "3.8" and python_version < "4.0" \ + --hash=sha256:6756834481d9ed5470f4a9393455154bc92fe7a64b7bc6ee2c804e78c52099b2 +cryptography==41.0.7 ; python_version > "3.8" and python_version < "4.0" \ + --hash=sha256:43f2552a2378b44869fe8827aa19e69512e3245a219104438692385b0ee119d1 From a2008ce2b48785ee10c5c136a72f19f4e53ced9d Mon Sep 17 00:00:00 2001 From: RZFeeser Date: Sat, 16 Dec 2023 19:58:35 -0500 Subject: [PATCH 3/3] Update README.md Updating README.md to instruct how to utilize `requirements.txt` --- README.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/README.md b/README.md index 070d87c01..41d1f069f 100644 --- a/README.md +++ b/README.md @@ -38,6 +38,12 @@ Install this collection using the Ansible Galaxy CLI: ansible-galaxy collection install paloaltonetworks.panos ``` +Install the collection's python dependencies using the Python pip installer and the `requirements.txt` file found in the root of this repository: + +```bash +pip install -r requirements.txt +``` + Usage -----