| *** rook has quit IRC | 00:07 | |
| *** rook has joined #tripleo | 00:07 | |
| *** david-lyle_ has joined #tripleo | 00:23 | |
| *** colonwq has quit IRC | 00:25 | |
| *** david-lyle_ has quit IRC | 00:27 | |
| *** jubapa has joined #tripleo | 00:34 | |
| *** jubapa has quit IRC | 00:39 | |
| *** limao has joined #tripleo | 00:39 | |
| *** saneax is now known as saneax_AFK | 00:42 | |
| *** mbound has joined #tripleo | 00:47 | |
| *** leanderthal|afk has quit IRC | 00:51 | |
| *** mbound has quit IRC | 00:52 | |
| *** chlong has quit IRC | 00:54 | |
| *** rain has joined #tripleo | 01:03 | |
| *** rain is now known as Guest3563 | 01:04 | |
| *** chlong has joined #tripleo | 01:11 | |
| *** bfournie has joined #tripleo | 01:13 | |
| *** chlong is now known as chlong_POffice | 01:21 | |
| *** akshai has joined #tripleo | 01:33 | |
| *** tbonds has quit IRC | 01:44 | |
| *** akshai has quit IRC | 01:44 | |
| *** tbonds has joined #tripleo | 01:52 | |
| *** coolsvap has joined #tripleo | 01:59 | |
| *** colonwq has joined #tripleo | 02:01 | |
| *** dmacpher has joined #tripleo | 02:01 | |
| *** ebalduf has quit IRC | 02:02 | |
| openstackgerrit | Xiang Chen proposed openstack/diskimage-builder: Explain the difference between 'DIB_OFFLINE' and 'DIB_DEBIAN_USE_DEBOOTSTRAP_CACHE' https://review.openstack.org/345935 | 02:19 |
|---|---|---|
| *** david-lyle_ has joined #tripleo | 02:25 | |
| *** david-lyle_ has quit IRC | 02:29 | |
| *** colonwq has quit IRC | 02:47 | |
| *** ebalduf has joined #tripleo | 02:49 | |
| *** colonwq has joined #tripleo | 02:49 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Add tests for building *-minimal images https://review.openstack.org/181162 | 03:02 |
| *** morazi has quit IRC | 03:24 | |
| *** david-lyle_ has joined #tripleo | 04:27 | |
| *** david-lyle_ has quit IRC | 04:32 | |
| *** jubapa has joined #tripleo | 04:34 | |
| *** jubapa has quit IRC | 04:39 | |
| *** saneax_AFK is now known as saneax | 04:41 | |
| *** oshvartz has quit IRC | 04:52 | |
| *** dmacpher_ has joined #tripleo | 05:01 | |
| *** dmacpher has quit IRC | 05:05 | |
| *** dmacpher_ is now known as dmacpher | 05:12 | |
| *** numans has joined #tripleo | 05:20 | |
| *** masco has joined #tripleo | 05:24 | |
| *** r-mibu has quit IRC | 05:41 | |
| *** radek_ has joined #tripleo | 05:54 | |
| *** skramaja has joined #tripleo | 05:54 | |
| *** skramaja has quit IRC | 05:55 | |
| *** r-mibu has joined #tripleo | 05:57 | |
| *** pgadiya has joined #tripleo | 06:04 | |
| *** rcernin has joined #tripleo | 06:05 | |
| *** skramaja has joined #tripleo | 06:10 | |
| openstackgerrit | Xiang Chen proposed openstack/diskimage-builder: check APT_SOURCES earlier https://review.openstack.org/346605 | 06:14 |
| *** oshvartz has joined #tripleo | 06:21 | |
| openstackgerrit | Saravanan KR proposed openstack/os-net-config: Add support for OVS DPDK Bridge and Port https://review.openstack.org/338694 | 06:23 |
| *** rasca has joined #tripleo | 06:28 | |
| *** jprovazn has joined #tripleo | 06:32 | |
| *** pcaruana has joined #tripleo | 06:37 | |
| *** liverpooler has joined #tripleo | 06:37 | |
| *** tesseract- has joined #tripleo | 06:41 | |
| *** bandini has joined #tripleo | 06:41 | |
| *** florianf has joined #tripleo | 06:44 | |
| *** milan has joined #tripleo | 06:50 | |
| *** pradiprwt has joined #tripleo | 06:52 | |
| pradiprwt | How to develop plugin for RHEL-OSP, Is there any document ..??? | 06:52 |
| *** tbonds has quit IRC | 06:52 | |
| pradiprwt | Is there any way to install undercloud and overcloud using virtual machines for testing ..??? | 06:54 |
| *** jtomasek has joined #tripleo | 06:55 | |
| *** dmacpher has quit IRC | 07:13 | |
| *** zoli_gone-proxy is now known as zoliXXL | 07:14 | |
| zoliXXL | good morning | 07:16 |
| *** ebarrera has joined #tripleo | 07:17 | |
| *** ccamacho has joined #tripleo | 07:18 | |
| *** jtomasek has quit IRC | 07:18 | |
| *** jtomasek has joined #tripleo | 07:19 | |
| *** paramite has joined #tripleo | 07:25 | |
| *** akuznetsov has joined #tripleo | 07:30 | |
| openstackgerrit | sandhya proposed openstack/diskimage-builder: Add support for building images capable of UEFI https://review.openstack.org/287784 | 07:30 |
| openstackgerrit | sandhya proposed openstack/diskimage-builder: Add support for building images capable of UEFI https://review.openstack.org/287784 | 07:32 |
| *** yamahata has quit IRC | 07:48 | |
| *** jpich has joined #tripleo | 07:52 | |
| *** ramishra has quit IRC | 07:52 | |
| *** ramishra has joined #tripleo | 07:54 | |
| *** Guest3563 is now known as leanderthal | 07:55 | |
| *** leanderthal is now known as leanderthal|afk | 07:55 | |
| *** pblaho has joined #tripleo | 07:55 | |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work https://review.openstack.org/314208 | 07:56 |
| bandini | pradiprwt: yes you can use virtual machines. take a look at the tripleo-quickstart project | 07:57 |
| *** jtomasek has quit IRC | 07:58 | |
| *** ebarrera has quit IRC | 07:58 | |
| *** jtomasek has joined #tripleo | 07:58 | |
| *** fzdarsky has joined #tripleo | 07:59 | |
| *** ebalduf has quit IRC | 07:59 | |
| *** fragatina has joined #tripleo | 08:02 | |
| *** mcornea has joined #tripleo | 08:05 | |
| *** jtomasek has quit IRC | 08:06 | |
| *** mbound has joined #tripleo | 08:07 | |
| *** shardy has joined #tripleo | 08:08 | |
| *** dsariel has joined #tripleo | 08:09 | |
| *** fragatina has quit IRC | 08:10 | |
| *** akuznetsov has quit IRC | 08:11 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Move nova::db data within service template https://review.openstack.org/335561 | 08:12 |
| *** dsariel has quit IRC | 08:16 | |
| *** dcotton has joined #tripleo | 08:27 | |
| *** snecklifter_ has joined #tripleo | 08:27 | |
| *** david-lyle_ has joined #tripleo | 08:31 | |
| *** david-lyle_ has quit IRC | 08:36 | |
| *** devvesa has joined #tripleo | 08:38 | |
| *** hewbrocca-afk is now known as hewbrocca | 08:39 | |
| pradiprwt | bandini: thanks for reply ...is this environment also have same functionalities as normal installation..?? | 08:39 |
| *** electrofelix has joined #tripleo | 08:40 | |
| *** pino|work has quit IRC | 08:41 | |
| *** derekh has joined #tripleo | 08:45 | |
| bandini | pradiprwt: it is the same yes, but being installed in virtual machine it obviously is more for proof of concepts | 08:53 |
| *** milan has quit IRC | 08:55 | |
| *** chem has joined #tripleo | 08:55 | |
| *** pino|work has joined #tripleo | 08:56 | |
| *** beagles has quit IRC | 08:56 | |
| *** milan has joined #tripleo | 08:59 | |
| *** limao has quit IRC | 08:59 | |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work https://review.openstack.org/314208 | 09:02 |
| *** limao has joined #tripleo | 09:03 | |
| *** jtomasek has joined #tripleo | 09:05 | |
| *** Goneri has joined #tripleo | 09:10 | |
| *** milan has quit IRC | 09:16 | |
| *** gfidente has joined #tripleo | 09:22 | |
| pradiprwt | bandini : I want to make some post installation changes to overcloud.... Is there is any way to make it such as plugin ..??? | 09:27 |
| shardy | https://review.openstack.org/#/q/topic:bug/1601857 and https://review.openstack.org/#/c/337803/ can probably land I think if anyone has a moment to take a look | 09:27 |
| shardy | pradiprwt: we have per-service templates which are kind of our "plugin" format now | 09:28 |
| shardy | http://docs.openstack.org/developer/tripleo-docs/tht_walkthrough/tht_walkthrough.html | 09:28 |
| *** mgould|afk is now known as mgould | 09:28 | |
| shardy | pradiprwt: you can also pass hieradata overrides to influence service configuration http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/node_config.html | 09:28 |
| shardy | or do specific pre or post-install tasks not handled via puppet: http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/extra_config.html | 09:29 |
| *** jubapa has joined #tripleo | 09:32 | |
| pradiprwt | shardy : Can we add new role like controller and compute ..? | 09:35 |
| marios | shardy: you got a pointer to the tht side which uses /#/c/337803/ handy? | 09:35 |
| *** limao_ has joined #tripleo | 09:36 | |
| *** limao has quit IRC | 09:36 | |
| shardy | pradiprwt: Not yet, but that is currently in-progress https://blueprints.launchpad.net/tripleo/+spec/custom-roles | 09:37 |
| *** jubapa has quit IRC | 09:37 | |
| marios | shardy: nm found it thnka | 09:37 |
| marios | thanks | 09:37 |
| marios | https://review.openstack.org/#/c/338551/ | 09:37 |
| shardy | pradiprwt: there are ways to do it by abusing the previous interfaces, but that's going to be the supported way of adding custom roles | 09:37 |
| marios | shardy: so sounds like you're ready to commit to what you have in the puppet class for now? I mean committed wrt it being required/depends on for the tht | 09:38 |
| shardy | marios: Yeah, it's basically just a cut/paste of the old ringbuilder.pp | 09:38 |
| marios | gfidente: just +2d that from under me | 09:39 |
| * marios shakes fist | 09:39 | |
| shardy | so we can land that as a refactoring step, like the rest of the composable services work | 09:39 |
| *** akrivoka has joined #tripleo | 09:39 | |
| shardy | I just made the comment in the commit message because I know we do have some additional work to do wrt ringbuilder, and I've not considered that in this refactoring patch | 09:39 |
| marios | shardy: ack thanks. gentle reminder on the manila if you get a chance. +1 on the 'migration review' imo this is the cleanest way to migrate things over th composable, fixes can come after, otherwise may not be clear what was done/why | 09:40 |
| marios | https://review.openstack.org/#/c/188137/ | 09:40 |
| shardy | Ah, I'm going to have to remove the Depends-On: I2cf59a1117156396b9da4104d8edbf49620396b1 because that patch can't land before this one | 09:41 |
| shardy | marios: ack, thanks will take a look now | 09:41 |
| marios | shardy: tht side you mean? | 09:41 |
| marios | https://review.openstack.org/#/c/345661/ | 09:41 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Convert Swift ringbuilder to composable services format https://review.openstack.org/338551 | 09:41 |
| marios | shardy: k, i was going to have a look at that but sounds ilke you'll revise | 09:42 |
| marios | heh | 09:42 |
| marios | k looking | 09:42 |
| pradiprwt | shardy : Is this role functionality going to supported in RHEL OSP 9 ..? | 09:42 |
| gfidente | marios hey I didn't even see you were chatting here | 09:42 |
| gfidente | I was actually looking at the submission | 09:42 |
| shardy | marios: I just pushed an edit, only changed the commit message | 09:42 |
| shardy | pradiprwt: no, it's under development for the current newton based cycle | 09:43 |
| * marios shakes fist at the faidentee. no one likes your excuses, giulio. | 09:43 | |
| *** pgadiya has quit IRC | 09:43 | |
| *** pgadiya has joined #tripleo | 09:43 | |
| pradiprwt | shardy : Can you please refer any link for implement role functionality. | 09:44 |
| shardy | pradiprwt: I just linked it https://blueprints.launchpad.net/tripleo/+spec/custom-roles | 09:45 |
| gfidente | shardy so swift-devices-and-proxy-config is effectively swift-base.yaml | 09:46 |
| *** fragatina has joined #tripleo | 09:47 | |
| shardy | gfidente: possibly, it's similar to the bind_ip problem, because those devices are output from the controller.yaml with the IPs etc | 09:47 |
| shardy | a subsequent step will to be refactoring swift-devices-and-proxy into the composable templates | 09:48 |
| shardy | but this one just moves the ringbuilder | 09:48 |
| gfidente | yep | 09:48 |
| marios | gfidente: shardy yeah... so i was just about to ask sthing else but i answered myself. so this on e is a bit different, since the puppet-modules are used in the existing puppet/config rather than puppet/services/swift-foo | 09:48 |
| gfidente | shardy I tried to cleanup the old ceph 'shared config' https://review.openstack.org/#/c/345273/ | 09:49 |
| gfidente | it didn't have "bind_ip" problem though | 09:49 |
| shardy | gfidente: Nice! :) | 09:50 |
| shardy | that's another barrier to custom-roles fixed, thanks! :) | 09:50 |
| gfidente | yeah looks like we can finally do a cleanup there | 09:50 |
| pradiprwt | shardy : I want to add new role in present deployment environment, how can I make it ....?? | 09:50 |
| shardy | pradiprwt: https://review.openstack.org/#/c/242267 shows one way to do it | 09:52 |
| shardy | it's not a clean interface though, it's an (ab)use of the current customization interfaces | 09:52 |
| shardy | maybe a workable interim solution for you, but be aware upgrading from that to the custom-roles interface may be tricky or even impossible | 09:53 |
| openstackgerrit | Merged openstack/python-tripleoclient: Adds ManilaPassword to the tripleoclient https://review.openstack.org/339437 | 09:53 |
| *** fragatina has quit IRC | 09:54 | |
| *** stendulker has joined #tripleo | 09:55 | |
| *** chem has quit IRC | 09:55 | |
| *** tosky has joined #tripleo | 09:55 | |
| pradiprwt | shardy : Thank you shardy, I will go through all links provides by you.. :) | 09:56 |
| shardy | pradiprwt: np :) | 09:56 |
| *** chem has joined #tripleo | 09:56 | |
| *** limao_ has quit IRC | 10:01 | |
| *** jubapa has joined #tripleo | 10:02 | |
| *** zoliXXL is now known as zoli|lunch | 10:03 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Remove NeutronEnableTunnelling from python-tripleoclient https://review.openstack.org/335234 | 10:06 |
| shardy | marios: Hey, it looks OK but some of my comments from ps79 remain unaddressed? | 10:06 |
| shardy | specifically, why can't the manila-generic stuff just be copied into a service template? | 10:06 |
| *** jubapa has quit IRC | 10:07 | |
| marios | shardy: thanks, so i thought we already agreed that the client/password thing was needed, but for the rest, it would be a migratino from what was proposed (with their extraconfig) and then we'd move later, as is the case with other services | 10:08 |
| shardy | marios: Ok, I guess I don't get why it can't just be cut/pasted now | 10:09 |
| marios | shardy: ok, just means I'll have to reise it later, then we'd need another ci pass. i can do that if you'd be happier, then we can try land it tomorrow | 10:09 |
| marios | shardy: i mean i'd like to run it through another test review first | 10:10 |
| marios | shardy: but thanks for the comments i see there are more of them and i'll respond and we take it from there | 10:10 |
| shardy | marios: sure, I'm not trying to be obstructive, but it could be confusing to folks if we land patches which are clearly a mixup of old/new style service integration? | 10:10 |
| shardy | it looks like it's a simple cut/paste job to me, but if it proves more complex them I'm OK with deferring it I guess | 10:11 |
| marios | shardy: yeah just i've seen it done for other stuff, some of the neutron/integrations were landed as is (with the existing extraconfig) to be fixed up later | 10:11 |
| shardy | re the passwords, we agreed to remove the defaults, so I'm not sure we need the password in the environment file now that we landed tripleoclient generation of it | 10:11 |
| marios | shardy: ok will do then | 10:11 |
| shardy | marios: I don't recall doing that, apart from perhaps for plumgrid, but that was a FFE and we wanted to backport it | 10:12 |
| marios | shardy: yeah didn't imply it was you i just mean i saw it done/discussed. this is a special case though since the 'existing' was just a review | 10:15 |
| marios | shardy: but i appreciate that someone else is also looking here so i'll do another pass for your comments today | 10:16 |
| *** panda|sick is now known as panda | 10:22 | |
| *** gchamoul is now known as gchamoul|afk | 10:26 | |
| *** gchamoul|afk is now known as gchamoul | 10:27 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Trove Integration https://review.openstack.org/233240 | 10:28 |
| shardy | gfidente: Re https://review.openstack.org/#/c/345273/2/overcloud.yaml - that changes things so the hieradata is now only deployed on the compute node | 10:32 |
| shardy | it looks OK, but can you confirm that deploying it on controller/cephstorage nodes is a historical mistake? | 10:33 |
| *** bfournie has joined #tripleo | 10:39 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Add swift ringbuilder profile https://review.openstack.org/337803 | 10:43 |
| ccamacho | shardy mind to check https://review.openstack.org/#/c/323436/ when having some time? | 10:44 |
| *** fragatina has joined #tripleo | 10:45 | |
| gfidente | shardy well it doesn't deploy the shared hieradata on computes either | 10:50 |
| gfidente | it's given by the role | 10:50 |
| gfidente | it wasn't a mistake to push it to all nodes before | 10:51 |
| gfidente | it was 'overly deployed' though | 10:51 |
| gfidente | the real problem with that submission is the one below it https://review.openstack.org/#/c/345277/1 | 10:52 |
| *** anshul has joined #tripleo | 10:52 | |
| gfidente | if I remove the depends_on the ceph nodes get to configure ceph-osd too early | 10:52 |
| gfidente | so I think if we want to land the cleanup I have to remove the dependency | 10:52 |
| gfidente | let me do it | 10:53 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Cleanup templates from the shared CephCluster config https://review.openstack.org/345273 | 10:55 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-heat-templates: GATE TEST, please ignore https://review.openstack.org/346698 | 10:58 |
| gfidente | shardy that shared cephcluster thing initially had a mix of hiera settings which were needed to the various nodes hosting a ceph service or acting as client | 11:01 |
| gfidente | then the data slowly relocated into roles | 11:01 |
| gfidente | I'm testing it locally | 11:02 |
| *** stendulker has quit IRC | 11:07 | |
| adarazs | derekh: hello! can you help me out with DLRN a bit? | 11:16 |
| adarazs | derekh: I'm having trouble with DLRN overwriting my checkouts. I had this workaround so far: https://github.com/redhat-openstack/ansible-role-tripleo-gate/blob/master/tasks/dlrn-build.yml#L29-L33 | 11:17 |
| adarazs | derekh: but now it doesn't seem to work. I looked around and I see this change: https://github.com/openstack-packages/DLRN/commit/194361456df6301f5646afdb3f4841388f958deb -- does this work with main repos or still only packaging ones? | 11:17 |
| derekh | adarazs: I can try, but you might have better luck in #rdo, I havn't really worked on delorean in nearly a year | 11:18 |
| *** fragatina has quit IRC | 11:19 | |
| adarazs | derekh: all right. I'm always confused which channels to use :) | 11:19 |
| adarazs | derekh: I asked you because I saw a recent commit from you, and jpena is on PTO, who seems to be committing regularly there. | 11:19 |
| derekh | adarazs: ok, this might be the same problem we had in tripleo-ci, one sec | 11:22 |
| shardy | gfidente: thanks for the additional info (sorry for slow reply, was in a meeting) | 11:23 |
| adarazs | derekh: hm, I tried --dev but it still gets overwritten :/ | 11:23 |
| derekh | adarazs: so are you using --local? | 11:24 |
| derekh | adarazs: see this thread http://lists.openstack.org/pipermail/openstack-dev/2016-July/099644.html | 11:24 |
| adarazs | $ delorean --config-file projects.ini --head-only --package-name openstack-tripleo-heat-templates --local --info-repo rdoinfo --build-env DELOREAN_DEV=1 --dev | 11:24 |
| derekh | adarazs: in particular, the behaviour of --local has changed, I've summarized it here http://lists.openstack.org/pipermail/openstack-dev/2016-July/099736.html | 11:26 |
| adarazs | derekh: reading the thread, thanks. | 11:26 |
| *** egafford has quit IRC | 11:26 | |
| derekh | adarazs: if that doesn't help out, just shout I'll I'll see if I can take a closer look | 11:26 |
| adarazs | derekh: thank you for the pointers | 11:26 |
| adarazs | reading through it | 11:26 |
| *** shardy is now known as shardy_lunch | 11:30 | |
| *** zoli|lunch is now known as zoli | 11:31 | |
| *** zoli is now known as zoliXXL | 11:32 | |
| *** david-lyle_ has joined #tripleo | 11:34 | |
| *** david-lyle_ has quit IRC | 11:39 | |
| *** rhallisey has joined #tripleo | 11:42 | |
| *** rbrady_ has quit IRC | 11:46 | |
| *** rodrigods has quit IRC | 11:47 | |
| *** weshay_afk is now known as weshay | 11:47 | |
| *** rodrigods has joined #tripleo | 11:47 | |
| *** thrash has quit IRC | 11:47 | |
| *** jdob has joined #tripleo | 11:48 | |
| *** pkovar has joined #tripleo | 11:48 | |
| *** skramaja has quit IRC | 11:55 | |
| *** tosky has quit IRC | 11:56 | |
| *** tosky has joined #tripleo | 11:57 | |
| *** milan has joined #tripleo | 11:59 | |
| *** morazi has joined #tripleo | 12:01 | |
| *** skramaja has joined #tripleo | 12:01 | |
| *** skramaja has quit IRC | 12:02 | |
| *** skramaja has joined #tripleo | 12:04 | |
| *** bfournie has quit IRC | 12:04 | |
| *** bfournie has joined #tripleo | 12:05 | |
| *** bfournie has quit IRC | 12:07 | |
| *** beagles has joined #tripleo | 12:09 | |
| *** masco has quit IRC | 12:09 | |
| *** Guest32906 is now known as flaper87 | 12:10 | |
| *** flaper87 has quit IRC | 12:10 | |
| *** flaper87 has joined #tripleo | 12:10 | |
| *** milan has quit IRC | 12:12 | |
| *** trown|outtypewww is now known as trown | 12:13 | |
| *** d0ugal_ is now known as d0ugal | 12:17 | |
| *** d0ugal has quit IRC | 12:17 | |
| *** d0ugal has joined #tripleo | 12:17 | |
| *** rbrady has joined #tripleo | 12:17 | |
| *** thrash has joined #tripleo | 12:22 | |
| *** thrash has joined #tripleo | 12:22 | |
| openstackgerrit | Justin Kilpatrick proposed openstack/tripleo-quickstart: Add documentation for addon roles https://review.openstack.org/346733 | 12:25 |
| EmilienM | hello | 12:26 |
| hewbrocca | you guys it's EmilienM | 12:26 |
| EmilienM | or his ghost | 12:26 |
| slagle | please stand up | 12:26 |
| EmilienM | did I break something? | 12:27 |
| hewbrocca | Only the mirror | 12:27 |
| hewbrocca | HAHAHA | 12:27 |
| *** tzumainn has joined #tripleo | 12:33 | |
| *** jprovazn has quit IRC | 12:35 | |
| *** rlandy has joined #tripleo | 12:41 | |
| *** bfournie has joined #tripleo | 12:43 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 12:45 |
| *** shardy_lunch is now known as shardy | 12:48 | |
| *** morazi has quit IRC | 12:50 | |
| *** jpeeler has quit IRC | 12:50 | |
| *** jpeeler has joined #tripleo | 12:51 | |
| *** fragatina has joined #tripleo | 12:52 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Trove Integration https://review.openstack.org/233240 | 12:52 |
| *** egafford has joined #tripleo | 12:52 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Trove Integration https://review.openstack.org/233240 | 12:53 |
| *** akuznetsov has joined #tripleo | 12:53 | |
| slagle | EmilienM: i believe i've reviewed all of your awesome uc patches | 12:54 |
| slagle | if i missed anything, let me know | 12:54 |
| EmilienM | thanks for your reviews, I am currently re-working the patches to address your comments and also to make it passing | 12:55 |
| EmilienM | the undercloud-only thing ^ | 12:55 |
| *** zaneb has joined #tripleo | 12:57 | |
| social | EmilienM: doesn't the package-upgrade exec break puppet package provider anyway? | 12:58 |
| *** jprovazn has joined #tripleo | 12:58 | |
| EmilienM | social: not if you execute after the Package resource | 12:58 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement basic undercloud sanity checks https://review.openstack.org/346551 | 13:00 |
| social | EmilienM: I'd preffer to have 50-puppet-stack-config puppet_apply ; yum update ; puppet_apply. on CI you could even make it red if package update outside of managed by puppet deps caused a change | 13:00 |
| *** akuznetsov has quit IRC | 13:00 | |
| EmilienM | the 2 puppet apply would be noop I guess | 13:01 |
| *** akuznetsov has joined #tripleo | 13:03 | |
| EmilienM | slagle: ha ! https://review.openstack.org/#/c/346147/6/scripts/deploy.sh | 13:04 |
| EmilienM | thats why the puppet run was failing :) thanks | 13:04 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 13:05 |
| *** snecklifter has quit IRC | 13:06 | |
| *** bootsha has joined #tripleo | 13:07 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Convert ceph_pools into a hash type https://review.openstack.org/346794 | 13:11 |
| EmilienM | gfidente: can you review https://review.openstack.org/#/c/344860/ and https://review.openstack.org/#/c/344985/ please? | 13:11 |
| EmilienM | composable firewall rules ^ | 13:11 |
| gfidente | EmilienM yeah sec | 13:12 |
| gfidente | morning there | 13:12 |
| *** jcoufal has joined #tripleo | 13:13 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Support arbitrary attributes for the Ceph pools https://review.openstack.org/346796 | 13:14 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: add composable services for Contrail Database, Config, Analytics, Control and Webui https://review.openstack.org/346799 | 13:17 |
| *** akshai has joined #tripleo | 13:17 | |
| *** jpich_ has joined #tripleo | 13:18 | |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: Use Webpack for application processing https://review.openstack.org/343592 | 13:19 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: Remove app.conf to use a JS-based config file https://review.openstack.org/343658 | 13:19 |
| *** tbonds has joined #tripleo | 13:20 | |
| *** jpich has quit IRC | 13:21 | |
| gfidente | EmilienM I will merge but I think 6800-7300 goes in ceph-osd.yaml | 13:22 |
| EmilienM | gfidente: ah, feel free to change it | 13:22 |
| *** bootsha has quit IRC | 13:22 | |
| *** snecklifter has joined #tripleo | 13:22 | |
| gfidente | EmilienM ack | 13:23 |
| EmilienM | gfidente: or land it and I can follow-up | 13:24 |
| EmilienM | gfidente: I just want it quickly to avoid rebases on this one | 13:24 |
| gfidente | EmilienM yeah maybe better to merge it first | 13:24 |
| EmilienM | good, let's do it! | 13:24 |
| gfidente | as the change is bigger than that | 13:24 |
| *** pgadiya has quit IRC | 13:26 | |
| *** mhenkel has joined #tripleo | 13:26 | |
| *** karthiks has quit IRC | 13:27 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Composable firewall rules https://review.openstack.org/344985 | 13:28 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Set MDS/OSD firewall ports from ceph-osd template https://review.openstack.org/346809 | 13:28 |
| gfidente | WTF I rebased the original one on master | 13:28 |
| *** anshul has quit IRC | 13:29 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 13:29 |
| *** anshul has joined #tripleo | 13:30 | |
| *** julim has joined #tripleo | 13:31 | |
| *** jpich_ is now known as jpich | 13:31 | |
| social | EmilienM: yes it should be noop but if issue happends with packaging it won't be and CI would catch that | 13:31 |
| EmilienM | social: i'm not sure to understand exactly what you means | 13:32 |
| *** weshay is now known as weshay_brb | 13:32 | |
| *** ayoung has joined #tripleo | 13:32 | |
| EmilienM | social: but feel free to comment on https://review.openstack.org/#/c/346124/ | 13:33 |
| *** bootsha has joined #tripleo | 13:33 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Remove tenant_id from nova v2.1 endpoint https://review.openstack.org/317036 | 13:34 |
| *** trown is now known as trown|brb | 13:34 | |
| social | EmilienM: deployment of undercloud by puppet means it should manage all packages required but yum update works on whole system. If situation happens when yum update triggers a change that makes secon't run of puppet not noop it means puppet or package is wrong | 13:34 |
| *** thrash has quit IRC | 13:34 | |
| *** skramaja is now known as skramaja_afk | 13:35 | |
| *** david-lyle_ has joined #tripleo | 13:36 | |
| *** mgarciam has joined #tripleo | 13:36 | |
| EmilienM | social: if you look at my patch, there are 2 things: Package<| tag == 'openstack' |> { ensure => latest } and Package<| tag == 'openstack' |> -> Exec['package-upgrade'] so we make sure we first upgrade OpenStack packages using Puppet package resource, that will trigger the services that we need and then we upgrade all the rest. So what you're saying is that upgrading the rest is a risk? | 13:37 |
| *** saneax is now known as saneax_AFK | 13:38 | |
| *** snecklifter has quit IRC | 13:38 | |
| *** zaneb has quit IRC | 13:39 | |
| *** zaneb has joined #tripleo | 13:39 | |
| social | EmilienM: yum update is always risk and should be done as standalone step ideally with reboot. Puppet should manage only things necessary for the deployment | 13:39 |
| EmilienM | social: this is true, until now, I didn't change the behavior, yum update was here before | 13:40 |
| social | EmilienM: yes, I'm strongly for removing it and having it as an separate operation | 13:41 |
| *** david-lyle_ has quit IRC | 13:41 | |
| EmilienM | shardy, slagle: do you have thoughts about keeping "yum update" into the undercloud puppet manifest? Should we make it "CI only" ? or is it intented? | 13:41 |
| *** karthiks has joined #tripleo | 13:41 | |
| *** karthiks has quit IRC | 13:41 | |
| *** karthiks has joined #tripleo | 13:41 | |
| social | EmilienM: puppet triggering dbsync on package upgrade is another issue but bit related to this, would be nice if we could have command to check if db-sync is necessary | 13:41 |
| *** thrash has joined #tripleo | 13:42 | |
| *** thrash has joined #tripleo | 13:42 | |
| shardy | EmilienM: I'm fine with puppet doing the yum update, but I would prefer that we always did a DB sync so things won't break if someone decides to do an out-of-band yum update for whatever reason | 13:42 |
| *** jeckersb_gone is now known as jeckersb | 13:43 | |
| shardy | social: wouldn't it be simpler to just always sync? On the undercloud we don't have to worry about service disruption, and all of the sync tools should do nothing if the version is already up to date? | 13:44 |
| *** skramaja_afk is now known as skramaja | 13:44 | |
| *** weshay_brb is now known as weshay | 13:44 | |
| *** morazi has joined #tripleo | 13:44 | |
| *** karthiks has quit IRC | 13:46 | |
| social | shardy: I'd preffer fixing the issue in openstack, eg provide command via which you can check if db sync is necessary and run it if needed | 13:47 |
| social | shardy: anything else sounds like hack on bug in openstack | 13:47 |
| shardy | social: sure, I'd support that if it was added to all the *sync commands | 13:47 |
| social | or just openstack-db ,) | 13:47 |
| shardy | not having every project use a slightly different syntax would be good too ;) | 13:48 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Enable Manila integration - as a composable controller service https://review.openstack.org/188137 | 13:48 |
| shardy | social: one question, how would having a command which is used for a conditional that then runs the sync differ from having a sync that does nothing when things are up to date? | 13:50 |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 13:50 |
| shardy | I guess the logging etc could me made clearer, but they seem functionally equivalent to me | 13:50 |
| openstackgerrit | Merged openstack/puppet-tripleo: Add base constraint so gnocchi metricd is tied to core-clone https://review.openstack.org/344822 | 13:51 |
| openstackgerrit | Merged openstack/puppet-tripleo: Add firewall::service_rules define https://review.openstack.org/344860 | 13:51 |
| social | shardy: if we run db-sync we disrupt the service? I know not caring about diruptions on undercloud is point now but I'd try to look bit more to the future | 13:51 |
| *** dsariel has joined #tripleo | 13:51 | |
| EmilienM | there is no disruption afik | 13:52 |
| *** akuznetsov has quit IRC | 13:52 | |
| *** milan has joined #tripleo | 13:52 | |
| EmilienM | and puppet already takes care of restarting the service if needed | 13:52 |
| *** akuznetsov has joined #tripleo | 13:52 | |
| *** akuznetsov has quit IRC | 13:53 | |
| social | EmilienM: so is there a reason for puppet to run db-sync only on package update? | 13:53 |
| shardy | you could do db_version, then sync, then only restart services if the version changed | 13:53 |
| *** trown|brb is now known as trown | 13:57 | |
| gfidente | have you guys seen recently overcloud deployment stuck on step4 when re-starting httpd? | 13:57 |
| EmilienM | shardy: I agree doing "yum update" at each puppet apply is not optimal. Is it what we want for production setups? | 13:57 |
| EmilienM | gfidente: yes, a sec | 13:57 |
| slagle | gfidente: there's a bug | 13:57 |
| gfidente | EmilienM oh cool | 13:57 |
| slagle | gfidente: looks like a httpd start timeout related to compressing js files | 13:58 |
| social | EmilienM: in production yum update can be even more disruptive. customer can have his own repos with conflicting packages for system libraries enabled and also just running yum update can leave system in unconsistent state | 13:58 |
| *** zoliXXL is now known as zoli|mtg | 13:58 | |
| slagle | EmilienM: i'd prefer to keep the yum update as the default behavior | 13:58 |
| EmilienM | slagle: ack | 13:59 |
| shardy | EmilienM: I imagine folks might like the choice to either only re-assert the configuration, or update then re-assert the configuration | 13:59 |
| shardy | personally I would find that useful | 13:59 |
| slagle | EmilienM: if for no other reason than that's what we've done, so there is an expectation that we always have latest packages | 13:59 |
| EmilienM | social: ^ | 13:59 |
| EmilienM | gfidente: I'm searching the bug URL | 13:59 |
| shardy | slagle: we could add a hiera value that toggles the update tho, and have undercloud install --noupdate or something? | 13:59 |
| EmilienM | https://bugs.launchpad.net/tripleo/+bug/1572976 | 14:00 |
| openstack | Launchpad bug 1572976 in tripleo "Periodically Apache httpd restart fails" [Medium,Triaged] | 14:00 |
| slagle | shardy: sure, optional is fine | 14:00 |
| EmilienM | gfidente: I think that's this one ^ | 14:00 |
| slagle | EmilienM: shardy : honestly, my hesitation to removing it is just that it'd have to be an added manual step | 14:01 |
| gfidente | EmilienM slagle interestingly I seem to be able to reproduce this consistently | 14:01 |
| slagle | "please make sure you have latest packages" | 14:01 |
| gfidente | maybe the env finally got slower than CI :) | 14:01 |
| slagle | EmilienM: shardy: but we could do that i suppose | 14:01 |
| EmilienM | slagle: it sounds risky, if people miss it | 14:01 |
| *** karthiks has joined #tripleo | 14:02 | |
| EmilienM | if we remove it from puppet apply, we need to document it | 14:02 |
| EmilienM | for now I would keep it | 14:02 |
| shardy | +1 to keep it, and maybe add an optional flag to disable | 14:02 |
| social | I'm for going ahead with EmilienM's patch + adding optional there | 14:03 |
| social | we might change it but not in newton | 14:03 |
| *** ebalduf has joined #tripleo | 14:07 | |
| *** fragatina has quit IRC | 14:08 | |
| *** fragatina has joined #tripleo | 14:08 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 14:09 |
| *** bootsha has quit IRC | 14:13 | |
| *** bootsha has joined #tripleo | 14:17 | |
| openstackgerrit | Steven Hardy proposed openstack/python-tripleoclient: WIP wire in jinja templating for custom roles https://review.openstack.org/336625 | 14:19 |
| *** anshul has quit IRC | 14:19 | |
| *** fultonj_ has joined #tripleo | 14:28 | |
| hewbrocca | slagle, shardy hey, so on this blueprint from paramite https://blueprints.launchpad.net/tripleo/+spec/tripleo-opstools-availability-monitoring | 14:28 |
| hewbrocca | re ops tools integration | 14:28 |
| hewbrocca | Can you guys provide some guidance for next steps? Details to gather and fill in, whatever? | 14:29 |
| hewbrocca | EmilienM: ^^^ you too | 14:29 |
| EmilienM | does it need a openstack/tripleo-specs ? | 14:30 |
| EmilienM | shardy: you ok with https://review.openstack.org/#/c/339236/ ? | 14:30 |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: WIP: Containerized Services for Composable Roles https://review.openstack.org/330659 | 14:34 |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Update heat-agents setup files https://review.openstack.org/346859 | 14:34 |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Add default value for `RoleData` https://review.openstack.org/346860 | 14:34 |
| *** Ryjedo has joined #tripleo | 14:35 | |
| *** pradiprwt has quit IRC | 14:35 | |
| shardy | hewbrocca: we might want a spec with some more details on that, but overall it looks OK | 14:37 |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 14:37 |
| shardy | we do need to figure how what to do re more converged deployments, e.g folks may want to deploy these tools on the undercloud node in some environments even if it potentially gives worse availability | 14:38 |
| shardy | particularly if we want to do anything in CI wrt this we'll probably need that | 14:38 |
| shardy | also, EmilienM already landed sensu support in the undercloud, so we need to figure out the ovelap there | 14:38 |
| shardy | paramite: ^^ | 14:39 |
| EmilienM | shardy: it's disable by default and it was a PoC | 14:39 |
| EmilienM | I guess we can change it | 14:39 |
| hewbrocca | shardy: thank you | 14:39 |
| *** jcoufal_ has joined #tripleo | 14:39 | |
| beekneemech | If we're going to do a hyperconverged setup for ops tools I'd rather converge on the overcloud. The compute nodes in CI are pretty un-loaded, so that might be good for testing. | 14:40 |
| *** beekneemech is now known as bnemec | 14:40 | |
| bnemec | Also a good test that composability works as expected. | 14:40 |
| paramite | shardy, ack, will work on the spec | 14:41 |
| paramite | spec(s) | 14:41 |
| *** paramite is now known as paramite|afk | 14:41 | |
| openstackgerrit | Merged openstack/os-net-config: Add support for Infiniband interfaces https://review.openstack.org/305039 | 14:41 |
| shardy | bnemec: Yeah, I was just thinking the same thing - if the new ops-stack reuses the same composable roles format that should be possible | 14:41 |
| EmilienM | bnemec: ++ | 14:41 |
| shardy | the ops stack could even be a custom role when we get that working, with the option for co-located services e.g for CI | 14:41 |
| *** jcoufal has quit IRC | 14:42 | |
| bnemec | shardy: +1 | 14:43 |
| openstackgerrit | Merged openstack/tripleo-docs: Add Liberty and Mitaka admonitions and use them https://review.openstack.org/330802 | 14:46 |
| bnemec | derekh: We have a bunch of patches in the CI queue, but I've only seen one testenv spun up since I started watching this morning. Any idea why? | 14:51 |
| derekh | bnemec: no idea, taking a look now | 14:51 |
| *** mburned has joined #tripleo | 14:52 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Properly direct MTU parameters and allow service defaults https://review.openstack.org/333333 | 14:53 |
| shardy | d0ugal, rbrady: Hey, what's the status of getting overcloud deploy working via mistral+tripleoclient? | 14:54 |
| shardy | I've found another thing I need to wire in between the plan generation and the overcloud deploy, so trying to figure out the best way to test it | 14:55 |
| rbrady | shardy: as of friday afternoon, I was still debugging. this morning I've been working on something else, planning to continue debugging after mistral upstream meeting | 14:55 |
| rbrady | shardy: what needs to be wired in? | 14:56 |
| shardy | rbrady: Ack, thanks - if you're able to prioritize it that would be great, as custom-roles is currently blocked on this | 14:56 |
| shardy | rbrady: I need two things, one is to run jinja2 to generate overcloud.yaml based on roles_data: https://review.openstack.org/#/c/336625/ | 14:56 |
| shardy | currently hacked that in via tripleoclient, but I am aware it'll need moving to a mistral action | 14:57 |
| shardy | the second thing is we need a way to map some old parameter values which aren't consistent with the naming created via custom-roles | 14:57 |
| derekh | bnemec: so looking at the zuul queue, there are 5 running jobs that have been running for some time and the rest have just started | 14:57 |
| d0ugal | I was blocked as the workflows didn't work when I tried, I'll see if I can help out more to finish them | 14:57 |
| *** dtrainor_ has joined #tripleo | 14:57 | |
| d0ugal | shardy: ^ | 14:57 |
| shardy | we'll have a deprecated mapping defined somewhere which can process user-parameters and warn if the old name is used | 14:58 |
| derekh | bnemec: and to correspond with this there are 5 quitupleo heat stacks | 14:58 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Explicitly set nova and neutron host on controllers https://review.openstack.org/304858 | 14:58 |
| *** fragatina has quit IRC | 14:58 | |
| *** fragatin_ has joined #tripleo | 14:58 | |
| derekh | bnemec: which looks correct | 14:58 |
| shardy | There is actually a third potential action, where we'll convert the RandomString resource values in overcloud.yaml into some parameter values on update | 14:58 |
| openstackgerrit | Merged openstack/tripleo-specs: Add expedited approvals policy https://review.openstack.org/339236 | 14:58 |
| bnemec | derekh: I see jobs on the zuul status page that have been queued for over 20 minutes. :-/ | 14:58 |
| bnemec | Does it really take that long for the job to get sent to the te-broker? | 14:59 |
| shardy | d0ugal, rbrady: ack, thanks for the update, let me know if there's anything I can do to help | 14:59 |
| shardy | hopefully we can get this working soon, i.e this week | 14:59 |
| *** dtrainor has joined #tripleo | 14:59 | |
| *** rcernin has quit IRC | 15:00 | |
| bnemec | derekh: Actually it's worse than that. There are a couple of jobs that have been there for an hour and a half and still don't have an env. | 15:00 |
| derekh | bnemec: have you an example? | 15:00 |
| d0ugal | shardy: If you are able to try the workflows and help figure out what is broken that would be good | 15:01 |
| derekh | bnemec: job start to testenv-client is usually about 6 or 7 minutes | 15:01 |
| d0ugal | shardy: I guess that is the next step for all of us | 15:01 |
| bnemec | derekh: 344985 | 15:01 |
| shardy | d0ugal: ack, I'll try to do that later and add notes to the review | 15:01 |
| derekh | bnemec: thats not how long its waiting on a test env, thats how long its waiting on a test slave, i.e. the job hasn't been started yet | 15:03 |
| bnemec | derekh: I wonder if there's a zuul issue. The check queue is also ridiculously long for a Monday morning. | 15:03 |
| *** dtrainor_ has quit IRC | 15:03 | |
| bnemec | derekh: Oh, six more envs just showed up. Strangely I don't see anything the te-broker logs. | 15:04 |
| rbrady | shardy: looking at the jinja patch, is this something that needs to happen once (e.g. during plan creation)? | 15:04 |
| bnemec | Ah, my ssh connection to that node hung. | 15:04 |
| *** milan has quit IRC | 15:04 | |
| shardy | rbrady: yes I think so, but not yet looked in detail at where in the workflow it should happen | 15:05 |
| openstackgerrit | Merged openstack/tripleo-common: Fix formatting in doc files https://review.openstack.org/346065 | 15:05 |
| derekh | bnemec: nothing here looks abnormal to me, the zuul queue has grown to similar lengths most days since moving to rh2 | 15:06 |
| rbrady | shardy: will the jinja file(s) exist in /usr/share/openstack-tripleo-heat-templates immediately after install or added to plan from user later? | 15:06 |
| bnemec | derekh: Yeah, I may have been confused because my ssh connection to the broker hung shortly after I started watching, so I didn't realize more envs had been spun up. | 15:06 |
| derekh | bnemec: I'm not saying its good, just saying its looks like what I'd expect | 15:06 |
| derekh | bnemec: ahh, ok | 15:07 |
| shardy | rbrady: the j2.yaml file will exist when the plan is created, it's going to replace overcloud.yaml in the t-h-t repo | 15:07 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: adjusted serivce names https://review.openstack.org/346879 | 15:07 |
| shardy | rbrady: then there's an additional step, where we run jinja to generate the overcloud.yaml | 15:07 |
| *** bootsha has quit IRC | 15:15 | |
| openstackgerrit | Saravanan KR proposed openstack/os-net-config: Add support for OVS DPDK Bridge and Port https://review.openstack.org/338694 | 15:15 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move AllNodesValidationDeployments into jinja template loop https://review.openstack.org/337587 | 15:19 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Convert overcloud.yaml to support jinja2 templating https://review.openstack.org/315679 | 15:19 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Allow map_replace substitution of network names https://review.openstack.org/345510 | 15:19 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move AllNodesDeployments into jinja template loop https://review.openstack.org/337267 | 15:19 |
| *** tosky has quit IRC | 15:19 | |
| *** tosky has joined #tripleo | 15:20 | |
| EmilienM | slagle, shardy: I'm preparing an email for ML about next deep dive, to gather feedback before doing it. | 15:20 |
| openstackgerrit | Merged openstack/tripleo-specs: Add co-author +2 policy https://review.openstack.org/341202 | 15:20 |
| hewbrocca | EmilienM: oh, I thought jistr already volunteered to do one about pacemaker | 15:23 |
| * hewbrocca snickers | 15:24 | |
| EmilienM | jistr: can you confirm? | 15:24 |
| jistr | :D no | 15:24 |
| jistr | EmilienM: hewbrocca is kidding | 15:24 |
| EmilienM | I can do mine next week otherwise | 15:24 |
| hewbrocca | EmilienM: he missed scrum | 15:24 |
| jistr | EmilienM: go ahead with your e-mail please | 15:24 |
| EmilienM | ahah ok :P | 15:24 |
| EmilienM | so you next week? :) | 15:24 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] add debug to the overcloud deployment https://review.openstack.org/346889 | 15:24 |
| *** pcaruana has quit IRC | 15:26 | |
| *** skramaja is now known as skramaja_afk | 15:26 | |
| mhenkel | @shardy: I changed the service name removing the “-“ so that is passes the test and matches the directory name | 15:27 |
| *** milan has joined #tripleo | 15:31 | |
| *** derekjhyang has quit IRC | 15:32 | |
| *** zoli|mtg is now known as zoli | 15:33 | |
| *** zoli is now known as zoliXXL | 15:33 | |
| *** dtrainor has quit IRC | 15:35 | |
| *** fragatin_ has quit IRC | 15:36 | |
| *** milan has quit IRC | 15:36 | |
| *** fragatina has joined #tripleo | 15:37 | |
| *** david-lyle_ has joined #tripleo | 15:38 | |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci: Enable Ceph in OVB/HA2 https://review.openstack.org/346897 | 15:39 |
| *** dtrainor has joined #tripleo | 15:39 | |
| *** rbrady has quit IRC | 15:41 | |
| openstackgerrit | Miles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package https://review.openstack.org/344214 | 15:41 |
| *** david-lyle_ has quit IRC | 15:43 | |
| shardy | mhenkel: Hi! Ack, yup that should work to get them passing validation | 15:44 |
| shardy | mhenkel: we can chat with dprince though, as we probably want more descriptive (and less potentially colliding) names than e.g "config" | 15:44 |
| shardy | mhenkel: I'd be tempted to just move them all into services/contrail and prefix the filenames with contrail- | 15:45 |
| shardy | we'd have duplication in the path/name but it'd ensure a descriptive service name, and pass validation | 15:45 |
| mhenkel | shardy: ok, that is why I used the contrail- in the first instance. If I’ll move the service name back to contrail-* won’t that break the validation? | 15:47 |
| shardy | mhenkel: yes, the filename and the service_name need to match | 15:47 |
| shardy | but the names also need to be unique, so e.g "database" isn't a good choice | 15:48 |
| mhenkel | shardy: but I can use a “-“ ? | 15:48 |
| shardy | contrail/contrail-database.yaml with a service_name: contrail-database would fix that I think | 15:48 |
| mhenkel | shardy: ok, will do that | 15:48 |
| shardy | mhenkel: thanks! | 15:48 |
| *** dtrainor has quit IRC | 15:49 | |
| *** skramaja_afk is now known as skramaja | 15:49 | |
| *** rbrady has joined #tripleo | 15:50 | |
| beagles | shardy, dprince: if you get a sec https://review.openstack.org/#/c/332147/ ... just want to know if that approach fits the bill | 15:51 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Trove Integration https://review.openstack.org/233240 | 15:51 |
| *** dtrainor has joined #tripleo | 15:51 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: DO NOT MERGE testing https://review.openstack.org/#/c/188137/ https://review.openstack.org/342961 | 15:52 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: rebased service naming as suggested by shardy https://review.openstack.org/346906 | 15:52 |
| shardy | beagles: lgtm, unless dprince has any objections I'm happy to land it | 15:56 |
| shardy | beagles: there is one unused parameter we might want to clean up in a follow-up patch | 15:56 |
| *** florianf has quit IRC | 15:57 | |
| beagles | shardy: ack. I'm keen to do an "audit" of sorts and do a general cleanup, but that's likely an "O" thing | 15:57 |
| shardy | mhenkel: Hey, I think you want to squash those three patches together? | 15:58 |
| shardy | e.g git rebase -i origin, then either squash or fixup the last two patches into the original one? | 15:58 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Convert Swift ringbuilder to composable services format https://review.openstack.org/338551 | 15:58 |
| *** fragatina has quit IRC | 15:58 | |
| *** penick has joined #tripleo | 15:59 | |
| shardy | beagles: tools/yaml-validate.py actually warns about such things, so I'm liable to do a cleanup towards the end of newton anyway | 15:59 |
| beagles | shardy: ah, right | 15:59 |
| *** penick_ has joined #tripleo | 16:00 | |
| *** fragatina has joined #tripleo | 16:00 | |
| *** milan has joined #tripleo | 16:02 | |
| *** penick has quit IRC | 16:03 | |
| *** penick_ is now known as penick | 16:03 | |
| *** Goneri has quit IRC | 16:04 | |
| *** devvesa has quit IRC | 16:09 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Migrate Puppet Hieradata to composable services https://review.openstack.org/341616 | 16:12 |
| *** oshvartz has quit IRC | 16:13 | |
| *** dsneddon_ has quit IRC | 16:14 | |
| *** dsneddon has quit IRC | 16:15 | |
| EmilienM | shardy: https://review.openstack.org/#/c/346124/ see comment - I'll let you approve also | 16:15 |
| *** dsneddon has joined #tripleo | 16:15 | |
| shardy | EmilienM: thanks, approved | 16:16 |
| EmilienM | thx | 16:17 |
| EmilienM | shardy: I'm working on next steps ;) | 16:17 |
| shardy | EmilienM: sounds good, I guess using the profiles for the undercloud isn't super high priority, but it would be nice | 16:17 |
| rbrady | shardy: re jinja templates, should the overcloud.yaml file be generated and added back to the swift container at plan creation, or do you prefer the generation to occur just before we call out to process_multiple_files_and_environments? | 16:18 |
| shardy | rbrady: I was thinking it'd be loaded back into swift as part of the plan | 16:18 |
| rbrady | ack, thanks | 16:18 |
| *** Goneri has joined #tripleo | 16:18 | |
| shardy | the operators have the opportunity to review it and potentially re-run the generation step if it's not what they expected | 16:18 |
| shardy | I imagine the default workflow will just generate it, store it to swift then call heat tho | 16:19 |
| EmilienM | shardy: right, I'm not working on it as high priority rather low now | 16:20 |
| *** tesseract- has quit IRC | 16:21 | |
| pabelanger | derekh: what is the status of tripleo-test-cloud-rh1? Is that down for a while yet? | 16:23 |
| pabelanger | derekh: The reason I ask, we're likely going to remove it from nodepool to cleaning up some warnings. | 16:23 |
| derekh | pabelanger: still working on it, I'd hope to have it running by the end of the week | 16:24 |
| pabelanger | derekh: Ah, okay. I'll hold off then | 16:24 |
| derekh | pabelanger: btw just saw you mail about the mirror server this afternoon, will reply later this evening | 16:24 |
| pabelanger | derekh: ack | 16:24 |
| derekh | pabelanger: sorry was on PTO yesterday | 16:24 |
| derekh | pabelanger: or friday I mean | 16:25 |
| pabelanger | derekh: np | 16:25 |
| shardy | Hrm, I just updated, and undercloud install seems broken: | 16:25 |
| *** hewbrocca is now known as hewbrocca-afk | 16:25 | |
| shardy | http://paste.openstack.org/show/541624/ | 16:25 |
| shardy | the tripleo::firewall profile expects a step, but isn't using the facter one? | 16:25 |
| pabelanger | derekh: actually, once we get tripleo-test-cloud-rh1 back online, we should talk about credentials in keystone. EG: creating a separate openstacknodepool user outside of the user you us to create local instances. The reason for this, so we remove warnings in nodepool about unknown instances running | 16:27 |
| *** davidlenwell has quit IRC | 16:28 | |
| derekh | pabelanger: ok makes sense | 16:28 |
| shardy | gah, it's probably because we still don't install puppet modules from delorean-current | 16:29 |
| shardy | yup | 16:29 |
| *** david-lyle_ has joined #tripleo | 16:35 | |
| *** numans has quit IRC | 16:36 | |
| *** trown is now known as trown|lunch | 16:36 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Implement tripleo::network::os_net_config class https://review.openstack.org/346118 | 16:37 |
| *** lucasagomes is now known as lucas-afk | 16:39 | |
| *** davidlenwell has joined #tripleo | 16:40 | |
| *** ChanServ sets mode: +v davidlenwell | 16:40 | |
| *** jubapa has joined #tripleo | 16:40 | |
| *** derekh has quit IRC | 16:43 | |
| openstackgerrit | Martin André proposed openstack/tripleo-heat-templates: Add steps to containerized compute deployment https://review.openstack.org/346927 | 16:44 |
| *** jubapa has quit IRC | 16:45 | |
| *** zoliXXL is now known as zoli|gone | 16:46 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail config puppet module for tripleO https://review.openstack.org/346938 | 16:46 |
| *** zoli|gone is now known as zoli_gone-proxy | 16:47 | |
| *** jpich has quit IRC | 16:49 | |
| *** fzdarsky is now known as fzdarsky|afk | 16:51 | |
| openstackgerrit | Merged openstack/instack-undercloud: Deploy undercloud in 1 step instead of 2 https://review.openstack.org/346124 | 16:54 |
| egafford | bnemec: Anyone in particular to ping to WF+1 https://review.openstack.org/#/c/346213/ (to allow a clean downstream cherry-pick?) | 16:55 |
| trozet | bnemec: can you remove the -1 on https://review.openstack.org/#/c/344969/ ? Do I need to list a 'backport' in the commit msg? | 16:55 |
| *** mbound has quit IRC | 16:55 | |
| bnemec | egafford: It passed CI, so I approved it. | 16:56 |
| egafford | bnemec: Excellent; thanks | 16:56 |
| bnemec | trozet: See my last comment. I think steps 2 and 3 still need to be done before that merges. | 16:57 |
| *** liverpooler has quit IRC | 16:57 | |
| *** stendulker has joined #tripleo | 16:57 | |
| bnemec | I'd prefer a mention of the backport in the commit message too. | 16:57 |
| bnemec | Even though it isn't a straight cherry-pick we should keep it in sync with what we did on master. | 16:57 |
| trozet | bnemec: what is the proper syntax for that, just 'backport: mitaka' ? I see that in a few other commits | 16:57 |
| trozet | bnemec: ah sorry I didn't see your reply on that patch | 16:58 |
| bnemec | trozet: I would just add a line like "This is a backport of Ieb793b9e9fd2dfc98584691412f9991aa99e0b47 in tripleo-heat-templates" | 16:58 |
| trozet | bnemec: ok got it will fix those things | 16:59 |
| trozet | thanks | 16:59 |
| bnemec | Oh, I guess they're both in tripleo-heat-templates. | 16:59 |
| bnemec | Maybe mention that it can't be a cherry-pick because of composable work. | 16:59 |
| bnemec | As long as there's a way to track the commit chain I'm happy though. | 16:59 |
| trozet | bnemec: ok got it | 17:00 |
| *** lucas-afk is now known as lucasagomes | 17:00 | |
| *** david-lyle_ has quit IRC | 17:01 | |
| openstackgerrit | Merged openstack/tripleo-ui: Remove app.conf to use a JS-based config file https://review.openstack.org/343658 | 17:02 |
| openstackgerrit | Merged openstack/tripleo-ui: Use Webpack for application processing https://review.openstack.org/343592 | 17:02 |
| *** pkovar has quit IRC | 17:03 | |
| *** mgould is now known as mgould|afk | 17:06 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic nonha test https://review.openstack.org/346948 | 17:07 |
| *** david-lyle_ has joined #tripleo | 17:07 | |
| *** rajinir has joined #tripleo | 17:07 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic ha test https://review.openstack.org/346949 | 17:07 |
| *** ifarkas is now known as ifarkas_afk | 17:07 | |
| openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates: Fixes nova host configuration on control nodes https://review.openstack.org/344969 | 17:08 |
| trozet | bnemec: do we need to tag the bug with 'mitaka-backport-potential'? If so, how do you tag it, just put it in a comment or? | 17:09 |
| *** dmsimard is now known as dmsimard|afk | 17:09 | |
| bnemec | trozet: I targeted it for Mitaka and Liberty (since I'm pretty sure we have the same problem there). | 17:09 |
| trozet | bnemec: ok | 17:09 |
| bnemec | trozet: I don't know if everyone has the ability to target, so if not then tagging would be your best bet. | 17:10 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Installing openstack-sahara-ui https://review.openstack.org/346213 | 17:11 |
| *** yamahata has joined #tripleo | 17:18 | |
| pabelanger | EmilienM: How do you manage external puppet modules (eg: puppetlabs-firewall) in tripleo? I think you cache them to the undercloud right? | 17:21 |
| EmilienM | pabelanger: tripleo-puppet-elements | 17:21 |
| pabelanger | EmilienM: okay, so next question. Why are you using git repos and not tarballs? | 17:22 |
| EmilienM | pabelanger: so we can use depends-on? | 17:22 |
| *** pcaruana has joined #tripleo | 17:23 | |
| pabelanger | EmilienM: well, puppetlabs-firewell wouldn't support that | 17:23 |
| EmilienM | right | 17:23 |
| EmilienM | but there is no tarball in t.o.o for this module | 17:23 |
| EmilienM | we could use tarball from github, is it what you mean? | 17:24 |
| pabelanger | EmilienM: you could fetch it from forge.puppetlabs.com | 17:24 |
| pabelanger | EmilienM: I'm trying to figure out why you are depending on github.com for production installs over using official tarballs of the projects | 17:24 |
| EmilienM | I think we could change it | 17:25 |
| pabelanger | okay, so that is good news :) | 17:25 |
| pabelanger | trying to figure out your mirror requirements, and from opentack-infra it is easier to mirror a tarball into the infra then a git repo | 17:26 |
| EmilienM | ok | 17:27 |
| shardy | EmilienM, pabelanger: since packaging changes were made to build individual puppet module packages (vs the old openstack-puppet-modules monolithic package), I'm not sure we need to build all the modules from source anymore | 17:27 |
| shardy | only those which are overlaid due to being in ZUUL_CHANGES, e.g a Depends-On | 17:28 |
| shardy | which may reduce the requirement for mirroring | 17:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Unit tests for HAproxy dual stack https://review.openstack.org/346958 | 17:28 |
| EmilienM | dsneddon: look ^ | 17:28 |
| shardy | https://review.openstack.org/#/c/341429/ aims to do that, but needs a rebase and reviews | 17:28 |
| EmilienM | dsneddon: I could not reproduce your data error | 17:28 |
| shardy | rather, it aims to stop using puppet source for everything, instead relying on the deloran trunk repo | 17:29 |
| EmilienM | dsneddon: look L47 https://review.openstack.org/#/c/346958/1/spec/defines/tripleo_haproxy_endpoint_spec.rb | 17:29 |
| EmilienM | the expected result is an array of hashes | 17:29 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Stop rh2 from preferring slower nodes https://review.openstack.org/346960 | 17:29 |
| shardy | it also seems we'll have to special-case puppet-ceph until we land ceph jewel support in tripleo | 17:29 |
| pabelanger | shardy: Right, openstack puppet modules don't have to change to date. We have the infra in place to ensure reliable clones. However, external git repos are hard to deal with, since we don't have the mirror infra for that today. If they were tarballs, that would infact be easier to work with | 17:29 |
| pabelanger | shardy: okay, cool. with 341429 that reduces things | 17:30 |
| shardy | pabelanger: we'll require git repos for everything in ZUUL_CHANGES, because we build packages for those | 17:30 |
| shardy | but all the other repo dependencies can probably be broken pretty soon | 17:30 |
| pabelanger | shardy: right, but ZUUL_CHANGES wouldn't work for external hosted git repos | 17:31 |
| pabelanger | shardy: which could move to tarballs | 17:31 |
| shardy | pabelanger: yup, for all external repos I think we should just use delorean-current | 17:31 |
| shardy | I'm pretty sure Depends-On doesn't work in that case anyway, since they're external to OpenStack | 17:32 |
| pabelanger | shardy: okay good. So there is some plan in motion to address this. Which reduces the amount of items in the mirrorlist | 17:32 |
| pabelanger | shardy: right | 17:32 |
| shardy | pabelanger: well the plan is that patch, so we'll need to push on getting that moving again | 17:32 |
| shardy | and figure out a solution to the ceph blocker mentioned | 17:33 |
| slagle | thrash: any chance you could take a look at https://bugs.launchpad.net/tripleo/+bug/1605876 ? | 17:33 |
| openstack | Launchpad bug 1605876 in tripleo "openstack baremetal import --json instackenv.json fails with "PluginAttributeError: _auth_params"" [Undecided,New] | 17:33 |
| *** mbound has joined #tripleo | 17:33 | |
| pabelanger | shardy: okay, I'll follow along and see how to help\ | 17:33 |
| shardy | pabelanger: sounds good, I'll try to catch up with derekh tomorrow and see how we can progress things | 17:33 |
| shardy | pabelanger: thanks! | 17:33 |
| gfidente | shardy I was thinking to add a tempurl plugin resource in heat to progress on the swift tempurl story? | 17:34 |
| *** fragatina has quit IRC | 17:35 | |
| shardy | gfidente: there is http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Swift::Container | 17:35 |
| shardy | but I'm not sure if it supports tempurl yet | 17:35 |
| gfidente | yeah it doesn't | 17:36 |
| gfidente | though there is tempurl in the swift client already | 17:36 |
| shardy | Yeah, would probably be fairly easy to add it as a new attribute I guess | 17:36 |
| gfidente | right so we get tempurl at the container level | 17:37 |
| gfidente | I did some coding already, let me play with it want to see if I can push something later or tomorrow | 17:38 |
| shardy | ++ sounds good | 17:38 |
| *** stendulker_ has joined #tripleo | 17:39 | |
| *** stendulker_ has quit IRC | 17:39 | |
| *** stendulker_ has joined #tripleo | 17:39 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 17:39 |
| *** trown|lunch is now known as trown | 17:40 | |
| gfidente | ak, ttyt | 17:40 |
| *** gfidente has quit IRC | 17:40 | |
| *** electrofelix has quit IRC | 17:42 | |
| *** stendulker has quit IRC | 17:42 | |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Add Contrail composable services https://review.openstack.org/346967 | 17:53 |
| mhenkel | shardy: getting there… ;_ | 17:53 |
| *** mcornea has quit IRC | 17:54 | |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Add Contrail composable services https://review.openstack.org/346967 | 17:56 |
| *** david-lyle_ is now known as david-lyle | 17:57 | |
| *** dtrainor has quit IRC | 17:58 | |
| *** stendulker_ has quit IRC | 17:59 | |
| *** fragatina has joined #tripleo | 17:59 | |
| *** stendulker has joined #tripleo | 18:00 | |
| thrash | slagle: sure. | 18:01 |
| *** lucasagomes is now known as lucas-dinner | 18:01 | |
| *** skramaja is now known as skramaja_afk | 18:02 | |
| *** stendulker_ has joined #tripleo | 18:02 | |
| *** fragatina has quit IRC | 18:04 | |
| *** tosky has quit IRC | 18:04 | |
| *** stendulker has quit IRC | 18:05 | |
| openstackgerrit | Sarath Kumar Sankaran Kutty proposed openstack/os-net-config: Enable os-net-config to support and configure NFVSwitch https://review.openstack.org/345599 | 18:09 |
| *** dtrainor has joined #tripleo | 18:10 | |
| *** dtrainor has quit IRC | 18:11 | |
| *** dtrainor has joined #tripleo | 18:11 | |
| *** dtrainor has quit IRC | 18:13 | |
| *** dtrainor has joined #tripleo | 18:13 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail config puppet module for tripleO https://review.openstack.org/346938 | 18:14 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo https://review.openstack.org/346975 | 18:14 |
| *** yamahata has quit IRC | 18:19 | |
| *** jubapa has joined #tripleo | 18:21 | |
| *** mbound has quit IRC | 18:23 | |
| *** mbound has joined #tripleo | 18:24 | |
| *** mbound has quit IRC | 18:25 | |
| *** jubapa has quit IRC | 18:25 | |
| *** ayoung has quit IRC | 18:30 | |
| EmilienM | gate-tripleo-ci-centos-7-undercloud-nv SUCCESS | 18:31 |
| chem | EmilienM: catching up, what is this testing ? | 18:32 |
| EmilienM | chem: to deploy an undercloud | 18:33 |
| EmilienM | chem: https://review.openstack.org/#/c/346147/ | 18:33 |
| chem | EmilienM: shorter feedback loop, nice | 18:33 |
| chem | EmilienM: thanks | 18:33 |
| EmilienM | it takes 30 min | 18:34 |
| EmilienM | I guess I can start the undercloud upgrade job | 18:34 |
| *** dtrainor has quit IRC | 18:36 | |
| *** oshvartz has joined #tripleo | 18:37 | |
| pabelanger | EmilienM: it would likely be shorter too if we switched away from devstack and used zuul-cloner directly | 18:40 |
| EmilienM | pabelanger: maybe, but I don't have all knowledge yet about that thing. Maybe you can kickoff an example and I could continue later | 18:41 |
| *** dtrainor has joined #tripleo | 18:43 | |
| *** julim has quit IRC | 18:43 | |
| *** julim has joined #tripleo | 18:45 | |
| *** fzdarsky|afk has quit IRC | 18:45 | |
| *** alhernan|away has joined #tripleo | 18:45 | |
| pabelanger | EmilienM: ya, I was going to add an experimental job to work on it | 18:46 |
| EmilienM | slagle, shardy, bnemec: https://review.openstack.org/346982 | 18:48 |
| bnemec | EmilienM: Are we actually planning to have both undercloud and undercloud-upgrade jobs, or is that just for testing? | 18:52 |
| EmilienM | bnemec: for now, just for testing | 18:55 |
| EmilienM | bnemec: if you feel like we don't need both, say it. For now I'm doing both | 18:55 |
| bnemec | EmilienM: I'm fine with that, just wondering if you had a specific case in mind that required both. | 18:58 |
| EmilienM | bnemec: until now, we have no upstream job that tested undercloud upgrade | 18:59 |
| EmilienM | that was my goal | 18:59 |
| EmilienM | so I iterated to have first an undercloud-only job | 18:59 |
| EmilienM | and now it's working, I'm working on the upgrade | 18:59 |
| EmilienM | at the end, we can keep both or only upgrade job | 18:59 |
| *** yamahata has joined #tripleo | 18:59 | |
| EmilienM | I don't mind. | 18:59 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: Stop Validation https://review.openstack.org/336561 | 19:00 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: Triggering validations and messaging https://review.openstack.org/336560 | 19:00 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: List Validations and Validation results https://review.openstack.org/334770 | 19:00 |
| slagle | EmilienM: have a look at we update openstack-tripleo-heat-templates in the overcloud upgrades job on how we can do this with all packages | 19:00 |
| bnemec | EmilienM: Okay, I guess we can discuss it when we're ready to start moving jobs out of experimental. | 19:00 |
| EmilienM | slagle: yeah I'll follow it | 19:01 |
| *** mgarciam has quit IRC | 19:01 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/tripleo-ci: Refactor mirror logic https://review.openstack.org/326143 | 19:01 |
| openstackgerrit | Paul Belanger proposed openstack-infra/tripleo-ci: Add conditional check for tripleo-test-cloud-rh2 https://review.openstack.org/346985 | 19:01 |
| pabelanger | EmilienM: ^ Some cleanup now that jobs are running on other clouds | 19:02 |
| *** alhernan|away has quit IRC | 19:04 | |
| *** dtrainor has quit IRC | 19:10 | |
| *** stendulker_ has quit IRC | 19:15 | |
| *** jcoufal_ has quit IRC | 19:15 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement basic undercloud sanity checks https://review.openstack.org/346551 | 19:16 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 19:16 |
| *** mhenkel has quit IRC | 19:19 | |
| EmilienM | mhh, I see this new error: | 19:19 |
| EmilienM | http://logs.openstack.org/47/346147/8/check/gate-tripleo-ci-centos-7-nonha-multinode-nv/a568147/logs/postci.txt.gz#_2016-07-25_18_36_44_000 | 19:19 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Don't spam postci with deployment time loop https://review.openstack.org/346990 | 19:19 |
| EmilienM | I think this is https://github.com/openstack/puppet-tripleo/commit/e1d2245f7c7a14d22bb022511fab836e224d61b1 | 19:19 |
| EmilienM | shardy: ^ | 19:20 |
| EmilienM | nevermind, doing recheck | 19:21 |
| slagle | EmilienM: i think we just need to recheck and then land https://review.openstack.org/#/c/345661/ | 19:22 |
| *** dtrainor has joined #tripleo | 19:23 | |
| EmilienM | ah that's why | 19:25 |
| *** mbound has joined #tripleo | 19:25 | |
| *** dmsimard|afk is now known as dmsimard | 19:26 | |
| slagle | EmilienM: there is already a standing bj meeting for the deep dive | 19:27 |
| slagle | EmilienM: it's in the etherpad | 19:27 |
| slagle | oh i see, you updated it there | 19:27 |
| slagle | ok, nm, this is fine | 19:28 |
| slagle | people should look at the etherpad anyway | 19:28 |
| EmilienM | ok | 19:29 |
| EmilienM | I'll send a reminder again over the ML | 19:29 |
| *** chlong_POffice has quit IRC | 19:30 | |
| *** chlong_POffice has joined #tripleo | 19:31 | |
| *** mbound has quit IRC | 19:31 | |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo: Adds OpenDaylight https://review.openstack.org/335173 | 19:33 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement undercloud upgrade job https://review.openstack.org/346995 | 19:38 |
| EmilienM | slagle, bnemec: can I have early feedback on how to upgrade undercloud? ^ | 19:38 |
| EmilienM | let me know if see another approach | 19:38 |
| trozet | beagles: can you take a look at https://review.openstack.org/#/c/335173/6 and let me know if that is more what you are asking for? | 19:39 |
| bnemec | EmilienM: Ideally I would like to see the undercloud upgrade be from Mitaka->master. | 19:42 |
| EmilienM | bnemec: I did the same way as tht now | 19:42 |
| *** jprovazn has quit IRC | 19:44 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use scp to copy deploy.env to subnodes https://review.openstack.org/346998 | 19:45 |
| EmilienM | slagle: I haven't looked into it yet but it would be great to have primary node logs already uncompressed in log server, at least /var/log/ like ovb jobs | 19:45 |
| EmilienM | for subnode2, all is fine iirc | 19:46 |
| EmilienM | bnemec: can you review https://review.openstack.org/#/c/346230/ please? | 19:46 |
| *** dtrainor has quit IRC | 19:51 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Allow toci_instack_multinode.sh to run single node https://review.openstack.org/346230 | 19:51 |
| EmilienM | slagle: I'm monitoring https://review.openstack.org/#/c/345661/ in zuul, if multinode pass, I'll approve the patch right away and not wait for ovb jobs | 19:55 |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo: Adds OpenDaylight https://review.openstack.org/335173 | 19:56 |
| slagle | ah....rebase hell :) | 19:57 |
| slagle | your patch must have merged | 19:57 |
| *** paramite|afk is now known as paramite | 19:58 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use scp to copy deploy.env to subnodes https://review.openstack.org/346998 | 20:03 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Break out bootstrapping of subnodes setup https://review.openstack.org/347008 | 20:03 |
| *** shardy has quit IRC | 20:04 | |
| EmilienM | slagle: did you already start investigating at how to run your multinode job outside tripleo, (ie: heat, nova)? | 20:04 |
| EmilienM | I mean, in heat's gate, etc | 20:04 |
| slagle | EmilienM: i don't think anything different will be needed. what did you have in mind? | 20:05 |
| EmilienM | slagle: I would like to propose your job into heat's gate as experimental | 20:05 |
| slagle | it's just project-config changes to do that i believe | 20:06 |
| EmilienM | slagle: but how heat would be deployed? | 20:06 |
| EmilienM | my question might be dumb but | 20:06 |
| EmilienM | how delorean-ci repo is created? | 20:06 |
| slagle | EmilienM: delorean should build an rpm on the fly baed on ZUUL_CHANGES | 20:06 |
| EmilienM | ahhhh | 20:06 |
| slagle | EmilienM: delorean_build_and_serve function should do it | 20:06 |
| EmilienM | wow awesome | 20:06 |
| EmilienM | for any project? | 20:07 |
| slagle | should work, yea. for any project delorean knows how to build | 20:07 |
| slagle | which is darn near everything | 20:07 |
| EmilienM | nice | 20:07 |
| *** dtrainor has joined #tripleo | 20:07 | |
| *** dtrainor has quit IRC | 20:08 | |
| *** dtrainor has joined #tripleo | 20:08 | |
| slagle | i might suggest waiting a couple more days to get any bugs worked out | 20:08 |
| slagle | https://bugs.launchpad.net/tripleo/+bug/1605596 for instance | 20:08 |
| openstack | Launchpad bug 1605596 in tripleo "Depends-On doesn't work for multinode job" [High,In progress] - Assigned to James Slagle (james-slagle) | 20:08 |
| dsneddon | EmilienM, I have an environment up which reproduces https://bugs.launchpad.net/puppet-tripleo/+bug/1606316 | 20:09 |
| openstack | Launchpad bug 1606316 in puppet-tripleo " loadbalancer.pp Not Properly Accepting Multiple Addresses" [Undecided,New] | 20:09 |
| dsneddon | EmilienM, We can discuss it here... | 20:09 |
| EmilienM | dsneddon: looking | 20:10 |
| *** shardy has joined #tripleo | 20:10 | |
| EmilienM | dsneddon: can you show me haproxy.cfg? | 20:10 |
| dsneddon | EmilienM, It appears that it this is all that is in haproxy.cfg: http://paste.openstack.org/show/541646/ | 20:12 |
| EmilienM | ok so puppet couldn't apply | 20:12 |
| dsneddon | EmilienM, The error appears to trying to run a config file with "/etc/haproxy/haproxy.cfg20160630-45596-yk8oyv" that failed. | 20:12 |
| EmilienM | shardy, slagle: https://review.openstack.org/347012 | 20:13 |
| EmilienM | dsneddon: I'm looking at your THT work now. | 20:13 |
| slagle | EmilienM: i would not put this in their check queue right now | 20:14 |
| slagle | experimental perhaps | 20:14 |
| shardy | Yeah I agree, lets start with experimental - folks are used to running check experimental for tripleo feedback I think | 20:15 |
| EmilienM | slagle: ok done. | 20:15 |
| shardy | slagle: what sort of quota is there for the multinode job? | 20:15 |
| slagle | dunno | 20:16 |
| slagle | i guess if we burn up too many cloud instances, infra will holler :) | 20:16 |
| shardy | I'm assuming if we did put it in the check queue for a fairly high traffic repo like heat we'd end up with jobs queuing | 20:16 |
| shardy | hehe | 20:16 |
| shardy | experimental seems saftest for now regardless I think | 20:16 |
| shardy | good to see that it's working pretty well tho | 20:17 |
| shardy | other than the depends-on thing, it seems to work great | 20:17 |
| EmilienM | I thought Red Hat would give some resources with RH cloud? | 20:17 |
| EmilienM | slagle: I also thought to switch puppet jobs from ovb to multinode, to release resources in RH cloud. wdyt? | 20:18 |
| slagle | EmilienM: you'll lose coverage of nova doing the ironic deployment | 20:19 |
| EmilienM | dsneddon: I found out | 20:19 |
| EmilienM | slagle: except for puppet-nova and puppet-ironic | 20:19 |
| slagle | EmilienM: and arent you running ha on pupet jobs? | 20:19 |
| EmilienM | no we aren't | 20:19 |
| slagle | ok | 20:19 |
| EmilienM | we run ovb non ha | 20:19 |
| EmilienM | well, I can keep the idea for later :) | 20:20 |
| dsneddon | EmilienM, Awesome, what did you find? | 20:20 |
| EmilienM | slagle: need your thoughts on undercloud-upgrade thing | 20:20 |
| slagle | EmilienM: yea i think i'd like to see it voting in tripleo before you did that | 20:20 |
| EmilienM | slagle: kk | 20:20 |
| EmilienM | dsneddon: a sec | 20:20 |
| EmilienM | dsneddon: commented on https://review.openstack.org/#/c/289279/15/overcloud.yaml | 20:22 |
| EmilienM | dsneddon: a few months ago, I told you to give an array of VIPs iirc | 20:23 |
| EmilienM | and this is not an array in THT ^ | 20:23 |
| EmilienM | or is it? | 20:23 |
| EmilienM | I think this is an hash | 20:23 |
| shardy | EmilienM: json can be a list or a hash | 20:23 |
| EmilienM | anyway, the datatype is wrong, I need an array to make it working, https://review.openstack.org/#/c/346958/ proves it works | 20:24 |
| EmilienM | shardy: puppet needs to see it like an array | 20:24 |
| shardy | the advantage of having json parameter type over comma_delimited_list is it can accept yaml arrays and json serialized lists | 20:24 |
| dsneddon | EmilienM, Support Heat types are "ring, number, comma_delimited_list, json and boolean" | 20:24 |
| shardy | EmilienM: if you pass a list to the parameter, it will be | 20:25 |
| dsneddon | EmilienM, String, not ring. | 20:25 |
| EmilienM | I don't know about Heat here, but I think Puppet doesn't see it as an array | 20:25 |
| shardy | you could use comma_delimited_list to enforce a list, but it won't work with json strings | 20:25 |
| EmilienM | but as a string with comma_delimited parameters | 20:25 |
| EmilienM | so I can try to fix it in Puppet, let me a second | 20:25 |
| EmilienM | gimme a sec, /me tries to fix it in puppet-tripleo | 20:26 |
| shardy | EmilienM: does puppet actually need it in json format? | 20:26 |
| dsneddon | EmilienM, Note that a comma_delimeted_list would output ["one", "two"] to hieradata, the only difference there being double instead of single quotes. | 20:27 |
| EmilienM | where do you see this error? 'bind' : invalid address: '2001:db8:fd00:1000::1010.34.107.18' in '2001:db8:fd00:1000::1010.34.107.18:80' | 20:28 |
| EmilienM | in puppet catalog? | 20:28 |
| EmilienM | dsneddon: ^ | 20:28 |
| dsneddon | EmilienM, No, it's on the Undercloud in "heat deployment-show 7269cb9e-2746-4b2d-8ffe-b573d9b06730" | 20:29 |
| EmilienM | dsneddon: can you try something? I have a patch coming | 20:30 |
| dsneddon | EmilienM, Sure, I can test something out. | 20:30 |
| EmilienM | a sec | 20:31 |
| *** dtrainor has quit IRC | 20:31 | |
| *** shardy has quit IRC | 20:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: haproxy/endpoints: ensure vips are arrays https://review.openstack.org/347018 | 20:35 |
| EmilienM | dsneddon: ^ please try that | 20:35 |
| dsneddon | EmilienM, Cool, will do. I should have the results by tomorrow, because I have to recreate the undercloud with puppet-tripleo being pulled from Git. Thanks for your help. | 20:36 |
| EmilienM | dsneddon: can you try to use CI to test it? | 20:37 |
| dsneddon | EmilienM, You mean by using depends-on? | 20:37 |
| EmilienM | yes | 20:38 |
| dsneddon | EmilienM, Yeah, I don't see why not. Thanks. | 20:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: haproxy/endpoints: ensure IPs are arrays https://review.openstack.org/347018 | 20:38 |
| EmilienM | dsneddon: wait | 20:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: haproxy/endpoints: ensure IPs are arrays https://review.openstack.org/347018 | 20:39 |
| EmilienM | dsneddon: adding unit tests | 20:39 |
| *** akrivoka has quit IRC | 20:39 | |
| *** akshai has quit IRC | 20:40 | |
| *** milan has quit IRC | 20:43 | |
| EmilienM | dsneddon: ok wait I found something weird in my code | 20:47 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move AllNodesValidationDeployments into jinja template loop https://review.openstack.org/337587 | 20:52 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Convert overcloud.yaml to support jinja2 templating https://review.openstack.org/315679 | 20:52 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Allow map_replace substitution of network names https://review.openstack.org/345510 | 20:52 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move AllNodesDeployments into jinja template loop https://review.openstack.org/337267 | 20:52 |
| EmilienM | dsneddon: do you have puppet catalog output? | 20:53 |
| dsneddon | EmilienM, Where is that stored? | 20:53 |
| EmilienM | you can see it in /var/log/message or in journalctl | 20:53 |
| *** fultonj_ has quit IRC | 20:55 | |
| dsneddon | EmilienM, On the controller? | 20:55 |
| dsneddon | EmilienM, I do, here is the output: http://paste.openstack.org/show/541648/ | 20:56 |
| EmilienM | yes.. | 20:56 |
| EmilienM | where haproxy runs | 20:56 |
| *** chlong_POffice has quit IRC | 20:56 | |
| dsneddon | EmilienM, It's there, it just didn't wrap the long line. | 20:56 |
| *** tbonds has quit IRC | 20:57 | |
| EmilienM | dsneddon: can I see /etc/haproxy/haproxy.cfg ? | 20:59 |
| EmilienM | dsneddon: is it before my patch or after? | 20:59 |
| dsneddon | EmilienM, No, this is before your patch. The old run. | 20:59 |
| dsneddon | EmilienM, Wait, not sure I understand the question | 21:00 |
| EmilienM | no its ok | 21:00 |
| EmilienM | can I just see /etc/haproxy/haproxy.cfg ? | 21:00 |
| EmilienM | dsneddon: good, i need to see old run | 21:00 |
| dsneddon | EmilienM, haproxy.cfg: http://paste.openstack.org/show/541658/ | 21:01 |
| EmilienM | mwhahaha: oh thanks for your review | 21:02 |
| mwhahaha | sure | 21:02 |
| *** morazi has quit IRC | 21:02 | |
| EmilienM | dsneddon: ok, a sec, I made a mistake in my patch | 21:02 |
| *** jeckersb is now known as jeckersb_gone | 21:03 | |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Add adapter teaming support using teamd for ifcfg-systems https://review.openstack.org/339854 | 21:04 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Composable firewall rules https://review.openstack.org/344985 | 21:08 |
| *** chlong_POffice has joined #tripleo | 21:09 | |
| *** trown is now known as trown|outtypewww | 21:09 | |
| *** fragatina has joined #tripleo | 21:15 | |
| *** rlandy has quit IRC | 21:16 | |
| *** Goneri has quit IRC | 21:16 | |
| *** egafford has quit IRC | 21:19 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Add SwiftRingBuilder to multinode job https://review.openstack.org/345661 | 21:20 |
| EmilienM | dsneddon: ok I found the bug maybe | 21:20 |
| EmilienM | dsneddon: I'll hack something on puppet-tripleo tonight or tomorrow | 21:20 |
| *** julim has quit IRC | 21:20 | |
| dsneddon | EmilienM, OK, thanks, let me know and I'll be ready to do some testing. | 21:20 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Enable IPv4/IPv6 dual-stack Public API endpoints https://review.openstack.org/289279 | 21:25 |
| EmilienM | dsneddon: rebasing ^ | 21:25 |
| EmilienM | I want to see CI logs | 21:25 |
| EmilienM | I'm still unsure why unit tests work find | 21:25 |
| EmilienM | fine* | 21:25 |
| EmilienM | see https://review.openstack.org/#/c/346958/1/spec/defines/tripleo_haproxy_endpoint_spec.rb | 21:25 |
| EmilienM | I give an array of VIPs, and it configures HAproxy like we expect | 21:25 |
| EmilienM | with an array of bindings | 21:25 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement basic undercloud sanity checks https://review.openstack.org/346551 | 21:27 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 21:27 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-specs: Spec: Add Adapter Teaming Support to os-net-config https://review.openstack.org/337886 | 21:27 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement undercloud upgrade job https://review.openstack.org/346995 | 21:27 |
| *** rhallisey has quit IRC | 21:30 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-specs: Spec: Add Adapter Teaming Support to os-net-config https://review.openstack.org/337886 | 21:30 |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Add adapter teaming support using teamd for ifcfg-systems https://review.openstack.org/339854 | 21:34 |
| *** egafford has joined #tripleo | 21:38 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Record testenv wait times in ovb jobs https://review.openstack.org/347037 | 21:38 |
| *** snecklifter_ has quit IRC | 21:40 | |
| *** bnemec has quit IRC | 21:40 | |
| *** jeckersb_gone is now known as jeckersb | 21:47 | |
| *** fragatin_ has joined #tripleo | 21:49 | |
| *** bnemec has joined #tripleo | 21:50 | |
| *** fragatin_ has quit IRC | 21:51 | |
| *** fragatin_ has joined #tripleo | 21:52 | |
| *** fragatina has quit IRC | 21:53 | |
| *** coolsvap has quit IRC | 22:01 | |
| *** rwsu has quit IRC | 22:04 | |
| *** bfournie has quit IRC | 22:05 | |
| *** paramite has quit IRC | 22:08 | |
| *** rlandy has joined #tripleo | 22:11 | |
| *** thrash is now known as thrash|g0ne | 22:17 | |
| *** jubapa has joined #tripleo | 22:17 | |
| *** akshai has joined #tripleo | 22:20 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move AllNodesValidationDeployments into jinja template loop https://review.openstack.org/337587 | 22:21 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Convert overcloud.yaml to support jinja2 templating https://review.openstack.org/315679 | 22:21 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Move AllNodesDeployments into jinja template loop https://review.openstack.org/337267 | 22:21 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Remove some properties from overcloud.yaml Controller group https://review.openstack.org/347050 | 22:21 |
| *** fragatina has joined #tripleo | 22:23 | |
| *** fragatin_ has quit IRC | 22:25 | |
| *** akshai has quit IRC | 22:29 | |
| *** ramishra has quit IRC | 22:30 | |
| *** jubapa has quit IRC | 22:30 | |
| *** jubapa has joined #tripleo | 22:31 | |
| *** tbonds has joined #tripleo | 22:31 | |
| *** ramishra has joined #tripleo | 22:32 | |
| *** chem has quit IRC | 22:36 | |
| *** chem has joined #tripleo | 22:36 | |
| *** rhallisey has joined #tripleo | 22:46 | |
| *** lucas-dinner has quit IRC | 22:51 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 22:51 |
| *** ramishra has quit IRC | 22:54 | |
| *** ramishra has joined #tripleo | 22:56 | |
| *** lucasagomes has joined #tripleo | 22:57 | |
| *** egafford has quit IRC | 23:06 | |
| *** dtrainor has joined #tripleo | 23:19 | |
| *** fragatina has quit IRC | 23:23 | |
| *** fragatina has joined #tripleo | 23:24 | |
| *** chlong_POffice has quit IRC | 23:25 | |
| *** bfournie has joined #tripleo | 23:25 | |
| *** chlong_POffice has joined #tripleo | 23:42 | |
| *** ramishra has quit IRC | 23:44 | |
| *** ramishra has joined #tripleo | 23:46 | |
| *** tzumainn has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!