| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #tripleo | 00:23 | |
| *** shivrao has joined #tripleo | 00:26 | |
| *** tiswanso has joined #tripleo | 00:29 | |
| *** shivrao_ has joined #tripleo | 00:29 | |
| *** shivrao has quit IRC | 00:31 | |
| *** shivrao_ is now known as shivrao | 00:31 | |
| *** mbound has joined #tripleo | 00:34 | |
| *** mbound has quit IRC | 00:39 | |
| *** tiswanso has quit IRC | 00:48 | |
| *** chlong has joined #tripleo | 00:50 | |
| *** mestery has quit IRC | 00:57 | |
| *** shivrao has quit IRC | 01:18 | |
| *** tiswanso has joined #tripleo | 01:44 | |
| openstackgerrit | Graeme Gillies proposed openstack/tripleo-docs: Add documentation on how to backup and restore the Undercloud https://review.openstack.org/233468 | 01:46 |
|---|---|---|
| *** tiswanso has quit IRC | 01:46 | |
| *** tiswanso has joined #tripleo | 01:46 | |
| *** tiswanso_ has joined #tripleo | 01:48 | |
| *** tiswanso has quit IRC | 01:51 | |
| openstackgerrit | Graeme Gillies proposed openstack/os-cloud-config: Add wol ironic driver support https://review.openstack.org/213956 | 02:10 |
| *** chlong has quit IRC | 02:18 | |
| *** cody-somerville has joined #tripleo | 02:26 | |
| *** cody-somerville has joined #tripleo | 02:26 | |
| *** penick has joined #tripleo | 02:27 | |
| *** david-ly_ has joined #tripleo | 02:31 | |
| *** david-lyle has quit IRC | 02:33 | |
| *** phschwartz has quit IRC | 02:33 | |
| *** cody-somerville has quit IRC | 02:43 | |
| *** penick has quit IRC | 02:44 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Remove extra install of release pkgs in fedora-minimal https://review.openstack.org/227131 | 02:45 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Reset yum/dnf cache to correct location https://review.openstack.org/232863 | 02:45 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Add basic Fedora 22 support to yum-minimal https://review.openstack.org/227125 | 02:45 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Cleanup the fedora-minimal yum/dnf path https://review.openstack.org/227127 | 02:45 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Update default fedora-minimal to f22 https://review.openstack.org/227126 | 02:45 |
| *** jobewan has quit IRC | 02:49 | |
| *** untriaged-bot has joined #tripleo | 03:00 | |
| untriaged-bot | Untriaged bugs so far: | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504137 | 03:00 |
| openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1504632 | 03:00 |
| openstack | Launchpad bug 1504632 in diskimage-builder "elements README documentation environment variables should be clearer" [Undecided,In progress] - Assigned to Augustina Ragwitz (missaugustina) | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504179 | 03:00 |
| openstack | Launchpad bug 1504179 in tripleo "instack-virt-setup can't use dnsmasq lease file anymore" [Undecided,In progress] - Assigned to Thomas Herve (therve) | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504263 | 03:00 |
| openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 03:00 |
| openstack | Launchpad bug 1504263 in tripleo "os-net-config mapping by MAC fails if trying to map in alternate order" [Undecided,Triaged] | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504363 | 03:00 |
| openstack | Launchpad bug 1504363 in tripleo "Updates to hiera configuration not propogating/making a puppet run" [Undecided,New] | 03:00 |
| *** untriaged-bot has quit IRC | 03:00 | |
| *** shivrao has joined #tripleo | 03:05 | |
| *** julim has joined #tripleo | 03:18 | |
| *** chlong has joined #tripleo | 03:31 | |
| *** david-lyle has joined #tripleo | 03:37 | |
| *** lazy_prince has joined #tripleo | 03:39 | |
| *** david-ly_ has quit IRC | 03:40 | |
| *** lazy_prince has quit IRC | 03:50 | |
| *** tzumainn has quit IRC | 04:03 | |
| *** julim has quit IRC | 04:16 | |
| *** shivrao has quit IRC | 04:25 | |
| *** chlong has quit IRC | 04:45 | |
| *** ukalifon has quit IRC | 04:52 | |
| *** chlong has joined #tripleo | 04:57 | |
| *** masco has joined #tripleo | 05:04 | |
| *** bandini has joined #tripleo | 05:12 | |
| *** shivrao has joined #tripleo | 05:18 | |
| *** lazy_prince has joined #tripleo | 05:40 | |
| *** jtomasek has joined #tripleo | 05:50 | |
| *** matbu has joined #tripleo | 05:54 | |
| *** matbu has quit IRC | 06:01 | |
| *** matbu has joined #tripleo | 06:01 | |
| *** jaosorior has joined #tripleo | 06:03 | |
| *** BharatK has joined #tripleo | 06:10 | |
| *** rasca has joined #tripleo | 06:16 | |
| *** rwsu has joined #tripleo | 06:23 | |
| *** bvandenh has joined #tripleo | 06:25 | |
| *** ukalifon has joined #tripleo | 06:27 | |
| *** aufi has joined #tripleo | 06:36 | |
| *** gfidente has joined #tripleo | 06:36 | |
| *** gfidente has quit IRC | 06:36 | |
| *** gfidente has joined #tripleo | 06:36 | |
| *** jcoufal has joined #tripleo | 06:43 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Ensure access to RabbitMQ is password protected https://review.openstack.org/210402 | 06:57 |
| jaosorior | what's the way to set the CloudName parameter from the templates? I've tried to set it from an environment file, but it seems it didn't go through. I suspect it's because it's one of the parameters coming from the tripleoclient; so it might not be passing through | 07:10 |
| spredzy | parameters: | 07:18 |
| spredzy | CloudName: mycloud | 07:18 |
| spredzy | with a -e /path/to/mylfie option to tripleoclient | 07:18 |
| spredzy | that should work jaosorior ^ | 07:18 |
| jaosorior | spredzy: That's what I had done | 07:18 |
| spredzy | jaosorior, ack. I am rebuilding my platform atm, I'll try when I have it backup | 07:20 |
| jaosorior | spredzy: Thanks, I would appreciate that | 07:20 |
| jaosorior | but yeah, If I understood correctly, that way it should set the fqdn for the overcloud nodes, right? | 07:20 |
| *** jprovazn has joined #tripleo | 07:20 | |
| *** shivrao has quit IRC | 07:25 | |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Allow one to specify horizon ALLOWED_HOSTS https://review.openstack.org/221255 | 07:27 |
| *** shardy has joined #tripleo | 07:29 | |
| *** pelix has joined #tripleo | 07:30 | |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Add more components virtual ip mapping into controller. https://review.openstack.org/231361 | 07:31 |
| *** hien_ has joined #tripleo | 07:33 | |
| *** ifarkas has joined #tripleo | 07:35 | |
| hien_ | hi | 07:37 |
| hien_ | after running "openstack undercloud install" command on the instack machine | 07:38 |
| hien_ | i got this error | 07:38 |
| hien_ | Error: Could not find class ::ironic::inspector for instack on node instack | 07:38 |
| hien_ | Error: Could not find class ::ironic::inspector for instack on node instack | 07:38 |
| hien_ | any ideas? | 07:40 |
| *** hien_ has quit IRC | 07:41 | |
| *** hien_ has joined #tripleo | 07:41 | |
| *** hien_ has quit IRC | 07:42 | |
| shardy | Morning all | 07:45 |
| shardy | https://review.openstack.org/#/c/227226/2 is all green and needs a final +A if anyone has a moment | 07:45 |
| jaosorior | shardy: Got a moment to check this one out https://review.openstack.org/#/c/229768/ ? Only missing +A, would be really useful for debugging the client | 07:46 |
| shardy | jaosorior: nice, thanks! :) | 07:49 |
| spredzy | Did anyone had this issue : Parse error: Invalid string: control characters from U+0000 through U+001F must be escaped at line 32, column 30 (while running instack-virt-setup) ? | 07:50 |
| jaosorior | shardy: Thanks Mr. :) | 07:50 |
| openstackgerrit | Merged openstack/python-tripleoclient: Add debug environment to tox https://review.openstack.org/229768 | 07:53 |
| marios | spredzy: i saw this when i had mixed up/downstream repos (there is also a BZ for it, is because of jq https://bugzilla.redhat.com/show_bug.cgi?id=1265403#c3 | 07:54 |
| openstack | bugzilla.redhat.com bug 1265403 in instack-undercloud "instack-undercloud fails to setup seed vm, parse error while creating ssh key" [High,Closed: notabug] - Assigned to jslagle | 07:54 |
| *** mkovacik has joined #tripleo | 07:55 | |
| spredzy | marios, thanks for the link. Unfortunately if I didn't enable EPEL it complained also. Will look into it | 07:55 |
| spredzy | EPEL repository is required to install python-pip for CentOS. | 07:56 |
| *** dtantsur|afk is now known as dtantsur | 07:57 | |
| openstackgerrit | Merged openstack/os-net-config: Add support for Linux Bonding to os-net-config ifcfg https://review.openstack.org/218134 | 07:57 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Ensure httpd is not enabled by puppet on system boot https://review.openstack.org/206463 | 08:03 |
| *** StevenK has quit IRC | 08:03 | |
| *** mbound has joined #tripleo | 08:03 | |
| marios | spredzy: np - just looked familiar as was relatively recently i hit that :) | 08:04 |
| *** pblaho has joined #tripleo | 08:04 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Move RHEL (un)registration to NodeExtraConfig https://review.openstack.org/227226 | 08:05 |
| *** StevenK has joined #tripleo | 08:07 | |
| jaosorior | spredzy: I think it's not getting the parameter I gave it because it's getting the default parameters that are specified in the tripleoclient. CloudName is set as overcloud and that's what it's getting regardless of what I give it in a separate environment file | 08:08 |
| *** Guest2082 is now known as d0ugal | 08:09 | |
| *** d0ugal has joined #tripleo | 08:09 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Allow enabling debug mode for config management (Puppet) https://review.openstack.org/202465 | 08:10 |
| spredzy | jaosorior, if you try directly with heat client does it take the paramter ? | 08:10 |
| jaosorior | spredzy: I'll try that out. Been trying only with tripleoclient | 08:11 |
| *** lucasagomes has joined #tripleo | 08:12 | |
| *** derekh has joined #tripleo | 08:12 | |
| shardy | jaosorior: are you using parameters or parameter_defaults in your environment file? | 08:12 |
| shardy | jaosorior: "parameters" should overwrite any hard-coded tripleoclient value, otherwise we still have bugs | 08:13 |
| shardy | (I previously raised a downstream bug about rdomanager-oscplugin due to this very issue) | 08:13 |
| spredzy | shardy, marios gfidente Can I have reviews on those two please ? https://review.openstack.org/#/c/221255/ https://review.openstack.org/#/c/231361/ | 08:14 |
| shardy | it's another reason why we really want all those hard-coded defaults to die, things will work much more intuitively if we maintain them in the heat templates, or maybe an environment file in tripleo-heat-templates | 08:14 |
| cmyster | +1 | 08:15 |
| cmyster | shardy: would it be out of the ordinary here to open a bug for hard coded defaults ? | 08:15 |
| *** yog_ has joined #tripleo | 08:15 | |
| shardy | cmyster: I already raised https://bugzilla.redhat.com/show_bug.cgi?id=1245737 downstream | 08:17 |
| openstack | bugzilla.redhat.com bug 1245737 in python-rdomanager-oscplugin "hard-coded parameters can't be overridden" [Urgent,New] - Assigned to jason.dobies | 08:17 |
| shardy | if tripleoclient still does it, we need an upstream bug | 08:17 |
| shardy | cmyster: FWIW I have a whole series up for t-h-t which fixes the defaults as a precursor to ripping them out of the client | 08:18 |
| shardy | unfortunately that has met more resistance than I expected, but when that's resolved hopefully we can remove the client-side defaults for good | 08:18 |
| *** athomas has joined #tripleo | 08:18 | |
| cmyster | shardy: rgr | 08:19 |
| shardy | d0ugal: Hey, any idea what happened to "openstack overcloud validate" ? | 08:23 |
| shardy | AFAICS it's not in the tripleoclient code, but it's mentioned in the docs? | 08:24 |
| d0ugal | shardy: hmm, odd. | 08:24 |
| shardy | Maybe I'm looking in the wrong place, or just need more coffee ;) | 08:24 |
| d0ugal | shardy: Checking. | 08:25 |
| *** hewbrocca has joined #tripleo | 08:25 | |
| *** shardy_ has joined #tripleo | 08:27 | |
| *** shardy has quit IRC | 08:29 | |
| *** mcornea has joined #tripleo | 08:29 | |
| d0ugal | shardy_: https://github.com/openstack/python-tripleoclient/commit/a8c069c6e794f0846bafa348defe48500701da56 | 08:30 |
| d0ugal | shardy_: So I remember it being changed so that it creates the tempest config only, I didn't notice that it got merged into the deploy command | 08:31 |
| d0ugal | so on every deploy it creates the tempest config it seems | 08:31 |
| *** yog_ has quit IRC | 08:31 | |
| *** shardy_ has quit IRC | 08:33 | |
| *** shardy has joined #tripleo | 08:33 | |
| *** chlong has quit IRC | 08:35 | |
| *** yog_ has joined #tripleo | 08:36 | |
| *** mbound has quit IRC | 08:46 | |
| *** jistr has joined #tripleo | 08:46 | |
| jaosorior | shardy: I'm using parameters not parameter_defaults. But it's from downstream | 08:56 |
| *** regebro has joined #tripleo | 08:58 | |
| *** BharatK has quit IRC | 08:58 | |
| *** untriaged-bot has joined #tripleo | 09:00 | |
| untriaged-bot | Untriaged bugs so far: | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504137 | 09:00 |
| openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1504632 | 09:00 |
| openstack | Launchpad bug 1504632 in diskimage-builder "elements README documentation environment variables should be clearer" [Undecided,In progress] - Assigned to Augustina Ragwitz (missaugustina) | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504179 | 09:00 |
| openstack | Launchpad bug 1504179 in tripleo "instack-virt-setup can't use dnsmasq lease file anymore" [Undecided,In progress] - Assigned to Thomas Herve (therve) | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 09:00 |
| openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504263 | 09:00 |
| openstack | Launchpad bug 1504263 in tripleo "os-net-config mapping by MAC fails if trying to map in alternate order" [Undecided,Triaged] | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1505129 | 09:00 |
| openstack | Launchpad bug 1505129 in tripleo "Docs describe non-existent validation options" [Undecided,New] | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504363 | 09:00 |
| openstack | Launchpad bug 1504363 in tripleo "Updates to hiera configuration not propogating/making a puppet run" [Undecided,Confirmed] | 09:00 |
| *** untriaged-bot has quit IRC | 09:00 | |
| *** mbound has joined #tripleo | 09:05 | |
| *** lucasagomes has quit IRC | 09:12 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-docs: Add note that qemu libvirt-type is needed for virt environment https://review.openstack.org/233535 | 09:12 |
| *** BharatK has joined #tripleo | 09:13 | |
| *** lucasagomes has joined #tripleo | 09:24 | |
| *** dtantsur is now known as dtantsur|brb | 09:24 | |
| *** olap has joined #tripleo | 09:27 | |
| shardy | derekh: Hey, I was wondering where we are wrt moving from the docker to mock based delorean for tripleo CI/.sh? | 09:29 |
| shardy | derekh: I'm looking to do a docs patch showing how the tripleo.sh works for building on-review changes, but I assume some of the initial steps will change when we move from docker | 09:30 |
| *** tosky has joined #tripleo | 09:31 | |
| derekh | shardy: we're close. I'm not quite sure why the ci switchover didn't work https://review.openstack.org/#/c/229425/ | 09:39 |
| derekh | shardy: I'm yting to spin up and CI env manually to reproduce | 09:40 |
| shardy | ramishra. d0ugal: Turns out there's already a docs patch up: https://review.openstack.org/#/c/221664/ | 09:40 |
| derekh | shardy: but have hit a timeout problem on our CI cloud and have to figure that out first | 09:40 |
| d0ugal | oh, neat | 09:40 |
| * d0ugal reads | 09:40 | |
| shardy | derekh: Ok, cool, I'll hold off documenting anything yet then, thanks! | 09:42 |
| d0ugal | shardy: Looks pretty good other than trown|outttypeww's comments | 09:42 |
| shardy | d0ugal: Cool, I've assigned the bug to ifarkas so we can hopefully get that updated/landed | 09:42 |
| derekh | shardy: the 2 patches up for tripleo.sh should be extremely close (if not complete) | 09:43 |
| shardy | derekh: Yeah, it was just stuff like having to mess with the docker group, which I assume won't be needed when we move to mock | 09:44 |
| derekh | shardy: the switch from docker to mock is done in PS11 https://review.openstack.org/#/c/228156/11 | 09:45 |
| derekh | shardy: the docker stuff never merged so I've switched it to mock | 09:46 |
| shardy | derekh: Oh, thanks, I hadn't spotted that and was still using the old docker version | 09:48 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-image-elements: Grab passthrough horizon.allowed_hosts https://review.openstack.org/233549 | 09:49 |
| shardy | I've been running out of memory on the undercloud building stuff there, so was going to refresh some steps to run builds on the host and export them to the undercloud as a local repo | 09:49 |
| derekh | shardy: sounds good | 09:49 |
| *** chlong has joined #tripleo | 09:49 | |
| *** regebro has quit IRC | 09:51 | |
| *** regebro has joined #tripleo | 09:51 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-image-elements: Grab passthrough horizon.allowed_hosts https://review.openstack.org/233549 | 09:54 |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Allow one to specify horizon ALLOWED_HOSTS https://review.openstack.org/221255 | 09:56 |
| *** marzif has joined #tripleo | 09:58 | |
| *** hien__ has joined #tripleo | 10:10 | |
| *** hien__ has quit IRC | 10:15 | |
| *** BharatK has quit IRC | 10:17 | |
| *** BharatK has joined #tripleo | 10:30 | |
| shardy | derekh: Hmm, one disadvantage of the mock based approach is it appears to use a fixed --installroot | 10:32 |
| shardy | I'm running short on /var space and I can't see an easy way to redirect the (large) mock output to e.g $HOME | 10:32 |
| * shardy starts grepping delorean tree | 10:33 | |
| derekh | shardy: not sure either off the top of my head, but ya there are probably places where we can make improvements. | 10:34 |
| jaosorior | shardy: What do you want to do? | 10:35 |
| shardy | jaosorior: I want mock, driven via delorean, to use a different build root instead of /var/lib/mock | 10:37 |
| shardy | obviously I can symlink it, but I was looking for a way to easily influence it via a config option | 10:38 |
| *** dtantsur|brb is now known as dtantsur | 10:39 | |
| openstackgerrit | Jan Provaznik proposed openstack/tripleo-heat-templates: Add a resource for node reboot https://review.openstack.org/233573 | 10:40 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Set shared secrets, keys and passwords as hidden https://review.openstack.org/233578 | 11:00 |
| *** tosky has quit IRC | 11:05 | |
| *** masterbound has joined #tripleo | 11:05 | |
| *** hien_ has joined #tripleo | 11:05 | |
| *** mbound has quit IRC | 11:08 | |
| *** lucasagomes is now known as lucas-hungry | 11:11 | |
| *** bandini has quit IRC | 11:14 | |
| *** bvandenh has quit IRC | 11:18 | |
| *** regebro has quit IRC | 11:20 | |
| *** yog_ has quit IRC | 11:24 | |
| *** bandini has joined #tripleo | 11:24 | |
| *** weshay has quit IRC | 11:26 | |
| *** mkovacik has quit IRC | 11:28 | |
| *** mburned_out is now known as mburned | 11:29 | |
| *** rhallisey has joined #tripleo | 11:34 | |
| *** bvandenh has joined #tripleo | 11:36 | |
| *** yog_ has joined #tripleo | 11:37 | |
| *** tosky has joined #tripleo | 11:43 | |
| *** thrash|g0ne is now known as thrash | 11:50 | |
| *** trown|outttypeww is now known as trown | 11:57 | |
| *** julim has joined #tripleo | 11:59 | |
| *** jdob has joined #tripleo | 12:02 | |
| *** weshay_xchat has joined #tripleo | 12:05 | |
| jaosorior | shardy: If you have time, could you take a quick look at this CR? https://review.openstack.org/#/c/231930/ started a basic implementation of TLs enabling in the overcloud | 12:15 |
| *** dprince has joined #tripleo | 12:16 | |
| *** jaosorior has quit IRC | 12:18 | |
| *** tzumainn has joined #tripleo | 12:19 | |
| dprince | derekh: hey, so this takes 2 image builds out of the CI job https://review.openstack.org/#/c/233449/. The nonha job ran in 66 minutes. | 12:25 |
| *** pozorvlak has joined #tripleo | 12:26 | |
| jistr | spredzy: did you somehow solve the issue when setting a value to comma_delimited_list parameter via parameter_defaults didn't work? | 12:28 |
| gfidente | marios, regarding the rabbit fd limit, I've fixed the spelling but the original problem mentioned in bug about | 12:28 |
| jistr | spredzy: i'm trying to set a value for a different parameter, which is a comma_delimited_list too, and i can't get it working | 12:29 |
| gfidente | standard limits.conf from systemd is transparently fixed by the puppet module now | 12:29 |
| *** jprovazn has quit IRC | 12:29 | |
| jistr | other params are fine | 12:29 |
| spredzy | jistr, I hit that https://bugs.launchpad.net/tripleo/+bug/1504137 | 12:29 |
| openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 12:29 |
| spredzy | jistr, via parameters: it does work, the issue is when I specify it from the tripleoclient | 12:30 |
| spredzy | somehow its not passed to the heat stack | 12:30 |
| *** masco has quit IRC | 12:31 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Parameterize RabbitMQ FD limit https://review.openstack.org/201796 | 12:31 |
| jistr | spredzy: hmm that sounds like it could be the same issue, as tripleoclient uses parameter_defaults instead of parameters... maybe the problem is not really in tripleoclient, but in Heat not processing parameter_defaults for comma_delimited_list properly | 12:31 |
| spredzy | shardy, ^ | 12:32 |
| shardy | jistr: it's not, we investigated and it works fine direct to heat | 12:32 |
| jistr | hmm do you have somewhere the syntax you used? | 12:32 |
| spredzy | <shardy> jaosorior: are you using parameters or parameter_defaults in your environment file? | 12:32 |
| spredzy | <shardy> jaosorior: "parameters" should overwrite any hard-coded tripleoclient value, otherwise we still have bugs | 12:32 |
| spredzy | jistr, the above was discussed this morning | 12:32 |
| shardy | spredzy: yeah, bugs in tripleoclient :\ | 12:33 |
| jistr | well yeah, 'parameters:' works for me too | 12:33 |
| jistr | but 'parameter_defaults:' doesn't | 12:33 |
| jistr | and i'm not feeding it through tripleoclient | 12:33 |
| shardy | jistr: Ok, I can investigate, what parameter are you changing? | 12:33 |
| marios | gfidente: sec, reading | 12:34 |
| marios | gfidente: thanks man that's fine, was just an early monday morning rant :) | 12:35 |
| jistr | shardy: CinderNfsServers. i want to switch storage_environment.yaml from parameters to parameter_defaults, because some of the parameters will no longer be top-level | 12:35 |
| jistr | shardy: https://gist.github.com/jistr/b2c921e1e02cc8405a92 | 12:35 |
| shardy | jistr: FWIW, I've been through this "parameter_defaults don't work" cycle several times now, and it's always been bugs in the client, where stuff is either merged in the wrong order, or some other value is passed via parameters, which overrides parameter_defaults | 12:35 |
| shardy | jistr: Ok, do you have debug output of the actual call to heat? | 12:37 |
| shardy | First thing to check is that the value in the POST to heat is as expected, and that there's not a duplicate value passed e.g in parameters | 12:37 |
| * shardy tries a test | 12:37 | |
| *** tiswanso_ has quit IRC | 12:37 | |
| *** lucas-hungry is now known as lucasagomes | 12:38 | |
| *** tiswanso has joined #tripleo | 12:38 | |
| *** tiswanso_ has joined #tripleo | 12:39 | |
| *** rbrady has joined #tripleo | 12:40 | |
| *** tiswanso has quit IRC | 12:40 | |
| *** tiswanso has joined #tripleo | 12:40 | |
| gfidente | marios, hey thank you for checking it, it's the purpose of the reviews :) | 12:40 |
| *** tiswanso_ has quit IRC | 12:40 | |
| jistr | shardy: in the debug output of heat-api the parameter value is received from client already wrong. I don't see any parameter_defaults there at all though. Is parameter_defaults a client thing, rather than a Heat API thing? | 12:40 |
| shardy | jistr: no, it's a heat thing, but obviously if you're passing wrong and/or missing values from the client, heat can't do what you want ;) | 12:41 |
| *** tiswanso_ has joined #tripleo | 12:41 | |
| shardy | jistr: the client confusion is that heat (API) expects exactly one environment, with exactly one parameters map, and exactly one (or zero) parameter_defaults map | 12:42 |
| shardy | jistr: the tripleoclient bugs have typically been around merging things before making the call to heat, where you have to resolve multiple environment files into one environment for heat | 12:42 |
| *** tiswanso has quit IRC | 12:44 | |
| jistr | shardy: ack, thanks for the info. i'll try to look around a bit more to find where the problem is | 12:46 |
| shardy | dprince: ^^ FYI this is why I'm pushing to make it much easier to create an overcloud directly via heatclient | 12:47 |
| shardy | it'd be great to have an easy binary test which proves immediately heat vs client bugs | 12:47 |
| *** egafford has joined #tripleo | 12:48 | |
| marios | np gfidente thanks for response | 12:49 |
| dprince | shardy: I'm sold on the idea, just not a fan of some of the tripleoclient defaults | 12:52 |
| shardy | dprince: cool, if I were to refactor into an environments/foo_defaults.yaml, what would I call it? | 12:52 |
| shardy | centos7_ci_defaults.yaml? | 12:52 |
| *** yog_ has quit IRC | 12:52 | |
| dprince | shardy: you are talking to someone who actually misses devtest_overcloud.sh in this regard... because it was less hard coded and just used heatclient | 12:53 |
| dprince | shardy: tripleo-docs.yaml? | 12:53 |
| shardy | dprince: ack - well I'm essentially trying to reassemble that workflow for myself ;) | 12:53 |
| dprince | shardy: you said you were aiming to accomidate the tripleo-docs defaults right? | 12:54 |
| shardy | dprince: Yeah, I want a new contributor to be able to either follow the docs, or run tripleo.sh (which should amount to the same thing), then create an overcloud direct via heat with the absolute minimum of complexity | 12:54 |
| shardy | heat stack-create overcloud -f overcloud.yaml -e overcloud_env.yaml | 12:55 |
| *** Goneri has quit IRC | 12:56 | |
| *** rbrady has quit IRC | 12:57 | |
| dprince | shardy: sure, I always have my own -e $HOME/custom.yaml in there too | 12:57 |
| *** rbrady has joined #tripleo | 12:57 | |
| shardy | dprince: yeah, I think that's great, it just shouldn't be required for basic virt day-1 users | 12:57 |
| *** mcornea has quit IRC | 12:58 | |
| dprince | shardy: as to what you call it either be super descriptive (and have a nice long env name) or perhaps just name it all-in-one-defaults.yaml | 12:58 |
| shardy | tripleo-virt-defaults.yaml? | 12:59 |
| *** mcornea has joined #tripleo | 12:59 | |
| dprince | shardy: that works for me | 12:59 |
| shardy | Or maybe tripleo-docs-defaults.yaml as only one arg is virt specific | 12:59 |
| dprince | shardy: you might even throw the word 'test' in there too | 12:59 |
| shardy | tripleo-docs-maybe-virt-test-proven-by-ci-use-these-please-defaults.yaml ;) | 13:00 |
| dprince | shardy: sure, the only thing super virt about it is what we choose for the KVM setting default | 13:00 |
| shardy | dprince: https://review.openstack.org/#/c/233535/ that's already catching folks out actually | 13:01 |
| shardy | as we don't document it at all, fixed by that patch | 13:01 |
| dprince | shardy: right, this is a case where I'd argue a default should not get set (like we have it already in the Heat templates) | 13:04 |
| dprince | shardy: because people just need to choose the right value for what they want | 13:05 |
| dprince | shardy: so if your environment has this setting set to qemu, then I'd name it virt-testing | 13:05 |
| dprince | shardy: otherwise it sort of targets testing on real hardware | 13:05 |
| *** yog_ has joined #tripleo | 13:06 | |
| *** mestery has joined #tripleo | 13:06 | |
| jistr | btw i created overcloud with heatclient (not successfully finished, just triggered the stack creation to see what params it will get) and i'm getting the same result. Default for a string parameter applied, default for comma_delimited_list didn't | 13:06 |
| jistr | http://fpaste.org/278353/65502714/ | 13:06 |
| jistr | http://paste.fedoraproject.org/278351/54801144/ | 13:06 |
| *** pozorvlak has left #tripleo | 13:07 | |
| jistr | i still wonder if i'm doing something wrong... tried to double-check the param spelling at least | 13:07 |
| *** jaosorior has joined #tripleo | 13:09 | |
| shardy | dprince: ack, maybe removing the default for the libvirt type is a good idea | 13:12 |
| shardy | but it's not unset in the templates, it's "" | 13:12 |
| shardy | which means heat won't ever give you a good validation error | 13:12 |
| dprince | shardy: probably all of the '' are wrong, or most of them | 13:12 |
| shardy | dprince: Cool, I'll take a look and see which may be better removed, or abstracted into a developer-orientated environment file | 13:13 |
| rhallisey | anyone having trouble with using tripleo.sh righjt now? | 13:14 |
| rhallisey | Error: Could not find class ::ironic::inspector for instack on node instack | 13:14 |
| dprince | rhallisey: your instack is missing puppet modules? | 13:15 |
| trown | rhallisey: need to use puppet from source | 13:15 |
| slagle | rhallisey: i think you need to export DIB_INSTALLTYPE_puppet_modules=source before installing undercloud | 13:15 |
| dprince | yes, do that ^^^ | 13:16 |
| rhallisey | lol thanks | 13:16 |
| slagle | we should add that to tripleo.sh | 13:16 |
| trown | rhallisey: the puppet-ironic change did not make it into the openstack-puppet-modules rpm | 13:16 |
| Slower | ah ok | 13:16 |
| Slower | thx guys | 13:16 |
| jistr | and here's a full debug output of the heat stack-create http://paste.fedoraproject.org/278359/55741144/ | 13:17 |
| dprince | shardy: hey, while we are on the topic of parameters would it make sense to outline our "guidelines" in a HACKING doc for t-h-t? | 13:18 |
| jistr | the correct CinderNfsServers value is right beside the only occurence of "retry=1" value (that one did get applied) | 13:18 |
| *** rpothier has joined #tripleo | 13:19 | |
| jistr | shardy: starts to look like a heatclient bug maybe? ^ | 13:19 |
| dprince | shardy: it occurs to me that perhaps we should agree on that first, and then I think it'll make reviewing your patch series much more clear | 13:19 |
| shardy | jistr: Hmm, heatclient is OK AFAICS, ""CinderNfsServers": "192.0.2.1:/export/cinder"" | 13:19 |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: [WIP] Use tripleo.sh to build packages https://review.openstack.org/229425 | 13:20 |
| shardy | so, it is being passed | 13:20 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-docs: EPEL needed for instack-virt-setup https://review.openstack.org/233624 | 13:20 |
| shardy | jistr: let me try to reproduce | 13:20 |
| shardy | dprince: +1, sounds like a good idea, and we can also cover stuff like preferred use of parameters vs parameter_defaults etc | 13:21 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-docs: Workaround for missing puppet-ironic patch in OPM https://review.openstack.org/233627 | 13:22 |
| trown | that is the patch I applied downstream ^ | 13:23 |
| *** morazi has joined #tripleo | 13:25 | |
| *** lblanchard has joined #tripleo | 13:27 | |
| dprince | trown, slagle: this saves us a good bit of CI time https://review.openstack.org/#/c/233449/ | 13:29 |
| trown | dprince: nice, I am surprised it was so easy :) | 13:30 |
| trown | thanks to IPA folks for making it support pxe_ssh driver | 13:31 |
| *** pradk has joined #tripleo | 13:31 | |
| dprince | trown: yeah, I just got IPA working on my bare metal last week | 13:31 |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Start the nova-cert service in HA setup https://review.openstack.org/223027 | 13:33 |
| *** BharatK has quit IRC | 13:36 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-specs: Support the use of an external load balancer as frontend for the Overcloud https://review.openstack.org/233634 | 13:39 |
| *** weshay_xchat is now known as weshay | 13:39 | |
| trown | thrash: https://review.openstack.org/#/c/233449/ is the patch to remove deploy ramdisk I said would come later when you were looking at the patch to build the agent ramdisk | 13:45 |
| thrash | trown: ack | 13:46 |
| *** BharatK has joined #tripleo | 13:51 | |
| derekh | dprince: you havn't changed anything on rh1 recently have you ? | 13:51 |
| dprince | derekh: no | 13:51 |
| dprince | derekh: whats up? | 13:51 |
| derekh | dprince: I'm having trouble accessing the overcloud api url (nova list timing out etc...) | 13:52 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: external loadbalancer: Add net_vip_map_external https://review.openstack.org/222189 | 13:53 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Make load balancer deployment optional via template param https://review.openstack.org/222305 | 13:53 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the Controller nodes https://review.openstack.org/222497 | 13:53 |
| dprince | derekh: locally? from your local machine to the API? | 13:53 |
| derekh | dprince: yup, looks like nodpool may be having similar problems, jobs starting to queue up a little | 13:54 |
| *** Goneri has joined #tripleo | 13:54 | |
| *** jprovazn has joined #tripleo | 13:54 | |
| dprince | derekh: I did a list --all-tenants on the seedhost and it didn't take too long | 13:57 |
| derekh | dprince: ack, is it working for you from outside the rack ? | 13:58 |
| dprince | derekh: no, it hangs for me locally | 13:59 |
| trown | derekh: when you were working on getting HA working in tripleoci, did you come across an issue with ceilometer db-sync failing? | 14:00 |
| derekh | dprince: I was looking in this a few hours ago and ended up restarting some of the controller services which seemed to help, but now its seem to be back | 14:00 |
| derekh | dprince: brb | 14:00 |
| *** mcornea has quit IRC | 14:10 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Allow one to specify horizon ALLOWED_HOSTS https://review.openstack.org/221255 | 14:10 |
| *** mcornea has joined #tripleo | 14:10 | |
| *** BharatK has quit IRC | 14:12 | |
| jistr | spredzy: doing the 3 reviews you requested. if we merge this, the passing of NtpServer param will stop working, right? (parameter_defaults + comma_delimited_list) | 14:12 |
| jistr | spredzy: i'm inclined to give it +2 -A, because if we merge it we won't have a way to set even a single NTP server until the bug you mentioned earlier today is fixed | 14:13 |
| jistr | spredzy: so the patch is good imo, we just need to wait with it | 14:14 |
| jistr | https://review.openstack.org/#/c/232051/5 | 14:14 |
| spredzy | jistr, you're right. It would break it I think | 14:14 |
| spredzy | ok let me comment | 14:14 |
| *** nijaba has joined #tripleo | 14:16 | |
| *** nijaba has quit IRC | 14:16 | |
| *** nijaba has joined #tripleo | 14:16 | |
| *** Goneri has quit IRC | 14:25 | |
| spredzy | jistr, last one if you still have some time :) https://review.openstack.org/#/c/223027/ | 14:29 |
| spredzy | marios, ^ I added the constraint does it look ok to you ? | 14:30 |
| openstackgerrit | Rob Pothier proposed openstack/tripleo-heat-templates: Cisco Nexus ML2 MD: Add Support for mulitple UCSMs https://review.openstack.org/233664 | 14:34 |
| marios | spredzy: reading back | 14:34 |
| *** yamahata has joined #tripleo | 14:34 | |
| *** mestery has quit IRC | 14:34 | |
| marios | spredzy: which review, the nova-cert one? | 14:35 |
| spredzy | marios, yes | 14:35 |
| marios | spredzy: yeah sorry was readying thx | 14:35 |
| *** cody-somerville has joined #tripleo | 14:38 | |
| marios | spredzy: so i don't see anything from poke at https://github.com/beekhof/osp-ha-deploy/search?utf8=%E2%9C%93&q=nova-cert&type=Code about nova-cert - unless i missed it. oterwise sounds sane enough to depend on nova-conductor | 14:44 |
| *** jobewan has joined #tripleo | 14:45 | |
| spredzy | marios, they are nothing there. But based on https://bugzilla.redhat.com/show_bug.cgi?id=1245740 they are no strong constraint. So just making one to make sure the service is colocated with the other nova services | 14:46 |
| openstack | bugzilla.redhat.com bug 1245740 in openstack-tripleo-heat-templates "openstack-nova-cert is installed, but not running on the controller nodes" [Unspecified,On_dev] - Assigned to yguenane | 14:46 |
| marios | spredzy: cool thanks for confirm | 14:47 |
| spredzy | marios, thx to you for the review :) | 14:48 |
| *** egafford has left #tripleo | 14:48 | |
| *** penick has joined #tripleo | 14:49 | |
| trown | slagle: dprince, I wonder if we could consider merging https://review.openstack.org/#/c/228034/ without CI... it is causing issues with jq 1.5 and CI is only failing because it cant build tripleo-incubator | 14:51 |
| *** spzala has joined #tripleo | 14:55 | |
| slagle | i thought fixing the project name map was going to fix that? | 14:55 |
| trown | ya, me too, but it is a bit of a rabbit hole | 14:55 |
| trown | slagle: or are you talking about fixing it in delorean proper? | 14:56 |
| slagle | wherever the mappings are maintained | 14:56 |
| slagle | what map-project-name uses | 14:56 |
| trown | map-project-name just looks at rdoinfo/rdo.yml for the name | 14:58 |
| *** untriaged-bot has joined #tripleo | 15:00 | |
| untriaged-bot | Untriaged bugs so far: | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504137 | 15:00 |
| openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1504632 | 15:00 |
| openstack | Launchpad bug 1504632 in diskimage-builder "elements README documentation environment variables should be clearer" [Undecided,In progress] - Assigned to Augustina Ragwitz (missaugustina) | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504179 | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504263 | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504363 | 15:00 |
| openstack | Launchpad bug 1504179 in tripleo "instack-virt-setup can't use dnsmasq lease file anymore" [Undecided,In progress] - Assigned to Thomas Herve (therve) | 15:00 |
| *** untriaged-bot has quit IRC | 15:00 | |
| openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 15:00 |
| openstack | Launchpad bug 1504263 in tripleo "os-net-config mapping by MAC fails if trying to map in alternate order" [Undecided,Triaged] | 15:00 |
| openstack | Launchpad bug 1504363 in tripleo "Updates to hiera configuration not propogating/making a puppet run" [Undecided,Confirmed] | 15:00 |
| *** paramite has joined #tripleo | 15:01 | |
| *** mkovacik has joined #tripleo | 15:01 | |
| *** hien_ has quit IRC | 15:02 | |
| trown | slagle: I wonder actually if we can change the project to tripleo-incubator and add openstack-tripleo name and it will "just work" https://github.com/redhat-openstack/rdoinfo/blob/master/rdo.yml#L281-L285 | 15:03 |
| *** HenryG_ is now known as HenryG | 15:03 | |
| trown | trying it locally... we will need to revert my hack in tripleoci too https://github.com/openstack-infra/tripleo-ci/commit/3db00edc76113cfd71e152b804c21cff94b07e0c | 15:04 |
| derekh | trown: sorry, missed your message earlier, yes, if you hit the same thing I did, here is the fix https://review.openstack.org/#/c/229183/ | 15:05 |
| *** aufi has quit IRC | 15:05 | |
| trown | derekh: odd, in RDO we are using puppet from source in the overcloud, so I should have that | 15:07 |
| derekh | trown: yup | 15:07 |
| *** hien_ has joined #tripleo | 15:14 | |
| *** absubram has joined #tripleo | 15:22 | |
| *** masterbound has quit IRC | 15:24 | |
| *** shardy_ has joined #tripleo | 15:25 | |
| openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: Revert "There is no tripleo-incubator package" https://review.openstack.org/233686 | 15:27 |
| *** shardy has quit IRC | 15:27 | |
| *** shardy_ has quit IRC | 15:31 | |
| *** shardy has joined #tripleo | 15:31 | |
| *** yog_ has quit IRC | 15:35 | |
| *** jprovazn has quit IRC | 15:37 | |
| *** dtantsur is now known as dtantsur|bbl | 15:47 | |
| jpeeler | dprince: i have this bash script to generate the JSON config files http://paste.fedoraproject.org/278420/64942144/ is this the best place to ask a few things about it? | 15:50 |
| *** penick has quit IRC | 15:50 | |
| *** mbound has joined #tripleo | 15:55 | |
| *** thrash is now known as thrash|biab | 15:57 | |
| *** derekh changes topic to "TripleO ci cloud DOWN https://etherpad.openstack.org/p/snqXyvqubi | CI status: http://tripleo.org/cistatus.html | Docs http://docs.openstack.org/developer/tripleo-docs/" | 16:04 | |
| *** ifarkas has quit IRC | 16:07 | |
| *** trellooobot has joined #tripleo | 16:10 | |
| trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 16:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 16:10 |
| trellooobot | | Title | URL | Members | Last Active | | 16:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 16:10 |
| trellooobot | | Tripleo ci cloud is DOWN | https://trello.com/c/ORiaa8pz | **NEEDS MEMBERS** | 5 min | | 16:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 16:10 |
| *** trellooobot has quit IRC | 16:10 | |
| tosky | o.O | 16:10 |
| *** alop has joined #tripleo | 16:11 | |
| *** sthillma has joined #tripleo | 16:11 | |
| *** olap has quit IRC | 16:13 | |
| *** dsneddon is now known as dsneddon_away_dm | 16:14 | |
| *** bvandenh has quit IRC | 16:19 | |
| *** mkovacik has quit IRC | 16:19 | |
| *** ukalifon has quit IRC | 16:19 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Support NFS backend for Glance (via Pacemaker) https://review.openstack.org/199152 | 16:20 |
| *** sthillma has quit IRC | 16:25 | |
| *** thrash|biab is now known as thrash | 16:27 | |
| *** sthillma has joined #tripleo | 16:29 | |
| *** jistr has quit IRC | 16:42 | |
| *** pradk has quit IRC | 16:42 | |
| *** sthillma has quit IRC | 16:42 | |
| *** pradk has joined #tripleo | 16:43 | |
| *** yamahata has quit IRC | 16:43 | |
| *** mbound has quit IRC | 16:45 | |
| *** gfidente is now known as gfidente|afk | 16:48 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Allow ctlplane network or ID, and default to "ctlplane" https://review.openstack.org/232101 | 16:51 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Update overcloud template description and Count constraints https://review.openstack.org/232084 | 16:51 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Align some neutron defaults with CI https://review.openstack.org/232123 | 16:51 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Align some defaults with with ControllerCount default https://review.openstack.org/232122 | 16:51 |
| derekh | dprince: The controller has been restarted (details of what I've done are on this etherpad https://etherpad.openstack.org/p/snqXyvqubi ) | 16:53 |
| derekh | dprince: api access from outside of rack is now working, I'll come back to continue in a few hours | 16:53 |
| derekh | dprince: next we need to being up the compute nodes, | 16:54 |
| derekh | dprince: I've added a iptable rule to block infra/nodepool intol we're back up and running | 16:55 |
| dprince | derekh: thanks | 16:55 |
| *** derekh is now known as derekh_afk | 16:56 | |
| dprince | derekh_afk: our ssl cert might have regressed since running os-refresh-config, we'll need to put back in place the updated one for 2015 | 16:56 |
| *** dtantsur|bbl is now known as dtantsur | 16:57 | |
| *** mkovacik has joined #tripleo | 16:59 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-docs: Add note that qemu libvirt-type is needed for virt environment https://review.openstack.org/233535 | 17:03 |
| slagle | thrash: i dont get what you're asking for on https://review.openstack.org/#/c/232105 | 17:04 |
| slagle | thrash: you want validation-errors-nonfatal defaulted to True? | 17:05 |
| *** yamahata has joined #tripleo | 17:05 | |
| slagle | in that case, i'd just leave the arg as validation-errors-fatal, and default that to False | 17:06 |
| thrash | slagle: yes, but then the 'action' would be store_false | 17:06 |
| thrash | true is the default. | 17:06 |
| thrash | then the check at line 959 would make more sense. | 17:06 |
| thrash | especiailly if you flip validation-warnings-fatal | 17:07 |
| slagle | it'd be the same check though (modulo the warnings change) | 17:07 |
| slagle | you'd still have to "if not parsed_args.validation_errors_nonfatal" | 17:07 |
| thrash | then that check at 959 becomes 'it not parsed_args.validation_warnings_fatal and parsed_args.validation_errors_nonfatal' | 17:07 |
| thrash | *if | 17:08 |
| slagle | ok, so why i reveresed the arg was so that the existing behavior would stay the same | 17:08 |
| thrash | I thought you *wanted* the existing behaviour to change? | 17:09 |
| thrash | You wanted the errors to become fatal, right? | 17:09 |
| thrash | by default... | 17:09 |
| slagle | they already were fatal by default, just not covered by the cli arg | 17:10 |
| *** trown is now known as trown|lunch | 17:10 | |
| *** trellooobot has joined #tripleo | 17:10 | |
| trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 17:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 17:10 |
| trellooobot | | Title | URL | Members | Last Active | | 17:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 17:10 |
| trellooobot | | Tripleo ci cloud is DOWN | https://trello.com/c/ORiaa8pz | **NEEDS MEMBERS** | 65 min | | 17:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 17:10 |
| *** trellooobot has quit IRC | 17:10 | |
| slagle | thrash: there was no way to say "not fatal" before for the errors that i made validations in the subsequent patches | 17:10 |
| slagle | but, if we don't care about those staying fatal by default, we can just keep the existing arg as-is | 17:11 |
| slagle | the node count check for instance | 17:12 |
| slagle | there's no way to disable that right now | 17:12 |
| thrash | slagle: I think I'm mostly concerned with the statement at 959 | 17:13 |
| thrash | reading it "naturally" it doesn't make sense to me. | 17:14 |
| *** mcornea has quit IRC | 17:14 | |
| thrash | slagle: I may be misreading that chunk of code tho | 17:14 |
| thrash | slagle: I see it now. The return is actually exiting the command... | 17:16 |
| thrash | slagle: I am convinced. I will remove my -1 if you fix L901. :) | 17:17 |
| slagle | thrash: i guess what it comes down to is...do we want warnings and errors fatal by default? | 17:18 |
| thrash | slagle: I think errors should be fatal by default. It's an error. | 17:19 |
| slagle | that implies which arg we need...fatal or nonfatal | 17:19 |
| slagle | ok, i'll go with that. and give warnigns the same treatment, just for consistency sake | 17:20 |
| thrash | warnings on the other hand I think makes sense to say non-fatal by default | 17:20 |
| thrash | as you have it seems fine to me. | 17:21 |
| slagle | ok | 17:21 |
| thrash | just fix the help string on L901. :) | 17:21 |
| slagle | you have the inconsistent cli args that way, but that's ok i suppose | 17:21 |
| thrash | slagle: but it makes sense at least. Warnings are non-fatal by default, so an arg to make them fatal. Errors are fatal by default, so an arg to make it non-fatal | 17:22 |
| *** penick has joined #tripleo | 17:22 | |
| *** lazy_prince has quit IRC | 17:29 | |
| *** shivrao has joined #tripleo | 17:31 | |
| *** paramite has quit IRC | 17:32 | |
| *** jcoufal has quit IRC | 17:32 | |
| dprince | looks like http://cbs.centos.org is out again | 17:40 |
| *** pelix has quit IRC | 17:42 | |
| *** sthillma has joined #tripleo | 17:42 | |
| dprince | trown|lunch, slagle: who do we reach out to for the cbs.centos.org failures? | 17:44 |
| *** mcornea has joined #tripleo | 17:46 | |
| *** tosky has quit IRC | 17:46 | |
| *** Marga_ has quit IRC | 17:47 | |
| *** rlandy has joined #tripleo | 17:48 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Fix cherry-pick syntax https://review.openstack.org/225342 | 17:54 |
| slagle | dprince: most likely #rdo or #centos-devel | 17:54 |
| *** bhunter has quit IRC | 17:57 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Fix cherry-pick syntax https://review.openstack.org/225342 | 17:57 |
| *** dsneddon_away_dm is now known as dsneddon | 17:59 | |
| *** bhunter has joined #tripleo | 17:59 | |
| *** adam_g` is now known as adam_g | 18:00 | |
| *** Marga_ has joined #tripleo | 18:02 | |
| *** dtantsur is now known as dtantsur|afk | 18:03 | |
| *** penick has quit IRC | 18:03 | |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Add Linux Bridge capability to os-net-config ifcfg https://review.openstack.org/219927 | 18:04 |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Fix kwarg assignment in objects.py https://review.openstack.org/231247 | 18:07 |
| *** lucasagomes is now known as lucas-dinner | 18:08 | |
| *** trown|lunch is now known as trown | 18:08 | |
| *** trellooobot has joined #tripleo | 18:10 | |
| trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 18:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 18:10 |
| trellooobot | | Title | URL | Members | Last Active | | 18:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 18:10 |
| trellooobot | | Tripleo ci cloud is DOWN | https://trello.com/c/ORiaa8pz | **NEEDS MEMBERS** | 125 min | | 18:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 18:10 |
| *** trellooobot has quit IRC | 18:10 | |
| *** weshay has quit IRC | 18:10 | |
| *** penick has joined #tripleo | 18:12 | |
| *** penick has quit IRC | 18:17 | |
| openstackgerrit | Ben Nemec proposed openstack/python-tripleoclient: Move common functions out into utils.py https://review.openstack.org/225244 | 18:24 |
| *** weshay has joined #tripleo | 18:25 | |
| *** Marga_ has quit IRC | 18:25 | |
| openstackgerrit | Ben Nemec proposed openstack/python-tripleoclient: Consolidate module constants. https://review.openstack.org/225245 | 18:25 |
| *** penick has joined #tripleo | 18:26 | |
| *** Marga_ has joined #tripleo | 18:26 | |
| openstackgerrit | Ben Nemec proposed openstack/python-tripleoclient: Move more functions to utils.py https://review.openstack.org/225246 | 18:26 |
| *** matbu has quit IRC | 18:31 | |
| *** shivrao has quit IRC | 18:35 | |
| *** penick has quit IRC | 18:40 | |
| openstackgerrit | James Slagle proposed openstack/python-tripleoclient: Make hypervisor stat check a validation https://review.openstack.org/232041 | 18:40 |
| openstackgerrit | James Slagle proposed openstack/python-tripleoclient: Add --skip-validation-checks cli arg https://review.openstack.org/232209 | 18:40 |
| openstackgerrit | James Slagle proposed openstack/python-tripleoclient: Make validation errros fatal by default https://review.openstack.org/232105 | 18:40 |
| openstackgerrit | James Slagle proposed openstack/python-tripleoclient: Make node count check a validation https://review.openstack.org/232106 | 18:40 |
| *** rlandy has quit IRC | 18:41 | |
| *** matbu has joined #tripleo | 18:47 | |
| *** greghaynes has joined #tripleo | 18:51 | |
| greghaynes | bnemec: OHAI. I was looking at https://review.openstack.org/#/c/222351/8 and a tangent thought occured to me - do the centos7 and rhel7 elements need to be very different? I wonder if we could make one element which does both so we can test the code (since theres no real hope of testing an element which uses publicly unavailable images) | 18:53 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Set shared secrets, keys and passwords as hidden https://review.openstack.org/233578 | 18:54 |
| *** julim has quit IRC | 19:01 | |
| *** jdob1 has joined #tripleo | 19:03 | |
| *** jdob1 has quit IRC | 19:03 | |
| *** shivrao has joined #tripleo | 19:06 | |
| *** trellooobot has joined #tripleo | 19:10 | |
| trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 19:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 19:10 |
| trellooobot | | Title | URL | Members | Last Active | | 19:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 19:10 |
| trellooobot | | Tripleo ci cloud is DOWN | https://trello.com/c/ORiaa8pz | **NEEDS MEMBERS** | 185 min | | 19:10 |
| trellooobot | +--------------------------+-------------------------------+-------------------+-------------+ | 19:10 |
| *** trellooobot has quit IRC | 19:10 | |
| *** hien_ has quit IRC | 19:15 | |
| openstackgerrit | Merged openstack/diskimage-builder: Update apt-conf elements README from free text to table formatting https://review.openstack.org/231770 | 19:23 |
| derekh_afk | dprince: ack, will figure out the ssl certs now and then hopfully start bringing things back | 19:24 |
| openstackgerrit | Merged openstack/diskimage-builder: Update apt-preferences element README from free text to table formatting https://review.openstack.org/232211 | 19:26 |
| openstackgerrit | James Slagle proposed openstack/python-tripleoclient: Add --skip-validation-checks cli arg https://review.openstack.org/232209 | 19:28 |
| *** derekh_afk is now known as derekh | 19:31 | |
| *** shivrao has quit IRC | 19:36 | |
| *** spzala has quit IRC | 19:46 | |
| openstackgerrit | James Slagle proposed openstack/os-cloud-config: Remove requirement of specifying hardware stats https://review.openstack.org/231613 | 20:00 |
| *** weshay_xchat has joined #tripleo | 20:00 | |
| *** rasca has quit IRC | 20:00 | |
| *** gfidente|afk is now known as gfidente | 20:02 | |
| *** weshay has quit IRC | 20:04 | |
| *** spzala has joined #tripleo | 20:04 | |
| *** jtomasek has quit IRC | 20:04 | |
| openstackgerrit | Merged openstack/diskimage-builder: Adds debian support to iso element https://review.openstack.org/211976 | 20:15 |
| *** spzala has quit IRC | 20:18 | |
| *** olap has joined #tripleo | 20:19 | |
| trown | dprince or derekh any chance you could look at a small patch to tripleo-ci, I think I fixed the issue with tripleo-incubator not building in CI, but we need to revert my previous hack https://review.openstack.org/#/c/233686/ | 20:19 |
| derekh | trown: looking | 20:20 |
| *** jaosorior has quit IRC | 20:20 | |
| dprince | trown: +2, will send once we get CI back online... | 20:21 |
| trown | awesome, thansk | 20:21 |
| *** weshay_xchat has quit IRC | 20:25 | |
| bnemec | greghaynes: There's actually almost nothing in the rhel7 element at this point. It's almost all been factored out into redhat-common and rhel-common. | 20:35 |
| greghaynes | bnemec: perfect! | 20:36 |
| bnemec | We do need to set up third party CI to run tests with rhel, but ENOTIME. | 20:36 |
| openstackgerrit | Ben Nemec proposed openstack/diskimage-builder: Update rhel7 element readme https://review.openstack.org/222351 | 20:40 |
| *** sthillma has quit IRC | 20:46 | |
| *** lblanchard has quit IRC | 20:50 | |
| *** yamahata has quit IRC | 20:52 | |
| *** yamahata has joined #tripleo | 20:52 | |
| trown | should ceilometer be trying to run dbsync on the controller in a HA setup? | 20:55 |
| *** dprince has quit IRC | 20:59 | |
| *** untriaged-bot has joined #tripleo | 21:00 | |
| untriaged-bot | Untriaged bugs so far: | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504137 | 21:00 |
| openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1504632 | 21:00 |
| openstack | Launchpad bug 1504632 in diskimage-builder "elements README documentation environment variables should be clearer" [Undecided,In progress] - Assigned to Augustina Ragwitz (missaugustina) | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504179 | 21:00 |
| openstack | Launchpad bug 1504179 in tripleo "instack-virt-setup can't use dnsmasq lease file anymore" [Undecided,In progress] - Assigned to Thomas Herve (therve) | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 21:00 |
| openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504263 | 21:00 |
| openstack | Launchpad bug 1504263 in tripleo "os-net-config mapping by MAC fails if trying to map in alternate order" [Undecided,Triaged] | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504363 | 21:00 |
| openstack | Launchpad bug 1504363 in tripleo "Updates to hiera configuration not propogating/making a puppet run" [Undecided,Confirmed] | 21:00 |
| *** untriaged-bot has quit IRC | 21:00 | |
| openstackgerrit | Jon Schlueter proposed openstack/tripleo-docs: Add step to basic deployment cli to verify images were built https://review.openstack.org/233817 | 21:04 |
| *** absubram has quit IRC | 21:15 | |
| *** trown is now known as trown|outttypeww | 21:17 | |
| *** Marga_ has quit IRC | 21:23 | |
| *** Marga_ has joined #tripleo | 21:24 | |
| *** rhefner has quit IRC | 21:25 | |
| *** matbu has quit IRC | 21:30 | |
| *** matbu has joined #tripleo | 21:31 | |
| *** Hazelesque_ is now known as Hazelesque | 21:35 | |
| *** rpothier has quit IRC | 21:38 | |
| *** morazi has quit IRC | 21:50 | |
| *** mcornea has quit IRC | 21:50 | |
| *** matbu has quit IRC | 21:52 | |
| *** matbu has joined #tripleo | 21:53 | |
| *** dsneddon has quit IRC | 21:57 | |
| *** dsneddon has joined #tripleo | 22:01 | |
| *** matbu has quit IRC | 22:02 | |
| *** matbu has joined #tripleo | 22:03 | |
| *** trellooobot has joined #tripleo | 22:10 | |
| trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 22:10 |
| trellooobot | +--------------------------+-------------------------------+---------------+-------------+ | 22:10 |
| trellooobot | | Title | URL | Members | Last Active | | 22:10 |
| trellooobot | +--------------------------+-------------------------------+---------------+-------------+ | 22:10 |
| trellooobot | | Tripleo ci cloud is DOWN | https://trello.com/c/ORiaa8pz | Derek Higgins | 160 min | | 22:10 |
| trellooobot | +--------------------------+-------------------------------+---------------+-------------+ | 22:10 |
| *** trellooobot has quit IRC | 22:10 | |
| openstackgerrit | Merged openstack/instack-undercloud: Update README to include tags recognized by release-tools https://review.openstack.org/221671 | 22:14 |
| *** olap has quit IRC | 22:17 | |
| *** gfidente has quit IRC | 22:18 | |
| *** mkovacik has quit IRC | 22:19 | |
| *** pradk has quit IRC | 22:30 | |
| *** spzala has joined #tripleo | 22:39 | |
| *** rhefner has joined #tripleo | 22:53 | |
| *** david-ly_ has joined #tripleo | 23:01 | |
| *** david-lyle has quit IRC | 23:02 | |
| *** david-lyle has joined #tripleo | 23:04 | |
| *** david-ly_ has quit IRC | 23:04 | |
| *** trellooobot has joined #tripleo | 23:10 | |
| trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 23:10 |
| trellooobot | +--------------------------+-------------------------------+---------------+-------------+ | 23:10 |
| trellooobot | | Title | URL | Members | Last Active | | 23:10 |
| trellooobot | +--------------------------+-------------------------------+---------------+-------------+ | 23:10 |
| trellooobot | | Tripleo ci cloud is DOWN | https://trello.com/c/ORiaa8pz | Derek Higgins | 220 min | | 23:10 |
| trellooobot | +--------------------------+-------------------------------+---------------+-------------+ | 23:10 |
| *** trellooobot has quit IRC | 23:10 | |
| *** beagles has quit IRC | 23:17 | |
| *** b3nt_pin has joined #tripleo | 23:27 | |
| *** jobewan has quit IRC | 23:31 | |
| *** lucas-dinner has quit IRC | 23:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!