| *** mlupton has joined #tripleo | 00:01 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add support for configuring the OVS firewall driver https://review.openstack.org/357556 | 00:04 |
|---|---|---|
| *** mlupton has quit IRC | 00:10 | |
| *** nyechiel has quit IRC | 00:10 | |
| *** mlupton has joined #tripleo | 00:11 | |
| *** bfournie has joined #tripleo | 00:16 | |
| *** mlupton has quit IRC | 00:19 | |
| *** mlupton has joined #tripleo | 00:20 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv ceilo, aodh, and gnocchi out of controller.yaml https://review.openstack.org/360536 | 00:27 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move Swift hiera settings into composable services https://review.openstack.org/360058 | 00:27 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv keystone, mongo, manila out of controller.yaml https://review.openstack.org/360060 | 00:27 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add constraint to prohibit balance-tcp from BondInterfaceOvsOptions https://review.openstack.org/355073 | 00:27 |
| *** jistr has quit IRC | 00:27 | |
| *** mlupton has quit IRC | 00:28 | |
| *** jistr has joined #tripleo | 00:30 | |
| *** mlupton has joined #tripleo | 00:31 | |
| *** mlupton has quit IRC | 00:39 | |
| *** mlupton has joined #tripleo | 00:41 | |
| *** bana_k has quit IRC | 00:41 | |
| *** limao has joined #tripleo | 00:42 | |
| *** mlupton has quit IRC | 00:51 | |
| *** rajinir has quit IRC | 00:55 | |
| *** mburned_out is now known as mburned | 00:58 | |
| *** mburned is now known as mburned_out | 00:58 | |
| *** mlupton has joined #tripleo | 00:59 | |
| *** bana_k has joined #tripleo | 00:59 | |
| *** bana_k has quit IRC | 01:01 | |
| trozet | bnemec, slagle: could you please be a 2nd reviewer to https://review.openstack.org/#/c/355147/, https://review.openstack.org/#/c/335173/11 | 01:02 |
| *** tzumainn has quit IRC | 01:05 | |
| *** yamahata has quit IRC | 01:13 | |
| *** mlupton has quit IRC | 01:13 | |
| *** mlupton has joined #tripleo | 01:18 | |
| *** chlong has joined #tripleo | 01:18 | |
| *** yamahata has joined #tripleo | 01:27 | |
| *** mlupton has quit IRC | 01:31 | |
| *** mlupton has joined #tripleo | 01:34 | |
| *** mlupton has quit IRC | 01:46 | |
| *** fultonj has quit IRC | 01:46 | |
| *** nyechiel has joined #tripleo | 01:49 | |
| *** thrash is now known as thrash|g0ne | 01:50 | |
| *** vijaykc4 has joined #tripleo | 01:57 | |
| openstackgerrit | Merged openstack/tripleo-common: Verify that neither the Swift container or Mistral environment exist https://review.openstack.org/357741 | 02:05 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: More cleanup of controller.yaml https://review.openstack.org/360751 | 02:08 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Fix use of snmp service https://review.openstack.org/360062 | 02:08 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Move Redis, Memcached, Sahara, OVS agent out of role templates https://review.openstack.org/360697 | 02:08 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/tripleo-heat-templates: Add fluentd client service https://review.openstack.org/353506 | 02:10 |
| *** nyechiel has quit IRC | 02:19 | |
| *** vijaykc4 has quit IRC | 02:34 | |
| *** vijaykc4 has joined #tripleo | 02:39 | |
| *** vijaykc4 has quit IRC | 02:40 | |
| *** rkolli has quit IRC | 02:40 | |
| *** yamahata has quit IRC | 02:43 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Move Swift hiera settings into composable services https://review.openstack.org/360058 | 02:43 |
| *** StevenK has quit IRC | 02:48 | |
| *** onovy has quit IRC | 02:48 | |
| *** onovy has joined #tripleo | 02:48 | |
| *** StevenK has joined #tripleo | 02:49 | |
| *** bana_k has joined #tripleo | 02:52 | |
| *** vijaykc4 has joined #tripleo | 03:00 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Mv keystone, mongo, manila out of controller.yaml https://review.openstack.org/360060 | 03:12 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Mv ceilo, aodh, and gnocchi out of controller.yaml https://review.openstack.org/360536 | 03:12 |
| *** Ryjedo_ has joined #tripleo | 03:29 | |
| *** Ryjedo has quit IRC | 03:29 | |
| *** Ryjedo_ is now known as Ryjedo | 03:29 | |
| *** numans has joined #tripleo | 03:34 | |
| *** numans has quit IRC | 03:35 | |
| *** numans has joined #tripleo | 03:36 | |
| *** chlong has quit IRC | 03:37 | |
| *** gfidente has quit IRC | 03:40 | |
| *** dmacpher has joined #tripleo | 03:51 | |
| *** dmacpher has quit IRC | 03:56 | |
| *** dmacpher has joined #tripleo | 03:58 | |
| *** bana_k has quit IRC | 04:01 | |
| *** vijaykc4 has quit IRC | 04:02 | |
| *** links has joined #tripleo | 04:07 | |
| *** rlandy|bbl is now known as rlandy | 04:07 | |
| *** chlong has joined #tripleo | 04:09 | |
| *** yamahata has joined #tripleo | 04:13 | |
| *** apetrich has quit IRC | 04:25 | |
| *** apetrich has joined #tripleo | 04:25 | |
| *** ebalduf has quit IRC | 04:25 | |
| *** jaosorior has joined #tripleo | 04:46 | |
| *** yamahata has quit IRC | 05:01 | |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Availability monitoring agents support https://review.openstack.org/254788 | 05:06 |
| jaosorior | bandini, ccamacho: Can you guys take a look at this? https://review.openstack.org/#/c/356416/ | 05:12 |
| *** skramaja has joined #tripleo | 05:13 | |
| *** yamahata has joined #tripleo | 05:14 | |
| *** chlong has quit IRC | 05:21 | |
| *** radeks has joined #tripleo | 05:46 | |
| *** akuznetsov has joined #tripleo | 05:48 | |
| *** vijaykc4 has joined #tripleo | 05:56 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-quickstart: newton/master add ssl option as default to True for the undercloud https://review.openstack.org/356431 | 05:58 |
| *** pgadiya has joined #tripleo | 06:04 | |
| *** eggmaster has quit IRC | 06:05 | |
| *** matbu|afk is now known as matbu | 06:07 | |
| *** akuznetsov has quit IRC | 06:08 | |
| *** numans has quit IRC | 06:08 | |
| *** fragatin_ has joined #tripleo | 06:12 | |
| *** fragatina has quit IRC | 06:15 | |
| *** fragatin_ has quit IRC | 06:17 | |
| *** milan has joined #tripleo | 06:20 | |
| *** bana_k has joined #tripleo | 06:20 | |
| *** masco has joined #tripleo | 06:21 | |
| *** shardy has joined #tripleo | 06:23 | |
| *** abehl has joined #tripleo | 06:24 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Customize scheduler_default_filters (nova.conf) https://review.openstack.org/337264 | 06:26 |
| *** rcernin has joined #tripleo | 06:26 | |
| *** fragatina has joined #tripleo | 06:36 | |
| ccamacho | jaosorior, last vote before going PTO :P see ya! | 06:36 |
| ccamacho | BTW a really good idea there #/356416 | 06:37 |
| jaosorior | ccamacho: aaah, didn't you you were gonna have PTO | 06:39 |
| jaosorior | ccamacho: enjoy dude! | 06:39 |
| ccamacho | thanks man! Should come back on Sept 7 (Now packing for going to the beach finally!) | 06:40 |
| ccamacho | :) | 06:40 |
| jaosorior | niiiiice | 06:40 |
| *** fragatina has quit IRC | 06:41 | |
| *** tesseract- has joined #tripleo | 06:44 | |
| *** paramite has joined #tripleo | 06:46 | |
| *** tobias-fiberdata has joined #tripleo | 06:48 | |
| *** bana_k has quit IRC | 06:48 | |
| *** aufi has joined #tripleo | 06:49 | |
| *** rasca has joined #tripleo | 06:50 | |
| jaosorior | shardy: Hey dude, did you figure out a solution to the service_enabled flag thingy? | 06:53 |
| *** jubapa has joined #tripleo | 06:55 | |
| *** tobias_fiberdata has joined #tripleo | 06:57 | |
| *** vijaykc4 has quit IRC | 06:57 | |
| *** ccamacho has quit IRC | 06:58 | |
| *** vijaykc4 has joined #tripleo | 06:58 | |
| *** tobias-fiberdata has quit IRC | 07:00 | |
| shardy | jaosorior: Hey, yes I did but I hit a heat bug, which I've now worked around, testing the patch now | 07:01 |
| *** tremble has joined #tripleo | 07:02 | |
| *** tdasilva has quit IRC | 07:04 | |
| jaosorior | shardy: daaamn dude, what was the bug about? | 07:08 |
| *** mhenkel has joined #tripleo | 07:09 | |
| jaosorior | shardy: when you get time, can you take a look at this? https://review.openstack.org/#/c/356416/ | 07:11 |
| *** hewbrocca-afk is now known as hewbrocca | 07:16 | |
| shardy | jaosorior: https://bugs.launchpad.net/heat/+bug/1617019 | 07:20 |
| openstack | Launchpad bug 1617019 in heat "comma_delimited_list validated too eagerly, breaks with list_join" [High,Triaged] | 07:20 |
| shardy | jaosorior: will do, when I get done with this patch | 07:21 |
| * shardy just made coffee so may work faster now :) | 07:21 | |
| jaosorior | hahaha nice | 07:22 |
| jaosorior | shardy: sure man, no biggie | 07:22 |
| *** jaosorior is now known as jaosorior_brb | 07:22 | |
| *** dmacpher has quit IRC | 07:25 | |
| *** r-mibu has quit IRC | 07:27 | |
| *** vijaykc4 has quit IRC | 07:27 | |
| jaosorior_brb | shardy: hey dude, one thing, for third party integration. If there's a package needed what's the policy? do we still includ it to in the overcloud-full image? or do deployers have to deal with getting the package there? | 07:27 |
| *** r-mibu has joined #tripleo | 07:27 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Set parameters https://review.openstack.org/360391 | 07:28 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command https://review.openstack.org/359784 | 07:28 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Add `openstack overcloud deploy --plan` https://review.openstack.org/360305 | 07:28 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 07:28 |
| shardy | jaosorior_brb: I think in general we suggest folks virt-customize the overcloud-full image | 07:28 |
| jaosorior_brb | shardy: but I guess that's not viable for customers using OSP, right? | 07:29 |
| *** florianf has joined #tripleo | 07:30 | |
| jaosorior_brb | hewbrocca: ^^ | 07:30 |
| *** akuznetsov has joined #tripleo | 07:31 | |
| d0ugal | shardy: IIRC, you had requested the Mistral execution ID's, what do you think about the -1 here? https://review.openstack.org/#/c/340711/ | 07:31 |
| d0ugal | (requested that they be displayed) | 07:31 |
| hewbrocca | jaosorior_brb: yeah, it's our supported solution for adding third-party packages to an image | 07:35 |
| shardy | d0ugal: I would prefer them to be a print, and I don't think exposing the ID is any more weird than printing the stack ID like we used to do for overcloud deploys direct via heat | 07:36 |
| hewbrocca | Hey did we get the nodepool vs. RH1 FIP thing sorted out last night, anybody know? | 07:36 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Set parameters https://review.openstack.org/360391 | 07:36 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command https://review.openstack.org/359784 | 07:36 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Add `openstack overcloud deploy --plan` https://review.openstack.org/360305 | 07:36 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 07:36 |
| shardy | d0ugal: that said, if folks want it to be a LOG.debug I'm OK with it, I just want an easy way to get the ID without needing to list stuff via mistralclient every time | 07:36 |
| shardy | I am inclined to approve it ;) | 07:37 |
| d0ugal | shardy: I'd be happy for you to do that ;) | 07:37 |
| d0ugal | shardy: I am sure the ID is already in the debug log, if you can find it. | 07:37 |
| shardy | Yeah it's not that easily accessible unless you tee the output and know what to grep for tho | 07:38 |
| d0ugal | Exactly, I find the debug log useless (except for error tracebacks) | 07:38 |
| d0ugal | shardy: For me it raises the question, do we want users to know and care about Mistral or are we trying to totally abstract it away? | 07:39 |
| shardy | d0ugal: I think the reality will be that, as with heat, some level of knowledge will be required when things go wrong | 07:39 |
| d0ugal | Yup, makes sense. So then the execution ID would be useful. | 07:40 |
| shardy | the error paths for the mistral stuff isn't good in some cases, so making users aware there is a mistral workflow that ran, in a terse and easy to consume way, seems worthwhile to me | 07:40 |
| d0ugal | +1 | 07:40 |
| shardy | if anyone is offended by the couple of lines of prints, they can simply ignore them | 07:40 |
| *** vijaykc4 has joined #tripleo | 07:40 | |
| shardy | but when something breaks, at least they have a tiny clue where to start digging, before there is nothing | 07:41 |
| *** ifarkas_afk is now known as ifarkas | 07:41 | |
| shardy | dtantsur|afk: ^^ FYI | 07:41 |
| *** tdasilva has joined #tripleo | 07:42 | |
| *** vijaykc4 has quit IRC | 07:44 | |
| *** yamahata has quit IRC | 07:45 | |
| *** dtantsur|afk is now known as dtantsur | 07:46 | |
| *** jpich has joined #tripleo | 07:47 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Add List Roles Action https://review.openstack.org/330283 | 07:49 |
| shardy | d0ugal: added a comment, perhaps dtantsur can revisit and we can discuss further - my only observation is if we're adding the prints it should probably be consistent e.g everywhere we run any workflow, not just the baremetal stuff | 07:50 |
| hewbrocca | Back on in an hour or so folks | 07:50 |
| d0ugal | shardy: Yup, good point - I think there were only Baremetal workflows in use when I wrote that. | 07:51 |
| d0ugal | Obviously a low priority thing, just doing a cleanup of all the review I have sitting around :) | 07:51 |
| shardy | d0ugal: Yeah, I figured that was the case | 07:51 |
| dtantsur | shardy, morning. I'm -1 on using prints where you must use log.debug | 07:51 |
| dtantsur | this is just bad UX | 07:51 |
| shardy | if you do rebase it I'd add the work Mistral in the output | 07:51 |
| dtantsur | of course you can override my -1 | 07:51 |
| shardy | dtantsur: I think the discussion is more around the log level | 07:51 |
| dtantsur | shardy, there is nothing to discuss, we have guidelines. execution ID is an implementation detail, unclear without understanding how tripleo work => log.debug :) | 07:52 |
| *** jprovazn has joined #tripleo | 07:52 | |
| shardy | I'd be fine if we say used LOG.info, I just think making it clear we're running a mistral workflow, and making it's ID obvious would be useful for non-debug usage | 07:52 |
| d0ugal | Really, all the prints should be updated to use log.info | 07:53 |
| dtantsur | d0ugal, ++ for s/pring/log.info/ or log.warning | 07:53 |
| d0ugal | I'll open a bug for this, I may have done so already. | 07:53 |
| *** saneax-_-|AFK is now known as saneax | 07:53 | |
| shardy | Sure, I'm fine with killing the prints, and we can argue later re s/info/debug | 07:53 |
| dtantsur | shardy, if we start log.info'ing such things, we need to make sure they don't appear out of blue. i.e. we give good context to people why it is there and what it means. | 07:54 |
| dtantsur | e.g. very bad: "Execution ID: abcd" | 07:54 |
| dtantsur | not so bad: "Starting introspection, workflow execution ID abcd" | 07:54 |
| shardy | Our current interfaces make it pretty clear that we're creating a heat stack, I'd just like to make it transparent that we're now running mistal, which does stuff and we can output the result | 07:54 |
| shardy | dtantsur: Yeah, as I said above, we should add mistral into the output | 07:55 |
| dtantsur | "Our current interfaces make it pretty clear that we're creating a heat stack" I don't agree with this | 07:55 |
| dtantsur | but it's probably not related | 07:55 |
| *** aufi has quit IRC | 07:56 | |
| shardy | [overcloud]: CREATE_IN_PROGRESS Stack CREATE started | 07:57 |
| shardy | but yeah, we should probably also print the ID at the start of that too | 07:57 |
| shardy | s/print/log | 07:57 |
| shardy | ;) | 07:57 |
| dtantsur | ++ | 07:57 |
| dtantsur | Like "Creating stack overcloud from templates in /usr/share/blah-blah" | 07:57 |
| dtantsur | instead of just "Deploying templates" | 07:58 |
| shardy | Yeah, I think I raised a but about that actually, that we should print the list of environment files | 07:58 |
| shardy | definitel room for improvement | 07:59 |
| shardy | what I'm trying to avoid is introducing mistral making things even more obfuscated and magical | 07:59 |
| shardy | if we make it very clear we're just running a mistral workflow, hopefully folks will understand and have a better chance of figuring things out when something breaks | 07:59 |
| *** hewbrocca is now known as hewbrocca-afk | 08:02 | |
| *** derekh has joined #tripleo | 08:05 | |
| *** mcornea has joined #tripleo | 08:08 | |
| *** ioggstream has joined #tripleo | 08:09 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Set parameters https://review.openstack.org/360391 | 08:11 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command https://review.openstack.org/359784 | 08:11 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Add `openstack overcloud deploy --plan` https://review.openstack.org/360305 | 08:11 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 08:11 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Generate reference documentation for Workbooks https://review.openstack.org/341572 | 08:13 |
| *** milan has quit IRC | 08:17 | |
| *** openstackgerrit has quit IRC | 08:18 | |
| *** openstackgerrit has joined #tripleo | 08:18 | |
| *** milan has joined #tripleo | 08:19 | |
| *** gfidente has joined #tripleo | 08:21 | |
| *** gfidente has quit IRC | 08:21 | |
| *** gfidente has joined #tripleo | 08:21 | |
| *** coolsvap_ has quit IRC | 08:21 | |
| cschwede | d0ugal: Good Morning! Quick question: do you mind if I update your tripleo-doc patch for Swift and address tdasilva’s comments? I’d like to make sure your patch and my followup are in a good shape to be ready for merging | 08:24 |
| cschwede | d0ugal: https://review.openstack.org/#/c/293311/8/doc/source/advanced_deployment/deploy_swift.rst | 08:24 |
| d0ugal | cschwede: Sure, please do. I've not had time to look at it for some time. It would be very much appreciated :) | 08:25 |
| cschwede | d0ugal: alright, thx! | 08:25 |
| *** aufi has joined #tripleo | 08:25 | |
| *** florianf has quit IRC | 08:25 | |
| *** electrofelix has joined #tripleo | 08:27 | |
| *** lucas-dinner is now known as lucasagomes | 08:28 | |
| openstackgerrit | Merged openstack/tripleo-common: Fixes missing action entry point https://review.openstack.org/359907 | 08:32 |
| paramite | shardy, slagle, jistr, gfidente: CI is green now, can we get this merged? | 08:35 |
| paramite | EmilienM, ^ | 08:35 |
| jpich | If the "Depends-On:" of a patch isn't merged, but the patch itself gets +2A, will CI block the merge until the dependency has gone through? | 08:35 |
| shardy | jpich: yes | 08:35 |
| shardy | you may need to re-approve or reverify when the Depends-On patch lands | 08:35 |
| jpich | shardy: Great, thanks! Wasn't sure if it was only for series in the same repo | 08:36 |
| shardy | paramite: sorry, can we merge what? | 08:36 |
| jpich | Understood | 08:36 |
| paramite | shardy, oh sorry, forgot to paste: https://review.openstack.org/#/c/254788/ | 08:36 |
| shardy | paramite: thanks, will check it out | 08:39 |
| paramite | shardy, thanks a lot | 08:39 |
| *** florianf has joined #tripleo | 08:40 | |
| *** uwe123 has joined #tripleo | 08:41 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Use Ironic's generic service name in help messages https://review.openstack.org/360755 | 08:42 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-docs: Deploy and Scale Swift https://review.openstack.org/293311 | 08:42 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-docs: Update advanced Swift deployment doc https://review.openstack.org/360353 | 08:42 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Add the ability to customize undercloud with virt-customize https://review.openstack.org/360637 | 08:42 |
| *** akuznetsov has quit IRC | 08:43 | |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Add the ability to customize undercloud with virt-customize https://review.openstack.org/360637 | 08:43 |
| *** jpena_ has joined #tripleo | 08:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-docs: [WIP] Documentation for installing and using Ironic in overcloud https://review.openstack.org/354016 | 08:44 |
| *** vijaykc4 has joined #tripleo | 08:45 | |
| gfidente | shardy, have a minute to help figure what I am doing wrong in https://review.openstack.org/#/c/336542/ ? | 08:53 |
| gfidente | fails with The Referenced Attribute (ComputeServiceChain role_data) is incorrect. | 08:53 |
| openstackgerrit | Andreas Jaeger proposed openstack/tripleo-validations: Cleanup tox.ini, enable constraints https://review.openstack.org/360994 | 08:56 |
| *** athomas has joined #tripleo | 08:59 | |
| *** jaosorior_brb is now known as jaosorior | 08:59 | |
| shardy | gfidente: I think the yaql is bad, you're trying to do a list transformation on a map? | 09:06 |
| gfidente | aaaaaaaaaaaaaaaaaaaaaaahhhh copy pastaaaaa thanks | 09:06 |
| gfidente | though wait, shouldn't that be a list of maps? | 09:07 |
| shardy | gfidente: no, the output of map_merge is a map | 09:08 |
| shardy | gfidente: maybe you need to move the map_merge outside the yaql? | 09:09 |
| gfidente | so can I can probably take map_merge out of yaql: | 09:09 |
| gfidente | yes that ack | 09:10 |
| shardy | ++ | 09:10 |
| uwe123 | Hi! Can I get help for a Triple-O deployment here or is this a dev channel? | 09:10 |
| shardy | uwe123: Hi! What's the problem, we'll try to help :) | 09:10 |
| uwe123 | I have an almost working deployment. The only thing not working as expected is that I cannot use both VLAN and VXLAN tenant networks. (This is on a Red Hat OSP 8) | 09:11 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add global_config_settings to services' output https://review.openstack.org/336542 | 09:12 |
| uwe123 | I can provide the templates and teh commandline I use to deploy. | 09:12 |
| uwe123 | In my networking template I have the following options: | 09:13 |
| uwe123 | NeutronExternalNetworkBridge: "''" | 09:13 |
| uwe123 | NeutronEnableTunnelling: true | 09:13 |
| uwe123 | NeutronNetworkType: "vxlan" | 09:13 |
| uwe123 | NeutronTunnelTypes: "vxlan" | 09:13 |
| uwe123 | NeutronNetworkVLANRange: "datacentre:3000:3199" | 09:13 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Fix outdated help text from undercloud VIPs https://review.openstack.org/361020 | 09:14 |
| uwe123 | after the deployment /etc/neutron/plugin.ini contains: | 09:15 |
| uwe123 | tenant_network_types = vxlan | 09:15 |
| uwe123 | network_vlan_ranges =datacentre:1:1000 | 09:15 |
| saneax | folks, this needs a simple workflow https://review.openstack.org/#/c/332190/ | 09:15 |
| uwe123 | not sure if I just need to add "vlan" to NeutronNetworkType or what else needs to be done so that the deployment uses the supplied VLAN IDs | 09:16 |
| saneax | Cores please help on this | 09:16 |
| gfidente | uwe123, sorry you mean you want to create tenant networks using one of VLAN , VXLAN | 09:16 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Add the ability to customize undercloud with virt-customize https://review.openstack.org/360637 | 09:17 |
| uwe123 | normal tenants should be able to create VXLANs, admins should also be able to create networks that use VLANs as mechanism | 09:17 |
| uwe123 | the hardware is already setup correctly (both switches and overcloud nodes) | 09:17 |
| uwe123 | only neutron is configured to only use VXLANs | 09:18 |
| gfidente | uwe123, ok let me check code | 09:18 |
| uwe123 | I have the need to connect external servers for specific tentants via VLAN | 09:18 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Added a new composable service for enabling DPDK on compute nodes. https://review.openstack.org/344084 | 09:21 |
| gfidente | uwe123, looks like that might just work yes | 09:22 |
| gfidente | NeutronNetworkType: vxlan,vlan | 09:23 |
| *** limao has quit IRC | 09:23 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Create hiera service_enabled keys for enabled services https://review.openstack.org/361029 | 09:25 |
| shardy | jaosorior: ^^ sorry for the delay, that should do what you need I think | 09:25 |
| gfidente | uwe123, should that fail, you can set neutron::plugins::ml2::tenant_network_types: ['vlan','vxvlan'] | 09:25 |
| gfidente | using ExtraConfig | 09:25 |
| *** abregman has joined #tripleo | 09:26 | |
| bandini | Anyone seen this error? https://paste.fedoraproject.org/414245/47220353/ am testing M/N upgrades and heat brakes when I do the major-upgrade-pacemaker-init.yaml step | 09:26 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Create hiera service_enabled keys for enabled services https://review.openstack.org/361029 | 09:26 |
| jaosorior | shardy: do you have a sample of how the hiera looks like in the end? | 09:27 |
| *** abregman has quit IRC | 09:27 | |
| shardy | jaosorior: http://paste.openstack.org/show/563818/ | 09:27 |
| shardy | jaosorior: that should "just work" with your haproxy patch I think? | 09:28 |
| jaosorior | shardy: ok, so is that name matching to the service profile names in t-h-t? | 09:28 |
| jaosorior | shardy: it should! I'm just checking that the names match as expected and that's it | 09:29 |
| shardy | http://paste.openstack.org/show/563819/ that's the full list | 09:29 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-docs: Update advanced Swift deployment doc https://review.openstack.org/360353 | 09:29 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-docs: Include Swift ring building example https://review.openstack.org/361032 | 09:29 |
| jaosorior | shardy: cause initially I defined them manually | 09:29 |
| shardy | jaosorior: yes it generates the hiera from the service_names in t-h-t | 09:29 |
| gfidente | shardy, that's cool | 09:29 |
| shardy | which is consistent with what I'm doing elsewhere for custom-roles | 09:29 |
| jaosorior | shardy: awesome! | 09:29 |
| jaosorior | let me try it out | 09:29 |
| shardy | We can simplify my patch a bit when bug #1617019 is fixed | 09:31 |
| openstack | bug 1617019 in heat "comma_delimited_list validated too eagerly, breaks with list_join" [High,Triaged] https://launchpad.net/bugs/1617019 - Assigned to Rabi Mishra (rabi) | 09:31 |
| shardy | bandini: not seen that, I'd check the heat-engine logs as it looks like a possible internal error | 09:33 |
| jaosorior | shardy: seems to me they match | 09:33 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Conditionally expose haproxy endpoints https://review.openstack.org/359645 | 09:34 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Configure keystone endpoints in service profile https://review.openstack.org/359680 | 09:35 |
| jaosorior | shardy: lets give it a go | 09:35 |
| jaosorior | there they are ^^ | 09:35 |
| *** numans has joined #tripleo | 09:36 | |
| shardy | nice, lets see if it works | 09:36 |
| bandini | shardy: thanks! | 09:36 |
| jaosorior | shardy: hey dude, could you check this out? https://review.openstack.org/#/c/356416/ | 09:38 |
| jaosorior | gfidente: have you had time to give it a thought? ^^ | 09:38 |
| gfidente | jaosorior, I think not appending the .network.localdomain part would be better | 09:39 |
| gfidente | jaosorior, but I am not -1 if others are fine with having those | 09:40 |
| jaosorior | I agree with bnemec that having a fqdn would be better. He even pointed out that he would like CloudName to be defaulted to a fully qualified name. Which would be good IMO (and could be done in a further patch) | 09:42 |
| gfidente | jaosorior, right so we can make those fqdn | 09:43 |
| gfidente | but they shouldn't happen to match our hosts naming I think | 09:43 |
| gfidente | the main issue I have there is that they look like as if they were built from the other parameters, like the domain part, as we do for the hostnames, but they are not | 09:43 |
| jaosorior | gfidente: but nowhere in the description does it desribe that they're generated in any way | 09:44 |
| gfidente | yep but why we want those to match .network.localdomain then? | 09:45 |
| jaosorior | gfidente: because the deployer didn't specify a domain. And to indicate that it's in another network. We can't have those domain match unless they are actually in the same network. | 09:46 |
| lucasagomes | derekh, shardy if you guys any time, mind taking a look at https://review.openstack.org/#/c/356539/ ? It would be great to have it merged in this cycle | 09:46 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Print the execution ID when starting workflows https://review.openstack.org/340711 | 09:47 |
| gfidente | jaosorior, exactly, so let there be fqdn | 09:48 |
| gfidente | internalvip.mydomain | 09:48 |
| gfidente | externalvip.mydomain | 09:48 |
| gfidente | ... | 09:48 |
| jaosorior | gfidente: that could be done if the deployer specified it. But it's very customary that if no domain is specified, localdomain is what's normally used :/ | 09:49 |
| derekh | lucasagomes: will take a look today | 09:49 |
| jaosorior | gfidente: which is still valid | 09:49 |
| gfidente | there is a domain, .mydomain | 09:49 |
| lucasagomes | derekh, thanks, this is the last essential bits to have a basic baremetal-to-tenant feature in the overcloud | 09:49 |
| jaosorior | but that's a random domain | 09:49 |
| gfidente | yes that is precisely my point | 09:50 |
| lucasagomes | derekh, shardy anyway, appreciate! | 09:50 |
| jaosorior | gfidente: localdomain is not random | 09:50 |
| gfidente | we could default them to a short name | 09:50 |
| gfidente | like we do for cloudname | 09:50 |
| gfidente | and call them just 'internalvip', 'storagevip' | 09:50 |
| jaosorior | but that default wasn't a good idea in the first place X_x | 09:50 |
| jaosorior | it's a bad practice | 09:50 |
| gfidente | jaosorior, the moment you want to make it a prodution ssl deployment | 09:50 |
| gfidente | you'll need to customize those strings right | 09:51 |
| gfidente | and they will be | 09:51 |
| jaosorior | gfidente: that's correct | 09:51 |
| *** tosky has joined #tripleo | 09:51 | |
| gfidente | publicname.publicdomain | 09:51 |
| jaosorior | gfidente: however, even for a deployment in a testing environment, you could use localdomain | 09:51 |
| gfidente | yes the thing is, the internal names could be built like we do for hostnames | 09:51 |
| gfidente | and set into endpoints correctly | 09:52 |
| gfidente | now if they are not built appending .network.localdomain | 09:52 |
| gfidente | I think that having a default which like they are, is misleading | 09:52 |
| gfidente | people has to customize those | 09:52 |
| gfidente | the internal ones too | 09:52 |
| openstackgerrit | Merged openstack/os-net-config: Add support for OVS DPDK Bridge and Port https://review.openstack.org/338694 | 09:53 |
| jaosorior | gfidente: what about doing a validation on those? That if people use CLOUDNAME in the endpoint map, the validation should detect if localdomain is used and throw a warning? | 09:53 |
| derekh | dtantsur: I'm confused, "openstack baremetal create: error: argument --driver is required" | 09:54 |
| derekh | dtantsur: but I have the most recent ironicclient, I think | 09:54 |
| gfidente | jaosorior, not worth more work :) if people is fine with that default it's okay | 09:54 |
| dtantsur | derekh, you need one from git. IIRC we now use upper-constraints for clients, so you don't get the latest even on master tripleo with the latest promotion | 09:55 |
| *** mpsairam has quit IRC | 09:55 | |
| dtantsur | derekh, I hope we release it early next week, and my docs will work more smoothly | 09:55 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Set parameters https://review.openstack.org/360391 | 09:55 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command https://review.openstack.org/359784 | 09:55 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Print the execution ID when starting Mistral workflows https://review.openstack.org/340711 | 09:55 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Add `openstack overcloud deploy --plan` https://review.openstack.org/360305 | 09:55 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 09:55 |
| derekh | dtantsur: ahh, ok | 09:55 |
| dtantsur | derekh, also depending on whether we got a promotion tonight, you might get broken ironic :( | 09:56 |
| dtantsur | the fix merged yesterday evening iirc | 09:56 |
| dtantsur | so it's all one big mess now, sorry | 09:56 |
| derekh | dtantsur: ok | 09:56 |
| openstackgerrit | Saravanan KR proposed openstack/os-net-config: Add support for OVS DPDK Bond https://review.openstack.org/349183 | 09:56 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-docs: Deploy and Scale Swift https://review.openstack.org/293311 | 09:58 |
| openstackgerrit | Saravanan KR proposed openstack/os-net-config: Supporting numbered nics for DPDK Port and DPDK Bond https://review.openstack.org/359610 | 10:01 |
| *** akrivoka has joined #tripleo | 10:03 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Migrate to using osc-lib https://review.openstack.org/335460 | 10:04 |
| derekh | Exception raised: 'module' object has no attribute 'CreateBaremetal' | 10:08 |
| derekh | dtantsur: ^^ will wait for a more recent ironicclient and working ironic to test again, for now I'll just review without testing | 10:09 |
| dtantsur | yeah, please just review for now.. I also can't test any more with so many bits in flight | 10:11 |
| *** vijaykc4 has quit IRC | 10:12 | |
| *** vijaykc4 has joined #tripleo | 10:12 | |
| *** vijaykc4 has quit IRC | 10:25 | |
| *** hewbrocca-afk is now known as hewbrocca | 10:27 | |
| uwe123 | gfidente: thanks, I'll try | 10:27 |
| *** aqkhan__ has joined #tripleo | 10:28 | |
| *** aqkhan_ has quit IRC | 10:29 | |
| *** aqkhan_ has joined #tripleo | 10:29 | |
| *** vijaykc4 has joined #tripleo | 10:29 | |
| hewbrocca | jaosorior: icanhaz os-cloud-config.die.die.die ? | 10:29 |
| openstackgerrit | Martin André proposed openstack/puppet-tripleo: UI profile for tripleo https://review.openstack.org/354630 | 10:30 |
| *** aqkhan__ has quit IRC | 10:33 | |
| *** milan has quit IRC | 10:33 | |
| *** skramaja has quit IRC | 10:35 | |
| *** skramaja has joined #tripleo | 10:35 | |
| *** numans has quit IRC | 10:38 | |
| dtantsur | hewbrocca, AttributeError: 'os-cloud-config' object has no attribute 'die' | 10:38 |
| *** numans has joined #tripleo | 10:39 | |
| jaosorior | hewbrocca: maybe. The patches are in the gate | 10:40 |
| hewbrocca | wooooooo | 10:41 |
| *** ioggstream has quit IRC | 10:41 | |
| *** rhallisey has joined #tripleo | 10:43 | |
| shardy | gfidente: Hey I wanted your opinion on https://review.openstack.org/#/c/355067/2/puppet/all-nodes-config.yaml | 10:44 |
| *** vijaykc4 has quit IRC | 10:44 | |
| shardy | I'm trying to move the ceph_mon_host lists into the service template, and I'm wondering if I should generate the _v6 bracketed lists for all services? | 10:45 |
| shardy | Or if it would be cleaner to just pass the unbracketed csv list into puppet-tripleo and add the brackets there? | 10:45 |
| *** skramaja_ has joined #tripleo | 10:45 | |
| shardy | https://review.openstack.org/#/c/355067/2/puppet/services/ceph-mon.yaml | 10:45 |
| *** skramaja has quit IRC | 10:45 | |
| shardy | It wouldn't be that hard to add a ceph_mon_node_ips_csv_v6 to hiera I guess | 10:46 |
| *** skramaja_ is now known as skramaja | 10:46 | |
| shardy | we're just getting quite a lot of duplicated lists in different formats, which isn't great | 10:46 |
| shardy | gfidente: the main problem with that is there isn't one _v6 format, e.g | 10:47 |
| shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/all-nodes-config.yaml#L141 | 10:47 |
| shardy | that one has a weird inet6: prefix | 10:48 |
| hewbrocca | "the main problem with that is there isn't one _v6 format" ... ARRRRGH... this makes me feel like it's 1999 or something | 10:48 |
| shardy | Yeah, it's a mess :( | 10:48 |
| *** adarazs is now known as adarazs_lunch | 10:48 | |
| hewbrocca | It demonstrates all our failings as an industry | 10:49 |
| hewbrocca | sigh | 10:49 |
| rook | shardy: heya! | 10:49 |
| shardy | rook: Hi! | 10:50 |
| hewbrocca | It's kinda like when the fire department from Baltimore traveled all the way to Chicago to help put out the fire | 10:50 |
| hewbrocca | only to discover that their hoses wouldn't attach to Chicago hydrants | 10:50 |
| hewbrocca | because nobody thought it was important to make all hydrants the same gauge and thread | 10:50 |
| rook | shardy: we are going to start the newton scaling work, however I want to make sure the patches you were expecting to drop are in. From what I hear from CI, newton is taking more resources to deploy a OC. | 10:50 |
| shadower | hewbrocca: ugh | 10:51 |
| hewbrocca | shadower: on the plus side, that catastrophe gave birth to ANSI | 10:51 |
| shadower | oh wow | 10:51 |
| shardy | rook: Yeah, that is mostly because folks keep adding new services and we're still working out a way to do a matrix of coverage vs duplicating tests | 10:52 |
| shardy | rook: the heat fixes have all landed I think, but it's possible we'll have some TripleO optimizations yet before the end of the cycle | 10:53 |
| shardy | rook: your feedback at this point would be very helpful, I'm sure there will be further tuning needed (in particular re the wsgi all-the-things that has been happening) | 10:54 |
| shardy | I'm hopeful that some of the heat memory usage issues and nested introspection pieces should be improved tho | 10:54 |
| rook | shardy: i see the wsgi stuff is changing for the better, so that is good. | 10:54 |
| *** skramaja has quit IRC | 10:54 | |
| rook | however, if we start tuning that -- we could see more memory utilization as well :/ | 10:55 |
| shardy | rook: Yeah, it just gave us some initially bad defaults I think | 10:55 |
| rook | shardy: well, we have the tooling to see if heat-api/engine is using less... so we should be able to see that. | 10:55 |
| rook | shardy: yeah :/ | 10:56 |
| openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-docs: [WIP] Documentation for installing and using Ironic in overcloud https://review.openstack.org/354016 | 11:00 |
| *** zoli_gone-proxy is now known as zoliXXL | 11:01 | |
| gfidente | shardy, yeah I think we need to hurry up and have ipv6 back in ci jobs | 11:01 |
| *** zoliXXL is now known as zoli|lunmch | 11:04 | |
| *** zoli|lunmch is now known as zoli|lunch | 11:04 | |
| gfidente | I'll do the mangling in puppet | 11:06 |
| gfidente | shardy, ^^ | 11:06 |
| gfidente | hopefully we have some ipv6 there | 11:06 |
| *** mburned_out is now known as mburned | 11:06 | |
| gfidente | (bool) | 11:06 |
| *** numans has quit IRC | 11:06 | |
| gfidente | I remember simila issue with memcached too somewhere in the templates | 11:07 |
| shardy | gfidente: Ok, that would be good, so we'll just pass in either a json or csv string with a list of IPs, then have puppet-tripleo do any further transformations? | 11:07 |
| shardy | gfidente: Yeah, we have the same issue with memcached | 11:07 |
| shardy | and the format it expects isn't even the same bracketed format needed by ceph | 11:08 |
| * hewbrocca wishes we could just use hostnames | 11:08 | |
| gfidente | though believe it or not I landed so is_ipv4 and is_ipv6 in puppet-stdlib | 11:08 |
| gfidente | https://github.com/puppetlabs/puppetlabs-stdlib/commit/d85aec41a3b57a13f16086cc1ff7ed2fe09602b8 | 11:08 |
| gfidente | which looks like turn useful now :P | 11:08 |
| gfidente | hewbrocca, yeah mee too | 11:08 |
| gfidente | shardy, actually, we have network mapped hostnames in hosts now | 11:09 |
| gfidente | does that help in allnodes ? | 11:09 |
| *** thrash|g0ne is now known as thrash | 11:09 | |
| shardy | gfidente: Not unless we do the network->ips lookup in puppet too | 11:09 |
| shardy | gfidente: I've been generating a per-service list of IPs (and hostnames) in allNodesConfig | 11:10 |
| shardy | which is fine, I just don't really want to do it in three different formats | 11:10 |
| gfidente | ok let's do it in puppet for now | 11:10 |
| gfidente | /me doing | 11:10 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move ceph cluster config out of allNodesConfig https://review.openstack.org/355067 | 11:10 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add service_ips in csv format to allNodes hieradata https://review.openstack.org/355066 | 11:10 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Generate composable service node_names lists https://review.openstack.org/355068 | 11:10 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Create composable mapping between enabled services and role ips https://review.openstack.org/348974 | 11:10 |
| shardy | gfidente: that's the series, rebased on the foo_enabled patch I did for jaosorior | 11:11 |
| gfidente | foo_enabled is cool :) | 11:11 |
| shardy | feedback welcome, but yeah if you were able to help remove some of the _v6 hiera keys that would really help me out :) | 11:11 |
| shardy | Those and the per-service vip mappings are the final pieces to enable custom-roles I think | 11:12 |
| *** shardy is now known as shardy_lunch | 11:13 | |
| *** tosky has quit IRC | 11:13 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: [WIP] Remove duplication of the Zaqar message format https://review.openstack.org/361088 | 11:13 |
| *** tosky has joined #tripleo | 11:15 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: [WIP] Remove duplication of the Zaqar message format https://review.openstack.org/361088 | 11:15 |
| *** ioggstream has joined #tripleo | 11:19 | |
| Jokke_ | Hi all, https://review.openstack.org/#/c/354047/ how do I recheck the ovb jobs? | 11:19 |
| jaosorior | Jokke_: I think you can only recheck everything | 11:20 |
| *** ramishra has quit IRC | 11:20 | |
| jaosorior | Jokke_: so you just write a comment starting with "recheck" | 11:20 |
| *** lucasagomes is now known as lucas-hungry | 11:21 | |
| Jokke_ | jaosorior: well by the looks on the change, it has rerun the "Jenkins check" group but not the "Jenkins" group that is ovb jobs | 11:21 |
| jaosorior | Jokke_: yeah, the ovb take considerably longer | 11:21 |
| jaosorior | Jokke_: you can check if they're running in zuul | 11:21 |
| jaosorior | http://status.openstack.org/zuul/ | 11:21 |
| *** ramishra has joined #tripleo | 11:22 | |
| *** matbu is now known as matbu|lch | 11:22 | |
| *** abehl has quit IRC | 11:23 | |
| Jokke_ | jaosorior: I mean that change has rechecked 2, we still have results from the original ovb run, no rechecks on that at all | 11:23 |
| jaosorior | Jokke_: for some reason it's not running in zuul | 11:24 |
| jaosorior | Jokke_: well, that's how I do it, and it usually works for ovb too. No clue why that one is not being taken | 11:24 |
| Jokke_ | jaosorior: if you toggle CI on, you see that ovb has reported only after the PS was uploaded, not after either of the rechecks | 11:24 |
| Jokke_ | jaosorior: me neither, that's why I was asking ;) | 11:25 |
| Jokke_ | by the looks of it it does not trigger on rechecks | 11:25 |
| Jokke_ | so looks like pointless to throw it in there if it runs only the infra runs | 11:26 |
| hewbrocca | derekh: hey, any idea if we got the FIP problem with infra sorted? | 11:29 |
| *** skramaja has joined #tripleo | 11:31 | |
| hewbrocca | Jokke_ and jaosorior: We were having some amount of delay on infra OVB jobs getting FIPs on RH1 | 11:31 |
| hewbrocca | that could be impacting you, I guess | 11:31 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: SR-IOV nic agent changes in line with composable roles. https://review.openstack.org/332190 | 11:32 |
| Jokke_ | hewbrocca: so it should be picked up just by recheck, but that might not happen due to some issue that is worked on? | 11:32 |
| Jokke_ | hewbrocca: just recapping so I undertood correctly | 11:32 |
| hewbrocca | I'm speculating on that but it could be true | 11:33 |
| *** bootsha has joined #tripleo | 11:34 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add sriov_numvfs in THT https://review.openstack.org/345352 | 11:39 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Customize "supported_pci_vendor_devs" for SR-IOV https://review.openstack.org/327735 | 11:39 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Adds brackets to mon_host, removing old _v6 param https://review.openstack.org/361108 | 11:41 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Ironic: add missing haproxy and firewall configuration https://review.openstack.org/356539 | 11:42 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Adds brackets to mon_host, removing old _v6 param https://review.openstack.org/361108 | 11:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-tripleo: Ironic: manage PXE environment for conductors https://review.openstack.org/361109 | 11:43 |
| *** asbishop has joined #tripleo | 11:43 | |
| *** ioggstream has quit IRC | 11:44 | |
| *** bootsha has quit IRC | 11:44 | |
| openstackgerrit | Merged openstack/tripleo-common: Workflow to setup validations https://review.openstack.org/313632 | 11:45 |
| *** nyechiel has joined #tripleo | 11:47 | |
| karthiks | shardy_lunch, gfidente, could you please look at https://review.openstack.org/#/c/343770/ https://review.openstack.org/#/c/337264/ https://review.openstack.org/#/c/327705/ | 11:47 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Add the ability to customize the overcloud with virt-customize https://review.openstack.org/361112 | 11:49 |
| *** masco has quit IRC | 11:52 | |
| *** zoli|lunch is now known as zoliXXL | 11:53 | |
| *** nyechiel has quit IRC | 11:53 | |
| *** jaosorior has quit IRC | 11:53 | |
| *** pgadiya has quit IRC | 11:54 | |
| *** jaosorior has joined #tripleo | 11:54 | |
| *** matbu|lch is now known as matbu | 11:56 | |
| *** pkovar has joined #tripleo | 11:57 | |
| *** fultonj has joined #tripleo | 11:59 | |
| gfidente | karthiks, nice, commented on https://review.openstack.org/#/c/343770/5 | 12:00 |
| *** jpena_ is now known as jpena|lunch | 12:01 | |
| *** oshvartz has joined #tripleo | 12:02 | |
| *** zoliXXL is now known as zoli|brb | 12:03 | |
| *** Goneri has joined #tripleo | 12:04 | |
| *** milan has joined #tripleo | 12:04 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Customize vcpu_pin_set in nova.conf https://review.openstack.org/343770 | 12:05 |
| karthiks | Thanks gfidente | 12:08 |
| thrash | d0ugal: so, it fixed your plugin problem, but created the requirements problem. | 12:09 |
| uwe123 | gfidente: I tried an update of the deployed overcloud with | 12:09 |
| uwe123 | parameter_defaults: | 12:09 |
| uwe123 | NeutronNetworkType: "vxlan,vlan" | 12:09 |
| uwe123 | which was updated in /etc/neutron/plugin.ini but | 12:09 |
| uwe123 | parameter_defaults: | 12:09 |
| uwe123 | NeutronNetworkVLANRange: "datacentre:3000:3199" | 12:09 |
| uwe123 | wasn't honoured as /etc/neutron/plugin.ini still has | 12:09 |
| uwe123 | [ml2_type_vlan] | 12:09 |
| uwe123 | network_vlan_ranges =datacentre:1:1000 | 12:09 |
| uwe123 | Any suggestions? Else I would try an update with | 12:09 |
| uwe123 | parameter_defaults: | 12:09 |
| uwe123 | ExtraConfig: | 12:09 |
| EmilienM | good morning | 12:10 |
| *** shardy_lunch is now known as shardy | 12:11 | |
| *** adarazs_lunch is now known as adarazs | 12:12 | |
| jaosorior | EmilienM: hey dude, would really appreciate your opinion on this CR https://review.openstack.org/#/c/356416/ | 12:14 |
| EmilienM | jaosorior: will look in a few | 12:15 |
| dtantsur | folks, do THT have "if" conditions or something like that? | 12:16 |
| dtantsur | I have 2 variables to set, one accepts string, the other - boolean... both has to be set to an equivalent value | 12:16 |
| *** dprince has joined #tripleo | 12:23 | |
| uwe123 | Could someone tell me if either "NeutronNetworkVLANRange" or "NeutronNetworkVLANRanges" (plural) is correct for networking template? If plural is correct, then I found a documentation bug | 12:23 |
| Jokke_ | jaosorior, hewbrocca: it got picked up by zuul on recheck now | 12:24 |
| jaosorior | Jokke_: nice! | 12:24 |
| *** ioggstream has joined #tripleo | 12:24 | |
| *** chem|off` has quit IRC | 12:25 | |
| *** lucas-hungry is now known as lucasagomes | 12:25 | |
| *** chem|off` has joined #tripleo | 12:26 | |
| shardy | uwe123: it should be NeutronNetworkVLANRanges I think (grep of tripleo-heat-templates says so) | 12:26 |
| *** vijaykc4 has joined #tripleo | 12:27 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates: Move ceilometer api to run under apache wsgi https://review.openstack.org/293642 | 12:28 |
| gfidente | uwe123, yeah and that's also the reason why it isn't reflected in the .ini file | 12:28 |
| d0ugal | thrash: Yeah, I am just going to copy the code over. I only need this. It's like 20 lines of code. https://github.com/openstack/tripleo-common/blob/master/tripleo_common/utils/tarball.py | 12:28 |
| gfidente | uwe123, good to hear the vlan,vxlan part worked | 12:28 |
| *** pradk has joined #tripleo | 12:29 | |
| *** numans has joined #tripleo | 12:29 | |
| thrash | d0ugal: that's a good idea. | 12:30 |
| *** rlandy has joined #tripleo | 12:31 | |
| jpich | d0ugal: Keep in mind florianf's https://review.openstack.org/#/c/355472/ for create_tarball too, if possible | 12:32 |
| *** trozet has quit IRC | 12:32 | |
| d0ugal | jpich: d'oh | 12:32 |
| d0ugal | jpich: I shall construct some hackery so it attempts to import it from tripleo_common. | 12:35 |
| thrash | d0ugal: maybe keep it a separate patch to make a revert easier? | 12:35 |
| d0ugal | thrash: hmm, that could work - I am currently doing a try/except ImportError | 12:35 |
| uwe123 | gfidente: Thanks for confirmation. Then there really is a documentation bug. I'll run the update a second time to see if the VLAN ranges are configuried correctly then. | 12:36 |
| openstackgerrit | Merged openstack/tripleo-ui: Update validations action name https://review.openstack.org/360643 | 12:36 |
| thrash | d0ugal: the ImportError catch could make the plugin check pass... | 12:37 |
| thrash | d0ugal: what's keeping us from releasing tripleo-common? | 12:38 |
| d0ugal | thrash: oh, that would be even more wonderfully hacky! | 12:38 |
| thrash | and just updating the requirements? | 12:38 |
| d0ugal | thrash: nothing I guess? | 12:38 |
| karthiks | gfidente, Commented on https://review.openstack.org/#/c/327705/ | 12:38 |
| d0ugal | thrash: Can you do releases? | 12:38 |
| thrash | d0ugal: maybe? :P | 12:38 |
| d0ugal | shardy: Can we just release tripleo-common? | 12:38 |
| d0ugal | thrash: I guess we are working towards 5.0 which has significance, so not sure what we would release under now. | 12:38 |
| shardy | d0ugal: yes, but I'm not quite sure what we do re the numbering | 12:39 |
| shardy | anyone can propose a release to openstack/releases | 12:39 |
| shardy | I'd suggest we take a recent periodic job and release everything, then look to release again next week for newton-3 | 12:40 |
| pradk | can i get some reviews on https://review.openstack.org/#/c/359387/ please | 12:41 |
| d0ugal | shardy: Sounds good, can you point me to something that tells me how to do it? | 12:42 |
| shardy | d0ugal: jump into #openstack-release, lets ask about the numbering | 12:42 |
| d0ugal | shardy: k | 12:42 |
| *** jdob has quit IRC | 12:43 | |
| *** jdob has joined #tripleo | 12:44 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Update pacemaker_resource_restart.sh for new HA arch https://review.openstack.org/354891 | 12:44 |
| *** trozet has joined #tripleo | 12:44 | |
| *** lblanchard has joined #tripleo | 12:48 | |
| jpeeler | EmilienM: do you mind taking a quick look at https://review.openstack.org/#/c/360790/ whenever you can? | 12:50 |
| EmilienM | jpeeler: +A | 12:51 |
| jpeeler | thanks | 12:51 |
| openstackgerrit | Erno Kuvaja proposed openstack/tripleo-heat-templates: Add integration with Manila CephFS Native driver https://review.openstack.org/358525 | 12:53 |
| *** zoli|brb is now known as zoliXXL | 12:53 | |
| EmilienM | jaosorior: before approving https://review.openstack.org/#/c/356416/ we might want to see ovb ha job, no? | 12:56 |
| EmilienM | jaosorior: also gfidente seems to have some opinion | 12:56 |
| EmilienM | but didn't vote | 12:56 |
| beagles | karthiks: I wonder if the numvfs stuff could go in puppet-tripleo? You could run it past shardy and dprince, etc. | 12:57 |
| *** fultonj has quit IRC | 12:58 | |
| *** tzumainn has joined #tripleo | 12:58 | |
| beagles | karthiks, we do stuff like configure firewalls, haproxy, etc, run os-net-config as well as the "adapter layer" between tht and the puppet modules | 12:59 |
| jaosorior | EmilienM: not sure why the ha is not showing up there | 12:59 |
| d0ugal | shardy: Thanks! | 12:59 |
| beagles | karthiks, if it is appropriate then the change the tht is actually pretty minimal as well. You just change which hieradata is being modified | 12:59 |
| *** morazi has joined #tripleo | 13:00 | |
| karthiks | beagles, do you mean moving the complete numvfs (puppet-neutron) stuff to puppet-tripleo | 13:01 |
| d0ugal | thrash, jpich: turns out this will solve our issues: https://review.openstack.org/#/c/360728/ | 13:01 |
| beagles | karthiks, yeah | 13:01 |
| jaosorior | EmilienM: gfidente has mentioned he doesn't agree with the defaults. But doesn't want to -1 either since people seem to be alright with them. | 13:01 |
| karthiks | beagles, I think it makes sense. | 13:01 |
| beagles | karthiks, just the numvfs stuff... neutron doesn't need it (right?) | 13:01 |
| jaosorior | EmilienM: by the way, at least in the comments from the patch, this is what jenkins shows: | 13:02 |
| karthiks | beagles, yes. its required for the hosts | 13:02 |
| jaosorior | gate-tripleo-ci-centos-7-ovb-ha SUCCESS in 2h 42m 02s | 13:02 |
| jaosorior | gate-tripleo-ci-centos-7-ovb-nonha SUCCESS in 2h 25m 15s | 13:02 |
| jaosorior | EmilienM: ^^ | 13:02 |
| beagles | karthiks, the neutron services need it? | 13:02 |
| *** fultonj has joined #tripleo | 13:02 | |
| EmilienM | gfidente: could you vote in the patch? I don't feel confortable to approve it without your vote in this case. | 13:02 |
| jpich | d0ugal: So, this means we'll have a red non-voting job till the n-3 release? | 13:03 |
| gfidente | EmilienM, jaosorior guys I just want to see if there are other comments from people on those defaults | 13:03 |
| jaosorior | EmilienM: you don't need to merge it if you don't want to. That's why I asked for your opinion on the patch. To get more input | 13:04 |
| karthiks | beagles, I dont think neutron services need it. | 13:04 |
| shardy | jpich: Yes, that's my understanding, but n-3 is next week | 13:05 |
| EmilienM | jaosorior: the defaults are good to me | 13:05 |
| beagles | karthiks, right.. so it shouldn't affect anything lifting it out of there. | 13:05 |
| *** jpena|lunch is now known as jpena_ | 13:05 | |
| jaosorior | EmilienM: can you vote on it? if you like them a +1 is fine too. | 13:05 |
| beagles | karthiks, it'd be nice if that is a suitable solution - after looking at the other ways to do it, the puppet approach seemed soooo much better | 13:06 |
| jaosorior | EmilienM: thanks for checking it out. | 13:06 |
| jpich | shardy: Got it, I wanted to confirm when it's time to start paying attention to the job again. Thanks! | 13:06 |
| beagles | karthiks, we may end up having to refine how we "hook stuff up" as time goes on but that's "normal" | 13:06 |
| karthiks | dprince, shardy beagles There is a conflict in configuring the persistence of numvfs in puppet-neutron with Fuel. Can we move this patch to puppet-tripleo https://review.openstack.org/#/c/352494/ | 13:07 |
| jpich | d0ugal: Well that's one resolved, yay ;) | 13:07 |
| EmilienM | jaosorior: +2 | 13:07 |
| * beagles snickers at using the word "normal" in this context | 13:07 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Set parameters https://review.openstack.org/360391 | 13:09 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command https://review.openstack.org/359784 | 13:09 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Print the execution ID when starting Mistral workflows https://review.openstack.org/340711 | 13:09 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: [WIP] Add `openstack overcloud deploy --plan` https://review.openstack.org/360305 | 13:09 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 13:09 |
| d0ugal | jpich: Yeah, I guess it will be red until tripleo-common is released next week. | 13:10 |
| * d0ugal just seen he was too slow | 13:10 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Extract /var/log from all nodes https://review.openstack.org/361164 | 13:11 |
| openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud: Handling defaults for pxe_bootfile_name and pxe_config_template moved to puppet https://review.openstack.org/361165 | 13:11 |
| jpich | d0ugal: At least the answer was the same ;) | 13:11 |
| d0ugal | haha, true! | 13:11 |
| shardy | karthiks: I guess we could do something similar to the fuel manifest in puppet tripleo | 13:11 |
| slagle | EmilienM: https://review.openstack.org/361164 should make log diving in multinode jobs easier | 13:11 |
| shardy | kinda sad that we then duplicate the fuel implementation tho | 13:11 |
| shardy | karthiks: perhaps EmilienM has ideas on another way to do it | 13:11 |
| EmilienM | oh dude I love you | 13:11 |
| EmilienM | no more tar xf on my laptop | 13:12 |
| slagle | i love you too man | 13:12 |
| EmilienM | shardy: what is the context? sorry | 13:12 |
| shardy | EmilienM: https://review.openstack.org/#/c/352494/ is blocked because of a conflict with fuel | 13:13 |
| EmilienM | let me check | 13:13 |
| shardy | EmilienM: karthiks is asking if we can move the logic into puppet-tripleo (which I guess will be similar to how fuel does it) | 13:13 |
| shardy | I don't have a strong opinion on it tbh, other than a desire to not wheel-reinvent if we can avoid it | 13:13 |
| EmilienM | "If you want to manage the persistence via neutron, please make it optional with default off." | 13:14 |
| EmilienM | sounds like a key comment ^ | 13:14 |
| shardy | Yeah a conditional seems like a reasonable compromise, but folks seemed to subsequently be saying to it in a different module | 13:15 |
| EmilienM | mhh it looks like they want the code outside | 13:15 |
| beagles | EmilienM, I think the general feeling was where this configuration is infrastructure only and doesn't affect neutron that it doesn't belong there -- but maybe we're off there? | 13:15 |
| EmilienM | yeah, maybe using puppet tripleo... | 13:16 |
| *** cdearborn has joined #tripleo | 13:16 | |
| EmilienM | I agree that thing has nothing to do with neutron itself | 13:16 |
| *** dtantsur is now known as dtantsur|brb | 13:16 | |
| EmilienM | I guess puppet-neutron would be a better candidate for this provider | 13:16 |
| EmilienM | and we can iterate later if we think it should be somewhere else | 13:17 |
| beagles | do you meant puppet-tripleo? | 13:17 |
| EmilienM | but puppet-tripleo would be a great incubator for this feature. | 13:17 |
| EmilienM | beagles: yes sorry | 13:17 |
| EmilienM | puppet-tripleo -> good place for now | 13:17 |
| *** nyechiel has joined #tripleo | 13:17 | |
| *** maeca1 has joined #tripleo | 13:17 | |
| karthiks | EmilienM, so we'll move it to puppet-tripleo ? | 13:17 |
| EmilienM | puppet-neutron -> maybe later but right now, not the best place because it's not neutron related | 13:17 |
| EmilienM | yes, please | 13:17 |
| beagles | awesome, thanks EmilienM, shardy | 13:18 |
| openstackgerrit | Adriano Petrich proposed openstack/tripleo-heat-templates: GATE TEST, please ignore https://review.openstack.org/361173 | 13:18 |
| karthiks | Thanks EmilienM beagles shardy | 13:18 |
| EmilienM | karthiks: yw, and sorry | 13:18 |
| pabelanger | derekh: replied to https://review.openstack.org/#/c/357308 There is a follow up patch to do just that | 13:18 |
| pabelanger | derekh: I wanted to land the shard network change first, so we could validate the process external to tripleo-ci | 13:18 |
| pabelanger | then land https://review.openstack.org/#/c/357341 | 13:19 |
| *** [1]cdearborn has joined #tripleo | 13:20 | |
| *** pkovar has quit IRC | 13:21 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT https://review.openstack.org/329415 | 13:21 |
| *** pkovar has joined #tripleo | 13:22 | |
| karthiks | gfidente, shardy EmilienM Regarding review https://review.openstack.org/329415, I've set the default value as "" . If not, the nova.conf has "pci_passthrough_whitelist =". Please advise. | 13:23 |
| EmilienM | karthiks: what if you set an empty list? [] | 13:25 |
| karthiks | pci_passthrough_whitelist = | 13:25 |
| openstackgerrit | Merged openstack/tripleo-quickstart: newton/master add ssl option as default to True for the undercloud https://review.openstack.org/356431 | 13:25 |
| EmilienM | you'll have to patch puppet nova like done for schedulers | 13:25 |
| karthiks | EmilienM, hieradata has '[]' . Puppet is expecting a string | 13:25 |
| EmilienM | a sec | 13:25 |
| *** abregman has joined #tripleo | 13:26 | |
| EmilienM | karthiks: you'll have to do something like https://review.openstack.org/#/c/359604/3/manifests/api.pp | 13:27 |
| EmilienM | and then you'll be able to give [] in THT | 13:27 |
| EmilienM | puppet will not configure the param if [] is given in THT | 13:27 |
| karthiks | ok EmilienM . Will do | 13:27 |
| *** jeckersb_gone is now known as jeckersb | 13:29 | |
| EmilienM | slagle, shardy: by doing scenario001, I found out sahara is not working | 13:30 |
| hewbrocca | EmilienM: #winning | 13:30 |
| EmilienM | I added a Sahara resource in the pingtest heat template and it fails to connect to sahara API | 13:30 |
| *** jobewan has joined #tripleo | 13:31 | |
| derekh | pabelanger: ok, sounds good | 13:31 |
| shardy | EmilienM: ack, well thats good to know I guess - maybe chat with thrash as I know he did a bunch of work very recently to get Sahara working again | 13:34 |
| *** cdearborn has quit IRC | 13:34 | |
| *** jobewan is now known as jobewan_away | 13:35 | |
| EmilienM | shardy: well in fact I think I missed something | 13:35 |
| EmilienM | shardy: I forgot to override the resource_registry | 13:36 |
| EmilienM | shardy: the resource is None by default | 13:36 |
| shardy | aha, that'll do it :) | 13:36 |
| EmilienM | shardy: https://review.openstack.org/#/c/360039/14..15/test-environments/scenario001-multinode.yaml | 13:37 |
| EmilienM | hewbrocca: /me facepalm | 13:37 |
| *** uwe123 has quit IRC | 13:39 | |
| *** pkovar1 has joined #tripleo | 13:39 | |
| *** pkovar has quit IRC | 13:39 | |
| *** uwe123 has joined #tripleo | 13:39 | |
| *** myoung|remote has joined #tripleo | 13:39 | |
| hewbrocca | rebooting, BRB folks | 13:40 |
| *** hewbrocca is now known as hewbrocca-afk | 13:40 | |
| *** jobewan_away is now known as jobewan | 13:41 | |
| karthiks | EmilienM, If I give [] in THT default, it gets converted to '[]'. And it will skip empty check in puppet-nova. Can I use a regex instead of empty() in puppet-nova | 13:41 |
| EmilienM | karthiks: why a regex | 13:42 |
| EmilienM | use the same code as I showed you | 13:42 |
| EmilienM | this is consistent | 13:42 |
| *** hewbrocca-afk is now known as hewbrocca | 13:44 | |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Add the ability to customize the overcloud with virt-customize https://review.openstack.org/361112 | 13:44 |
| *** links has quit IRC | 13:45 | |
| openstackgerrit | Thiago da Silva proposed openstack/tripleo-docs: Update advanced Swift deployment doc https://review.openstack.org/360353 | 13:46 |
| *** maeca1 has quit IRC | 13:47 | |
| pradk | can i get some reviews on https://review.openstack.org/#/c/293642/ https://review.openstack.org/#/c/360729/ https://review.openstack.org/#/c/360731/ | 13:49 |
| *** myoung|remote has quit IRC | 13:49 | |
| *** maeca1 has joined #tripleo | 13:50 | |
| karthiks | EmilienM, Right now puppet-nova takes pci_passthrough as a string. Do you want me to change this to an array (as per the example you've given ) ? | 13:56 |
| EmilienM | karthiks: probably it's best, right? | 13:57 |
| EmilienM | it should be an array, isn't? | 13:57 |
| uwe123 | gfidente: Using the plural resulted in a correctly updated plugin.ini. Thanks a lot! | 13:57 |
| gfidente | uwe123, good to know | 13:57 |
| karthiks | Wont it affect anyone using puppet-nova ? | 13:57 |
| karthiks | EmilienM, ^^ | 13:58 |
| EmilienM | karthiks: yes it will, that's what you might need to use any2array | 13:59 |
| EmilienM | wait, let me find an example | 13:59 |
| EmilienM | oh wait, let me write the code | 13:59 |
| EmilienM | :) | 13:59 |
| dprince | gfidente: see comment here https://review.openstack.org/#/c/360751/2 | 14:00 |
| karthiks | Thanks EmilienM :) | 14:00 |
| *** uwe123 has left #tripleo | 14:00 | |
| dprince | gfidente: can I take over your patches? | 14:00 |
| dprince | gfidente: I'm concerned we are stepping on toes with rebases here? | 14:00 |
| jaosorior | simple commit fixing help text: https://review.openstack.org/#/c/361020/ | 14:02 |
| gfidente | dprince, sure | 14:02 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Customize scheduler_default_filters (nova.conf) https://review.openstack.org/337264 | 14:02 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Move Redis, Memcached, Sahara, OVS agent out of role templates https://review.openstack.org/360697 | 14:02 |
| *** rbowen has joined #tripleo | 14:03 | |
| gfidente | we just got one merged luckily :) | 14:03 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Populate HAProxy endpoint ports from EndpointMap https://review.openstack.org/355366 | 14:03 |
| *** bootsha has joined #tripleo | 14:05 | |
| *** ebalduf has joined #tripleo | 14:09 | |
| *** saneax is now known as saneax-_-|AFK | 14:11 | |
| *** cdearborn has joined #tripleo | 14:13 | |
| *** dtantsur|brb is now known as dtantsur | 14:15 | |
| larsks | Is there a location to place docs for composable roles? | 14:16 |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Availability monitoring agents support https://review.openstack.org/254788 | 14:17 |
| *** eggmaster has joined #tripleo | 14:17 | |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-heat-templates: Added DPDK environment file and nic configs samples https://review.openstack.org/361238 | 14:18 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Use relative paths in ci-scripts for quickstart.sh https://review.openstack.org/360649 | 14:18 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Document Third Party CI and developer usage https://review.openstack.org/360007 | 14:18 |
| EmilienM | karthiks: almost finished, let me a few min | 14:19 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Handle image location based on CI_ENV and REL_TYPE https://review.openstack.org/359796 | 14:21 |
| *** ifarkas is now known as ifarkas_afk | 14:21 | |
| *** bootsha has quit IRC | 14:21 | |
| shardy | larsks: there is a composable services tutiorial in tripleo-docs which you are welcome to add to and/or refactor | 14:22 |
| karthiks | No problem EmilienM | 14:22 |
| larsks | shardy: I wasn't thinking so much of a tutorial as much as per-role documentation. But I'll take a look... | 14:22 |
| openstackgerrit | Thiago da Silva proposed openstack/tripleo-docs: Include Swift ring building example https://review.openstack.org/361032 | 14:22 |
| shardy | larsks: ah, yeah there's nothing like that yet except the per-service descriptions in the templates and associated puppet profiles | 14:23 |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Stop mirroring puppet modules https://review.openstack.org/361241 | 14:23 |
| openstackgerrit | Merged openstack/instack-undercloud: Modify improper augeas set command https://review.openstack.org/360790 | 14:23 |
| *** tremble has quit IRC | 14:25 | |
| *** Ng has quit IRC | 14:25 | |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Start using the image cache again https://review.openstack.org/361245 | 14:25 |
| *** [1]cdearborn has quit IRC | 14:27 | |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/tripleo-heat-templates: Add fluentd client service https://review.openstack.org/353506 | 14:27 |
| *** Ng has joined #tripleo | 14:28 | |
| *** ChanServ sets mode: +v Ng | 14:28 | |
| *** jpena_ has quit IRC | 14:30 | |
| *** dprince has quit IRC | 14:30 | |
| jrist | EmilienM: are your glasses for when your eyes really tired? | 14:30 |
| EmilienM | jrist: yes | 14:30 |
| EmilienM | jrist: only when I'm on computers | 14:30 |
| jrist | :) | 14:30 |
| *** vijaykc4 has quit IRC | 14:30 | |
| *** dprince has joined #tripleo | 14:31 | |
| EmilienM | jrist: it also makes me serious | 14:31 |
| *** chlong has joined #tripleo | 14:32 | |
| *** jeckersb is now known as jeckersb_gone | 14:32 | |
| *** vijaykc4 has joined #tripleo | 14:33 | |
| *** jpena_ has joined #tripleo | 14:33 | |
| jrist | I see that EmilienM | 14:33 |
| jrist | so serious | 14:33 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Update undercloud flavor stats https://review.openstack.org/360756 | 14:35 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Enable usage of FQDNs for the endpoints https://review.openstack.org/356416 | 14:35 |
| gfidente | anybody willing to check some bash https://review.openstack.org/#/c/357130/ ? :) | 14:37 |
| *** jeckersb_gone is now known as jeckersb | 14:39 | |
| *** maeca1 has quit IRC | 14:39 | |
| tbarron | is there a way to just recheck gate-tripleo-ci-centos-7-ovb-ha ? or do I have to throw everything back to the overloaded CI ? | 14:40 |
| *** pblaho has quit IRC | 14:40 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: More cleanup of controller.yaml https://review.openstack.org/360751 | 14:40 |
| *** cdearborn has quit IRC | 14:42 | |
| *** yamahata has joined #tripleo | 14:42 | |
| *** atom_ has joined #tripleo | 14:42 | |
| openstackgerrit | Martin André proposed openstack/puppet-tripleo: UI profile for tripleo https://review.openstack.org/354630 | 14:43 |
| *** cdearborn has joined #tripleo | 14:43 | |
| *** yamahata has quit IRC | 14:45 | |
| *** yamahata has joined #tripleo | 14:45 | |
| *** jprovazn has quit IRC | 14:47 | |
| *** jpena_ has quit IRC | 14:49 | |
| jrist | dprince: I like your shirt | 14:49 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Add a function to upgrade from full HA to NG HA https://review.openstack.org/358626 | 14:50 |
| jaosorior | EmilienM: can you check this one out? https://review.openstack.org/#/c/353037/ | 14:51 |
| dprince | jrist: thanks :) | 14:51 |
| slagle | rbrady: can i mark these as implemented? https://blueprints.launchpad.net/tripleo/+spec/plan-list-action and https://blueprints.launchpad.net/tripleo/+spec/plan-delete-action | 14:52 |
| EmilienM | I don't see dprince's tee shirt | 14:53 |
| rbrady | slagle: yes | 14:53 |
| EmilienM | jaosorior: looking | 14:53 |
| *** coolsvap_ has joined #tripleo | 14:53 | |
| dprince | EmilienM: Triple(Owl) man | 14:54 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Create entries for overcloud VIPs in /etc/hosts https://review.openstack.org/357765 | 14:54 |
| pradk | can someone please +a these https://review.openstack.org/#/c/359387/ | 14:54 |
| EmilienM | dprince: ++ | 14:54 |
| EmilienM | jaosorior: +A | 14:54 |
| jaosorior | EmilienM: this one starts using it in CI | 14:54 |
| jaosorior | https://review.openstack.org/#/c/357791/ | 14:55 |
| *** myoung is now known as myoung|afk | 14:55 | |
| *** dobson has quit IRC | 14:55 | |
| rook | lucasagomes: ping - got another scenario for ya. | 14:56 |
| *** HenryG has quit IRC | 14:56 | |
| *** ppradhan has joined #tripleo | 14:57 | |
| openstackgerrit | Derek Higgins proposed openstack/tripleo-heat-templates: Remove outdated upgrade steps https://review.openstack.org/361276 | 14:57 |
| rook | lucasagomes: i have seen where the provisioning interface fails to DHCP (dunno why) but rebooting the hosts gets it unwedged. it owuld be cool if ironic didn't get stuck and had a timeout period of waiting to run a command on a remote host, and if it times out ironic restarts the host? | 14:57 |
| jaosorior | EmilienM: thanks dude | 14:58 |
| ppradhan | is there a way to login to overcloud nodes?. I have a failed deployment and need to check the network interfaces | 15:00 |
| *** jistr is now known as jistr|call | 15:01 | |
| *** HenryG has joined #tripleo | 15:01 | |
| jaosorior | ppradhan: openstack server list will show you the addresses of the nodes | 15:02 |
| jaosorior | then you can ssh to them | 15:02 |
| jaosorior | ssh heat-admin@$NODE_IP | 15:02 |
| *** paramite has quit IRC | 15:03 | |
| ppradhan | jaosorior: I could'nt ssh/ping the nodes after NetwrokDeployment | 15:03 |
| *** rcernin has quit IRC | 15:04 | |
| *** dobson has joined #tripleo | 15:05 | |
| ppradhan | they pxe boot fine and NetwokDeploy stucked for hours and failed with Authentican error | 15:05 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Add -L option for curl during image downloads https://review.openstack.org/360665 | 15:05 |
| *** dprince has quit IRC | 15:07 | |
| *** dprince has joined #tripleo | 15:07 | |
| *** dprince has quit IRC | 15:08 | |
| *** dprince has joined #tripleo | 15:08 | |
| *** maeca1 has joined #tripleo | 15:10 | |
| openstackgerrit | Merged openstack/os-net-config: Add support for OVS DPDK Bond https://review.openstack.org/349183 | 15:11 |
| lucasagomes | rook, right, so the machine gets reboot and fail to PXE boot the first time | 15:12 |
| lucasagomes | rook, then you reboot it and it works ? | 15:12 |
| rook | neg, not PXE | 15:12 |
| rook | the host provisioned just fine, but when it loads the OS the prov. interface fails to get a DHCP addy | 15:12 |
| *** jistr|call is now known as jistr | 15:12 | |
| rook | I have seen this as 1 or two hosts. | 15:12 |
| openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud: Stop using deprecated ironic::drivers::deploy https://review.openstack.org/361291 | 15:12 |
| rook | so, the deployment is still rolling -- so i manually issue a ironic node-set-power-state <bad-node> off | 15:13 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Add swap usage to dstat capture https://review.openstack.org/359515 | 15:13 |
| rook | then on | 15:13 |
| *** mcornea has quit IRC | 15:13 | |
| rook | lucasagomes: which fixes the dhcp issue | 15:13 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Remove white space between print and () https://review.openstack.org/341482 | 15:13 |
| lucasagomes | rook, oh right... and looking at the console does it give any hint why it failed to get an addr ? | 15:16 |
| slagle | rbrady: is https://blueprints.launchpad.net/tripleo/+spec/template-processing-action completed by https://review.openstack.org/#/c/337615/? | 15:17 |
| lucasagomes | rook, maybe we need to make it more failure resistent and have a service or something that keeps trying to dhcp | 15:17 |
| rbrady | slagle: yes | 15:17 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-quickstart: Add objectstorage flavor and extra blockdevices option https://review.openstack.org/359630 | 15:19 |
| slagle | rbrady: same for https://blueprints.launchpad.net/tripleo/+spec/deployment-workflow and https://review.openstack.org/#/c/298732/? | 15:19 |
| shardy | larsks: Does the whiteboard for https://blueprints.launchpad.net/tripleo/+spec/tripleo-opstools-centralized-logging reference all the outstanding ops/logging integration patches? | 15:20 |
| rbrady | slagle: yep | 15:20 |
| rook | lucasagomes: sure - maybe os-collect-config check that the provisioning interface can ping the pxe host? | 15:20 |
| slagle | rbrady: :), https://blueprints.launchpad.net/tripleo/+spec/add-parameters-actions completed by https://review.openstack.org/#/c/298682/? | 15:20 |
| shardy | Doesn't look like it and I'm trying to figure out if we just need reviews, or if there are more patches todo | 15:21 |
| rbrady | slagle: yes | 15:21 |
| karthiks | EmilienM, Can you have a look at https://review.openstack.org/#/c/343770/ ? | 15:21 |
| EmilienM | karthiks: currently working your puppet-nova stuff, please hold on a bit | 15:22 |
| karthiks | Sure EmilienM | 15:22 |
| *** bootsha has joined #tripleo | 15:22 | |
| larsks | shardy: https://review.openstack.org/#/q/topic:%5Ebp/tripleo-opstools-centralized-logging+is:open | 15:22 |
| lucasagomes | rook, so, what is the state of the ironic node ? | 15:22 |
| lucasagomes | provision state* | 15:23 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: Composable Mistral services https://review.openstack.org/323436 | 15:23 |
| shardy | larsks: Ok, thanks - the BP hasn't been tracked for newton-3, but I'll add it so we can track getting those landed | 15:23 |
| slagle | rbrady: jpeeler : is https://blueprints.launchpad.net/tripleo/+spec/change-capabilities-input fully completed by https://review.openstack.org/#/c/349732/? | 15:24 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: Do not merge, checking composable mistral. https://review.openstack.org/352947 | 15:24 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: update the native quickstart to use a variable publish location https://review.openstack.org/361304 | 15:24 |
| rbrady | slagle: yes | 15:25 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Customize undercloud and overcloud with virt-customize https://review.openstack.org/361112 | 15:25 |
| *** bootsha has quit IRC | 15:27 | |
| atom_ | Hi, I've spawned up a vagrant box to test instack and set up the undercloud - however I'm getting a rather cryptic error https://paste.fedoraproject.org/414393/22250911/ | 15:27 |
| atom_ | does anybody know why this could be happening? | 15:27 |
| atom_ | this VM has 8G of RAM and plenty of HDD | 15:28 |
| atom_ | I executed instack with this snippet | 15:28 |
| atom_ | https://paste.fedoraproject.org/414394/72225203/ | 15:28 |
| beagles | atom_, nested virtualization support isn't enabled I think | 15:28 |
| atom_ | but in the docs it says that the undercloud can be provisioned in a virtual machine :-/ | 15:29 |
| larsks | beagles: atom_ : I am puzzled by that chattr failure. What sort of filesystem is /var/lib/libvirt/images? | 15:29 |
| beagles | atom_, at least that was the standard answer whenever something like that showed up in the past | 15:29 |
| *** maeca1 has quit IRC | 15:29 | |
| *** derekh has quit IRC | 15:30 | |
| slagle | d0ugal: would https://blueprints.launchpad.net/tripleo/+spec/roles-list-action be better assigned to you instead of jpeeler ? | 15:31 |
| slagle | looked like you may have taken over https://review.openstack.org/#/c/330283/ | 15:31 |
| rook | lucasagomes: available | 15:31 |
| atom_ | larsks: it's an xfs partition | 15:32 |
| *** pradk has quit IRC | 15:32 | |
| bnemec | atom_: The undercloud can be installed in a VM, but you can't run instack-virt-setup from there. You install it directly in that case. | 15:32 |
| lucasagomes | rook, ? I mean, when that error that you have to reboot the node occur | 15:33 |
| lucasagomes | rook, what is the provision state at that time ? | 15:33 |
| *** pcaruana has quit IRC | 15:33 | |
| *** pradk has joined #tripleo | 15:33 | |
| *** pblaho has joined #tripleo | 15:34 | |
| rook | ah lucasagomes so I simply have a ping of nova list (grabbing IP) | 15:34 |
| atom_ | bnemec: what do you mean? how can I install it directly? | 15:34 |
| *** myoung|afk is now known as myoung | 15:34 | |
| rook | and if it fails after deploying, i know the host is fubar | 15:34 |
| atom_ | so instack-virt-setup should be ran from a RHEL-based baremetal server, and that would deploy a virtual instance of the undercloud? | 15:34 |
| beagles | hah, good point | 15:34 |
| bnemec | atom_: Right, that's the idea. | 15:34 |
| bnemec | atom_: To install directly, skip the instack-virt-setup part and go straight to http://docs.openstack.org/developer/tripleo-docs/installation/installation.html#installing-the-undercloud | 15:35 |
| bnemec | Although you won't be able to use it for anything unless you've also configured other vms for it to control. | 15:35 |
| bnemec | Which gets...complicated. | 15:35 |
| beagles | heheh | 15:35 |
| *** bootsha has joined #tripleo | 15:36 | |
| *** numans has quit IRC | 15:36 | |
| atom_ | bnemec: I'm just getting started with tripleo, wanted to see how the instalation of the undercloud was done :) thanks | 15:37 |
| atom_ | I'll request a physical server to run instack-virt-setup | 15:38 |
| atom_ | thanks | 15:38 |
| *** tesseract- has quit IRC | 15:38 | |
| d0ugal | I just rebased it, but yeah, I could take it on. | 15:38 |
| d0ugal | slagle: ^ | 15:38 |
| bnemec | atom_: Sounds good. That's your best bet at this point. | 15:39 |
| d0ugal | slagle: I'll check with jpeeler | 15:39 |
| jpeeler | d0ugal: either way, i think you simply rebased it. and i didn't actually write it | 15:39 |
| d0ugal | jpeeler: hah, we are equally unqualified :) | 15:39 |
| jpeeler | d0ugal: i can handle rebasing it again though... | 15:40 |
| d0ugal | jpeeler: Thanks | 15:41 |
| *** nyechiel has quit IRC | 15:43 | |
| *** hewbrocca is now known as hewbrocca-afk | 15:51 | |
| *** zoliXXL is now known as zoli|gone | 15:57 | |
| *** jaosorior has quit IRC | 15:58 | |
| zoli|gone | have a good weekend | 15:58 |
| pabelanger | EmilienM: bnemec: slagle: do you mind helping land https://review.openstack.org/#/c/357308/? This is for our effort to split ovb nodes and nodepool nodes into different proces | 15:59 |
| pabelanger | projects* | 15:59 |
| *** zoli|gone is now known as zoli_gone-proxy | 15:59 | |
| *** milan has quit IRC | 16:01 | |
| EmilienM | pabelanger: +2 now, want to see dprince approving it | 16:02 |
| *** atom_ has quit IRC | 16:03 | |
| bnemec | Yeah, I'm fine with that patch since it doesn't change anything major, but I didn't hear a resolution to the discussion yesterday about whether we should proceed with this. | 16:03 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Relocate changes to staged publishing. https://review.openstack.org/355720 | 16:04 |
| dprince | EmilienM, pabelanger: I don't think this is critical. Lets hold on it because it because there is risk and little benefit to doing this now I think vs. waiting a week or two | 16:05 |
| *** rkolli has joined #tripleo | 16:05 | |
| *** chlong has quit IRC | 16:07 | |
| dprince | pabelanger: I don't want to make any risky CI changes that aren't absolutely required for the next few weeks FWIW | 16:07 |
| *** pkovar1 has quit IRC | 16:08 | |
| dprince | pabelanger: I think this has to be manually deployed anyway on our te-broker so unless it is already deployed I'd like to hold on it | 16:09 |
| openstackgerrit | Merged openstack/os-net-config: Supporting numbered nics for DPDK Port and DPDK Bond https://review.openstack.org/359610 | 16:09 |
| *** rasca has quit IRC | 16:10 | |
| bnemec | dprince: I'm pretty sure this script gets updated every hour from whatever is in git. | 16:10 |
| bnemec | There's one script in te-broker that doesn't because it requires a destructive restart to apply, but the rest will take effect automatically. | 16:11 |
| *** pkovar has joined #tripleo | 16:11 | |
| dprince | bnemec: oh, okay. Maybe we postponed that a few weeks back. But good to know | 16:11 |
| dprince | bnemec: what I mean is we had breakages last time we changed the credentials to the cloud | 16:12 |
| dprince | bnemec: in troubleshooting it both derek and I were changing things on the fly... and ultimately the infra change that broke us was reverted | 16:12 |
| bnemec | dprince: Yeah, I'm fine with holding off on this. | 16:12 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Add a function to upgrade from full HA to NG HA https://review.openstack.org/358626 | 16:17 |
| *** chlong has joined #tripleo | 16:19 | |
| *** vijaykc4 has quit IRC | 16:21 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Add ci-script to upload images in OVB https://review.openstack.org/355476 | 16:22 |
| openstackgerrit | Karthik S proposed openstack/puppet-tripleo: Configure the numvfs for SRIOV interfaces https://review.openstack.org/361350 | 16:23 |
| *** bootsha has quit IRC | 16:25 | |
| *** morazi has quit IRC | 16:25 | |
| beagles | ummm | 16:25 |
| *** akrivoka has quit IRC | 16:26 | |
| beagles | oh crap | 16:26 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add deployment of CAs via hieradata https://review.openstack.org/353037 | 16:27 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart: Add documentation for add-on roles https://review.openstack.org/346733 | 16:29 |
| beagles | shardy... I was just looking at puppet/services/neutron-sriov-agent.yaml. Aren't we missing something there? Don't we have to include the neutron-base and pull into the those config settings or can we skip it? | 16:29 |
| *** dtantsur is now known as dtantsur|afk | 16:30 | |
| beagles | it seems like rabbit passwords might be required at the very least | 16:30 |
| *** fragatina has joined #tripleo | 16:31 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement scenario001 CI job https://review.openstack.org/360039 | 16:31 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Test new scenario001 job https://review.openstack.org/360040 | 16:31 |
| *** fragatina has quit IRC | 16:33 | |
| gfidente | are we deploying horizon in the overcloud? | 16:33 |
| *** fragatina has joined #tripleo | 16:33 | |
| gfidente | (in ci) | 16:34 |
| *** jdob has quit IRC | 16:34 | |
| * beagles concedes it might just *happen* to work because that neutron stuff ends up on the compute nodes anyways because of "other stuff" | 16:35 | |
| *** jdob has joined #tripleo | 16:35 | |
| trozet | dprince, shardy: i fixed the merge conflict: https://review.openstack.org/#/c/355147/ | 16:35 |
| shardy | beagles: Hmm, it wasn't that clear looking at the puppet stuff if it was needed, but it sounds like you're saying it is ;) | 16:36 |
| *** david-lyle has joined #tripleo | 16:36 | |
| shardy | beagles: indeed, if any other service includes the rabbit stuff on the same node, it will still work | 16:36 |
| trozet | also shardy if you could please review the dependent patch for puppet: https://review.openstack.org/#/c/335173/ | 16:36 |
| shardy | beagles: thanks for catching that, feel like pushing a patch with a fix? :) | 16:36 |
| beagles | shardy, I don't know ;) I'm guessing it will end up okay | 16:37 |
| dprince | trozet: +2 | 16:37 |
| beagles | shardy, already on it :) | 16:37 |
| *** milan has joined #tripleo | 16:37 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: More cleanup of controller.yaml https://review.openstack.org/360751 | 16:37 |
| *** lucasagomes is now known as lucas-afk | 16:39 | |
| *** electrofelix has quit IRC | 16:39 | |
| *** jpich has quit IRC | 16:40 | |
| *** jobewan has quit IRC | 16:41 | |
| *** jobewan has joined #tripleo | 16:42 | |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart: Allow reusable sockets across multiple quickstart.sh calls https://review.openstack.org/361358 | 16:42 |
| shardy | trozet: done - how shall we handle the blueprint for newton? | 16:42 |
| shardy | https://blueprints.launchpad.net/tripleo/+spec/opendaylight-integration | 16:42 |
| shardy | can we close that when these two patches land, then follow up with bugs if needed to get the ODL service to run on a non-controller role? | 16:43 |
| shardy | seems like the controller integrated case is an achievable first step for newton-3, then we'll need custom roles to enable the next step? | 16:43 |
| *** pradk has quit IRC | 16:44 | |
| *** links has joined #tripleo | 16:44 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml https://review.openstack.org/361362 | 16:44 |
| dprince | gfidente: ^^^ could we just go with that? | 16:44 |
| *** rcernin has joined #tripleo | 16:45 | |
| gfidente | dprince, yep | 16:45 |
| dprince | gfidente: I removed the entire set of Nova, Neutron, and Horizon configs... not just bind IPs | 16:45 |
| pabelanger | dprince: EmilienM: slagle: Thanks for the review of 357308. I have left a reply about the feature freeze on CI. I'd like to discuss it more since this does affect openstack-infra today (comments explain more) | 16:45 |
| *** pradk has joined #tripleo | 16:45 | |
| pabelanger | also, if anybody can review python-tripleoclient https://review.openstack.org/#/c/353929/ adds bindep support | 16:46 |
| pabelanger | which is needed for the experimental job I am working on | 16:46 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement non-ovb overcloud update job - Newton -> Newton https://review.openstack.org/351330 | 16:49 |
| pabelanger | dprince: EmilienM: slagle: If the concern is about breaking tripleo-test-cloud-rh1, we can do the work in tripleo-test-cloud-rh2 first, demo everything works correctly then roll the change to tripleo-test-cloud-rh1 | 16:49 |
| dprince | pabelanger: ack, I replied. I think we agree. It can happen before Barcelona I think. But lets just hold off on it for a few weeks | 16:49 |
| *** bana_k has joined #tripleo | 16:50 | |
| dprince | pabelanger: I'd prefer to just hold the change in general | 16:50 |
| dprince | pabelanger: to me this still doesn't qualify as a critical thing | 16:50 |
| dprince | pabelanger: annoying, yes. But nothing close to critical | 16:50 |
| *** gfidente has quit IRC | 16:50 | |
| *** gfidente has joined #tripleo | 16:51 | |
| gfidente | dprince, I think I spotted a couple of things and commented, but hopefully it passes then | 16:51 |
| pabelanger | dprince: Okay, I'll follow up in a week or so | 16:51 |
| gfidente | good weekend all | 16:52 |
| dprince | gfidente: ack, thanks | 16:52 |
| EmilienM | gfidente: take care | 16:52 |
| gfidente | if earthquakes agree | 16:53 |
| EmilienM | :( | 16:54 |
| gfidente | yeah not a friday topic | 16:54 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add base neutron service configuration https://review.openstack.org/361367 | 16:55 |
| *** mburned is now known as mburned_out | 16:56 | |
| beagles | dprince, in files like puppet/services/neutron-compute-plugin-nuage.yaml that aren't using heat parameters like ServiceNetMap in their outputs, can we just removed the unused parameters? | 16:57 |
| beagles | s/removed/remove | 16:57 |
| *** bfournie has quit IRC | 16:58 | |
| *** mlupton has joined #tripleo | 16:58 | |
| dprince | beagles: no, they are required because of puppet/services/service.yaml | 16:58 |
| beagles | ah | 16:58 |
| beagles | dprince, thanks | 16:58 |
| dprince | beagles: there are essentially a few parameters that get passed in explicitly via the Heat ResourceChain. Those parameters have to be there. | 16:59 |
| beagles | dprince, ServiceNetMap, DefaultPasswords and EndpointMap? | 16:59 |
| dprince | beagles: we typically have a check for required parameters in tools/validate-yaml.py in t-h-t too | 16:59 |
| *** jpena is now known as jpena|off | 16:59 | |
| *** gfidente has quit IRC | 16:59 | |
| dprince | beagles: so if you run tox -epep8 in your t-h-t checkout it should complain if they are missing | 16:59 |
| beagles | dprince, oh crap.. I knew that, and if they truly are unused they'll show up as a warning right? | 17:00 |
| *** morazi has joined #tripleo | 17:00 | |
| dprince | beagles: yeah | 17:00 |
| beagles | dprince, cool, thanks | 17:00 |
| dprince | beagles: we should skip the warning for the required ones. That is a bug in the test that should be fixable I think | 17:00 |
| *** chlong has quit IRC | 17:02 | |
| *** bootsha has joined #tripleo | 17:02 | |
| *** vijaykc4 has joined #tripleo | 17:03 | |
| *** ioggstream has quit IRC | 17:03 | |
| *** vijaykc4 has quit IRC | 17:09 | |
| *** pkovar has quit IRC | 17:10 | |
| *** vijaykc4 has joined #tripleo | 17:10 | |
| beagles | dprince, just an fyi https://review.openstack.org/#/c/361384/ | 17:11 |
| *** [1]cdearborn has joined #tripleo | 17:11 | |
| dprince | beagles: +2 | 17:13 |
| *** mburned_out is now known as mburned | 17:14 | |
| *** mwhahaha has quit IRC | 17:16 | |
| *** bana_k has quit IRC | 17:22 | |
| *** rcernin has quit IRC | 17:22 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services https://review.openstack.org/360062 | 17:23 |
| *** fragatina has quit IRC | 17:23 | |
| *** yamahata has quit IRC | 17:25 | |
| *** cdearborn has quit IRC | 17:25 | |
| dprince | EmilienM: I swipped these from gfidente so I'm +2'ing as well | 17:26 |
| dprince | EmilienM: https://review.openstack.org/#/c/360062/ | 17:26 |
| EmilienM | dprince: ack will review shortly | 17:26 |
| *** bfournie has joined #tripleo | 17:27 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates: M/N upgrade set scheduler_host_manager right. https://review.openstack.org/358005 | 17:28 |
| *** tosky has quit IRC | 17:28 | |
| openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Add List Roles Action https://review.openstack.org/330283 | 17:33 |
| *** bana_k has joined #tripleo | 17:35 | |
| *** akrivoka has joined #tripleo | 17:35 | |
| *** jubapa has quit IRC | 17:39 | |
| *** bootsha has quit IRC | 17:42 | |
| *** rhallisey has quit IRC | 17:53 | |
| karthiks | EmilienM, what shall be the format of the input in THT ? we understand that check_array_of_hash () expects a string in json format | 17:54 |
| karthiks | https://github.com/openstack/puppet-nova/blob/40a62aba4182a9a418a9ab59b325a93cff1072bc/lib/puppet/parser/functions/check_array_of_hash.rb#L14 | 17:54 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml https://review.openstack.org/361362 | 17:54 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services https://review.openstack.org/360062 | 17:54 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller https://review.openstack.org/361398 | 17:54 |
| *** akrivoka has quit IRC | 17:55 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates: Refactor upgrade checks. https://review.openstack.org/357750 | 17:56 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates: Fix check of rpm-python. https://review.openstack.org/357022 | 17:56 |
| dprince | EmilienM: https://review.openstack.org/#/c/361362/ | 17:56 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates: M/N upgrade fix galera restart. https://review.openstack.org/354713 | 17:56 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates: M/M upgrade sahara-api fails to restart. https://review.openstack.org/358022 | 17:57 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/tripleo-heat-templates: Add fluentd client service https://review.openstack.org/353506 | 17:57 |
| *** bootsha has joined #tripleo | 17:58 | |
| *** fragatina has joined #tripleo | 17:58 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml https://review.openstack.org/361362 | 17:58 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller https://review.openstack.org/361398 | 17:58 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services https://review.openstack.org/360062 | 17:58 |
| EmilienM | karthiks: an hash | 17:58 |
| *** bootsha has quit IRC | 17:59 | |
| EmilienM | dprince: ack | 17:59 |
| openstackgerrit | Merged openstack/puppet-tripleo: Expose enable_combination_alarm setting https://review.openstack.org/359387 | 18:01 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix base service type inheriting ceilometer service pcmk templates https://review.openstack.org/360729 | 18:01 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix base service type inheriting gnocchi service templates https://review.openstack.org/360731 | 18:02 |
| *** fragatin_ has joined #tripleo | 18:02 | |
| *** yamahata has joined #tripleo | 18:03 | |
| karthiks | EmilienM, current THT review is - https://review.openstack.org/#/c/329415/, which is taking a json param and applying 'str_replace' to convert. | 18:05 |
| *** fragatina has quit IRC | 18:06 | |
| karthiks | EmilienM, if we give this way and make the default as [], then the hiera data looks like - nova::compute::pci_passthrough: '[]' | 18:06 |
| openstackgerrit | Merged openstack/tripleo-common: Add boot option argument to the Node Registration Mistral workflow https://review.openstack.org/348875 | 18:06 |
| openstackgerrit | Merged openstack/puppet-tripleo: Adds OpenDaylight https://review.openstack.org/335173 | 18:06 |
| karthiks | EmilienM, which fails the newly added empty check | 18:06 |
| trozet | dprince, shardy: thanks for the reviews on the patches. The puppet tripleO change went through. Do you mind setting workflow +1 on https://review.openstack.org/#/c/355147/ ? | 18:08 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Add a function to upgrade from full HA to NG HA https://review.openstack.org/358626 | 18:08 |
| dprince | trozet: once CI passes on it again we can send it,yes | 18:08 |
| dprince | trozet: I will look for it this afternoon | 18:08 |
| dprince | should be soon | 18:08 |
| trozet | dprince: oh didn't realize still waiting on the other jobs. thanks | 18:09 |
| EmilienM | karthiks: and '' ? | 18:09 |
| karthiks | EmilienM, we will test and get back.. | 18:09 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml https://review.openstack.org/361362 | 18:11 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller https://review.openstack.org/361398 | 18:11 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services https://review.openstack.org/360062 | 18:11 |
| *** rbrady has quit IRC | 18:12 | |
| *** rbrady has joined #tripleo | 18:12 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-common: Fix ACTIVE node detection. https://review.openstack.org/361402 | 18:12 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller https://review.openstack.org/361398 | 18:14 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-common: M/N upgrade: Fix ACTIVE node detection. https://review.openstack.org/361402 | 18:14 |
| chem|off` | new bug for M/N upgrade, when you have time, this is an easy one ^ | 18:14 |
| *** chem|off` is now known as chem | 18:14 | |
| *** bnemec is now known as beekneemech | 18:16 | |
| beekneemech | awk != easy :-P | 18:16 |
| *** jprovazn has joined #tripleo | 18:17 | |
| *** rbrady has quit IRC | 18:18 | |
| *** rbrady has joined #tripleo | 18:19 | |
| *** bootsha has joined #tripleo | 18:20 | |
| *** bootsha has quit IRC | 18:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: configure heat with new authtoken parameters https://review.openstack.org/361409 | 18:22 |
| *** dprince has quit IRC | 18:26 | |
| *** bnemec has joined #tripleo | 18:26 | |
| *** beekneemech has quit IRC | 18:26 | |
| *** bnemec is now known as beekneemech | 18:27 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: configure Heat with new authtoken parameters https://review.openstack.org/361412 | 18:28 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Extract /var/log and /etc from all nodes https://review.openstack.org/361164 | 18:29 |
| slagle | EmilienM: dtantsur|afk : i aim to please ^ | 18:29 |
| EmilienM | slagle: +2 | 18:30 |
| EmilienM | slagle: https://review.openstack.org/#/c/360039/ | 18:33 |
| EmilienM | slagle: look https://review.openstack.org/#/c/360040/ | 18:33 |
| EmilienM | scenario001 pass | 18:33 |
| EmilienM | sahara is working | 18:33 |
| *** shardy has quit IRC | 18:33 | |
| EmilienM | Overcloud pingtest SUCCEEDED | 18:33 |
| EmilienM | slagle: you'll notice I used a Sahara resource in the heat template: https://review.openstack.org/#/c/360039/16/templates/scenario001-multinode-pingtest.yaml | 18:34 |
| EmilienM | so Sahara is actually tested | 18:34 |
| slagle | cool | 18:35 |
| EmilienM | thanks for the +2 | 18:39 |
| openstackgerrit | Ade Lee proposed openstack/tripleo-heat-templates: Add Barbican to the overcloud - WIP https://review.openstack.org/352458 | 18:39 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Extract /var/log and /etc from all nodes https://review.openstack.org/361164 | 18:53 |
| EmilienM | slagle: did you miss the create the dir before? | 18:54 |
| slagle | beekneemech: EmilienM: i think it's right now ^^ | 18:55 |
| slagle | $WORKSPACE/logs already contains /etc for the primary node (some of it) and other logs | 18:55 |
| slagle | so i think we want it in there | 18:55 |
| EmilienM | beekneemech: if you have some time, can you look https://review.openstack.org/#/c/360039/ please? I want to go ahead with scenarios | 18:55 |
| EmilienM | slagle: ok fair enough | 18:55 |
| openstackgerrit | Merged openstack/python-tripleoclient: Add 'openstack overcloud plan delete' command https://review.openstack.org/350617 | 18:56 |
| karthiks | EmilienM, its nova::compute::pci_passthrough: '' | 18:57 |
| karthiks | in hieradata | 18:57 |
| karthiks | EmilienM, I think this should solve with the !empty () check added | 18:58 |
| EmilienM | karthiks: cool ! | 18:58 |
| *** rhallisey has joined #tripleo | 19:00 | |
| skramaja | beagles: https://review.openstack.org/#/c/361238/1/environments/neutron-ovs-dpdk.yaml@7 | 19:00 |
| skramaja | beagles: do you think constraints will be apt? | 19:01 |
| karthiks | Thanks EmilienM | 19:01 |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Remove the ceilo wsgi transient flag https://review.openstack.org/361424 | 19:01 |
| beagles | skramaja, not sure really.. 1s | 19:01 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-pacemaker: Fixes for race condition when adding primitives without OCF present https://review.openstack.org/356086 | 19:02 |
| skramaja | beagles: ok | 19:02 |
| beagles | skramaja, is there a problem with removing the defaults from the template? | 19:02 |
| *** fragatin_ has quit IRC | 19:03 | |
| beagles | skramaja, I think that is the most hardcore validation you can against empty values :) | 19:03 |
| *** fragatina has joined #tripleo | 19:04 | |
| skramaja | beagles: i am not sure, default is mandatory for parameter.. if it is set from env, whether default can be removed or not is not clear.. i have to test to figure it out.. | 19:04 |
| skramaja | beagles: but an empty string is aslo not valid.. DPDK PMD needs a core dedicated for sure.. | 19:05 |
| skramaja | beagles: without it vswitchd will fail.. | 19:05 |
| beagles | skramaja, IIRC (and this is just because I ran into it once) is that if you don't define a default in your heat template and try to use it, heat will choke and things will stop right there - which is what you want for required values | 19:05 |
| beagles | skramaja, but you are right empty values are also not allowed so I guess you could put something there for that as well.. sure | 19:06 |
| beagles | skramaja, actually a good idea if it is going to cause "bad things"^TM | 19:06 |
| karthiks | EmilienM, checked nova.conf and default: '' works with your patch. | 19:06 |
| skramaja | ok beagles i will add constraints.. thanks.. | 19:07 |
| EmilienM | karthiks: WOUHOUWH | 19:08 |
| EmilienM | sorry | 19:08 |
| openstackgerrit | Merged openstack/tripleo-common: Generate reference documentation for Workbooks https://review.openstack.org/341572 | 19:10 |
| *** aufi has quit IRC | 19:11 | |
| *** rkolli has quit IRC | 19:13 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT https://review.openstack.org/329415 | 19:13 |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT https://review.openstack.org/329415 | 19:16 |
| *** bootsha has joined #tripleo | 19:16 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Support promoting based on $TOCI_JOBTYPE https://review.openstack.org/361429 | 19:20 |
| *** bootsha has quit IRC | 19:21 | |
| *** rodrigods has quit IRC | 19:21 | |
| *** rodrigods has joined #tripleo | 19:21 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Modify sriov_numvfs config from puppet-neutron to tripleo https://review.openstack.org/361430 | 19:22 |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Modify sriov_numvfs config from puppet-neutron to tripleo https://review.openstack.org/361430 | 19:25 |
| *** ioggstream has joined #tripleo | 19:25 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Modify sriov_numvfs config from puppet-neutron to tripleo https://review.openstack.org/361430 | 19:26 |
| *** bootsha has joined #tripleo | 19:27 | |
| *** kbyrne has quit IRC | 19:29 | |
| *** rbowen has quit IRC | 19:30 | |
| karthiks | EmilienM, Could you please look at https://review.openstack.org/#/c/327705/ https://review.openstack.org/#/c/343770/ | 19:30 |
| karthiks | https://review.openstack.org/#/c/329415/ | 19:30 |
| EmilienM | I can't | 19:31 |
| *** bootsha has quit IRC | 19:31 | |
| EmilienM | I'm joking | 19:32 |
| EmilienM | let me some time to look | 19:32 |
| karthiks | :) | 19:32 |
| *** kbyrne has joined #tripleo | 19:32 | |
| EmilienM | karthiks: you missed the depends-on isn't? | 19:33 |
| karthiks | yes 1 min will add | 19:33 |
| *** links has quit IRC | 19:33 | |
| EmilienM | ok | 19:34 |
| EmilienM | other patches are +A | 19:34 |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT https://review.openstack.org/329415 | 19:35 |
| beagles | I've just gone and blown all the good-will I may have built with karthiks ;) | 19:36 |
| EmilienM | karthiks: +2 | 19:36 |
| karthiks | beagles, :) | 19:37 |
| beagles | EmilienM, if you can check out my comments on https://review.openstack.org/#/c/361350/ that'd be cool | 19:37 |
| karthiks | Thanks ebalduf | 19:37 |
| beagles | EmilienM, I dont' want to lead karthiks astray | 19:37 |
| karthiks | Thanks EmilienM | 19:37 |
| *** bootsha has joined #tripleo | 19:39 | |
| EmilienM | beagles: nice commment indeed | 19:39 |
| EmilienM | I agree | 19:39 |
| beagles | EmilienM, okay so I'm not out to lunch whew | 19:39 |
| EmilienM | ahah | 19:39 |
| *** dprince has joined #tripleo | 19:43 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Set Deploymnet Params https://review.openstack.org/361440 | 19:47 |
| karthiks | beagles, EmilienM Is it OK if we keep this tested code for n3 and enhance it later [OR] if it is must, do we have time on Monday before freeze to complete it? | 19:49 |
| karthiks | its just very late now.. 1.20AM | 19:50 |
| EmilienM | I guess it's fine | 19:50 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Set Deployment Parameters https://review.openstack.org/361440 | 19:50 |
| EmilienM | go to bed we see that on monday | 19:50 |
| karthiks | wuuuhooooo | 19:51 |
| *** bootsha has quit IRC | 19:51 | |
| beagles | karthiks, have a good weekend! | 19:52 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml https://review.openstack.org/361362 | 19:52 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller https://review.openstack.org/361398 | 19:52 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services https://review.openstack.org/360062 | 19:52 |
| karthiks | Have a nice weekend. beagles EmilienM | 19:53 |
| EmilienM | cheers | 19:53 |
| *** vijaykc4 has quit IRC | 19:53 | |
| *** rbowen has joined #tripleo | 20:00 | |
| *** jprovazn has quit IRC | 20:01 | |
| *** gregwork has quit IRC | 20:02 | |
| *** NachoDuck has quit IRC | 20:03 | |
| *** NobodyCam has quit IRC | 20:04 | |
| *** bootsha has joined #tripleo | 20:05 | |
| *** rbowen has quit IRC | 20:13 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Fix debug parameter in gnocchi-base.yaml https://review.openstack.org/361450 | 20:14 |
| dprince | EmilienM: pradk: ^^ | 20:14 |
| EmilienM | dprince: ack | 20:14 |
| * EmilienM afk 1h | 20:14 | |
| dprince | ironic is messed up too, fixing that now | 20:14 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Move ceilometer api to run under apache wsgi https://review.openstack.org/293642 | 20:15 |
| *** Goneri has quit IRC | 20:19 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Extract /var/log and /etc from all nodes https://review.openstack.org/361164 | 20:23 |
| *** rbowen has joined #tripleo | 20:24 | |
| *** jdob has quit IRC | 20:27 | |
| *** abregman has quit IRC | 20:31 | |
| *** NachoDuck has joined #tripleo | 20:33 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] replace qemu-kvm w/ qemu-kvm-ev https://review.openstack.org/356639 | 20:36 |
| *** bootsha has quit IRC | 20:36 | |
| *** jeckersb is now known as jeckersb_gone | 20:36 | |
| *** lblanchard has quit IRC | 20:37 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Fix Ironic service bind IPs https://review.openstack.org/361459 | 20:40 |
| openstackgerrit | Merged openstack/tripleo-common: M/N upgrade: Fix ACTIVE node detection. https://review.openstack.org/361402 | 20:47 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: WIP: Avoid using MysqlNetwork to set Ironic my_ip https://review.openstack.org/361464 | 20:49 |
| *** vijaykc4 has joined #tripleo | 20:50 | |
| *** gregwork has joined #tripleo | 20:52 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Customize vcpu_pin_set in nova.conf https://review.openstack.org/343770 | 20:53 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: replace qemu-kvm w/ qemu-kvm-ev https://review.openstack.org/356639 | 20:53 |
| *** vijaykc4 has quit IRC | 20:56 | |
| *** bfournie has quit IRC | 20:56 | |
| *** dprince has quit IRC | 20:57 | |
| *** mhenkel has quit IRC | 20:57 | |
| *** ayoung has joined #tripleo | 20:59 | |
| openstackgerrit | Ryan Brady proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command https://review.openstack.org/359784 | 20:59 |
| openstackgerrit | Ryan Brady proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 20:59 |
| *** florianf has quit IRC | 21:00 | |
| *** asbishop has quit IRC | 21:01 | |
| *** NobodyCam has joined #tripleo | 21:08 | |
| *** Goneri has joined #tripleo | 21:08 | |
| *** milan has quit IRC | 21:11 | |
| *** slagle has quit IRC | 21:12 | |
| *** rbowen has quit IRC | 21:12 | |
| pradk | EmilienM, ugh did we break ci by merging https://review.openstack.org/293642 .. the nonha job seems to fail on apache restart | 21:14 |
| *** morazi has quit IRC | 21:14 | |
| pradk | i'll revert and see | 21:14 |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates: Revert "Move ceilometer api to run under apache wsgi" https://review.openstack.org/361472 | 21:14 |
| *** mburned is now known as mburned_out | 21:14 | |
| *** akuznetsov has joined #tripleo | 21:16 | |
| pradk | the patch did pass ci.. so not sure why it would fail | 21:16 |
| EmilienM | pradk: I told you in private | 21:18 |
| EmilienM | because we needed the hiera param to enable ceilometer_wsgi... | 21:18 |
| pradk | weird then why did ci pass? | 21:18 |
| EmilienM | mhh | 21:19 |
| EmilienM | maybe it's unrelated | 21:19 |
| pradk | i can submit a patch to set that param and see if that passes | 21:19 |
| pradk | if so we can just merge that and not revert | 21:19 |
| EmilienM | where did you see the failure? | 21:19 |
| EmilienM | your puppet tripleo patch should do the job | 21:20 |
| pradk | yea | 21:20 |
| EmilienM | I think we need https://review.openstack.org/#/c/361424/ | 21:20 |
| EmilienM | actually I don't think I'm sure | 21:20 |
| pradk | i see some patches in the queue failing.. but the last one passed | 21:21 |
| EmilienM | ovb jobs are in queue | 21:21 |
| pradk | ok we can merge this and see https://review.openstack.org/#/c/361424/ | 21:21 |
| pradk | i abandoned the revert for now | 21:21 |
| EmilienM | is there a core around? | 21:21 |
| EmilienM | otherwise I'll approve it | 21:21 |
| EmilienM | slagle and dprince are not around | 21:22 |
| EmilienM | beekneemech: can you approve it maybe? | 21:22 |
| pradk | sorry man, i just went by ci and asked you to approve.. not your fault | 21:22 |
| EmilienM | no worries | 21:22 |
| EmilienM | it's not a big deal I think, we need this puppet tripleo patch | 21:23 |
| *** ioggstream has quit IRC | 21:23 | |
| EmilienM | I'm approving it, it should be good. | 21:23 |
| *** mhenkel has joined #tripleo | 21:23 | |
| beekneemech | That hasn't even had CI run on it yet. | 21:23 |
| beekneemech | Also, I would strongly prefer not to be moving services into apache at this point in the cycle. | 21:23 |
| pradk | yea its friday, i'm ok with waiting for ci run before approving | 21:23 |
| EmilienM | ok I didn't approve | 21:24 |
| EmilienM | I'll let ovb jobs to finish | 21:24 |
| beekneemech | I mean, I guess if we're broken then it won't make things any worse, but I'm not clear on whether that's the case from the previous discussion. | 21:25 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Adds OpenDaylight composable service https://review.openstack.org/355147 | 21:26 |
| pradk | yea i see some patches fail.. but some are merging so may be i'm over reacting here | 21:27 |
| pradk | for example https://review.openstack.org/#/c/361362/ .. nonha job fails | 21:29 |
| pradk | and the logs indicate httpd failed to restart hence i started panicking | 21:29 |
| EmilienM | pradk: no panic ! | 21:30 |
| EmilienM | pradk: in *any case* we need your patch | 21:30 |
| EmilienM | in puppet tripleo | 21:30 |
| pradk | yea we do | 21:30 |
| EmilienM | maybe beekneemech can already review it and +2 if good, and someone will +A when CI is good? | 21:30 |
| EmilienM | oh he did it alraedy | 21:30 |
| beekneemech | Already done. :-) | 21:30 |
| EmilienM | dude you're too fast | 21:31 |
| *** abregman has joined #tripleo | 21:31 | |
| beekneemech | I'm fine with merging that, it looks right. I just don't want us to get in the habit of circumventing tripleo-ci. | 21:31 |
| EmilienM | yes, you're right | 21:33 |
| EmilienM | let's wait for ovb | 21:33 |
| *** mhenkel has quit IRC | 21:33 | |
| EmilienM | I'll be online this evening | 21:33 |
| pradk | cool, thx guys | 21:33 |
| openstackgerrit | Merged openstack/instack-undercloud: Ironic: Configure iPXE to work with UEFI https://review.openstack.org/350627 | 21:35 |
| *** akuznetsov has quit IRC | 21:38 | |
| *** rhallisey has quit IRC | 21:48 | |
| *** ppradhan has left #tripleo | 22:00 | |
| openstackgerrit | Merged openstack/tripleo-docs: Mention nested guest support https://review.openstack.org/357223 | 22:01 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: WIP: Add directory to store documentation files https://review.openstack.org/361483 | 22:02 |
| openstackgerrit | Merged openstack/tripleo-docs: Add TripleO UI to the components docs https://review.openstack.org/340703 | 22:02 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Extract /var/log from all nodes https://review.openstack.org/361164 | 22:03 |
| *** rlandy has quit IRC | 22:04 | |
| *** abregman has quit IRC | 22:04 | |
| *** [1]cdearborn has quit IRC | 22:04 | |
| *** dmsimard is now known as dmsimard|pto | 22:12 | |
| *** fultonj has quit IRC | 22:14 | |
| openstackgerrit | Ryan Brady proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral https://review.openstack.org/342261 | 22:19 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml https://review.openstack.org/361362 | 22:27 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller https://review.openstack.org/361398 | 22:27 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services https://review.openstack.org/360062 | 22:27 |
| *** mlupton has quit IRC | 22:33 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Add 'openstack overcloud node import' command https://review.openstack.org/339727 | 22:54 |
| *** NobodyCam has quit IRC | 23:01 | |
| *** gregwork has quit IRC | 23:01 | |
| *** NachoDuck has quit IRC | 23:02 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Long options should go first in help for all the options https://review.openstack.org/347698 | 23:13 |
| *** NobodyCam has joined #tripleo | 23:15 | |
| *** jobewan has quit IRC | 23:16 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: M/N upgrade fail to restart nova-scheduler. https://review.openstack.org/358006 | 23:18 |
| *** gregwork has joined #tripleo | 23:20 | |
| *** NachoDuck has joined #tripleo | 23:29 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Initial support for bindep https://review.openstack.org/353929 | 23:39 |
| *** yamahata has quit IRC | 23:43 | |
| *** NachoDuck has quit IRC | 23:46 | |
| *** NobodyCam has quit IRC | 23:46 | |
| *** gregwork has quit IRC | 23:46 | |
| *** yamahata has joined #tripleo | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!