*** markvoelker has quit IRC | 00:01 | |
openstackgerrit | zhongshengping proposed openstack/puppet-heat master: Add manage_backend_package parameter https://review.openstack.org/614418 | 01:07 |
---|---|---|
openstackgerrit | zhongshengping proposed openstack/puppet-nova master: Add manage_backend_package parameter https://review.openstack.org/614419 | 01:08 |
*** markvoelker has joined #puppet-openstack | 02:03 | |
*** markvoelker has quit IRC | 02:35 | |
openstackgerrit | Merged openstack/puppet-keystone stable/rocky: Add manage_backend_package parameter https://review.openstack.org/611633 | 02:42 |
*** markvoelker has joined #puppet-openstack | 03:32 | |
*** markvoelker has quit IRC | 04:06 | |
*** markvoelker has joined #puppet-openstack | 05:02 | |
*** markvoelker has quit IRC | 05:36 | |
*** agurenko has joined #puppet-openstack | 06:11 | |
*** agurenko has quit IRC | 06:20 | |
*** chkumar|off is now known as chandankumar | 06:21 | |
*** agurenko has joined #puppet-openstack | 06:22 | |
*** markvoelker has joined #puppet-openstack | 06:33 | |
*** markvoelker has quit IRC | 07:05 | |
*** markvoelker has joined #puppet-openstack | 08:03 | |
*** shardy has joined #puppet-openstack | 08:08 | |
*** shardy_ has joined #puppet-openstack | 08:08 | |
*** shardy_ has quit IRC | 08:10 | |
*** shardy_ has joined #puppet-openstack | 08:11 | |
*** ykarel has joined #puppet-openstack | 08:16 | |
*** markvoelker has quit IRC | 08:36 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Stop running non voting jobs in the gate https://review.openstack.org/614305 | 08:38 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 09:05 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 09:08 |
*** derekh has joined #puppet-openstack | 09:16 | |
openstackgerrit | yatin proposed openstack/puppet-keystone master: Revert "Install python3-keystoneclient in Fedora or RedHat > 7" https://review.openstack.org/614352 | 09:32 |
*** markvoelker has joined #puppet-openstack | 09:33 | |
*** PagliaccisCloud has quit IRC | 09:34 | |
*** PagliaccisCloud has joined #puppet-openstack | 09:34 | |
*** ykarel is now known as ykarel|lunch | 09:38 | |
*** markvoelker has quit IRC | 10:07 | |
*** chandankumar has quit IRC | 10:09 | |
*** salmankhan has joined #puppet-openstack | 10:12 | |
*** jaosorior has quit IRC | 10:23 | |
*** chandankumar has joined #puppet-openstack | 10:31 | |
*** morazi has quit IRC | 10:33 | |
*** agurenko has quit IRC | 10:37 | |
*** agurenko has joined #puppet-openstack | 10:39 | |
*** jtomasek has quit IRC | 10:43 | |
*** jtomasek has joined #puppet-openstack | 10:45 | |
*** iurygregory is now known as iury|lunch | 11:00 | |
*** markvoelker has joined #puppet-openstack | 11:04 | |
*** pma78 has quit IRC | 11:05 | |
*** jaosorior has joined #puppet-openstack | 11:35 | |
*** markvoelker has quit IRC | 11:36 | |
*** iury|lunch is now known as iurygregory | 11:36 | |
*** morazi has joined #puppet-openstack | 11:47 | |
*** trown|outtypewww is now known as trown | 12:18 | |
*** ykarel_ has joined #puppet-openstack | 12:32 | |
*** ykarel|lunch has quit IRC | 12:35 | |
openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 12:39 |
*** ykarel_ is now known as ykarel | 12:57 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts https://review.openstack.org/614577 | 13:07 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 13:08 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-murano master: Use puppet4 functions-api https://review.openstack.org/614238 | 13:15 |
tobias-urdin | mwhahaha: ^ the spec testing fix should pass now | 13:16 |
tobias-urdin | also 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-urdin | would 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-openstack | 13:36 | |
*** dgurtner has quit IRC | 13:39 | |
*** dgurtner has joined #puppet-openstack | 13:39 | |
*** ykarel_ has joined #puppet-openstack | 13:40 | |
*** ykarel has quit IRC | 13:42 | |
*** ykarel_ is now known as ykarel | 13:43 | |
mwhahaha | tobias-urdin: you probably need to squash it with https://review.openstack.org/#/c/614628/ | 13:44 |
ykarel | mwhahaha, tobias-urdin so is there some other plan then revert for facts patches? | 13:58 |
mwhahaha | i think we just need to fix the tests | 13:59 |
ykarel | mwhahaha, tobias-urdin i mean https://review.openstack.org/#/c/614352/ and https://review.openstack.org/#/c/614391/ | 13:59 |
mwhahaha | and bump our rspec-puppet-facts to >3 | 13:59 |
ykarel | mwhahaha, okk | 13:59 |
ykarel | so putting reverts on hold? | 13:59 |
openstackgerrit | Alan Bishop proposed openstack/puppet-cinder master: Deprecate several iscsi_XXX parameters https://review.openstack.org/614536 | 13:59 |
mwhahaha | yea we have other random issues at the moment | 13:59 |
ykarel | mwhahaha, scenario003? | 14:00 |
ykarel | horizon | 14:00 |
ykarel | or something else also | 14:00 |
mwhahaha | unit tests/lint issues | 14:00 |
mwhahaha | in addition to horizon | 14:00 |
ykarel | okk | 14:00 |
mwhahaha | we just need to exclude that horizon test | 14:00 |
mwhahaha | i'm sick of it | 14:00 |
ykarel | yes that's good plan | 14:00 |
ykarel | it's putting unnecessary load on ci | 14:00 |
ykarel | due to rechecks | 14:01 |
ykarel | mwhahaha, also i think it's good to keep some tracking of such thins in some etherpad etc | 14:01 |
ykarel | so if some temp measure is taken it's keep on record | 14:02 |
mwhahaha | probably. i've been so under water with the tripleoci that i don't know the exact state of the puppet stuff | 14:02 |
ykarel | hmm too many things for you :) | 14:02 |
ykarel | so if all the issue are put at some place it's good to recall efforts | 14:02 |
openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 14:11 |
*** dgurtner has quit IRC | 14:16 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack_extras master: Convert spec testing to use rspec-puppet-facts https://review.openstack.org/614782 | 14:41 |
mgagne | can someone review https://review.openstack.org/#/c/612455/ ? thanks | 14:45 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Cleanup spec testing https://review.openstack.org/614785 | 15:08 |
tobias-urdin | mwhahaha: ykarel i'm working on fixing all the tests | 15:11 |
tobias-urdin | mwhahaha: mind fixing the depends for that lint ignore? | 15:13 |
mwhahaha | tobias-urdin: so i think you need to include it in your spec fix | 15:13 |
*** ykarel_ has joined #puppet-openstack | 15:13 | |
tobias-urdin | oh ok | 15:13 |
mwhahaha | or they won't merge since one will fail on lint and the other on unit tests | 15:13 |
tobias-urdin | yy, i was kind of confused on how it would get merged | 15:13 |
* ykarel_ network issue | 15:13 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts https://review.openstack.org/614577 | 15:15 |
*** ykarel has quit IRC | 15:16 | |
*** ykarel_ is now known as ykarel | 15:16 | |
*** ykarel is now known as ykarel|away | 15:22 | |
ykarel|away | tobias-urdin, cool, in case it's some heavy change, let me know i can give some time there | 15:27 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api https://review.openstack.org/614226 | 15:31 |
tobias-urdin | mwhahaha: seems like when calling a custom function in another custom function function_* name was used | 15:37 |
tobias-urdin | for example function_normalize_ip_for_uri is called in os_transport_url, seems like this isn't the case anymore | 15:37 |
tobias-urdin | do you know if we can just remove function_ prefix with the new functions api? | 15:37 |
mwhahaha | yea there's supposed to be a new way to do it, though not sure it works | 15:37 |
tobias-urdin | seems like it's call_function | 15:41 |
tobias-urdin | https://github.com/puppetlabs/puppet-specifications/blob/master/language/func-api.md#reserved-method-names | 15:41 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api https://review.openstack.org/614231 | 15:46 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api https://review.openstack.org/614241 | 15:48 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Cleanup spec testing https://review.openstack.org/614785 | 16:03 |
*** ykarel|away has quit IRC | 16:21 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Cleanup spec testing https://review.openstack.org/614803 | 16:25 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Add puppet6 testing https://review.openstack.org/612201 | 16:39 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Cleanup spec testing https://review.openstack.org/614785 | 16:54 |
*** trown is now known as trown|lunch | 17:12 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts https://review.openstack.org/614577 | 17:35 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api https://review.openstack.org/614231 | 17:51 |
*** derekh has quit IRC | 18:00 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Cleanup spec testing https://review.openstack.org/614803 | 18:01 |
*** salmankhan has quit IRC | 18:02 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api https://review.openstack.org/614246 | 18:24 |
*** trown|lunch is now known as trown | 19:14 | |
*** salmankhan has joined #puppet-openstack | 19:38 | |
*** agurenko has quit IRC | 19:57 | |
*** salmankhan has quit IRC | 19:59 | |
*** sai_p has joined #puppet-openstack | 20:05 | |
*** salmankhan has joined #puppet-openstack | 20:09 | |
tobias-urdin | mwhahaha: think this broke some logic https://review.openstack.org/#/c/613138/ | 20:42 |
tobias-urdin | http://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_325176 | 20:42 |
tobias-urdin | was no unit testing before, adding it here https://review.openstack.org/#/c/614241/ seems to reveal it's broken | 20:43 |
tobias-urdin | i'll see if i can solve it in that commit | 20:43 |
mwhahaha | hrm ok | 20:44 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api https://review.openstack.org/614241 | 20:47 |
*** xarses_ has quit IRC | 21:04 | |
*** trown is now known as trown|outtypewww | 21:11 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api https://review.openstack.org/614231 | 21:12 |
*** morazi has quit IRC | 21:34 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api https://review.openstack.org/614246 | 21:54 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate nova-network code https://review.openstack.org/613522 | 21:58 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: nova::quota fix correct conf group and rename params https://review.openstack.org/614183 | 21:59 |
*** salmankhan has quit IRC | 22:01 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack_spec_helper stable/ocata: Remove duplicate puppet declaration https://review.openstack.org/614882 | 22:02 |
*** salmankhan has joined #puppet-openstack | 22:09 | |
*** salmankhan has quit IRC | 22:38 | |
*** owalsh_ has joined #puppet-openstack | 23:14 | |
*** owalsh has quit IRC | 23:15 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!