Bug 1880003 - [RFE] Migrate disaster-recover Ansible role into oVirt Ansible Collection
Summary: [RFE] Migrate disaster-recover Ansible role into oVirt Ansible Collection
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-ansible-collection
Classification: oVirt
Component: disaster-recovery
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ovirt-4.4.4
: 1.2.1
Assignee: Pavel Bar
QA Contact: Ilan Zuckerman
URL:
Whiteboard:
Depends On: 1879995
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-09-17 14:17 UTC by Martin Perina
Modified: 2020-12-22 11:26 UTC (History)
5 users (show)

Fixed In Version: ovirt-ansible-collection-1.2.1
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-12-21 12:36:37 UTC
oVirt Team: Storage
Embargoed:
mperina: ovirt-4.4?
izuckerm: testing_plan_complete+
mperina: planning_ack?
pm-rhel: devel_ack+
pm-rhel: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github oVirt ovirt-ansible-collection pull 159 0 None closed Add disaster recovery 2021-02-02 10:47:59 UTC
Github oVirt ovirt-ansible-collection pull 160 0 None closed Disaster Recovery fixes after added to collections 2021-02-02 10:47:59 UTC

Description Martin Perina 2020-09-17 14:17:27 UTC
Newly introduced ovirt-ansible-collection package will contain not only all existing oVirt Ansible modules, but we also want to migrate disaster-recovery standalone role into ovirt-ansible-collection package.

oVirt Ansible Collection will not only be distributed as RPM package, but it should be also available from Red Hat Automation Hub as supported collection

Comment 3 Pavel Bar 2020-11-18 11:58:54 UTC
Comments to QE regarding what has changed while moving to Ansible Collection: in general not much from the functionality PoV.
1) CI warnings - didn't change the logic.
2) README.md updates
3) Multiple code changes that *do not* change the logic:
  a) Removing unused variables.
  b) Renaming variables.
  c) Fixing typos in fixing/improving user messages/logs.
  d) Code simplification & reformatting.
  e) Some internal implementation files were moved to different location under the new project.
  f) Updating the default paths.
  g) Removing redundant configuration parameter "vault" from the validator step.
4) Minot bug-fixes:
  a) Validator step - using default var file path correctly (instead of an empty path if user just presses <ENTER>, thus choosing the suggested default path) when the configured var file was not properly initialized (does not appear in "dr.conf" or doesn't exist on disk).
  b) Same as above issue with the default Ansible play file in failover & failback flows.
  c) Add default parameter for the var file for the "failover" & "failback" flows.
5) Bug-fixes:
  a) Adding "~" support in paths received from the user (either from configuration file "dr.conf" or from CLI when the file path received from "dr.conf" file doesn't exist).

So generally speaking, from the user/QE no functionality changed, except moving to Ansible Collection.

Comment 4 Ilan Zuckerman 2020-11-26 07:10:39 UTC
Verified on rhv 4.4.4-2 using the full flow of 'active - passive' DR manually done, including failover and failback flows. Using NFS storage.

Comment 5 Sandro Bonazzola 2020-12-21 12:36:37 UTC
This bugzilla is included in oVirt 4.4.4 release, published on December 21st 2020.

Since the problem described in this bug report should be resolved in oVirt 4.4.4 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.