| *** jerrygb has quit IRC | 00:18 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #tripleo | 00:23 | |
| *** larstobi has quit IRC | 01:33 | |
| *** rainya has quit IRC | 01:33 | |
| *** davidlenwell has quit IRC | 01:33 | |
| *** Nakato has quit IRC | 01:33 | |
| *** gus has quit IRC | 01:33 | |
| *** mgagne has quit IRC | 01:33 | |
| *** wendar has quit IRC | 01:33 | |
| *** wendar_ has joined #tripleo | 01:33 | |
| *** davidlenwell has joined #tripleo | 01:34 | |
| *** ChanServ sets mode: +v davidlenwell | 01:34 | |
| *** dsneddon has quit IRC | 01:35 | |
| *** adam_g has quit IRC | 01:35 | |
| *** jroll has quit IRC | 01:35 | |
| *** eggmaste` has joined #tripleo | 01:35 | |
| *** Nakato has joined #tripleo | 01:35 | |
| *** eggmaster has quit IRC | 01:36 | |
| *** larstobi has joined #tripleo | 01:37 | |
| *** gus has joined #tripleo | 01:37 | |
| *** jroll has joined #tripleo | 01:38 | |
| *** mgagne has joined #tripleo | 01:38 | |
| *** adam_g has joined #tripleo | 01:38 | |
| *** rainya has joined #tripleo | 01:38 | |
| *** adam_g has quit IRC | 01:38 | |
| *** adam_g has joined #tripleo | 01:38 | |
| *** mgagne is now known as Guest91043 | 01:38 | |
| *** dsneddon has joined #tripleo | 01:38 | |
| *** jerrygb has joined #tripleo | 01:39 | |
| *** jerrygb has quit IRC | 02:04 | |
| *** noslzzp has quit IRC | 02:05 | |
| *** noslzzp has joined #tripleo | 02:07 | |
| *** eggmaste` is now known as eggmaster | 02:13 | |
| *** jerrygb has joined #tripleo | 02:26 | |
| *** noslzzp has quit IRC | 02:26 | |
| *** noslzzp has joined #tripleo | 02:34 | |
| *** cwolferh has joined #tripleo | 03:08 | |
| *** noslzzp has quit IRC | 03:09 | |
| *** tiswanso has joined #tripleo | 03:14 | |
| openstackgerrit | Steve Baker proposed openstack/python-tripleoclient: Log all resource events during overcloud deploy https://review.openstack.org/235032 | 03:27 |
|---|---|---|
| *** links has joined #tripleo | 04:11 | |
| *** tiswanso has quit IRC | 04:16 | |
| *** masco__ has joined #tripleo | 04:20 | |
| *** masco__ has quit IRC | 04:26 | |
| *** masco__ has joined #tripleo | 04:26 | |
| *** noslzzp has joined #tripleo | 04:46 | |
| *** masco_ has joined #tripleo | 05:01 | |
| *** masco__ has quit IRC | 05:05 | |
| *** noslzzp has quit IRC | 05:12 | |
| *** noslzzp has joined #tripleo | 05:23 | |
| *** anande has joined #tripleo | 05:31 | |
| *** anande has quit IRC | 05:31 | |
| *** anande has joined #tripleo | 05:33 | |
| *** anande has quit IRC | 05:33 | |
| *** anande has joined #tripleo | 05:34 | |
| openstackgerrit | Rhys Oxenham proposed openstack/tripleo-heat-templates: Add network templates for multiple NICs with flat and VLAN tagged networks https://review.openstack.org/214372 | 05:59 |
| *** Marga_ has joined #tripleo | 06:04 | |
| *** rcernin has joined #tripleo | 06:16 | |
| openstackgerrit | Rhys Oxenham proposed openstack/tripleo-heat-templates: Add network templates for multiple NICs with flat and VLAN tagged networks https://review.openstack.org/214372 | 06:18 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function https://review.openstack.org/245643 | 06:19 |
| *** shakamunyi has joined #tripleo | 06:21 | |
| *** dhellmann has quit IRC | 06:28 | |
| *** dhellmann has joined #tripleo | 06:28 | |
| *** dhellmann has quit IRC | 06:34 | |
| *** dhellmann has joined #tripleo | 06:34 | |
| *** shakamunyi has quit IRC | 06:39 | |
| *** shakamunyi has joined #tripleo | 06:53 | |
| *** rasca has joined #tripleo | 06:55 | |
| *** jaosorior has joined #tripleo | 06:59 | |
| *** lazy_prince has joined #tripleo | 06:59 | |
| *** anande has quit IRC | 07:00 | |
| *** anande has joined #tripleo | 07:00 | |
| *** jerrygb has quit IRC | 07:03 | |
| *** anande has quit IRC | 07:05 | |
| *** anande has joined #tripleo | 07:05 | |
| *** Nakato_ has joined #tripleo | 07:09 | |
| *** davidlenwell_ has joined #tripleo | 07:10 | |
| *** slagle_ has joined #tripleo | 07:11 | |
| *** jrist has quit IRC | 07:15 | |
| *** davidlenwell has quit IRC | 07:15 | |
| *** Nakato has quit IRC | 07:15 | |
| *** slagle has quit IRC | 07:15 | |
| *** Ng has quit IRC | 07:15 | |
| *** Hunner has quit IRC | 07:15 | |
| *** jrist has joined #tripleo | 07:15 | |
| *** jrist has joined #tripleo | 07:15 | |
| *** Hunner has joined #tripleo | 07:15 | |
| *** Hunner has joined #tripleo | 07:15 | |
| *** Ng has joined #tripleo | 07:20 | |
| *** ChanServ sets mode: +v Ng | 07:20 | |
| *** Nakato_ is now known as Nakato | 07:20 | |
| *** ukalifon has joined #tripleo | 07:24 | |
| *** jtomasek has joined #tripleo | 07:34 | |
| *** olap has joined #tripleo | 07:40 | |
| openstackgerrit | Jane.zhang proposed openstack/diskimage-builder: Fix grub-efi-amd64-signed install failure https://review.openstack.org/219612 | 07:43 |
| *** liverpooler has joined #tripleo | 07:46 | |
| *** bvandenh has joined #tripleo | 07:48 | |
| *** jprovazn has joined #tripleo | 07:55 | |
| *** bvandenh has quit IRC | 08:06 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/python-tripleoclient: Make root directory configurable https://review.openstack.org/230275 | 08:07 |
| *** lsmola has joined #tripleo | 08:07 | |
| *** bvandenh has joined #tripleo | 08:09 | |
| *** bvandenh has quit IRC | 08:16 | |
| *** ifarkas has joined #tripleo | 08:16 | |
| *** shivrao has joined #tripleo | 08:16 | |
| *** akuznetsov has joined #tripleo | 08:17 | |
| *** akuznetsov has quit IRC | 08:22 | |
| *** tchaypo1 has joined #tripleo | 08:25 | |
| *** olap has quit IRC | 08:25 | |
| *** tchaypo1 is now known as tchaypo | 08:25 | |
| *** tchaypo is now known as Guest88883 | 08:26 | |
| *** tremble has joined #tripleo | 08:26 | |
| *** Guest88883 is now known as tchaypo | 08:28 | |
| *** shardy has joined #tripleo | 08:40 | |
| *** jaosorior has quit IRC | 08:44 | |
| *** jaosorior has joined #tripleo | 08:51 | |
| *** olap has joined #tripleo | 09:01 | |
| *** devvesa has joined #tripleo | 09:01 | |
| *** jerrygb has joined #tripleo | 09:04 | |
| *** jerrygb has quit IRC | 09:08 | |
| *** dtantsur|afk is now known as dtantsur | 09:08 | |
| *** shivrao has quit IRC | 09:15 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Ensure memcached resource uses interleave=true https://review.openstack.org/245670 | 09:21 |
| *** hewbrocca has joined #tripleo | 09:21 | |
| *** athomas has joined #tripleo | 09:22 | |
| *** gfidente has joined #tripleo | 09:23 | |
| *** gfidente has quit IRC | 09:23 | |
| *** gfidente has joined #tripleo | 09:23 | |
| *** masco_ has quit IRC | 09:28 | |
| *** killer_prince has joined #tripleo | 09:32 | |
| *** paramite has joined #tripleo | 09:32 | |
| *** killer_prince has quit IRC | 09:32 | |
| *** killer_prince has joined #tripleo | 09:32 | |
| *** lazy_prince has quit IRC | 09:35 | |
| *** electrofelix has joined #tripleo | 09:39 | |
| *** jistr has joined #tripleo | 09:40 | |
| *** killer_prince has quit IRC | 09:40 | |
| *** killer_prince has joined #tripleo | 09:40 | |
| *** mgould has joined #tripleo | 09:44 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add missing constraints in yum_update.sh https://review.openstack.org/244321 | 09:47 |
| *** olap has quit IRC | 09:48 | |
| *** mcornea has joined #tripleo | 09:49 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Set start/stop pacemaker resource timeouts for updates https://review.openstack.org/245174 | 09:50 |
| *** olap has joined #tripleo | 09:50 | |
| *** stendulker has joined #tripleo | 09:53 | |
| *** untriaged-bot has joined #tripleo | 10:00 | |
| untriaged-bot | Untriaged bugs so far: | 10:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1505900 | 10:00 |
| openstack | Launchpad bug 1505900 in diskimage-builder "Failed to create 'wily' release with DIB_RELEASE" [Undecided,New] | 10:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1515996 | 10:00 |
| openstack | Launchpad bug 1515996 in tripleo "Typo in NeutronMechanismDrivers description in heat templates" [Undecided,New] - Assigned to shravya Gaddam (shravya-g90) | 10:00 |
| *** untriaged-bot has quit IRC | 10:00 | |
| *** mbound has joined #tripleo | 10:00 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-puppet-elements: Add python-memcached to list of installed pkgs on controllers https://review.openstack.org/245686 | 10:11 |
| *** regebro has joined #tripleo | 10:13 | |
| openstackgerrit | Steven Hardy proposed openstack/instack-undercloud: puppet: restart mysql if config changes https://review.openstack.org/245689 | 10:14 |
| *** openstackgerrit has quit IRC | 10:16 | |
| *** paramite is now known as paramite|afk | 10:16 | |
| gfidente | shardy, I see you started backporting stuff | 10:16 |
| *** openstackgerrit has joined #tripleo | 10:17 | |
| gfidente | I will check my submissions as well and add you on CC | 10:17 |
| shardy | gfidente: I'm just pushing one commit per repo to see if CI works | 10:17 |
| shardy | gfidente: I spotted an issue with t-h-t already | 10:17 |
| gfidente | sigh | 10:17 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Bump further the stop/start timeout for pcmk/systemd services https://review.openstack.org/245161 | 10:17 |
| shardy | Yeah, it's frustrating as project-config patches take so long to get reviewed/approved :( | 10:17 |
| shardy | I think I can see what the problem is though, about to push the fix | 10:18 |
| tremble | shardy: Is that just because there's so few approvers? | 10:18 |
| shardy | gfidente: https://etherpad.openstack.org/p/tripleo-stable-branches I'm collecting notes re validating the CI at the bottom of there | 10:18 |
| shardy | tremble: Yeah, I would assume so | 10:19 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-common: Fix cherry-pick comments https://review.openstack.org/245691 | 10:20 |
| *** paramite|afk is now known as paramite | 10:26 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Add missing constraints in yum_update.sh https://review.openstack.org/245694 | 10:28 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Set start/stop pacemaker resource timeouts for updates https://review.openstack.org/245695 | 10:28 |
| *** akrivoka has joined #tripleo | 10:35 | |
| *** mkovacik has joined #tripleo | 10:36 | |
| *** stendulker has quit IRC | 10:37 | |
| devvesa | dtantsur: shardy redirect me to you for a problem I have using tripleo.sh for build the undercloud | 10:44 |
| dtantsur | devvesa, hi! Not sure I'm the right person, but what's the problem? | 10:45 |
| devvesa | Hi! I'm building the undercloud, first using the --repo-setup attribute and then with the --undercloud attribute | 10:46 |
| devvesa | But it fails | 10:46 |
| devvesa | Warning: Scope(Class[Ceilometer::Expirer]): Parameter "time_to_live" is deprecated and will be removed in next release. Use metering_time_to_live in "ceilometer" class instead. | 10:46 |
| devvesa | Error: Could not find class ::ironic::inspector for instack on node instack | 10:46 |
| devvesa | Error: Could not find class ::ironic::inspector for instack on node instack | 10:46 |
| devvesa | It is like the version of 'openstack-puppet; package is too old.. | 10:46 |
| devvesa | But is the same one available on: http://trunk.rdoproject.org/centos7/current-tripleo/ | 10:47 |
| devvesa | -> openstack-puppet-modules-2015.2-dev.232f6155275c8defe1d50c675d702041c108cb30_8eef2746.el7.centos.noarch | 10:47 |
| devvesa | shardy says that should work because you are using it on the CI | 10:48 |
| *** olaph has quit IRC | 10:48 | |
| dtantsur | so yeah, you're right that puppet version is too old, otherwise I'm not sure what to do. please wait until trown|outttypeww gets online | 10:48 |
| devvesa | ok, thanks! but... how you manage to get the CI run then? | 10:49 |
| *** olaph has joined #tripleo | 10:53 | |
| dtantsur | I don't know, I'm not a TripleO person, so I'm not sure why you got redirected to me :) | 10:58 |
| dtantsur | I suspect something might be wrong about your environment, e.g. some yum/dnf pinning | 10:58 |
| dtantsur | devvesa, https://github.com/openstack/tripleo-common/blob/master/scripts/tripleo.sh#L159-L205 is what gate used to my best knowledge | 11:00 |
| *** mbound has quit IRC | 11:03 | |
| devvesa | Sorry dtantsur, blame on shardy :) | 11:03 |
| shardy | dtantsur: I redirected to you because I assumed you were testing inspector in TripleO, and this is an inspector related issue | 11:03 |
| shardy | anyway, sorry devvesa that we've not yet resolved your problem, I'll try to test later, if nobody else has resolved it by then | 11:04 |
| devvesa | I'll try to build upon centos-liberty repos | 11:04 |
| shardy | it must be a repo mismatch problem, or CI would be broken | 11:05 |
| devvesa | No problem at all, any help is appreciated! | 11:05 |
| shardy | You could use export DIB_INSTALLTYPE_puppet_modules=source to pull in the latest puppet modules | 11:05 |
| shardy | http://docs.openstack.org/developer/tripleo-docs/installation/installation.html#installing-the-undercloud | 11:06 |
| shardy | I can't see that being used in CI, but perhaps that will allow for a workaround | 11:06 |
| *** tosky has joined #tripleo | 11:06 | |
| devvesa | ok, I'll try the centos-liberty packages first (I don't think It can be a problem on my development) | 11:07 |
| dtantsur | shardy, last time it was trown|outttypeww who tested it | 11:08 |
| devvesa | But... the DIB_INSTALLTYPE_puppet_modules also applies to puppet modules on the undercloud? | 11:08 |
| shardy | dtantsur: Ok, cool, thanks for the clarification | 11:08 |
| devvesa | I thought that only was applicable on the overcloud image | 11:08 |
| *** lazy_prince has joined #tripleo | 11:08 | |
| shardy | devvesa: we use diskimage-builder elements for both undercloud package installs and overcloud image building | 11:09 |
| devvesa | ack! | 11:09 |
| *** killer_prince has quit IRC | 11:11 | |
| *** links has quit IRC | 11:21 | |
| *** mbound has joined #tripleo | 11:31 | |
| *** paramite is now known as paramite|afk | 11:41 | |
| *** mbound has quit IRC | 11:45 | |
| *** mbound has joined #tripleo | 11:45 | |
| cmyster | Hi, did anyone see an error when I have not enough valid hosts even when I do ? | 11:51 |
| cmyster | looks like this http://paste.openstack.org/show/478963/ | 11:53 |
| cmyster | introspection passed successfully, ironic node-list sees all VMs correctly, and only this issue is preventing me from moving on | 11:54 |
| cmyster | oh and hi shardy, all well I hope? | 11:55 |
| *** links has joined #tripleo | 11:55 | |
| *** links has quit IRC | 11:59 | |
| gfidente | marios, thanks for the feedback on the external lb stuff :P | 12:04 |
| *** jerrygb has joined #tripleo | 12:04 | |
| *** masterbound has joined #tripleo | 12:06 | |
| *** mbound has quit IRC | 12:06 | |
| *** akrivoka has quit IRC | 12:08 | |
| marios | gfidente: yeah sry, there's a lot goign on. my main reservation was understanding the changes required to the 'default' - with no network iso, the vip ports are nooped, with network iso, they are neutron ports, when external lb, they are nooped but the controller ports themserles are 'created' from the _pool templates | 12:08 |
| *** jerrygb has quit IRC | 12:09 | |
| gfidente | marios, sorry? | 12:09 |
| marios | gfidente: for https://review.openstack.org/#/c/222497/25 | 12:09 |
| gfidente | marios, I think bnemec had a different proposals which would make the templates create the neutron ports lying down a little less custom code | 12:10 |
| marios | gfidente: so the new "vip ports" are NOOP here https://review.openstack.org/#/c/222497/25/overcloud-resource-registry-puppet.yaml | 12:10 |
| gfidente | I'm going to submit that as an alternative implementation as well I think | 12:10 |
| gfidente | yes I see your point there, but the problem is those are switched into actual neutron ports by the network-isolation.yaml file | 12:10 |
| gfidente | in the existing implementation we want those to stay nooped instead | 12:11 |
| gfidente | I'll add you as reviewer for the other implementation as well, you're committed now, no excuses | 12:11 |
| marios | gfidente: as i said, i'd like another pass, or even wait until you've revised if you're going to do that. | 12:11 |
| marios | gfidente: yeah sure, i just didn't feel confident enough to -1 or +2 at this point. I saw a discussion with bnemec and you fly by so i am aware of that so didn't want to stand in the way either way, not at this point anyway ;) | 12:12 |
| gfidente | it's good to receive eyes on those | 12:13 |
| gfidente | better later than never | 12:13 |
| *** akrivoka has joined #tripleo | 12:21 | |
| *** jeff_phillips has joined #tripleo | 12:34 | |
| *** lucasagomes is now known as lucas-hungry | 12:35 | |
| *** paramite|afk is now known as paramite | 12:36 | |
| *** jdob has joined #tripleo | 12:37 | |
| *** rhallisey_ has joined #tripleo | 12:39 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays https://review.openstack.org/187506 | 12:44 |
| *** Guest51663 is now known as beagles | 12:47 | |
| *** openstack has joined #tripleo | 12:49 | |
| *** rhallisey_ is now known as rhallisey | 12:49 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Make all network ports type to consume FixedIPs https://review.openstack.org/245749 | 12:50 |
| *** trown|outttypeww is now known as trown | 12:55 | |
| *** jdob has quit IRC | 12:56 | |
| *** jdob has joined #tripleo | 12:57 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow the deployer to pick a predefined IP for VIPs https://review.openstack.org/245750 | 12:57 |
| *** tzumainn has joined #tripleo | 12:59 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow the deployer to pick a predefined IP for VIPs https://review.openstack.org/245750 | 12:59 |
| *** dprince has joined #tripleo | 13:00 | |
| *** olaph has quit IRC | 13:01 | |
| *** jdob has quit IRC | 13:01 | |
| *** olaph has joined #tripleo | 13:01 | |
| *** jdob has joined #tripleo | 13:01 | |
| *** julim has joined #tripleo | 13:06 | |
| *** jayg|g0n3 is now known as jayg | 13:15 | |
| *** pradk has joined #tripleo | 13:16 | |
| *** jaosorior has quit IRC | 13:16 | |
| *** jaosorior has joined #tripleo | 13:17 | |
| *** Marga_ has joined #tripleo | 13:19 | |
| *** Marga_ has quit IRC | 13:24 | |
| *** thrash|wknd is now known as thrash | 13:25 | |
| *** morazi has joined #tripleo | 13:28 | |
| *** jprovazn has quit IRC | 13:32 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays https://review.openstack.org/187506 | 13:36 |
| *** rbrady has joined #tripleo | 13:38 | |
| *** pblaho has quit IRC | 13:50 | |
| *** rpothier has joined #tripleo | 13:50 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Remove class_parameter_defaults puppet-lint check https://review.openstack.org/244607 | 13:50 |
| *** Ng has quit IRC | 13:53 | |
| *** BadCub has quit IRC | 13:53 | |
| *** greghaynes has quit IRC | 13:53 | |
| *** greghaynes has joined #tripleo | 13:54 | |
| *** Ng has joined #tripleo | 13:55 | |
| *** ChanServ sets mode: +v Ng | 13:55 | |
| *** BadCub has joined #tripleo | 13:56 | |
| *** mcornea has quit IRC | 13:59 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Make CloudName available for Endpoints https://review.openstack.org/239960 | 13:59 |
| *** mcornea has joined #tripleo | 14:00 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Expose the IP of the Endpoints https://review.openstack.org/241274 | 14:00 |
| *** lucas-hungry is now known as lucasagomes | 14:01 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add the GlanceRegistry and Horizon endpoints to EndpointMap https://review.openstack.org/240497 | 14:01 |
| *** ukalifon has quit IRC | 14:02 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Support network isolation without external nets https://review.openstack.org/199599 | 14:05 |
| *** julim has quit IRC | 14:06 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add output for host entries https://review.openstack.org/240171 | 14:06 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add environment for isolated networks without tunneling VLAN https://review.openstack.org/218535 | 14:08 |
| tremble | shardy: while you're in a reviewing mood, could you take a look at https://review.openstack.org/#/c/231930/ and https://review.openstack.org/#/c/241200/ | 14:09 |
| *** dtantsur is now known as dtantsur|brb | 14:09 | |
| shardy | tremble: will do | 14:09 |
| tremble | shardy++ | 14:09 |
| tremble | Thanks shardy | 14:10 |
| *** jprovazn has joined #tripleo | 14:11 | |
| shardy | I was a bit unsure re that series tbh as there's been such a high rate of re-posts, I got the impression it was a bit WIP still as a result | 14:11 |
| tremble | shardy: We made the mistake of chaining a lot of CRs together, so they kept getting rebased | 14:12 |
| shardy | tremble: Ok, thanks for the clarification | 14:12 |
| tremble | shardy: And originally they all depended on the puppet endpoints change that was a little wonky | 14:13 |
| *** ukalifon has joined #tripleo | 14:16 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Don't log exceptions redundantly https://review.openstack.org/237097 | 14:17 |
| *** Marga_ has joined #tripleo | 14:19 | |
| shardy | tremble, jaosorior: Have you had any discussion re removing the top-level parameters in overcloud-without-mergepy? | 14:20 |
| tremble | Which ones? | 14:21 |
| shardy | Normally I've viewed that as the default surface for user interaction, so I'm wondering if it might be better to leave the existing parameters in place | 14:21 |
| shardy | https://review.openstack.org/#/c/231930/46/overcloud-without-mergepy.yaml | 14:21 |
| shardy | SSLCertificate/SSLKey etc | 14:21 |
| shardy | I normally view the parameter_defaults technique as a good way to wire in additional stuff e.g third-party pieces where the needed parameters aren't known ahead of time | 14:22 |
| *** tiswanso has joined #tripleo | 14:22 | |
| tremble | Hmm, good point.... jaosorior ? | 14:22 |
| shardy | but when you do know them, the advantage of using the top-level parameters schema is the options become more easily discoverable, e.g by a UI or whatever | 14:22 |
| shardy | It's a judgement call either way, but given they already exist, I'm leaning towards leaving them and making the existing options actually work? | 14:23 |
| tremble | shardy: The existing options are just file locations, which still leaves you needing to land the certs. | 14:23 |
| jaosorior | shardy: Well, they weren't used at all. And I removed them since actually taking them into use was more verbose than the patch that was posted | 14:23 |
| *** rlandy has joined #tripleo | 14:24 | |
| jaosorior | on the other hand, I didn't find the existing parameters used at all. Just passed around, and thought that would end up being even more confusing for the end-user | 14:24 |
| *** Marga_ has quit IRC | 14:24 | |
| shardy | Yeah, I can see that argument, but it's not a lot more verbose, few additional get_param's passed in to OS::TripleO::NodeTLSData | 14:26 |
| shardy | I'll start a ML thread, as I think it'd be good to get consensus on when to use parameters vs parameter_defaults in general | 14:27 |
| shardy | and this is an ideal example to focus that discussion | 14:27 |
| shardy | other than that the patch looks good to me | 14:27 |
| shardy | To clarify, I'm not blocking on the parameters thing, I just want to discuss it a bit | 14:27 |
| jaosorior | shardy: well, the CAcertificate would need to be taken into use too | 14:27 |
| jaosorior | which goes to every resourcegroup | 14:27 |
| jaosorior | * almost every | 14:28 |
| jaosorior | sure | 14:28 |
| shardy | jaosorior: Yeah, it's a bit more wiring in from the top level template | 14:28 |
| *** jerrygb has joined #tripleo | 14:29 | |
| *** jerrygb has quit IRC | 14:29 | |
| *** jerrygb has joined #tripleo | 14:29 | |
| jaosorior | shardy: I could re-add it, but the downsides are that both the CR will get a bit more cluttered and on the other hand, we would then be restricting the user to using those parameters specifically. When they wouldn't necessarilly want to just deploy one cert/key for all the controllers, but actually deploy an array like it was done in a previous proposal of that patch | 14:30 |
| *** lblanchard has joined #tripleo | 14:31 | |
| jaosorior | (which could still be done, they would need to write their own environment, but all the tools are there) | 14:31 |
| shardy | jaosorior: ack, I wouldn't change anything yet, lets get some wider feedbackk first | 14:31 |
| jaosorior | ack | 14:31 |
| *** Goneri has joined #tripleo | 14:33 | |
| *** mkovacik has quit IRC | 14:34 | |
| *** pblaho has joined #tripleo | 14:35 | |
| *** daemontool_ has joined #tripleo | 14:38 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add support for enabling L2 population in Neutron https://review.openstack.org/244723 | 14:47 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add support for enabling ARP Responder in Neutron https://review.openstack.org/244801 | 14:50 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-specs: New spec: composable services within roles https://review.openstack.org/245804 | 14:53 |
| *** julim has joined #tripleo | 14:59 | |
| d0ugal | rbrady: In https://review.openstack.org/#/c/227297 | 15:04 |
| d0ugal | rbrady: I think lots of exceptions will be logged twice | 15:05 |
| *** ukalifon1 has joined #tripleo | 15:05 | |
| d0ugal | rbrady: As they are logged in plan_storage.py and re-raised and then logged in plan.py | 15:05 |
| rbrady | d0ugal: I'll take a look before I push up the plan model patchset | 15:05 |
| d0ugal | rbrady: ok, I am not sure if I am miss-reading or not, so I may be wrong | 15:06 |
| d0ugal | rbrady: also, we should really use LOG.exception rather than LOG.error | 15:06 |
| d0ugal | (sorry I didn't notice that before) | 15:06 |
| *** ukalifon has quit IRC | 15:07 | |
| gfidente | slagle_, shardy, jistr on the 'remember to backport' comments, I'm for backport everything (assuming it passes update) if not labeled with do-not-backport | 15:07 |
| *** jaosorior has quit IRC | 15:08 | |
| d0ugal | rbrady: Added some comments so you can see where I mean | 15:08 |
| *** jaosorior has joined #tripleo | 15:08 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 15:09 |
| gfidente | No-Backport: tah as per release spec | 15:09 |
| slagle_ | gfidente: sure, i just put those there to remind myself | 15:10 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 15:10 |
| gfidente | slagle_, ack so as soon as we get stable branches to work I'll probably resubmit a bunch of those | 15:10 |
| *** mcornea has quit IRC | 15:16 | |
| *** olap has quit IRC | 15:17 | |
| *** mcornea has joined #tripleo | 15:17 | |
| *** Marga_ has joined #tripleo | 15:20 | |
| *** masterbound is now known as mbound | 15:23 | |
| *** Marga_ has quit IRC | 15:25 | |
| shardy | gfidente: Yeah, my intention was to write a script or something to help with auto-proposing everything without No-Backport | 15:26 |
| shardy | but not got there yet due to delays getting CI working | 15:27 |
| dprince | shardy: wouldn't it be easier to use 'Backport' instead of 'No-Backport'? | 15:27 |
| shardy | dprince: we had that discussion in the spec, and the consensus was to prefer backporting by default | 15:28 |
| shardy | either is fine with me provided it's consistent and well understood | 15:28 |
| dprince | shardy: defaulting to backporting seems dangerous | 15:28 |
| shardy | dprince: at the moment everything is 100% manual, so it's really a non-issue | 15:28 |
| *** pblaho has quit IRC | 15:28 | |
| dmsimard | dprince: I've bumped into that neutron pbr version issue too (on RDO ci), have you found anything ? Ihar is on PTO until wednesday.. | 15:29 |
| dprince | shardy: sure, just saying if we get a job, I'd rather see it positive rather than nedative | 15:29 |
| shardy | it's not like anything would ever be reviewed or approved by default | 15:29 |
| shardy | dprince: ack, OK | 15:29 |
| shardy | that conflicts with what everyone said they wanted in the spec, but fine | 15:29 |
| dprince | dmsimard: is there a patch I can revert in openstack-packages/neutron? | 15:29 |
| dmsimard | dprince: I have no clue tbh, if there's one it would probably be https://github.com/openstack-packages/neutron/commit/3e81eef03980cf650d4f26b579621475e2df2b3d | 15:30 |
| dmsimard | dprince: in liberty that would be https://github.com/openstack-packages/neutron/commit/d016fc88d1cc6030f84cbced1bb32c643155572c | 15:30 |
| shardy | dprince: btw, can you pls sanity check https://review.openstack.org/#/c/245693/ (project-config) when you get a sec? | 15:31 |
| shardy | derekh isn't around, and I'd like confirmation before chasing for reviews | 15:32 |
| shardy | I think thats why the CI jobs aren't running on our stable branches yet | 15:32 |
| dprince | dmsimard: can you propose a revert for that referencing your bug? | 15:32 |
| dprince | dmsimard: or I can | 15:32 |
| dmsimard | dprince: I'll try and ask if any other neutron core can look into it before reverting that | 15:33 |
| dprince | dmsimard: perhaps our PBR dependency in RDO trunk isn't new enough now | 15:33 |
| dprince | dmsimard: regardless, RDO trunk is hosed | 15:34 |
| dprince | dmsimard: FWIW, if we can prove that patch breaks us I can land the revert | 15:34 |
| dprince | dmsimard: will try and test it in a bit... | 15:34 |
| dmsimard | dprince: Yeah I'm hoping we can start doing actual trunk CI sometime this week, there were some issues to address in liberty first and I didn't want to do the same fixes twice | 15:34 |
| dmsimard | dprince: sure, please report back | 15:34 |
| dmsimard | dprince: do you have a link to a failing job somewhere, I'll add that to the bugzilla | 15:37 |
| dmsimard | I have some eyes on the issue now | 15:37 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/diskimage-builder: Extend root device hints for different types of WWN https://review.openstack.org/245839 | 15:43 |
| *** dtantsur|brb is now known as dtantsur | 15:46 | |
| trown | dprince: want to +A the inspection CI patch? :) https://review.openstack.org/#/c/225934/ | 15:48 |
| *** mbound has quit IRC | 15:54 | |
| *** mbound has joined #tripleo | 15:55 | |
| *** akuznetsov has joined #tripleo | 15:58 | |
| *** untriaged-bot has joined #tripleo | 16:00 | |
| untriaged-bot | Untriaged bugs so far: | 16:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1505900 | 16:00 |
| openstack | Launchpad bug 1505900 in diskimage-builder "Failed to create 'wily' release with DIB_RELEASE" [Undecided,New] | 16:00 |
| *** untriaged-bot has quit IRC | 16:00 | |
| *** pblaho has joined #tripleo | 16:03 | |
| *** tiswanso has quit IRC | 16:03 | |
| *** tiswanso has joined #tripleo | 16:03 | |
| *** mkovacik has joined #tripleo | 16:05 | |
| *** pblaho has quit IRC | 16:07 | |
| *** masterbound has joined #tripleo | 16:09 | |
| *** pblaho has joined #tripleo | 16:09 | |
| *** mbound has quit IRC | 16:11 | |
| *** absubram has joined #tripleo | 16:18 | |
| *** Marga_ has joined #tripleo | 16:21 | |
| *** wendar_ is now known as wendar | 16:26 | |
| *** Marga_ has quit IRC | 16:26 | |
| *** jaosorior has quit IRC | 16:31 | |
| *** jaosorior has joined #tripleo | 16:31 | |
| openstackgerrit | Jaume Devesa proposed openstack/tripleo-puppet-elements: Adding networking MidoNet element https://review.openstack.org/232551 | 16:31 |
| *** akuznetsov has quit IRC | 16:34 | |
| *** noslzzp has quit IRC | 16:34 | |
| *** jerrygb has quit IRC | 16:39 | |
| *** alop has joined #tripleo | 16:46 | |
| *** jerrygb has joined #tripleo | 16:47 | |
| *** sthillma has joined #tripleo | 16:48 | |
| *** mcornea has quit IRC | 16:55 | |
| *** mcornea has joined #tripleo | 16:56 | |
| *** noslzzp has joined #tripleo | 16:58 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Update tripleoclient to use the common deploy logic https://review.openstack.org/237920 | 16:59 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-common: Update gitreview on stable/liberty https://review.openstack.org/244643 | 17:02 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-puppet-elements: Update gitreview on stable/liberty https://review.openstack.org/244644 | 17:03 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-puppet-elements: Add python-memcached to list of installed pkgs on controllers https://review.openstack.org/245686 | 17:03 |
| bnemec | gfidente: What I was saying about the external lb patch is that I don't think the fixed ips version is compatible with the existing one. | 17:05 |
| bnemec | gfidente: In the fixed ips implementation you have to put the ips inside the allocation pool for Neutron. | 17:05 |
| *** panda has joined #tripleo | 17:05 | |
| bnemec | In the current one you have to keep them outside it. | 17:05 |
| gfidente | bnemec, yeah not 'compatible' | 17:05 |
| bnemec | Also the syntax for specifying the ips would have to change. | 17:05 |
| gfidente | I was thinking we can have both | 17:06 |
| bnemec | Both would be issues for upgrades between the two different methods, so I'd rather not try to switch post-merge. | 17:06 |
| gfidente | agreed, switch amongst the two is probably not trivial | 17:06 |
| bnemec | gfidente: I'd rather just eat the technical debt that I think we'd accumulate with the current implementation than have two separate versions of the same feature. | 17:07 |
| gfidente | I was thinking we can have a different environment file with appropriate _defaults: for the fixed ips version if we wanted | 17:07 |
| bnemec | Technical debt has become my fifth food group lately anyway. | 17:07 |
| gfidente | that is really about extracting the IP from IPPool in controller.yaml though | 17:08 |
| gfidente | so the submissions for FixedIPs should be valid 'standalone' | 17:08 |
| gfidente | should we decide to try the FixedIPs thing for external lb, we can always do it | 17:08 |
| gfidente | makes sense? | 17:08 |
| bnemec | gfidente: Yeah, but I'd rather not do it at all than do it in addition to the current thing. My whole argument here is that it makes the templates really hard to read, and layering more on top of it isn't going to help. | 17:09 |
| *** lblanchard has quit IRC | 17:10 | |
| gfidente | ok so to start with I'll rebase the existing changes | 17:11 |
| *** tosky_ has joined #tripleo | 17:13 | |
| *** davidlenwell_ is now known as davidlenwell | 17:15 | |
| *** ChanServ sets mode: +v davidlenwell | 17:15 | |
| *** tosky has quit IRC | 17:16 | |
| jistr | gfidente, slagle_, shardy: just a heads up for now -- i'm seeing the same result as slagle_ when testing https://review.openstack.org/#/c/245093 -- the Pre/PostUpdateDeployment is CREATE_COMPLETE for all 3 servers but seems that nothing happened | 17:18 |
| jistr | and it's not even to be found in /var/run/heat-config/deployed | 17:19 |
| gfidente | jistr, /me cherry-picking | 17:19 |
| dmsimard | dprince: found the root cause for the pbr version problem, there's part of pbr that replaces non-alphanumeric characters by dashes - hence why the + could be interpreted as a - | 17:20 |
| *** pblaho has quit IRC | 17:20 | |
| slagle_ | jistr: yea i think it's prettly clear those deployments aren't getting triggered on stack-update if they get created the first time | 17:21 |
| slagle_ | which, actually kind of makes sense | 17:22 |
| slagle_ | i guess :) | 17:22 |
| *** Marga_ has joined #tripleo | 17:22 | |
| jistr | kind of, yeah :) though it would also kind of make sense if it referred to stack update/create instead of resource update/create | 17:24 |
| *** bvandenh has joined #tripleo | 17:24 | |
| gfidente | slagle_, you mean if they *didn | 17:26 |
| gfidente | 't* get created? | 17:26 |
| slagle_ | gfidente: no | 17:26 |
| openstackgerrit | Mike Burns proposed openstack/tripleo-heat-templates: change the default satellite tools rpm repo. https://review.openstack.org/245895 | 17:27 |
| marios | slagle_: thanks for the notes | 17:27 |
| jistr | good news is i got another successful stack update, using the constraints + timeouts patches and doing a manual maintenance mode after discovering that the automation didn't work on that | 17:27 |
| *** tosky_ is now known as tosky | 17:27 | |
| *** masterbound has quit IRC | 17:27 | |
| *** Marga_ has quit IRC | 17:27 | |
| *** Guest91043 is now known as mgagne | 17:28 | |
| *** mgagne has joined #tripleo | 17:28 | |
| gfidente | slagle_, my expectations would be that ControllerPostUpdateDeployment is not deployed on the first update attempt because that is when it is created | 17:29 |
| gfidente | isn't that the case? | 17:29 |
| *** mgagne is now known as Guest91043 | 17:29 | |
| *** Guest91043 is now known as mgagne | 17:29 | |
| *** dmsimard is now known as dmsimard|food | 17:29 | |
| dmsimard|food | dprince: https://bugzilla.redhat.com/show_bug.cgi?id=1281920#c4 | 17:31 |
| openstack | bugzilla.redhat.com bug 1281920 in openstack-neutron "neutron-server will not start: fails with pbr version issue" [High,New] - Assigned to ihrachys | 17:31 |
| jistr | gfidente: yeah it is the case, though your assumption is different than mine and shardy's assumption was :) | 17:31 |
| *** paramite has quit IRC | 17:31 | |
| jistr | which means... this is still a problem | 17:31 |
| *** bvandenh has quit IRC | 17:31 | |
| gfidente | jistr, ok that is because it works on a per-resource not per-stack | 17:31 |
| jistr | yeah, i thought it would be per-stack | 17:32 |
| gfidente | so | 17:32 |
| jistr | anyways | 17:32 |
| gfidente | is that really a problem to have pcmk in maintenance mode during initial deployment? | 17:32 |
| jistr | gfidente: yeah i think so, i think it wouldn't perform service starts etc., no? | 17:32 |
| gfidente | right :P | 17:33 |
| *** lazy_prince has quit IRC | 17:33 | |
| slagle_ | also, pacemaker isn't even up on initial deployment | 17:33 |
| *** slagle_ is now known as slagle | 17:33 | |
| jistr | gfidente: yeah, we need the step separation e.g. (n) galera up, (n+1) db syncs, (n+2) service startup | 17:33 |
| slagle | so the command would probably fail | 17:34 |
| gfidente | so the thing is that it would only work for new deployments as-is | 17:34 |
| jistr | slagle: yeah that too | 17:34 |
| *** bvandenh has joined #tripleo | 17:34 | |
| slagle | sounds like another use case for heat template conditionals :) | 17:35 |
| *** bvandenh has quit IRC | 17:36 | |
| *** athomas has quit IRC | 17:36 | |
| *** sthillma has quit IRC | 17:36 | |
| jistr | heh yeah | 17:36 |
| gfidente | slagle, could use the command failure to discriminate then? | 17:37 |
| *** mcornea has quit IRC | 17:37 | |
| openstackgerrit | Jaume Devesa proposed openstack/tripleo-heat-templates: Add simple linux bridge as network environment https://review.openstack.org/243531 | 17:37 |
| openstackgerrit | Jaume Devesa proposed openstack/tripleo-heat-templates: MidoNet heat templates https://review.openstack.org/238998 | 17:37 |
| gfidente | slagle, jistr something like we check if a tripleo cluster exists and if so, we run maintenance | 17:37 |
| jistr | yeah actually those two errors would cancel out each other | 17:37 |
| slagle | gfidente: i suppose, have the script do the same systemctl is-active pacemaker that yum_update.sh does | 17:38 |
| jistr | hehe | 17:38 |
| slagle | and then have the conditional in the script that way | 17:38 |
| gfidente | and that, removing 'UPDATE" should do the job | 17:38 |
| gfidente | for both new deployments and old ones no? | 17:38 |
| jistr | the downside of this is that we're getting rid of the 'pre-update' semantics of those scripts | 17:39 |
| jistr | for our specific pacemaker hack | 17:39 |
| slagle | yes | 17:39 |
| slagle | it should be just pre-puppet-run | 17:40 |
| gfidente | so if the is-active thing reports fail, we dont signal failure, but just carry on | 17:40 |
| slagle | right | 17:42 |
| *** chmouel_ is now known as chmouel | 17:42 | |
| *** rcernin has quit IRC | 17:42 | |
| gfidente | it's not great but /me doesn't have other ideas | 17:42 |
| slagle | so we already have the ControllerExtraConfigPre | 17:43 |
| *** akuznetsov has joined #tripleo | 17:43 | |
| jistr | yeah... however using those would collide with a lot of other use cases | 17:44 |
| slagle | true | 17:44 |
| *** ifarkas has quit IRC | 17:45 | |
| slagle | i guess we need a Step-1 then | 17:48 |
| slagle | that's "Step Negative 1" if it's not obvious :) | 17:48 |
| slagle | or we have another resource right before *ExtraConfigPre which is reserved for tht internal use | 17:49 |
| slagle | we had this same problem with the userdata, and shardy came up with the solution to use OS::Heat::MultipartMime solution, but i can't think of any equivalent for SoftwareDeployments | 17:50 |
| jistr | i kinda like the resource reserved for tht internal use | 17:51 |
| *** ukalifon1 has quit IRC | 17:51 | |
| jistr | i think it can be done to have multiple software configs created https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/extraconfig/pre_deploy/controller/multiple.yaml | 17:52 |
| jistr | but still | 17:52 |
| gfidente | well, we could add a couple of steps which are implemented by empty yamls by default | 17:52 |
| gfidente | so they don't prevent use of Pre and Post | 17:53 |
| jistr | anyone who used ExtraConfigPre for something would need to produce their own template like that ^ to be able to update their cloud | 17:53 |
| *** electrofelix has quit IRC | 17:53 | |
| openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests https://review.openstack.org/244184 | 17:55 |
| jistr | so what about ControllerPrePuppetConfig / ControllerPostPuppetConfig | 17:56 |
| gfidente | yeah isn't that what they are meant to do? | 17:56 |
| jistr | or maybe we can put it into the ::Tasks:: namespace to make it more apparent that it's not an expected integration point | 17:58 |
| jistr | OS::TripleO::Tasks::ControllerPrePuppet | 17:59 |
| jistr | OS::TripleO::Tasks::ControllerPostPuppet | 17:59 |
| slagle | hmm, i don't think Tasks implies that :) some will think it implies the opposite. i can run tasks! | 17:59 |
| slagle | we could just name it for what it does ControllerPrePuppetPacemakerMaintenanceMode | 18:00 |
| slagle | or something shorter :) | 18:00 |
| *** dtantsur is now known as dtantsur|afk | 18:00 | |
| gfidente | I like the ControllerPrePuppetConfig thing | 18:00 |
| gfidente | one can change what that is meant to do from registry | 18:00 |
| gfidente | we use it for pcmk in pcmk scenario, but who knows what it will be useful for in different scenarios? :) | 18:01 |
| *** tosky has quit IRC | 18:07 | |
| *** trown is now known as trown|lunch | 18:09 | |
| *** leanderthal has quit IRC | 18:09 | |
| *** sthillma has joined #tripleo | 18:11 | |
| *** rcernin has joined #tripleo | 18:11 | |
| jistr | well we already use ::Tasks:: for things which cannot be used like "i can run tasks!" anyway i'm fine naming it whatever, but already started working on the tasks bit, i'll submit that one for now and if we want to shuffle it a bit, that's no problem | 18:12 |
| *** olap has joined #tripleo | 18:14 | |
| *** lucasagomes is now known as lucas-dinner | 18:15 | |
| *** mgould has quit IRC | 18:19 | |
| *** Goneri has quit IRC | 18:21 | |
| *** leanderthal has joined #tripleo | 18:22 | |
| *** penick has joined #tripleo | 18:23 | |
| *** Marga_ has joined #tripleo | 18:23 | |
| *** akuznetsov has quit IRC | 18:25 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 18:25 |
| *** Marga__ has joined #tripleo | 18:26 | |
| *** Marga__ has quit IRC | 18:27 | |
| gfidente | jistr, /me likes | 18:27 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 18:27 |
| *** Marga_ has quit IRC | 18:27 | |
| *** lucas-dinner has quit IRC | 18:27 | |
| *** Marga_ has joined #tripleo | 18:28 | |
| jistr | gfidente: cool :) as i said i'm fine re-shuffling if desired | 18:28 |
| jistr | i didn't test it e2e in a deployment yet | 18:28 |
| jistr | but tested the script snippets by pasting them to a bash session on controller-0 and they did what they should do | 18:28 |
| jistr | o/ | 18:29 |
| *** jistr has quit IRC | 18:29 | |
| *** penick has quit IRC | 18:30 | |
| *** penick has joined #tripleo | 18:32 | |
| *** shivrao has joined #tripleo | 18:33 | |
| openstackgerrit | Jaume Devesa proposed openstack/tripleo-heat-templates: Add simple linux bridge as network environment https://review.openstack.org/243531 | 18:34 |
| *** lucasagomes has joined #tripleo | 18:35 | |
| *** devvesa has quit IRC | 18:37 | |
| *** Goneri has joined #tripleo | 18:37 | |
| *** akuznetsov has joined #tripleo | 18:39 | |
| gfidente | rhallisey, why NovaImage had to be in parameters: instead of parameter_defaults: ? | 18:51 |
| rhallisey | gfidente, good question. I think it could go in parameter_defaults | 18:52 |
| gfidente | I'd like to standardize on that as well, as per shardy email | 18:53 |
| gfidente | that's why I asked | 18:53 |
| rhallisey | standardize on using parameter_defaults in this example? | 18:53 |
| rhallisey | did not see shardy's email | 18:54 |
| gfidente | standardize on when we use parameters: and when _defaults: across all the envs | 18:54 |
| *** akuznetsov has quit IRC | 18:54 | |
| rhallisey | gotcha | 18:54 |
| dprince | if there is already a top level parameter for NovaImage I see no reason not to use it | 18:55 |
| gfidente | dprince, there is already a parameter yes, I was asking why we control it via parameters: instead of parameter_defaults: in the env file | 18:57 |
| dprince | gfidente: if you want to allow it to be overridden then parameter_defaults maybe. Otherwise I'd use parameters | 18:58 |
| dprince | which is still overridable, if another environment were to set that same parameter and come later in the -e environments list | 18:58 |
| gfidente | dprince yeah but in general parameter_defaults seems safer to use because we can move the params around if we need it ... like specific roles or to cleanup the top-level yaml | 18:59 |
| gfidente | and that won't be cause of troubles into the custom env files users might be using | 18:59 |
| gfidente | so I was thinking we could try to use always parameter_defaults: in the envs | 19:00 |
| gfidente | even if the param is top-level | 19:00 |
| *** penick has quit IRC | 19:00 | |
| dprince | gfidente: that works for newer heat | 19:00 |
| dprince | gfidente: but old heat versions I think had issues with that approach | 19:01 |
| *** akuznetsov has joined #tripleo | 19:01 | |
| *** trown|lunch is now known as trown | 19:01 | |
| dprince | gfidente: personally, if there is a top level param I use it | 19:01 |
| dprince | gfidente: and I'm leaning towards using a lot more top level params w/ the role stuff. But only for generic Openstack services things | 19:02 |
| gfidente | I think the problem with parameter_defaults not applying to top-level params applied only to some param types | 19:02 |
| dprince | gfidente: nothing, puppet related for example should be top level | 19:02 |
| gfidente | dprince, but what is the benefit of using parameters: ? | 19:02 |
| gfidente | dprince, I like that parameter_defaults: would work wherever we put the param | 19:03 |
| dprince | gfidente: I will comment on the thread | 19:03 |
| *** penick has joined #tripleo | 19:03 | |
| dprince | gfidente: but when I added parameter defaults for the network isolation stuff I always wanted a development time check on the descriptions being the same | 19:03 |
| gfidente | dprince, ack, me replying as well, the thread is on a slightly different topic though | 19:03 |
| dprince | gfidente: right now any parameter that is duplicated in name would be a "risk" | 19:03 |
| dprince | gfidente: while it may seem convient... it is a bit risky too | 19:04 |
| dprince | convienient | 19:04 |
| *** penick has quit IRC | 19:05 | |
| *** penick has joined #tripleo | 19:14 | |
| gfidente | bnemec, I think the l3/arp changes, I wanted to test those first though | 19:17 |
| gfidente | *I think are safe | 19:17 |
| bnemec | gfidente: ??? | 19:17 |
| gfidente | test update | 19:18 |
| gfidente | I just saw your comment on https://review.openstack.org/244723 | 19:18 |
| bnemec | gfidente: That changes nothing in the case of an existing deployment that doesn't set the param. The puppet module already defaulted the value he's setting to false. | 19:19 |
| bnemec | It's possible that if you change the value after deployment then all hell breaks loose, but I think we have a bunch of params like that already. :-) | 19:20 |
| gfidente | bnemec, sure not going to contemplate the case where you change its value during update, we had the same issue for rabbitmq user/pass | 19:20 |
| openstackgerrit | shravya Gaddam proposed openstack/tripleo-heat-templates: Typo in NeutronMechanismDrivers in heat templates https://review.openstack.org/245935 | 19:22 |
| *** akuznetsov has quit IRC | 19:23 | |
| *** dmsimard|food is now known as dmsimard | 19:31 | |
| *** Goneri has quit IRC | 19:38 | |
| openstackgerrit | Rhys Oxenham proposed openstack/tripleo-heat-templates: Add network templates for multiple NICs with flat and VLAN tagged networks https://review.openstack.org/214372 | 19:42 |
| *** noslzzp has quit IRC | 19:42 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add net_vip_map_external to be used for an external balancer https://review.openstack.org/222189 | 19:44 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Make load balancer deployment optional via template param https://review.openstack.org/222305 | 19:44 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of pre-defined IPs https://review.openstack.org/241606 | 19:44 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the controller nodes https://review.openstack.org/222497 | 19:44 |
| dmsimard | dprince: they're rolling back the last commit | 19:51 |
| dmsimard | dprince: it'll be built soon | 19:51 |
| dprince | dmsimard: awesome, I think that'll fix us | 19:51 |
| dmsimard | dprince: the fix might be in for mitaka but I'm not sure yet - liberty definitely not in yet | 19:52 |
| *** Goneri has joined #tripleo | 19:54 | |
| *** lblanchard has joined #tripleo | 19:55 | |
| *** dprince_ has joined #tripleo | 19:57 | |
| *** dprince has quit IRC | 19:57 | |
| *** spredzy_ has joined #tripleo | 19:57 | |
| *** spredzy has quit IRC | 19:58 | |
| *** shadower has quit IRC | 19:58 | |
| *** rcernin has quit IRC | 19:58 | |
| *** penick has quit IRC | 20:02 | |
| *** gfidente has quit IRC | 20:02 | |
| *** sthillma has quit IRC | 20:04 | |
| *** penick has joined #tripleo | 20:06 | |
| *** olaph has quit IRC | 20:07 | |
| *** olaph has joined #tripleo | 20:10 | |
| *** Marga_ has quit IRC | 20:12 | |
| *** akuznetsov has joined #tripleo | 20:13 | |
| *** akuznetsov has quit IRC | 20:17 | |
| *** liverpooler has quit IRC | 20:23 | |
| *** lucasagomes is now known as lucas-afk | 20:23 | |
| dprince_ | thrash: you okay w/ this one https://review.openstack.org/#/c/244350/ | 20:37 |
| *** mbound has joined #tripleo | 20:37 | |
| thrash | dprince_: lgtm | 20:41 |
| openstackgerrit | Merged openstack/python-tripleoclient: Generate a unique DeployIdentifier on updates https://review.openstack.org/244350 | 20:46 |
| *** dsneddon is now known as dsneddon_biab | 20:46 | |
| *** jaosorior has quit IRC | 20:50 | |
| rhefner | marios: can we try to get +A on this tripleo-puppet-elements patch: https://review.openstack.org/#/c/244582/ to see if that fixes out CI issues on 188137? | 20:50 |
| dprince_ | rhefner: sent | 20:54 |
| rhefner | dprince_: thanks Dan! | 20:55 |
| *** yamahata has joined #tripleo | 20:55 | |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Adds puppet-manila to install from source list https://review.openstack.org/244582 | 21:02 |
| *** panda has quit IRC | 21:04 | |
| *** panda has joined #tripleo | 21:05 | |
| *** dsneddon_biab is now known as dsneddon | 21:07 | |
| *** noslzzp has joined #tripleo | 21:10 | |
| *** jprovazn has quit IRC | 21:12 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Log all resource events during overcloud deploy https://review.openstack.org/235032 | 21:13 |
| *** akrivoka has quit IRC | 21:16 | |
| *** sthillma has joined #tripleo | 21:20 | |
| *** olap has quit IRC | 21:28 | |
| *** shardy has quit IRC | 21:33 | |
| *** jayg is now known as jayg|g0n3 | 21:44 | |
| *** Marga_ has joined #tripleo | 21:52 | |
| *** dprince_ has quit IRC | 21:53 | |
| *** untriaged-bot has joined #tripleo | 22:00 | |
| untriaged-bot | Untriaged bugs so far: | 22:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1505900 | 22:00 |
| openstack | Launchpad bug 1505900 in diskimage-builder "Failed to create 'wily' release with DIB_RELEASE" [Undecided,New] | 22:00 |
| *** untriaged-bot has quit IRC | 22:00 | |
| *** tiswanso has quit IRC | 22:00 | |
| *** jeff_phillips has quit IRC | 22:00 | |
| *** rhallisey has quit IRC | 22:03 | |
| *** trown is now known as trown|outttypeww | 22:12 | |
| *** bvandenh has joined #tripleo | 22:20 | |
| *** julim has quit IRC | 22:22 | |
| *** yamahata has quit IRC | 22:27 | |
| *** bvandenh has quit IRC | 22:28 | |
| *** rpothier has quit IRC | 22:29 | |
| openstackgerrit | Lokesh Jain proposed openstack/tripleo-heat-templates: Changes for configuring Nuage https://review.openstack.org/230116 | 22:39 |
| *** lblanchard has quit IRC | 22:39 | |
| openstackgerrit | Rohit Pagedar proposed openstack/tripleo-heat-templates: Making nova parameters configurable https://review.openstack.org/234965 | 22:40 |
| *** Goneri has quit IRC | 22:40 | |
| *** hewbrocc` has joined #tripleo | 22:53 | |
| *** greghaynes_ has quit IRC | 22:53 | |
| *** rbrady has quit IRC | 22:54 | |
| *** hewbrocca has quit IRC | 22:54 | |
| *** cmyster has quit IRC | 22:54 | |
| *** pleia2 has quit IRC | 22:54 | |
| *** Cameron_C_ has quit IRC | 22:54 | |
| *** thrash has quit IRC | 22:54 | |
| *** cmyster has joined #tripleo | 22:54 | |
| *** cmyster has quit IRC | 22:54 | |
| *** cmyster has joined #tripleo | 22:54 | |
| *** rbrady has joined #tripleo | 22:54 | |
| *** morazi has quit IRC | 22:54 | |
| *** therve has quit IRC | 22:54 | |
| *** pleia2 has joined #tripleo | 22:55 | |
| *** ryansb_ has joined #tripleo | 22:58 | |
| *** ryansb_ has joined #tripleo | 22:58 | |
| *** thegodfather has joined #tripleo | 22:59 | |
| *** therve has joined #tripleo | 22:59 | |
| *** thrash has joined #tripleo | 23:01 | |
| *** thrash has joined #tripleo | 23:01 | |
| *** beekneemech has joined #tripleo | 23:05 | |
| *** spredzy has joined #tripleo | 23:06 | |
| *** spredzy_ has quit IRC | 23:06 | |
| *** lucas-afk has quit IRC | 23:06 | |
| *** pradk has quit IRC | 23:06 | |
| *** tchaypo has quit IRC | 23:06 | |
| *** ryansb has quit IRC | 23:06 | |
| *** zaneb has quit IRC | 23:06 | |
| *** fabbione has quit IRC | 23:06 | |
| *** bnemec has quit IRC | 23:06 | |
| *** mburned has quit IRC | 23:06 | |
| *** lifeless has quit IRC | 23:06 | |
| *** dobson has quit IRC | 23:06 | |
| *** ryansb_ is now known as ryansb | 23:06 | |
| *** dobson has joined #tripleo | 23:07 | |
| *** morazi has joined #tripleo | 23:07 | |
| *** greghaynes_ has joined #tripleo | 23:08 | |
| *** lifeless has joined #tripleo | 23:09 | |
| *** lucasagomes has joined #tripleo | 23:09 | |
| *** lucasagomes is now known as lucas-afk | 23:11 | |
| *** zaneb has joined #tripleo | 23:12 | |
| *** tchaypo has joined #tripleo | 23:13 | |
| *** tchaypo is now known as Guest83847 | 23:13 | |
| openstackgerrit | Rohit Pagedar proposed openstack/tripleo-heat-templates: Changes for configuring Nuage https://review.openstack.org/234970 | 23:16 |
| *** rwsu has quit IRC | 23:22 | |
| *** mburned has joined #tripleo | 23:23 | |
| openstackgerrit | Rohit Pagedar proposed openstack/tripleo-heat-templates: Changes for configuring Nuage https://review.openstack.org/234970 | 23:25 |
| *** alop has quit IRC | 23:29 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function https://review.openstack.org/245643 | 23:32 |
| *** absubram has quit IRC | 23:34 | |
| *** Marga__ has joined #tripleo | 23:37 | |
| *** Marga_ has quit IRC | 23:39 | |
| *** shivrao has quit IRC | 23:39 | |
| *** shivrao has joined #tripleo | 23:43 | |
| *** thrash is now known as thrash|g0ne | 23:45 | |
| *** cwolferh has quit IRC | 23:56 | |
| *** Guest83847 is now known as tchaypo | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!