Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing restore task file and vars file #180

Merged
merged 2 commits into from
Nov 18, 2020

Conversation

kobihk
Copy link
Contributor

@kobihk kobihk commented Nov 16, 2020

when running restore flow we failed with the following error:
TASK [ovirt.ovirt.engine_setup : Restore engine from file] *********************
fatal: [example.com]: FAILED! => {"reason": "Could not find or access
'/path/to/file/restore_engine_from_file.yml' on the Ansible Controller."}

when running restore flow we failed with the following error:
TASK [ovirt.ovirt.engine_setup : Restore engine from file] *********************
fatal: [example.com]: FAILED! => {"reason": "Could not find or access
'/path/to/file/restore_engine_from_file.yml' on the Ansible Controller."}
@mwperina
Copy link
Member

@didib

Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but I would Didi to take a look

Copy link
Member

@didib didib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not use the restore in hosted-engine (we have there our own code for that. Perhaps worth considering merging, not that important right now), so I do not know current code well. It looks ok (other than engine-cleanup). I assume it was a simple omission when copying it from ovirt-ansible-engine-setup. Did you check the current diff between them? There are a few other things not copied - perhaps none are important, but better make sure - things around tests, meta, requirements, etc.

@@ -0,0 +1,19 @@
---
- name: Run engine cleanup command
command: "engine-cleanup"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without an answerfile? It would get stuck asking questions. Was this ever tested?

Perhaps also add 'ignore_errors: true'.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes you right!!
we have an open issue about it in:
https://github.com/oVirt/ovirt-ansible-engine-setup/issues/93

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll move the issue to the collection so we have everything here. (Please let me know if I missed some important issue which should be moved here)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kobihk
Copy link
Contributor Author

kobihk commented Nov 18, 2020

The content of this PR merged in:
https://github.com/oVirt/ovirt-ansible-engine-setup/pull/87/files
but I assume by mistake ommited in:
#113

@didib
Copy link
Member

didib commented Nov 18, 2020

Fine for me to merge current, if you also keep tracking open issues in the old project (or copy them here).

Copy link
Member

@mnecas mnecas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, sorry for missing this when migrating

@mnecas mnecas merged commit af73080 into oVirt:master Nov 18, 2020
@kobihk kobihk deleted the missing_restore_task_file_master branch November 18, 2020 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants