| *** sthillma has joined #tripleo | 00:02 | |
| *** jcoufal has quit IRC | 00:16 | |
| *** rasca has joined #tripleo | 00:16 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration. https://review.openstack.org/226057 | 00:17 |
|---|---|---|
| *** penick has quit IRC | 00:21 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add all isolated networks to all nodes. https://review.openstack.org/249533 | 00:25 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration. https://review.openstack.org/226057 | 00:25 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics https://review.openstack.org/252683 | 00:27 |
| *** eil397 has quit IRC | 00:39 | |
| *** dmacpher has joined #tripleo | 00:50 | |
| *** openstack has joined #tripleo | 01:06 | |
| *** mbound_ has quit IRC | 01:07 | |
| *** julim has joined #tripleo | 01:28 | |
| *** cwolferh has quit IRC | 01:56 | |
| *** shivrao has quit IRC | 02:05 | |
| *** eil397 has joined #tripleo | 02:08 | |
| *** sthillma has quit IRC | 02:54 | |
| openstackgerrit | Steve Baker proposed openstack/tripleo-heat-templates: Add SoftwareConfigTransport for switching transports https://review.openstack.org/250593 | 02:54 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-heat-templates: Switch to POLL_TEMP_URL for config transport https://review.openstack.org/257657 | 02:54 |
| *** yamahata has quit IRC | 03:05 | |
| *** rhallisey has quit IRC | 03:05 | |
| *** yuanying has quit IRC | 03:20 | |
| *** lazy_prince has joined #tripleo | 03:48 | |
| *** david-lyle has quit IRC | 03:50 | |
| *** david-lyle has joined #tripleo | 03:58 | |
| *** david-lyle has quit IRC | 04:01 | |
| *** david-lyle has joined #tripleo | 04:02 | |
| *** yuanying has joined #tripleo | 04:07 | |
| *** links has joined #tripleo | 04:07 | |
| *** eil397 has quit IRC | 04:12 | |
| *** yamahata has joined #tripleo | 04:19 | |
| *** openstackstatus has quit IRC | 04:24 | |
| *** openstack has joined #tripleo | 04:25 | |
| *** openstackstatus has joined #tripleo | 04:26 | |
| *** ChanServ sets mode: +v openstackstatus | 04:26 | |
| *** tzumainn has quit IRC | 05:06 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function https://review.openstack.org/245643 | 05:23 |
| *** masco has joined #tripleo | 05:49 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function https://review.openstack.org/245643 | 06:09 |
| *** aufi has joined #tripleo | 06:32 | |
| *** trembleWork has quit IRC | 07:07 | |
| *** stendulker has joined #tripleo | 07:10 | |
| *** dmacpher has quit IRC | 07:13 | |
| *** ukalifon1 has joined #tripleo | 07:29 | |
| *** jprovazn has joined #tripleo | 07:35 | |
| *** tremble has joined #tripleo | 07:57 | |
| *** tremble has joined #tripleo | 07:57 | |
| *** yuanying has quit IRC | 08:05 | |
| *** mkovacik has joined #tripleo | 08:15 | |
| *** shivrao has joined #tripleo | 08:25 | |
| *** yamahata has quit IRC | 08:26 | |
| *** shivrao has quit IRC | 08:27 | |
| *** athomas has joined #tripleo | 08:28 | |
| *** rleander has joined #tripleo | 08:29 | |
| *** jcoufal has joined #tripleo | 08:30 | |
| *** ifarkas has joined #tripleo | 08:34 | |
| *** shivrao has joined #tripleo | 08:36 | |
| *** shardy has joined #tripleo | 08:37 | |
| *** morazi has joined #tripleo | 08:50 | |
| *** mbound_ has joined #tripleo | 08:51 | |
| shardy | marios: Hey g'morning - any chance you can review/approve these two please? | 08:51 |
| shardy | https://review.openstack.org/#/c/256933/ | 08:51 |
| shardy | https://review.openstack.org/#/c/247379/ | 08:52 |
| openstackgerrit | Shu Muto proposed openstack/python-tripleoclient: Drop py33 support https://review.openstack.org/257740 | 08:52 |
| shardy | midonet related stable backports, both passing CI | 08:52 |
| *** mbound_ has quit IRC | 08:52 | |
| *** mbound_ has joined #tripleo | 08:52 | |
| marios | shardy: sure looking# | 08:53 |
| *** jcoufal has quit IRC | 08:56 | |
| *** jistr has joined #tripleo | 08:56 | |
| *** jcoufal has joined #tripleo | 08:57 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add simple linux bridge as network environment https://review.openstack.org/247379 | 08:58 |
| shardy | marios: thanks - hey I was wondering, what's the status of https://review.openstack.org/#/c/241167/ ? | 08:59 |
| shardy | that could be pretty useful in the context of upgrade testing | 08:59 |
| marios | shardy: :( I know. every dayi plan to work on it, then blockers. | 09:00 |
| shardy | It'd be cool if we could translate that into launching the VM via heat, to gain a little more API coverage | 09:00 |
| shardy | marios: No worries, I figured that may be the case :( | 09:00 |
| marios | shardy: i will try today to get somethign quick out to iterate on | 09:00 |
| shardy | marios: no pressure, I was just wondering that's all | 09:00 |
| marios | shardy: has been quiet but still early in the day | 09:00 |
| shardy | marios: also, I left a comment re fedora/cirros | 09:00 |
| marios | shardy: ack thanks | 09:00 |
| shardy | our experience with images in heat CI is the cirros image boots way, way faster | 09:01 |
| shardy | so maybe worth considering | 09:01 |
| marios | shardy: so i played with it | 09:01 |
| marios | shardy: i had this discussino with derekh | 09:01 |
| marios | who also preferred cirros | 09:01 |
| marios | shardy: but i consistently had issues pinging into the cirros, | 09:01 |
| marios | shardy: at the time i didn't investigate further | 09:01 |
| marios | shardy: especially cos, we build the fedora-user anyway and it is there | 09:02 |
| shardy | marios: ack, Ok that's weird | 09:02 |
| marios | shardy: cirros we need to download, so there is an element of network issues | 09:02 |
| marios | but yeay, i don't mind really | 09:02 |
| marios | so i did try it | 09:02 |
| shardy | I guess we can always start with fedora then optimize, provided the performance hit isn't too huge | 09:02 |
| shardy | I've used cirros a lot locally and never had a problem | 09:02 |
| *** olap has quit IRC | 09:04 | |
| marios | shardy: right, not sure what was up. i can try it again when i get back to it | 09:04 |
| shardy | marios: kk, cool, thanks for the update :) | 09:04 |
| *** julim has quit IRC | 09:10 | |
| *** julim has joined #tripleo | 09:10 | |
| openstackgerrit | Shu Muto proposed openstack/python-tuskarclient: Drop py33 support https://review.openstack.org/257754 | 09:13 |
| *** shivrao has quit IRC | 09:22 | |
| shardy | Simple tripleo.sh review which will enable testing deletes in CI: | 09:32 |
| shardy | https://review.openstack.org/#/c/250486/ | 09:32 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Set swift replicas = min(device_count, replicas) https://review.openstack.org/257540 | 09:32 |
| *** mcornea has joined #tripleo | 09:34 | |
| *** nico_auvv has joined #tripleo | 09:42 | |
| *** devvesa has joined #tripleo | 09:43 | |
| devvesa | Hi. Do you have any idea why this patch is on workflow+1 since yesterday but not merged? | 09:46 |
| devvesa | https://review.openstack.org/#/c/251319/ | 09:46 |
| devvesa | Can it be because the Depends-On is related on two reviews (master and stable/liberty) | 09:46 |
| devvesa | ? | 09:46 |
| *** openstackgerrit has quit IRC | 09:47 | |
| *** openstackgerrit has joined #tripleo | 09:47 | |
| *** akrivoka has joined #tripleo | 09:49 | |
| *** jtomasek has joined #tripleo | 09:51 | |
| openstackgerrit | Merged openstack/instack-undercloud: Remove tripleo::ringbuilder cruft https://review.openstack.org/256631 | 09:53 |
| jistr | devvesa: hi, yes i think that's the cause, i've hit it on one of my patches a while ago. It's a bit unfortunate that it works like this, but i don't know of any workaround. I +2'd the stable/liberty backport of the dependency review, and shardy already requested a recheck, so if CI passes there, we should be able to merge both patches. | 10:00 |
| devvesa | jistr: Great. Thanks for the rechecks shardy :) | 10:01 |
| *** derekh has joined #tripleo | 10:01 | |
| *** dtantsur|afk is now known as dtantsur | 10:02 | |
| *** mgould has joined #tripleo | 10:11 | |
| shardy | devvesa: Yes, Depends-On is not branch aware, which is why I generally wait until all patches have landed to master before proposing stable backports | 10:13 |
| shardy | that also allows for a little time to work out any issues on master, vs immediately backporting | 10:13 |
| mgould | hi everyone! | 10:14 |
| shardy | Hi mgould! | 10:15 |
| mgould | my patch to python-tripleoclient is failing the tripleo CI gate; what's the best way for me to duplicate the failure locally? | 10:15 |
| devvesa | shardy: No problem. As long as I understand there's nothing wrong on the patches or the CI gates, it is ok | 10:15 |
| mgould | I do not currently have a working install of TripleO | 10:15 |
| mgould | https://review.openstack.org/#/c/235158/ | 10:15 |
| mgould | shardy, hi! | 10:15 |
| devvesa | mgould: your patch has failed on the overcloud | 10:17 |
| devvesa | Can it be unrelated to your patch? | 10:18 |
| devvesa | I see a 'No hosts matched' on the Jenkins jobs | 10:18 |
| mgould | devvesa, there was one error that looked unrelated, but another which looked related to my patch | 10:18 |
| shardy | mgould: for developers, most folks are using a VM setup, as documented here: | 10:19 |
| shardy | http://docs.openstack.org/developer/tripleo-docs/environments/environments.html#virtual-environment | 10:19 |
| shardy | mgould: there is a shell script which automates some steps after "instack-virt-setup" | 10:19 |
| shardy | https://github.com/openstack/tripleo-common/blob/master/scripts/tripleo.sh | 10:19 |
| shardy | that is what we use in CI, so many folks will use that to reproduce gate issues | 10:19 |
| mgould | the line https://review.openstack.org/#/c/235158/ just before failure is logging from my code | 10:20 |
| mgould | shardy, awesome | 10:20 |
| shardy | mgould: there is a docs patch for tripleo.sh, but it seems to have stalled: | 10:21 |
| shardy | https://review.openstack.org/#/c/230027/ | 10:21 |
| mgould | shardy, excellent | 10:21 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Drop py33 support https://review.openstack.org/257808 | 10:23 |
| mgould | shardy, so I clone that repo onto a clean VM and run the script? | 10:23 |
| shardy | mgould: no, follow the virtual-environment docs until you've run "instack-virt-setup" | 10:23 |
| shardy | then, ssh onto the instack VM, and clone tripleo-common | 10:23 |
| mgould | right, thanks | 10:24 |
| shardy | after that you can run triple.sh and follow the options in the order they are listed | 10:24 |
| *** electrofelix has joined #tripleo | 10:24 | |
| mgould | OK | 10:25 |
| shardy | mgould: There is a "--all" option, but I prefer to run only the following steps, manually, as it's easier if something goes wrong | 10:25 |
| shardy | --repo-setup, then --undercloud, then --overcloud-images then --register-nodes, then --flavors then --overcloud-deploy | 10:26 |
| shardy | mgould: the --all does some optional steps like node introspection which aren't strictly necessary for development usage | 10:26 |
| shardy | also, the delorean steps are useful for building packages, but not required for basic usage | 10:26 |
| mgould | shardy, my patch relates to node introspection :-) | 10:27 |
| mgould | so yeah, that's something I'll have to do at some point | 10:27 |
| shardy | mgould: Aha, well you'll want to do that then! :) | 10:27 |
| * shardy normally skips it | 10:27 | |
| mgould | I'll maybe skip it during setup, git-checkout my patch, then run introspection | 10:27 |
| shardy | mgould: note for development, the --delorean-setup option configures a local delorean setup | 10:28 |
| shardy | which can be run on the host, or the undercloud if you have enough memory/disk | 10:28 |
| shardy | personally I run it on the host machine | 10:28 |
| shardy | then you can do e.g tripleo.sh --delorean-build openstack/ironic | 10:28 |
| shardy | which will pull the ironic git master and build a local delorean RPM in ~/tripleo/delorean/data/repos/current | 10:29 |
| *** SlickNik has quit IRC | 10:29 | |
| shardy | you can then pull whatever patches you need into e.g ~/tripleo/ironic and re-run --delorean-build | 10:29 |
| * shardy notes we need a docs patch for this | 10:29 | |
| * mgould googles "delorean" | 10:30 | |
| mgould | ah, it builds rpms from git commits | 10:30 |
| shardy | mgould: Yeah, it's basically a wrapper for mock | 10:31 |
| shardy | we use it in CI | 10:31 |
| mgould | cool | 10:31 |
| mgould | OpenStack has a lot of moving parts with non-obvious names :-( | 10:32 |
| shardy | Yeah, true | 10:32 |
| *** openstackgerrit has quit IRC | 10:32 | |
| dtantsur | mgould, I think I've spotted problem with your patch | 10:32 |
| *** openstackgerrit has joined #tripleo | 10:32 | |
| *** epheo has quit IRC | 10:32 | |
| *** epheo has joined #tripleo | 10:33 | |
| * dtantsur got hit by the same mistake just yesterday | 10:33 | |
| mgould | dtantsur, thanks! | 10:34 |
| *** SlickNik has joined #tripleo | 10:35 | |
| mgould | right, I'll make that change, kick off a CI build, and meanwhile continue trying to duplicate it locally, because that will be a useful thing to know how to do | 10:37 |
| mgould | thanks very much! | 10:42 |
| *** stendulker has quit IRC | 10:45 | |
| devvesa | An easy +2 here, please? https://review.openstack.org/#/c/257420/ | 10:46 |
| openstackgerrit | Miles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state https://review.openstack.org/235158 | 10:48 |
| *** paramite has joined #tripleo | 10:50 | |
| dtantsur | mgould, for future debugging: the "last_error" line in http://logs.openstack.org/58/235158/10/check-tripleo/gate-tripleo-ci-f22-nonha/b40b4fb/console.html.gz#_2015-12-10_15_51_17_757 is an error message from AttributeError | 10:50 |
| dtantsur | which raises a good question of if we should run everything in debug mode in gate really | 10:51 |
| dtantsur | shardy, wdyt ^^ | 10:51 |
| shardy | dtantsur: yes, I think we should | 10:52 |
| shardy | some things are also hidden inside tripleo.sh, where we should be set -x'ing to ensure visibility when things go wrong | 10:52 |
| shardy | but that's not what you want for developers running the script so we probably need a variable to enable super-verbose mode for CI | 10:53 |
| mgould | dtantsur, can we make that error message more explicit? | 10:56 |
| mgould | I saw it but had no idea where it was coming from | 10:56 |
| *** tosky has joined #tripleo | 10:56 | |
| dtantsur | mgould, that's how OpenStackClient works IIRC.. it probably should be discussed on #openstack-sdk with them | 10:57 |
| dtantsur | I also find it annoying FWIW | 10:57 |
| mgould | dtantsur, which repo does that happen in? | 11:01 |
| dtantsur | mgould, python-openstackclient is what we use as a basis for our CLI | 11:03 |
| *** rebrego has joined #tripleo | 11:05 | |
| *** regebro has quit IRC | 11:05 | |
| *** rebrego is now known as regebro | 11:05 | |
| * mgould clones Yet Another Repo | 11:05 | |
| dtantsur | $ ls ~/Projects/ | wc -l | 11:07 |
| dtantsur | 63 | 11:07 |
| dtantsur | mgould, ^^ ;) | 11:07 |
| *** rasca has quit IRC | 11:07 | |
| *** rasca has joined #tripleo | 11:08 | |
| mgould | 17, I have a way to go :-/ | 11:08 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: neutron: enable nova-event-callback by default https://review.openstack.org/255226 | 11:09 |
| shardy | https://etherpad.openstack.org/p/tripleo-reviews-ready-to-land | 11:11 |
| shardy | ^^ there are 8 easy stable/liberty reviews there, +2'd and passing CI | 11:11 |
| shardy | just needs approving if someone has a moment | 11:11 |
| devvesa | Can I add this one, shardy? https://review.openstack.org/#/c/257420/ | 11:16 |
| *** Marga_ has joined #tripleo | 11:16 | |
| *** Marga_ has quit IRC | 11:17 | |
| *** Marga_ has joined #tripleo | 11:17 | |
| *** regebro has quit IRC | 11:18 | |
| *** lucas-dinner is now known as lucasagomes | 11:18 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: [WIP] Add the Plan deploy endpoint to start a Heat deploy https://review.openstack.org/256384 | 11:18 |
| *** regebro has joined #tripleo | 11:19 | |
| *** mburned is now known as mburned_out | 11:22 | |
| dtantsur | shardy, could you please take a look at https://review.openstack.org/#/c/253476/ ? it will make CI actually cover introspection | 11:29 |
| *** Marga_ has quit IRC | 11:30 | |
| *** paramite is now known as paramite|afk | 11:33 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Raise the Swift exception if we get one and it isn't a 404 https://review.openstack.org/257843 | 11:34 |
| *** paramite|afk is now known as paramite | 11:35 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Raise the Swift exception if we get one and it isn't a 404 https://review.openstack.org/257843 | 11:35 |
| shardy | dtantsur: lgtm, maybe d0ugal can give it a final +2/A | 11:35 |
| *** gfidente has joined #tripleo | 11:35 | |
| *** gfidente has quit IRC | 11:35 | |
| *** gfidente has joined #tripleo | 11:35 | |
| d0ugal | shardy, dtantsur - sure. | 11:35 |
| *** Marga_ has joined #tripleo | 11:36 | |
| d0ugal | well, I'll look before I promise the +2 actually :P | 11:36 |
| dtantsur | good idea :D | 11:36 |
| shardy | hehe | 11:37 |
| d0ugal | dtantsur: lgtm | 11:38 |
| dtantsur | thanks | 11:39 |
| *** Marga_ has quit IRC | 11:40 | |
| *** Marga__ has joined #tripleo | 11:40 | |
| *** trown|outttypeww is now known as trown | 11:40 | |
| trown | dtantsur: I think your lio patch broke tripleo | 11:40 |
| trown | dtantsur: also good morning :) | 11:41 |
| dtantsur | trown, yeah, good morning :) | 11:41 |
| openstackgerrit | Merged openstack/python-tripleoclient: Fail the introspection command if introspection has failed https://review.openstack.org/253476 | 11:41 |
| dtantsur | trown, where is it broken and how? | 11:41 |
| trown | dtantsur: so first was the rtslib import... but that was a simple packaging fix | 11:41 |
| *** trown has left #tripleo | 11:41 | |
| *** trown has joined #tripleo | 11:43 | |
| trown | wierd | 11:43 |
| dtantsur | ok, you're back :) | 11:43 |
| dtantsur | so what's the 2nd problem? | 11:43 |
| trown | dtantsur: I was saying before whatever happened there... | 11:43 |
| trown | dtantsur: now it is failing to mount the root device here https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/image.py#L96 | 11:43 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: honor the rabbit user and password provided https://review.openstack.org/255959 | 11:44 |
| trown | that returns "/dev/sda2 already mounted" | 11:44 |
| dtantsur | hmmm | 11:44 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of predictable IPs https://review.openstack.org/241606 | 11:44 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the controller nodes https://review.openstack.org/222497 | 11:45 |
| trown | dtantsur: these images have the issue https://ci.centos.org/artifacts/rdo/images/mitaka/delorean/testing/ | 11:45 |
| gfidente | marios, rebased ^^ thanks for reviewing | 11:45 |
| shardy | gfidente: Hey, FYI the heat patch fixing the neutron net id/name issue we discussed yesterday was just approved | 11:45 |
| shardy | https://review.openstack.org/#/c/257581/2 | 11:45 |
| gfidente | yeah I noticed | 11:45 |
| dtantsur | trown, I'll think what can be done here.. | 11:46 |
| gfidente | shardy, you were pretty fast, I will link that from the BZ | 11:46 |
| gfidente | I think the change to the client is still valid | 11:46 |
| *** mgould has quit IRC | 11:46 | |
| gfidente | so it'd be good to merge them all | 11:46 |
| dtantsur | trown, do you have an upstream bug for it? | 11:46 |
| shardy | gfidente: agreed, the client change is still good, this should just make things a little easier in future | 11:46 |
| trown | dtantsur: cool, I will continue to think on it as well... I am actually wondering if it is some other patch than the lio one since we are past the lio stuff by that point | 11:46 |
| trown | dtantsur: no I am filing one this morning... just got the second issue figured out as I was leaving last night | 11:47 |
| shardy | gfidente: we may be able to handle the DELETE_FAILED issue discussed in bug #1521944 via similar special-casing | 11:48 |
| openstack | bug 1521944 in heat "Failed update attempts might not preserve in heat the actual status of a resource" [Undecided,New] https://launchpad.net/bugs/1521944 | 11:48 |
| shardy | gfidente: I'll look into that later | 11:48 |
| gfidente | shardy, oh the heat change will be backported to liberty as well? | 11:48 |
| shardy | gfidente: Yes, I'll propose it after it lands on master | 11:48 |
| *** SlickNik has quit IRC | 11:50 | |
| *** paramite is now known as paramite|afk | 11:52 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of predictable IPs https://review.openstack.org/241606 | 11:52 |
| *** SlickNik has joined #tripleo | 11:54 | |
| trown | dtantsur: is there a way to get logs from the IPA ramdisk on failure? I have been pausing the virt console with systemd logging to console to see the traceback, but that seems not ideal | 11:55 |
| dtantsur | trown, you can build IPA with dynamic-login element, and use it (I think lucasagomes has a patch up for it) | 11:55 |
| *** paramite|afk is now known as paramite | 11:55 | |
| lucasagomes | trown, dtantsur yeah there's a patch proposed to tripleoclient | 11:56 |
| trown | dtantsur: but for deploy the ramdisk gets shutdown pretty fast | 11:56 |
| lucasagomes | trown, yeah I always edit ironic to avoid the shutdown tho :-( | 11:56 |
| lucasagomes | I had a patch to not shutdown on failures but it was turned down | 11:57 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Fail the introspection command if introspection has failed https://review.openstack.org/257855 | 11:57 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add update yaml backward compatibe with PublicVirtualIP on ctlplane https://review.openstack.org/257856 | 11:57 |
| trown | lucasagomes: hmmm... seems like we should be trying to send back the journal for the agent service, or at least that we should be able to | 11:59 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add update yaml backward compatibe with PublicVirtualIP on ctlplane https://review.openstack.org/257856 | 11:59 |
| dtantsur | folks, when I just issue "openstack overcloud deploy --templates", where does number of nodes come from? | 12:01 |
| openstackgerrit | Jaume Devesa proposed openstack/tripleo-common: Create an overcloud-full function https://review.openstack.org/257859 | 12:01 |
| dtantsur | currently we only validate profiles if --XXX-scale is given, which is not entirely correct | 12:01 |
| lucasagomes | trown, I'm on that as well, gotta find the patch 1 sec | 12:01 |
| lucasagomes | trown, https://review.openstack.org/#/c/248832/ | 12:02 |
| lucasagomes | I will update it | 12:02 |
| lucasagomes | trown, needs the bits in Ironic too to get the logs, problem is where to store it | 12:02 |
| lucasagomes | we can do it in swift or base64 | 12:02 |
| trown | lucasagomes: my vote would be swift | 12:03 |
| lucasagomes | trown, yeah, I want to make it configurable | 12:04 |
| lucasagomes | like the configdrive | 12:04 |
| trown | but that is biased to tripleo since we already have swift setup | 12:04 |
| dtantsur | also folks, is this command even correct http://docs.openstack.org/developer/tripleo-docs/post_deployment/scale_roles.html#scaling-overcloud-roles ? why doesn't it use --compute-flavor? | 12:05 |
| lucasagomes | trown, ++ | 12:05 |
| *** rhallisey has joined #tripleo | 12:06 | |
| trown | dtantsur: it is correct if you did not use --compute-flavor originally... it will default to baremetal flavor | 12:06 |
| dtantsur | trown, aha, so with profile matching we have to use --XX-flavor... it's not error-proof to be honest | 12:07 |
| dtantsur | e.g. people could expect that `openstack overcloud deploy --templates --compute-scale 5` would verify profiles, while it actually won't.. | 12:07 |
| trown | dtantsur: ah right... | 12:08 |
| dtantsur | trown, also this commands seems wrong too: http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/profile_matching.html#use-the-flavors-to-deploy | 12:08 |
| dtantsur | it does not provide --XXX-scale, so again, validation is skipped (and I'm not sure what is actually deployed) | 12:08 |
| trown | dtantsur: ah ya that scale command is definitely wrong... it is use tuskar notation with that '-r Compute-1' | 12:09 |
| dtantsur | trown, no, I meant one above | 12:10 |
| trown | dtantsur: that one I think would be ok... it would deploy the default of 1 control and 1 compute | 12:11 |
| dtantsur | trown, yeah, but profiles won't be checked/processed | 12:11 |
| *** mburned_out is now known as mburned | 12:11 | |
| dtantsur | as it's 0 from validation point of view | 12:11 |
| trown | hmmm... that seems not ideal | 12:12 |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Ceph cluster validation https://review.openstack.org/225248 | 12:12 |
| openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-docs: [WIP] Rewrite completely outdated information on the profile matching https://review.openstack.org/257867 | 12:17 |
| dtantsur | trown, ^^ | 12:17 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities https://review.openstack.org/250405 | 12:19 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Add --dry-run argument to overcloud deploy command https://review.openstack.org/257348 | 12:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities https://review.openstack.org/250405 | 12:22 |
| trown | dtantsur: I filed https://bugs.launchpad.net/ironic-python-agent/+bug/1526304 for the IPA issue... I confirmed that the liberty IPA image works so it has to be LIO that is causing the problem | 12:24 |
| openstack | Launchpad bug 1526304 in ironic-python-agent "Using LIO for ISCSI results in failure to mount root device" [Undecided,New] | 12:24 |
| *** dprince has joined #tripleo | 12:26 | |
| dtantsur | thanks | 12:26 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Fix description of control plane route parameter https://review.openstack.org/257873 | 12:26 |
| dtantsur | trown, if I make a patch, is it possible you test it? | 12:30 |
| trown | dtantsur: ya I can test it | 12:32 |
| *** olap has joined #tripleo | 12:32 | |
| *** jprovazn has quit IRC | 12:34 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays https://review.openstack.org/187506 | 12:39 |
| *** stendulker has joined #tripleo | 12:49 | |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Remove deleted Cinder rows https://review.openstack.org/209019 | 12:50 |
| *** weshay_xchat has joined #tripleo | 12:59 | |
| *** weshay_xchat is now known as weshay | 12:59 | |
| *** jprovazn has joined #tripleo | 12:59 | |
| *** thrash|g0ne is now known as thrash | 13:03 | |
| *** links has quit IRC | 13:09 | |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Adding networking MidoNet element https://review.openstack.org/247382 | 13:10 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Remove Cassandra repository for MidoNet element https://review.openstack.org/256912 | 13:11 |
| *** jayg|g0n3 is now known as jayg | 13:15 | |
| *** regebro has quit IRC | 13:16 | |
| *** regebro has joined #tripleo | 13:16 | |
| *** trown is now known as trown|brb | 13:21 | |
| *** mgould has joined #tripleo | 13:21 | |
| *** masco has quit IRC | 13:22 | |
| *** slagle has quit IRC | 13:25 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 13:26 |
| *** trown|brb is now known as trown | 13:31 | |
| *** whydidyoustealmy has joined #tripleo | 13:35 | |
| *** shakamunyi has joined #tripleo | 13:35 | |
| *** whydidyoustealmy is now known as superflyy | 13:36 | |
| trown | dprince: I think the overcloud swift fix breaks the undercloud install | 13:39 |
| trown | dprince: I do not think $device_array is defined there | 13:40 |
| openstackgerrit | Merged openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/251319 | 13:43 |
| *** hewbrocca has joined #tripleo | 13:45 | |
| *** ukalifon1 has quit IRC | 13:49 | |
| *** links has joined #tripleo | 13:51 | |
| *** slagle has joined #tripleo | 13:52 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Add the missing plan name to the PlanAlreadyExistsError https://review.openstack.org/257343 | 13:54 |
| openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/257908 | 13:56 |
| *** tzumainn has joined #tripleo | 13:56 | |
| *** yamahata has joined #tripleo | 13:58 | |
| openstackgerrit | Miles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state https://review.openstack.org/235158 | 14:00 |
| *** rcernin has joined #tripleo | 14:02 | |
| dprince | meeting time | 14:03 |
| d0ugal | \o/ | 14:04 |
| openstackgerrit | Qasim Sarfraz proposed openstack/tripleo-heat-templates: PLUMgrid Neutron integration https://review.openstack.org/257916 | 14:07 |
| openstackgerrit | Qasim Sarfraz proposed openstack/tripleo-heat-templates: PLUMgrid Neutron integration https://review.openstack.org/257916 | 14:10 |
| *** SlickNik has quit IRC | 14:14 | |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API https://review.openstack.org/230432 | 14:18 |
| *** dmacpher has joined #tripleo | 14:19 | |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Remove deleted Nova rows https://review.openstack.org/207909 | 14:21 |
| *** masco has joined #tripleo | 14:21 | |
| *** SlickNik has joined #tripleo | 14:24 | |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Switch for Keystone DB cron job https://review.openstack.org/209533 | 14:24 |
| *** lblanchard has joined #tripleo | 14:25 | |
| openstackgerrit | Merged openstack/tripleo-common: [stable/liberty]Fix node group name when deleting a node https://review.openstack.org/255955 | 14:26 |
| *** rpothier has joined #tripleo | 14:29 | |
| *** stendulker has quit IRC | 14:32 | |
| *** mgould has quit IRC | 14:35 | |
| *** lazy_prince has quit IRC | 14:40 | |
| *** ChuckC has joined #tripleo | 14:41 | |
| *** qasims has joined #tripleo | 14:44 | |
| *** qasims has quit IRC | 14:44 | |
| *** ChuckC has quit IRC | 14:45 | |
| *** ChuckC has joined #tripleo | 14:46 | |
| *** mgould has joined #tripleo | 14:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-puppet-elements: Add logging components https://review.openstack.org/256640 | 14:51 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-puppet-elements: Add logging components https://review.openstack.org/256640 | 14:52 |
| *** lucasagomes is now known as lucas-hungry | 14:56 | |
| *** links has quit IRC | 14:58 | |
| *** links has joined #tripleo | 14:59 | |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Keystone domain for Heat https://review.openstack.org/180566 | 15:01 |
| *** paramite has quit IRC | 15:19 | |
| *** rleander is now known as leanderthal | 15:22 | |
| *** Goneri has quit IRC | 15:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-docs: Rewrite completely outdated information on the profile matching https://review.openstack.org/257867 | 15:25 |
| *** bnemec has quit IRC | 15:25 | |
| *** tosky has quit IRC | 15:27 | |
| *** ChuckC has quit IRC | 15:27 | |
| *** tosky has joined #tripleo | 15:28 | |
| *** mkovacik has quit IRC | 15:28 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 15:30 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 15:31 |
| *** bnemec has joined #tripleo | 15:32 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Add fixup for pcs order constraints after update to new templates https://review.openstack.org/248863 | 15:35 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 15:35 |
| *** stendulker has joined #tripleo | 15:36 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 15:37 |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-heat-templates: Add capabilities map https://review.openstack.org/242439 | 15:37 |
| *** aufi has quit IRC | 15:39 | |
| *** pradk has joined #tripleo | 15:39 | |
| *** lucas-hungry is now known as lucasagomes | 15:40 | |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-heat-templates: Add capabilities map https://review.openstack.org/242439 | 15:48 |
| *** Marga__ has quit IRC | 15:53 | |
| tzumainn | shardy, hiya! I know you and jtomasek talked about the capabilities map a bunch, does https://review.openstack.org/#/c/242439/ look like what you had in mind? | 15:57 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/257976 | 15:58 |
| jtomasek | tzumainn: did I just +1'd my own patch?:))) | 15:58 |
| *** rcernin has quit IRC | 15:58 | |
| tzumainn | jtomasek, lol, apparently so! | 15:59 |
| *** jaosorior has joined #tripleo | 16:03 | |
| shardy | tzumainn: Yup, lgtm | 16:04 |
| shardy | tzumainn: one thought, we probably need a test pretty soon after we start using it, e.g a check job which enforces that any new environment/foo.yaml has an entry in there | 16:04 |
| shardy | and that nothing gets removed or renamed if it's in a requires: list | 16:04 |
| tzumainn | shardy, that makes sense | 16:05 |
| *** dtantsur is now known as dtantsur|afk | 16:06 | |
| *** sthillma has joined #tripleo | 16:08 | |
| *** Goneri has joined #tripleo | 16:09 | |
| *** sthillma_ has joined #tripleo | 16:10 | |
| *** sthillma has quit IRC | 16:12 | |
| *** sthillma_ is now known as sthillma | 16:12 | |
| jaosorior | bnemec, are you around? | 16:19 |
| bnemec | shardy: tzumainn jtomasek: https://review.openstack.org/#/c/253638/ for automatically generating those environment files. | 16:21 |
| bnemec | We may want to combine the two. | 16:21 |
| bnemec | jaosorior: Yes | 16:21 |
| jtomasek | bnemec: thanks! I'll take a look | 16:22 |
| jaosorior | bnemec: Sorry to ping you so much, but you were the one working with TLS too, so yeah :/ ... Anyway, since it seems we're not gonna have the keystone endpoints configured through puppet any time soon, I have this patch to enable them through os-cloud-config in the python-tripleoclient, if you have time during the day, can you take a look? https://review.openstack.org/#/c/254613/3 | 16:23 |
| *** Marga_ has joined #tripleo | 16:25 | |
| bnemec | jaosorior: I'll try. My plan for today is to try to work through my upstream review backlog, so I'll hopefully get to it. | 16:26 |
| jaosorior | bnemec: thanks man | 16:26 |
| *** yamahata has quit IRC | 16:27 | |
| *** stendulker has quit IRC | 16:33 | |
| shardy | bnemec: if you want some easy stable/liberty ones to start with, there's some ready to +A here: | 16:34 |
| shardy | https://etherpad.openstack.org/p/tripleo-reviews-ready-to-land | 16:34 |
| *** dshulyak has joined #tripleo | 16:36 | |
| *** tremble has quit IRC | 16:40 | |
| *** jcoufal has quit IRC | 16:43 | |
| *** ifarkas has quit IRC | 16:44 | |
| *** mgould has quit IRC | 16:45 | |
| *** dshulyak has quit IRC | 16:45 | |
| lucasagomes | hi folks, if you have time mind taking a look at this patch? https://review.openstack.org/#/c/257369/ this is a 3 line patch (incl. test) so it should be very quick | 16:46 |
| trown | dprince: for your comment that we should not be using swift::ringbuilder class in the undercloud, that would mean not having swift in the undercloud no? | 16:47 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Inject TLS certificate and keys for the Overcloud https://review.openstack.org/254831 | 16:53 |
| *** yamahata has joined #tripleo | 16:53 | |
| *** lazy_prince has joined #tripleo | 16:54 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Enable trust anchor injection https://review.openstack.org/254832 | 16:55 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Output the SSL Certificate and Key modulus https://review.openstack.org/254833 | 16:56 |
| *** cwolferh has joined #tripleo | 16:56 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Minor fixes to TLS related resources https://review.openstack.org/254834 | 16:56 |
| jaosorior | yay :D | 16:56 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 16:58 |
| *** david-lyle has quit IRC | 17:02 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Update docker compute environment to use json config https://review.openstack.org/254408 | 17:04 |
| dprince | trown: we use swift::ringbuilder | 17:04 |
| dprince | trown: but not tripleo::ringbuilder | 17:04 |
| dprince | trown: thus I don't think the overcloud change should effect us directly | 17:05 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add local docker registry support https://review.openstack.org/254409 | 17:06 |
| *** akrivoka has quit IRC | 17:06 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Nova-libvirt needs to bind to /sys/fs/cgroup https://review.openstack.org/254410 | 17:06 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Change the Atomic image name so it's less specific https://review.openstack.org/254411 | 17:07 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Point registry at tripleoupstream https://review.openstack.org/254412 | 17:07 |
| trown | dprince: I am confused... https://review.openstack.org/#/c/255995/4/elements/puppet-stack-config/puppet-stack-config.yaml.template defitely uses swift::ringbuilder | 17:08 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 17:09 |
| trown | dprince: and https://review.openstack.org/#/c/257540/ definitely changes swift::ringbuilder | 17:09 |
| dprince | trown: right, but that shouldn't effect the undercloud | 17:10 |
| dprince | trown: sorry, this morning I thought you said the overcloud change broke the undercloud | 17:10 |
| trown | dprince: the first change there is to the undercloud puppet-stack-config element... | 17:10 |
| trown | dprince: if I manually apply the puppet-swift change, undercloud install fails | 17:11 |
| dprince | trown: got a paste of the error | 17:11 |
| trown | dprince: I can recreate it, one sec | 17:11 |
| dprince | trown: because setting replicas = 1 in the undercloud fixes it for me | 17:11 |
| openstackgerrit | Miles Gould proposed openstack/python-tripleoclient: Support new ironic "enroll" state https://review.openstack.org/235158 | 17:12 |
| trown | dprince: have you tried to apply the puppet-swift change before running the undercloud install? | 17:12 |
| *** david-lyle has joined #tripleo | 17:12 | |
| *** mgould has joined #tripleo | 17:12 | |
| openstackgerrit | Miles Gould proposed openstack/python-tripleoclient: Support new ironic "enroll" state https://review.openstack.org/235158 | 17:13 |
| dprince | trown: what puppet-swift change? | 17:13 |
| dprince | trown: I'm using puppet-swift master... | 17:13 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh https://review.openstack.org/241167 | 17:13 |
| trown | dprince: ugh.... just realized my error... your change is in tht not puppet-swift | 17:15 |
| dprince | trown: correct | 17:15 |
| trown | I was patching puppet-swift which breaks undercloud, but I think we are good | 17:16 |
| *** mcornea has quit IRC | 17:16 | |
| *** thrash is now known as thrash|f00dz | 17:16 | |
| *** trown is now known as trown|lunch | 17:19 | |
| *** chem has joined #tripleo | 17:19 | |
| chem | hi, I've done a modification on a OPM module and I would like to rerun the puppet apply portion of the heat template, is there a way to do that without "updating" the stack ? | 17:20 |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-heat-templates: Sample environment generator https://review.openstack.org/253638 | 17:20 |
| *** derekh has quit IRC | 17:21 | |
| *** masco has quit IRC | 17:25 | |
| *** lazy_prince has quit IRC | 17:29 | |
| *** dshulyak has joined #tripleo | 17:33 | |
| chem | so no way to do a puppet apply directly on the concerned node ? | 17:41 |
| *** rebrego has joined #tripleo | 17:50 | |
| *** regebro has quit IRC | 17:50 | |
| *** rebrego is now known as regebro | 17:50 | |
| *** rasca has quit IRC | 17:50 | |
| openstackgerrit | Ben Nemec proposed openstack/instack-undercloud: Remove option of installing tuskar https://review.openstack.org/258028 | 17:50 |
| *** links has quit IRC | 17:50 | |
| *** shivrao has joined #tripleo | 17:56 | |
| openstackgerrit | Ben Nemec proposed openstack/instack-undercloud: Allow setting the hostname during undercloud install https://review.openstack.org/249457 | 17:59 |
| *** devvesa has quit IRC | 18:04 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add capabilities map https://review.openstack.org/242439 | 18:06 |
| lucasagomes | bnemec, hi there, re 257369. It's the other way around. It will not work with local boot | 18:09 |
| lucasagomes | because we have no control over the kernel cmdline when local boot is used (it's written in the grub configuration in the disk) | 18:09 |
| bnemec | lucasagomes: Why not? Can't I stop at the grub prompt and add the kernel cmdline option? | 18:10 |
| bnemec | I guess it would be a headache to type an entire encrypted password in that way, but it should work. | 18:10 |
| *** nico_auvv has quit IRC | 18:10 | |
| lucasagomes | bnemec, yeah you can | 18:10 |
| bnemec | And if you're using a remote KVM that does copy-paste it wouldn't be too bad. | 18:10 |
| lucasagomes | ok if that's acceptable I'm alright adding it to common | 18:10 |
| lucasagomes | cool | 18:11 |
| bnemec | lucasagomes: Yeah, I think it's good. Like I said, we get people requesting this all the time so I'm very happy to see it go in. | 18:11 |
| lucasagomes | bnemec, cool. I will edit the patch | 18:12 |
| lucasagomes | I was just making sure we are on the same page about hw it works | 18:12 |
| lucasagomes | but yeah if you have console access you can edit the grub before booting and set it up | 18:12 |
| *** Marga_ has quit IRC | 18:18 | |
| *** jistr has quit IRC | 18:22 | |
| EmilienM | slagle: can you consider https://review.openstack.org/#/c/234704/ again please? | 18:24 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Deploy Monitoring on the undercloud with Puppet https://review.openstack.org/253709 | 18:24 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Deploy Aodh services, replacing Ceilometer Alarm https://review.openstack.org/241412 | 18:25 |
| *** gfidente is now known as gfidente|afk | 18:26 | |
| slagle | EmilienM: looking now | 18:28 |
| EmilienM | thanks! | 18:28 |
| *** rasca has joined #tripleo | 18:29 | |
| openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila https://review.openstack.org/188138 | 18:32 |
| openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila https://review.openstack.org/230936 | 18:32 |
| *** mgould is now known as mgould|afk | 18:36 | |
| *** olap has quit IRC | 18:38 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks https://review.openstack.org/235423 | 18:39 |
| *** Guest73233 is now known as mgagne | 18:42 | |
| slagle | EmilienM: i'm getting an error still, http://paste.openstack.org/show/481987/ | 18:42 |
| *** mgagne is now known as Guest76434 | 18:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-cloud-config: Updated from global requirements https://review.openstack.org/247624 | 18:43 |
| slagle | EmilienM: here are the endpoints i have, http://paste.openstack.org/show/481988/ | 18:43 |
| EmilienM | slagle: ok, looking | 18:44 |
| EmilienM | slagle: why do you have dupplicates in your endpoints? | 18:46 |
| *** trown|lunch is now known as trown | 18:46 | |
| EmilienM | because of this patch? | 18:47 |
| slagle | i guess so | 18:47 |
| EmilienM | slagle: I wonder if adding Keystone_service<||> -> Keystone_endpoint<||> would help | 18:48 |
| EmilienM | I thought no, but in fact we might need it | 18:48 |
| EmilienM | I'm going to try it | 18:49 |
| slagle | ok, so in the ironic duplicate, there is this difference, http://192.0.2.1:6385/ vs http://192.0.2.1:6385 | 18:49 |
| slagle | the trailing slash is the only difference | 18:49 |
| slagle | are the puppet modules looking up urls to see if an endpoint already exists? | 18:49 |
| EmilienM | yes but it needs to match a uniq service name | 18:50 |
| EmilienM | but from what i'm seeing we might check something else | 18:50 |
| slagle | can it be less strict a bout the trailing slash? | 18:50 |
| EmilienM | slagle: chem did that recently: https://github.com/openstack/puppet-keystone/commit/0a4e06abb0f5b3f324464ff5219d2885816311ce | 18:51 |
| chem | EmilienM: catching up, what is the problem ? | 18:51 |
| EmilienM | chem: still trying to make upgrades pass with https://review.openstack.org/#/c/234704/ | 18:52 |
| chem | EmilienM: slagle the prbolem is the double tuskar entry ? in the paste http://paste.openstack.org/show/481988/ | 18:53 |
| EmilienM | chem: when we run puppet, it created dupplicated endpoints | 18:53 |
| EmilienM | yes and ceilometer and tuskar and ironic | 18:53 |
| chem | EmilienM: nope :) | 18:53 |
| slagle | the only difference i see between the duplicates is the trailing slash in the url's | 18:54 |
| jpeeler | thrash|f00dz: have you been in contact with anybody about CI difficulties? it looks like the latest CI failures aren't due to the patch | 18:55 |
| *** jtomasek has quit IRC | 18:55 | |
| EmilienM | slagle: are you able to retry again by adding Keystone_service<||> -> Keystone_endpoint<||> in the manifest? | 18:55 |
| EmilienM | I remember dprince suggested me to add it | 18:55 |
| dprince | slagle: yeah, I'd be curious if that fixes it for you | 18:55 |
| EmilienM | but I thought our provider would do the job with Autorequire | 18:55 |
| *** tosky has quit IRC | 18:56 | |
| slagle | EmilienM: i can try, but i'd need to rollback somewhow to what i had before | 18:56 |
| slagle | i dont have a backup of the keystone db | 18:56 |
| slagle | i dont know which endpoints puppet created, the ones with or without the slash | 18:56 |
| slagle | i can try it both ways i guess though | 18:56 |
| dprince | slagle: do you have a paste of the error you got? | 18:57 |
| EmilienM | dprince: http://paste.openstack.org/show/481987/ | 18:57 |
| EmilienM | Error: Execution of '/bin/openstack endpoint create --format shell nova admin http://192.0.2.1:8774/v2/%(tenant_id)s --region regionOne' returned 1: No service with a type, name or ID of 'nova' exists. | 18:57 |
| dprince | EmilienM: yeah, it sounds like it isn't running early enough | 18:57 |
| dprince | EmilienM: lets put in the resource chain explicitly and see if it fixes it | 18:58 |
| EmilienM | ok | 18:58 |
| dprince | EmilienM: that was what I tested (initially). Perhaps there are subtle catalog ordering issues where sometimes it works without it | 18:58 |
| dprince | EmilienM: but making the relationship explicit would make it always work | 18:59 |
| EmilienM | ok | 18:59 |
| EmilienM | the bot is dead but I sent a patch | 19:00 |
| chem | dprince: EmilienM slagle in the last puppet keystone_endpoint the autorequire should work ok and the duplicate entry shoudn't exist... I wonder what is missing | 19:00 |
| chem | slagle: could you paste puppet resource keystone_endpoint and puppet resource keystone_service at the time you get the error ? | 19:01 |
| dprince | chem: perhaps it autorequires another (specific) instance of the service resource. Where as in our case we want to make sure all keystone_service deletes occur before puppet tries to re-add subsequent services and endpoints | 19:02 |
| slagle | chem: what do you mean at the time i get the error? | 19:02 |
| slagle | the puppet apply of the manifest failed, can i run those now and show you the output? | 19:02 |
| chem | slagle: yep | 19:03 |
| slagle | chem: http://paste.openstack.org/show/481995/ | 19:04 |
| chem | dprince: you shoudn't need to delete and recreate, the keystone_endpoint should detect the existing one and do recreate them again. I must be missing something here | 19:04 |
| chem | slagle: thanks | 19:04 |
| *** pblaho_ has joined #tripleo | 19:05 | |
| *** pino|work_ has joined #tripleo | 19:05 | |
| dprince | chem: okay, so that functionality isn't working | 19:05 |
| *** rleander_ has joined #tripleo | 19:05 | |
| dprince | chem: I think because in tripleo there were two endpoints tied to a service named 'nova' | 19:05 |
| chem | dprince: the latest keystone_endpoint shouldn't be fooled anymore by this | 19:06 |
| dprince | chem: I tested this just over the weekend (latest keystone_endpoint code then) | 19:06 |
| *** dtantsur|afk has quit IRC | 19:06 | |
| dprince | chem: I think the bug is that TripleO creates two endpoints for the same service/type | 19:07 |
| *** pblaho has quit IRC | 19:08 | |
| dprince | chem: so while the keystone_endpoint handles updates nicely, in this case it blows up because it can't determine which one to update | 19:08 |
| *** athomas has quit IRC | 19:08 | |
| *** hewbrocca has quit IRC | 19:08 | |
| *** pino|work has quit IRC | 19:08 | |
| *** morazi has quit IRC | 19:08 | |
| *** leanderthal has quit IRC | 19:08 | |
| *** eil397 has joined #tripleo | 19:09 | |
| chem | dprince: could you show me the two "endpoint creation" part in tripleo ? | 19:09 |
| dprince | chem: yep, just a sec... | 19:09 |
| *** hewbrocca has joined #tripleo | 19:09 | |
| *** sthillma has quit IRC | 19:09 | |
| dprince | chem: http://git.openstack.org/cgit/openstack/os-cloud-config/tree/os_cloud_config/keystone.py#n70 | 19:10 |
| chem | dprince: thanks | 19:10 |
| dprince | chem: note that the novav3 endpoint uses the same service name (for TripleO) as the normal nova endpoint | 19:10 |
| dprince | chem: I think that case is what keystone_endpoint isn't handling. The explicit delete seemed (to me) to be the cleanest way to handle this | 19:11 |
| dprince | EmilienM: the latest version of this patch removes all of the fixes? https://review.openstack.org/#/c/234704/10 | 19:12 |
| *** rleander_ has quit IRC | 19:12 | |
| dprince | EmilienM: oh, nm. I was on a previous patch! | 19:12 |
| *** pino|work has joined #tripleo | 19:12 | |
| EmilienM | ok | 19:12 |
| *** pino|work_ has quit IRC | 19:13 | |
| *** pblaho_ has quit IRC | 19:13 | |
| dprince | EmilienM: yeah, that should work. It did for me | 19:13 |
| *** dtantsur has joined #tripleo | 19:13 | |
| dprince | by my testing over the weekend... | 19:13 |
| chem | dprince: to match this you have no choice but specify regionOne/nova::computev3 and regionOne/nova::compute as kestone_endpoint name (or add the type parameter to the resource) | 19:13 |
| chem | dprince: the patch made this possible while it wasn't before | 19:14 |
| chem | dprince: it then fetch the correct service by "id" not by name | 19:14 |
| EmilienM | dprince: that's why I sent a new patchset with the orchestration and I wondered if slagle could also check | 19:14 |
| dprince | chem: right, but the nova::keystone::auth manifests forces us to use unique services for nova vs. novav3 though | 19:14 |
| *** pblaho has joined #tripleo | 19:14 | |
| *** penick has joined #tripleo | 19:14 | |
| dprince | chem: I tried changing the parameters to nova::keystone::auth to match what you suggested... and it would work except for the fact that nova::keystone::auth throws a validation error for the previous TripleO default case | 19:15 |
| dprince | chem: http://git.openstack.org/cgit/openstack/puppet-nova/tree/manifests/keystone/auth.pp#n259 | 19:16 |
| chem | dprince: ack. It was taking into account the fact that it was not supported. This should be removed... anyway this is going down the rabbit hole. | 19:19 |
| dprince | chem: sorry, appreciate you trying to help us sort this out | 19:19 |
| slagle | EmilienM: ok, should i manually clean up the dupes first before i try the latest patchset? | 19:19 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/python-tripleoclient: Add dynamic-login element to the image creation https://review.openstack.org/257369 | 19:20 |
| chem | slagle: you should | 19:20 |
| EmilienM | yes so we have the previous state | 19:20 |
| slagle | ok, i'm not sure which ones puppet created | 19:20 |
| slagle | i'll guess | 19:20 |
| *** ukalifon1 has joined #tripleo | 19:20 | |
| chem | slagle: this paste http://paste.openstack.org/show/481995/ is comming from the platform where you have the duplicate ? | 19:21 |
| slagle | yes | 19:21 |
| dprince | slagle: my guess is running the new patch will just work | 19:21 |
| dprince | slagle: but I'm more interested in if it fixes the clean upgrade... | 19:21 |
| dprince | slagle: because it did for me | 19:22 |
| slagle | i'll try it a couple different ways | 19:22 |
| *** jaosorior has quit IRC | 19:23 | |
| *** shardy is now known as shardy_afk | 19:23 | |
| *** sthillma has joined #tripleo | 19:24 | |
| dprince | slagle: cool, thanks for trying this | 19:24 |
| *** rleander_ has joined #tripleo | 19:25 | |
| chem | dprince: I have this patch as a quick workaround this nova/{compute/computev3} issue at the ready that would solve all this : https://review.openstack.org/#/c/242198/ | 19:25 |
| chem | dprince: it just lacks another +2 | 19:26 |
| dprince | chem: cool, let me check it out then | 19:26 |
| dprince | chem: would that delete the old one though? | 19:27 |
| chem | dprince: not sure I understand ? | 19:27 |
| chem | dprince: what would delete what ? | 19:28 |
| dprince | chem: it fixes it if someone installs a new version and upgrades, but that doesn't delete the previous nova/computev3 service right? | 19:29 |
| chem | dprince: nope. It would only work on fresh install and "fresh" update. I see your problematic here... | 19:30 |
| *** gfidente|afk has quit IRC | 19:30 | |
| *** openstackgerrit has quit IRC | 19:32 | |
| *** openstackgerrit has joined #tripleo | 19:32 | |
| *** rwsu has quit IRC | 19:39 | |
| chem | dprince: oki, on my side, I'll will report the nova::keystone::auth problem (and maybe others *::keystone::auth modules) and try to solve it. It's no fun to have feature if the wrapper classes do not support them. | 19:41 |
| *** jaosorior has joined #tripleo | 19:42 | |
| *** jprovazn has quit IRC | 19:42 | |
| dprince | chem: fair enough. all of the wrappers worked for us except for nova. My view is these wrappers really work well for common cases and what we were dealing with was just a one-off mismatch error | 19:43 |
| dprince | chem: if it is easy to improve the wrappers in this regard... then fine but I think they already perform quite acceptably | 19:43 |
| *** dshulyak has quit IRC | 19:44 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests https://review.openstack.org/244162 | 19:46 |
| *** dprince has quit IRC | 19:47 | |
| *** electrofelix has quit IRC | 19:48 | |
| openstackgerrit | yolanda.robla proposed openstack/diskimage-builder: Split vm and bootloader elements https://review.openstack.org/258126 | 19:49 |
| *** olap has joined #tripleo | 19:52 | |
| slagle | EmilienM: k, it seems to be working fine now | 19:56 |
| slagle | +2'd | 19:57 |
| EmilienM | dan was right then | 19:57 |
| EmilienM | dp | 19:57 |
| EmilienM | slagle: thanks... for your help | 19:57 |
| slagle | i'll mash the big red button once it passes ci | 19:57 |
| EmilienM | very nice | 19:57 |
| EmilienM | a nice step that will us to make progress | 19:57 |
| EmilienM | thanks for your help chem, dprince & slagle | 19:57 |
| *** sthillma has quit IRC | 20:00 | |
| *** thrash|f00dz is now known as thrash|brb | 20:09 | |
| *** rpothier has left #tripleo | 20:12 | |
| *** jaosorior has quit IRC | 20:12 | |
| *** lucasagomes is now known as lucas-dinner | 20:17 | |
| *** rwsu has joined #tripleo | 20:19 | |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-docs: Document installing the undercloud with SSL endpoints https://review.openstack.org/221891 | 20:23 |
| *** Marga_ has joined #tripleo | 20:25 | |
| *** sthillma has joined #tripleo | 20:26 | |
| openstackgerrit | Ben Nemec proposed openstack/python-tripleoclient: Always install yum-plugin-priorities https://review.openstack.org/242169 | 20:30 |
| *** sthillma has quit IRC | 20:31 | |
| *** jtomasek has joined #tripleo | 20:32 | |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-common: Remove yum-plugin-priorities hack https://review.openstack.org/242228 | 20:32 |
| openstackgerrit | Jeff Peeler proposed openstack/instack-undercloud: Disable auto activation of all volumes https://review.openstack.org/258139 | 20:38 |
| *** Marga_ has quit IRC | 20:44 | |
| openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: DO NOT MERGE changes needed for delorean promotion https://review.openstack.org/258140 | 20:45 |
| *** lblanchard has quit IRC | 20:56 | |
| *** Marga_ has joined #tripleo | 21:01 | |
| *** jcoufal has joined #tripleo | 21:05 | |
| *** thrash|brb is now known as thrash | 21:07 | |
| *** ukalifon1 has quit IRC | 21:10 | |
| openstackgerrit | Merged openstack/tripleo-docs: Document using a pending review https://review.openstack.org/226570 | 21:10 |
| *** jaosorior has joined #tripleo | 21:11 | |
| *** lblanchard has joined #tripleo | 21:22 | |
| *** sthillma has joined #tripleo | 21:26 | |
| *** mkovacik has joined #tripleo | 21:29 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use renamed overcloud.yaml template https://review.openstack.org/253354 | 21:30 |
| *** pradk has quit IRC | 21:33 | |
| *** Guest76434 is now known as mgagne | 21:33 | |
| *** mgagne is now known as Guest160 | 21:34 | |
| *** Guest160 has quit IRC | 21:34 | |
| *** Guest160 has joined #tripleo | 21:34 | |
| *** Guest160 is now known as mgagne | 21:35 | |
| *** cwolferh has quit IRC | 21:38 | |
| *** jaosorior has quit IRC | 21:41 | |
| *** Marga_ has quit IRC | 21:41 | |
| *** dshulyak has joined #tripleo | 21:43 | |
| *** mcornea has joined #tripleo | 21:44 | |
| *** jcoufal has quit IRC | 21:45 | |
| *** jayg is now known as jayg|g0n3 | 21:55 | |
| *** david-lyle has quit IRC | 21:58 | |
| *** lblanchard has quit IRC | 21:58 | |
| *** jtomasek has quit IRC | 21:59 | |
| *** david-lyle has joined #tripleo | 22:00 | |
| bnemec | dsneddon: I just left a comment on https://review.openstack.org/#/c/226057/ | 22:03 |
| bnemec | I didn't really want to -1 it at this point for such a minor thing, but it does need to be fixed at some point. | 22:03 |
| *** trown is now known as trown|outttypeww | 22:04 | |
| dsneddon | bnemec, Might as well fix it now, I lost all reviews when I had to rebase it. | 22:05 |
| dsneddon | bnemec, On the other hand, as soon as I fix this, I really want to get this merged. It's too big a rebase magnet. | 22:05 |
| bnemec | dsneddon: I'm still showing a +2 from shardy_afk on that one. | 22:05 |
| dsneddon | bnemec, Oh, you're right, he did review it a few hours ago. | 22:05 |
| bnemec | Although, honestly at this point I think that's a candidate for a fast merge. | 22:06 |
| bnemec | Everyone's been basically good with it for a while, it just keeps hitting rebase problems. | 22:06 |
| dsneddon | bnemec, I'm going to fix the minor issue, and send shardy a note. Thanks. | 22:06 |
| *** rcernin has joined #tripleo | 22:16 | |
| *** mkovacik has quit IRC | 22:18 | |
| *** shardy_afk has quit IRC | 22:23 | |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-docs: Document installing the undercloud with SSL endpoints https://review.openstack.org/221891 | 22:30 |
| *** mcornea has quit IRC | 22:38 | |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics templates https://review.openstack.org/252683 | 22:40 |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-heat-templates: Rename validate tox env to pep8 https://review.openstack.org/258173 | 22:40 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration. https://review.openstack.org/226057 | 22:41 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add all isolated networks to all nodes. https://review.openstack.org/249533 | 22:42 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics templates https://review.openstack.org/252683 | 22:44 |
| dsneddon | bnemec, Sorry to be rude, but I just rebased your change on top of my management network change. | 22:45 |
| dsneddon | bnemec, Thought you wouldn't mind, since you convinced me to fix it. ;-) | 22:45 |
| bnemec | dsneddon: Yeah, no problem. :-) I was trying to be really careful not to mess up your commit when I pushed those. | 22:52 |
| *** david-lyle has quit IRC | 23:01 | |
| *** david-lyle has joined #tripleo | 23:01 | |
| *** yuanying has joined #tripleo | 23:02 | |
| *** Goneri has quit IRC | 23:03 | |
| *** yuanying has quit IRC | 23:03 | |
| *** yuanying has joined #tripleo | 23:04 | |
| *** jtomasek has joined #tripleo | 23:11 | |
| *** mburned is now known as mburned_out | 23:15 | |
| *** jtomasek has quit IRC | 23:16 | |
| *** rcernin has quit IRC | 23:20 | |
| openstackgerrit | Merged openstack/diskimage-builder: Add dib-python element https://review.openstack.org/256071 | 23:24 |
| *** stevebaker has quit IRC | 23:30 | |
| *** stevebaker has joined #tripleo | 23:31 | |
| *** david-lyle_ has joined #tripleo | 23:33 | |
| *** david-lyle has quit IRC | 23:36 | |
| openstackgerrit | Abel Lopez proposed openstack/diskimage-builder: Document byte-to-inode ratio https://review.openstack.org/258195 | 23:47 |
| *** sthillma has quit IRC | 23:52 | |
| *** mburned_out is now known as mburned | 23:53 | |
| *** chlong has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!