Conversation
The verbose option prints the yaml plan to stdout
|
I hope I am not being obnoxious, but I'd like to push back on the premise of this change. Given the relevant report said (plainly, but inside a toggled-closed section) "no combination of preferred sizes and regions was found", I don't think the problem was going to be fixed by adding even more text to read, especially when it would look sort of like a wall of YAML. That sentence hadn't been noticed, so adding text to explain it feels premature. I think we can make a two-pronged approach that is more likely to have positive results. First: when a collapsed section fails, add a failure marker outside the collapsed section. The displayed text was: Given that the job has failed on the second of those lines, the failure "must" be in the first, but it requires attention. We can instead emit: Hopefully, then, the cross mark will catch the eye. Second, rather than spit out the YAML, we can print planning decisions in plain English, and I wouldn't require I don't feel confident this would've helped, but I do feel pretty confident that this would've been more useful than the YAML dump. |
|
If you like, I will carry out the above two steps, but I don't want to step on your toes. If you're interested in doing it, you'll want to look at my $success = try {
...
} catch ($error) {
say "Error while setting up box: $error";
}
print "section-end-stuff";
say "\N{CROSS MARK} Droplet creation failed";
$success || die...Note that these commands are meant to run in both terminal and CI, so "look at section above" is not really sensible all the time. The other change would go in |
Hit an issue where CI wasn't successfully spinning up a box, problem was simple but took a while to figure out. Would have been a LOT faster if the ci plan was printed to see in gitlab output.
I don't love that i've made a _write_plan_file() sub print to stdout, but the alternative was doing it in the ci-plan directly which would mean converting it to yaml in two places. I'm rather ambivalent though how this is done, so if you prefer another way just let me know