Wednesday, 2015-12-09

*** jerrygb has quit IRC00:01
*** bvandenh has quit IRC00:02
*** panda has quit IRC00:03
*** panda has joined #tripleo00:03
*** yuanying has quit IRC00:04
*** yuanying has joined #tripleo00:05
*** eil397 has quit IRC00:13
*** psanchez has quit IRC00:21
*** eil397 has joined #tripleo00:23
*** psanchez has joined #tripleo00:23
*** weshay has quit IRC00:27
*** chlong has joined #tripleo00:27
*** yuanying has quit IRC00:32
*** yuanying has joined #tripleo00:34
*** dmacpher-afk has quit IRC00:38
*** mbound_ has quit IRC00:41
*** yuanying has quit IRC00:44
*** shivrao has quit IRC00:45
*** shivrao has joined #tripleo00:46
*** yuanying has joined #tripleo00:47
*** openstackstatus has quit IRC00:50
*** openstack has joined #tripleo00:51
*** openstackstatus has joined #tripleo00:53
*** ChanServ sets mode: +v openstackstatus00:53
*** rebrego has joined #tripleo01:04
*** regebro has quit IRC01:04
*** rebrego is now known as regebro01:04
openstackgerritMerged openstack/diskimage-builder: Fix grub-efi-amd64-signed install failure  https://review.openstack.org/21961201:07
*** dmacpher has joined #tripleo01:18
*** shivrao has quit IRC01:19
*** rhallisey has joined #tripleo01:30
openstackgerritMerged openstack/diskimage-builder: Load the 8021q kernel module in simple-init  https://review.openstack.org/25001201:31
*** NobodyCam has joined #tripleo01:49
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140801:55
*** chlong has quit IRC02:03
*** alop has quit IRC02:15
*** eil397 has quit IRC02:17
*** sthillma has quit IRC02:20
rhalliseyEmilienM, how come you didn't come to westford this week?02:30
EmilienMrhallisey: nobody invivted me02:30
rhalliseyoh02:30
rhalliseymaybe it's just release & CI folks02:31
EmilienMrhallisey: if you want, I can plan a trip for early next year02:31
EmilienMand we hack together02:31
EmilienMI live in Quebec, it's not that far02:31
rhalliseythat'd be cool02:31
rhalliseyis there an office there?02:32
EmilienMrhallisey: yes, my home :)02:32
rhalliseyhehe02:32
EmilienMrhallisey: I'm off for tnight02:32
rhalliseyEmilienM, cool man goodnight!02:33
EmilienMrhallisey: ttyl02:33
* rhallisey is not giving up on network iso patch just yet!02:33
*** sthillma has joined #tripleo02:35
*** sthillma has quit IRC02:51
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140802:53
*** jerrygb has joined #tripleo02:57
*** yamahata has quit IRC03:01
*** greghayn1 has joined #tripleo03:02
*** greghayn1 has quit IRC03:02
*** jerrygb has quit IRC03:02
*** thrash is now known as thrash|g0ne03:03
*** chlong has joined #tripleo03:15
*** morazi has quit IRC03:17
*** jerrygb has joined #tripleo03:18
*** masco has joined #tripleo03:23
*** yuanying has quit IRC03:23
*** links has joined #tripleo03:24
*** rhallisey has quit IRC03:46
*** jerrygb has quit IRC03:48
*** jerrygb has joined #tripleo03:48
*** chlong has quit IRC03:49
*** tzumainn has quit IRC03:52
*** jerrygb has quit IRC03:52
*** chlong has joined #tripleo03:53
*** untriaged-bot has joined #tripleo04:01
untriaged-botUntriaged bugs so far:04:01
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/152012004:01
openstackLaunchpad bug 1520120 in tripleo "Could not find /dev/mapper/loop0p1 error in CI" [Undecided,New]04:01
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/152402804:01
openstackLaunchpad bug 1524028 in tripleo "Ceph CI error: Can't connect to any member of replicaset tripleo" [Undecided,New]04:01
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/152405204:01
openstackLaunchpad bug 1524052 in tripleo "Configuration of neutron QoS feature missing" [Undecided,In progress] - Assigned to Brent Eagles (beagles)04:01
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/151781404:01
openstackLaunchpad bug 1517814 in diskimage-builder "ironic-python-agent service failed to be started automatically" [Undecided,Incomplete] - Assigned to Ligong Duan (duanlg)04:01
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/150590004:01
openstackLaunchpad bug 1505900 in diskimage-builder "Failed to create 'wily' release with DIB_RELEASE" [Undecided,New]04:01
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/151920904:01
openstackLaunchpad bug 1519209 in tripleo " disk-image-create fail with opensuse element" [Undecided,In progress] - Assigned to Baodong (Robert) Li (baoli)04:01
*** untriaged-bot has quit IRC04:01
*** links has quit IRC04:03
*** panda has quit IRC04:03
*** jerrygb has joined #tripleo04:03
*** panda has joined #tripleo04:04
*** rlandy has quit IRC04:17
*** yuanying has joined #tripleo04:28
*** shivrao has joined #tripleo04:37
*** links has joined #tripleo04:37
*** masco has quit IRC04:40
*** yuanying has quit IRC04:46
*** jerrygb has quit IRC04:49
*** jerrygb has joined #tripleo04:51
*** jerrygb has quit IRC04:51
*** jerrygb has joined #tripleo04:51
*** sthillma has joined #tripleo04:53
*** sthillma_ has joined #tripleo04:55
*** sthillma has quit IRC04:58
*** sthillma_ is now known as sthillma04:58
*** yuanying has joined #tripleo05:00
*** Marga_ has quit IRC05:28
*** shivrao has quit IRC05:45
*** sthillma has quit IRC05:56
*** Marga_ has joined #tripleo06:12
*** Marga_ has quit IRC06:14
*** Marga_ has joined #tripleo06:14
*** Marga_ has quit IRC06:15
*** Marga_ has joined #tripleo06:15
*** jerrygb has quit IRC06:17
*** jerrygb has joined #tripleo06:17
*** jerrygb has quit IRC06:21
*** chlong has quit IRC06:40
*** masco has joined #tripleo06:47
*** aufi has joined #tripleo06:59
*** jerrygb has joined #tripleo07:00
*** jtomasek has joined #tripleo07:03
*** jtomasek has quit IRC07:06
*** jerrygb has quit IRC07:06
*** shardy has joined #tripleo07:06
*** ukalifon1 has joined #tripleo07:06
*** ukalifon1 has quit IRC07:06
*** rpothier has joined #tripleo07:13
*** ukalifon has joined #tripleo07:17
*** jaosorior has joined #tripleo07:19
*** jaosorior has quit IRC07:19
*** jaosorior has joined #tripleo07:20
*** rcernin has joined #tripleo07:20
*** jtomasek has joined #tripleo07:21
openstackgerritJuan Antonio Osorio Robles proposed openstack/instack-undercloud: Support relative certificate paths  https://review.openstack.org/22188607:24
openstackgerritJuan Antonio Osorio Robles proposed openstack/instack-undercloud: stackrc improvements for SSL undercloud  https://review.openstack.org/23863907:24
*** devvesa has joined #tripleo07:32
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Remove all 'validate_array' statements  https://review.openstack.org/25076307:35
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Inject TLS certificate and keys for the Overcloud  https://review.openstack.org/25483107:38
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Enable trust anchor injection  https://review.openstack.org/25483207:38
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Output the SSL Certificate and Key modulus  https://review.openstack.org/25483307:38
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Minor fixes to TLS related resources  https://review.openstack.org/25483407:38
*** rasca has joined #tripleo07:45
*** tremble has joined #tripleo07:59
*** tremble has joined #tripleo07:59
*** panda has quit IRC08:03
*** panda has joined #tripleo08:04
*** jprovazn has joined #tripleo08:05
*** liverpooler has joined #tripleo08:15
*** tosky has joined #tripleo08:25
*** paramite has joined #tripleo08:36
*** ifarkas has joined #tripleo08:37
*** dtantsur|afk is now known as dtantsur08:39
*** olap has joined #tripleo08:41
*** shardy has quit IRC08:42
*** shardy has joined #tripleo08:43
*** dmacpher has quit IRC08:47
*** regebro has quit IRC08:48
*** shardy has quit IRC08:49
*** regebro has joined #tripleo08:49
*** shardy has joined #tripleo08:51
*** gfidente has joined #tripleo08:52
*** gfidente has quit IRC08:52
*** gfidente has joined #tripleo08:52
*** jaosorior has quit IRC08:55
*** tosky_ has joined #tripleo08:58
*** tosky has quit IRC08:59
*** nico_auv has joined #tripleo09:01
*** yamahata has joined #tripleo09:01
*** jerrygb has joined #tripleo09:02
*** openstackstatus has quit IRC09:05
*** openstack has joined #tripleo09:06
*** openstackstatus has joined #tripleo09:08
*** ChanServ sets mode: +v openstackstatus09:08
*** jerrygb has quit IRC09:08
*** athomas has joined #tripleo09:12
*** yamahata has quit IRC09:15
*** bvandenh has joined #tripleo09:16
*** jaosorior has joined #tripleo09:16
*** akrivoka has joined #tripleo09:25
*** olap has quit IRC09:26
*** olap has joined #tripleo09:27
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Inject TLS certificate and keys for the Overcloud  https://review.openstack.org/25483109:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Enable trust anchor injection  https://review.openstack.org/25483209:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Output the SSL Certificate and Key modulus  https://review.openstack.org/25483309:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Minor fixes to TLS related resources  https://review.openstack.org/25483409:29
jaosoriorshardy: I re-did the backports with cherry-pick -x ^^09:30
jaosoriorlearned a valuable lesson about backports here09:30
*** openstackgerrit has quit IRC09:32
*** openstackgerrit has joined #tripleo09:32
*** mcornea has joined #tripleo09:34
*** mbound_ has joined #tripleo09:35
shardyjaosorior: thanks - sorry to nitpick :)09:41
jaosoriorshardy: No biggie dude, sometimes it can be quite useful.09:42
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Enable TLS endpoints through os-cloud-config  https://review.openstack.org/25461309:48
gfidentejaosorior, was it the -x ?09:55
jaosoriorgfidente the above CRs to tripleo-heat-templates, yes09:55
jaosoriorthe one to python-tripleoclient is not a backport09:56
jaosoriorgfidente: But it would be cool if you have time to check it out. It enables us to have the keystone endpoints configured with TLS, through os-cloud-config. And that one we can actually backport nicely09:58
*** nijaba has quit IRC10:02
*** electrofelix has joined #tripleo10:02
*** openstackgerrit has quit IRC10:02
*** openstackgerrit has joined #tripleo10:03
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates: Ceph cluster validation  https://review.openstack.org/22524810:03
d0ugalregebro: ping10:04
d0ugalregebro: I was going to land https://review.openstack.org/#/c/238016/, but looks like it needs a rebase.10:05
regebrod0ugal: pong!10:05
d0ugalI tried to do it, but I got confused :)10:05
d0ugaloh, it's because the conflict is in the dependant review. i see.10:06
regebrod0ugal: yes, exactly.10:06
jaosoriord0ugal, regebro: It probably better to remove that dependency10:06
d0ugaljaosorior: Yeah, I wasn't sure if it was needed, but that makes sense10:06
regebrojaosorior, d0ugal  Can't, they go together.10:06
jaosoriorseems that this one https://review.openstack.org/#/c/230275/6 is not merging any time soon :/10:06
d0ugalah10:07
jaosoriorregebro: unless you manage to convince the dev that -1ed it10:07
d0ugalA fair bit of discussion there10:07
d0ugalI think I might have opened a can of worms10:07
d0ugal:)10:07
regebroIt's all about making sure you don't have to have things in /home/stack specifically.10:07
d0ugalright, makes sense.10:08
jaosoriord0ugal: Hey, got a couple of minutes to check this one out? https://review.openstack.org/#/c/254613/10:13
d0ugaljaosorior: sure10:13
*** mkovacik has joined #tripleo10:18
*** dtantsur is now known as dtantsur|brb10:19
openstackgerritMerged openstack/instack-undercloud: Enable coverage testing  https://review.openstack.org/25157510:20
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Enable TLS endpoints through os-cloud-config  https://review.openstack.org/25461310:21
jaosoriord0ugal: Thanks for the review. Addressed the comment there ^^10:22
d0ugaljaosorior: Thanks!10:22
*** jcoufal has joined #tripleo10:25
openstackgerritMerged openstack/tripleo-common: Make coverage tests work  https://review.openstack.org/25155910:25
*** jistr|mobi has joined #tripleo10:26
jistr|mobihi folks, i have a hw problem with my laptop, so i'm off internal irc for now10:27
mariosjistr|mobi: ack10:27
jaosoriorshardy: You commented on this CR https://review.openstack.org/#/c/221416/ regarding the CI failures10:34
jaosoriorshardy: I'm seeing those same failures sporadically on other CRs too10:34
*** olaph has quit IRC10:36
*** olaph has joined #tripleo10:36
shardyjaosorior: Ok, we should probably raise bugs if none already exist10:36
*** jistr|mobi has quit IRC10:39
*** jistr has joined #tripleo10:39
jaosoriorshardy: This commit seems fishy, but honestly I'm not sure :/ https://github.com/openstack/puppet-swift/commit/5a7d18975f891c628cdfce798c5c426046203fcb10:42
jaosoriorthe error complaigns that swift::storage::server already imports swift::storage:account10:42
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays  https://review.openstack.org/18750610:51
gfidenteshardy, do you know if there is a better way to do ^^ ?10:52
*** mgould has joined #tripleo10:52
shardygfidente: what format does puppet expect the list in - will a json serialized list work?10:54
shardyit looks like that's what we're constructing via the str_replace/list_join?10:55
gfidenteshardy, yep exactly10:55
shardygfidente: Ok, yeah there's a better way from liberty, I'll comment on the review10:55
gfidenteshardy, so we can backport in stable/liberty too10:56
gfidentewhich is great :)10:56
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Allow the deployer to pick a predefined IP for VIPs  https://review.openstack.org/24575011:04
shardygfidente: commented, there's a slightly cleaner syntax available with liberty heat.11:04
*** jerrygb has joined #tripleo11:04
shardyotherwise looks good11:04
shardygfidente: Hey I also wanted your feedback on https://review.openstack.org/#/c/254667/11:05
shardyIt's a way to get predictable placement of nodes via scheduler hints11:05
shardywhich I think should work well with your pre-allocated IP addresses patch11:05
*** yuanying has quit IRC11:07
gfidentewow it's pretty simple11:09
*** jerrygb has quit IRC11:09
gfidenteI mean, not 'intrusive'11:09
gfidenteshardy, it's a little harder to control hostname, but gives predictable placement11:12
gfidenteso I think it's fine11:13
gfidentehostnames can be configured already, the only limitation is the index progression but why one wouldn't want that11:13
*** lucas-dinner is now known as lucasagomes11:16
shardygfidente: Yeah, it is pretty simple :)11:16
shardygfidente: and yeah, the HostnameFormat approach is a bit limited, but folks could still override the default hostname with a mapping file if needed11:17
gfidenteI failed at figuring it initially11:17
gfidentebecause I didn't get how close the nova scheduling was to the ironic scheduling11:17
shardyI'd propose the hostname mapping thing as being something we document via an ExtraConfig script, similar to https://review.openstack.org/#/c/215116/11:18
shardyYeah, I think we can use the scheduler_hints exactly like the flavor matching that's already documented11:19
shardywould be good to get confirmation from someone else it all works though, I only tested it on some VMs11:19
openstackgerritMiles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state  https://review.openstack.org/23515811:19
openstackgerritDougal Matthews proposed openstack/tripleo-common: TripleO API v1  https://review.openstack.org/24373711:21
openstackgerritDougal Matthews proposed openstack/tripleo-common: Add environment capabilities  https://review.openstack.org/23253411:21
trembleshardy: https://review.openstack.org/#/c/254667/ it would be good if those hints can be added when you import the nodes...11:23
shardytremble: Yeah, I was thinking the same thing11:24
shardywe can potentially add support for that11:24
shardyhttps://bugs.launchpad.net/tripleo/+bug/152429911:24
openstackLaunchpad bug 1524299 in tripleo "CI error Error: Could not find dependent Achor[nova::config::end]" [Undecided,New]11:24
shardyJust raised that as we're seeing lots of those errors in CI11:25
*** dtantsur|brb is now known as dtantsur11:25
shardylooks like a puppet regression, but I'm not quite sure where11:25
*** links has quit IRC11:26
*** mgould has quit IRC11:32
*** mgould has joined #tripleo11:45
*** akrivoka has quit IRC11:45
*** akrivoka has joined #tripleo11:47
*** mburned_out is now known as mburned11:58
*** panda has quit IRC12:03
*** panda has joined #tripleo12:04
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the controller nodes  https://review.openstack.org/22249712:10
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: neutron: enable nova-event-callback by default  https://review.openstack.org/25522612:12
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of predictable IPs  https://review.openstack.org/24160612:13
openstackgerritSteven Hardy proposed openstack/python-tripleoclient: Allow node import via yaml not only csv/json  https://review.openstack.org/25522812:16
*** links has joined #tripleo12:16
shardytremble: ^^12:16
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: neutron: enable nova-event-callback by default  https://review.openstack.org/25522612:16
shardyneeds testing as I've only done the unit tests so far12:16
trembleshardy: :)12:16
* shardy heads to lunch12:16
*** devvesa has quit IRC12:20
*** marios_ has joined #tripleo12:21
marios_jistr: did you manage to get on? /me having vpn issues now12:24
*** marios_ has quit IRC12:25
*** marios_ has joined #tripleo12:27
*** thrash|g0ne is now known as thrash12:28
jistrmarios: sent a mojo link to gfidente how to set up VPN to BRQ endpoint, will pvt that link to you too12:32
jistrmarios: re my hw troubles, it was different, HDD in my laptop slipped off its connection bus because the the casing is not correct size for it12:33
*** weshay_xchat has joined #tripleo12:39
*** weshay_xchat is now known as weshay12:39
*** chlong has joined #tripleo12:41
*** jaosorior has quit IRC12:42
*** jaosorior has joined #tripleo12:42
*** rhallisey has joined #tripleo12:43
*** dmacpher has joined #tripleo12:43
mariosjistr: thanks gfidente it worked12:46
*** marios_ has quit IRC12:47
mariosjistr: oh dear. did they manage to reconnect or you need replace?#12:48
*** shardy has quit IRC12:49
*** shardy has joined #tripleo12:51
*** rpothier has quit IRC12:52
*** Goneri has joined #tripleo12:53
*** jerrygb has joined #tripleo12:53
jistrmarios: yeah just reconnected, plus put a piece of pencil eraser there to make things hold in place :))12:54
*** dprince has joined #tripleo13:01
*** julim has joined #tripleo13:01
*** ayoung has quit IRC13:02
*** morazi has joined #tripleo13:08
*** jayg|g0n3 is now known as jayg13:14
openstackgerritMerged openstack/python-tripleoclient: [liberty backport] Disable command run under root user  https://review.openstack.org/24311613:16
*** hewbrocca has quit IRC13:18
*** hewbrocca has joined #tripleo13:22
jistrhmm weird things with the CI. I've just hit a 2nd legit bug which breaks my dev env but hasn't been caught by the CI. This has been merged 2 days ago to puppet-swift and breaks swift deployment on the controller (duplicate declaration in puppet), yet cistatus.html is mostly green. https://github.com/openstack/puppet-swift/commit/5a7d18975f891c628cdfce798c5c426046203fcb13:26
slaglejistr: i'm seeing that failure in CI13:28
slaglehttp://logs.openstack.org/93/245093/17/check-tripleo/gate-tripleo-ci-f21-ha/feb6128/console.html13:28
slaglewas just looking at why https://review.openstack.org/#/c/245093/ failed :)13:28
jistryeah that's the failure i meant... hmm how come we have green jobs which are more recent than that though http://tripleo.org/cistatus.html13:30
jistrthis is not an intermittent thing13:30
* jistr checks if they did further updates to puppet-swift13:30
jistrhmm nope13:31
jistrand no updates to tripleo.sh or tripleo-ci either13:31
jistrwat... :)13:31
slaglejistr: i'm seeing a difference between the job that failed and the one that passed13:33
slaglein the one that passed puppet-swift appears to be from opm13:33
slagle2015-12-09 10:02:00.870 | puppet-swift install type not set to source13:34
slaglein the one that failed, that's not the case13:34
slaglei thought we were always setting DIB_INSTALLTYPE_puppet_modules=source in CI, but apparently not13:34
slaglethat string appears nowhere here: http://logs.openstack.org/32/254832/3/check-tripleo/gate-tripleo-ci-f21-ha/b1a9260/console.html (passed)13:35
slaglebut is set here: http://logs.openstack.org/93/245093/17/check-tripleo/gate-tripleo-ci-f21-ha/feb6128/console.html (failed)13:35
jistryeah not setting DIB_INSTALLTYPE_puppet_modules=source explains the failure13:35
jistrhmm13:35
jistrand it's the same type of job, f21-ha. interesting13:36
*** psanchez has quit IRC13:37
*** masco has quit IRC13:43
slaglejistr: ok i think it's just that the ha jobs that have passed are against stable/liberty and not master13:43
slagleand on stable/liberty opm is used13:43
jistrslagle: ahh, good spot. So CI works as expected, stable is passing and master is failing due to the puppet-swift issue. I'll see what can i do on the puppet-swift issue, i have it reproduced locally.13:45
slaglejistr: should we ask for a revert?13:46
slagleEmilienM: https://github.com/openstack/puppet-swift/commit/5a7d18975f891c628cdfce798c5c426046203fcb is breaking tripleo ci13:47
EmilienMlooking13:47
slagleEmilienM: failed log here: http://logs.openstack.org/93/245093/17/check-tripleo/gate-tripleo-ci-f21-ha/feb6128/console.html13:47
EmilienMargh13:47
EmilienMI checked tripleo logs before13:47
EmilienMlet me verify13:47
EmilienMslagle: https://review.openstack.org/#/c/203220/13:48
jistrEmilienM: it's that one statement in storage/server.pp changed from "include ..." to "class ...", and we are already defining that class, because we need to pass manage_service params to it13:48
EmilienMthe patch passed tripleo CI13:48
EmilienMso it breaks HA jobs13:48
slagleEmilienM: i see it runs the nonha job though. could it be just that it's breaking ha?13:49
EmilienMslagle: yes13:49
EmilienMbecause we manage_service to false, because we use pacemaker13:49
EmilienM...13:49
jistryea exactly13:49
EmilienMjistr: what if we set the param with Hiera?13:50
jistrok lemme see if i can fix it on tripleo side first, that patch would be a big revert13:50
jistrEmilienM: we cannot have a custom hiera for pacemaker at the moment :(13:50
jistri had a patch for it but didn't pass review stylistically13:50
EmilienMargh13:50
jistrand dprince's composalbe roles should solve it more elegantly13:50
jistrbut we don't have that in master yet13:51
*** yamahata has joined #tripleo13:51
*** mgould has quit IRC14:01
jistrEmilienM: can you think of some way how to set a class parameter default? i think Swift::Storage::Account { ... } syntax works only for resources, same for resource collectors, right?14:01
*** yamahata has quit IRC14:02
dtantsurslagle, hi! could you do something about https://review.openstack.org/#/c/235236/ please? it's approved but does not merge..14:02
slagledtantsur: what do you want me to do?14:06
*** Goneri has quit IRC14:07
dtantsurslagle, maybe W+1 once more, maybe I can remove Depends-On and you reapprove.. whatever you feel appropriate14:07
*** jerrygb_ has joined #tripleo14:08
*** tosky_ is now known as tosky14:11
*** jerrygb has quit IRC14:11
*** mgould has joined #tripleo14:13
openstackgerritJames Slagle proposed openstack/instack-undercloud: Make 98-undercloud-setup rerunnable  https://review.openstack.org/25494614:18
*** lblanchard has joined #tripleo14:20
*** tzumainn has joined #tripleo14:25
openstackgerritDmitry Tantsur proposed openstack/diskimage-builder: Add kmod to package-installs of ironic-agent  https://review.openstack.org/23523614:27
dtantsurslagle, could you reapprove ^^ please? it's only a commit message change14:27
*** ayoung has joined #tripleo14:28
beaglesdprince, shardy: EmilienM had an excellent point on https://review.openstack.org/#/c/247056/ where I clumsily pulled in some sriov support. What are your thoughts on this? Should I yank that out until sriov support is "officially" (read deliberately) added or is okay to leave in case somebody does add the sr-iov mechanism driver when QoS is enableD?14:32
*** devvesa has joined #tripleo14:33
jistrslagle: there's no way around that puppet-swift issue, even setting things via hiera might not help because the $enabled and $manage_service values are actually generated dynamically within the manifest. ($step is one of the inputs). I'm trying to put together a fix to both puppet-swift and t-h-t, but in the meantime we might want either a pin or a revert. Since it's 2 patches i don't think it will land blazingly fast, and ATM i'm just 80%14:33
jistrsure that the approach i'm taking is correct.14:33
slaglejistr: ok. EmilienM, what do you think?14:33
slaglei tend to favor reverts14:34
*** jdob_lt has joined #tripleo14:34
EmilienMwhy me14:34
slagledoesn't this potentially affect any consumer of puppet-swift who might be using it the same way as tripleo does?14:34
EmilienMgo on #puppet-openstack and talk with Vinsh14:34
EmilienMhe's the author14:34
slaglejust thought you might have an opinion on it14:35
EmilienMbefore revert, I prefer talking with Vinsh, the author14:35
EmilienMand eventually find a consensus14:35
EmilienMslagle: just kidding :-)14:35
dprincebeagles: no oppinion. I'm probably fine to leave sriov if it would be useful for someone14:36
beaglesdprince, okay. Thanks.14:37
slagleEmilienM: :) you know i don't like talking in #puppet-openstack b/c i sound like an idiot14:37
slaglei'm not hip enough to talk about puppet or ruby14:37
EmilienMslagle: let jistr, he's a puppet guy :P14:37
EmilienMahah14:37
EmilienMyou're hype, OOO is very hype nowadays14:38
slagleoh, it's definitely a lot of hype.14:38
*** nico_auv has quit IRC14:38
trown|BOSlol14:41
*** pradk has joined #tripleo14:42
*** pradk has quit IRC14:43
*** pradk has joined #tripleo14:44
*** lucasagomes is now known as lucas-hungry14:46
EmilienMslagle: in the meantime jistr is fixing puppet-swift, I highly suggest you to pin your CI to an older puppet swift14:50
jaosoriorshardy: I got busy and left for lunch, so didn't check the stuff from the CI, did you figure out something? I'm getting back to checking it out14:50
*** jprovazn has quit IRC14:50
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Verify galera is sync'd in yum_update.sh  https://review.openstack.org/25529814:54
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add constraints and timeouts from file in single step  https://review.openstack.org/25529914:54
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Apply mongod timeout via cib-push  https://review.openstack.org/25530014:54
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add pcmk constraints against haproxy-clone only if applicable  https://review.openstack.org/25530114:54
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Update: clean keepalived and radvd instances after pcs cluster stop  https://review.openstack.org/25530214:54
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Fixup neutron constraints in older overclouds before updating  https://review.openstack.org/25530314:54
shardyjaosorior: Hey, I've been at lunch too, I assume the discussion in the scrollback about broken puppet has something to do with it ;)14:56
jaosoriorshardy: Yeah, now I noticed jistr is taking a shot at it in the puppet-openstack channel14:56
*** mbound_ has quit IRC14:58
jistrslagle: actually we're probably gonna go with Vinsh's suggestion which should fix it fully, not just for TripleO's use case of $manage_service and $enable. Basically it's going to be a "partial rollback" in the problematic spot, to use the original include-like style.14:58
jistri'm going to submit a pin to tripleo.sh for the time being14:58
slaglejistr: ok. should we pin in tripleo-ci in the meantime?14:58
slagleok :)14:58
jistrcc jaosorior ^14:58
*** nico_auv has joined #tripleo15:00
jistrwill report a bug too with the messages pasted, to make sure it makes sense to folks who weren't looking at the issue15:02
jistrhttps://bugs.launchpad.net/tripleo/+bug/152438115:09
openstackLaunchpad bug 1524381 in tripleo "Duplicate declaration when defining swift::storage::* outside swift::storage::server" [Critical,Triaged] - Assigned to Jiří Stránský (jistr)15:09
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140815:15
rhalliseytrown|BOS, looks like the ironic issue is because we have an old version of ipxe in the puddle15:16
*** derekh has joined #tripleo15:18
openstackgerritJiri Stransky proposed openstack/tripleo-common: Pin puppet-swift due to duplicate declaration  https://review.openstack.org/25531415:18
jistrderekh: hello :) can you please skim through that ^^ if it looks correct to you (mainly the variable name and where i put the variable)15:20
derekhjistr: looking15:21
openstackgerritMiles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state  https://review.openstack.org/25531915:27
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Do not merge: testing a puppet-swift fix  https://review.openstack.org/25532015:29
derekhjistr: it looks correct, but since the fix is also proposed maybe we should wait and see if it passes, if I approve it the pin will prevent testing that fix15:30
jistrderekh: yeah we could do that. The puppet-swift bug only breaks HA job though, and the only non-HA job is being triggered on commits to puppet modules. So i submitted another patch ^^ to get a full CI run on the puppet-swift patch via Depends-On.15:31
derekhjistr: or we could merge the pin and then you test could also repend on a revert of it15:32
jistrderekh: yeah, either way i'd wait for the CI to finish on the tripleo.sh pin patch15:32
derekhjistr: cool, I'll check back in a bit and see whats passed15:33
*** dprince has quit IRC15:33
*** jprovazn has joined #tripleo15:35
*** mkovacik has quit IRC15:36
*** lucas-hungry is now known as lucasagomes15:42
*** rpothier has joined #tripleo15:43
*** paramite has quit IRC15:45
ayoungjaosorior, re https://review.openstack.org/#/c/213175/10/puppet/manifests/overcloud_controller_pacemaker.pp,cm  I just reased this, had not even tested.  You think it would work if I just removed the section above that?15:45
jaosoriorayoung: Let me recheck those modules, I'm not entirely sure15:46
ayoungjaosorior, I'm assuming modulo the changes from Keystone to apache on lines 1587 and 159015:46
jaosoriormarios: Are these series clean cherry-picks? https://review.openstack.org/#/c/255298/115:46
*** nijaba has joined #tripleo15:47
*** nijaba has quit IRC15:47
*** nijaba has joined #tripleo15:47
jaosoriorayoung: I think it'll be alright. mostly what those manifests ( <service>::keystone::auth ) seem to do is setup the keystone services and endpoints, and that currently is not done from puppet (there are plans to change that but there are blockers for it)15:49
ayoungjaosorior, these are the values  that ends up in the database?15:49
jaosoriorayoung: So it should be alright to remove from line 1592 to 161115:49
mariosjaosorior: yeah no conflicts, but only cos/is why i did them in that order15:50
jaosoriormarios: cool15:51
jaosoriorayoung: Yeah, so those puppet resources create the service users, services and endpoints in keystone15:51
openstackgerritMiles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state  https://review.openstack.org/23515815:51
ayoungjaosorior, so, not necessary for this review, but we will need in the futre15:52
jistrtrown|BOS: hi, are you the right person to talk to about bumping http://trunk.rdoproject.org/centos7/current-tripleo ?15:52
jaosoriorayoung: But we won't get that in puppet yet, until there's a gate that tests updates15:52
jaosoriorayoung: indeed, and it's done in this CR, which is blocked cause of the lack of that gate: https://review.openstack.org/#/c/244162/515:52
ayoungcool15:53
trown|BOSjistr: nope derekh would be the one who could update that... I do not think there is a newer repo that will work right now though for a few reasons15:54
ayoungjaosorior, not sure how I feel having the endpoints managed by puppet.  I think that might be too far15:54
jaosoriorayoung: well, currently it's done via os-cloud-config15:54
*** liverpooler has quit IRC15:55
jaosoriorayoung: which is triggered by the python-tripleoclient15:55
jistrtrown|BOS, derekh: ok... the repo which is there doesn't work either though. But i already know what to cherry-pick to make it work so if we cannot move to something fully working for now, maybe we shouldn't move it at all :) I'll push code somewhere which will show how i fix the env.15:55
ayoungjaosorior, yeah, but that is initialization, which is OK.  If we go with full Puppet master/style support, then this is probably right.  I just would not want to upstae my endpoints and then find out the y were reverted on me during an upgrade15:56
ayoungupstae->update15:56
derekhjistr: I'll take a look at it once we sort out the puppet-swift thing15:56
jistrderekh: ok. just to paste the relevant info so that i don't forget about it -- the issue is that we're getting openstack-ironic-common-4.2.1-dev242.el7.centos.noarch, which has this bug present but not the fix https://bugs.launchpad.net/tripleo/+bug/150773815:57
openstackLaunchpad bug 1507738 in tripleo "ipxe fails on Centos 7 (inc: command not found)" [Critical,Triaged]15:57
jistrafter applying the fix manually in place, ironic works15:57
jistri'm puzzled how come CI doesn't break on that15:58
jaosoriorayoung: That's a pretty good point15:58
trown|BOSjistr: is there a delorean repo that has the fix?15:59
jistrtrown|BOS: didn't search for it yet but i guess there would be (?) the fix has been merged to ironic already15:59
trown|BOSjistr: ah right ya16:00
jaosoriorayoung: The thing is that now to do changes in the endpoints, one would need to make a change in the tripleo-heat-templates. Because that's actually handlling the configuration for HAproxy, which is the one that's actually listening and then forwarding the request to the actual servers16:00
jaosoriorayoung: So at least you would need to do everything in one place, instead of having to re-configure the frontends in HAProxy, and then having those match the keystone endpoints16:01
ayoungjaosorior, BTW, does tox run on tripelo-heat-templates?16:03
*** panda has quit IRC16:03
*** panda has joined #tripleo16:03
jaosoriorayoung: no16:04
*** pradk has quit IRC16:05
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Pass the endpoint ports and protocols through to hiera  https://review.openstack.org/24049816:06
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols  https://review.openstack.org/24020816:08
*** aufi has quit IRC16:08
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the controller nodes  https://review.openstack.org/22249716:08
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add sample environment file to document usage of predictable IPs  https://review.openstack.org/24160616:09
openstackgerritJeff Peeler proposed openstack/instack-undercloud: Install local docker registry with puppet  https://review.openstack.org/23823816:09
jaosoriorgfidente: rebased your commit above ^^16:09
*** pradk has joined #tripleo16:10
gfidentejaosorior, ack16:11
*** Goneri has joined #tripleo16:12
*** tzumainn has quit IRC16:14
*** jerrygb_ has quit IRC16:18
*** jcoufal has quit IRC16:20
openstackgerritMerged openstack/diskimage-builder: Add kmod to package-installs of ironic-agent  https://review.openstack.org/23523616:22
*** tremble has quit IRC16:28
*** jdob_lt has quit IRC16:29
jistrjaosorior, ayoung: fwiw inlunch *should* be working now. I didn't test it in a single end-to-end run, but rather added things which i've done manually to work around issues. Old workarounds removed, switched to using tripleo.sh, 2 new workarounds added :)16:29
jistrjaosorior: thanks for the pull request16:29
beaglesEmilienM, (or anybody) I wonder if I'm operating under a conceptual mismatch with respect to the tripleo templates and wonder if you can set me straight...16:30
ayoungjistr, cool.  I've been using it with jaosorior 's patch16:30
ayoungjistr, I do think it might be time to move some of the scripts in answers... into the Ansible.16:30
jistrayoung, jaosorior: right. That means you're not as close to upstream master branches as possible, but most probably that is not a problem.16:30
jistrayoung: nooo ;)16:31
beaglesEmilienM, (or again anybody ;)) my concern that I've been thinking more of this as a pseudo installer vs a simple deployment mechanism and that is influencing my expectations, decisions, etc.16:31
ayoungjistr, yes...you can still chose to do two different variation based on an env var, and not all of them, but have a "expected path" from ansible, with the answers file providing the override16:31
jaosoriorjistr: It's close enough to do some productive work. but I'll try your new patches for inlunch soon. Thanks for the fixes dude :D16:31
beaglesEmilienM, case in point.. should it be easy to control a feature from the templates or is the goal simply to expose the raw configuration elements so someone can do the rest through specialized/customized environment files16:31
jaosoriorgotta go guys, have a good evening/day16:32
*** jaosorior has quit IRC16:32
EmilienMbeagles: I'll let tripleo gurus reply :P16:33
jistrayoung: having those bash snippets is very useful imho, and easily customizable. It's not just forking here and there, you can set up completely different envs with that. It's not worth spending the time and debugging on implementing proper ansible task for each mini-step. The purpose of the tool is to get a dev env spinned up quickly, be super-easy to customize, and not spend too much time maintaining it :)16:33
beaglesshardy, can you comment on the questions I posed to EmilienM above ^^^16:33
*** tzumainn has joined #tripleo16:34
dtantsurfolks, could someone please release DIB now? we need it for Linux-IO support for IPA16:35
jistrayoung: i actually started with all-in-ansible approach like i'm used to, but it was a pain to maintain, as tripleo tends to change fast16:35
shardybeagles: what's the question - when do we expose configuration options as parameters?16:37
shardybeagles: vs hacking things in via additional hieradata in environment files?16:38
beaglesshardy: actually I think that's pretty much it :)16:38
shardybeagles: There's no simple answer tbh, it's a judgement call - the idea is the heat template parameters provide a reasonable schema with which to influence the deployment of your cloud16:39
shardybeagles: but the ExtraConfig parameters allow for transparent override of anything via hieradata, so advanced operators aren't constrained by the parameters16:40
shardyI always thing "would this make sense in a UI"16:40
shardyif the answer is yes, you probably want a heat parameter for it16:40
shardybecause the way folks are discovering what to ask users for in terms if UX input is looking at the heat parameters16:40
shardybut if it's some super-advanced ceph-hacker optimization or whatever, then I think documenting using the ExtraConfig parameters is fine16:41
openstackgerritRyan Brady proposed openstack/tripleo-common: Adds file deletion to plan manager  https://review.openstack.org/25536616:41
shardyIt's impossible to expose every concieveable option as a top level parameter, so we have to draw a line somewhere.16:41
jistrderekh: this is the fix i do for ironic, not sure if we want to add something like that to tripleo.sh (or if we can do cherry-picks like we used to in tripleo-ci?) https://github.com/jistr/inlunch/commit/21cff69e75cd5a628959f9b683f113dff0a62a6816:44
*** jprovazn has quit IRC16:46
beaglesshardy, actually - I think I'm still missing the gist by reading too much into your answer. I'll try to reword the question: should a heat parameter always map to a single configuration element or is a "switch" that influences several configuration elements across the board okay as well16:47
derekhjistr: ack, thanks16:47
derekhjistr: not sure if cherry-picks still work, I'm not sure if I've done one since that change to tripleo.sh, will look16:47
beaglesshardy, while the QoS thing is an example, another might be the nova-neutron event callback mechanism16:48
*** jprovazn has joined #tripleo16:48
beagles(though they might be different enough to warrant two different answers, I realize)16:48
derekhjistr: at the moment, I'm hoping to try a more recent trunk repo once the swift issue is resolved, that should make the workaround no needed, if it works I'll update currenct-tripleo16:49
beaglesshardy, so do you have "EnableEventCallbacks" as a top level parameter or do you have a file that is "EventCallbacks.yaml"16:49
jistrderekh: cool, if that turns out as an achievable solution, that would surely be better16:49
shardyderekh: heads up, oslo.utils just landed a change which breaks master delorean16:50
shardybecause of a missing dep on funcsigs16:50
derekhshardy: ack thanks, I'll try something before that so16:51
shardyso we may need to pin that, my delorean builds are breaking locally16:51
shardyI'm about to propose a patch to the specfile16:51
shardybut funcsigs is still under review according to BZ, or only just approved16:51
shardyI don't see it in the repos yet16:51
*** rlandy has joined #tripleo16:52
beaglesshardy, or do you just expose the constituent variables and it is somebody-else's-problem and they figure it out (event callbacks is probably a bad choice here because they actually can't figure all of that out at deployment time)16:52
derekhshardy:  is it a dep that should be in delorean trunk?16:52
shardybeagles: IMO both is fine, we already have parameters which map directly into configuration, and stuff which acts like a conditional16:52
shardybeagles: but when there's a new feature which requires mapping resource implementation and parameters, we prefer to pass it encapsulated in an environment file16:53
shardye.g -e network_isolation -e neutron_ml2_foo.yaml etc16:53
shardythe differentiator is a parameter can only influence the configuration (e.g manifests) we enable by default16:54
shardyso any extra or non-default config is likely to require an additional environment, which, aligns with your required deployment environment16:55
shardybeagles: that's why we don't use parameters to e.g select cinder backends or neutron plugins - a conditional parameter isn't flexible enough in that case16:55
beaglesshardy, ahhhh I get it16:55
shardyderekh: I think so, here's the fedora package review https://bugzilla.redhat.com/show_bug.cgi?id=128789916:55
openstackbugzilla.redhat.com bug 1287899 in Package Review "Review Request: python-funcsigs - Python function signatures from PEP362 for Python 2.6, 2.7 and 3.2+" [Medium,Assigned] - Assigned to jpena16:55
shardybeagles: All of that is IMHO/IME - we should formally document it somewhere16:56
beaglesshardy, because our default stuff is oriented to a sort of "reference implementation" of sorts...16:56
shardybeagles: Yeah, pretty much that's it16:56
shardywe have a relatively opinionated default setup, with very flexible ways to wire in additional/optional pieces16:56
derekhshardy: just spoke to number80 , he is going to import it16:57
derekhshardy: funcsigs that is16:57
shardyderekh: Cool, I'll push a patch to the spec file Requireing it16:57
beaglesshardy, that actually answers my question. The QoS stuff is suspiciously linked to my question, I admit, but I was more concerned with totally "missing the point" of this stuff :)16:57
beaglesshardy thanks for that16:58
*** dtantsur is now known as dtantsur|afk16:58
*** sthillma has joined #tripleo17:00
*** ukalifon has quit IRC17:01
*** links has quit IRC17:01
*** sthillma_ has joined #tripleo17:01
*** sthillma has quit IRC17:05
*** sthillma_ is now known as sthillma17:05
*** jprovazn has quit IRC17:06
*** Marga_ has quit IRC17:06
*** jprovazn has joined #tripleo17:06
*** dprince has joined #tripleo17:10
*** psanchez has joined #tripleo17:11
*** Marga_ has joined #tripleo17:11
*** rcernin has quit IRC17:11
*** shivrao has joined #tripleo17:12
*** Marga_ has quit IRC17:12
*** jistr has quit IRC17:19
*** shivrao has quit IRC17:21
*** shivrao has joined #tripleo17:22
*** tosky has quit IRC17:23
*** tosky has joined #tripleo17:23
trown|BOSd0ugal: for https://review.openstack.org/254885 we will have to merge with failing CI... that was the idea with splitting it to a purelyish additive noop patch17:26
d0ugaltrown|BOS: Gotcha, I understood that after thrash explained it to me :)17:26
trown|BOSd0ugal: once that lands, we can have passing CI on the packaging change, and then we should be able to get passing CI on the dependant patches that add the functionality17:27
*** olaph has quit IRC17:27
trown|BOSd0ugal: just saw you rechecked so thought I would explain :)17:27
*** olaph has joined #tripleo17:27
d0ugaltrown|BOS: Thanks17:27
*** eil397 has joined #tripleo17:30
*** Goneri has quit IRC17:31
*** olap has quit IRC17:31
*** nico_auv has quit IRC17:34
*** devvesa has quit IRC17:36
*** ayoung has quit IRC17:37
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks  https://review.openstack.org/23542317:39
*** shivrao has quit IRC17:41
*** alop has joined #tripleo17:42
*** cmyster has quit IRC17:42
*** cmyster has joined #tripleo17:43
*** cmyster has quit IRC17:43
*** cmyster has joined #tripleo17:43
openstackgerritRyan Brady proposed openstack/tripleo-common: Ensures plan exists  https://review.openstack.org/25540017:43
*** alop has quit IRC17:56
*** alop has joined #tripleo17:57
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove deprecated overcloud-resource-registry.yaml  https://review.openstack.org/25540717:58
shardydprince: ^^ Hey FYI I noticed we missed the registry referencing all the old o-a-c templates17:58
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove deprecated overcloud-resource-registry.yaml  https://review.openstack.org/25540718:00
dprinceshardy: cool. +218:05
dprinceshardy: nice to have all this gone18:05
dprincesorry about all the rebasing it may be causing though18:06
shardyjtomasek: Hey I just saw your ML reply re nested validation & parameters18:10
shardythat is a heat bug, see https://bugs.launchpad.net/heat/+bug/150885718:10
openstackLaunchpad bug 1508857 in python-heatclient "nested validation requires parameters, but they are not accepted" [Medium,Triaged] - Assigned to Steven Hardy (shardy)18:10
shardywe need to accept the parameters if they are passed, but also not require them18:10
*** athomas has quit IRC18:13
openstackgerritMiles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state  https://review.openstack.org/23515818:14
*** electrofelix has quit IRC18:22
*** mcornea has quit IRC18:25
*** mcornea has joined #tripleo18:27
jtomasekshardy: thanks18:28
*** edmondk has quit IRC18:29
shardyjtomasek: a workaround so you're not blocked is to pass an environment file with dummy values for all the mandatory parameters18:29
jtomasekshadower: yeah, that's what I did for now18:29
shardyjtomasek: Ok, cool, will try to get that fixed soon18:29
jtomasekshardy: in nested params I see parameters without default and value, that means those need the value to get set and are required, right?18:30
shardyjtomasek: Yes that's right18:30
jtomasek(some of them are like that to be exact)18:30
jtomasekok18:31
shardyjtomasek: do you have an example we can check?18:31
*** mgould has quit IRC18:31
jtomasekshardy: yep, I can send you a json output I am receiving18:31
*** tosky has quit IRC18:33
jtomasekshardy: this is what I get from the validation when I use tripleo-heat-templates with just with overcloud-resource-registry-puppet.yaml environment18:33
jtomasekshardy: http://pastebin.test.redhat.com/33414118:34
jtomasekshardy: one thing which is not clear to me: the root parameter values are not set yet. Do I need to set the root parameter values and run the validation again to receive a value in nested parameters?18:35
jtomasekshardy: how can I identify if the nested parameter receives the value from parent in this output? I think it has been answered somewhere but I can't find it just now18:36
*** mcornea has quit IRC18:37
shardyjtomasek: the "Value" key is set if the template recieved a value from the parent18:41
*** Goneri has joined #tripleo18:41
shardyjtomasek: so, if you do a (non nested) validation of the parent (overcloud.yaml, root template), and pass dummy values for all parameters which don't have defaults, you should see that value as "Value" in the NestedParameters, when it's passed from parent->child template18:41
*** dsneddon has quit IRC18:42
shardyjtomasek: it would be better (when we fix that heat bug) if we just had e.g "Value": None when a value will be passed from the parent, but it's not yet defined18:42
*** sthillma has quit IRC18:43
shardyjtomasek: we seem to have a similar problem with e.g blockstorage_servers which is passed via an attribute from a resource in the parent template18:43
shardyI expected to see that have a Value: key assigned18:43
shardyjtomasek: You can see it working correctly e.g in PingTestIps, where the Value key is set to the spaces generated by list_join in overcloud.yaml18:45
*** Marga_ has joined #tripleo18:46
shardyjdob: Hey you may be interested in this, we have a few more nits than I thought to work out of nested validation ^^18:46
*** dsneddon has joined #tripleo18:46
jdobshardy: reading18:46
shardyjdob: we have https://bugs.launchpad.net/heat/+bug/1508857 but the paste from jtomasek shows we've got a few additional issues18:47
openstackLaunchpad bug 1508857 in python-heatclient "nested validation requires parameters, but they are not accepted" [Medium,Triaged] - Assigned to Steven Hardy (shardy)18:47
jdobshardy: oh ya, is this about the fact that it has to be valid in the first place?18:47
jtomasekshadower: yeah, I was wondering why the blockstorage_servers etc. were not set because their value should be set in root18:47
jdobshardy: want me to take that bug so you're not distracted from your PoC? i'm largely waiting on a shit ton of reviews on my other stuff and have some time18:47
shardyjdob: Yeah, that's one part, but also we're not assigning "Value" when the value is passed in from the parent template via an attribute18:48
jdobwell, shit, that sounds like a problem :)18:48
shardyjdob: I was hoping you'd say that :D18:48
shardyYeah, if you have bandwidth to pick it up, that would be great, thanks18:48
jdobthe flip side is now I'm going to harass you to look at my multi-env and chains reviews :D18:49
jdobthough you're largely live testing the chains stuff, so you get brownie points for that18:49
*** Marga__ has joined #tripleo18:49
shardyjdob: yeah, I took a look earlier actually at the chains one, looking good18:49
derekhperiodic jobs are still failing, could I get some eyes on these fixes https://review.openstack.org/#/c/244519/ https://review.openstack.org/#/c/244526/218:49
jdobshardy: I grabbed the bug in launchpad, I'll take a look in a bit18:50
jdober, get started in a bit18:50
*** Marga_ has quit IRC18:50
*** Marga__ has quit IRC18:50
*** ifarkas has quit IRC18:51
*** Marga_ has joined #tripleo18:51
shardyjdob: thanks, I'm adding some notes now18:51
*** nico_auv has joined #tripleo18:53
*** rbrady has quit IRC18:57
*** david-ly_ has joined #tripleo19:00
*** rpothier has quit IRC19:01
*** akrivoka has quit IRC19:01
*** dprince has quit IRC19:03
*** david-ly_ is now known as david-lyle_19:03
*** david-lyle has quit IRC19:03
*** david-lyle_ is now known as david-lyle19:04
*** tzumainn has quit IRC19:05
openstackgerritayoung proposed openstack/tripleo-heat-templates: puppet: run keystone in wsgi  https://review.openstack.org/21317519:06
*** tzumainn has joined #tripleo19:06
openstackgerritJeff Peeler proposed openstack/instack-undercloud: Install local docker registry with puppet  https://review.openstack.org/23823819:06
*** mbound_ has joined #tripleo19:09
*** derekh has quit IRC19:10
*** gfidente is now known as gfidente|afk19:15
*** sthillma has joined #tripleo19:16
*** nico_auv has quit IRC19:29
*** shivrao has joined #tripleo19:36
*** lucasagomes is now known as lucas-dinner19:51
*** sthillma has quit IRC19:57
*** david-lyle has quit IRC19:58
*** david-lyle has joined #tripleo19:58
*** dprince has joined #tripleo19:58
*** panda has quit IRC20:03
*** panda has joined #tripleo20:03
*** Marga_ has quit IRC20:11
*** Marga_ has joined #tripleo20:12
*** olap has joined #tripleo20:13
*** athomas has joined #tripleo20:16
jdobjtomasek: you still around by any chance?20:17
jtomasekjdob: kind of, yeah20:17
jdobwhat version of Heat are you using when you see that OvercloudBlockStorageFlavor param missing with --show-nested?20:18
jtomasekjdob: [root@instack ~]# heat --version20:19
jtomasek0.8.120:19
jtomasekjdob: is that any helpful?20:19
jdobhrm20:20
jdobprobably not; how did you install instack? from the upstream instructions?20:20
jtomasekjdob: http://docs.openstack.org/developer/tripleo-docs/environments/virtual.html20:21
jtomasekjdob: I am on centos20:21
jtomasekjdob: I did the installation about 14 days ago20:22
jdobslagle: the instructions at ^, those install from delorean right? are they the nightly builds or pinned to some point in time20:22
jdobjtomasek: ok, the 14 days might help20:22
jtomasekjdob: I think so20:22
jdoboh oh, I have a better idea20:22
jdobwhere I'm going with this is that I can't reproduce it with a simple template that I thought *should* replicate it20:23
jdobso i'll send you that to run on your instack, but it can wait until tomorrow20:23
jdobsince its like, what, 9pm over there?20:23
jdobunless it's a box you can just give me access to20:23
jdob(can totally wait until tomorrow, that's fine too)20:23
jtomasekjdob: yeah, I can give you access to the box20:24
jdobkk, PM or email me if you have time20:24
slaglejdob: it mostly uses delorean where available20:24
jdobwhich is built nightly right?20:24
jdobor per commit?20:25
*** sthillma has joined #tripleo20:26
slagleper commit20:28
*** jprovazn has quit IRC20:32
*** mcornea has joined #tripleo20:39
*** Marga_ has quit IRC20:42
*** derekh has joined #tripleo20:44
*** derekh has quit IRC20:45
*** sthillma has quit IRC20:45
*** mbound_ has quit IRC20:47
dprincethis passes CI. Could we move forward w/ V3 support for the undercloud here: https://review.openstack.org/#/c/234704/20:50
dprincetime goes on... and TripleO is still using keystone v2!20:50
slagledprince: just to clear the air before...i'm -1 on that until someone says they've tested an update20:55
slagledprince: b/c it was breaking really bad for me earlier, if you check my earlier reviews20:56
dprinceslagle: hmmm. okay. So here is my take. Lets forget keystone updates for the undercloud20:56
dprinceslagle: we have no business doing them20:56
dprinceslagle: simply delete the keystone database and start fresh on each update20:57
dprinceslagle: think of it as a step towards a stateless undercloud...20:57
dprinceslagle: would you buy that?20:57
slagledprince: Hmm. what if you have an overcloud deployed?20:58
slagleyou couldn't just delete the keystone db20:58
dprinceslagle: just keystone20:58
dprinceslagle: no other databases20:58
slaglearen't there users in keystone that Heat needs?20:58
slaglefor deployed stacks and what not?20:58
dprinceslagle: So long as we create equivalent users we'd be fine. We aren't creating undercloud keystone users with Heat anyways20:59
*** Marga_ has joined #tripleo20:59
slagleok. i don't understand how it works then. but if it does, it does :)20:59
slaglei must be mistaken in what i was thinking20:59
dprinceslagle: I haven't tried this mind you20:59
dprinceslagle: but we've got to stop the undercloud state creep21:00
dprinceslagle: we need to maintain some state... yes21:00
*** jayg is now known as jayg|g0n321:00
slaglebut yes...all i want is a way to update with an overcloud already deployed21:00
dprinceslagle: but anything we can ditch we should21:00
slagleand still be able to fully manage that overcloud21:00
slagleor rerun the installer at any time to apply config changes, and have it work21:00
*** mkovacik has joined #tripleo21:03
*** Marga_ has quit IRC21:03
dprinceslagle: is anyone helping shardy on the updates job?21:03
dprinceslagle: CI job?21:03
*** lblanchard has quit IRC21:03
slaglei'm not aware of anyone21:04
slaglebut i'm only aware of what a small number of people are actually working on21:04
dprinceslagle: okay, yeah if updates is the blocking concern we need it automated. Asking for manual testing isn't going to scale well.21:05
dprinceand IMO that job should go upstream first21:05
slagledprince: i'm well aware of that21:07
slagledprince: i can only do what i'm able to do21:07
slaglenot everyone is empowered to do what you are21:07
slagledprince: i'm full aware that manual testing doesn't scale. but thanks for the tip21:07
dprinceslagle: right, I'm just saying if we really want to block this on upgrades... then lets say we are waiting on a CI job, not manual testing21:08
slaglemaking sure that patches don't break our expectations is a perfectly valid -121:08
slaglenot everything is CI'd21:08
slaglemaybe one day it will be. until then, testing patches is ok21:08
slagledprince: ok. i'll -2 the patch and it can be blocked on the CI job and i won't bother with the testing21:08
dprinceslagle: hindsight, landing things like network isolation without CI was a mistake21:09
dprinceslagle: maybe I'll -2 all the network isolation patches up for review then too :)21:09
dprinceslagle: seriously!!!21:10
slaglethat sounds like a fine plan to me21:10
slagleseriously for me too. i have no objection21:10
*** lblanchard has joined #tripleo21:12
openstackgerritDan Prince proposed openstack/instack-undercloud: Install mistral (workflow) in the undercloud  https://review.openstack.org/24850621:14
*** Marga_ has joined #tripleo21:15
*** sthillma has joined #tripleo21:16
*** dprince has quit IRC21:19
*** tzumainn has quit IRC21:19
shardydprince: Hey, if anyone wants to help with update/upgrade CI that would be good21:19
shardyI started looking at it, but just getting stable/liberty CI working was so much work I've kind of lost momentum a bit21:20
shardywhen we have manual update/upgrade steps proven I was going to modify tripleo.sh to optionally install from master, then update to the tripleo-ci repo21:20
*** tzumainn has joined #tripleo21:20
shardythere's some basic missing stuff we've already discussed like wiring repo files into the nodes, getting some stuff running on the overcloud before update/upgrade (marios had a tripleo.sh patch for this)21:21
shardyI'll try to do an etherpad for it tomorrow and we can see if the tasks can be shared about to those interested in helping21:21
*** Marga_ has quit IRC21:23
shardyslagle: I was looking at https://review.openstack.org/#/c/222227/, are you happy for that to land so we can approve the stable backport?21:23
shardyI hesitated because it's been around for a while21:23
shardyslagle: also, I had a question about the "delete all users" thing - do we really delete *all* keystone users?21:24
shardywe don't want to delete all the undercloud keystone users in the heat domain21:24
shardythat would be very bad21:24
openstackgerritJeff Peeler proposed openstack/python-tripleoclient: Add ability to push docker images to registry  https://review.openstack.org/25553521:32
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks  https://review.openstack.org/23542321:33
*** trown|BOS is now known as trown|outttypeww21:35
slagleshardy: openstack user list isn't going to show those users21:37
slagleshardy: but the other issue you point out about manually adding a user it would if they were under v221:38
shardyslagle: Ok, if we're sure then OK - I was concerned because we're using the magic admin token, which bypasses all of keystone's RBAC21:38
openstackgerritJeff Peeler proposed openstack/tripleo-common: Upload docker image files from yaml config file  https://review.openstack.org/24772421:38
slaglewell let me test it21:39
shardyslagle: would it be safer to explicitly pass arguments to ensure e.g only users in the default domain and services project are listed?21:39
slagleshardy: probably yes. sounds like a safer thing to do21:39
slagleactually...21:40
slaglethere is an "openstack user set" which takes a password arg21:40
shardyJust adding --project service would probably do the trick21:40
slaglei didnt see this in keystone cli, and just assumed that using openstack with v2 it wouldnt be there either21:40
shardyah, yeah updating what's there would be even better21:40
slaglei will try that21:41
shardyslagle: thanks, sorry to hold things up21:41
shardyotherwise lgtm21:41
*** Marga_ has joined #tripleo21:51
*** tzumainn has quit IRC21:53
*** tserong has quit IRC21:57
*** Marga_ has quit IRC22:00
*** cwolferh has quit IRC22:03
*** tzumainn has joined #tripleo22:06
*** lblanchard has quit IRC22:10
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Convert JSON generations from bash to python  https://review.openstack.org/25313522:14
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Network Isolation support for containerized compute  https://review.openstack.org/25430422:14
*** bvandenh has quit IRC22:16
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Convert JSON generations from bash to python  https://review.openstack.org/25313522:20
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Network Isolation support for containerized compute  https://review.openstack.org/25430422:21
*** Marga_ has joined #tripleo22:24
*** rasca has quit IRC22:28
*** Marga_ has quit IRC22:28
*** mkovacik has quit IRC22:33
*** rhallisey has quit IRC22:34
*** david-lyle has quit IRC22:34
*** david-ly_ has joined #tripleo22:39
*** patrickeast has quit IRC22:40
*** shardy has quit IRC22:45
*** david-ly_ is now known as david-lyle22:45
*** Marga_ has joined #tripleo22:55
*** yuanying has joined #tripleo22:56
*** Marga_ has quit IRC22:59
*** morazi has quit IRC23:00
*** rebrego has joined #tripleo23:04
*** regebro is now known as Guest2677023:04
*** rebrego is now known as regebro23:04
*** sthillma has quit IRC23:05
*** Marga_ has joined #tripleo23:11
*** thrash is now known as thrash|g0ne23:12
*** Marga_ has quit IRC23:12
*** Marga_ has joined #tripleo23:13
*** Marga_ has quit IRC23:13
*** Marga_ has joined #tripleo23:14
openstackgerritMerged openstack/diskimage-builder: Use ironic-agent for source-repositories  https://review.openstack.org/23308823:19
*** sthillma has joined #tripleo23:36
*** rcernin has joined #tripleo23:37
*** cwolferh has joined #tripleo23:38
*** pradk has quit IRC23:40
*** gfidente|afk is now known as gfidente23:42
*** weshay has quit IRC23:42
*** patrickeast has joined #tripleo23:51
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140823:53
*** rlandy has quit IRC23:54
openstackgerritEmilien Macchi proposed openstack/tripleo-incubator: Add Gnocchi & Aodh passwords  https://review.openstack.org/25559423:57
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm  https://review.openstack.org/24140823:58

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