OpenNebula / one

The open source Cloud & Edge Computing Platform bringing real freedom to your Enterprise Cloud 🚀
http://opennebula.io
Apache License 2.0
1.23k stars 479 forks source link

Backup jobs argument missing in the developer docs for the storage drivers #6378

Closed atodorov-storpool closed 10 months ago

atodorov-storpool commented 11 months ago

Description Commit https://github.com/OpenNebula/one/commit/c14869eb84c56e671ecff11ee0a469e255f40e3f introduces a new argument in the middle of the arguments of TM_MAD/{pre,post}backup{,_live} that is not addressed in Integration and Development » Infrastructure Integration » Storage Driver

To Reproduce

Expected behavior Integrations and Development docs is in sync with the code.

Details

Additional context The change in the TM_MAD drivers for the backups looks irrelevant, as the variable looks like not used there. But rearranging the arguments is breaking the backward compatibility with the latest upgradeable CE version... Anyway, I've handled it in https://github.com/OpenNebula/addon-storpool/commit/df73afd42d8a366abe715d3a842b6b3179034656

Progress Status

atodorov-storpool commented 10 months ago

Closing as merged https://github.com/OpenNebula/docs/commit/4a90e28dca5a1b809a1dca9ef9e5a37b0a22d063