*** openstackgerrit has joined #puppet-openstack | 00:29 | |
openstackgerrit | zhongshengping proposed openstack/puppet-placement master: Prepare Stein M2 https://review.openstack.org/629709 | 00:29 |
---|---|---|
openstackgerrit | zhongshengping proposed openstack/puppet-senlin master: Prepare Stein M2 https://review.openstack.org/629956 | 00:31 |
*** openstackgerrit has quit IRC | 00:34 | |
*** tosky has quit IRC | 01:05 | |
*** markvoelker has joined #puppet-openstack | 01:15 | |
*** openstackgerrit has joined #puppet-openstack | 01:21 | |
openstackgerrit | Merged openstack/puppet-ovn master: Prepare Stein M2 https://review.openstack.org/629712 | 01:21 |
openstackgerrit | Merged openstack/puppet-rally master: Prepare Stein M2 https://review.openstack.org/629714 | 01:22 |
openstackgerrit | 98k proposed openstack/puppet-nova master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629968 | 02:35 |
*** markvoelker has quit IRC | 02:42 | |
openstackgerrit | 98k proposed openstack/puppet-zaqar master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629971 | 02:45 |
openstackgerrit | 98k proposed openstack/puppet-watcher master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629972 | 02:46 |
openstackgerrit | 98k proposed openstack/puppet-vswitch master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629973 | 02:47 |
openstackgerrit | 98k proposed openstack/puppet-vitrage master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629974 | 02:48 |
openstackgerrit | 98k proposed openstack/puppet-trove master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629976 | 02:50 |
openstackgerrit | 98k proposed openstack/puppet-tempest master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629978 | 02:51 |
*** markvoelker has joined #puppet-openstack | 02:52 | |
openstackgerrit | 98k proposed openstack/puppet-tacker master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629979 | 02:52 |
openstackgerrit | 98k proposed openstack/puppet-swift master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629980 | 02:53 |
openstackgerrit | 98k proposed openstack/puppet-senlin master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629981 | 02:55 |
openstackgerrit | 98k proposed openstack/puppet-sahara master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629982 | 02:56 |
openstackgerrit | 98k proposed openstack/puppet-qdr master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629984 | 02:58 |
openstackgerrit | 98k proposed openstack/puppet-placement master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629985 | 02:59 |
openstackgerrit | 98k proposed openstack/puppet-panko master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629986 | 03:00 |
openstackgerrit | 98k proposed openstack/puppet-ovn master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629988 | 03:02 |
openstackgerrit | 98k proposed openstack/puppet-oslo master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629989 | 03:03 |
openstackgerrit | 98k proposed openstack/puppet-openstacklib master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629991 | 03:04 |
openstackgerrit | 98k proposed openstack/puppet-openstack_spec_helper master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629992 | 03:05 |
openstackgerrit | 98k proposed openstack/puppet-openstack_extras master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629993 | 03:05 |
openstackgerrit | Merged openstack/puppet-mistral master: Prepare Stein M2 https://review.openstack.org/629724 | 03:07 |
openstackgerrit | 98k proposed openstack/puppet-openstack-specs master: Add upper-constraints.txt to releasenotes tox environment https://review.openstack.org/629994 | 03:07 |
openstackgerrit | Merged openstack/puppet-swift master: Prepare Stein M2 https://review.openstack.org/629731 | 03:08 |
openstackgerrit | Merged openstack/puppet-aodh master: Prepare Stein M2 https://review.openstack.org/629718 | 03:08 |
openstackgerrit | Merged openstack/puppet-barbican master: Prepare Stein M2 https://review.openstack.org/629726 | 03:08 |
openstackgerrit | Merged openstack/puppet-monasca master: Prepare Stein M2 https://review.openstack.org/629704 | 03:08 |
openstackgerrit | Merged openstack/puppet-tempest master: Prepare Stein M2 https://review.openstack.org/629710 | 03:08 |
openstackgerrit | Merged openstack/puppet-vitrage master: Prepare Stein M2 https://review.openstack.org/629728 | 03:09 |
openstackgerrit | Merged openstack/puppet-manila master: Prepare Stein M2 https://review.openstack.org/629722 | 03:09 |
openstackgerrit | Merged openstack/puppet-vswitch master: Prepare Stein M2 https://review.openstack.org/629715 | 03:09 |
openstackgerrit | 98k proposed openstack/puppet-octavia master: Add doc/requirements.txt to docs tox environment https://review.openstack.org/629995 | 03:09 |
openstackgerrit | Merged openstack/puppet-nova master: Prepare Stein M2 https://review.openstack.org/629720 | 03:09 |
openstackgerrit | Merged openstack/puppet-horizon master: Prepare Stein M2 https://review.openstack.org/629717 | 03:09 |
openstackgerrit | Merged openstack/puppet-glare master: Prepare Stein M2 https://review.openstack.org/629735 | 03:09 |
openstackgerrit | Merged openstack/puppet-watcher master: Prepare Stein M2 https://review.openstack.org/629738 | 03:09 |
openstackgerrit | Merged openstack/puppet-cloudkitty master: Prepare Stein M2 https://review.openstack.org/629703 | 03:10 |
openstackgerrit | Merged openstack/puppet-trove master: Prepare Stein M2 https://review.openstack.org/629737 | 03:10 |
openstackgerrit | 98k proposed openstack/puppet-neutron master: Add doc/requirements.txt to docs tox environment https://review.openstack.org/629996 | 03:10 |
openstackgerrit | Merged openstack/puppet-neutron master: Prepare Stein M2 https://review.openstack.org/629736 | 03:10 |
openstackgerrit | Merged openstack/puppet-openstack_extras master: Prepare Stein M2 https://review.openstack.org/629719 | 03:10 |
openstackgerrit | Merged openstack/puppet-cinder master: Prepare Stein M2 https://review.openstack.org/629702 | 03:10 |
openstackgerrit | Merged openstack/puppet-murano master: Prepare Stein M2 https://review.openstack.org/629725 | 03:10 |
openstackgerrit | Merged openstack/puppet-ec2api master: Prepare Stein M2 https://review.openstack.org/629733 | 03:10 |
openstackgerrit | Merged openstack/puppet-oslo master: Prepare Stein M2 https://review.openstack.org/629723 | 03:11 |
openstackgerrit | Merged openstack/puppet-panko master: Prepare Stein M2 https://review.openstack.org/629706 | 03:11 |
openstackgerrit | 98k proposed openstack/puppet-murano master: Add doc/requirements.txt to docs tox environment https://review.openstack.org/629997 | 03:11 |
openstackgerrit | Merged openstack/puppet-gnocchi master: Prepare Stein M2 https://review.openstack.org/629698 | 03:11 |
openstackgerrit | Merged openstack/puppet-designate master: Prepare Stein M2 https://review.openstack.org/629727 | 03:11 |
openstackgerrit | Merged openstack/puppet-keystone master: Prepare Stein M2 https://review.openstack.org/629734 | 03:12 |
openstackgerrit | Merged openstack/puppet-ceilometer master: Prepare Stein M2 https://review.openstack.org/629716 | 03:12 |
openstackgerrit | 98k proposed openstack/puppet-monasca master: Add doc/requirements.txt to docs tox environment https://review.openstack.org/629998 | 03:12 |
openstackgerrit | Merged openstack/puppet-congress master: Prepare Stein M2 https://review.openstack.org/629708 | 03:12 |
openstackgerrit | Merged openstack/puppet-heat master: Prepare Stein M2 https://review.openstack.org/629730 | 03:12 |
openstackgerrit | Merged openstack/puppet-tacker master: Prepare Stein M2 https://review.openstack.org/629729 | 03:12 |
openstackgerrit | Merged openstack/puppet-qdr master: Prepare Stein M2 https://review.openstack.org/629711 | 03:13 |
openstackgerrit | 98k proposed openstack/puppet-mistral master: Add doc/requirements.txt to docs tox environment https://review.openstack.org/629999 | 03:13 |
*** vnogin has joined #puppet-openstack | 03:14 | |
openstackgerrit | Merged openstack/puppet-freezer master: Prepare Stein M2 https://review.openstack.org/629713 | 03:14 |
openstackgerrit | Merged openstack/puppet-octavia master: Prepare Stein M2 https://review.openstack.org/629721 | 03:15 |
openstackgerrit | 98k proposed openstack/puppet-manila master: Add doc/requirements.txt to docs tox environment https://review.openstack.org/630000 | 03:15 |
openstackgerrit | Merged openstack/puppet-zaqar master: Prepare Stein M2 https://review.openstack.org/629707 | 03:15 |
openstackgerrit | 98k proposed openstack/puppet-magnum master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630001 | 03:16 |
openstackgerrit | Merged openstack/puppet-glance master: Prepare Stein M2 https://review.openstack.org/629700 | 03:16 |
openstackgerrit | Merged openstack/puppet-openstacklib master: Prepare Stein M2 https://review.openstack.org/629732 | 03:16 |
openstackgerrit | 98k proposed openstack/puppet-keystone master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630002 | 03:17 |
openstackgerrit | 98k proposed openstack/puppet-ironic master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630003 | 03:17 |
openstackgerrit | Merged openstack/puppet-sahara master: Prepare Stein M2 https://review.openstack.org/629705 | 03:18 |
*** vnogin has quit IRC | 03:18 | |
openstackgerrit | 98k proposed openstack/puppet-horizon master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630004 | 03:18 |
openstackgerrit | Merged openstack/puppet-magnum master: Prepare Stein M2 https://review.openstack.org/629699 | 03:18 |
openstackgerrit | 98k proposed openstack/puppet-heat master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630006 | 03:19 |
openstackgerrit | 98k proposed openstack/puppet-gnocchi master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630007 | 03:20 |
openstackgerrit | 98k proposed openstack/puppet-glare master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630008 | 03:21 |
openstackgerrit | 98k proposed openstack/puppet-glance master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630009 | 03:21 |
openstackgerrit | 98k proposed openstack/puppet-freezer master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630010 | 03:22 |
openstackgerrit | 98k proposed openstack/puppet-ec2api master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630012 | 03:23 |
openstackgerrit | 98k proposed openstack/puppet-designate master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630013 | 03:25 |
openstackgerrit | 98k proposed openstack/puppet-congress master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630014 | 03:25 |
openstackgerrit | 98k proposed openstack/puppet-cloudkitty master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630015 | 03:26 |
openstackgerrit | 98k proposed openstack/puppet-cinder master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630016 | 03:27 |
openstackgerrit | 98k proposed openstack/puppet-ceph master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630017 | 03:28 |
*** markvoelker has quit IRC | 03:28 | |
openstackgerrit | 98k proposed openstack/puppet-ceilometer master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630018 | 03:29 |
openstackgerrit | 98k proposed openstack/puppet-barbican master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630019 | 03:29 |
openstackgerrit | 98k proposed openstack/puppet-aodh master: Add doc/requirements.txt to releasenotes tox environment https://review.openstack.org/630020 | 03:30 |
openstackgerrit | Merged openstack/puppet-placement master: Prepare Stein M2 https://review.openstack.org/629709 | 03:41 |
openstackgerrit | Merged openstack/puppet-senlin master: Prepare Stein M2 https://review.openstack.org/629956 | 03:46 |
*** ykarel|away has joined #puppet-openstack | 03:46 | |
*** markvoelker has joined #puppet-openstack | 03:48 | |
*** skramaja has joined #puppet-openstack | 03:59 | |
*** ykarel|away has quit IRC | 04:07 | |
*** ykarel has joined #puppet-openstack | 04:15 | |
*** ykarel_ has joined #puppet-openstack | 04:18 | |
*** ykarel has quit IRC | 04:21 | |
*** markvoelker has quit IRC | 04:26 | |
*** ykarel_ has quit IRC | 04:28 | |
*** ykarel has joined #puppet-openstack | 04:32 | |
*** ykarel has quit IRC | 04:39 | |
*** ykarel has joined #puppet-openstack | 04:42 | |
*** markvoelker has joined #puppet-openstack | 05:05 | |
*** ykarel has quit IRC | 05:05 | |
*** markvoelker has quit IRC | 05:11 | |
*** markvoelker has joined #puppet-openstack | 05:11 | |
*** markvoelker has quit IRC | 05:15 | |
*** markvoelker has joined #puppet-openstack | 05:16 | |
*** ykarel has joined #puppet-openstack | 05:26 | |
*** rcernin has quit IRC | 05:58 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/611996 | 06:02 |
*** rcernin has joined #puppet-openstack | 06:04 | |
*** markvoelker has quit IRC | 06:20 | |
*** markvoelker has joined #puppet-openstack | 06:31 | |
*** markvoelker has quit IRC | 06:36 | |
*** markvoelker has joined #puppet-openstack | 07:02 | |
openstackgerrit | Pranali Deore proposed openstack/puppet-glance master: Add missing glance cinder store settings https://review.openstack.org/626847 | 07:03 |
*** rcernin has quit IRC | 07:06 | |
*** markvoelker has quit IRC | 07:07 | |
*** markvoelker has joined #puppet-openstack | 07:08 | |
*** markvoelker has quit IRC | 07:13 | |
*** vnogin has joined #puppet-openstack | 07:14 | |
*** gfidente has joined #puppet-openstack | 07:15 | |
*** vnogin has quit IRC | 07:18 | |
*** mschuppert has joined #puppet-openstack | 07:24 | |
openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM][all scenarios] Test glance without SSL, rest with SSL https://review.openstack.org/620561 | 07:34 |
*** ykarel is now known as ykarel|lunch | 07:34 | |
*** dpawlik has joined #puppet-openstack | 07:47 | |
*** rpittau has joined #puppet-openstack | 07:48 | |
*** markvoelker has joined #puppet-openstack | 08:16 | |
*** ykarel|lunch is now known as ykarel | 08:23 | |
*** markvoelker has quit IRC | 08:23 | |
*** tosky has joined #puppet-openstack | 08:40 | |
*** jpena|off is now known as jpena | 08:54 | |
*** chem has joined #puppet-openstack | 09:03 | |
*** bandini has quit IRC | 09:07 | |
lyarwood | https://review.openstack.org/#/q/topic:ubuntu-py3+(status:open+OR+status:merged) - is anyone able to take a look at the first three changes here so we can start fixing this with https://review.openstack.org/#/c/629450/ | 09:18 |
*** shardy has joined #puppet-openstack | 09:20 | |
* lyarwood rebuilds his test env again with this | 09:21 | |
lyarwood | on the plus side this is refreshing my Ubuntu knowledge after almost a decade ^_^ | 09:21 |
*** gfidente has quit IRC | 09:22 | |
*** rascasoft has joined #puppet-openstack | 09:23 | |
*** gfidente has joined #puppet-openstack | 09:27 | |
openstackgerrit | Andriy Shevchenko proposed openstack/puppet-openstack-guide master: Updatae home-page https://review.openstack.org/630102 | 09:48 |
*** derekh has joined #puppet-openstack | 09:48 | |
*** jtomasek has quit IRC | 10:03 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Cap puppetlabs-mysql to 6.x release https://review.openstack.org/630121 | 10:11 |
*** shardy has quit IRC | 10:12 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/611996 | 10:15 |
*** shardy has joined #puppet-openstack | 10:15 | |
*** skramaja has quit IRC | 10:25 | |
*** shardy_ has joined #puppet-openstack | 10:26 | |
*** shardy has quit IRC | 10:29 | |
tobias-urdin | iurygregory: you around? :) could you review https://review.openstack.org/#/c/629509/ | 10:31 |
iurygregory | tobias-urdin, doing now o/ | 10:31 |
tobias-urdin | ty | 10:32 |
iurygregory | yw | 10:33 |
tobias-urdin | iurygregory: could you check the msgs from lyarwood above, so he can get unblocked on his placement work and we can start working on ubuntu stein + py3 | 10:34 |
iurygregory | tobias-urdin, sure | 10:34 |
tobias-urdin | ty again :D | 10:35 |
iurygregory | LMAO "his will break the world since all stein packages in ubuntu is py3." | 10:35 |
iurygregory | ops forgot the T | 10:35 |
tobias-urdin | lyarwood: to get the ubuntu work rolling we need to work on the "pyvers in openstacklib::default which needs to be inherited by the <module>::params classes" that we talked about earlier | 10:36 |
*** shardy_ has quit IRC | 10:36 | |
*** shardy has joined #puppet-openstack | 10:49 | |
*** ykarel_ has joined #puppet-openstack | 10:51 | |
*** ykarel has quit IRC | 10:54 | |
*** gfidente has quit IRC | 10:57 | |
*** shardy has quit IRC | 10:58 | |
*** ykarel has joined #puppet-openstack | 11:02 | |
lyarwood | tobias-urdin: ack sorry was on a call, planning to work on that now once I've got this local env up | 11:02 |
*** ykarel_ has quit IRC | 11:03 | |
lyarwood | iurygregory: thanks for acking that and FWIW these were tobias-urdin's words not mine ;) | 11:03 |
*** gfidente has joined #puppet-openstack | 11:03 | |
tobias-urdin | hehe | 11:05 |
openstackgerrit | Merged openstack/puppet-openstack_spec_helper master: Workaround for installing correct version of ruby bundler https://review.openstack.org/629509 | 11:09 |
iurygregory | lyarwood, LOL | 11:21 |
*** dtantsur|afk is now known as dtantsur | 11:26 | |
*** morazi has quit IRC | 11:27 | |
*** vnogin has joined #puppet-openstack | 11:31 | |
*** vnogin has quit IRC | 11:41 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Set Ubuntu jobs to non-voting https://review.openstack.org/628737 | 11:45 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-openstacklib master: DNM: Test setting pyvers in defaults https://review.openstack.org/629450 | 12:07 |
lyarwood | tobias-urdin: so even with https://review.openstack.org/#/c/629450/2/manifests/defaults.pp I'm still seeing pyvers overwritten in modules like nova | 12:27 |
tobias-urdin | yeah, what we need to do is change nova::params to not "include" openstacklib::defaults but to have nova::params inherit openstacklib::defaults, the "pyvers" variables in nova::params will still override the openstacklib::defaults so after that we need to *remove* the pyvers variables from being explicitly defined in nova::params | 12:30 |
tobias-urdin | the good thing is that we won't break centos while doing it since we can remove explicitly defined stuff in nova::params after fixing openstacklib::defaults and changing it to inherits | 12:30 |
tobias-urdin | (and debian) | 12:30 |
tobias-urdin | we probably need some input from ykarel and "check experimental" for fedora/py3 support before merging the changes as well | 12:31 |
lyarwood | kk, I'll hack on that now | 12:32 |
* ykarel reading back | 12:32 | |
ykarel | tobias-urdin, ok to run fedora jobs, it's check rdo experimental | 12:34 |
*** morazi has joined #puppet-openstack | 12:35 | |
tobias-urdin | ykarel: ack | 12:38 |
*** jpena is now known as jpena|lunch | 12:38 | |
*** rpittau is now known as rpittau|lunch | 12:46 | |
*** dtantsur is now known as dtantsur|brb | 12:59 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-openstacklib master: WIP Set pyvers in defaults https://review.openstack.org/629450 | 13:05 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-nova master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630212 | 13:05 |
tobias-urdin | iurygregory: EmilienM fixed https://review.openstack.org/#/c/611996/ and proposed to not cause failures https://review.openstack.org/#/c/630121/ we recently merged https://review.openstack.org/#/c/616222/ that increased puppetlabs-mysql, but it seems like it was increased to a too high one | 13:05 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-neutron master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630213 | 13:05 |
lyarwood | tobias-urdin: ^ so basically this is what you're looking for right? This works for me locally now with 18.04 | 13:06 |
*** vnogin has joined #puppet-openstack | 13:06 | |
tobias-urdin | lyarwood: yes! :) | 13:06 |
lyarwood | tobias-urdin: ack thanks, I'll sort the rest out this afternoon | 13:07 |
tobias-urdin | lyarwood: the only one that i can think of needing something special is puppet-gnocchi https://review.openstack.org/#/c/624802/ | 13:10 |
tobias-urdin | hopefully the debian introduction with py3 support cleared out most of the stuff required | 13:10 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-aodh master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630214 | 13:13 |
*** mmethot has quit IRC | 13:13 | |
lyarwood | tobias-urdin: kk | 13:13 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-barbican master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630215 | 13:15 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-cinder master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630217 | 13:17 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-glance master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630218 | 13:18 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-heat master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630220 | 13:19 |
lyarwood | ah my bad, lint isn't happy | 13:20 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630222 | 13:22 |
lyarwood | http://logs.openstack.org/12/630212/1/check/puppet-openstack-lint/8de4320/job-output.txt.gz#_2019-01-11_13_12_33_370825 - so a WARNING is enough to knock over the lint job? | 13:25 |
lyarwood | tobias-urdin: ^ | 13:25 |
iurygregory | tobias-urdin, ack will look at the patches | 13:27 |
*** vnogin has quit IRC | 13:27 | |
tobias-urdin | lyarwood: checking | 13:31 |
tobias-urdin | lyarwood: hm, not liking those warnings, maybe it's bad praxis to actually do that | 13:32 |
tobias-urdin | lyarwood: yes we set it to fail on warnings https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/rake_tasks.rb#L19 | 13:33 |
tobias-urdin | maybe it was a bad idea, I assume it's against puppet policy to have dependencies between modules in that way | 13:34 |
tobias-urdin | sorry i never actually checked lint on my poc code | 13:34 |
lyarwood | np np | 13:36 |
tobias-urdin | lyarwood: https://puppet.com/docs/puppet/6.1/style_guide.html#class-inheritance | 13:37 |
tobias-urdin | i guess there isn't much we can do then but to keep the original approach :( | 13:38 |
tobias-urdin | i was probably assuming if was ok because the include statement was ok | 13:39 |
*** janki has joined #puppet-openstack | 13:39 | |
*** janki has quit IRC | 13:39 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-nova master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630212 | 13:40 |
lyarwood | tobias-urdin: ^ that should actually work | 13:40 |
lyarwood | tobias-urdin: setting the local pyvers to ::openstacklib::defaults::pyvers | 13:41 |
lyarwood | appears to locally at least | 13:41 |
tobias-urdin | yeah that should work, EmilienM any feedback on ^ is it bad? can't be worse since we have the include there anyways | 13:42 |
EmilienM | tobias-urdin: one sec, let me look | 13:44 |
*** jpena|lunch is now known as jpena | 13:44 | |
EmilienM | tobias-urdin: it sounds fine to me | 13:46 |
lyarwood | thanks EmilienM | 13:46 |
*** rpittau|lunch is now known as rpittau | 13:46 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-aodh master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630214 | 13:47 |
tobias-urdin | yeah that's probably the approach we need to do, doesn't complain on lint and doesn't cause any issues | 13:48 |
tobias-urdin | and doesn't change from the concept today where we include it anyways | 13:48 |
tobias-urdin | what i dont like though is the naming of "pyver3" not that it matter :p | 13:48 |
tobias-urdin | let's go with that then, see how it looks when we get the first scenario passing, how many patches is required | 13:49 |
tobias-urdin | lyarwood: ^ | 13:49 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-barbican master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630215 | 13:50 |
lyarwood | tobias-urdin: ack | 13:50 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-cinder master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630217 | 13:52 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-glance master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630218 | 13:55 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-heat master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630220 | 13:56 |
*** shardy has joined #puppet-openstack | 13:57 | |
openstackgerrit | Merged openstack/puppet-openstacklib master: Cap puppetlabs-mysql to 6.x release https://review.openstack.org/630121 | 14:00 |
tobias-urdin | lyarwood: commented on https://review.openstack.org/#/c/630218/2 | 14:01 |
lyarwood | ah sorry | 14:01 |
lyarwood | that was stupid | 14:01 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-glance master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630218 | 14:02 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630213 | 14:03 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-keystone master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630236 | 14:05 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-tempest master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630237 | 14:09 |
*** jtomasek has joined #puppet-openstack | 14:10 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-nova master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630212 | 14:13 |
*** ykarel is now known as ykarelaway | 14:27 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630222 | 14:32 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630222 | 14:33 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack_spec_helper stable/rocky: Workaround for installing correct version of ruby bundler https://review.openstack.org/630243 | 14:33 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-openstacklib master: Set pyvers in defaults https://review.openstack.org/629450 | 14:36 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-aodh master: Inherit pyvers from openstacklib::defaults https://review.openstack.org/630214 | 14:38 |
*** vnogin has joined #puppet-openstack | 14:40 | |
*** ykarelaway has quit IRC | 14:54 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/611996 | 14:57 |
*** vnogin has quit IRC | 15:01 | |
*** rpittau has quit IRC | 15:03 | |
*** rpittau has joined #puppet-openstack | 15:03 | |
*** dtantsur|brb is now known as dtantsur | 15:06 | |
*** rpittau has quit IRC | 15:15 | |
*** rpittau has joined #puppet-openstack | 15:16 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Change ubuntu repos to stein https://review.openstack.org/620055 | 15:17 |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Install libapache-wsgi-py3 https://review.openstack.org/624944 | 15:26 |
*** rpittau has quit IRC | 15:29 | |
*** rpittau has joined #puppet-openstack | 15:30 | |
*** ykarel has joined #puppet-openstack | 15:31 | |
*** LarsErikP has quit IRC | 15:41 | |
*** ekultails has joined #puppet-openstack | 15:46 | |
*** jistr is now known as jistr|mtg | 15:47 | |
*** trown|outtypewww is now known as trown|lunch | 15:53 | |
*** markvoelker has joined #puppet-openstack | 15:58 | |
*** jistr|mtg is now known as jistr | 15:58 | |
*** bnemec is now known as beekneemech | 16:02 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-openstack-integration master: placement: Extract the service from nova.pp https://review.openstack.org/615568 | 16:04 |
*** dpawlik has quit IRC | 16:12 | |
*** EmilienM is now known as EvilienM | 16:26 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-openstack-integration master: Set Ubuntu Beaker jobs to non-voting https://review.openstack.org/630317 | 16:33 |
lyarwood | tobias-urdin: ^ do we also need these jobs disabled while working on the Stein UCA switch? | 16:33 |
tobias-urdin | shouldn't need that "libraries" job, that only runs on puppet-openstack_spec_helper and puppet-openstack-cookiecutter | 16:39 |
tobias-urdin | we need to disable this: https://github.com/openstack/puppet-openstack-integration/blob/92e6e17668c67c220b9c64a779b20310d6854324/zuul.d/base.yaml#L180 | 16:39 |
tobias-urdin | the beaker test on ubuntu | 16:39 |
tobias-urdin | nice catch btw, i totally forgot about that | 16:40 |
*** shardy has quit IRC | 16:40 | |
lyarwood | tobias-urdin: ack, I'll rework that in a second | 16:41 |
*** iurygregory has quit IRC | 16:45 | |
* tobias-urdin leaving for today | 16:46 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-openstack-integration master: Set Ubuntu Beaker jobs to non-voting https://review.openstack.org/630317 | 16:47 |
lyarwood | \o_ enjoy your weekend | 16:47 |
*** rpittau has quit IRC | 16:52 | |
*** ykarel is now known as ykarel|away | 16:55 | |
openstackgerrit | Merged openstack/puppet-placement master: Add logging params to placement::wsgi::apache https://review.openstack.org/629497 | 17:00 |
openstackgerrit | Merged openstack/puppet-glance stable/rocky: Enable image inject metadata properties & user roles to be ignored https://review.openstack.org/629813 | 17:00 |
*** trown|lunch is now known as trown | 17:28 | |
*** jtomasek has quit IRC | 17:30 | |
*** tosky has quit IRC | 17:40 | |
*** dtantsur is now known as dtantsur|afk | 17:48 | |
*** gfidente has quit IRC | 17:58 | |
*** jtomasek has joined #puppet-openstack | 17:59 | |
*** derekh has quit IRC | 18:01 | |
*** jtomasek_ has joined #puppet-openstack | 18:08 | |
*** jtomasek has quit IRC | 18:08 | |
openstackgerrit | Bryan Wyatt proposed openstack/puppet-ceph master: Add support for encrypted OSDs https://review.openstack.org/604882 | 18:10 |
*** jtomasek_ has quit IRC | 18:15 | |
*** ykarel|away has quit IRC | 18:21 | |
*** jpena is now known as jpena|off | 18:44 | |
*** gouthamr has quit IRC | 18:51 | |
*** ekultails has quit IRC | 19:39 | |
*** openstackstatus has quit IRC | 19:43 | |
*** openstackstatus has joined #puppet-openstack | 19:44 | |
*** ChanServ sets mode: +v openstackstatus | 19:44 | |
*** dpawlik has joined #puppet-openstack | 19:48 | |
*** gouthamr has joined #puppet-openstack | 19:53 | |
*** openstack has joined #puppet-openstack | 20:18 | |
*** ChanServ sets mode: +o openstack | 20:18 | |
*** dpawlik has quit IRC | 20:53 | |
openstackgerrit | Bryan Wyatt proposed openstack/puppet-ceph master: Add support for encrypted OSDs https://review.openstack.org/604882 | 21:22 |
*** tosky has joined #puppet-openstack | 21:42 | |
*** morazi has quit IRC | 21:45 | |
*** trown is now known as trown|outtypewww | 21:52 | |
*** EvilienM is now known as EmilienM | 22:14 | |
*** markvoelker has quit IRC | 22:32 | |
*** rascasoft has quit IRC | 23:19 | |
*** rascasoft has joined #puppet-openstack | 23:26 | |
*** rascasoft has quit IRC | 23:37 | |
*** rascasoft has joined #puppet-openstack | 23:44 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!