Tuesday, 2016-01-05

« Monday, 2016-01-04 Index Wednesday, 2016-01-06 »
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26348300:03
openstackgerritMerged openstack/puppet-nova: Remove EC2 support  https://review.openstack.org/26109100:03
EmilienM_ody: I'm afraid https://review.openstack.org/#/c/261091/ landed - but we can talk about a follow-up here00:06
EmilienMthe idea was to 1/ drop ec2 2/ deprecate the interface00:07
openstackgerritAlex Schultz proposed openstack/puppet-openstack-cookiecutter: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26341000:08
*** ntpttr1 has left #puppet-openstack00:08
*** dfisher has quit IRC00:08
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26348300:09
openstackgerritAlex Schultz proposed openstack/puppet-aodh: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26348100:10
openstackgerritAlex Schultz proposed openstack/puppet-vitrage: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347800:11
openstackgerritMerged openstack/puppet-swift: Change defaults fro incoming/outgoing chmod (rsync)  https://review.openstack.org/26132700:20
*** ducttape_ has joined #puppet-openstack00:22
*** Vinsh has quit IRC00:23
*** Vinsh has joined #puppet-openstack00:24
*** jfluhmann has quit IRC00:47
openstackgerritMerged openstack/puppet-tempest: Allow to enable/disable Ironic service  https://review.openstack.org/26296400:48
_odyEmilienM: Yeah.  Once I commented I noticed it had already been approved.  I kinda want to just fail the catalog.  I usually don't mind having a parameter around that doesn't do anything but in this case we've completely stopped managing an entire "thing" as opposed to removing a simple piece of un-needed configuration.00:49
iurygregory_ody, can you take a look https://review.openstack.org/#/c/263252/ ?00:50
*** richm has quit IRC00:59
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Implement WSGI support for Ironic API  https://review.openstack.org/26345001:01
openstackgerritEmilien Macchi proposed openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type  https://review.openstack.org/26350301:03
openstackgerritEmilien Macchi proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix  https://review.openstack.org/25988501:04
_odyiurygregory: hu.  Weird.01:07
iurygregory_ody,  yep kkkk tks man01:08
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/25563501:08
*** ducttape_ has quit IRC01:10
openstackgerritMerged openstack/puppet-ironic: Remove duplicate rabbit_virtual_host  https://review.openstack.org/26325201:18
openstackgerritMerged openstack/puppet-glance: Set os_region_name to $::os_service_default and drop warning  https://review.openstack.org/26304901:21
*** ducttape_ has joined #puppet-openstack01:24
*** ducttape_ has quit IRC01:30
openstackgerritMerged openstack/puppet-swift: Stop calling ::concat::setup  https://review.openstack.org/26306001:31
*** Vinsh has quit IRC01:32
*** Vinsh has joined #puppet-openstack01:33
*** rahulait has joined #puppet-openstack01:34
*** ducttape_ has joined #puppet-openstack01:41
EmilienMVinsh: I might need your help on swift02:07
EmilienMtrying to deploy Mitaka with puppet-swift: https://review.openstack.org/#/c/255635/02:07
EmilienMscenario002 fails also because of swift02:08
EmilienMVinsh: http://logs.openstack.org/35/255635/8/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/c469d03/console.html#_2016-01-05_01_47_50_67102:08
EmilienMswift logs: http://logs.openstack.org/35/255635/8/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/c469d03/logs/swift/swift.txt.gz02:08
EmilienMit seems like /etc/swift/account.ring.gz does not exist, which should be created by puppet somehow02:09
EmilienMdmsimard: have you seen that in RDO ^?02:09
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/25563502:10
openstackgerritXiaohua Yuan proposed openstack/puppet-neutron: Add related parameters to ovs section  https://review.openstack.org/26352802:17
dmsimardEmilienM: I never deployer Swift on RDO02:26
dmsimardthe account.ring.gz file definitely doesn't come from packaging, that's a ring database file02:27
dmsimardIt has to be generated through swift-ring-builder, either through puppet or other means02:27
dmsimardlogging off for now, catch me tomorrow if you need a hand02:30
*** cwolferh has quit IRC02:30
*** Vinsh_ has joined #puppet-openstack02:46
*** ducttape_ has quit IRC02:47
*** Vinsh has quit IRC02:50
*** cwolferh has joined #puppet-openstack03:04
*** rahulait has quit IRC04:11
Vinsh_EmilienM: Will take a look closer in the morning. Just wanted to let you know I saw your message.04:30
*** jaosorior has joined #puppet-openstack05:37
*** jaosorior has quit IRC06:05
*** jaosorior has joined #puppet-openstack06:26
*** samuelBartel has joined #puppet-openstack07:52
*** dgurtner has joined #puppet-openstack07:58
*** jtomasek has joined #puppet-openstack08:02
*** rmart04 has joined #puppet-openstack08:08
*** tdb has quit IRC08:16
*** foxhunt_away is now known as foxhunt7208:20
*** tdb has joined #puppet-openstack08:23
*** iurygregory has quit IRC09:09
*** jistr has joined #puppet-openstack09:21
*** paramite has joined #puppet-openstack09:41
*** jaosorior has quit IRC09:44
*** jaosorior has joined #puppet-openstack09:44
*** jaosorior has quit IRC09:56
*** jaosorior has joined #puppet-openstack09:56
*** krickwix has quit IRC09:58
openstackgerritReedip proposed openstack/puppet-zaqar: Update typo  https://review.openstack.org/26317610:04
*** paramite is now known as paramite|afk10:07
*** bpiotrowski has left #puppet-openstack10:29
openstackgerritGael Chamoulaud proposed openstack/puppet-swift: Require swift package installation before swift user.  https://review.openstack.org/26367010:45
*** andybotting has quit IRC10:54
*** samuelBartel has quit IRC10:58
*** paramite|afk is now known as paramite11:19
*** rmart04_ has joined #puppet-openstack11:36
*** rmart04 has quit IRC11:36
*** rmart04_ is now known as rmart0411:36
*** chem has joined #puppet-openstack11:39
*** ericksonsantos has joined #puppet-openstack11:43
*** rmart04 has quit IRC11:56
*** cgascoig has quit IRC12:05
*** cgascoig has joined #puppet-openstack12:06
*** rmart04 has joined #puppet-openstack12:15
*** iurygregory has joined #puppet-openstack12:19
*** slagle has quit IRC12:38
*** trown|outttypeww is now known as trown12:43
*** gfidente has joined #puppet-openstack12:45
*** gfidente has quit IRC12:45
*** gfidente has joined #puppet-openstack12:45
EmilienMgood morning12:49
EmilienMVinsh_: thanks!12:49
socialEmilienM: moin12:49
iurygregorymorning =D12:51
*** cgascoig has quit IRC12:52
*** dprince has joined #puppet-openstack12:54
*** cgascoig has joined #puppet-openstack12:54
*** ducttape_ has joined #puppet-openstack13:06
*** ducttape_ has quit IRC13:07
*** ducttape_ has joined #puppet-openstack13:13
*** samuelBartel has joined #puppet-openstack13:24
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Fix typo  https://review.openstack.org/26373113:25
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Fix typo  https://review.openstack.org/26373113:27
iurygregoryEmilienM, https://review.openstack.org/#/c/256642/ \o/13:29
iurygregorywhen you have time =D13:29
*** morazi has joined #puppet-openstack13:33
*** jayg|g0n3 is now known as jayg13:34
*** slagle has joined #puppet-openstack13:35
*** ducttape_ has quit IRC13:36
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Fix metadata.json  https://review.openstack.org/26373613:36
chemhi cores, if you have 2 min to review a very simple one https://review.openstack.org/#/c/258483/13:39
chemit fixes bug https://launchpad.net/bugs/152678613:39
openstackLaunchpad bug 1526786 in tripleo "Haproxy has non-working Horizon session persistence." [Undecided,New]13:39
EmilienMchem: it's tripleo related, you'll be luckier on #tripleo channel13:43
EmilienMchem: in your commit message you set a reference to [1] which does not exist13:43
chemEmilienM: thanks.13:44
chemEmilienM: for the tripleo thing I'm surprise as it's a puppet module, I though that it would be good here13:44
EmilienMchem: this module is a bit special, it's a composition layer used by tripleo only13:45
EmilienMso it makes sense to let tripleo folks reviewing it13:45
EmilienMbecause you can break tripleo easily with this module13:45
EmilienMour group is core on it, for historical reasons I think :)13:45
chemEmilienM: oki, thanks for the information13:45
*** dslevin has quit IRC13:45
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: Haproxy has non-working Horizon session persistence.  https://review.openstack.org/25848313:47
*** iurygregory has quit IRC13:55
*** rahulait has joined #puppet-openstack13:59
*** b3nt_pin is now known as beagles14:01
*** rochaporto has joined #puppet-openstack14:02
openstackgerritMerged openstack/puppet-keystone: Federation support for mellon  https://review.openstack.org/25664214:10
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: fwaas config needs to notify l3 agent service  https://review.openstack.org/26374514:16
*** ericksonsantos has quit IRC14:22
*** ducttape_ has joined #puppet-openstack14:22
*** ducttape_ has quit IRC14:28
*** rahulait has quit IRC14:28
*** Vinsh_ has quit IRC14:30
*** Vinsh has joined #puppet-openstack14:30
*** rochaporto has quit IRC14:31
*** iurygregory has joined #puppet-openstack14:33
*** kbyrne has quit IRC14:33
*** Vinsh_ has joined #puppet-openstack14:36
*** Vinsh has quit IRC14:36
*** kbyrne has joined #puppet-openstack14:40
EmilienMour weekly meeting will start in 15min https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016010514:46
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Set $keystone::endpoint::version to '' by default  https://review.openstack.org/26110514:47
openstackgerritAlex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347514:47
*** ducttape_ has joined #puppet-openstack14:48
openstackgerritAlex Schultz proposed openstack/puppet-barbican: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347914:50
EmilienMmwhahaha: thx a lot ^14:50
*** dfisher has joined #puppet-openstack14:51
mwhahahano problem, i left a note on the trello card. the mistral module needs some attention as it doesn't have a mistral::db class14:52
EmilienMok14:52
*** kbyrne has quit IRC14:52
openstackgerritAlex Schultz proposed openstack/puppet-vitrage: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347814:53
*** kbyrne has joined #puppet-openstack14:53
*** richm has joined #puppet-openstack14:54
*** kbyrne has quit IRC14:55
openstackgerritEmilien Macchi proposed openstack/puppet-swift: Set mount_check to True by default  https://review.openstack.org/26133314:57
openstackgerritAlex Schultz proposed openstack/puppet-aodh: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26348114:57
*** kbyrne has joined #puppet-openstack15:00
*** kbyrne has quit IRC15:01
*** kbyrne has joined #puppet-openstack15:02
*** slagle has quit IRC15:02
openstackgerritAdam Vinsh proposed openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/26376015:03
Vinsh_EmilienM: If I got the array syntax right, this ^^^ should solve your mitaka swift ring woes.15:03
EmilienMVinsh_: meeting15:03
*** Vinsh_ is now known as Vinsh15:04
Vinshjoined. thanks for the reminder!15:04
*** rahulait has joined #puppet-openstack15:05
*** dhollinger has joined #puppet-openstack15:09
*** paramite is now known as paramite|afk15:19
*** rahulait has quit IRC15:24
VinshEmilienM: yep syntax is good in 263760.. that is backward compat safe and will also fix scenario002 for mitaka swift.15:37
VinshYou ran into this new swift patch: https://review.openstack.org/#/c/241571/15:38
EmilienMVinsh: thanks a lot :-) you rocks man15:38
VinshThey now enforce that ring devices be >= replica count.15:38
EmilienMVinsh: could you rebase on top of my refacto patch please?15:38
EmilienMotherwise your patch will merge conflict today15:38
Vinshd'oh. good point. lemme fix.15:39
EmilienMthanks15:39
EmilienMchem: same error: http://logs.openstack.org/05/261105/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/6e200ad/console.html#_2016-01-05_15_30_57_40215:41
openstackgerritAdam Vinsh proposed openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/26376015:42
chemyes, but I think the problem is with the API, not puppet15:42
chemEmilienM: ^15:42
Vinshrebased.15:42
chemEmilienM: or slightly with puppet :)15:43
openstackgerritMartin Millnert proposed openstack/puppet-nova: Allow spice protocol to be deployed  https://review.openstack.org/26377715:43
EmilienMVinsh: thx, will look after CI result15:43
chemEmilienM: I'll check later in the day and give you my final answer15:44
*** paramite|afk is now known as paramite15:45
*** dslevin has joined #puppet-openstack15:54
openstackgerritLucas Alvares Gomes proposed openstack/puppet-ironic: Add uefi_pxe_config_template parameter  https://review.openstack.org/26332115:54
*** baker has joined #puppet-openstack15:58
*** slagle has joined #puppet-openstack16:05
*** dslevin has quit IRC16:06
*** rmart04 has quit IRC16:13
crinkleEmilienM: jan 25-27 works for me16:22
*** imcsk8 is now known as imcsk8|afk16:23
openstackgerritAlex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347516:23
*** zeus has quit IRC16:24
EmilienMcrinkle: ack16:25
EmilienMcrinkle: we just need to know for nibalizer and we're good I guess16:25
*** baker has quit IRC16:27
*** zeus has joined #puppet-openstack16:29
*** zeus is now known as Guest5433216:30
*** foxhunt72 is now known as foxhunt_away16:33
mwhahahaHey EmilienM who do you talk to about ubuntu package problems?16:41
mwhahahaneutron (liberty) fails if you configure it with syslog because of Bug #1499620 which from an oslo  issue16:42
openstackbug 1499620 in python-oslo.log (Ubuntu Wily) "[SRU] Unintended assignment of "syslog"" [High,In progress] https://launchpad.net/bugs/1499620 - Assigned to Edward Hope-Morley (hopem)16:42
openstackgerritMerged openstack/puppet-tripleo: Enable X-Forwarded-Proto header for keystone_public  https://review.openstack.org/25477116:43
*** dprince has quit IRC16:45
nibalizerEmilienM: hi16:45
nibalizervirtual or physical?16:45
EmilienMnibalizer: hey, could you confirm that jan 25-27 works works for you too? virtual16:46
nibalizerya that works16:47
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26341016:47
EmilienMnibalizer: cool thanks16:48
*** mkarpin1 has quit IRC16:49
*** dprince has joined #puppet-openstack16:50
*** mkarpin has joined #puppet-openstack16:50
openstackgerritMerged openstack/puppet-tripleo: Enable X-Forwarded-Proto header for Heat and Nova  https://review.openstack.org/25863416:54
*** rmart04 has joined #puppet-openstack16:55
*** ericksonsantos has joined #puppet-openstack16:55
dprinceEmilienM: hey, I've got a few patches blocked on this one https://review.openstack.org/#/c/253493/16:56
EmilienMdprince: ok looking right after my meeting16:57
dprincesbadia: might you have a chance to review this one as well? ^^16:58
EmilienMdprince: +216:59
*** Guest54332 is now known as zeus17:01
openstackgerritMerged openstack/puppet-mistral: Update typo  https://review.openstack.org/26318017:01
openstackgerritMerged openstack/puppet-mistral: Reflect provider change in puppet-openstacklib  https://review.openstack.org/26330117:01
*** zeus has quit IRC17:01
*** zeus has joined #puppet-openstack17:01
*** imcsk8|afk is now known as imcsk817:04
*** rmart04 has quit IRC17:05
*** rmart04 has joined #puppet-openstack17:06
*** paramite is now known as paramite|afk17:09
VinshEmilienM: swift issue fixed.. looks like other dependency issues though http://logs.openstack.org/60/263760/2/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/8881db2/console.html#_2016-01-05_16_12_47_31917:11
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb support  https://review.openstack.org/25454817:12
*** ducttape_ has quit IRC17:14
EmilienMVinsh: tha's not an error17:14
EmilienMit's a warning17:14
*** ducttape_ has joined #puppet-openstack17:14
EmilienMthe error is puppet run is not idempotent17:14
*** baker has joined #puppet-openstack17:15
EmilienMthe real issue is http://logs.openstack.org/60/263760/2/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/8881db2/console.html#_2016-01-05_16_28_23_38717:15
EmilienMsomething broke up in sahara/liberty in RDO17:16
EmilienMdmsimard: ^17:16
VinshAh-hah.17:16
EmilienMno logs: http://logs.openstack.org/60/263760/2/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/8881db2/logs/17:16
EmilienMJan 05 16:28:24 template-devstack-centos7-1451830444.novalocal systemd[1]: openstack-sahara-engine.service failed.17:17
EmilienMhttp://logs.openstack.org/60/263760/2/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/8881db2/logs/syslog.txt.gz17:17
dmsimardEmilienM: stable repositories ?17:17
EmilienMdmsimard: yes.17:17
EmilienMI guess17:17
* EmilienM checks17:17
EmilienMlol17:18
EmilienMVinsh rebased on top of wrong patch :(17:18
EmilienMdmsimard: no you're good but sahara looks broken in Mitaka17:18
EmilienMsee logs17:18
*** samuelBartel has quit IRC17:18
VinshI rebased on the patch you had up yesterday..17:18
dmsimardEmilienM: so is the issue in liberty or in mitaka then ?17:18
EmilienMVinsh: https://review.openstack.org/26111517:18
EmilienMnope that's the refacto patch ^17:18
EmilienM"Refactor fixtures & manifests to reduce code lines"  :)17:18
EmilienMto avoid conflict with your patch you sent first17:19
EmilienMdmsimard: mitaka master17:19
*** jistr has quit IRC17:19
EmilienMsahara does not start17:19
VinshEmilienM: oh. I was thinking you meant the one that was failing for you yesterday. oops.17:20
EmilienMVinsh: can you rebase on top of the right one please?17:20
openstackgerritMerged openstack/puppet-tripleo: Sahara integration  https://review.openstack.org/22085917:21
*** paramite|afk is now known as paramite17:21
openstackgerritMerged openstack/puppet-tripleo: Trove integration  https://review.openstack.org/23323217:23
*** rmart04 has quit IRC17:29
*** ntpttr has joined #puppet-openstack17:30
openstackgerritAdam Vinsh proposed openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/26376017:31
EmilienMVinsh: thx ^17:31
VinshMakes much more sense on that rebase.  (facepalm)17:31
EmilienMVinsh: :-P stop champagne !17:32
Vinshbut it has vitamins!17:32
EmilienMahah, vitamins matter17:32
openstackgerritTim Rozet proposed openstack/puppet-neutron: Adds the ability to force metadata with dhcp agent  https://review.openstack.org/26108417:32
colonwqEmilienM, I think I have taken care of all of the civetweb changes recommended in the reviews. (254548).17:39
EmilienMI'm not a puppet-ceph core17:39
*** _ducttape_ has joined #puppet-openstack17:40
*** ducttape_ has quit IRC17:40
colonwqEmilienM, not a problem. you got me mostly on my bad code style.17:41
openstackgerritMerged openstack/puppet-openstack-integration: Refactor fixtures & manifests to reduce code lines  https://review.openstack.org/26111517:42
openstackgerritMerged openstack/puppet-openstack-integration: swift: set mount_check to false  https://review.openstack.org/26280117:42
EmilienMI'm that guy?17:42
openstackgerritMerged openstack/puppet-ironic: Add ironic::keystone::auth_inspector  https://review.openstack.org/25349317:44
*** jaosorior has quit IRC17:44
colonwqonly minor things. I litter extra white spaces out of habit.17:44
*** jaosorior has joined #puppet-openstack17:44
*** jaosorior has quit IRC17:45
*** jaosorior has joined #puppet-openstack17:45
openstackgerritMerged openstack/puppet-glance: Use identity_uri and auth_uri by default  https://review.openstack.org/26112017:51
*** mdorman has joined #puppet-openstack17:53
*** trown is now known as trown|lunch17:54
openstackgerritMerged openstack/puppet-glance: Default service_name to 'Image Service'  https://review.openstack.org/26131717:58
*** Anticimex has joined #puppet-openstack18:01
Anticimexwe're deploying using puppet-nova, and had a run at deploying spice consoles18:02
Anticimexran into a bug, https://bugs.launchpad.net/puppet-nova/+bug/151858018:03
openstackLaunchpad bug 1518580 in puppet-nova "Unable to deploy spice on Nova Compute" [Undecided,In progress] - Assigned to Martin Millnert (r-martin-5)18:03
Anticimexthe issue is a conflict of ini settings blocking deployment of non-vnc consoles, on by default18:04
Anticimexi got gate errors however, and there are two tests that expect these settings to exist even when vnc is disabled:18:04
Anticimexaround http://logs.openstack.org/77/263777/1/check/gate-puppet-nova-puppet-unit-3.3-dsvm-centos7/645bfc0/console.html#_2016-01-05_16_21_13_13518:04
Anticimexthe patch works for us at any rate.18:05
Anticimexso i'd like to learn more about the rationale for those two tests, since they block spice deployment18:06
mwhahahayour patch changes the way it functions so the tests would need to be updated as well18:06
* Anticimex is digging after where the spec of the tests live18:07
mwhahahathe log tells you, ./spec/classes/nova_compute_spec.rb:15518:07
Anticimexthx18:07
Anticimexok, so i'll update the tests and resubmit, and see what that gives18:10
mwhahahamy assumption based on the logs is that the test was wrong and what it's doing now is correct18:10
mwhahahanot sure why we would set the vnc proxy address if we're disabling vnc :D18:11
Anticimexpointer to rspec tutorial gladly accepted =)18:12
Anticimex(want to flip the test to assert variable not defined)18:12
mwhahahasure let me pull up the test18:12
Anticimexi ahve it up too, just need som docs18:12
Anticimexgoogle "rspec tutorial" wasn't too bad though.  i was using a bit too specific search terms18:13
mwhahahahttp://rspec-puppet.com/tutorial/18:13
Anticimexthx!18:13
mwhahahabut you can just switch it to is_expected.to_not contain_nova_config('DEFAULT/vncserver_proxyclient_address')18:13
Anticimexthx18:14
mwhahahaactually18:14
mwhahahashould be18:14
mwhahahais_expected.to contain_nova_config('DEFAULT/vncserver_proxyclient_address').with_ensure('absent')18:14
mwhahahabased on your change18:14
Anticimexright, based on the patch18:14
Anticimexack18:14
Anticimexthat's to handle switches back and forth18:15
mwhahahaand also should add a similar line for vnc_keymap18:15
Anticimexit seems the tests are just modeled after the code18:15
Anticimexack18:15
mwhahahayea they just copied https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_compute_spec.rb#L92-L9818:15
mwhahahashould also update that block to reflect the vnc_keymap thing as well18:16
Anticimexright, will do18:18
*** paramite has quit IRC18:22
openstackgerritMartin Millnert proposed openstack/puppet-nova: Allow spice protocol to be deployed  https://review.openstack.org/26377718:23
*** angdraug has joined #puppet-openstack18:25
*** rmart04 has joined #puppet-openstack18:30
*** dgurtner has quit IRC18:34
Anticimexmwhahaha: the review got stuck it seems on "Needs Workflow"18:37
Anticimexis that due to changes in tests?18:37
mwhahahanope, that means it needs to be reviewed18:37
*** _ducttape_ has quit IRC18:37
Anticimexack18:38
mwhahahaonce the tests pass and the cores will review and merge if there are no issues18:39
*** jaosorior has quit IRC18:43
*** rmart04 has quit IRC18:44
Anticimexmwhahaha: i guess tests are just on queue for now? there's no need to manually issue recheck (and where has that gone in this new gerrit web anyway?)18:58
Anticimexi'm familiar with the work flow in general18:58
mwhahahaAnticimex: yea you can check the status of the queue over at http://status.openstack.org/zuul/19:06
mwhahahaif for some reason they fail because of rubygem issues or whatever you can just reply with 'recheck' and it'll get requeued, but it looks like it passed all the unit tests, or at least the 3.x version so far19:07
Anticimexyeah there seemed to be one external "infra" error in the first batch of checks. couldn't create some machines or whatnot19:07
*** trown|lunch is now known as trown19:20
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: Neutron parses the wrong json revert to shell/csv.  https://review.openstack.org/26280919:28
*** ducttape_ has joined #puppet-openstack19:30
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: Correctly parse csv even with extra output.  https://review.openstack.org/26387419:34
openstackgerritAlex Schultz proposed openstack/puppet-neutron: Remove server_service from params  https://review.openstack.org/26387519:35
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add keystone domain specific configuration.  https://review.openstack.org/21928919:36
*** gfidente has quit IRC19:38
dprinceEmilienM: I'm leaving some feedback on the auth_uri/identity_uri patches https://review.openstack.org/#/c/261326/19:40
dprinceEmilienM: basically wondering if our defaults there are correct...19:40
dprinceEmilienM: I think keystone would very much like to see hard coded v2.0 go away19:41
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add keystone domain specific configuration.  https://review.openstack.org/21928919:42
*** foxhunt_away is now known as foxhunt7219:44
EmilienMdprince: youre right19:51
EmilienMdprince: fwiw, other patches are version less19:52
openstackgerritMerged openstack/puppet-swift: Set mount_check to True by default  https://review.openstack.org/26133319:53
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Implement WSGI support for Ironic API  https://review.openstack.org/26345020:07
openstackgerritAdam Vinsh proposed openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/26376021:01
VinshIndentation fix.21:02
openstackgerritBradley Klein proposed openstack/puppet-monasca: Add script to prune data older than a retention period  https://review.openstack.org/26390821:05
openstackgerritAdam Vinsh proposed openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/26376021:16
*** Vinsh has quit IRC21:29
*** Vinsh has joined #puppet-openstack21:30
openstackgerritEmilien Macchi proposed openstack/puppet-tuskar: Retire openstack/puppet-tuskar  https://review.openstack.org/26391921:32
EmilienMslagle, dprince: please review https://review.openstack.org/26391921:32
EmilienMit's puppet-tuskar killing21:32
EmilienMoh I think it won't pass CI21:33
*** Vinsh_ has joined #puppet-openstack21:33
*** Vinsh has quit IRC21:37
openstackgerritBradley Klein proposed openstack/puppet-monasca: Add script to prune data older than a retention period  https://review.openstack.org/26390821:38
*** baker has quit IRC21:44
*** foxhunt72 is now known as foxhunt_away21:45
*** baker has joined #puppet-openstack21:49
EmilienMInfr21:51
EmilienMargh21:51
*** cdelatte has quit IRC21:56
*** dgurtner has joined #puppet-openstack22:05
*** dgurtner has quit IRC22:05
*** dgurtner has joined #puppet-openstack22:05
*** trown is now known as trown|outttypeww22:09
*** baker has quit IRC22:16
*** baker has joined #puppet-openstack22:19
*** dgurtner has quit IRC22:21
*** jayg is now known as jayg|g0n322:26
*** dprince has quit IRC22:40
openstackgerritMerged openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type  https://review.openstack.org/26350323:02
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: rabbitmq: make sure RabbitMQ resources are created before running OpenStack  https://review.openstack.org/26307323:05
openstackgerritMerged openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix  https://review.openstack.org/25988523:06
openstackgerritMerged openstack/puppet-ceilometer: deprecate mongodb_replica_set parameter  https://review.openstack.org/26060423:08
*** dhollinger has quit IRC23:10
openstackgerritMerged openstack/puppet-openstack-integration: rabbitmq: make sure RabbitMQ resources are created before running OpenStack  https://review.openstack.org/26307323:13
*** angdraug has quit IRC23:16
openstackgerritEmilien Macchi proposed openstack/puppet-aodh: Support MongoDB as backend db  https://review.openstack.org/26394523:17
openstackgerritMerged openstack/puppet-neutron: fwaas config needs to notify l3 agent service  https://review.openstack.org/26374523:18
*** morazi has quit IRC23:18
*** baker has quit IRC23:26
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26394823:33
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Fix ::gnocchi::statsd class parameters documentation  https://review.openstack.org/26395023:41
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Implement ::gnocchi::client  https://review.openstack.org/26395123:42
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26395423:47
openstackgerritEmilien Macchi proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26395523:52
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26395623:53
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26395723:53
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Follow-up on PyMySQL support for Red Hat platforms  https://review.openstack.org/26395923:55
« Monday, 2016-01-04 Index Wednesday, 2016-01-06 »

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