Monday, 2016-08-29

*** Ryjedo_ has joined #tripleo00:02
*** Ryjedo has quit IRC00:03
*** Ryjedo_ is now known as Ryjedo00:03
*** dmacpher has joined #tripleo00:11
*** jobewan has quit IRC00:13
*** limao has joined #tripleo00:46
*** fragatina has joined #tripleo00:48
*** bana_k has joined #tripleo00:56
*** Goneri has joined #tripleo01:00
*** bfournie has joined #tripleo01:08
*** gregwork has quit IRC01:10
*** NachoDuck has quit IRC01:11
*** NachoDuck has joined #tripleo01:24
*** gregwork has joined #tripleo01:26
*** Goneri has quit IRC01:46
*** Goneri has joined #tripleo01:52
*** bana_k has quit IRC01:56
*** yamahata has quit IRC02:14
*** Goneri has quit IRC02:30
*** apetrich has quit IRC02:38
*** apetrich has joined #tripleo02:39
*** limao has quit IRC03:11
*** rhefner has joined #tripleo03:42
openstackgerritKarthik S proposed openstack/puppet-tripleo: Configure the numvfs for SRIOV interfaces
*** bana_k has joined #tripleo03:57
*** limao has joined #tripleo04:10
*** chlong has quit IRC04:12
*** limao has quit IRC04:15
*** chlong has joined #tripleo04:25
*** coolsvap_ has joined #tripleo04:28
*** limao has joined #tripleo04:28
*** coolsvap_ is now known as coolsvap_\04:28
*** coolsvap_\ is now known as coolsvap04:28
*** bana_k has quit IRC04:29
*** links has joined #tripleo04:33
*** numans has joined #tripleo04:34
*** fragatin_ has joined #tripleo04:45
*** fragati__ has joined #tripleo04:48
*** fragatina has quit IRC04:48
*** fragatin_ has quit IRC04:50
*** jaosorior has joined #tripleo04:52
*** fragati__ has quit IRC04:53
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Create entries for overcloud VIPs in /etc/hosts
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add HAProxy TLS handled by certmonger as composable service
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: DO NOT MERGE - TESTING BARBICAN
*** shardy has joined #tripleo05:17
*** pgadiya has joined #tripleo05:24
*** bana_k has joined #tripleo05:27
*** akuznetsov has joined #tripleo05:28
*** florianf has joined #tripleo05:40
Jokke_morning folks05:44
*** skramaja has quit IRC05:45
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Create entries for overcloud VIPs in /etc/hosts
*** liverpooler has joined #tripleo05:50
*** skramaja has joined #tripleo05:50
*** dmacpher has quit IRC05:51
jaosoriorshardy: there was an error in heat that was causing some errors that look like: UPDATE_FAILED NotFound_Remote: resources[0]: Software config with id aff00d7a-88ea-4775-a247-77b69614e401 not found05:53
jaosoriorwas taht fixed?05:54
*** aufi has joined #tripleo06:01
*** saneax-_-|AFK is now known as saneax06:06
*** leanderthal|afk is now known as leanderthal06:07
openstackgerritBabu Shanmugam proposed openstack/puppet-tripleo: Add ML2/OVN support for neutron profile
shardyjaosorior: Hi, yeah I saw the same thing on update, I believe the problem has been fixed in heat master, but not yet promoted to current-tripleo06:10
shardyjaosorior: was the final part of the fix I believe06:11
*** akuznetsov has quit IRC06:14
openstackgerritOpenStack Proposal Bot proposed openstack/os-collect-config: Updated from global requirements
openstackgerritOpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-validations: Updated from global requirements
Jokke_jaosorior, shardy: any chance to get +A for couple of +2s already and finally all tests green?06:25
*** fragatina has joined #tripleo06:25
*** fragatina has quit IRC06:26
*** fragatina has joined #tripleo06:26
Jokke_and would really like to get this reviewed as well, last link on the chain
shardyJokke_: approved the first one06:28
shardythe pattern with defaulting the parameters to hiera for a different module is weird, but I see it's been discussed earlier in the series06:28
Jokke_cheers ... yeah we had bit of back and forth how this should be done06:29
shardywhy are we adding all backends to manila-base.yaml?06:30
mariosmorning o/06:31
shardyI commented on another patch adding a netapp backend, we'll end up with one manila-base that contains all the backends06:31
shardymarios: aha, good morning, was just about to ping you re this manila stuff :)06:31
Jokke_shardy: I followed the example of the previous work and what marios did in ... so from my part that was for consistency06:32
mariosshardy: i had a quick look at the comments thanks for looking. yeah so it is possible for multiple backends to be enabled at the same time. it is kinda weird, you're right. for example, having a dedicated 'ManilaGenericEnabled' and also 'ManilaNetapEnabled'06:32
Jokke_morning marios06:32
mariosshardy: so i thought it would somehow keep it simpler by having all the config in the base class, enabled only by that param from the environment file,06:33
mariosJokke_: o/06:33
shardymarios: Yeah, but it's not that consistent with the new composable services interface06:33
shardyI'm wondering if there's a way to rework it so it's more like how we enable backends for neutron06:33
mariosshardy: so it is just a matter of moving the config to a dedicated file. right. so one thing i wanted to do but didn't get to it before i left06:34
mariosshardy: was to tidy up puppet-tripleo seide, since you may have seen, we are setting hiera from tht as well as re-resetting it on puppet-tripleo side unnecessarily06:34
shardymarios: given that we now have two backends trying to add to -base, do you think it's reasonable to just say we've got to break out the backends now?06:35
mariosshardy: so yeah i will have another pass probably first a tidy up to include ^^^ and what you're saying about moving the generic stuff to dedicated  service file06:35
shardyYeah I noticed that06:35
shardyOk, I'll +1 the patches and hopefully we can get those cleanups in06:35
shardyin general alarm bells start going off every time I see an EnableFoo parameter nowadays ;)06:35
mariosshardy: so sure man, i will move/tidy up today at some point. problem is time, my involvement here is as you know 'assist' purely because I have other stuff that people will call me up on if i don't do :)06:36
Jokke_I'm happy to attack it now if someone would give me a pointer how to get it done (would really like to see this work reaching Newton)06:36
mariosshardy: yeah, this was the first questino i asked, about the enable foo06:36
mariosshardy: there is some discussino on the tht06:36
Jokke_fwiw I kind of liked the approach marios took. Looked way cleaner and easier to follow ;)06:37
mariosshardy: i just copied what was in the original reviews at first, but clarified that it is indeed the case that we can have >1 backend enabled simultaneously. we could ofcourse make that a list param 'EnabledBackends'06:37
*** mcornea has joined #tripleo06:37
*** bana_k has quit IRC06:37
mariosshardy: but i think that would add complexity to handle the list. we *do* have a dedicated env file for each backend, and the inclusion of either of those will deploy that backend06:38
shardymarios: Yeah, there is a heat feature to help with that, ramishra finished it, let me see if it landed now06:39
shardymarios: basically instead of defining the entire ControllerServices list, it's possible to define a merge strategy so your environment file just appends to the list06:39
shardywhich will make this much easier/cleaner06:40
shardytwo patches remaining to merge :(06:40
shardyI'll chase for reviews06:40
mariosshardy: ack .. interesting thanks noted06:42
shardymarios: if we have to, I suppose we can land the monolithic -base stuff, provided folks are OK with pretty quickly refactoring to split out the backends06:43
mariosshardy: sure, won't be for a couple hours/after lunch i have a look there so let's see if there are any other thoughts/opinions and i'll put up a revision and we can discuss there?06:44
*** rasca has joined #tripleo06:44
shardymarios: yup, sounds good - it's a public holiday here so I'll be inconsistently available, but will be around later to discuss further if needed06:44
mariosack thanks shardy enjoy06:45
*** rcernin has joined #tripleo06:47
*** jubapa has joined #tripleo06:47
Jokke_shardy & marios: as said I have the cycles to attack that if needed (I just need some pointers how to approach it). I'm fine for either one, hitting it (quite literally) now or doing/helping out the refactoring after my PTO (I'll be back at 12th of Sept)06:49
Jokke_shardy: enjoy the bank holiday. Thanks for jumping in even during it.06:50
mariosJokke_: thanks feel free to do so i will ping you later when i am going to look /and/or you update it before then thanks for any help06:51
mariosJokke_: wrt pointers, maybe more specific question/unknown ? i guess the one question now is whether we keep the config in base and iterate, or we split. the split is simple enough ... look at neutron as shardy says06:52
mariosJokke_: 'simple enough' ... * t&c apply I mean should be but i can't see why it wouldn't be, need to move into dedicated files for each backend06:53
mariosJokke_: so either way, it would be very useful if you would pose that question on the review for other reviewers as a start ?06:53
Jokke_marios: ok, I'll have a look into how the neutron is done and if it is simple enough to understand I propose new PS to that ceph driver first so we can see how it works06:53
mariosJokke_: ceph driver?06:54
shardythere's a ceph and netapp manila backend/driver, both modifying manila-base06:54
mariosJokke_: shardy ah thanks I missed that part of the discussion/those reviews06:55
shardycan you have multiple manila backends?06:55
mariosshardy: yes06:55
mariosshardy: simultaneously apparently06:55
Jokke_shardy: yes and that's the direction we have been working this towards06:55
mariosshardy: see qestion in the early comments at
mariosshardy: and discussino with tbarron06:56
shardyOk, thanks, will check the earlier discussion06:56
* Jokke_ has no idea how one is supposed to schedule those, but that's Manila's problem, not ours. The request was to provide integration for these drivers and wishing that multiple drivers could be enabled simultaneously so apparently they can handle it06:56
*** dsariel has joined #tripleo06:56
*** akuznetsov has joined #tripleo06:57
*** jprovazn has joined #tripleo06:59
*** pcaruana has joined #tripleo07:00
jaosorioreasy commit updating help text if anyone feel like merging code :D
*** tesseract- has joined #tripleo07:01
*** akuznetsov has quit IRC07:02
jaosoriorshardy: aww, still getting that error even with the code you passed me :(07:04
shardyjaosorior: did you pull that and all of the related patches?07:05
shardyI think there were at least two patches related to that bug07:05
jaosoriornot the related patches, no07:05
jaosorioronly that07:05
shardyI'd check the bug to make sure you got them all or pull latest master heat07:05
*** panda has joined #tripleo07:06
jaosoriorshardy: I think I rather just get the latest heat07:06
jaosoriorshardy: is there a way to fetch it from the repos?07:07
*** anshul has joined #tripleo07:08
*** ohamada has joined #tripleo07:09
shardyjaosorior: I just build a delorean package normally, then update the installed version on the undercloud07:09
shardyjaosorior: Or, you can add it to includpkgs in the delorean-current yum.repos.d file07:09
jaosoriorshardy: alright, thanks dude07:10
*** nyechiel has joined #tripleo07:11
*** jpena|off is now known as jpena07:13
*** tremble has joined #tripleo07:14
jaosorioranybody up for merging this commit by shardy? :D07:15
*** ifarkas_afk is now known as ifarkas07:16
shardythere are two more patches in that series too which are passing CI and ready for review:07:17
shardyreviews appreciated! :)07:17
shadowershardy: the first one lgtm. May I?07:18
shardyshadower: sure, thanks!07:19
*** atom_ has joined #tripleo07:19
mariosshardy: ack i have in my list for a revisit, but looks like i will have to stare at the tht filters/yaql for a while but will get to it. also, if you're not going to use your day off, can i have it please?07:19
shardylol :)07:20
* shardy is planning to use some of his day off shortly :D07:20
shardymarios: Yeah, some of the nested intrinsic functions are a little hairy, I'm planning a blog post to hopefully demystify it07:21
shadowershardy: btw the mistral validation series is passing CI, has a +2 and several +1s and is good to go imho:
shardywhen you break it down it's not that hard, but combined I agree it's slightly brain teasing07:21
mariosshardy: cool thanks. yeah i saw a much earlier versino of that where you had a couple of filters but i think you have changed it now so make it much more complicated07:21
mariossimpler :)07:21
openstackgerritMerged openstack/puppet-tripleo: Add Manila CephFS backend to manila class
shardyshadower: thanks, I'll check it out07:22
shardywe need to figure out how to stop the ovb jobs timing out all the time ref the POST_FAILURE's :(07:23
shardygtg for a bit, bbl07:26
*** shardy has quit IRC07:26
Jokke_and Manila folks would be really happy if this one-liner merged, has +2 from shardy already and finally passed all the tests so should be ~freebie
*** r-mibu has quit IRC07:27
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Create entries for overcloud VIPs in /etc/hosts
*** fzdarsky has joined #tripleo07:29
*** akuznetsov has joined #tripleo07:30
*** karthiks_afk is now known as karthikc07:31
*** karthikc is now known as karthiks07:31
*** jpich has joined #tripleo07:37
Jokke_thanks marios :)07:37
*** hewbrocca-afk is now known as hewbrocca07:38
*** ioggstream has joined #tripleo07:39
*** zoli_gone-proxy is now known as zoliXXL07:39
mariosnp Jokke_ :)07:40
hewbroccaMorning folks07:40
zoliXXLgood morning07:41
jaosoriorjistr: ping07:43
hewbroccamarios: welcome back!07:45
marioshewbrocca: o/07:47
mariosthanks :)07:47
hewbroccagfidente was trying to make more bugs while you were away07:48
hewbroccabut I told him to stop it07:48
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Increase timeout for ironic power status report
*** zoliXXL is now known as zoli|brb07:48
jaosoriorsshnaidm: should you be using single quote there?
mariosthanks hewbrocca. but you know, i think we have to accept that it is futile. the gfidente has been released. there is no earthly power to restrain the gfidente. such is the nature of, the gfidente. :(07:49
sshnaidmjaosorior, not sure, but singles are more safe, thanks07:50
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Increase timeout for ironic power status report
jaosoriorsshnaidm: also, I suggest you do that here instead
jaosoriorsshnaidm: that way it will be persisted on the undercloud and will even work if we get an update gate07:54
*** iogg has joined #tripleo07:54
*** ioggstream has quit IRC07:55
sshnaidmjaosorior, yeah, it seems better, what is path to this parameter though?07:55
jaosoriorsshnaidm: I'm figuring that out.07:56
jaosoriorsshnaidm: is post_deploy_get_power_state_retries in the DEFAULT group?07:58
sshnaidmjaosorior, no, it's in [agent[07:58
jaosorioryou can use this07:59
*** tvignaud has quit IRC07:59
*** oshvartz has joined #tripleo07:59
jaosoriorsshnaidm: but the best thing to do is to actually add post_deploy_get_power_state_retries to puppet-ironic08:00
sshnaidmjaosorior, cool, so I can change any value in configurations, right?08:00
jaosoriorsshnaidm: yeah08:00
jaosoriorand actually I wrote it wrong08:00
jaosoriorsshnaidm: ^^08:01
jaosoriorsshnaidm: but yeah, the best thing to do is to add that parameter to puppet-ironic. That way if someone adds that parameter there, they won't break our CI08:01
*** r-mibu has joined #tripleo08:03
*** mpsairam has quit IRC08:03
sshnaidmjaosorior, I'm not sure I can do it, puppet modules are still in my tolearn list08:03
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Increase timeout for ironic power status report
jaosoriorsshnaidm: alright, lets see if we can manage with this for now. I'll try to poke people into how to add taht to pupept-ironic08:05
sshnaidmjaosorior, do you have any example maybe?08:06
*** links has quit IRC08:06
jaosoriorsshnaidm: so, for instance, here I added some parameters to puppet-nova
*** dsariel has quit IRC08:09
sshnaidmjaosorior, ok, maybe next time :)08:09
*** dsariel has joined #tripleo08:09
*** derekh has joined #tripleo08:12
*** vijaykc4 has joined #tripleo08:12
*** tvignaud has joined #tripleo08:13
*** greghaynes has quit IRC08:13
*** lucas-afk is now known as lucasagomes08:21
*** coolsvap is now known as coolsvap_08:21
*** pblaho has joined #tripleo08:21
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: [WIP] Set parameters
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Add 'openstack overcloud plan create' command
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Print the execution ID when starting Mistral workflows
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: [WIP] Add `openstack overcloud deploy --plan`
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Update the 'overcloud deploy --templates' command to use Mistral
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Migrate to using osc-lib
jistrjaosorior: pong, hi08:28
openstackgerritChristian Schwede proposed openstack/tripleo-heat-templates: Enable Swift account_quotas middleware by default
jaosoriorjistr: hey dude, was re-reading your comment here
openstackgerritDougal Matthews proposed openstack/tripleo-common: Fix an autoclass reference and add missing pages to the toctree
jaosoriorjistr: I could add the balancermember reload. But do you think I should go for the refresh on the concat? I think that might be a bit of overkill and quite focused on an implementation detail, no?08:30
openstackgerritMerged openstack/tripleo-ui: Fix multiple default exports in single module
openstackgerritDougal Matthews proposed openstack/tripleo-common: Better namespace the list of TripleO actions
* d0ugal is rebasing all the things08:32
jistrjaosorior: yea true that it would rely on puppet-haproxy implementation. I'm fine with whichever you prefer.08:33
jaosoriorjistr: alright, just wanted to know your opinion08:33
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Refactor DeploymentPlan Component
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Mistral Actions error handling
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Parameters listing and updating
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Reload HAProxy on refresh
jaosoriorjistr: done ^^08:35
jistr+2'd, thanks08:36
jaosoriorjistr: thanks dude08:37
*** zoli|brb is now known as zoli08:37
*** zoli is now known as zoliXXL08:37
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Disable sahara tests in tempest
*** pgadiya has quit IRC08:38
*** greghaynes has joined #tripleo08:40
openstackgerritMerged openstack/tripleo-heat-templates: Create hiera service_enabled keys for enabled services
hewbroccaFolks, shardy was looking for reviews on these:
hewbroccaPassing CI, ready to look at08:41
hewbroccahe's out today so I am advocating on his behalf08:41
*** jbadiapa has joined #tripleo08:41
*** pgadiya has joined #tripleo08:41
openstackgerritMerged openstack/tripleo-puppet-elements: Install OpenStack Manila UI
shadowerhewbrocca: #1 is merged, #2 waiting for the gate08:42
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add flag to assert that puppet manages the keystone endpoints
shadowerhewbrocca: is he really out today? /me has talked to him couple hours ago08:45
*** abregman has joined #tripleo08:47
hewbroccashadower: well, it's a bank holiday in the UK today08:47
hewbroccaso I think he's hiding08:47
hewbroccaNow these two appear to be gfidente fault:08:48
*** akrivoka has joined #tripleo08:48
openstackgerritMerged openstack/tripleo-heat-templates: Rename opendaylight service to opendaylight_api
openstackgerritBabu Shanmugam proposed openstack/puppet-tripleo: Add ML2/OVN support for neutron profile
shadowerDoes this need a rechec?
shadowerJenkins says +1 and the "Jenkins check" runs pass but the "Jenkins" ones are POST_FAILURE08:51
jaosoriorshadower: I think it does. both ha and nonha got that :/08:53
shadowerwill do, thanks jaosorior08:53
*** gfidente has joined #tripleo08:53
*** gfidente has quit IRC08:54
*** gfidente has joined #tripleo08:54
jaosoriorshadower: even though ha passed the pingtest, it timed out shortly after. So that's not a big deal. BUT, in that run, nonha timed out in the introspection. So yeah, I think recheck is necessary08:54
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Migrate Deploy action to Mistral
shadowerjaosorior: right. I was a bit confused because of the +1 and the fact that the two checks are separate from the rest08:54
jaosoriorshadower: yeah. Jenkins doesn't take into account the ovb ha and nonha for voting08:55
*** electrofelix has joined #tripleo09:00
*** vijaykc4 has quit IRC09:07
jistrbtw folks i'm a bit confused -- i saw some mentions of feature freeze today, but i think the official OpenStack one is Sept 1? When do we aim for feature freeze?09:08
hewbroccaMy understanding was that the official freeze is August 29 -- but I have no basis for that other than somebody said it at some point09:08
jaosoriorjistr: that's a great question. Cause it seems I need to apply for an FFE O_o09:08
hewbroccajaosorior: alternatively, you could just land all your patches :D09:09
jaosoriorhewbrocca: interesting proposition O_O09:10
Jokke_I have heard everyone talking about today, official date 'though by the release schedule is 1st as that's the Thu of FF week where all dates defaults unless otherwise specified and with FF there is no such specification09:11
jistr^^ that09:12
jpichThat's my experience as well, I think folks are just hoping to merge things earlier in the week so the branch can be cut on Thursday without having to wait for the gate queue to empty09:13
*** sshnaidm is now known as sshnaidm|afk09:13
Jokke_jpich: the stable branches should not be cut for 2 weeks still (that's aligned with -RC1 not with -3)09:14
jistrbtw i found this
jpichJokke_: I guess I meant apply the tag then?09:14
Jokke_jpich: that works :P09:14
gfidenteJokke_, commented on the manila submission09:15
*** oshvartz has quit IRC09:15
gfidenteI think that's an issue we haven't solved in neutron because neutron plugins are exclusive09:15
Jokke_thanks gfidente!09:16
gfidenteI wanted to add a reference to the services _enabled which shardy did though09:17
gfidentecause I think it will be useful09:17
gfidentecan't find it09:17
*** limao has quit IRC09:17
gfidenteit's basically a mechanism to push in hiera a list of _enabled profiles09:17
Jokke_gfidente: I was looking that neutron implementation for past couple of hours and really couldn't figure out how that would work. Thanzks for confirming thta it wouldn't09:17
jaosoriorgfidente: it merged09:18
gfidentejaosorior tks :)09:18
jaosoriorgfidente: here it is
jaosoriorah, I was too slow :P09:19
gfidentejaosorior morning :)09:19
*** akuznetsov has quit IRC09:19
Jokke_yeah, I'd like to understand what we want to do there and would be happy to work towards it. But something shardy referred this morning apparently is still missing couple of patches before it can be consumed here09:19
jaosoriorgfidente: morning indeed09:19
jaosoriorJokke_: what's the issue?09:19
Jokke_08:39 < shardy> marios: basically instead of defining the entire ControllerServices list, it's possible to define a merge strategy so your environment file just appends to the list09:20
Jokke_08:40 < shardy> which will make this much easier/cleaner09:20
Jokke_08:40 < shardy>
*** limao has joined #tripleo09:21
gfidenteI am going to use that for ceph/osd on computes if I can figure how :)09:21
gfidentejust do add more bugs09:21
jaosoriorJokke_: is the manila backend exclusive? Or can you have mutliple backends there?09:23
Jokke_so by the looks of it the two missing pieces merged into heat this morning09:23
Jokke_jaosorior: there can be multiple simultanious ones09:23
jaosoriorI see09:23
gfidentejaosorior, same problem is with cinder09:23
gfidenteI was thinking if we shouldn't address this in the puppet module09:23
gfidenteand build there the list of enabled backends from the enabled drivers09:24
jaosoriorgfidente, Jokke_ cause the way I'm dealing with a pluggable option in HAProxy is to add another substack
jaosoriorgfidente, Jokke_ but then again, I don't have the requirement of multiple backends there09:24
Jokke_jaosorior: yeah ... and how would you do the same dynamically?09:25
jaosoriorthe word dynamically is a bit overused... what do you mean?09:26
gfidenteramishra, you around?09:27
Jokke_jaosorior: so how do you do that based on one or multiple environment files listed on the call?09:27
hewbroccagfidente: you're looking at
hewbrocca right?09:28
hewbroccashardy explicitly mentioned those need to merge for comp roles09:28
*** oshvartz has joined #tripleo09:28
hewbroccajpich: shardy also asked one other thing, if you have time:09:29
hewbroccaThen I can convert this tripleoclient hack to a mistral action and add it09:29
hewbroccato the deployment workflow:09:29
hewbroccaWe could perhaps ask someone (rbrady or jpich?) to help create the mistral09:29
hewbroccaaction should they have time to do it, in theory it should be a pretty09:29
hewbroccasimple cut/paste from my patch.  If not then I'll do it tomorrow.09:29
Jokke_Also while I do understand shardy's comment of composability, I kind of disagree the need of backends of composable roles being composable. At least with current capabilities this seems like cleaner approach09:29
ramishragfidente: hi09:30
jaosoriorJokke_: so on my path it's just a matter of adding the environment file, and it will replace that heat stack from None to the one I'm giving there. But yeah, like I said, it only works with one backend.09:30
Jokke_jaosorior: yeah09:30
*** pgadiya has quit IRC09:30
jaosoriorJokke_: I guess you could do some yaql trickery. But it would overcomplicate things. I'm fine with what you posted.09:31
gfidenteramishra, can you comment on to see how we can make it merge a single line change into ComputeServicesList ?09:32
ramishragfidente: sure09:33
gfidenteramishra, thanks :)09:34
ramishragfidente: heatclient patch for merge strategies is not merged yet though.09:34
gfidenteramishra, is it expected to land soon? also read as, can we count on it be available?09:35
ramishrayep, n309:35
*** flaper87 has quit IRC09:35
openstackgerritMerged openstack/tripleo-heat-templates: M/N upgrade set scheduler_host_manager right.
ramishrapanned for n3, just someone has to approve it. But, I don't know if tripleo would need a new heatclient release or not.09:36
jistrready for landing and awesome  (another spot where we got rid of misusing Heat->Hiera bug... one day we'll be able to fix it!!)09:36
jpichhewbrocca: Thanks! dbecker also pointed me to this. I'm having a look but I haven't really worked on deployments yet and I'm afraid a lot of it still looks like black magic to me. I'll see how far I can get, if anywhere, before rbrady gets up but I can't say I'm super confident about it09:36
*** tosky has joined #tripleo09:37
gfidentejistr, just w+109:38
*** social has quit IRC09:42
*** social has joined #tripleo09:43
mariosjistr: gfidente i was reviewing that, but in any case would not have +2 yet ;)09:43
mariosjistr: gfidente i mean
*** dtantsur|afk is now known as dtantsur09:44
mariosjistr: gfidente also o/ :)09:45
*** social has quit IRC09:46
gfidentemarios, mongo name has been replaced by mongodb in puppet-tripleo09:46
*** xuao has joined #tripleo09:46
*** xuao has quit IRC09:47
*** xuao has joined #tripleo09:47
jistrmarios, gfidente: yea in fact in the original code removed by the patch ^^ it had mongo_node_ips as heat param but it generated mongodb_node_ips hiera key09:48
jistrso all should be well?09:48
*** vijaykc4 has joined #tripleo09:49
*** vijaykc4 has quit IRC09:51
*** oshvartz has quit IRC09:51
dtantsurmorning folks! how much time do we have left before the feature freeze?09:52
mariosjistr: gfidente ack it was the discrepancy between the hiera key and heat param in all-nodes-config that caught my eye so thanks, seems ok then09:52
*** flaper87 has joined #tripleo09:53
*** flaper87 has quit IRC09:54
*** flaper87 has joined #tripleo09:54
*** vijaykc4 has joined #tripleo09:54
*** limao has quit IRC09:55
shadowerd0ugal: a docs job is now failing because this: expects "input" in every workflow09:55
shadowerd0ugal: but e.g. this doesn't take any so the key is not there:
shadowerd0ugal: is that a bug in the sphinx extension (just checking, happy to write)09:55
*** zigo_ is now known as zigo09:56
d0ugalshadower: dang, yeah, it is. I can update it if you like. Should be super easy.09:56
d0ugalshadower: just give me 1 min09:56
*** iogg has quit IRC09:58
*** iogg has joined #tripleo09:59
hewbroccadtantsur: it's a ticking time bomb09:59
openstackgerritDerek Higgins proposed openstack/tripleo-heat-templates: Remove outdated upgrade steps
dtantsurhewbrocca, yeah, I get it. I need to plan my ironic work accordingly. We've landed all required bits, but there quite a few optional ones.10:00
hewbroccafolks ^^^ merge early, merge often10:00
*** vijaykc4 has quit IRC10:02
openstackgerritDougal Matthews proposed openstack/tripleo-common: Don't expect input to be in every workflow in doc generation
d0ugalshadower: ^10:03
*** oshvartz has joined #tripleo10:03
shadowerd0ugal: nice one, thanks!10:04
d0ugalshadower: FWIW, we have avoided using workflows when they just call one action - users are expected to call the action directly10:04
d0ugalshadower: (I am not a fan of this, I prefer your way, but I was the only one)10:04
*** vijaykc4 has joined #tripleo10:05
shadowerd0ugal: hm, good to know10:05
*** vijaykc4 has quit IRC10:06
gfidentejistr, so as I understand it hiera() will return string anyway10:06
gfidentebut any2array should do?10:06
jistrgfidente: btw as shardy wrote about the interpolation here -- do you know if we need that somewhere? I don't think it can produce anything else than a string, regardless what the input is.10:06
*** coolsvap_ is now known as coolsvap10:06
gfidentenice was writing about same thing in same moment :)10:06
jistrhehe yea :))10:07
d0ugalshadower: but maybe if we can sneak that in then I have an excuse to make the others match it ;)10:07
jpichd0ugal: 0_010:08
shadowerd0ugal: well as soon as it passes CI, it's yours to +A :-)10:08
jaosoriorderekh: I actually don't see the pki directory in the dump we get of the overcloud nodes. What's up with that?10:09
derekhjaosorior: looking10:10
openstackgerritDmitry Tantsur proposed openstack/instack-undercloud: Handling defaults for pxe_bootfile_name and pxe_config_template moved to puppet
d0ugalshadower, jpich - hah, well, at this point I don't want to suggest changing it as I assume the UI is already using that approach10:10
dtantsurlucasagomes, mind taking a look please ^^^10:10
* lucasagomes looks10:11
jpichd0ugal: tbh if you create the one-action-workflows so they're available it probably works too, that won't prevent anyone from calling the actions directly10:11
lucasagomesdtantsur, btw you will need to rebase that I believe10:11
d0ugaljpich: Indeed, and then the workflows could be the documented interface - if you want to dive behind that, go for it.10:12
dtantsurlucasagomes, just rebased10:12
lucasagomesdtantsur, a-ha, right lgtm10:12
d0ugalAnyone got a moment to review an if statement? :)10:12
jpichd0ugal: Hmmm I'd discuss a bit more widely before implying that only workflow calls are "supported"/"approved" and the actions are unstable10:13
d0ugaljpich: they are already unstable because we don't version them like we do with workflows ... ;)10:13
*** zoliXXL is now known as zoli|lunch10:14
derekhjaosorior: ahh I see it now
derekhjaosorior:  --exclude=etc/pki10:14
derekhjaosorior: so the patch is be used I guess?10:14
openstackgerritJuan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: Inject CA certificate via service profile
jaosoriorderekh: no. I just figured out a problem10:14
jaosoriorderekh: thanks for checking it out10:15
derekhjaosorior: no prob, looks like I came to the correct conclusion for the wrong reason ...10:15
openstackgerritDmitry Tantsur proposed openstack/instack-undercloud: Stop using deprecated ironic::drivers::deploy
*** pgadiya has joined #tripleo10:16
jaosoriorderekh: it was a good review either way :D10:17
jaosoriordtantsur: hey man, currently nothing under the "agent" section in the ironic configuration can be set through puppet-ironic, right?\10:18
dtantsurjaosorior, I'm afraid you're right10:18
jaosoriordtantsur: I see. So currently we don't tweak any of that in tripleo?10:18
dtantsurI guess so10:19
dtantsurwhat exactly do you have in mind?10:19
openstackgerritLukas Bezdicka proposed openstack/instack-undercloud: Use ironic::inspector::ipxe class to setup iPXE
openstackgerritLukas Bezdicka proposed openstack/instack-undercloud: Update puppet dependancies before running puppet
*** paramite has joined #tripleo10:20
openstackgerritLukas Bezdicka proposed openstack/instack-undercloud: Use ironic::pxe class to setup PXE
openstackgerritLukas Bezdicka proposed openstack/instack-undercloud: Update puppet dependancies before running puppet
jaosoriordtantsur: I was just wondering how necessary is it to add those to puppet-ironic. This commit sets post_deploy_get_power_state_retries which we needed to tweak (apparently) for the undercloud. And I was thinking if we should leave it like that or actually get a proper parameter in the puppet manifests for it10:20
dtantsurjaosorior, ugh, this is ugly. please do expose it in puppet-ironic10:21
*** vijaykc4 has joined #tripleo10:21
dtantsurotherwise we'll break tripleo CI if we start exposing it10:21
jaosoriordtantsur: you know someone that might have time to add this to puppet-ironic? This just came to my mind when reviewing that code, but unfortunately am quite caught with other things/10:22
dtantsurjaosorior, or in this case I'd even consider increasing the ironic default10:22
dtantsurjaosorior, I can do it in 15-20 minutes :)10:23
dtantsurbut if someone wants to increase the upstream default, they should bring some data to show the problem10:23
Jokke_gfidente, jaosorior: added comment to Would appreciate your insight if I got it right10:27
shadowerFolks, basically a one-liner that's fixing some of the broken docs jobs now:
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Port plan-edit file listing to use Swift
jaosoriorshadower: merged10:28
jaosoriorI mean, approved10:28
jaosoriorshadower: can you check this out? also quite quick review10:29
shadowerjaosorior: thanks and will do10:30
jaosoriorshadower: do you have time to review a t-h-t patch?10:35
*** vijaykc4 has quit IRC10:36
shadowerjaosorior: I do, but I'm still not that familiar with the new changes10:36
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart: Devmode: rename delorean current so any existing repo is overwritten
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart: POC: new devmode
*** vijaykc4 has joined #tripleo10:39
*** fultonj has joined #tripleo10:39
jaosoriorshadower: wait up10:41
*** akuznetsov has joined #tripleo10:42
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Conditionally expose haproxy endpoints
jaosoriorshadower: this version ^^10:43
shadowerall right10:43
*** zoli|lunch is now known as zoli10:44
*** zoli is now known as zoliXXL10:44
jaosoriorshadower: those *_enabled values come from this commit
jaosoriorshadower: and are available in all-nodes-config10:45
shadoweryeah, thanks10:45
*** oshvartz has quit IRC10:45
openstackgerritTom Barron proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class
jistrgfidente: oooh apparently Hiera 2.0 has an alias function, which doesn't convert everything to string
*** social has joined #tripleo10:50
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Populate HAProxy endpoint ports from EndpointMap
gfidentejistr, right I think we have 1.3 though10:51
openstackgerritMerged openstack/tripleo-heat-templates: Create composable mapping between enabled services and role ips
gfidentewe also wanted to upgrade at some point to be able to do nested hashes lookups10:51
gfidentebut couldn't manage to10:51
gfidenteas existing manifests had issues10:51
jistryea... seems like this would be a very helpful upgrade10:51
jistrgfidente: do you know if it was oonly our manifests or is this a general openstack-puppet problem?10:52
gfidenteso I think I can take out the brackets10:52
gfidenteand proceed with split10:52
gfidenteit's the only way I can think of at least10:52
jistrgfidente: can we remove this altogether, since inerpolating an array value isn't any good
jistrgfidente: and do a ceph_mon_node_ips lookup from the .pp file itself?10:54
jistrgfidente: until we switch to hiera 210:55
gfidentejistr, yes for ceph10:55
gfidentejistr, but we'll have same problem with the other modules probably?10:55
gfidentebut I am good doing it with ceph profile for now10:56
gfidenteand deal with the others on a case by case10:57
jistrgfidente: yea we might have to do the aliasing in puppet-tripleo for the time being for other modules too? Or do you see another option? E.g. i don't see even the CSV output approach solving this, because that would work only for modules which actually accept CSV and not arrays, which is a weird thing to do and AFAIK only puppet-ceph does that.10:57
*** oshvartz has joined #tripleo10:57
jaosoriorgfidente: hey dude, so regarding this
jistrgfidente: yea +1 for case by case. As i see it we don't have a choice. (?)10:58
jaosoriorit depends on this patch, right?
gfidentejistr+1 :)10:58
gfidentejaosorior, should be the other way around10:58
jaosoriorgfidente: you might want to rebase to stop depending on that one. Since the other patch seems to have issues.10:58
gfidentejaosorior, ah I see what you mean yeah10:59
mariosJokke_: o/ hey am getting ready to look at the manila-netap a bit, fyi cos you said you might do that too10:59
gfidentejistr, doing ceph_mon_ips lookup for now then10:59
mariosJokke_: tbarron just updated the puppet-tripleo side for the merge conflict10:59
mariosthanks tbarron and sorry for the non-reply to email, i was away last week11:00
tbarronmarios: thanks for all *your* help, i did hear you were out.11:00
Jokke_marios, tbarron: cool, havin my lunch atm. Still trying to figure out how we want to do that or if the current approach is ok11:01
mariostbarron: heh np didn't really do much ... we had a chat with shardy this morning and i think he *may* be ameanable to landing with all in 'base' and revisit, but i will have a go at a split out and see if it is ok that way too, shouldn't be too much work11:02
mariostbarron: either way Jokke_ we need to decide on a common way forward for both and
*** sshnaidm|afk is now known as sshnaidm11:03
tbarronmarios: yeah, we need a common approach11:03
*** panda is now known as panda|lunch11:03
Jokke_marios: agreed!11:04
tbarronmarios: Jokke_ need to work towards being able to apply multiple backends, non-exclusively, for both manila and cinder11:04
openstackgerritMerged openstack/tripleo-heat-templates: Set the default gnocchi driver to swift
tbarronmarios: Jokke_ AND (maybe harder ??) multiple instances of the same backend, e.g. cephfsnative1 (to cluster ceph1) and cephfsnative2 (to cluster2)11:05
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Move snmp settings into composable services
gfidentejaosorior, ^^11:05
tbarronmarios: Jokke_ but right now minimum viable solution for N is get netapp deployed as it's the only backend that's not tech preview for Newton11:06
jaosoriorgfidente: awesome! I'll marge it when it passes CI11:06
tbarronJokke_: of courxe I want to see cephfs native land too, have been doing a lot of work on the non OOO part of that myself, etc.11:06
Jokke_tbarron: cephfs native is close ... THT change left, the two others have been merged11:07
tbarronJokke_: yeah, I saw.  You are ahead of netapp :)11:08
*** rodrigods has quit IRC11:13
*** rodrigods has joined #tripleo11:13
*** vijaykc4 has quit IRC11:13
*** vijaykc4 has joined #tripleo11:14
sshnaidmcores, please take a look and merge if possible:  passed ci11:17
jaosoriorsshnaidm: done11:18
sshnaidmjaosorior, thanks!11:18
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Adds brackets to mon_host, removing old _v6 param
gfidentejistr, ^^ can you double check?11:19
gfidenteI want to update tht part too11:19
*** ramishra has quit IRC11:21
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Move ceph cluster config out of allNodesConfig
jistrgfidente: found 2 issues otherwise lgtm11:22
gfidentejistr, yeah thanks11:22
*** ramishra has joined #tripleo11:22
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Adds brackets to mon_host, removing old _v6 param
*** thrash|wknd is now known as thrash11:23
jistrgfidente: on the other one i just wonder about the removal of CephIPv6 param -- if the puppet-tripleo place where we removed it is the only place where we used it then probably it's ok, but we need to remove it from the net iso v6 file too11:26
gfidentejistr, commented there we're using it in ceph-mon.yaml11:26
jistrah ha! ok +211:27
gfidentethough I think both changes should pass CI11:27
*** rbowen has joined #tripleo11:32
*** maeca1 has joined #tripleo11:37
openstackgerritMerged openstack/tripleo-validations: Updated from global requirements
*** pradk has joined #tripleo11:42
hewbroccaEmilienM: good morning!11:42
openstackgerritBrad P. Crochet proposed openstack/tripleo-heat-templates: Composable Zaqar services
*** panda|lunch is now known as panda11:43
*** Guest85979 is now known as beagles11:44
openstackgerritMerged openstack/tripleo-common: Don't expect input to be in every workflow in doc generation
*** asbishop has joined #tripleo11:44
*** pradk has quit IRC11:46
karthiksEmilienM,   can you please have a look at
EmilienMkarthiks: I will11:47
karthiksThanks EmilienM .11:48
openstackgerritBrad P. Crochet proposed openstack/tripleo-heat-templates: Composable Mistral services
d0ugalshadower, mandre: Can you rebase 353898? The docs change has landed.11:50
*** shardy has joined #tripleo11:50
d0ugaldocs fix*11:50
*** jeckersb_gone is now known as jeckersb11:50
mandred0ugal: on it11:50
*** asbishop has quit IRC11:50
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Hanlde galera_node_names being an array
jistrgfidente: ^^ here's for galera node names11:51
*** shardy_ has joined #tripleo11:52
*** shardy_ has quit IRC11:52
gfidentejistr, nice11:53
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Handle galera_node_names being an array
*** jayg|g0n3 is now known as jayg11:54
gfidentecause it's safe and will enable
jistrgfidente: yea at least for the galera_node_names part11:55
gfidenteso for controller_node_names we should actually add something in ServiceNetMap11:55
*** lucasagomes is now known as lucas-hungry11:55
EmilienMjaosorior: can you look please?11:56
jistrgfidente: wondering what do we need controller_node_names for. Should we rather use service specific naming like pacemaker_node_names and haproxy_node_names etc?11:56
gfidenteyeah I see those two and midonet plugin using it11:56
gfidentethough list of controller names seems fair to have to me11:57
openstackgerritMartin André proposed openstack/tripleo-common: Allow running validation against different plans
openstackgerritMartin André proposed openstack/tripleo-common: Workflows to run validations
openstackgerritMartin André proposed openstack/tripleo-common: Workflows to load validations
openstackgerritMartin André proposed openstack/tripleo-common: Disable retry files for ansible validations
*** limao has joined #tripleo11:58
*** jpena is now known as jpena|lunch11:58
gfidentejistr, we can probably do that per-role11:58
jistrgfidente: maybe yea, but probably not for the use cases we have right now? e.g. the midonet plugin should be using something neutron-specific?11:58
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Generate composable service node_names lists
*** dprince has joined #tripleo12:00
gfidentehaproxy is the one consuming controller_node_ips too12:00
*** jaosorior has quit IRC12:02
*** jaosorior has joined #tripleo12:03
*** limao_ has joined #tripleo12:03
mandred0ugal: quick question...12:03
mandred0ugal: there is a bit of manual setup required after deploying the undercloud to boostrap the ansible validations framework (namely copy the SSH key to the overcloud nodes)12:04
mandreit's all bundled in a mistral workflow12:04
mandredo we want to make it a tripleoclient command?12:04
d0ugalmandre: I guess we might want to run that workflow at the end of the undercloud install?12:05
d0ugalmandre: at the same time we make the default plan12:05
shadowerI like that12:05
EmilienMgfidente, jaosorior: I think I found an issue in CI, it seems like puppet modules are not deployed from source but from delorean12:05
EmilienMand since we haven't had promotion for 5 days, our CI is not deploying latest bits12:06
*** limao has quit IRC12:06
EmilienMsshnaidm: ^12:06
d0ugalmandre, shadower - akrivoka recently done this for the default plan creation, so she probably knows exactly where to do it :)12:06
hewbroccaEmilienM: blearrgh12:07
mandred0ugal: can we do it conditionally? i.e. when we enable validations12:07
sshnaidmEmilienM, afaik puppet modules should be installed from packages, not from sources now12:07
d0ugalmandre: Good question, I assume so but I've not looked into the process enough12:07
*** rlandy has joined #tripleo12:07
mandred0ugal: ok. I'll dig around12:08
sshnaidmEmilienM, but puppet modules are installed from current rdo trunk, not from promoted current-tripleo12:08
EmilienMsshnaidm: that's a problem if our promotion system doesn't happen12:08
EmilienMsshnaidm: are you sure?12:08
sshnaidmEmilienM, yep:
sshnaidmEmilienM, puppet-* is installed from
EmilienMok, let me check why we have old packages in our CI12:09
*** pradk has joined #tripleo12:09
EmilienMsshnaidm: probably a false alarm :)12:10
sshnaidmEmilienM, should be installed those:
sshnaidmEmilienM, do you have logs?12:11
*** mburned_out is now known as mburned12:12
EmilienMsshnaidm: it was a false alarm, nevermind.12:12
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: configure heat with new authtoken parameters
sshnaidmEmilienM, ok12:13
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class and tidy up generic
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class and tidy up generic
*** rhallisey has joined #tripleo12:19
jaosoriorEmilienM, gfidente, marios: keystone endpoints via puppet is green :D and so is the commit that disables post-config from tripleoclient
gfidentejaosorior, great12:23
hewbroccajaosorior: woooooooooooooo12:23
openstackgerritMartin André proposed openstack/instack-undercloud: Setup CORS settings for tripleo-ui
openstackgerritMartin André proposed openstack/instack-undercloud: Introduce 'enable_ui' option
shardygfidente: thanks for updating
shardy+2 on moving *node_ips lookup into puppet-tripleo12:25
shardyI did the same for rabbitmq in
shardyso we can remove the yaml aliases in allNodesConfig12:25
gfidenteshardy, we worked on that with jistr and were looking at the two submissions above12:26
openstackgerritMartin André proposed openstack/instack-undercloud: Setup CORS settings for tripleo-ui
openstackgerritMartin André proposed openstack/instack-undercloud: Introduce 'enable_ui' option
gfidenteshardy, looks like either we add pacemaker and haproxy into servicenetmap or keep _controller_names,_ips in there for now12:27
shardygfidente: I'd say we should put the services into ServiceNetMap12:27
gfidenteshardy, ok12:27
*** chem has joined #tripleo12:27
jaosoriorshardy: is there a way you can think of of fetching on which network a service is listening on? (on puppet)12:27
shardyjaosorior: Yes, we set the bind_ip by doing a substitution of the IP for the network assigned to the service in ServiceNetMap12:28
* shardy finds the patch that does it12:28
EmilienMjaosorior: reviewing it too today12:29
shardyjaosorior: ^^ like that12:29
jaosoriorshardy: right; in the internal TLS case, the service's internal endpoint will point in most of the cases to the internal API VIP, but in other cases it might point to storage and such.12:30
mandrethanks akrivoka, very helpful12:30
jaosoriorshardy: so, we need a certificate per VIP endpoint (or at least the used ones)12:30
jaosoriorshardy: so I need a way to tell haproxy which certificate to use in the internal endpoint depending on the service12:31
EmilienMjaosorior: +A, excellent work here12:31
jaosoriorEmilienM: thanks :D12:31
* hewbrocca does happy dance12:32
* jaosorior puts celebration beer on the fridge12:32
* EmilienM still at breakfast12:32
jaosoriorEmilienM: breakfast beer?12:33
EmilienMyou're asking a french guy if he takes beer at breakfast. Sigh.12:33
hewbroccaBreakfast wine, whole other story12:34
EmilienMhewbrocca knows french better12:34
*** pkovar has joined #tripleo12:36
sshnaidmjaosorior, should it be so then? "ironic::drivers::agent:post_deploy_get_power_state_retries: 15"12:36
sshnaidmjaosorior, all (double)colons are in place I hope12:36
jaosoriorshardy: sounds about right12:36
sshnaidmjaosorior, great, then will update the patch12:37
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Increase timeout for ironic power status report
numansEmilienM, gfidente can you please add this into your review queue -, Babu has submitted another patchset addressing gfidente's comments and resolving merge conflict. Thanks12:38
jaosoriorsshnaidm: it looks about right, but you forgot the depends-on12:39
sshnaidmjaosorior, yeah, I'm looking how to do it12:39
*** milan has joined #tripleo12:40
jaosoriorsshnaidm: Add this to the commit messsage  ->   Depends-On: If2c6987d0f38aecabf41c09505c4f0dca111603412:40
sshnaidmjaosorior, thanks12:40
openstackgerritMerged openstack/tripleo-common: Exclude more unneeded files from default plan
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Increase timeout for ironic power status report
gfidentenumans, I see it defaults to eth0 but if the encap ip should be from one of the provisioned networks12:42
gfidenteyou can collect that from heat12:42
gfidentenumans, should it be the network where tenants networks go?12:43
gfidentenumans, or rather the network where the neutron api is?12:43
shadowermandre: question about this validation:
gfidenteor none of the two?12:43
*** vijaykc4 has quit IRC12:43
shadowermandre: are the values (e.g. controller_ctlplane_ip_address) coming from anywhere? I don't see how they could12:43
mandreshadower: you have to pass the values as ansible environment variables, which we don't do12:45
gfidenteshardy, so shall I add haproxy/pacemaker in servicenetmap in ?12:45
shadowermandre: ah, right. Might be something we could figure out from Heat or Mistral later. So by default this just runs the gateway ping, correct?12:45
mandreshadower: I think it was our first ansible validation, so it was a bit experimental12:45
*** egafford has quit IRC12:46
mandreshadower: yup, we should be able to get what we need from the environment12:46
jistrgfidente: just replied there12:46
*** pradk has quit IRC12:46
karthiksgfidente, jaosorior shardy  for review  has two +2s and now CI is all green now. PTAL12:46
jistrgfidente: i don't think we need them in servicenetmap essentially? if we are only concerned about the hostnames12:47
shardygfidente: Yes, please do, thanks!12:47
jaosoriorkarthiks: done12:47
shardythen we can adjust the profiles to match12:47
mandreshadower: also, do note there is currently no way to pass ansible variables via our mistral workflow12:47
karthiksThanks jaosorior12:47
mandreshardy: it shouldn't be too difficult to add, though12:48
jaosoriorkarthiks: what does PTAL stand for?12:48
mandreshadower: ^^12:48
sshnaidmderekh, hi12:48
shadowermandre: yeah fair enough. I think that's fine atm12:48
shardygfidente: if it's easier, we can just not remove controller_node_names12:48
gfidenteshardy, jistr we need to find agreement :)12:48
mandresorry shardy, please ignore12:48
*** jdob has joined #tripleo12:48
hewbroccaPlease Take A Look!12:48
shardyI think I actually removed that by mistake :(12:48
hewbroccajaosorior: ^^^ :D12:48
karthiksjaosorior,  yes hewbrocca12:48
jaosoriorhewbrocca: I see. Thanks12:48
gfidenteshardy, jistr ok then maybe let's do it in follow up12:48
gfidentejistr, will you update it to keep _names?12:49
shardygfidente: Yeah, let's add controller_node_ips back for now, and potentially add haproxy/pacemaker to ServiceNetMap later12:49
jistrgfidente: sorry maybe i just don't understand all the context... is about hostnames only, no? ServiceNetMap doesn't play any role in that does it?12:49
shardyI only meant to remove the *service* names, not the role names12:49
*** Goneri has joined #tripleo12:50
jistrbut yea in general +1 for keeping controller_names12:50
jistrgfidente: yea i'll update it12:50
shardyjistr: you're right, we only filter *node_ips based on ServiceNetMap12:50
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Port plan-edit file listing to use Swift
*** vijaykc4 has joined #tripleo12:51
gfidenteyeah I raised that cause haproxy was using node_ips too12:51
jaosoriorand what was the issue on adding haproxy and pacemaker to the servicenetmap?12:51
gfidenteguys I'm going for lunch, back soon12:51
jistri just noticed i'll need to update the puppet-tripleo galera_node_names patch too as it now will be mysql_node_names12:51
*** vijaykc4 has quit IRC12:51
shardyjaosorior: just that we probably don't need to do it right now if we fix my patch12:52
shardyjistr, gfidente: FYI I had planned to put per-role *_names in for all roles12:52
shardybut I've since started to prefer using per-service data wherever possible, as it'll work much better when folks come to move services between roles12:53
shardyjistr: are you OK to update or shall I do it?12:55
*** lucas-hungry is now known as lucasagomes12:56
openstackgerritMartin André proposed openstack/instack-undercloud: Setup CORS settings for tripleo-ui
jistrshardy: i'll update, just wanted to do if after i update the galera part too12:58
jistrand i wanted to test that...12:58
jistrbut given the time constraints, i may just update them both right now12:58
shardyjistr: ack, sounds good, thanks!12:58
jistrand test them subsequently12:58
*** tzumainn has joined #tripleo12:58
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Handle galera_node_names being an array
*** anshul has quit IRC12:59
shardyjistr: Ok, sounds good13:00
shardyI just noticed another issue with so I may have to push another revision too13:00
*** trozet has joined #tripleo13:02
openstackgerritMartin André proposed openstack/instack-undercloud: Setup CORS settings for tripleo-ui
numansgfidente, the encap ip should be the host ip.13:02
*** pcaruana has quit IRC13:02
jistrshardy, gfidente: i'll also remove this for the same reason as with the IPs, wdyt?
jistrit gave me a rebase conflict so that's why i noticed it13:03
*** social has quit IRC13:03
shardyjistr: +113:03
*** jpena|lunch is now known as jpena13:04
*** social has joined #tripleo13:04
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Availability monitoring agents support
numansgfidente, this ip is internal ip of the host -
*** Goneri has quit IRC13:06
EmilienMshardy: taking over 361730 to fix lint13:06
shardyEmilienM: ack, thanks!13:06
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Make service profiles default to rabbitmq_node_ips
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Generate composable service node_names lists
*** vijaykc4 has joined #tripleo13:10
jistrshardy, gfidente: ^ updated & rebased on top of the latest patchset of the patch underneath13:11
shardyjistr: thanks!13:11
*** anbu has joined #tripleo13:11
anbugfidente, Hi Giulio.13:12
anbuThank you for the review.13:12
shadowermandre: the controller/gateway ping test is in the tripleo-heat-templates anyway so I think we can just drop it13:13
mandreshadower: right, it's not super useful right now13:14
*** rbrady has joined #tripleo13:14
*** bkopilov has quit IRC13:14
*** shardy has quit IRC13:17
*** pcaruana has joined #tripleo13:17
openstackgerritBabu Shanmugam proposed openstack/puppet-tripleo: Add ML2/OVN support for neutron profile
*** pgadiya has quit IRC13:19
*** vijaykc4 has quit IRC13:22
*** pkovar has quit IRC13:22
*** abregman_ has joined #tripleo13:27
*** abregman has quit IRC13:30
*** myoung|afk is now known as myoung13:32
*** nyechiel has quit IRC13:34
*** dmacpher has joined #tripleo13:38
*** chem has quit IRC13:40
*** chem has joined #tripleo13:40
jaosoriorjistr: ping13:40
jistrjaosorior: pong, heya13:40
jaosoriorjistr: hey dude, any idea why would the HA job fail with "pcs command not found" here?
jistrjaosorior: ah yea the exec should pass `path =>`13:41
jistrjaosorior: actually it does but i think pcs is in sbin maybe?13:42
jistrjaosorior: you might try with:    path => ['/usr/bin', '/usr/sbin'],13:43
EmilienMor give the path to pcs13:43
EmilienMlike /usr/sbin/pcs13:43
*** dtrainor has quit IRC13:43
*** dtrainor has joined #tripleo13:43
jaosoriorEmilienM, jistr thanks. Will do13:43
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add NetApp Manila driver integration and tidy up generic
*** skramaja_ has joined #tripleo13:46
*** skramaja has quit IRC13:46
*** skramaja_ is now known as skramaja13:46
*** ayoung has joined #tripleo13:47
*** jubapa has quit IRC13:48
dtantsurdprince, morning! re ironic patches, mind merging both into one final for me to test locally? please see a few comments there13:51
*** pkovar has joined #tripleo13:51
openstackgerritKarthik S proposed openstack/puppet-tripleo: Configure the numvfs for SRIOV interfaces
openstackgerritMerged openstack/tripleo-quickstart: Add ci-script to upload images in OVB
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Implement scenario001 CI job
EmilienMbnemec: I addressed your comments on
dprincedtantsur: ack. They are slightly different but I don't mind merging them I guess13:54
gfidentejistr, so to make pass we need to patch p-t to set mon names from array as well13:54
*** xuao has quit IRC13:54
gfidentejistr, do you think I can make it in so we have less changes around?13:54
jaosoriorEmilienM: do the puppet execs get ran with the root user? or which?13:55
*** pcaruana has quit IRC13:55
jistrgfidente: yea +1 on that13:56
EmilienMjaosorior: sudo afik13:56
*** zoliXXL is now known as zoli|brb13:56
*** egafford has joined #tripleo13:57
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Reload HAProxy on refresh
jistrjaosorior: +2 ^^14:00
jaosoriorjistr: thanks14:00
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Fix Ironic service bind IPs
dprincedtantsur: ^^14:00
*** pradk has joined #tripleo14:02
jaosoriorgfidente, dprince: where does the replacement take place again? I remember it happens but not how exactly14:02
jaosoriorthe IP replacement14:02
dprincedtantsur: NP, with regards to the my_ip change note that I for me (and likely other network isolation users) the MySql traffic runs on the internal_api network which is a vlan (tagged traffic). This wouldn't lend itself well to PXE booting other instances so I think we want a separate setting that can default to 'ctlplane' instead here14:03
dtantsurdprince, yeah, this is totally fair14:04
dtantsurI still have to learn a lot about our networking :(14:04
d0ugalthrash: ping14:04
*** marcusvrn_ has quit IRC14:04
thrashd0ugal: pong14:05
dprincedtantsur: it would be cool to have another isolated network on different NICS for overcloud tenant baremetal provisioning. We would actually support that with os-net-config but it would require custom network templates14:05
*** mhenkel has joined #tripleo14:05
jaosoriorgfidente: I see. So with that, if I try to pass what network is a service on, it will get replaced, right?14:05
dtantsurthis is what I would recommend people to do honestly14:05
dprincedtantsur: but as an out of the box I think re-using the ctlplane for this would make testing, etc. easiest so long as the overcloud Ironic doesn't conflict with the undercloud Ironic.14:05
*** jprovazn has quit IRC14:06
gfidentejaosorior, if a key value is a network name, it gets replaced with the local ip on that network14:06
dtantsurdprince, the "conflict" bit is worrying. it has high chances of clashing with ironic-inspector. but for now, yes, we can delay it14:06
jaosoriorgfidente: well, crap14:06
dtantsurdprince, while you're on it, is it possible you skim through in case I write something stupid there from network pov?14:06
jaosoriorgfidente: I guess I'll have to figure out something else then.14:07
dprincedtantsur: ack, anything running a wildcard DNS would be concerning on a shared network14:07
*** lmiccini has quit IRC14:07
d0ugalthrash: I am adding --plan back!
dprincedtantsur: DHCP rather14:07
*** akuznetsov has quit IRC14:07
thrashd0ugal: wow14:07
gfidentejaosorior, sorry I am not sure what you're dealing with but purpose there was to set the local ip address of any given network14:07
d0ugalthrash: I want that to never be used with another arg, but I don't want to put everything in a mutually exclusive group. Any clever tricks?14:07
jaosoriorgfidente: it makes sense.14:07
thrashd0ugal: I've struggled with that one for other things. And no, I haven't come up with anything yet.14:08
*** adarazs is now known as adarazs_afk14:08
thrashd0ugal: other than *maybe* using a subparser.14:08
thrashd0ugal: but that gets tricky too...14:08
*** pcaruana has joined #tripleo14:09
jaosoriorgfidente: the thing is that, for TLS, we will need a certificate per-VIP. And, some services don't listen on the internalapi network. So for these cases I need to figure out which network these services are listening on on HAProxy and use an appropriate certificate14:09
d0ugalthrash: damn. I did think about checking sys.argv... but that might be a bit gross.14:09
jaosoriorgfidente: for that I was thinking of using the network name14:09
thrashd0ugal: let me think about that one. It's a bit different than what I was trying to accomplish.14:09
d0ugalthrash: k, thanks!14:09
jaosoriorgfidente: so I wanted to pass the servicenetmap via hieradata14:09
thrashd0ugal: and you probably don't want to use return from take_action.14:10
thrashd0ugal: that will always return 0, which is not ideal for scripting14:10
thrashd0ugal: unless _start_mistral_deploy throws exceptions14:10
*** abregman_ is now known as abregman_|mtg14:11
d0ugalthrash: oh, damn. How does it know I am returning? (given no return, returns None anyway?)14:11
openstackgerritKarthik S proposed openstack/puppet-tripleo: Configure the numvfs for SRIOV interfaces
d0ugalthrash: I am tempted to just go with a new command to make things easier. `openstack overcloud plan deploy`14:12
thrashd0ugal: if _start_mistral_deploy throws an exception, that should be sufficient. I'm not saying the return is bad, just don't have that as your *only* way of returning14:12
*** iogg is now known as ioggstream14:12
d0ugalthrash: gotcha, that makes sense.14:12
thrashd0ugal: I would not be *completely* opposed to a new command. It's the same problem I have with tripleo-build-images vs. openstack overcloud image build14:13
*** Goneri has joined #tripleo14:14
*** lmiccini has joined #tripleo14:14
d0ugalthrash: we already have commands to create/delete/list plans. So it actually might even make more sense now that I think about it.14:15
*** skramaja has quit IRC14:15
*** skramaja has joined #tripleo14:15
*** zoli|brb is now known as zoli14:16
*** zoli is now known as zoliXXL14:16
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Defaults to ceph_mon_node_ips and ceph_mon_node_names
gfidentejaosorior, and it gets replaced14:17
jaosoriorgfidente: yeah, that's the issue I have14:17
sshnaidmdprince, hi, could we reconsider caching images again?14:18
openstackgerritDmitry Tantsur proposed openstack/tripleo-heat-templates: Ironic: enabled iPXE by default and make it configurable
dprincesshnaidm: um, are we still having job timeouts? which jobs?14:19
jaosoriorgfidente: Do you know if map_replace does a deep search of the values?14:19
openstackgerritMerged openstack/puppet-tripleo: Configure keystone endpoints in service profile
openstackgerritMerged openstack/tripleo-common: Add Mistral action for boot configuration
openstackgerritMerged openstack/tripleo-common: Add Mistral action & workflow for root device configuration
openstackgerritMerged openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT
*** zoliXXL is now known as zoli|dentist14:21
sshnaidmdprince, it's much much better, but still timeouts both for ha and nonha, you can see here: http://status-tripleoci.rhcloud.com14:21
openstackgerritDmitry Tantsur proposed openstack/tripleo-heat-templates: Ironic: enabled iPXE by default and make it configurable
gfidentejaosorior, it should be happening here
gfidenteand I doesn't look like it does deep search14:22
sshnaidmdprince, 3 for nonha and 7-8 for ha from Fri, not sure if it's critical though14:22
*** dsariel has quit IRC14:23
*** oshvartz has quit IRC14:23
socialdtantsur: so it'll work fine without ipxe element, I can add the include but I prefer to do so after the other review is in :)14:24
lucasagomesderekh, in the tripleo-ci we don't have any function like iniset from devstack to edit .conf files do we ? (/me was looking but couldn't find it)14:24
dprincesshnaidm: is that mostly the periodic jobs? if so would it be safer to just disable the extra stack delete the periodic jobs perform?14:24
sshnaidmdprince, no, it's gate jobs14:25
*** jaosorior is now known as jaosorior_away14:25
dtantsursocial, sorry, I don't quite get it.. how does it work without including the class?14:25
sshnaidmdprince, periodical must build images14:25
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add hyperconverged-ceph environment to include CephOSD on computes
socialdtantsur: the class is in ironic::inspector because of backwards compatibility14:25
socialdtantsur: the warning with note that after newton cycle it'll be removed + releasenotes ,)14:25
dtantsursocial, oh true14:26
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Customize the reservation of memory for host processes
sshnaidmdprince, you are welcome to comment here about periodic:
gfidenterhallisey, shall we pull in openstack-selinux explicitly?14:27
gfidenteor is it installed as dep?14:27
dprincesshnaidm: could we enable caching next Friday evening? Pending no other outages?14:28
sshnaidmdprince, sure14:28
derekhlucasagomes: no don't think so14:28
*** saneax is now known as saneax-_-|AFK14:29
dprincesshnaidm: so essentially we are scheduling the upgrade to occur then (if others agree) so long as no other outages block us14:29
lucasagomesderekh, ack, I'll add one to fix that MTU thingy with neutron and ironic14:29
*** jtomasek|afk has quit IRC14:30
sshnaidmdprince, although I don't understand the value of building images in each gate job, we lived with cached images pretty well before.14:30
derekhlucasagomes: ok,14:30
shadowerjaosorior_away: what does "recheck timed out" do?14:30
dprincesshnaidm: I like image caching very much14:30
derekhsshnaidm: just saw your mail setting new keys now14:31
sshnaidmderekh, yeah.. :(14:31
dprincesshnaidm: just want to flip the switch to test it at a time that doesn't impact the work week, landing code, etc.14:31
sshnaidmdprince, sure14:32
sshnaidmdprince, I can keep eye on it even in Sun14:32
openstackgerritSaravanan KR proposed openstack/tripleo-heat-templates: Added DPDK environment file and nic configs samples
dprincejaosorior_away: good catch on
dprincejaosorior_away: fixing...14:34
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Upgrade ceph-mon
*** rhallisey has quit IRC14:35
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Upgrade ceph-osd
thrashd0ugal: I'd go with that.14:36
thrashd0ugal: perhaps add --plan to the existing command, and deprecate.14:37
hewbroccadprince: I think scheduling it for the weekend is sensible14:38
hewbroccadprince: pabelanger tells me they're going to restart nodepool.o.o later this week, with some shade changes that will hopefully fix our FIP problem14:39
hewbroccabut we'll see14:39
dprincehewbrocca: ideally we'd queue up this so we can verify them one at a time14:40
*** rhallisey has joined #tripleo14:41
openstackgerritLars Kellogg-Stedman proposed openstack/tripleo-heat-templates: Add fluentd client service
openstackgerritLucas Alvares Gomes proposed openstack-infra/tripleo-ci: Do not reuse dnsmasq-ironic.conf for forcing MTU 1400
lucasagomesderekh, ^ not tested, waiting on jenkins14:43
gfidenterhallisey, I saw those selinux rules in the .spec as well14:46
gfidenterhallisey, I don't see it installed so I am going to add it to tpe14:47
rhalliseyit may add ~1 min or two to deployment, but if you're running in enforcing you'll need it14:48
gfidenteI think we can use a depends-on and see if we pass as well14:49
*** rbowen has quit IRC14:49
paramitedprince, EmilienM, larsks: Hi guys, can we join a BJ and talk about and ? Because those two patches are doing basically the same thing and one received +2 from Dan and the other -1 from Emilien. It would be great if we could agree if those patches are OK or not and eventually merge them today.14:50
*** jtomasek has joined #tripleo14:53
dprinceparamite: your patch needs a rebase.14:54
dprinceparamite: I think I'd be happy for both of these to land14:54
dprinceEmilienM: ^^14:54
paramitedprince, ok, thanks14:54
dprinceparamite: agree the approaches are similar. I +2'd fluent because it was passing CI at the time14:56
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Do not use selinux-permissive for the CentOS image
*** vijaykc4 has joined #tripleo14:57
larsksdprince: so, this is more of an architecture question, rather than a "does it pass ci question".  Both approaches are the same, so we're sort of getting conflicting feedback on our chosen approach...14:57
*** chem has quit IRC14:58
larsksdprince: but yeah, so many rebases! :)14:58
dprincelarsks: there are only a couple of ways to do this I think and much of the decision is where to manage the extra Fluent, and Sensu parameters I think14:58
socialhmm lately successfull deploy of tripleo master means everything is broken it just passed the deployment :(14:58
*** abregman_|mtg has quit IRC14:59
larsksdprince: Exactly.  I'm not sure I see a good alternative for aggregating parameters like we're doing.  I'm hoping we can convince EmilienM that this is an okay solution...14:59
dprincelarsks: like we spoke last week so long as we treat these parameters as pseudo internal (i.e. no descriptions on them, used for advanced settings) I thik we are ookay14:59
dprinceOoOkay even15:00
EmilienMdprince: will look after mtg15:00
larsksEmilienM: thanks!15:00
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Availability monitoring agents support
socialhiera(): Cannot find datafile /etc/puppet/hieradata/extraconfig.json but still CREATE_COMPLETE :/15:00
*** rbowen has joined #tripleo15:01
*** adarazs_afk is now known as adarazs15:02
*** rhallisey has quit IRC15:03
*** rhallisey has joined #tripleo15:04
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Mv Nova, Neutron, Horizon out of controller.yaml
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Mv pacemaker and firewall out of controller
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move snmp settings into composable services
*** jaosorior_away is now known as jaosorior15:10
jaosoriorshadower: recheck does the regular recheck, timed out was just the reason I did the recheck15:10
shadowerjaosorior: ah cool. Thanks15:10
*** dprince has quit IRC15:11
*** zoli|dentist is now known as zoli15:11
*** zoli is now known as zoliXXL15:11
*** abregman_|mtg has joined #tripleo15:12
*** beagles is now known as beagles_brb15:15
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Add `overcloud parameters set` to set Heat params is a plan
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Print the execution ID when starting Mistral workflows
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Add `openstack overcloud plan deploy`
openstackgerritMerged openstack/python-tripleoclient: Add 'openstack overcloud node configure' command
openstackgerritMerged openstack/python-tripleoclient: Use Mistral for baremetal boot configuration
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Add `overcloud parameters set` to set Heat params in a plan
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Print the execution ID when starting Mistral workflows
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Add `openstack overcloud plan deploy`
jaosoriorayoung: hey man, so how did the mellon stuff go? Did you get a chance to get that working?15:20
ayoungjaosorior, never could get the module to load15:20
jaosoriorayoung: I saw there was some package-related stuff missing from puppet. Is that needed in the upstream apache modules?15:21
openstackgerritMerged openstack/instack-undercloud: Fix outdated help text from undercloud VIPs
openstackgerritMerged openstack/puppet-tripleo: Removing WARNING: line has more than 140 characters in puppet-tripleo profiles
ayoungjaosorior, I tried to keep the patch in sync with my understanding/what I was trying15:21
ayoungI think that it is necessary, but not yet sufficient15:21
ayoungjaosorior, that coupled with the Jenkis failures makes me somewhat puzzled15:22
hewbroccaAnybody working on doing a release of oooq this week15:22
*** nyechiel has joined #tripleo15:23
openstackgerritMartin André proposed openstack/instack-undercloud: Introduce 'enable_validations' option
openstackgerritMartin André proposed openstack/instack-undercloud: Deploy SSH keys to overcloud in post config
jaosoriorayoung: the errors in the gate seem unrelated15:23
ayoungjaosorior, ah, those are gone now, too15:24
ayoungthat was older, yeah, we are down to getting the Package to install15:24
ayoungjaosorior, any suggestion? Does it need to be in the Puppet module?  Is that something I can test?15:25
jaosoriorayoung: first lets get it to work. Then we get it into puppetlabs-apache15:25
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Optionally include mod auth mellon for apache
jaosoriorayoung: do you have a log of the errors you were seeing?15:28
jaosoriorayoung: I got the package to install in my undercloud testing manually the stuff I just put up. But I'm not sure if that's all that's needed15:28
ayoungjaosorior, this is overcloud deploy15:29
jaosoriorayoung: yeah, just wanted to see if it actually installs. (gotta test it somehow cause right now I'm doing another deployment testing HAProxy TLS)15:30
ayoungjaosorior, I am not concerned about Federation for the undercloud, only the overcloud right now15:30
jaosoriorayoung: you're using the beaker machine, right?15:31
ayoungjaosorior, nope15:31
ayoungI am suing the Dell on my desktop right next to me15:31
jaosoriorayoung: is it accessible somehow?15:31
ayoungjaosorior, so, yeah, but it is a PITA due to OOOqs using Keys and proxies.  Let me put your key in the right places...15:32
*** thrash is now known as thrash|biab15:33
*** nyechiel has quit IRC15:34
ayoungjaosorior, .....and here is where I find my redeploy from Friday night failed.15:36
jaosoriorayoung: actually, mod_auth_mellon SHOULD be installed by the include we did. we shouldn't need that package statement15:36
ayoungjaosorior, one moment while I bring it back up.15:36
*** yamahata has joined #tripleo15:37
ayoungjaosorior, and, by "moment" I mean I am doing an undercloud deploy...\15:37
*** xuao has joined #tripleo15:38
*** tremble has quit IRC15:40
jaosoriorayoung: just ran on my undercloud (like I mentioned earlier, to test that the puppet manifest is doing what it should) and it seems it removed my keystone configuration15:40
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Defaults to ceph_mon_node_ips and ceph_mon_node_names
jaosoriorEmilienM: Any idea why including ::apache::mod::auth_mellon  would remove my keystone configuration?
*** tesseract- has quit IRC15:42
*** pcaruana has quit IRC15:43
EmilienMjaosorior: I guess you didn't include other keystone classes15:43
jaosoriorEmilienM: didn't know it would purge every other configuration there15:44
jaosoriorwell crap15:44
*** rcernin has quit IRC15:44
EmilienMjaosorior: puppet manages apache config15:44
*** leanderthal is now known as leanderthal|afk15:45
jaosoriorEmilienM: fair enough. It was just unexpected to me that it would purge all the rest if it's not in the catalog15:45
*** chem has joined #tripleo15:47
*** yamahata has quit IRC15:48
*** vijaykc4 has quit IRC15:48
*** chem has quit IRC15:48
*** chem has joined #tripleo15:49
*** chem has quit IRC15:49
*** chem has joined #tripleo15:49
*** chem has quit IRC15:50
jaosoriorEmilienM: have you seen something like this?15:51
jaosoriorError: /Stage[main]/Main/Augeas[lvm.conf]: Could not evaluate: Saving failed, see debug15:51
*** limao_ has quit IRC15:51
EmilienMbut it looks related to the undercloud thing with lvm15:51
jaosoriorEmilienM: which thing?15:51
jaosoriorJust hit that while trying to re-deploy my undercloud15:52
*** chem has joined #tripleo15:54
trozetdtantsur: hi15:57
*** zoliXXL is now known as zoli|gone15:58
*** tobias_fiberdata has quit IRC15:58
openstackgerritMerged openstack-infra/tripleo-ci: Print services statuses of nodes in CI
*** zoli|gone is now known as zoli_gone-proxy15:59
*** yamahata has joined #tripleo15:59
*** limao has joined #tripleo16:00
openstackgerritMerged openstack/tripleo-heat-templates: Add flag to assert that puppet manages the keystone endpoints
*** chem has quit IRC16:00
dtantsurtrozet, hi16:00
*** thrash|biab is now known as thrash16:01
*** david-lyle_ is now known as david-lyle16:01
trozetdtantsur: I'm seeing in an OOO deployment to baremetal, the node fails to ipxe boot, and says "undionly".  It seems to get an IP, then says configuring net0 unidonly and times out16:01
*** chem has joined #tripleo16:02
trozetdtantsur: i see multiple bugs filed on this from last year:,
openstackLaunchpad bug 1454332 in ironic (Ubuntu) "Fail to chainload iPXE (undionly.kpxe) " [Undecided,Confirmed]16:02
openstackLaunchpad bug 1415017 in Ironic "failed deploy undionly.kpxe then can't get ipxe script boot.ipxe" [Undecided,Fix released]16:02
trozetdtantsur: these are old Kontron servers, any ideas?16:02
dtantsurtrozet, my first guess is old iPXE ROM on servers16:02
dtantsurlucasagomes, any ideas here ^^^16:03
trozetdtantsur: i have a screen shot of the boot let me get it for you16:03
* lucasagomes looks16:03
*** numans has quit IRC16:04
jaosoriorhewbrocca: time to open my celebration beer16:04
jaosoriorhewbrocca: overcloud deploy should now be skipping the post-config :D16:05
jaosoriorguess we can't delete o-c-c from being called yet, but it's it's progress :D16:05
lucasagomestrozet, hmm strange so it keeps looping and fetching undionly.kpxe over and over ?16:06
hewbroccaso happy16:06
jaosoriorlife is good :D16:06
trozetlucasagomes, dtantsur:
lucasagomestrozet, that looks fine actually, so it loaded iPXE and now it's configuring the nic16:07
gfidentejaosorior, I think we can because pki init was already supefluous16:07
lucasagomestrozet, what happens next ?16:07
trozetlucasagomes: right hten it just sits here and times out16:07
trozetlucasagomes: i can see on the undercloud it says sent tftpboot to the IP16:08
dtantsurtrozet, the iPXE version looks correct16:08
jaosoriorgfidente: thought we had to support n-1 t-h-t ?16:08
lucasagomestrozet, probably it's failing to get an IP from the DHCP server, can you check with tcpdump and see if iPXE is sending DHCPREQUEST packets ?16:08
*** fzdarsky is now known as fzdarsky|afk16:09
trozetlucasagomes: yeah I need to get vpn access to this pod so it will take me a little bit16:09
lucasagomestrozet, $ sudo tcpdump -i any port 69 should do it16:09
trozetlucasagomes: but you suspect hte pxe boot is fine, and its some network problem?16:09
trozetlucasagomes: or some configuration problem on undercloud16:09
dtantsurI would take -i port 67 or port 68 or port 6916:09
jaosoriorEmilienM: any idea where the logs go to from the calls that puppet-augeas do?16:10
trozetdtantsur, lucasagomes: ok will do that hten report back with what I find16:10
gfidentejaosorior, right sorry, forgot n-1 :)16:10
*** ifarkas is now known as ifarkas_afk16:11
lucasagomestrozet, right, it could be network problem but it's a bit unlikely since the PXE worked before (it chainloaded to iPXE)16:11
dtantsurfolks, have we had a promotion recently?16:11
lucasagomestrozet, it could be a problem wiht the ipxe image itself, I also would recommend testing with the ipxe image from to see if the problem for that nic was fixed already16:12
lucasagomestrozet, you can download it from here (the latest version already compiled from master)16:12
*** beagles_brb is now known as beagles16:12
lucasagomestrozet, see this too:
lucasagomesdtantsur, ?16:13
openstackgerritMerged openstack-infra/tripleo-ci: Extract /var/log from all nodes
* lucasagomes might be missing the promotion context16:13
dtantsurlucasagomes, this is unrelated to ironic question, please ignore :)16:13
trozetlucasagomes: ok thanks16:13
*** abregman_|mtg has quit IRC16:17
*** ohamada has quit IRC16:19
*** atom_ has quit IRC16:20
*** dprince has joined #tripleo16:21
*** bana_k has joined #tripleo16:23
*** jpich has quit IRC16:25
hewbroccaFolks, I gotta head out16:25
hewbroccaKeep those patches coming16:25
*** limao has quit IRC16:26
dtantsurhmmm, is it a known problem on master:16:27
dtantsurInvalid input [name=tripleo.baremetal.v1.register_or_update, class=DirectWorkflowSpec, unexpected=[u'instance_boot_option']]16:27
dtantsursome mismatch between the client and workflows?16:28
dtantsurdprince, have you tried locally anyhow? My new local environment does not work...16:28
*** yamahata has quit IRC16:28
*** abregman has joined #tripleo16:30
*** jprovazn has joined #tripleo16:30
dprincedtantsur: haven't tried it16:31
dprincedtantsur: this is why I separated the patches FWIW16:31
dtantsursigh... ok, I'll continue repairing my env, will report back16:31
dprincedtantsur: but the local IP configuration was definately wrong in the first patch16:31
dtantsurdprince, it was wrong but working :) so now I want to make sure we switch to a correct and working, not to a correct and broken16:32
*** abregman has quit IRC16:34
paramitedprince, EmilienM: seems like both are green, can you guys check it before there will be another merge conflic?16:34
dprincedtantsur: could it be Ironic API needs to be on the ctplane too since IPA will try to contact back on it?16:36
*** bana_k has quit IRC16:36
dtantsurdprince, yeah, this is correct. good catch16:36
dtantsurbut that goes to haproxy configuration, right?16:37
dprincedtantsur: we should just be able to change the value in ServiceNetMap I think16:37
dtantsurnot sure I understand it :(16:38
dtantsurhmm, or maybe I do. sorry, pretty late here, my brain does not work too well16:39
*** Guest35918 is now known as mgagne16:39
dtantsurdprince, are you planning to update your patch with it?16:39
*** mgagne has quit IRC16:39
*** mgagne has joined #tripleo16:39
dprincedtantsur: commented on the patch, I will update if you agree16:39
dtantsurdprince, yeah, I agree. thanks!16:40
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Fix Ironic service bind IPs
dprincedtantsur: okay, lets see how this plays16:40
dprincedtantsur: thanks for the feedback on this16:40
dtantsurthank you! my understanding of OoO networking leaves a lot to be desired still16:41
dtantsurI'll test it as soon as I figure out broken workflows16:41
*** tobias_fiberdata has joined #tripleo16:41
*** hewbrocca is now known as hewbrocca-afk16:43
*** lucasagomes is now known as lucas-afk16:45
*** aufi has quit IRC16:45
*** jcoufal has joined #tripleo16:46
*** jbadiapa has quit IRC16:48
*** ioggstream has quit IRC16:49
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Remove hard-coded per-service rabbitmq hieradata
*** jtomasek has quit IRC16:50
bnemecjaosorior: So we want now?16:51
jaosoriorbnemec: I believe we do16:52
jaosoriorbnemec: that is not getting ran if you use master :D16:52
EmilienMparamite: done16:52
*** Arkady_Kanevsky has joined #tripleo16:52
EmilienMparamite: with some comments though16:52
bnemecjaosorior: Okay, I'll update that then.  If we have puppet init for Newton then we probably want the deprecation warning for Newton too.16:53
paramiteEmilienM, well LoggigSOurces and LoggingGroups is also on the similar base as MonitoringSubscriptions or ServiceNames16:54
Arkady_Kanevskygiulio, per you request bringing swift/ceph chat here16:54
openstackgerritOpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements
Arkady_Kanevskyquestion is do we want to support an option that we have single proxy on controllers that will be swift one, and does that config automatically use local ceph RGW on controller node16:55
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements
gfidentetty tomorrow guys16:55
gfidentebye all16:56
EmilienMgfidente: o/16:56
jaosoriorgfidente: have a good one16:56
EmilienMjaosorior: not sure about logs16:56
Arkady_KanevskyAlternative is to use current 2 proxies: one for swift via HA proxy, and one for Ceph. Not sure if Ceph will be udner ha-proxy or not16:56
EmilienMjaosorior: iscsi?16:56
jaosoriorEmilienM: not sure either, and didn't figure it out16:56
jaosoriorEmilienM: for now, I'm getting a strange augeas error related to lvm.conf16:56
jaosoriorwith the latest undercloud16:57
EmilienMwe might need to revert it16:57
jaosoriorEmilienM: what merged related to lvm.conf?16:57
tdasilvaArkady_Kanevsky: what's the use case for having both a swift proxy and RGW node?16:57
jaosoriorEmilienM: it's failing on update, FWIW16:58
EmilienMwe really need the undercloud upgrade job I did16:58
Arkady_Kanevskyswift front (API & horizon) and ceph providing object stoarge via RGW16:58
jaosoriorEmilienM: might be this then
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Implement undercloud upgrade job - Mitaka -> Newton
EmilienMlet's see16:58
Arkady_Kanevskymy appologies to jumpinng in the middle of other discusoisn16:59
*** gfidente has quit IRC16:59
jaosoriorI gotta go, but will probably get back to this tomorrow16:59
EmilienMjaosorior: thx I'll look too this aft16:59
jaosoriorHave a good rest of the day folks!16:59
*** lucas-afk is now known as lucasagomes17:00
*** jaosorior has quit IRC17:00
dprincebnemec: you might review this. I need this setting for tenant vlan networks
*** vijaykc4 has joined #tripleo17:00
tdasilvaArkady_Kanevsky: sorry, I must be missing some context. I'm not sure what you mean by "swift front (API & horizon)" If ceph is going to provide object storage via RGW, what's the need for a swift proxy behind HAProxy and vice versa?17:02
dprincebnemec: specifically my external net requires it when using network isolation17:02
*** chem has quit IRC17:03
vijaykc4dprince, we are waiting for our patches to get past the OVB CI and get our workflows... considering today's the last day how does it work?17:04
openstackgerritJohn Browning proposed openstack/tripleo-validations: Added check for keystone token_flush cronjob.
dprincevijaykc4: after today you'd apply for feature freeze exceptions17:05
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: [WIP] Heat stack for monitoring/logging node
*** yamahata has joined #tripleo17:07
bnemecdprince: Bleh, MTU17:07
bnemec+2 on the patch17:08
*** jpena is now known as jpena|off17:09
*** tobias_fiberdata has quit IRC17:09
*** jtomasek has joined #tripleo17:10
*** tobias_fiberdata has joined #tripleo17:11
*** paramite has quit IRC17:11
vijaykc4dprince, thanks.17:12
larsksdprince: EmilienM: well, not mmagr as a +2 and I have a -1, so we're at about the same place.  EmilienM, re: your comment, I can't put those configs in the individual service definition because they need the aggregate values which are only available from the ServiceChain.  I could probably move those up into overcloud.yaml instead; does that make sense?17:12
Arkady_Kanevsky<tdasilva>: as a user I want to use swift APIs. But I donot want to deplpoy swift. Instead I want to use new or existing Ceph cluster. and configure swift to use Ceph for it.17:14
Arkady_KanevskyThis works now. But currently there are 2 proxys. One for swift as part of swift confug. And another part of ceph RGW config17:14
trownderekh: did the mirror server move? or does tripleo not publish any images any more? been out for a few weeks :)17:14
dsneddondprince, This change should be done in conjunction with removing the NeutronTenantMtu and instead using the NeutronGlobalPhysnetMtu to calculate things like path_mtu. If you'd prefer, I can do that in a follow up patch to
dprincedsneddon: NeutronTenantMTU was removed almost 2 months ago17:18
dprincedsneddon: I just pushed this because I needed it to get my local dev environment working17:19
dsneddondprince, Oops, my git repo is not on master branch.17:19
dsneddondprince, thanks17:19
dprincedsneddon: apparently nobody is testing network isolation with an external network on a Vlan besides me :)17:19
dprincedsneddon: I mentioned this in my commit message, but there is a potential upgrade impact here17:20
dsneddondprince, I've been having a tough time testing master because I don't have enough RAM to handle Newton virt envs, so I'm behind on my testng.17:20
dprincedsneddon: Neutron as I understand it is supposed to leave the MTU's of previously existing networks alone, but users deploying from scratch may hit issues due to this config change17:20
dprincedsneddon: so in hindsight perhaps we should have treated the removal of NeutronTenantMTU a bit more carefullly in TripleO...17:21
dsneddondprince, Hmm, I'm surprised that you would need to set GlobalPhysnetMtu lower than physical. I was under the impression that Neutron would do the right thing and subtract the overhead from the physical MTU for VXLAN.17:22
dsneddondprince, But I'll take a look at that and see if there is someplace we need to take that into account, like we did with forcing the MTU through dnsmasq based on the NeutronTenantMtu.17:22
beaglesdsneddon, dprince, if neutron isn't subtracting the tunnel overhead there is a bug somewhere17:23
dprincedsneddon, beagles: neutron only claims to auto magically set the MTU for tunnelled networks like GRE and VXLAN I think17:24
beaglesdsneddon, dprince, unless the MTU of the actual underlying physical network is less than 1500, you shouldn't have to set anything at all17:24
beaglesdprince, yeah.. you are talking about VLAN right?17:24
dprincedsneddon, beagles: for VLAN, flat networks the *global* setting gets used instead17:24
beaglesdprince, yup that's right17:24
dsneddonbeagles, And if the physical network is less than 1500, you should still only have to set the global physical mtu equal to the real MTU.17:24
beaglesdsneddon, correct17:24
EmilienMlarsks: I'm ok with a first iteratio17:25
EmilienMlarsks: just keep in mind the files won't exist anymore17:25
EmilienM+2 on both17:25
larsksEmilienM: what do you think about my proposal for moving those settings into overcloud.yaml?  Or are you thinking of a different solution?17:26
EmilienMlarsks: it would be great indeed17:28
openstackgerritHarry Rybacki proposed openstack/tripleo-quickstart: Revert "Move unique socket generation into"
*** bana_k has joined #tripleo17:31
*** trown is now known as trown|lunch17:32
*** yamahata__ has quit IRC17:33
beaglesdsneddon: in your patch, when you say "previous" and "has" are you referring to before this patch or before newton?17:33
beaglesdsneddon, the global physnet mtu patch, I'm referring to17:33
*** maeca1 has quit IRC17:34
*** electrofelix has quit IRC17:37
beaglesdprince, is your "infrastructure"'s network MTU less than 1500?17:38
dprincebeagles: no17:38
dprincebeagles: just normal I think17:39
*** pkovar has quit IRC17:39
* beagles sees the three letter MTU and goes snaky these days ;)17:41
dprincebeagles: once I apply that setting I set my global_physnet_mtu: 140017:43
dprincebeagles: and it has the effect of making my vlan network show up like that which is correct17:43
*** tosky has quit IRC17:43
dprincebeagles: without it I'm able to ping instances, but I can't ssh to them. Classic MTU issue17:44
*** fragatina has quit IRC17:44
beaglesdprince, is this a fresh system or post upgrade?17:44
dprincebeagles: totally fresh17:44
karthiksEmilienM,  For, we've used recursion in puppet for iterating a list.  Do you see any issues here ?17:44
dprincebeagles: this setting is required for me for all new deployments from master17:44
dprincebeagles: to make it work like it used to... i.e. having an external network on a vlan (segment 10)17:45
*** tobias_fiberdata has quit IRC17:45
dtantsurfolks, is it time already to request FFEs? I see people doing that...17:46
beaglesdprince, is this an ovb thing?17:46
dprincebeagles: I don't use OVB locally :)17:46
dprincebeagles: these are real machines.17:46
dprincebeagles: its a *real* thing17:46
EmilienMkarthiks: where?17:46
dprincebeagles: real machines, Cisco switch, real vlans. Same setup that worked for Mitaka....17:47
EmilienMkarthiks: this should work17:47
karthiksThanks EmilienM . Just to check if the approach is ok :)17:48
beaglesdprince, this strikes me as odd17:49
dprincebeagles: perhaps I'm missing something? Is my setup odd you mean? Or the fact that this need to be set?17:49
beaglesdprince, that this needs to be set17:50
dprincebeagles: well, if you read the Neutron release notes I think it is intended behavior17:50
skramajadtantsur: we are already at the end of 29th accorrding to IST :)17:51
dprincebeagles: and yes, I am impling there is a hidden bug in that Neutron has refactored the way MTU's are handled. They have upgrades code in place... but this will still confuse people deploying with the new code17:52
dtantsurskramaja, FF is not on 29th, it's on the week of 29th, hence my question17:53
*** xuao has quit IRC17:54
*** rajinir has joined #tripleo17:55
karthiksdtantsur, we thought FF is on 29th. isn't it dprince  ?17:57
dprincekarthiks, dtantsur: see the Plans re newton-3 release and feature freeze exceptions'17:59
dprinceemail to the list from Steve17:59
dtantsurdprince, it does not have dates, that's the problem :)17:59
dprinceI think the plan is to tag the release tomorrow17:59
dprincedtantsur: ^^17:59
dtantsur(unless I've missed them)17:59
dtantsurah cool. so after tomorrow I start begging for FFE for ironic work, right? :)17:59
dprincedtantsur: I guess :)17:59
rookAnyone have a osp+ceph cluster up... curious if you could try something :/18:01
*** isq has joined #tripleo18:01
rooknewton cloud.18:01
dprincerook: talk to me18:01
rookdprince hey man, if you have a instance on your overcloud... if you issue nova image-create <guest you want to copy> <name to put into glance> ; does it work?18:02
rookdprince: I am getting a traceback on the computes :
dprincerook: ack, trying18:02
rookhowever I am not finding much out there...18:02
rookw/o ceph, this operation works great18:02
*** fragatina has joined #tripleo18:02
*** dtantsur is now known as dtantsur|afk18:02
dprincerook: worked for me, the image went 'ACTIVE' at least18:03
dprincerook: looking at the logs...18:03
rookdprince interesting.18:03
dprincerook: did your image go active?18:04
rookdprince negative.. the guest went down, and never came back up.. no image was created.18:04
rookdprince was this image raw?18:04
rookmaybe that is the hiccup18:04
dprincerook: could be18:04
dprincerook: I was using RBD, ceph backed instance18:04
rooki import into glance a qcow image (stupid) instead of raw...18:04
rookyeah, me too18:04
beaglesdprince, sorry, was off down a rabbit hole. Are you setting an MTU of 1400 somewhere, like in an undercloud.conf or something?18:05
dprincebeagles: no, just this new setting18:05
beaglesdprince, oh.. it wasn't 1400 before?18:05
rookdprince: let me try setting the image to raw and try again... I am also seeing a MTU issue with Newton :/18:05
dprincerook: then my patch might fix you!18:06
beaglesdprince,  wati.. backing up18:06
beaglesdprince, the paste you sent before, was that 1400 before or after your patch?18:06
dprincebeagles: correct, before it wasn't since we revoved NeutronTenantMTU18:06
dprincebeagles: after18:06
beaglesdprince, ah okay18:06
dprincebeagles: if I didn't apply my patch then ext-net would ping but ssh fails18:07
rookdprince: word... can we please figure out a way to get MTU right :P18:07
rookyeah, ssh fails... but so does large user-data payloads from nova18:07
dprincerook: set that to 1400 and it WFM18:07
ayoungdo we havea doc or something explaining how mistral is going to work in with Heat for openstack overcloud deploy?18:09
ayoungdprince, thanks18:10
*** trown|lunch is now known as trown18:11
*** jcoufal has quit IRC18:11
ayoungdprince, and Mistral is going to be part of the Newton release, right?18:11
dprinceayoung: it has to be, critical feature18:12
dprinceayoung: Mistral itself was in the undercloud for Mitaka. Just new...18:13
dprinceayoung: we are only now starting to use it18:13
ayoungdprince, and it is only going to be used to kick off the Heat workflow from the undercloud, right?  It will not e something that we can use during an overcloud deployment to do complex workflows?18:15
dprinceayoung: it is already used for several things. It is a generic in-cloud workflow service18:16
dprinceayoung: I used it to implement 'openstack overcloud execute' for example18:16
*** mcornea has quit IRC18:17
ayoungdprince, so when we deploy an overcloud, we need to do some IPA operations that really should not be executed on the controller or compute nodes.  THey would have to happen in the middle of an overcloud deploy.  So heat would call Mistral.  That is not part of what we can do now, right?  But, I could have a bunch of stuff done in mistral prior to calling heat for, say an overcloud deployment?18:18
dprinceayoung: right, we haven't gotten into that business yet. But we can.18:18
dprinceayoung: I would like to do a similar thing for Swift rings for example18:19
*** isq has quit IRC18:19
dprinceayoung: we want to add to our composable services with "composable workflows" interface I think18:19
dprinceayoung: so that any service can tie into it and do some custom workflow'y thinks18:19
dprinceayoung: haven't built that yet... I think it is more of an O thing18:20
*** isq has joined #tripleo18:20
ayoungdprince, OK, I'll go do my homework.  The thing I like about working through Mistral is we can set it up with a Kerberos principal, set up access controls , and do all the security hardening properly to do things like create new hosts and services, grant new hosts elevated degrees of permissions if required and all that. It really should not be done on the controller, but right now that is our only option.18:21
openstackgerritSarath Kumar Sankaran Kutty proposed openstack/os-net-config: Cleanups and new unit test for IVS
*** maeca1 has joined #tripleo18:24
*** jcoufal has joined #tripleo18:25
*** isq has quit IRC18:26
*** chem has joined #tripleo18:28
*** lucasagomes is now known as lucas-dinner18:33
*** isq has joined #tripleo18:34
*** jcoufal has quit IRC18:36
*** jcoufal has joined #tripleo18:37
openstackgerritMerged openstack/tripleo-common: Workflows to load validations
*** liverpooler has quit IRC18:47
*** jbadiapa has joined #tripleo18:50
*** jcoufal has quit IRC18:51
*** jcoufal has joined #tripleo18:52
*** oshvartz has joined #tripleo18:59
EmilienMdprince: any chance to review ? slagle had +2 on it before I rebased19:00
dprinceEmilienM: looking19:00
EmilienMdprince: it's non voting jobs FYI19:01
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: configure Heat with new authtoken parameters
*** jcoufal has quit IRC19:06
*** jcoufal has joined #tripleo19:07
*** fzdarsky|afk has quit IRC19:14
*** mhenkel has quit IRC19:14
openstackgerritJeff Peeler proposed openstack/tripleo-common: Add List Roles Action
*** rasca has quit IRC19:21
*** rbowen has quit IRC19:24
openstackgerritMichele Baldessari proposed openstack/puppet-pacemaker: WIP DO NOT MERGE for location_rule constraints
*** rbowen has joined #tripleo19:25
*** radeks has joined #tripleo19:25
rookso dprince looked at this MTU thing with beagles some..19:31
rookI am not convinced using a 1400 mtu is the right thing..19:31
rookI was able to leave my MTUs at 1450 across the env-- except the L3, i set that  to 1446 and everything still worked.19:32
rook*accounting for a 4byte vlan* which I am not sure why that isn't being stripped19:32
rookdsneddon ^^^^19:32
dprincerook: what you set it to is up to you19:33
dprincerook: the point is making it configurable19:33
rookdprince: ok - as long as the default isn't taking us back in time to a 1350 MTU19:33
dprincerook: I just used 1400 because that was the default value of our old NeutronTenantMtu setting in t-h-t19:33
rookdprince just making sure.19:33
dprincerook: so that setting worked for you then?19:33
rookthere is a configuration issue somewhere -- i believe we should be stripping tags before it hits the tunnel19:34
dprincerook: or you did it manually some other way?19:34
dsneddondprince, rook: Yeah, 1400 should never be necessary. You only need 50 bytes in the worst case.19:34
rookdprince manually tinkering19:34
rookdsneddon yeah man, 50 for the vxlan oh.19:34
rookoverhead, i am getting bad about shorting words.19:34
rookdsneddon: however, i think the vlan tag is making it into the vxlan tunnel, which bumps things another 4bytes.19:35
rookthis is a hunch. /me waits for beagles to backme up or say I am crazy19:35
dprincerook: all my patch does is make use of an existing Neutron flag. I think it is safe to expose it in t-h-t19:35
dsneddonrook, What VXLAN tunnel? I thought dprince was having trouble on his external network, which is a VLAN network.19:35
dprincedsneddon: yes, my ext-net is a vlan network19:36
dprincedsneddon: which gets used for the floating IPs for my instances19:36
rookdsneddon ah, same problem different envs.19:36
rookthe instances use a vxlan tunnel?19:36
rookor not19:36
rookmine was, ext-net (vlan) tenant-net(vxlan)19:36
*** skramaja has quit IRC19:38
*** fragatina has quit IRC19:39
dprincerook: exactly, that is what I'm doing too. ext-net is the vlan network19:39
*** fragatina has joined #tripleo19:39
vijaykc4EmilienM, dprince  the builds are done for these patches: need your attention please.19:41
*** pradk has quit IRC19:46
EmilienMvijaykc4: ack19:46
*** vijaykc4 has quit IRC19:47
openstackgerritMerged openstack/tripleo-docs: Deploy and Scale Swift
colonwqEmilienM, For the scenario004 patch, I am working on getting RGW to work as an endpoint with SSL.19:49
EmilienMcolonwq: ok19:50
dprincerbrady: hi, left some feedback on
dprincerbrady: hit an odd template error19:54
dprincerbrady: something munged my network template or something...19:54
rbradydprince: did you happen to add a template that did not contain json or yaml?19:56
*** sshnaidm is now known as sshnaidm|afk19:57
openstackgerritChristian Schwede proposed openstack/tripleo-docs: Update advanced Swift deployment doc
openstackgerritChristian Schwede proposed openstack/tripleo-docs: Include Swift ring building example
*** pradk has joined #tripleo19:59
dprincerbrady: nothing out of the ordinary20:02
dprincerbrady: I'm using the same templates from t-h-t master20:02
dprincerbrady: I use network isolation, perhaps that is the difference here?20:03
*** derekh has quit IRC20:03
*** radeks has quit IRC20:04
*** maeca1 has quit IRC20:04
larsksdoes anyone have a moment to chat about "split stack" deployments as we have previously discussed w/r/t the opstools work?20:09
rbradydprince: maybe. can you tell if this error is being thrown by template_utils.process_multiple_environments_and_files or is it from heat.stacks.create ?20:10
rbradydprince: and what is in custom.yaml?20:10
*** vijaykc4 has joined #tripleo20:12
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: WIP DO NOT MERGE Initial Composable HA
*** trown is now known as trown|outtypewww20:16
vijaykc4EmilienM, dprince : thanks... hope we can close these patches tomorrow and the CI errors will be resolved. Good night20:19
*** vijaykc4 has quit IRC20:22
*** milan has quit IRC20:26
dprincerbrady: custom.yaml is my own YAML file20:32
dprincerbrady: it is how I inject parameters in convieniently20:32
beaglesrook: not crazy.. there is something weird if bumping your physical device's MTU to 9000 but nothing else works, or that dropping the effective MTU of the tenant network at the router by 4 bytes does the trick.. there is something else really weird20:38
openstackgerritBen Nemec proposed openstack/python-tripleoclient: Re-enable keystone init deprecation message
*** jbadiapa has quit IRC20:42
*** akrivoka has quit IRC20:45
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Port plan-edit file listing to use Swift
*** dprince has quit IRC20:49
*** florianf has quit IRC20:50
openstackgerritAthlan-Guyot sofer proposed openstack/tripleo-heat-templates: M/M upgrade sahara-api fails to restart.
*** jayg is now known as jayg|g0n320:54
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: pin the undercloud-post and overcloud roles
openstackgerritMathieu Mitchell proposed openstack/diskimage-builder: Generate and use upper-constraints for ironic-agent
*** Goneri has quit IRC21:09
*** ioggstream has joined #tripleo21:18
*** jprovazn has quit IRC21:22
*** jcoufal has quit IRC21:24
*** dmacpher has quit IRC21:24
*** Arkady_Kanevsky has quit IRC21:24
*** jcoufal has joined #tripleo21:24
*** dmacpher has joined #tripleo21:36
*** lucas-dinner has quit IRC21:56
openstackgerritRyan Brady proposed openstack/tripleo-common: WIP wire in jinja templating for custom roles
*** wfoster has quit IRC21:56
*** sai has quit IRC21:58
*** rook has quit IRC21:58
*** wfoster has joined #tripleo21:59
*** sai has joined #tripleo21:59
*** lucasagomes has joined #tripleo21:59
*** rook has joined #tripleo22:00
*** rook is now known as Guest6478222:00
*** rhallisey has quit IRC22:01
*** ayoung has quit IRC22:07
*** assassin has quit IRC22:12
*** ioggstream has quit IRC22:33
*** fragatin_ has joined #tripleo22:35
*** yamahata has quit IRC22:36
*** fragatina has quit IRC22:38
*** rlandy has quit IRC22:43
*** Goneri has joined #tripleo22:43
*** yamahata has joined #tripleo22:55
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: configure Heat with new authtoken parameters
*** rajinir has quit IRC23:05
*** mpsairam has joined #tripleo23:06
*** egafford has quit IRC23:12
*** jmiu has quit IRC23:12
*** jmiu has joined #tripleo23:13
*** pradk has quit IRC23:17
*** ayoung has joined #tripleo23:20
*** saneax-_-|AFK is now known as saneax23:20
openstackgerritMerged openstack/puppet-tripleo: Configure the numvfs for SRIOV interfaces
*** myoung is now known as myoung|afk23:31
*** fultonj has quit IRC23:34
*** Guest64782 is now known as rook23:34
openstackgerritMerged openstack-infra/tripleo-ci: Implement scenario001 CI job

Generated by 2.14.0 by Marius Gedminas - find it at!