Turns out 99 isn't a good prefix.
Also, remove the move argument now that we're merging with the existing
Also, stop running zuul unittests on migration script changes. They are
Unmatched single quotes is what killed us before. Add an option which
does a test with ansible-playbook --syntax-check to validate whether a
playbook emitted with shell instead of script works.
Running that way takes around 90 minutes. So default to shell, but
provide --syntax-check as a way to check using shell and fallback to
script. This can then be used to fix pre-migration jobs if desired.
Also separate task entries with an extra '\n' for readability.
And only collect the generated playbooks - not all of them.
We need to be able to generate a new v3 config based on our old v2
config. It won't be perfect, but should ultimately be no worse than the
v2.5 auto-generated playbooks.