VeeamHub / veeam-ansible

Sample code for deploying and configuring various Veeam solutions using Ansible.
GNU Lesser General Public License v3.0
47 stars 18 forks source link

Dependencies need updated #29

Closed wdlarg01 closed 1 year ago

wdlarg01 commented 2 years ago

Describe the bug Attempted to update the collection using community-general-ansible-galaxy-install-module, which doesn't actually handle failures very well. I had previously installed the collection manually without incident, latest being v1.3.2. On researching the failure, it appears there are dependencies that have since been updated. According to your manifest: "dependencies": { "ansible.windows": "1.7.3", "community.windows": "1.7.0" Both of those are now at v1.9.0.

We've not yet actually used this collection, but is definitely in scope for us for the future. Thanks.

To Reproduce Here's the main update task:

If veeamhub.veeam is included in the requirements file, it fails due to unable to satisfy the dependencies.

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

carceneaux commented 2 years ago

Hello 😄

It's unclear whether you experienced an error or if you're asking for the dependencies to be updated. Can you please clarify?

Also, it's highly probable the latest versions of the collections you specified will work but it's untested at this moment. The versions currently listed were the most recent as of a few months ago. Software is always pushing forward... 😊

Feel free to test using the newer versions and I'll update the dependencies on my side once I've been able to test them.

wdlarg01 commented 2 years ago

The job run via the module mentioned was an error, due to the dependencies. I assume doing it manually would either downgrade or at least prompt for some action. I can understand you would not want to update the dependencies without testing, so will wait for that. As much as anything, just making sure that was a data point you were tracking. Staying current is always a challenge.

Thanks for the response.

From: Chris Arceneaux @.> Sent: Thursday, March 24, 2022 11:45 AM To: VeeamHub/veeam-ansible @.> Cc: Large,Dennis @.>; Author @.> Subject: Re: [VeeamHub/veeam-ansible] Dependencies need updated (Issue #29)

You don't often get email from @.*** Learn why this is importanthttp://aka.ms/LearnAboutSenderIdentification CAUTION: This email originated from outside of our organization. Do not click links, open attachments, or respond unless you recognize the sender's email address and know the contents are safe.

Hello 😄

It's unclear whether you experienced an error or if you're asking for the dependencies to be updated. Can you please clarify?

Also, it's highly probable the latest versions of the collections you specified will work but it's untested at this moment. The versions currently listed were the most recent as of a few months ago. Software is always pushing forward... 😊

Feel free to test using the newer versions and I'll update the dependencies on my side once I've been able to test them.

— Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FVeeamHub%2Fveeam-ansible%2Fissues%2F29%23issuecomment-1077772411&data=04%7C01%7Cdennis.large%40louisville.edu%7Cd6af1f96ed594a2f953008da0dad499c%7Cdd246e4a54344e158ae391ad9797b209%7C0%7C0%7C637837335165795497%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=aEJR9CzY%2Bs221N2KVqmIf%2B9I9fg%2BfNjpX9O5MK%2FziCE%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEMRDVJLUQMAEN2HAHS4GBTVBSEYTANCNFSM5RRRDXXQ&data=04%7C01%7Cdennis.large%40louisville.edu%7Cd6af1f96ed594a2f953008da0dad499c%7Cdd246e4a54344e158ae391ad9797b209%7C0%7C0%7C637837335165795497%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=tBUDdODUKs40jpZIgrKb6KedCHGYuJFNLhGKN3%2F6jow%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

carceneaux commented 1 year ago

The latest PR #32 updated the dependency versions.

Currently working on another PR where I'll most likely updated these dependencies as well.

Closing out this request as this will always be a continual process.