-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Introduce concept of pre-flight sanity playbook #2152
Comments
|
We definitely want to do some sanity checking for conflicting variable settings to avoid discovering those down the line. Also I like the idea of doing some testing around detected host IP addresses and the service/pod networks to make sure that there isn't a conflict there. Obviously we can't get all use cases, but if we can catch the obvious ones it would be a huge win. |
And lint ansible.cfg against known limitations such as # of forks. |
@jeremyeder validating ansible.cfg may be a difficult task. Some of the values are bubbled up in a way that they can be tested, but others like the value of forks, do not get used outside of the ansible-playbook command itself. Challenge will be to see if there is a way to get the value without diving too deep into internal-use only code that might be fragile with different ansible verisons going forward. |
There really should be a way for playbooks to introspect the ansible environment they're working in. Surprised that there isn't one :-( ... hopefully we can do something short term and perhaps file something with Ansible to provide that guaranteed interface? |
Gather state and dump before upgrade. |
Hey, so I have a few more days left as a writer on the OpenShift documentation team before I leave the company, but this issue echoes an idea I've been chewing on. Initially I was thinking about a script that would be run pre-install, but this pre-flight sanity playbook is a similar idea. I'd hope something could be scripted for the quick install method as well. There are a lotttt of things listed in the installation prerequisites in the OpenShift documentation. Too many, really. It would be great if all of those things could be incorporated into a pre-installation check, where the tasks are either automated or the user is prompted to enter Y/N or values to make it easy. I reckon it'd require a great effort, but in my eyes, it would be ideal to include all of these:
I know this issue is aimed at the Ansible playbook method, but would it be possible to include this kind of pre-install check for the quick installer as well? Thank you to @adellape for pointing me to this GitHub issue when I mentioned the concept of a pre-install checker. |
@tpoitras thanks for the feedback, it definitely helps! |
With #4079 we introduced Anybody who wants to add more checks is welcome to do so (see the existing checks). The items mentioned here should probably be ported and groomed on the Customer Success Trello board - but shall we consider this issue completed? |
How do we get the remaining items ported/groomed? |
The use-case is laid out in the trello card [1], but I'll copy it here:
The openstack performance and scale team has developed a set of
pre-flight sanity checks for RHEL OSP.
We could do the same for openshift (does something already exist?)
https://github.com/openstack/browbeat/tree/master/ansible/check
A pre-flight playbook could do things like: - verify nodes are reachable in the first place (sometimes in the real
world, not all nodes are equal. timeouts for ansible may have to be
increased).
[1] https://trello.com/c/3Zb2d9zl/98-3-cfme-integration-prerequisites-playbook
The text was updated successfully, but these errors were encountered: