*** sdake_ has joined #openstack-ansible | 00:07 | |
*** sdake has quit IRC | 00:11 | |
*** sdake has joined #openstack-ansible | 00:18 | |
*** sdake_ has quit IRC | 00:21 | |
*** darrenc is now known as darrenc_afk | 00:34 | |
*** annashen_ has joined #openstack-ansible | 00:35 | |
*** annashen_ has quit IRC | 00:40 | |
*** darrenc_afk is now known as darrenc | 01:00 | |
*** javeriak has quit IRC | 01:08 | |
*** mmasaki_ has joined #openstack-ansible | 01:12 | |
*** sdake_ has joined #openstack-ansible | 01:29 | |
*** sdake has quit IRC | 01:32 | |
*** alop has quit IRC | 01:36 | |
*** annashen has joined #openstack-ansible | 01:36 | |
*** sdake has joined #openstack-ansible | 01:38 | |
*** annashen has quit IRC | 01:41 | |
*** sdake_ has quit IRC | 01:41 | |
*** mmasaki_ has quit IRC | 02:07 | |
*** mmasaki has joined #openstack-ansible | 02:14 | |
*** mmasaki has quit IRC | 02:19 | |
*** tlian2 has joined #openstack-ansible | 02:19 | |
*** mmasaki has joined #openstack-ansible | 02:19 | |
*** tlian has quit IRC | 02:21 | |
*** annashen has joined #openstack-ansible | 02:37 | |
*** annashen has quit IRC | 02:42 | |
openstackgerrit | Miguel Grinberg proposed stackforge/os-ansible-deployment: [WIP] Keystone SP configuration https://review.openstack.org/194395 | 03:08 |
---|---|---|
*** tlian2 has quit IRC | 03:11 | |
*** annashen has joined #openstack-ansible | 03:38 | |
*** annashen has quit IRC | 03:43 | |
*** darrenc is now known as darrenc_afk | 04:15 | |
*** darrenc_afk is now known as darrenc | 04:30 | |
*** annashen has joined #openstack-ansible | 04:39 | |
*** shausy has joined #openstack-ansible | 04:41 | |
*** annashen has quit IRC | 04:43 | |
*** sdake_ has joined #openstack-ansible | 04:58 | |
*** sdake has quit IRC | 05:00 | |
*** sdake has joined #openstack-ansible | 05:00 | |
*** fawadkhaliq has joined #openstack-ansible | 05:03 | |
*** sdake_ has quit IRC | 05:03 | |
*** markvoelker has quit IRC | 05:32 | |
*** annashen_ has joined #openstack-ansible | 05:40 | |
*** annashen_ has quit IRC | 05:44 | |
*** ig0r__ has joined #openstack-ansible | 05:52 | |
*** ig0r_ has quit IRC | 05:55 | |
*** prometheanfire has quit IRC | 06:03 | |
*** radek__ has joined #openstack-ansible | 06:07 | |
*** subscope has quit IRC | 06:25 | |
*** markvoelker has joined #openstack-ansible | 06:32 | |
*** markvoelker has quit IRC | 06:37 | |
*** subscope has joined #openstack-ansible | 06:38 | |
*** annashen_ has joined #openstack-ansible | 06:40 | |
*** javeriak has joined #openstack-ansible | 06:46 | |
*** annashen_ has quit IRC | 06:46 | |
*** fawadkhaliq has quit IRC | 07:21 | |
*** fawadk has joined #openstack-ansible | 07:21 | |
*** b3rnard0 has quit IRC | 07:25 | |
*** cloudnull has quit IRC | 07:26 | |
*** shausy has quit IRC | 07:26 | |
*** sigmavirus24_awa has quit IRC | 07:27 | |
*** shausy has joined #openstack-ansible | 07:27 | |
*** d34dh0r53 has quit IRC | 07:30 | |
*** eglute has quit IRC | 07:30 | |
*** dolphm has quit IRC | 07:30 | |
*** jwagner_away has quit IRC | 07:30 | |
*** stevelle has quit IRC | 07:32 | |
odyssey4me | o/ | 07:38 |
*** odyssey4me has quit IRC | 07:45 | |
*** prometheanfire has joined #openstack-ansible | 07:49 | |
*** odyssey4me has joined #openstack-ansible | 08:07 | |
odyssey4me | hughsaunders you around yet? | 08:09 |
hughsaunders | yep | 08:10 |
*** javeriak has quit IRC | 08:13 | |
odyssey4me | hughsaunders why is the SP patch adding a user? | 08:23 |
hughsaunders | odyssey4me: the user that represents all the remote users | 08:23 |
odyssey4me | hughsaunders that wasn't necessary in my testing | 08:24 |
odyssey4me | as long as you have a group/role mapping to a domain/project, you're good to go :) | 08:24 |
hughsaunders | ok, will remove that step | 08:25 |
*** fawadk has quit IRC | 08:27 | |
*** markvoelker has joined #openstack-ansible | 08:34 | |
*** fawadkhaliq has joined #openstack-ansible | 08:35 | |
*** markvoelker has quit IRC | 08:38 | |
*** annashen has joined #openstack-ansible | 08:42 | |
*** odyssey4me_ has joined #openstack-ansible | 08:42 | |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: Add v3 calls for federation to keystone module https://review.openstack.org/197677 | 08:44 |
*** odyssey4me_ has quit IRC | 08:47 | |
*** annashen has quit IRC | 08:47 | |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone SSL cert/key distribution and configuration https://review.openstack.org/194474 | 09:01 |
*** fawadkhaliq has quit IRC | 09:04 | |
*** fawadkhaliq has joined #openstack-ansible | 09:05 | |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: Don't deploy ceilometer by default https://review.openstack.org/200252 | 09:07 |
*** fawadkhaliq has quit IRC | 09:23 | |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: SSL support for haproxy https://review.openstack.org/198957 | 09:35 |
*** fawadkhaliq has joined #openstack-ansible | 09:36 | |
*** annashen has joined #openstack-ansible | 09:43 | |
*** radek_ has joined #openstack-ansible | 09:47 | |
*** annashen has quit IRC | 09:47 | |
*** radek__ has quit IRC | 09:47 | |
*** radek__ has joined #openstack-ansible | 09:50 | |
*** radek_ has quit IRC | 09:53 | |
hughsaunders | odyssey4me: why is the ksv3 patch on top of the insecure patch? | 10:06 |
odyssey4me | hughsaunders just to merge them cleanly - unfortunately the haproxy config is broken in my last patch set - working on that now | 10:06 |
*** radek_ has joined #openstack-ansible | 10:07 | |
*** radek__ has quit IRC | 10:09 | |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone SSL cert/key distribution and configuration https://review.openstack.org/194474 | 10:13 |
odyssey4me | hughsaunders ah sorry, the insecure patch is fine - I got confused with the keystone ssl patch | 10:14 |
odyssey4me | I did the dependency to merge them cleanly because they conflicted. I shouldn't have done that - it was a bad shortcut. | 10:15 |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone SSL cert/key distribution and configuration https://review.openstack.org/194474 | 10:27 |
git-harry | odyssey4me: is that keystone patch a wip? | 10:29 |
git-harry | I started reviewing it but you keep uploading new patchsets | 10:30 |
odyssey4me | git-harry it wasn't, then I found a bug - fixed now | 10:30 |
odyssey4me | sorry - ready for review again | 10:30 |
git-harry | okay, np just checking | 10:30 |
*** markvoelker has joined #openstack-ansible | 10:35 | |
*** markvoelker has quit IRC | 10:39 | |
*** annashen has joined #openstack-ansible | 10:44 | |
*** annashen has quit IRC | 10:48 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Fixed minor typo in etc/README.rst "Global Options" section https://review.openstack.org/199856 | 10:53 |
*** eglute has joined #openstack-ansible | 10:53 | |
*** dolphm has joined #openstack-ansible | 10:53 | |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Fixed minor typo in etc/README.rst "Global Options" section https://review.openstack.org/200469 | 10:55 |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Fixed minor typo in etc/README.rst "Global Options" section https://review.openstack.org/200470 | 10:56 |
*** fawadkhaliq has quit IRC | 11:10 | |
*** odyssey4me has quit IRC | 11:20 | |
hughsaunders | odyssey4me: annoying that choices is case sensitive :( | 11:20 |
*** fawadkhaliq has joined #openstack-ansible | 11:22 | |
hughsaunders | od<tab> come back | 11:27 |
* hughsaunders smacks ZNC around a bit with a wet fish | 11:28 | |
*** markvoelker has joined #openstack-ansible | 11:35 | |
*** tlian has joined #openstack-ansible | 11:35 | |
*** markvoelker has quit IRC | 11:40 | |
*** fawadkhaliq has quit IRC | 11:44 | |
*** annashen has joined #openstack-ansible | 11:45 | |
*** fawadkhaliq has joined #openstack-ansible | 11:47 | |
*** annashen has quit IRC | 11:50 | |
*** markvoelker has joined #openstack-ansible | 12:07 | |
*** fawadkhaliq has quit IRC | 12:15 | |
*** ig0r__ has quit IRC | 12:17 | |
*** javeriak has joined #openstack-ansible | 12:20 | |
*** jaypipes has joined #openstack-ansible | 12:20 | |
*** ig0r_ has joined #openstack-ansible | 12:23 | |
mancdaz | hughsaunders is this still needing to be an open bug https://bugs.launchpad.net/openstack-ansible/+bug/1445421 ? | 12:30 |
openstack | Launchpad bug 1445421 in openstack-ansible trunk "Juno to Kilo upgrades should be tested" [High,Triaged] | 12:30 |
hughsaunders | mancdaz: I'm not planning to commit any more code against it, but I didn't close it because we still don't have upgrade testing. | 12:31 |
hughsaunders | in short: yesno | 12:31 |
mancdaz | hughsaunders I would postulate that would really need to be a blueprint | 12:31 |
mancdaz | blueprint/spec | 12:32 |
hughsaunders | ah yes, that is from a process long ago | 12:32 |
mancdaz | in an ansible galaxy far away? | 12:32 |
hughsaunders | yep | 12:32 |
hughsaunders | mancdaz: I was asked about this bug a few days ago by theIntern, I said it could be closed, obviously didn't get close though.. | 12:33 |
openstackgerrit | git-harry proposed stackforge/os-ansible-deployment: Restart mysql when config changed https://review.openstack.org/200054 | 12:36 |
*** ccrouch has joined #openstack-ansible | 12:37 | |
*** annashen has joined #openstack-ansible | 12:45 | |
*** odyssey4me has joined #openstack-ansible | 12:47 | |
mancdaz | git-harry https://review.openstack.org/#/c/200054/3/playbooks/library/mysql_status_facts | 12:48 |
mancdaz | do you not need to handle a failure in the call to check_output? | 12:49 |
mancdaz | and fail gracefully? | 12:49 |
git-harry | mancdaz: no | 12:49 |
mancdaz | ok, I wasn't sure | 12:50 |
mancdaz | why not? | 12:50 |
git-harry | It might look nicer if I did, but it works if I don't | 12:50 |
*** sdake_ has joined #openstack-ansible | 12:50 | |
*** annashen has quit IRC | 12:50 | |
mancdaz | I see other mdules calling self.module.fail_json, just wondered why that's not required | 12:51 |
git-harry | mancdaz: I was being lazy | 12:51 |
mancdaz | git-harry I gathered | 12:52 |
mancdaz | the consequence being you don't pass back a nice failure message? | 12:52 |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Enable all services to use Keystone 'insecurely' https://review.openstack.org/199307 | 12:52 |
git-harry | mancdaz: essentially it means you see the python trackback | 12:53 |
mancdaz | git-harry does ansible capture the exception for you? | 12:53 |
mancdaz | right | 12:53 |
git-harry | so it's informative | 12:53 |
git-harry | but not necessarily formatted as nicely as it could be | 12:53 |
*** sdake has quit IRC | 12:54 | |
*** cloudnull has joined #openstack-ansible | 12:56 | |
*** javeriak has quit IRC | 12:56 | |
cloudnull | Morning. | 12:57 |
mancdaz | hi cloudnull | 12:57 |
odyssey4me | hughsaunders wut? | 13:00 |
cloudnull | How's it mancdaz | 13:01 |
mancdaz | cloudnull pretty good, how bout you? | 13:01 |
cloudnull | Very well. | 13:01 |
hughsaunders | odyssey4me: Instead of adding (default) after the default choice, specify default: <default value> | 13:01 |
odyssey4me | hughsaunders ah, I see that in the keystone module | 13:04 |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Enable all services to use Keystone 'insecurely' https://review.openstack.org/199307 | 13:06 |
odyssey4me | hughsaunders I started with the glance module, which had the bad convention as an example that I copied | 13:06 |
odyssey4me | that's now resolve :) | 13:07 |
hughsaunders | odyssey4me: thanks :) | 13:07 |
openstackgerrit | git-harry proposed stackforge/os-ansible-deployment: Restart mysql when config changed https://review.openstack.org/200054 | 13:17 |
*** fawadkhaliq has joined #openstack-ansible | 13:18 | |
*** TheIntern has joined #openstack-ansible | 13:22 | |
*** b3rnard0 has joined #openstack-ansible | 13:27 | |
*** galstrom_zzz is now known as galstrom | 13:32 | |
alextricity | hey | 13:32 |
alextricity | So is master still blocked? | 13:32 |
odyssey4me | alextricity we did spot the bugs for why switching to 'no' wasn't working - but master isn't blocked | 13:33 |
odyssey4me | it is, unfortunately, not very good at being successful :( | 13:33 |
odyssey4me | check the updated version of the unblock master patch and see the review comments and updated patch sets | 13:34 |
alextricity | odyssey4me: Interesting..alright. Yeah I'll check it out. Thanks! | 13:35 |
alextricity | odyssey4me: Good suggestion with using the mongo ping command | 13:38 |
alextricity | I'll create a new patch set with that DEPLOY_CEILOMETER conditional fixed, and a proper wait for the mongodb service. | 13:39 |
odyssey4me | alextricity sure - if you can rework that patch into a bug fix and register a bug in launchpad for it then that'd be great | 13:39 |
odyssey4me | no need for a new patch set - just undo the disabling bits and fix up the other bits | 13:39 |
odyssey4me | then add a bug ref in the commit msg | 13:39 |
alextricity | odyssey4me: Sure thing. I appreciate it man! | 13:40 |
*** annashen has joined #openstack-ansible | 13:46 | |
*** sdake_ has quit IRC | 13:51 | |
*** annashen has quit IRC | 13:51 | |
*** fawadkhaliq has quit IRC | 13:54 | |
TheIntern | Hey, so does https://bugs.launchpad.net/openstack-ansible/+bug/1473230 need a fix? It seems like it is upstream. | 13:55 |
openstack | Launchpad bug 1473230 in openstack-ansible "Reduce neutron configuration" [Low,Triaged] | 13:55 |
odyssey4me | TheIntern it refers to changes made upstream in docs and indicates that we could reduce our config file size by learning from the upstream changes | 13:56 |
TheIntern | Ah, so learn the upstream changes and then apply them in the configs. | 13:57 |
*** sigmavirus24_awa has joined #openstack-ansible | 14:04 | |
odyssey4me | TheIntern yep :) | 14:04 |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:04 | |
TheIntern | Let's see what I can learn before I start touching config files | 14:04 |
*** sigmavirus24 is now known as sigmavirus24_awa | 14:05 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:05 | |
*** shausy has quit IRC | 14:14 | |
*** Mudpuppy has joined #openstack-ansible | 14:15 | |
*** galstrom is now known as galstrom_zzz | 14:17 | |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: [DO NOT MERGE] Test to isolate devstack-trusty-hpcloud-b4 issue https://review.openstack.org/200557 | 14:17 |
mattt | ^^^ odyssey4me let's see if that does it | 14:21 |
odyssey4me | mattt we could make it permanent if: 1) an env var set whether to do that or not, 2) we collect the apt logs as suggested by cloudnull | 14:22 |
mattt | yeah but those aren't logged to apt logs | 14:22 |
palendae | I don't think collecting the apt logs is a bad idea | 14:22 |
mattt | i want to do a bit of testing first to see how useful this is | 14:22 |
palendae | Even ignoring hpb4 | 14:22 |
mattt | i can happily make this more permanent tho, let's see how a run or two goes first | 14:23 |
palendae | Sure | 14:23 |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: Wait until mongo responds after restart https://review.openstack.org/200252 | 14:24 |
palendae | More eyes/runs of https://review.openstack.org/#/c/198311/ would be helpful. Upgrades aren't tested in a gate, so we'll need to do this manually | 14:25 |
*** galstrom_zzz is now known as galstrom | 14:26 | |
mattt | things you didn't want to see in gate: "reverify - unrelated mongo issue" | 14:27 |
* mattt sads | 14:27 | |
odyssey4me | mattt you have a bug there - check the review :) | 14:27 |
odyssey4me | alextricity ^ hughsaunders appears to be doing a patch to fix things | 14:27 |
palendae | mattt: Well I don't like seeing reverify - hpcloud either :( | 14:28 |
mattt | odyssey4me: derp! -e not needed, but can add anyway | 14:28 |
alextricity | odyssey4me: hughsaunders Thanks! I have my hands tied with a customer right now. I really appreciate it guys. you rock! | 14:28 |
hughsaunders | odyssey4me: only implementing your comments really, since the patch to disable testing had 2x -2, I thought we should probably start fixing it. | 14:29 |
palendae | hughsaunders: Good call; unfortunately I'm on rpc-openstack QE blockers :( | 14:29 |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: [DO NOT MERGE] Test to isolate devstack-trusty-hpcloud-b4 issue https://review.openstack.org/200557 | 14:30 |
palendae | hughsaunders: Heh, didn't realize you just renamed the review. Sneaky | 14:31 |
hughsaunders | palendae: twice | 14:31 |
palendae | mattt: -W'ing that change should make it gate and prevent someone from merging, no? | 14:31 |
mattt | palendae: yep good call | 14:33 |
*** d34dh0r53 has joined #openstack-ansible | 14:36 | |
hughsaunders | cloudnull, palendae, this no longer disables ceilometer testing: https://review.openstack.org/#/c/200252/5 | 14:40 |
palendae | hughsaunders: Thanks | 14:40 |
*** jwagner has joined #openstack-ansible | 14:44 | |
*** annashen has joined #openstack-ansible | 14:47 | |
*** markvoelker has quit IRC | 14:48 | |
*** annashen has quit IRC | 14:52 | |
*** britthouser has quit IRC | 14:53 | |
hughsaunders | anyone free to review odyssey4me's keystone insecure patch: https://review.openstack.org/#/c/199307/ bottom of a growing chain | 14:57 |
odyssey4me | we also need more eyes on this spec, ideally to merge it today: https://review.openstack.org/194147 | 14:59 |
*** jaypipes has quit IRC | 14:59 | |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: Upgrade the Keystone library to use v3 https://review.openstack.org/196943 | 15:00 |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: Add v3 calls for federation to keystone module https://review.openstack.org/197677 | 15:00 |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: [WIP] Keystone IdP configuration https://review.openstack.org/194259 | 15:01 |
*** weezS has joined #openstack-ansible | 15:01 | |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: [WIP] Keystone SP configuration https://review.openstack.org/194395 | 15:01 |
mancdaz | git-harry after properly testing, I actually got your mysql restart patch to work | 15:05 |
mancdaz | nice | 15:06 |
git-harry | mancdaz: actually, you were right the first time | 15:06 |
git-harry | There is a mistake. | 15:06 |
mancdaz | wait what? | 15:06 |
mancdaz | it just worked | 15:07 |
git-harry | Yes, but if you used the tags it wouldn't | 15:07 |
mancdaz | git-harry the first time when it failed when I used tags? | 15:07 |
mancdaz | right | 15:07 |
mancdaz | I assume that's the case for a lot of our tagged tasks | 15:07 |
git-harry | Yes, I added a variable to the role but it's not set by default | 15:08 |
mancdaz | ah ok | 15:08 |
openstackgerrit | Merged stackforge/os-ansible-deployment-specs: Keystone Federation Deployment https://review.openstack.org/194147 | 15:13 |
*** ig0r__ has joined #openstack-ansible | 15:19 | |
*** sdake has joined #openstack-ansible | 15:20 | |
*** ig0r_ has quit IRC | 15:22 | |
odyssey4me | \o/ :) | 15:27 |
odyssey4me | now we need some eyes on this one: https://review.openstack.org/194255 | 15:27 |
b3rnard0 | the first spec is away! | 15:28 |
openstackgerrit | Andy McCrae proposed stackforge/os-ansible-deployment: Adjust key distribution mechanism for Swift https://review.openstack.org/199992 | 15:30 |
*** galstrom is now known as galstrom_zzz | 15:35 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Fix scripts/run-upgrade.sh https://review.openstack.org/198311 | 15:35 |
openstackgerrit | Darren Birkett proposed stackforge/os-ansible-deployment: Fix scripts/run-upgrade.sh https://review.openstack.org/200597 | 15:36 |
*** annashen has joined #openstack-ansible | 15:48 | |
*** annashen has quit IRC | 15:53 | |
*** radek__ has joined #openstack-ansible | 15:53 | |
*** radek_ has quit IRC | 15:55 | |
mancdaz | palendae cloudnull d34dh0r53 the second check of the mongodb is successful, and merges, it might be worth rebasing the other changes in the queue to incorporate that fix, so they don't all need to be rechecked 17 times to merge | 15:57 |
cloudnull | https://review.openstack.org/197677 https://review.openstack.org/194259 https://review.openstack.org/194395 are all failing lint checks | 15:57 |
mancdaz | this one I mean https://review.openstack.org/#/c/200252/ | 15:57 |
palendae | mancdaz: Ok. I triggered another recheck out of paranoia | 15:59 |
mancdaz | palendae yeah I meant after that one (hopefully) works, and that patch merges | 16:01 |
palendae | mancdaz: Ah, looks like you missed an 'if' at the start of your sentence :) | 16:01 |
b3rnard0 | swift team room is 4827 | 16:01 |
*** alop has joined #openstack-ansible | 16:01 | |
mancdaz | palendae oh yeah | 16:01 |
palendae | But yeah, that's probably a good idea in any case | 16:01 |
palendae | As long as it works it removes that pain | 16:02 |
*** stevelle has joined #openstack-ansible | 16:04 | |
*** radek__ has quit IRC | 16:11 | |
*** yaya has joined #openstack-ansible | 16:18 | |
*** sdake_ has joined #openstack-ansible | 16:19 | |
*** sdake has quit IRC | 16:22 | |
openstackgerrit | Jesse Pretorius proposed stackforge/os-ansible-deployment: Keystone SSL cert/key distribution and configuration https://review.openstack.org/194474 | 16:28 |
*** markvoelker has joined #openstack-ansible | 16:31 | |
*** galstrom_zzz is now known as galstrom | 16:40 | |
sigmavirus24 | So https://review.openstack.org/#/c/196943/ needs to wait for https://review.openstack.org/#/c/199307/ to merge so I can rebase everything off of master to get the ceilometer work into it and can update the endpoint work | 16:41 |
sigmavirus24 | wooo | 16:41 |
*** tlian has quit IRC | 16:41 | |
sigmavirus24 | let's hope 199307 doesn't have to be reverified all day long | 16:41 |
*** galstrom is now known as galstrom_zzz | 16:42 | |
sigmavirus24 | (another reason not to needlessly make patches dependent on each other) | 16:42 |
* odyssey4me beats himself up | 16:42 | |
openstackgerrit | Hugh Saunders proposed stackforge/os-ansible-deployment: Add v3 calls for federation to keystone module https://review.openstack.org/197677 | 16:42 |
hughsaunders | miguelgrinberg: pushed domain bits to https://review.openstack.org/#/c/197677/ | 16:43 |
miguelgrinberg | hughsaunders: thanks! | 16:43 |
*** sdake_ has quit IRC | 16:43 | |
odyssey4me | sigmavirus24 nice catch, obviously the ceilometer role wasn't around at the patch's inception | 16:43 |
sigmavirus24 | odyssey4me: Not trying to beat up on you | 16:44 |
sigmavirus24 | Just hoping the rest of the channel learns from our sins | 16:44 |
sigmavirus24 | That wedges the whole patch chain now though | 16:44 |
sigmavirus24 | I can't fix that patch's gate until yours merges and I can rebase | 16:44 |
*** sdake has joined #openstack-ansible | 16:45 | |
odyssey4me | sigmavirus24 I know, I know. :/ | 16:47 |
* sigmavirus24 goes back to fixing projects broken by mock 1.1.0 | 16:48 | |
*** annashen has joined #openstack-ansible | 16:49 | |
*** annashen has quit IRC | 16:50 | |
*** annashen has joined #openstack-ansible | 16:50 | |
hughsaunders | miguelgrinberg: also pushed to sp patch, added domain creation task and removed keystone lib fixes. Haven't added those to the v3 patch though. | 16:50 |
hughsaunders | Am off now, have a great afternoon mericans! | 16:51 |
miguelgrinberg | hughsaunders: have a good one | 16:51 |
openstackgerrit | Merged stackforge/os-ansible-deployment: Enable all services to use Keystone 'insecurely' https://review.openstack.org/199307 | 16:53 |
*** yaya has quit IRC | 16:57 | |
*** alextricity has quit IRC | 16:57 | |
*** bitblt has joined #openstack-ansible | 17:01 | |
*** yaya has joined #openstack-ansible | 17:05 | |
miguelgrinberg | hughsaunders: you're probably gone but in case you are still there, I don't see a new SP patch from you, looks like it did not go through. | 17:05 |
cloudnull | odyssey4me: mattt: it seems that hpb4 is still unhappy and theres not additional logging in the current patch set failed: [aio1_memcached_container-80216962] => {"attempts": 5, "failed": true, "parsed": false} < failed on apt-get update | 17:09 |
*** yaya has quit IRC | 17:10 | |
*** yaya has joined #openstack-ansible | 17:11 | |
*** britthouser has joined #openstack-ansible | 17:13 | |
*** yaya_ has joined #openstack-ansible | 17:16 | |
*** yaya has quit IRC | 17:17 | |
*** yaya_ is now known as yaya | 17:17 | |
*** sdake_ has joined #openstack-ansible | 17:19 | |
*** sdake has quit IRC | 17:23 | |
*** jmckind has joined #openstack-ansible | 17:25 | |
*** yaya has quit IRC | 17:36 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Fixed minor typo in etc/README.rst "Global Options" section https://review.openstack.org/200470 | 17:48 |
openstackgerrit | Merged stackforge/os-ansible-deployment: Fixed minor typo in etc/README.rst "Global Options" section https://review.openstack.org/200469 | 17:48 |
*** galstrom_zzz is now known as galstrom | 17:49 | |
*** javeriak has joined #openstack-ansible | 17:50 | |
*** sdake has joined #openstack-ansible | 17:55 | |
*** alop has quit IRC | 17:56 | |
*** alop has joined #openstack-ansible | 17:57 | |
*** sdake_ has quit IRC | 17:59 | |
*** yaya has joined #openstack-ansible | 18:09 | |
*** sdake_ has joined #openstack-ansible | 18:12 | |
*** yaya has quit IRC | 18:13 | |
*** sdake has quit IRC | 18:16 | |
*** sdake_ is now known as sdake | 18:25 | |
*** javeriak has quit IRC | 18:42 | |
*** annashen has quit IRC | 18:45 | |
*** galstrom is now known as galstrom_zzz | 18:49 | |
*** logan2 has quit IRC | 18:51 | |
*** logan2 has joined #openstack-ansible | 18:52 | |
*** javeriak has joined #openstack-ansible | 19:02 | |
*** georgem1 has joined #openstack-ansible | 19:03 | |
*** d34dh0r53 is now known as th31nt3rn | 19:05 | |
*** alop has quit IRC | 19:07 | |
*** galstrom_zzz is now known as galstrom | 19:10 | |
sigmavirus24 | cloudnull: do we agree with this level of indirection? https://github.com/stackforge/os-ansible-deployment/blob/master/playbooks/roles/os_ceilometer/tasks/ceilometer_service_setup.yml | 19:16 |
*** ig0r__ has quit IRC | 19:19 | |
*** ig0r_ has joined #openstack-ansible | 19:20 | |
cloudnull | what ? | 19:25 |
*** yaya has joined #openstack-ansible | 19:25 | |
*** annashen has joined #openstack-ansible | 19:26 | |
cloudnull | sigmavirus24: its similar to how we do other service additions https://github.com/stackforge/os-ansible-deployment/blob/master/playbooks/roles/os_nova/tasks/nova_service_setup.yml | 19:26 |
sigmavirus24 | cloudnull: ah okay i hadn't seen that | 19:27 |
sigmavirus24 | most other services have it with less indirection | 19:27 |
cloudnull | maybe not amazing way to do it | 19:27 |
sigmavirus24 | No it's fine | 19:27 |
sigmavirus24 | It caught me off gurad | 19:27 |
sigmavirus24 | *guard | 19:27 |
cloudnull | how are thing s? | 19:28 |
sigmavirus24 | Going well. Just helped me catch a potential problem with nova's service endpoints being added properly :D | 19:31 |
sigmavirus24 | nevermind. I had already caught those | 19:32 |
sigmavirus24 | heh | 19:32 |
openstackgerrit | Miguel Grinberg proposed stackforge/os-ansible-deployment: [WIP] Keystone SP configuration https://review.openstack.org/194395 | 19:32 |
cloudnull | fix all the bugs ! :) | 19:34 |
cloudnull | or make more, whats ever comes first . | 19:35 |
*** odyssey4me has quit IRC | 19:37 | |
sigmavirus24 | something like that | 19:37 |
javeriak | hey guys, is OSAD modifying iptables directly through ansible anywhere in the plays, i could only find 'roles/lxc_common/files/lxc-system-manage' and that seems to be doing stuff in bulk for the containers | 19:41 |
*** yaya has quit IRC | 19:41 | |
palendae | javeriak: To my knowledge, no...but the bootstrap and gating scripts definitely do | 19:41 |
palendae | So if you're doing an all-in-one build they'll be changing | 19:41 |
javeriak | palendae: yep i see those, they are bash scripts though, was wondering if there was a clean way thru ansible | 19:42 |
palendae | Right; I don't know of any plays/roles that modify iptables within OSAD | 19:43 |
*** th31nt3rn is now known as d34dh0r53 | 19:45 | |
miguelgrinberg | sigmavirus24: FYI the latest rev of the keystone v3 patch has domain and domain_name args. Most commands use domain, but hughsaunders added ensure_domain, which uses domain_name. Do we want to go with domain_name for everything, since we use project_name, user_name, etc? | 19:50 |
sigmavirus24 | miguelgrinberg: yeah I was updating my patch with taht stuff | 19:51 |
miguelgrinberg | ah, good, then I'll wait for the update | 19:51 |
sigmavirus24 | Yeah working on it now | 19:51 |
sigmavirus24 | sorry for the delay | 19:51 |
miguelgrinberg | I'm not blocked, I can use it as it is for now | 19:51 |
*** alop has joined #openstack-ansible | 19:52 | |
*** sdake_ has joined #openstack-ansible | 19:52 | |
*** sdake has quit IRC | 19:56 | |
cloudnull | javeriak: do you need to create some rules or remove them ? | 19:58 |
javeriak | cloudnull: create | 19:58 |
cloudnull | theres https://github.com/geerlingguy/ansible-role-firewall that should work to all for simple iptables rule management. | 19:59 |
*** galstrom is now known as galstrom_zzz | 19:59 | |
cloudnull | IE https://galaxy.ansible.com/list#/roles/451 | 20:01 |
cloudnull | so that could be added to the role as a meta dep, then installed using the role requirements file | 20:01 |
*** britthou_ has joined #openstack-ansible | 20:04 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Sort dynamic inventory files in juno https://review.openstack.org/199229 | 20:04 |
*** yaya has joined #openstack-ansible | 20:04 | |
javeriak | cloudnull: thanks! i'll give it a try | 20:06 |
*** britthouser has quit IRC | 20:08 | |
cloudnull | javeriak: missed you at the plumgrid event the other day. tell your bosses to send you out next time. :) | 20:09 |
javeriak | cloudnull: yes I heard good things yesterday, hopefully next time :) | 20:11 |
cloudnull | +1 | 20:11 |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Add v3 calls for federation to keystone module https://review.openstack.org/197677 | 20:13 |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Upgrade the Keystone library to use v3 https://review.openstack.org/196943 | 20:13 |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: [WIP] Keystone IdP configuration https://review.openstack.org/194259 | 20:13 |
*** bitblt has quit IRC | 20:14 | |
cloudnull | go sigmavirus24 go ! | 20:14 |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Add v3 calls for federation to keystone module https://review.openstack.org/197677 | 20:16 |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: [WIP] Keystone IdP configuration https://review.openstack.org/194259 | 20:16 |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: [WIP] Keystone SP configuration https://review.openstack.org/194395 | 20:19 |
*** jmckind has quit IRC | 20:22 | |
*** jaypipes has joined #openstack-ansible | 20:22 | |
*** annashen has quit IRC | 20:26 | |
palendae | Huh | 20:29 |
palendae | Surprised debops doesn't have an iptables-managing role | 20:29 |
*** britthou_ has quit IRC | 20:30 | |
*** jaypipes is now known as leakypipes | 20:31 | |
*** yaya_ has joined #openstack-ansible | 20:35 | |
*** yaya has quit IRC | 20:37 | |
*** yaya_ is now known as yaya | 20:37 | |
*** annashen has joined #openstack-ansible | 20:38 | |
*** georgem1 has quit IRC | 20:44 | |
stevelle | review.openstack.org slow for others? | 20:45 |
palendae | stevelle: Has been most of the day | 20:45 |
palendae | Zuul, too | 20:45 |
stevelle | wasn't sure if it was just me all day | 20:46 |
*** sdake_ is now known as sdake | 20:46 | |
*** jwagner is now known as jwagner_away | 20:54 | |
openstackgerrit | Bjoern Teipel proposed stackforge/os-ansible-deployment: Adding galera_root_password token during upgrade Closes-Bug: #1473548 https://review.openstack.org/200721 | 20:58 |
openstack | bug 1473548 in openstack-ansible "Missing variables from Juno to Kilo upgrade" [Undecided,New] https://launchpad.net/bugs/1473548 | 20:58 |
-openstackstatus- NOTICE: Gerrit will be unavailable from 22:00 to 22:30 UTC for project renames | 21:02 | |
*** ig0r_ has quit IRC | 21:10 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Parameterize galera slow/unindexed logging options https://review.openstack.org/200274 | 21:11 |
openstackgerrit | Nolan Brubaker proposed stackforge/os-ansible-deployment: Parameterize galera slow/unindexed logging options https://review.openstack.org/200728 | 21:14 |
sigmavirus24 | cloudnull: btw, https://review.openstack.org/#/c/200730/ | 21:22 |
sigmavirus24 | I can Workflow-1 that though if we didn't want to do that too soon | 21:22 |
palendae | I am +1 | 21:22 |
cloudnull | no, i thought that was already in flight :\ | 21:22 |
sigmavirus24 | cloudnull: did someone submit a review for it? | 21:23 |
sigmavirus24 | I didn't see it if so | 21:23 |
palendae | I assume we'll be involved in the project renames from the openstackstatus notice | 21:23 |
sigmavirus24 | palendae: no | 21:23 |
openstackgerrit | David Wilde proposed stackforge/os-ansible-deployment: Parameterize galera slow/unindexed logging options https://review.openstack.org/200728 | 21:23 |
sigmavirus24 | palendae: project-config is frozen for the ones currently planned | 21:23 |
cloudnull | no. i assumed it was taken care of by the TC | 21:23 |
sigmavirus24 | cloudnull: it isn't | 21:23 |
cloudnull | thats what i get for assuming | 21:24 |
sigmavirus24 | Just talked to jeblair about that in -infra | 21:24 |
sigmavirus24 | hence my review | 21:24 |
cloudnull | +9000 from me | 21:24 |
*** sdake_ has joined #openstack-ansible | 21:24 | |
cloudnull | good catch | 21:24 |
palendae | d34dh0r53: What's the different from my backport there? | 21:24 |
palendae | Did I miss something? | 21:24 |
d34dh0r53 | lol no, I didn't know you were doing it | 21:24 |
d34dh0r53 | sorry | 21:24 |
palendae | I thought you were gone for the day | 21:24 |
palendae | My bad :) | 21:24 |
cloudnull | thanks sigmavirus24! | 21:24 |
d34dh0r53 | you were first | 21:25 |
*** sdake has quit IRC | 21:27 | |
sigmavirus24 | cloudnull: yw | 21:28 |
palendae | sigmavirus24: Thanks! | 21:28 |
sigmavirus24 | Let's hope it passes the gate | 21:28 |
sigmavirus24 | palendae: are you on ff39? | 21:29 |
palendae | sigmavirus24: Yes | 21:29 |
sigmavirus24 | do you use HTTPS Everywhere? | 21:29 |
palendae | Not right now | 21:30 |
palendae | Doesn't it break openstack.org all over? | 21:30 |
sigmavirus24 | yeah | 21:31 |
sigmavirus24 | I think | 21:31 |
sigmavirus24 | but | 21:31 |
sigmavirus24 | if I disable the OpenStack rule | 21:31 |
sigmavirus24 | It still breaks | 21:31 |
palendae | >.< | 21:31 |
sigmavirus24 | It's super annoying | 21:31 |
cloudnull | use chrome, google owns all your data anyway. its not use to fight it anymore. | 21:33 |
cloudnull | :) | 21:33 |
*** Mudpuppy has quit IRC | 21:33 | |
palendae | cloudnull: Not all of it yet, I don't have Google fiber | 21:33 |
palendae | Then I'll be MitMed all day | 21:33 |
palendae | I should really go | 21:34 |
palendae | I need to make some purchases that are blocking my workout today | 21:34 |
palendae | But the couch is so nice | 21:34 |
cloudnull | soon | 21:36 |
cloudnull | :) | 21:36 |
openstackgerrit | Merged stackforge/os-ansible-deployment: Remove haproxy galera monitoring user https://review.openstack.org/199625 | 21:39 |
*** annashen has quit IRC | 21:41 | |
*** yaya has left #openstack-ansible | 21:41 | |
*** javeriak has quit IRC | 21:41 | |
*** annashen has joined #openstack-ansible | 21:42 | |
openstackgerrit | Nolan Brubaker proposed stackforge/os-ansible-deployment: Remove haproxy galera monitoring user https://review.openstack.org/200742 | 21:43 |
*** javeriak has joined #openstack-ansible | 21:45 | |
*** annashen has quit IRC | 21:46 | |
*** alop has quit IRC | 21:48 | |
-openstackstatus- NOTICE: Gerrit is unavailable from approximately 22:00 to 22:30 UTC for project renames | 21:56 | |
*** ChanServ changes topic to "Gerrit is unavailable from approximately 22:00 to 22:30 UTC for project renames" | 21:56 | |
*** alop has joined #openstack-ansible | 21:58 | |
*** annashen has joined #openstack-ansible | 22:00 | |
*** alop has quit IRC | 22:02 | |
*** sdake has joined #openstack-ansible | 22:11 | |
*** sdake_ has quit IRC | 22:14 | |
*** annashen has quit IRC | 22:15 | |
*** britthouser has joined #openstack-ansible | 22:19 | |
*** annashen has joined #openstack-ansible | 22:22 | |
*** annashen has quit IRC | 22:27 | |
*** ChanServ changes topic to "Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible" | 22:30 | |
*** alop has joined #openstack-ansible | 22:31 | |
miguelgrinberg | sigmavirus24: I'm having issues getting ensure_group to work, it fails with a 401. Before I spend too much time investigating, want to ask if you have seen this. Other ensure methods work fine for me. | 22:49 |
*** klindgren has quit IRC | 22:51 | |
*** klindgren has joined #openstack-ansible | 22:52 | |
*** misc has quit IRC | 22:54 | |
*** misc has joined #openstack-ansible | 22:55 | |
*** britthouser has quit IRC | 23:08 | |
*** TheIntern has quit IRC | 23:36 | |
*** alop has quit IRC | 23:38 | |
*** openstackgerrit has quit IRC | 23:39 | |
*** openstackgerrit has joined #openstack-ansible | 23:39 | |
*** weezS has quit IRC | 23:53 | |
*** rromans is now known as rromans_afk | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!