Friday, 2019-01-25

*** mmethot has quit IRC00:00
*** tdb has quit IRC00:00
*** mmethot has joined #puppet-openstack00:00
*** mmethot has quit IRC00:04
*** mschuppert has quit IRC00:11
*** ade_lee has quit IRC00:16
*** ade_lee has joined #puppet-openstack00:16
*** rcernin has quit IRC00:18
*** rcernin has joined #puppet-openstack00:19
*** markvoelker has quit IRC00:20
*** tdb has joined #puppet-openstack00:36
*** mmethot has joined #puppet-openstack02:16
*** markvoelker has joined #puppet-openstack02:25
openstackgerritMerged openstack/puppet-keystone master: Deprecate keystone::token_driver  https://review.openstack.org/63274902:48
*** markvoelker has quit IRC02:55
*** skramaja has joined #puppet-openstack03:44
*** markvoelker has joined #puppet-openstack03:52
*** ykarel|away has joined #puppet-openstack03:52
*** morazi has quit IRC03:58
openstackgerritzhongshengping proposed openstack/puppet-glance master: Fix lint issue  https://review.openstack.org/63310904:24
openstackgerritzhongshengping proposed openstack/puppet-cinder master: Fix lint issue  https://review.openstack.org/63311004:24
openstackgerritzhongshengping proposed openstack/puppet-nova master: Fix lint issue  https://review.openstack.org/63311204:24
openstackgerritzhongshengping proposed openstack/puppet-murano master: Fix lint issue  https://review.openstack.org/63311104:24
openstackgerritzhongshengping proposed openstack/puppet-ceilometer master: Fix lint issue  https://review.openstack.org/63311304:24
openstackgerritzhongshengping proposed openstack/puppet-heat master: Fix lint issue  https://review.openstack.org/63311404:24
openstackgerritzhongshengping proposed openstack/puppet-barbican master: Fix lint issue  https://review.openstack.org/63311504:24
openstackgerritzhongshengping proposed openstack/puppet-manila master: Fix lint issue  https://review.openstack.org/63311604:24
openstackgerritzhongshengping proposed openstack/puppet-designate master: Fix lint issue  https://review.openstack.org/63311704:24
openstackgerritzhongshengping proposed openstack/puppet-zaqar master: Fix lint issue  https://review.openstack.org/63311804:24
openstackgerritzhongshengping proposed openstack/puppet-freezer master: Fix lint issue  https://review.openstack.org/63311904:24
openstackgerritzhongshengping proposed openstack/puppet-glare master: Fix lint issue  https://review.openstack.org/63312004:24
openstackgerritzhongshengping proposed openstack/puppet-congress master: Fix lint issue  https://review.openstack.org/63312104:24
openstackgerritzhongshengping proposed openstack/puppet-tacker master: Fix lint issue  https://review.openstack.org/63312204:24
openstackgerritzhongshengping proposed openstack/puppet-rally master: Fix lint issue  https://review.openstack.org/63312504:24
openstackgerritzhongshengping proposed openstack/puppet-panko master: Fix lint issue  https://review.openstack.org/63312604:24
openstackgerritzhongshengping proposed openstack/puppet-watcher master: Fix lint issue  https://review.openstack.org/63312404:24
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty master: Fix lint issue  https://review.openstack.org/63312304:24
openstackgerritzhongshengping proposed openstack/puppet-magnum master: Fix lint issue  https://review.openstack.org/63312704:24
openstackgerritzhongshengping proposed openstack/puppet-octavia master: Fix lint issue  https://review.openstack.org/63312804:24
openstackgerritzhongshengping proposed openstack/puppet-trove master: Fix lint issue  https://review.openstack.org/63312904:25
openstackgerritzhongshengping proposed openstack/puppet-neutron master: Fix lint issue  https://review.openstack.org/63313004:25
*** ykarel|away is now known as ykarel04:25
*** markvoelker has quit IRC04:25
openstackgerritzhongshengping proposed openstack/puppet-keystone master: Fix lint issue  https://review.openstack.org/63313904:55
*** markvoelker has joined #puppet-openstack05:22
*** markvoelker has quit IRC05:54
*** ykarel has quit IRC06:02
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/63191106:09
*** ykarel has joined #puppet-openstack06:26
openstackgerritzhongshengping proposed openstack/puppet-murano master: Fix unit test failed  https://review.openstack.org/63314806:43
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Fix unit test failed  https://review.openstack.org/63314906:54
openstackgerritzhongshengping proposed openstack/puppet-murano master: Fix unit test failed  https://review.openstack.org/63314807:10
openstackgerritMerged openstack/puppet-freezer master: Fix lint issue  https://review.openstack.org/63311907:15
openstackgerritMerged openstack/puppet-ceilometer master: Fix lint issue  https://review.openstack.org/63311307:15
openstackgerritMerged openstack/puppet-senlin master: Fix lint issue  https://review.openstack.org/63313507:16
openstackgerritMerged openstack/puppet-barbican master: Fix lint issue  https://review.openstack.org/63311507:16
openstackgerritMerged openstack/puppet-ec2api master: Fix lint issue  https://review.openstack.org/63313707:16
openstackgerritMerged openstack/puppet-vitrage master: Fix lint issue  https://review.openstack.org/63313107:17
openstackgerritMerged openstack/puppet-magnum master: Fix lint issue  https://review.openstack.org/63312707:17
openstackgerritMerged openstack/puppet-cloudkitty master: Fix lint issue  https://review.openstack.org/63312307:17
openstackgerritMerged openstack/puppet-gnocchi master: Fix lint issue  https://review.openstack.org/63313207:18
openstackgerritMerged openstack/puppet-manila master: Fix lint issue  https://review.openstack.org/63311607:18
openstackgerritMerged openstack/puppet-rally master: Fix lint issue  https://review.openstack.org/63312507:18
openstackgerritMerged openstack/puppet-designate master: Fix lint issue  https://review.openstack.org/63311707:20
openstackgerritMerged openstack/puppet-octavia master: Fix lint issue  https://review.openstack.org/63312807:20
openstackgerritMerged openstack/puppet-watcher master: Fix lint issue  https://review.openstack.org/63312407:22
openstackgerritMerged openstack/puppet-aodh master: Fix lint issue  https://review.openstack.org/63313307:23
openstackgerritMerged openstack/puppet-glare master: Fix lint issue  https://review.openstack.org/63312007:24
openstackgerritMerged openstack/puppet-tacker master: Fix lint issue  https://review.openstack.org/63312207:26
*** mschuppert has joined #puppet-openstack07:28
openstackgerritzhongshengping proposed openstack/puppet-murano master: Fix unit test failed  https://review.openstack.org/63314807:30
openstackgerritMerged openstack/puppet-panko master: Fix lint issue  https://review.openstack.org/63312607:34
openstackgerritMerged openstack/puppet-congress master: Fix lint issue  https://review.openstack.org/63312107:35
*** pgaxatte has joined #puppet-openstack07:40
openstackgerritzhongshengping proposed openstack/puppet-murano master: Fix unit test failed  https://review.openstack.org/63314807:41
jaosoriordoes anybody know why the beaker tests are failing?07:49
ykarelyes there is a fix proposed related to puppet 5.507:51
jaosoriorykarel: cool! is it merging yet?07:51
ykareljaosorior, finding linkk07:51
ykarellooks like it merged 13 hours ago07:52
ykarelhttps://review.openstack.org/#/c/632962/07:52
ykarelwhen ur test failed07:52
*** markvoelker has joined #puppet-openstack07:52
jaosoriorykarel: re-ran the tests for this one https://review.openstack.org/#/c/632459/ this morning. And it seems it's still failing.07:52
jaosoriorlet me reabse it to see if that does the trick.07:53
ykarelrecheck should work07:53
jaosoriorykarel: recheck didn't though07:53
jaosoriorykarel: http://logs.openstack.org/59/632459/8/check/puppet-openstack-beaker-centos-7/4b61975/07:54
ykarelubuntu beaker also failing,07:54
ykarelAn error occurred while installing puppet (6.2.0), and Bundler cannot continue.07:55
jaosoriorX_X07:55
ykarelthere are some recent success and failure too: http://zuul.openstack.org/builds?job_name=puppet-openstack-beaker-centos-707:58
ykarellooks like that fix was not complete08:00
jaosorior:(08:01
ykareltobias-urdin, can u check ^^08:02
*** rpittau has joined #puppet-openstack08:10
*** ykarel is now known as ykarel|lunch08:14
tobias-urdinreading backlog08:19
*** markvoelker has quit IRC08:25
*** shardy has joined #puppet-openstack08:25
tobias-urdinsuper weird since it has passed before08:38
tobias-urdinah maybe we are not setting the environment variable in the beaker playbook08:44
openstackgerritMerged openstack/puppet-trove master: Fix lint issue  https://review.openstack.org/63312908:48
*** tosky has joined #puppet-openstack08:49
*** shardy has quit IRC08:50
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Make beaker tests honour puppet_gem_version  https://review.openstack.org/63316208:50
*** jpena|off is now known as jpena08:51
tobias-urdinykarel|lunch: ^08:51
*** shardy has joined #puppet-openstack08:51
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Unpin puppet gem version selection  https://review.openstack.org/63316408:58
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Inherit pyvers from openstacklib::defaults  https://review.openstack.org/63023609:05
tobias-urdin^ needed to fix conflict in this one, so testing the beaker fix there at the same time09:05
*** strigazi has quit IRC09:15
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Fix unit test failed  https://review.openstack.org/63314909:21
*** markvoelker has joined #puppet-openstack09:22
*** derekh has joined #puppet-openstack09:31
*** ykarel|lunch is now known as ykarel09:40
openstackgerritMerged openstack/puppet-keystone master: Fix lint issue  https://review.openstack.org/63313909:48
openstackgerritMerged openstack/puppet-zaqar master: Fix lint issue  https://review.openstack.org/63311809:48
openstackgerritMerged openstack/puppet-mistral master: Fix lint issue  https://review.openstack.org/63313609:49
ykareltobias-urdin, ack09:49
*** rpittau has quit IRC09:51
*** rpittau has joined #puppet-openstack09:53
*** markvoelker has quit IRC09:55
openstackgerritMerged openstack/puppet-glance master: Fix lint issue  https://review.openstack.org/63310909:56
openstackgerritMerged openstack/puppet-heat master: Fix lint issue  https://review.openstack.org/63311410:02
openstackgerritMerged openstack/puppet-cinder master: Fix lint issue  https://review.openstack.org/63311010:05
openstackgerritMerged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/63191110:08
*** shardy has quit IRC10:19
*** shardy has joined #puppet-openstack10:20
*** salmankhan has joined #puppet-openstack10:27
*** salmankhan has quit IRC10:33
*** salmankhan has joined #puppet-openstack10:34
*** markvoelker has joined #puppet-openstack10:52
*** shardy has quit IRC10:55
*** jaosorior has quit IRC11:07
openstackgerritMerged openstack/puppet-neutron master: Fix lint issue  https://review.openstack.org/63313011:09
*** shardy has joined #puppet-openstack11:10
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Remove Ryu workaround  https://review.openstack.org/63318611:14
*** shardy has quit IRC11:15
*** shardy has joined #puppet-openstack11:17
*** shardy has quit IRC11:22
*** markvoelker has quit IRC11:25
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Make beaker tests honour puppet_gem_version  https://review.openstack.org/63316211:31
*** shardy has joined #puppet-openstack11:35
*** shardy has quit IRC11:39
*** shardy has joined #puppet-openstack11:53
*** skramaja has quit IRC12:40
*** jpena is now known as jpena|lunch12:44
*** morazi has joined #puppet-openstack12:52
tobias-urdinykarel: yay the ugly hack can be removed :)13:09
ykareltobias-urdin, ack13:09
*** iurygregory has quit IRC13:11
*** rcernin has quit IRC13:30
*** jpena|lunch is now known as jpena13:49
tobias-urdinmwhahaha: you around and have minute?13:49
tobias-urdinykarel: https://review.openstack.org/#/c/633162/2 confirmed solving the issue (https://review.openstack.org/#/c/630236/ is passing) i'll merge it since it blocks a lot of stuff13:53
ykareltobias-urdin, ack13:54
ykareltobias-urdin, looks like it's broken ubuntu bionic beaker13:54
ykarelThe task includes an option with an undefined variable. The error was: 'puppet_gem_version' is undefined13:54
ykareli see that job was already non-voting and failing but ^^ seems to be with the patch13:55
tobias-urdini'll fix that in a new commit since bionic is broken because of rework to py3 + stein anyway13:55
ykarelack just noticed, so shared13:56
tobias-urdinor maybe not, i'll just fix it i guess13:56
tobias-urdinsaw that a line in my change is not needed since template doesn't build anyway13:56
*** trown|outtypewww is now known as trown13:57
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Make beaker tests honour puppet_gem_version  https://review.openstack.org/63316213:58
*** mmethot has quit IRC14:02
tobias-urdinykarel: thanks for noting that btw14:03
*** mmethot has joined #puppet-openstack14:03
*** mmethot has quit IRC14:28
*** mmethot has joined #puppet-openstack14:29
mwhahahatobias-urdin: sup14:42
*** ekultails has joined #puppet-openstack14:43
tobias-urdinthree things, two of them pretty random14:45
mwhahaharandom, aka normal for openstack :D14:45
tobias-urdin1) i was thinking about the providers in puppet modules that uses openstackclient and just remembered these two that i saw while browsing the specs repo a while ago https://github.com/openstack/puppet-openstack-specs/blob/master/specs/icehouse/use-aviator-in-module-resources.rst https://github.com/openstack/puppet-openstack-specs/blob/master/specs/juno/use-openstackclient-in-aviator-resource14:46
tobias-urdins.rst14:46
tobias-urdindo you know if there ever was any discussions about moving providers to ruby-openstack (which isn't really maintained), aviator (which is dead), fog (which seems to be in well shape)?14:46
tobias-urdinor just plain HTTP calls or w/e14:46
tobias-urdinwhile it's aloooooooooooooooot of work, perhaps there was or could be a long time goal14:47
mwhahahait was a topic long ago, nothing came of it due to lack of ruby support for the apis14:47
tobias-urdinit's pretty random because i just got thinking about it while having to dive by right now bad ruby-fu into this https://github.com/dgolja/golja-influxdb/pull/7514:48
tobias-urdinsecond thing 2) i started this one https://review.openstack.org/#/c/630714/ i was hoping it was gonna go smooth but i've been researching a little and i'm starting to think14:49
tobias-urdinthat we need to move to a single approach like reading the openrc file and requiring deployments to have one before providers is used, like openstack-ansible reads clouds.yaml14:49
tobias-urdinor to be more precise the openstack ansible module reads that14:49
tobias-urdinand 3) was https://review.openstack.org/#/c/633162/ but saw now that you +2'd it ^^14:50
tobias-urdinmwhahaha: my first approach to #2 was saving admin password in keystone.conf which seems like a huge security risk and since puppet runs as root it should instead only read a file owned14:51
tobias-urdinby root with the appropriate permissions to make the access to that file very strict14:52
mwhahahawell we don't use an openrc file in tripleo14:52
mwhahahawe always set env vars or config values14:52
mwhahahaso not sure if we should require that14:53
mwhahahasome operators get mad about leaving credentials around in random places :D14:53
tobias-urdinyeah, it works very well for services that have keystone_authtoken section so that's probably not something we need to change, but not should how we should do with keystone after it's bootstrapped14:53
tobias-urdinthat english... not sure how we should do after keystone is bootstrapped14:54
mwhahahai didn't think we kept using the authtoken14:54
mwhahahai need to refresh my memory on what we actually do14:54
tobias-urdinthe providers is like being lost in amazonas, you always have to track how things work and most of the stuff isn't covered by testing even though some modules have pretty decent unit tests14:56
mwhahahapretty much14:56
tobias-urdinhttps://github.com/fog/fog-openstack seems well maintained and we could probably hook that into a base provider in openstacklib, require that gem from there, then have all providers just use that SDK14:57
tobias-urdinwe don't have any advanced openstack providers, even though I think that sdk is probably missing some stuff we have14:57
tobias-urdinpuppet is under so much changes it's crazy, did you know they even started a new "resource api" that'll replace the old providers format?14:59
tobias-urdinhttps://puppet.com/docs/puppet/6.0/about_the_resource_api.html14:59
tobias-urdinimagine having the providers just do basic stuff like http://paste.openstack.org/show/743474/ ... flavor.create etc, would be so much more satisfying15:02
mwhahahayea would be nice if it works15:05
mwhahahabut i think fog has a lot of random ruby version dependencies15:05
mwhahahawhich is what we keep hitting when they bump versions :D15:05
tobias-urdinnot that bad though http://paste.openstack.org/show/743475/15:09
tobias-urdinwe could probably pin versions in openstacklib15:09
tobias-urdinatleast i can dream about it :D15:10
mwhahahaindeed :D15:12
*** ykarel is now known as ykarel|away15:22
*** ykarel|away has quit IRC15:29
*** bnemec has quit IRC15:43
*** chem has quit IRC15:48
*** chem has joined #puppet-openstack15:49
*** ykarel|away has joined #puppet-openstack15:52
openstackgerritTobias Urdin proposed openstack/puppet-glance master: Inherit pyvers from openstacklib::defaults  https://review.openstack.org/63021815:54
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Inherit pyvers from openstacklib::defaults  https://review.openstack.org/63163215:55
*** pgaxatte has quit IRC15:57
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Inherit pyvers from openstacklib::defaults  https://review.openstack.org/63021315:58
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Inherit pyvers from openstacklib::defaults  https://review.openstack.org/63021215:58
openstackgerritTobias Urdin proposed openstack/puppet-placement master: Inherit pyvers from openstacklib::defaults  https://review.openstack.org/63163615:59
openstackgerritMerged openstack/puppet-murano master: Fix unit test failed  https://review.openstack.org/63314816:15
toskytobias-urdin: hi! Do you thin kthat the failure of puppet-openstack-beaker-centos-7 here http://review.openstack.org/583881 is really linked to the change in the patch?16:32
tobias-urdintosky: yes, but I missed one part, so if you change your patch to depend on https://review.openstack.org/#/c/633162/ it will pass16:35
toskytobias-urdin: oh, sure, thanks16:36
openstackgerritLuigi Toscano proposed openstack/puppet-sahara master: Deploy Sahara with unversioned endpoints  https://review.openstack.org/58388116:37
*** chem has quit IRC16:48
*** chem has joined #puppet-openstack16:50
*** rpittau has quit IRC16:54
tobias-urdinEmilienM: could you +w https://review.openstack.org/#/c/633162/ to unblock17:01
* tobias-urdin takes weekend17:02
*** derekh has quit IRC17:05
*** mmethot_ has joined #puppet-openstack17:11
*** mmethot has quit IRC17:14
*** mmethot_ has quit IRC17:34
*** mmethot_ has joined #puppet-openstack17:35
*** mmethot_ has quit IRC17:36
*** jpena is now known as jpena|off17:44
*** chem has quit IRC17:48
*** chem has joined #puppet-openstack17:49
*** ykarel|away has quit IRC18:02
*** shardy has quit IRC18:06
openstackgerritMerged openstack/puppet-ironic master: Fix unit test failed  https://review.openstack.org/63314918:27
*** trown is now known as trown|lunch18:30
EmilienMtobias-urdin: ok18:39
*** chem has quit IRC18:49
*** chem has joined #puppet-openstack18:50
openstackgerritMerged openstack/puppet-nova master: Fix lint issue  https://review.openstack.org/63311219:00
*** bnemec has joined #puppet-openstack19:02
*** bnemec is now known as beekneemech19:11
*** beekneemech has quit IRC19:42
*** mmethot has joined #puppet-openstack19:49
*** trown|lunch is now known as trown20:01
openstackgerritMerged openstack/puppet-openstack-integration master: Make beaker tests honour puppet_gem_version  https://review.openstack.org/63316220:03
*** mmethot has quit IRC20:29
*** mmethot has joined #puppet-openstack20:31
*** salmankhan has quit IRC20:41
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Unpin puppet gem version selection  https://review.openstack.org/63316420:54
*** morazi has quit IRC21:19
*** trown is now known as trown|outtypewww21:40
*** ekultails has quit IRC21:41
openstackgerritMerged openstack/puppet-octavia master: update apache.pp [oportunity to opportunity]  https://review.openstack.org/62786422:06
*** chem has quit IRC22:51
*** chem has joined #puppet-openstack22:52
openstackgerritMerged openstack/puppet-openstack-integration master: Set selective rspec-puppet versions  https://review.openstack.org/61891923:09
*** salmankhan has joined #puppet-openstack23:10
*** bnemec has joined #puppet-openstack23:12
*** salmankhan has quit IRC23:14
*** mmethot has quit IRC23:32
*** mmethot has joined #puppet-openstack23:32
*** mmethot has quit IRC23:33
*** mmethot has joined #puppet-openstack23:33
*** salmankhan has joined #puppet-openstack23:34
*** mmethot has quit IRC23:34
*** mmethot has joined #puppet-openstack23:35
*** salmankhan1 has joined #puppet-openstack23:35
*** salmankhan has quit IRC23:38
*** salmankhan1 is now known as salmankhan23:39
*** mmethot has quit IRC23:41
*** mmethot has joined #puppet-openstack23:42
*** mmethot has quit IRC23:43
*** bnemec has quit IRC23:43
*** mmethot has joined #puppet-openstack23:44
*** bnemec has joined #puppet-openstack23:44
*** mmethot has quit IRC23:44
*** salmankhan has quit IRC23:45
*** mmethot has joined #puppet-openstack23:45
*** salmankhan has joined #puppet-openstack23:45
*** chem has quit IRC23:52
*** chem has joined #puppet-openstack23:53
*** bnemec has quit IRC23:59
*** bnemec has joined #puppet-openstack23:59

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