Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-08-15

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:01 ducttape_ joined #puppet-openstack
01:22 vinsh joined #puppet-openstack
01:47 ilbot3 joined #puppet-openstack
01:47 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
02:32 ducttape_ joined #puppet-openstack
04:02 ducttape_ joined #puppet-openstack
04:13 vinsh joined #puppet-openstack
04:14 vinsh joined #puppet-openstack
04:46 jaosorior joined #puppet-openstack
05:03 ducttape_ joined #puppet-openstack
05:25 rcernin joined #puppet-openstack
05:26 dmsimard joined #puppet-openstack
06:04 ducttape_ joined #puppet-openstack
06:13 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/355159
06:44 rwsu joined #puppet-openstack
06:58 hjensas joined #puppet-openstack
06:58 hjensas joined #puppet-openstack
07:05 ducttape_ joined #puppet-openstack
08:03 paramite joined #puppet-openstack
08:06 ducttape_ joined #puppet-openstack
08:15 dgurtner joined #puppet-openstack
08:15 dgurtner joined #puppet-openstack
08:20 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
08:36 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
08:41 markvoelker joined #puppet-openstack
08:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add ironic inspector as a terminated HAProxy endpoint  https://review.openstack.org/345183
08:59 Jokke_ joined #puppet-openstack
09:00 Jokke_ gDay
09:01 Jokke_ anyone has idea why this  https://review.openstack.org/#/c/353928/ fails with http://logs.openstack.org/28/353928/5/check/gate-puppet-manila-puppet-unit-3.6-centos-7/cbf7da9/console.html.gz#_2016-08-12_10_25_27_958013 ?
09:07 ducttape_ joined #puppet-openstack
09:44 aderyugin hi, Jokke_ as I can see you've created a define, but wrote a test for class, so you should rewrite your test for cephfsnative in a form like https://github.com/openstack/puppet-manila/blob/master/spec/defines/manila_backend_glusterfs_spec.rb and put it in folder defines instead of classes
09:46 Jokke_ aderyugin: oh, thanks a million! ... trying to learn bit by bit. Had no idea.
09:52 openstackgerrit Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic  https://review.openstack.org/353928
10:02 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services  https://review.openstack.org/348905
10:02 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Sorted variables alphabetically  https://review.openstack.org/355400
10:07 ducttape_ joined #puppet-openstack
10:19 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
10:27 openstackgerrit Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic  https://review.openstack.org/353928
10:32 openstackgerrit Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::ipxe class  https://review.openstack.org/354125
10:42 markvoelker joined #puppet-openstack
10:48 openstackgerrit Carlos Camacho proposed openstack/puppet-nova: Removing WARNING: line has more than 140 characters in puppet-nova profiles  https://review.openstack.org/355409
10:53 Jokke_ degorenko: Thanks for your review ... this looks promising \\o \o/ o// o/7
10:53 degorenko Jokke_, for which patch? :D
10:54 Jokke_ degorenko: the Manila CephFS
10:54 degorenko ah
10:54 degorenko :)
10:54 Jokke_ thanks guys for bearing with me ... Puppet is really not the strong field for me ;)
10:57 degorenko no problem
11:00 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services  https://review.openstack.org/348905
11:08 ducttape_ joined #puppet-openstack
11:25 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services  https://review.openstack.org/348905
11:35 jaosorior joined #puppet-openstack
11:42 EmilienM good morning!
11:50 johnmilton joined #puppet-openstack
11:51 openstackgerrit Merged openstack/puppet-cinder: Allow to manage Cinder Volume types from backends  https://review.openstack.org/353775
11:51 furlongm joined #puppet-openstack
11:52 dprince joined #puppet-openstack
11:52 openstackgerrit Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::ipxe class  https://review.openstack.org/354125
12:09 ducttape_ joined #puppet-openstack
12:12 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: puppet4: fix ordering for compute agent  https://review.openstack.org/354756
12:13 ducttape_ joined #puppet-openstack
12:14 moravec joined #puppet-openstack
12:20 moravec joined #puppet-openstack
12:22 foxhunt72 joined #puppet-openstack
12:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_spec_helper: mitaka-only: force to clone poi on stable/mitaka  https://review.openstack.org/355442
12:30 markvoelker joined #puppet-openstack
12:33 iurygregory morning!
12:46 iurygregory hey EmilienM o/
12:47 iurygregory I've try to fix the nova_flavor type but with no luck  https://review.openstack.org/#/c/337389/25/lib/puppet/type/nova_flavor.rb =(
12:50 openstackgerrit Merged openstack/puppet-aodh: Move other-requirements.txt to bindep.txt  https://review.openstack.org/354973
12:50 openstackgerrit Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/355159
12:51 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Configure keystone authtoken options  https://review.openstack.org/351272
12:52 EmilienM iurygregory: yeah, I noticed this morning too, I had no chance to find why yet
12:52 iurygregory EmilienM, oh no problem o/
12:53 amoralej joined #puppet-openstack
12:55 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Configure keystone authtoken options  https://review.openstack.org/351272
12:57 EmilienM iberezovskiy, degorenko: can you please look https://review.openstack.org/#/c/355442/ and https://review.openstack.org/#/c/354837/ ?
12:57 EmilienM I'm trying to bring puppet-ceph stable/hammer CI back
12:58 degorenko ack +2
12:59 EmilienM thanks
12:59 EmilienM also https://review.openstack.org/#/c/354762/ and https://review.openstack.org/#/c/354756/ please
12:59 EmilienM i'm trying to bring all puppet4 jobs green
12:59 EmilienM so we can enable voting on them :)
13:07 iurygregory EmilienM, puppet-unit-latest-centos-7 is non-voting because of ceilometer only?
13:07 EmilienM this job has never voted
13:07 EmilienM http://docs.openstack.org/developer/puppet-openstack-guide/ci.html
13:07 iurygregory lol i'm still as sleep hahah
13:07 iberezovskiy reviewed
13:08 iberezovskiy need to talk with packaging guys to fix 'user' issue
13:09 moravec joined #puppet-openstack
13:10 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer  https://review.openstack.org/354835
13:10 openstackgerrit Merged openstack/puppet-ec2api: Remove S3 service  https://review.openstack.org/354732
13:11 EmilienM iberezovskiy: which one?
13:11 iberezovskiy EmilienM, in ceilometer
13:11 iberezovskiy I'll talk to guys
13:14 ducttape_ joined #puppet-openstack
13:16 openstackgerrit Merged openstack/puppet-openstack_spec_helper: mitaka-only: force to clone poi on stable/mitaka  https://review.openstack.org/355442
13:16 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services  https://review.openstack.org/348905
13:22 fultonj joined #puppet-openstack
13:24 iurygregory EmilienM, https://github.com/openstack/puppet-cinder/blob/master/lib/puppet/provider/cinder.rb#L52  https://github.com/openstack/puppet-neutron/blob/436b01f5426470dc251b131e0fa957211b552471/lib/puppet/provider/neutron.rb#L32 Do you know why the difference in auth?
13:31 EmilienM iurygregory: no
13:32 iurygregory skolekonov, ^
13:32 iurygregory do you know?
13:32 EmilienM one needs auth_url other auth_uri
13:32 iurygregory this is strange I think
13:40 EmilienM degorenko, iberezovskiy: about https://review.openstack.org/#/c/354756/ - do we need for something? just to make sure
13:40 EmilienM iurygregory: I'm looking nova today, trying to reproduce
13:40 iurygregory EmilienM, ack
13:41 degorenko EmilienM, +A, we +2'ed in one time
13:41 iurygregory degorenko, i think iberezovskiy did not +A because need a fix in fuel no?
13:42 degorenko iurygregory, for collector?
13:42 EmilienM degorenko: thx sir
13:42 iurygregory ' need to talk with packaging guys to fix 'user' issue'
13:43 degorenko iurygregory, to fix it in proper way with dependency, no?
13:43 iurygregory degorenko, oh yeah
13:43 iurygregory sorry XD
13:43 degorenko :)
13:44 iurygregory degorenko, do you know why neutron use auth_url and other components use auth_uri? https://github.com/openstack/puppet-cinder/blob/master/lib/puppet/provider/cinder.rb#L52  https://github.com/openstack/puppet-neutron/blob/436b01f5426470dc251b131e0fa957211b552471/lib/puppet/provider/neutron.rb#L32
13:44 degorenko skolekonov, ^
13:45 EmilienM I think it just depends on the providers
13:46 EmilienM like, cinder needs auth_uri and neutron auth_url
13:46 skolekonov iurygregory, AFAIK auth_url is the replacement for identity_uri which was used before. auth_uri has a bit different meaning
13:46 skolekonov but it seems they work both
13:46 skolekonov not completely sure, however
13:47 iurygregory skolekonov, yeah  the replace for identity_uri is auth_url and the difference for auth_uri is the port (admin) =)
13:48 iurygregory thanks god both work XD
13:48 skolekonov :)
13:55 ayoung joined #puppet-openstack
13:59 openstackgerrit Merged openstack/puppet-gnocchi: Keystone authoken resource change  https://review.openstack.org/347417
14:00 openstackgerrit Merged openstack/puppet-heat: Add log directory to .gitignore  https://review.openstack.org/354581
14:00 openstackgerrit Merged openstack/puppet-ironic: Add log directory to .gitignore  https://review.openstack.org/354583
14:00 openstackgerrit Merged openstack/puppet-mistral: Add log directory to .gitignore  https://review.openstack.org/354794
14:00 openstackgerrit Merged openstack/puppet-tripleo: Add ironic inspector as a terminated HAProxy endpoint  https://review.openstack.org/345183
14:03 skolekonov iurygregory, btw, about project_domain_id and user_domain_id. They are not covered by new authtoken class, so I use service_default for them, just for compatibility
14:04 ducttape_ joined #puppet-openstack
14:04 iurygregory skolekonov, yeah i notice, i comment about that we do not need both?
14:05 skolekonov right, we don't them both set to smth, but it's possible that someone used only *_id options (without *_name)
14:05 skolekonov *don't need
14:06 iurygregory yeah =)
14:06 iurygregory skolekonov, ack when you update i'll look =)
14:06 skolekonov thanks for comments :)
14:06 iurygregory np ^^
14:06 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Fix identation in docs for authtoken  https://review.openstack.org/355480
14:08 alee joined #puppet-openstack
14:08 ebalduf joined #puppet-openstack
14:14 devnull0 joined #puppet-openstack
14:15 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing"  https://review.openstack.org/339813
14:18 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-manila: Add authtoken class  https://review.openstack.org/351792
14:26 openstackgerrit Merged openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer  https://review.openstack.org/354837
14:27 dprince joined #puppet-openstack
14:27 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Fix authtoken and metadata  https://review.openstack.org/355480
14:28 EmilienM degorenko, iberezovskiy : https://review.openstack.org/#/c/354835/
14:28 EmilienM thanks
14:29 openstackgerrit Emilien Macchi proposed openstack/puppet-ceph: Update .gitreview to point at stable/hammer  https://review.openstack.org/350511
14:29 openstackgerrit Emilien Macchi proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/350512
14:30 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Fix metadata to require keystone  https://review.openstack.org/355494
14:38 EmilienM iurygregory: ok I reproduced the nova/authtoken issue, now I'm trying something
14:39 iurygregory EmilienM, you rock man \o/ thanks
14:40 EmilienM well, I havent found it yet
14:40 iurygregory but you have reproduced :P
14:42 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Fix metadata for authtoken  https://review.openstack.org/355505
14:44 openstackgerrit Merged openstack/puppet-openstack-integration: puppet4: ensure rabbitmq is ready before starting nova  https://review.openstack.org/354762
14:44 iurygregory mjblack, you around?
14:53 xarses joined #puppet-openstack
15:03 mjblack iurygregory: am now
15:03 mjblack whats up?
15:04 iurygregory mjblack, hey o/ can I take the work in heat for authtoken?
15:05 mjblack iurygregory: sure
15:05 iurygregory mjblack, ack o/
15:09 johnmilton joined #puppet-openstack
15:09 openstackgerrit Merged openstack/puppet-ceilometer: puppet4: fix ordering for compute agent  https://review.openstack.org/354756
15:11 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/355519
15:14 EmilienM iurygregory: in self.get_auth_endpoint I changed auth_uri to be auth_url and it worked
15:14 EmilienM let me double check
15:14 johnmilton joined #puppet-openstack
15:16 EmilienM mhh it might be something else
15:23 EmilienM I think the latest release of puppet is breaking unit tests for puppet-nova
15:24 EmilienM for nova::conductor and nova::scheduler
15:26 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Move nova to authtoken  https://review.openstack.org/337389
15:26 EmilienM iurygregory: let's try something^
15:26 EmilienM https://review.openstack.org/#/c/337389/25..26
15:27 EmilienM for unit tests failures, http://logs.openstack.org/89/337389/25/check/gate-puppet-nova-puppet-unit-latest-centos-7/b7844cd/console.html#_2016-08-14_14_32_49_099147
15:36 EmilienM I reported https://bugs.launchpad.net/puppet-neutron/+bug/1613329
15:36 openstack Launchpad bug 1613329 in puppet-nova "puppet 4.6.0 unit tests are broken" [Undecided,New]
15:36 EmilienM I'll work on it today but feel free to help
15:41 haplo37__ joined #puppet-openstack
15:56 hogepodge EmilienM: is there a publicly available repository of daily Newton RDO package builds that I can run the modules off of?
15:57 xarses joined #puppet-openstack
15:59 iurygregory EmilienM, ok :D
16:06 iurygregory hogepodge, something like https://trunk.rdoproject.org/centos7-master/45/36/45369b608aea665e38ad203deb6936c1004e082d_6c60bd37/ ?
16:07 hogepodge iurygregory: are there docs on how to activate it? I know that in the past I sometimes need to augment epel with newer packages
16:08 iurygregory hogepodge, in integration we use repos.pp https://review.openstack.org/#/c/355159/3/manifests/repos.pp
16:08 cwolferh joined #puppet-openstack
16:08 iurygregory i don't think there is doc about how activate
16:09 hogepodge iurygregory: that should all be enough. I want to start working from master so when I have to do the newton release I'm not scrambling to figure out what broke for my modules in the previous cycle
16:10 iurygregory hogepodge, ok if you need help just ask here =)
16:11 dfisher joined #puppet-openstack
16:17 ducttape_ joined #puppet-openstack
16:22 dhollinger joined #puppet-openstack
16:24 dprince joined #puppet-openstack
16:42 beagles wow, CI seems cranky .. at least on mitaka
16:42 iurygregory beagles wich patch?
16:43 beagles iurygregory, https://review.openstack.org/#/c/355235/
16:44 iurygregory beagles, recheck is the only thing you can do
16:45 beagles iurygregory, *shrug* ack
16:45 iurygregory beagles, i have put recheck o/
16:45 beagles iurygregory, thanks!!!
16:46 iurygregory beagles, np, i'm wondering if ubuntu nodes in infra are ok ...
16:56 openstackgerrit Merged openstack/puppet-openstack-integration: Disable barbican  https://review.openstack.org/355221
17:16 beagles iurygregory, good question.. I wonder if I should pint openstack-infra
17:17 beagles or ping rather.. they probably wouldn't mind pint either
17:36 dprince joined #puppet-openstack
17:37 rcernin joined #puppet-openstack
17:41 ayoung joined #puppet-openstack
17:52 EmilienM beagles: did you investigate in logs why mitaka is failing on trusty?
17:52 EmilienM looks like a problem with zuul-cloner and git servers
17:52 EmilienM http://logs.openstack.org/35/355235/1/gate/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/aaf8252/console.html#_2016-08-15_13_04_26_268615
17:52 iurygregory joined #puppet-openstack
17:54 beagles EmilienM, that's about as far as I got, I haven't asked infra yet
17:54 EmilienM ah :)
17:54 EmilienM ask them
17:54 beagles yep
18:12 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Move nova to authtoken  https://review.openstack.org/337389
18:21 EmilienM iurygregory: I think I found why
18:21 EmilienM the nova thing
18:21 EmilienM it sounds like an orchestration thing in poi
18:29 grimlock86 joined #puppet-openstack
18:30 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: provision: require admin user before creating nova flavor  https://review.openstack.org/355592
18:31 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Move nova to authtoken  https://review.openstack.org/337389
18:36 iurygregory EmilienM, oh the problem is in the poi? :O
18:37 iurygregory sorry i was in a meeting
18:37 EmilienM _ody: you around? need your help on some puppet4 things
18:51 dmsimard EmilienM: hm, https://github.com/openstack/puppet-gnocchi/commit/7aa0c6052b76bb3be94d6a52ccba8f7d163fbee4 broke Packstack.
18:52 dmsimard Probably packstack's fault though, just saying
18:52 EmilienM weird, poi didnt break
18:52 dmsimard Duplicate declaration: Gnocchi_config[keystone_authtoken/auth_version] is already declared in file /var/tmp/packstack/258ede9892104ffd980efac9019b6897/modules/packstack/manifests/gnocchi.pp:34; cannot redeclare at /var/tmp/packstack/258ede9892104ffd980efac9019b6897/modules/keystone/manifests/resource/authtoken.pp:326 on node n45.ci.centos.org
18:52 mwhahaha yup that'll happen
18:53 iurygregory lol
18:53 mwhahaha need to remove the auth_version from packstack and use the new params available
18:53 mwhahaha should get packstack visible in CI ;)
18:53 dmsimard eh it's alright
18:53 dmsimard nothing hard to fix
18:53 EmilienM well, we have enough jobs I think
18:54 EmilienM how many jobs have packstack, 3?
18:54 EmilienM dmsimard: you think we could add them into check?
18:54 EmilienM maybe as non voting
18:54 mwhahaha i don't think we have any external coverage on gnocchi
18:54 dmsimard I don't think it's worth it
18:54 dmsimard In many ways p-o-i has better and more modern coverage than Packstack
18:55 mwhahaha k
18:55 iurygregory beagles, the CI does not like you XD
18:56 beagles iurygregory, no it does not :) apparently there was an issue with zuul (or something). Infra says they have a patch pending
18:56 iurygregory beagles, oh awesome :D now only 2 failling i put recheck again
18:57 beagles iurygregory, :) thanks. It's like the "monte carlo" method of getting CI passing
18:57 mwhahaha looks like puppet-pacemaker is broken on xenial
18:59 EmilienM so much things are broken
18:59 EmilienM I'm working on puppet4 breakages this week, unit & functional
18:59 colonwq mwhahaha,  s/puppet-pacemaker/xenial/ :)
18:59 EmilienM and I'm helping Iury with the nova/authtoken thing
19:00 * mwhahaha fast forwards to Friday
19:01 EmilienM lol if only
19:02 EmilienM which reminds me: i'm on pto friday and monday, will be far in the middle of nowhere
19:02 colonwq Have a good time away from everything.
19:02 mwhahaha polarbear hunting again?
19:03 EmilienM I won't tell anyone, it's a secret place in the world, where nobody goes :P
19:03 dfisher Sun Microsystems!
19:03 * dfisher single tear
19:03 EmilienM lol ahahahah
19:04 haplo37__ joined #puppet-openstack
19:04 EmilienM mwhahaha: https://goo.gl/maps/fX4bUGSm3q82
19:05 mwhahaha Good ol' maine
19:05 dfisher "We're still a state!"  - official Maine state motto
19:06 mwhahaha north woods law, mildly amusing reality tv
19:07 colonwq All I know about Maine I learned from reading Steven King.
19:08 EmilienM are you scaring me? I'll lock the door.
19:08 dfisher won't matter.
19:08 dfisher U DED
19:09 colonwq just bury the body in the old Indian cemetery up on the hill.
19:10 EmilienM I knew I should have said no and stay in Canada
19:12 EmilienM mwhahaha: do you remember Sofer's bug in puppet-keystone with ldap thing?
19:12 EmilienM I think we're hitting that everywhere in our modules, with unit tests
19:12 mwhahaha the bool2num thing
19:12 mwhahaha ?
19:12 EmilienM it seems like when you pick() parameters from a different class you need to include the class
19:12 EmilienM see his patch https://review.openstack.org/#/c/354872/
19:12 EmilienM I have to do the same to fix puppet 4.6 unit tests or puppet-nova
19:13 mwhahaha that's annoying
19:14 EmilienM yes
19:14 mwhahaha so is that a breaking change in the 4.5 to 4.6 path?
19:14 EmilienM yeah
19:14 EmilienM that's why any help from _ody on this thing would be great
19:14 EmilienM I started to investigate the diff between 4.5 and 4.6 and I gave up
19:15 mwhahaha does it just flat out error or are we getting weird data in places?
19:16 _ody Sorry in a meeting all day. I'll try to hunt so,some down to help.
19:16 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Fix nvoa::db and puppet 4.6  https://review.openstack.org/355609
19:16 EmilienM mwhahaha: ^
19:16 EmilienM _ody: please help
19:16 EmilienM when you can
19:17 EmilienM mwhahaha: no, unit tests fail
19:17 EmilienM the pick() function doesn't work
19:18 mwhahaha seems like maybe the fix is in the pick function then
19:20 johnmilton joined #puppet-openstack
19:21 mwhahaha so it's the acceptance test right? where we're getting a "" from pick
19:21 mwhahaha which should get cleaned up by https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/lib/puppet/parser/functions/pick.rb#L22
19:21 haplo37__ joined #puppet-openstack
19:22 mwhahaha wonder if an undeclared variable in 4.6 is ending up not being an empty string but some other non printable char or something
19:23 EmilienM mwhahaha: no unit tests
19:23 mwhahaha but i thought his bug was showing up in the xenial acceptance tests not the unit
19:23 EmilienM mwhahaha: see http://logs.openstack.org/89/337389/25/check/gate-puppet-nova-puppet-unit-latest-centos-7/b7844cd/console.html#_2016-08-14_14_32_49_099016
19:23 mwhahaha http://logs.openstack.org/87/354087/3/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/e89d782/console.html#_2016-08-11_23_56_40_060367
19:23 EmilienM mwhahaha: it's another problem
19:23 EmilienM but maybe related
19:24 mwhahaha oh right the other one where we determined it was probably parse order related
19:24 mwhahaha where the p-o-i includes weren't necessarily being evaluated prior to the running of a resource
19:25 mwhahaha maybe we need some contains or some thing
19:29 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Defaults 'undef' to kernel_modules and sysctl_settings  https://review.openstack.org/355615
19:32 jonnytpuppet joined #puppet-openstack
19:33 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Fix nvoa::db and puppet 4.6  https://review.openstack.org/355609
19:34 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Fix nova::db and puppet 4.6  https://review.openstack.org/355609
19:35 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix neutron::db and puppet 4.6  https://review.openstack.org/355617
19:35 iurygregory spam time?
19:37 EmilienM so a lot of modules are affected
19:37 EmilienM I'm going to wait for feedback on nova and neutron and I'll patch the rest later
19:38 EmilienM I'm investigating a weird timeout issue  on puppet4 and gnocchi http://logs.openstack.org/19/355519/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7-nv/e5807ba/console.html#_2016-08-15_15_37_05_598922
19:40 mwhahaha so the puppet pacemaker issue lead me to this bit of code, https://github.com/puppetlabs/puppet/blob/master/lib/puppet/property.rb#L374-L398
19:41 mwhahaha anytime i see 'Here be dragons' (╯°□°)╯︵ ┻━┻
19:41 iurygregory LOL
19:42 iurygregory 'you need to be carreful?'
19:42 iurygregory careful*
19:48 ayoung joined #puppet-openstack
19:52 EmilienM mwhahaha: in 4.6?
19:52 EmilienM I'm wondering what kind of QA puppetlabs is running really
19:53 mwhahaha I'm sure its always been there but yea
19:53 mwhahaha It's erroring on data comparisons, haven't gotten farther yey
19:53 EmilienM I'm more talking about the bug Sofer and I have
19:53 EmilienM it looks like a non backward compatible thing
19:54 EmilienM my stuff sounds related to pick indeed
19:59 mwhahaha oh yea in 4.6
20:08 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Fine tuning of stack script  https://review.openstack.org/355629
20:09 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: copy_logs.sh: add support for Puppet4 modulepath  https://review.openstack.org/355630
20:12 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Fine tuning of stack script  https://review.openstack.org/355629
20:13 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: conductor: add api_url parameter  https://review.openstack.org/355632
20:14 mwhahaha http://logs.openstack.org/02/355202/1/check/gate-puppet-pacemaker-puppet-beaker-rspec-ubuntu-xenial/1879471/console.html#_2016-08-13_20_14_22_379463 and the other errors seems to point to some datatyping issue which i wonder if it's also the reason for the pick() issues
20:18 EmilienM mwhahaha: ugly error
20:18 mwhahaha pretty much
20:19 jonnytpuppet joined #puppet-openstack
20:21 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Fine tuning of stack script  https://review.openstack.org/355629
20:30 jonnytpuppet joined #puppet-openstack
20:36 Hunner EmilienM: Hmm. Before https://review.openstack.org/#/c/355609/ how did ::nova get included? neither nova::deps nor nova::params declares the base class as far as I see
20:37 mwhahaha it was included in the p-o-i manifest
20:37 mwhahaha or not as the case may be
20:38 mwhahaha the point is to try and maintain some type of backwards compatibility if that variable was set use it instead of the one to the class
20:39 EmilienM Hunner: yeah, something broke the unit tests
20:39 EmilienM and when I add ::nova in nova::db, it works again
20:39 Hunner :/
20:39 Hunner EmilienM: but something IS declaring ::nova elsewhere?
20:39 Hunner and you have to declare it there also?
20:39 EmilienM in unit tests, yes
20:40 Hunner I wonder if :pre_conditions broke or something...
20:40 EmilienM Hunner: here https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_conductor_spec.rb#L61
20:40 EmilienM yeah
20:57 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings  https://review.openstack.org/355615
21:01 iurygregory see ya o/ i'll be back latter
21:05 openstackgerrit Merged openstack/puppet-manila: Add authtoken class  https://review.openstack.org/351792
21:18 haplo37__ joined #puppet-openstack
21:25 grimlock86 joined #puppet-openstack
21:35 mwhahaha stupid data types
21:35 * mwhahaha cries
21:41 dmsimard Are we super positive that this is_service_default method works well ? https://github.com/openstack/puppet-gnocchi/blob/7aa0c6052b76bb3be94d6a52ccba8f7d163fbee4/manifests/keystone/authtoken.pp#L226
21:41 dmsimard I'm passing a password to gnocchi::keystone::authtoken and it doesn't look like it's picking it up
21:42 dmsimard i.e, in Packstack: https://review.openstack.org/#/c/355601/2/packstack/puppet/modules/packstack/manifests/gnocchi.pp
21:42 dmsimard Pasted some error snippets in my last comment in https://review.openstack.org/#/c/355601/ ..
22:00 mwhahaha dmsimard: works fine provided you have openstacklib
22:01 xarses joined #puppet-openstack
22:02 mwhahaha dmsimard: do you have a debug log for the hiera loading?
22:05 mwhahaha dmsimard: you need to include the keystone::authtoken prior to the api
22:05 mwhahaha i will comment on the review
22:19 jistr joined #puppet-openstack
22:28 xarses is there an etherpad for tomorrow? https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20160816 I'd guess
22:28 bkero sounds like you're making it
22:38 xarses seems so
23:24 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings  https://review.openstack.org/355615
23:34 xarses joined #puppet-openstack
23:52 dmsimard mwhahaha: ohhhhhh, thanks
23:52 dmsimard <3

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary