15:59:57 <Qalthos> #startmeeting Ansible Network Working Group
15:59:57 <zodbot> Meeting started Wed Jul 24 15:59:57 2019 UTC.
15:59:57 <zodbot> This meeting is logged and archived in a public location.
15:59:57 <zodbot> The chair is Qalthos. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:59:57 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic.
15:59:57 <zodbot> The meeting name has been set to 'ansible_network_working_group'
16:00:26 <pabelanger> o/
16:00:30 <Qalthos> #chair pabelanger ganeshrn ikhan justjais privateip trishnag
16:00:30 <zodbot> Current chairs: Qalthos ganeshrn ikhan justjais pabelanger privateip trishnag
16:00:49 <Qalthos> #chair dmellado
16:00:49 <zodbot> Current chairs: Qalthos dmellado ganeshrn ikhan justjais pabelanger privateip trishnag
16:00:53 <Qalthos> There we go
16:01:01 <Qalthos> #topic Core Updates
16:01:20 <Qalthos> Mostly a rehash of last week
16:01:32 <Qalthos> #info AnsibleFest is coming up, Sept 24-26 in Atlanta, GA
16:01:54 <Qalthos> #link https://www.ansible.com/ansiblefest
16:02:36 <Qalthos> #info Contributor's Summit will be Monday the 23rd
16:02:46 <Qalthos> #link https://github.com/ansible/community/issues/449 for info on that
16:03:11 <Qalthos> #info if you have anything you want to bring up there, please add it to the etherpad
16:03:19 <Qalthos> #link https://etherpad.openstack.org/p/ansible-summit-atlanta-2019
16:04:10 <Qalthos> #info As a reminder, it will be possible to participate remotely
16:04:28 <Qalthos> Details are in the Etherpad
16:05:49 <Qalthos> #info we are still exploring having some version of this meeting earlier in the day to better accommodate EMEA/APAC folks as well as possible
16:06:36 <Qalthos> AFAIR, I haven't heard from anyone who would actually prefer that, but please do mention something if you would be interested
16:07:30 <Qalthos> I think that's everything.
16:08:03 <Qalthos> #topic Meraki PRs
16:08:30 <mrproper> I had the 4 or 5 PRs in the agenda. I believe you've reviewed and approved all of them,
16:08:33 <Qalthos> We can probably merge all of these today, I think?
16:08:53 <mrproper> That's my suspicion.
16:09:29 <pabelanger> +1 I know we triaged one today in our meeting
16:09:36 <pabelanger> and think I assigned it to me to review
16:11:09 <Qalthos> I've rerun Shippable on most of them... we do /try/ not to merge things with stale_ci showing, but that shouldn't add too much friction
16:11:34 <Qalthos> Oh, and it looks like thay've all finished anyway
16:11:43 <mrproper> CI is fast today.
16:12:05 <Qalthos> mrproper: Any luck on getting unit tests on Meraki?
16:12:46 <mrproper> I was working on it last night. I'm really struggling on getting mocks to work for the API responses.
16:14:02 <Qalthos> I'll see if I can find some time to look into it
16:14:02 <mrproper> If there are other unit tests you think are important, I can work on those. I do have a feature I want in 2.9 which requires unit tests for development (rate limit handling) so unit tests are a priority for me.
16:14:28 <mrproper> Thanks. I have a file committed with gists to show output. Should I message you later and can walk you through what I'm seeing?
16:16:18 <Qalthos> mrproper: I probably won't be able to do much synchronous communication, but if you can put something together I will see what I can do
16:16:33 <mrproper> IRC best? Even if it's not sync?
16:17:06 <Qalthos> mrproper: My bouncer is (almost) always up (:
16:17:23 <mrproper> Sounds good. I think I'm good for this week then. Thank you!
16:18:37 <Qalthos> pabelanger: Do you want to take a pass over these modules before merging?
16:19:16 <eramirez> Hello all, I have been trying to use ansible to backup a few IOS devices and checked the document a few times but without any luck. If I use the copy module in the playbook I get SELinux module is not installed, and when I use the ios_config command, the playbook fails because backup: yes and backup_options: are not valid
16:19:17 <pabelanger> sure I can
16:19:41 <eramirez> can anyone let me know if they were able to use ansible to take a backup of IOS devices?
16:19:41 <Qalthos> #action pabelanger Review & merge Meraki PRs
16:20:30 <Qalthos> #topic New module avi_user
16:21:47 <Qalthos> I am assuming there is some thing here that I am completely failing to grasp
16:25:26 <Qalthos> grastogi: Going over your comments, do these references sometimes come in the form of complete URIs?
16:25:31 <grastogi> yes
16:25:34 <grastogi> they do
16:26:52 <grastogi> Also, the test case in the other PR covers that use case... and there are situations when one can do a lookup, extract the URI and without modification pass that. The diff logic in ansible utils handles the comparison properly even though simple string match will fail
16:27:12 <Qalthos> grastogi: Would you mind please expanding the documentation and adding examples to that effect so that someone attemting to understand how the module can be used will know that?
16:27:38 <grastogi> sure. we will add examples and description about it. good point!
16:28:40 <Qalthos> Now that I finally understand what we have been arguing about for... almost two months, that's all I need to approve it
16:29:57 <grastogi> ok.. I am glad to hear that. Also, we learned the gaps in our documentation. we will fix
16:29:59 <Qalthos> I apologise for completely failing to grasp that, but nothing I had seen so far short of your one comment last week gave me any indication that such a thing was expected to be supported
16:30:36 <Qalthos> Though I now see why you were going on about ARNs
16:31:15 <Qalthos> That should finally resolve that
16:31:20 <Qalthos> #topic Open Floor
16:31:33 <grastogi> Also, we plan a big change to the documentation by adding sub-objects. This came out previous review... however, we wanted this module to be merged before we do that massive change in documentation.
16:33:59 <Qalthos> eramirez: More details would be helpful... have you opened an issue about this?
16:34:31 <eramirez> Qalthos, no I have not open an issue
16:36:35 <Qalthos> ios_config should be able to handle backups. What does your playbook look like?
16:39:01 <eramirez> getting it now
16:39:54 <eramirez> I tried this
16:40:00 <eramirez> https://paste.gnome.org/plxaco4yn
16:41:30 <eramirez> and https://paste.gnome.org/ptolbhwhc
16:41:49 <eramirez> append -u username -k
16:42:00 <eramirez> when I run the command
16:43:01 <eramirez> for the first playbook, I get the SELinux errors
16:43:51 <eramirez> Aborting, target uses selinux but python bindings (libselinux-python) aren't installed
16:44:28 <eramirez> and for the send playbook, I get 'backup' is not a valid option
16:44:49 <eramirez> sorry not a valid attribute for a task
16:45:50 <Qalthos> eramirez: right... They need to be attributes of ios_config, not the task itself. Indent backup and backup_options so that they are children of ios_config
16:46:42 <Qalthos> As for copy... if you're on a selinux-aware platform, you need selinux python bindings installed.
16:47:01 <eramirez> on the control stations?
16:47:09 <eramirez> as on my laptop
16:47:32 <Qalthos> On the machine ansible is running on
16:47:43 <eramirez> Qalthos, thanks I will make the indents
16:48:13 <eramirez> okay, it showed the target machine, thanks will do that
16:48:14 <Qalthos> But ios_config is probably the better way anyway. See the last example on https://docs.ansible.com/ansible/latest/modules/ios_config_module.html
16:48:43 <eramirez> I missed the indents
16:48:59 <eramirez> thanks for the link
16:49:13 <Qalthos> eramirez: You're welcome
16:49:29 <Qalthos> Anyone else have anything they want to bring up in the last few minutes?
16:50:16 <mrproper> Qalthos: I did a writeup of my unit test issue. https://gist.github.com/kbreit/7f7623ce1bab82730d05d6522feb334c
16:51:07 <Qalthos> mrproper: That looks like what I wanted to see. I'll let you know if I come up with anything
16:51:13 <mrproper> Thanks.
16:53:45 <Qalthos> Thanks everyone for coming.
16:54:04 <Qalthos> #endmeeting