Tuesday, 2015-12-15

*** sthillma has joined #tripleo00:02
*** jcoufal has quit IRC00:16
*** rasca has joined #tripleo00:16
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration.  https://review.openstack.org/22605700:17
*** penick has quit IRC00:21
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add all isolated networks to all nodes.  https://review.openstack.org/24953300:25
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration.  https://review.openstack.org/22605700:25
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics  https://review.openstack.org/25268300:27
*** eil397 has quit IRC00:39
*** dmacpher has joined #tripleo00:50
*** openstack has joined #tripleo01:06
*** mbound_ has quit IRC01:07
*** julim has joined #tripleo01:28
*** cwolferh has quit IRC01:56
*** shivrao has quit IRC02:05
*** eil397 has joined #tripleo02:08
*** sthillma has quit IRC02:54
openstackgerritSteve Baker proposed openstack/tripleo-heat-templates: Add SoftwareConfigTransport for switching transports  https://review.openstack.org/25059302:54
openstackgerritSteve Baker proposed openstack/tripleo-heat-templates: Switch to POLL_TEMP_URL for config transport  https://review.openstack.org/25765702:54
*** yamahata has quit IRC03:05
*** rhallisey has quit IRC03:05
*** yuanying has quit IRC03:20
*** lazy_prince has joined #tripleo03:48
*** david-lyle has quit IRC03:50
*** david-lyle has joined #tripleo03:58
*** david-lyle has quit IRC04:01
*** david-lyle has joined #tripleo04:02
*** yuanying has joined #tripleo04:07
*** links has joined #tripleo04:07
*** eil397 has quit IRC04:12
*** yamahata has joined #tripleo04:19
*** openstackstatus has quit IRC04:24
*** openstack has joined #tripleo04:25
*** openstackstatus has joined #tripleo04:26
*** ChanServ sets mode: +v openstackstatus04:26
*** tzumainn has quit IRC05:06
openstackgerritGilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function  https://review.openstack.org/24564305:23
*** masco has joined #tripleo05:49
openstackgerritGilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function  https://review.openstack.org/24564306:09
*** aufi has joined #tripleo06:32
*** trembleWork has quit IRC07:07
*** stendulker has joined #tripleo07:10
*** dmacpher has quit IRC07:13
*** ukalifon1 has joined #tripleo07:29
*** jprovazn has joined #tripleo07:35
*** tremble has joined #tripleo07:57
*** tremble has joined #tripleo07:57
*** yuanying has quit IRC08:05
*** mkovacik has joined #tripleo08:15
*** shivrao has joined #tripleo08:25
*** yamahata has quit IRC08:26
*** shivrao has quit IRC08:27
*** athomas has joined #tripleo08:28
*** rleander has joined #tripleo08:29
*** jcoufal has joined #tripleo08:30
*** ifarkas has joined #tripleo08:34
*** shivrao has joined #tripleo08:36
*** shardy has joined #tripleo08:37
*** morazi has joined #tripleo08:50
*** mbound_ has joined #tripleo08:51
shardymarios: Hey g'morning - any chance you can review/approve these two please?08:51
openstackgerritShu Muto proposed openstack/python-tripleoclient: Drop py33 support  https://review.openstack.org/25774008:52
shardymidonet related stable backports, both passing CI08:52
*** mbound_ has quit IRC08:52
*** mbound_ has joined #tripleo08:52
mariosshardy: sure looking#08:53
*** jcoufal has quit IRC08:56
*** jistr has joined #tripleo08:56
*** jcoufal has joined #tripleo08:57
openstackgerritMerged openstack/tripleo-heat-templates: Add simple linux bridge as network environment  https://review.openstack.org/24737908:58
shardymarios: thanks - hey I was wondering, what's the status of https://review.openstack.org/#/c/241167/ ?08:59
shardythat could be pretty useful in the context of upgrade testing08:59
mariosshardy: :( I know. every dayi plan to work on it, then blockers.09:00
shardyIt'd be cool if we could translate that into launching the VM via heat, to gain a little more API coverage09:00
shardymarios: No worries, I figured that may be the case :(09:00
mariosshardy: i will try today to get somethign quick out to iterate on09:00
shardymarios: no pressure, I was just wondering that's all09:00
mariosshardy: has been quiet but still early in the day09:00
shardymarios: also, I left a comment re fedora/cirros09:00
mariosshardy: ack thanks09:00
shardyour experience with images in heat CI is the cirros image boots way, way faster09:01
shardyso maybe worth considering09:01
mariosshardy: so i played with it09:01
mariosshardy: i had this discussino with derekh09:01
marioswho also preferred cirros09:01
mariosshardy: but i consistently had issues pinging into the cirros,09:01
mariosshardy: at the time i didn't investigate further09:01
mariosshardy: especially cos, we build the fedora-user anyway and it is there09:02
shardymarios: ack, Ok that's weird09:02
mariosshardy: cirros we need to download, so there is an element of network issues09:02
mariosbut yeay, i don't mind really09:02
mariosso i did try it09:02
shardyI guess we can always start with fedora then optimize, provided the performance hit isn't too huge09:02
shardyI've used cirros a lot locally and never had a problem09:02
*** olap has quit IRC09:04
mariosshardy: right, not sure what was up. i can try it again when i get back to it09:04
shardymarios: kk, cool, thanks for the update :)09:04
*** julim has quit IRC09:10
*** julim has joined #tripleo09:10
openstackgerritShu Muto proposed openstack/python-tuskarclient: Drop py33 support  https://review.openstack.org/25775409:13
*** shivrao has quit IRC09:22
shardySimple tripleo.sh review which will enable testing deletes in CI:09:32
openstackgerritMerged openstack/tripleo-heat-templates: Set swift replicas = min(device_count, replicas)  https://review.openstack.org/25754009:32
*** mcornea has joined #tripleo09:34
*** nico_auvv has joined #tripleo09:42
*** devvesa has joined #tripleo09:43
devvesaHi. Do you have any idea why this patch is on workflow+1 since yesterday but not merged?09:46
devvesaCan it be because the Depends-On is related on two reviews (master and stable/liberty)09:46
*** openstackgerrit has quit IRC09:47
*** openstackgerrit has joined #tripleo09:47
*** akrivoka has joined #tripleo09:49
*** jtomasek has joined #tripleo09:51
openstackgerritMerged openstack/instack-undercloud: Remove tripleo::ringbuilder cruft  https://review.openstack.org/25663109:53
jistrdevvesa: 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
devvesajistr: Great. Thanks for the rechecks shardy :)10:01
*** derekh has joined #tripleo10:01
*** dtantsur|afk is now known as dtantsur10:02
*** mgould has joined #tripleo10:11
shardydevvesa: Yes, Depends-On is not branch aware, which is why I generally wait until all patches have landed to master before proposing stable backports10:13
shardythat also allows for a little time to work out any issues on master, vs immediately backporting10:13
mgouldhi everyone!10:14
shardyHi mgould!10:15
mgouldmy patch to python-tripleoclient is failing the tripleo CI gate; what's the best way for me to duplicate the failure locally?10:15
devvesashardy: No problem. As long as I understand there's nothing wrong on the patches or the CI gates, it is ok10:15
mgouldI do not currently have a working install of TripleO10:15
mgouldshardy, hi!10:15
devvesamgould: your patch has failed on the overcloud10:17
devvesaCan it be unrelated to your patch?10:18
devvesaI see a 'No hosts matched' on the Jenkins jobs10:18
mgoulddevvesa, there was one error that looked unrelated, but another which looked related to my patch10:18
shardymgould: for developers, most folks are using a VM setup, as documented here:10:19
shardymgould: there is a shell script which automates some steps after "instack-virt-setup"10:19
shardythat is what we use in CI, so many folks will use that to reproduce gate issues10:19
mgouldthe line https://review.openstack.org/#/c/235158/ just before failure is logging from my code10:20
mgouldshardy, awesome10:20
shardymgould: there is a docs patch for tripleo.sh, but it seems to have stalled:10:21
mgouldshardy, excellent10:21
openstackgerritDougal Matthews proposed openstack/tripleo-common: Drop py33 support  https://review.openstack.org/25780810:23
mgouldshardy, so I clone that repo onto a clean VM and run the script?10:23
shardymgould: no, follow the virtual-environment docs until you've run "instack-virt-setup"10:23
shardythen, ssh onto the instack VM, and clone tripleo-common10:23
mgouldright, thanks10:24
shardyafter that you can run triple.sh and follow the options in the order they are listed10:24
*** electrofelix has joined #tripleo10:24
shardymgould: There is a "--all" option, but I prefer to run only the following steps, manually, as it's easier if something goes wrong10:25
shardy--repo-setup, then --undercloud, then --overcloud-images then --register-nodes, then --flavors then --overcloud-deploy10:26
shardymgould: the --all does some optional steps like node introspection which aren't strictly necessary for development usage10:26
shardyalso, the delorean steps are useful for building packages, but not required for basic usage10:26
mgouldshardy, my patch relates to node introspection :-)10:27
mgouldso yeah, that's something I'll have to do at some point10:27
shardymgould: Aha, well you'll want to do that then! :)10:27
* shardy normally skips it 10:27
mgouldI'll maybe skip it during setup, git-checkout my patch, then run introspection10:27
shardymgould: note for development, the --delorean-setup option configures a local delorean setup10:28
shardywhich can be run on the host, or the undercloud if you have enough memory/disk10:28
shardypersonally I run it on the host machine10:28
shardythen you can do e.g tripleo.sh --delorean-build openstack/ironic10:28
shardywhich will pull the ironic git master and build a local delorean RPM in ~/tripleo/delorean/data/repos/current10:29
*** SlickNik has quit IRC10:29
shardyyou can then pull whatever patches you need into e.g ~/tripleo/ironic and re-run --delorean-build10:29
* shardy notes we need a docs patch for this10:29
* mgould googles "delorean"10:30
mgouldah, it builds rpms from git commits10:30
shardymgould: Yeah, it's basically a wrapper for mock10:31
shardywe use it in CI10:31
mgouldOpenStack has a lot of moving parts with non-obvious names :-(10:32
shardyYeah, true10:32
*** openstackgerrit has quit IRC10:32
dtantsurmgould, I think I've spotted problem with your patch10:32
*** openstackgerrit has joined #tripleo10:32
*** epheo has quit IRC10:32
*** epheo has joined #tripleo10:33
* dtantsur got hit by the same mistake just yesterday10:33
mgoulddtantsur, thanks!10:34
*** SlickNik has joined #tripleo10:35
mgouldright, 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 do10:37
mgouldthanks very much!10:42
*** stendulker has quit IRC10:45
devvesaAn easy +2 here, please? https://review.openstack.org/#/c/257420/10:46
openstackgerritMiles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state  https://review.openstack.org/23515810:48
*** paramite has joined #tripleo10:50
dtantsurmgould, 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 AttributeError10:50
dtantsurwhich raises a good question of if we should run everything in debug mode in gate really10:51
dtantsurshardy, wdyt ^^10:51
shardydtantsur: yes, I think we should10:52
shardysome things are also hidden inside tripleo.sh, where we should be set -x'ing to ensure visibility when things go wrong10:52
shardybut that's not what you want for developers running the script so we probably need a variable to enable super-verbose mode for CI10:53
mgoulddtantsur, can we make that error message more explicit?10:56
mgouldI saw it but had no idea where it was coming from10:56
*** tosky has joined #tripleo10:56
dtantsurmgould, that's how OpenStackClient works IIRC.. it probably should be discussed on #openstack-sdk with them10:57
dtantsurI also find it annoying FWIW10:57
mgoulddtantsur, which repo does that happen in?11:01
dtantsurmgould, python-openstackclient is what we use as a basis for our CLI11:03
*** rebrego has joined #tripleo11:05
*** regebro has quit IRC11:05
*** rebrego is now known as regebro11:05
* mgould clones Yet Another Repo11:05
dtantsur$ ls ~/Projects/ | wc -l11:07
dtantsurmgould, ^^ ;)11:07
*** rasca has quit IRC11:07
*** rasca has joined #tripleo11:08
mgould17, I have a way to go :-/11:08
openstackgerritMerged openstack/tripleo-heat-templates: neutron: enable nova-event-callback by default  https://review.openstack.org/25522611:09
shardy^^ there are 8 easy stable/liberty reviews there, +2'd and passing CI11:11
shardyjust needs approving if someone has a moment11:11
devvesaCan I add this one, shardy? https://review.openstack.org/#/c/257420/11:16
*** Marga_ has joined #tripleo11:16
*** Marga_ has quit IRC11:17
*** Marga_ has joined #tripleo11:17
*** regebro has quit IRC11:18
*** lucas-dinner is now known as lucasagomes11:18
openstackgerritDougal Matthews proposed openstack/tripleo-common: [WIP] Add the Plan deploy endpoint to start a Heat deploy  https://review.openstack.org/25638411:18
*** regebro has joined #tripleo11:19
*** mburned is now known as mburned_out11:22
dtantsurshardy, could you please take a look at https://review.openstack.org/#/c/253476/ ? it will make CI actually cover introspection11:29
*** Marga_ has quit IRC11:30
*** paramite is now known as paramite|afk11:33
openstackgerritDougal Matthews proposed openstack/tripleo-common: Raise the Swift exception if we get one and it isn't a 404  https://review.openstack.org/25784311:34
*** paramite|afk is now known as paramite11:35
openstackgerritDougal Matthews proposed openstack/tripleo-common: Raise the Swift exception if we get one and it isn't a 404  https://review.openstack.org/25784311:35
shardydtantsur: lgtm, maybe d0ugal can give it a final +2/A11:35
*** gfidente has joined #tripleo11:35
*** gfidente has quit IRC11:35
*** gfidente has joined #tripleo11:35
d0ugalshardy, dtantsur - sure.11:35
*** Marga_ has joined #tripleo11:36
d0ugalwell, I'll look before I promise the +2 actually :P11:36
dtantsurgood idea :D11:36
d0ugaldtantsur: lgtm11:38
*** Marga_ has quit IRC11:40
*** Marga__ has joined #tripleo11:40
*** trown|outttypeww is now known as trown11:40
trowndtantsur: I think your lio patch broke tripleo11:40
trowndtantsur: also good morning :)11:41
dtantsurtrown, yeah, good morning :)11:41
openstackgerritMerged openstack/python-tripleoclient: Fail the introspection command if introspection has failed  https://review.openstack.org/25347611:41
dtantsurtrown, where is it broken and how?11:41
trowndtantsur: so first was the rtslib import... but that was a simple packaging fix11:41
*** trown has left #tripleo11:41
*** trown has joined #tripleo11:43
dtantsurok, you're back :)11:43
dtantsurso what's the 2nd problem?11:43
trowndtantsur: I was saying before whatever happened there...11:43
trowndtantsur: 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#L9611:43
openstackgerritMerged openstack/tripleo-heat-templates: honor the rabbit user and password provided  https://review.openstack.org/25595911:44
trownthat returns "/dev/sda2 already mounted"11:44
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of predictable IPs  https://review.openstack.org/24160611:44
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the controller nodes  https://review.openstack.org/22249711:45
trowndtantsur: these images have the issue https://ci.centos.org/artifacts/rdo/images/mitaka/delorean/testing/11:45
gfidentemarios, rebased ^^ thanks for reviewing11:45
shardygfidente: Hey, FYI the heat patch fixing the neutron net id/name issue we discussed yesterday was just approved11:45
gfidenteyeah I noticed11:45
dtantsurtrown, I'll think what can be done here..11:46
gfidenteshardy, you were pretty fast, I will link that from the BZ11:46
gfidenteI think the change to the client is still valid11:46
*** mgould has quit IRC11:46
gfidenteso it'd be good to merge them all11:46
dtantsurtrown, do you have an upstream bug for it?11:46
shardygfidente: agreed, the client change is still good, this should just make things a little easier in future11:46
trowndtantsur: 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 point11:46
trowndtantsur: no I am filing one this morning... just got the second issue figured out as I was leaving last night11:47
shardygfidente: we may be able to handle the DELETE_FAILED issue discussed in bug #1521944 via similar special-casing11:48
openstackbug 1521944 in heat "Failed update attempts might not preserve in heat the actual status of a resource" [Undecided,New] https://launchpad.net/bugs/152194411:48
shardygfidente: I'll look into that later11:48
gfidenteshardy, oh the heat change will be backported to liberty as well?11:48
shardygfidente: Yes, I'll propose it after it lands on master11:48
*** SlickNik has quit IRC11:50
*** paramite is now known as paramite|afk11:52
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of predictable IPs  https://review.openstack.org/24160611:52
*** SlickNik has joined #tripleo11:54
trowndtantsur: 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 ideal11:55
dtantsurtrown, 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 paramite11:55
lucasagomestrown, dtantsur yeah there's a patch proposed to tripleoclient11:56
trowndtantsur: but for deploy the ramdisk gets shutdown pretty fast11:56
lucasagomestrown, yeah I always edit ironic to avoid the shutdown tho :-(11:56
lucasagomesI had a patch to not shutdown on failures but it was turned down11:57
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Fail the introspection command if introspection has failed  https://review.openstack.org/25785511:57
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add update yaml backward compatibe with PublicVirtualIP on ctlplane  https://review.openstack.org/25785611:57
trownlucasagomes: hmmm... seems like we should be trying to send back the journal for the agent service, or at least that we should be able to11:59
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add update yaml backward compatibe with PublicVirtualIP on ctlplane  https://review.openstack.org/25785611:59
dtantsurfolks, when I just issue "openstack overcloud deploy --templates", where does number of nodes come from?12:01
openstackgerritJaume Devesa proposed openstack/tripleo-common: Create an overcloud-full function  https://review.openstack.org/25785912:01
dtantsurcurrently we only validate profiles if --XXX-scale is given, which is not entirely correct12:01
lucasagomestrown, I'm on that as well, gotta find the patch 1 sec12:01
lucasagomestrown, https://review.openstack.org/#/c/248832/12:02
lucasagomesI will update it12:02
lucasagomestrown, needs the bits in Ironic too to get the logs, problem is where to store it12:02
lucasagomeswe can do it in swift or base6412:02
trownlucasagomes: my vote would be swift12:03
lucasagomestrown, yeah, I want to make it configurable12:04
lucasagomeslike the configdrive12:04
trownbut that is biased to tripleo since we already have swift setup12:04
dtantsuralso 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
lucasagomestrown, ++12:05
*** rhallisey has joined #tripleo12:06
trowndtantsur: it is correct if you did not use --compute-flavor originally... it will default to baremetal flavor12:06
dtantsurtrown, aha, so with profile matching we have to use --XX-flavor... it's not error-proof to be honest12:07
dtantsure.g. people could expect that `openstack overcloud deploy --templates --compute-scale 5` would verify profiles, while it actually won't..12:07
trowndtantsur: ah right...12:08
dtantsurtrown, also this commands seems wrong too: http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/profile_matching.html#use-the-flavors-to-deploy12:08
dtantsurit does not provide --XXX-scale, so again, validation is skipped (and I'm not sure what is actually deployed)12:08
trowndtantsur: ah ya that scale command is definitely wrong... it is use tuskar notation with that '-r Compute-1'12:09
dtantsurtrown, no, I meant one above12:10
trowndtantsur: that one I think would be ok... it would deploy the default of 1 control and 1 compute12:11
dtantsurtrown, yeah, but profiles won't be checked/processed12:11
*** mburned_out is now known as mburned12:11
dtantsuras it's 0 from validation point of view12:11
trownhmmm... that seems not ideal12:12
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Ceph cluster validation  https://review.openstack.org/22524812:12
openstackgerritDmitry Tantsur proposed openstack/tripleo-docs: [WIP] Rewrite completely outdated information on the profile matching  https://review.openstack.org/25786712:17
dtantsurtrown, ^^12:17
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities  https://review.openstack.org/25040512:19
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Add --dry-run argument to overcloud deploy command  https://review.openstack.org/25734812:22
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities  https://review.openstack.org/25040512:22
trowndtantsur: 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 problem12:24
openstackLaunchpad bug 1526304 in ironic-python-agent "Using LIO for ISCSI results in failure to mount root device" [Undecided,New]12:24
*** dprince has joined #tripleo12:26
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Fix description of control plane route parameter  https://review.openstack.org/25787312:26
dtantsurtrown, if I make a patch, is it possible you test it?12:30
trowndtantsur: ya I can test it12:32
*** olap has joined #tripleo12:32
*** jprovazn has quit IRC12:34
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays  https://review.openstack.org/18750612:39
*** stendulker has joined #tripleo12:49
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Remove deleted Cinder rows  https://review.openstack.org/20901912:50
*** weshay_xchat has joined #tripleo12:59
*** weshay_xchat is now known as weshay12:59
*** jprovazn has joined #tripleo12:59
*** thrash|g0ne is now known as thrash13:03
*** links has quit IRC13:09
openstackgerritMerged openstack/tripleo-puppet-elements: Adding networking MidoNet element  https://review.openstack.org/24738213:10
openstackgerritMerged openstack/tripleo-puppet-elements: Remove Cassandra repository for MidoNet element  https://review.openstack.org/25691213:11
*** jayg|g0n3 is now known as jayg13:15
*** regebro has quit IRC13:16
*** regebro has joined #tripleo13:16
*** trown is now known as trown|brb13:21
*** mgould has joined #tripleo13:21
*** masco has quit IRC13:22
*** slagle has quit IRC13:25
openstackgerritMerged openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update  https://review.openstack.org/24509313:26
*** trown|brb is now known as trown13:31
*** whydidyoustealmy has joined #tripleo13:35
*** shakamunyi has joined #tripleo13:35
*** whydidyoustealmy is now known as superflyy13:36
trowndprince: I think the overcloud swift fix breaks the undercloud install13:39
trowndprince: I do not think $device_array is defined there13:40
openstackgerritMerged openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25131913:43
*** hewbrocca has joined #tripleo13:45
*** ukalifon1 has quit IRC13:49
*** links has joined #tripleo13:51
*** slagle has joined #tripleo13:52
openstackgerritRyan Brady proposed openstack/tripleo-common: Add the missing plan name to the PlanAlreadyExistsError  https://review.openstack.org/25734313:54
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25790813:56
*** tzumainn has joined #tripleo13:56
*** yamahata has joined #tripleo13:58
openstackgerritMiles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state  https://review.openstack.org/23515814:00
*** rcernin has joined #tripleo14:02
dprincemeeting time14:03
openstackgerritQasim Sarfraz proposed openstack/tripleo-heat-templates: PLUMgrid Neutron integration  https://review.openstack.org/25791614:07
openstackgerritQasim Sarfraz proposed openstack/tripleo-heat-templates: PLUMgrid Neutron integration  https://review.openstack.org/25791614:10
*** SlickNik has quit IRC14:14
openstackgerritTzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API  https://review.openstack.org/23043214:18
*** dmacpher has joined #tripleo14:19
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Remove deleted Nova rows  https://review.openstack.org/20790914:21
*** masco has joined #tripleo14:21
*** SlickNik has joined #tripleo14:24
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Switch for Keystone DB cron job  https://review.openstack.org/20953314:24
*** lblanchard has joined #tripleo14:25
openstackgerritMerged openstack/tripleo-common: [stable/liberty]Fix node group name when deleting a node  https://review.openstack.org/25595514:26
*** rpothier has joined #tripleo14:29
*** stendulker has quit IRC14:32
*** mgould has quit IRC14:35
*** lazy_prince has quit IRC14:40
*** ChuckC has joined #tripleo14:41
*** qasims has joined #tripleo14:44
*** qasims has quit IRC14:44
*** ChuckC has quit IRC14:45
*** ChuckC has joined #tripleo14:46
*** mgould has joined #tripleo14:50
openstackgerritEmilien Macchi proposed openstack/tripleo-puppet-elements: Add logging components  https://review.openstack.org/25664014:51
openstackgerritEmilien Macchi proposed openstack/tripleo-puppet-elements: Add logging components  https://review.openstack.org/25664014:52
*** lucasagomes is now known as lucas-hungry14:56
*** links has quit IRC14:58
*** links has joined #tripleo14:59
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Keystone domain for Heat  https://review.openstack.org/18056615:01
*** paramite has quit IRC15:19
*** rleander is now known as leanderthal15:22
*** Goneri has quit IRC15:22
openstackgerritDmitry Tantsur proposed openstack/tripleo-docs: Rewrite completely outdated information on the profile matching  https://review.openstack.org/25786715:25
*** bnemec has quit IRC15:25
*** tosky has quit IRC15:27
*** ChuckC has quit IRC15:27
*** tosky has joined #tripleo15:28
*** mkovacik has quit IRC15:28
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116715:30
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116715:31
*** bnemec has joined #tripleo15:32
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Add fixup for pcs order constraints after update to new templates  https://review.openstack.org/24886315:35
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116715:35
*** stendulker has joined #tripleo15:36
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116715:37
openstackgerritTzu-Mainn Chen proposed openstack/tripleo-heat-templates: Add capabilities map  https://review.openstack.org/24243915:37
*** aufi has quit IRC15:39
*** pradk has joined #tripleo15:39
*** lucas-hungry is now known as lucasagomes15:40
openstackgerritTzu-Mainn Chen proposed openstack/tripleo-heat-templates: Add capabilities map  https://review.openstack.org/24243915:48
*** Marga__ has quit IRC15:53
tzumainnshardy, 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
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update  https://review.openstack.org/25797615:58
jtomasektzumainn: did I just +1'd my own patch?:)))15:58
*** rcernin has quit IRC15:58
tzumainnjtomasek, lol, apparently so!15:59
*** jaosorior has joined #tripleo16:03
shardytzumainn: Yup, lgtm16:04
shardytzumainn: 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 there16:04
shardyand that nothing gets removed or renamed if it's in a requires: list16:04
tzumainnshardy, that makes sense16:05
*** dtantsur is now known as dtantsur|afk16:06
*** sthillma has joined #tripleo16:08
*** Goneri has joined #tripleo16:09
*** sthillma_ has joined #tripleo16:10
*** sthillma has quit IRC16:12
*** sthillma_ is now known as sthillma16:12
jaosoriorbnemec, are you around?16:19
bnemecshardy: tzumainn jtomasek: https://review.openstack.org/#/c/253638/ for automatically generating those environment files.16:21
bnemecWe may want to combine the two.16:21
bnemecjaosorior: Yes16:21
jtomasekbnemec: thanks! I'll take a look16:22
jaosoriorbnemec: 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/316:23
*** Marga_ has joined #tripleo16:25
bnemecjaosorior: 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
jaosoriorbnemec: thanks man16:26
*** yamahata has quit IRC16:27
*** stendulker has quit IRC16:33
shardybnemec: if you want some easy stable/liberty ones to start with, there's some ready to +A here:16:34
*** dshulyak has joined #tripleo16:36
*** tremble has quit IRC16:40
*** jcoufal has quit IRC16:43
*** ifarkas has quit IRC16:44
*** mgould has quit IRC16:45
*** dshulyak has quit IRC16:45
lucasagomeshi 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 quick16:46
trowndprince: 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
openstackgerritMerged openstack/tripleo-heat-templates: Inject TLS certificate and keys for the Overcloud  https://review.openstack.org/25483116:53
*** yamahata has joined #tripleo16:53
*** lazy_prince has joined #tripleo16:54
openstackgerritMerged openstack/tripleo-heat-templates: Enable trust anchor injection  https://review.openstack.org/25483216:55
openstackgerritMerged openstack/tripleo-heat-templates: Output the SSL Certificate and Key modulus  https://review.openstack.org/25483316:56
*** cwolferh has joined #tripleo16:56
openstackgerritMerged openstack/tripleo-heat-templates: Minor fixes to TLS related resources  https://review.openstack.org/25483416:56
jaosorioryay :D16:56
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116716:58
*** david-lyle has quit IRC17:02
openstackgerritMerged openstack/tripleo-heat-templates: Update docker compute environment to use json config  https://review.openstack.org/25440817:04
dprincetrown: we use swift::ringbuilder17:04
dprincetrown: but not tripleo::ringbuilder17:04
dprincetrown: thus I don't think the overcloud change should effect us directly17:05
openstackgerritMerged openstack/tripleo-heat-templates: Add local docker registry support  https://review.openstack.org/25440917:06
*** akrivoka has quit IRC17:06
openstackgerritMerged openstack/tripleo-heat-templates: Nova-libvirt needs to bind to /sys/fs/cgroup  https://review.openstack.org/25441017:06
openstackgerritMerged openstack/tripleo-heat-templates: Change the Atomic image name so it's less specific  https://review.openstack.org/25441117:07
openstackgerritMerged openstack/tripleo-heat-templates: Point registry at tripleoupstream  https://review.openstack.org/25441217:07
trowndprince: I am confused... https://review.openstack.org/#/c/255995/4/elements/puppet-stack-config/puppet-stack-config.yaml.template defitely uses swift::ringbuilder17:08
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116717:09
trowndprince: and https://review.openstack.org/#/c/257540/ definitely changes swift::ringbuilder17:09
dprincetrown: right, but that shouldn't effect the undercloud17:10
dprincetrown: sorry, this morning I thought you said the overcloud change broke the undercloud17:10
trowndprince: the first change there is to the undercloud puppet-stack-config element...17:10
trowndprince: if I manually apply the puppet-swift change, undercloud install fails17:11
dprincetrown: got a paste of the error17:11
trowndprince: I can recreate it, one sec17:11
dprincetrown: because setting replicas = 1 in the undercloud fixes it for me17:11
openstackgerritMiles Gould proposed openstack/python-tripleoclient: Support new ironic "enroll" state  https://review.openstack.org/23515817:12
trowndprince: have you tried to apply the puppet-swift change before running the undercloud install?17:12
*** david-lyle has joined #tripleo17:12
*** mgould has joined #tripleo17:12
openstackgerritMiles Gould proposed openstack/python-tripleoclient: Support new ironic "enroll" state  https://review.openstack.org/23515817:13
dprincetrown: what puppet-swift change?17:13
dprincetrown: I'm using puppet-swift master...17:13
openstackgerritMarios Andreou proposed openstack/tripleo-common: WIP: Adds a simple overcloud tenant vm ping test to tripleo.sh  https://review.openstack.org/24116717:13
trowndprince: ugh.... just realized my error... your change is in tht not puppet-swift17:15
dprincetrown: correct17:15
trownI was patching puppet-swift which breaks undercloud, but I think we are good17:16
*** mcornea has quit IRC17:16
*** thrash is now known as thrash|f00dz17:16
*** trown is now known as trown|lunch17:19
*** chem has joined #tripleo17:19
chemhi, 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
openstackgerritBen Nemec proposed openstack/tripleo-heat-templates: Sample environment generator  https://review.openstack.org/25363817:20
*** derekh has quit IRC17:21
*** masco has quit IRC17:25
*** lazy_prince has quit IRC17:29
*** dshulyak has joined #tripleo17:33
chemso no way to do a puppet apply directly on the concerned node ?17:41
*** rebrego has joined #tripleo17:50
*** regebro has quit IRC17:50
*** rebrego is now known as regebro17:50
*** rasca has quit IRC17:50
openstackgerritBen Nemec proposed openstack/instack-undercloud: Remove option of installing tuskar  https://review.openstack.org/25802817:50
*** links has quit IRC17:50
*** shivrao has joined #tripleo17:56
openstackgerritBen Nemec proposed openstack/instack-undercloud: Allow setting the hostname during undercloud install  https://review.openstack.org/24945717:59
*** devvesa has quit IRC18:04
openstackgerritMerged openstack/tripleo-heat-templates: Add capabilities map  https://review.openstack.org/24243918:06
lucasagomesbnemec, hi there, re 257369. It's the other way around. It will not work with local boot18:09
lucasagomesbecause 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
bnemeclucasagomes: Why not?  Can't I stop at the grub prompt and add the kernel cmdline option?18:10
bnemecI 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 IRC18:10
lucasagomesbnemec, yeah you can18:10
bnemecAnd if you're using a remote KVM that does copy-paste it wouldn't be too bad.18:10
lucasagomesok if that's acceptable I'm alright adding it to common18:10
bnemeclucasagomes: 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
lucasagomesbnemec, cool. I will edit the patch18:12
lucasagomesI was just making sure we are on the same page about hw it works18:12
lucasagomesbut yeah if you have console access you can edit the grub before booting and set it up18:12
*** Marga_ has quit IRC18:18
*** jistr has quit IRC18:22
EmilienMslagle: can you consider https://review.openstack.org/#/c/234704/ again please?18:24
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Deploy Monitoring on the undercloud with Puppet  https://review.openstack.org/25370918:24
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24141218:25
*** gfidente is now known as gfidente|afk18:26
slagleEmilienM: looking now18:28
*** rasca has joined #tripleo18:29
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila  https://review.openstack.org/18813818:32
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila  https://review.openstack.org/23093618:32
*** mgould is now known as mgould|afk18:36
*** olap has quit IRC18:38
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks  https://review.openstack.org/23542318:39
*** Guest73233 is now known as mgagne18:42
slagleEmilienM: i'm getting an error still, http://paste.openstack.org/show/481987/18:42
*** mgagne is now known as Guest7643418:42
openstackgerritOpenStack Proposal Bot proposed openstack/os-cloud-config: Updated from global requirements  https://review.openstack.org/24762418:43
slagleEmilienM: here are the endpoints i have, http://paste.openstack.org/show/481988/18:43
EmilienMslagle: ok, looking18:44
EmilienMslagle: why do you have dupplicates in your endpoints?18:46
*** trown|lunch is now known as trown18:46
EmilienMbecause of this patch?18:47
slaglei guess so18:47
EmilienMslagle: I wonder if adding Keystone_service<||> -> Keystone_endpoint<||> would help18:48
EmilienMI thought no, but in fact we might need it18:48
EmilienMI'm going to try it18:49
slagleok, so in the ironic duplicate, there is this difference, vs
slaglethe trailing slash is the only difference18:49
slagleare the puppet modules looking up urls to see if an endpoint already exists?18:49
EmilienMyes but it needs to match a uniq service name18:50
EmilienMbut from what i'm seeing we might check something else18:50
slaglecan it be less strict a bout the trailing slash?18:50
EmilienMslagle: chem did that recently: https://github.com/openstack/puppet-keystone/commit/0a4e06abb0f5b3f324464ff5219d2885816311ce18:51
chemEmilienM: catching up, what is the problem ?18:51
EmilienMchem: still trying to make upgrades pass with https://review.openstack.org/#/c/234704/18:52
chemEmilienM: slagle the prbolem is the double tuskar entry ? in the paste http://paste.openstack.org/show/481988/18:53
EmilienMchem: when we run puppet, it created dupplicated endpoints18:53
EmilienMyes and ceilometer and tuskar and ironic18:53
chemEmilienM: nope :)18:53
slaglethe only difference i see between the duplicates is the trailing slash in the url's18:54
jpeelerthrash|f00dz: have you been in contact with anybody about CI difficulties? it looks like the latest CI failures aren't due to the patch18:55
*** jtomasek has quit IRC18:55
EmilienMslagle: are you able to retry again by adding Keystone_service<||> -> Keystone_endpoint<||> in the manifest?18:55
EmilienMI remember dprince suggested me to add it18:55
dprinceslagle: yeah, I'd be curious if that fixes it for you18:55
EmilienMbut I thought our provider would do the job with Autorequire18:55
*** tosky has quit IRC18:56
slagleEmilienM: i can try, but i'd need to rollback somewhow to what i had before18:56
slaglei dont have a backup of the keystone db18:56
slaglei dont know which endpoints puppet created, the ones with or without the slash18:56
slaglei can try it both ways i guess though18:56
dprinceslagle: do you have a paste of the error you got?18:57
EmilienMdprince: http://paste.openstack.org/show/481987/18:57
EmilienMError: Execution of '/bin/openstack endpoint create --format shell nova admin --region regionOne' returned 1: No service with a type, name or ID of 'nova' exists.18:57
dprinceEmilienM: yeah, it sounds like it isn't running early enough18:57
dprinceEmilienM: lets put in the resource chain explicitly and see if it fixes it18:58
dprinceEmilienM: that was what I tested (initially). Perhaps there are subtle catalog ordering issues where sometimes it works without it18:58
dprinceEmilienM: but making the relationship explicit would make it always work18:59
EmilienMthe bot is dead but I sent a patch19:00
chemdprince: EmilienM slagle in the last puppet keystone_endpoint the autorequire should work ok and the duplicate entry shoudn't exist... I wonder what is missing19:00
chemslagle: could you paste puppet resource keystone_endpoint and puppet resource keystone_service at the time you get the error ?19:01
dprincechem: 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 endpoints19:02
slaglechem: what do you mean at the time i get the error?19:02
slaglethe puppet apply of the manifest failed, can i run those now and show you the output?19:02
chemslagle: yep19:03
slaglechem: http://paste.openstack.org/show/481995/19:04
chemdprince: 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 here19:04
chemslagle: thanks19:04
*** pblaho_ has joined #tripleo19:05
*** pino|work_ has joined #tripleo19:05
dprincechem: okay, so that functionality isn't working19:05
*** rleander_ has joined #tripleo19:05
dprincechem: I think because in tripleo there were two endpoints tied to a service named 'nova'19:05
chemdprince: the latest keystone_endpoint shouldn't be fooled anymore by this19:06
dprincechem: I tested this just over the weekend (latest keystone_endpoint code then)19:06
*** dtantsur|afk has quit IRC19:06
dprincechem: I think the bug is that TripleO creates two endpoints for the same service/type19:07
*** pblaho has quit IRC19:08
dprincechem: so while the keystone_endpoint handles updates nicely, in this case it blows up because it can't determine which one to update19:08
*** athomas has quit IRC19:08
*** hewbrocca has quit IRC19:08
*** pino|work has quit IRC19:08
*** morazi has quit IRC19:08
*** leanderthal has quit IRC19:08
*** eil397 has joined #tripleo19:09
chemdprince: could you show me the two "endpoint creation" part in tripleo ?19:09
dprincechem: yep, just a sec...19:09
*** hewbrocca has joined #tripleo19:09
*** sthillma has quit IRC19:09
dprincechem: http://git.openstack.org/cgit/openstack/os-cloud-config/tree/os_cloud_config/keystone.py#n7019:10
chemdprince: thanks19:10
dprincechem: note that the novav3 endpoint uses the same service name (for TripleO) as the normal nova endpoint19:10
dprincechem: I think that case is what keystone_endpoint isn't handling. The explicit delete seemed (to me) to be the cleanest way to handle this19:11
dprinceEmilienM: the latest version of this patch removes all of the fixes? https://review.openstack.org/#/c/234704/1019:12
*** rleander_ has quit IRC19:12
dprinceEmilienM: oh, nm. I was on a previous patch!19:12
*** pino|work has joined #tripleo19:12
*** pino|work_ has quit IRC19:13
*** pblaho_ has quit IRC19:13
dprinceEmilienM: yeah, that should work. It did for me19:13
*** dtantsur has joined #tripleo19:13
dprinceby my testing over the weekend...19:13
chemdprince: 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
chemdprince: the patch made this possible while it wasn't before19:14
chemdprince: it then fetch the correct service by "id" not by name19:14
EmilienMdprince: that's why I sent a new patchset with the orchestration and I wondered if slagle could also check19:14
dprincechem: right, but the nova::keystone::auth manifests forces us to use unique services for nova vs. novav3 though19:14
*** pblaho has joined #tripleo19:14
*** penick has joined #tripleo19:14
dprincechem: 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 case19:15
dprincechem: http://git.openstack.org/cgit/openstack/puppet-nova/tree/manifests/keystone/auth.pp#n25919:16
chemdprince: 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
dprincechem: sorry, appreciate you trying to help us sort this out19:19
slagleEmilienM: ok, should i manually clean up the dupes first before i try the latest patchset?19:19
openstackgerritLucas Alvares Gomes proposed openstack/python-tripleoclient: Add dynamic-login element to the image creation  https://review.openstack.org/25736919:20
chemslagle: you should19:20
EmilienMyes so we have the previous state19:20
slagleok, i'm not sure which ones puppet created19:20
slaglei'll guess19:20
*** ukalifon1 has joined #tripleo19:20
chemslagle: this paste http://paste.openstack.org/show/481995/ is comming from the platform where you have the duplicate ?19:21
dprinceslagle: my guess is running the new patch will just work19:21
dprinceslagle: but I'm more interested in if it fixes the clean upgrade...19:21
dprinceslagle: because it did for me19:22
slaglei'll try it a couple different ways19:22
*** jaosorior has quit IRC19:23
*** shardy is now known as shardy_afk19:23
*** sthillma has joined #tripleo19:24
dprinceslagle: cool, thanks for trying this19:24
*** rleander_ has joined #tripleo19:25
chemdprince: 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
chemdprince: it just lacks another +219:26
dprincechem: cool, let me check it out then19:26
dprincechem: would that delete the old one though?19:27
chemdprince: not sure I understand ?19:27
chemdprince: what would delete what ?19:28
dprincechem: it fixes it if someone installs a new version and upgrades, but that doesn't delete the previous nova/computev3 service right?19:29
chemdprince: nope.  It would only work on fresh install and "fresh" update.  I see your problematic here...19:30
*** gfidente|afk has quit IRC19:30
*** openstackgerrit has quit IRC19:32
*** openstackgerrit has joined #tripleo19:32
*** rwsu has quit IRC19:39
chemdprince: 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 #tripleo19:42
*** jprovazn has quit IRC19:42
dprincechem: 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 error19:43
dprincechem: if it is easy to improve the wrappers in this regard... then fine but I think they already perform quite acceptably19:43
*** dshulyak has quit IRC19:44
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests  https://review.openstack.org/24416219:46
*** dprince has quit IRC19:47
*** electrofelix has quit IRC19:48
openstackgerrityolanda.robla proposed openstack/diskimage-builder: Split vm and bootloader elements  https://review.openstack.org/25812619:49
*** olap has joined #tripleo19:52
slagleEmilienM: k, it seems to be working fine now19:56
EmilienMdan was right then19:57
EmilienMslagle: thanks... for your help19:57
slaglei'll mash the big red button once it passes ci19:57
EmilienMvery nice19:57
EmilienMa nice step that will us to make progress19:57
EmilienMthanks for your help chem, dprince & slagle19:57
*** sthillma has quit IRC20:00
*** thrash|f00dz is now known as thrash|brb20:09
*** rpothier has left #tripleo20:12
*** jaosorior has quit IRC20:12
*** lucasagomes is now known as lucas-dinner20:17
*** rwsu has joined #tripleo20:19
openstackgerritBen Nemec proposed openstack/tripleo-docs: Document installing the undercloud with SSL endpoints  https://review.openstack.org/22189120:23
*** Marga_ has joined #tripleo20:25
*** sthillma has joined #tripleo20:26
openstackgerritBen Nemec proposed openstack/python-tripleoclient: Always install yum-plugin-priorities  https://review.openstack.org/24216920:30
*** sthillma has quit IRC20:31
*** jtomasek has joined #tripleo20:32
openstackgerritBen Nemec proposed openstack/tripleo-common: Remove yum-plugin-priorities hack  https://review.openstack.org/24222820:32
openstackgerritJeff Peeler proposed openstack/instack-undercloud: Disable auto activation of all volumes  https://review.openstack.org/25813920:38
*** Marga_ has quit IRC20:44
openstackgerritJohn Trowbridge proposed openstack-infra/tripleo-ci: DO NOT MERGE changes needed for delorean promotion  https://review.openstack.org/25814020:45
*** lblanchard has quit IRC20:56
*** Marga_ has joined #tripleo21:01
*** jcoufal has joined #tripleo21:05
*** thrash|brb is now known as thrash21:07
*** ukalifon1 has quit IRC21:10
openstackgerritMerged openstack/tripleo-docs: Document using a pending review  https://review.openstack.org/22657021:10
*** jaosorior has joined #tripleo21:11
*** lblanchard has joined #tripleo21:22
*** sthillma has joined #tripleo21:26
*** mkovacik has joined #tripleo21:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use renamed overcloud.yaml template  https://review.openstack.org/25335421:30
*** pradk has quit IRC21:33
*** Guest76434 is now known as mgagne21:33
*** mgagne is now known as Guest16021:34
*** Guest160 has quit IRC21:34
*** Guest160 has joined #tripleo21:34
*** Guest160 is now known as mgagne21:35
*** cwolferh has quit IRC21:38
*** jaosorior has quit IRC21:41
*** Marga_ has quit IRC21:41
*** dshulyak has joined #tripleo21:43
*** mcornea has joined #tripleo21:44
*** jcoufal has quit IRC21:45
*** jayg is now known as jayg|g0n321:55
*** david-lyle has quit IRC21:58
*** lblanchard has quit IRC21:58
*** jtomasek has quit IRC21:59
*** david-lyle has joined #tripleo22:00
bnemecdsneddon: I just left a comment on https://review.openstack.org/#/c/226057/22:03
bnemecI 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|outttypeww22:04
dsneddonbnemec, Might as well fix it now, I lost all reviews when I had to rebase it.22:05
dsneddonbnemec, 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
bnemecdsneddon: I'm still showing a +2 from shardy_afk on that one.22:05
dsneddonbnemec, Oh, you're right, he did review it a few hours ago.22:05
bnemecAlthough, honestly at this point I think that's a candidate for a fast merge.22:06
bnemecEveryone's been basically good with it for a while, it just keeps hitting rebase problems.22:06
dsneddonbnemec, I'm going to fix the minor issue, and send shardy a note. Thanks.22:06
*** rcernin has joined #tripleo22:16
*** mkovacik has quit IRC22:18
*** shardy_afk has quit IRC22:23
openstackgerritBen Nemec proposed openstack/tripleo-docs: Document installing the undercloud with SSL endpoints  https://review.openstack.org/22189122:30
*** mcornea has quit IRC22:38
openstackgerritBen Nemec proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics templates  https://review.openstack.org/25268322:40
openstackgerritBen Nemec proposed openstack/tripleo-heat-templates: Rename validate tox env to pep8  https://review.openstack.org/25817322:40
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration.  https://review.openstack.org/22605722:41
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add all isolated networks to all nodes.  https://review.openstack.org/24953322:42
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics templates  https://review.openstack.org/25268322:44
dsneddonbnemec, Sorry to be rude, but I just rebased your change on top of my management network change.22:45
dsneddonbnemec, Thought you wouldn't mind, since you convinced me to fix it. ;-)22:45
bnemecdsneddon: 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 IRC23:01
*** david-lyle has joined #tripleo23:01
*** yuanying has joined #tripleo23:02
*** Goneri has quit IRC23:03
*** yuanying has quit IRC23:03
*** yuanying has joined #tripleo23:04
*** jtomasek has joined #tripleo23:11
*** mburned is now known as mburned_out23:15
*** jtomasek has quit IRC23:16
*** rcernin has quit IRC23:20
openstackgerritMerged openstack/diskimage-builder: Add dib-python element  https://review.openstack.org/25607123:24
*** stevebaker has quit IRC23:30
*** stevebaker has joined #tripleo23:31
*** david-lyle_ has joined #tripleo23:33
*** david-lyle has quit IRC23:36
openstackgerritAbel Lopez proposed openstack/diskimage-builder: Document byte-to-inode ratio  https://review.openstack.org/25819523:47
*** sthillma has quit IRC23:52
*** mburned_out is now known as mburned23:53
*** chlong has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!