Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:11 michchap_ joined #puppet-openstack
00:35 ducttape_ joined #puppet-openstack
00:50 ducttape_ joined #puppet-openstack
00:53 EmilienM _ody: probably a packaging problem
00:53 EmilienM probably in https://github.com/rdo-packages/murano-dis​tgit/blob/mitaka-rdo/openstack-murano.spec
00:54 EmilienM I don't see a package for tempest
00:54 EmilienM iurygregory: yes
00:55 markvoelker joined #puppet-openstack
00:55 iurygregory_ EmilienM, ack tks
00:55 iurygregory_ swift is the last one \o/
00:56 iurygregory_ all other patches are ready and upstream :D (heat, magnum, neutron, trove)
00:58 EmilienM iurygregory_: excellent work here
00:58 iurygregory_ EmilienM, tks :D
01:36 openstackgerrit Merged openstack/puppet-swift: Add log directory to .gitignore  https://review.openstack.org/359337
01:47 ilbot3 joined #puppet-openstack
01:47 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules http://docs.openstack.org/deve​loper/puppet-openstack-guide/
01:48 openstackgerrit Merged openstack/puppet-keystone: Add CORS settings  https://review.openstack.org/354087
01:52 ducttape_ joined #puppet-openstack
02:02 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004  https://review.openstack.org/339106
02:12 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Move magnum to authtoken  https://review.openstack.org/360146
02:15 ducttape_ joined #puppet-openstack
02:27 honglei joined #puppet-openstack
02:28 simonmcc joined #puppet-openstack
02:53 ducttape_ joined #puppet-openstack
02:59 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Allow setting of object name and namespace name max length  https://review.openstack.org/358029
03:05 openstackgerrit Merged openstack/puppet-nova: Remove manager option  https://review.openstack.org/358413
03:20 openstackgerrit ayoung proposed openstack/puppet-tripleo: Optionally include mod auth mellon for apache  https://review.openstack.org/359942
03:22 ducttape_ joined #puppet-openstack
04:20 openstackgerrit Michael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken  https://review.openstack.org/351980
04:23 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-swift: Change swift authtoken  https://review.openstack.org/360216
04:23 ducttape_ joined #puppet-openstack
04:26 iurygregory_ Vinsh, fyi https://review.openstack.org/360216, going to sleep now =)
04:55 morazi joined #puppet-openstack
05:10 jaosorior joined #puppet-openstack
05:25 ducttape_ joined #puppet-openstack
05:27 johnmilton joined #puppet-openstack
05:52 simonmcc joined #puppet-openstack
05:59 skramaja joined #puppet-openstack
06:26 ducttape_ joined #puppet-openstack
06:27 johnmilton joined #puppet-openstack
06:28 rcernin joined #puppet-openstack
06:37 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/359973
06:44 openstackgerrit Karthik S proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots  https://review.openstack.org/352494
07:01 dgurtner joined #puppet-openstack
07:08 tesseract- joined #puppet-openstack
07:20 johnmilton joined #puppet-openstack
07:26 ducttape_ joined #puppet-openstack
07:30 davidmichaelkarr joined #puppet-openstack
07:32 AndyWojo joined #puppet-openstack
07:47 shardy joined #puppet-openstack
07:57 jmiu joined #puppet-openstack
08:22 chem joined #puppet-openstack
08:22 skramaja joined #puppet-openstack
08:27 ducttape_ joined #puppet-openstack
08:30 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/360323
08:37 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Terminate Zaqar websocket endpoint in HAProxy  https://review.openstack.org/360329
08:43 aarefiev_ joined #puppet-openstack
08:45 mkarpin1 joined #puppet-openstack
08:47 openstackgerrit Martin André proposed openstack/puppet-swift: Trim useless blank lines in generated config file  https://review.openstack.org/359342
08:49 openstackgerrit joined #puppet-openstack
08:56 openstackgerrit Javier Peña proposed openstack/puppet-openstack-integration: Move all lbaas to v2  https://review.openstack.org/360354
09:00 derekh joined #puppet-openstack
09:01 davideagle joined #puppet-openstack
09:27 openstackgerrit Martin André proposed openstack/puppet-tripleo: [WIP] UI profile for tripleo  https://review.openstack.org/354630
09:28 ducttape_ joined #puppet-openstack
09:46 openstackgerrit Karthik S proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots  https://review.openstack.org/352494
09:52 danpawlik joined #puppet-openstack
09:56 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Update  Midonet neutron plugin to 5.x  https://review.openstack.org/360390
10:02 xarses_ joined #puppet-openstack
10:07 richm joined #puppet-openstack
10:13 ddmitriev joined #puppet-openstack
10:22 chem joined #puppet-openstack
10:26 social chem: ping about https://review.openstack.org/#/c/354125/ what do you think about iurygregory point to use "Defaults to ::ironic::params::variable"?
10:29 ducttape_ joined #puppet-openstack
10:29 chem iurygregory, regarding the social review ^, do you have other examples where a variable is used for a default ?  IMHO, it's more readable in a doc to have a something fixed as a default than a variable.  WDYT ?
11:09 dtantsur iberezovskiy|off or somebody: this breaks Ironic: https://github.com/openstack/puppet-ironic/co​mmit/eef5b11679e1c0e96a782f51f5e0c8aa6abfb3bd
11:10 dtantsur I think we should revert it and then provide a complete set of credentials
11:13 dtantsur or we should make sure we fill in all other credentials that used to fall back to the removed ones... which I think is a bit too optimistic right before the release
11:14 * dtantsur looks what can be fixed on ironic side
11:26 jgr joined #puppet-openstack
11:34 iberezovskiy|off dtantsur, what's actually wrong? could you please provide logs and steps to reproduce the issue
11:34 dtantsur so, I've filed a bug, really not sure who to proceed: https://bugs.launchpad.net/​puppet-ironic/+bug/1616873
11:34 openstack Launchpad bug 1616873 in puppet-ironic "Accessing keystone catalog from conductor broken after recent refactoring" [Undecided,New]
11:34 dtantsur iberezovskiy, see the bug ^^^ you can reproduce by installing ironic in overcloud, but it will take you some time
11:34 dtantsur tripleo overrides several options that probably make this not reproducible for it
11:35 dtantsur but tl;dr is: you've removed some options that are used as fall back for (still missing) new options
11:40 iberezovskiy hah, interesting
11:42 dtantsur yeah, it's a bit hacky, I know... but it's tricky to keep backward compatibility when migrating to a completely new set of options that is not even controlled by us
11:46 iberezovskiy from your log, it can't find username
11:46 iberezovskiy but it will be set if you pass it
11:48 iberezovskiy dtantsur, I need help with reproducing the issue
11:49 dtantsur well, we no longer pass it, right?
11:50 dtantsur I mean, we used to pass admin_username, which is something we expect
11:50 dtantsur now it passes just username, ironic does not know to fall back to it
11:50 dtantsur (this is why I call it hacky)
11:51 iberezovskiy I don't see admin_username option in default ironic conf
11:52 jaosorior joined #puppet-openstack
11:53 dtantsur me neither, but it might be due to deprecation
11:53 iberezovskiy if so, it should be marked as deprecated
11:54 * dtantsur has no clue, but it clearly no longer finds the username option
11:55 dprince joined #puppet-openstack
11:56 iberezovskiy this is how we configure ironic auth in fuel https://github.com/openstack/fuel-librar​y/blob/master/deployment/puppet/openstac​k_tasks/manifests/ironic/ironic.pp#L94
12:00 dtantsur iberezovskiy, yeah, but tripleo got switched to new options
12:00 dtantsur iberezovskiy, https://github.com/openstack/tripleo-heat-template​s/commit/b5a54bf9852ab24c8c7e0501b258b9fa26e2ad01
12:00 * iberezovskiy looking
12:02 dtantsur iberezovskiy, I guess this bit gets broken: https://github.com/openstack/ironic/blob/​master/ironic/common/keystone.py#L95-L98
12:02 dtantsur it still relies on old names
12:04 iberezovskiy damn
12:05 dtantsur I can't even find where these names come from Oo neither in keystonemiddleware not in keystoneauth
12:06 * dtantsur adds ironic to the bug
12:06 dgurtner joined #puppet-openstack
12:06 dgurtner joined #puppet-openstack
12:11 ducttape_ joined #puppet-openstack
12:12 iberezovskiy https://github.com/openstack/keyston​emiddleware/blob/master/keystonemidd​leware/auth_token/_auth.py#L34-L40
12:13 iberezovskiy https://github.com/openstack/keystone​middleware/blob/master/keystonemiddle​ware/auth_token/__init__.py#L191-L198
12:13 degorenko joined #puppet-openstack
12:14 iberezovskiy joined #puppet-openstack
12:14 degorenko joined #puppet-openstack
12:14 iberezovskiy dtantsur, ^
12:14 openstackgerrit Jiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary  https://review.openstack.org/358510
12:14 * dtantsur looking
12:15 skolekonov joined #puppet-openstack
12:20 colonwq joined #puppet-openstack
12:22 shardy joined #puppet-openstack
12:23 mwhahaha_ joined #puppet-openstack
12:24 vern joined #puppet-openstack
12:24 dtantsur iberezovskiy, hmm, I'm trying to switch ironic to newer options and I get "no such option in group keystone_authtoken: username"
12:24 ansmith joined #puppet-openstack
12:24 dtantsur wut
12:25 mfisch joined #puppet-openstack
12:25 iberezovskiy omg
12:25 mfisch joined #puppet-openstack
12:26 dtantsur iberezovskiy, I suspect the easiest thing we can do on our side (while I'm figuring our ironic side) is to bring back setting admin_* stuff to the same values
12:27 iberezovskiy so I don't understand only one thing. how did it work on CI
12:27 dtantsur which CI you mean?
12:27 iberezovskiy openstack-integration checks
12:28 dtantsur do we launch ironic instances there?
12:28 iberezovskiy we have scenario with ironic
12:28 iberezovskiy and we run tempest for it
12:29 * dtantsur checks what happened there
12:29 iberezovskiy http://logs.openstack.org/28/346028/14/gate​/gate-puppet-openstack-integration-3-scenar​io002-tempest-centos-7/59c18ce/console.html
12:30 mtreinish joined #puppet-openstack
12:31 iberezovskiy i don't see the sest for ironic instances
12:31 iberezovskiy test*
12:32 dtantsur right, we don't boot anything http://logs.openstack.org/28/346028/14/ch​eck/gate-puppet-openstack-integration-3-s​cenario002-tempest-centos-7/7dd01e0/logs/​ironic/ironic-conductor.txt.gz?level=INFO
12:32 dtantsur this is how it passed
12:33 iberezovskiy :(
12:33 dtantsur now, what to do...
12:34 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ironic: Revert "Configure keystone authtoken options"  https://review.openstack.org/360499
12:35 iberezovskiy dtantsur, but we still need to update ironic
12:36 dtantsur iberezovskiy, wait a second with the revert, I think we can do it easier
12:36 iberezovskiy ok
12:37 EmilienM morning
12:37 morazi joined #puppet-openstack
12:37 iberezovskiy hi
12:37 dtantsur morning EmilienM, a lot of fun today
12:38 EmilienM dtantsur: excellent !
12:39 iberezovskiy joined #puppet-openstack
12:39 skolekonov joined #puppet-openstack
12:40 * iberezovskiy have some troubles with connectivity
12:40 EmilienM review.openstack.org is unreachable for me
12:41 chem hi
12:42 dtantsur wonderful
12:42 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Set fallback authentication options for Ironic  https://review.openstack.org/360506
12:42 cdelatte joined #puppet-openstack
12:42 EmilienM dtantsur: thx for helping
12:42 EmilienM (as usual)
12:43 dtantsur :)
12:44 degorenko joined #puppet-openstack
12:44 degorenko_ joined #puppet-openstack
12:45 * dtantsur writes tests for his patc
12:45 iberezovskiy joined #puppet-openstack
12:47 derekh joined #puppet-openstack
12:47 fxpester joined #puppet-openstack
12:48 markvoelker joined #puppet-openstack
12:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing"  https://review.openstack.org/339813
12:52 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Set fallback authentication options for Ironic  https://review.openstack.org/360506
12:52 dtantsur EmilienM, iberezovskiy ^^^
12:52 AJaeger joined #puppet-openstack
12:53 EmilienM dtantsur: looking
12:53 AJaeger EmilienM, could you +W https://review.openstack.org/#/c/355001/ , please? It has two +2s already
12:53 dtantsur this is much less invasive than revert the whole thing
12:53 iberezovskiy dtantsur, thanks
12:53 EmilienM dtantsur: commented
12:54 dtantsur oops, gimme a second to fix
12:54 dtantsur EmilienM, actually it's missing because it's missing in the place where I copy-pasted it from
12:54 EmilienM oops
12:55 dtantsur I'll fix both
12:56 mkarpin joined #puppet-openstack
13:01 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Set fallback authentication options for Ironic  https://review.openstack.org/360506
13:01 dtantsur EmilienM, iberezovskiy, updated, please check ^^^
13:01 openstackgerrit Javier Peña proposed openstack/puppet-openstack-integration: Move all lbaas to v2  https://review.openstack.org/360354
13:02 EmilienM ok I want to see tripleo CI
13:02 iurygregory morning
13:02 EmilienM but yeah +2
13:03 social iurygregory: ping about the ironic::pxe review and options
13:03 iurygregory social, go ahead
13:04 iberezovskiy once all CI are green, I'll approve
13:04 social iurygregory: I commented yesterday also on review. Do you really prefer if I specify variable ironic::params::var instead of what the pick function will default to?
13:04 dtantsur thanks (I wonder how much OoO CI will take though...)
13:06 iurygregory social, in my mind i think the pick should be at other classes that will use.. that's why I put 'Not sure this is the best way' hahah =) I would appreciate a review from any core about this ^^
13:07 morazi joined #puppet-openstack
13:09 paramite joined #puppet-openstack
13:11 skramaja joined #puppet-openstack
13:15 johnmilton joined #puppet-openstack
13:15 social iurygregory: thanks, well if I use it in the other classes the code would look much uglyer, it actually makes it nice if I use params class fro this
13:20 iurygregory social, humm ok o/
13:20 iurygregory let's wait for someelse before you send the patch, i can remove the -1 so people will review I think haha
13:22 social chem: ^^
13:22 * dtantsur does not have enough understanding of puppet to review this aspect :)
13:22 ducttape_ joined #puppet-openstack
13:24 chem social: ack, will do later, buzy right now
13:28 Ahharu joined #puppet-openstack
13:29 Ahharu I am having trouble with this patch https://review.openstack.org/#/c/359973/3 because I keep being told to do a cherry-pick from master but that's not possible since the code is not exactly the same for the 2 branches, on MASTER the command(neutron) is defined at the "neutron" top scope provider and on stable/mitaka it is defined on each type
13:29 Ahharu What should I do?
13:33 iurygregory Ahharu, the same fix from master will not work in stable/mitaka? well this is strange because we should make patches backward compatible .-.
13:34 iurygregory degorenko, i think heat broken fuel, did you check? i saw your +2 =)
13:35 Ahharu nope, it will break it
13:35 Ahharu because it will say that command neutron is not found
13:35 degorenko iurygregory, no, job failed not because of your patch
13:35 iurygregory Ahharu, lol O.o
13:35 iurygregory degorenko, ack, tks man :D
13:36 Ahharu the thing is that is the command neutron declaration was moved into the "neutron.rb" provider
13:36 Ahharu on which other types inherit
13:36 Ahharu makes sense
13:39 fultonj joined #puppet-openstack
13:42 ansmith_ joined #puppet-openstack
13:48 Ahharu I think I can do it this way..
13:49 Ahharu cherry pick this 8a85bb0860e334bced12c86771e4ef23c6731dc2 into stable/mitaka
13:49 Ahharu and backport from master
13:49 iurygregory what?
13:49 iurygregory backport to master? O.o
13:49 haaj joined #puppet-openstack
13:49 Ahharu from master
13:49 dfisher joined #puppet-openstack
13:50 Ahharu this change 8a85bb0860e334bced12c86771e4ef23c6731dc2 takes the command neutron to top-scope
13:50 iurygregory Ahharu, sorry still sleeping hahah
13:50 Ahharu :D :D it's already afternoon here
13:51 openstackgerrit Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::pxe class  https://review.openstack.org/354125
13:51 dsavineau joined #puppet-openstack
13:52 alee joined #puppet-openstack
13:57 ebalduf joined #puppet-openstack
14:04 NachoDuck joined #puppet-openstack
14:07 AJaeger EmilienM, while I have your attention on reviewing ;), could you +W https://review.openstack.org/#/c/355001/ , please? It has two +2s already
14:08 iurygregory degorenko, ^
14:08 EmilienM AJaeger: done
14:08 AJaeger thanks
14:10 iurygregory EmilienM, since magnum only have release 9.0.0 and 9.1.0 the authtoken should be backward compatible?
14:10 EmilienM iurygregory: no
14:10 EmilienM break magnum
14:10 iurygregory EmilienM, ok, degorenko you're right :D
14:10 EmilienM I'm not sure someone is using it
14:11 EmilienM and we have no real functional testing yet on it
14:11 AJaeger left #puppet-openstack
14:11 iurygregory EmilienM, yeah, we have two patches for authtoken i'll abandon my patch since the other is older
14:11 EmilienM ok
14:12 ducttape_ joined #puppet-openstack
14:19 AJaeger joined #puppet-openstack
14:20 AJaeger EmilienM: my change is stacked on top of https://review.openstack.org/#/c/355202 ;(
14:20 AJaeger Any cores around to review that one, please? It's needed to pass the testsuite...
14:21 openstackgerrit Matt Fischer proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/344464
14:22 EmilienM iberezovskiy, degorenko: ^ can you review it please?
14:23 iurygregory mwhahaha, you around?
14:23 mwhahaha for a few
14:23 mwhahaha what's up
14:24 iurygregory mwhahaha, auth_strategy for new modules make sense? we don't need to include the class in the api.pp since we don't have release...
14:25 iurygregory like: ec2api, watcher, magnum...
14:25 mwhahaha my concern around not having is that we won't notify the end user that they need to provide the other class for keystone enablement
14:25 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/360584
14:25 mwhahaha if you include it, then automatically they get an error about having to configure the keystone options
14:26 mwhahaha otherwise you'll end up with services with possibly no auth by default
14:26 iurygregory mwhahaha, ack ;)
14:27 mwhahaha time to bus children to school, back in 30
14:33 karimb joined #puppet-openstack
14:36 Ahharu ok, so after this is backported, I can backport the neutron port fix thing from master
14:36 Ahharu :)
14:37 iurygregory Ahharu, after the patch is merged, yes :D
14:38 Ahharu sure, will wait for it
14:43 bogdando joined #puppet-openstack
14:48 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/360608
14:48 degorenko EmilienM, i'm ready to +2 it :D waiting for ci
14:49 openstackgerrit joined #puppet-openstack
14:50 EmilienM degorenko: thx
14:50 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken  https://review.openstack.org/351980
14:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests"  https://review.openstack.org/360610
14:50 EmilienM dmburmistrov: FYI  ^
14:51 degorenko not sure that Dima is person who knows about this :D
14:53 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Move magnum to authtoken  https://review.openstack.org/351980
14:54 finchd joined #puppet-openstack
14:54 aviramb joined #puppet-openstack
14:56 xarses_ joined #puppet-openstack
14:56 iurygregory degorenko, can you review magnum? :D
14:57 AJaeger left #puppet-openstack
14:58 finchd left #puppet-openstack
14:59 degorenko iurygregory, is it fair that you updated magnum patch? :)
15:00 iurygregory degorenko, yeah because we need to finish authtoken i think XD
15:01 degorenko iurygregory, but Michael is working on that right now, last update from his side was today :)
15:01 iurygregory degorenko, oh D: i was fast sorry .-.
15:01 degorenko iurygregory, ok, don't worry :)
15:01 iurygregory but he can update if you see something wrong =)
15:03 openstackgerrit Jiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary  https://review.openstack.org/358510
15:04 degorenko iurygregory, i'm ok with patch, one nit inline, also probably we need to update commit message, to reflect real goal of this change
15:05 iurygregory degorenko, ok I'll let Michael do this =) put in the review about the commit message please =)
15:06 degorenko ya
15:13 bnemec joined #puppet-openstack
15:17 openstackgerrit Jiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary  https://review.openstack.org/358510
15:19 furlongm_ joined #puppet-openstack
15:21 degorenko EmilienM, iurygregory , mwhahaha how we missed that one https://github.com/openstack/puppet-nova/blob​/master/manifests/keystone/authtoken.pp#L226 ? :(
15:22 EmilienM oops
15:22 EmilienM degorenko: can you submit a patch?
15:22 mwhahaha cause 4.6.0
15:22 mwhahaha :(
15:22 degorenko not exactly
15:22 degorenko that's because our acceptance doesn't have include for main nova class
15:23 degorenko i don't know why :D
15:23 degorenko ok, i'll do patch
15:23 mwhahaha interesting
15:26 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Remove unneeded include for main class  https://review.openstack.org/360632
15:26 degorenko EmilienM, mwhahaha ^
15:27 EmilienM degorenko: +6
15:27 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/359967
15:27 degorenko :D
15:30 openstackgerrit Emilien Macchi proposed openstack/puppet-panko: puppet-panko: Initial commit  https://review.openstack.org/360639
15:30 Ahharu it is some kind of weird neutron cli thing
15:31 mwhahaha puppet-panko wtf is that
15:31 mwhahaha does it takes delicious on chicken?
15:31 EmilienM lol
15:31 Ahharu the network name is defined by checking the first thing that is "alone" , I mean without -- or containing "=" sign
15:31 EmilienM mwhahaha: https://github.com/openstack/panko
15:31 EmilienM mwhahaha: a new telemetry service
15:32 degorenko lol
15:32 EmilienM mwhahaha: for event storage this time
15:32 mwhahaha of course it is
15:32 degorenko we need more telemetry services!
15:33 degorenko ah, it's just renaming
15:33 EmilienM degorenko: no, it's moving the code
15:33 EmilienM mwhahaha: I actually like how telemetry split things, well it brings us more work but it help to scale the services
15:33 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/359967
15:33 EmilienM pradk from telemetry team is going to take over my patch
15:33 degorenko ah, ok
15:34 mwhahaha more things to eat ram </troll>
15:34 EmilienM and will write the module
15:34 EmilienM he's already familiar with it
15:34 derekh joined #puppet-openstack
15:36 pradk joined #puppet-openstack
16:10 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade  https://review.openstack.org/360660
16:10 EmilienM dtantsur: ^
16:12 dtantsur EmilienM, I suspect the debian package name is wrong there
16:12 EmilienM dtantsur: http://ubuntu-cloud.archive.canoni​cal.com/ubuntu/pool/main/i/ironic/
16:12 EmilienM it's python-ironic
16:12 dtantsur no, I don't think so
16:12 dtantsur https://packages.debian.org/e​xperimental/python-ironic-lib
16:13 iurygregory degorenko, well the reason to include was because memcached was not evaluated
16:13 dtantsur EmilienM, I think python-ironic is just python modules extracted from the main Ironic package
16:13 dtantsur EmilienM, here you go http://ubuntu-cloud.archive.canonical.co​m/ubuntu/pool/main/p/python-ironic-lib/ :)
16:13 EmilienM dtantsur: i'm not sure it's in ubuntu though
16:13 EmilienM oh
16:13 EmilienM you got it
16:13 EmilienM damn !
16:13 EmilienM your the best
16:13 dtantsur :)
16:15 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade  https://review.openstack.org/360660
16:15 degorenko iurygregory, then this class should be included before authtoken
16:16 degorenko not inside
16:16 iurygregory degorenko, oh sorry .-.
16:16 degorenko ok, no problem
16:17 iurygregory social, EmilienM put the same comment =)
16:19 dprince joined #puppet-openstack
16:21 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/360608
16:22 iurygregory EmilienM, why you depend on the same patch?
16:22 iurygregory '-'
16:22 iurygregory Depends-On: I8f5b7f3d7392bfa4215fccd7defe953857cef3f6
16:22 iurygregory Change-Id: I8f5b7f3d7392bfa4215fccd7defe953857cef3f6
16:23 EmilienM ah
16:23 EmilienM I'm tired
16:23 iurygregory it happens
16:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/360608
16:25 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests"  https://review.openstack.org/360610
16:47 chem internet is slow today
16:48 EmilienM chem: gerrit is slow today
16:53 dprince joined #puppet-openstack
16:57 _ducttape_ joined #puppet-openstack
16:57 dgurtner joined #puppet-openstack
17:08 skramaja joined #puppet-openstack
17:10 7JTABL8FZ joined #puppet-openstack
17:14 openstackgerrit Merged openstack/puppet-swift: Add statsd configuration options  https://review.openstack.org/360032
17:22 AJaeger joined #puppet-openstack
17:25 AJaeger left #puppet-openstack
17:27 AnarchyAo joined #puppet-openstack
17:31 dsavineau left #puppet-openstack
17:51 ducttape_ joined #puppet-openstack
17:54 rcernin joined #puppet-openstack
18:12 openstackgerrit Ben Swartzlander proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class  https://review.openstack.org/354014
18:23 social EmilienM: is there any other way how to satisfy dtantsur's requirement for having a class for global override of options of classes that are independant? Or you would just preffer to have the pick function in the classes and the classes would inherit the common one?
18:44 iurygregory gerrit is slow only here? .-.
18:45 paramite joined #puppet-openstack
18:49 Epx998 joined #puppet-openstack
18:49 mwhahaha no it's  pretty bad today
18:50 Epx998 left #puppet-openstack
19:14 openstackgerrit Merged openstack/puppet-trove: Policy support for Trove  https://review.openstack.org/359948
19:17 openstackgerrit Merged openstack/puppet-nova: Add support for new vendordata plugins  https://review.openstack.org/359604
19:26 openstackgerrit Merged openstack/puppet-openstack-integration: Move all lbaas to v2  https://review.openstack.org/360354
19:43 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade  https://review.openstack.org/360660
19:46 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/360608
19:46 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests"  https://review.openstack.org/360610
19:47 openstackstatus NOTICE: The Gerrit service on review.openstack.org is restarting to implement some performance tuning adjustments, and should return to working order momentarily.
19:48 johnmilton joined #puppet-openstack
19:50 _stowa joined #puppet-openstack
20:02 openstackgerrit Merged openstack/puppet-cinder: Add support for allow_availability_zone_fallback parameter  https://review.openstack.org/356359
20:09 johnmilton joined #puppet-openstack
20:10 iurygregory EmilienM, TripleO does not run in puppet-heat?
20:10 openstackgerrit Brent Eagles proposed openstack/puppet-neutron: Adding support for overlay_ip_version  https://review.openstack.org/360766
20:10 EmilienM it should be
20:10 EmilienM iurygregory: show me patch
20:11 iurygregory EmilienM, https://review.openstack.org/#/c/359543/
20:11 iurygregory authtoken haha XD
20:11 EmilienM iurygregory: gate-tripleo-ci-centos-7-nonha-multinode
20:11 EmilienM it's green
20:11 EmilienM only fuel is red
20:12 iurygregory i've talked to denis it's not related
20:13 EmilienM ok I'll review it shortly
20:13 iurygregory EmilienM, tks take your time o/
20:14 iurygregory tomorrow i'll take a closer look in all modules to fix some nits like metadata is ok, and make sure we are using auth_strategy =)
20:14 ducttape_ joined #puppet-openstack
20:15 EmilienM excellent
20:15 EmilienM we're getting close to release and feature freeze
20:15 EmilienM we might want to stabilize a bit our modules
20:15 iurygregory yeah, i'll make sure they are in the same shape for authtoken o/
20:16 iurygregory if you need any reviews let me know
20:20 EmilienM iurygregory: will do
20:28 johnmilton joined #puppet-openstack
20:41 EmilienM iurygregory: heat reviewed
20:42 iurygregory EmilienM, ack fixing right now
20:59 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004  https://review.openstack.org/339106
21:08 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-heat: Move Heat to authtoken  https://review.openstack.org/359543
21:18 openstackgerrit Merged openstack/puppet-nova: Remove unneeded include for main class  https://review.openstack.org/360632
21:27 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/359967
21:33 openstackgerrit Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/359967
21:35 Ahharu_ joined #puppet-openstack
21:35 Ahharu_ @EmilienM how should I proceed with https://review.openstack.org/#/c/360390/ ?
21:36 Ahharu_ code is different from master and mitaka
21:36 Ahharu_ in terms of Anchors, requires and stuff
21:39 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: Change authtoken to class  https://review.openstack.org/356792
22:01 openstackgerrit Merged openstack/puppet-swift: Trim useless blank lines in generated config file  https://review.openstack.org/359342
22:03 EmilienM Ahharu: ah :( indeed
22:03 EmilienM Ahharu: please specify it in commit message then
22:07 openstackgerrit Merged openstack/puppet-swift: Change swift authtoken  https://review.openstack.org/360216
22:07 openstackgerrit Merged openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
22:18 chem|off joined #puppet-openstack
22:26 ducttape_ joined #puppet-openstack
22:36 ntpttr joined #puppet-openstack
22:46 chem|off` joined #puppet-openstack
22:53 openstackgerrit Merged openstack/puppet-tripleo: SR-IOV nic agent changes in line with composable roles.  https://review.openstack.org/332191
23:00 ducttape_ joined #puppet-openstack
23:18 ebalduf joined #puppet-openstack
23:23 ducttape_ joined #puppet-openstack
23:26 chem|off` joined #puppet-openstack
23:32 beagles left #puppet-openstack
23:47 ducttape_ joined #puppet-openstack
23:56 xarses joined #puppet-openstack

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