Thursday, 2018-11-22

*** tosky has quit IRC01:07
*** rcernin has quit IRC01:39
*** rcernin has joined #puppet-openstack01:39
*** mschuppert has quit IRC02:31
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940203:43
openstackgerritzhongshengping proposed openstack/puppet-octavia master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940303:43
openstackgerritzhongshengping proposed openstack/puppet-mistral master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940403:43
openstackgerritzhongshengping proposed openstack/puppet-trove master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940503:43
openstackgerritzhongshengping proposed openstack/puppet-senlin master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940603:43
openstackgerritzhongshengping proposed openstack/puppet-vitrage master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940803:43
openstackgerritzhongshengping proposed openstack/puppet-panko master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941003:43
openstackgerritzhongshengping proposed openstack/puppet-freezer master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940703:43
openstackgerritzhongshengping proposed openstack/puppet-glare master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61940903:43
openstackgerritzhongshengping proposed openstack/puppet-heat master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941103:43
openstackgerritzhongshengping proposed openstack/puppet-ceilometer master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941203:43
openstackgerritzhongshengping proposed openstack/puppet-openstack-cookiecutter master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941303:43
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941403:43
openstackgerritzhongshengping proposed openstack/puppet-glance master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941503:43
openstackgerritzhongshengping proposed openstack/puppet-barbican master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941603:43
openstackgerritzhongshengping proposed openstack/puppet-cinder master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941703:43
openstackgerritzhongshengping proposed openstack/puppet-designate master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941803:43
openstackgerritzhongshengping proposed openstack/puppet-manila master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61941903:43
openstackgerritzhongshengping proposed openstack/puppet-congress master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61942003:43
openstackgerritzhongshengping proposed openstack/puppet-tacker master: Deprecate check_revocations_for_cached option  https://review.openstack.org/61942103:43
*** ykarel|away has joined #puppet-openstack03:49
*** ykarel|away is now known as ykarel03:56
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/puppet-ceilometer master: Add a parameter  https://review.openstack.org/61943304:09
*** ykarel has quit IRC05:24
*** chandankumar has joined #puppet-openstack05:33
*** chandankumar is now known as chkumar|ruck05:33
*** ykarel has joined #puppet-openstack05:40
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/61199606:05
*** mschuppert has joined #puppet-openstack07:08
*** agurenko has joined #puppet-openstack07:28
*** agurenko has quit IRC07:31
*** Tengu has left #puppet-openstack07:33
*** jaosorior has joined #puppet-openstack07:41
*** ykarel is now known as ykarel|lunch07:44
*** rpittau has joined #puppet-openstack07:44
*** skramaja has joined #puppet-openstack07:44
iurygregorygood morning08:05
*** rcernin has quit IRC08:13
*** shardy has joined #puppet-openstack08:26
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove port 35357 deployment  https://review.openstack.org/61925708:36
*** ykarel|lunch is now known as ykarel08:36
*** jpena|off is now known as jpena08:37
*** chem` has joined #puppet-openstack08:41
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Move out debug to keystone::logging  https://review.openstack.org/61947508:44
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove deprecated logging parameters  https://review.openstack.org/61937908:44
*** tosky has joined #puppet-openstack08:49
openstackgerritTobias Urdin proposed openstack/puppet-heat master: Install heat cron job after all heat packages  https://review.openstack.org/61949009:30
*** gfidente has joined #puppet-openstack09:31
tobias-urdinchkumar|ruck: seems like the horizon tempest job is not blacklisted on ubuntu bionic https://review.openstack.org/#/c/611996/09:32
tobias-urdinstill some that fails on that, but they are non-voting since before09:32
*** derekh has joined #puppet-openstack09:34
*** dtantsur|afk is now known as dtantsur09:54
gfidentetobias-urdin hey10:01
gfidentewas wondering10:01
gfidentehow do we get a new puppet-cinder release to include that fix about the rbd config?10:01
gfidenteI mean, what is the process, are releases created on a fixed schedule?10:02
tobias-urdinthe process is 1) bump version in metadata.json 2) submit patch to openstack/releases repo10:03
tobias-urdinit's on a per needed basis, usually go through all modules sometimes when a lot of changes has been made since release10:04
gfidentetobias-urdin ah ok thanks10:04
tobias-urdinthe backport to stable/rocky for your rbd config fix just missed 13.3.110:04
gfidenteI'll try!10:04
gfidenteyeah I noticed10:04
tobias-urdinwe can release 13.3.2 if you need it10:04
gfidenteand queens too10:04
gfidenteso that I was going to try10:04
gfidentetobias-urdin and, thanks!10:05
tobias-urdinno worries, go for it :)10:05
tobias-urdinadd me to the patches to openstack/releases and i'll approve10:05
openstackgerritGiulio Fidente proposed openstack/puppet-cinder stable/rocky: Release 13.3.2  https://review.openstack.org/61949410:08
openstackgerritGiulio Fidente proposed openstack/puppet-cinder stable/queens: Release 12.4.1  https://review.openstack.org/61949510:09
tobias-urdiniurygregory: can you check those two new stable releases ^10:21
iurygregorytobias-urdin, sure10:22
tobias-urdinty10:22
iurygregoryanytime o/10:23
tobias-urdinhigh five o/ \o10:23
tobias-urdin:D10:23
iurygregory=D10:23
openstackgerritMerged openstack/puppet-cinder stable/rocky: Release 13.3.2  https://review.openstack.org/61949410:33
openstackgerritMerged openstack/puppet-cinder stable/queens: Release 12.4.1  https://review.openstack.org/61949510:33
*** dpawlik has quit IRC10:37
*** dpawlik_ has joined #puppet-openstack10:37
*** jpena is now known as jpena|off10:37
*** salmankhan has joined #puppet-openstack10:49
*** salmankhan1 has joined #puppet-openstack10:52
*** salmankhan has quit IRC10:54
*** salmankhan1 is now known as salmankhan10:54
tobias-urdingfidente: see comments on releases change11:05
gfidentetobias-urdin ok need some help understanding why we don't want to change those lines though11:15
gfidenteI thought that was just a pointer to the version to use11:15
tobias-urdinthose are only for when creating branches, doesn't do anything after that and shouldn't be touched11:15
tobias-urdinwhen you are in releases repo you can do "tox -e venv -- new-release rocky puppet-cinder bugfix" if you dont want to touch the yaml files11:17
tobias-urdinare a lot of options and bugfix can be replaced with feature etc etc, can check the help on new-release cmd11:17
gfidenteah ah11:17
gfidenteupdated releases submission11:17
gfidentetrying the command11:18
tobias-urdinthe only difference between puppet-* release and all other projects is that we need to update metadata.json11:18
openstackgerritchenge proposed openstack/puppet-ceilometer master: fix mistakes  https://review.openstack.org/61954012:07
chkumar|rucktobias-urdin, I will take a look12:07
tobias-urdinoh saw that it was an old with just a new PS12:28
tobias-urdinperhaps it just isn't rebased12:28
chkumar|rucktobias-urdin, ok12:28
chkumar|ruckthanks for the info :-)12:29
tobias-urdinsorry i'll rebase and keep and eye out, seems like it's an old patch that has been updated by the periodic job from zuul12:33
tobias-urdinchkumar|ruck: it actually was rebased, so yeah something is wrong12:34
*** chem` has quit IRC13:50
*** salmankhan has left #puppet-openstack13:53
*** chkumar|ruck has quit IRC14:02
*** chem` has joined #puppet-openstack14:24
*** ykarel is now known as ykarel|away14:34
*** ykarel|away has quit IRC14:47
*** skramaja has quit IRC14:49
*** jistr is now known as jistr|call15:20
*** jistr|call is now known as jistr15:28
tobias-urdinEmilienM: tripleo jobs is broken, have they been fixed today?15:59
EmilienMtobias-urdin: link?16:02
EmilienMyeah we should have fixed everything16:02
EmilienM(tm)16:02
tobias-urdinEmilienM: see the first in this list https://review.openstack.org/#/q/project:%255Eopenstack/puppet%255C-.*+is:open+branch:master+message:%255ERemove.*16:03
tobias-urdinif so i'll just start rechecking those that have failed tripleo jobs16:04
EmilienMtobias-urdin: recheck16:04
EmilienMtobias-urdin: ping me if still failing16:04
tobias-urdinok ty16:04
*** ykarel|away has joined #puppet-openstack16:11
zigoEmilienM: tobias-urdin: When I try running the agent on a volume node, I get this:16:32
zigoError: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Virtual Query, Could not autoload puppet/type/cinder_type: Could not autoload puppet/provider/cinder_type/openstack: cannot load such file -- puppet/provider/openstack at /usr/share/puppet/modules/cinder/manifests/deps.pp:72:616:32
zigoon node zigocluster-volume-116:32
zigoWhat's going on?!?16:32
zigoIt's working perfectly on the controller nodes ...16:32
tobias-urdindo you have openstacklib module?16:33
zigotobias-urdin: I do.16:33
zigoWhich is why I don't understand.16:34
tobias-urdinit fails here https://github.com/openstack/puppet-cinder/blob/master/lib/puppet/provider/cinder.rb#L216:35
tobias-urdincan you try to add this like to that cinder.rb file at the top, like nova module does https://github.com/openstack/puppet-nova/blob/master/lib/puppet/provider/nova.rb#L516:36
zigoAlso because it works on another node with the exact same puppet manifest... :/16:36
tobias-urdinyeah that is weird, do you use a puppetmaster or bring all code to the modules?16:36
tobias-urdins/modules/nodes/g16:36
zigoSure, thanks !16:36
zigotobias-urdin: I use a puppet master node.16:39
zigoAll the puppet modules are installed on it using packages that I maintain, though it's the same as if you were using r10k or similar.16:40
zigotobias-urdin: I get: Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, undefined method `map' for :undef:Symbol16:40
zigoDid you mean?  tap at /usr/share/puppet/modules/oci/manifests/volume.pp:46:30 on node zigocluster-volume-116:40
zigoThough I had this, and sometimes the other error message I just pasted.16:41
zigoSo it should be the same problem.16:41
zigoHere's my manifest content: http://paste.openstack.org/show/735952/16:41
zigoOnly this, it still fails ...16:41
tobias-urdinthat's probably the "hosts" part given to os_transport_url16:45
tobias-urdincausing normalise_ip_for_uri or os_transport_url to fail16:45
zigotobias-urdin: You're right, my ENC is wrong !!! :)16:46
zigoI know how to fix ... :P16:46
zigoSorry for the noise.16:46
zigo$all_masters isn't being populated.16:46
tobias-urdindid it fix the cinder_type thing?16:47
zigotobias-urdin: I'm not sure if this fix is needed, probably it is.16:50
zigotobias-urdin: I'll write the patch if I see the issue again.16:50
zigotobias-urdin: Thanks a lot for your help.16:50
* zigo goes back home16:50
tobias-urdinok np16:50
*** shardy has quit IRC17:05
*** vnogin has joined #puppet-openstack17:16
*** vnogin has quit IRC17:16
*** dtantsur is now known as dtantsur|afk17:35
*** gfidente is now known as gfidente|afk17:44
*** derekh has quit IRC18:01
*** vnogin has joined #puppet-openstack18:04
*** ykarel|away has quit IRC18:09
*** rpittau has quit IRC18:46
*** vnogin has quit IRC19:04
*** rmart04 has joined #puppet-openstack19:53
*** rmart04 has left #puppet-openstack20:16
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418220:33
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418220:41
*** gfidente|afk has quit IRC21:04
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove deprecated PKI  https://review.openstack.org/61938021:39
*** rcernin has joined #puppet-openstack22:05
*** jtomasek has quit IRC22:20
*** tosky has quit IRC23:31

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