*** darrenc is now known as darrenc_afk | 00:20 | |
*** javeriak has quit IRC | 00:28 | |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Upgrade the Keystone library to use v3 https://review.openstack.org/196943 | 00:30 |
---|---|---|
*** CheKoLyN has joined #openstack-ansible | 00:32 | |
*** JRobinson__ has quit IRC | 00:40 | |
*** sdake has quit IRC | 00:56 | |
*** sdake has joined #openstack-ansible | 00:56 | |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Upgrade the Keystone library to use v3 https://review.openstack.org/196943 | 01:00 |
*** jaypipes has quit IRC | 01:16 | |
*** mfisch has quit IRC | 01:20 | |
*** darrenc_afk is now known as darrenc | 01:20 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Fix link to OSAD configuration files page. https://review.openstack.org/198069 | 01:25 |
*** sigmavirus24 is now known as sigmavirus24_awa | 01:30 | |
*** sdake_ has joined #openstack-ansible | 01:40 | |
*** sdake has quit IRC | 01:43 | |
*** sdake_ has quit IRC | 01:49 | |
*** sdake has joined #openstack-ansible | 01:51 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Clear expired Django sessions regularly from DB https://review.openstack.org/197744 | 02:11 |
*** mfisch has joined #openstack-ansible | 02:42 | |
openstackgerrit | Kevin Carter proposed stackforge/os-ansible-deployment: Rabbit config changes dont initiate a restart https://review.openstack.org/198193 | 02:42 |
*** mfisch is now known as Guest61548 | 02:42 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Updated the clone process to exclude unneeded files https://review.openstack.org/198014 | 03:00 |
*** CheKoLyN has quit IRC | 03:00 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Rabbit config changes dont initiate a restart https://review.openstack.org/198134 | 03:31 |
*** JRobinson__ has joined #openstack-ansible | 04:03 | |
*** annashen has joined #openstack-ansible | 04:09 | |
*** annashen has quit IRC | 04:25 | |
*** sdake_ has joined #openstack-ansible | 04:39 | |
*** sdake has quit IRC | 04:43 | |
*** shausy has joined #openstack-ansible | 04:57 | |
*** javeriak has joined #openstack-ansible | 05:31 | |
*** fawadkhaliq has joined #openstack-ansible | 05:54 | |
*** fawadk has joined #openstack-ansible | 05:55 | |
*** fawadkhaliq has quit IRC | 05:58 | |
*** annashen has joined #openstack-ansible | 05:59 | |
*** annashen has quit IRC | 06:13 | |
*** annashen has joined #openstack-ansible | 06:15 | |
*** annashen has quit IRC | 06:22 | |
*** radek_ has joined #openstack-ansible | 06:33 | |
*** subscope has quit IRC | 07:11 | |
*** subscope has joined #openstack-ansible | 07:12 | |
*** svg has quit IRC | 07:32 | |
*** svg_ has joined #openstack-ansible | 07:33 | |
*** svg_ is now known as svg | 07:33 | |
*** JRobinson__ has quit IRC | 07:34 | |
*** openstackstatus has quit IRC | 07:49 | |
*** openstackstatus has joined #openstack-ansible | 07:50 | |
*** ChanServ sets mode: +v openstackstatus | 07:50 | |
*** fawadk has quit IRC | 07:51 | |
evrardjp | hello everyone | 08:31 |
*** vdo has joined #openstack-ansible | 08:36 | |
mancdaz | morning evrardjp | 08:43 |
openstackgerrit | Darren Birkett proposed stackforge/os-ansible-deployment: Rabbit config changes dont initiate a restart https://review.openstack.org/198193 | 08:43 |
*** javeriak has quit IRC | 08:45 | |
svg | good morning all | 08:48 |
evrardjp | good morning svg | 08:54 |
evrardjp | is there a reason that we are not using ansible handlers ? I'm thinking at your commit mancdaz | 08:55 |
evrardjp | I don't know how everything fits together, and I'm ready to learn about it | 08:56 |
evrardjp | but typically restarting a service, even on multiple hosts, could be done by handlers | 08:56 |
mancdaz | evrardjp we do use handlers in a lot of the cases. I'm honestly not sure why one wasn't used here since I was only cherry-picking the backport | 08:57 |
*** openstackgerrit has quit IRC | 08:57 | |
*** openstackgerrit has joined #openstack-ansible | 08:58 | |
evrardjp | It means twice the work to fix it in both branches if this kind of change is ever done | 08:59 |
evrardjp | but that wasn't my question, it was more to know how to work in OSAD | 08:59 |
mancdaz | evrardjp the double commit thing is indeed a consequence of how we're maintaining the master and kilo branches right now | 09:00 |
evrardjp | I don't see a problem with the double commit | 09:00 |
evrardjp | I see a problem with double fixing ;) | 09:00 |
evrardjp | if fixing happens | 09:01 |
mancdaz | evrardjp no fixing ever needs to happen, because nothing is broken :) | 09:01 |
evrardjp | I meant if it requires fixing | 09:01 |
mancdaz | we have no bugs | 09:01 |
evrardjp | then it's problem solved :p | 09:01 |
mancdaz | lol yup | 09:01 |
evrardjp | mancdaz: FYI I didn't see the comment on https://review.openstack.org/#/c/198193/ by git-harry before posting here :/ | 09:06 |
mancdaz | evrardjp no probs - it means you're all on the same page, so your assertions are good | 09:07 |
evrardjp | still, duplicate info ;) | 09:07 |
mancdaz | but yes, if we decide to change it, it means a double fix | 09:07 |
* svg is confused by that git-harry username | 09:08 | |
*** fawadkhaliq has joined #openstack-ansible | 09:09 | |
svg | name sounds like a bot, but the comments seem Turing complete | 09:09 |
git-harry | svg: sorry, I do not understand. Please try rephrasing the question | 09:10 |
evrardjp | :D | 09:10 |
evrardjp | svg Imagine something this is like a guitar that can speak.. It's guit(h)ar(r)y :p | 09:11 |
evrardjp | it's friday, sorry for that... Too bad I can't remove this comment from the log. | 09:11 |
svg | I'd welcome git-Sally | 09:12 |
evrardjp | classic! | 09:12 |
mattt | are you two peaking belgian | 09:12 |
mattt | *speaking :P | 09:12 |
mancdaz | msg /philipw | 09:12 |
mancdaz | yes, how not to irc | 09:13 |
mattt | mancdaz: bra srsly | 09:13 |
mancdaz | lols | 09:13 |
mancdaz | I'm going home | 09:13 |
mattt | need a music suggestion ... anyone | 09:14 |
mancdaz | whigfield | 09:14 |
mancdaz | Saturday night | 09:14 |
mattt | anyone* | 09:14 |
mattt | *but mancdaz | 09:14 |
evrardjp | svg, while I'm waiting for a new /48 and discussions about l3ha, do you mind if I help you on ceph support? | 09:15 |
svg | sure | 09:15 |
svg | did you have something particular in mind evrardjp | 09:16 |
svg | mattt we are chatting Belgians! | 09:16 |
evrardjp | I'll check on the clashing variables and on leseb comment on review https://review.openstack.org/#/c/181957/ | 09:17 |
svg | evrardjp: I already did some partial updates on his comments, see last commit on https://github.com/srvg/os-ansible-deployment/tree/master-ceph-rev27 - search for TODO | 09:19 |
svg | evrardjp: I'm trying to figure out a solution for https://github.com/srvg/os-ansible-deployment/commit/276cc72639a5f0eb42c0aac122a2ad7f5b82b088#diff-1fc9f748c2103f182beae1045d73cc09R69 | 09:22 |
svg | given you can't overrule ssh errors in ansible, that is tricky | 09:22 |
*** annashen has joined #openstack-ansible | 09:23 | |
vincent_vdk | svg: you mean Dutch speaking | 09:26 |
*** annashen has quit IRC | 09:27 | |
svg | vincent_vdk: euh, no, I didn;t mention language | 09:28 |
openstackgerrit | Merged stackforge/os-ansible-deployment: Configurable memcached connections limit & threads https://review.openstack.org/197740 | 09:33 |
*** shausy has quit IRC | 09:36 | |
*** hughsaunders has quit IRC | 10:07 | |
*** hughsaunders has joined #openstack-ansible | 10:07 | |
*** sdake_ has quit IRC | 10:14 | |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Copy env.d files in scripts/run-upgrade.sh https://review.openstack.org/198311 | 11:04 |
openstackgerrit | Merged stackforge/os-ansible-deployment: Update port definition for PLUMgrid in Juno https://review.openstack.org/197705 | 11:12 |
*** fawadkhaliq has quit IRC | 11:24 | |
*** fawadkhaliq has joined #openstack-ansible | 11:25 | |
*** subscope has quit IRC | 11:39 | |
*** subscope has joined #openstack-ansible | 11:54 | |
*** jaypipes has joined #openstack-ansible | 11:54 | |
openstackgerrit | philip williams proposed stackforge/os-ansible-deployment-specs: Implements: blueprint multi-region-swift Change-Id: I3c718ece6643fe932cb45bf5baea620c5c8a9408 https://review.openstack.org/198322 | 11:54 |
*** subscope has quit IRC | 12:05 | |
*** subscope has joined #openstack-ansible | 12:21 | |
*** fawadkhaliq has quit IRC | 12:25 | |
openstackgerrit | Darren Birkett proposed stackforge/os-ansible-deployment-specs: Multi-region swift https://review.openstack.org/198322 | 12:26 |
*** fawadkhaliq has joined #openstack-ansible | 12:39 | |
*** fawadkhaliq has quit IRC | 12:50 | |
*** markvoelker has quit IRC | 12:53 | |
*** markvoelker has joined #openstack-ansible | 13:00 | |
*** jaypipes is now known as leakypipes | 13:01 | |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Copy env.d files in scripts/run-upgrade.sh https://review.openstack.org/198311 | 13:19 |
*** sdake has joined #openstack-ansible | 14:16 | |
*** sdake_ has joined #openstack-ansible | 14:17 | |
*** sdake has quit IRC | 14:21 | |
*** sdake_ is now known as sdake | 14:22 | |
openstackgerrit | Ian Cordasco proposed stackforge/os-ansible-deployment: Upgrade the Keystone library to use v3 https://review.openstack.org/196943 | 14:24 |
*** sdake has quit IRC | 14:31 | |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: Fix scripts/run-upgrade.sh https://review.openstack.org/198311 | 14:45 |
*** vdo has quit IRC | 14:52 | |
*** markvoelker has quit IRC | 15:00 | |
*** annashen has joined #openstack-ansible | 15:07 | |
*** annashen has quit IRC | 15:18 | |
openstackgerrit | Matt Thompson proposed stackforge/os-ansible-deployment: [WIP] Fix scripts/run-upgrade.sh https://review.openstack.org/198311 | 15:30 |
*** subscope has quit IRC | 15:35 | |
*** subscope has joined #openstack-ansible | 15:36 | |
mattt | cloudnull: you're not in are you ? | 15:36 |
cloudnull | No | 15:37 |
cloudnull | ;) | 15:37 |
cloudnull | hows it mattt | 15:37 |
mattt | cloudnull: :P | 15:38 |
mattt | cloudnull: hey, regarding https://review.openstack.org/198311, we currently do not pull in the old environment file | 15:38 |
mattt | so as it stands there is neither a new or old environment in /etc/openstack_deploy | 15:38 |
mattt | also, you have to skip those groups because when those commands are run the containers have yet to be created so the script will abort because it hits hosts that are down | 15:39 |
mattt | have been chatting w/ andymccr and agree that just pulling in the files into env.d is probably going to result in a pretty bad time | 15:40 |
cloudnull | if you re-organize that funciton you should be ok. | 15:41 |
mattt | cloudnull: yeah we could build the containers earlier but then you have a mix of old and new containers, and i'm not sure what implications that will have | 15:42 |
cloudnull | IE run https://github.com/stackforge/os-ansible-deployment/blob/master/scripts/run-upgrade.sh#L308-L325 then https://github.com/stackforge/os-ansible-deployment/blob/master/scripts/run-upgrade.sh#L300-L306 | 15:42 |
mattt | i'm going to assume that some operation expecting an old container is going to bomb when it finds the new container | 15:42 |
cloudnull | that way you destroy the old one and then build the new ones. | 15:42 |
mattt | i thought the new containers were getting created in https://github.com/stackforge/os-ansible-deployment/blob/master/scripts/run-upgrade.sh#L534 | 15:43 |
cloudnull | also the rpc_environment is rewritten here https://github.com/stackforge/os-ansible-deployment/blob/master/scripts/run-upgrade.sh#L230-L261 | 15:43 |
mattt | cloudnull: that code isn't working | 15:45 |
mattt | i end up with an empty openstack_environment.yml file | 15:45 |
mattt | cloudnull: my file looks exactly like this afterwards https://github.com/stackforge/os-ansible-deployment/blob/master/etc/openstack_deploy/openstack_environment.yml | 15:45 |
mattt | probably because we only alter anything if onmetal exists | 15:46 |
cloudnull | it should only alter but itll write out the file regardless. | 15:46 |
cloudnull | atleast it should | 15:46 |
mattt | it's writing out os_environment (the new file, which is empty), not environment (the old /etc/rpc_deploy.OLD/rpc_environment.yml) | 15:47 |
cloudnull | we copy over the env file here https://github.com/stackforge/os-ansible-deployment/blob/master/scripts/run-upgrade.sh#L98 | 15:48 |
mattt | cloudnull: but that file doesn't contain anything https://github.com/stackforge/os-ansible-deployment/blob/master/etc/openstack_deploy/openstack_environment.yml | 15:48 |
mattt | it has some empty dicts in it | 15:48 |
mattt | does that not matter? | 15:49 |
mattt | i had an error which i thought was due to that, but maybe it's nto? | 15:49 |
cloudnull | true ah... | 15:49 |
cloudnull | that function will need to be changed to hunt through the new env.d files. | 15:50 |
cloudnull | so thats a problem | 15:50 |
cloudnull | youre right there. | 15:50 |
mattt | cloudnull: check md5sum of /etc/rpc_deploy.OLD/rpc_environment.yml, if it's different abort upgrade :D | 15:50 |
mattt | otherwise blindly copy in env.d files and move on | 15:51 |
mattt | naw that's not great, need a programatic way to fix this | 15:51 |
*** fawadkhaliq has joined #openstack-ansible | 15:51 | |
*** fawadkhaliq has quit IRC | 15:55 | |
cloudnull | looking at how it was, simply copying over the env.d files and the new empty environment file should be feature compatible. | 15:58 |
cloudnull | thats what we were doing before. | 15:58 |
cloudnull | however we still need to fix that function to pic trough the new files and make changes where neded. | 15:59 |
*** fawadkhaliq has joined #openstack-ansible | 15:59 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Use the correct upstream repos. https://review.openstack.org/185099 | 16:06 |
cloudnull | mattt: something like https://gist.github.com/cloudnull/adef604ce948fce32211 should work | 16:08 |
cloudnull | mattt: updated again https://gist.github.com/cloudnull/adef604ce948fce32211 - should only modify the found files that have changed | 16:17 |
*** sdake has joined #openstack-ansible | 16:23 | |
*** fawadkhaliq has quit IRC | 16:27 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Rabbit config changes dont initiate a restart https://review.openstack.org/198193 | 16:31 |
odyssey4me | mattt / andymccr is that upgrade issue with env.d files pertinent to the kilo branch - or only master? | 16:36 |
andymccr | both | 16:37 |
odyssey4me | blast, that patch backport should probably have been held back for the 11.1 release instead of being backported so happily :/ | 16:39 |
odyssey4me | oh well, such is life | 16:39 |
odyssey4me | (it's probably my fault, sorry) | 16:39 |
stevelle | odyssey4me: the rabbit restart? | 16:39 |
odyssey4me | stevelle nope, the split of the environment file into many pieces | 16:39 |
stevelle | ah, np then | 16:40 |
*** markvoelker has joined #openstack-ansible | 16:49 | |
*** serverascode has quit IRC | 17:04 | |
*** serverascode has joined #openstack-ansible | 17:06 | |
*** subscope has quit IRC | 17:09 | |
*** sdake_ has joined #openstack-ansible | 17:10 | |
*** sdake has quit IRC | 17:13 | |
*** sdake has joined #openstack-ansible | 17:21 | |
openstackgerrit | Merged stackforge/os-ansible-deployment: Fixed the upgrade script edge case https://review.openstack.org/197657 | 17:25 |
*** sdake_ has quit IRC | 17:26 | |
openstackgerrit | Kevin Carter proposed stackforge/os-ansible-deployment: Fixed the upgrade script edge case https://review.openstack.org/198441 | 18:12 |
*** fawadkhaliq has joined #openstack-ansible | 18:45 | |
*** sdake_ has joined #openstack-ansible | 18:52 | |
*** sdake has quit IRC | 18:56 | |
-openstackstatus- NOTICE: etherpad.openstack.org is offline for scheduled database maintenance, ETA 19:30 UTC | 19:01 | |
*** ChanServ changes topic to "etherpad.openstack.org is offline for scheduled database maintenance, ETA 19:30 UTC" | 19:01 | |
*** sdake_ has quit IRC | 19:05 | |
*** fawadkhaliq has quit IRC | 19:14 | |
*** sdake has joined #openstack-ansible | 19:15 | |
-openstackstatus- NOTICE: etherpad.openstack.org is still offline for scheduled database maintenance, ETA 19:45 UTC | 19:29 | |
*** ChanServ changes topic to "etherpad.openstack.org is still offline for scheduled database maintenance, ETA 19:45 UTC" | 19:29 | |
*** javeriak has joined #openstack-ansible | 19:51 | |
*** ChanServ changes topic to "Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible" | 19:52 | |
*** javeriak has quit IRC | 20:16 | |
*** sdake_ has joined #openstack-ansible | 20:52 | |
*** sdake has quit IRC | 20:55 | |
*** sdake_ is now known as sdake | 21:34 | |
*** radek_ has quit IRC | 21:42 | |
*** fawadkhaliq has joined #openstack-ansible | 22:39 | |
*** fawadkhaliq has quit IRC | 22:52 | |
*** sdake has quit IRC | 23:01 | |
*** sdake has joined #openstack-ansible | 23:03 | |
*** andymccr has quit IRC | 23:30 | |
*** andymccr has joined #openstack-ansible | 23:32 | |
*** weezS has joined #openstack-ansible | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!