Thursday, 2018-11-01

*** markvoelker has quit IRC00:01
openstackgerritzhongshengping proposed openstack/puppet-heat master: Add manage_backend_package parameter  https://review.openstack.org/61441801:07
openstackgerritzhongshengping proposed openstack/puppet-nova master: Add manage_backend_package parameter  https://review.openstack.org/61441901:08
*** markvoelker has joined #puppet-openstack02:03
*** markvoelker has quit IRC02:35
openstackgerritMerged openstack/puppet-keystone stable/rocky: Add manage_backend_package parameter  https://review.openstack.org/61163302:42
*** markvoelker has joined #puppet-openstack03:32
*** markvoelker has quit IRC04:06
*** markvoelker has joined #puppet-openstack05:02
*** markvoelker has quit IRC05:36
*** agurenko has joined #puppet-openstack06:11
*** agurenko has quit IRC06:20
*** chkumar|off is now known as chandankumar06:21
*** agurenko has joined #puppet-openstack06:22
*** markvoelker has joined #puppet-openstack06:33
*** markvoelker has quit IRC07:05
*** markvoelker has joined #puppet-openstack08:03
*** shardy has joined #puppet-openstack08:08
*** shardy_ has joined #puppet-openstack08:08
*** shardy_ has quit IRC08:10
*** shardy_ has joined #puppet-openstack08:11
*** ykarel has joined #puppet-openstack08:16
*** markvoelker has quit IRC08:36
openstackgerritMerged openstack/puppet-openstack-integration master: Stop running non voting jobs in the gate  https://review.openstack.org/61430508:38
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418209:05
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418209:08
*** derekh has joined #puppet-openstack09:16
openstackgerrityatin proposed openstack/puppet-keystone master: Revert "Install python3-keystoneclient in Fedora or RedHat > 7"  https://review.openstack.org/61435209:32
*** markvoelker has joined #puppet-openstack09:33
*** PagliaccisCloud has quit IRC09:34
*** PagliaccisCloud has joined #puppet-openstack09:34
*** ykarel is now known as ykarel|lunch09:38
*** markvoelker has quit IRC10:07
*** chandankumar has quit IRC10:09
*** salmankhan has joined #puppet-openstack10:12
*** jaosorior has quit IRC10:23
*** chandankumar has joined #puppet-openstack10:31
*** morazi has quit IRC10:33
*** agurenko has quit IRC10:37
*** agurenko has joined #puppet-openstack10:39
*** jtomasek has quit IRC10:43
*** jtomasek has joined #puppet-openstack10:45
*** iurygregory is now known as iury|lunch11:00
*** markvoelker has joined #puppet-openstack11:04
*** pma78 has quit IRC11:05
*** jaosorior has joined #puppet-openstack11:35
*** markvoelker has quit IRC11:36
*** iury|lunch is now known as iurygregory11:36
*** morazi has joined #puppet-openstack11:47
*** trown|outtypewww is now known as trown12:18
*** ykarel_ has joined #puppet-openstack12:32
*** ykarel|lunch has quit IRC12:35
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418212:39
*** ykarel_ is now known as ykarel12:57
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts  https://review.openstack.org/61457713:07
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418213:08
openstackgerritTobias Urdin proposed openstack/puppet-murano master: Use puppet4 functions-api  https://review.openstack.org/61423813:15
tobias-urdinmwhahaha: ^ the spec testing fix should pass now13:16
tobias-urdinalso updating puppet-lint to 2.3.6 seems to make it fail properly on the WARNINGs atleast https://review.openstack.org/#/c/614589/13:17
tobias-urdinwould have been good if we can merge that later and test if it solves the lint jobs that reports OK but has an ERROR here: https://review.openstack.org/#/c/613449/13:17
*** dgurtner has joined #puppet-openstack13:36
*** dgurtner has quit IRC13:39
*** dgurtner has joined #puppet-openstack13:39
*** ykarel_ has joined #puppet-openstack13:40
*** ykarel has quit IRC13:42
*** ykarel_ is now known as ykarel13:43
mwhahahatobias-urdin: you probably need to squash it with https://review.openstack.org/#/c/614628/13:44
ykarelmwhahaha, tobias-urdin so is there some other plan then revert for facts patches?13:58
mwhahahai think we just need to fix the tests13:59
ykarelmwhahaha, tobias-urdin i mean https://review.openstack.org/#/c/614352/ and https://review.openstack.org/#/c/614391/13:59
mwhahahaand bump our rspec-puppet-facts to >313:59
ykarelmwhahaha, okk13:59
ykarelso putting reverts on hold?13:59
openstackgerritAlan Bishop proposed openstack/puppet-cinder master: Deprecate several iscsi_XXX parameters  https://review.openstack.org/61453613:59
mwhahahayea we have other random issues at the moment13:59
ykarelmwhahaha, scenario003?14:00
ykarelhorizon14:00
ykarelor something else also14:00
mwhahahaunit tests/lint issues14:00
mwhahahain addition to horizon14:00
ykarelokk14:00
mwhahahawe just need to exclude that horizon test14:00
mwhahahai'm sick of it14:00
ykarelyes that's good plan14:00
ykarelit's putting unnecessary load on ci14:00
ykareldue to rechecks14:01
ykarelmwhahaha, also i think it's good to keep some tracking of such thins in some etherpad etc14:01
ykarelso if some temp measure is taken it's keep on record14:02
mwhahahaprobably. i've been so under water with the tripleoci that i don't know the exact state of the puppet stuff14:02
ykarelhmm too many things for you :)14:02
ykarelso if all the issue are put at some place it's good to recall efforts14:02
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418214:11
*** dgurtner has quit IRC14:16
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Convert spec testing to use rspec-puppet-facts  https://review.openstack.org/61478214:41
mgagnecan someone review https://review.openstack.org/#/c/612455/ ? thanks14:45
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Cleanup spec testing  https://review.openstack.org/61478515:08
tobias-urdinmwhahaha: ykarel i'm working on fixing all the tests15:11
tobias-urdinmwhahaha: mind fixing the depends for that lint ignore?15:13
mwhahahatobias-urdin: so i think you need to include it in your spec fix15:13
*** ykarel_ has joined #puppet-openstack15:13
tobias-urdinoh ok15:13
mwhahahaor they won't merge since one will fail on lint and the other on unit tests15:13
tobias-urdinyy, i was kind of confused on how it would get merged15:13
* ykarel_ network issue15:13
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts  https://review.openstack.org/61457715:15
*** ykarel has quit IRC15:16
*** ykarel_ is now known as ykarel15:16
*** ykarel is now known as ykarel|away15:22
ykarel|awaytobias-urdin, cool, in case it's some heavy change, let me know i can give some time there15:27
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422615:31
tobias-urdinmwhahaha: seems like when calling a custom function in another custom function function_* name was used15:37
tobias-urdinfor example function_normalize_ip_for_uri is called in os_transport_url, seems like this isn't the case anymore15:37
tobias-urdindo you know if we can just remove function_ prefix with the new functions api?15:37
mwhahahayea there's supposed to be a new way to do it, though not sure it works15:37
tobias-urdinseems like it's call_function15:41
tobias-urdinhttps://github.com/puppetlabs/puppet-specifications/blob/master/language/func-api.md#reserved-method-names15:41
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api  https://review.openstack.org/61423115:46
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api  https://review.openstack.org/61424115:48
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Cleanup spec testing  https://review.openstack.org/61478516:03
*** ykarel|away has quit IRC16:21
openstackgerritTobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Cleanup spec testing  https://review.openstack.org/61480316:25
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Add puppet6 testing  https://review.openstack.org/61220116:39
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Cleanup spec testing  https://review.openstack.org/61478516:54
*** trown is now known as trown|lunch17:12
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts  https://review.openstack.org/61457717:35
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api  https://review.openstack.org/61423117:51
*** derekh has quit IRC18:00
openstackgerritTobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Cleanup spec testing  https://review.openstack.org/61480318:01
*** salmankhan has quit IRC18:02
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api  https://review.openstack.org/61424618:24
*** trown|lunch is now known as trown19:14
*** salmankhan has joined #puppet-openstack19:38
*** agurenko has quit IRC19:57
*** salmankhan has quit IRC19:59
*** sai_p has joined #puppet-openstack20:05
*** salmankhan has joined #puppet-openstack20:09
tobias-urdinmwhahaha: think this broke some logic https://review.openstack.org/#/c/613138/20:42
tobias-urdinhttp://logs.openstack.org/41/614241/3/check/puppet-openstack-unit-5.5-centos-7/52673ac/job-output.txt.gz#_2018-11-01_18_47_57_32517620:42
tobias-urdinwas no unit testing before, adding it here https://review.openstack.org/#/c/614241/ seems to reveal it's broken20:43
tobias-urdini'll see if i can solve it in that commit20:43
mwhahahahrm ok20:44
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api  https://review.openstack.org/61424120:47
*** xarses_ has quit IRC21:04
*** trown is now known as trown|outtypewww21:11
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api  https://review.openstack.org/61423121:12
*** morazi has quit IRC21:34
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api  https://review.openstack.org/61424621:54
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Deprecate nova-network code  https://review.openstack.org/61352221:58
openstackgerritTobias Urdin proposed openstack/puppet-nova master: nova::quota fix correct conf group and rename params  https://review.openstack.org/61418321:59
*** salmankhan has quit IRC22:01
openstackgerritTobias Urdin proposed openstack/puppet-openstack_spec_helper stable/ocata: Remove duplicate puppet declaration  https://review.openstack.org/61488222:02
*** salmankhan has joined #puppet-openstack22:09
*** salmankhan has quit IRC22:38
*** owalsh_ has joined #puppet-openstack23:14
*** owalsh has quit IRC23:15

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