19:00:19 #startmeeting ansible core 19:00:19 Meeting started Tue Feb 6 19:00:19 2018 UTC. The chair is thaumos. Information about MeetBot at http://wiki.debian.org/MeetBot. 19:00:19 Useful Commands: #action #agreed #halp #info #idea #link #topic. 19:00:19 The meeting name has been set to 'ansible_core' 19:00:23 blorp 19:00:28 #chair alikins 19:00:28 Current chairs: alikins thaumos 19:00:48 .hello2 19:00:49 sivel: sivel 'Matt Martz' 19:00:54 #chair sivel 19:00:54 Current chairs: alikins sivel thaumos 19:01:20 o/ 19:01:23 * shertel waves 19:01:27 no idea why it's hello2, I just type that because others do :) 19:01:28 .hello2 19:01:29 maxamillion: maxamillion 'Adam Miller' 19:02:17 sivel: I'm actually not sure either 19:02:37 nirik: ping - do you happen to know why it's .hello2? 19:03:11 maybe .hello was taken and engineers are terrible at naming things :) 19:03:22 #chair sdoran maxamillion 19:03:22 Current chairs: alikins maxamillion sdoran sivel thaumos 19:04:11 .hello2 19:04:12 thaumos: thaumos 'Dylan Silva' 19:04:18 nifty 19:04:22 That is pretty neat 19:04:34 * thaumos realises he needs to change the email associated to irc handle 19:05:03 * gundalow waves 19:05:05 .hello2 19:05:06 gundalow: gundalow 'John Barker' 19:05:12 eek 19:05:40 heh 19:05:48 So I am not sure what's on the agenda folks 19:05:48 Not sure who John is 19:05:54 do we have anything in queue or should i go make lunch? 19:05:56 I apologise for missing the past couple of days. 19:06:02 at least it didnt call you bungalow 19:06:04 I don't think so... 19:06:39 sivel: did you see my comment on 'inventory pathing with ,'? 19:06:51 bcoca: yes, and no 19:07:04 ^ some inventory_hostnames might be paths, so for chroot, your patch might create problems, just fyi 19:07:10 i think it 'should' be mostly ok 19:07:17 although I'd be surprised if someone had a chroot that had a `,` in the path to the chroot 19:07:21 but had similar issue with varsmanager and chroot hosts 19:07:41 sivel: i still want to slap anyone witha , in ANY path 19:07:59 * bcoca goes to his happy place 19:08:15 yeah, a ',' in a path seems odd 19:08:16 well, it worked in 2.3 with commas, so we'll see where this gets us 19:08:29 Like I said, windows users... ;) 19:08:38 thought it was OS X user ... 19:08:49 /Users <= not a windoez thing 19:08:52 my opinion, if someone does do that... we can say, "Sorry, out of scope." 19:09:04 he may have been using osx... 19:09:05 we can't conceivably cover every use case. 19:09:10 thaumos: fix is alreayd in, ticket is 2 weeks old, just caught it in 'catching up' 19:09:12 after PTO 19:09:14 \,\h\g\j\m,\f\nj\h 19:09:21 that's how all dumb osx paths look 19:09:31 thaumos: so you read the ticket? 19:09:38 I did before 19:09:48 moving on, anything else we got? 19:09:49 It was another thing that I had a spike in blood pressure from 19:09:49 the way it worked in 2.3 was unexpected, but it worked. this should be less unexpected if you read the code 19:09:58 we can look to handle edge cases if it comes to that 19:10:10 sivel: im shocked it worked in ANY version 19:10:20 same 19:10:22 but yeah, anyone have anything? 19:10:24 probably by accident 19:10:34 if not we can end in a few minutes and give some time back to everybody. 19:10:53 * bcoca puts apron on 19:10:57 bcoca: it worked in 2.3, because we recorded the dirname of the inventory file before doing any processing, and then we both exluced it because it had a comma, but later included it due to the recorded dirname 19:11:33 if it was a real path we recorded the dirname* 19:11:39 sivel: we did a lot of weird things i removed, i find it much simpler now 19:11:41 can also have hostnames like /looks/like/a/path 19:11:54 but with 'simpler' you also get 'quirky things that used to work now fail' 19:12:05 alikins: chroot connection uses those 19:12:16 thaumos: doesn't look like there's anything ... light the fuse when you're ready :) 19:12:16 ^ why i brought this up 19:12:22 .... 19:12:23 ..... 19:12:24 ..... 19:12:25 ....... 19:12:28 #endmeeting