search

ansible-collections / community.network

Ansible Community Network Collection
http://galaxy.ansible.com/community/network
Other
124 stars 90 forks source link

Add "Permission denied" error handling for Extreme Exos #571

Closed Thorbijoern closed 6 months ago

Thorbijoern commented 10 months ago
SUMMARY

I stumbled over an error with the scp2 command on a 5320 model switch with EXOS Version 31.x

Couldn't open local file \"/scratch/summit_arm-32.4.1.10.xos\" for writing: Permission denied

Ansible/the exos modules don't recognize it as error and pretend it's fine. So i just want to add it in case somebody else has a similar problem and it might help.

ISSUE TYPE
COMPONENT NAME

exos

ADDITIONAL INFORMATION

It only happens on a switch that is freshly unconfigured. It's probably a bug, but i use download image for now. I don't know the exact Exos version anymore, i would have to downgrade the switch and test a bit.

Andersson007 commented 10 months ago

@Thorbijoern hello, thanks for the fix! Could you please add a changelog fragment?

Andersson007 commented 10 months ago

@Thorbijoern ^ ?

Thorbijoern commented 6 months ago

Sorry for forgetting to follow up with this PR. I added the changelog fragment, is it ok like this?

patchback[bot] commented 6 months ago

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/4dffff1f9453fa93d5835f0f18c870328116e89e/pr-571

Backported as https://github.com/ansible-collections/community.network/pull/595

🤖 @patchback I'm built with octomachinery and my source is open — https://github.com/sanitizers/patchback-github-app.

patchback[bot] commented 6 months ago

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/4dffff1f9453fa93d5835f0f18c870328116e89e/pr-571

Backported as https://github.com/ansible-collections/community.network/pull/596

🤖 @patchback I'm built with octomachinery and my source is open — https://github.com/sanitizers/patchback-github-app.

Andersson007 commented 6 months ago

@Thorbijoern that's OK, thanks for adding the fragment and for the bugfix! Just a fyi, this is the only merged PR since the previous release, so I wouldn't expect the fix to be available in the package soon: I would wait for a bit more fixes/new features to release them at once.