*** michchap_ has joined #puppet-openstack | 00:11 | |
*** michchap has quit IRC | 00:13 | |
*** ducttape_ has quit IRC | 00:21 | |
*** Ahharu has quit IRC | 00:32 | |
*** ducttape_ has joined #puppet-openstack | 00:35 | |
*** ducttape_ has quit IRC | 00:40 | |
*** ducttape_ has joined #puppet-openstack | 00:50 | |
*** ducttape_ has quit IRC | 00:51 | |
EmilienM | _ody: probably a packaging problem | 00:53 |
---|---|---|
EmilienM | probably in https://github.com/rdo-packages/murano-distgit/blob/mitaka-rdo/openstack-murano.spec | 00:53 |
EmilienM | I don't see a package for tempest | 00:54 |
EmilienM | iurygregory: yes | 00:54 |
*** markvoelker has joined #puppet-openstack | 00:55 | |
iurygregory_ | EmilienM, ack tks | 00:55 |
iurygregory_ | swift is the last one \o/ | 00:55 |
iurygregory_ | all other patches are ready and upstream :D (heat, magnum, neutron, trove) | 00:56 |
EmilienM | iurygregory_: excellent work here | 00:58 |
iurygregory_ | EmilienM, tks :D | 00:58 |
*** saneax-_-|AFK is now known as saneax | 01:16 | |
*** ebalduf has quit IRC | 01:22 | |
*** saneax is now known as saneax-_-|AFK | 01:32 | |
*** markvoelker has quit IRC | 01:36 | |
openstackgerrit | Merged openstack/puppet-swift: Add log directory to .gitignore https://review.openstack.org/359337 | 01:36 |
*** ilbot3 has quit IRC | 01:45 | |
*** ilbot3 has joined #puppet-openstack | 01:47 | |
openstackgerrit | Merged openstack/puppet-keystone: Add CORS settings https://review.openstack.org/354087 | 01:48 |
*** ducttape_ has joined #puppet-openstack | 01:52 | |
*** ducttape_ has quit IRC | 01:57 | |
*** saneax-_-|AFK is now known as saneax | 01:58 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 02:02 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Move magnum to authtoken https://review.openstack.org/360146 | 02:12 |
*** ducttape_ has joined #puppet-openstack | 02:15 | |
*** saneax is now known as saneax-_-|AFK | 02:17 | |
*** simonmcc has quit IRC | 02:26 | |
*** honglei has joined #puppet-openstack | 02:27 | |
*** simonmcc has joined #puppet-openstack | 02:28 | |
*** honglei has quit IRC | 02:33 | |
*** ducttape_ has quit IRC | 02:46 | |
*** ducttape_ has joined #puppet-openstack | 02:53 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Allow setting of object name and namespace name max length https://review.openstack.org/358029 | 02:59 |
openstackgerrit | Merged openstack/puppet-nova: Remove manager option https://review.openstack.org/358413 | 03:05 |
*** saneax-_-|AFK is now known as saneax | 03:13 | |
*** ducttape_ has quit IRC | 03:16 | |
openstackgerrit | ayoung proposed openstack/puppet-tripleo: Optionally include mod auth mellon for apache https://review.openstack.org/359942 | 03:20 |
*** ducttape_ has joined #puppet-openstack | 03:22 | |
*** ducttape_ has quit IRC | 03:26 | |
*** finchd has quit IRC | 03:39 | |
*** morazi has quit IRC | 03:58 | |
*** saneax is now known as saneax-_-|AFK | 04:03 | |
openstackgerrit | Michael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken https://review.openstack.org/351980 | 04:20 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-swift: Change swift authtoken https://review.openstack.org/360216 | 04:23 |
*** ducttape_ has joined #puppet-openstack | 04:23 | |
iurygregory_ | Vinsh, fyi https://review.openstack.org/360216, going to sleep now =) | 04:26 |
*** iurygregory_ has quit IRC | 04:28 | |
*** ducttape_ has quit IRC | 04:28 | |
*** morazi has joined #puppet-openstack | 04:55 | |
*** ayoung has quit IRC | 05:03 | |
*** johnmilton has quit IRC | 05:07 | |
*** jaosorior has joined #puppet-openstack | 05:10 | |
*** ducttape_ has joined #puppet-openstack | 05:25 | |
*** johnmilton has joined #puppet-openstack | 05:27 | |
*** ducttape_ has quit IRC | 05:29 | |
*** richm has quit IRC | 05:40 | |
*** morazi has quit IRC | 05:46 | |
*** simonmcc has quit IRC | 05:51 | |
*** simonmcc has joined #puppet-openstack | 05:52 | |
*** Sanjay has quit IRC | 05:56 | |
*** skramaja has joined #puppet-openstack | 05:59 | |
*** johnmilton has quit IRC | 06:19 | |
*** ducttape_ has joined #puppet-openstack | 06:26 | |
*** johnmilton has joined #puppet-openstack | 06:27 | |
*** rcernin has joined #puppet-openstack | 06:28 | |
*** ducttape_ has quit IRC | 06:30 | |
openstackgerrit | Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options https://review.openstack.org/359973 | 06:37 |
openstackgerrit | Karthik S proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots https://review.openstack.org/352494 | 06:44 |
*** saneax-_-|AFK is now known as saneax | 06:55 | |
*** johnmilton has quit IRC | 06:56 | |
*** dgurtner has joined #puppet-openstack | 07:01 | |
*** tesseract- has joined #puppet-openstack | 07:08 | |
*** ifarkas_afk is now known as ifarkas | 07:09 | |
*** dgurtner has quit IRC | 07:13 | |
*** johnmilton has joined #puppet-openstack | 07:20 | |
*** ducttape_ has joined #puppet-openstack | 07:26 | |
*** davidmichaelkarr has quit IRC | 07:28 | |
*** AndyWojo has quit IRC | 07:29 | |
*** davidmichaelkarr has joined #puppet-openstack | 07:30 | |
*** saneax is now known as saneax-_-|AFK | 07:30 | |
*** ducttape_ has quit IRC | 07:31 | |
*** AndyWojo has joined #puppet-openstack | 07:32 | |
*** shardy has joined #puppet-openstack | 07:47 | |
*** jmiu has quit IRC | 07:51 | |
*** jmiu has joined #puppet-openstack | 07:57 | |
*** jpena|away is now known as jpena | 08:09 | |
*** skramaja has quit IRC | 08:15 | |
*** chem has joined #puppet-openstack | 08:22 | |
*** skramaja has joined #puppet-openstack | 08:23 | |
*** ducttape_ has joined #puppet-openstack | 08:27 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/360323 | 08:30 |
*** ducttape_ has quit IRC | 08:32 | |
*** dtantsur|afk is now known as dtantsur | 08:32 | |
*** saneax-_-|AFK is now known as saneax | 08:33 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Terminate Zaqar websocket endpoint in HAProxy https://review.openstack.org/360329 | 08:37 |
*** aarefiev_ has joined #puppet-openstack | 08:43 | |
*** mkarpin1 has joined #puppet-openstack | 08:45 | |
*** mkarpin has quit IRC | 08:46 | |
*** ddmitriev has quit IRC | 08:46 | |
*** aarefiev has quit IRC | 08:47 | |
openstackgerrit | Martin André proposed openstack/puppet-swift: Trim useless blank lines in generated config file https://review.openstack.org/359342 | 08:47 |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #puppet-openstack | 08:49 | |
*** aarefiev_ is now known as aarefiev | 08:50 | |
openstackgerrit | Javier Peña proposed openstack/puppet-openstack-integration: Move all lbaas to v2 https://review.openstack.org/360354 | 08:57 |
*** derekh has joined #puppet-openstack | 09:00 | |
*** davideagle has joined #puppet-openstack | 09:01 | |
openstackgerrit | Martin André proposed openstack/puppet-tripleo: [WIP] UI profile for tripleo https://review.openstack.org/354630 | 09:27 |
*** ducttape_ has joined #puppet-openstack | 09:28 | |
*** ducttape_ has quit IRC | 09:33 | |
openstackgerrit | Karthik S proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots https://review.openstack.org/352494 | 09:46 |
*** danpawlik has joined #puppet-openstack | 09:52 | |
openstackgerrit | Alex Ruiz Estradera proposed openstack/puppet-neutron: Update Midonet neutron plugin to 5.x https://review.openstack.org/360390 | 09:56 |
*** gfidente|afk is now known as gfidente | 09:56 | |
*** xarses_ has joined #puppet-openstack | 10:02 | |
*** xarses has quit IRC | 10:05 | |
*** richm has joined #puppet-openstack | 10:07 | |
*** ddmitriev has joined #puppet-openstack | 10:13 | |
*** chem has quit IRC | 10:22 | |
*** chem has joined #puppet-openstack | 10:22 | |
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:26 |
*** ducttape_ has 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 ? | 10:29 |
*** ducttape_ has quit IRC | 10:34 | |
dtantsur | iberezovskiy|off or somebody: this breaks Ironic: https://github.com/openstack/puppet-ironic/commit/eef5b11679e1c0e96a782f51f5e0c8aa6abfb3bd | 11:09 |
dtantsur | I think we should revert it and then provide a complete set of credentials | 11:10 |
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:13 |
* dtantsur looks what can be fixed on ironic side | 11:14 | |
*** karthiks_afk is now known as karthiks | 11:23 | |
*** _degorenko|afk is now known as degorenko | 11:24 | |
*** jgr has joined #puppet-openstack | 11:26 | |
*** jgr has quit IRC | 11:26 | |
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 |
*** iberezovskiy|off is now known as iberezovskiy | 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:34 |
dtantsur | but tl;dr is: you've removed some options that are used as fall back for (still missing) new options | 11:35 |
iberezovskiy | hah, interesting | 11:40 |
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:42 |
iberezovskiy | from your log, it can't find username | 11:46 |
iberezovskiy | but it will be set if you pass it | 11:46 |
iberezovskiy | dtantsur, I need help with reproducing the issue | 11:48 |
dtantsur | well, we no longer pass it, right? | 11:49 |
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:50 |
iberezovskiy | I don't see admin_username option in default ironic conf | 11:51 |
*** jaosorior has quit IRC | 11:52 | |
*** jaosorior has joined #puppet-openstack | 11:52 | |
dtantsur | me neither, but it might be due to deprecation | 11:53 |
iberezovskiy | if so, it should be marked as deprecated | 11:53 |
* dtantsur has no clue, but it clearly no longer finds the username option | 11:54 | |
*** dprince has joined #puppet-openstack | 11:55 | |
iberezovskiy | this is how we configure ironic auth in fuel https://github.com/openstack/fuel-library/blob/master/deployment/puppet/openstack_tasks/manifests/ironic/ironic.pp#L94 | 11:56 |
dtantsur | iberezovskiy, yeah, but tripleo got switched to new options | 12:00 |
dtantsur | iberezovskiy, https://github.com/openstack/tripleo-heat-templates/commit/b5a54bf9852ab24c8c7e0501b258b9fa26e2ad01 | 12:00 |
* iberezovskiy looking | 12:00 | |
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:02 |
*** jpena is now known as jpena|lunch | 12:04 | |
iberezovskiy | damn | 12:04 |
dtantsur | I can't even find where these names come from Oo neither in keystonemiddleware not in keystoneauth | 12:05 |
* dtantsur adds ironic to the bug | 12:06 | |
*** dgurtner has joined #puppet-openstack | 12:06 | |
*** dgurtner has quit IRC | 12:06 | |
*** dgurtner has joined #puppet-openstack | 12:06 | |
*** ducttape_ has joined #puppet-openstack | 12:11 | |
iberezovskiy | https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/auth_token/_auth.py#L34-L40 | 12:12 |
*** degorenko has quit IRC | 12:12 | |
iberezovskiy | https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/auth_token/__init__.py#L191-L198 | 12:13 |
*** degorenko has joined #puppet-openstack | 12:13 | |
*** iberezovskiy has quit IRC | 12:14 | |
*** skolekonov has quit IRC | 12:14 | |
*** degorenko has quit IRC | 12:14 | |
*** iberezovskiy has joined #puppet-openstack | 12:14 | |
*** degorenko has 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:14 | |
*** skolekonov has joined #puppet-openstack | 12:15 | |
*** colonwq has quit IRC | 12:19 | |
*** colonwq has joined #puppet-openstack | 12:20 | |
*** shardy has quit IRC | 12:21 | |
*** mfisch has quit IRC | 12:21 | |
*** shardy has joined #puppet-openstack | 12:22 | |
*** mwhahaha_ has joined #puppet-openstack | 12:23 | |
*** vern has quit IRC | 12:23 | |
*** mwhahaha has quit IRC | 12:23 | |
*** mwhahaha_ is now known as mwhahaha | 12:24 | |
*** vern has 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 has joined #puppet-openstack | 12:24 | |
dtantsur | wut | 12:24 |
*** mfisch has joined #puppet-openstack | 12:25 | |
iberezovskiy | omg | 12:25 |
*** mfisch has quit IRC | 12:25 | |
*** mfisch has joined #puppet-openstack | 12:25 | |
*** jaosorior is now known as jaosorior_away | 12:25 | |
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:26 |
*** mtreinish has quit IRC | 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:27 |
dtantsur | do we launch ironic instances there? | 12:28 |
*** ducttape_ has quit IRC | 12:28 | |
iberezovskiy | we have scenario with ironic | 12:28 |
iberezovskiy | and we run tempest for it | 12:28 |
* dtantsur checks what happened there | 12:29 | |
iberezovskiy | http://logs.openstack.org/28/346028/14/gate/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/59c18ce/console.html | 12:29 |
*** mtreinish has joined #puppet-openstack | 12:30 | |
iberezovskiy | i don't see the sest for ironic instances | 12:31 |
iberezovskiy | test* | 12:31 |
dtantsur | right, we don't boot anything http://logs.openstack.org/28/346028/14/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/7dd01e0/logs/ironic/ironic-conductor.txt.gz?level=INFO | 12:32 |
dtantsur | this is how it passed | 12:32 |
iberezovskiy | :( | 12:33 |
dtantsur | now, what to do... | 12:33 |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ironic: Revert "Configure keystone authtoken options" https://review.openstack.org/360499 | 12:34 |
iberezovskiy | dtantsur, but we still need to update ironic | 12:35 |
dtantsur | iberezovskiy, wait a second with the revert, I think we can do it easier | 12:36 |
iberezovskiy | ok | 12:36 |
EmilienM | morning | 12:37 |
*** morazi has joined #puppet-openstack | 12:37 | |
iberezovskiy | hi | 12:37 |
dtantsur | morning EmilienM, a lot of fun today | 12:37 |
*** iberezovskiy has quit IRC | 12:37 | |
*** skolekonov has quit IRC | 12:37 | |
EmilienM | dtantsur: excellent ! | 12:38 |
*** iberezovskiy has joined #puppet-openstack | 12:39 | |
*** skolekonov has joined #puppet-openstack | 12:39 | |
* iberezovskiy have some troubles with connectivity | 12:40 | |
EmilienM | review.openstack.org is unreachable for me | 12:40 |
chem | hi | 12:41 |
*** degorenko has quit IRC | 12:41 | |
*** iberezovskiy has quit IRC | 12:41 | |
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 has joined #puppet-openstack | 12:42 | |
EmilienM | dtantsur: thx for helping | 12:42 |
EmilienM | (as usual) | 12:42 |
dtantsur | :) | 12:43 |
*** derekh has quit IRC | 12:44 | |
*** ddmitriev has quit IRC | 12:44 | |
*** degorenko has joined #puppet-openstack | 12:44 | |
*** degorenko_ has joined #puppet-openstack | 12:44 | |
* dtantsur writes tests for his patc | 12:45 | |
*** iberezovskiy has joined #puppet-openstack | 12:45 | |
*** cdelatte has quit IRC | 12:46 | |
*** derekh has joined #puppet-openstack | 12:47 | |
*** fxpester has joined #puppet-openstack | 12:47 | |
*** markvoelker has joined #puppet-openstack | 12:48 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing" https://review.openstack.org/339813 | 12:50 |
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 has joined #puppet-openstack | 12:52 | |
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:53 |
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 |
*** mgagne_ is now known as mgagne | 12:54 | |
EmilienM | oops | 12:54 |
dtantsur | I'll fix both | 12:55 |
*** mkarpin1 has quit IRC | 12:55 | |
*** mkarpin has joined #puppet-openstack | 12:56 | |
*** jpena|lunch is now known as jpena | 12:56 | |
*** sanjayu|AFK is now known as sanjayu_ | 12:57 | |
*** morazi has quit IRC | 12:57 | |
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:01 |
EmilienM | ok I want to see tripleo CI | 13:02 |
iurygregory | morning | 13:02 |
EmilienM | but yeah +2 | 13:02 |
social | iurygregory: ping about the ironic::pxe review and options | 13:03 |
iurygregory | social, go ahead | 13:03 |
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:05 |
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:06 |
*** alee has quit IRC | 13:07 | |
*** morazi has joined #puppet-openstack | 13:07 | |
*** paramite has joined #puppet-openstack | 13:09 | |
*** skramaja has quit IRC | 13:10 | |
*** skramaja has joined #puppet-openstack | 13:11 | |
*** johnmilton has quit IRC | 13:14 | |
*** johnmilton has 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:15 |
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:20 |
social | chem: ^^ | 13:22 |
* dtantsur does not have enough understanding of puppet to review this aspect :) | 13:22 | |
*** ducttape_ has joined #puppet-openstack | 13:22 | |
chem | social: ack, will do later, buzy right now | 13:24 |
*** Ahharu has joined #puppet-openstack | 13:28 | |
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:29 |
*** ducttape_ has quit IRC | 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:33 |
iurygregory | degorenko, i think heat broken fuel, did you check? i saw your +2 =) | 13:34 |
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:35 |
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:36 |
*** fultonj has joined #puppet-openstack | 13:39 | |
*** ansmith_ has joined #puppet-openstack | 13:42 | |
*** jaosorior_away is now known as jaosorior | 13:48 | |
Ahharu | I think I can do it this way.. | 13:48 |
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 has joined #puppet-openstack | 13:49 | |
Ahharu | from master | 13:49 |
*** dfisher has 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 |
*** haaj has quit IRC | 13:50 | |
Ahharu | :D :D it's already afternoon here | 13:50 |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::pxe class https://review.openstack.org/354125 | 13:51 |
*** dsavineau has joined #puppet-openstack | 13:51 | |
*** alee has joined #puppet-openstack | 13:52 | |
*** ebalduf has joined #puppet-openstack | 13:57 | |
*** NachoDuck has quit IRC | 14:01 | |
*** NachoDuck has joined #puppet-openstack | 14:04 | |
*** xarses_ has quit IRC | 14:06 | |
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:07 |
iurygregory | degorenko, ^ | 14:08 |
*** jaosorior has quit IRC | 14:08 | |
EmilienM | AJaeger: done | 14:08 |
AJaeger | thanks | 14:09 |
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:10 |
EmilienM | and we have no real functional testing yet on it | 14:11 |
*** AJaeger has 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:11 |
*** ducttape_ has joined #puppet-openstack | 14:12 | |
*** AJaeger has joined #puppet-openstack | 14:19 | |
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:20 |
openstackgerrit | Matt Fischer proposed openstack/puppet-cinder: Cinder hooks support https://review.openstack.org/344464 | 14:21 |
EmilienM | iberezovskiy, degorenko: ^ can you review it please? | 14:22 |
iurygregory | mwhahaha, you around? | 14:23 |
mwhahaha | for a few | 14:23 |
mwhahaha | what's up | 14:23 |
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:24 |
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:25 |
mwhahaha | otherwise you'll end up with services with possibly no auth by default | 14:26 |
iurygregory | mwhahaha, ack ;) | 14:26 |
mwhahaha | time to bus children to school, back in 30 | 14:27 |
*** karimb has joined #puppet-openstack | 14:33 | |
Ahharu | ok, so after this is backported, I can backport the neutron port fix thing from master | 14:36 |
Ahharu | :) | 14:36 |
iurygregory | Ahharu, after the patch is merged, yes :D | 14:37 |
Ahharu | sure, will wait for it | 14:38 |
*** bogdando has quit IRC | 14:39 | |
*** bogdando has joined #puppet-openstack | 14:43 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method https://review.openstack.org/360608 | 14:48 |
*** openstackgerrit has quit IRC | 14:48 | |
degorenko | EmilienM, i'm ready to +2 it :D waiting for ci | 14:49 |
*** openstackgerrit has joined #puppet-openstack | 14:49 | |
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:50 |
degorenko | not sure that Dima is person who knows about this :D | 14:51 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Move magnum to authtoken https://review.openstack.org/351980 | 14:53 |
*** finchd has joined #puppet-openstack | 14:54 | |
*** aviramb has joined #puppet-openstack | 14:54 | |
*** saneax is now known as saneax-_-|AFK | 14:56 | |
*** xarses_ has joined #puppet-openstack | 14:56 | |
iurygregory | degorenko, can you review magnum? :D | 14:56 |
*** AJaeger has left #puppet-openstack | 14:57 | |
*** finchd has left #puppet-openstack | 14:58 | |
degorenko | iurygregory, is it fair that you updated magnum patch? :) | 14:59 |
iurygregory | degorenko, yeah because we need to finish authtoken i think XD | 15:00 |
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:01 |
openstackgerrit | Jiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary https://review.openstack.org/358510 | 15:03 |
*** sanjayu_ is now known as sanjayu|AFK | 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:04 |
iurygregory | degorenko, ok I'll let Michael do this =) put in the review about the commit message please =) | 15:05 |
degorenko | ya | 15:06 |
*** dgurtner has quit IRC | 15:08 | |
*** bnemec has quit IRC | 15:08 | |
*** dprince has quit IRC | 15:09 | |
*** bnemec has joined #puppet-openstack | 15:13 | |
openstackgerrit | Jiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary https://review.openstack.org/358510 | 15:17 |
*** furlongm_ has joined #puppet-openstack | 15:19 | |
*** furlongm has quit IRC | 15:20 | |
degorenko | EmilienM, iurygregory , mwhahaha how we missed that one https://github.com/openstack/puppet-nova/blob/master/manifests/keystone/authtoken.pp#L226 ? :( | 15:21 |
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:23 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Remove unneeded include for main class https://review.openstack.org/360632 | 15:26 |
degorenko | EmilienM, mwhahaha ^ | 15:26 |
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:27 |
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:30 |
mwhahaha | puppet-panko wtf is that | 15:31 |
mwhahaha | does it takes delicious on chicken? | 15:31 |
*** derekh has quit IRC | 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:31 |
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:32 |
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:33 |
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 has joined #puppet-openstack | 15:34 | |
*** pradk has joined #puppet-openstack | 15:36 | |
*** xarses_ is now known as xarses | 15:37 | |
*** aviramb has quit IRC | 15:56 | |
*** derekh has quit IRC | 16:02 | |
*** danpawlik has quit IRC | 16:09 | |
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:10 |
dtantsur | EmilienM, I suspect the debian package name is wrong there | 16:12 |
EmilienM | dtantsur: http://ubuntu-cloud.archive.canonical.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/experimental/python-ironic-lib | 16:12 |
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.com/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:13 |
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:15 |
degorenko | not inside | 16:16 |
iurygregory | degorenko, oh sorry .-. | 16:16 |
degorenko | ok, no problem | 16:16 |
iurygregory | social, EmilienM put the same comment =) | 16:17 |
*** dprince has joined #puppet-openstack | 16:19 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method https://review.openstack.org/360608 | 16:21 |
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:22 |
*** skolekonov has quit IRC | 16:22 | |
EmilienM | ah | 16:23 |
EmilienM | I'm tired | 16:23 |
iurygregory | it happens | 16:23 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method https://review.openstack.org/360608 | 16:24 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests" https://review.openstack.org/360610 | 16:25 |
*** dprince has quit IRC | 16:37 | |
*** ifarkas is now known as ifarkas_afk | 16:42 | |
*** rcernin has quit IRC | 16:45 | |
chem | internet is slow today | 16:47 |
EmilienM | chem: gerrit is slow today | 16:48 |
*** hjensas has quit IRC | 16:52 | |
*** dprince has joined #puppet-openstack | 16:53 | |
*** davideagle has quit IRC | 16:54 | |
*** tesseract- has quit IRC | 16:56 | |
*** _ducttape_ has joined #puppet-openstack | 16:57 | |
*** dgurtner has joined #puppet-openstack | 16:57 | |
*** ducttape_ has quit IRC | 17:01 | |
*** skramaja has quit IRC | 17:06 | |
*** iberezovskiy is now known as iberezovskiy|off | 17:07 | |
*** skramaja has joined #puppet-openstack | 17:08 | |
*** 7JTABL8FZ has joined #puppet-openstack | 17:10 | |
*** ebalduf has quit IRC | 17:12 | |
*** chem is now known as chem|off | 17:13 | |
openstackgerrit | Merged openstack/puppet-swift: Add statsd configuration options https://review.openstack.org/360032 | 17:14 |
*** AJaeger has joined #puppet-openstack | 17:22 | |
*** _ducttape_ has quit IRC | 17:25 | |
*** AJaeger has left #puppet-openstack | 17:25 | |
*** AnarchyAo has joined #puppet-openstack | 17:27 | |
*** dsavineau has left #puppet-openstack | 17:31 | |
*** dtantsur is now known as dtantsur|afk | 17:36 | |
*** degorenko is now known as _degorenko|afk | 17:37 | |
*** karimb has quit IRC | 17:41 | |
*** ducttape_ has joined #puppet-openstack | 17:51 | |
*** rcernin has joined #puppet-openstack | 17:54 | |
*** dgurtner has quit IRC | 18:02 | |
openstackgerrit | Ben Swartzlander proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class https://review.openstack.org/354014 | 18:12 |
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:23 |
*** paramite has quit IRC | 18:33 | |
iurygregory | gerrit is slow only here? .-. | 18:44 |
*** paramite has joined #puppet-openstack | 18:45 | |
*** Epx998 has joined #puppet-openstack | 18:49 | |
mwhahaha | no it's pretty bad today | 18:49 |
*** Epx998 has left #puppet-openstack | 18:50 | |
*** paramite has quit IRC | 18:57 | |
openstackgerrit | Merged openstack/puppet-trove: Policy support for Trove https://review.openstack.org/359948 | 19:14 |
openstackgerrit | Merged openstack/puppet-nova: Add support for new vendordata plugins https://review.openstack.org/359604 | 19:17 |
openstackgerrit | Merged openstack/puppet-openstack-integration: Move all lbaas to v2 https://review.openstack.org/360354 | 19:26 |
*** johnmilton has quit IRC | 19:32 | |
*** foxhunt_away is now known as foxhunt72 | 19:38 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade https://review.openstack.org/360660 | 19:43 |
*** foxhunt72 is now known as foxhunt_away | 19:44 | |
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:46 |
-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:47 | |
*** johnmilton has joined #puppet-openstack | 19:48 | |
*** _stowa has joined #puppet-openstack | 19:50 | |
*** skramaja has quit IRC | 19:50 | |
*** johnmilton has quit IRC | 19:53 | |
*** _stowa has quit IRC | 19:55 | |
openstackgerrit | Merged openstack/puppet-cinder: Add support for allow_availability_zone_fallback parameter https://review.openstack.org/356359 | 20:02 |
*** dprince has quit IRC | 20:07 | |
*** johnmilton has joined #puppet-openstack | 20:09 | |
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:10 |
iurygregory | EmilienM, https://review.openstack.org/#/c/359543/ | 20:11 |
*** ducttape_ has quit IRC | 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:11 |
iurygregory | i've talked to denis it's not related | 20:12 |
EmilienM | ok I'll review it shortly | 20:13 |
iurygregory | EmilienM, tks take your time o/ | 20:13 |
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_ has joined #puppet-openstack | 20:14 | |
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:15 |
*** johnmilton has quit IRC | 20:15 | |
iurygregory | if you need any reviews let me know | 20:16 |
*** alee has quit IRC | 20:18 | |
EmilienM | iurygregory: will do | 20:20 |
*** johnmilton has joined #puppet-openstack | 20:28 | |
EmilienM | iurygregory: heat reviewed | 20:41 |
*** ansmith has quit IRC | 20:42 | |
iurygregory | EmilienM, ack fixing right now | 20:42 |
*** ansmith_ has quit IRC | 20:43 | |
*** morazi has quit IRC | 20:46 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004 https://review.openstack.org/339106 | 20:59 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-heat: Move Heat to authtoken https://review.openstack.org/359543 | 21:08 |
openstackgerrit | Merged openstack/puppet-nova: Remove unneeded include for main class https://review.openstack.org/360632 | 21:18 |
openstackgerrit | Alex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options https://review.openstack.org/359967 | 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 |
*** Ahharu_ has joined #puppet-openstack | 21:35 | |
Ahharu_ | @EmilienM how should I proceed with https://review.openstack.org/#/c/360390/ ? | 21:35 |
Ahharu_ | code is different from master and mitaka | 21:36 |
Ahharu_ | in terms of Anchors, requires and stuff | 21:36 |
openstackgerrit | Daniel Pawlik proposed openstack/puppet-watcher: Change authtoken to class https://review.openstack.org/356792 | 21:39 |
*** pradk has quit IRC | 21:41 | |
*** shardy has quit IRC | 21:57 | |
*** Ahharu_ has quit IRC | 21:59 | |
openstackgerrit | Merged openstack/puppet-swift: Trim useless blank lines in generated config file https://review.openstack.org/359342 | 22:01 |
EmilienM | Ahharu: ah :( indeed | 22:03 |
EmilienM | Ahharu: please specify it in commit message then | 22:03 |
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:07 |
*** chem|off has quit IRC | 22:16 | |
*** chem|off has joined #puppet-openstack | 22:18 | |
*** ducttape_ has quit IRC | 22:25 | |
*** ducttape_ has joined #puppet-openstack | 22:26 | |
*** ducttape_ has quit IRC | 22:30 | |
*** ntpttr has quit IRC | 22:31 | |
*** ntpttr has joined #puppet-openstack | 22:36 | |
*** 7JTABL8FZ has quit IRC | 22:40 | |
*** chem|off` has joined #puppet-openstack | 22:46 | |
*** chem|off has quit IRC | 22:48 | |
openstackgerrit | Merged openstack/puppet-tripleo: SR-IOV nic agent changes in line with composable roles. https://review.openstack.org/332191 | 22:53 |
*** ducttape_ has joined #puppet-openstack | 23:00 | |
*** ducttape_ has quit IRC | 23:08 | |
*** dfisher has quit IRC | 23:14 | |
*** ebalduf has joined #puppet-openstack | 23:18 | |
*** ducttape_ has joined #puppet-openstack | 23:23 | |
*** chem|off` has quit IRC | 23:25 | |
*** chem|off` has joined #puppet-openstack | 23:26 | |
*** xarses has quit IRC | 23:29 | |
*** beagles has left #puppet-openstack | 23:32 | |
*** ducttape_ has quit IRC | 23:33 | |
*** AnarchyAo has quit IRC | 23:41 | |
*** ducttape_ has joined #puppet-openstack | 23:47 | |
*** rcernin has quit IRC | 23:53 | |
*** xarses has joined #puppet-openstack | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!