Tuesday, 2018-10-30

« Monday, 2018-10-29 Index Wednesday, 2018-10-31 »
*** ykarel|away has joined #puppet-openstack02:55
*** ykarel|away is now known as ykarel02:56
*** mschuppert has quit IRC02:57
ykarelEmilienM, hi02:58
ykarelEmilienM, can u please review https://review.openstack.org/#/c/608268/ and https://review.openstack.org/#/c/613510/02:59
ykarelthese are need to unblock fedora job02:59
ykarelokk checked from logs, u are out, will wait03:01
openstackgerritMerged openstack/puppet-nova master: Split off metadata vendordata plugin in own class  https://review.openstack.org/61296403:59
*** ykarel has quit IRC05:20
*** jaosorior has quit IRC05:32
*** jaosorior has joined #puppet-openstack05:32
*** ykarel has joined #puppet-openstack05:42
*** mnaser has quit IRC06:16
*** mnaser has joined #puppet-openstack06:17
*** tonyb has quit IRC06:50
*** mschuppert has joined #puppet-openstack06:52
openstackgerritMartin Schuppert proposed openstack/puppet-nova stable/rocky: Split off metadata vendordata plugin in own class  https://review.openstack.org/61408306:58
*** iurygregory has quit IRC07:00
*** dpawlik has joined #puppet-openstack07:26
*** dpawlik has quit IRC07:27
*** dpawlik has joined #puppet-openstack07:27
*** ykarel is now known as ykarel|lunch07:35
*** shardy has joined #puppet-openstack07:46
*** iurygregory has joined #puppet-openstack08:17
*** ykarel|lunch is now known as ykarel08:28
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration stable/queens: Bump puppetlabs-mysql to 6.0.0  https://review.openstack.org/61414308:31
*** chem has joined #puppet-openstack08:42
*** shardy_ has joined #puppet-openstack08:54
openstackgerritTobias Urdin proposed openstack/puppet-nova stable/queens: Fixup functions for 5.5.7+  https://review.openstack.org/61344608:57
*** jpena|off is now known as jpena08:57
*** agurenko has joined #puppet-openstack09:09
*** derekh has joined #puppet-openstack09:25
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate nova-network code  https://review.openstack.org/61352209:27
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate neutron options and add alternatives  https://review.openstack.org/61352909:27
openstackgerritMerged openstack/puppet-neutron master: Adds new neutron nsx plugin parameters  https://review.openstack.org/60486509:36
zigoEmilienM: It's just cool to say "I work for IBM", then you don't need to tell more, everyone knows what IBM does ... :P09:40
zigomwhahaha: tobias-urdin: EmilienM: mnaser: Which one of you is comming to the summit in Berlin?09:41
iurygregoryyou are from IBM zigo?09:43
zigoiurygregory: Not at all.09:44
iurygregorylol09:44
zigoiurygregory: I've been maintaining OpenStack in Debian since 2011, in various companies.09:44
iurygregoryzigo, got it09:44
*** salmankhan has joined #puppet-openstack10:01
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate nova-network code  https://review.openstack.org/61352210:08
*** jtomasek has quit IRC10:30
*** dpawlik has quit IRC10:37
*** dpawlik has joined #puppet-openstack10:37
openstackgerritTobias Urdin proposed openstack/puppet-nova master: nova::quota fix correct conf group and rename params  https://review.openstack.org/61418310:55
*** agurenko has quit IRC10:57
*** iurygregory is now known as iury|lunch10:59
openstackgerritTobias Urdin proposed openstack/puppet-nova stable/ocata: Fixup functions for 5.5.7+  https://review.openstack.org/61418611:01
openstackgerrityatin proposed openstack/puppet-ironic master: Install python3-ironicclient in Fedora or RedHat > 7  https://review.openstack.org/61366111:02
openstackgerritMerged openstack/puppet-nova stable/rocky: Deprecate fping_path option  https://review.openstack.org/61383411:05
openstackgerrityatin proposed openstack/puppet-openstack-integration master: [DNM] Testing openstacklib change with Fedora current  https://review.openstack.org/61328211:14
openstackgerritMerged openstack/puppet-openstack-guide master: Add puppet-senlin to module list  https://review.openstack.org/61343211:14
openstackgerritMerged openstack/puppet-openstack-guide master: Update module versions and add puppet-senlin  https://review.openstack.org/61343411:18
openstackgerritMerged openstack/puppet-openstack-guide master: Update supported platforms  https://review.openstack.org/61343711:18
openstackgerritMerged openstack/puppet-openstack-guide master: Fix broken CI link on supported platforms page  https://review.openstack.org/61344311:18
openstackgerritJanki Chhatbar proposed openstack/puppet-neutron master: IPv6 support for ODL-OVS deployment  https://review.openstack.org/58474011:19
*** salmankhan has quit IRC11:26
*** morazi has joined #puppet-openstack11:39
*** dtantsur|afk is now known as dtantsur11:50
*** jpena is now known as jpena|lunch11:57
*** jtomasek has joined #puppet-openstack12:01
*** salmankhan has joined #puppet-openstack12:02
tobias-urdinzigo: not me, unfortunately, hopefully the next one12:08
*** gfidente has joined #puppet-openstack12:18
ykarelEmilienM, review please https://review.openstack.org/#/c/608268/3 and https://review.openstack.org/#/c/613510/ to promote fedora packages12:23
EmilienMok12:23
chandankumarEmilienM: on this review https://review.openstack.org/609936 horizon tests are failing constantly I have no idea how to fix it http://logs.openstack.org/36/609936/9/check/puppet-openstack-integration-5-scenario003-tempest-centos-7/bfceca9/logs/testr_results.html.gz12:24
chandankumarafter two rechecks also no success12:24
ykarelEmilienM, Thanks12:24
EmilienMpuppet-openstack-integration-5-scenario003-tempest-centos-7 might be broken12:25
EmilienMthe horizon integration maybe?12:25
ykarelEmilienM, also review other, an attempt to enable ironic,heat,zaqar and mistral in fedora job:- https://review.openstack.org/#/q/topic:enable-py3+status:open12:27
ykareli am testing those https://review.openstack.org/#/c/613282/12:27
*** ykarel is now known as ykarel|afk12:29
*** ade_lee has quit IRC12:32
*** trown|outtypewww is now known as trown12:33
*** radez has joined #puppet-openstack12:37
*** ykarel|afk has quit IRC12:44
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Checkout stable/pike for spec_helper  https://review.openstack.org/61420512:57
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Checkout stable/ocata for spec_helper  https://review.openstack.org/61420612:57
*** ykarel has joined #puppet-openstack13:05
openstackgerritJanki Chhatbar proposed openstack/puppet-neutron master: IPv6 support for ODL-OVS deployment  https://review.openstack.org/58474013:08
openstackgerritEmilien Macchi proposed openstack/puppet-keystone master: Drop undercloud job  https://review.openstack.org/61190813:10
*** jpena|lunch is now known as jpena13:12
*** jtomasek_ has joined #puppet-openstack13:13
chandankumarEmilienM: can we disable that job?13:16
*** jtomasek has quit IRC13:17
openstackgerritTobias Urdin proposed openstack/puppet-nova master: nova::quota fix correct conf group and rename params  https://review.openstack.org/61418313:17
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate nova-network code  https://review.openstack.org/61352213:19
EmilienMtobias-urdin: have you seen failures on puppet-openstack-integration-5-scenario003-tempest-centos-7 ?13:21
EmilienMseems like the job isn't working well today13:21
EmilienMhorizon tempest test13:21
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate neutron options and add alternatives  https://review.openstack.org/61352913:23
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate neutron options and add alternatives  https://review.openstack.org/61352913:23
tobias-urdinEmilienM: are we talking the normal 'Overview not in' when it cannot login? or some timeout?13:25
*** salmankhan has left #puppet-openstack13:25
EmilienMtobias-urdin: yea13:26
EmilienM"normal"13:26
EmilienMok :D13:26
zigoEmilienM: I wonder if we could increase the timeout, as it always does timeout for Debian, your thoughts?13:29
tobias-urdinheh we've had that for months, i never understood what caused it13:34
tobias-urdinsame as we have a gateway timeout for horizon on ubuntu13:34
tobias-urdini tried working on a retry logic in tempest https://review.openstack.org/#/q/topic:horizon-tempest-bug-t+(status:open+OR+status:merged)13:34
tobias-urdinbut never got around to solving it, but yeah, it's somewhere on my list13:34
tobias-urdiniirc ykarel assigned somebody to follow up13:34
ykarelyes he was rdopiera but i didn't follow on the progress13:35
tobias-urdinit's a pain though, scenario003 is very unstable because of horizon primarly13:35
openstackgerritMerged openstack/puppet-openstacklib master: Install python3-openstackclient in Fedora or RedHat > 7  https://review.openstack.org/60826813:47
*** social has quit IRC14:11
*** social has joined #puppet-openstack14:14
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422614:23
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422614:24
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422614:37
*** ade_lee has joined #puppet-openstack14:37
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api  https://review.openstack.org/61423114:39
openstackgerritTobias Urdin proposed openstack/puppet-horizon master: Remove os_any2array function  https://review.openstack.org/61423414:47
openstackgerritMerged openstack/puppet-keystone master: Install python3-keystoneclient in Fedora or RedHat > 7  https://review.openstack.org/61351014:47
*** dtantsur is now known as dtantsur|brb14:50
openstackgerritTobias Urdin proposed openstack/puppet-murano master: Use puppet4 functions-api  https://review.openstack.org/61423814:52
openstackgerritMartin Schuppert proposed openstack/puppet-nova stable/rocky: Split off metadata vendordata plugin in own class  https://review.openstack.org/61408314:55
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api  https://review.openstack.org/61424114:56
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api  https://review.openstack.org/61424615:04
tobias-urdinEmilienM: first attempt at remodeling those functions, there is a lot in puppet-tripleo and some in puppet-pacemaker and puppet-midonet which i haven't done15:05
EmilienMwow nice15:05
mwhahahatobias-urdin: i've got most of the ones in puppet-tripleo. except the two that use lookupvar (which isn't a thing anymore)15:08
mwhahahatobias-urdin: you should write some tests for your functions :D though i'm not sure how to properly mock the fact stuff which is why i haven't migrated those last two functions out puppet-tripleo15:09
mwhahahatobias-urdin: https://review.openstack.org/#/c/613411/2 for reference15:10
tobias-urdinyeah, i was hoping that most already had testing but it didn't15:11
tobias-urdinmy changes is actually pretty minor, it haven't touch the functionality more so it uses the new api15:11
tobias-urdinbut i can look into it, i need to learn more ruby anyway :)15:12
*** iury|lunch is now known as iurygregory15:16
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422615:26
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422615:26
openstackgerritTong Liu proposed openstack/puppet-neutron stable/rocky: Adds new neutron nsx plugin parameters  https://review.openstack.org/61426415:33
*** dpawlik has quit IRC15:43
*** dpawlik has joined #puppet-openstack15:44
*** dpawlik has quit IRC15:49
*** ykarel is now known as ykarel|away15:50
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api  https://review.openstack.org/61424115:52
openstackgerritCarlos Goncalves proposed openstack/puppet-octavia master: WIP: Set sharable controller and DB configurations  https://review.openstack.org/61239515:54
tobias-urdinmwhahaha: reading docs to learn how to stub ruby stuff i saw https://rspec-puppet.com/documentation/functions/#accessing-the-parser-scope16:01
tobias-urdinseems to be something related to lookupvar you are refering to16:01
mwhahahayea i tired it and it doesn't really work in the new stuff16:01
mwhahahabecuse lookupvar goes away and you have to use some other var16:02
* mwhahaha goes and tries to find it16:02
mwhahahathe puppet folks pointed me at https://gist.github.com/hlindberg/8c250c54f5adb76679576ae5cffff86816:02
mwhahahabut still didn't solve my problem16:02
mwhahahaclosure_scope is the way of accessing it in 4.x but i couldn't find a way to mock that correctly16:03
mwhahahahttps://puppet.com/docs/puppet/4.10/functions_ruby_implementation.html#accessing-puppet-variables16:03
tobias-urdinhm16:08
*** jtomasek_ has quit IRC16:12
tobias-urdinmwhahaha: looks like a pain in the ass, this one uses closure_scope and somehow implements there own mock provider in puppet_x folder16:12
tobias-urdinhttps://github.com/simp/pupmod-simp-libkv16:12
mwhahahayea16:13
tobias-urdinmwhahaha: can you have `context` where you run `before` in then have `context` inside that context that also has a `before`?16:18
tobias-urdininception16:18
mwhahahayea16:18
tobias-urdinty16:20
*** dtantsur|brb is now known as dtantsur16:26
*** trown is now known as trown|lunch16:33
*** morazi has quit IRC16:34
*** ykarel has joined #puppet-openstack16:46
*** ykarel|away has quit IRC16:48
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api  https://review.openstack.org/61424616:55
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api  https://review.openstack.org/61424616:56
tobias-urdinmwhahaha: any idea what might be broken in the syntax tests there?17:01
tobias-urdinfrom the patch you abandoned i noticed this was missing https://review.openstack.org/#/c/614206/ that fixed the lint but the syntax jobs are still failing17:01
openstackgerritClark Boylan proposed openstack/puppet-openstack-integration master: Stop running non voting jobs in the gate  https://review.openstack.org/61430517:02
mwhahahatobias-urdin: not sure17:06
mwhahahatobias-urdin: are we manually munging the gemspec in ci?17:09
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Deprecate all paste code  https://review.openstack.org/61344917:09
tobias-urdinmwhahaha: what is that?17:10
mwhahahatrying to figure out if we're messing with the versions in the gemspec or something17:10
mwhahahahttp://logs.openstack.org/06/614206/1/check/puppet-openstack-syntax-5/cb2526f/job-output.txt.gz#_2018-10-30_13_57_40_35412417:10
*** ykarel is now known as ykarel|away17:11
mwhahahasince it's erroring on the version limits17:11
mwhahahahttps://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/run-syntax-tests.yaml#L517:11
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate neutron options and add alternatives  https://review.openstack.org/61352917:12
mwhahahahmm i wonder if we need to be running the ocata version17:13
mwhahahahttp://logs.openstack.org/06/614206/1/check/puppet-openstack-syntax-5/cb2526f/job-output.txt.gz#_2018-10-30_13_57_28_54547817:13
tobias-urdinhm can't we start actually running from the proper branch? was there any reasoning why we are running from master for older versions?17:16
*** ykarel|away has quit IRC17:16
tobias-urdindamn CI is a pain right now...17:16
mwhahahamagical zuul branch matching would be my assumption17:16
*** ykarel|away has joined #puppet-openstack17:26
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/puppet-qdr master: Add a broker connector configuration  https://review.openstack.org/61399217:29
openstackgerritMerged openstack/puppet-mistral master: Install python3-mistralclient in Fedora or RedHat > 7  https://review.openstack.org/61356517:29
*** derekh has quit IRC17:30
*** morazi has joined #puppet-openstack17:34
*** trown|lunch is now known as trown17:50
*** morazi has quit IRC17:50
*** morazi has joined #puppet-openstack17:51
*** shardy_ has quit IRC17:55
*** dtantsur is now known as dtantsur|afk17:57
*** dpawlik has joined #puppet-openstack18:03
*** chandankumar is now known as chkumar|off18:05
*** jpena is now known as jpena|off18:10
*** dpawlik has quit IRC18:20
mnaserzigo: ill be in berlin :)18:22
*** dpawlik has joined #puppet-openstack18:36
*** dpawlik has quit IRC18:40
*** shardy has quit IRC19:06
*** shardy has joined #puppet-openstack19:18
*** ykarel|away has quit IRC19:32
*** ykarel|away has joined #puppet-openstack19:32
*** ykarel|away has quit IRC19:38
*** gfidente has quit IRC19:47
*** shardy has quit IRC20:03
*** social has quit IRC20:05
*** shardy has joined #puppet-openstack20:12
*** rascasoft has quit IRC20:13
*** rnoriega has quit IRC20:13
*** rnoriega has joined #puppet-openstack20:14
*** rascasoft has joined #puppet-openstack20:16
openstackgerritTobias Urdin proposed openstack/puppet-murano master: Use puppet4 functions-api  https://review.openstack.org/61423820:52
*** morazi has quit IRC20:53
tobias-urdinmwhahaha: still around?20:53
mwhahahayea20:53
tobias-urdinthis change https://review.openstack.org/#/c/613510/20:54
tobias-urdinkilled all unit tests20:54
tobias-urdinsince all includes the keystone module and those facts are not set20:54
tobias-urdinwill probably be the same for most other similar changes unless we fix all unit tests20:54
mwhahahaoh damn it20:54
tobias-urdinwe might need to go through facterdb stuff before merging all those20:55
tobias-urdinor do some custom hack in spec_helper for now to make sure we set those in unit tests20:55
mwhahahais it just because those are undefined?20:55
*** shardy has quit IRC20:55
tobias-urdinhttp://logs.openstack.org/22/613522/4/check/puppet-openstack-unit-5.5-centos-7/19aa9e3/job-output.txt.gz#_2018-10-30_16_22_41_12681820:56
mwhahahaor is it just $::os['release']['major'] w/ an undefined access20:56
tobias-urdinEvaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Operator '[]' is not applicable to an Undef Value20:56
tobias-urdin(file: /home/zuul/workspace/spec/fixtures/modules/keystone/manifests/params.pp, line: 6, column: 43)20:56
mwhahahalame20:57
mwhahahalet's back that out20:57
mwhahahaand we need to update the factor db stuff20:57
openstackgerritAlex Schultz proposed openstack/puppet-keystone master: Revert "Install python3-keystoneclient in Fedora or RedHat > 7"  https://review.openstack.org/61435220:58
mwhahahastupid new facts20:58
tobias-urdinthere are about 4-5 that failed to merge, we should perhaps block those until we know more20:59
mwhahahathey probably failed because of keystone :D20:59
tobias-urdinseems like they got blocked by undercloud gate job21:01
tobias-urdinto get forward with the effort perhaps we'll need to use the older facts21:01
tobias-urdinwill probably be some hassle getting it merged in facterdb and publish new release21:02
tobias-urdiniirc i did it a while ago21:02
tobias-urdinyeah https://github.com/camptocamp/facterdb/pull/8821:02
tobias-urdinmight need to pull in facterdb from git, no release since april 201821:05
tobias-urdincentos-7 | Installing facterdb 0.5.221:06
tobias-urdincentos-7 | Installing facter 2.5.121:06
tobias-urdinmwhahaha: feel like we should get it though https://github.com/camptocamp/facterdb/blob/0.5.2/facts/2.5/centos-7-x86_64.facts#L2121:08
* tobias-urdin confused21:08
mwhahahasec21:08
*** trown is now known as trown|outtypewww21:09
tobias-urdinthat file looks sad though, comparing to newer facter versions like in my PR21:12
*** ade_lee has quit IRC21:18
*** salmankhan has joined #puppet-openstack21:20
* tobias-urdin logout21:31
mwhahahatobias-urdin: maybe the failure is because the tests don't use rspec-puppet-facts. we had some missing ones, maybe we need to update them21:50
*** salmankhan has quit IRC22:43
*** tonyb has joined #puppet-openstack23:03
*** salmankhan has joined #puppet-openstack23:09
openstackgerritMerged openstack/puppet-openstack-integration stable/queens: Bump puppetlabs-mysql to 6.0.0  https://review.openstack.org/61414323:31
« Monday, 2018-10-29 Index Wednesday, 2018-10-31 »

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