-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
cs_instance: doc: fix typo in examples #46035
Conversation
The test
|
The test
|
The test
|
The test
|
Discusses |
@@ -77,11 +77,12 @@ | |||
- Name the hypervisor to be used for creating the new instance. | |||
- Relevant when using I(state=present), but only considered if not set on ISO/template. | |||
- If not set or found on ISO/template, first found hypervisor will be used. | |||
choices: [ KVM, VMware, BareMetal, XenServer, LXC, HyperV, UCS, OVM, Simulator ] | |||
choices: [KVM, kvm, VMware, vmware, BareMetal, baremetal, XenServer, xenserver, LXC, lxc, HyperV, hyperv, UCS, ucs, OVM, ovm, Simulator, simulator] | |||
keyboard: | |||
description: | |||
- Keyboard device type for the instance. | |||
choices: [ de, de-ch, es, fi, fr, fr-be, fr-ch, is, it, jp, nl-be, no, pt, uk, us ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quote the no
to avoid parsing as a bool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
facepalm
The tests were unstable. I've restarted them. Assuming they pass, they may still warrant further investigation, as they're likely to be unstable in the future as well. |
@mattclay I was able to reproduce the issue, it looks like a bug in cloudstack. |
Might be worth backporting this |
preparing a couple of backports. Thanks! |
SUMMARY
fixes sanity
ISSUE TYPE
COMPONENT NAME
cs_instance
ANSIBLE VERSION
ADDITIONAL INFORMATION