Skip to content

Actually support storing backups on Azure

username-removed-470592 requested to merge benalbrecht/gitlab-ce:replace-fog-azure into master

What does this MR do?

This MR replaces the fog-azure gem with the fog-azure-rm gem to actually support storing backups on Azure.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

fog-azure was added by !4444 (merged) to support storing backups on Azure.
Sadly, fog-azure does only support Compute resources. (see https://github.com/fog/fog-azure/blob/master/lib/fog/bin/azure.rb#L26-L30, http://stackoverflow.com/a/32050560)
As a result, storing backups on Azure is still not possible.

fog-azure-rm does actually include support for Storage resources.
The gem dependency issue that has caused fog-azure to be included instead of fog-azure-rm in the first place has been resolved.

What are the relevant issue numbers?

#1819 (closed) #15352 (closed)

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

Merge request reports