Thursday, 2015-05-14

*** Marga_ has quit IRC00:25
*** cwolferh has quit IRC00:59
*** Marga_ has joined #tripleo01:58
*** tzumainn has quit IRC01:59
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Update neutron local_ip to use the tenant network  https://review.openstack.org/17871602:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a network ports IP mapping resource  https://review.openstack.org/17871402:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to block storage roles  https://review.openstack.org/18082402:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to swift roles  https://review.openstack.org/18082302:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to ceph roles  https://review.openstack.org/18082202:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to controller roles  https://review.openstack.org/17784602:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to compute roles  https://review.openstack.org/18082102:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a ports (ip address) abstraction layer  https://review.openstack.org/17784502:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated net parameters to net-config stacks  https://review.openstack.org/18082002:07
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Wire in optional network creation for overcloud  https://review.openstack.org/17784402:07
*** sdake has quit IRC02:09
*** sdake has joined #tripleo02:12
*** spzala has joined #tripleo02:16
*** eghobo has quit IRC02:16
*** yamahata has quit IRC02:16
*** Marga_ has quit IRC02:22
*** Marga_ has joined #tripleo02:23
*** Marga_ has quit IRC02:46
*** Marga_ has joined #tripleo02:54
*** untriaged-bot has joined #tripleo03:00
untriaged-botUntriaged bugs so far:03:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985203:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985403:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480303:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275203:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480203:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/145240003:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,Fix committed] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)03:00
*** untriaged-bot has quit IRC03:00
openstackLaunchpad bug 1454803 in tripleo "puppet: Neutron is not configured with L2 population" [Undecided,New]03:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]03:00
openstackLaunchpad bug 1454802 in tripleo "puppet: Neutron does not use Nova notifications" [Undecided,New]03:00
openstackLaunchpad bug 1452400 in diskimage-builder "debian-minimal builds still busted, now on locale issues" [Undecided,New]03:00
*** ccrouch has left #tripleo03:02
*** panda has quit IRC03:13
*** panda has joined #tripleo03:14
*** Marga_ has quit IRC03:26
*** Marga_ has joined #tripleo03:38
*** morazi has quit IRC03:43
*** yamahata has joined #tripleo03:46
*** eghobo has joined #tripleo04:04
*** eghobo_ has joined #tripleo04:07
*** spzala has quit IRC04:10
*** daneyon has quit IRC04:10
*** eghobo has quit IRC04:10
*** eghobo has joined #tripleo04:39
*** eghobo_ has quit IRC04:40
*** ukalifon has quit IRC05:12
*** toabctl has quit IRC05:21
*** sdake_ has joined #tripleo05:22
*** sdake has quit IRC05:26
*** sdake_ is now known as sdake05:38
*** yog_ has joined #tripleo05:40
*** dshulyak has joined #tripleo05:55
*** ishant has joined #tripleo06:03
*** masco has joined #tripleo06:05
*** gfidente has joined #tripleo06:06
*** adrianopetrich has joined #tripleo06:08
*** Marga_ has quit IRC06:10
openstackgerritMerged openstack-infra/tripleo-ci: Add a ceph job  https://review.openstack.org/17629206:15
*** ukalifon1 has joined #tripleo06:18
*** jcoufal has joined #tripleo06:24
gfidentejcoufal, I tried, but was unable to reproduce the issue with os-apply-config, have you got any luck with that?06:25
*** GheRiver1 is now known as GheRivero06:25
*** dasm|afk is now known as dasm06:26
*** shardy has joined #tripleo06:29
jcoufalgfidente: it was already fixed06:30
jcoufalthey were pulling juno version of 0s-*06:30
jcoufalwhich was... broken06:30
jcoufalgfidente: you have rights to push to isntack-undercloud?06:30
*** sdake has quit IRC06:31
gfidentejcoufal, don't we use gerrithub for that?06:34
jcoufalgfidente: we do and I am asking if you have the rights06:34
gfidenteno I can't approve changes there06:34
*** jprovazn has joined #tripleo06:37
shardymorning all06:39
*** eghobo has quit IRC06:40
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt Galera ocf resource to puppet-pacemaker last update  https://review.openstack.org/18294706:43
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt MongoDB systemd resource to puppet-pacemaker last update  https://review.openstack.org/18294806:50
*** ishant has quit IRC07:06
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt HAProxy systemd resource to puppet-pacemaker last update  https://review.openstack.org/18295507:10
spredzymorning everybody07:10
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt Galera ocf resource to puppet-pacemaker last update  https://review.openstack.org/18294707:14
*** dshulyak has quit IRC07:20
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt RabbitMQ ocf resource to puppet-pacemaker last update  https://review.openstack.org/18295807:21
dasmo/07:25
*** yamahata has quit IRC07:26
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt Galera ocf resource to puppet-pacemaker last update  https://review.openstack.org/18294707:30
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt MongoDB systemd resource to puppet-pacemaker last update  https://review.openstack.org/18294807:30
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Adapt RabbitMQ ocf resource to puppet-pacemaker last update  https://review.openstack.org/18295807:30
*** mmagr has joined #tripleo07:31
*** jistr has joined #tripleo07:34
gfidenteukalifon1, you around?07:34
ukalifon1gfidente: yea, hi07:35
gfidenteso regarding the pacemaker stuff, I think we can discuss it here so we get more people involved07:35
ukalifon1sure07:35
gfidentewith that card, we were supposed to cover setup of the pacemaker cluster itself plus the vips and haproxy resources07:36
gfidenteso you should see all those three managed by pacemaker07:36
gfidentelike it was for rabbit07:36
gfidenteplus you should have a working pacemaker cluster07:36
gfidentenow to some extent, to test this, you just use pcs status07:37
gfidentebut there are at least two more things we might want to test07:37
gfidente1. it actually survive failures (and we can't test this yet)07:37
gfidente2. it matches the configuration described here https://github.com/beekhof/osp-ha-deploy/blob/master/ha-openstack.md07:38
gfidenteand again, 2 only applies to those few services which already are managed by pacemaker07:39
gfidenteso in the plan you can add the two vips and haproxy amongst the resources to be checked07:40
ukalifon1gfidente: we can't test it yet because we can't increase the number of controllers and nodes? And when will that be ready for testing?07:41
*** aufi has joined #tripleo07:42
gfidenteexactly07:42
gfidenteso that works upstream now07:42
gfidentewe'll need to cherry pick some changes into midstream for testing07:43
gfidenteso that one can test the cluster and services managed by it survive failure07:43
gfidentespredzy, I think it would be better to make the puppet-pacemaker changes a single one07:45
gfidentespredzy, easier to track and to merge07:45
*** jtomasek has joined #tripleo07:45
*** ifarkas has joined #tripleo07:45
spredzygfidente, ok will do that :)07:46
gfidentespredzy, so we can also make sure the job goes back into green state from a single CI change and just merge it07:46
spredzygfidente, yeah that is kinda of why I made them depend on each other to ensure that if top one was green prev were ok. But I am ok with making a single commit also07:47
gfidentespredzy, galera is tip right? https://review.openstack.org/#/c/182947/07:48
spredzygfidente, yes, but the order doesn't really matter anyway07:48
spredzyI am pushing the singlle commit07:48
gfidentesec07:48
gfidenteit does for CI07:48
gfidenteonly the tip will have green, which is fine07:49
spredzyyes, separately they will all be red07:49
spredzysince they depend on each other all together they'll be green, hence the chaining07:49
gfidentewell tip should be green07:49
gfidentebut the dependencies should be merged despite failures07:50
gfidenteI think we can do that, so you don't have to redo work07:50
spredzyonly non-voting job will be red07:50
spredzyso yes they should be mergeable07:50
gfidenteyeah07:50
gfidenteso don't do extra work now and let's wait for TIP results first07:51
spredzygfidente, ack :)07:51
spredzygfidente, but as I say in the comment TIP job won't be green until https://github.com/redhat-openstack/puppet-pacemaker/pull/48  is merged upstream07:53
jcoufalgfidente: http://pastebin.test.redhat.com/283092 overcloud deployment is failing, looks like problem with starting mongo_db07:53
jcoufalgfidente: any idea what can be wrong?07:53
spredzyso I rely on you pinging jayg|g0n3 while I am away :p07:53
spredzyjcoufal, This is due to the mongodb packaging08:00
*** dshulyak has joined #tripleo08:00
spredzyone relies on /etc/mongod.conf and the other on /etc/mongodb.conf (when I say one depending if the package comes from RDO or EPEL) upstream uses one, the version you are using uses another08:00
spredzywhen my overcloud boot I'll be able to tell you the exact line you need to add in your hiera file08:00
jcoufalspredzy: thanks08:01
*** links has joined #tripleo08:02
spredzyjcoufal, upstream uses mongodb-server-2.4.13-1.fc21.x86_64 and the configuration file is /etc/mongodb.conf08:02
spredzyso your is most certainly /etc/mongod.conf08:02
jistr /o\08:03
*** athomas has joined #tripleo08:04
spredzyjcoufal, you need to add "mongodb::server::config: /etc/mongod.conf" in the hiera file08:04
spredzycontroller.yaml08:04
*** derekh has joined #tripleo08:05
jistrexcellent08:11
jistrso it's fedora vs. EPEL using different conventions in the packages08:11
jistron F21 the package comes from @updates08:11
spredzyjistr, yes, packstack and spinal stack had been bitten by it08:12
spredzyand not so long ago marios and jdob also08:12
jcoufaland now everybody08:12
*** ishant has joined #tripleo08:13
spredzythere is a pull request upstream (puppetlabs-mongodb) https://github.com/puppetlabs/puppetlabs-mongodb/pull/19308:15
spredzyto solve that issue, just didn't get merged yet08:15
jistrspredzy: thx that's vital info. I was just looking at that file in master but not at the pull request.08:17
*** lucasagomes has joined #tripleo08:19
spredzyjistr, np :)08:21
spredzyjistr, gfidente so with https://review.openstack.org/#/c/182947/ , the dependencies and the PR in puppet-pacemaker I get a succesful pacemaker cluster with the latest commit of puppet-pacemaker. I noticed a small issue but that is unrelated to the change itself is that somtimes if fails saying Unable to get CIB I think Giulio already faced that. I had to add a sleep 5 to test the patch was working just fine08:22
spredzyI am gonna be off for the rest of the day, so talk to you later gusy08:22
jistrspredzy: ack, thanks08:23
*** spredzy is now known as spredzy|afk08:23
jistrjcoufal: what's your `rpm -q mongodb-server`?08:26
*** yog_ has quit IRC08:27
jcoufaljistr: mongodb-server-2.6.9-1.el7.x86_6408:27
jistrmmagr: looks like this might be a solution then? https://github.com/puppetlabs/puppetlabs-mongodb/pull/19308:31
jistrmmagr: not sure if we can/should get it somehow to OPM before it's merged upstream?08:31
mmagrjistr, yup, seem like it is ... if it is breaking our stuff we should get it to OPM08:32
gfidentejistr, spredzy|afk I remember we discussed the mongod issue a while back08:33
gfidentejistr, spredzy|afk the version in epel is newer08:34
jistrgfidente: does the PR above seem like a possible fix to you as well?08:35
jistrjcoufal: wanna share the machine with me and test the fix?08:35
jcoufaljistr: send me your public key08:35
jistrjcoufal: haha i'll just post this again :P https://github.com/jistr.keys08:36
jcoufaljistr: sorry, I will never get it into my bookmarks :D08:36
gfidentejcoufal, I think your mongod comes from epel08:36
jistryeah it does08:36
jistrand it's 2.6.908:36
gfidentejistr, spredzy|afk it seems to me, more than distro based, this depends on the version of mongodb08:36
gfidenteso except for the title, the PR looks fine to me08:37
*** mcornea has joined #tripleo08:40
*** yog_ has joined #tripleo08:43
gfidentejistr, spredzy|afk checking builds of mongodb-server https://apps.fedoraproject.org/packages/mongodb-server/08:46
gfidentelooks like newer 2.6.9 in fedora will use mongod.conf as well08:47
*** pdb has quit IRC08:48
*** pdb has joined #tripleo08:48
*** yog_ has quit IRC08:53
*** untriaged-bot has joined #tripleo09:00
untriaged-botUntriaged bugs so far:09:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275209:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]09:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985209:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480309:00
openstackLaunchpad bug 1454803 in tripleo "puppet: Neutron is not configured with L2 population" [Undecided,New]09:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985409:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,Fix committed] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480209:00
openstackLaunchpad bug 1454802 in tripleo "puppet: Neutron does not use Nova notifications" [Undecided,New]09:00
*** untriaged-bot has quit IRC09:00
*** dtantsur is now known as dtantsur|bbl09:04
*** mkerrin has joined #tripleo09:05
*** rlandy has joined #tripleo09:07
*** mmagr is now known as mmagr|afk09:08
*** bnemec has quit IRC09:13
*** bnemec has joined #tripleo09:13
*** slagle has quit IRC09:14
*** pelix has joined #tripleo09:17
*** slagle has joined #tripleo09:21
*** mmagr|afk is now known as mmagr09:22
*** trown|outttypeww is now known as trown09:27
jprovaznWhat is the right way to add a new param into THT so this param is visible after THT is processed by tsukar? If I add a new parameter "ComputeRemovalPolicies" into overcloud-without-mergepy.yaml and then I run "instack-deploy-overcloud --tuskar" which adds ruskar roles and creates plan, then this parameter is missing in the templates generated by tuskar?09:35
jprovaznhttp://paste.openstack.org/show/222633/09:35
jprovaznIt almost seem that some other file than /usr/share/openstack-tripleo-heat-templates/overcloud-without-mergepy.yaml is used for creating tuskar roles/plan09:36
jprovaznmarios: pblaho: d0ugal: ^ any idea?09:37
gfidente<marios> roles=`tuskar role-list | grep OpenStack | awk '{print $2}'`09:37
gfidente<marios> tuskar plan-delete overcloud09:37
gfidente<marios> tuskar-delete-roles --config-file /etc/tuskar/tuskar.conf --uuids $roles09:37
gfidente<marios> curl http://paste.openstack.org/raw/219769/ | bash -x09:37
gfidenteI think we should stick in the TOPIC :)09:37
d0ugaljprovazn: Not sure, I'd try somebody working on the templates and/or tuskar api09:38
gfidentejistr, why unsure about the PR pupeptlabs-mongo?09:39
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add metering agent as neutron-server plugin  https://review.openstack.org/17937009:40
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add neutron-lbaas as neutron-server plugin & neutron-lbaas-agent  https://review.openstack.org/18053509:40
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Adds neutron-server and agents as pacemaker resources  https://review.openstack.org/18101509:40
jistrgfidente: 1) it only does the fix if $manage_package_repo is true, and i think for us it's false 2) it also does the fix wrong so it doesn't cover all cases -- it shouldn't be accepted upstream09:41
mariosjprovazn: that is definitely the right templates directory fwiw09:41
mariosjprovazn: (sorry ettomany screens)09:41
mariosjprovazn: at leaset, that is where i change templates. but gfidente is right, you need to reload roles09:41
gfidentejistr, ack on manage_package_repo09:42
gfidentenot sure about it's false for us?09:42
jistrgfidente: if it's true, it enables mongodb's own repos09:42
gfidenteah you mean the manage_repo is always false09:43
jistrhttps://github.com/puppetlabs/puppetlabs-mongodb/blob/master/manifests/repo.pp09:43
gfidenteI thought the comparison09:43
gfidentesure :)09:43
jistryeah09:43
jprovaznmarios: gfidente: yea, I deleted all roles before deployment, but if I dig deeper and check "stored_file" table in DB, I can see that overcloud plan references:09:44
jprovazn11314424-964b-4130-86d4-8062f8fbd988    {"environment_file_uuid": "ec598894-916d-4daf-a62f-9bb1fb4f6300", "master_template_uuid": "d7948e2e-3e58-4b19-be45-001ba5eb7f25"}       deployment_plan overcloud       NULL    2015-05-14 09:18:10     NULL09:44
jprovaznbut "d7948e2e-3e58-4b19-be45-001ba5eb7f25" template does *not* contain this param I added, bu I can see this param in DB for a different template which doesn't seem to be referenced :(09:44
*** derekh has quit IRC09:55
*** athomas has quit IRC09:55
jistrgfidente: also the $version variable probably needs to be set manually for the other bits to take the correct effect, so that's not really a solution... that would be the same as setting the config path as we do now09:59
jistrgfidente: i'm testing an alternate fix now if it doesn't break F21, then i'll test it if it fixes EPEL, and if both are OK i'll submit the change to puppetlabs09:59
*** regebro has joined #tripleo10:00
*** weshay has quit IRC10:07
*** weshay has joined #tripleo10:09
*** athomas has joined #tripleo10:24
*** akrivoka has joined #tripleo10:29
*** sdake has joined #tripleo10:35
*** Hazelesque has quit IRC10:48
*** Hazelesque has joined #tripleo10:50
*** sdake_ has joined #tripleo10:59
*** sdake has quit IRC11:02
*** dtantsur|bbl is now known as dtantsur11:03
*** weshay has quit IRC11:06
*** lucasagomes is now known as lucas-hungry11:10
*** derekh has joined #tripleo11:11
*** pelix has quit IRC11:13
*** panda has quit IRC11:13
*** pelix has joined #tripleo11:14
*** panda has joined #tripleo11:14
*** Goneri has joined #tripleo11:16
*** lazy_prince has joined #tripleo11:18
*** mmagr is now known as mmagr|afk11:20
*** rhallisey has joined #tripleo11:21
*** dshulyak has quit IRC11:21
*** mmagr|afk is now known as mmagr11:41
*** al has quit IRC11:44
*** yog_ has joined #tripleo11:44
*** weshay has joined #tripleo11:44
*** al has joined #tripleo11:44
*** chlong has joined #tripleo11:48
*** Marga_ has joined #tripleo11:49
*** akrivoka has quit IRC11:50
*** lucas-hungry is now known as lucasagomes11:50
*** thrash|g0ne is now known as thrash11:50
*** rhallisey has quit IRC11:52
*** rhallisey has joined #tripleo11:54
mariosjistr: finally passing builds, can i please get another pass for https://review.openstack.org/#/c/181015/ - if u are busy whenever/tomorrow thanks very much!11:59
*** rbrady has quit IRC12:06
*** tzumainn has joined #tripleo12:07
*** dprince has joined #tripleo12:16
*** ishant has quit IRC12:16
*** jayg|g0n3 is now known as jayg12:21
jprovaznjdob: morning, I have a tuskar question12:25
*** morazi has joined #tripleo12:29
*** masco has quit IRC12:32
*** Marga_ has quit IRC12:33
*** Marga_ has joined #tripleo12:36
EmilienMjtomasek: I think we use this chan for tripleo elements12:42
*** sdake_ has quit IRC12:43
jtomasekEmilienM: ok, instack-undercloud recently switched to using packages install instead of source install in tripleo-puppet-elements. This means, that it now uses openstack-puppet-modules package12:43
jtomasekEmilienM: I'd need to get horizon module up to date there, so I get new build here: http://trunk.rdoproject.org/centos70/report.html12:44
*** sdake has joined #tripleo12:44
EmilienMjtomasek: ok so you need an update of puppet-horizon in OPM, right?12:45
jtomasekEmilienM: yes12:45
EmilienMmmagr: ^12:45
*** Marga_ has quit IRC12:46
mmagrjtomasek, you need latest master?12:46
jtomasekmmagr: yes12:47
*** Marga_ has joined #tripleo12:47
openstackgerritJan Provaznik proposed openstack/tripleo-image-elements: Include yum update extra files  https://review.openstack.org/18219812:54
*** radez_g0n3 is now known as radez12:59
*** Marga_ has quit IRC13:00
*** regebro has quit IRC13:00
*** regebro has joined #tripleo13:02
*** Goneri has quit IRC13:03
*** derekh_ has joined #tripleo13:07
jaygjistr: was spredzy|afk 's question for me earlier answered?  lot of backchat, hard to see13:07
jaygpacemaker PR is merged, btw13:07
jaygso expect everything to be broken  :)13:07
*** derekh has quit IRC13:10
jistrjayg: i don't see the question (lot of backchat :) ) but i'm testing spredzy|afk 's fixes, so far no success13:11
EmilienMmmagr: thx for update BTW :)13:11
jistri think we might have to do some updates and also squash the fixes to one commit "deal with puppet-pacemaker update" to have saner CI testing. All the changes are really small so squashing should be ok13:11
jistrhah failed on this  Error: pcs create failed: Error: unable to get cib13:14
jistrthis has to be either race or an ordering issue13:14
mmagrEmilienM, np13:16
jaygjistr: it may be related to the sleep spredzy|afk mentioned, I know we _have_ to use those in previous versions13:17
jaygand they are supported, as are tries in the newest puppet-pacemaker13:18
jistrjayg: ack, but the default tries is 1, right?13:18
jaygcorrect13:18
jaygand sleep 0 I think13:18
*** masco has joined #tripleo13:19
jistrjayg: ack, i'm now looking if we can set it to something else on all pcmk_resource without replicating that setting on all places in the manifest13:19
jistri think puppet can do something like that, i just don't recall the syntax13:19
jaygcollector would be one option13:20
jaygor, it could go in hiera file to be picked up for all resources perhaps?13:21
*** jprovazn has quit IRC13:21
jistrjayg: hmm i thought hiera only works for classes13:25
jistrjayg: i found resource defaults https://docs.puppetlabs.com/puppet/latest/reference/lang_defaults.html13:26
jistrlooks like what we want13:26
jaygjistr: that could work, but it looks like you would have to do it per type still, no?  collector would get you all matches, and can be scoped I think13:30
jistrjayg: i think collectors are for dependencies rather https://docs.puppetlabs.com/puppet/3.8/reference/lang_collectors.html13:34
jistrjayg: but maybe i just don't know what you mean, could you paste an example?13:34
jaygsure, sec, we do this in quickstack, lemme find one13:35
jistrmy idea was this http://fpaste.org/221886/31610522/13:35
jaygjistr: if it works, great, I was thinking similar but with collectors, we do it with cron here (think we may have borrowed this from spinal stack actually) https://github.com/redhat-openstack/astapor/blob/master/puppet/modules/quickstack/manifests/ceilometer/control.pp#L10713:37
EmilienMjayg: thanks for credits :P we don't use pacemaker anymore in SpS :P13:38
jaygEmilienM: yeah, this was just basic ceilo config outside pacemaker13:39
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Makes sure galera can handle the connections  https://review.openstack.org/18304413:39
jistrjayg: ack. If the resource defaults don't work, i'll try the collector. Thanks :)13:40
gfidentemarios, think we need those in the _pacemaker version as well13:40
openstackgerritGiulio Fidente proposed stackforge/puppet-tripleo: Increase global maxconn to 10000 and remove per-instance limit  https://review.openstack.org/18304613:41
jaygjistr: sure thing, I am sure one of the two will do it13:41
jistrjayg: actually i'm afraid that this will override the default https://github.com/redhat-openstack/puppet-pacemaker/blob/master/manifests/resource/ocf.pp#L53-L5413:41
mariosgfidente: ack looking13:41
jistrjayg: now i see what you meant by doing it per type13:41
jaygright, that is what I was worried about13:42
EmilienMgfidente: looking at https://review.openstack.org/#/c/183046/13:42
EmilienMgfidente: the patch is good, I wonder if we should be able to change it13:42
gfidenteEmilienM, we should13:42
EmilienMa "no" works, just wondering13:42
gfidentewe wanted to split change into a smaller one which could cherry-pick13:42
jistrjayg: ok i go for the collector first. thx ;)13:42
gfidenteand later the good one which makes it configurable13:42
EmilienMgfidente: yeah13:43
gfidenteI can make it configurable later today though, maybe we can do that from start13:43
EmilienMgfidente: you know what, we go ahead with that patch now13:43
EmilienMgfidente: +213:43
EmilienMgfidente: we will improve it later13:43
gfidenteI like the refine later process13:43
gfidente:)13:44
gfidenteEmilienM, or maybe, we make it configurable in the module now13:44
gfidentebut expose it in template later13:44
jaygjistr: if you think of some way that would be easier if we structures puppet-pacemaker params differently (if needed), I don't think either of us would be resistant to improvements13:44
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Makes sure galera can handle the connections  https://review.openstack.org/18304413:47
mariosgfidente: updated ^^13:47
*** jehb has joined #tripleo13:49
*** bvandenh has joined #tripleo13:50
hewbroccanice work tracking that down guys13:58
*** jprovazn has joined #tripleo14:01
*** BadCub-Hiding is now known as BadCub14:02
*** bvandenh has quit IRC14:03
jistrjayg: seems like the collector didn't work, trying the resource defaults per-type now14:04
jayg:(14:05
jistrhuh it might also be a dependency issue14:08
* jistr already deleted the overcloud though :/14:08
*** rhallisey has quit IRC14:08
jistrjayg: https://github.com/openstack/tripleo-heat-templates/blob/2c6ea075cf0eeeea5cc7cf6ea9b654c325096d32/puppet/manifests/overcloud_controller_pacemaker.pp#L7114:08
jistrgonna change the RHS to Pcmk_resource and we'll see14:09
jistrand i'll keep the collector approach for now still, since it might not have been the problem spot14:09
*** cody-somerville has quit IRC14:10
jaygok14:12
jprovaznjdob, Hi14:13
jdobjprovazn: ahoy14:14
jprovaznjdob, I need your tuskar expertise...14:14
hewbrocca"Let me through, I'm a Tuskar expert!"14:14
jprovaznI'd like to add a new property to compute (and other) resource groups, more specifically14:15
jprovaznI'd like to add "removal_policies" bellow https://github.com/rdo-management/tripleo-heat-templates/blob/mgt-master/overcloud-without-mergepy.yaml#L64514:15
jprovaznjdob, after some experiments and digging, it seems that tuskar parses and regenerates master template and ignores newely added property14:16
* jprovazn digs for the right place in tuskar14:16
jdobright14:17
jdobwthats added at the properties level, not the resource_def?14:17
jdobso same level as count14:18
jprovaznjdob, yes, same level, removal_policies - http://docs.openstack.org/hot-reference/content/OS__Heat__ResourceGroup.html14:18
jdobhrm, thats gonna be a problem14:19
jdobtuskar is creating those resources on its own14:19
jdoband manually adding count when it wraps the resource def in an auto scaling group14:19
jdobthere's currently no way to introduce new parameters on the group itself14:19
jprovaznepic14:19
jprovazncouldn't you just change your answer to "hey, that is piece of cake!"??14:20
*** masco has quit IRC14:20
jprovaznjdob, to give you more context...14:20
jdobhaha, its actually a giant pain in the ass14:20
jprovazn:)14:21
jprovaznI *think* I need to use this removal_policies param if I want to remove a specific node from overcloud14:21
jdobwould we ever want to change the value of it?14:22
jdobor can tuskar just always introduce it14:22
jdobi mean, hell, tuskar could always introduce it anyway, with a sensible default14:22
jdobat the same time it adds the count parameter14:22
jprovaznjdob, default should be empty list, but when removing a node, I want to call stack-update where removal_policies refers to nodes being removed14:22
jdobit also depends on when you need it; in liberty we need to rethink a lot of tuskar given how the THT templates are evolving14:22
jdobthat shouldn't be too bad to get tuskar to add14:23
jdobits already adding count, creating a parameter for it, and likely defaulting that14:23
jdobso it should be able to follow the same model14:23
hewbroccajdob: where do you see it going?14:23
jdobin the tuskar code itself14:23
hewbroccameaning, what direction14:23
jdobwait14:24
*** rhallisey has joined #tripleo14:24
jprovaznjdob, yes, so if tuskar can always set this property to [] and allow me to pass custom list by param, it would be exactly what I need14:24
hewbroccado you see tuskar going14:24
jdobwhat is "it" in this context14:24
jdoboh, tuskar?14:24
hewbroccasorry, that was unclear even for me14:24
jdobhewbrocca: pretty big ass redesign, to be honest14:24
*** rhallisey has quit IRC14:24
jdobjprovazn: that should be a fairly unoffensive change14:24
hewbroccaThe main use case has changed a lot given how t-h-t has gone14:24
jdobright, the gist of what the Background section in my wall of text email was saying14:24
jprovaznjdob, this matches my hope of "hey, that is piece of cake!" :)14:25
jdobya, now i'm changing my answer to less ass and more cake14:25
jdob...14:25
jdobnot my best choice of words there14:25
hewbroccathat's a good rule for life14:25
jprovaznhehe14:25
*** mmagr is now known as mmagr|afk14:25
jdobits times like this I realize why my old teams would keep a running list of jdob-isms14:25
jdobwhich included such nuggets of wisdom as "I would guess that I don't know"14:26
jdobbut I digress14:26
jdobhewbrocca: now i see tuskar more as resource regsitry management14:26
jdobwhich would make it easier to plug in stuff14:26
hewbrocca*nod*14:26
jprovaznjdob, how is your todo list doing? is it realistic you could put together a patch for this or is it something I should look at?14:26
*** mkerrin has quit IRC14:27
*** rhallisey has joined #tripleo14:27
jdobjprovazn: depends on when you need it. i'm dealing with an auth bug in tuskar now and have summit next week, so i *may* get to it this afternoon/tomorrow, but if not, it likely wont be for another week14:27
*** mmagr|afk is now known as mmagr14:28
jprovaznjdob, I need this soon, tomorrow ideally :), how about this: if you manage to get to it till your EOD today, it would be cool, otherwise I can try tomorrow myself?14:29
*** shakamunyi has joined #tripleo14:29
jprovazn(I don't mean to  have it merged tomorrow, but at least working on my local setup)14:30
jdobthat sounds good. i'll send you an email before I leave today that will either be a review or some pointers on where to look in the code14:30
jprovaznjdob, excellent, thanks!14:30
jdobnp :)14:31
gfidentemarios, shall we add the settings for the pacemaker ragent in same patch?14:31
*** aufi has quit IRC14:37
mariosomnomnom late lunch is best lunch14:39
*** mordred has quit IRC14:40
marios'now thats the best damned cracker i ever ate!'14:40
*** mordred has joined #tripleo14:42
*** barra204 has joined #tripleo14:43
gfidenteamahaha14:47
*** shardy_ has joined #tripleo14:48
*** mmagr has quit IRC14:49
*** shardy has quit IRC14:50
*** shardy_ has quit IRC14:54
*** shardy has joined #tripleo14:54
*** cody-somerville has joined #tripleo14:55
*** yamahata has joined #tripleo14:55
gfidenteEmilienM, going back to https://review.openstack.org/#/c/183046/14:55
*** alop has joined #tripleo14:56
gfidentewill make the global param configurable,14:56
*** MasterPiece has joined #tripleo14:58
shardydprince, slagle: I've been thinking about how to reduce the delay between SoftwareDeployments due to os-c-c polling_interval14:59
shardyI'm considering wiring in logic which kicks os-collect-config to poll every time we send a signal back to heat15:00
shardywould you be cool with that?15:00
*** untriaged-bot has joined #tripleo15:00
untriaged-botUntriaged bugs so far:15:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275215:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]15:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985215:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480315:00
openstackLaunchpad bug 1454803 in tripleo "puppet: Neutron is not configured with L2 population" [Undecided,New]15:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985415:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,Fix committed] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480215:00
openstackLaunchpad bug 1454802 in tripleo "puppet: Neutron does not use Nova notifications" [Undecided,New]15:00
*** untriaged-bot has quit IRC15:00
shardyI'm thinking we add something where we can trigger a couple of quick polls which should catch the next Deployment very fast after we finish and signal back to heat, either via o-r-c or heat-config-notify15:01
slagleshardy: so occ would tell itself it to repoll if it signals heat?15:01
shardyslagle: occ doesn't signal heat, either an o-r-c script or heat-config-notify does15:01
shardyso I was thinking maybe wire in a signal handler which can be triggered to tell occ to do some additional polls15:01
shardyor have a timestamp of the last signal it can poll on the filesystem15:02
shardy(or in /tmp if thats a ramfs)15:02
shardyeither that or we just reduce the polling_interval, but this would put less load on heat15:04
slagleit sounds fine to me15:04
*** Marga_ has joined #tripleo15:04
slagleocc re-exec's the command (orc) on a metadata change, i guess that would be ok15:05
shardyslagle: Ok, cool - looking at the default poll interval and number of deployments it might win us quite a bit of time15:05
shardyAFAICS the default poll interval is 30s15:05
slagleocc has a signal handler already for reexec, is that what you're thinking about tieing into?15:07
shardyslagle: aha, I acutally didn't know about that :)15:07
* shardy was still at the "thinking is this a good idea stage" ;)15:07
* shardy looks at occ code15:08
openstackgerritGiulio Fidente proposed stackforge/puppet-tripleo: Increase global maxconn to 10000 and remove per-instance limit  https://review.openstack.org/18304615:09
slaglei'm not sure if you'll be able to signal that from something executed by orc, maybe so15:09
shardyslagle: thanks, that looks like it might work15:10
shardyI was thinking more like a SIGIO which defeats the poll time, but same basic result I think15:10
*** yog_ has quit IRC15:10
shardyslagle: I was thinking something like a killall here:15:10
shardyhttps://github.com/openstack/tripleo-image-elements/blob/master/elements/os-refresh-config/os-refresh-config/post-configure.d/99-refresh-completed15:10
shardywith the appropriate signal number set15:11
shardyand the equivalent in the heat-config-notify code15:11
shardyor a little helper script which looks up the occ pid and signals it which both can use15:11
slagleyea, i think looking up the occ pid would work15:12
shardyslagle: cool, thanks for going through it15:12
slaglesounds like a good plan to me anyway15:12
shardyjust wanted to sanity check before I spent too much time on it :)15:12
*** rhallisey has quit IRC15:12
gfidenteshardy, regarding https://review.openstack.org/#/c/181271/1 so you prefer to move those into t-h-t? I remember dprince being unsure as well, I wanted to use those (adding ha) for the CI jobs instead of pasting inline the params15:17
gfidente*pasting inline the custom-env file15:17
dprincegfidente: lets move them to t-h-t. We'll need to be careful with naming... because some of these files may be appropriate for CI/dev environments only (i.e. not production because of hard coded IDs)15:20
gfidentedprince, I see what you mean, ending up with same ceph key for all non-demo ceph deployments15:21
gfidente:P15:21
shardygfidente: Yeah, I've not got a strong opinion, but I was thinking it'd be best if the environment examples all lived in t-h-t15:23
shardythen you could have e.g a directory with demo/enable_ceph.yaml or something, and many others, providing an easy way to glue together the deployment you want from environment fragments15:24
shardyobviously we'll need to find a way to document loudly when there are hard-coded things like in that patch15:24
slaglefwiw, i like them living in tht as well15:24
*** links has quit IRC15:24
slaglethat's what i did with https://review.openstack.org/#/c/172065/15:24
shardyOr make them deliberately invalid so folks have to update them15:24
slagleit feels natural to provide the example environment alongside the example resource15:25
shardye.g have the parameter keys but no values in t-h-t with a comment saying "fill these in"15:25
shardyslagle: +115:25
gfidenteso we put in extraconfig15:26
shardywell extraconfig was intended for extra deployer stuff, not really configuring the features of the deployment15:27
shardyor at least that was my thinking15:27
shardymaybe have a samples/environment/enable_ceph.yaml ?15:28
dprinceshardy: Yeah, ceph can go in a directory like that I think.15:30
gfidenteokay I'll try to add the ceph job meantime15:31
gfidentewe can replace the inline cat with a pointer to the sample files, if we want, later15:31
dprincegfidente: If you'll hold off a minute I'll move this :)15:31
gfidentedprince, yep but the change to project-infra is unrelated I think15:32
dprincegfidente: oh, I see. You already landed the tripleo-ci change I did too.15:33
dprincegfidente: sure. the infra change is probably fine then15:33
gfidentedprince, yep I wanted to use that15:33
dprincegfidente: and eventually we can make it use the t-h-t ENV once it lands...15:33
gfidentethen realized we could use the environments instead of the inline cat15:33
gfidenteyep15:33
gfidenteso wanted to add to the envs the ha config, you can do it, deprecate enablepacemaker because we don't have it anymore the .pp anyway15:34
gfidenteand point to the envs15:34
jistrjayg: fwiw15:35
jistr[root@ov-gakwybgkly4-1-xtdfqtm2xaaf-controller-u6fuzecu3sib ~]# pcs resource list | grep galera15:35
jistrocf:heartbeat:galera - Manages a galara instance15:35
gfidenteshall we make ceph voting?15:35
gfidenteprobably not?15:35
jaygjistr: hmm, ok, that is good news!15:36
dprincegfidente: initially no I think15:36
gfidenteack15:36
jaygsurprising, but less change needed if it works15:36
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove NO_SIGNAL from puppet role templates  https://review.openstack.org/18308515:36
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove NO_SIGNAL from ControllerBootstrapNodeDeployment  https://review.openstack.org/18308615:36
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove NO_SIGNAL from Controller|ObjectSwiftDeployment  https://review.openstack.org/18308715:36
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove NO_SIGNAL from ControllerClusterConfig  https://review.openstack.org/18308815:36
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove DefaultSignalTransport from top-level template  https://review.openstack.org/18308915:36
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Wire in DeploymentSignalTransport global parameter  https://review.openstack.org/18309015:36
shardydiaf NO_SIGNAL! ;)15:36
*** jcoufal has quit IRC15:37
gfidentefeels like we also don't neet puppet-ha for tripleo-incubator changes, but we can probably remove later15:37
shardyzaneb: ^^ DeploymentSignalTransport change I mentioned15:37
shardythat's only the signals, not the polling for now15:38
zanebkk15:38
openstackgerritDan Prince proposed openstack-infra/tripleo-ci: Testing the ceph job.  https://review.openstack.org/18309215:38
zanebstevebaker: ^^15:38
dprincegfidente: actually, before you push it lets see what the latest result is ^^^15:38
dprincegfidente: I would, rather. Would be nice to know if it is passing.15:39
shardyzaneb: I probably won't get to the polling now before summit now, so if stevebaker is picking that up that's cool15:39
dprincegfidente: then we can decide if it votes or not I think15:39
gfidentethanks :)15:39
zanebshardy: I think he is, so that should work perfectly :)15:39
dprinceshardy: re, the NO_SIGNAL stuff. Does that cost us in terms of deployment time?15:39
dprinceshardy: as in more iteration of os-collect-config15:39
dprinceshardy: probably fine I think just wanted to ask15:40
shardydprince: AFAICS not much, but that's why I was discussing ways to minimise the poll time above with slagle15:40
shardythe NO_SIGNAL stuff is just wrong, so IMO we have to fix it15:40
shardylike, we're doing depends_on combined with NO_SIGNAL, which makes *no* sense15:40
dprinceshardy: understood, but I'd like to not increase our CI runtime in the process15:40
shardydprince: if the CI shows a significant slowdown I'm happy to WIP that series while I get the occ signal/hup thing just discussed working15:41
shardydprince: FWIW my local devtest_overcloud run takes pretty much the exact same time with and without those patches15:43
dprinceshardy: okay, sound good15:44
*** Goneri has joined #tripleo15:44
slagledprince: if you didn't notice, i set the topic for the first tripleo work session to the network architecture15:45
slagle*to be about15:45
dprinceslagle: I haven't. slammed actually implementing it :)15:46
dprinceslagle: what day again?15:46
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Provide RabbitMQ clients with a list of servers instead of VIP  https://review.openstack.org/18108115:47
marioshey tripleo o/ would appreciate some eyes on https://review.openstack.org/#/c/181015/ - i think i've addressed everyones concerns and we have green build now15:48
mariosthanks15:48
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Ensure haproxy is not enabled by puppet-tripleo when EnablePacemaker  https://review.openstack.org/18049015:48
shardySimple incubator review if anyone has a moment: https://review.openstack.org/#/c/163108/315:49
shardystepped deployment is also ready to go now: https://review.openstack.org/#/c/162109/15:49
*** rhallisey has joined #tripleo15:49
gfidentemarios, jistr that got green *after* the puppet-pacemaker PR merged?15:49
jistrgfidente: i think marios means green for the non-HA job15:50
slagledprince: thursday at 3:10pm https://libertydesignsummit.sched.org/event/84683930ab834188d80c34c9c494f66815:50
jistrHA is still down, i'm working on finishing spredzy|afk 's fixes15:50
gfidentejistr, ack15:51
dprinceslagle: cool, thanks for scheduling that15:51
* dprince needs to download the app15:51
gfidentejistr so marios has to do some more updates :)15:51
mariosjistr: ah good point sorry,15:51
mariosjistr: am just sure it was working (neutron bits anyway) cos i tested it enough15:52
mariosjistr: but yeah youre right15:52
mariosmy bad15:52
mariosgfidente: i will hit you15:52
*** daneyon has joined #tripleo15:52
*** daneyon has quit IRC15:52
*** daneyon has joined #tripleo15:53
gfidentemarios, it's not my fault!15:53
*** shadower has quit IRC15:53
gfidente;)15:53
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a directory for overcloud heat environments  https://review.openstack.org/18309615:54
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Environment which configures puppet pacemaker.  https://review.openstack.org/18309715:54
jistrgfidente: btw this doesn't seem to do the trick always https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/manifests/overcloud_controller_pacemaker.pp#L8615:54
jistrpuppet still attempted to start it for me15:54
jistrand failed15:54
jistri'll see if i can find a way around that15:54
*** dtantsur is now known as dtantsur|afk15:54
mariosgfidente: i will hit you anyway15:54
*** yog_ has joined #tripleo15:54
jistruntil we have this merged https://github.com/puppetlabs/puppetlabs-mongodb/pull/19815:55
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Environment which configures puppet pacemaker.  https://review.openstack.org/18309715:58
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a directory for overcloud heat environments  https://review.openstack.org/18309615:58
*** ukalifon1 has quit IRC15:58
openstackgerritDan Prince proposed openstack-infra/tripleo-ci: Switch to environments in tripleo-heat-templates.  https://review.openstack.org/18309815:59
dprincegfidente: ^^15:59
gfidentedprince, ack, thanks :)16:00
*** regebro has quit IRC16:02
*** Goneri has quit IRC16:02
* jistr hoping that aside from making the HA job green again we'll be able to crack down on a few intermittent issues16:06
jistrthat's high hopes tho :)16:06
mariosjistr:  you're a visionary amongst men!16:06
jistrlol16:06
gfidentejistr, if you did the updates already, I can check the mongodb part16:06
gfidentemaybe removing entirely the need for the = undef16:07
jistri guess i'll submit a new patch rather than amending spredzy|afk 's, i'd like it to be one patch to reduce the amount of rechecks necessary until we get a green HA16:08
openstackgerritDan Prince proposed openstack/tripleo-incubator: Add a puppet disk images YAML file  https://review.openstack.org/18111416:10
openstackgerritDan Prince proposed openstack/tripleo-incubator: Wire in load-images  https://review.openstack.org/17552216:10
openstackgerritDan Prince proposed openstack/tripleo-incubator: Add diskimage-builder/bin to PATH.  https://review.openstack.org/17719816:10
gfidentejistr, a new one on top of https://review.openstack.org/#/c/182947 ?16:10
*** daneyon_ has joined #tripleo16:11
jistrgfidente: no i'd like a new one on top of master, encompassing all the changes needed to get a green CI. It's not *that* big patch anyway, and we won't have to recheck on X patches everytime CI has an intermittent issue.16:11
gfidentejistr, ah I see, that would be instead of the tree below https://review.openstack.org/#/c/182947 right?16:12
gfidenteI was talking about this with spredzy|afk earlier today16:12
gfidentejistr, so I'm looking at the mongodb thing16:12
gfidentethanks for pointing it out16:12
jistrgfidente: i might have the mongodb thing solved16:12
jistrgimme a few mins16:13
*** daneyon has quit IRC16:13
gfidentejistr, so if you do that16:14
gfidentecan you also include https://review.openstack.org/#/c/180570/1/puppet/manifests/overcloud_controller.pp,cm16:14
gfidenteso we close that16:14
gfidentejistr, or merge that!16:14
jistrgfidente: the one you linked is in merge conflict, i'll include it16:16
gfidentehehe I was updating16:16
gfidenteyou decide16:16
gfidenteso long as we close16:16
*** shardy has quit IRC16:16
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Use $service_name from module when adding resource to pcmk  https://review.openstack.org/18057016:18
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Update to reflect puppet-pacemaker changes  https://review.openstack.org/18310316:18
openstackgerritMerged openstack/tripleo-incubator: Add diskimage-builder/bin to PATH.  https://review.openstack.org/17719816:19
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Update to reflect puppet-pacemaker changes  https://review.openstack.org/18310316:21
jistrgfidente: the trick for the cib errors is this: Class['::pacemaker::corosync'] -> Pcmk_resource <| |>16:22
gfidenteI'm not puppeteer at heart16:22
jistrgfidente: previously we only had it for systemd resources, which meant that VIPs weren't included in that ordering16:22
jistrso they were sometimes created before cluster was set up16:23
gfidenteah so now any resource will wait for pacemaker::corosync16:23
jistri mean attempted to create16:23
jistryeah16:23
gfidenteVIPs included16:23
gfidenteI see16:23
*** derekh_ has quit IRC16:24
gfidentejistr, L107?16:25
gfidentedprince, I see you set 2 ceph osds in the envs? you wanted that in CI as well?16:27
*** shakamunyi has quit IRC16:27
jistrgfidente, jayg: yeah i actually wonder if the clone_params should have true as a value or an empty string16:28
jistror if it doesn't matter16:28
*** barra204 has quit IRC16:28
dprincegfidente: we could I guess. Might help us catch races.16:28
jistrgfidente: i changed L107 after i hit some issue there but it might not be related16:28
dprincegfidente: perhaps it would slow us down a bit though16:28
dprincegfidente: maybe one then...16:28
gfidentedprince, honestly, as demo, 3 makes more sense because the replica is configured to make 3 copies to stay safe16:29
*** ukalifon1 has joined #tripleo16:30
dprincegfidente: hmmm. so maybe we do need to override this in tripleo-ci still?16:30
*** cody-somerville has quit IRC16:30
gfidenteyeah16:30
dprincegfidente: it is a developer focussed demo though, most devs may not have enough space for that16:30
gfidentewhatever :P I'll +2 menatime16:31
gfidente:)16:31
jistrgfidente: good eye there ;) i'll change it back to true now as i think the issue was more related to the service_ensure => undef16:31
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Update to reflect puppet-pacemaker changes  https://review.openstack.org/18310316:31
gfidentedprince, on https://review.openstack.org/#/c/183097/2/environments/puppet-pacemaker.yaml,cm I think the path is related to the where the env file is16:32
gfidentedprince, https://review.openstack.org/#/c/182553/16:33
openstackgerritMerged openstack/tripleo-heat-templates: Use pacemaker::resource::service instead of ::systemd  https://review.openstack.org/18050616:35
*** cwolferh has joined #tripleo16:38
*** masco has joined #tripleo16:39
dprincegfidente: okay, I'll try it out first.16:41
dprincegfidente: ci will tell us16:41
dprincegfidente: I thinkj that is correct though16:41
gfidentedprince, and it will be useful to know16:41
jistrre https://review.openstack.org/183103 -- still not quite there yet, i'll follow up tomorrow. If anyone wants to make changes to the patch in the meantime, feel free :)16:42
jistro/16:42
*** jistr has quit IRC16:42
gfidenteEmilienM, https://review.openstack.org/#/c/183046/16:43
dprincedsneddon: hi, I think there are a couple issues w/ vlans on bonds we'll need to fix in order to pull off https://review.openstack.org/#/c/181999/16:43
dprincedsneddon: I'm tinkering with a patch that does these two things: http://paste.openstack.org/show/222964/16:43
hewbroccadprince: Love the idea that you're using OVS for the bond config though16:44
dprincedsneddon: normal VLANs work fine. And I think I had vlans on Ovs bridges (or bonds) working at one point but they have regressed. No CI on it...16:45
hewbroccamakes a ton of sense16:45
dprincedsneddon: anwyay, this morning Flavio Leitner helped me work out the last few config details and basically it was just a simple config change16:46
dprincedsneddon: VLAN=true is only for non-OVS vlans. When on an Ovs bridge the Vlans are "internal" ports so VLAN=true should not be specified. Having it there makes 'ifup' very unhappy in fact.16:47
dprincedsneddon: oh, and the reason I've been spinning on this a bit is the other approaches (using 5-6 nics) or whatever don't fit the devtest model quite so nicely16:48
dprincedsneddon: ETOOMANYNICS16:48
dprincedsneddon: with this approach we can model a 3 NIC setup, 1 for the provisioning network, and 2 for all the overcloud traffic and layer the vlans on top of that.16:48
dsneddondprince, Yes, that makes sense16:49
*** eghobo has joined #tripleo16:49
dprincedsneddon: perhaps some hard coded conventions around VLAN ids... or we can make Heat parameters for those too. I don't care initially16:49
dprincedsneddon: I'm all set to wire up a bunch of stuff now, I'm still a bit fuzzy on what exactly to do with the external net though16:50
dsneddondprince, I figured we might add on Heat parameters for VLAN IDs, but initially it will just be whatever is in the os-net-config configuration templates.16:50
dsneddondprince, Most of the external net functionality is already wired in through the templates that set up the external bridge16:52
dprincedsneddon: right, that was for the single controller case sort of. In our case all controllers would get an external IP (which would/ or could be on one of these vlans). How to expose that so devtest still passes is how I've confused myself I think16:53
dprincedsneddon: I think we may need to muck w/ the seed a bit to get devtest driven CI happy with regards to that network16:54
dprincedsneddon: because by default the external (public) network VIP is just on the ctlplane16:54
dprincedsneddon: perhaps we just continue using that for now...16:55
dsneddondprince, OK, I'll have to take a look at how the public VIP is handled.16:55
dsneddondprince, I will be back in 5, I need to reboot my workstation16:56
*** dsneddon has quit IRC16:56
*** masco has quit IRC16:57
*** yamahata has quit IRC16:58
*** morazi has quit IRC16:58
*** dsneddon has joined #tripleo16:59
openstackgerritMerged stackforge/puppet-tripleo: Increase global maxconn to 10000 and remove per-instance limit  https://review.openstack.org/18304616:59
*** morazi has joined #tripleo17:01
*** mestery has joined #tripleo17:03
*** sdake has quit IRC17:06
dsneddondprince, It looks like the existing stuff is a mess. overcloud-vlan-port.yaml is deprecated, and it looks like it was never really working. There are comments saying it needs a Neutron fix, and there is a hardcoded subnet mask, etc.17:06
*** sdake has joined #tripleo17:06
dprincedsneddon: I don't even remember what that template is. Anything in deprecated can be ignored17:08
dsneddondprince, So you're right, unfortunately we're going to need to wire up the external network.17:08
*** Marga_ has quit IRC17:09
dprincedsneddon: right, so the I think most of the rest of this is boilerplate (lots of patches incoming soon) expect that.17:09
dprincedsneddon: perhaps we can sort through it next week17:09
dsneddondprince, Well I definitely don't think I'm going to get it sorted this week. ;-)17:09
dprincedsneddon: I'm pretty sure I can hack out my own dev environment to support it. Getting it cleanly working in devtest may require a few extra things though17:09
*** athomas has quit IRC17:19
*** mcornea has quit IRC17:20
*** yamahata has joined #tripleo17:21
*** pdb has quit IRC17:22
*** mcornea has joined #tripleo17:24
*** lucasagomes has quit IRC17:36
*** mcornea has quit IRC17:41
*** Marga_ has joined #tripleo17:45
openstackgerritDan Sneddon proposed openstack/os-net-config: Add 'eno*' interfaces to embedded_nics  https://review.openstack.org/18313217:50
openstackgerritDan Sneddon proposed openstack/os-net-config: Add NM_CONTROLLED=no to each interface configuration  https://review.openstack.org/18313618:00
*** morazi has quit IRC18:04
*** gfidente has quit IRC18:12
openstackgerritMerged openstack/tuskar: Avoid MODULEPATH environment var in config generator  https://review.openstack.org/18202518:16
openstackgerritDan Sneddon proposed openstack/os-net-config: Add NM_CONTROLLED=no to each interface configuration  https://review.openstack.org/18313618:17
*** morazi has joined #tripleo18:18
*** spzala has joined #tripleo18:18
openstackgerritJay Dobies proposed openstack/tuskar: Added removal_policies support for role resource groups  https://review.openstack.org/18314818:18
*** lazy_prince has quit IRC18:30
*** rhallisey has quit IRC18:30
*** russell_h has left #tripleo18:36
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Update neutron local_ip to use the tenant network  https://review.openstack.org/17871618:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a network ports IP mapping resource  https://review.openstack.org/17871418:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to block storage roles  https://review.openstack.org/18082418:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to swift roles  https://review.openstack.org/18082318:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to ceph roles  https://review.openstack.org/18082218:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to controller roles  https://review.openstack.org/17784618:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated network ports to compute roles  https://review.openstack.org/18082118:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a ports (ip address) abstraction layer  https://review.openstack.org/17784518:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add isolated net parameters to net-config stacks  https://review.openstack.org/18082018:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Wire in optional network creation for overcloud  https://review.openstack.org/17784418:39
*** mcornea has joined #tripleo18:46
*** mcornea has quit IRC18:50
openstackgerritDan Prince proposed openstack/os-net-config: ifcfg: don't set VLAN=yes if is an ovs_port  https://review.openstack.org/18317119:00
*** dsneddon is now known as dsneddon_lunch19:05
*** erikmack has quit IRC19:18
openstackgerritDan Prince proposed openstack/os-net-config: Don't set OVSDHCPINTERFACES if bridge is static  https://review.openstack.org/18318419:20
*** weshay has quit IRC19:33
*** dsneddon_lunch has quit IRC19:35
*** ifarkas has quit IRC19:40
openstackgerritJames Slagle proposed openstack/diskimage-builder: Use Centos 7 cloud image symlink  https://review.openstack.org/18318519:41
*** eghobo has quit IRC19:41
*** rhallisey has joined #tripleo19:47
*** pdb has joined #tripleo19:48
*** radez is now known as radez_g0n319:49
*** daneyon has joined #tripleo19:51
*** openstackgerrit has quit IRC19:52
*** openstackgerrit has joined #tripleo19:52
*** daneyon_ has quit IRC19:54
*** trown is now known as trown|outttypeww20:01
*** jtomasek has quit IRC20:02
*** pdb has quit IRC20:05
*** dsneddon has joined #tripleo20:12
*** russell_h has joined #tripleo20:27
*** eghobo has joined #tripleo20:27
*** weshay has joined #tripleo20:34
*** russell_h has left #tripleo20:36
*** dshulyak has joined #tripleo20:37
*** rhallisey has quit IRC20:46
*** rhallisey has joined #tripleo20:48
*** rhallisey has quit IRC20:53
*** dshulyak has quit IRC20:56
*** dshulyak has joined #tripleo20:56
stevebakerI'm a bit blocked on these tripleo-common changes landing https://review.openstack.org/#/q/project:openstack/tripleo-common,n,z20:56
stevebakerAny chance they could get some review attention?20:56
*** dshulyak has quit IRC20:56
*** dprince has quit IRC20:57
*** untriaged-bot has joined #tripleo21:00
untriaged-botUntriaged bugs so far:21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145517521:00
openstackLaunchpad bug 1455175 in tripleo "Option to configure gateway through keepalived" [Undecided,New] - Assigned to Mayank (mayank0107)21:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985221:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)21:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985421:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,Fix committed] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480321:00
openstackLaunchpad bug 1454803 in tripleo "puppet: Neutron is not configured with L2 population" [Undecided,New]21:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275221:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/145480221:00
openstackLaunchpad bug 1454802 in tripleo "puppet: Neutron does not use Nova notifications" [Undecided,New]21:00
*** untriaged-bot has quit IRC21:00
*** jprovazn has quit IRC21:03
*** dshulyak has joined #tripleo21:07
*** chlong has quit IRC21:08
*** jayg is now known as jayg|g0n321:10
*** dshulyak has quit IRC21:12
*** dshulyak has joined #tripleo21:18
*** dshulyak has quit IRC21:19
*** openstackgerrit has quit IRC21:22
*** openstackgerrit has joined #tripleo21:22
*** pdb has joined #tripleo21:23
*** dshulyak has joined #tripleo21:27
*** dshulyak has quit IRC21:27
*** yamahata has quit IRC21:32
openstackgerritJeremy Stanley proposed openstack-infra/tripleo-ci: Replace ci.o.o links with docs.o.o/infra  https://review.openstack.org/18330621:45
*** spzala has quit IRC21:49
*** weshay has quit IRC21:52
*** pdb has quit IRC21:57
*** eghobo has quit IRC22:09
*** eghobo has joined #tripleo22:10
*** spzala has joined #tripleo22:13
*** sdake has quit IRC22:17
*** sdake has joined #tripleo22:18
*** MasterPiece has quit IRC22:20
*** spzala has quit IRC22:20
*** rlandy has quit IRC22:25
*** morazi has quit IRC22:27
*** daneyon has quit IRC22:39
*** shakamunyi has joined #tripleo22:47
*** barra204 has joined #tripleo22:47
openstackgerritOpenStack Proposal Bot proposed openstack/tuskar: Updated from global requirements  https://review.openstack.org/17988022:50
*** barra204 has quit IRC23:04
*** shakamunyi has quit IRC23:04
*** SamYaple has quit IRC23:05
*** chlong has joined #tripleo23:10
*** panda has quit IRC23:13
*** panda has joined #tripleo23:13
*** shakamunyi has joined #tripleo23:17
*** barra204 has joined #tripleo23:17
*** alop has quit IRC23:37
*** bvandenh has joined #tripleo23:46
*** yuanying has quit IRC23:46
openstackgerritSteve Baker proposed openstack/os-collect-config: Refactor merging out of cfn collector  https://review.openstack.org/18335523:48
openstackgerritSteve Baker proposed openstack/os-collect-config: Merge deployments data for collectors heat, request  https://review.openstack.org/18335623:48
*** bvandenh has quit IRC23:53
openstackgerritSteve Baker proposed openstack/os-collect-config: Log with info instead of warn when collector not configured  https://review.openstack.org/18336023:56

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