20:00:15 #startmeeting Ansible Windows Working Group 20:00:15 Meeting started Tue Dec 11 20:00:15 2018 UTC. 20:00:15 This meeting is logged and archived in a public location. 20:00:15 The chair is nitzmahone. Information about MeetBot at http://wiki.debian.org/MeetBot. 20:00:15 Useful Commands: #action #agreed #halp #info #idea #link #topic. 20:00:15 The meeting name has been set to 'ansible_windows_working_group' 20:00:23 hey 20:00:23 #chair jborean93 20:00:23 Current chairs: jborean93 nitzmahone 20:00:31 #chair jhawkesworth_ 20:00:31 Current chairs: jborean93 jhawkesworth_ nitzmahone 20:00:34 Hey Jon! 20:00:39 hiya 20:00:45 hey 20:01:06 Looks like nothing new on agenda for today... 20:01:45 yeah. I was going to propose postponing meetings until say 8th Jan as I think there's a fair bit of holiday coming up for people 20:02:07 Works for me :) 20:02:33 I'm guessing jborean93 won't object either 20:02:41 that's fine with me 20:02:54 I'll make a note on the agenda 20:03:07 any objections from lurkers? 20:05:58 all quiet. sounds like a decision then 20:06:02 OK, note made on the agenda 20:06:06 #topic open floor 20:07:33 Nothing exciting from our end right now that I can think of 20:07:50 If nobody's got anything, we can close it up until the new year... 20:08:25 Hey! 20:08:30 A bit late sorry 20:08:37 hey ksubileau 20:08:40 No worries- got something to bring up? 20:08:53 Maybe just a little update about win_nssm 20:09:30 As you may have seen I impletemented the depreciations we discussed two weeks ago 20:10:21 just FYI 20:10:55 this one isn't it? https://github.com/ansible/ansible/pull/45693 20:11:07 * jhawkesworth_ reads latest 20:11:08 Yes exactly 20:11:45 Next steps are adding some potential new features 20:11:48 I assume you want to hold off merging until all the things you've laid out are complete? 20:13:00 Let's say the goal remains merging it for 2.8 20:13:36 Cool 20:13:46 I think its ok to have both `application` and `executable` as long as they are described in the docs 20:14:07 So I'll try to complete if I have enough time, if not it will be for a future PR/version 20:14:38 Ok so I'll name it executable 20:14:55 OK- keep us posted. It seems like you're already well on track to have stuff that's good for 2.8, so if you run out of time to add other goodies, we can merge what you've got 20:15:41 yeah I was going to say if it makes sense to have a separate 'feature' PR it makes reviewing a bit easier. 20:15:43 Yes exactly. Is there any planned date for code freeze of 2.8 ? 20:15:59 It'll probably be March-April sometime 20:16:03 (longer release this time) 20:16:14 So you should have plenty of time 20:16:26 Oh yes perfect 20:17:03 Looks like March 21 is non-core freeze date 20:17:15 https://docs.ansible.com/ansible/devel/roadmap/ROADMAP_2_8.html 20:17:48 And maybe one more thing about the RDS modules : https://github.com/ansible/ansible/pull/43406 20:18:37 I've pushed a litlle update on this last week, if anyone have some time to look at this and tell me if it's all OK ;) 20:18:54 I think I was generally fine with those, but haven't looked at the latest commits 20:19:26 any particular reason for switching from wmi to CIM? 20:19:35 * jhawkesworth_ just curious 20:20:00 WMI to CIM, it's because of a sanity rule 20:20:15 WMI seem deprecated 20:20:18 WMI is the older deprecated cmdlets used to interact with WMI/CIM objects 20:20:21 doh, I see it now 20:21:06 And Powershell RDS PSProvider, it's because I didn't find a way to convert and SID to UPN 20:21:25 * And Powershell RDS PSProvider to WMI/CMI 20:22:18 we have a module_util to convert sids to a UPN or are you talking about something else? 20:22:32 hmm maybe not, that would probably give the netlogon name 20:23:15 Yes Netlogon name, not supported by the RDS PSProvider 20:24:08 brief Google search indicates you can use some .net functions https://stackoverflow.com/questions/2503011/given-a-users-sid-how-do-i-get-their-userprincipalname 20:25:05 I tried it, but seem to work only in AD context 20:25:10 here are the docs https://docs.microsoft.com/en-us/dotnet/api/system.directoryservices.accountmanagement.userprincipal?view=netframework-4.7.2 20:25:12 not for local groups 20:26:01 do local accounts have a UPN? 20:26:35 The PSProvider expects something like administrators@BUILTIN 20:27:24 in that case that's just the netlogon parts with @ instead of \ and reversed. Is it only expecting groups or can you specify accounts 20:27:59 Only groups 20:28:17 local or domain groups? 20:28:27 both 20:29:13 what a fun provider 20:33:32 So finally I found it more difficult to find the right Windows API that works in all cases than to switch to CIM to use the netlogon name returned by the module_util 20:34:16 Despite the inconsistency that leads to the RDS modules code 20:38:27 if there's something that could be done better or make use of the shared module code, that would be cool, but I'm unlikely to be able to test these modules any better than the tests already do, so if they work for others then I'm ok with them being merged. 20:45:57 * ksubileau lost connection 20:52:49 cool, is there anything else we wish to discuss? 20:52:57 not from me 20:53:11 not here 20:53:27 That's all for me 20:53:33 sounds good 20:53:42 Til the new year, then! 20:53:44 have a good holiday, if you're lucky, everyone 20:53:45 Thanks all... 20:53:48 #endmeeting