Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1880003

Summary:	[RFE] Migrate disaster-recover Ansible role into oVirt Ansible Collection
Product:	[oVirt] ovirt-ansible-collection	Reporter:	Martin Perina <mperina>
Component:	disaster-recovery	Assignee:	Pavel Bar <pbar>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ilan Zuckerman <izuckerm>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified	CC:	aefrat, bugs, eshenitz, sfishbai, tnisan
Target Milestone:	ovirt-4.4.4	Keywords:	FutureFeature
Target Release:	1.2.1	Flags:	mperina: ovirt-4.4? izuckerm: testing_plan_complete+ mperina: planning_ack? pm-rhel: devel_ack+ pm-rhel: testing_ack+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	ovirt-ansible-collection-1.2.1	Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-12-21 12:36:37 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1879995
Bug Blocks:

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.