Tuesday, 2015-04-14

*** dsneddon has quit IRC00:15
*** dsneddon has joined #tripleo00:15
*** cwolferh has quit IRC00:16
openstackgerritMerged stackforge/kolla: Sets debug logging to false  https://review.openstack.org/17307200:24
*** cwolferh has joined #tripleo00:26
openstackgerritDan Prince proposed openstack/tripleo-image-elements: Add an element to pin to Qemu 2.1.3-3 on Fedora 21  https://review.openstack.org/17298600:32
*** cwolferh has quit IRC00:35
openstackgerritDan Prince proposed openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298900:35
openstackgerritSteve Baker proposed openstack/diskimage-builder: Mount with -o nouuid for XFS base images  https://review.openstack.org/17313800:39
*** daneyon has joined #tripleo01:27
*** daneyon has quit IRC01:31
*** eghobo has joined #tripleo01:37
*** julim has quit IRC01:38
openstackgerritgreghaynes proposed openstack/diskimage-builder: Add functional smoke test for disk-image-create  https://review.openstack.org/16916701:48
openstackgerritgreghaynes proposed openstack/diskimage-builder: Fix image size to fit filesystem journal  https://review.openstack.org/16916601:50
openstackgerritgreghaynes proposed openstack/diskimage-builder: Add functional smoke test for disk-image-create  https://review.openstack.org/16916701:50
*** wuhg has joined #tripleo01:53
*** jerryz has joined #tripleo02:02
*** eghobo has quit IRC02:04
openstackgerritDan Prince proposed openstack/tripleo-image-elements: An element to force install Qemu 2.1.3-3 on F21  https://review.openstack.org/17298602:07
openstackgerritDan Prince proposed openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298902:09
*** cwolferh has joined #tripleo02:21
*** yamahata has quit IRC02:23
*** thrash is now known as thrash|g0ne02:26
*** cwolferh has quit IRC02:40
*** spzala has quit IRC02:41
*** tzumainn has quit IRC02:48
*** eghobo has joined #tripleo02:48
*** regebro has joined #tripleo03:01
*** wuhg has quit IRC03:06
*** aberdine has quit IRC03:06
*** cwolferh has joined #tripleo03:08
*** wuhg has joined #tripleo03:13
*** aberdine has joined #tripleo03:13
*** daneyon has joined #tripleo03:16
*** eghobo has quit IRC03:20
*** daneyon has quit IRC03:21
*** yamahata has joined #tripleo03:27
*** rlandy has joined #tripleo03:34
*** sdake has joined #tripleo03:56
*** tzumainn has joined #tripleo03:57
*** sdake_ has quit IRC03:58
*** eghobo has joined #tripleo04:00
*** rlandy has quit IRC04:01
openstackgerritSteven Dake proposed stackforge/kolla: Prepare compose for custom namespace  https://review.openstack.org/17304404:01
*** sdake_ has joined #tripleo04:02
*** eghobo_ has joined #tripleo04:03
openstackgerritSteven Dake proposed stackforge/kolla: Update developer documentation  https://review.openstack.org/17299804:04
*** sdake has quit IRC04:04
*** eghobo has quit IRC04:06
*** tzumainn has quit IRC04:29
openstackgerritMartin André proposed stackforge/kolla: Fix validate-all-yaml.sh to check for files with yml extension  https://review.openstack.org/17316804:46
*** pradeep has quit IRC04:50
*** jcoufal has joined #tripleo04:53
*** masco has joined #tripleo04:57
*** maufart has joined #tripleo04:58
*** daneyon has joined #tripleo05:04
*** maufart has quit IRC05:06
*** daneyon has quit IRC05:09
*** ishant has joined #tripleo05:14
*** pradeep has joined #tripleo05:16
*** pradeep has quit IRC05:22
*** eghobo_ has quit IRC05:36
*** eghobo has joined #tripleo05:38
*** pradeep has joined #tripleo05:40
*** jtomasek has joined #tripleo05:54
*** jprovazn has joined #tripleo05:57
*** yamahata has quit IRC05:58
*** yamahata has joined #tripleo05:59
*** jprovazn has quit IRC06:03
*** devananda has quit IRC06:10
*** devananda has joined #tripleo06:17
*** spredzy|afk is now known as spredzy06:24
*** Marga_ has quit IRC06:33
*** maufart has joined #tripleo06:47
*** ukalifon has joined #tripleo06:50
*** maufart has quit IRC06:53
*** daneyon has joined #tripleo06:53
*** daneyon has quit IRC06:58
*** jprovazn has joined #tripleo07:02
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: puppet: implement MongoDB on controller nodes  https://review.openstack.org/16547407:05
*** dmellado has quit IRC07:10
*** eghobo has quit IRC07:13
*** mcornea has joined #tripleo07:24
*** dmellado has joined #tripleo07:24
*** chlong has quit IRC07:27
*** mmagr has joined #tripleo07:28
*** jistr has joined #tripleo07:29
*** yamahata has quit IRC07:30
*** dtantsur|afk is now known as dtantsur07:35
openstackgerritJan Provaznik proposed openstack/tripleo-common: Add basic project structure  https://review.openstack.org/17290707:38
*** mmagr is now known as mmagr|afk07:38
*** ifarkas has joined #tripleo07:42
openstackgerritJan Provaznik proposed openstack/tripleo-common: Add basic project structure  https://review.openstack.org/17290707:50
*** bvandenh has joined #tripleo07:51
*** mmagr|afk is now known as mmagr07:54
*** yog__ has joined #tripleo07:56
*** mcornea has quit IRC07:57
*** yog__ has quit IRC07:57
*** mcornea has joined #tripleo07:58
*** wuhg has quit IRC07:59
spredzyjistr, unfortunately, in 6 different run I couldnt reproduce the rabbitmq issue. Could you paste me the puppet log of step 2 about RabbiMQ resources ?08:01
jistrspredzy: atm i don't have it deployed, trying to deploy a potential fix08:02
spredzyjistr, ack. What angle did you take for the fix ?08:02
jistri found this http://rabbitmq.1065348.n5.nabble.com/Rabbitmq-boot-failure-with-quot-tables-not-present-quot-td24494.html08:02
jistrwhich simply suggests that we cannot start all nodes at the same time08:03
*** gfidente has joined #tripleo08:03
*** gfidente has joined #tripleo08:03
*** pradeep has quit IRC08:03
jistri'm trying to add one more step which will deploy RabbitMQ on one node first, and then in a next step it will deploy on the rest08:03
spredzyI though about that yesterday, and suggested the master get deployed in step 2 the other on step 308:04
spredzybut it randomly works, so is it just orchestration luck08:04
spredzylike master got enough time to do the job before salves went up08:04
spredzyBecause in puppet-openstack-cloud we didn't have this issue, because during step1 (which RabbitMQ is part), the provisioning is done sequentially, not in //08:05
openstackgerritYanyan Hu proposed openstack/os-collect-config: Use non-versioned auth_url for keystone  https://review.openstack.org/13824608:05
*** jcoufal_ has joined #tripleo08:06
jistrah right, i was wondering if you hit that previously in the other composition layer... that explains it08:06
openstackgerritJan Provaznik proposed openstack/tripleo-common: Add basic project structure  https://review.openstack.org/17290708:06
*** pradeep has joined #tripleo08:08
*** jcoufal has quit IRC08:09
*** derekh has joined #tripleo08:10
gfidentejistr, spredzy we can't make resources of the same resourcegroup depend on each other08:15
gfidenteif we're down to that I'd add I'd rather a random sleep so one node will get there first08:16
*** athomas has joined #tripleo08:16
gfidentehu, good keyboarding everyone08:16
gfidenteI meant I'd rather add08:16
* gfidente ♥08:17
jistrok i've just had my second successful deployment using the step split approach...08:17
gfidentewhere the other nodes come up in the second step?08:18
spredzyjistr, can you show us the diff.08:18
spredzygfidente, not sure if the sleep will be accepted upstream08:18
gfidenteah that's nice from me, I forgot we could do that :)08:18
gfidentespredzy, don't let me start on the puppet-corosync module ...08:18
gfidente... it actually prevents you from using udpu on Red Hat forcing to pass params which can't go toggether08:19
gfidentespredzy, so the sleep is the worst idea possible08:20
jistri added one more step because i wanted all rabbitmq nodes to be ready before the OpenStack services start to get configured08:20
jistrthe issue there is that i cannot do:08:21
jistrstep 2 - include rabbit if i'm bootstrap node08:21
jistrstep 3 - include rabbit always08:21
gfidentejistr, can we shift one node to step1 ?08:21
jistrbecause i cannot cut out the rabbit part out for inclusion, we have to put everyting into one file the way it is now08:21
*** sdake has joined #tripleo08:22
hewbroccaI don't guess the rabbit resource agent helps with this bootstrapping stuff at all?08:22
jistrit might08:22
spredzyjistr, I think to keep it shorter gfidente is right. We could change the if with [Step 1 AND bootstrap node] OR [step2+]08:25
spredzywe do nothing in step 2 that requires rabbitmq08:25
spredzyso initiating it in the bootstrap node on step 1 won't hurt08:25
*** lucasagomes has joined #tripleo08:26
spredzythis way the change stay concise, only puppet related08:26
*** sdake_ has quit IRC08:26
spredzyhewbrocca, in the euro timezone ? :08:26
jistrspredzy, gfidente: do you know why we currently initialize HAproxy in its own step outside everything else? do we need it in step 2?08:27
* jistr looks further into the code08:27
hewbroccaspredzy: I am -- I have moved permanently to Brno :)08:28
gfidentejistr, stuff in step 2 relies on the VIP08:29
*** mcornea has quit IRC08:29
*** sdake has quit IRC08:30
gfidentejistr, I think the ::mysql might have attempted sync08:30
spredzyjistr, yes all the X::db::mysql on step 2 relies on the vip08:31
spredzyhewbrocca, nice, big change though.08:31
jistrgfidente, spredzy: ack, thanks. I've updated the code for step 1/2 instead of 2/3, testing now.08:32
hewbroccaspredzy: yep!08:34
*** yog__ has joined #tripleo08:34
*** mcornea has joined #tripleo08:40
*** daneyon has joined #tripleo08:42
*** mmagr is now known as mmagr|afk08:43
*** shardy has joined #tripleo08:46
*** daneyon has quit IRC08:47
*** mmagr|afk is now known as mmagr08:53
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Initialize RabbitMQ on bootstrap node first  https://review.openstack.org/17322208:58
jistr^^ gfidente spredzy08:58
jistrhmm i think we should get to a green multi-node controller deployment as soon as we can. RabbitMQ will start now but the deployment still fails later on.08:59
spredzyjistr, what the following issue ?09:00
*** untriaged-bot has joined #tripleo09:00
untriaged-botUntriaged bugs so far:09:00
*** untriaged-bot has quit IRC09:00
openstackLaunchpad bug 1443706 in diskimage-builder "Failed to mount: Filesystem has duplicate UUID" [Undecided,In progress] - Assigned to Steve Baker (steve-stevebaker)09:00
jistrDB syncs09:00
jistri tried with the Galera patch too but it failed on Ceilometer DB sync too there09:00
spredzyjistr, which one ?09:00
*** pradeep has quit IRC09:02
jistrspredzy: this one https://review.openstack.org/#/c/164983/ and the dependent one in puppet-tripleo. I now didn't deploy with it to get rid of any side effects though, so i don't have the error at hand right now.09:02
jistri think we should get to a green HA deployment as soon as we can, no matter if we have to remove some features initially (from HA only). Right now when the deployment constantly fails, it's time consuming to verify if the breakage is due to my change or if its "the normal breakage"...09:04
jistrwhich reminds me of an issue that's even higher priority i'd say -- our non-HA CI is broken again09:05
hewbroccajistr: +109:05
spredzyjistr, dprince pushed a patch to tripleo-ci09:05
spredzyit needs to be approved for CI to be back in shape09:06
*** dtantsur is now known as dtantsur|brb09:06
openstackgerritDerek Higgins proposed openstack/tripleo-image-elements: An element to force install Qemu 2.1.3-3 on F21  https://review.openstack.org/17298609:06
jistrspredzy: thx for the info09:08
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298909:08
jistrhmm the CI failed on it though09:08
spredzyjistr, for the Galera I'll be interested in knowing the issue you've found. Just re run one and got the stack up and running09:08
spredzyjistr, check derekh last change ^09:09
*** pelix has joined #tripleo09:10
derekhjistr: ya, the puppet job is currently failing because of a regression in the qemu package, were trying to get CI to use an old version09:10
*** Goneri has joined #tripleo09:10
jistrderekh, spredzy: thanks, i've +2'd both patches, we'll see what Jenkins will post09:12
derekhjistr: k,thanks, the first shouldn't be merged its only there so we can cherrypick it, I'll WIP it now09:13
openstackgerritMerged openstack/tripleo-image-elements: An element to force install Qemu 2.1.3-3 on F21  https://review.openstack.org/17298609:13
derekhsh*t, that was the oposite to what I wanted to do09:14
jistrderekh: :D09:14
*** bus-104_ has quit IRC09:15
jistrnvm it won't break anything09:15
derekhnope, we can remove it again later, stupid stupid stupid09:15
jistryeah we can prepare a revert patch and remove it anytime09:15
derekhstill gotta wait on cI to see what happens09:16
* derekh goes to wake himself up properly09:16
*** pradeep has joined #tripleo09:17
*** akrivoka has joined #tripleo09:26
spredzyjistr, derekh where are 'precise' and 'nonha' defined for the CI ? I suspect its on toci_gate_test.sh but I am wondering where is TOCI_JOBTYPE coming from09:31
derekhspredzy: TOCI_JOBTYPE is coming from the infra job definition here http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/tripleo.yaml09:33
derekhspredzy: the next question would be where does {type} come from, one sec and I'll find it09:33
spredzyderekh, hehe yes indeed09:34
derekhspredzy: that first file is a template for all the tripleo jobs, which is used here09:35
*** yuanying has quit IRC09:35
spredzyderekh, thanks. great. So noramlly adding a f20puppet-ha job is a simple as creating an extra stanza in this file. Since f20puppet and ha are already part of toci_gate_test.sh things will picked up autimatically, right ?09:37
spredzywell apparently there is something else to do since 'ironic-vlan-precise-nonha' is in the file but not enabled, so I should also enable it properly09:39
derekhspredzy: yup, instead of adding an extra entry, I'm thinking we should replace overcloud-precise-ha with overcloud-f20puppet-ha because the precise one is consistently failing and nobody is maintaining it09:40
derekhspredzy: ahh, yes you would also need to define which projects to run the new job against, looking for a link09:41
spredzyok I ll make the review. I'll specify that we overide precise-ha due to the fact that precise-ha is unmaintained09:41
derekhspredzy: cool, here is the file where you define what projects to run against http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n262809:43
* spredzy checking09:44
spredzyderekh, thanks09:45
derekhspredzy: gfidente was looking into doing the same thing I think, but was waiting until the puppet ha job would pass09:45
derekhspredzy: if its not expected to pass yet, make sure its left as non voting, we can then switch it to voting once its good09:45
gfidentederekh, yeah I was a little concerned that during migration it will not that much useful09:45
gfidentebut non-voting is bettet than not having09:45
spredzyderekh, that was my idea, so we can see jistr patch passing and then probably enable it09:46
spredzycurrently with jistr patch I have a succesful deployment09:46
derekhspredzy: ack09:46
spredzywith 3 nodes09:46
spredzycontroller nodes I meant09:46
gfidentederekh, I was also thinking why we insist on 2 computes09:47
gfidentethat seems to me waste of resources, we're not testing the nova scheduler09:47
gfidenteI'd drop all jobs to 1 ctrl and 1 compute09:47
gfidenteand use those resources to play a bit with the block storage scaling, or ceph nodes09:47
gfidenteceph probably more useful than cinder scaling09:48
gfidentecan I attempt that after spredzy switches to puppetha?09:48
*** cody-somerville has quit IRC09:49
*** cody-somerville has joined #tripleo09:49
openstackgerritDerek Higgins proposed openstack/tripleo-image-elements: Mask yum downgrade qemu errors  https://review.openstack.org/17323609:50
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298909:51
derekhgfidente: sounds ok to me, I'm trying to remember why we originally kicked off 2 compute nodes but can't think of why09:53
*** lifeless1 has joined #tripleo09:54
*** mandre_ has joined #tripleo10:01
*** cwolferh has quit IRC10:02
*** dsneddon has quit IRC10:02
*** lifeless has quit IRC10:02
*** mandre has quit IRC10:02
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298910:07
*** dtantsur|brb is now known as dtantsur10:09
*** pblaho has joined #tripleo10:11
*** pblaho has quit IRC10:12
*** pblaho has joined #tripleo10:12
gfidentederekh, https://review.openstack.org/#/c/172989 seems like one of the needed changes merged10:13
derekhgfidente: thanks for pointing that out, I merged it by accident, when I intended to -2 it .... the comment in that patch needs to be updated to reference a different patch, I'll sort it out once I see what ci does10:15
gfidentebut the code is using the qemu-pinned element10:17
gfidenteyou need to remove that too?10:17
derekhgfidente: no we need to use the qemu-pinned element, maybe I'm missunderstanding what your asking, the qemu-pinned element merged but now we are pulling in an update to it https://review.openstack.org/#/c/173236/10:19
openstackgerritGiulio Fidente proposed openstack/tripleo-incubator: Switch default COMPUTESCALE setting down to 1 from 2  https://review.openstack.org/17324810:19
gfidenteah the update is what I missed10:19
gfidenteso you wanted to -2 initially to make the update there from the start10:20
derekhgfidente: yup, I should never have merged the qemu-pinned element10:21
*** daneyon has joined #tripleo10:31
spredzyderekh, gfidente jistr ^10:34
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298910:34
*** daneyon has quit IRC10:36
derekhspredzy: looks fairly close to me, comments added10:41
gfidentederekh, looking https://github.com/openstack-infra/tripleo-ci/blob/master/toci_gate_test.sh10:44
gfidentegiven USE_MERGEPY is set in _variables.sh to 0 I think all jobs are actually running without merge at this point?10:44
ukalifonshardy: Hello. Can I ask you about the stepwise installation which uses the heat breakpoints? I want to know where the breakpoints will be placed on the stack, so I can test the feature similarly to how it will be used. Which tests will run in each phase ?10:44
spredzyderekh, thank you. updated.10:45
openstackgerritGiulio Fidente proposed openstack/tripleo-puppet-elements: Add pacemaker, pacemaker_remote and pcs packages to images  https://review.openstack.org/17256510:46
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Perform basic setup of pacemaker cluster on controllers  https://review.openstack.org/17257210:46
derekhgfidente: on ubuntu we are setting USE_MERGEPY to 1 in that file and nothing is changing it back to 010:47
derekhgfidente: its only set it devtest_variables if it hasn't been set already10:47
openstackgerritMerged openstack/tuskar-ui: Fix node list  https://review.openstack.org/17239910:47
gfidentederekh, ah so this is actually setting it10:47
gfidenteI was looking at how combine things to get only a single overcloud-precise-nonha job use merge.oy10:48
shardyukalifon: Hi, currently the breakpoints will be placed in the *NodesPostDeployment resources, e.g:10:49
shardyukalifon: EmilienM is working on serverspec tests which will test each *StepN of the configuration10:49
*** ishant has quit IRC10:50
openstackgerritGiulio Fidente proposed openstack-infra/tripleo-ci: Use merge.py only for overcloud-precise-nonha job  https://review.openstack.org/17326310:52
ukalifonEmilienM: Can you outline what in general the scripts will test in every step? What is installed and ready for test in step1, what in step2 etc...10:57
*** lsmola_ has joined #tripleo10:58
gfidentespredzy, derekh the ha job, instead of adding it to tripleo-check-jobs I was thinking if we shouldn't run it instead only were very pertinent11:00
gfidentefor instance, not sure what is benefit of running it against diskimage-builder, we test that on f20 already11:00
gfidenteI'm even unsure about tripleo-image-elements given it runs on puppet now11:01
spredzygfidente, which project would be pertinent ? puppet-tripleo, tripleo-heat-templates and what else ?11:01
gfidentespredzy, yeah I would take the route of only adding it were makes sense11:01
gfidenteso I would add it to puppet-tripleo for instance, exactly, remove from dib11:02
spredzygfidente, can you -1 it dropping this comment11:02
derekhgfidente: spredzy yup shaving the list of projects down a bit would be a good idea11:02
spredzythere, so we have 'history' and I'll amend it11:02
spredzyderekh, gfidente ok this is a proposition. put f20puppet-ha (remove it from check-tripleo-jobs) and add it on tripleo-heat-templates. Would that make sense ? (total projects would be: puppet-tirpleo, tripleo-heat-templates, tripleo-puppe-element)11:05
spredzysorry put f20puppet-ha in check-tripleo-puppet-jobs*11:05
gfidentespredzy, those three plus tripleo-incubator11:06
gfidenteso four in total, like they already are, only reshuffled :P11:06
gfidentederekh, the CONTROLSCALE thing, why is it there?11:08
gfidentewe don't enforce COMPUTESCALE so why shall we enforce CONTROLSCALE?11:08
gfidentehonestly was only an attempt to cleanup, I think either we enforce both or none of the two and rely on defaults11:09
*** masco has quit IRC11:09
spredzygfidente, thx. updated. Does it look more accurate to you ?11:11
gfidentespredzy, needs to be added to tripleo-puppet-elements and puppet-tripleo?11:12
spredzygfidente, puppet-tripleo yes, to test changes to the tripleo::loadbalancer class11:14
*** thrash|g0ne is now known as thrash11:14
spredzyand other stuff we might be adding to it in relation w/ HA11:14
*** aberdine has quit IRC11:15
gfidenteit's missing in the change I mean11:15
spredzygfidente, nop. the job is now part of check-tripleo-puppet-jobs11:15
spredzyand both puppet-tripleo and tripleo-puppet-elements run checks in that group11:15
gfidenteAH, sorry missed that, thanks11:15
derekhgfidente: I guess it was put there to be explicit about what we are doing11:17
*** rhallisey has joined #tripleo11:17
derekhgfidente: I know git blame will tell you it was me that did it, but git blame is wrong ;-)11:18
derekhgfidente: I took the config from one repo and moved it to another11:18
derekhgfidente: https://review.openstack.org/#/c/107468/11:18
gfidenteso I dunno, it's a stupid thing to ask but, for the glory of coherence, would you add COMPUTESCALE or remove CONTROLSCALE ?11:19
*** aberdine has joined #tripleo11:20
derekhgfidente: I think any variable we set in CI at all in some cases we should set in all cases, that would mean leaving CONTROLSCALE the way it is so it obvious what we are doing when looking at that file11:21
derekhfor COMPUTE scale if it is alway going to be default, then I have no problem with leaving it out of the file11:21
gfidentederekh, ack, but I don't like leaving COMPUTESCALE depend on default then :P11:21
derekhbut I'm also happy top see it in there mainly for consistency11:22
gfidentein another change though, if we want11:22
*** spredzy is now known as spredzy|afk11:27
openstackgerritGiulio Fidente proposed openstack-infra/tripleo-ci: Use merge.py only for overcloud-precise-nonha job  https://review.openstack.org/17326311:30
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Perform basic setup of pacemaker cluster on controllers  https://review.openstack.org/17257211:30
*** yog__ has quit IRC11:41
*** weshay has joined #tripleo11:43
jistrspredzy|afk: tried deploying again with the galera patch, no failed db syncs reported it seems, but neutron fails to start because of a DB state issue: http://paste.fedoraproject.org/210753/42901174/11:47
* jistr -> english class11:47
*** jistr is now known as jistr|class11:47
*** lucasagomes is now known as lucas-hungry11:49
*** trown|outttypeww is now known as trown11:53
*** mcornea has quit IRC11:57
*** Marga_ has joined #tripleo11:57
*** ukalifon has quit IRC11:58
*** mcornea has joined #tripleo11:58
*** panda|afk is now known as panda11:59
derekhthat qemu pin on the puppet job worked https://jenkins04.openstack.org/job/check-tripleo-ironic-overcloud-f20puppet-nonha/196/ , back in a few plase merge when the f20 job passes12:01
*** ukalifon has joined #tripleo12:03
*** morazi has joined #tripleo12:06
*** dprince has joined #tripleo12:07
openstackgerritJan Provaznik proposed openstack/tripleo-common: Scale out heat stack  https://review.openstack.org/17328312:12
*** daneyon has joined #tripleo12:20
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Preformat node IPs list in the template  https://review.openstack.org/17328712:24
openstackgerritMerged openstack-infra/tripleo-ci: Pin qemu-img on Fedora 21 jobs.  https://review.openstack.org/17298912:25
*** daneyon has quit IRC12:25
dprincederekh: ^^ thanks12:25
dprincederekh: FWIW, I was testing yesterday with a seed VM and my qemu pin'ing patches worked there.12:27
dprincederekh: then we made changes around the DIB up-to-date patch. A long road to simply get a package 'pinned'.12:29
dprincederekh: seems like it shouldn't have to be this hard to do something so simple :(12:29
* hewbrocca notes Pulp has versioned repos12:32
*** regebro has quit IRC12:34
*** spredzy|afk is now known as spredzy12:42
*** rlandy has joined #tripleo12:44
*** mmagr is now known as mmagr|afk12:44
*** lucas-hungry is now known as lucasagomes12:46
*** jdob has quit IRC12:46
*** jdob has joined #tripleo12:47
openstackgerritMerged openstack/tuskar-ui: Only show charts for provisioned nodes  https://review.openstack.org/17240612:50
slagleshardy: hi, got a few mins to help me with getting a software deployment DELETE action to execute?12:51
spredzyjistr|class, hmmm never had that :/12:52
spredzyLet's try to get the f20puppet-ha in so we can troubleshoot from a common setup12:53
*** tzumainn has joined #tripleo12:53
spredzyjistr|class, dprince ^12:53
*** ukalifon has quit IRC12:57
dprincespredzy: thanks, looks good.12:59
dprincespredzy: I gotta say... after the latest round of Fedora 21 breakages I'm about to say lets rip all the Fedora 21 jobs out and go with Centos instead12:59
dprincespredzy: but for now this is fine I think13:00
dprincespredzy: what are your thoughts on running ceph?13:01
dprincespredzy: do we want to do that in the HA job? Or perhaps in the normal fedora20 job instead.13:01
dprincespredzy: I've got a couple patches in the works to move towards that...13:01
spredzydprince, for CentOS from my point of view I'll be fine with it. The less the CI is broken, the faster we will move on13:02
derekhdprince: yup, it should have been easier, btw I didn't mean to merge the t-e-i patch, hit the wrong button when I meant to -213:02
spredzydprince, for Ceph, even if it might be overkill I tend to think that ceph should be tested in both ha and nonha setup13:03
*** jistr|class is now known as jistr13:03
dprincederekh: oh, right. I hadn't noticed.13:03
spredzyit would be sad to miss a patch that break the setup in a really specific case if we could avoid it13:03
dprincespredzy: right, so for the HA setup I think we are going to have to have breakpoints13:03
spredzythis way we will be confident in the CI13:03
shardyslagle: Hi, sure13:04
dprincespredzy: for Ceph, without breakpoints I have to enable this: https://review.openstack.org/#/c/172597/13:04
* spredzy checks13:04
dprincespredzy: otherwise we have a race between when Cinder recognizes how much Ceph storage there actually is13:05
dprincespredzy: what I'm getting at is we've had really flaky HA jobs up until this point. I'd like to suggest we use breakpoints to guarantee things come up in an expected order13:06
spredzydprince, what's the actual status of breakpoints ?13:07
dprincespredzy: feature wise it is all landed I think. We just need to outfit devtest_overcloud.sh to support it13:07
shardyspredzy: all the heat stuff has landed, I'm working on making it easier to use via devtest_overcloud right now13:07
dprincespredzy: yup, shardy is on it13:08
hewbroccashardy: excellent!13:08
spredzydprince, shardy awesome.13:10
spredzydprince, So the question is which scenario we want to test in nonha. It seems that you need this patch if you want Ceph on the controller. but what about a setup with a CEPHSTORAGE node ?13:12
spredzywe shouldn't heat the issue you're trying to bypass with the patch you mentioned13:12
dprincespredzy: SO I had issues in both cases.13:12
dprincespredzy: because there is a race with Cinder volume's periodic task13:13
dprincespredzy: if cinder-volume gets started before the Ceph cluster is fully up CI is hosed13:13
dprincespredzy: because it would register 0.0 space13:13
dprincespredzy: and thus volume's fail to get scheduled via cinder scheduler13:13
dprincespredzy: if we put a storage on the controller (in step2) we guarantee there is some storage available, and thus cinder-scheduler is happy for the CI case13:14
dprincespredzy: again, the right solution here is breakpoints13:14
*** jprovazn has quit IRC13:14
dprincespredzy: but for the simple All-in-one testing case having Ceph storage on the controller is useful too (and matches what we already do w/ Swift)13:15
*** regebro has joined #tripleo13:15
spredzydprince, ack. Then IMO lets work toward making CI relying on it first  and then improve the setup deployed in the CI in both ha and nonha scenario, once orchestration won't be an issue anymore13:15
dprincespredzy: note that I'm also suggesting we add a flag to disable Swift on the controller here as well: https://review.openstack.org/#/c/172616/13:15
dprincespredzy: exactly13:15
dprincespredzy: the reason I'm proposing these things is they are required to make devtest.sh (using Ceph) actually run to completion13:16
dprincespredzy: using these patches I am able to enable RBD for Nova, Cinder, Glance and we are good to go13:16
*** julim has joined #tripleo13:17
slagleshardy: hi, so i'm not seeing the deployments that have the DELETE action get run when i delete my stack13:18
slagleshardy: i added a deployment for DELETE here: https://review.openstack.org/#/c/172065/13:18
slagleshardy: was wondering if i was missing something somewheres, like a wait condition perhaps?13:20
spredzydprince, those 2 patch are usefull even w/o breakpoint. It allows to test a all-in-one setup13:20
shardyslagle: No, that looks OK to me, maybe a bug, let me try to reproduce13:21
dprincespredzy: exactly, Swift storage is unfortunately already enabled by default on the controller though so I set the default there to true13:21
shardyslagle: provided you don't specify NO_SIGNAL in the deployment, things should get correctly serialized13:22
shardyslagle: with SoftwareDeployment resource, WaitConditions are generally not needed, because there's signalling built-in13:22
slagleshardy: ok13:22
spredzydprince, so first iteration CI ha setup w/o ceph, when breakpoints gets fully supported in devtest we will apply the necessary change to have ceph deployed in the CI in both setup13:23
dprincespredzy: exactly13:23
slagleshardy: i also tried moving the deployment to not be part of ExtraConfig, but that didnt seem to make any difference13:23
spredzydprince, deal13:24
spredzyCI is back to green on reviews. Thanks dprince derekh and all13:28
dprincebnemec, helped alot too. A team effort... and a lesson in frustration (for me at least)13:33
*** lblanchard has joined #tripleo13:33
spredzyand bnemec :)13:33
*** mcornea has quit IRC13:33
*** sdake has joined #tripleo13:35
*** mcornea has joined #tripleo13:36
*** pradeep has quit IRC13:38
*** sdake has quit IRC13:40
spredzydprince, do you know any one core on openstack-infra I should add as a reviewer on the f20puppet-ha review ?13:41
dprincespredzy: I'd ask on IRC13:43
*** weshay is now known as weshay_mtg13:45
dprincespredzy: since it is in the projects repo maybe anteaya, and AJaeger13:46
spredzydprince, ok thank you.13:46
dprincespredzy: they are all quite helpful (but busy too)13:46
*** jprovazn has joined #tripleo13:53
*** jprovazn has quit IRC13:53
*** jprovazn has joined #tripleo13:53
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: puppet: implement MongoDB on controller nodes  https://review.openstack.org/16547413:55
*** sdake has joined #tripleo13:57
openstackgerritMerged openstack/tripleo-puppet-elements: Add newline to end of source-repository-puppet-modules  https://review.openstack.org/17297313:59
*** mmagr|afk is now known as mmagr14:02
*** daneyon has joined #tripleo14:09
jistrspredzy: thx for tackling the HA CI, that's going to be extremely helpful14:12
*** daneyon has quit IRC14:13
hewbroccait will also be a thing we have never had before14:14
gfidentehewbrocca, not true, we're indeed switching the elements based job to puppet14:16
hewbroccagfidente: ahh, sure, fair point14:18
gfidentehewbrocca, jprovazn could be around so ...14:18
gfidentedprince, if you worry about the number of nodes in CI, can you check this https://review.openstack.org/#/c/173248/14:19
gfidenteand also I think the sooner we switch this the bette https://review.openstack.org/#/c/173263/14:19
hewbroccajankomat :D14:19
dprincegfidente: I'm not too worried about that now.14:20
dprincegfidente: for the HA job I think we can adjust it14:20
gfidentedprince, why you want the ceph node on controllers then?14:20
dprincegfidente: but for the normal default I think two computes is fine (it allows you to test migrations for example)14:20
gfidentedprince, except we never tested it14:21
dprincegfidente: I want ceph on the controller so with a smaller setup we can use ceph14:21
gfidenteand continue spawning 2 computes for each job14:21
dprincegfidente: 1 controller (with ceph), 1 compute14:21
dprincegfidente: I want flexability14:21
dprincegfidente: people can argue about defaults...14:21
dprincegfidente: also, see my notes to spredzy about why (functionally) Ceph on the controller today is required14:22
gfidenteyou mean the ceph on controller default or the number of computes default?14:22
* jprovazn reads back14:22
dprincegfidente: without breakpoints to orchestrate things coming online in an orderly fashion devtest will fail14:22
dprincegfidente: using these patches was the *only* way I could get devtest (with ceph) actually running14:22
dprincegfidente: I'll push a tripleo-ci patch soon which uses these things...14:23
dprincegfidente: but these would be a requirement14:23
gfidentedprince, so the problem here is that devtest attempts to start a demo gues14:23
gfidentebefore cinder-volume realized there is some amount in the ceph storage14:24
dprincegfidente: that isn't a problem. That is a requirement14:24
shardydprince: doh, I just hacked out a patch for adding custom environments before seeing https://review.openstack.org/#/c/172257/ ;)14:24
gfidentewhich it checks every minute, did i get it right?14:24
dprinceshardy: I'm working on making the images customizable too FWIW14:24
shardyI was thinking it'd be good to enable any number of additional custom environment files14:24
dprinceshardy: oh, cool. I could go for that as well14:25
shardydprince: my patch does that, I may push it to see if there's a combined version we could agree on14:25
jprovazngfidente, hewbrocca: apparently I joined in the middle of some chat, not sure what was the question, but you made a good point with jankomat ;) (LFMF - never ever lend money to jcoufal_ )14:25
shardyyour patch looks fine tho so we could just land that for now :)14:25
jcoufal_jprovazn: what? do you owe me some money? :)14:26
jprovaznjcoufal_, almost correct14:26
dprinceshardy: where is your patch?14:27
dprinceshardy: I want to get this right, so if merging efforts makes sense lets do it now14:27
openstackgerritSteven Hardy proposed openstack/tripleo-incubator: Allow additional heat environment files  https://review.openstack.org/17334614:28
shardydprince: ^^14:28
shardyyour interface is a bit cleaner, I took the easy option of requiring -e in the variable string14:28
shardywe could add a loop which just adds that though I guess14:29
spredzyjistr, if I want to see how pacemaker is used in staypuft/quickstack what link should I look at ?14:29
jistrspredzy: here https://github.com/redhat-openstack/astapor/tree/master/puppet/modules/quickstack/manifests/pacemaker14:29
dprinceshardy: either way works. A bash array with a loop could be nice14:29
spredzyjistr, thanks14:30
jistrspredzy: in quickstack it's an essential part of the deployment process so there's *a lot* of puppet code dealing somehow with pacemaker14:30
shardydprince: since yours is already reviewed, I guess we just land that, I can add loop & support for multiple files after14:30
jistrspredzy: e.g. because quickstack deploys in a single step, but it still needs to somehow perform ordering between the nodes, pacemaker properties are used to do that synchronization/"orchestration"14:31
spredzyjistr, ok, looking into it14:32
hewbroccajistr: currently quickstack uses the Galera resource agent to handle that orchestration, correct?14:33
dprinceshardy: I'm okay w/ that I guess. Although if we are going to change the variable name we should perhaps do that now14:33
jistrhewbrocca: hmm i don't think so, lemme dig up what i meant14:34
*** Marga_ has quit IRC14:34
shardydprince: I'm fine with OVERCLOUD_CUSTOM_HEAT_ENV provided we can make that allow multiple space-seperated paths later14:34
shardyOVERCLOUD_CUSTOM_HEAT_ENV="firstboot_params.yaml post_deploy_config.yaml" etc14:35
dprinceshardy: So in that case: would OVERCLOUD_CUSTOM_HEAT_ENVIRONMENTS make more sense?14:35
dprinceshardy: longer, but more specific14:35
jistrhewbrocca, spredzy ^^14:35
shardydprince: yeah possibly - I went for "extra" due to the historical connection with extra config14:36
jistrhewbrocca, spredzy: that's essentially a workaround for not being able to do multi-step deployment, and the resulting ordering is even more strict than what we have in the steps. The quickstack manifests are quite complex and have a lot of explicit ordering, but as a result the deployment is pretty predictable.14:39
jistr(not suggesting we should follow the same path though ;) )14:40
hewbroccaThen there is also the method that jprovazn / jankomat implemented in the image elements14:40
jistrjprovazn: can you please point me where i could find the important bits re ^ ?14:42
jprovaznjistr, hewbrocca: TIE script is more complex because it doesn't expect multi step deployment, the important bits are here - since the orig patch this has been rewritten by better experts (so "implemented by jankomat is not true ;):14:44
jprovaznbut I suggest to at least read through to get better idea what all pitfalls you can hit14:45
*** noslzzp has joined #tripleo14:45
jistrhmm that's pretty complex logic there14:47
jprovaznjistr, in case of multi step deployment it might be easier, nice thing about the above script is that it will make a cluster of nodes started in various order and it will deal with failed nodes14:51
*** pradeep has joined #tripleo14:53
gfidentespredzy, regarding the pacemaker change, resource-agents are installed as deps so I didn't include it explicitly14:53
* jistr rebuilding images with the pacemaker bits included14:55
gfidentejistr, spredzy so the reason I had to use broadcast is that there is a problem in puppet-corosync as to use udpu you're supposed to pass unicast_addresses AND quorum_members14:56
gfidentejistr, spredzy but if you do that the resulting config turns invalid (for RH based only, where votequorum is enabled)14:57
gfidenteI think we should fix this at some point though and use udpu14:57
* spredzy back from meeting, reading backlog about pacemaker14:59
*** untriaged-bot has joined #tripleo15:00
untriaged-botNo untriaged bugs so far! \o/15:00
*** untriaged-bot has quit IRC15:00
*** noslzzp_ has joined #tripleo15:06
*** noslzzp has quit IRC15:08
openstackgerritGiulio Fidente proposed openstack-infra/tripleo-ci: Enforce COMPUTESCALE to 1 for CI, except HA jobs  https://review.openstack.org/17337215:09
*** noslzzp_ has quit IRC15:11
*** pdb has quit IRC15:20
*** pdb has joined #tripleo15:21
*** wfoster has quit IRC15:23
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Remove heat-templates cherrypick  https://review.openstack.org/17293715:23
*** yamahata has joined #tripleo15:24
*** mmagr is now known as mmagr|afk15:28
*** mmagr|afk is now known as mmagr15:30
*** wfoster has joined #tripleo15:32
*** jerryz has quit IRC15:39
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder container  https://review.openstack.org/17096515:41
*** dprince has quit IRC15:46
*** rhallisey has quit IRC15:48
*** dtantsur is now known as dtantsur|afk15:48
*** ifarkas has quit IRC15:48
*** rhallisey has joined #tripleo15:50
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Enable passing optional first-boot user-data  https://review.openstack.org/17013715:50
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Example of post-deploy hook doing rhel registration  https://review.openstack.org/17206515:50
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Add hooks for extra post-deployment config  https://review.openstack.org/17205015:50
*** jistr has quit IRC15:51
*** spredzy is now known as spredzy|afk15:52
slagleshardy: so in https://review.openstack.org/#/c/172065/6 i attempted moving the delete deployments to the top level stack15:52
*** eghobo has joined #tripleo15:53
slagleshardy: didnt seem to help though15:53
*** mmagr has quit IRC15:55
*** daneyon has joined #tripleo15:58
shardyslagle: Ok, thanks, I'm trying to reproduce on devstack with some minimal examples16:01
*** spzala has joined #tripleo16:01
*** bvandenh has quit IRC16:02
*** daneyon has quit IRC16:02
*** jcoufal_ has quit IRC16:11
*** sdake_ has joined #tripleo16:11
*** Marga_ has joined #tripleo16:12
*** sdake_ has joined #tripleo16:12
*** Marga_ has quit IRC16:14
*** Marga_ has joined #tripleo16:15
*** noslzzp has joined #tripleo16:15
*** sdake has quit IRC16:15
*** Marga_ has quit IRC16:18
*** Marga_ has joined #tripleo16:19
*** noslzzp has quit IRC16:20
*** mcornea has quit IRC16:22
*** Marga_ has quit IRC16:23
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder container  https://review.openstack.org/17096516:27
mordredgreghaynes: our new image works on both vexxhost and runabove16:31
greghayneshow many distros?16:32
mordredgreghaynes: I'm going to kick off a new end-to-end test with all of the clouds and regions with your latest patch16:32
mordredgreghaynes: only tried ubuntu for now16:32
greghaynesah, ok16:32
greghaynesI verified fedora gets networking and then the others bulid16:32
greghaynesbefore this last round of changes16:32
mordredfedora and ubuntu cover most of teh crazy choices16:32
greghaynesalthough they were only rebases for !simple-init16:32
mordredgreghaynes: well, I haven't tested since you fixed the source-repositories depend16:36
mordredgreghaynes: so I';m removing my hackaroudn16:36
greghaynesok, I was able to build and boot those16:36
greghaynesdidnt verify networking and such16:36
mordredgreghaynes: my build/boot script does a "wait_for: port=22"16:37
mordredgreghaynes: so if it doesn't get networking and a working ssh - it errors16:37
greghayneshah, I havent added ssh in16:37
greghaynesdo you have an element that adds it or something?16:37
*** Goneri has quit IRC16:38
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Preformat node IPs list in the template  https://review.openstack.org/17328716:39
mordredgreghaynes: github.com:emonty/ansible-build-image16:41
mordredgreghaynes: is what I'm using to test16:41
mordredgreghaynes: it requires the infra element from https://review.openstack.org/#/c/140840/16:42
mordredgreghaynes: which I've just updated with my latest playbook16:42
*** derekh has quit IRC16:43
*** noslzzp has joined #tripleo16:44
*** Marga_ has joined #tripleo16:46
*** Marga_ has quit IRC16:46
*** Marga_ has joined #tripleo16:51
*** Marga_ has quit IRC16:58
*** sdake has joined #tripleo17:07
*** Marga_ has joined #tripleo17:08
*** sdake_ has quit IRC17:11
*** sdake_ has joined #tripleo17:14
*** dprince has joined #tripleo17:18
*** sdake has quit IRC17:18
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Preformat node IPs list in the template  https://review.openstack.org/17328717:21
*** spredzy|afk is now known as spredzy17:22
*** trown is now known as trown|lunch17:26
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Preformat node IPs list in the template  https://review.openstack.org/17328717:26
*** sdake has joined #tripleo17:31
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder container  https://review.openstack.org/17096517:31
*** pelix has quit IRC17:32
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder container  https://review.openstack.org/17096517:34
mordredgreghaynes: qemu-img: Could not open '/tmp/image.64ZHgR8r/image.raw': Could not open '/tmp/image.64ZHgR8r/image.raw': No such file or directory17:34
*** sdake_ has quit IRC17:35
*** Marga_ has quit IRC17:35
*** Marga_ has joined #tripleo17:35
*** Marga_ has quit IRC17:36
*** Marga_ has joined #tripleo17:36
mordredgreghaynes: hrm - maybe that wasn't the real error ...17:38
greghaynesmordred: huh, is https://review.openstack.org/#/c/169911/ in your ancestry?17:38
shardyslagle: sorry for the slow response, I've reproduced your problem - it's a heat bug, I've got a fix, just working on tests now17:38
mordredgreghaynes: nope. it is not17:39
mordredgreghaynes: rebased - trying again17:39
*** lucasagomes is now known as lucas-dinner17:42
*** sdake_ has joined #tripleo17:44
*** sdake has quit IRC17:45
*** sdake_ has quit IRC17:48
*** sdake has joined #tripleo17:48
openstackgerritMerged openstack/tripleo-heat-templates: Make rabbitmq bind on ctlplane instead of  https://review.openstack.org/17184317:50
*** sdake has quit IRC17:52
*** sdake has joined #tripleo17:52
openstackgerritMerged openstack/tripleo-heat-templates: puppet: implement MongoDB on controller nodes  https://review.openstack.org/16547417:52
openstackgerritSteven Dake proposed stackforge/kolla: Make glance work reliably  https://review.openstack.org/17311317:56
openstackgerritSteven Dake proposed stackforge/kolla: CentOS 7.1 image doesn't work with libvirt  https://review.openstack.org/17344017:56
openstackgerritMerged stackforge/kolla: CentOS 7.1 image doesn't work with libvirt  https://review.openstack.org/17344018:01
openstackgerritSteven Dake proposed stackforge/kolla: Make glance work reliably  https://review.openstack.org/17311318:14
*** daneyon has joined #tripleo18:16
shardyslagle: https://review.openstack.org/#/c/173446/118:16
shardyshould fix your DELETE deployment issue18:16
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: puppet: Ceilometer use MongoDB datastore backend  https://review.openstack.org/16548018:16
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add support for Redis configuration  https://review.openstack.org/16753818:20
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Enable use of coordination_url in ceilometer  https://review.openstack.org/17199618:20
*** daneyon_ has joined #tripleo18:20
openstackgerritSteven Dake proposed stackforge/kolla: remove wait_for from glance-api  https://review.openstack.org/17344718:21
*** yamahata has quit IRC18:22
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: galera: Allow installer to setup galera cluster  https://review.openstack.org/16498318:22
*** daneyon has quit IRC18:23
*** sdake_ has joined #tripleo18:25
*** sdake has quit IRC18:29
*** Marga_ has quit IRC18:29
slagleshardy: cool, i'll give it a go18:31
*** trown|lunch is now known as trown18:32
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Do not downcase IP addresses  https://review.openstack.org/17345418:36
*** sdake has joined #tripleo18:39
openstackgerritSteven Dake proposed stackforge/kolla: Update developer documentation  https://review.openstack.org/17299818:41
*** sdake_ has quit IRC18:43
*** yamahata has joined #tripleo18:45
*** Marga_ has joined #tripleo18:50
*** rhallisey has quit IRC18:53
*** Goneri has joined #tripleo18:53
*** rhallisey has joined #tripleo18:54
*** pradeep has quit IRC18:57
*** rhallisey has quit IRC18:59
gfidentehas 3x +2 but was failing CI https://review.openstack.org/#/c/171840/ :)19:01
*** rhallisey has joined #tripleo19:01
*** rhallisey has quit IRC19:02
openstackgerritMerged openstack/tripleo-heat-templates: Make memcached bind on ctlplane instead of  https://review.openstack.org/17184019:03
jdobdidnt think you were still around, but i'm keeping that HA etherpad up to date19:05
gfidentejdob, nice :)19:06
*** bvandenh has joined #tripleo19:10
*** mmagr has joined #tripleo19:10
spredzyjdob, can I get your review in this one https://review.openstack.org/#/c/165480/19:10
spredzyplease. Just removed the conflict19:10
gfidentespredzy, !19:11
*** Goneri has quit IRC19:11
spredzyo/ still around :p19:12
*** pradeep has joined #tripleo19:12
*** jprovazn has quit IRC19:12
openstackgerritSteven Dake proposed stackforge/kolla: Add a conf-novanet to configure nova  https://review.openstack.org/17030619:15
*** akrivoka has quit IRC19:19
*** rhallisey has joined #tripleo19:19
tchaypogfidente: I’m curious19:19
tchaypogfidente: does the downcasing cause problems, or is it just silly?19:19
tchaypogfidente: but also - is it required or useful for ipv6 addresses?19:19
gfidentetchaypo, absolutely just me being silly :)19:20
gfidentetchaypo, I was trying to make the list propagate into .pp as an array preformatting it in the .yaml19:20
gfidenteI was able to, with str_replace19:20
gfidentebut I still had to use any2array in the .pp19:20
gfidenteso I abandoned that which seemed not worth19:20
gfidentebut noticed there were these downcase things :)19:21
gfidentetchaypo, btw, how are you doing?  :)19:21
tchaypoaside from being up at 5am for a meeting?19:21
spredzygfidente, it is totally worth it. Spent an hour trouble shooting an issue where the hostname from one side was xxx-Controller-xxx and from the fact it was xxx-controller-xxx19:21
*** mmagr has quit IRC19:21
gfidentespredzy, it is totally needed19:22
spredzyso w/o the downcase the equality test was failing19:22
gfidenteI remember we discussed that before19:22
spredzyand it never got into the if it was supposed to19:22
gfidentebut for hostnames, not ips19:22
tchaypospredzy: could you comment on the review?19:22
gfidentehostnames are based on the resource name "Controller"19:22
spredzytchaypo, sorry which review are yu guys talking about ? :)19:22
spredzyjust catched the question19:22
gfidentethat's why we need the downcase19:22
tchaypowait, that’s different19:23
spredzywell on ips its not necessary19:23
spredzyonly on hostname X_names stuff19:23
gfidentespredzy, so to avoid the downcase I was thinking to switch rabbit into ips as well19:24
gfidenteto avoid the Controller trap19:24
spredzygfidente, ack. My point was mainly be careful when using _names/hostname, and do not ofrget the downcase() function19:25
gfidentespredzy, exactly, I remember we hit this before19:25
spredzybut if it could be replaced by IPs sure.19:25
*** weshay_mtg has quit IRC19:29
jdobspredzy: looking19:34
slagleshardy: worked :)19:34
openstackgerritSteven Dake proposed stackforge/kolla: Add a conf-novanet to configure nova  https://review.openstack.org/17030619:35
openstackgerritSteven Dake proposed stackforge/kolla: Add a conf-novanet to configure nova  https://review.openstack.org/17030619:36
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Example of post-deploy hook doing rhel registration  https://review.openstack.org/17206519:36
*** sdake_ has joined #tripleo19:37
*** sdake has quit IRC19:41
*** sdake has joined #tripleo19:51
*** sdake_ has quit IRC19:54
openstackgerritMerged openstack-infra/tripleo-ci: Remove heat-templates cherrypick  https://review.openstack.org/17293720:03
openstackgerritMerged openstack-infra/tripleo-ci: Filter session-XXX.scope files from logs  https://review.openstack.org/16967020:04
*** sdake_ has joined #tripleo20:05
openstackgerritDaneyon Hansen proposed stackforge/kolla: Adds Neutron to Start/Stop Scripts  https://review.openstack.org/17349420:06
*** sdake has quit IRC20:07
openstackgerritDaneyon Hansen proposed stackforge/kolla: Adds Neutron to Start/Stop Scripts  https://review.openstack.org/17349420:08
*** gfidente has quit IRC20:13
openstackgerritThomas Bechtold proposed openstack/diskimage-builder: openSUSE update  https://review.openstack.org/17349820:15
*** lucas-dinner has quit IRC20:17
openstackgerritDaneyon Hansen proposed stackforge/kolla: Adds Neutron to Start/Stop Scripts  https://review.openstack.org/17349420:23
*** sdake has joined #tripleo20:23
*** sdake_ has quit IRC20:25
*** Marga_ has quit IRC20:28
openstackgerritRyan Hallisey proposed stackforge/kolla: Add cinder into the docker compose setup  https://review.openstack.org/17350720:31
*** lblanchard has quit IRC20:31
*** mmagr has joined #tripleo20:37
*** lifeless1 is now known as lifeless20:37
*** MasterPiece has joined #tripleo20:38
openstackgerritDaneyon Hansen proposed stackforge/kolla: Adds a Status Script  https://review.openstack.org/17351120:43
*** mmagr has quit IRC20:43
*** Goneri has joined #tripleo20:48
*** jsidhu has joined #tripleo20:49
openstackgerritMerged openstack/diskimage-builder: Do not export REG_HALT_UNREGISTER between hook scripts  https://review.openstack.org/16223920:50
*** jtomasek has quit IRC20:51
*** untriaged-bot has joined #tripleo21:00
untriaged-botUntriaged bugs so far:21:00
openstackLaunchpad bug 1444068 in tripleo "Rabbitmq_plugin[rabbitmq_management]: Could not evaluate: execution expired"Puppet deployment fails with "" [Undecided,Triaged]21:00
*** untriaged-bot has quit IRC21:00
*** pblaho has quit IRC21:02
openstackgerritDaneyon Hansen proposed stackforge/kolla: Adds a Remove Script  https://review.openstack.org/17352021:05
*** trown is now known as trown|outttypeww21:05
shardyslagle: thanks for the confirmation, chasing some reviews so we can hopefully land the fix soon21:09
*** dprince has quit IRC21:14
*** dsneddon has joined #tripleo21:18
*** cwolferh has joined #tripleo21:18
*** rhallisey has quit IRC21:20
*** daneyon has joined #tripleo21:30
*** daneyon_ has quit IRC21:32
*** shardy is now known as shardy_z21:33
openstackgerritMerged stackforge/kolla: Adds Neutron to Start/Stop Scripts  https://review.openstack.org/17349421:35
*** Goneri has quit IRC21:35
*** weshay has joined #tripleo21:35
*** daneyon has quit IRC21:44
*** tzumainn has quit IRC21:47
*** bvandenh has quit IRC21:52
*** julim has quit IRC21:52
*** Marga_ has joined #tripleo22:21
openstackgerritwes hayutin proposed openstack/diskimage-builder: update the dib centos7 baseurl to use a mirrorlist  https://review.openstack.org/17261522:28
*** dsneddon has quit IRC22:34
*** dsneddon has joined #tripleo22:38
*** rhallisey has joined #tripleo22:39
*** Marga_ has quit IRC22:49
*** jsidhu has quit IRC22:52
*** jsidhu has joined #tripleo22:56
openstackgerritSteven Dake proposed stackforge/kolla: genenv generated invalid variable  https://review.openstack.org/17359022:56
*** yuanying has joined #tripleo22:57
*** jsidhu has quit IRC22:58
*** Marga_ has joined #tripleo23:07
*** chlong has joined #tripleo23:10
*** regebro has quit IRC23:38
*** jerryz has joined #tripleo23:42
*** chlong has quit IRC23:43
*** rlandy has quit IRC23:44
*** chlong has joined #tripleo23:45
*** alop has quit IRC23:53

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