Wednesday, 2016-02-24

*** tiswanso has joined #tripleo00:08
*** eil397 has quit IRC00:13
*** tiswanso has quit IRC00:13
*** dmacpher-afk has quit IRC00:20
*** sshnaidm has quit IRC00:25
*** sshnaidm has joined #tripleo00:31
*** thrash is now known as thrash|g0ne00:33
*** sshnaidm has quit IRC00:36
*** sshnaidm has joined #tripleo00:38
*** lblanchard has joined #tripleo00:49
openstackgerritMerged openstack/tripleo-common: Allow updating images via `tripleo.sh --overcloud-images`  https://review.openstack.org/28360700:55
*** saneax is now known as saneax_AFK00:56
*** Goneri has joined #tripleo00:59
*** dmacpher has joined #tripleo01:09
*** tiswanso has joined #tripleo01:10
*** masco has joined #tripleo01:11
*** Goneri has quit IRC01:12
*** Goneri has joined #tripleo01:12
*** tiswanso has quit IRC01:20
*** penick has quit IRC01:28
openstackgerritGiulio Fidente proposed openstack/tripleo-docs: Update list of pkgs in includepkgs  https://review.openstack.org/28389901:34
*** shivrao has quit IRC01:35
*** trown|outtypewww has quit IRC01:37
openstackgerritMerged openstack/instack: Remove Py33 support  https://review.openstack.org/26611601:39
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements  https://review.openstack.org/26854901:40
openstackgerritMerged openstack/tripleo-common: Reuse puppet modules already downloaded  https://review.openstack.org/28369901:52
*** tiswanso has joined #tripleo01:54
*** tiswanso has quit IRC01:54
*** tiswanso has joined #tripleo01:55
openstackgerritMichael Chapman proposed openstack/tripleo-heat-templates: Adds OpenDaylight support  https://review.openstack.org/20025301:57
*** yamahata has quit IRC02:00
*** masco has quit IRC02:02
*** Goneri has quit IRC02:06
*** pcaruana has quit IRC02:07
*** shivrao has joined #tripleo02:17
*** shivrao_ has joined #tripleo02:19
*** pcaruana has joined #tripleo02:19
*** lblanchard has quit IRC02:21
*** shivrao has quit IRC02:21
*** shivrao_ is now known as shivrao02:21
*** Marga_ has quit IRC02:39
*** shivrao has quit IRC02:47
*** shivrao has joined #tripleo02:49
*** shivrao has quit IRC02:54
*** pcaruana has quit IRC03:01
*** pcaruana has joined #tripleo03:16
*** links has joined #tripleo03:46
*** rlandy has quit IRC03:49
*** pcaruana has quit IRC04:01
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements  https://review.openstack.org/26854904:02
*** xinwu has quit IRC04:09
openstackgerritMerged openstack/puppet-tripleo: Override X-Forwarded-Proto header  https://review.openstack.org/28307404:15
*** pcaruana has joined #tripleo04:15
*** masco has joined #tripleo04:25
*** xinwu has joined #tripleo04:35
*** Marga_ has joined #tripleo04:36
*** Marga_ has quit IRC04:40
*** Marga_ has joined #tripleo04:43
*** tiswanso has quit IRC04:48
*** penick has joined #tripleo04:48
*** penick_ has joined #tripleo04:50
*** penick has quit IRC04:53
*** penick_ is now known as penick04:53
*** Marga_ has quit IRC04:57
*** Marga_ has joined #tripleo04:58
*** anande has joined #tripleo04:59
openstackgerritMartin AndrĂ© proposed openstack/tripleo-specs: Spec for TripleO validations  https://review.openstack.org/25579205:08
*** trozet has quit IRC05:22
*** liverpooler has quit IRC05:27
*** anande has quit IRC05:36
*** weshay has quit IRC05:36
*** pcaruana has quit IRC05:38
*** pcaruana has joined #tripleo05:53
*** akuznetsov has joined #tripleo05:56
*** saneax_AFK is now known as saneax06:16
*** akuznetsov has quit IRC06:19
*** openstack has joined #tripleo13:21
*** openstackstatus has joined #tripleo13:22
*** ChanServ sets mode: +v openstackstatus13:22
bandini_jistr: yeah I saw that the wsgi change went in only after the fact. I would have supported it only within the lightweight arch discussion. As is, things get rather complicated not only upgrade wise, but also the whole constraint setup13:26
bandini_aka we make everything depend on httpd which hosts keystone and horizon, which sounds really wrong13:26
trownpradk: tripleo pin for delorean has been updated, so you should have a recent version of ceilo packages to test against now13:27
trownderekh++13:27
derekhtrown: actually didn't update the current-tripleo link as I was giving existing jobs a chance to finish, I guess thats why you set it in tripleo.sh13:28
derekhtrown: will update it now13:28
trownderekh: ya, that part is more a formality though, since we are pinned in tripleo.sh13:29
derekhtrown: hmm, looks like I don't have permission on the new server, will ask jpena13:29
derekhtrown: yup13:29
trownderekh: I think he copied keys, but you would need to ssh as root13:29
derekhtrown: yup, root works, thanks13:30
bandini_jistr: could you send me a cib file of a latest deployment, so I can take a look?13:31
jistrbandini_: yeah i thought so too... EmilienM, jayg: ^^ can we revert keystone WSGI and land it again with correct constraints and upgrade support? For one thing, i think the lack of migration logic will block us from getting the upgrade job from liberty to mitaka working, and given that the constraints seem incorrect, we can't just jump into implementing the migration logic for what has been merged. It might be better to revert it first and13:32
jistrland it again with correct constraints and migration support.13:32
jistrbandini_: i should have a fresh deployment shortly, will do13:33
jistrEmilienM, jayg: i mean, if we had an upgrade CI job working already, the patch wouldn't be allowed to land in the first place13:33
derekhtrown: done13:34
trownderekh: can we switch this to consistent? https://github.com/openstack-infra/tripleo-ci/blob/master/toci_gate_test.sh#L6213:34
trownI will put up a patch for that and for tripleo.sh to put back current-tripleo13:35
derekhtrown: yes, make more sense13:35
ayoungbandini_, we need to rip off this bandage13:35
derekh*makes13:36
ayoungtripleo should never have been written with Keystone in Eventlet in the first place.13:36
ayoungso...lets solve this, and solve it right13:36
jaosoriorjistr: Do we really need to revert it? Isn't it possible to propose another patch with the logic to upgrade?13:36
jaygjistr: could there not just be a new patch fixing the constraints instead of reverting whatever made it in? ayoung was pretty adamant that this is a non-negotiable change since eventlet support is gone in mitaka13:37
jaygand he beat me :)13:37
hewbroccaayoung: there are several things we are trying to solve, and solve right13:37
fabbionefixing the constraints is easy13:37
fabbioneand safe13:37
hewbroccaand they are in conflict at the moment13:37
fabbionei gave you the steps13:37
fabbionebut that bug keeps being pushed around from "urgent" to "nobody-cares"13:38
ayoungfabbione, link?13:38
jaygwhich bug is this?13:38
fabbioneand i understand why13:38
openstackgerritJohn Trowbridge proposed openstack/tripleo-common: Return current-tripleo symlink  https://review.openstack.org/28413013:38
fabbionesec13:38
hewbroccafabbione: is referring to the change to add the dummy resource to our pacemaker config13:38
fabbionei need to dig the num13:38
fabbioneyes that one13:38
hewbroccaso that restarting keystone doesn't restart the entire cluster13:38
jistrjaosorior, jayg: yeah i know it's gone. We can surely have a patch which fixes the constraints *and* prevents upgrades from breaking. My only concern is that until we do have that patch, we cannot start gating on upgrades, because liberty->mitaka upgrade will not work with the current state of things. (It wouldn't work even with correct constraints. This is mainly about the missing upgrade logic.)13:38
hewbrocca(or httpd)13:38
hewbroccaAnd13:39
hewbroccathe feature freeze is in one week13:39
fabbionehttps://bugzilla.redhat.com/show_bug.cgi?id=129012113:39
openstackbugzilla.redhat.com bug 1290121 in rhel-osp-director "[RFE] Remove keystone constraints and add the openstack-core dummy resource in its place" [High,Assigned] - Assigned to jstransk13:39
ayoungso really https://bugs.launchpad.net/tripleo/+bug/153788513:39
openstackLaunchpad bug 1537885 in tripleo "Remove keystone constraints and add the openstack-core dummy resource in its place" [Undecided,New]13:40
fabbionei mentioned several times that moving keystone to http requires that change13:40
jistrayoung: yup, this, plus a migration logic for the same thing13:40
pradktrown, cool, thx13:40
ayoungMake it happen.  It would take me months to figure out the logic here13:40
ayoungI am sorry, I can't master every technology in OpenStack13:40
jistryea i guess we need to find someone who isn't tasked through the roof with Liberty things13:41
ayoungI am heads down tryin to do about 5 different things, and I trust you guys to do the right thing by Keystone.13:41
jistror un-task someone from Liberty things13:41
jaygjistr: did this patch go to liberty as well as master?  can you repaste the review link, I dont have it handy13:41
*** tzumainn has joined #tripleo13:41
jistrjayg: just master https://review.openstack.org/#/c/213175/13:42
openstackgerritJohn Trowbridge proposed openstack-infra/tripleo-ci: Use consistent delorean symlink for periodic job  https://review.openstack.org/28413113:42
jaygso if we are worried about feature freeze for liberty, how is this a problem for right now?13:43
hewbroccajayg: +113:43
ayoungIt went to master only, AFAICT13:43
jaygI agree it needs to be fixed, lets put it on the backlog13:43
hewbroccato be clear, the problem is13:43
jistrjayg: that means kilo->liberty upgrades (main downstream concern) aren't broken by that, but liberty->mitaka upgrades (upstream upgrade job which is being baked) are broken13:43
hewbroccayes13:43
ayoungI didn't backport, and would not suggest it get backported to liberty.13:43
jistrjayg: we're worried about Mitaka freeze, not Liberty freeze13:43
jaygah, ok13:44
jaygso switching to this dummy resource doesn't look hard froma quick read, we have used such a thing before13:44
bandini_when is the mitaka freeza again?13:44
jaygso why not do a patch to get that in, and another for whatever upgrade stuff is needed?13:44
hewbroccaIf you can do it quickly13:45
hewbroccaI mean, it's either that, or revert the whole thing13:45
hewbroccathose are the two options13:45
jaygwell, I am not suggesting I can, but I dont see how a revert is an option13:45
jaygwe have broken keystone or broken upgrades without this13:45
jistrjayg: +1 such two patches would fix it. I brought it up because if we can't do that in time (as i mentioned folks are tasked with Liberty a lot), then we have a broken Mitaka on the freeze date.13:46
jaygjistr: ok, then I am agreeing, I think I missed some of the backchat13:46
ayoungjistr, lets not focus on Freeze date.  That is feature freeze, and this would be a bug fix13:46
jaygah, that is a good point13:47
jistrok yea13:47
ayoungthe time from M3 to release is for stabilization, just this sort of thing.13:47
EmilienMjistr: I'm not in favor of reverts in general, except if CI is broken or if something is blocked.13:47
hewbroccathe problem is it is blocking upgrade CI13:47
*** Goneri has joined #tripleo13:47
ayounghewbrocca, agreed, so lets get that working.13:47
hewbroccaand that is a huge problem for us right this second13:47
EmilienMbut since OOO CI does not have testing for upgrade, I'm not in favor of any revert around keystone/wgsi13:47
jaygI think we have agreed it just needs to be fixed, not reverted13:48
EmilienMsorry I did not follow all discussion13:48
jaygit is just a prioritization and timing discussion now13:48
jaygs'okay, it was long :)13:48
ayoungfabbione, can you work up a WIP patch along the lines of what you wrote in https://bugs.launchpad.net/tripleo/+bug/153788513:48
openstackLaunchpad bug 1537885 in tripleo "Remove keystone constraints and add the openstack-core dummy resource in its place" [Undecided,New]13:48
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140813:48
fabbioneayoung: no13:48
hewbroccaAll right, but if it is not fixed *very quickly*, then we need to take another look at reverting13:48
ayoungwho can?13:48
fabbioneayoung: i never ever touched tripleo or puppet13:48
fabbionepeople in this channel13:49
fabbione?13:49
hewbroccaI think you guys are missing the point of the time crunch we are in here13:49
jaygso we need to have some people focused on mitaka, since others are focused on liberty/downstream is basically the thing, imo13:49
jayghewbrocca: I dont see how a revert is an option, that may fix upgrade, but it breaks the deployment13:49
fabbioneok extra question13:49
jistryeah, this is not as much of what needs to happen kind of problem, it's probably more of a "who is able *and* has bandwidth to do this before things go very bad" problem13:49
fabbionecan somebody show me the upgrade error?13:50
fabbionemaybe there is a simpler fix13:50
ayounghewbrocca, Tripleo of a large community here, and by going with the wrong qway of deploying Keystone it has essentially introduced a regression that kills a huge swath of functionality.13:50
ayoungso, lets fix it13:50
ayoungfabbione, what *do* you know here?  Pacemaker?13:50
hewbroccafabbione: the "upgrade error" is that the wsgi patch doesn't include any code to change a running eventlet setup over to become a running wsgi setup13:50
hewbroccathat needs to be written13:51
jaygayoung fabbione: be nice  :)13:51
ayoungjayg, no, that was not a ding13:51
fabbionejayg: uh?13:51
ayoungjayg, it was me asking what is his specialty13:51
jaygok ok13:51
jaygyes, he is the pacemaker guru13:51
fabbioneayoung: look at orgchart13:51
jistrfabbione: yeah, i got an error when running our current upgrade code from mitaka->mitaka too, because it doesn't count with openstack-keystone resource not existing. I already fixed that. But the greater problem is migration for existing eventlet deployments to wsgi, as hewbrocca mentioned.13:51
ayoungI don't know who knows what. We need to pull the pieces together.  I know Keystone, and can gfake a few other things13:52
fabbionehewbrocca: ok, then i don't think changing constraint is going to fix it13:52
ayoungI have the utmost respect for all the people here, just trying to let everyone know how important this is13:52
fabbionehewbrocca: it's going to make it hella easier to upgrade13:52
hewbroccafabbione: no. It's a *prerequisite* for fixing it13:52
hewbroccabut it is not itself going to fix it13:52
jistrexactly13:52
fabbioneayoung: we got it. you told that N times, we are overloaded, not hard-headed13:52
jistrfabbione: +1 :)13:53
ayoungfabbione, I'm the heard headed one13:53
fabbioneayoung: and yes I do pacemkaer/HA and a bunch of other stuff13:53
ayounghard13:53
fabbionei just don't write code for OOO13:53
ayoungfabbione, and you are awesome.  Never let it be thought that I indicated differently...13:53
ayoungCool.13:53
bandini_jistr, hewbrocca: so the fix is 1) conversion from eventlet to wsgi part 2) fixing the constraints, yes?13:53
ayoungbandini_, in that order?13:53
fabbionebandini_: fix constraints then conversation13:54
jistrbandini_: yup, exactly. In the other order though.13:54
jistryea13:54
hewbroccaright13:54
bandini_ok13:54
*** ohamada has joined #tripleo13:54
bandini_ayoung: reverse roder ;)13:54
ayoungOK.. fabbione did a pretty clear layout in the bug of what needs to change for the constraints....who can do a WIP patch for that?13:54
bandini_jistr: while we have a fairly good idea in terms of constraint fixing. what's the story about the eventlet conversion? what needs to be done exactly?13:55
fabbionebandini_: basically, stop delete the keystone systemd resource is the first step13:55
jistrbandini_: i think stop and undefine openstack-keystone resource and all its constraints13:55
jistrbandini_: then create the openstack-core and its constraints13:55
fabbionebandini_: then i guess it's moving openstack-keystone under httpd and restart httpd in pacemaker13:55
jistrbandini_: since this can happen during major upgrade, we will have a window where everything is stopped and we can work with the constraints on stopped rather than running resources, if that helps13:56
ayoungbandini_, my guess is that we need to trigger the HTTPD support in the puppet modules.  We do that on a clean install, but I don't know what it means for upgrade13:56
ayoungI know what the end state is supposed to look like13:56
bandini_fabbione, jistr: do we care about the fact that "httpd" at that point mens both keystone and horizon (and possibly more)?13:56
bandini_s/mens/means/13:57
fabbionebandini_: no, we can worry about renaming it at some other times13:57
fabbionefor mikata+113:57
fabbioneor whatever13:57
ohamadahello guys, could you please help - i need to reduce the keystone timeout in the overcloud before deploying the overcloud - like to edit the templates. Could you please point me where to look?13:57
bandini_fabbione: am not worried about the name, am more concerned about "pcs resource restart httpd" now means *two* things in one13:57
jistrbandini_: that's bad, but i'm not sure we can do anything about it13:58
bandini_if we care about that, that is13:58
gfidentejistr, fabbione, ayoung ... any chance we switch to httpd only for new deployments?13:58
ayoungbandini_, from a depednecy graph, we should be OK.  If we restart, say, Nova, we should not restart HTTPD when it means only Horizon.  And with Keystone in there, obviously not restarting HTTPD is the right thing.13:58
ayounggfidente, It means we are going to be chasing Eventlet bugs in the new release13:58
bandini_ayoung: ack13:59
jistrgfidente: i don't think so... puppet manifest in new t-h-t will try to create the new constraints, so you'd have 2 sets of competing constraints (old + new), or something would plainly error out, if you don't have the migration logic13:59
bandini_ayoung: the only minor thing is that now restarting the httpd service means restarting both horizon and keystone. no biggie I guess, but worth being aware of13:59
ayoungit also means there is no way to get the Keystone HTTPD specific features, and that is a pretty major bummer.  And eventlet support is depracated.13:59
gfidentejistr, yeah was just raising a point about making httpd an optional thing13:59
jaygbandini_: I would propose dealing with that is patch 3, and maybe can fall to bugfix time?14:00
jistrgfidente: oh you mean support *both* options in t-h-t... hmm i think it's easier to do the fix maybe..14:00
gfidenteahahah14:00
gfidenteI meant both yes14:00
jaygalso, there is no eventlet support in mitaka, so how can we support it?14:00
ayoungbandini_, it won't hurt anything if it is just a restart.  Shutting it off and leaving it off is no different than doing that with Just Keystone...I think it is status quo?14:00
bandini_jayg: +1 definitely the last thing to look at in terms of priorities14:00
gfidenteok sorry about noise14:01
bandini_ayoung: yeah I think it is not a big deal, as long as we document it somewhere14:01
ayoungOK...I had a WIP patch that I killed...let me see:14:01
fabbionebandini_: I am still waiting a response for that.14:01
ayounghttps://review.openstack.org/#/c/276285/14:01
fabbionebandini_: ideally we would have separated http resources for each WSGI service14:01
jaosoriorWhen is the freeze?14:02
*** saneax_AFK is now known as saneax14:02
hewbrocca3 March14:02
hewbrocca-ish14:02
fabbionebandini_: but apparently we don't have time to get it right and IIRC OSPd right now deploys with systemd:httpd and not ocf::14:02
ayoungjaosorior, M3.  But that is Feature freeze. The real deal is that CI is broken14:02
fabbionebandini_: multiple httpd instance require ocf14:02
ayoungfor upgrades from L->M14:02
jistrto clarify: we don't have that ^ CI yet, but this would be another blocker on the road between now and having it14:03
ayoungOK...so, first question.  Is  https://review.openstack.org/#/c/276285/1/manifests/wsgi/apache.pp   even the right file?14:03
*** weshay has joined #tripleo14:03
bandini_fabbione: ah righto, so that would be another step to go through14:04
fabbioneyes14:04
ayoungBack in a bit..appointment14:04
*** ayoung has quit IRC14:04
fabbionesomebody decided to go for this intermediate dirty solution as interim step14:04
fabbionenot my decision, but who am I to enforce it?14:05
jaygjistr: could the upgrade CI perhaps be held off until bugfix time, to allow a bit of breathing room for this?14:05
hewbroccajayg: unfortunately it means we have no way to test upgrades upstream14:06
hewbroccawhich is really important14:06
jaygfwiw, this patch has been wip for many months, and we have been unable to get everyone who should care to be involved, largely due to focus on liberty instead of mitaka14:06
hewbroccajayg: *facepalm*14:06
jistrjayg: i'm actually thinking now that it's a possibility that we'll only get the upgrade CI working from mitaka to newton. Not that i like that, but it might happen. That doesn't mean we can skip solving the problem, it only means we'll hit it downstream rather than upstream (sigh, again).14:07
jaygI think we need to get a bit of split here to allow people to focus on each release14:07
hewbroccaWell, hopefully we can fix it14:07
hewbroccaEmilienM, mburned WDYT14:07
jaygso a team focused on master +upgrade, and another focused on stabilization of master -114:07
jaygI feel everyones attention is currently too divided14:08
hewbroccajayg: it's not a bad idea, although I think the first thing is a prequisite for the second thing14:08
hewbroccabecause we're so late with Liberty14:08
*** chem has joined #tripleo14:08
hewbrocca(otherwise it wouldn't be, necessarily)14:08
jaygyeah, but if we keep focusing on only master-1, it will keep happening that way for each release14:09
*** bandini_ is now known as bandini14:09
hewbroccajayg: totally agree. We should be focusing on master +upgrade right now14:09
hewbroccab/c if we don't we have no CI and we're just kicking the can down the road14:09
hewbrocca(which is why I would rather not land this wsgi thing right this minute, but apparently we have no choice)14:10
jaygright, maybe this should be a 'feature team', even though those are mostly right now for liberty?14:10
jaygsadly we do not, we got to this point because we did not have enough people focused on mitaka in the particular area14:10
jaygso we have steps, we just need to allocate people to fix it14:11
hewbroccaEmilienM: ^^^ this is for you to sort out when you get back I think14:11
dtantsurfolks, I have an excellent offer for you! you can reduce your agent images by ~ 80 MiB (!!) by only approving https://review.openstack.org/#/c/279453/ :)14:14
dtantsuralready has 1 +2 and passed the gate14:14
hewbroccadprince: ^^^ little help please?14:14
*** julim has joined #tripleo14:15
jaygwhile we wait for EmilienM to return, anyone have a pointer to the doc ohamada needs for configuring keystone timeout in the overcloud? I do not know where to point him14:15
jistrbandini: this is current cib http://paste.fedoraproject.org/328410/32331314/14:15
* jayg uses this lull to take the dogs for a walk, bbs14:15
*** jaosorior has quit IRC14:18
*** jaosorior has joined #tripleo14:19
*** lblanchard has joined #tripleo14:22
jistrohamada, jayg: depends on what timeout is meant, but if it's something in keystone.conf, then that can be customized in hiera here https://github.com/openstack/tripleo-heat-templates/blob/838c998d3d6b3f3703f1ba3fea7a47c862e09212/puppet/hieradata/controller.yaml#L50-L5414:22
*** tiswanso has joined #tripleo14:22
openstackgerritDmitry Tantsur proposed openstack/instack-undercloud: Enable extra hardware data collection and processing for ironic-inspector  https://review.openstack.org/27797414:23
jistrohamada: instead of editing that file, the hiera parameter can be overriden via the ExtraConfig heat stack parameter: https://github.com/openstack/tripleo-heat-templates/blob/9e473e4b54197811ede59b07bfb3d0e79475e2e1/overcloud.yaml#L40614:23
jistrohamada: could be done via a Heat environment file passed into `openstack overcloud deploy`. The environment file would look something like this: http://fpaste.org/328418/23926145/14:26
ohamadajistr: thank you, i want to reduce the keystone default timeout of 14400 secs - my deployment of overcloud is failing constantly but i do not want to wait four hours each time14:26
jistroh then i guess you want to set *undercloud* keystone timeout14:26
jistrohamada: and i guess you mean the keystone token validity timeout?14:27
ohamadajistr: probably - i'm not sure. The deployment always get stuck and then 'Authentication error' occurs which kills it14:28
jistrohamada: alternatively, i think if you want to get a heat deployment stuck in CREATE_IN_PROGRESS to go to CREATE_FAILED, you can `systemctl restart openstack-heat-engine` on the undercloud. I think (but i'm not sure) that this shouldn't have any disastrous side effects for the overcloud stack.14:29
hewbroccajistr: when are we getting abort support in Heat14:29
hewbroccaMitaka?14:29
hewbroccaI know there was talk of it14:29
* jistr doesn't know14:29
hewbroccazaneb: ^^^ "abort" support in Heat was coming, wasn't it?14:29
jistrohamada: the token expiration can be set in keystone.conf in [token] section in the expiration key, and then you would restart keystone for the change to take effect. But i think the heat-engine restart might be a preferred solution.14:33
dprincehewbrocca: sorry, what exactly did you need help with?14:34
ohamadajistr: thanks, the heat-engine restart works, however there are more failed jobs then before14:34
*** saneax is now known as saneax_AFK14:37
hewbroccadprince: dtantsur needs a +A14:41
hewbrocca<dtantsur> folks, I have an excellent offer for you! you can reduce your agent14:41
hewbrocca           images by ~ 80 MiB (!!) by only approving14:41
hewbrocca           https://review.openstack.org/#/c/279453/ :)  [15:14]14:41
hewbrocca<dtantsur> already has 1 +2 and passed the gate14:41
hewbrocca14:41
dtantsur:)14:41
mburned+114:42
mburnedhewbrocca: sorry, was not watching this channel14:43
mburnedhewbrocca: the thought of splitting focus between master+1 and master-1 for upgrades?14:43
fabbionebandini: ping14:44
bandinifabbione: pong14:44
fabbionebandini: change of plans, can you please help hewbrocca and jistr with those pacemaker constraints 100% of your time?14:44
fabbionehewbrocca: ^^ please make sure somebody can at least review the patches and help if necessary14:45
fabbionebandini: put autofencing and IHA on hold14:45
*** ayoung has joined #tripleo14:46
bandinifabbione: ack14:47
bandinijistr: would you have 10mins this afternoon for a quick call (I need to understand how I am going to test the upgrade exactly as I am not sure atm)?14:48
dtantsurfolks, also an important patch bring UEFI local boot support: https://review.openstack.org/#/c/277842/ (backport to liberty, CI passed)14:49
hewbroccawut UEFI14:49
jistrbandini: sure14:51
ayoungdid you guys solve everything while I was away?14:52
jaygayoung: yep, we are going to just not use keystone  ;)14:52
ayoungjayg, I'd be fine with that14:52
bandinilol14:53
jayghaha14:53
ayoungKeystone really should not exist anyway14:53
fabbionedirect access to /etc/passwd14:53
fabbioneun-encrypted!14:53
jaygbut then won't the arch collapse?14:53
* jayg steps out of the entryway14:53
ayoungfabbione, Kerberos and LDAP.14:53
ayoungjayg, the only thing that Keystone really provides is the service catalog.  It could be replaced with a static JSON page rturned when you do curl $OS_AUTH_URL14:54
ayoungbut then you would have to authenticate at Nova, Neutron, and Glance etc...and then all those services would have to be run in HTTPD14:55
ayoung:)14:55
jaygsounds simple enough </sarcasm>14:55
*** ayoung has quit IRC14:59
lucasagomeshi all, someone can take a look at https://review.openstack.org/#/c/277842/ please? This is the liberty backport needed for EFI15:02
*** pblaho has quit IRC15:05
*** eggmaste` is now known as eggmaster15:06
zanebhewbrocca: depends what you mean by 'abort'. if something fails then all of the nested stacks will be aborted as of Liberty (in fact they also were pre-Kilo - only Kilo was broken)15:07
zanebhewbrocca: we also have had an 'update-cancel' API since at least Liberty, BUT it always does a rollback15:08
zaneband rollback is basically never what you want for TripleO15:08
*** fgimenez has quit IRC15:11
*** fgimenez has joined #tripleo15:13
*** fgimenez has quit IRC15:13
*** fgimenez has joined #tripleo15:13
*** xinwu has joined #tripleo15:15
EmilienMjistr: why not landing https://review.openstack.org/#/c/270700/ ?15:16
EmilienMCI is green, 3 +2 !!15:16
hewbroccazaneb: ahh, cancel15:16
hewbroccathat was the thing we were looking for15:16
hewbroccaexcept, you're right, always doing a rollback is not what's needed I don't guess15:17
zanebhewbrocca: yeah, you don't want to try rolling back a tripleo deployment15:17
hewbroccaweeeeeeeee15:17
jistrEmilienM: it depends on an unmerged patch15:17
EmilienMjistr: what do we want to land it?15:18
*** fgimenez has quit IRC15:20
*** tosky has quit IRC15:22
*** nkinder has joined #tripleo15:23
*** tosky has joined #tripleo15:23
*** fgimenez has joined #tripleo15:24
*** fgimenez has quit IRC15:24
*** fgimenez has joined #tripleo15:24
*** mcornea has quit IRC15:26
derekhFYI : Via #rdo meeting #agreed rename openstack-packages/delorean to fluzo15:28
*** penick has joined #tripleo15:29
paramitefluzo?15:29
paramitederekh, what's wrong with delorean?15:30
trownparamite: already a project on pypi15:30
derekhdprince: the next time you are playing on the rh1 rack, sshuttle is extremely useful, no more messing with tunnels and X forwarding https://goodsquishy.com/2016/02/sshuttle-where-have-you-been-all-my-life/15:30
*** mcornea has joined #tripleo15:30
derekhparamite: and a possible trademark issue15:30
paramiteah I see15:30
dprincederekh: nice15:31
* derekh didn't worry about such things 2 years ago15:31
* trown has to change his github picture now15:32
*** paramite is now known as paramite|afk15:32
dprincederekh/paramite: I'm a holdout for delorean15:32
dprincefluzo sounds like a disease man15:33
trownlol15:33
derekh:-)15:33
trownit is a mistranslation of flux in spanish that was used in the subtitles for back to the future15:33
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Source undercloud environment variable from a file  https://review.openstack.org/27566715:35
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Split the deploy script into its own file  https://review.openstack.org/27566815:35
*** thrash is now known as thrash|biab15:37
*** panda has quit IRC15:37
*** paramite|afk has quit IRC15:38
*** panda has joined #tripleo15:38
*** yamahata has joined #tripleo15:39
*** fgimenez has quit IRC15:51
*** fgimenez has joined #tripleo15:52
*** fgimenez has quit IRC15:52
*** fgimenez has joined #tripleo15:52
*** paramite|afk has joined #tripleo15:53
*** jaosorior has quit IRC15:55
*** penick has quit IRC15:58
*** paramite|afk has quit IRC15:58
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Set notification driver for nova to send  https://review.openstack.org/28368615:58
*** oshvartz has quit IRC16:00
*** hjensas has quit IRC16:10
*** yamahata has quit IRC16:10
*** leanderthal is now known as leanderthal|afk16:14
openstackgerritEvgeny Bagdasaryan proposed openstack/tripleo-heat-templates: Change the default value for NetworkNexusVxlanGlobalConfig  https://review.openstack.org/28195716:14
*** rajinir has joined #tripleo16:19
*** ayoung has joined #tripleo16:20
*** mcornea has quit IRC16:21
*** liverpooler has quit IRC16:22
*** rpothier has joined #tripleo16:26
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks  https://review.openstack.org/23542316:28
*** tiswanso has quit IRC16:31
*** tiswanso has joined #tripleo16:32
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Nova RPC unpinning  https://review.openstack.org/28423916:32
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration  https://review.openstack.org/18813716:36
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Add integration with NetApp Manila driver  https://review.openstack.org/18813816:37
*** jdob has quit IRC16:43
dmsimardzaneb: re: endpoint map16:44
zanebdmsimard: yeah16:44
dmsimardtbh I have no idea, there's a lot of things I don't understand in tripleo - I made the patch with the help of other people here that pointed me in the right direction16:44
*** jistr has quit IRC16:45
dmsimardIf there's something wrong, it didn't come up in the reviews until now16:45
trownzaneb: we can always do it right now that we have moved CI into the present wrt delorean16:45
zanebdmsimard: that's fine, I don't think it's critical. we can easily fix up with a follow-up patch16:45
trownzaneb: we were testing tripleo against code from 30 days ago in other projects though16:46
openstackgerritZane Bitter proposed openstack/tripleo-heat-templates: Generate the endpoint map statically  https://review.openstack.org/27543716:48
trownzaneb: glad to be fodder for your argument on that patch :)16:49
trownI definitely agree16:49
*** Goneri has quit IRC16:49
zanebtrown: can't overlook a gift like that ;)16:50
*** pblaho has joined #tripleo16:50
*** aufi has quit IRC16:51
*** mcornea has joined #tripleo16:52
*** chem has quit IRC16:52
dmsimardhaha, sorry, it was definitely a time sensitive patch and even then it took weeks to get it merged16:54
dmsimardwe definitely wanted to base our patch on top of yours16:54
trowndmsimard: funny (well not so funny really) thing is that if we stuck with the endpoint generator based patch, it was correct16:55
trownso we could have updated trunk 2 weeks ago16:56
dmsimardbleh16:56
dmsimardbut the generator patch hasn't landed yet16:56
*** Guest51435 is now known as mgagne16:56
*** mgagne has quit IRC16:56
*** mgagne has joined #tripleo16:56
trownI switched to the master one to try to speed things up, but then got lost in an issue I did not attribute to a mistake in that patch16:56
trowna pretty strong argument for the generator patch16:57
*** trown is now known as trown|lunch16:57
*** nico_auv has quit IRC17:01
*** ifarkas has quit IRC17:02
*** mikelk has quit IRC17:02
EmilienMayoung: chem (offline now) is working on the puppet-pacemaker provider17:04
EmilienMwe hope we'll be able to tell puppet "do not manage the service with systemd, just run pcs"17:04
ayoungEmilienM, cool.  Feel free to bug me (and pass that on) if there are any questions on the Eventlet to HTTPD transition on Upgrade.  Or anything else around this, of course17:05
*** yamahata has joined #tripleo17:06
*** CaptTofu___ is now known as CaptTofu17:07
*** tosky has quit IRC17:12
openstackgerritZane Bitter proposed openstack/tripleo-heat-templates: Don't have separate protocols/ports for Keystone v3  https://review.openstack.org/28426517:13
zanebdmsimard, trown|lunch: follow-up ^17:14
dmsimardzaneb: I think that might require a change in the heat/hiera files but I'm a noob17:15
dmsimardi.e https://review.openstack.org/#/c/279184/3/puppet/compute.yaml and https://review.openstack.org/#/c/279184/3/puppet/controller.yaml17:15
zanebdmsimard: I don't believe so. those files only look at the *output* of that stack. I'm changing the *input*17:15
dmsimardbut those are expecting a key in the endpoint map to be called KeystoneV3Admin (for example) no ?17:16
dmsimardwith your patch that no longer exists iiuc17:17
zanebso what's confusing is that both the input and the output are called EndpointMap17:17
dmsimardconfusing things? in tripleo? :P17:17
zanebbut there's nothing called "uri" in the input (only host, port, protocol). that's definitely the output17:18
dmsimardyou're the expert though, just wanted to point that out and if you see it as a non-issue that's fine17:18
zaneblol, I'm really not ;)17:18
zanebbut how it works is you have an input called 'EndpointMap' with host, port, protocol. you then launder it through that stack, and also call the output (with host, port, protocol, uri, base_uri) EndpointMap. then you use the output as an input to other things like puppet/compute.yaml17:20
openstackgerritEvgeny Bagdasaryan proposed openstack/tripleo-heat-templates: Add BondInterfaceOvsOptions parameter to net-config-bond.yaml  https://review.openstack.org/24508617:23
openstackgerritJason Guiditta proposed openstack/puppet-tripleo: loadbalancer: fix MySQL timeout HAproxy config  https://review.openstack.org/28427417:26
*** fgimenez has quit IRC17:27
*** dtantsur is now known as dtantsur|afk17:31
*** Goneri has joined #tripleo17:33
*** xinwu has quit IRC17:45
*** trown|lunch is now known as trown17:50
*** lucasagomes is now known as lucas-hungry17:51
*** rdopiera has quit IRC17:54
*** Marga_ has quit IRC17:55
*** mkovacik_ has quit IRC17:59
*** thrash|biab is now known as thrash18:00
*** jdob_lt has joined #tripleo18:01
*** eil397 has joined #tripleo18:01
*** derekh has quit IRC18:01
*** olap has quit IRC18:02
*** julim has quit IRC18:03
*** julim has joined #tripleo18:07
*** gfidente has quit IRC18:14
*** xinwu has joined #tripleo18:16
*** jtomasek has quit IRC18:21
*** ohamada has quit IRC18:30
*** electrofelix has quit IRC18:30
*** shivrao has joined #tripleo18:31
*** Marga_ has joined #tripleo18:34
*** olap has joined #tripleo18:38
*** liverpooler has joined #tripleo18:41
openstackgerritMerged openstack/tripleo-heat-templates: Enable configuration of Neutron QoS  https://review.openstack.org/27213118:42
openstackgerritMerged openstack/tripleo-heat-templates: Adds v6 capability to the deploy validation test (pings)  https://review.openstack.org/26810318:46
*** jcoufal has quit IRC18:48
*** Marga_ has quit IRC18:50
*** Marga_ has joined #tripleo18:51
*** paramite|afk has joined #tripleo18:51
*** rajinir has quit IRC18:55
*** dustins has joined #tripleo18:57
*** mgould has quit IRC19:02
*** paramite|afk has quit IRC19:09
zigoCould someone review this one? https://review.openstack.org/#/c/211859/ It's blocking the addition of a Debian image in the infra, and it's really a huge blocker for me.19:14
*** athomas has quit IRC19:14
zigoIt's been there since October, so I'm really worried here...19:14
zigoSpamapS: Since nobody is reviewing it, maybe you could workflow it? ^19:15
*** mcornea has quit IRC19:17
*** mkovacik has joined #tripleo19:24
*** eil397 has quit IRC19:29
SpamapSzigo: ACK, I'll pioritize today19:29
SpamapSzigo: oh, I +2'd ...19:30
SpamapSzigo: greghaynes is usually pretty quick on those reviews but he's out on leave for a bit.19:30
SpamapSianw: can you maybe take a look at 211859 ?19:30
b3nt_pinsanity check: are the instructions on http://docs.openstack.org/developer/tripleo-incubator/devtest.html expected to work? (more or less)19:32
trowndoes anyone know if these scale docs are accurate? http://docs.openstack.org/developer/tripleo-docs/post_deployment/scale_roles.html19:32
trownb3nt_pin: highly doubtful19:32
b3nt_pintrown: heh okay19:32
b3nt_pintrown, the next question is do we *want* them to work? ;)19:33
b3nt_pinin other words, is it basically obsolete or is it just something that is bit-rotting at the moment19:33
trownb3nt_pin: I think they are obsolete, devtest is not used in CI19:34
b3nt_pintrown: okay cool19:34
trownb3nt_pin: http://docs.openstack.org/developer/tripleo-docs/ is what you want19:34
*** dustins_ has joined #tripleo19:34
trownb3nt_pin: if those don't work, bugs and patches are welcome19:35
*** dustins has quit IRC19:35
b3nt_pintrown: is that how people are making patches? hand craft in that deployed environment and then patch the code in git?19:36
trownb3nt_pin: probably depends on the person, and on the patch... for tripleo-heat-templates, I just git clone to the home dir and deploy using those templates19:37
trownso putting up a patch is simple from taht19:37
*** dustins_ is now known as dustins19:38
trownother stuff, I build a package for the change using delorean and then install that package to test19:38
trownwhich is what CI does19:38
b3nt_pintrown: gotcha19:39
*** yamahata has quit IRC19:41
*** trown is now known as trown|brb19:45
*** trown|brb is now known as trown19:49
*** dustins has quit IRC19:50
openstackgerritBen Nemec proposed openstack/python-tripleoclient: Refresh node power state at start of loop  https://review.openstack.org/28380919:50
*** rpothier has left #tripleo19:56
*** penick has joined #tripleo20:01
dmsimarddang20:02
dmsimardthe tripleo owl has some serious competition20:02
dmsimardhttps://github.com/openstack/dragonflow20:02
*** mkovacik has quit IRC20:05
trownfire-breathing cat5? no thanks20:11
*** dustins has joined #tripleo20:17
Erming_trown: can I have question for you?20:32
trownErming_: sure20:33
Erming_trown: awesome. After your undercloud env setup (undercloud.qcow2), then I run the following command:20:33
Erming_oenstack overcloud deploy --templates --control-scale 3 -e /usr/share/openstack-tripleo-heat-templates/environments/puppet-pacemaker.yaml --ntp-server pool.ntp.org  --ceph-storage-scale 1 --templates -e /usr/share/openstack-tripleo-heat-templates/environments/storage-environment.yaml  --libvirt-type qemu20:34
trownErming_: with you so far :)20:34
Erming_trown: while it had a ' no valid host' error. I found it could be because that it creates a new instance on a already assigned virtual node.20:35
Erming_that is, it may create more vms than the number of nodes specified in instackenv.json. (I didn't change it at all)20:36
Erming_trown: so do you think my command is right?20:36
trownErming_: the command is right, I do not follow your logic on the no valid host though20:37
Erming_trown: because in your instackenv.json, there are 3 controls, 1 compute, 1 ceph20:37
trownErming_: for that, you want to look in nova-scheduler.log for which filter returned 0 hosts20:38
Erming_I give 3 controls and 1 ceph in the command. But in the log file, I saw20:38
Erming_an instance was trying to be created on a node that has already a VM running.20:39
trownErming_: so there is a nova scheduler bug like that, but I do not see it often these days20:40
trownErming_: https://bugs.launchpad.net/nova/+bug/134142020:40
openstackLaunchpad bug 1341420 in OpenStack Compute (nova) "gap between scheduler selection and claim causes spurious failures when the instance is the last one to fit" [Wishlist,In progress]20:40
trownErming_: do you reproduce it every time?20:41
Erming_trown: I just tried once. and just get time to resume this stuff.20:43
trownErming_: ok I would try to delete the stack and redeploy20:46
Erming_trown: another suspicion is that I only have 90GB left in / space, could it be the no valid host cause?20:46
trownErming_: well, no valid host is a super generic error, that is why I said look in the scheduler logs20:46
Erming_trown: unfortunately I had removed the stack. so scheduler logs .. let me find the original error for you.20:47
*** Marga_ has quit IRC20:48
*** Marga_ has joined #tripleo20:48
Erming_trown: I checked the flavor and the node themselves, don't see inconsistencies.20:49
*** mkovacik has joined #tripleo20:51
Erming_trown: sorry I don't know why the previous error log is missing in the nova-scheduler log. I am going to redo it. you think the 90GB / space is not a issue? and the openstack undercloud deploy command is also good?20:52
trownErming_: deploy command definitely looks right... I doubt that the 90GB of storage was related20:53
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140820:54
trownErming_: if you saw nodes being scheduled to already scheduled nodes, that sounds like that nova scheduler bug I posted... which is a race condition, so maybe you will get luckier the second time around :)20:54
Erming_trown: Hmm. OK. I will do lvresize, and then re-try. before that, just to confirm, for doing it on virtual environment instead of baremetal, I don't need do anything extra other than the this instrcution:20:55
Erming_"Then proceed with the upstream documentation for the rest of the deployment."  --> http://docs.openstack.org/developer/tripleo-docs/basic_deployment/basic_deployment_cli.html#upload-images20:56
trownErming_: yep, assuming you ran `openstack undercloud install` before that20:56
*** penick has quit IRC21:00
Erming_trown: if one vm needs 40GB, 5 needs 200GB, where are the VM files located?21:08
Erming_trown: I am looking at the bug you mentioned at the same time.21:08
trownErming_: by default they would be in /var/lib/libvirt/images/, but the backing storage is thin provisioned21:10
*** pcaruana has quit IRC21:10
trownErming_: so you probably only need 20-40G of space21:10
Erming_trown: OK if so , it shouldn't an issue. as I linked  /var/lib/libvirt/images/ to /cloud which has 300GB21:11
Erming_trown: sounds that I only need try again with running the same command.21:11
trownErming_: ya, and get scheduler logs if it fails again... ironic-conductor logs would be useful if it was actually some storage related issue21:12
trownbut I dont think that is the problem21:12
Erming_OK. trying!21:13
*** Goneri has quit IRC21:13
trownErming_: I am taking off, but let me know how it goes21:14
*** trown is now known as trown|outtypewww21:14
Erming_trown|outtypewww: I will. thanks!21:14
ianwSpamapS: sorry, i probably should have stated my opinion with a vote there.  but i just see it as a really undefined layer that will accumulate "stuff" that makes sense for someone at some time, but never leaves21:15
ianwSpamapS: i much prefer to see a more fine grained approach where elements that need things add those dependencies directly21:16
*** dustins has quit IRC21:36
*** dustins has joined #tripleo21:38
*** rbrady-run is now known as rbrady21:40
openstackgerritMerged openstack/tripleo-heat-templates: Add TripleO Heat Template Parameters for Neutron Tenant MTU  https://review.openstack.org/28085221:41
*** dshulyak has quit IRC21:45
openstackgerritMerged openstack/tripleo-heat-templates: Allow the deployer to pick a predefined IP for VIPs  https://review.openstack.org/27631621:47
openstackgerritOpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements  https://review.openstack.org/26852821:50
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements  https://review.openstack.org/26854921:51
openstackgerritBen Nemec proposed openstack/python-tripleoclient: Refresh node power state at start of loop  https://review.openstack.org/28380921:52
*** jdob_lt has quit IRC22:01
openstackgerritxin wu proposed openstack/os-net-config: Enable os_net_config to configure IVS  https://review.openstack.org/27449222:07
openstackgerritxin wu proposed openstack/os-net-config: Enable os_net_config to configure IVS  https://review.openstack.org/27449222:08
*** jayg is now known as jayg|g0n322:09
openstackgerritMerged openstack/instack: remove python 2.6 trove classifier  https://review.openstack.org/26079022:11
*** lblanchard has quit IRC22:14
*** dustins has quit IRC22:17
openstackgerritMerged openstack/tripleo-heat-templates: Rename tox env to pep8  https://review.openstack.org/27922522:25
*** dprince has quit IRC22:26
*** penick has joined #tripleo22:27
*** julim has quit IRC22:33
*** tiswanso has quit IRC22:37
openstackgerritBen Nemec proposed openstack/instack-undercloud: Unset DIB_YUM_REPO_CONF before running undercloud install  https://review.openstack.org/28444422:43
openstackgerritMerged openstack/instack-undercloud: Unset DIB_YUM_REPO_CONF before running undercloud install  https://review.openstack.org/27155722:46
openstackgerritMerged openstack/os-apply-config: Deprecated tox -downloadcache option removed  https://review.openstack.org/25680322:55
openstackgerritMerged openstack/os-refresh-config: Deprecated tox -downloadcache option removed  https://review.openstack.org/25680522:56
openstackgerritMerged openstack/python-tripleoclient: Install bigswitch networking agent by default  https://review.openstack.org/28253422:56
openstackgerritMerged openstack/tripleo-puppet-elements: Deprecated tox -downloadcache option removed  https://review.openstack.org/25684223:01
openstackgerritBen Nemec proposed openstack/instack-undercloud: Enable notifications on undercloud  https://review.openstack.org/24134123:08
openstackgerritBen Nemec proposed openstack/instack-undercloud: Enable notifications on undercloud  https://review.openstack.org/24134123:09
*** thrash is now known as thrash|g0ne23:12
openstackgerrityolanda.robla proposed openstack/diskimage-builder: Add dib element to generate logical volumes  https://review.openstack.org/25204123:12
openstackgerritRyan Brady proposed openstack/tripleo-common: Add Heat Capabilities Actions  https://review.openstack.org/28446023:19
*** saneax_AFK is now known as saneax23:21
openstackgerrityolanda.robla proposed openstack/diskimage-builder: Add dib element to generate logical volumes  https://review.openstack.org/25204123:22
*** pradk has quit IRC23:24
*** panda has quit IRC23:37
*** panda has joined #tripleo23:38
openstackgerritBen Nemec proposed openstack/instack-undercloud: Enable notifications on undercloud  https://review.openstack.org/24134123:44

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