| *** tlian has quit IRC | 00:07 | |
| *** darrenc_afk is now known as darrenc | 00:14 | |
| *** annashen has joined #openstack-ansible | 00:29 | |
| *** annashen has quit IRC | 00:34 | |
| *** daneyon has quit IRC | 00:48 | |
| *** daneyon has joined #openstack-ansible | 00:48 | |
| *** daneyon has quit IRC | 00:49 | |
| *** daneyon has joined #openstack-ansible | 00:49 | |
| *** daneyon has quit IRC | 00:51 | |
| *** alop has quit IRC | 00:51 | |
| *** daneyon has joined #openstack-ansible | 00:52 | |
| *** daneyon has quit IRC | 00:56 | |
| *** abitha has quit IRC | 01:24 | |
| *** annashen has joined #openstack-ansible | 01:30 | |
| *** metral has quit IRC | 01:33 | |
| openstackgerrit | Christopher H. Laco proposed stackforge/os-ansible-deployment: Remove reference to missing user_group_vars file https://review.openstack.org/207264 | 01:33 |
|---|---|---|
| *** annashen has quit IRC | 01:35 | |
| *** abitha has joined #openstack-ansible | 02:16 | |
| *** annashen_ has joined #openstack-ansible | 02:31 | |
| *** annashen_ has quit IRC | 02:36 | |
| *** sdake has joined #openstack-ansible | 03:05 | |
| *** markvoelker_ has quit IRC | 03:20 | |
| *** markvoelker_ has joined #openstack-ansible | 03:22 | |
| *** metral_zzz has joined #openstack-ansible | 03:27 | |
| *** metral_zzz is now known as metral | 03:27 | |
| *** annashen_ has joined #openstack-ansible | 03:32 | |
| *** sdake has quit IRC | 03:36 | |
| *** annashen_ has quit IRC | 03:37 | |
| *** JRobinson__ is now known as JRobinson__afk | 04:25 | |
| *** fawadkhaliq has joined #openstack-ansible | 04:27 | |
| *** jlvillal has quit IRC | 04:32 | |
| *** annashen has joined #openstack-ansible | 04:33 | |
| *** annashen has quit IRC | 04:38 | |
| *** misc has quit IRC | 04:50 | |
| *** misc has joined #openstack-ansible | 04:52 | |
| *** fawadkhaliq has quit IRC | 04:52 | |
| *** yapeng has joined #openstack-ansible | 04:54 | |
| *** darrenc is now known as darrenc_afk | 04:56 | |
| *** shausy has joined #openstack-ansible | 04:57 | |
| *** JRobinson__afk is now known as JRobinson__ | 04:58 | |
| *** yapeng has quit IRC | 04:59 | |
| *** fawadkhaliq has joined #openstack-ansible | 05:05 | |
| *** yapeng has joined #openstack-ansible | 05:10 | |
| *** abitha has quit IRC | 05:11 | |
| *** annashen has joined #openstack-ansible | 05:34 | |
| *** shausy has quit IRC | 05:37 | |
| *** shausy has joined #openstack-ansible | 05:38 | |
| *** annashen has quit IRC | 05:39 | |
| *** darrenc_afk is now known as darrenc | 05:40 | |
| *** javeriak has joined #openstack-ansible | 05:55 | |
| *** javeriak has quit IRC | 05:58 | |
| *** javeriak has joined #openstack-ansible | 06:05 | |
| *** javeriak_ has joined #openstack-ansible | 06:07 | |
| *** javeriak has quit IRC | 06:07 | |
| *** shausy has quit IRC | 06:15 | |
| *** shausy has joined #openstack-ansible | 06:15 | |
| *** annashen_ has joined #openstack-ansible | 06:35 | |
| *** annashen_ has quit IRC | 06:39 | |
| evrardjp | hello everyone | 07:15 |
| evrardjp | should I vote for some talks to bring more OSAD visibility for the next summit? | 07:16 |
| *** gparaskevas has joined #openstack-ansible | 07:35 | |
| *** annashen has joined #openstack-ansible | 07:35 | |
| stevelle | evrardjp: there are probably a half dozen or more related talks | 07:37 |
| *** gparaskevas has quit IRC | 07:38 | |
| *** gparaskevas has joined #openstack-ansible | 07:39 | |
| evrardjp | That's why I'd vote for a pre-defined list ;) | 07:39 |
| -openstackstatus- NOTICE: Our CI system is broken again today, jobs are not getting processed at all. | 07:39 | |
| evrardjp | I've voted for what I've found | 07:39 |
| *** ChanServ changes topic to "Our CI system is broken again today, jobs are not getting processed at all." | 07:39 | |
| *** annashen has quit IRC | 07:40 | |
| *** JRobinson__ has quit IRC | 07:47 | |
| -openstackstatus- NOTICE: CI system is broken and very far behind. Please do not approve any changes for a while. | 07:48 | |
| *** ChanServ changes topic to "CI system is broken and very far behind. Please do not approve any changes for a while." | 07:48 | |
| *** abitha has joined #openstack-ansible | 08:11 | |
| *** abitha has quit IRC | 08:16 | |
| *** javeriak_ has quit IRC | 08:21 | |
| *** javeriak has joined #openstack-ansible | 08:33 | |
| *** annashen has joined #openstack-ansible | 08:36 | |
| *** annashen has quit IRC | 08:41 | |
| *** fawadkhaliq has quit IRC | 08:53 | |
| *** ChanServ changes topic to "Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible" | 08:57 | |
| -openstackstatus- NOTICE: CI is back online but has a huge backlog. Please be patient and if possible delay approving changes until it has caught up. | 08:57 | |
| openstackgerrit | Merged stackforge/os-ansible-deployment: set correct swift dispersion tenant https://review.openstack.org/206913 | 09:24 |
| *** fawadkhaliq has joined #openstack-ansible | 09:25 | |
| *** fawadkhaliq has quit IRC | 09:26 | |
| *** fawadkhaliq has joined #openstack-ansible | 09:26 | |
| *** ig0r_ has joined #openstack-ansible | 09:32 | |
| *** annashen has joined #openstack-ansible | 09:37 | |
| openstackgerrit | Merged stackforge/os-ansible-deployment: Set default inventory file for inventory-manage.py https://review.openstack.org/207158 | 09:38 |
| *** annashen has quit IRC | 09:42 | |
| *** javeriak has quit IRC | 10:07 | |
| mattt | svg: would it be possible to push up your changes from yesterday so we can do further testing? | 10:09 |
| svg | Sure, was waiting for you to ask that :) | 10:10 |
| mattt | svg: thanks :) | 10:11 |
| *** shausy has quit IRC | 10:12 | |
| openstackgerrit | Serge van Ginderachter proposed stackforge/os-ansible-deployment: Ceph/RBD support https://review.openstack.org/181957 | 10:12 |
| *** shausy has joined #openstack-ansible | 10:13 | |
| svg | mattt: ^^ | 10:13 |
| mattt | thanks svg! | 10:18 |
| git-harry | svg: are you going to have any time today to address the last few outstanding issues? | 10:22 |
| git-harry | We'd like to try and get this patch merged in tomorrow | 10:23 |
| git-harry | If not do you mind if mattt and I upload some changes to address those things? | 10:23 |
| svg | Let me see, IIRC, besides the obvious #TODO's, there was your last comment about 'services are configured before ceph_client installs'. Anything I am mising? | 10:25 |
| *** javeriak has joined #openstack-ansible | 10:26 | |
| svg | about that latter, I'm not sure which of your proposed solutions, is the best. Perhaps having the ceph role restart those services? | 10:26 |
| git-harry | svg: I think that's everything. | 10:28 |
| git-harry | svg: I was inclined to have the role install libvirt and run it before the service role because virsh is required by the role | 10:29 |
| svg | Of the TODO's in the ceph role, so far I left them there b/c I was waiting for feedback from leseb, but given he seems offline, I suspect he might be on holidays | 10:29 |
| svg | git-harry: which service-role are you referring to exactly, os-cinder os-nova os glance I suppose? | 10:31 |
| *** shausy has quit IRC | 10:32 | |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone Federation Identity Provider Configuration https://review.openstack.org/207386 | 10:32 |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone Federation Service Provider Configuration https://review.openstack.org/194395 | 10:37 |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Enable Horizon to consume a Keystone v3 API endpoint https://review.openstack.org/206575 | 10:37 |
| *** annashen has joined #openstack-ansible | 10:38 | |
| *** shausy has joined #openstack-ansible | 10:38 | |
| svg | hmz, git-harry, doing an implementation of ceph before service thingz: this won't help if ceph support is installed later, in a subsequent run, as then the os services won't be restarted either | 10:42 |
| *** annashen has quit IRC | 10:42 | |
| svg | so doing a handler restart from the ceph role seems better. thoughts? | 10:42 |
| git-harry | svg: the conf files for the openstack services would be updated to include the ceph stuff so the services should get restarted | 11:01 |
| *** jlvillal has joined #openstack-ansible | 11:04 | |
| svg | yes, that was already clear | 11:05 |
| svg | I'm looking at implementing handlers, but that would mean I need to reimplement some data, lik ethe list of *_service_names | 11:05 |
| odyssey4me | mancdaz there's some discussion about the ceilometer bit in https://review.openstack.org/181560 which could warrant a minor update, or a follow-on patch. Thoughts? | 11:06 |
| odyssey4me | If you could do a follow-on patch to improve that today, then we can +w that patch now. | 11:07 |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Add sample Keystone Federation SP configuration for ADFS https://review.openstack.org/203736 | 11:09 |
| mattt | svg git-harry : any thoughts on this approach to removing ceph_conf from defaults/main.yml and either a) providing a path to ceph.conf or b) obtaining from a running mon? | 11:12 |
| mattt | svg git-harry : https://gist.github.com/mattt416/f00a30f9a7d9b3c9df5c | 11:12 |
| mattt | (still need to rebase against latest patchset) | 11:12 |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone SSL cert/key distribution and configuration https://review.openstack.org/194474 | 11:12 |
| svg | mattt is it b/c of not rebased yet you are removing that # look for 1 ceph monitor host that is up task? | 11:15 |
| svg | O, dit you move that to include: ceph_get_mon_host.yml ? that one's not in the diff | 11:17 |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Remove hardcoded config drive enforcement https://review.openstack.org/195403 | 11:18 |
| svg | mattt perhaps better to do a local_action: command instead of delegate_to: localhost for name: Get ceph.conf and store contents as a var | 11:21 |
| odyssey4me | mancdaz there are some notes in the reviews for https://review.openstack.org/199572 which I think are pertinent... it is probably better to configure the logging rather than just remove the package | 11:21 |
| git-harry | does local_action give all hosts access to the var? | 11:22 |
| svg | sure, it's actyally more of a syntactic difference, though locat_action will do a connection: local, whilst delegate might still go over ssh iirc | 11:23 |
| mattt | svg: cool, i can make that change | 11:23 |
| svg | it's a connection thing, doesnt change scope of host | 11:23 |
| mancdaz | odyssey4me well, after discussion (in the bug) it was decided we don't want the package there at all | 11:23 |
| mattt | svg: if i remove ceph_conf from defaults/main.yml will that break your workflow significantly? | 11:24 |
| svg | mattt so the local ceph file, where would that one live? personally, I'd rather make that a variable in inventory, than a loose file somewhere | 11:24 |
| mancdaz | odyssey4me but I see the note from kevin around moving the task out somewhere else | 11:24 |
| mattt | svg: it will read it from anywhere on the deployment host i guess, so location doesn't matter | 11:25 |
| mancdaz | not sure I agree. If you want to install the package again in the future you just remove it from the uninstall list and add it to the install list | 11:25 |
| odyssey4me | mancdaz I see that it's merged into juno already, and also see the discussion in the bug - I think perhaps a subsequent patch can do the suggested reconfiguration? your thoughts? | 11:25 |
| svg | which could be anywhere, so also outside of a regular dir with data for osad | 11:25 |
| svg | I'd rather enforce it to be in some user var file | 11:25 |
| svg | but gotta go or i don;t have lunch! brb | 11:26 |
| mancdaz | odyssey4me I'm not sure I understand the value of moving the task into a separate file | 11:26 |
| mattt | svg: go go ! | 11:26 |
| odyssey4me | mancdaz I think the idea is similar to pinning - it has broader use beyond just neutron | 11:26 |
| mancdaz | odyssey4me well the argument being made is that moving it out makes it easier to remove in future? | 11:27 |
| odyssey4me | meh, I don't agree with that | 11:27 |
| odyssey4me | I think we should merge this as-is. It's come specifically from a deployer and has been agreed to by the SME's. | 11:28 |
| odyssey4me | The only thing that has been disagreed to was the method of how we do it in the playbooks/roles. | 11:29 |
| odyssey4me | Andy's -1 is related to rather doing the config change instead of removing the package. | 11:29 |
| mancdaz | odyssey4me I responded to both comments | 11:31 |
| *** annashen has joined #openstack-ansible | 11:39 | |
| *** yapeng has quit IRC | 11:39 | |
| odyssey4me | mancdaz thanks - I've approved it and am registering a wish list bug for the generic removal method | 11:40 |
| *** annashen has quit IRC | 11:43 | |
| mancdaz | odyssey4me https://bugs.launchpad.net/openstack-ansible/+bug/1400881 | 11:56 |
| openstack | Launchpad bug 1400881 in openstack-ansible "Cannot rebuild a VM created from a Cinder volume backed by NetApp" [Medium,Triaged] | 11:56 |
| odyssey4me | mancdaz sure, we could do that - but it seems silly... do we do this for every other upstream patch? | 11:57 |
| odyssey4me | we will automatically consume the fixes as they merge, if they merge | 11:57 |
| mancdaz | odyssey4me yes but being able to show the users that we have bumped our sha to consume that fix, is useful | 11:57 |
| odyssey4me | mancdaz I really doubt those backports will end up being merged though | 11:58 |
| mancdaz | so wontfix when they get rejected? | 11:59 |
| odyssey4me | mancdaz fair enough, we'll just keep moving the milestone until there's a conclusion of some sort | 11:59 |
| mancdaz | either way, I think if you're wontfixing something, you need to remove the target milestone | 11:59 |
| mancdaz | odyssey4me yeah I bumped juno to 10.1.12 | 12:00 |
| *** fawadk has joined #openstack-ansible | 12:02 | |
| *** fawadkhaliq has quit IRC | 12:03 | |
| odyssey4me | mancdaz the same has been done for kilo and they're marked as in-progress | 12:05 |
| odyssey4me | I'll keep tabs on them (they're assigned to me) | 12:05 |
| mancdaz | odyssey4me kewl | 12:05 |
| odyssey4me | this review is a bit pointless: https://review.openstack.org/196702 | 12:06 |
| odyssey4me | we need to add the reserved port, but in containers it can't be done due to apparmor | 12:06 |
| odyssey4me | so the role only does it on a physical keystone server, which we don't have (except in an edge case where someone chooses to do that) | 12:07 |
| odyssey4me | I think we need an alternative approach and that perhaps this bug should be moved along to the next release. | 12:07 |
| *** tlian has joined #openstack-ansible | 12:10 | |
| *** fawadk has quit IRC | 12:10 | |
| *** javeriak has quit IRC | 12:30 | |
| mattt | svg: updated diff https://gist.github.com/mattt416/84fd07c8dd70378ff0e4 (this one includes the file i missed before) | 12:36 |
| mattt | svg: so were you suggesting to create a variable w/ contents of your local ceph.conf file? how would that be done? | 12:37 |
| *** annashen has joined #openstack-ansible | 12:40 | |
| openstackgerrit | Merged stackforge/os-ansible-deployment: remove conntrackd package https://review.openstack.org/199572 | 12:40 |
| openstackgerrit | Merged stackforge/os-ansible-deployment: Remove reference to missing user_group_vars file https://review.openstack.org/207264 | 12:44 |
| *** annashen has quit IRC | 12:44 | |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: remove conntrackd package https://review.openstack.org/207441 | 12:47 |
| *** javeriak has joined #openstack-ansible | 12:55 | |
| *** javeriak has quit IRC | 13:07 | |
| *** ig0r_ has quit IRC | 13:10 | |
| *** fawadkhaliq has joined #openstack-ansible | 13:11 | |
| mattt | svg: oops, updated that commit w/ your local_action suggestion | 13:15 |
| *** fawadkhaliq has quit IRC | 13:16 | |
| *** ig0r_ has joined #openstack-ansible | 13:17 | |
| *** Mudpuppy has joined #openstack-ansible | 13:21 | |
| *** Mudpuppy has quit IRC | 13:23 | |
| *** Mudpuppy has joined #openstack-ansible | 13:24 | |
| *** ig0r_ has quit IRC | 13:31 | |
| *** TheIntern has joined #openstack-ansible | 13:34 | |
| *** KLevenstein has joined #openstack-ansible | 13:39 | |
| *** annashen has joined #openstack-ansible | 13:40 | |
| svg | mattt: sorry, had also a blitz meeting when I got back :) | 13:43 |
| svg | see http://sprunge.us/LYNU on how to put ceph conf in a var | 13:43 |
| *** annashen has quit IRC | 13:45 | |
| *** Mudpuppy has quit IRC | 13:56 | |
| mattt | svg: maybe it's best to just create the var as you did before, and have it then build the ceph.conf via the template | 13:58 |
| mattt | svg: that way we can obtain the mon list from the variable without having to have to respecify ceph_mons | 13:59 |
| svg | Well, thought of that too, the difference is small | 13:59 |
| svg | hm, but how do you then say "I have no cephconf, go pick it up"? | 13:59 |
| mattt | assuming there is no variable specified then it'd go out and pick it up | 14:00 |
| mattt | (ceph_conf_file or whatever you want to call it) | 14:00 |
| mattt | svg: maybe i'll just implement what you suggested, we can iterate on this later as needs arise | 14:01 |
| svg | a list of mon hosts is always necessary, either way | 14:01 |
| *** spotz_zzz is now known as spotz | 14:01 | |
| *** Mudpuppy has joined #openstack-ansible | 14:02 | |
| mattt | svg: not if you can do ceph_conf.global.mon_host as your patch currently does | 14:02 |
| svg | Well, then it still is needed :) | 14:03 |
| mattt | svg: well, not as a separate variable :) | 14:03 |
| mattt | (that's gleaned from the config you've dumped in) | 14:03 |
| *** Mudpuppy has quit IRC | 14:04 | |
| svg | but I mean, if you dont provide the ceph conf, he has to fetch it, he needs the list, so a spearate variable in that case - better keep it that way in all cases, no? | 14:04 |
| *** yaya has joined #openstack-ansible | 14:04 | |
| mattt | svg: suppose it's more consistent yes | 14:04 |
| svg | yes | 14:04 |
| *** Mudpuppy has joined #openstack-ansible | 14:04 | |
| svg | unless we ight need other components from ceph-conf later on, but currently not the case | 14:05 |
| *** prad has joined #openstack-ansible | 14:05 | |
| mattt | svg: actually good point, we will need something to indicate if you're using cephx or not | 14:06 |
| *** markvoelker_ has quit IRC | 14:06 | |
| *** alop has joined #openstack-ansible | 14:10 | |
| evrardjp | Am I the only one to find interesting to have a line log_path= in ansible.cfg in my playbooks folder? | 14:12 |
| *** javeriak has joined #openstack-ansible | 14:13 | |
| *** spotz is now known as spotz_zzz | 14:13 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:13 | |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Change ansible forks used https://review.openstack.org/207474 | 14:14 |
| odyssey4me | evrardjp that's primarily there for gating - I'd like to change it so that it doesn't do it when you test with an AIO | 14:16 |
| odyssey4me | in fact, let me do that now - it causes issues for development and testing | 14:16 |
| *** javeriak has quit IRC | 14:17 | |
| *** Mudpuppy has quit IRC | 14:18 | |
| evrardjp | what do you mean by "that"? the ansible.cfg file? | 14:18 |
| *** spotz_zzz is now known as spotz | 14:19 | |
| evrardjp | other question, I've been away for 2 weeks, should I run all the playbooks to upgrade my environment, or just the openstack ones? | 14:20 |
| odyssey4me | evrardjp I need to adjust the bootstrapping to not edit ansible.cfg unless it's part of a gate check process, and also need to not setup the link to the logs directory unless the execution is part of a gate check | 14:20 |
| odyssey4me | evrardjp well, what tag/branch were you on and where are you now? | 14:21 |
| evrardjp | kilo | 14:21 |
| evrardjp | I just did a git pull so hard to say... I guess .4 | 14:21 |
| evrardjp | let me check | 14:21 |
| palendae | Within the last 2 weeks kilo had some rabbitmq bugs that affect upgrades there; so I'd say run all | 14:22 |
| odyssey4me | evrardjp you can check /etc/*release on any of your hosts | 14:22 |
| *** jwagner_away is now known as jwagner | 14:23 | |
| evrardjp | yup | 14:24 |
| evrardjp | .4 | 14:24 |
| evrardjp | 11.0.4 to be complete | 14:24 |
| evrardjp | so no big changes | 14:24 |
| odyssey4me | ok, so that was the last point release | 14:24 |
| *** javeriak has joined #openstack-ansible | 14:25 | |
| odyssey4me | we're planning to release 11.1.0 tomorrow night - so perhaps you should wait a little | 14:25 |
| *** Mudpuppy has joined #openstack-ansible | 14:25 | |
| evrardjp | ok | 14:25 |
| odyssey4me | also, how you do the upgrade is really up to you - the deployment is designed in such a way that things shouldn't change what you already have - but you may wish to inspect all the changes and run one play at a time so that you can inspect the changes and catch anything unexpected | 14:26 |
| *** alop has quit IRC | 14:26 | |
| *** markvoelker has joined #openstack-ansible | 14:27 | |
| odyssey4me | the list of patches in review and not yet merged for 11.1.0 is https://review.openstack.org/#/q/starredby:%22Jesse+Pretorius%22+project:stackforge/os-ansible-deployment,n,z if you're curious | 14:27 |
| palendae | Does anyone know off the top of their head if we have an example of running an ansible task/role only on the deployment host? | 14:27 |
| evrardjp | I'd like to test (in a near future) the upgrade process for minor versions and also test new features (SSL support on HAproxy + ceph + federation) | 14:28 |
| evrardjp | so I'll wait for 11.1 | 14:28 |
| *** markvoelker_ has joined #openstack-ansible | 14:28 | |
| prometheanfire | Sam-I-Am: only if you ask in here | 14:29 |
| svg | mattt: right, the current ceph_conf var is also used in the conditional in playbooks/roles/ceph_client/tasks/ceph_all.yml | 14:29 |
| svg | so we will need some ceph_auth_cephx: True var | 14:30 |
| Sam-I-Am | prometheanfire: what? | 14:31 |
| prometheanfire | Sam-I-Am: don't ask for reviews in that channel :P | 14:32 |
| Sam-I-Am | oh | 14:32 |
| odyssey4me | palendae something like this: echo 'localhost ansible_connection=local' > /tmp/ansible_inventory; ansible localhost -i /tmp/ansible_inventory -m shell -a 'ls -al /tmp' | 14:32 |
| Sam-I-Am | Apsu prometheanfire rackertom can y'all look at https://review.openstack.org/#/c/204597/ | 14:32 |
| Sam-I-Am | there :) | 14:32 |
| prometheanfire | thanks | 14:32 |
| *** markvoelker has quit IRC | 14:32 | |
| * prometheanfire puts away the open source bat | 14:32 | |
| *** ig0r_ has joined #openstack-ansible | 14:32 | |
| Apsu | Sam-I-Am: hi | 14:32 |
| git-harry | palendae: is the 'Wait for ssh to be available' task in playbooks/lxc-containers-create.yml what you're looking for? | 14:33 |
| *** alop has joined #openstack-ansible | 14:34 | |
| palendae | odyssey4me: Well, I'm doing it with the OSAD inventory; I'd like to distribute an SSH key that's only on the deploy host. The containers are failing the role because they don't have that file present | 14:34 |
| odyssey4me | palendae we have a precent for distributing stuff from the deployment host | 14:34 |
| odyssey4me | palendae eg: https://review.openstack.org/#/c/194474/19/playbooks/roles/os_keystone/tasks/keystone_ssl_user_provided.yml,cm | 14:35 |
| odyssey4me | *precedent | 14:35 |
| *** Mudpuppy has quit IRC | 14:36 | |
| odyssey4me | palendae we do it for horizon in the same way: https://github.com/stackforge/os-ansible-deployment/blob/master/playbooks/roles/os_horizon/tasks/horizon_ssl_user_provided.yml | 14:37 |
| *** javeriak_ has joined #openstack-ansible | 14:37 | |
| palendae | odyssey4me: Ok, thanks, I'll dig into these. I figured we had precedent, wanted to follow those | 14:38 |
| *** jmckind has joined #openstack-ansible | 14:38 | |
| mattt | svg: https://gist.github.com/mattt416/84fd07c8dd70378ff0e4 updated, i added a 'cephx' variable cheekily since that's what ceph-ansible also uses | 14:39 |
| Apsu | Sam-I-Am: In the l3_agent.ini.j2 template, line 25 of your changed version... you've got an ansible assignment: {{ neutron_max_l3_agents_per_router | default(groups['neutron_agent'] | length) }} | 14:40 |
| Apsu | Sam-I-Am: Are you sure you can put a jinja pipe inside of a jinja function like default()? | 14:41 |
| *** javeriak has quit IRC | 14:41 | |
| *** annashen has joined #openstack-ansible | 14:41 | |
| *** ccrouch has joined #openstack-ansible | 14:42 | |
| odyssey4me | Apsu after trying some fancy stuff in default() recently, I doubt it | 14:43 |
| odyssey4me | that said, you should probably review in the review so that the discussion and changes have history :) | 14:43 |
| *** bapalm_ has joined #openstack-ansible | 14:44 | |
| Apsu | odyssey4me: mmhmm | 14:44 |
| prometheanfire | Sam-I-Am: done | 14:45 |
| *** annashen has quit IRC | 14:46 | |
| svg | mattt: ok; cephx might also need a reference in the example vars file in etc/... | 14:47 |
| mattt | svg: i'm going to leave that out only because i'm guessing most installs will not be using ceph | 14:52 |
| svg | Do you know if we need to restart all the os services after ceph setup? | 14:54 |
| mattt | svg: git-harry and i were discussing that this morning, i don't know 100% but i don't think so | 14:57 |
| svg | Also: I'm inclined to leave the ceph_client role run after the services role, and restart os services from a handler, which allows to just use the defaults defined in those roles | 14:58 |
| svg | doing it with a handler makes sure it restarts after a tiny change too | 14:58 |
| svg | I was looking at the vars where the services are listed, and those are quite extensive, especially for nova; I think it's better to avoid needing to reimplement all those? | 14:59 |
| *** jmckind has quit IRC | 15:00 | |
| git-harry | mattt: svg I think they all need restarting because the rdb/rados python libraries don't exist when the opnestack services are started | 15:01 |
| git-harry | When ceph_client is run for the first time | 15:01 |
| svg | either way, restarting them all is the safest and simpliest | 15:01 |
| *** klindgren_ is now known as klindgren | 15:01 | |
| git-harry | svg: I'm not too concerned as to how it is done. Each way has its pluses and minuses, but it does need to be done | 15:02 |
| *** ig0r_ has quit IRC | 15:03 | |
| *** ig0r__ has joined #openstack-ansible | 15:04 | |
| *** jmckind has joined #openstack-ansible | 15:05 | |
| *** logan2 has quit IRC | 15:08 | |
| *** ig0r__ has quit IRC | 15:09 | |
| *** ig0r_ has joined #openstack-ansible | 15:09 | |
| *** fawadkhaliq has joined #openstack-ansible | 15:11 | |
| *** ig0r__ has joined #openstack-ansible | 15:13 | |
| *** ig0r__ has quit IRC | 15:15 | |
| gparaskevas | Hello there | 15:15 |
| *** ig0r_ has quit IRC | 15:15 | |
| gparaskevas | a small problem when running the swift installation playbook | 15:15 |
| gparaskevas | i get this erros "'dict object' has no attribute 'swift_vars'" | 15:16 |
| gparaskevas | http://paste.openstack.org/show/406450/ | 15:16 |
| gparaskevas | more here | 15:16 |
| *** Mudpuppy has joined #openstack-ansible | 15:17 | |
| svg | mattt: I'm going to need your patch on ceph-conf to put min to update the service restarts on top of it | 15:18 |
| mattt | svg: you want me to update your review with https://gist.github.com/mattt416/84fd07c8dd70378ff0e4? | 15:20 |
| openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Move ansible logging to gate-check script only https://review.openstack.org/207502 | 15:20 |
| svg | mattt: I gues that's the easies? | 15:21 |
| svg | t | 15:21 |
| mattt | svg: yeah i can do that, sec | 15:21 |
| *** phalmos has joined #openstack-ansible | 15:24 | |
| openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Ceph/RBD support https://review.openstack.org/207504 | 15:27 |
| *** sdake has joined #openstack-ansible | 15:27 | |
| mattt | bleh, ignore | 15:27 |
| svg | :) | 15:28 |
| svg | or I could just apply your patch on what I'm doing here | 15:28 |
| *** rackertom has quit IRC | 15:29 | |
| prometheanfire | abandoned, nice | 15:29 |
| openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Ceph/RBD support https://review.openstack.org/181957 | 15:29 |
| mattt | svg: ^^ | 15:30 |
| *** gparaskevas has quit IRC | 15:32 | |
| sigmavirus24 | zuul is really struggling: http://status.openstack.org/zuul/ | 15:35 |
| sigmavirus24 | I know we want to tag 11.1 tomorrow but it might not be realistic | 15:35 |
| palendae | Help them help us | 15:36 |
| palendae | Wonder if we can ask OSIC to lend some capacity to Zuul | 15:37 |
| palendae | I guess we can and they can say no :) | 15:37 |
| *** jmckind has quit IRC | 15:38 | |
| *** jmckind has joined #openstack-ansible | 15:39 | |
| *** jmckind has quit IRC | 15:39 | |
| *** jmckind has joined #openstack-ansible | 15:40 | |
| prometheanfire | I would be happy with that :D | 15:41 |
| sigmavirus24 | prometheanfire: "that"? | 15:42 |
| prometheanfire | zuul going faster | 15:42 |
| prometheanfire | both locally and remotely... | 15:42 |
| palendae | Make zuul use web workers and farm jobs out to browsers with the page open :) | 15:42 |
| *** annashen has joined #openstack-ansible | 15:42 | |
| prometheanfire | palendae: must have been what happened to me :( | 15:43 |
| prometheanfire | http://bellard.org/jslinux/ | 15:43 |
| openstackgerrit | Serge van Ginderachter proposed stackforge/os-ansible-deployment: Add Ceph/RBD support to playbooks https://review.openstack.org/181957 | 15:43 |
| svg | mattt: git-harry ^^ | 15:44 |
| prometheanfire | ohgod | 15:45 |
| Apsu | prometheanfire: Been a while since I've seen the jslinux | 15:46 |
| prometheanfire | it only came out a year or two ago | 15:46 |
| mattt | im hoping the js means NODEJS | 15:46 |
| mattt | oh. | 15:46 |
| prometheanfire | mattt: ffffffuuuuu | 15:46 |
| *** annashen has quit IRC | 15:47 | |
| Sam-I-Am | prometheanfire: ping | 15:48 |
| prometheanfire | Sam-I-Am: wat | 15:48 |
| Sam-I-Am | prometheanfire: your comment on my patch | 15:50 |
| Sam-I-Am | you +1d it, then -1d it? | 15:50 |
| prometheanfire | yes, didn't see that comment | 15:50 |
| prometheanfire | then I did | 15:51 |
| Sam-I-Am | the vxlan_udp_port business | 15:51 |
| svg | mattt: git-harry that leaves 2 tasks with todo's in playbooks/roles/ceph_client/tasks/ceph_auth.yml | 15:51 |
| git-harry | svg: great | 15:52 |
| prometheanfire | Sam-I-Am: ya, you remove it yet? | 15:52 |
| git-harry | svg: would you like us to pick those task up or are you working on fixes for those | 15:52 |
| *** ig0r_ has joined #openstack-ansible | 15:53 | |
| svg | Do you guys think the first one (TODO: also be able to create users, keys and pools on ceph) ca be deferred to later? | 15:53 |
| Sam-I-Am | prometheanfire: i guess it is not needed with the lb agent, so i s'pose we could | 15:53 |
| mattt | svg: we're going to handle that separately | 15:53 |
| svg | That's one that will prolly pull in roles from ansible-ceph | 15:53 |
| git-harry | svg: yeah I don't think that's an issue for now | 15:53 |
| prometheanfire | Sam-I-Am: it's really a nit, but I'd prefer to do it now rather then forget about it | 15:53 |
| * prometheanfire is very anti cruft | 15:53 | |
| prometheanfire | I wonder if there's a type of coverage tool for ansible | 15:54 |
| svg | ok, so for now just needing to fix the group owner on the ceph keyring file | 15:54 |
| Sam-I-Am | prometheanfire: makes sense | 15:55 |
| svg | git-harry: mattt I'm going afk for the day, I might have a look at that tomorrow - if you think there is a timeconstraint let me know and feel free to go forward? | 15:55 |
| *** rward has quit IRC | 15:55 | |
| svg | those latest patches will need some extensive testing though, and right now I can't easily do that | 15:56 |
| git-harry | svg:we'd like to get it merged tomorrow or at least have it ready for review by others. | 15:57 |
| git-harry | svg: mattt and I will test it tomorrow | 15:57 |
| svg | I'm not sure by what time I'll be able to implement that last todo | 15:57 |
| git-harry | svg: don't worry about the group issue, if you haven't got time that is fine. mattt and I will get it sorted | 15:57 |
| openstackgerrit | Matthew Kassawara proposed stackforge/os-ansible-deployment: Reduce neutron configuration https://review.openstack.org/204597 | 15:58 |
| Sam-I-Am | prometheanfire: fixeraterated | 15:58 |
| svg | I'll get back to you tomorrow then, we'll see from there then | 15:58 |
| svg | cya | 15:58 |
| git-harry | svg:bye | 15:58 |
| evrardjp | cya svg | 15:58 |
| prometheanfire | Sam-I-Am: reviewing now, was that the only change you made? | 15:59 |
| Sam-I-Am | prometheanfire: yaep | 15:59 |
| prometheanfire | ok | 15:59 |
| prometheanfire | making it easy then | 15:59 |
| odyssey4me | cloudnull, mattt, andymccr, d34dh0r53, hughsaunders, b3rnard0, palendae, Sam-I-Am, odyssey4me, serverascode, rromans, mancdaz, dolphm, _shaps_, BjoernT, claco, echiu, dstanek, jwagner, ayoung meeting in #openstack-meeting-4 starting in a minute | 15:59 |
| dstanek | o/ | 16:01 |
| mattt | svg: cool, have a good night, talk to you tomorrow | 16:01 |
| odyssey4me | dstanek in #openstack-meeting-4 :) | 16:01 |
| dstanek | odyssey4me: ha, then why are you pinging me here :-) | 16:02 |
| odyssey4me | miguelgrinberg can you join us in #openstack-meeting-4 please :) | 16:02 |
| odyssey4me | dstanek 'cos this is where you like to hang out :p | 16:02 |
| palendae | d34dh0r53: If you can get https://bugs.launchpad.net/openstack-ansible/+bug/1479870 fixed, that would be awesome | 16:03 |
| openstack | Launchpad bug 1479870 in openstack-ansible "teardown.sh does not cleanup lvm.conf" [Undecided,New] | 16:03 |
| b3rnard0 | hello | 16:03 |
| palendae | I think rackertom was looking at that issue, too | 16:03 |
| *** TheIntern has quit IRC | 16:05 | |
| *** TheIntern has joined #openstack-ansible | 16:07 | |
| *** jmckind has quit IRC | 16:11 | |
| openstackgerrit | David Wilde proposed stackforge/os-ansible-deployment: Enable Horizon to consume a Keystone v3 API endpoint https://review.openstack.org/206575 | 16:11 |
| *** jmckind has joined #openstack-ansible | 16:11 | |
| *** jmckind has quit IRC | 16:11 | |
| *** jmckind has joined #openstack-ansible | 16:11 | |
| *** woodard has joined #openstack-ansible | 16:14 | |
| *** logan2 has joined #openstack-ansible | 16:19 | |
| *** shausy has quit IRC | 16:24 | |
| openstackgerrit | git-harry proposed stackforge/os-ansible-deployment: Add Ceph/RBD support to playbooks https://review.openstack.org/181957 | 16:27 |
| *** jmckind has quit IRC | 16:28 | |
| git-harry | svg: I've uploaded a new patchset that creates the group. We can discuss if it's the best approach tomorrow but I wanted d34dh0r53 to be able to test a patchset that should work. | 16:29 |
| *** TheIntern has quit IRC | 16:31 | |
| *** jmckind has joined #openstack-ansible | 16:32 | |
| evrardjp | about https://review.openstack.org/194395, I'm willing to test, but I may need some help to grasp the concepts (Ok for federation, not ok for integration with openstack right now) | 16:32 |
| evrardjp | can give a review afterwards | 16:32 |
| *** yaya has quit IRC | 16:36 | |
| dstanek | where's chapter 1 in the install-guide? | 16:43 |
| *** annashen_ has joined #openstack-ansible | 16:43 | |
| *** rackertom has joined #openstack-ansible | 16:43 | |
| dstanek | also is there any reason my configuring-ceilometer.rst isn't linked in the configuration section? | 16:47 |
| *** annashen_ has quit IRC | 16:47 | |
| palendae | dstanek: Oversight probably | 16:52 |
| dstanek | palendae: i'd like to get it in there. is there any particular ordering or is at the end ok? | 16:54 |
| palendae | dstanek: None that I'm aware of | 16:54 |
| palendae | Sam-I-Am may have something to say about it :) | 16:55 |
| dstanek | palendae: Sam-I-Am: not being in there is wrecking my patch! | 17:00 |
| sigmavirus24 | evrardjp: if you want to help get the ceph work into 11.2 we'll need thorough reviews for at least the functionality if not also correctness of approach :D | 17:00 |
| evrardjp | agreed | 17:00 |
| sigmavirus24 | dstanek: patch ch-ch-ch-ch-chain | 17:00 |
| *** rward has joined #openstack-ansible | 17:00 | |
| odyssey4me | evrardjp so it seems likely that ceph client support will merge pretty soon into master, and will possibly be backported into kilo for a point release like 11.1.1 depending on the state of it | 17:00 |
| sigmavirus24 | evrardjp: I personally have no knowledge of ceph or a way to test against it (yet) so for me reviewing that patch will be pointless because I can only offer helpful criticisms around the ansibles and style (which svg already is better at than me) | 17:01 |
| sigmavirus24 | odyssey4me: ceph support should be 11.x not 11.1.y | 17:01 |
| sigmavirus24 | odyssey4me: that's a significant feature and since we're following semver, that belongs in a 11.x release | 17:01 |
| odyssey4me | it looks like mattt git-harry and svg are close to a client patch merge for the major starting point - they'll then iterate in smaller patches afterwards which are easier to review | 17:01 |
| sigmavirus24 | evrardjp: regardless, please also review the ceph work. more eyes will make it better sooner | 17:02 |
| evrardjp | I have to go for today, let's continue this conversation tomorrow. I'll stick with the team to test them | 17:02 |
| odyssey4me | sigmavirus24 well, there's ceph cluster deployment and simply enabling the use of a ceph back-end - the current patch is just enabling the back-end | 17:02 |
| evrardjp | to test the integration* | 17:02 |
| evrardjp | sigmavirus24: ofc, same for other stuff like federation | 17:02 |
| odyssey4me | but yeah, when that merges into kilo is above my pay grade :p | 17:03 |
| odyssey4me | is there such a thing as pay grade in open source? ;) | 17:03 |
| evrardjp | it will never be if you're not interested by it ;p | 17:03 |
| evrardjp | anyway I'm off | 17:03 |
| *** abitha has joined #openstack-ansible | 17:04 | |
| d34dh0r53 | yeah, I'm testing the ceph-client side stuff today, the server side deployment will live in rpc-openstack AFAIK | 17:04 |
| d34dh0r53 | ceph is slated for 11.2 right now, but that is not firm | 17:05 |
| d34dh0r53 | afk lunch | 17:05 |
| sigmavirus24 | odyssey4me: right I understand the patch | 17:12 |
| sigmavirus24 | hence "no way to test against it (yet)" | 17:12 |
| palendae | d34dh0r53: Should specify "Rackspace's server side deployment will live in rpc-openstack" :) | 17:15 |
| palendae | Other implementations can go elsewhere | 17:15 |
| *** woodard has quit IRC | 17:16 | |
| openstackgerrit | Nolan Brubaker proposed stackforge/os-ansible-deployment: Upgrade Elasticsearch and Kibana https://review.openstack.org/207552 | 17:22 |
| odyssey4me | palendae there were some other playbook/role changes in rpc-openstack for ELK which may be required for that build to work - did you test it all? | 17:23 |
| palendae | Not yet | 17:23 |
| palendae | Will mark WIP | 17:24 |
| odyssey4me | ah ok, cool | 17:24 |
| palendae | Ideally we'd bump versions in Kilo...but those all have big changes | 17:24 |
| palendae | So that's not really on deck for Juno in my mind | 17:24 |
| *** bapalm_ has quit IRC | 17:25 | |
| palendae | We being RAX, not OSAD | 17:25 |
| *** abitha has quit IRC | 17:31 | |
| *** abitha has joined #openstack-ansible | 17:32 | |
| openstackgerrit | Merged stackforge/os-ansible-deployment: Add python-cinderclient to os_glance role https://review.openstack.org/207130 | 17:38 |
| *** daneyon has joined #openstack-ansible | 17:43 | |
| *** annashen has joined #openstack-ansible | 17:44 | |
| *** daneyon_ has joined #openstack-ansible | 17:44 | |
| *** daneyon has quit IRC | 17:47 | |
| *** annashen has quit IRC | 17:49 | |
| *** javeriak_ has quit IRC | 17:52 | |
| *** javeriak has joined #openstack-ansible | 17:52 | |
| *** TheIntern has joined #openstack-ansible | 17:54 | |
| *** annashen has joined #openstack-ansible | 17:56 | |
| *** woodard has joined #openstack-ansible | 17:59 | |
| *** woodard has quit IRC | 18:04 | |
| *** jmckind has quit IRC | 18:14 | |
| *** woodard has joined #openstack-ansible | 18:14 | |
| *** jmckind has joined #openstack-ansible | 18:14 | |
| *** jmckind has quit IRC | 18:15 | |
| *** ig0r__ has joined #openstack-ansible | 18:25 | |
| *** woodard has quit IRC | 18:26 | |
| meteorfox | quick question, I'm drafting the spec for the Rally role. I think I understand the role of an administrator, but is there are difference between deployer and operator? | 18:28 |
| meteorfox | in my mind the operator is the same as the deployer, in the case of RPC, that would be us (Rackspace) | 18:29 |
| *** javeriak_ has joined #openstack-ansible | 18:32 | |
| *** misc_ has joined #openstack-ansible | 18:34 | |
| *** jlvillal_ has joined #openstack-ansible | 18:34 | |
| *** jmccrory has quit IRC | 18:34 | |
| *** andymccr has quit IRC | 18:34 | |
| *** jlvillal has quit IRC | 18:34 | |
| *** misc has quit IRC | 18:34 | |
| *** jmccrory has joined #openstack-ansible | 18:34 | |
| *** javeriak has quit IRC | 18:35 | |
| *** jlvillal_ is now known as jlvillal | 18:35 | |
| *** woodard has joined #openstack-ansible | 18:39 | |
| *** alop has quit IRC | 18:40 | |
| *** britthouser has quit IRC | 18:43 | |
| *** britthouser has joined #openstack-ansible | 18:44 | |
| *** ig0r_ has quit IRC | 18:50 | |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Adds ceilometer link in configure.rst https://review.openstack.org/207576 | 18:54 |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Uses tox for automating documentation builds https://review.openstack.org/207577 | 18:54 |
| *** bapalm_ has joined #openstack-ansible | 19:00 | |
| dstanek | if we like those changes i'll propose a docs job to infra | 19:01 |
| dstanek | ^ | 19:01 |
| dstanek | late lunch time! | 19:03 |
| *** ig0r_ has joined #openstack-ansible | 19:04 | |
| *** bapalm_ has quit IRC | 19:05 | |
| palendae | dstanek: Will try to check this afternoon | 19:06 |
| alextricity | Hey..I have a big problem I was hoping someone might help me with. | 19:08 |
| *** jmckind has joined #openstack-ansible | 19:08 | |
| *** annashen has quit IRC | 19:08 | |
| alextricity | Turns out some servers that were provisioned for OSAD only have 4G on the / moun | 19:08 |
| *** jmckind has quit IRC | 19:08 | |
| sigmavirus24 | So fwiw, I think we should be prioritizing what we review and while I want dstanek's stuff, I'd like that to go in next week at the earliest | 19:08 |
| alextricity | Any way I can configure the containers to use the /var mount for storage? This mount was configured with 500G | 19:09 |
| palendae | sigmavirus24: True | 19:09 |
| *** jmckind has joined #openstack-ansible | 19:09 | |
| *** annashen has joined #openstack-ansible | 19:09 | |
| *** jmckind has quit IRC | 19:10 | |
| *** javeriak_ has quit IRC | 19:10 | |
| *** annashen has quit IRC | 19:10 | |
| *** jmckind has joined #openstack-ansible | 19:10 | |
| *** annashen has joined #openstack-ansible | 19:10 | |
| *** jmckind has quit IRC | 19:11 | |
| *** jmckind has joined #openstack-ansible | 19:12 | |
| *** jmckind has quit IRC | 19:13 | |
| *** jmckind has joined #openstack-ansible | 19:14 | |
| *** jmckind has quit IRC | 19:14 | |
| dstanek | sigmavirus24: i'm in no hurry. i saw my bug get tagged with a release number last night and addressed it this morning | 19:20 |
| *** jmckind has joined #openstack-ansible | 19:20 | |
| *** jmckind has quit IRC | 19:21 | |
| sigmavirus24 | dstanek: cool | 19:21 |
| sigmavirus24 | I appreciate the tox stuff myself | 19:22 |
| sigmavirus24 | I was hoping to tackle that + flake8 + bash8 etc for our stuff soonish | 19:22 |
| mgariepy | alextricity, you could create a directory on /var/ and bind-mount it to /openstack ? | 19:23 |
| *** jmckind has joined #openstack-ansible | 19:23 | |
| *** annashen has quit IRC | 19:26 | |
| *** annashen has joined #openstack-ansible | 19:26 | |
| *** jwagner is now known as jwagner_away | 19:27 | |
| alextricity | mgarie..don't know. How would I tell lxc to use that mount for disks? | 19:27 |
| *** andymccr has joined #openstack-ansible | 19:28 | |
| mgariepy | alextricity, containers will use /openstack/<container_name> folder, so everything will finish on the /var disk | 19:35 |
| mgariepy | alextricity, for my part tho, i use lvm on the nodes and will create /openstack with enough room on the lvm. | 19:37 |
| alextricity | Turns out I won't really get very far with a 4gb root partition on each node anyway | 19:37 |
| alextricity | I'm just going to re-provision | 19:38 |
| mgariepy | alextricity, http://osad.readthedocs.org/en/latest/install-guide/targethosts-configlvm.html | 19:38 |
| *** alop has joined #openstack-ansible | 19:38 | |
| mgariepy | alextricity, what are you using to provision your server ? | 19:38 |
| alextricity | just PXE | 19:38 |
| mgariepy | ok | 19:38 |
| alextricity | mount | 19:39 |
| *** annashen has quit IRC | 19:40 | |
| *** annashen has joined #openstack-ansible | 19:41 | |
| *** fawadkhaliq has quit IRC | 19:42 | |
| *** fawadkhaliq has joined #openstack-ansible | 19:43 | |
| *** openstackgerrit has quit IRC | 19:46 | |
| *** openstackgerrit has joined #openstack-ansible | 19:47 | |
| *** alop has quit IRC | 19:47 | |
| *** alop has joined #openstack-ansible | 19:50 | |
| *** TheIntern has quit IRC | 19:51 | |
| *** alop has quit IRC | 19:53 | |
| *** jwagner_away is now known as jwagner | 19:55 | |
| *** annashen has quit IRC | 19:56 | |
| *** ig0r_ has quit IRC | 19:56 | |
| *** annashen has joined #openstack-ansible | 19:57 | |
| *** woodard has quit IRC | 19:57 | |
| *** jmckind has quit IRC | 20:04 | |
| *** TheIntern has joined #openstack-ansible | 20:12 | |
| *** woodard has joined #openstack-ansible | 20:12 | |
| *** openstackgerrit has quit IRC | 20:16 | |
| *** openstackgerrit has joined #openstack-ansible | 20:17 | |
| *** woodard has quit IRC | 20:18 | |
| palendae | sigmavirus24 posted this http://lists.openstack.org/pipermail/openstack-dev/2015-July/071034.html | 20:33 |
| dstanek | sigmavirus24: ha, i have bash8 and spell checker patches that i haven't pushed yet | 20:34 |
| sigmavirus24 | dstanek: go go go go | 20:34 |
| palendae | dstanek: Run bash8 against run-upgrade.sh | 20:34 |
| *** jmckind has joined #openstack-ansible | 20:36 | |
| dstanek | sigmavirus24: i was just pulling over the keystone stuff i find useful | 20:39 |
| dstanek | palendae: yeah, that's why i haven't pushed yet :-P there are a few hundred things that need to be cleaned up | 20:39 |
| palendae | dstanek: laffo | 20:39 |
| sigmavirus24 | dstanek: naaaaw | 20:39 |
| palendae | Well, fwiw, I think the plan for that script in Liberty is to have most, if not all that logic removed | 20:40 |
| palendae | Because we're only doing jumps of 1 | 20:40 |
| palendae | Juno to Kilo, Kilo to Liberty, Liberty to M | 20:40 |
| palendae | So Kilo -> Liberty will not have the scrubbing of rax-isms | 20:40 |
| palendae | But yeah, the kilo version of that file needs to be linted with like, 4 programs | 20:40 |
| dstanek | so.... 588 bashate error(s) found | 20:42 |
| palendae | Right | 20:42 |
| dstanek | that's across all shell scripts | 20:42 |
| palendae | It's 50% bash, 50% yaml, 50% python | 20:42 |
| palendae | And 90% of the bash errors are probably in the bash :) | 20:42 |
| openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Fix Keystone URI/URL defaults https://review.openstack.org/205192 | 20:43 |
| dstanek | so the question is....is it worth it to have that stuff cleaned up? | 20:44 |
| sigmavirus24 | probably | 20:44 |
| sigmavirus24 | most of the bash scripts (besides run-upgrade.sh) keep with us | 20:44 |
| palendae | ^ | 20:45 |
| palendae | I would say exclude run-upgrade.sh | 20:45 |
| sigmavirus24 | for now | 20:45 |
| sigmavirus24 | dstanek: do we know anyone that can get us a new release of bash8? | 20:45 |
| sigmavirus24 | the one that's out is about a year old and missing many bug fixes | 20:45 |
| palendae | The in-line ansible playbooks were to prevent people from re-running them. However, i think a better approach in the future is to put them in an upgrade directory and use that | 20:45 |
| sigmavirus24 | == palendae | 20:46 |
| palendae | because we're leaving them behind in the current version | 20:46 |
| sigmavirus24 | that way ansible lint check will run against them | 20:46 |
| palendae | Similarly with any python scripts | 20:46 |
| palendae | Right | 20:46 |
| palendae | Each thing can then be linted | 20:46 |
| palendae | As intended | 20:46 |
| *** openstackgerrit has quit IRC | 20:46 | |
| palendae | Make an 'upgrade/' junk drawer | 20:46 |
| sigmavirus24 | clean it out every spring | 20:46 |
| *** openstackgerrit has joined #openstack-ansible | 20:47 | |
| palendae | s/spring/6 months/ | 20:47 |
| sigmavirus24 | spring comes every 6 months | 20:47 |
| palendae | More like ~8 months | 20:47 |
| dstanek | sigmavirus24: i don't know who runs that | 20:48 |
| sigmavirus24 | dstanek: me either | 20:49 |
| sigmavirus24 | I think the person who started it disappeared | 20:49 |
| sigmavirus24 | it's an openstack thing | 20:49 |
| dstanek | oh, it's joe gordon | 20:50 |
| sigmavirus24 | is it? | 20:50 |
| dstanek | he's at least one of the committers - since it's an openstack-dev thing i'm surprise that it hasn't seen a release | 20:51 |
| *** TheIntern has quit IRC | 20:52 | |
| *** JRobinson__ has joined #openstack-ansible | 20:52 | |
| dstanek | i'll submit a patch where i ignore all of the errors and another one for each check to start enabling the checks | 20:52 |
| dstanek | i have a vacation day tomorrow and a long drive so i might as well get something done | 20:53 |
| palendae | dstanek: Nah | 20:53 |
| palendae | I forget, are you presenting at PyOhio? | 20:53 |
| dstanek | no, i didn't have to have to worry about it at mid-cycle | 20:54 |
| dstanek | i would have probably had to work on it there instead of making friends and influencing people | 20:54 |
| palendae | Haha | 20:55 |
| *** jwagner is now known as jwagner_away | 20:56 | |
| *** yapeng has joined #openstack-ansible | 20:58 | |
| *** jmckind has quit IRC | 20:59 | |
| *** jmckind has joined #openstack-ansible | 21:00 | |
| *** openstack has joined #openstack-ansible | 21:11 | |
| Sam-I-Am | dstanek: ping | 21:11 |
| *** prad has quit IRC | 21:12 | |
| *** tlian has quit IRC | 21:13 | |
| *** annashen has quit IRC | 21:15 | |
| sigmavirus24 | dstanek: fly you fool | 21:16 |
| *** annashen has joined #openstack-ansible | 21:18 | |
| *** tomjoseka has joined #openstack-ansible | 21:20 | |
| Sam-I-Am | sigmavirus24: haha | 21:23 |
| *** yapeng has quit IRC | 21:24 | |
| *** yapeng has joined #openstack-ansible | 21:31 | |
| *** yapeng has quit IRC | 21:32 | |
| *** yapeng has joined #openstack-ansible | 21:38 | |
| *** yapeng has quit IRC | 21:38 | |
| *** markvoelker_ has quit IRC | 21:46 | |
| dstanek | Sam-I-Am: pong | 21:49 |
| *** alop has joined #openstack-ansible | 21:49 | |
| Sam-I-Am | dstanek: you were asking me about some ceilometer docs? | 21:50 |
| Sam-I-Am | or asking someone... | 21:50 |
| *** ig0r__ has quit IRC | 21:51 | |
| dstanek | Sam-I-Am: i didn't know where to link it so i did this: https://review.openstack.org/207576 | 21:52 |
| Sam-I-Am | dstanek: o hey cool, there's another patch underneath this one | 21:54 |
| Sam-I-Am | something we needed to do | 21:54 |
| dstanek | Sam-I-Am: i added a tox target for docs and i wanted it to fail for warnings and the ceilometer thing was the only warning | 21:55 |
| Sam-I-Am | yep, thats slick | 21:55 |
| *** jmckind has quit IRC | 21:55 | |
| Sam-I-Am | not sure why ceilometer wasn't fully added. i didnt see it come through. | 21:55 |
| dstanek | i have a few more in the queue to add bashate and some others things i find useful | 21:56 |
| dstanek | just have to polish the code a little | 21:56 |
| Sam-I-Am | those look pretty good | 21:57 |
| Sam-I-Am | is there any CI on that stuff? | 21:57 |
| Sam-I-Am | i think our docs ci job broke (or something) | 21:57 |
| dstanek | i plan on adding ci jobs for these | 21:57 |
| dstanek | do we currently publish the docs anywhere? | 21:58 |
| Sam-I-Am | yeah... sort of | 21:59 |
| Sam-I-Am | they're on readthedocs | 21:59 |
| Sam-I-Am | (which is sort of funny, because no one ever does) | 21:59 |
| Sam-I-Am | i dont see why the ci couldn't publish them somewhere closer to the repo | 22:00 |
| Sam-I-Am | not sure what the plans are for docs with big tent stuff | 22:01 |
| Sam-I-Am | devref is not the place, imho... unless its dev stuff | 22:01 |
| dstanek | no idea, but when i make the docs job i'll ask infra | 22:01 |
| *** yapeng has joined #openstack-ansible | 22:01 | |
| dstanek | i would think this would go on docs.openstack.org | 22:01 |
| *** annashen has quit IRC | 22:02 | |
| *** yapeng has quit IRC | 22:03 | |
| Sam-I-Am | dstanek: thats my thought too | 22:03 |
| Sam-I-Am | but i dont know if its set up for that yet | 22:04 |
| *** sdake has quit IRC | 22:05 | |
| *** sdake has joined #openstack-ansible | 22:05 | |
| *** phalmos has quit IRC | 22:08 | |
| *** JRobinson__ has quit IRC | 22:08 | |
| *** markvoelker has joined #openstack-ansible | 22:08 | |
| *** Mudpuppy has quit IRC | 22:10 | |
| *** sdake has quit IRC | 22:10 | |
| *** sdake has joined #openstack-ansible | 22:11 | |
| dstanek | Sam-I-Am: i'll let you know when i start poking at the CI jobs | 22:14 |
| Sam-I-Am | dstanek: yeh, keep me in the loop | 22:15 |
| *** yapeng has joined #openstack-ansible | 22:15 | |
| *** yapeng has quit IRC | 22:18 | |
| *** markvoelker_ has joined #openstack-ansible | 22:32 | |
| *** markvoelker has quit IRC | 22:35 | |
| *** harlowja has quit IRC | 22:44 | |
| *** harlowja has joined #openstack-ansible | 22:44 | |
| *** KLevenstein has quit IRC | 22:52 | |
| *** markvoelker_ has quit IRC | 23:14 | |
| *** Mudpuppy has joined #openstack-ansible | 23:21 | |
| *** jmckind has joined #openstack-ansible | 23:26 | |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Adds a pep8 target to tox.ini https://review.openstack.org/207662 | 23:27 |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Removes trailing whitespace for bashate https://review.openstack.org/207663 | 23:27 |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Removes tab indents for bashate https://review.openstack.org/207664 | 23:27 |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Fixes deprecated arithmetic expansion for bashate https://review.openstack.org/207665 | 23:27 |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Fixes function declarations for bashate https://review.openstack.org/207666 | 23:27 |
| openstackgerrit | David Stanek proposed stackforge/os-ansible-deployment: Fixes loops for bashate https://review.openstack.org/207667 | 23:27 |
| *** spotz is now known as spotz_zzz | 23:28 | |
| *** daneyon_ has quit IRC | 23:41 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!