Monday, 2016-04-11

*** jerrygb has joined #tripleo00:22
*** lblanchard has joined #tripleo00:26
*** chlong has quit IRC00:27
*** fragatina has quit IRC00:29
*** ayoung has joined #tripleo00:32
*** chlong has joined #tripleo00:34
*** tiswanso has joined #tripleo00:45
*** tiswanso has quit IRC00:49
*** dmacpher has joined #tripleo01:05
*** julim has quit IRC01:09
*** julim has joined #tripleo01:11
openstackgerritMerged openstack/tripleo-heat-templates: Replace extraconfig/tasks/noop.yaml w/ Heat::None  https://review.openstack.org/30337001:14
*** tiswanso has joined #tripleo01:15
*** tiswanso has quit IRC01:16
*** tiswanso has joined #tripleo01:17
*** ramishra has joined #tripleo01:17
*** fragatina has joined #tripleo01:25
*** lblanchard has quit IRC01:36
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron dhcp service  https://review.openstack.org/30338602:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron metadata service  https://review.openstack.org/30361802:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron l3 service  https://review.openstack.org/30356202:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable glance services  https://review.openstack.org/23737002:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add GlanceRegistry to the endpoint map  https://review.openstack.org/30372802:11
*** tiswanso has quit IRC02:16
*** ramishra has quit IRC02:21
*** ramishra has joined #tripleo02:21
*** jerrygb has quit IRC02:29
*** jerrygb has joined #tripleo02:42
*** shivrao has joined #tripleo02:45
*** yuanying has quit IRC02:46
*** akshai has quit IRC02:46
*** MaxPC has joined #tripleo02:59
*** yamahata has joined #tripleo03:13
*** jerrygb has quit IRC03:41
*** Ryjedo has joined #tripleo03:43
*** links has joined #tripleo03:45
*** yuanying has joined #tripleo03:49
*** jerrygb has joined #tripleo03:51
*** jerrygb has quit IRC03:51
*** coolsvap has joined #tripleo03:51
*** jerrygb has joined #tripleo03:51
*** jerrygb has quit IRC03:51
*** MaxPC has quit IRC04:10
*** cwolferh has quit IRC04:17
*** liverpooler has joined #tripleo04:27
*** liverpooler has quit IRC04:27
*** liverpooler has joined #tripleo04:27
*** jerrygb has joined #tripleo04:28
*** aparnav has joined #tripleo04:28
*** r-mibu has quit IRC04:30
*** oshvartz has quit IRC04:34
*** r-mibu has joined #tripleo04:35
*** cwolferh has joined #tripleo04:37
*** jerrygb has quit IRC04:37
*** jerrygb has joined #tripleo04:38
openstackgerritMerged openstack/os-net-config: Bump hacking in test-requirements.txt  https://review.openstack.org/30186304:38
openstackgerritMerged openstack-infra/tripleo-ci: Upload the ironic-python-agent images to cache  https://review.openstack.org/30169804:39
openstackgerritMerged openstack/tripleo-puppet-elements: Gnocchi Integration  https://review.openstack.org/23801304:41
*** jerrygb has quit IRC04:43
openstackgerritMerged openstack/tripleo-common: Fix setting deploy kernel and image when updating nodes  https://review.openstack.org/30109604:46
*** jaosorior has joined #tripleo04:46
jaosoriormarios: Got a +A for this? https://review.openstack.org/#/c/292799/ :D also would really use a review here: https://review.openstack.org/#/c/299475/04:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Don't have separate protocols/ports for Keystone v3  https://review.openstack.org/28426504:50
*** florianf has joined #tripleo04:55
*** ramishra has quit IRC05:04
*** ramishra has joined #tripleo05:04
*** saneax_AFK is now known as saneax05:18
*** numans has joined #tripleo05:24
*** pcaruana has joined #tripleo05:39
*** liverpooler has quit IRC05:39
*** liverpooler has joined #tripleo05:41
mariosjaosorior: morning - will do in  a sec (gerrit is a bit slow for me - miht be my connection here05:55
*** oshvartz has joined #tripleo05:59
jaosoriormarios: No problem, thanks dude06:01
jaosoriorhow's it going?06:01
mariosjaosorior: still trying to loginto gerrit :/06:08
openstackgerritMerged openstack/diskimage-builder: Turn down tracing for source-repo cache  https://review.openstack.org/29556906:12
*** chlong has quit IRC06:13
*** rcernin has joined #tripleo06:15
openstackgerritMerged openstack/tripleo-common: Expose documented --seconds option  https://review.openstack.org/29964206:18
*** hjensas has quit IRC06:21
*** aufi has joined #tripleo06:28
openstackgerritMerged openstack/tripleo-heat-templates: Change /etc/hosts format and add domain  https://review.openstack.org/29279906:29
jaosoriorthanks marios :D06:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add EndpointMap as output  https://review.openstack.org/30391706:31
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add EndpointMap as output  https://review.openstack.org/30391806:33
*** tesseract has joined #tripleo06:35
*** tesseract is now known as Guest898906:35
*** jtomasek has joined #tripleo06:39
*** coolsvap has quit IRC06:43
*** tremble has joined #tripleo06:43
*** tremble has joined #tripleo06:43
*** shivrao_ has joined #tripleo06:55
*** coolsvap has joined #tripleo06:57
*** shivrao has quit IRC06:58
*** shivrao_ is now known as shivrao06:58
*** jaosorior has quit IRC07:03
*** jaosorior has joined #tripleo07:04
aparnavHey good morning all07:05
aparnavCan anyone take a look at this patch https://review.openstack.org/#/c/295203/ ? Thank you :)07:05
*** jprovazn has joined #tripleo07:09
*** mikelk has joined #tripleo07:11
*** ccamacho has joined #tripleo07:18
jaosorioraparnav: done07:20
aparnavjaosorior: thank you. I will look into it.07:21
*** shivrao has quit IRC07:24
*** paramite has joined #tripleo07:25
*** mcornea has joined #tripleo07:25
*** hjensas has joined #tripleo07:27
*** hjensas has quit IRC07:27
*** hjensas has joined #tripleo07:27
*** dsneddon has quit IRC07:29
*** dmacpher has quit IRC07:29
*** chlong has joined #tripleo07:30
*** shardy has joined #tripleo07:34
*** jpena|off is now known as jpena07:37
*** jaosorior has quit IRC07:47
*** mkovacik has joined #tripleo07:48
*** jaosorior has joined #tripleo07:49
jaosoriorshardy: Hey dude, so it's clear that at the moment those alarms are not being used, right? https://review.openstack.org/#/c/302896/07:51
shardyjaosorior: I just commented, AFAIK yes07:51
shardywe don't use them for anything, and there was a discussion which I copied from IRC logs that indicates cloudforms also don't use them07:51
shardyin the event folks want them, they simply flip the boolean in the undercloud.conf07:52
jaosoriorshardy: Alright, that's what I had understood but just wanted to make sure. Had +2ed anyway07:52
shardyjaosorior: I did consider leaving it enabled by default, but our ram usage keeps going up, and we've added stuff like sensu which uses a lot of ram and is arguably a more appropriate/useful tool than ceilometer in this case07:53
shardyIf folks complain, we'll just enable it by default and turn it off for CI07:53
jaosoriorthat makes sense to me07:53
*** mgould has joined #tripleo07:53
shardyI'd prefer our default to only enable things we actually document/use tho07:54
*** hjensas has quit IRC07:54
*** mbound has joined #tripleo07:57
*** pblaho has joined #tripleo07:57
*** chem has joined #tripleo07:57
*** hjensas has joined #tripleo08:02
jaosoriormarios, shardy: small commit all green in the CI https://review.openstack.org/#/c/302573/1 :D08:02
jaosoriornevermind :/ messed up the documentation08:03
*** zoli|wfh is now known as zoliXXL08:06
*** gfidente has joined #tripleo08:08
mariosjaosorior: looking now08:09
mariosjaosorior: k you gonna update?08:09
jaosoriormarios: Thinking about pushing a later addition. There are 3 changes in that chain, and it would be a bummer to run them all again08:10
jaosoriorsince they are all passing CI08:10
*** yamahata has quit IRC08:10
mariosjaosorior: from shortlog i don't see a chain though ? /me checks again https://review.openstack.org/gitweb?p=openstack/puppet-tripleo.git;a=shortlog;h=f3bb9638788d51fe72684cfabdba142a66acc0af08:12
jaosoriormarios: unfortunately gerrit no longer shows you chains properly... only the Depends-On stuff, but not if a commit'08:13
jaosoriorcommit'08:13
jaosoriorcommit's parent is not master or stuff like that08:13
gfidenteyeah that 'related chages' thing is confusing for me as well08:14
jaosoriormarios: All of these commits except the one for python-tripleoclient https://review.openstack.org/#/q/topic:tls_enablement+status:open+owner:"Juan+Antonio+Osorio+Robles+%3Cjaosorior%40redhat.com%3E" are a chain08:14
mariosjaosorior: yeah i thought that is what you meant by chain, like you have 2/3 commits ontop of master and you publish them together08:14
jaosorioryep08:14
mariosjaosorior: you mean same topic08:14
mariosjaosorior: k08:14
jaosoriorthey have the same topic08:14
jaosoriorso rebasing "Add missing services ports to service_ports map" also rebases "Remove individual service certificates" and "Add generic manifest for loadbalancer endpoints" and eventually "Add support for internal/admin endpoint TLS in HAProxy" which is the leaf08:15
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port documentation  https://review.openstack.org/30395408:16
jaosoriorgfidente: Yeah... that sucks. Gerrit used to show which was the parent commit properly. Don't know why they got rid of that08:17
*** gfidente has quit IRC08:19
jaosoriormarios: Anyway, the chain looks like this in git http://paste.openstack.org/show/493609/08:19
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Deployment Plan page updates  https://review.openstack.org/30395808:23
*** gfidente has joined #tripleo08:25
mariosjaosorior: ack thanks, comment/clarification on review08:27
jaosoriormarios: Responded in the review. Thanks dude!08:28
jaosoriormarios: But yeah, there will be no issues as that is just the documentation. The actual code has the correct port08:28
mariosjaosorior: lol, completely missed the commment :)08:29
mariosjaosorior: /me done for reviews today08:29
mariosjaosorior: updated08:29
jaosoriormarios: haha fair enough. Thanks man08:30
openstackgerritMerged openstack/diskimage-builder: Debian: dont set always the hostname to debian  https://review.openstack.org/20504708:31
*** jistr has joined #tripleo08:32
*** derekh has joined #tripleo08:33
shardyguys I just refactored all the summit topic suggestions into a proposed agenda:08:34
shardyhttps://etherpad.openstack.org/p/newton-tripleo-sessions08:34
shardyI'll mail the list but now is a good time to say if you have any changes/additions ;)08:35
jaosoriorshardy: Is there still a chance to update that and add topics?08:35
*** snecklifter has joined #tripleo08:36
*** aparnav has quit IRC08:37
*** appu103_ has joined #tripleo08:38
*** ramishra has quit IRC08:39
shardyjaosorior: Yes, hence my comment :)08:42
shardyjaosorior: although we've not got any session space remaining, so we'd need to add topics to those already listed, or decide to remove one that's there08:42
*** bvandenh has joined #tripleo08:43
shardyso far I avoided rejecting any of the propoals by combining things which were related08:43
*** fragatina has quit IRC08:43
shardyI have to formally propose the sessions for the summit schedule very soon, so if you have changes you'd like, now is the time to discuss them :)08:43
*** appu103_ is now known as aparnav08:46
gfidenteshardy added a link to migrations, we were discussing it with jistr marios and bandini the other day08:47
jistr++08:48
gfidentethat's about 'how to move from ceilometer alarm to aodh in an upgrade'08:48
gfidentesort of problems08:48
bandiniyes, we definitely need a good discussion about that08:48
shardygfidente: You need to fix the etherpad link, it's internal08:49
gfidentesorry I didn't notice :(08:49
gfidentethanks :)08:49
shardyAgree on the topic tho08:49
shardyI was thinking similar re the move to Gnocchi08:50
gfidentethought it was the public pad08:50
gfidentegnocchi yeah08:50
shardyit's pretty bad to just switch out services without offering any migration path IMO08:50
* jistr copies the pad to etherpad.openstack08:50
bandiniack08:50
ccamachoQuestion, are you planning to record this one "Composable Services and beyond", in the Summit? BTW, good morning!08:51
bandinimorning ccamacho08:52
bandini(no idea about the recordings going on at the meeting ;)08:52
shardyccamacho: Previous summit there was no provision for recording sessions - I'm not sure if there will be this time08:52
ccamachook, np then :)08:52
shardyccamacho: if you're not attending we can try to loop you in via video-call providing the summit wifi works well enough08:52
ccamachoshardy, I didn't got any info about attending to the summit, but a video-call would be great if possible08:54
mariosshardy: how about a 'things we've added to ci recently' (like the upgrades job and the pingtest and net-iso etc etc ) and how to continue to improve/plans/work you can get involved in etc08:56
mariosshardy: there's usually at least one discussion about this stuff anyway08:56
shardymarios: sure, that can go in the CI workroom session, maybe we can adjust the title08:56
shardymarios: added08:58
*** pcaruana has quit IRC09:00
mariosthanks shardy09:01
*** zoliXXL is now known as zoli|llunch09:10
*** sambetts|afk is now known as sambetts09:12
*** ramishra has joined #tripleo09:16
*** ramishra has quit IRC09:16
*** liverpooler has quit IRC09:20
*** liverpooler has joined #tripleo09:20
*** ramishra has joined #tripleo09:21
openstackgerritMerged openstack/puppet-tripleo: Add missing services ports to service_ports map  https://review.openstack.org/30257309:27
openstackgerritMerged openstack/puppet-tripleo: Remove individual service certificates  https://review.openstack.org/30149309:29
openstackgerritMerged openstack/tripleo-incubator: Add bnemec as ci admin  https://review.openstack.org/30353609:29
*** akrivoka has joined #tripleo09:35
*** social has quit IRC09:49
*** social has joined #tripleo09:51
*** social has quit IRC10:03
*** social has joined #tripleo10:03
openstackgerritCyril Lopez proposed openstack/tripleo-docs: The control placement is needed more information to aviod a no valid host found  https://review.openstack.org/30118510:05
*** bvandenh has quit IRC10:07
*** coolsvap has quit IRC10:15
*** bvandenh has joined #tripleo10:16
*** ramishra has quit IRC10:19
*** zoli|llunch is now known as zoli10:23
*** manfredp has joined #tripleo10:23
openstackgerritMerged openstack/tripleo-puppet-elements: Remove tuskar from puppet-modules  https://review.openstack.org/30352110:24
*** manfredp has quit IRC10:24
*** ramishra has joined #tripleo10:24
openstackgerritCarlos Camacho proposed openstack-infra/tripleo-ci: Update deprecated commands in tripleo.sh for cleanup_pingtest method  https://review.openstack.org/30402010:26
openstackgerritCarlos Camacho proposed openstack-infra/tripleo-ci: Update deprecated commands in tripleo.sh for cleanup_pingtest method  https://review.openstack.org/30402010:32
*** apetrich has joined #tripleo10:42
openstackgerritMerged openstack/puppet-tripleo: Fix comparison to control_virtual_ip  https://review.openstack.org/30121610:45
*** jcoufal has joined #tripleo10:48
*** ifarkas has joined #tripleo10:50
*** leanderthal|afk is now known as leanderthal10:59
openstackgerritMerged openstack/tripleo-heat-templates: Don't have separate protocols/ports for Keystone v3  https://review.openstack.org/28426511:00
*** rhallisey has joined #tripleo11:06
*** ramishra has quit IRC11:16
*** Goneri has joined #tripleo11:24
*** ramishra has joined #tripleo11:27
*** mgould has quit IRC11:32
*** weshay has quit IRC11:36
*** MaxPC has joined #tripleo11:39
*** ramishra has quit IRC11:43
*** hjensas has quit IRC11:50
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Map gnocchi vip to haproxy_listen_bind_param  https://review.openstack.org/30179811:51
*** sshnaidm has quit IRC11:51
*** sshnaidm has joined #tripleo11:57
openstackgerritAparna proposed openstack/diskimage-builder: Support to add certificate in ironic-agent  https://review.openstack.org/29520311:58
openstackgerritCarlos Camacho proposed openstack-infra/tripleo-ci: Update deprecated commands in tripleo.sh (Old heat * calls)  https://review.openstack.org/30108712:03
openstackgerritAparna proposed openstack/diskimage-builder: Support to add certificate in ironic-agent  https://review.openstack.org/29520312:10
*** rodrigods has quit IRC12:13
*** rodrigods has joined #tripleo12:13
jaosoriorEmilienM: When you have time, I would really appreciate your input on https://review.openstack.org/#/c/302574/ I'm trying to re-work the loadbalancer.pp from puppet-tripleo to have it easier to modify, as part of the work for enabling TLS on the internal/admin interfaces12:13
*** jayg|g0n3 is now known as jayg12:14
EmilienMjaosorior: sure, will look today12:15
*** bvandenh has quit IRC12:15
*** pradk has joined #tripleo12:17
tremblejaosorior: Did you see the work I'd done a while back?12:17
*** dprince has joined #tripleo12:21
tremblejaosorior: Something very similar: https://review.openstack.org/#/c/240208/13/manifests/loadbalancer/balancer.pp12:21
jaosoriortremble: I actually didn't check it out (I should have). But it ends up kinda similar. Only thing is that in this case I disabled the choice of having a certificate per-service (that was part of another patch) and was trying to make it easier for me to enable TLS for internal endpoints also, as here: https://review.openstack.org/#/c/303342/12:21
jaosoriorbecause now I went ahead and found it again. and damn that would have just been easier to take12:21
jaosoriorX_x12:21
trembleAh well12:22
tremblejaosorior: But yeah that code needed a refactor, way too much Copy&Paste12:23
*** mgould has joined #tripleo12:23
pradkHi can we get this reviewed soon https://review.openstack.org/#/c/301798/ i just had to rebase, pretty minor change.. its blocking https://review.openstack.org/#/c/252032/ from merging12:24
jistrgfidente: hi, i need these changes https://review.openstack.org/#/c/296592 to submit the t-h-t part of fixes for https://bugs.launchpad.net/tripleo/+bug/1567384 and https://bugs.launchpad.net/tripleo/+bug/156738512:25
openstackLaunchpad bug 1567384 in tripleo "Services not restarted on stack-update - config changes can go unapplied" [High,Triaged] - Assigned to Jiří Stránský (jistr)12:25
openstackLaunchpad bug 1567385 in tripleo "Minor update always triggered on first stack-deploy after major upgrade" [High,In progress] - Assigned to Jiří Stránský (jistr)12:25
jistri was about to do the amend we talked about on the review and submit12:25
*** jdob has joined #tripleo12:25
gfidentepradk I that change for gnocci has to go into mitaka right12:25
gfidente?12:25
*** dtantsur is now known as dtantsur|mtg12:25
jistrbut i noticed it depends on bandini's patch12:25
*** weshay has joined #tripleo12:25
jistrcan i un-depend that?12:25
*** hjensas has joined #tripleo12:25
jaosoriortremble: god damn, I see that that commit you had even had the internal SSL already!12:25
pradkgfidente, yes i have backport exception for mitaka.. so once this is merged i'll submit backports12:25
gfidentepradk ack12:26
*** saneax is now known as saneax_AFK12:26
tremblejaosorior: yeah it got hung up waiting on approvals and then I disappeared for a while on Parental Leave12:26
openstackgerritPradeep Kilambi proposed openstack/tripleo-puppet-elements: Gnocchi Integration  https://review.openstack.org/30409012:26
jistrgfidente: if not i'll probably submit a separate patch just for the changes in the pacemaker_resource_restart.sh, as those are the ones which prevent my patch from working if i don't have them12:26
gfidentejistr yeah it doesn't really depend on it12:26
gfidentesec12:26
jistrack, i'll un-depend12:26
jistrand remove the ifs12:26
jaosoriortremble: Congrats on the offspring :D12:27
tremblejaosorior: :)12:27
jaosoriortremble: How long did you take the leave for?12:29
tremblejaosorior: 1 full month, and then 2 months at 50% (where I was so swamped with normal work I definitely couldn't play with tripleo12:29
jistrgfidente: i have it ready, can i submit? (not sure what you meant by 'sec', if you have doubts about the proposal or it was sth else)12:29
*** aparnav has quit IRC12:29
*** rlandy has joined #tripleo12:30
gfidentesure go aheads12:30
jaosoriortremble: Well that's not bad at all!12:30
jistrthx12:30
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Update .sh references from openstack-keystone to openstack-core  https://review.openstack.org/29659212:30
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Fix distinguishing between stack-create and stack-update  https://review.openstack.org/30409412:31
dprincejaosorior, shardy: re https://review.openstack.org/#/c/303728/. I commented again. I would like to refine the use of EndpointMap such that we can consume it both internally and externally12:33
dprincejaosorior, shardy: we need a place for glance registry in the map however because that services has a VIP, and is used in several places internally12:34
dprincegfidente: you apparently did this recently too? ^^^12:35
gfidentedprince so I removed it actually because we don't need it in the endpoints12:35
jaosoriordprince: commenting12:35
gfidentedprince it's a bit like the mysql uri12:36
dprincegfidente: sure, but we do need a handle to that VIP internally12:36
gfidenteor the redis uri12:36
dprincegfidente: I'd like a generic interface for the composable service roles12:36
*** qasims has joined #tripleo12:36
gfidenteyeah there are some services which use a vip not just the list of services12:36
dprincegfidente: and endpointmap is the best fit12:36
jaosoriordprince: Alright, commented in the patch12:37
dprincejaosorior: this is only the begining I think. If I can't mold EndpointMap a bit I'm going to be forced to add duplicate parameters to all sorts of things here: http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/puppet/services/services.yaml#n3212:37
*** jpena is now known as jpena|lunch12:37
jaosoriordprince: I recommend you re-read my comment12:37
jaosoriorlike I said, for me it's fine that glanceregistry is added there12:38
dprincejaosorior: oh, I see12:38
qasimsfolks, Is gate-tripleo-ci-f22-upgrades pipeline on stable/liberty still broken?12:38
dprincejaosorior: but your patch hasn't landed yet right?12:38
dprincejaosorior: would it not be better to name it GlanceApi, and GlanceRegistry?12:38
dprincejaosorior: Glance is ambiguous right?12:39
jaosoriordprince: it would be better, but then I would need to change glance here https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/constants.py#L2312:39
jaosoriorsince that's the list that gets itterated, and adapted to get the endpoints from the map12:40
jaosoriorSo currently it relies on the format being <service name><interface>12:41
jaosorioras to match the names in python-tripleoclient12:41
dprincejaosorior: okay, and I'll remove the GlanceRegistryPublic bit then?12:42
shardydprince: I'm fine with using EndpointMap for it provided we can figure out a way that it's not added to the keystone endpoint list12:43
dprincejaosorior: that way your code can skip over Glance registry entirely because there is no public12:43
jaosoriorshardy: It won't be added there12:43
dprinceshardy: I can simply not add the public part of the glance registry interface12:43
dprinceshardy: easy to remove it12:43
shardyjaosorior: but we're moving to a model where puppet creates all the endpoints in keystone, I had assumed we'd create the entire endpoint map12:43
shardydprince: ack, that sounds like a reasonable compromise12:44
shardymy objection was really that we'd added it, then removed then added - so we need to clarify the scope of EndpointMap and how it relates to the service catalog12:44
jaosoriorshardy: the stuff I'm doing is meant to be backported12:44
jaosoriorthere is a bug that I'm addressing with that stuff12:45
dprinceshardy: agree we should clarify it. I'm going to need to add some internal services to that mapping though12:45
jaosoriorshardy: But yeah, the goal is to stop using os-cloud-config for this stuff12:45
*** ccamacho has quit IRC12:45
dprinceshardy: and if I can't... basically I'm going to end up building another mapping that I can add the internal stuff to anyways12:45
dprinceshardy: basically, it makes sense to share it I think12:45
dprinceshardy: because it can support both use cases quite nicely12:45
shardydprince: Ok, sounds good12:46
jaosoriorshardy: The good thing is that stuff will only be added to the endpoint map now if they are explicitly marked for such by tripleoclient here: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/constants.py#L1812:46
jaosoriorso any other services that appear in the endpoint map will be skipped12:46
jaosoriorthat's both with the old implementation and with the stuff I'm adding12:46
shardyjaosorior: sure, but we want to move away from that client-side keystone config, and thus we need a clear way of defining internal endpoints vs what ends up in the keystone endpoint list12:47
trembleshardy dprince : FWIW my original intent was for internal consumption but that the internal consumption could then feed anything being exposed externally...12:49
jaosoriorshardy: Well, honestly I'm not sure how to do that difference once we move to puppet-generated keystone endpoints12:50
dprincetremble: yes, I like this12:50
openstackgerritMerged openstack/tripleo-heat-templates: composable keystone services  https://review.openstack.org/29558812:51
*** akshai has joined #tripleo12:51
gfidentedprince given there is a similar problem for galera/redis already, any chance we can propagate the glance registry like we do for services like rabbit, from a different hiera?12:51
*** tzumainn has joined #tripleo12:56
*** tzumainn has quit IRC13:00
shardyIs anyone investigating issues where glance fails to start on stable/liberty?13:00
*** tzumainn has joined #tripleo13:01
*** dustins has joined #tripleo13:09
*** trown|outtypewww is now known as trown13:12
*** dmacpher has joined #tripleo13:16
*** jerrygb has joined #tripleo13:16
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Nothing to see here  https://review.openstack.org/11101113:17
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Temporarily disable mistral on the undercloud  https://review.openstack.org/30283213:17
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests  https://review.openstack.org/24416213:18
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/30412513:18
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Move Validations to right sidebar  https://review.openstack.org/30412713:20
*** links has quit IRC13:21
*** tiswanso has joined #tripleo13:21
*** lblanchard has joined #tripleo13:24
jaosoriordprince: Hey dude, the keystone profile work you did conflicted with the creation of the keystone endpoints in t-h-t. So I went ahead and added the puppet code in the keystone profile manifests, was that the right place to put it? https://review.openstack.org/30412513:25
trownshardy: do you know where the tripleo-quickstart-core group would get defined? I see how to create the ACL's in openstack-infra/project-config, but not where to define who belongs to the group13:26
tremblejaosorior: Gerrit/Launchpad isn't letting me log in right now, did you mean to have only ceilometer::keystone::auth depend on keystone::endpoint ?13:29
shardytrown: I thought you do it via the gerrit webui under "people", but I don't have superpowers to do it13:30
shardymaybe that process changed13:30
shardytrown: I do notice we already have core groups for UI and CI13:30
trownyep13:30
jaosoriortremble: I did not, that was unintented, thanks dude!13:30
rhalliseydprince, hey, I left you a question in you glance composable roles review13:31
tremblejaosorior: Oh, actuallyI think you just created a loop with the extra ->13:31
rhalliseydprince, probably easier I mention it instead of writing it out here for discussion13:32
dprincerhallisey: thanks. The last issue I saw was the fact that glance_api_servers isn't getting set correctly13:32
dprincerhallisey: nova::glance_api_servers: ://:13:32
dprincerhallisey: that was causing hiera errors, that ultimately gives a somewhat cryptic puppet apply failure13:33
jaosoriorooh crap13:34
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/30412513:34
rhalliseydprince, crazy13:34
rhalliseydprince, so I was prototyping the container stuff.  I think I can plug right into what you have13:35
rhalliseydprince, I just need to have a way to get a list of tags for puppet13:35
rhalliseyI post what I have soon so we can discuss13:36
dprincerhallisey: so, we would add a 'tags' section to each composable service I think, for the docker stuff13:36
dprincerhallisey: at least that was my plan for it13:36
dprincerhallisey: so like a 'contianer_configs' section13:37
rhalliseyok that's right along the lines of what I was thinking13:37
dprincerhallisey: and {container_configs: puppet_tags} or something13:37
jaosoriortremble: fixed13:37
jaosoriortrown: Hey dude, mind taking another look at this patch? https://review.openstack.org/#/c/299475/13:38
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: WIP: Adds a progress bar for the current deployment  https://review.openstack.org/30343613:39
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Adds a progress bar for the current deployment  https://review.openstack.org/30343613:40
trownjaosorior: sure, I would like someone more familiar with upgrades to take a look too13:40
*** jtomasek has quit IRC13:41
trownjistr: mind checking out https://review.openstack.org/#/c/299475/ from the upgrades angle?13:41
jaosoriortrown: Well, one thing is that that shouldn't be ran for upgrades. If the command is an upgrade, the postconfig gets skipped here: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L93213:42
trownjaosorior: ah good point, I am good with it then13:46
*** jtomasek has joined #tripleo13:48
jaosorioryay :D13:49
*** jpena|lunch is now known as jpena13:49
*** jtomasek has quit IRC13:50
*** egafford has joined #tripleo13:50
*** jtomasek has joined #tripleo13:50
openstackgerritMerged openstack/python-tripleoclient: Raise instead of returning False when take_action fails  https://review.openstack.org/29949413:52
jaosoriornooooo13:52
jaosoriormerge conflict13:52
trownshardy: dprince probably need your +1s on https://review.openstack.org/#/c/304145/13:54
*** lblanchard has quit IRC13:55
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use values from endpoint map for service endpoints  https://review.openstack.org/29947513:56
*** ccamacho has joined #tripleo13:57
jaosoriortrown, thrash: sorry to bother you again guys, there was a merge conflict just now :/ when you get time can you check it out again? https://review.openstack.org/#/c/29947513:57
*** mcornea has quit IRC13:58
*** mcornea has joined #tripleo13:59
trownjaosorior: np :)13:59
thrashjaosorior: ack13:59
jaosoriorthanks!14:00
*** jprovazn has quit IRC14:04
*** lblanchard has joined #tripleo14:05
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron dhcp service  https://review.openstack.org/30338614:08
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron metadata service  https://review.openstack.org/30361814:08
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron l3 service  https://review.openstack.org/30356214:08
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable glance services  https://review.openstack.org/23737014:08
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add GlanceRegistry to the endpoint map  https://review.openstack.org/30372814:08
dprinceshardy, jaosorior ^^^ update the endpoint map to re-add glance registry14:09
jaosoriordprince: one small detail and it's good for me14:10
jaosoriorcommented in the patch14:10
*** jtomasek has quit IRC14:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron dhcp service  https://review.openstack.org/30338614:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron metadata service  https://review.openstack.org/30361814:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron l3 service  https://review.openstack.org/30356214:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable glance services  https://review.openstack.org/23737014:11
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add GlanceRegistry to the endpoint map  https://review.openstack.org/30372814:11
dprincejaosorior: thanks, fixed14:11
dprincejaosorior: the fact that we duplicate the entire set of parameters in enable-tls isn't ideal14:12
jaosoriordprince: We can probably fix that by using map_merge14:13
jaosoriorthat was done before it was available I think14:13
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Use a different ceph key for admin/client user  https://review.openstack.org/30415414:13
dtantsur|mtghey folks! please approve the patch deprecating os_cloud_config.nodes: https://review.openstack.org/#/c/298344/14:13
dtantsur|mtgalready 1x +2 and gate passed14:14
trowndtantsur|mtg: merged14:16
dtantsur|mtgthnx!14:16
openstackgerritMerged openstack/os-cloud-config: Formally deprecate os_cloud_config.nodes in favor of tripleo-common  https://review.openstack.org/29834414:20
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Optionally skip Keystone post-deployment initialization  https://review.openstack.org/25026814:23
jaosoriorbnemec: any preference for the port?14:27
bnemecjaosorior: 13386 would be consistent with our pattern elsewhere.14:28
bnemecI don't see that one in use yet either.14:29
beaglesEmilienM, just  a heads up on that problem I pinged you about on Friday - that was a concat related issue. I guess I'm running into it because I was going by the docs and not using the tripleo ci scripts.14:29
EmilienMbeagles: ok :)14:29
beaglesEmilienM, swift defines those file resources via concat but sets up the dependencies via File and voila - error :)14:29
bnemecNot using tripleo.sh -> that way lies madness.14:30
beagleslol14:30
beaglesbnemec, apparently this is true :)14:30
beaglesbnemec, unfortunately those using the u/s docs on the wiki will be inflicted with said madness14:30
* beagles shuts up before inadvertently volunteering to adopt said docs14:30
trowntoo late14:31
bnemecI hate it, but I gave up trying to make the docs sane a while back.14:31
bnemecThere are too many hacks/workarounds in tripleo.sh.14:31
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port  https://review.openstack.org/30395414:31
* beagles disappears for a sec cuz of "update monday"14:31
bnemecjaosorior: 13838?14:32
jaosoriorbnemec: crap14:32
jaosoriorwait up14:32
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port  https://review.openstack.org/30395414:34
jaosoriorbnemec: Done14:34
EmilienMjaosorior: +2 on https://review.openstack.org/#/c/302574/ -- excellent work here14:35
*** yamahata has joined #tripleo14:35
jaosoriorEmilienM: thanks man14:36
EmilienMjaosorior: we might need to merge is asap14:36
EmilienMso we avoid rebasing14:36
EmilienMdprince: can you look at it please?14:36
jaosoriorbnemec: Can you also look at this? https://review.openstack.org/#/c/302574/14:36
pradkEmilienM, if you can +a https://review.openstack.org/#/c/301798/ before that14:37
bnemecjaosorior: Yeah, I'll try to.14:37
pradkEmilienM, this is blocking gnocchi to merge https://review.openstack.org/#/c/252032/14:37
openstackgerritStig Telfer proposed openstack/tripleo-quickstart: Add libselinux-python to install_deps  https://review.openstack.org/30416714:39
*** greghaynes has quit IRC14:39
*** jaosorior is now known as jaosorior_away14:42
*** manfredp has joined #tripleo14:44
*** jtomasek has joined #tripleo14:46
pradkEmilienM, the ha and nonha already passed in zuul and upgrade finished deploy and finishing up.. if we can wait and get https://review.openstack.org/#/c/301798/ merged, all gnocchi dependencies will be done and tht patch should merge14:46
*** TSCHAK has quit IRC14:49
beaglesbnemec, sanity check on this concat thing - the bug is in the puppet modules, not puppet-concat, right?. In other words, we need to line up patches for the puppet modules to work with the new behavior - correct?14:52
*** greghaynes has joined #tripleo14:52
*** jprovazn has joined #tripleo14:52
weshaytrown, do you see any issue w/ adding an option to the tripleo-quickstart to build w/ the puppet modules from src?14:54
trownweshay: it is possible now14:54
jistrtrown: re https://review.openstack.org/#/c/299475 -- hard to tell there what could happen on upgrade unless manually tried, unfortunately i can't do that ATM14:55
weshayadarazs, ^14:55
*** oshvartz has quit IRC14:55
trownnot the build images part, but tripleo-quickstart can pass arbitrary environment variables to the undercloud install, and there is a deploy artifacts ability to use source puppet modules instead of the opm in the overcloud14:56
jistrgenerally the upgrade perspective should be always covered by the submission, it's not scalable to have just 1 or 2 folks checking changes manually if the don't break upgrades14:56
weshaytrown, cool thanks.. adarazs something to look into there14:56
trownweshay: adarazs the latter is not wired directly in yet, but the former works https://etherpad.openstack.org/p/trunk-undercloud-image14:57
trownweshay: adarazs, specifically lines 28-3114:57
EmilienMpradk: maybe we can ask jaosorior_away to rebase on top of yours14:59
*** jtomasek has quit IRC14:59
pradkEmilienM, that would be helpful.. we want to backport my patch to mitaka, so better to keep it separate14:59
EmilienMpradk: please go ahead15:00
EmilienMjaosorior_away seems... away15:00
EmilienMso we save time15:00
pradkjaosorior_away, would you be ok with that?15:01
pradkEmilienM, ok15:01
*** mcornea_ has joined #tripleo15:01
*** mcornea has quit IRC15:03
*** jtomasek has joined #tripleo15:04
adarazstrown, weshay: ack. so it looks like I should go to play with khaleesi + quickstart instead of implement anything around this myself :)15:04
weshayadarazs, ack.. khaleesi in that case just consumes both the undercloud and overcloud images from quickstart15:07
weshayif you need to build.. obviously the job will be slower.. which then begs the question if we want a sister promotion job that uses the upstream delorean hash and puppet from src15:07
pradkEmilienM, ci passed if you can +a i can rebase his change15:08
pradkEmilienM,  https://review.openstack.org/#/c/301798/15:08
michchapEmilienM: dprince is there a rationale for having 2 profiles for the base glance services (api, registry) but only for the pacemaker HA?15:08
EmilienMpradk: done15:09
pradkEmilienM, ty sir15:09
michchaponly one*15:09
EmilienMmichchap: not really15:09
michchapEmilienM: other than that, looks good15:09
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Nothing to see here  https://review.openstack.org/11101115:11
*** dtrainor has joined #tripleo15:13
*** dtrainor has quit IRC15:14
*** dtrainor has joined #tripleo15:15
*** yamahata has quit IRC15:15
*** openstackgerrit has quit IRC15:18
*** openstackgerrit has joined #tripleo15:18
openstackgerritMerged openstack/puppet-tripleo: Map gnocchi vip to haproxy_listen_bind_param  https://review.openstack.org/30179815:20
*** manfredp has quit IRC15:21
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints  https://review.openstack.org/30257415:23
pradkEmilienM, ^^ rebased15:23
EmilienMpradk: thanks15:26
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Move Validations to right sidebar  https://review.openstack.org/30412715:28
shardyjistr: Hey I have a question about environments/major-upgrade-pacemaker.yaml15:32
shardywe disable all *PostDeployment, so we don't run puppet at all on an upgrade, but leave nova pinned ready for the compute manual updates right?15:33
*** tremble has quit IRC15:34
shardyI am wondering at what point it's safe to do another stack update via overcloud deploy, e.g if you have ExtraConfig resources you want to apply to the upgraded nodes15:35
*** Guest8989 has quit IRC15:35
*** leanderthal is now known as leanderthal|afk15:35
shardyI'm assuming you cannot do that at all until all compute nodes have been upgraded, or you risk erasing the crudini and nova pin config we apply (outside of puppet)?15:36
*** zoli has quit IRC15:36
jistrshardy: yes, the pinning on controllers happens via a shell script after the cluster is brought down, before it's brought back up https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/tasks/major_upgrade_controller_pacemaker_1.sh#L4915:36
jistrshardy: on computes it is pinned before upgrading a particular compute https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/tasks/major_upgrade_compute.sh#L1715:37
*** sanjay__u has joined #tripleo15:37
EmilienMgfidente: can you -A https://review.openstack.org/#/c/252032/4 please ?15:38
EmilienMso we can retrigger Zuul15:38
jistrshardy: i think it should be safe provided that you again pass the `UpgradeLevelNovaCompute: liberty` among the parameters15:39
* jistr digs through code a bit15:39
EmilienMpradk: retrying with recheck but not sure it will work. Otherwise, rebase and review again...15:40
jistrshardy: when major-upgrade-pacemaker-converge.yaml is used, the Nova upgrade level is unpinned via Puppet https://github.com/openstack/tripleo-heat-templates/commit/02b3025585db9134ea2c01e4a644c602a1c4886215:41
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/tasks/major_upgrade_controller_pacemaker_1.sh#L5515:41
shardyjistr: I'm looking at stuff like the crudini stuff there15:41
jistrshardy: but you can use the same mechanism to pin it, or keep it pinned, just depends on value of `UpgradeLevelNovaCompute`15:41
shardywill that be overrwritten by puppet?15:41
jistrshardy: yeah that could be. I don't know off the top of my head, but generally puppet could rewrite those. Are you thinking of using the pre-upgrade t-h-t or post-upgrade t-h-t to apply the extraconfigs?15:43
*** chem has quit IRC15:44
*** dtantsur|mtg is now known as dtantsur|afk15:44
shardyjistr: I'm just thinking about site-specific changes wired in e.g via OS::TripleO::NodeExtraConfigPost15:44
shardyor, really any kind of update15:44
shardyat some point post-upgrade, you'll want to update the stack again, which will re-run puppet15:44
shardyideally we'd have all those hooks applied during the upgrade so external integrators could do something to upgrade their stuff, but we probably need a better defined interface for that15:45
*** mgould has quit IRC15:45
shardythe current status appears to be any ExtraConfigPost stuff won't do anything on upgrade, and a workaround could be doing an upgrade, then an update15:46
*** mkovacik has quit IRC15:46
*** chem has joined #tripleo15:46
pradkEmilienM, cool its in queue now15:46
EmilienMpradk: in check.. let's see if it goes in gate afterward15:47
pradkcool15:47
jistrshardy: one thing worth mentioning -- in general running puppet out of "the usual schedule" during upgrade isn't safe. E.g. one of the things that a converge Puppet run can do is that it can change the rabbitmq password. If that happens only on a subset of nodes, and some no longer have a correct password in cfg files, it can be an issue. And i'm not sure if running puppet on all nodes while only some of them have been upgraded would work15:47
jistreither.15:47
*** jaosorior_away has quit IRC15:47
jistrshardy: ok so you want to run the stack-update *after* the upgrade has been fully completed?15:47
* jistr not sure i followed the gist fully, re-reading15:47
*** jaosorior_away has joined #tripleo15:48
*** ifarkas has quit IRC15:49
dprincemichchap: re. the single glance pacemaker profile. It was because the Glance pacemaker stuff is so entangled I didn't see a good way to split it apart15:49
michchapdprince: fair enough15:49
shardyjistr: I'm trying to figure out two things:15:49
jistrshardy: the last part of upgrade (the "converge" step) is basically a normal stack-update, so anything that runs during a stack-update should run there15:50
jistrdoing a puppet run sooner than that isn't safe15:50
jistr(reasons wrote above)15:50
shardy1 - What of our Documented ExtraConfig interfaces actually do anything during upgrade, I think the answer is only the pre-deploy ones that aren't in *post.yaml15:50
shardy2 - At what point can puppet safely be run after an upgrade, and I think you answered that (after the converge environment is passed?)15:51
shardyjistr: where do we reset the resource_registry mappinsg from *PostDeployment to actually run puppet again?15:52
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/environments/major-upgrade-pacemaker-init.yaml#L915:52
jistrre 1) it's needed to distinguish between *controller upgrade* (just step N in the upgrade process) vs. *full upgrade* (all steps). During controller upgrade everything in -post.yaml is no-opped15:52
shardythat stubs out all the puppet completely15:52
jistrshardy: here https://github.com/openstack/tripleo-heat-templates/blob/02b3025585db9134ea2c01e4a644c602a1c48862/environments/major-upgrade-pacemaker-converge.yaml15:52
jistrshardy: ^^ that is step 6 (line 24) in https://etherpad.openstack.org/p/tripleo-migrations15:53
shardyjistr: don't we use a PATCH update tho, so those mappings to OS::Heat::None are sticky?15:53
shardye.g after an upgrade, puppet will never run ever again?15:53
jistrshardy: i think it worked ok for everyone running it, but i think everyone may have been passing the resource-registry-puppet.yaml explicitly15:54
shardyOr do we document passing overcloud-resource-registry-puppet.yaml explicitly along with the converge environment?15:54
shardyah, yeah that would explain it15:54
shardyI'm just trying to clarify the expected operator flow15:55
shardythanks for bearing with me :)15:55
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Deploy Gnocchi as a Ceilometer metrics storage backend  https://review.openstack.org/25203215:56
jistrre question 2) yeah, when major-upgrade-pacemaker-converge.yaml is passed, puppet should re-run and everything else, just like a normal stack-update15:57
*** dprince has quit IRC15:57
shardyjistr: ack, thanks for all the info15:58
*** dprince has joined #tripleo15:58
*** TSCHAK has joined #tripleo15:58
*** mikelk has quit IRC15:58
sshnaidmI'm not sure whom to report to: ERROR: Connection was broken: java.io.IOException: Unexpected termination of the channel  http://logs.openstack.org/62/244162/30/check-tripleo/gate-tripleo-ci-f22-nonha/3788178/console.html15:59
sshnaidmdprince, ^^15:59
EmilienMsshnaidm: it sounds like an openstack-infra thing, might need to recheck16:00
sshnaidmok16:00
jistrshardy: hehe i was just digging up info re ExtraConfig not reapplying correctly and i saw your reply here https://bugzilla.redhat.com/show_bug.cgi?id=1312007#c2416:00
openstackbugzilla.redhat.com bug 1312007 in openstack-tripleo-heat-templates "When updating overcloud, OS::TripleO::NodeExtraConfigPost: isn't reapplied" [Medium,On_qa] - Assigned to sbaker16:00
jistrso i see you got that covered16:00
dprincesshnaidm: yeah, sounds like a Jenkins slave issue16:01
shardyjistr: Yeah I posted https://review.openstack.org/#/c/296488/ which shows how to wire in DeployIdentifier16:02
shardyreviews welcome ;)16:02
*** jaosorior_away has quit IRC16:04
*** jaosorior has joined #tripleo16:05
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy  https://review.openstack.org/30334216:06
*** openstackstatus has joined #tripleo16:12
*** ChanServ sets mode: +v openstackstatus16:12
*** apetrich has quit IRC16:13
*** mbound has quit IRC16:13
*** jcoufal has quit IRC16:14
*** mbound has joined #tripleo16:14
*** julim has quit IRC16:14
*** david-lyle_ is now known as david-lyle16:15
*** rwsu has joined #tripleo16:16
*** mbound has quit IRC16:18
jaosoriorpdradk: I saw you rebased my patch, thanks16:18
-openstackstatus- NOTICE: Reminder, Gerrit will be offline from 20:00 to 21:00 UTC for a server replacement http://lists.openstack.org/pipermail/openstack-dev/2016-April/091274.html16:20
jaosoriorbnemec: Hey dude, could you check this one out? It's for having internal TLS https://review.openstack.org/#/c/303342/ it was all green no the CI, but had to rebase it16:20
pradkcan anyone shed light on why this is not getting into the gate https://review.openstack.org/#/c/252032/16:21
pradkEmilienM and i tried different combinations16:21
pradki even rebased but still sits there16:21
pradki have a dependent patch in master thats merged, but i cherry picked to mitaka16:22
pradkwhich is abandoned, does it know the difference between branches?16:22
bnemecpradk: I think that's your problem.16:22
shardypradk: Depends-On is not branch aware16:22
shardybut abandoning the backport should be enough I think?16:23
slagleit's not16:23
pradkthats what i thought16:23
slaglewill have to remove the Depends-On16:23
bnemecYou'll need to remove the depends-on for that patch.16:23
shardygah :(16:23
pradkouch hmm.. or get the mitaka one merged i guess16:23
openstackgerritDmitry Ilyin proposed openstack/puppet-pacemaker: Merge with fuel-infra/puppet-pacemaker  https://review.openstack.org/29644016:27
openstackgerritDmitry Ilyin proposed openstack/puppet-pacemaker: Merge with fuel-infra/puppet-pacemaker  https://review.openstack.org/29644016:27
*** yamahata has joined #tripleo16:27
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Deploy Gnocchi as a Ceilometer metrics storage backend  https://review.openstack.org/25203216:28
openstackgerritJames Slagle proposed openstack/os-net-config: Add support for OVS tunnels  https://review.openstack.org/30421516:28
pradkslagle, bnemec, can we fast approve ^^ .. i just removed the depends on16:28
pradkci passed green in previous runs16:29
pradkjaosorior, ty sir!16:32
pradkfingers crossed it get in the gate this time16:32
*** jaosorior has quit IRC16:33
*** apetrich has joined #tripleo16:33
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Add Glance profiles  https://review.openstack.org/29607616:34
openstackgerritJames Slagle proposed openstack/os-net-config: Fix typos  https://review.openstack.org/30421916:34
*** mcornea_ has quit IRC16:37
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: deploy glance using puppet-tripleo manifests  https://review.openstack.org/28946616:39
EmilienMdprince: rebased ^16:39
*** rcernin has quit IRC16:40
snecklifterI don't believe it16:41
sneckliftermy patch passed Jenkins check16:41
rhalliseydprince, are you seeing neutron work for you locally?16:41
*** Marga_ has joined #tripleo16:42
openstackgerritJames Slagle proposed openstack/os-net-config: Add support for OVS tunnels  https://review.openstack.org/30421516:42
rhalliseyEmilienM, what's the difference between 'include ::tripleo::profile ...' and 'include tripleo::profile ...'16:43
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Add missing ManagementIpSubnet  https://review.openstack.org/30126616:44
EmilienMrhallisey: puppet4 syntax16:44
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Add net-config-static.yaml  https://review.openstack.org/30126716:44
EmilienMboth work, but puppet4 new syntax it is16:44
openstackgerritMerged openstack/tripleo-heat-templates: Remove hack the pulls latest docker  https://review.openstack.org/30014716:44
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates: Use already Deployed/Installed servers  https://review.openstack.org/22277216:44
*** jaosorior has joined #tripleo16:44
rhalliseyEmilienM, which one is puppet4?16:44
*** dtrainor has quit IRC16:45
*** aufi has quit IRC16:45
openstackgerritRyan Hallisey proposed openstack-infra/tripleo-ci: Allow the container job to run again  https://review.openstack.org/28891516:45
EmilienMrhallisey: with ::16:46
rhalliseyEmilienM, cool thx16:46
*** dtrainor has joined #tripleo16:46
rhalliseydprince, ^ Dan you're using a mix of puppet4 and its precursor which I'm assuming is pupept316:46
rhalliseydprince, the neutron stuff is puppet316:47
*** rwsu has quit IRC16:47
dprincerhallisey: all of the composable roles stuff should pass puppet 3 I think16:50
*** aparnav has joined #tripleo16:50
openstackgerritMerged openstack-infra/tripleo-ci: Temporarily disable mistral on the undercloud  https://review.openstack.org/30283216:51
rhalliseydprince, I'm just mentioning because glance + keystone are using 4 and neutron is using 316:51
rhalliseyshould they all just use puppet4?16:51
*** Marga_ has quit IRC16:51
dprincerhallisey: where are you seeing versions?16:51
openstackgerritMerged openstack/os-net-config: Use interface operstate to determine nic status  https://review.openstack.org/29124316:51
*** Marga_ has joined #tripleo16:52
rhalliseydprince, https://review.openstack.org/#/c/303386/616:53
*** Marga_ has quit IRC16:53
*** rwsu has joined #tripleo16:53
*** jerrygb has quit IRC16:53
*** paramite has quit IRC16:53
*** jerrygb has joined #tripleo16:54
*** dtantsur|afk is now known as dtantsur16:55
rhalliseydprince, oh btw I like the swift artifact stuff16:56
*** derekh has quit IRC16:57
*** dprince has quit IRC16:57
*** dprince has joined #tripleo16:58
*** julim has joined #tripleo16:59
*** Marga_ has joined #tripleo17:00
*** Marga_ has quit IRC17:00
*** Marga_ has joined #tripleo17:01
openstackgerritMerged openstack/tripleo-heat-templates: Deploy Gnocchi as a Ceilometer metrics storage backend  https://review.openstack.org/25203217:04
openstackgerritBen Nemec proposed openstack/instack-undercloud: Generate most of the pystache context automatically  https://review.openstack.org/29731917:04
*** ccamacho has quit IRC17:04
*** ccamacho has joined #tripleo17:04
trowndprince: can you re-+1 https://review.openstack.org/#/c/304145/ had to make a minor tweak17:07
*** dsneddon has joined #tripleo17:09
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests  https://review.openstack.org/24416217:09
*** appu103_ has joined #tripleo17:12
bnemecSimple CI patch in need of a +2A: https://review.openstack.org/#/c/30299917:12
*** penick has joined #tripleo17:13
*** aparnav has quit IRC17:14
*** appu103_ has quit IRC17:20
*** Marga_ has quit IRC17:21
*** Marga_ has joined #tripleo17:23
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Metrics tracking for TripleO deployment tasks  https://review.openstack.org/29139317:24
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Add common bash functions to help track metrics.  https://review.openstack.org/29139217:24
*** jpena is now known as jpena|off17:27
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Nova needs the proper volumes to use Cinder  https://review.openstack.org/30159217:27
*** trown is now known as trown|lunch17:28
*** Marga_ has quit IRC17:28
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Centos gives /dev/pts/ptmx the wrong perms  https://review.openstack.org/29987717:28
pradkisnt there a stable/mitaka branch for puppet-tripleo?17:31
*** jaosorior has quit IRC17:31
shardypradk: There should be but looks like we missed it17:33
pradkyea17:34
shardywe'll need to figure out the best hash to branch from and cut it now17:34
dprinceshardy: it kind of makes me sad to see us still landing this now: https://review.openstack.org/#/c/252032/17:36
dprinceshardy: kind of like piling on technical debt even though we have enough of the new architecture to use it17:36
shardy6c25f57d78d47419365e2238647228a186a0137f appears to be used in the most recent periodic job pass17:36
dprinceshardy: I can go around and -2 things I guess, but it would be better if the core team agreed on what architecture are going for...17:37
dprinceshardy: because continuing to land things in the old format will likely put us in a never finished with composability stance17:37
shardydprince: that's been agreed as a backport exception, so we need to land it in the old architecture17:37
shardyI agree for anything new we should use the new composable services17:37
dprinceshardy: sure, I guess this is a fine backport exception17:38
dprinceshardy: but no more, please !17:38
pradkdprince, i'm happy to help out with converting telemetry services into composable services17:38
shardydprince: completely agreed :)17:38
shardydprince: FWIW I asked for feedback on the ML, and IMO the main reason this didn't land before we branched was lack of reviews and CI17:39
dprincepradk: cool, let me know if you have questions on the approach17:39
dprincepradk: the keystone service just landed and would be a good example17:39
pradkdprince, cool will do, I'll start with ceilometer and then extend to aodh and gnocchi17:40
shardytrown|lunch, slagle: are you happy with 6c25f57d78d47419365e2238647228a186a0137f as a branch point for stable/mitaka puppet-tripleo?17:40
dprincepradk: cool, There may be some stepping on toes. Because controller.yaml will get touched a lot by me17:40
dprincepradk: I'm trying to untangle the neutron code right now. It is a rats nest of partner integration, etc.17:41
pradksure understand17:41
*** jerrygb has quit IRC17:41
dprincepradk: partner integration stuff which is good to have from a features standpoint, but that our v1 architecture couldn't handle properly17:41
shardyI'm using http://logs.openstack.org/periodic/periodic-tripleo-ci-f22-ha-mitaka/f8eebaa/console.html as the reference to that commit17:42
dprincepradk: But ++ on the ceilometer changes ,etc I'd be happy to have you jump in and do those17:42
dprincepradk: https://etherpad.openstack.org/p/tripleo-composable-services17:42
pradkdprince, cool i'm happy to help .. is there a place you're tracking the composable service work, so i can note my name down17:42
dprincepradk: list any work you do there if/when you proceed17:42
pradkawesome thx17:42
openstackgerritPradeep Kilambi proposed openstack/python-tripleoclient: Add gnocchi password as a deployment param  https://review.openstack.org/30424017:43
bnemecerror: RPC failed; result=6, HTTP code = 017:48
bnemecfatal: The remote end hung up unexpectedly17:48
bnemecRuh roh17:48
bnemecSeeing a bunch of random 20 minute failures in CI jobs again.17:48
*** shardy has quit IRC17:49
*** jistr has quit IRC17:51
rhalliseydprince, are you in the process of rebasing glance? Or should I do it?17:52
rhalliseyI can't push my patch until it gets rebased sadly :(17:52
dprincerhallisey: I'm rebasing17:53
rhalliseydprince, ok thanks17:53
dprincerhallisey: was just checking to see if the latest CI failure indicated anything else WTR to the glance_api_servers issue I saw yesterday17:53
*** shardy has joined #tripleo17:53
*** penick has quit IRC17:58
openstackgerritRyan Hallisey proposed openstack-infra/tripleo-ci: Allow the container job to run again  https://review.openstack.org/28891518:00
*** Marga_ has joined #tripleo18:00
*** penick has joined #tripleo18:01
*** sambetts is now known as sambetts|afk18:01
*** dtantsur is now known as dtantsur|afk18:01
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron dhcp service  https://review.openstack.org/30338618:02
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron metadata service  https://review.openstack.org/30361818:02
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable neutron l3 service  https://review.openstack.org/30356218:02
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: composable glance services  https://review.openstack.org/23737018:02
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add GlanceRegistry to the endpoint map  https://review.openstack.org/30372818:02
dprincerhallisey: ^^18:02
rhalliseysweet18:05
*** ccamacho has quit IRC18:06
*** shivrao has joined #tripleo18:07
*** rwsu_ has joined #tripleo18:09
*** dtrainor has quit IRC18:11
*** rwsu has quit IRC18:12
*** trown|lunch is now known as trown18:14
*** tiswanso has quit IRC18:14
trownshardy: yep, that is what is in the stable/mitaka openstack-puppet-modules18:14
*** tiswanso has joined #tripleo18:15
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Always use parameter_defaults in environment files  https://review.openstack.org/30330518:15
shardytrown: thanks, branch created18:20
shardypradk: thanks for spotting that!18:20
pradkcool18:20
*** shardy is now known as shardy_afk18:20
*** ccamacho has joined #tripleo18:23
*** apetrich has quit IRC18:24
openstackgerritOpenStack Proposal Bot proposed openstack/diskimage-builder: Updated from global requirements  https://review.openstack.org/30341618:26
*** tiswanso has quit IRC18:28
*** tiswanso has joined #tripleo18:28
openstackgerritOpenStack Proposal Bot proposed openstack/os-cloud-config: Updated from global requirements  https://review.openstack.org/28504918:31
openstackgerritOpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements  https://review.openstack.org/26852818:32
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements  https://review.openstack.org/30062918:33
*** apetrich has joined #tripleo18:36
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: composable neutron dhcp service  https://review.openstack.org/30338618:36
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: composable neutron metadata service  https://review.openstack.org/30361818:36
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: composable neutron l3 service  https://review.openstack.org/30356218:36
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: composable glance services  https://review.openstack.org/23737018:36
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Composable Keystone Containers  https://review.openstack.org/30428218:36
*** rwsu_ has quit IRC18:38
*** julim has quit IRC18:42
*** penick has quit IRC18:42
*** qasims has quit IRC18:43
*** penick has joined #tripleo18:44
openstackgerritmathieu bultel proposed openstack/tripleo-quickstart: Add ignore error on virsh destroy  https://review.openstack.org/30428618:51
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Map gnocchi vip to haproxy_listen_bind_param  https://review.openstack.org/30428718:51
*** apetrich has quit IRC18:52
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Map gnocchi vip to haproxy_listen_bind_param  https://review.openstack.org/30428718:54
*** tiswanso has quit IRC18:56
*** tiswanso has joined #tripleo18:58
*** apetrich has joined #tripleo18:58
*** julim has joined #tripleo19:01
*** rwsu has joined #tripleo19:02
slaglehmm, CI choking cloning from github, but github claims 100% operational19:05
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Deploy Gnocchi as a Ceilometer metrics storage backend  https://review.openstack.org/30429519:05
rhalliseydprince, line 233 https://review.openstack.org/#/c/304282/1/docker/controller-post.yaml19:10
pradkat8code19:10
* slagle writes down pradk's root pw19:10
rhalliseydprince, I simulate puppet here.  The only issue is I can' really pass a string the way I'm doing it19:11
pradklol and you couldnt let that slide unnoticed19:11
rhalliseydprince, is there a way to pass a config or something19:12
dprincerhallisey: you can pass in json right?19:12
dprincerhallisey: I think we can pass whatever you'd like19:13
rhalliseydprince, ok I'll try a json19:14
dprincerhallisey: a lot of this stuff should roll up from the heat resource chain19:14
dprincerhallisey: like instead of handling the chain at the top level overcloud.yaml we could just pass the list of services into this template19:15
rhalliseydprince, I think I'm doing it right, it's just the format19:15
dprincerhallisey: then create the resource chain there and you could then pull out docker specific things from each nested stack19:15
dprincerhallisey: like each nested service stack (keystone for example) should have it's own tags19:15
rhalliseydprince, I use your resource chain in this patch19:15
dprincerhallisey: oh, and one big different with docker vs. the baremetal config is for docker we'll generate each service config individually I think19:16
dprincerhallisey: or we could19:16
dprincerhallisey: I know we haven't been doing that, but once the roles are split up we can19:16
dprincerhallisey: so 'puppet apply' for keystone, then 'puppet apply' for neutron-ovs19:16
dprincerhallisey: since we are only using puppet for configs it would run pretty quickly and by isolating it we'd have a cleaner config trees I think19:17
*** olap has quit IRC19:17
dprincerhallisey: that is an idea anyway19:17
rhalliseydprince, ok so how would you get the list of services19:17
rhalliseyenv file?19:18
dprincerhallisey: ControllerServices is a top level parameter right now19:18
dprincerhallisey: http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/overcloud.yaml#n70019:18
rhalliseydprince, so would a user have a list in the environment file to overwrite the default?19:19
dprincerhallisey: sure19:19
bnemecslagle: Looks like we're hitting this again: https://bugs.launchpad.net/tripleo/+bug/156758019:19
openstackLaunchpad bug 1567580 in tripleo "Network problems in CI environment" [Critical,Triaged]19:19
rhalliseydprince, ok then we both can plug into that interface19:19
dprincerhallisey: see what I did in puppet-pacemaker19:19
trownbnemec: any idea why CI didn't ever run on https://review.openstack.org/#/c/291808/19:20
dprincerhallisey: if the default eventually becomes containers we could even modify the default in the base resource registry19:20
dprincerhallisey: for now having an overlay mechanism makes sense19:21
openstackgerritIan Pilcher proposed openstack/tripleo-heat-templates: Document IPv6 syntax for GlanceFilePcmkDevice  https://review.openstack.org/30429819:21
rhalliseydprince, ok19:21
bnemectrown: That was a long time ago, but probably a zuul restart or something like that.19:21
slaglebnemec: yea, i did see that error19:21
slaglean also a dns error19:21
bnemecslagle: Yeah, and the dns makes me think it isn't github at fault here.19:22
bnemecAlthough it's terrible that we depend on github as much as we do.19:22
bnemecslagle: On a related note, it would be nice to merge https://review.openstack.org/#/c/302999/ so we can logstash for these kind of errors.19:23
slagleyea apparently all you need is a github account and a repo called puppet-something, and you too can become a hard dependency for tripleo19:23
bnemeclol19:23
trownbnemec:  is there a reason we tar the undercloud logs in the first place?19:25
openstackgerritMerged openstack-infra/tripleo-ci: Dump end of logs to console.log on failure  https://review.openstack.org/30299919:25
bnemectrown: Because otherwise console.log gets absurdly long, and _usually_ failures aren't in the undercloud install or image build.19:26
bnemecEven in this case it's an infrastructure problem, not an actual error in the patch.19:26
trownbnemec: right, I get why we pipe the output to a separate log, just not why the logs are not browsable19:26
bnemectrown: Oh, that I don't know.  They used to be, probably back in the incubator days.19:27
*** sanjay__u has quit IRC19:29
*** akrivoka has quit IRC19:34
trowneasy review https://review.openstack.org/#/c/304219 I think we should ignore CI for that19:35
*** jprovazn has quit IRC19:36
slaglei'd like to recheck that 10 times19:36
trownlol19:36
gfidentetrown -1 !19:38
gfidenteahauhauha19:38
trownnot my patch :)19:38
gfidente:(19:38
gfidenteslagle I actually learned about the new release process19:39
slagleah, you -1'd me b/c you want me to fix more typos???19:39
gfidentewhere people doesn't need to be in the release group to make releases19:39
gfidenteI've updated the wiki19:39
gfidentebasically we should submit gerrit changes against the openstack/release repo19:39
trowngfidente: don't we have to change our tags in the governance repo19:42
trowngfidente: currently all the tripleo projects are release:independent19:43
trownhttps://github.com/openstack/governance/blob/master/reference/projects.yaml#L4143-L425119:44
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Move Validations to right sidebar  https://review.openstack.org/30412719:44
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Deployment Plan page updates  https://review.openstack.org/30395819:44
gfidentetrown there is an _independent directory in there19:44
gfidentehttps://review.openstack.org/#/c/303986/19:44
trownoh cool19:44
-openstackstatus- NOTICE: Gerrit will be offline from 20:00 to 21:00 UTC (starting 10 minutes from now) for a server replacement http://lists.openstack.org/pipermail/openstack-dev/2016-April/091274.html19:51
*** trown is now known as trown|outtypewww20:01
*** openstackgerrit has quit IRC20:05
*** florianf has quit IRC20:08
-openstackstatus- NOTICE: Gerrit is offline until 21:00 UTC for a server replacement http://lists.openstack.org/pipermail/openstack-dev/2016-April/091274.html20:08
*** ChanServ changes topic to "Gerrit is offline until 21:00 UTC for a server replacement http://lists.openstack.org/pipermail/openstack-dev/2016-April/091274.html"20:08
*** jayg is now known as jayg|g0n320:15
*** rcernin has joined #tripleo20:16
*** yamahata has quit IRC20:17
*** shardy_afk has quit IRC20:18
*** MaxPC has quit IRC20:20
*** rcernin has quit IRC20:21
*** egafford has quit IRC20:26
*** openstackgerrit has joined #tripleo20:27
*** julim has quit IRC20:28
*** dprince has quit IRC20:36
*** openstackgerrit has quit IRC20:41
*** openstackgerrit has joined #tripleo20:55
*** rhallisey has quit IRC20:58
*** lblanchard has quit IRC21:02
*** weshay has quit IRC21:03
openstackgerritJeff Peeler proposed openstack/python-tripleoclient: Add ability to push docker images to registry  https://review.openstack.org/25553521:05
*** ChanServ changes topic to "TripleO | CI status: http://tripleo.org/cistatus.html | Docs: http://tripleo.org/"21:08
-openstackstatus- NOTICE: Gerrit move maintenance completed successfully; note that DNS has been updated to new IP addresses as indicated in http://lists.openstack.org/pipermail/openstack-dev/2016-April/091274.html21:08
openstackgerritDmitry Ilyin proposed openstack/puppet-pacemaker: Merge with fuel-infra/puppet-pacemaker  https://review.openstack.org/29644021:09
openstackgerritDmitry Ilyin proposed openstack/puppet-pacemaker: Merge with fuel-infra/puppet-pacemaker  https://review.openstack.org/29644021:10
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Nothing to see here  https://review.openstack.org/11101121:26
openstackgerritMerged openstack/puppet-tripleo: Fix Sahara SSL default port  https://review.openstack.org/30395421:28
*** pradk has quit IRC21:40
*** tiswanso has quit IRC21:49
*** dustins has quit IRC21:50
*** lblanchard has joined #tripleo22:11
*** gfidente has quit IRC22:27
*** penick has quit IRC22:34
*** dtrainor has joined #tripleo23:12
*** rhallisey has joined #tripleo23:18
*** Ryjedo has quit IRC23:20
*** TSCHAK has quit IRC23:23
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Nothing to see here  https://review.openstack.org/11101123:24
*** yamahata has joined #tripleo23:24
*** lblanchard has quit IRC23:28
*** panda has quit IRC23:41
*** panda has joined #tripleo23:42
*** akshai has quit IRC23:49

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