Thursday, 2015-09-24

*** diogogmt has joined #kolla00:03
*** mfalatic has quit IRC00:09
*** mduggan has quit IRC00:16
*** mduggan has joined #kolla00:17
*** jtriley has joined #kolla00:18
*** pbourke_ has quit IRC00:22
*** pbourke_ has joined #kolla00:22
*** achanda has joined #kolla00:33
*** achanda has quit IRC00:38
*** sdake has joined #kolla01:03
*** zhiwei has joined #kolla01:07
*** jtriley has quit IRC01:22
*** rhallisey has quit IRC01:26
*** tfukushima has joined #kolla01:30
*** tummy has joined #kolla01:33
*** tfukushima has quit IRC01:34
*** dolpher has joined #kolla01:35
*** dolpher has quit IRC01:41
*** tfukushima has joined #kolla01:43
*** dolpher has joined #kolla01:43
*** tfukushima has quit IRC01:43
*** tfukushima has joined #kolla01:44
*** tummy has quit IRC01:45
*** dolpher1 has joined #kolla01:45
*** dolpher1 has left #kolla01:45
*** dolpher has quit IRC01:48
*** erkules_ has joined #kolla01:55
*** erkules has quit IRC01:58
*** rajalokan has joined #kolla01:59
*** tfukushima has quit IRC01:59
*** achanda has joined #kolla02:01
*** sdake has quit IRC02:06
*** tfukushima has joined #kolla02:08
*** dayou has joined #kolla02:11
*** jtriley has joined #kolla02:19
*** unicell has quit IRC02:23
*** jtriley has quit IRC02:49
*** diogogmt has quit IRC03:04
*** dayou has quit IRC03:04
*** tfukushima has quit IRC03:17
*** dayou has joined #kolla03:34
*** tfukushima has joined #kolla03:37
*** diga has joined #kolla03:45
*** diga has quit IRC03:51
*** achanda has quit IRC03:55
*** diga has joined #kolla04:09
*** chandankumar has joined #kolla04:39
*** inc0 has joined #kolla04:42
inc0hello, any chances for second +2 on this one? https://review.openstack.org/#/c/215062/1704:43
*** sdake has joined #kolla04:54
*** achanda has joined #kolla04:55
*** inc0 has quit IRC05:00
*** diga has quit IRC05:37
*** sdake has quit IRC05:40
openstackgerritSam Yaple proposed openstack/kolla: Ceph Ansible support  https://review.openstack.org/21857005:44
SamYaplepbourke_: you around?05:45
*** inc0 has joined #kolla05:47
inc0good morning05:51
SamYaplemorning inc005:59
SamYapleill take a look at that patch in a moment for a +206:00
inc0you already +2 it06:00
SamYapleoh ok06:00
inc0and it sits there with this lonely +206:00
inc0thanks anyway06:00
inc0;)06:00
SamYaplekinda like all my ceph stuff :/06:00
SamYaplethats be up and mostly unchanged for a while06:00
SamYaplelots of review promises06:00
inc0I'll try this in a moment, let me rebase my patches locally06:02
inc0however I don't think my +1 will mean much at this point06:03
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Ansible bits to rsyslog  https://review.openstack.org/21506206:22
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Glance logging to rsyslog  https://review.openstack.org/22563706:22
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Nova logging to rsyslog  https://review.openstack.org/22676006:22
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Heat logging to rsyslog  https://review.openstack.org/22682006:22
inc0SamYaple, would you be kind enough to take a look at service logging patches?06:23
inc0afk 30mins06:23
*** chandankumar has quit IRC06:24
SamYaplesyre06:28
*** chandankumar has joined #kolla06:34
*** daneyon has quit IRC07:00
openstackgerritMartin André proposed openstack/kolla: Allow different defaults for different providers  https://review.openstack.org/22714707:00
openstackgerritMartin André proposed openstack/kolla: Remove fake DHCP server in Vagrantfile  https://review.openstack.org/22714807:00
openstackgerritMartin André proposed openstack/kolla: Fix git instruction in vagrant doc  https://review.openstack.org/22714907:00
*** daneyon has joined #kolla07:01
*** rajalokan has quit IRC07:02
*** diga has joined #kolla07:06
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Glance logging to rsyslog  https://review.openstack.org/22563707:07
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Nova logging to rsyslog  https://review.openstack.org/22676007:07
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Heat logging to rsyslog  https://review.openstack.org/22682007:07
*** akwasnie has joined #kolla07:15
*** jmccarthy has quit IRC07:17
*** rajalokan has joined #kolla07:27
*** jmccarthy has joined #kolla07:27
*** achanda has quit IRC07:28
*** tfukushima has quit IRC07:29
*** tfukushima has joined #kolla07:54
*** inc0 has quit IRC07:55
*** athomas has joined #kolla08:08
SamYaplepbourke_: up yet?08:11
openstackgerritSam Yaple proposed openstack/kolla: Replace config-external with a JSON file for Neutron (thin)  https://review.openstack.org/22482908:12
openstackgerritSam Yaple proposed openstack/kolla: Replace config-external with a JSON file for Cinder  https://review.openstack.org/22487208:12
openstackgerritSam Yaple proposed openstack/kolla: Replace config-external with a JSON file for Heat  https://review.openstack.org/22489108:13
*** diga has quit IRC08:14
*** tfukushima has quit IRC08:16
*** tfukushima has joined #kolla08:17
*** tfukushima has quit IRC08:29
*** tfukushima has joined #kolla08:29
*** tfukushima has quit IRC08:31
*** tfukushima has joined #kolla08:31
*** shardy has joined #kolla08:34
openstackgerritMerged openstack/kolla: Replace config-external with a JSON file for Neutron (thin)  https://review.openstack.org/22482908:42
*** tfukushima has quit IRC08:46
*** mduggan has left #kolla08:47
*** diga has joined #kolla08:48
openstackgerritMerged openstack/kolla: Replace config-external with a JSON file for Cinder  https://review.openstack.org/22487208:49
openstackgerritMerged openstack/kolla: Replace config-external with a JSON file for Heat  https://review.openstack.org/22489108:49
*** athomas has quit IRC08:54
*** dayou has quit IRC09:02
*** gfidente has joined #kolla09:02
*** athomas has joined #kolla09:03
*** daneyon has quit IRC09:06
*** daneyon has joined #kolla09:07
pbourke_SamYaple: am now09:08
[Leeloo]Hi. I have a "functional" openstack using (oraclelinux/binary [except: magnum, murano...]) on a an OVH server. But can't have access outside or apply floating IP. Is this doc --> http://dohuyhoang.github.io/openstack/ovh/neutron/2014/07/07/Configure-OpenStack-network-on-OVH.html applicable to docker openstack install ? Another question, when I want to change thing on the docker neutron,...09:15
[Leeloo]instance how can I do ? thanks :)09:15
*** diga has quit IRC09:36
openstackgerritSam Yaple proposed openstack/kolla: Make OpenStack use Ceph  https://review.openstack.org/22593309:48
openstackgerritSam Yaple proposed openstack/kolla: Ceph Ansible support  https://review.openstack.org/21857009:48
SamYaplepbourke_: sorry i have to keep rebasing this ceph patch due to the json changes09:49
*** rajalokan has quit IRC10:19
*** diga has joined #kolla10:34
*** erkules_ has quit IRC10:38
*** inc0 has joined #kolla10:40
inc0hey there10:40
*** zhiwei has quit IRC10:40
*** pbourke_ has quit IRC10:43
*** pbourke_ has joined #kolla10:45
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Ansible bits to rsyslog  https://review.openstack.org/21506210:47
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Glance logging to rsyslog  https://review.openstack.org/22563710:47
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Nova logging to rsyslog  https://review.openstack.org/22676010:47
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Heat logging to rsyslog  https://review.openstack.org/22682010:47
inc0pbourke_, SamYaple could you please re-review10:49
inc0SamYaple, is it required to have delegate_to in ceph task distribute_keyrings?10:57
SamYapleyes10:58
SamYapleand we use delegate_to elsewhere. its not really an obscure ansible command10:58
inc0because it breaks on basic aio, it doesn't understand connection local10:58
SamYaplei know its been discussed10:59
SamYapleints all in the review10:59
inc0allright reading10:59
SamYaplethe fact is we use delegate_to in other places. we need to change the default AIO to use ssh. this isnt the first error we have encountered because of this problem10:59
inc0yeah makes sense11:00
SamYaplethe inventory file is just an example document anyway, so if Kolla cant use connection=local its not teh end of the world in my opinion11:00
inc0sure, all I'm saying is by default everyone tries aio11:00
inc0so we should make it work, even if that require people to do something in inventory first (as long as it's clearly stated in docs)11:01
SamYaplei know. and ceph+AIO isn't going to really work anyway (it can but that requires ceph knowledge)11:01
SamYapleall this is going in a ceph document I am writting once these patchses merge11:01
inc0aaa yeah it won't have anywhere to put replicas11:03
inc0right11:03
SamYaplepbourke_: fyi i have tgtd working with cinder (for the lvm stuff)11:03
SamYapleinc0: correct. again ceph trickery (crush map modification) can be done to get it into HEALTH_OK but thats ceph knowlege11:03
inc0yeah, I'm not rly a ceph expert, just want to get this running;)11:04
SamYapleinc0: it requires a disk (or a loop device) be created on your host11:07
SamYaplehard requirement11:08
*** gfidente has quit IRC11:08
inc0can you please submit docs as dependant patch?11:09
SamYapleno, i dont have the docs ready. this is going to have to merge by tommorrow11:09
SamYapleDocImpact flag is set so it wont get lost11:10
inc0ok, then I'll just review code and test it out at nearest occasion11:12
inc0we can always submit bugs11:12
SamYapleyea thats what these next 3 weeks are for11:12
SamYaplekeep in mind this code has been (successfully) demo'ed on both virtual and real hardware. And its structure is basically what I had in yaodu so its been around for ~6 months11:13
SamYapleim very confident in it so that should reassure you a bit even if you are unable to test it11:14
inc0you don't need to reasure me, I'm confident you did good job, just this is complicated stuff out there, it's always good to have additional test11:17
SamYaplei agree and i wish the docs were rady but reviews have been scarce the past few weeks11:17
inc0https://review.openstack.org/#/c/218570/14/docker/openstack-base/Dockerfile.j2 why we needed numpy in the first place?11:18
SamYaplewe didn't need it there. that was a performance increase thing i found was to remove numpy, it _should_ have been another patch11:19
SamYaplenumpy gets installing in neutron-base i think11:19
SamYaplei think neutron is the only project its used in11:19
inc0ok11:19
inc0but since you don't add it to neutron...won't that break?11:20
SamYaplerequirements.txt pulls it in11:20
SamYapleanything in the openstack-base container is just to consolidate-deps11:20
SamYapleno source thing requires the openstack-base container, its just to save space/speed up build11:20
inc0I'd be more comfortable with having it as separate patch, but won't -1 on tha11:20
inc0t11:20
SamYapleto be honest, I would too but deadlines and all11:21
inc0hence no -1, normally I would11:21
SamYapleyep agree with your statements11:21
SamYaplepbourke_: lets get this merged so we dont have to rereview the working logging patches: https://review.openstack.org/#/c/215062/11:26
*** chandankumar has quit IRC11:28
*** achanda has joined #kolla11:29
SamYapledammit ryan never tested his patches.11:32
SamYapleplaybooks are broken11:32
SamYapleor maybe its me11:33
SamYapleno thats me11:33
SamYaplesorry ryan11:33
SamYapleno dammit it was ryan! its in master11:34
SamYaplehttps://github.com/openstack/kolla/blob/master/ansible/roles/nova/tasks/config.yml#L15211:34
SamYapleugh patch incoming hold on11:34
openstackgerritMerged openstack/kolla: Ansible bits to rsyslog  https://review.openstack.org/21506211:35
openstackgerritMerged openstack/kolla: Glance logging to rsyslog  https://review.openstack.org/22563711:36
openstackgerritMerged openstack/kolla: Nova logging to rsyslog  https://review.openstack.org/22676011:36
openstackgerritSam Yaple proposed openstack/kolla: Create nova-libvirt directory before copying files  https://review.openstack.org/22724211:37
SamYaplepbourke_ coolsvap https://review.openstack.org/22724211:37
SamYaplethat patch is needed, master is broken11:37
SamYapleharmw if you are around ^^11:37
*** rhallisey has joined #kolla11:38
inc0Ryan, don't look at logs from last 10minutes;)11:40
SamYapleno do look at them and test things! people should test major changes :|11:41
*** achanda has quit IRC11:41
*** smeisner has quit IRC11:46
*** gfidente has joined #kolla11:48
*** gfidente has quit IRC11:48
*** gfidente has joined #kolla11:48
inc0we definetly need CI11:48
inc0(a good one)11:48
SamYaplethe playbooks arent too gangly yet, but they also need to be cleaned up11:50
SamYaplei wish ansible was like saltstack and you could write tasks entirely in python11:51
SamYapleor yaml11:51
SamYapleand mix them11:51
SamYaplemmmm choices11:51
SamYaplebirth cirtificate arrives today so I might actually get my password in time. yay11:54
rhalliseyinc0, eh?11:55
SamYaplerhallisey: i was complaining becuase master playbooks were borked11:55
SamYapleyou didnt test your patches11:55
rhalliseyI did11:55
rhalliseythey worked fine11:55
SamYaplenot in a clean environment apparently11:55
rhalliseywhats breaking?11:55
rhalliseyI was using AIO11:55
SamYaplehttps://review.openstack.org/#/c/227242/11:55
SamYapleyou reviewed it11:56
rhalliseyare you using multinode?11:56
SamYapleit matters not, it would break in both situations11:56
rhalliseySamYaple, I must've added that dir by hand when getting it going11:56
SamYaplewhatever the reason is it is busted, this is why inc0 said we need good CI which is agreed11:56
SamYaplerhallisey: yea ive done it too :)11:56
rhalliseySamYaple, so what's the deal is it totally busted or something?11:57
SamYapleyou can't run through the playbooks cleanly, no11:57
rhalliseyI'm confused because I had no problems when testing11:57
SamYaplelike you said, you probably added the directory by hand. that would resolve the issue11:58
SamYapleunfortunately i kick fresh vms for each test and if the playbooks are broken so is my ability to test11:58
inc0SamYaple, using vagrant?11:59
SamYapleinc0: no11:59
rhalliseySamYaple, ya I definitely did and forgot about it11:59
SamYaplenot a problem :) its found and fixed11:59
rhalliseyya sorry for the headache11:59
SamYapleyou can make up for it by reviewing ceph :[]12:00
SamYapleit really needs to merge its been a while12:00
rhalliseyI've looked at it only, I don't know how to test it12:00
rhalliseySamYaple, was the libvirt dir the only issue?12:00
SamYapleas sdake would say, testing is not required for reviewers12:00
inc0we need docs... <flies away>12:00
SamYapleyea rhallisey ive got a docs patch brewing12:01
rhalliseykk12:01
SamYaplebut its not done yet. the docimpact flag is set12:01
SamYapleas long as when you run the playbooks with ceph skipped it doesnt bust anything, its safe to merge12:01
SamYaplewe can hammer out the docs and any bugs after RC112:01
rhalliseySamYaple, going back to json config again, is libvirt the only issue or is it breaking elsewhere?12:01
SamYaplerhallisey: running a fresh test now, but that looked like it12:02
rhalliseySamYaple, ok cool12:02
SamYaplebut you did forget the when conditionals on some of those tasks12:02
SamYapleonly noticed now not in the review12:02
rhalliseySamYaple, that's for the cases when say heat is enabled/disabled12:03
rhalliseyya I'll add those in12:03
SamYapleno i mean for the hosts, you copy the json configs to all hosts in the nova group12:03
SamYaplenot a big deal. thats non-breaking12:04
rhalliseythe piece with heat is though I think12:04
rhalliseyI made note of it at my desk12:04
rhalliseysomewhere..12:04
inc0I wonder how your desk looks like then...12:04
rhalliseyI have lots of sticky notes12:05
rhalliseyof things to add into kolla12:05
SamYaplerhallisey: ive have tgt working for cinder, can we discuss what issues you were having12:06
SamYapleit seems to all work for me12:06
rhalliseyI think ansible bits were just missing for that12:06
SamYaplewell there is no tgt container so more than ansible was missing12:06
SamYaplecinder is not working at all with Kolla12:07
rhalliseyI wasn't using tgt in its own container12:07
SamYapleoh12:07
rhalliseyit was inside cinder-volumes12:07
*** chandankumar has joined #kolla12:07
rhalliseydam I didn't know cinder broke again all over :(12:07
rhalliseyI didn't do anything!12:07
SamYapleim confused. so tgt was running in the cinder-volumes container?12:08
rhalliseyya12:08
SamYaplehow?12:08
SamYapletgtd has to be up and running12:08
rhalliseyI would startd tgtd first12:08
SamYaple_in_ the cinder volumes container?12:09
rhalliseyyes12:09
SamYapleoh well that means cinder was never really working12:09
SamYapleand thats not documented anywhere12:09
rhalliseywhat wouldn't that work12:09
rhalliseyIt did for a while12:09
SamYaplewhy wouldn't manual steps that have to run in the container be considered working?12:10
rhalliseyit was a start script12:11
SamYaplehttps://github.com/openstack/kolla/blob/master/docker/cinder/cinder-volume/start.sh12:11
rhalliseywell so the reason to cinder breaking is cause there's no tgt in there12:13
rhalliseyuse to be in there12:13
rhalliseythere use to be a script to start tgt12:13
SamYapleno the reason its breaking is because tgtd doesn't run anywhere. exactly where would tgtd have been run?12:13
*** dwalsh has joined #kolla12:13
SamYaplethat hasnt been there in a long long time then12:14
rhalliseyI was running in cinder-columes12:14
rhalliseyI'm confused I fixed this in like l-112:14
rhalliseywhere did tgt go?12:14
SamYaplecinder has never worked with ansible afaik12:15
openstackgerritMerged openstack/kolla: Create nova-libvirt directory before copying files  https://review.openstack.org/22724212:15
rhalliseyI think when I testing it, I was using older images because I know for a fact I had tgt around12:15
rhalliseyI'm just confused when it went.. again12:16
rhalliseySamYaple, so before, I had tgt in cinder-volumes with a start sciprt12:16
rhalliseylet me go grab that12:16
*** smeisner has joined #kolla12:19
*** dims__ has quit IRC12:19
*** dims_ has joined #kolla12:20
rhalliseySamYaple, https://review.openstack.org/#/c/200139/12:21
rhalliseythere it is12:21
rhalliseydon't know where that went12:22
rhalliseyI'll throw up a fix for it then.  That's what it should b12:23
SamYaplerhallisey: dont bother, we shouldnt be running tgt in that container anyway12:23
SamYapleive got a new tgt container to do it12:23
rhalliseywfm12:24
*** tfukushima has joined #kolla12:49
*** larsks has quit IRC13:07
*** diga has quit IRC13:07
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Heat logging to rsyslog  https://review.openstack.org/22682013:09
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Neutron logging to rsyslog  https://review.openstack.org/22727513:09
*** thumpba has joined #kolla13:11
*** banix has joined #kolla13:17
*** dwalsh has quit IRC13:19
openstackgerritSam Yaple proposed openstack/kolla: Make OpenStack use Ceph  https://review.openstack.org/22593313:26
openstackgerritSam Yaple proposed openstack/kolla: Ceph Ansible support  https://review.openstack.org/21857013:26
*** banix has quit IRC13:26
SamYaplealright guys that should bring the ceph patches up to date with your most recent changes to the json stuff rhallisey13:26
SamYapleplease review them people. docs and bug fixes can be done during the bug fix period, but these need to merge before end of RC1 (tomorrow)13:27
*** erkules_ has joined #kolla13:29
rhalliseySamYaple, k will do13:34
*** banix has joined #kolla13:35
inc0damn you keystone and your "let's run on apache" idea13:37
*** thumpba has quit IRC13:39
*** thumpba has joined #kolla13:40
*** banix has quit IRC13:42
*** jtriley has joined #kolla13:46
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Keystone logging to rsyslog  https://review.openstack.org/22731113:48
*** thumpba has quit IRC13:52
*** chandankumar has quit IRC13:55
openstackgerritMerged openstack/kolla: Heat logging to rsyslog  https://review.openstack.org/22682013:55
*** achanda has joined #kolla13:57
*** achanda has quit IRC13:59
openstackgerritMerged openstack/kolla: Fix memcached user on Debian/Ubuntu based distro  https://review.openstack.org/22611214:01
*** dwalsh has joined #kolla14:06
*** tfukushima has quit IRC14:08
*** diogogmt has joined #kolla14:09
*** dims__ has joined #kolla14:11
*** banix has joined #kolla14:11
*** dims_ has quit IRC14:14
*** diogogmt has quit IRC14:23
*** gfidente has quit IRC14:25
*** cloudnull has joined #kolla14:33
*** dwalsh_ has joined #kolla14:35
*** banix has quit IRC14:44
*** thumpba has joined #kolla14:52
*** larsks has joined #kolla14:57
*** dwalsh has quit IRC15:03
*** dwalsh_ has quit IRC15:03
rhalliseySamYaple, around?15:07
*** Windir has quit IRC15:08
openstackgerritHui Kang proposed openstack/kolla: Add pub key to the VM booted by vagrant  https://review.openstack.org/22611515:14
*** tfukushima has joined #kolla15:14
*** dwalsh has joined #kolla15:16
*** dwalsh_ has joined #kolla15:16
*** diogogmt has joined #kolla15:21
*** CBR09 has joined #kolla15:27
CBR09Hello all15:27
CBR09Is there anyone who is online?15:28
*** akwasnie has quit IRC15:29
CBR09I am testing Kolla, and I have a question.15:31
pbourke_CBR09: ask away15:33
CBR09All docker images (glance, keystoner, nova,...) in github master used for OpenStack kilo version?, or any other version?15:33
pbourke_CBR09: they are master (liberty)15:34
CBR09so, images in stable/Kilo for OpenStack kilo version?15:35
*** larsks has quit IRC15:37
*** larsks has joined #kolla15:38
CBR09Thanks you!15:41
rhalliseyCBR09, use liberty-2 tag, those are the latest stable15:43
CBR09But I want to deploy OpenStack Kilo version, liberty-2 tag usage possible?15:47
CBR09I think every version of OpenStack, there will be difference docker images15:50
*** diogogmt has quit IRC15:55
*** diogogmt has joined #kolla15:56
*** blahRus has joined #kolla15:58
pbourke_CBR09: check out this conf: https://github.com/openstack/kolla/blob/master/etc/kolla/kolla-build.conf#L46-L5716:00
pbourke_it will allow you to use source installs with any git tag16:01
CBR09@pbourke_: Oh, it's useful for me, thanks you!16:09
*** mfalatic has joined #kolla16:09
CBR09But if I use binary install, is there any way?16:11
pbourke_CBR09: I think if you comment out these lines https://github.com/openstack/kolla/blob/master/docker/base/Dockerfile.j2#L33-L3516:14
pbourke_it will use kilo packages16:14
pbourke_at least for the rpm based distros16:14
pbourke_not sure about ubuntu/debian16:15
openstackgerritMichal Jastrzebski (inc0) proposed openstack/kolla: Cinder logging to rsyslog  https://review.openstack.org/22741216:18
inc0uhh done16:18
inc0all of our basic services are logging to rsyslog nopw16:18
inc0let me know if I've missed anything16:18
CBR09That's right!, I saw16:20
CBR09for ubuntu/debian, change sources.list file.16:20
*** unicell has joined #kolla16:24
*** CBR09 has quit IRC16:24
*** CBR09 has joined #kolla16:27
*** CBR09 has quit IRC16:29
*** dwalsh has quit IRC16:30
*** dwalsh_ has quit IRC16:30
*** unicell has quit IRC16:35
*** banix has joined #kolla16:40
*** inc0 has quit IRC16:44
*** achanda has joined #kolla17:00
*** unicell has joined #kolla17:10
*** larsks has quit IRC17:18
*** achanda has quit IRC17:36
openstackgerritRyan Hallisey proposed openstack/kolla: Add missing when statments when copying JSON files in ansible  https://review.openstack.org/22743717:39
*** dwalsh has joined #kolla17:39
*** dwalsh_ has joined #kolla17:39
*** achanda has joined #kolla17:51
*** athomas has quit IRC18:04
*** exploreshaifali has joined #kolla18:15
*** athomas has joined #kolla18:29
*** achanda has quit IRC18:29
*** achanda has joined #kolla18:31
*** athomas has quit IRC18:42
*** tummy has joined #kolla18:43
*** tfukushima has quit IRC18:55
*** thumpba has quit IRC18:57
SamYaplerhallisey: around for a bit19:34
rhalliseySamYaple, no worries, I just threw up some questions on gerrit19:35
SamYaplewhich commit?19:35
rhalliseyhttps://review.openstack.org/#/c/225933/19:35
rhalliseyjust pinged you to see if I could ask irc19:35
SamYaplekk looking now19:35
*** banix has quit IRC19:39
SamYaplereplied19:42
rhalliseySamYaple, so you with regards to the cinder task in common, should that stuff be moved under the cinder task?19:45
rhalliseyI just don't understand it's placement in common19:46
SamYapleit has to be placed in common to run against the ceph-mon host19:47
SamYaplelet me look again if I can change that. i did change the implementation recently19:47
rhalliseydoes that stuff need to be run before everything like the ansible or rsyslog container19:48
SamYapleit does but that doesn't mean i cant change it, looking now19:48
rhalliseyso for instance when you ensure some cinder dirs are available.  They could be moved down into ansible/roles/cinder/task/config.yml19:50
SamYaplerhallisey: no not if the tasks are in the common role19:50
SamYapleits a dep role so it runs first19:50
SamYapleyea with the site.yml change I can move these back into the individual roles I believe19:52
rhalliseythe reason for the comment would be that's were I'd expect them to be19:52
rhalliseythat's given the my experience thus far with ansible19:52
SamYaplefor logisitical reasons they couldn't actually be there before (since some tasks have to run on the ceph-mon hosts)19:53
rhalliseyit's hard for me to argue that point because I don't know ceph unfortunatly19:54
SamYapleits not a ceph thing, its an ansible thing19:54
SamYaplebut since the ceph-mon hosts are in the site.yml in this iteration I can move the tasks back19:54
rhalliseyok I see19:55
rhalliseyso that would be the same for glance and nova task in here then19:55
SamYaplecorrect, i can move them all out since i added the ceph-mon to the hosts run and filter with the when conditionals19:56
*** tfukushima has joined #kolla19:56
rhalliseyya let's go with that model19:56
rhalliseySamYaple, ok cool thanks19:56
rhalliseyI hope that's not too painful19:57
SamYaplenot at all. its a simple move19:57
SamYaplethe playbooks stills have to run before everything else19:57
*** tfukushima has quit IRC20:01
*** absubram has joined #kolla20:08
SamYaplerhallisey: would you mind reviewing its depedant patch since that has to merge first...20:10
*** tummy has quit IRC20:13
*** larsks has joined #kolla20:20
openstackgerritSam Yaple proposed openstack/kolla: Make OpenStack use Ceph  https://review.openstack.org/22593320:29
*** achanda has quit IRC20:30
rhalliseySamYaple, ya sure20:32
rhalliseySamYaple, I'll get to ceph tmr20:33
SamYaplepeople have been saying that for 3 weeks now :/20:33
rhalliseyI put it off I'm sorry20:34
rhalliseytunnel vision on the json stuff took me away from the review q20:35
rhalliseySamYaple, I'll try and get to it tonight. So in a few hours20:37
*** exploreshaifali has quit IRC20:41
*** jmccarthy has quit IRC20:52
*** banix has joined #kolla20:53
openstackgerritHui Kang proposed openstack/kolla: Add pub key to the VM booted by vagrant  https://review.openstack.org/22611521:02
*** tummy has joined #kolla21:02
*** rhallisey has quit IRC21:04
*** daneyon has quit IRC21:04
*** achanda has joined #kolla21:05
*** daneyon has joined #kolla21:05
*** achanda has quit IRC21:09
*** jtriley has quit IRC21:11
*** shardy has quit IRC21:16
*** absubram is now known as absubram_afk21:23
*** dims__ has quit IRC21:29
*** dwalsh_ has quit IRC21:33
*** dwalsh has quit IRC21:33
*** daneyon has quit IRC21:49
*** daneyon has joined #kolla21:50
*** dims_ has joined #kolla21:52
*** tfukushima has joined #kolla21:57
*** tfukushima has quit IRC22:01
*** thumpba has joined #kolla22:06
*** achanda has joined #kolla22:06
*** thumpba_ has joined #kolla22:17
*** thumpba has quit IRC22:20
*** tummy has quit IRC22:22
*** banix has quit IRC22:24
*** banix has joined #kolla22:29
*** absubram_afk has quit IRC22:32
*** jtriley has joined #kolla22:32
*** loth has quit IRC22:47
*** loth has joined #kolla22:48
*** thumpba_ has quit IRC22:48
*** thumpba has joined #kolla22:48
*** echoingumesh has joined #kolla22:51
*** tummy has joined #kolla22:52
*** tummy has quit IRC23:02
*** jtriley has quit IRC23:03
*** dims_ has quit IRC23:05
*** thumpba has quit IRC23:06
*** diogogmt has quit IRC23:16
*** jpeeler has quit IRC23:16
*** echoingumesh has quit IRC23:17
*** echoingumesh has joined #kolla23:18
*** echoingumesh has quit IRC23:22
*** thumpba has joined #kolla23:25
*** tfukushima has joined #kolla23:28
*** alisonh has quit IRC23:28
*** blahRus has quit IRC23:29
*** tfukushima has quit IRC23:32
*** alisonh has joined #kolla23:47
*** thumpba_ has joined #kolla23:54
*** thumpba has quit IRC23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!