16:02:37 #startmeeting OpenStack Ansible Meeting 16:02:37 Meeting started Thu Dec 10 16:02:37 2015 UTC and is due to finish in 60 minutes. The chair is odyssey4me. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:02:38 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 16:02:40 The meeting name has been set to 'openstack_ansible_meeting' 16:02:44 hi 16:02:44 o/ 16:02:46 moo 16:02:53 #topic Agenda & rollcall 16:03:01 o/ 16:03:16 #link https://wiki.openstack.org/wiki/Meetings/openstack-ansible#Agenda_for_next_meeting 16:04:40 presente 16:04:42 o/ 16:04:58 o/ 16:05:02 o/ 16:05:04 0/ 16:06:23 o/ 16:06:50 welcome everyone - no action items from last week, so we can head onwards 16:07:00 o/ 16:07:00 #topic Topics for Discussion 16:07:12 #topic Independent repository for plugins and libraries 16:07:17 * odyssey4me hands the mic to cloudnull 16:07:43 hi 16:08:12 so i would like to know how we should handle libs and plugins 16:08:22 within our new independent role repos approach 16:08:32 lots of the roles need access to the libs 16:08:40 and i'd like to not copy them all over the place 16:09:16 You're talking about the openstack service libs? Would a logical place be their associated role? 16:09:33 at present i've been doing this https://github.com/openstack/openstack-ansible-galera_server/blob/master/tests/ansible-role-requirements.yml#L1-L4 16:09:48 palendae: the keystone, neutron, etc 16:09:51 for ansible 16:09:51 palendae if they're only used by that role, yes - but we have cross-role libs 16:09:58 what kind of plugins are we talking about, service plugins? 16:10:07 javeriak_: https://github.com/os-cloud/openstack-ansible-plugins 16:10:10 those ones 16:10:26 Ah, right, we have them up higher now so Ansible can see them across the roles 16:10:37 which is a collection of https://github.com/openstack/openstack-ansible/tree/master/playbooks/library and https://github.com/openstack/openstack-ansible/tree/master/playbooks/plugins 16:10:46 Dependencies could help with that, no? 16:10:58 palendae: ansible doesnt have deps for libs and plugins sadly 16:11:15 cloudnull: I thought you could put a lib/plugin in a role and it'd be counted through role deps 16:11:21 Or does their lookup not account for that? 16:11:22 so we can submodule or do something similar to what i've done in the role requirements file 16:11:34 palendae: we can add it to a role 16:11:41 but then we have to add it to all the roles 16:11:44 Ok 16:11:46 or submodule etc 16:11:48 So it's scoped to the role 16:11:53 yes 16:11:58 cloudnull, you're also trying to upstream the genetric ones to ansible i believe 16:12:02 Ansible won't use it's role deps to search for libs 16:12:16 javeriak_: yes 16:12:33 the advantage of using https://github.com/openstack/openstack-ansible-galera_server/blob/master/tests/ansible-role-requirements.yml#L1-L4 to me is that we never have to manage submodules in each of the repositories... we can fix to a sha if we want to, but don't have to 16:12:44 so we have to have a place to put them if the new independent roles are to be used as stand alone roles 16:13:42 So I wanted to poll everyone and see what they thought was best. 16:13:56 Submodule, ansible role requires , other? 16:14:03 my initial feel was that perhaps we could do thing differently (eg: make the utility container to interact with keystone, neutron, etc) to isolate where the libraries are needed... but we need the config_template library everywhere... so that's off the table 16:14:37 are we caching those roles on the repo server ? 16:14:39 We also need the other plugins throughout the stack 16:15:30 Bjoernt no. The libs and plugins exist within the main git tree 16:15:37 I think the role-requirements file makes sense 16:15:43 cloudnull yep, there are some others that are needed... and while we should endeavour to upstream them, we also need the flexibility of having them around as an interim step to allow us to keep moving forward 16:16:10 so, vendoring? 16:16:23 using ansible role requirements seems like the cleaner approach to me; but that would incur maintaining a seperate repository for these now 16:16:24 So I'm good with whatever we deem is best. I just would like a group consensus. 16:16:33 I would prefer anything over git submodule :-) 16:16:46 Javeriak_ yes it would. 16:17:05 I've not proposed the new repo yet because I wasn't sure what to do with them. 16:17:25 I personally prefer using the role reqs as well 16:17:56 well the OSA project is growing, like the others, we will at some point have to separate sub-repos 16:18:01 shall we do a vote? 16:18:26 although, there are no objections to the role reqs approach so far... do we need to bother voting? 16:18:39 are there any specific objections to the role reqs approach? 16:18:53 aw... i was hoping to see the vote bot again... 16:18:59 Hahaha 16:19:09 javeriak_: Yeah, I'm not super happy about more repos, but at the same time, we're going to be making a lot of repos anyway, a single one's probably not that big of a deal in that case 16:19:15 Odyssey4me .doit() 16:19:42 cloudnull: .net? 16:19:42 heh, ok 16:19:43 palendae agreed 16:20:05 #vote Shall we use the role requirements approach for plugins & libs? yes, no 16:20:16 bah, I'm no good at this 16:20:16 #vote yes 16:20:24 #startvote Shall we use the role requirements approach for plugins & libs? yes, no 16:20:24 Begin voting on: Shall we use the role requirements approach for plugins & libs? Valid vote options are yes, no. 16:20:26 Vote using '#vote OPTION'. Only your last vote counts. 16:20:26 #vote yes 16:20:32 #vote yes 16:20:34 #vote yes 16:20:34 #vote yes 16:20:35 #vote yes 16:20:41 #vote yes 16:21:05 Somebody vote no just because. ;-) 16:21:14 #vote yes 16:21:16 one of us one of us 16:21:23 cloudnull: you're someone 16:21:43 I'm nobody. 16:21:53 #vote no 16:21:54 boom 16:22:06 Hahaha ! 16:22:12 #endvote 16:22:13 Voted on "Shall we use the role requirements approach for plugins & libs?" Results are 16:22:14 yes (6): palendae, odyssey4me, BjoernT, prometheanfire, cloudnull, javeriak_ 16:22:15 no (1): andymccr 16:22:17 Andymccr for prez! 16:22:22 I'm a visionary 16:22:22 lol 16:22:35 or terrorist 16:22:36 andymccr: trump 16:22:41 right, another question was whether we do one repo or two 16:22:42 OK I'm done. 16:22:49 i'll be proved right in 2 weeks time!! 16:22:50 that was fun :), i know what my fav part of these meetings is now.. 16:22:57 (or we'll all forget that i said anything) 16:23:14 I think the single new repo is the best approach. 16:24:01 yep, more is usually harder to maintain.. 16:24:14 yeah, after reviewing https://github.com/ansible/ansible/blob/devel/lib/ansible/constants.py I see that there are no defaults paths in /etc/ansible for plugins or libraries... so I'm happy with a single repo and the paths referenced in ansible.cfg 16:24:19 I would also like to limit the repo sprawl. 16:25:06 some of the roles could possible consolidate ;) 16:25:15 but that's another story, let's not go there 16:25:30 cloudnull: We could put them all in one repo! :) 16:25:30 would the new repo be under /openstack too? 16:25:55 javeriak_: I believe so, cloudnull's got stuff in to the infra team to create them 16:26:17 palendae the new repositories are already there 16:26:36 https://github.com/openstack/?utf8=%E2%9C%93&query=openstack-ansible 16:26:36 Cool 16:27:08 all the way through to 'setup-infrastructure' is there - the next roles to move over are the openstack roles 16:27:17 wow... 16:27:40 each role has testing focused on that specific role, so the feedback time is much shorter 16:27:42 Yup. Once https://review.openstack.org/#/c/255921/ goes through its off to the openstack roles. 16:28:54 ok, so one repo it is - unless there are any objections within the next two mins 16:29:04 btw is the documentation in place, the deployment flow will change now i believe? 16:29:06 (one repo for the plugins and libs) 16:29:39 javeriak_ the deployment flow doesn't actually change - the entry points are the same 16:29:46 where stuff goes changes 16:30:06 the result is also the same 16:30:13 so we're still cloning openstack-ansible and role-requirements will pull in the rest 16:30:22 Yes 16:30:22 yep 16:30:26 ok cool 16:30:56 The change is simply where the role code lives in terms of a full stack deployment. 16:30:56 what's also nice is that each role's individual test method shows how to consume them without the dynamic inventory 16:31:26 but yeah, I would like to spend some time beefing up each role's docs a bit - as developer orientated documentation 16:31:38 We will also be able to create role specific docs. In the future. 16:32:10 https://wiki.openstack.org/wiki/OpenStackAnsible 16:32:33 I updated it yesterday for the roles we split out. 16:32:42 But I'll need to add the new ones. 16:32:47 with IRR are the roles going to be pulled in to /etc/ansible/roles during bootstrap-ansible like the role-requirements currently work? 16:32:52 Role docs are limited currently. 16:33:08 logan- yes 16:33:56 ok thanks 16:36:13 ok, next topic 16:36:22 #topic Mid Cycle Meetup 16:36:31 #link http://lists.openstack.org/pipermail/openstack-dev/2015-December/081847.html 16:36:38 #link http://lists.openstack.org/pipermail/openstack-operators/2015-December/009154.html 16:38:06 Please can we have everyone respond to the email indicating your interest. 16:38:43 This doesn't have to be considered a firm RSVP, so don't worry about that - it's just simply a 'are you interested?' email. 16:38:46 ++ replied already 16:41:20 next! 16:42:05 alright, 16:42:09 #topic Open discussion 16:42:25 Other than the call for input in https://wiki.openstack.org/wiki/Meetings/openstack-ansible#Agenda_for_next_meeting we have nothing further planned 16:42:34 is there anything specific anyone would like to discuss? 16:44:07 heh, a quiet bunch today then :) 16:44:15 yep 16:44:26 I've got nothing else 16:44:34 odyssey4me: hi 16:44:58 you said to bring this up thursday 16:45:01 https://review.openstack.org/#/c/253702/ 16:46:02 ah yes, sure 16:46:27 * odyssey4me hands prometheanfire the mic 16:46:44 the patch pulls comments out, stored in an ordered list, then inserts them before the regenerated 'data' 16:47:12 it's useful for commented out key value options and general notes people leave in the config 16:47:37 I've brought it up with support and they liked it, major isn't here but he did as well 16:48:35 what is the purpose of it, considering that the returned data is unordered 16:48:48 how do you separate from a comment or deactivated tokens which have been added for testing ? 16:48:51 ie a comment about a line below will no longer be above the line 16:49:32 this only ever matters when the pwgen is run, which I expect is not often 16:49:51 so I really just don't see the point... this just seems like dev work done to solve no problem 16:50:06 so, personally, I'd like to understand the problem this is trying to solve 16:50:40 Yeah, I can get why someone might like to keep comments associated with the given line, but this doesn't appear to do that 16:50:54 ^ that 16:51:13 From just glancing over the code, it drops the comments at the top of the file 16:51:16 also, I don't see why someone would want to run pwgen more than once 16:51:37 if it is, there's always a backup 16:51:50 if anything, I'd say there may be value in keeping datestamped backups 16:52:00 it keeps a backup of the file it overwrites? 16:52:02 thats already there 16:52:06 that'd be better 16:52:08 yes it does 16:52:13 ok 16:52:14 Yeah, we already have backed up copies of files 16:52:32 anytime there is a change in the file it writes the old one to a backup 16:52:48 abandoned 16:53:03 so it only keeps one tarballed backup - I can see value in perhaps having 5 backups, datetimestamped or something 16:53:12 I'd be interested to see if there's a change that associates a line with it's comments, but the way the yaml dump works would make that difficult 16:53:26 its* 16:53:27 yeah, hardly worth the effort 16:53:40 close associated bug as well? 16:53:43 I think the right approach for config storage anyway is to use something like git to keep history 16:53:54 prometheanfire yep, please 16:55:18 anything else - we have 5 mins? 16:57:43 alright, that seems to be it 16:57:50 thank you everyone! 16:57:55 #endmeeting