| owalsh | it matters because the default cell is required, you can't not have a single cell setup in ocata | 00:00 |
|---|---|---|
| mwhahaha | right but you're getting hung up on the defaultness of it | 00:01 |
| mwhahaha | there's nothing that says i can't name it and i can't just use the command to pass in the transport/database from the config file | 00:01 |
| owalsh | we don't necessary know the transport/db, might be default | 00:02 |
| mwhahaha | the operator who is constructing their manifest will | 00:02 |
| * mwhahaha throws things at gerrit | 00:04 | |
| mwhahaha | it seems that i always try and submit right at midnight utc when they are backing up | 00:04 |
| mwhahaha | so what you're proposing is less flexible for the operator for those who want to use the puppet module to perform these actions. | 00:04 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup https://review.openstack.org/422248 | 00:04 |
| mwhahaha | so there -^ | 00:05 |
| mwhahaha | i didn't update all the tests | 00:05 |
| mwhahaha | but that drops the ability to pass in a transport url or database connection into the simple setup thus requiring the operator to ensure that the nova.conf is used to populate a cell named 'default' | 00:06 |
| mwhahaha | actually since we reverted the stable/newton cell v2 stuff i think we can clean it up further | 00:08 |
| owalsh | pass the transport url/dbconn for the case where they are on a different host you mean? | 00:10 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 00:10 |
| mwhahaha | owalsh: yea, say you split the apidb off from the main db | 00:11 |
| mwhahaha | owalsh: there could be a case where the 'default' cell is not the same as the api cell | 00:11 |
| mwhahaha | because a cell is essentially the main nova db for the cloud right? and this is the info that gets shoved into the api db? | 00:13 |
| mwhahaha | so api db could be on host1 and the main nova 'default' cell db could be on host2 | 00:13 |
| owalsh | yea, IIUC | 00:13 |
| owalsh | especailly for multi-cell | 00:13 |
| mwhahaha | so nova-manage is using the api db creds from the config, but maybe i don't want to put the main nova db on the same host | 00:13 |
| mwhahaha | thats why i didn't want to artificially restrict the user | 00:14 |
| mwhahaha | so by still allowing that, we allow for more advanced configurations using the same bit of code | 00:14 |
| mwhahaha | owalsh: so thoughts? my preference is to still accept transport/database url in the simple setup to ease the end user experience but by default it would be what you're describing. | 00:17 |
| *** xarses has quit IRC | 00:17 | |
| *** social has quit IRC | 00:17 | |
| *** ducttape_ has joined #puppet-openstack | 00:20 | |
| owalsh | mwhahaha: thinking... how about os_service_default as the default for transport_url/db in this case | 00:20 |
| mwhahaha | no because that doesn't mean anything outside of config files | 00:20 |
| mwhahaha | undef is correct | 00:21 |
| owalsh | mwhahaha: but... it means use the default from the config files, which is what we would do | 00:21 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 00:21 |
| mwhahaha | owalsh: no it really doesn't, it's a special string we use in our providers to nuke config values out of the files | 00:21 |
| mwhahaha | owalsh: which has the effect of using service defaults as defined in code | 00:21 |
| owalsh | mwhahaha: yes, but isn't that what we are telling create_cell to do? | 00:22 |
| mwhahaha | no, it gets passed to as a command line parameter | 00:23 |
| owalsh | if is_service_default(foo) { foo_real=''} | 00:23 |
| mwhahaha | so what would get run is nova-manage cell_v2 create_cell --name default --transport_url <SERVICE_DEFAULT> --database_connection <SERVICE_DEFAULT> | 00:23 |
| mwhahaha | trust me, for this undef is the correct value | 00:24 |
| owalsh | problem is the unless command | 00:24 |
| mwhahaha | but it's not because we're using a name | 00:24 |
| owalsh | it's not correct... what if renaming cells becomes a feature | 00:24 |
| mwhahaha | then we switch to a provider | 00:25 |
| mwhahaha | because then we need to handle updates at which point execs aren't going to cut it | 00:25 |
| mwhahaha | also deletes become viable because we'll have the uuid | 00:26 |
| mwhahaha | owalsh: anyway i've got to go. I thin patch set 3 + the removal of the ensure part from the nova::cell_v2::cell is teh correct implementation for an initial setup | 00:31 |
| *** ducttape_ has quit IRC | 00:31 | |
| mwhahaha | owalsh: we can always work on improving it later via providers or whatever but it gives us a point to start with that works | 00:31 |
| mwhahaha | owalsh: additionally we can work around the upgrade process by shuffling when the commands run | 00:32 |
| owalsh | mwhahaha: ok, I'll think it all through again in the morning | 00:32 |
| mwhahaha | owalsh: i had started that effort over https://review.openstack.org/#/c/419121/ | 00:32 |
| mwhahaha | owalsh: so i get what you've been saying and there are some gotchas that we have to document, but honestly they aren't that bad | 00:32 |
| mwhahaha | owalsh: and we like to keep things at least somewhat modular so that operators/deployers can use the tools we're providing in other ways | 00:33 |
| * mwhahaha wanders off | 00:33 | |
| owalsh | mwhahaha: yea, saw all of the upgrade fun. I've some thoughts on that too but they can wait | 00:33 |
| owalsh | mwhahaha: thanks, again | 00:33 |
| *** agrebennikov_ has quit IRC | 01:27 | |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 01:37 |
| *** saneax-_-|AFK is now known as saneax | 01:43 | |
| *** ducttape_ has joined #puppet-openstack | 01:55 | |
| *** acormier has joined #puppet-openstack | 01:55 | |
| *** ducttape_ has quit IRC | 01:59 | |
| *** sofie has joined #puppet-openstack | 01:59 | |
| *** ducttape_ has joined #puppet-openstack | 02:13 | |
| *** dprince has quit IRC | 02:19 | |
| *** acormier has quit IRC | 02:31 | |
| *** acormier has joined #puppet-openstack | 02:31 | |
| *** morazi has quit IRC | 02:50 | |
| *** ducttape_ has quit IRC | 02:54 | |
| *** imcsk8 has joined #puppet-openstack | 03:20 | |
| *** saneax is now known as saneax-_-|AFK | 03:24 | |
| *** higgins has joined #puppet-openstack | 03:33 | |
| *** higgins` has quit IRC | 03:33 | |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 03:43 |
| *** agrebennikov_ has joined #puppet-openstack | 03:52 | |
| *** saneax-_-|AFK is now known as saneax | 03:55 | |
| *** acormier has quit IRC | 04:06 | |
| *** acormier has joined #puppet-openstack | 04:06 | |
| *** spredzy has quit IRC | 04:07 | |
| *** spredzy has joined #puppet-openstack | 04:08 | |
| *** acormier has quit IRC | 04:11 | |
| *** udesale has joined #puppet-openstack | 04:25 | |
| *** agrebennikov_ has quit IRC | 04:25 | |
| *** udesale has quit IRC | 04:30 | |
| mfisch | mwhahaha: you there? | 04:56 |
| *** agrebennikov_ has joined #puppet-openstack | 04:59 | |
| mfisch | mwhahaha: will catch you tomorrow or next week | 05:08 |
| *** udesale has joined #puppet-openstack | 05:09 | |
| *** udesale has quit IRC | 05:10 | |
| *** udesale has joined #puppet-openstack | 05:11 | |
| *** imcsk8_ has joined #puppet-openstack | 05:30 | |
| *** imcsk8 has quit IRC | 05:33 | |
| *** timrc has quit IRC | 05:34 | |
| *** timrc has joined #puppet-openstack | 05:35 | |
| *** agrebennikov_ has quit IRC | 05:50 | |
| openstackgerrit | zhangyanxian proposed openstack/puppet-aodh: Fix typo in aodh_api_paste_ini_spec.rb https://review.openstack.org/423053 | 06:06 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-aodh: Fix typo in aodh_api_paste_ini_spec.rb https://review.openstack.org/423053 | 06:06 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-ceilometer: Fix typo in ceilometer_api_paste_ini_spec.rb https://review.openstack.org/423054 | 06:07 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-ceilometer: Fix typo in ceilometer_api_paste_ini_spec.rb https://review.openstack.org/423054 | 06:07 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-gnocchi: Fix typo in gnocchi_api_paste_ini_spec.rb https://review.openstack.org/423056 | 06:08 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-gnocchi: Fix typo in gnocchi_api_paste_ini_spec.rb https://review.openstack.org/423056 | 06:08 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-heat: Fix typo in heat_api_paste_ini_spec.rb https://review.openstack.org/423057 | 06:09 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-heat: Fix typo in heat_api_paste_ini_spec.rb https://review.openstack.org/423057 | 06:09 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-ironic: Fix typo in ironic_api_paste_ini_spec.rb https://review.openstack.org/423058 | 06:10 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-ironic: Fix typo in ironic_api_paste_ini_spec.rb https://review.openstack.org/423058 | 06:10 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-keystone: Fix typos in keystone_paste_ini_spec.rb & keystone_config_spec.rb https://review.openstack.org/423060 | 06:11 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-keystone: Fix typos in keystone_paste_ini_spec.rb & keystone_config_spec.rb https://review.openstack.org/423060 | 06:12 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-neutron: Fix typo in neutron_api_paste_ini_spec.rb https://review.openstack.org/423062 | 06:13 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-neutron: Fix typo in neutron_api_paste_ini_spec.rb https://review.openstack.org/423062 | 06:13 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-sahara: Fix typo in sahara_api_paste_ini_spec.rb https://review.openstack.org/423063 | 06:14 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-sahara: Fix typo in sahara_api_paste_ini_spec.rb https://review.openstack.org/423063 | 06:14 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-swift: Fix typo in swift_keystone_test.rb & swift.rb https://review.openstack.org/423065 | 06:17 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-swift: Fix typo in swift_keystone_test.rb & swift.rb https://review.openstack.org/423065 | 06:18 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-swift: Fix typo in swift_keystone_test.rb & swift.rb https://review.openstack.org/423065 | 06:18 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-vitrage: Fix typo in vitrage_api_paste_ini_spec.rb https://review.openstack.org/423068 | 06:19 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-vitrage: Fix typo in vitrage_api_paste_ini_spec.rb https://review.openstack.org/423068 | 06:19 |
| *** richm has quit IRC | 06:42 | |
| *** furlongm_ has quit IRC | 06:51 | |
| *** hjensas has quit IRC | 07:03 | |
| *** yprokule has joined #puppet-openstack | 07:10 | |
| *** rajinir has quit IRC | 07:16 | |
| *** hjensas has joined #puppet-openstack | 07:23 | |
| *** hjensas has joined #puppet-openstack | 07:23 | |
| openstackgerrit | zhangyanxian proposed openstack/puppet-watcher: Fix typo in api.pp https://review.openstack.org/423102 | 07:36 |
| openstackgerrit | zhangyanxian proposed openstack/puppet-watcher: Fix typo in api.pp https://review.openstack.org/423102 | 07:36 |
| *** LarsErikP has joined #puppet-openstack | 07:44 | |
| *** tdb_ has joined #puppet-openstack | 07:46 | |
| *** rasca has joined #puppet-openstack | 07:48 | |
| tobasco | mwhahaha: any reason why the new oslo_messaing config for transport_url and notification_transport_url was merged in puppet-glance master but not backported to newton? | 07:49 |
| *** ntpttr_ has joined #puppet-openstack | 07:50 | |
| *** Hunner_ has joined #puppet-openstack | 07:50 | |
| *** Hunner_ has joined #puppet-openstack | 07:50 | |
| *** LarsErik1 has quit IRC | 07:51 | |
| *** tdasilva has quit IRC | 07:51 | |
| *** tdb has quit IRC | 07:51 | |
| *** Hunner has quit IRC | 07:51 | |
| *** ntpttr has quit IRC | 07:51 | |
| *** tdasilva has joined #puppet-openstack | 07:51 | |
| *** tesseract has joined #puppet-openstack | 08:05 | |
| *** paramite has joined #puppet-openstack | 08:09 | |
| *** shardy has joined #puppet-openstack | 08:14 | |
| *** shardy is now known as shardy_afk | 08:26 | |
| *** jpena|off is now known as jpena | 08:37 | |
| *** amoralej|off is now known as amoralej | 08:39 | |
| *** jpich has joined #puppet-openstack | 08:43 | |
| *** jpena is now known as jpena|off | 08:51 | |
| *** shardy_afk is now known as shardy | 08:52 | |
| *** jpena|off is now known as jpena | 08:54 | |
| mkarpin | tobasco: mwhahaha: I am also wondering if there is any reason for these modules and for swift not to add transport_url in newton, btw I am backporting it here - https://review.openstack.org/#/c/422509/ https://review.openstack.org/#/c/422545/ | 08:55 |
| saneax | I was hearing there is an effort to have os-net-config via puppet module, is there any effort on that direction? | 08:59 |
| *** ccamacho has joined #puppet-openstack | 09:01 | |
| *** openstackgerrit has quit IRC | 09:02 | |
| tobasco | mkarpin: awesome, thanks for looking into it | 09:04 |
| tobasco | i was wondering why it was never backported when the other options is already deprecated in newton | 09:04 |
| tobasco | need to get this merged :) | 09:04 |
| tobasco | mkarpin: we need to cherry pick the notification_transport_url aswell, atleast for puppet-glance | 09:07 |
| *** social has joined #puppet-openstack | 09:09 | |
| mkarpin | tobasco: I'll try to look into it a bit later, or maybe you could do it? | 09:10 |
| tobasco | yeah, I'm gonna cherry pick the notification_transport_url commit that mwhahaha merged into master | 09:11 |
| *** openstackgerrit has joined #puppet-openstack | 09:14 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-magnum: Add unit test for client.pp https://review.openstack.org/423135 | 09:14 |
| *** derekh has joined #puppet-openstack | 09:15 | |
| *** gfidente has joined #puppet-openstack | 09:21 | |
| *** gfidente has quit IRC | 09:21 | |
| *** gfidente has joined #puppet-openstack | 09:21 | |
| *** fxpester has joined #puppet-openstack | 09:22 | |
| *** nmathew has joined #puppet-openstack | 09:23 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-aodh: Fix annotation of the ensure parameter https://review.openstack.org/423144 | 09:30 |
| *** lucas-afk is now known as lucasagomes | 09:34 | |
| *** zhongshengping has joined #puppet-openstack | 09:36 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron: Deprecate rabbitmq connection parameters https://review.openstack.org/423147 | 09:42 |
| *** yprokule has quit IRC | 09:54 | |
| *** owalsh has quit IRC | 09:59 | |
| *** owalsh has joined #puppet-openstack | 09:59 | |
| *** salmankhan has joined #puppet-openstack | 10:00 | |
| *** salmankhan1 has joined #puppet-openstack | 10:10 | |
| *** salmankhan has quit IRC | 10:15 | |
| *** salmankhan1 is now known as salmankhan | 10:15 | |
| *** aarefiev_afk is now known as aarefiev | 10:26 | |
| *** ansiwen_ has joined #puppet-openstack | 10:27 | |
| *** ansiwen has quit IRC | 10:27 | |
| *** guimaluf has joined #puppet-openstack | 10:27 | |
| *** owalsh has quit IRC | 10:30 | |
| *** saneax is now known as saneax-_-|AFK | 10:31 | |
| *** salmankhan has quit IRC | 10:35 | |
| *** owalsh has joined #puppet-openstack | 10:38 | |
| *** dtantsur|afk is now known as dtantsur | 10:43 | |
| *** panda|Zz is now known as panda | 10:52 | |
| openstackgerrit | Merged openstack/puppet-horizon: Manage password_validator with puppet manifest https://review.openstack.org/413653 | 11:02 |
| *** zhongshengping has quit IRC | 11:10 | |
| *** richm has joined #puppet-openstack | 11:13 | |
| *** nmathew has quit IRC | 11:27 | |
| *** ducttape_ has joined #puppet-openstack | 11:29 | |
| *** ducttape_ has quit IRC | 11:31 | |
| *** yprokule has joined #puppet-openstack | 11:47 | |
| *** saneax-_-|AFK is now known as saneax | 12:02 | |
| *** ansmith has joined #puppet-openstack | 12:03 | |
| *** cdelatte has joined #puppet-openstack | 12:08 | |
| openstackgerrit | Merged openstack/puppet-gnocchi: Remove carbonara package reference https://review.openstack.org/422918 | 12:08 |
| *** ansmith has quit IRC | 12:11 | |
| *** nmathew has joined #puppet-openstack | 12:18 | |
| EmilienM | hi | 12:24 |
| mkarpin | Hi, guys, mwhahaha, EmilienM I saw the mail, thank you for your trust! | 12:33 |
| *** shardy is now known as shardy_lunch | 12:36 | |
| *** lucasagomes is now known as lucas-hungry | 12:37 | |
| EmilienM | mkarpin: thank *you* for your work! | 12:37 |
| *** sofie has quit IRC | 12:48 | |
| openstackgerrit | Brent Eagles proposed openstack/puppet-octavia: Move db include from ::octavia to ::octavia::api https://review.openstack.org/422031 | 12:49 |
| *** morazi has joined #puppet-openstack | 12:53 | |
| *** hjensas has quit IRC | 12:54 | |
| *** nmathew has quit IRC | 12:59 | |
| *** morazi has quit IRC | 12:59 | |
| *** dprince has joined #puppet-openstack | 12:59 | |
| *** hjensas has joined #puppet-openstack | 13:00 | |
| *** zhongshengping has joined #puppet-openstack | 13:04 | |
| *** ducttape_ has joined #puppet-openstack | 13:11 | |
| *** morazi has joined #puppet-openstack | 13:11 | |
| *** shardy_lunch is now known as shardy | 13:14 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Allow configuring node_not_found_hook in inspector https://review.openstack.org/423256 | 13:15 |
| EmilienM | degorenko: o/ | 13:17 |
| EmilienM | why +1 only on https://review.openstack.org/#/c/420127/ ? | 13:17 |
| EmilienM | oh you were waiting for fuel CI i guess | 13:17 |
| degorenko | EmilienM, o/ | 13:17 |
| degorenko | let me check | 13:17 |
| degorenko | EmilienM, ah, right, approved | 13:18 |
| EmilienM | thanks | 13:18 |
| *** ducttape_ has quit IRC | 13:21 | |
| *** trown|outtypewww is now known as trown | 13:22 | |
| *** cgfbee has quit IRC | 13:22 | |
| *** iurygregory has quit IRC | 13:23 | |
| *** ducttape_ has joined #puppet-openstack | 13:24 | |
| *** jayg|g0n3 is now known as jayg | 13:24 | |
| pradk | can i get some reviews on https://review.openstack.org/#/c/420908/ please | 13:24 |
| EmilienM | pradk: I just +2 it | 13:25 |
| EmilienM | it was on my list | 13:25 |
| pradk | ah thx :) | 13:25 |
| *** shardy has quit IRC | 13:27 | |
| *** shardy has joined #puppet-openstack | 13:27 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Configure ironic-inspector parameters related to node discovery https://review.openstack.org/423256 | 13:31 |
| *** ducttape_ has quit IRC | 13:34 | |
| *** jpena is now known as jpena|lunch | 13:37 | |
| *** acormier has joined #puppet-openstack | 13:43 | |
| *** paramite has quit IRC | 13:47 | |
| *** hjensas has quit IRC | 13:47 | |
| *** lucas-hungry is now known as lucasagomes | 13:48 | |
| *** ansmith has joined #puppet-openstack | 13:51 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-nova: Fix api config https://review.openstack.org/423279 | 13:56 |
| *** acormier has quit IRC | 13:57 | |
| *** acormier has joined #puppet-openstack | 13:57 | |
| openstackgerrit | Merged openstack/puppet-nova: Allow python-memcache install from authtoken class https://review.openstack.org/420127 | 14:00 |
| *** acormier has quit IRC | 14:02 | |
| *** skramaja has quit IRC | 14:02 | |
| *** cgfbee has joined #puppet-openstack | 14:05 | |
| openstackgerrit | Brent Eagles proposed openstack/puppet-neutron: Remove several deprecated configuration items in Ocata https://review.openstack.org/422755 | 14:11 |
| *** zhongshengping has quit IRC | 14:14 | |
| *** saneax is now known as saneax-_-|AFK | 14:18 | |
| dtantsur | EmilienM, quick puppet question: does "pick" work with $::os_service_default? | 14:36 |
| dtantsur | like in https://github.com/openstack/puppet-ironic/blob/master/manifests/conductor.pp#L161 ? | 14:36 |
| EmilienM | hmm, I don't think so | 14:37 |
| EmilienM | pick will pick the first param with a value | 14:37 |
| EmilienM | and $::os_service_default is a value | 14:37 |
| dtantsur | ugh. my bad, this is broken then. I should rather set the old option to "unset" and pass it first to pick, right? | 14:37 |
| dtantsur | the report from our QE: https://bugzilla.redhat.com/show_bug.cgi?id=1412754 | 14:38 |
| openstack | bugzilla.redhat.com bug 1412754 in puppet-ironic "ironic - provider network cleaning[cleaning_network_uuid] not working" [Medium,Assigned] - Assigned to dtantsur | 14:38 |
| EmilienM | dtantsur: probably | 14:39 |
| * EmilienM wanders off - on PTO today | 14:39 | |
| * dtantsur cooks a patch | 14:39 | |
| dtantsur | see you EmilienM, enjoy your pto :) | 14:39 |
| EmilienM | thx | 14:39 |
| dtantsur | another dumg puppet question: with "undef" values, should I use "if $value" or "if $value != undef"? | 14:40 |
| llua | you can do if $value | 14:41 |
| openstackgerrit | zhongshengping proposed openstack/puppet-nova: Fix api config https://review.openstack.org/423279 | 14:41 |
| *** amoralej is now known as amoralej|lunch | 14:41 | |
| dtantsur | thanks llua | 14:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Fix deprecated cleaning_network_uuid and provisioning_network_uuid https://review.openstack.org/423308 | 14:44 |
| dtantsur | this should hopefully fix it ^^^ | 14:44 |
| *** zhongshengping has joined #puppet-openstack | 14:45 | |
| openstackgerrit | Merged openstack/puppet-ironic: Fix typo in ironic_api_paste_ini_spec.rb https://review.openstack.org/423058 | 14:49 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Sort the output of rpm -qa for easier comparisons https://review.openstack.org/422301 | 14:49 |
| *** jpena|lunch is now known as jpena | 14:49 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: logs: add keystone-resources for debug https://review.openstack.org/422302 | 14:49 |
| *** iurygregory has joined #puppet-openstack | 14:51 | |
| *** ducttape_ has joined #puppet-openstack | 14:53 | |
| *** ducttape_ has quit IRC | 15:02 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-cinder: Allow blank rabbit password if using transport_url https://review.openstack.org/423320 | 15:03 |
| *** acormier has joined #puppet-openstack | 15:09 | |
| *** acormier has quit IRC | 15:09 | |
| *** acormier has joined #puppet-openstack | 15:09 | |
| *** acormier has quit IRC | 15:11 | |
| *** acormier has joined #puppet-openstack | 15:12 | |
| *** amoralej|lunch is now known as amoralej | 15:13 | |
| *** fxpester has quit IRC | 15:19 | |
| *** udesale has quit IRC | 15:21 | |
| *** udesale has joined #puppet-openstack | 15:21 | |
| *** udesale has quit IRC | 15:24 | |
| *** udesale has joined #puppet-openstack | 15:25 | |
| *** ducttape_ has joined #puppet-openstack | 15:29 | |
| *** bnemec is now known as beekneemech | 15:33 | |
| *** Hunner_ is now known as Hunner | 15:34 | |
| *** jaosorior has joined #puppet-openstack | 15:39 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: Fix deprecated cleaning_network_uuid and provisioning_network_uuid https://review.openstack.org/423308 | 15:45 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 15:45 |
| mjblack | quick, question. In the keystone module, it looks like pki_setup was removed. Is there a replacement for creating the pki files? | 15:46 |
| openstackgerrit | Brent Eagles proposed openstack/puppet-neutron: Remove several deprecated configuration items in Ocata https://review.openstack.org/422755 | 15:46 |
| *** shardy has quit IRC | 15:48 | |
| *** shardy has joined #puppet-openstack | 15:49 | |
| zhongshengping | @mjblack There is no replacement , becuase PKI token support has been removed in keystone. You can use other certifications, for example uuid or fernet. | 15:55 |
| mjblack | zhongshengping: pki token might be | 15:57 |
| *** xarses has joined #puppet-openstack | 15:58 | |
| mjblack | but the pki is still there for others | 15:58 |
| *** xarses_ has joined #puppet-openstack | 15:59 | |
| *** zhongshengping has quit IRC | 16:00 | |
| *** zhongshengping has joined #puppet-openstack | 16:01 | |
| zhongshengping | What puppet modules? or What parameters? | 16:01 |
| *** xarses has quit IRC | 16:02 | |
| mwhahaha | mjblack: https://review.openstack.org/#/c/374479/ | 16:02 |
| mjblack | enable_pki, the ceph radosgw has a req, which I was told it doesnt for v3, for the ssl certs for pki | 16:02 |
| mwhahaha | so that's the problem for us is that keystone has removed it for ocata so if you still need it you need the newton version of stuff | 16:03 |
| mjblack | mwhahaha: I'm using Ocata RDO packages :D | 16:03 |
| *** rpittau has quit IRC | 16:05 | |
| mwhahaha | ah so they changed pki_setup to be for revocation lists | 16:05 |
| mwhahaha | so we need to revert that and update it to indicate it's still there | 16:05 |
| mwhahaha | but it's still deprecated | 16:06 |
| mwhahaha | so they just removed support for pki *tokens* but not using pki for revocation list | 16:06 |
| openstackgerrit | zhongshengping proposed openstack/puppet-nova: Fix api config https://review.openstack.org/423279 | 16:07 |
| mwhahaha | well lets just revert the keystone change then | 16:08 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-keystone: Revert "Remove PKI signing related parameters" https://review.openstack.org/423349 | 16:09 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-keystone: Revert "Remove PKI signing related parameters" https://review.openstack.org/423349 | 16:10 |
| *** dfisher has joined #puppet-openstack | 16:10 | |
| *** panda is now known as panda|bbl | 16:34 | |
| *** paramite has joined #puppet-openstack | 16:43 | |
| *** jaosorior has quit IRC | 16:48 | |
| *** jaosorior has joined #puppet-openstack | 16:49 | |
| *** paramite has quit IRC | 16:50 | |
| *** jaosorior has quit IRC | 16:51 | |
| *** _ducttape_ has joined #puppet-openstack | 16:59 | |
| *** derekh has quit IRC | 17:00 | |
| *** udesale has quit IRC | 17:02 | |
| *** ducttape_ has quit IRC | 17:02 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup https://review.openstack.org/422248 | 17:05 |
| mwhahaha | owalsh: -^ reorganized some stuff and threw in an acceptance test so we'll see if it works | 17:06 |
| mkarpin | mwhahaha: EmilienM: we with tobasco want to backport some urls here - https://review.openstack.org/#/c/422509/ https://review.openstack.org/#/c/422545/ https://review.openstack.org/#/c/423147/ tobasco suggests that we should backport deprecations of rabbit-* parameters also, what do you think about this? | 17:07 |
| *** dtantsur is now known as dtantsur|afk | 17:08 | |
| mwhahaha | mkarpin: i tend to agree that we shouldn't backport the deprecation warning | 17:08 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 17:09 |
| *** xarses_ is now known as xarses | 17:09 | |
| mkarpin | mwhahaha: ok, thanks, I have the same opinion | 17:12 |
| *** zhongshengping has quit IRC | 17:15 | |
| *** agrebennikov_ has joined #puppet-openstack | 17:17 | |
| *** rasca has quit IRC | 17:20 | |
| *** dprince has quit IRC | 17:23 | |
| *** lucasagomes is now known as lucas-afk | 17:23 | |
| pradk | mwhahaha, EmilienM, does this error ring any bells? "Error: No set of title patterns matched the title "@service". on node" | 17:24 |
| openstackgerrit | Merged openstack/puppet-neutron: Add new MidoNet plugin package https://review.openstack.org/421847 | 17:25 |
| *** jaosorior has joined #puppet-openstack | 17:32 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Configure compute params in config https://review.openstack.org/420908 | 17:33 |
| *** _ducttape_ has quit IRC | 17:35 | |
| *** ducttape_ has joined #puppet-openstack | 17:36 | |
| *** jpich has quit IRC | 17:36 | |
| iurygregory | hello | 17:45 |
| mwhahaha | pradk: sounds like a provider error | 17:56 |
| *** dprince has joined #puppet-openstack | 18:01 | |
| openstackgerrit | Ricardo Noriega proposed openstack/puppet-neutron: Add support for BGPVPN plugin https://review.openstack.org/422051 | 18:12 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 18:14 |
| *** ntpttr_ is now known as ntpttr | 18:17 | |
| *** jpena is now known as jpena|off | 18:19 | |
| *** ccamacho has quit IRC | 18:26 | |
| *** jaosorior has quit IRC | 18:35 | |
| *** trown is now known as trown|lunch | 18:35 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup https://review.openstack.org/422248 | 18:37 |
| *** hecliunyx has quit IRC | 18:40 | |
| *** owalsh is now known as owalsh-brb | 18:42 | |
| *** yprokule has quit IRC | 18:45 | |
| *** ccamacho has joined #puppet-openstack | 18:45 | |
| *** owalsh-brb has quit IRC | 18:48 | |
| *** owalsh-brb has joined #puppet-openstack | 18:49 | |
| *** hecliunyx has joined #puppet-openstack | 18:50 | |
| *** gfidente has quit IRC | 19:03 | |
| *** nkinder has joined #puppet-openstack | 19:27 | |
| *** paramite has joined #puppet-openstack | 19:31 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-gnocchi: Set auth mode in gnocchi to keystone https://review.openstack.org/423497 | 19:31 |
| *** tesseract has quit IRC | 19:32 | |
| pradk | amoralej, ^^ | 19:32 |
| amoralej | EmilienM, mwhahaha we need ^ for puppet promotion | 19:32 |
| mwhahaha | K | 19:34 |
| pradk | amoralej, since the default is keystone everything should just work without any change to poi | 19:34 |
| pradk | me hopes | 19:34 |
| amoralej | me too, :) | 19:34 |
| *** trown|lunch is now known as trown | 19:36 | |
| dprince | Hunner: hi again, getting back to these noop providers again https://review.openstack.org/#/c/209030/ | 19:42 |
| dprince | Hunner: an idea, is there anybody out there creating these sort of types dynamically? Like I think one simple noop provider could register itself off of a global regex or something and override other providers right? | 19:43 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 19:47 |
| *** owalsh-brb is now known as owalsh | 19:47 | |
| *** jaosorior has joined #puppet-openstack | 19:52 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-gnocchi: Remove api-paste configuration https://review.openstack.org/423504 | 19:54 |
| *** agrebennikov_ has quit IRC | 19:58 | |
| openstackgerrit | Merged openstack/puppet-vitrage: Fix typo in vitrage_api_paste_ini_spec.rb https://review.openstack.org/423068 | 20:02 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 20:03 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup https://review.openstack.org/422248 | 20:04 |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 20:06 |
| *** panda|bbl is now known as panda | 20:11 | |
| *** hecliunyx has quit IRC | 20:12 | |
| *** owalsh is now known as owalsh-afk | 20:12 | |
| pradk | mwhahaha, quick question .. when we set a param to $::os_workers is that just letting the service default kick in if nothing is passed? | 20:14 |
| mwhahaha | no | 20:14 |
| mwhahaha | it's a calculated fact | 20:14 |
| pradk | mwhahaha, where is the fact calculated? | 20:15 |
| mwhahaha | we use it instead of $::processorcount | 20:15 |
| mwhahaha | https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_workers.rb | 20:15 |
| dfisher | you're welcome! :) | 20:15 |
| pradk | ah its the same we use to have in the module.. just moved it there | 20:15 |
| mwhahaha | dfisher: you coming out to the openstack meetup next week? :D | 20:16 |
| dfisher | more info! | 20:16 |
| dfisher | U SEND | 20:16 |
| dfisher | because … things are changing 'round here. | 20:17 |
| dfisher | :( | 20:17 |
| mwhahaha | dfisher: https://www.meetup.com/OpenStack-Colorado/events/236828606/ | 20:17 |
| dfisher | yeah, i can probably make that. | 20:18 |
| mwhahaha | i figure its in your neck of the woods anyway | 20:18 |
| dfisher | you going? | 20:18 |
| mwhahaha | ya | 20:18 |
| dfisher | nice. | 20:18 |
| dfisher | so yeah, i'm sure y'all seen the noise about Solaris | 20:18 |
| mwhahaha | mfisch and vinsh are rsvp'd as well | 20:18 |
| mwhahaha | yea | 20:19 |
| vinsh | I hope there are snacks.. i love snacks! | 20:19 |
| dfisher | there's a killer Thai place right by there. | 20:19 |
| dfisher | so if you want amazing Thai ... | 20:19 |
| *** hecliunyx has joined #puppet-openstack | 20:20 | |
| vinsh | I do enjoy Thai | 20:22 |
| dfisher | ok, so .. i'm trying to create a network node on OL 7.3 (basically RHEL 7.3) with OVS. I know I need vs_bridge { 'br-ex' } and vs_port { … } but my node already has networking. I assume I have to do some kind of replumbing where I bring eth0 down first, unconfigure it, do the OVS bridge/port, bring up the new interface with eth0 bits .. right? | 20:22 |
| dfisher | vinsh: Busaba on the map from the link | 20:23 |
| mwhahaha | not sure about the networking order, what you say sounds reasonable | 20:24 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-nova: Make novajoin bind address and port configurable https://review.openstack.org/423507 | 20:24 |
| dfisher | so, p-o-i has something similar through a loopback interface | 20:24 |
| vinsh | dfisher: Hey cool.. that's only 15 minutes south of me. Lets do a dinner after if anyone is interested. | 20:25 |
| dfisher | just don't know wtf i'm doing with this linux shit :) | 20:26 |
| *** hecliunyx has quit IRC | 20:26 | |
| vinsh | good to try a new place | 20:26 |
| dfisher | vinsh: you'll love it. | 20:26 |
| dfisher | it's by far the best Thai i've ever had. | 20:26 |
| *** amoralej is now known as amoralej|off | 20:27 | |
| vinsh | The best I know of in the state is up in FoCo "cafe de bangkok" | 20:28 |
| vinsh | I'm ready to judge :) | 20:28 |
| *** morazi has quit IRC | 20:31 | |
| dmsimard | dfisher: I read about solaris recently, thought about you :( | 20:32 |
| dfisher | :( | 20:32 |
| dfisher | still employed | 20:32 |
| dfisher | but | 20:32 |
| dfisher | :( | 20:32 |
| dfisher | it's all because solaris sed(1) doesn't have -i | 20:33 |
| dmsimard | I know right | 20:33 |
| dmsimard | Is that merged yet | 20:33 |
| dfisher | nope! | 20:34 |
| dmsimard | and now it will never get merged | 20:34 |
| *** hecliunyx has joined #puppet-openstack | 20:34 | |
| dfisher | probably not, no. | 20:36 |
| dfisher | most people just used GNU sed instead. | 20:36 |
| dfisher | since it's had -i for like 26 years. | 20:36 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Fix typo in ceilometer_api_paste_ini_spec.rb https://review.openstack.org/423054 | 20:36 |
| openstackgerrit | Merged openstack/puppet-neutron: Fix typo in neutron_api_paste_ini_spec.rb https://review.openstack.org/423062 | 20:37 |
| dfisher | well, if people want to know more about the Solaris stuff, lemme know. | 20:39 |
| *** jaosorior has quit IRC | 20:57 | |
| *** jayg is now known as jayg|g0n3 | 21:12 | |
| *** ansmith has quit IRC | 21:32 | |
| *** rajinir has joined #puppet-openstack | 21:42 | |
| *** salmankhan has joined #puppet-openstack | 21:48 | |
| *** _ducttape_ has joined #puppet-openstack | 21:48 | |
| *** _ducttape_ has quit IRC | 21:49 | |
| *** _ducttape_ has joined #puppet-openstack | 21:50 | |
| *** salmankhan has quit IRC | 21:50 | |
| *** salmankhan has joined #puppet-openstack | 21:50 | |
| *** ducttape_ has quit IRC | 21:51 | |
| openstackgerrit | David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals https://review.openstack.org/420992 | 22:00 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup https://review.openstack.org/422248 | 22:01 |
| *** paramite has quit IRC | 22:12 | |
| *** ccamacho has quit IRC | 22:13 | |
| *** trown is now known as trown|outtypewww | 22:17 | |
| *** shardy has quit IRC | 22:22 | |
| *** fultonj has quit IRC | 22:26 | |
| *** chem has quit IRC | 22:27 | |
| *** dprince has quit IRC | 22:52 | |
| *** xarses_ has joined #puppet-openstack | 22:59 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack-integration: Add python-gabbi for tempest on Ubuntu https://review.openstack.org/423547 | 23:00 |
| *** xarses has quit IRC | 23:02 | |
| EmilienM | mwhahaha: what is blocking promotion? | 23:09 |
| EmilienM | for puppet CI | 23:09 |
| mwhahaha | https://review.openstack.org/#/c/423497/ | 23:10 |
| EmilienM | mwhahaha: approved | 23:10 |
| EmilienM | mwhahaha: is there any patch I need to look? | 23:11 |
| EmilienM | I'm here ~30 min | 23:11 |
| mwhahaha | no | 23:11 |
| mwhahaha | unless you want to cry over cell_v2, https://review.openstack.org/422248 | 23:12 |
| EmilienM | mwhahaha: is it ready for review or WIP? | 23:12 |
| EmilienM | omfg | 23:12 |
| EmilienM | I need 2 gin tonic before | 23:12 |
| dfisher | lol | 23:12 |
| mwhahaha | that doesn't even include the stuff for upgrades | 23:12 |
| mwhahaha | that's just the basics for new installs | 23:13 |
| EmilienM | I need 2 bottle of Gin and no Tonic | 23:13 |
| EmilienM | "Adds the ability to perform a simple cell v2 setup using map_cell0" | 23:13 |
| EmilienM | is it a joke? it's really good. | 23:13 |
| EmilienM | oh and I guess we want to backport it? :) | 23:13 |
| mwhahaha | i don't | 23:14 |
| mwhahaha | it won't work | 23:14 |
| mwhahaha | because it relies on code landed in ocata | 23:14 |
| mwhahaha | we just have to expose the other commands that we need for upgrades | 23:14 |
| mwhahaha | so that'll be next | 23:14 |
| mwhahaha | it also doesn't currently work on ubuntu either because UCA is only up to M2 and it needs stuff that landed last week | 23:15 |
| EmilienM | lol | 23:15 |
| mwhahaha | so right now it's bleeding edge only which is why it's not enabled by default | 23:15 |
| mwhahaha | happy friday | 23:15 |
| EmilienM | mwhahaha: why do you deprecate setup_cell0 ? | 23:16 |
| EmilienM | it was added in ocata no? | 23:16 |
| mwhahaha | yea | 23:16 |
| mwhahaha | but we use it in tripleo | 23:16 |
| EmilienM | we used no? | 23:16 |
| mwhahaha | dprince put it back in | 23:16 |
| mwhahaha | let me dig up the patch | 23:16 |
| EmilienM | ah | 23:16 |
| EmilienM | right | 23:16 |
| mwhahaha | we can remove it later but i didn't want to break stuff | 23:16 |
| EmilienM | well that's useless to deprecate it | 23:17 |
| EmilienM | let's clean up in tripleo | 23:17 |
| EmilienM | ok | 23:17 |
| EmilienM | sure | 23:17 |
| EmilienM | I'll comment | 23:17 |
| mwhahaha | https://review.openstack.org/#/c/421999/ | 23:17 |
| mwhahaha | as to not break containers i left it | 23:17 |
| mwhahaha | that one var deprecation is the least of my concerns about the whole thing :D | 23:17 |
| EmilienM | mwhahaha: I commented | 23:19 |
| mwhahaha | i'll deal with it on monday :D | 23:20 |
| mwhahaha | or the other people can do it if they get to it first | 23:20 |
| EmilienM | mwhahaha: where are we on the ubuntu front? | 23:20 |
| mwhahaha | https://review.openstack.org/#/c/423547/ | 23:20 |
| mwhahaha | for the tempest stuff | 23:21 |
| EmilienM | RETRY_LIMIT is the multinode job? | 23:21 |
| EmilienM | it's only tripleo or all openstack multinode? | 23:21 |
| mwhahaha | just tripleo i think | 23:21 |
| EmilienM | mhh k | 23:21 |
| EmilienM | and no progress on https://bugs.launchpad.net/ubuntu/+source/glance/+bug/1657459 | 23:22 |
| openstack | Launchpad bug 1657459 in glance (Ubuntu) "WebOb>=1.2.3 requirement for Glance will lead to 0 bytes backing image files on OpenStack Newton, although the image file sent to the python client does not have 0 bytes" [Low,Triaged] | 23:22 |
| mwhahaha | https://review.openstack.org/#/c/423546/ isnt' that the fix? | 23:22 |
| EmilienM | wonderful | 23:22 |
| *** _ducttape_ has quit IRC | 23:22 | |
| mwhahaha | i saw a ML not about they respun the webob package | 23:22 |
| mwhahaha | to < 1.7 | 23:22 |
| mwhahaha | i hadn't checked to see if that was the case yet | 23:22 |
| EmilienM | most probably | 23:22 |
| EmilienM | panda might know better | 23:22 |
| openstackgerrit | Merged openstack/puppet-ironic: Fix deprecated cleaning_network_uuid and provisioning_network_uuid https://review.openstack.org/423308 | 23:23 |
| panda | mwhahaha: yes, should be the fix | 23:30 |
| panda | mwhahaha: if that doesn't work, we'll revert the patch the broke everything | 23:30 |
| openstackgerrit | Merged openstack/puppet-cinder: Support for ScaleIO Cinder backend https://review.openstack.org/422778 | 23:31 |
| EmilienM | mwhahaha: I might disappear | 23:34 |
| EmilienM | enjoy the weekend folks | 23:34 |
| mwhahaha | later | 23:34 |
| * mwhahaha gives up for the week as well | 23:34 | |
| EmilienM | have rest and fun | 23:35 |
| openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: catch SElinux alerts in permissive mode https://review.openstack.org/423554 | 23:41 |
| dmsimard | ^ a backport that didn't make it's way back to mitaka | 23:42 |
| dmsimard | made it so mitaka was running in enforcing on environments where enforcing is default | 23:42 |
| *** dfisher has quit IRC | 23:43 | |
| EmilienM | dmsimard: ok if it pass CI i'll approve it over the weekend | 23:43 |
| EmilienM | I'm off now | 23:44 |
| openstackgerrit | Merged openstack/puppet-openstack-guide: Add meeting #104 https://review.openstack.org/421314 | 23:45 |
| *** panda is now known as panda|Zz | 23:49 | |
| openstackgerrit | Merged openstack/puppet-magnum: Add unit test for client.pp https://review.openstack.org/423135 | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!