[01:47] PR please https://github.com/juju/juju/pull/11515 [03:46] hpidcock: looking [03:50] hpidcock: approved [03:50] babbageclunk: thanks [04:48] babbageclunk: piling on, i have a couple of PRs up if you have a little time to look [04:51] wallyworld_: otp with thumper, will look after that [04:51] oh sorry, ty [10:54] stickupkid: hey, so if this gets landed, it will make experiemtng with centos and windows images much easier https://github.com/juju/juju/pull/11513 [10:54] wallyworld_, nice, let me qa it [10:56] stickupkid: i used it to try centos on aws etc. we still need to teach juju how to inperpret an error saying "no image for you" as something other than a "these creds are broken" error [11:00] can I get a PR reviewed at some point pls https://github.com/juju/juju/pull/11509 [11:04] hpidcock: lgtm [12:37] Facing a strange issue while running "juju status". it's failing with an error "ERROR cannot unmarshal accounts: yaml: control characters are not allowed" [15:00] hml: stickupkid followup fix for the constraints-with-spaces bug; can either of you review? https://github.com/juju/juju/pull/11519 [15:05] achilleasa, so "a b" will be "a\ \ b" right? [15:06] stickupkid: yes. Just as you would do in a shell command [15:06] fair [15:07] ideally we should also support quoted kv pairs [15:07] but it's way more effort than this small change (you'd need a tokenizer with depth counters for nested quotes etc.) [15:08] stickupkid: also, if you take a look at https://bugs.launchpad.net/juju/+bug/1847259, you will see that Pedro tried to slash-escape the spaces [15:08] Bug #1847259: Juju --constraints "zones=NAME WITH SPACES" fails because of space chars [15:09] achilleasa, it's actually easier to read [15:09] achilleasa, I used the prior to the previous commit to compare against [16:15] hml: can you take a look at https://github.com/juju/juju/pull/11521? [16:15] achilleasa: looking [16:18] achilleasa: approved [16:25] stickupkid: last one :-) https://github.com/juju/juju/pull/11522 [16:26] wallyworld_, looking