| *** bootsha has quit IRC | 00:03 | |
| *** mbound has quit IRC | 00:04 | |
| *** bootsha has joined #tripleo | 00:06 | |
| *** bootsha has quit IRC | 00:11 | |
| *** mburned is now known as mburned_out | 00:28 | |
| *** mburned_out is now known as mburned | 00:30 | |
| *** limao has joined #tripleo | 00:32 | |
| *** saneax is now known as saneax_AFK | 00:37 | |
| *** seanatcisco has quit IRC | 00:40 | |
| *** seanatcisco has joined #tripleo | 00:46 | |
| *** seanatcisco_ has joined #tripleo | 00:49 | |
| *** seanatcisco has quit IRC | 00:51 | |
| *** seanatcisco_ is now known as seanatcisco | 00:51 | |
| *** fultonj has quit IRC | 00:52 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Fix CI status report to show all job results on 1 line https://review.openstack.org/348093 | 00:53 |
|---|---|---|
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Fix CI status report to show all job results on 1 line https://review.openstack.org/348093 | 00:54 |
| *** egafford has quit IRC | 00:54 | |
| *** apetrich has quit IRC | 01:00 | |
| *** apetrich has joined #tripleo | 01:01 | |
| *** mbound has joined #tripleo | 01:03 | |
| *** chlong has joined #tripleo | 01:05 | |
| *** mbound has quit IRC | 01:08 | |
| *** dmacpher has joined #tripleo | 01:14 | |
| *** seanatcisco has quit IRC | 01:21 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Fix CI status report to show all job results on 1 line https://review.openstack.org/348093 | 01:22 |
| *** coolsvap has joined #tripleo | 01:59 | |
| *** yamahata has quit IRC | 02:13 | |
| *** tzumainn has quit IRC | 02:17 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Enable keystone to use the SSL middleware https://review.openstack.org/341354 | 02:21 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Enable glance to use the SSL middleware https://review.openstack.org/341367 | 02:22 |
| *** thrash is now known as thrash|g0ne | 02:25 | |
| *** ayoung has quit IRC | 02:25 | |
| *** ayoung has joined #tripleo | 02:27 | |
| *** rwsu has quit IRC | 02:28 | |
| *** ayoung has quit IRC | 02:32 | |
| *** ayoung has joined #tripleo | 02:36 | |
| *** rwsu has joined #tripleo | 02:40 | |
| *** jrist has quit IRC | 02:55 | |
| *** myoung has quit IRC | 02:55 | |
| *** rlandy has quit IRC | 02:58 | |
| openstackgerrit | Xiang Chen proposed openstack/diskimage-builder: check APT_SOURCES earlier https://review.openstack.org/346605 | 03:09 |
| *** ayoung has quit IRC | 03:14 | |
| *** larstobi has quit IRC | 03:26 | |
| *** larstobi has joined #tripleo | 03:33 | |
| *** julim has quit IRC | 03:59 | |
| *** julim has joined #tripleo | 04:02 | |
| *** julim has quit IRC | 04:02 | |
| *** links has joined #tripleo | 04:03 | |
| *** liverpooler has quit IRC | 04:19 | |
| *** leanderthal|afk is now known as leanderthal | 04:26 | |
| *** masco has joined #tripleo | 04:35 | |
| *** pgadiya has joined #tripleo | 04:43 | |
| *** jrist has joined #tripleo | 05:03 | |
| *** ramishra_ has quit IRC | 05:22 | |
| *** ramishra has joined #tripleo | 05:24 | |
| *** rbrady has quit IRC | 05:34 | |
| *** jprovazn has joined #tripleo | 05:47 | |
| *** yamahata has joined #tripleo | 05:55 | |
| *** dmacpher has quit IRC | 06:06 | |
| *** oshvartz has joined #tripleo | 06:06 | |
| *** xion has joined #tripleo | 06:14 | |
| *** pcaruana has joined #tripleo | 06:17 | |
| *** dmacpher has joined #tripleo | 06:18 | |
| *** liverpooler has joined #tripleo | 06:41 | |
| *** tesseract- has joined #tripleo | 06:42 | |
| *** dsariel has joined #tripleo | 06:45 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT https://review.openstack.org/329415 | 06:55 |
| *** zoli_gone-proxy is now known as zoliXXL | 06:57 | |
| *** mhenkel has joined #tripleo | 07:00 | |
| *** jtomasek has joined #tripleo | 07:02 | |
| *** bootsha has joined #tripleo | 07:03 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: SR-IOV nic agent changes in line with composable roles. https://review.openstack.org/332190 | 07:06 |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Add max_reserved_vfs in THT https://review.openstack.org/345352 | 07:06 |
| bandini | morning | 07:10 |
| *** devvesa has joined #tripleo | 07:10 | |
| *** akuznetsov has joined #tripleo | 07:11 | |
| *** dculik|gone has joined #tripleo | 07:12 | |
| ccamacho | bandini happy Thursday! | 07:14 |
| *** hjensas has joined #tripleo | 07:14 | |
| ccamacho | Its really hot in Madrid, can't sleep ;( | 07:14 |
| bandini | ccamacho: hey hey Carlos | 07:14 |
| bandini | ccamacho: 15C here :) | 07:15 |
| *** ifarkas_away is now known as ifarkas | 07:15 | |
| ccamacho | :O 30.5C in my living room right now | 07:16 |
| marios | ccamacho: i feel your pain | 07:17 |
| marios | aircon from 0930 | 07:17 |
| marios | sall good till the bill comes :/ | 07:17 |
| *** dmacpher has quit IRC | 07:22 | |
| *** fzdarsky has joined #tripleo | 07:30 | |
| *** akuznets_ has joined #tripleo | 07:38 | |
| *** akuznetsov has quit IRC | 07:38 | |
| *** jbadiapa has joined #tripleo | 07:40 | |
| *** mcornea has joined #tripleo | 07:43 | |
| *** chlong has quit IRC | 07:47 | |
| *** rbrady has joined #tripleo | 07:50 | |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Update heat-agents setup files https://review.openstack.org/346859 | 07:55 |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Add default value for `RoleData` https://review.openstack.org/346860 | 07:55 |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: WIP: Containerized Services for Composable Roles https://review.openstack.org/330659 | 07:55 |
| openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Bind mount files to run DiD in latest atomic host https://review.openstack.org/347218 | 07:55 |
| *** jpich has joined #tripleo | 07:56 | |
| *** shardy has joined #tripleo | 08:00 | |
| *** akuznets_ has quit IRC | 08:01 | |
| *** akuznetsov has joined #tripleo | 08:03 | |
| *** hanchao has joined #tripleo | 08:06 | |
| *** mcornea has quit IRC | 08:09 | |
| *** mcornea has joined #tripleo | 08:11 | |
| *** hparekh has quit IRC | 08:11 | |
| *** akuznetsov has quit IRC | 08:13 | |
| *** derekh has joined #tripleo | 08:14 | |
| *** stendulker has joined #tripleo | 08:16 | |
| *** masco is now known as masco|lunch | 08:16 | |
| *** dculik|gone has quit IRC | 08:18 | |
| *** anshul has joined #tripleo | 08:20 | |
| *** lucas-dinner is now known as lucasagomes | 08:21 | |
| *** hparekh has joined #tripleo | 08:24 | |
| *** dculik|gone has joined #tripleo | 08:27 | |
| *** r-mibu has quit IRC | 08:31 | |
| *** xion has quit IRC | 08:31 | |
| *** numans has joined #tripleo | 08:34 | |
| *** paramite has joined #tripleo | 08:37 | |
| *** paramite has quit IRC | 08:37 | |
| *** paramite has joined #tripleo | 08:37 | |
| *** r-mibu has joined #tripleo | 08:41 | |
| *** ramishra has quit IRC | 08:43 | |
| *** ramishra has joined #tripleo | 08:44 | |
| *** saneax_AFK is now known as saneax | 08:51 | |
| *** hewbrocca-afk is now known as hewbrocca | 08:56 | |
| shardy | Hey all - EmilienM's big hieradata refactoring patch is now passing all CI, including RDO: | 09:00 |
| shardy | https://review.openstack.org/#/c/341616/20 | 09:00 |
| shardy | If anyone would like to review/approve that would be good, so we don't have to do another rebase | 09:00 |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Don't spam postci with deployment time loop https://review.openstack.org/346990 | 09:02 |
| hewbrocca | zoliXXL: time for my daily nag re the rack | 09:07 |
| hewbrocca | rh1, I mean | 09:07 |
| zoliXXL | hewbrocca, redeploy is almost done. Just need "peer review" from derekh for final confirmation | 09:08 |
| hewbrocca | wooooooooo | 09:08 |
| * hewbrocca excited for tasty, fresh CI goodness | 09:09 | |
| derekh | zoliXXL: hewbrocca all looks good | 09:10 |
| derekh | zoliXXL: next is to add it back to nodepool in infra, will push up a patch in a few minutes | 09:11 |
| shardy | \o/ | 09:11 |
| shardy | Nice work :) | 09:11 |
| *** saneax is now known as saneax_AFK | 09:12 | |
| mandre | shardy: quick question, do we need to follow a 2 cores +2 policy in tripleo-validations? | 09:15 |
| *** masco|lunch is now known as masco | 09:17 | |
| hewbrocca | woooooo | 09:17 |
| hewbrocca | derekh, zoliXXL I guess once we know everything is working, we can start turning back on some of the jobs? | 09:18 |
| derekh | hewbrocca: yup | 09:18 |
| * hewbrocca would really, really like to have an update job again | 09:18 | |
| hewbrocca | and, and, and :) | 09:18 |
| shardy | mandre: I think it should follow the same process as all other tripleo repos, so yes, but we do have allowance for trivial patches and stuff that's previously been reviewed then rebased | 09:19 |
| shardy | so I'd say use your judgement and ask if you need more reviews on things | 09:19 |
| shardy | clearly it's a bit tricky as there's not many folks working on tripleo-validations | 09:20 |
| *** limao has quit IRC | 09:22 | |
| mandre | shardy: sounds good, we'll follow the general tripleo process and see how it goes | 09:24 |
| *** electrofelix has joined #tripleo | 09:26 | |
| openstackgerrit | Merged openstack/tripleo-validations: Use tripleo bug tracker on launchpad https://review.openstack.org/347706 | 09:36 |
| *** electrofelix has quit IRC | 09:39 | |
| *** hanchao has quit IRC | 09:43 | |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Availability monitoring agents support https://review.openstack.org/254788 | 09:46 |
| *** mcornea has quit IRC | 09:50 | |
| *** mgould|afk is now known as mgould | 09:54 | |
| *** gfidente has joined #tripleo | 09:55 | |
| *** mbound has joined #tripleo | 09:56 | |
| *** yamahata has quit IRC | 09:58 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Migrate Puppet Hieradata to composable services https://review.openstack.org/341616 | 10:01 |
| *** mcornea has joined #tripleo | 10:05 | |
| *** electrofelix has joined #tripleo | 10:05 | |
| *** devvesa has quit IRC | 10:08 | |
| *** panda|Zz is now known as panda | 10:13 | |
| *** saneax_AFK is now known as saneax | 10:16 | |
| openstackgerrit | Julie Pichon proposed openstack/instack-undercloud: Increase Mistral limit for Task Input size https://review.openstack.org/348216 | 10:16 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Set secure_proxy_ssl_header 'HTTP_X_FORWARDED_PROTO' for nova-api https://review.openstack.org/347806 | 10:17 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Set secure_proxy_ssl_header 'HTTP_X_FORWARDED_PROTO' for nova-api https://review.openstack.org/347806 | 10:18 |
| EmilienM | hello | 10:22 |
| hewbrocca | EmilienM: good morning sir | 10:22 |
| gfidente | EmilienM, ^^ merged finally :) | 10:23 |
| ansiwen | EmilienM: hi | 10:23 |
| ansiwen | EmilienM: https://review.openstack.org/#/c/347981 removes the horizon-package, not keystone-package... probably just wrong commit message? | 10:24 |
| EmilienM | ansiwen: I woke up for you | 10:24 |
| EmilienM | gfidente: /me happy | 10:24 |
| Jokke_ | derekh & zoliXXL: well done with rh1. That was quick. | 10:25 |
| EmilienM | ansiwen: patches will merge this morning | 10:25 |
| EmilienM | ansiwen: a guy is fixing Fuel CI first | 10:25 |
| gfidente | EmilienM, though help NG HA now, check https://review.openstack.org/#/c/347315/ , https://review.openstack.org/#/c/347472/ which won't pass CI withouth > https://review.openstack.org/#/c/347314/ which passed | 10:26 |
| ansiwen | EmilienM: you woke up for me? | 10:26 |
| *** devvesa has joined #tripleo | 10:26 | |
| gfidente | EmilienM, and ceph too https://review.openstack.org/#/c/340286/ | 10:27 |
| derekh | shardy: bnemec slagle zoliXXL so I'm proposing we add back rh1 with only 2 slaves now to make sure things are working ok | 10:27 |
| derekh | https://review.openstack.org/#/c/348212/ | 10:27 |
| zoliXXL | derekh, I am OK with it | 10:28 |
| derekh | assuming alls ok, we can switch over all the jobs to rh1 along with the old jobs we removed | 10:28 |
| shardy | derekh: +1, sounds good | 10:29 |
| EmilienM | ansiwen: https://review.openstack.org/#/c/328347/ | 10:29 |
| *** bootsha has quit IRC | 10:29 | |
| ansiwen | EmilienM: yes, I know, gave my humble +1 already | 10:29 |
| *** tosky has joined #tripleo | 10:31 | |
| *** mbound has quit IRC | 10:32 | |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Validation to detect if undercloud disks use 512e https://review.openstack.org/322179 | 10:34 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Add documentation and dynamic inventory script https://review.openstack.org/322178 | 10:34 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Add network gateway validation https://review.openstack.org/329385 | 10:34 |
| derekh | shardy: bnemec zoliXXL slagle also starting next week I'm on PTO for 2 weeks so should problems crop up somebody will have to get stuck in, rh1 is pretty much the same as rh2 though so we should be done with the teething problems we had whne we switched to OVB | 10:36 |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Add pacemaker mistral profile https://review.openstack.org/343346 | 10:40 |
| shardy | derekh: Ok, should there be issues, who has access to investigate? | 10:44 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services https://review.openstack.org/323436 | 10:44 |
| derekh | shardy: people with access are bnemec slagle dprince and zoliXXL | 10:45 |
| derekh | shardy: Also if some bit of info is needed that for some reason I only know about you'll get me on hangouts if needed, I'll reply as soon as I get a chance | 10:49 |
| shardy | derekh: ack, thanks - we'll only interrupt your PTO as a last resort ;) | 10:49 |
| derekh | shardy: ok | 10:49 |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci: WIP - Merge same patchset jobs in cistatus https://review.openstack.org/348231 | 10:49 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Validation to detect if undercloud disks use 512e https://review.openstack.org/322179 | 10:52 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Add documentation and dynamic inventory script https://review.openstack.org/322178 | 10:52 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Add network gateway validation https://review.openstack.org/329385 | 10:52 |
| *** milan has joined #tripleo | 10:55 | |
| derekh | conundrum: how does one figure out which cloud they are running on from inside the VM.... rh1 or rh2? | 10:58 |
| *** dculik|gone has quit IRC | 10:59 | |
| *** bootsha has joined #tripleo | 11:02 | |
| *** zoliXXL is now known as zoli|lunch | 11:02 | |
| *** zoli|lunch is now known as zoli_gone-proxy | 11:03 | |
| *** fultonj has joined #tripleo | 11:04 | |
| *** ramishra has quit IRC | 11:08 | |
| *** weshay has joined #tripleo | 11:09 | |
| *** seanatcisco has joined #tripleo | 11:12 | |
| *** ramishra has joined #tripleo | 11:12 | |
| *** seanatcisco_ has joined #tripleo | 11:12 | |
| *** dculik|gone has joined #tripleo | 11:14 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: DO NOT MERGE testing manila https://review.openstack.org/342961 | 11:14 |
| *** seanatcisco has quit IRC | 11:16 | |
| *** seanatcisco_ is now known as seanatcisco | 11:16 | |
| *** devvesa has quit IRC | 11:18 | |
| *** bootsha has quit IRC | 11:19 | |
| *** bootsha has joined #tripleo | 11:20 | |
| *** stendulker has quit IRC | 11:20 | |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci: Merge same patchset jobs in cistatus https://review.openstack.org/348231 | 11:21 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Configure Nova [cache] options correctly (Mitaka only) https://review.openstack.org/348255 | 11:22 |
| *** devvesa has joined #tripleo | 11:30 | |
| *** bootsha has quit IRC | 11:30 | |
| *** bootsha has joined #tripleo | 11:31 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: allow RHEL virthosts to use an updated ipxe-roms-qemu https://review.openstack.org/348017 | 11:32 |
| *** tbonds has quit IRC | 11:34 | |
| gfidente | EmilienM, check my comment on the openstack-core submission | 11:34 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: allow RHEL virthosts to use an updated ipxe-roms-qemu https://review.openstack.org/348017 | 11:35 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Configure Nova [cache] options correctly (Mitaka only) https://review.openstack.org/348255 | 11:35 |
| EmilienM | gfidente: o | 11:35 |
| EmilienM | ok | 11:35 |
| EmilienM | gfidente: can we merge it now? | 11:36 |
| EmilienM | gfidente: I was wondering | 11:36 |
| EmilienM | gfidente: why don't we add core to the existing pacemaker service? | 11:36 |
| EmilienM | https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/pacemaker.yaml | 11:37 |
| *** rhallisey has joined #tripleo | 11:37 | |
| *** seanatcisco has quit IRC | 11:38 | |
| gfidente | because in NG HA we want the pacemaker service but we don't want the openstack-core resources | 11:38 |
| slagle | EmilienM: I guess the infra patch to clean resolv.conf is now breaking our OVB jobs: http://logs.openstack.org/36/323436/44/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/af7210f/console.html | 11:39 |
| EmilienM | gfidente: it makes sense now | 11:39 |
| EmilienM | slagle: f**** | 11:39 |
| slagle | EmilienM: can we just tell puppet what hostname to "retrieve the facts" for? | 11:40 |
| slagle | otherwise i think we're going to keep breaking one side or the other | 11:40 |
| EmilienM | slagle: yes we can, by exporting FACTER_fqdn | 11:40 |
| *** dculik|gone has quit IRC | 11:40 | |
| slagle | ok, let me try that | 11:40 |
| EmilienM | marios: can you +2 https://review.openstack.org/#/c/347314/ please? | 11:41 |
| *** ccamacho is now known as ccamacho|lunch | 11:41 | |
| *** bootsha has quit IRC | 11:41 | |
| *** bootsha has joined #tripleo | 11:42 | |
| *** ramishra has quit IRC | 11:42 | |
| gfidente | EmilienM, marios thanks for following up on this was a bit messy | 11:43 |
| *** skramaja has quit IRC | 11:43 | |
| gfidente | bandini, ^^ | 11:43 |
| *** skramaja has joined #tripleo | 11:43 | |
| EmilienM | gfidente: I'll let +a the patches once marios +2 the last one | 11:43 |
| *** karthiks has quit IRC | 11:45 | |
| bandini | awesome, thanks EmilienM & marios ! | 11:45 |
| slagle | derekh: hey, would this have an effect on our images in rh2? https://review.openstack.org/#/c/347970/ | 11:47 |
| *** bootsha has quit IRC | 11:47 | |
| slagle | re: https://bugs.launchpad.net/tripleo/+bug/1607334 | 11:47 |
| openstack | Launchpad bug 1607334 in tripleo "tripleo-ci OVB: Error: Could not run: Could not retrieve facts for localhost.localhost: no address for localhost.localhost" [Critical,In progress] - Assigned to James Slagle (james-slagle) | 11:47 |
| *** bootsha has joined #tripleo | 11:48 | |
| shardy | all - FYI I'm going to be offline for an hour or so catching a train, should be back in time for the deep dive | 11:50 |
| *** shardy has quit IRC | 11:50 | |
| derekh | slagle: could be, but I'm not sure how, looking | 11:52 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Explicitly set $FACTER_fqdn https://review.openstack.org/348264 | 11:52 |
| *** saneax is now known as saneax_AFK | 11:52 | |
| *** dculik|gone has joined #tripleo | 11:53 | |
| *** lucasagomes is now known as lucas-hungry | 11:54 | |
| *** chlong has joined #tripleo | 11:54 | |
| panda | what's the parameter that sets the controller public VIP ? | 11:55 |
| *** karthiks has joined #tripleo | 11:58 | |
| derekh | slagle: dhclient-eth0.conf has definitely changed so we're using the puppet module | 11:58 |
| derekh | slagle: see before and after hare http://paste.openstack.org/show/543250/ | 11:58 |
| *** skramaja has quit IRC | 11:58 | |
| slagle | derekh: k. figured | 11:58 |
| slagle | this is my proposed fix, https://review.openstack.org/#/c/348264/ | 11:58 |
| slagle | i see an impending blood bath of red coming on the ci status page :) | 11:59 |
| *** saneax_AFK is now known as saneax | 12:00 | |
| *** zoli_gone-proxy is now known as zoliXXL | 12:00 | |
| *** morazi has joined #tripleo | 12:00 | |
| bandini | the red wedding | 12:01 |
| EmilienM | pabelanger: fyi ^ | 12:02 |
| *** ramishra has joined #tripleo | 12:02 | |
| EmilienM | slagle: wow, fixing it in instack? | 12:02 |
| EmilienM | slagle: let's see how it works | 12:02 |
| slagle | EmilienM: where should I fix it? | 12:02 |
| EmilienM | slagle: I'm amazed by your puppet skills | 12:03 |
| EmilienM | slagle: no, it's perfect | 12:03 |
| slagle | LOL | 12:03 |
| EmilienM | dude, a few months ago you were trolling me | 12:03 |
| slagle | EmilienM: I guess I don't see any reason why we don't want "puppet apply" to just work, every time | 12:03 |
| slagle | and not detect a fqdn | 12:03 |
| EmilienM | yeah, I think it's good | 12:03 |
| slagle | "I don't always use puppet. But when I do, I control it via bash environment variables." | 12:04 |
| *** bootsha has quit IRC | 12:04 | |
| bandini | lol | 12:04 |
| EmilienM | ahah | 12:05 |
| *** bootsha has joined #tripleo | 12:05 | |
| *** radek_ has quit IRC | 12:05 | |
| EmilienM | bnemec: hey, I was about re-using your spec about +2 policy into Puppet OpenStack project. Do you mind? | 12:06 |
| *** morazi has quit IRC | 12:08 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add environment file to enable DVR https://review.openstack.org/332147 | 12:09 |
| *** gfidente has quit IRC | 12:09 | |
| *** morazi has joined #tripleo | 12:10 | |
| EmilienM | slagle: so iiuc it needs to work on RAX this time? | 12:11 |
| slagle | EmilienM: it's in rh2 (ovb jobs) where it is failing | 12:14 |
| *** thrash|g0ne is now known as thrash | 12:14 | |
| EmilienM | slagle: oh right... /me facepalm | 12:16 |
| EmilienM | slagle: it looks like it works then | 12:16 |
| *** cbits has joined #tripleo | 12:16 | |
| *** cbits has left #tripleo | 12:17 | |
| slagle | it's 8 minutes into the undercloud install... | 12:17 |
| slagle | we might be past the failure | 12:17 |
| openstackgerrit | Sanjay Upadhyay proposed openstack/tripleo-heat-templates: SR-IOV nic agent changes in line with composable roles. https://review.openstack.org/332190 | 12:20 |
| EmilienM | slagle: dude | 12:24 |
| EmilienM | it works | 12:24 |
| EmilienM | you are now my puppet hero | 12:24 |
| hewbrocca | OMG slagle PUPPET HERO | 12:24 |
| openstackgerrit | Merged openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly. https://review.openstack.org/328347 | 12:26 |
| * slagle cries a little | 12:28 | |
| EmilienM | lol | 12:33 |
| beagles | it's so nice to see such positive things in this channel :) | 12:33 |
| *** jdob has quit IRC | 12:33 | |
| slagle | EmilienM: cool, it's on to images now | 12:34 |
| *** jdob has joined #tripleo | 12:34 | |
| slagle | EmilienM: once the ovb job finishes, we should probably recheck a few times to make sure the multinode job works at least on rax and ovh | 12:34 |
| *** tbonds has joined #tripleo | 12:34 | |
| *** zoliXXL is now known as zoli|brb | 12:35 | |
| EmilienM | slagle: ok | 12:35 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Enable Aodh SSL binding in HAproxy https://review.openstack.org/348286 | 12:37 |
| *** radek_ has joined #tripleo | 12:37 | |
| *** tzumainn has joined #tripleo | 12:39 | |
| *** links has quit IRC | 12:40 | |
| *** rlandy has joined #tripleo | 12:40 | |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Enable Aodh SSL binding in HAproxy (Mitaka only) https://review.openstack.org/348288 | 12:40 |
| EmilienM | pradk: done, master & mitaka | 12:40 |
| pradk | can someone please review https://review.openstack.org/#/c/344864/ .. need this downstream asap | 12:40 |
| pradk | EmilienM, awesome thx.. i'll link the patches to the bug | 12:41 |
| EmilienM | cool | 12:42 |
| *** julim has joined #tripleo | 12:43 | |
| *** apetrich has quit IRC | 12:43 | |
| *** chem is now known as chem|lunch | 12:43 | |
| *** trown|outtypewww is now known as trown | 12:43 | |
| *** apetrich has joined #tripleo | 12:43 | |
| *** egafford has joined #tripleo | 12:44 | |
| *** liverpooler has quit IRC | 12:45 | |
| EmilienM | can a core +2 https://review.openstack.org/#/c/347314/ ? | 12:45 |
| EmilienM | so we can land both patches together and gfidente can make progress | 12:46 |
| *** mbound has joined #tripleo | 12:49 | |
| trown | kind of hard to understand what that is for from just the review | 12:49 |
| marios | EmilienM: i did it i am already +2 puppet-tripleo side | 12:51 |
| *** panda is now known as panda|bootcamp | 12:51 | |
| EmilienM | marios: tht is missing | 12:51 |
| EmilienM | ok let's merge it now | 12:52 |
| EmilienM | I approved the puppet-tripleo patch | 12:52 |
| marios | EmilienM: done sry i thought i hit post | 12:52 |
| EmilienM | and once it merges I'll approve THT | 12:52 |
| EmilienM | CI is currently broken anyway until slagle's patch is merged. | 12:52 |
| *** rbrady has quit IRC | 12:53 | |
| EmilienM | (only ovb job is currently broken) | 12:53 |
| *** rbrady has joined #tripleo | 12:53 | |
| *** bfournie has quit IRC | 12:54 | |
| *** masco has quit IRC | 12:54 | |
| *** ccamacho|lunch is now known as ccamacho | 12:55 | |
| ccamacho | derekh mind to review? https://review.openstack.org/#/c/348231/ this is the submission to fix the CI status page table | 12:56 |
| *** pradk has quit IRC | 12:58 | |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart: Make the number of tempest workers configurable https://review.openstack.org/348300 | 12:59 |
| EmilienM | slagle: tested on internap, it worked | 13:00 |
| *** lucas-hungry is now known as lucasagomes | 13:03 | |
| EmilienM | ccamacho: what is progress on aodh? | 13:03 |
| *** numans has quit IRC | 13:06 | |
| *** ramishra has quit IRC | 13:07 | |
| ccamacho | EmilienM I thought pradk was working on it, but, let me push it, Ill test it see whats going on there | 13:07 |
| ccamacho | EmilienM Mistral its finished, and disabled now by default in the overcloud | 13:08 |
| *** zoli|brb is now known as zoliXXL | 13:08 | |
| *** saneax is now known as saneax_AFK | 13:09 | |
| EmilienM | ccamacho: we need to push aodh before mistral | 13:11 |
| EmilienM | aodh is the last service in THT | 13:11 |
| EmilienM | that has puppet code | 13:11 |
| ccamacho | EmilienM quick question, there are some pending submissions to land re to composable services mistral, aodh, zaqar, manila | 13:12 |
| ccamacho | EmilienM can you point me with some prio there? | 13:12 |
| ccamacho | to work on the most relevant first | 13:12 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Documentation how validations work https://review.openstack.org/322178 | 13:13 |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles https://review.openstack.org/332854 | 13:13 |
| openstackgerrit | Merged openstack/puppet-tripleo: Create role for the fake openstack-core resource https://review.openstack.org/347315 | 13:14 |
| derekh | ccamacho: I just noticed slagle beat you too it https://review.openstack.org/#/c/348093/3 looking at them now | 13:14 |
| EmilienM | ccamacho: aodh is top prio | 13:14 |
| openstackgerrit | Merged openstack/puppet-tripleo: Remove global openstack-core resource https://review.openstack.org/347472 | 13:14 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Add a dynamic ansible inventory script https://review.openstack.org/348305 | 13:14 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 13:14 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 13:15 |
| *** chem|lunch has quit IRC | 13:16 | |
| *** chem|lunch has joined #tripleo | 13:16 | |
| *** gfidente has joined #tripleo | 13:16 | |
| *** gfidente has quit IRC | 13:16 | |
| *** gfidente has joined #tripleo | 13:16 | |
| *** numans has joined #tripleo | 13:17 | |
| *** akshai has joined #tripleo | 13:18 | |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles https://review.openstack.org/333556 | 13:19 |
| ccamacho | derekh, man didnt saw that submission, well, I did not see it.. | 13:20 |
| ccamacho | s/saw/see | 13:20 |
| *** jcoufal has joined #tripleo | 13:21 | |
| ccamacho | EmilienM ack, will check it then | 13:21 |
| *** myoung has joined #tripleo | 13:22 | |
| EmilienM | slagle: do you want to recheck CI jobs on https://review.openstack.org/#/c/348264 or could we land it? | 13:23 |
| *** ramishra has joined #tripleo | 13:23 | |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Fix roles based CI jobs https://review.openstack.org/348309 | 13:23 |
| slagle | EmilienM: i guess we can land it | 13:25 |
| slagle | go4it | 13:25 |
| EmilienM | done | 13:25 |
| *** karthiks has quit IRC | 13:26 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Create role for the fake openstack-core resource https://review.openstack.org/347314 | 13:26 |
| trown | we must have solved this some other way ya? https://review.openstack.org/#/c/330148/ | 13:27 |
| trown | just looking through open reviews... feels odd to abandon one with +9 | 13:27 |
| trown | but we cant be hitting that | 13:27 |
| *** seanatcisco has joined #tripleo | 13:28 | |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Port the network environment validation https://review.openstack.org/341586 | 13:29 |
| *** devvesa has quit IRC | 13:30 | |
| *** bootsha has quit IRC | 13:30 | |
| *** morazi has quit IRC | 13:30 | |
| slagle | trown: was fixed by a different patch | 13:31 |
| derekh | trown: | 13:31 |
| derekh | trown: yup, what he said | 13:31 |
| slagle | https://review.openstack.org/#/c/334467/ | 13:31 |
| trown | thanks! | 13:31 |
| flaper87 | Anyway I can tell instack to create a bigger disk for the undercloud ? | 13:31 |
| slagle | i came in late after derekh, and took all the credit | 13:31 |
| flaper87 | Any way* | 13:31 |
| *** trozet has quit IRC | 13:32 | |
| slagle | flaper87: using instack-virt-setup? or something else | 13:33 |
| openstackgerrit | Merged openstack/instack-undercloud: Explicitly set $FACTER_fqdn https://review.openstack.org/348264 | 13:33 |
| flaper87 | slagle: yeah, using instack-virt-setup | 13:33 |
| flaper87 | Tried to find an env var but there doesn't seem to be one | 13:33 |
| * flaper87 is missing something | 13:34 | |
| slagle | flaper87: UNDERCLOUD_NODE_DISK i think | 13:34 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement basic undercloud sanity checks https://review.openstack.org/346551 | 13:34 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Validation to detect if undercloud disks use 512e https://review.openstack.org/322179 | 13:34 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 13:34 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 13:34 |
| slagle | flaper87: it was recently added | 13:34 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Configure Nova [cache] options correctly (Mitaka only) https://review.openstack.org/348255 | 13:35 |
| *** bfournie has joined #tripleo | 13:35 | |
| *** trozet has joined #tripleo | 13:35 | |
| flaper87 | gotcha, thanks | 13:35 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Allow to configure Ironic [ssh] section https://review.openstack.org/347455 | 13:35 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Add network gateway validation https://review.openstack.org/329385 | 13:36 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Remove usage of ::nova class in THT https://review.openstack.org/325983 | 13:36 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Fix CI status report to show all job results on 1 line https://review.openstack.org/348093 | 13:38 |
| *** karthiks has joined #tripleo | 13:38 | |
| openstackgerrit | Imre Farkas proposed openstack/instack-undercloud: Default to Ironic API v1.15 on undercloud https://review.openstack.org/348323 | 13:39 |
| trown | EmilienM: so issue was legit on https://review.openstack.org/#/c/341616/20 then? | 13:42 |
| trown | that controller_classes var was missiing | 13:42 |
| EmilienM | trown: yes | 13:42 |
| trown | good thing I tested it | 13:42 |
| EmilienM | trown: it was not failing only in RDO CI | 13:42 |
| trown | why was nobody else able to reproduce it? | 13:42 |
| EmilienM | it was failing everywhere | 13:42 |
| EmilienM | trown: the error appeared yesterday or so, when I did a wrong rebase | 13:43 |
| *** ayoung has joined #tripleo | 13:43 | |
| trown | k | 13:43 |
| *** morazi has joined #tripleo | 13:43 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Adds Deployment Action & Workflow https://review.openstack.org/298732 | 13:43 |
| *** shardy has joined #tripleo | 13:44 | |
| openstackgerrit | Merged openstack/tripleo-quickstart: Only run tempest on the minimal config https://review.openstack.org/348010 | 13:45 |
| *** jcoufal has quit IRC | 13:46 | |
| *** jcoufal has joined #tripleo | 13:47 | |
| EmilienM | deep dive session starts in 10 min, on bluejeans: https://bluejeans.com/275264340 | 13:51 |
| EmilienM | as a reminder, this week we'll talk about Puppet things | 13:51 |
| *** pgadiya has quit IRC | 13:51 | |
| pabelanger | morning | 13:51 |
| pabelanger | EmilienM: issues with dhclient.conf? | 13:51 |
| jistr | mitigation of our always-restarting issue -- any takers? :)) https://review.openstack.org/#/c/326364/ | 13:51 |
| EmilienM | pabelanger: fqdn, we fixed with FACTER_fqdn | 13:53 |
| pabelanger | EmilienM: great | 13:53 |
| pabelanger | EmilienM: was there an issue with the puppet integration tests? | 13:54 |
| *** dsariel has quit IRC | 13:54 | |
| slagle | is was actually in rh2 with the ovb jobs | 13:54 |
| slagle | once our images got updated | 13:54 |
| pabelanger | slagle: ya, what I am curious about is if it is limited to tripleo-ci or also affected all puppet testing for centos-7 | 13:56 |
| pabelanger | if tripleo, we can then debug why it is only failing here | 13:56 |
| *** ramishra has quit IRC | 13:57 | |
| jistr | thanks shardy | 13:57 |
| *** mcornea has quit IRC | 13:58 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Allow to manually disable post-puppet restarts https://review.openstack.org/326364 | 13:59 |
| slagle | dd starting now bluejeans.com/275264340/ | 13:59 |
| derekh | slagle: could this hostname setting be confusing things ? http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/toci_instack_ovb.sh#n21 | 13:59 |
| *** mcornea has joined #tripleo | 13:59 | |
| *** jbadiapa has quit IRC | 13:59 | |
| slagle | derekh: i'm not sure. YIL puppet uses resolv.conf to get the domainname | 14:00 |
| *** skramaja has joined #tripleo | 14:00 | |
| *** dmacpher has joined #tripleo | 14:00 | |
| *** saneax_AFK is now known as saneax | 14:00 | |
| derekh | TIL YIL | 14:01 |
| slagle | i just made it up :) | 14:01 |
| derekh | slagle: the internet says its good | 14:01 |
| *** pradk has joined #tripleo | 14:02 | |
| *** ramishra has joined #tripleo | 14:02 | |
| panda|bootcamp | uhm ... deep dive ? | 14:03 |
| beagles | panda|bootcamp, different room | 14:04 |
| beagles | the correct link is in the etherpad | 14:04 |
| slagle | 09:59 < slagle> dd starting now bluejeans.com/275264340/ | 14:04 |
| *** leanderthal is now known as leanderthal|afk | 14:05 | |
| *** panda|bootcamp is now known as panda|deepdive | 14:06 | |
| *** thrash is now known as thrash|biab | 14:07 | |
| pabelanger | slagle: derekh: EmilienM: I don't think that is the right fix of this issue. For some reason, the hostname is no longer getting setup in tripleo-test-cloud-rh2, this should be done with glean. If I log into a VM, I see the hostname is simply localhost, when it should be tripleo-centos-7-ovb-tripleo-test-cloud-rh2-2984012. Let me see what is happening, but don't believe we should be setting using facter to | 14:07 |
| pabelanger | setup the FQDN | 14:07 |
| slagle | pabelanger: the fix might not address the root cause...but i still think we want it in | 14:08 |
| slagle | 100% of the time we just want puppet apply to run against the system that executed it | 14:09 |
| pabelanger | slagle: have you tried --certname= with facter? | 14:09 |
| slagle | pabelanger: no. where would we add that? | 14:10 |
| pabelanger | puppet apply | 14:10 |
| slagle | i did not see that in the help | 14:10 |
| pradk | can i get a +a on this https://review.openstack.org/#/c/344864/ please? | 14:11 |
| pabelanger | I am not sure if facter will us it, but when I don't want to deal with DNS issue on puppet runs, I manually setup my certname to what my site.pp expects | 14:11 |
| pabelanger | but, I still think setting FDQN in facter is not correct. If FDQN is really an issue, we should be using linux commands to do it. EG: hostname / domainname. But, this is what glean does | 14:12 |
| slagle | pabelanger: it has been set via linux commands | 14:12 |
| slagle | i dont think this causes puppet to set that hostname | 14:13 |
| slagle | it just overrides the fact value | 14:13 |
| pabelanger | right, but facter should be just reading output of hostname | 14:13 |
| pabelanger | which in tripleo-test-cloud-rh2 is localhost | 14:14 |
| pabelanger | I'm looking into why glean is failing, so I should know more information soon | 14:14 |
| slagle | ok | 14:14 |
| slagle | we can always go with a better fix | 14:14 |
| derekh | slagle: if you setup a devenv on rh2 you might be able to see whats going wrong https://etherpad.openstack.org/p/tripleo-ci-devenvs | 14:14 |
| slagle | if puppet just used hostname to get fqdn, then we wouldnt keep having these issues... i think | 14:15 |
| slagle | instead it has implemented it's own thing in the fqdn fact | 14:15 |
| pabelanger | Ya, I don't disagree it fixes things now, and you need a working CI, but I'm not a fan of ignoring the problem by adding code to mask it. And since this is problem with infra, we can fix it in openstack-infra now. | 14:15 |
| *** oshvartz has quit IRC | 14:16 | |
| *** chem|lunch has quit IRC | 14:17 | |
| slagle | i'm not a fan of that either | 14:17 |
| slagle | but i also disagree that's what we've done | 14:17 |
| pabelanger | derekh: is tripleo-test-cloud-rh2 running config-drive? | 14:18 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 14:19 |
| derekh | pabelanger: it would be, those images wouldn't work at all without it | 14:19 |
| pabelanger | derekh: Ya, that's what I figured too. Same me some time looking into logs | 14:20 |
| pabelanger | derekh: okay, /mnt/config is missing on tripleo-test-cloud-rh2, that is where glean looks for config-drive. Let me see why that is missing | 14:24 |
| hewbrocca | ruh roh | 14:24 |
| derekh | pabelanger: I've just run a VM with $ nova boot --config-drive=true --image template-centos-7-1469674033 --flavor undercloud --nic net-id=7b8661c6-b8fa-4bde-be13-bcbae87b1107 --key-name default derekh-undercloud | 14:26 |
| derekh | pabelanger: and /mnt/config is there | 14:26 |
| derekh | pabelanger: does nodepool assume its config drive by default? | 14:27 |
| pabelanger | derekh: yes, I think I see the problem. nodepool is not setup to use it for tripleo-test-cloud-rh2, let me get a patch up to enable it | 14:27 |
| derekh | pabelanger: so we're relying on the fact that ssh keys are baked into the image at the moment ? | 14:28 |
| pabelanger | derekh: baked how? images are created by nodepool today. which, should have the proper keys | 14:29 |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Customize the reservation of memory for host processes https://review.openstack.org/327705 | 14:29 |
| pabelanger | derekh: lets confirm in openstack-infra | 14:30 |
| derekh | pabelanger: thats what I mean, the keys are built into the images when its created, we're not relying on glean to set up keys at all | 14:30 |
| *** dsariel has joined #tripleo | 14:30 | |
| *** mcornea has quit IRC | 14:31 | |
| pabelanger | derekh: that is fine, we don't pass keys via glean today | 14:31 |
| *** mcornea has joined #tripleo | 14:31 | |
| derekh | pabelanger: ok | 14:31 |
| *** shardy has quit IRC | 14:31 | |
| *** shardy has joined #tripleo | 14:34 | |
| openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT https://review.openstack.org/329415 | 14:34 |
| *** jistr is now known as jistr|call | 14:35 | |
| *** ramishra has quit IRC | 14:36 | |
| *** thrash|biab is now known as thrash | 14:36 | |
| *** ramishra has joined #tripleo | 14:37 | |
| beagles | the problem with giving a pretty complete overview in a deep-dive is that when it comes time to ask if there are any questions there aren't going to be many :) | 14:38 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO https://review.openstack.org/347739 | 14:40 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo https://review.openstack.org/347250 | 14:40 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo https://review.openstack.org/346975 | 14:40 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Corrected syntax mistakes https://review.openstack.org/347385 | 14:40 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail config puppet module for tripleO https://review.openstack.org/346938 | 14:40 |
| *** chem has joined #tripleo | 14:40 | |
| *** jbadiapa has joined #tripleo | 14:43 | |
| *** radek_ has quit IRC | 14:43 | |
| pradk | trown, thx for the ooo tshirt :) | 14:47 |
| *** jistr|call is now known as jistr | 14:49 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail database puppet module for tripleO https://review.openstack.org/348406 | 14:50 |
| beagles | EmilienM, great deep-dive session! | 14:51 |
| trown | pradk: I was just the courier... thanks goes to jrist | 14:51 |
| EmilienM | did you like french accent? | 14:51 |
| beagles | frickin' awesome! | 14:51 |
| beagles | :) | 14:51 |
| beagles | adds flavour | 14:51 |
| trown | great job EmilienM | 14:51 |
| pradk | trown, ah cool .. thx jrist | 14:51 |
| chem | EmilienM: arghh, could not be there, looking forward to get the video | 14:52 |
| bfournie | EmilienM: great presentation, thanks! | 14:52 |
| jrist | pradk: :) | 14:52 |
| jrist | trown deserves credit too though :) | 14:52 |
| jmiu_ | "adds flavour" > yes | 14:52 |
| *** panda|deepdive is now known as panda|bootcamp | 14:53 | |
| ayoung | EmilienM, shardy one thing I would like to get documented is how to trace values from the original openstack deploy command line all the way through to the puppet apply. I tried adding a -e <file> to my deploy command libne, as we discussed yesterday and it just did not show up in the hiera data, so I want to be able to trace through what happened. | 14:53 |
| ayoung | Its possible that I had a value wrong, or that it was the CLI or many other things. | 14:54 |
| ayoung | EmilienM got me as far as being able to check the hiera data on the compute node, but I would like to be able to trace from both ends and meet in the middle to understand where the values got dropped | 14:55 |
| beagles | ayoung, this can sometime happen if you set configuration for a puppet class that hasn't been included yet | 14:55 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Add Contrail composable service Database https://review.openstack.org/348408 | 14:55 |
| ayoung | beagles, can you expand on that? | 14:55 |
| beagles | ayoung, I'm not entirely certain how that plays though... | 14:56 |
| ayoung | beagles, nevermind that, in this case, I am adding things to ControllerPostConfig for Keystone, | 14:56 |
| beagles | ayoung, ahhh | 14:56 |
| beagles | ayoung, okay.. I was thinking of ExtraConfig stuff | 14:56 |
| bnemec | EmilienM: Go for it, re: the +2 spec. | 14:56 |
| bnemec | It's not really _my_ spec anyway. :-) | 14:56 |
| ayoung | I added a http://paste.openstack.org/show/543289/ | 14:56 |
| bnemec | Just a set of guidelines that tripleo has collectively come up with over the years. | 14:56 |
| beagles | ayoung, "making assumptions" in other words | 14:57 |
| openstackgerrit | Martin André proposed openstack/tripleo-validations: Port the network environment validation https://review.openstack.org/341586 | 14:57 |
| EmilienM | bnemec: ack thx | 14:57 |
| EmilienM | bnemec: I found it very useful | 14:57 |
| ayoung | beagles, and added to the call -e /home/stack/keystone_extra_config.yml as the last parameter | 14:57 |
| *** ifarkas is now known as ifarkas_afk | 14:58 | |
| ayoung | beagles, so, after the controller deployed, I looked in the hiera data like this | 14:58 |
| ayoung | sudo grep -r keystone:: /etc/puppet/hieradata/ | sed 's!.*yaml:!!g' | 14:58 |
| shardy | ayoung: maybe I should write another blog post about the value tracing thing | 14:58 |
| * ayoung realizes it might not be called yaml... | 14:58 | |
| shardy | it's pretty similar to my previous tripleo debugging post really | 14:58 |
| openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys and cluster FSID via generate_overcloud_passwords https://review.openstack.org/348412 | 14:59 |
| ayoung | shardy, that one has been super valuable and helpful | 14:59 |
| jmiu_ | shardy: by the way your tripleo debugging post is gold. thank you! | 14:59 |
| shardy | you pass -e extra_stuff.yaml, which for example contains ControllerExtraConfig as a parameter_default | 14:59 |
| *** numans has quit IRC | 14:59 | |
| ayoung | shardy, right...now that is processed by osc first, but passed to the tripleo client? | 14:59 |
| shardy | then you can introspect the nested stack (e.g the OS::TripleO::Controller one) via heat stack-show <id> | 14:59 |
| ayoung | shardy, so that shows the Parameters | 15:00 |
| shardy | and see the updated ControllerExtraConfig | 15:00 |
| shardy | then on the node, you should see the corresponding updated hieradata | 15:00 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 15:00 |
| shardy | in /etc/puppet/hieradata/. | 15:00 |
| shardy | ayoung: tripleoclient is an osc plugin, it just grabs all the data and calls heat | 15:01 |
| shardy | although in the future it will call a mistral workflow instead of heat directly | 15:01 |
| shardy | if you run it with --debug, you can see the call to heat with your parameter_defaults | 15:01 |
| ayoung | shardy, so I do | 15:01 |
| ayoung | openstack stack show overcloud | less | 15:01 |
| ayoung | and searcgh for | 15:01 |
| ayoung | ontrollerExtraConfig: which I find | 15:02 |
| ayoung | controllerExtraConfig: '{u''heat::api_cloudwatch::enabled'': False, u''heat::config::heat_config'': | 15:02 |
| ayoung | looks like it is truncated | 15:02 |
| ayoung | is there a way I can see that value? | 15:02 |
| ayoung | and, I note that it starts with a lowercase, and I used upper. Maybe that is part of the problem? | 15:02 |
| shardy | ayoung: we actually support both, controllerExtraConfig is the old deprecated format | 15:03 |
| shardy | either should work on recent templates | 15:03 |
| ayoung | shardy, can I somehow dump out that whole parameter value? | 15:03 |
| derekh | zoliXXL: so one thing I forgot was that post deployment there were 4 things I had changed on rh2, we'll need them on rh1 also | 15:04 |
| shardy | ayoung: if you're using ControllerExtraConfig, that isn't the one you want, sec | 15:04 |
| slagle | jistr: could we use the postpuppetrestart noop on initial deployment? | 15:04 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: We don't need to set a default for the CephX keys and cluster FSID https://review.openstack.org/348415 | 15:04 |
| * shardy spins up an overcloud | 15:04 | |
| derekh | zoliXXL: 1. I've asked in infra for a release of gear with the fix but for now it still needs to be applied manually | 15:04 |
| derekh | zoliXXL: 2. I've asked in #openstack-ironic about a release of pyghmi but for now it still needs to be applied manually | 15:05 |
| derekh | zoliXXL: 3, mightn't be needed | 15:05 |
| derekh | lets wait and see | 15:05 |
| derekh | zoliXXL: and 4. I'm applying now to the overcloud | 15:05 |
| *** wgd3 has joined #tripleo | 15:08 | |
| *** ccamacho is now known as ccamacho|afk | 15:08 | |
| *** jmiu has quit IRC | 15:09 | |
| *** jmiu_ is now known as jmiu | 15:09 | |
| *** jmiu_ has joined #tripleo | 15:09 | |
| shardy | ayoung: https://paste.fedoraproject.org/396910/69718599/ | 15:10 |
| ayoung | shardy, so, just to start from basics. If I have -e <file> that contains parameter_defaults:\n ControllerExtraConfig:\n keystone::using_domain_config: true\n (in proper yaml) I should see something in the parameters for the Heat stack that spins up | 15:10 |
| shardy | ayoung: Yes, in the nested OS::TripleO::Controller stack, see paste | 15:10 |
| ayoung | shardy, so only in the nested stack? | 15:11 |
| shardy | if osc eats the output, try heat stack-show, the truncation could be an osc bug | 15:11 |
| shardy | ayoung: yes | 15:11 |
| ayoung | OK...let's start with that... | 15:11 |
| shardy | ayoung: parameter_defaults are made available to all stacks, which means we don't have to pass like 500 parameters via the overcloud.yaml stack | 15:11 |
| ayoung | shardy, ok so I have 3 resources that match that grep | 15:12 |
| ayoung | and all have overcloud-Controller-y2lmuipmynnt | 15:12 |
| ayoung | I assume that is the stack name...confirming | 15:12 |
| ayoung | OK, so I want to do a stack show on that one... | 15:13 |
| ayoung | for parameters that has {"OS::project_id": "24addb62df304757bfc19c4b7f93d0b5","OS::stack_id": "1fc7433f-4c5d-40b2-a526-5f1b1ae8a23c","OS::stack_name": "overcloud-Controller-y2lmuipmynnt"} | 15:14 |
| shardy | ayoung: You missed the space from my example | 15:17 |
| shardy | "OS::TripleO::Controller " | 15:17 |
| shardy | then you should get one hit for each controller (in my env it's one) | 15:17 |
| ayoung | shardy, actually, I made a different mistake, but I did make a mistake | 15:17 |
| shardy | hehe :) | 15:17 |
| *** openstackgerrit has quit IRC | 15:18 | |
| shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/controller.yaml#L15 | 15:18 |
| *** openstackgerrit has joined #tripleo | 15:18 | |
| shardy | you want to look at that, and that heat stack is mapped to OS::TripleO::Controller | 15:18 |
| shardy | so you just need the ID of it | 15:18 |
| ayoung | shardy, if I cut and paste your line exactly, I get 3 resources. Each has the same stack name https://paste.fedoraproject.org/396915/46971914/ | 15:19 |
| ayoung | I searched each of those and did not see my values...like this | 15:19 |
| ayoung | for stackid in b4a558a2-297d-46c6-b658-46f9dc0fcd51 5b93eee2-97f6-4b8e-b9a0-b5edde6b4795 1fdfdfa9-759b-483c-a943-94f4c7b04d3b ; do heat stack-show $stackid | grep ControllerExtraConfig ; done | 15:20 |
| ayoung | and...shorting the output | 15:20 |
| ayoung | "ControllerExtraConfig": "{u'heat::api_cloudwatch::enabled': False, u'heat::config::heat_config': {u'DEFAULT/num_engine_workers': {u'value': 1}}, u'heat::api_cfn::enabled': False}", | 15:20 |
| ayoung | "ControllerExtraConfig": "{u'heat::api_cloudwatch::enabled': False, u'heat::config::heat_config': {u'DEFAULT/num_engine_workers': {u'value': 1}}, u'heat::api_cfn::enabled': False}", | 15:20 |
| ayoung | "ControllerExtraConfig": "{u'heat::api_cloudwatch::enabled': False, u'heat::config::heat_config': {u'DEFAULT/num_engine_workers': {u'value': 1}}, u'heat::api_cfn::enabled': False}", | 15:20 |
| ayoung | It seems like it has ignored my file. | 15:20 |
| ayoung | let me try running the overcloud deploy again with debug enabled and see what it gets | 15:21 |
| shardy | ayoung: where does that heat stuff come from? | 15:21 |
| shardy | by default it should be empty | 15:21 |
| shardy | do you define ControllerExtraConfig in more than one environment file? | 15:21 |
| ayoung | shardy, I'm running quickstart. I assume it was one of the env vars set in there... | 15:21 |
| *** pcaruana has quit IRC | 15:22 | |
| ayoung | ah...maybe one of those other files has already defined the ExtraConfig.... | 15:22 |
| ayoung | OK..glad I didn't run the redeploy...let me look | 15:22 |
| shardy | ayoung: You can only define it once, "last one wins" | 15:22 |
| shardy | we just do a dict.update internally | 15:22 |
| ayoung | hmmm...mine should be the last one | 15:22 |
| shardy | it's not deep merged | 15:22 |
| ayoung | but...let' | 15:22 |
| ayoung | let's see. | 15:22 |
| shardy | ayoung: I'd do openstack overcloud deploy --templates /path/to/templates -e the_env.yaml | 15:23 |
| shardy | I personally don't like having a wrapper script for development | 15:23 |
| shardy | I know quickstart gives you one tho | 15:23 |
| ayoung | shardy, yeah, but I edit the wrapper, so I have a consistent env each time I run | 15:24 |
| shardy | ayoung: ack, I guess that works, something must be adding that heat stuff there I suppose | 15:24 |
| ayoung | openstack overcloud deploy --debug --templates --libvirt-type qemu --control-flavor oooq_control --compute-flavor oooq_compute --ceph-storage-flavor oooq_ceph --timeout 90 -e /usr/share/openstack-tripleo-heat-templates/environments/network-isolation.yaml -e /usr/share/openstack-tripleo-heat-templates/environments/net-single-nic-with-vlans.yaml -e $HOME/network-environment.yaml --control-scale 3 --neutron-network-type vxlan --neutron-tunnel-t | 15:24 |
| ayoung | ypes vxlan -e /usr/share/openstack-tripleo-heat-templates/environments/puppet-pacemaker.yaml --ntp-server pool.ntp.org \ | 15:24 |
| ayoung | ${DEPLOY_ENV_YAML:+-e $DEPLOY_ENV_YAML} -e /home/stack/keystone_extra_config.yml "$@"|| deploy_status=1 | 15:24 |
| ayoung | its pretty lengthy. I would not want to keep track of all that each time I ran the command | 15:25 |
| ayoung | ah | 15:25 |
| shardy | ayoung: For development, you really don't need all that | 15:25 |
| ayoung | /tmp/deploy_env.yaml:2: controllerExtraConfig: | 15:25 |
| ayoung | ok...so I am battling the config file he generated...got it | 15:25 |
| jistr | slagle: yes you could use it i think, but it's already no-opped by default afaik | 15:27 |
| jistr | i mean the no-op restart for initial deployment | 15:27 |
| ayoung | shardy, yeah, quickstart might diverge from expected development paths. Something we need to address. | 15:27 |
| jistr | slagle: https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/tasks/pacemaker_resource_restart.sh#L11 | 15:28 |
| *** dtrainor has quit IRC | 15:28 | |
| EmilienM | can someone +A this bugfix please? https://review.openstack.org/#/c/348286/ | 15:28 |
| EmilienM | and https://review.openstack.org/#/c/348288/ | 15:28 |
| *** dtrainor has joined #tripleo | 15:28 | |
| slagle | jistr: oh ok, forgot you added that :) | 15:29 |
| EmilienM | slagle, bnemec: do you mind reviewing https://review.openstack.org/#/c/346551/ and https://review.openstack.org/#/c/346147/ ? things look more stable now, and I want to make progress on undercloud jobs | 15:29 |
| trown | EmilienM: the second one "counts" as a backport in that I can +A? | 15:30 |
| trown | EmilienM: oh second is not passing CI though | 15:30 |
| EmilienM | trown: it won't pass multijob | 15:30 |
| EmilienM | trown: multijob will not pass before Newton | 15:30 |
| EmilienM | slagle: btw, I'm preparing a patch to not run multinode job before newton ok? | 15:31 |
| ayoung | shardy, OK, accounted for that, running again. If this is run against a deployed overcloud, I should expect to see the updated keystone conf file anyway, right? | 15:31 |
| trown | EmilienM: ah k, thanks | 15:31 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Convert service_name to underscore syntax https://review.openstack.org/348440 | 15:31 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add EnabledServices output to overcloud.yaml https://review.openstack.org/348441 | 15:31 |
| shardy | All would like feedback on ^^ - I want to change the service_name from the-service to the_service | 15:32 |
| shardy | this should enable us to use the service names to dynamically generate things like the per service ips lists in overcloud.yaml | 15:32 |
| EmilienM | shardy: looking | 15:36 |
| *** saneax is now known as saneax_AFK | 15:36 | |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart: Make the number of tempest workers configurable https://review.openstack.org/348300 | 15:37 |
| openstackgerrit | James Slagle proposed openstack/tripleo-quickstart: Use sudo for yum install https://review.openstack.org/348446 | 15:40 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: oooq support check, exit on anything other than CentOS and RHEL https://review.openstack.org/348062 | 15:41 |
| *** jistr is now known as jistr|afk | 15:42 | |
| *** shardy has quit IRC | 15:43 | |
| openstackgerrit | Merged openstack/instack-undercloud: Enable Aodh SSL binding in HAproxy https://review.openstack.org/348286 | 15:45 |
| openstackgerrit | Merged openstack/instack-undercloud: Enable Aodh SSL binding in HAproxy (Mitaka only) https://review.openstack.org/348288 | 15:45 |
| slagle | trown: i didnt see that in the readme... | 15:45 |
| slagle | trown: is it just meant to use sudo when calling quickstart.sh? | 15:46 |
| trown | slagle: I still think your patch is better than having it hidden in the readme and inconsistent with the other usage of calling quickstart.sh | 15:46 |
| openstackgerrit | James Slagle proposed openstack/tripleo-quickstart: Use sudo for yum install https://review.openstack.org/348446 | 15:46 |
| slagle | k, wasn't sure | 15:47 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Fix --help output for --config https://review.openstack.org/347252 | 15:47 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: We don't need to set a default for the CephX keys and cluster FSID https://review.openstack.org/348415 | 15:47 |
| *** mcornea has quit IRC | 15:47 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 15:50 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 15:50 |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-specs: Availability monitoring support https://review.openstack.org/347399 | 15:50 |
| slagle | weshay: are the rdo CI failures on https://review.openstack.org/#/c/348446/ valid? | 15:53 |
| slagle | weshay: the links 404 | 15:53 |
| weshay | slagle, it's 404 because that build was aborted due to a new patch set | 15:54 |
| weshay | le> weshay: are the rdo CI failures on https://review.openstack.org/#/c/348446/ valid? | 15:54 |
| weshay | <slagl | 15:54 |
| weshay | whoops | 15:54 |
| weshay | https://ci.centos.org/view/rdo/view/tripleo-gate/job/tripleo-quickstart-gate-master-delorean-quick-basic/239/ | 15:54 |
| weshay | is what I meant to paste | 15:55 |
| tzumainn | rbrady, hi! I have a question about some of the tripleo-common logic when you have a moment | 15:55 |
| weshay | slagle, if you submit a patch before the test job is over, we kill the first test to save resources | 15:55 |
| rbrady | tzumainn: what's up? | 15:56 |
| *** dsariel has quit IRC | 15:57 | |
| EmilienM | can someone look this patch to deploy ironic in wsgi with apache? all green https://review.openstack.org/#/c/309140/ | 15:57 |
| tzumainn | rbrady, hey - I was playing around with the old tripleo-api code, and I noticed a potential issue: when we pass environments into process_multiple_environments_and_files, we don't guarantee that the root-environment goes first | 15:57 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Remove usage of ::nova class in THT https://review.openstack.org/325983 | 15:57 |
| tzumainn | rbrady, the code has changed since, so I dunno if it applies to the way tripleo-common works right now, but I thought I'd let you know | 15:57 |
| slagle | weshay: ah ok. maybe that could be changed to not post a comment in that case | 15:58 |
| rbrady | tzumainn: one of the previous implementations did use an ordered dict to store the envs. It was requested to be removed. | 15:59 |
| tzumainn | rbrady, what was the reasoning? | 15:59 |
| rbrady | tzumainn: that's a good question for jtomasek | 15:59 |
| rbrady | tzumainn: tbh, the current code and direction has changed a lot since tripleo-api was forked for rhci | 16:00 |
| tzumainn | rbrady, fair enough - all I know is that the previous implementation has issues if you don't use an ordered dict and explicitly order the, uh, ordered dict | 16:00 |
| rbrady | tzumainn: and I'm just now finishing up (hopefully) the last part of the deployment and interaction with heat to create the stack | 16:01 |
| *** liverpooler has joined #tripleo | 16:01 | |
| tzumainn | jtomasek, any input into whether you think ^ might still be an issue with the updated direction? | 16:01 |
| rbrady | tzumainn: the current convention for merging environments is to pass them to https://github.com/openstack/python-heatclient/blob/master/heatclient/common/template_utils.py#L200 | 16:02 |
| marios | bnemec: o/ hey man thanks for pointers on https://review.openstack.org/#/c/347749/ - I updated the bug with the context and abandoned. appreciate a review of the now mitaka only at https://review.openstack.org/#/c/347806 if you get some time | 16:02 |
| tzumainn | rbrady, that's what the old tripleo-api does as well : ) | 16:02 |
| tzumainn | rbrady, and what that method does is process each environment in array order of environments | 16:02 |
| * jtomasek looks | 16:02 | |
| tzumainn | so if the first environment *isn't* the root environment, you'll have issues | 16:03 |
| *** tosky has quit IRC | 16:03 | |
| weshay | slagle, true | 16:03 |
| jtomasek | rbrady, tzumainn: the order should be: overcloud-resource-registry-puppet.yaml < {all other environments enabled by user (order here is random} < temporary_environment (holds parameter values specified by user vie GUI) | 16:04 |
| rbrady | tzumainn: at the time the code was forked, we did the merge in tripleo-common | 16:04 |
| openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys and cluster FSID via generate_overcloud_passwords https://review.openstack.org/348412 | 16:04 |
| openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords https://review.openstack.org/348412 | 16:04 |
| jtomasek | rbrady: afaik recently heat got the capability to accept multiple environments | 16:05 |
| tzumainn | rbrady, jtomasek okay, fair enough! I can't comment on how the code is right now, all I'm saying is that it's an issue that I hit with the old tripleo-api, that at a cursory glance *might* hit you guys right now | 16:05 |
| jtomasek | tzumainn: yeah, we need to make sure that the function that merges them (in heatclient) does it in the correct order | 16:06 |
| jtomasek | which means root environment and then everything else | 16:06 |
| jtomasek | rbrady, tzumainn: https://blueprints.launchpad.net/heat/+spec/environment-merging | 16:06 |
| tzumainn | jtomasek, rbrady, okay, right - heat currently does it in 'last-one-wins' order, that might be an issue that needs to be taken into account in tripleo-common? | 16:07 |
| jtomasek | would be good to discuss this with shardy | 16:08 |
| jtomasek | rbrady: we should track it in launchpad as a tripleo-common bug | 16:08 |
| rbrady | jtomasek: let's make sure we're actually running into the bug first. I don't want to create a bug in tripleo -common, based on heat, used by some other code entirely | 16:10 |
| *** anshul has quit IRC | 16:11 | |
| jtomasek | rbrady: ack | 16:11 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Remove another private OSC attribute usage https://review.openstack.org/348460 | 16:11 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 16:14 |
| *** gfidente has quit IRC | 16:19 | |
| *** zoliXXL is now known as zoli|gone | 16:21 | |
| *** mbound has quit IRC | 16:22 | |
| *** zoli|gone is now known as zoli_gone-proxy | 16:22 | |
| marios | bnemec: thank you sir | 16:22 |
| *** mbound has joined #tripleo | 16:23 | |
| bnemec | marios: np. I was just freaking out when I first saw that patch because I thought we had fixed it already. :-) | 16:23 |
| bnemec | Then I realized the bug was against mitaka where we hadn't. | 16:23 |
| bandini | marios, jistr: can I entice you with a fairly simple one https://review.openstack.org/#/c/347310 (and its puppet-tripleo dep) ? ;) | 16:23 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Remove another private OSC attribute usage https://review.openstack.org/348460 | 16:24 |
| marios | bandini: thats a sneaky one .. it has a depends on | 16:24 |
| marios | not very nice of you bandini | 16:24 |
| marios | tsk tsk | 16:24 |
| bandini | ahahah | 16:24 |
| marios | bandini: so i don't hink i should review antyhing at this point. i can promise to look at them if they are still around tomorrow morning if that is ok? | 16:25 |
| bandini | marios: no worries at all, sir ;) | 16:26 |
| marios | bandini: i mean the tht side is simple enough but puppet-tripleo side my eyes just see a bunch of text on the screen i need to spend more than 2 seconds checking the things | 16:26 |
| bandini | marios: ahahah you're eyes are bleeding from tripleo patches, that's when it's time to call it a day ;) | 16:27 |
| bandini | *your | 16:28 |
| *** dsariel has joined #tripleo | 16:28 | |
| marios | bandini: time to go stare at something else for a bit i think indeed. dinner hopefully, maybe some tv. AHHHH such is the life! | 16:28 |
| marios | bai | 16:29 |
| *** rbrady is now known as rbrady-food | 16:29 | |
| bandini | eheh take care! | 16:29 |
| *** jistr|afk is now known as jistr | 16:30 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-quickstart: Use sudo for yum install https://review.openstack.org/348446 | 16:30 |
| *** tesseract- has quit IRC | 16:32 | |
| *** coolsvap is now known as coolsvap_ | 16:34 | |
| *** dculik|gone has quit IRC | 16:38 | |
| *** derekh has quit IRC | 16:39 | |
| *** jpich has quit IRC | 16:42 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Add details for rh1 deployments https://review.openstack.org/347797 | 16:49 |
| pradk | https://review.openstack.org/#/c/344864/ can i get a +a on this please | 16:54 |
| *** numans has joined #tripleo | 16:54 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add a second proxy-logging middleware entry to swift-proxy https://review.openstack.org/341321 | 16:55 |
| *** yamahata has joined #tripleo | 17:00 | |
| *** chem is now known as chem|away | 17:01 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Add specs dir https://review.openstack.org/336109 | 17:01 |
| *** paramite has quit IRC | 17:02 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Spec for adding diskimage_builder module in-chroot https://review.openstack.org/336664 | 17:04 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Spec for adding diskimage_builder module in-chroot https://review.openstack.org/336664 | 17:04 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Make map-packages use dib-python https://review.openstack.org/339065 | 17:07 |
| *** trown is now known as trown|lunch | 17:08 | |
| weshay | thrash, when you have time.. can we push on this https://review.openstack.org/#/c/318087 | 17:09 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Use sudo for yum install https://review.openstack.org/348446 | 17:10 |
| *** akshai has quit IRC | 17:10 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 17:12 |
| *** akshai has joined #tripleo | 17:12 | |
| thrash | weshay: looks like the last comment I had hasn't been addressed... | 17:12 |
| weshay | thrash, k.. :( | 17:13 |
| weshay | thanks | 17:13 |
| *** jcoufal_ has joined #tripleo | 17:14 | |
| *** jcoufal has quit IRC | 17:17 | |
| *** fragatina has joined #tripleo | 17:17 | |
| *** fragatina has quit IRC | 17:19 | |
| *** mgould is now known as mgould|afk | 17:20 | |
| *** rbrady-food is now known as rbrady | 17:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: move hieradata/ceph into ceph-base service https://review.openstack.org/348489 | 17:24 |
| pabelanger | EmilienM: slagle: okay, glean is setting up the hostname properly for tripleo-test-cloud-rh2 now | 17:25 |
| pabelanger | new images are coming online | 17:25 |
| pabelanger | sorry, new nodes are coming online | 17:25 |
| slagle | cool, we can put up a revert of the patch to see if CI passes | 17:26 |
| pabelanger | yup, doing that now | 17:26 |
| *** fragatina has joined #tripleo | 17:26 | |
| *** fragatina has quit IRC | 17:27 | |
| slagle | EmilienM: can you point me at the code in facter that calculates fqdn? | 17:29 |
| slagle | i still think it's doing the wrong thing | 17:29 |
| slagle | not that we'd ever get that changed | 17:29 |
| EmilienM | whitch version of Facter | 17:29 |
| EmilienM | the one in ruby or in C++? | 17:29 |
| slagle | which one do we use? :) | 17:29 |
| EmilienM | I think the one in ruby | 17:30 |
| EmilienM | let me check | 17:30 |
| EmilienM | facter-2.4.4-3.el7.x86_64 | 17:30 |
| openstackgerrit | Paul Belanger proposed openstack/instack-undercloud: Revert "Explicitly set $FACTER_fqdn" https://review.openstack.org/348493 | 17:31 |
| EmilienM | slagle: a sec | 17:31 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 17:32 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 17:32 |
| slagle | EmilienM: here is what I was looking for: /usr/share/ruby/vendor_ruby/facter/domain.rb | 17:32 |
| EmilienM | so fqdn uses hostname + domain | 17:33 |
| EmilienM | hostname is computed like this https://github.com/puppetlabs/facter/blob/2.4.4/lib/facter/hostname.rb#L17-L24 | 17:33 |
| EmilienM | and for domain it's a bit more complex https://github.com/puppetlabs/facter/blob/2.4.4/lib/facter/domain.rb | 17:33 |
| EmilienM | it checks /etc/resolv.conf | 17:33 |
| slagle | i dont get why it has all this logic | 17:34 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add environment file to enable DVR https://review.openstack.org/332147 | 17:34 |
| EmilienM | https://github.com/puppetlabs/facter/blob/2.4.4/lib/facter/domain.rb#L44-L66 | 17:34 |
| slagle | hostname -f | 17:34 |
| EmilienM | slagle: me neither honestly. | 17:34 |
| slagle | the only way we could protect against this via a validation would be to reimplement all this logic in a check | 17:34 |
| slagle | which would be silly | 17:35 |
| slagle | well i should say if we want to actually set the hostname puppet expects | 17:35 |
| slagle | we could always just do a compare and bail out early i guess | 17:35 |
| pabelanger | do you need a domain? | 17:35 |
| slagle | we've added a check for one, but that was just b/c of puppet iirc | 17:36 |
| pabelanger | okay, with the latest changes to dhclient.conf, we're actively stripping out the domain-name from DHCP in the clouds. So keep that in mind | 17:37 |
| *** akshai has quit IRC | 17:39 | |
| *** fragatina has joined #tripleo | 17:39 | |
| *** fragatina has quit IRC | 17:40 | |
| *** fragatina has joined #tripleo | 17:41 | |
| bnemec | Yeah, we need a domain. I don't recall what broke, but something definitely does if you only set a short hostname. | 17:41 |
| bnemec | slagle: Long-term, my preference is to start setting hostname via https://github.com/openstack/instack-undercloud/blob/master/undercloud.conf.sample#L12 | 17:42 |
| bnemec | That way we don't have to guess. | 17:42 |
| slagle | bnemec: yea, we tried that fix first, but it didnt fully work | 17:42 |
| slagle | b/c puppet reads domain from resolv.conf | 17:42 |
| slagle | so we'd have to write something there | 17:42 |
| bnemec | fml | 17:42 |
| slagle | if it wasnt there already | 17:42 |
| slagle | bnemec: yes, exactly :) | 17:43 |
| bnemec | We could do that though. | 17:43 |
| bnemec | The goal of undercloud_hostname was to set _everything_ related to the hostname, so if facter needs it somewhere else then we should set it there too. | 17:44 |
| bnemec | Although I don't understand how this is working today. | 17:44 |
| bnemec | My resolv.conf doesn't match the actual hostname either: http://paste.openstack.org/show/543415/ | 17:45 |
| EmilienM | slagle, pabelanger: https://review.openstack.org/#/c/347964/ is ready for review, when you have time | 17:46 |
| *** fragatina has quit IRC | 17:46 | |
| *** akshai has joined #tripleo | 17:47 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix ceilometer auth url for compute https://review.openstack.org/344864 | 17:49 |
| *** jbadiapa has quit IRC | 17:56 | |
| *** akshai has quit IRC | 18:00 | |
| openstackgerrit | Merged openstack/tripleo-quickstart: oooq support check, exit on anything other than CentOS and RHEL https://review.openstack.org/348062 | 18:01 |
| *** akshai has joined #tripleo | 18:01 | |
| *** wgd3 has quit IRC | 18:05 | |
| *** trown|lunch is now known as trown | 18:07 | |
| jcoufal_ | hey all, can we get some eyes on https://review.openstack.org/#/c/339048 just when happen to have some spare few minutes, thanks | 18:08 |
| *** liverpooler has quit IRC | 18:08 | |
| openstackgerrit | Merged openstack/tripleo-quickstart: Fix roles based CI jobs https://review.openstack.org/348309 | 18:09 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Make the number of tempest workers configurable https://review.openstack.org/348300 | 18:09 |
| jcoufal_ | and one more for a bit of attention: https://review.openstack.org/#/c/341146/ | 18:12 |
| jrist | honza: so the stuff you did for https://review.openstack.org/#/c/344932/11 is based on discussion with apevec? | 18:14 |
| openstackgerrit | Tim Rozet proposed openstack/python-tripleoclient: Removes setting NeutronL3HA and NeutronAllowL3AgentFailover https://review.openstack.org/345663 | 18:14 |
| honza | jrist: yes, general discussion in #rdo | 18:14 |
| jrist | honza: nice. | 18:14 |
| openstackgerrit | Ben Nemec proposed openstack/diskimage-builder: Generalize logic for skipping final image generation https://review.openstack.org/334042 | 18:15 |
| jrist | jcoufal_: while we're at it, lets get eyes on this https://review.openstack.org/#/c/344932/11 :) | 18:15 |
| jrist | hehe | 18:15 |
| openstackgerrit | Tim Rozet proposed openstack/python-tripleoclient: Removes setting NeutronL3HA and NeutronAllowL3AgentFailover https://review.openstack.org/345663 | 18:15 |
| *** ayoung has quit IRC | 18:24 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 18:28 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart: Improve debug output for failed overcloud deployments https://review.openstack.org/347828 | 18:28 |
| *** electrofelix has quit IRC | 18:33 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Properly direct MTU parameters and allow service defaults https://review.openstack.org/333333 | 18:34 |
| *** tbonds has quit IRC | 18:41 | |
| *** tbonds has joined #tripleo | 18:43 | |
| beagles | since we seem to be polling for review attention: how about https://review.openstack.org/#/c/333333/ ? ;) | 18:43 |
| *** fzdarsky is now known as fzdarsky|afk | 18:45 | |
| EmilienM | beagles: let's wait for CI first | 18:45 |
| beagles | EmilienM, ack! | 18:45 |
| beagles | the mtu stuff in neutron is tangly and while mitaka makes it less so, it only works if we "get out of its way" | 18:45 |
| pradk | beagles, 333333 nicely done | 18:46 |
| beagles | pradk, :) | 18:47 |
| * beagles is sketching plans for a CI shrine | 18:48 | |
| EmilienM | beagles: you should play loto tonight | 18:50 |
| EmilienM | slagle: can you please approve https://review.openstack.org/#/c/346551 ? | 18:50 |
| beagles | EmilienM, lol.. good idea | 18:50 |
| *** dsariel has quit IRC | 18:51 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 18:52 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 18:55 |
| *** anshul has joined #tripleo | 19:00 | |
| *** anshul has quit IRC | 19:00 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Set secure_proxy_ssl_header 'HTTP_X_FORWARDED_PROTO' for nova-api https://review.openstack.org/347806 | 19:00 |
| *** akshai has quit IRC | 19:02 | |
| *** fzdarsky|afk has quit IRC | 19:11 | |
| *** numans has quit IRC | 19:12 | |
| *** jcoufal_ has quit IRC | 19:13 | |
| *** jcoufal has joined #tripleo | 19:18 | |
| *** Slower_ is now known as Slower | 19:24 | |
| *** fultonj has quit IRC | 19:25 | |
| *** fzdarsky|afk has joined #tripleo | 19:25 | |
| *** dsariel has joined #tripleo | 19:31 | |
| *** hjensas has quit IRC | 19:33 | |
| *** hjensas has joined #tripleo | 19:35 | |
| rhallisey | anyone know how I can run a puppet command on compute/controller host and load the hieradata? | 19:36 |
| rhallisey | for example, if I only wanted to run puppet on the nova-compute class on a compute node | 19:40 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 19:46 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 19:46 |
| slagle | trown: what is the significance of delegate_to: undercloud in the "Configure Ironic pxe_ssh driver | 19:48 |
| slagle | task | 19:48 |
| slagle | isn't that already taken care of by applying this tasks to the "undercloud" host? | 19:48 |
| *** jcoufal has quit IRC | 19:49 | |
| *** jcoufal has joined #tripleo | 19:50 | |
| trown | slagle: hmm... that task was moved from the libvirt/setup role that runs against localhost, and we just missed that | 19:50 |
| *** ccamacho has joined #tripleo | 19:51 | |
| trown | slagle: you are right though it does nothing in the current location of that task | 19:51 |
| trown | hadnt noticed that | 19:51 |
| slagle | ok, it was actually failing for me, and removing that line makes it work | 19:52 |
| slagle | granted, i've gone off the rails a bit with what I'm doing | 19:52 |
| trown | ya here is where we moved it https://github.com/openstack/tripleo-quickstart/commit/7e588cc7f5a4e62d9dc22c52e3f45e9674aecc84 | 19:52 |
| trown | which was almost two months ago, so I am pretty confident it is not "broken" as is, but a patch to remove it would be good anyways | 19:52 |
| trown | now I am curious... what are you doing? :) | 19:53 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use oooq to install the undercloud https://review.openstack.org/348530 | 19:54 |
| trown | ah | 19:54 |
| slagle | seems to work i guess | 19:56 |
| *** radek_ has joined #tripleo | 19:56 | |
| trown | neat | 19:56 |
| trown | weshay: checkout https://review.openstack.org/348530 | 19:57 |
| slagle | trown: i actually don't want that task to run at all. can i switch it to a different tag? undercloud-virt-setup maybe? | 19:58 |
| slagle | and then add that tag to the defaults | 19:58 |
| trown | slagle: makes sense | 19:58 |
| trown | slagle: that is kind of why I had in the libvirt role orginally even though it is an undercloud task | 20:00 |
| trown | it is very specific to the unprivledged virt setup | 20:00 |
| slagle | right | 20:00 |
| weshay | trown, oh nice | 20:00 |
| weshay | I need to retry it now that i have the right ipxe-rom on my test box | 20:01 |
| *** jprovazn has quit IRC | 20:01 | |
| weshay | trown, I'll build on that.. I was also hoping to update the tripleo doc for tripleo.sh re: instack-virt-setup | 20:01 |
| trown | weshay: ah sweet... I did not get to that... and dont think I will before the end of week | 20:02 |
| weshay | I'm not 100% sure where that doc lives.. but can dig around | 20:02 |
| trown | weshay: are you talking about https://github.com/openstack/tripleo-docs | 20:03 |
| weshay | ya | 20:05 |
| weshay | thanks | 20:05 |
| rhallisey | EmilienM, found it | 20:05 |
| EmilienM | rhallisey: how? | 20:05 |
| rhallisey | /var/lib/heat-config/heat-config-puppet/<uuid>.pp | 20:06 |
| rhallisey | puppet apply of that | 20:06 |
| EmilienM | rhallisey: I think it's what I told you in pm :P | 20:07 |
| EmilienM | or almost :P | 20:07 |
| EmilienM | sorry I was not precise enough | 20:07 |
| EmilienM | I'm glad you found it, I'll note it | 20:07 |
| rhallisey | ya you were super close | 20:08 |
| rhallisey | pointed me in the right direction though thank! | 20:08 |
| rhallisey | thanks! | 20:08 |
| *** mbound has quit IRC | 20:08 | |
| EmilienM | cool | 20:08 |
| openstackgerrit | Matthew Thode proposed openstack/diskimage-builder: Optionally remove portage files https://review.openstack.org/348536 | 20:13 |
| openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction https://review.openstack.org/348537 | 20:14 |
| openstackgerrit | James Slagle proposed openstack/tripleo-docs: Remove enabling epel https://review.openstack.org/348538 | 20:16 |
| *** fultonj has joined #tripleo | 20:16 | |
| jrist | +1 | 20:20 |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo: Adds OpenDaylight https://review.openstack.org/335173 | 20:25 |
| *** greghaynes has quit IRC | 20:25 | |
| *** fragatina has joined #tripleo | 20:27 | |
| *** coolsvap_ has quit IRC | 20:31 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Implement basic undercloud sanity checks https://review.openstack.org/346551 | 20:32 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Add facter output to host_info.txt https://review.openstack.org/347460 | 20:33 |
| openstackgerrit | James Slagle proposed openstack/tripleo-quickstart: Use undercloud-virt-setup tag for Ironic pxe_ssh configuration https://review.openstack.org/348545 | 20:34 |
| openstackgerrit | Merged openstack/tripleo-quickstart: add debug to the overcloud deployment https://review.openstack.org/346889 | 20:36 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Improve debug output for failed overcloud deployments https://review.openstack.org/347828 | 20:36 |
| *** greghaynes has joined #tripleo | 20:37 | |
| *** ccamacho has quit IRC | 20:39 | |
| *** ccamacho|afk has quit IRC | 20:39 | |
| weshay | trown++ | 20:40 |
| pabelanger | weshay: trown: once tripleo-quickstart is working, I'd like to setup an experimental job and see if we can run it outside of the devstack hook. | 20:43 |
| pabelanger | weshay: trown: that effort will lay the ground work to stop using devstack to clone the git repos locally, and shave some time off your builds | 20:44 |
| *** Ryjedo has quit IRC | 20:45 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Break out bootstrapping of subnodes setup https://review.openstack.org/347008 | 20:47 |
| *** radek_ has quit IRC | 20:47 | |
| weshay | pabelanger, oh ya.. nice | 20:48 |
| *** Ryjedo has joined #tripleo | 20:48 | |
| pabelanger | weshay: infact, we can add the job to tripleo-quickstart first, since you only have noop jobs. | 20:49 |
| pabelanger | weshay: maybe I'll do that tomorrow | 20:49 |
| *** fzdarsky|afk has quit IRC | 20:54 | |
| *** trown is now known as trown|outtypewww | 20:57 | |
| openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Add tox -elinters https://review.openstack.org/348554 | 21:03 |
| pabelanger | weshay: Okay, going to start with ^. As a way to better understand triple-quickstart | 21:03 |
| *** rhallisey has quit IRC | 21:03 | |
| pabelanger | weshay: I'll also add some non-voting jobs to run tox -elinters in the gate | 21:03 |
| *** mbound has joined #tripleo | 21:09 | |
| openstackgerrit | Alan Pevec proposed openstack/python-tripleoclient: Remove another private OSC attribute usage https://review.openstack.org/348460 | 21:11 |
| bnemec | pabelanger: I thought the linters environment got nixed? | 21:11 |
| pabelanger | bnemec: nixed where? | 21:12 |
| bnemec | pabelanger: It got proposed as the new target for linting jobs, but because of the PTI those changes were undone and everything got moved back to pep8. | 21:13 |
| bnemec | I'm only familiar with this because I had to push several new patch sets related to it when I was adding linting to t-h-t. | 21:13 |
| pabelanger | bnemec: Oh, maybe that was for TC projects. But, I can confirm with -infra in the morning | 21:13 |
| *** mbound has quit IRC | 21:14 | |
| pabelanger | but ya, it might be the pep8 target now | 21:14 |
| bnemec | pabelanger: http://lists.openstack.org/pipermail/openstack-dev/2016-February/086268.html | 21:15 |
| pabelanger | bnemec: ya, I have a question out to AJaeger about it | 21:16 |
| pabelanger | since we still have some jobs using -linters | 21:16 |
| pabelanger | but ya, and easy fix | 21:16 |
| bnemec | I'm also giving myself a big pat on the back for leaving a link to that in my commit message so I could find it easily. :-) | 21:16 |
| openstackgerrit | Merged openstack/tripleo-docs: Add instructions on how to provision a custom Cinder backend https://review.openstack.org/323833 | 21:21 |
| *** dsariel has quit IRC | 21:23 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add a second proxy-logging middleware entry to swift-proxy https://review.openstack.org/341322 | 21:28 |
| dsneddon | EmilienM, After you rebased this change, it fails and I haven't figured out why. Unfortunately, the rebase removed/changed quite a few lines, and I haven't been able to figure out what's getting jammed yet. | 21:33 |
| dsneddon | https://review.openstack.org/#/c/289279/ | 21:34 |
| dsneddon | EmilienM, I'm going to try to get it back into working shape so you can see the CI logs. | 21:34 |
| *** seanatcisco has quit IRC | 21:36 | |
| *** seanatcisco has joined #tripleo | 21:36 | |
| *** mhenkel has quit IRC | 21:38 | |
| *** tbonds has quit IRC | 21:46 | |
| *** bfournie has quit IRC | 21:57 | |
| EmilienM | dsneddon: you should rebase/recheck, CI was flappy lately | 21:59 |
| dsneddon | EmilienM, Will try that, too. | 22:03 |
| *** fultonj has quit IRC | 22:09 | |
| *** morazi has quit IRC | 22:11 | |
| *** TSCHAK_ has quit IRC | 22:17 | |
| *** Ryjedo has quit IRC | 22:17 | |
| *** Ryjedo has joined #tripleo | 22:21 | |
| *** dbecker has joined #tripleo | 22:25 | |
| *** dtrainor has quit IRC | 22:31 | |
| *** dtrainor has joined #tripleo | 22:39 | |
| *** dtrainor has quit IRC | 22:42 | |
| *** dtrainor has joined #tripleo | 22:43 | |
| *** bfournie has joined #tripleo | 22:46 | |
| *** yamahata has quit IRC | 22:47 | |
| *** seanatcisco has quit IRC | 22:50 | |
| *** yamahata has joined #tripleo | 23:00 | |
| *** pradk has quit IRC | 23:04 | |
| *** panda|bootcamp is now known as panda|Zz | 23:07 | |
| *** dbecker has quit IRC | 23:19 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Updates and fixes to NIC config template routes https://review.openstack.org/318389 | 23:27 |
| *** saneax_AFK is now known as saneax | 23:34 | |
| *** egafford has quit IRC | 23:43 | |
| *** tbonds has joined #tripleo | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!