Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-01-20

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

All times shown according to UTC.

Time Nick Message
00:00 owalsh it matters because the default cell is required, you can't not have a single cell setup in ocata
00:01 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:02 owalsh we don't necessary know the transport/db, might be default
00:02 mwhahaha the operator who is constructing their manifest will
00:04 * 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:05 mwhahaha so there -^
00:05 mwhahaha i didn't update all the tests
00:06 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:08 mwhahaha actually since we reverted the stable/newton cell v2 stuff i think we can clean it up further
00:10 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:11 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:12 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:14 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:17 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:20 ducttape_ 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:21 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:22 owalsh mwhahaha: yes, but isn't that what we are telling create_cell to do?
00:23 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:24 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:26 mwhahaha also deletes become viable because we'll have the uuid
00:31 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 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:32 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:33 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
01:37 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
01:54 ducttape_ joined #puppet-openstack
01:55 acormier joined #puppet-openstack
01:59 sofie joined #puppet-openstack
02:13 ducttape_ joined #puppet-openstack
02:31 acormier joined #puppet-openstack
03:20 imcsk8 joined #puppet-openstack
03:33 higgins joined #puppet-openstack
03:43 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
03:52 agrebennikov_ joined #puppet-openstack
04:06 acormier joined #puppet-openstack
04:08 spredzy joined #puppet-openstack
04:25 udesale joined #puppet-openstack
04:56 mfisch mwhahaha: you there?
04:59 agrebennikov_ joined #puppet-openstack
05:08 mfisch mwhahaha: will catch you tomorrow or next week
05:09 udesale joined #puppet-openstack
05:11 udesale joined #puppet-openstack
05:30 imcsk8_ joined #puppet-openstack
05:35 timrc joined #puppet-openstack
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-aodh: Fix typo in aodh_api_paste_ini_spec.rb  https://review.openstack.org/423053
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-ceilometer: Fix typo in ceilometer_api_paste_ini_spec.rb  https://review.openstack.org/423054
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-gnocchi: Fix typo in gnocchi_api_paste_ini_spec.rb  https://review.openstack.org/423056
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-heat: Fix typo in heat_api_paste_ini_spec.rb  https://review.openstack.org/423057
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-ironic: Fix typo in ironic_api_paste_ini_spec.rb  https://review.openstack.org/423058
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-keystone: Fix typos in keystone_paste_ini_spec.rb & keystone_config_spec.rb  https://review.openstack.org/423060
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-neutron: Fix typo in neutron_api_paste_ini_spec.rb  https://review.openstack.org/423062
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-sahara: Fix typo in sahara_api_paste_ini_spec.rb  https://review.openstack.org/423063
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-swift: Fix typo in swift_keystone_test.rb & swift.rb  https://review.openstack.org/423065
06:19 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
07:10 yprokule joined #puppet-openstack
07:23 hjensas joined #puppet-openstack
07:23 hjensas joined #puppet-openstack
07:36 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:44 LarsErikP joined #puppet-openstack
07:46 tdb_ joined #puppet-openstack
07:48 rasca joined #puppet-openstack
07:49 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:50 ntpttr_ joined #puppet-openstack
07:50 Hunner_ joined #puppet-openstack
07:50 Hunner_ joined #puppet-openstack
07:51 tdasilva joined #puppet-openstack
08:05 tesseract joined #puppet-openstack
08:09 paramite joined #puppet-openstack
08:14 shardy joined #puppet-openstack
08:43 jpich joined #puppet-openstack
08:55 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:59 saneax I was hearing there is an effort to have os-net-config via puppet module, is there any effort on that direction?
09:01 ccamacho joined #puppet-openstack
09:04 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:07 tobasco mkarpin: we need to cherry pick the notification_transport_url aswell, atleast for puppet-glance
09:09 social joined #puppet-openstack
09:10 mkarpin tobasco: I'll try to look into it a bit later, or maybe you could do it?
09:11 tobasco yeah, I'm gonna cherry pick the notification_transport_url commit that mwhahaha merged into master
09:14 openstackgerrit joined #puppet-openstack
09:14 openstackgerrit zhongshengping proposed openstack/puppet-magnum: Add unit test for client.pp  https://review.openstack.org/423135
09:15 derekh joined #puppet-openstack
09:21 gfidente joined #puppet-openstack
09:21 gfidente joined #puppet-openstack
09:22 fxpester joined #puppet-openstack
09:23 nmathew joined #puppet-openstack
09:30 openstackgerrit zhongshengping proposed openstack/puppet-aodh: Fix annotation of the ensure parameter  https://review.openstack.org/423144
09:36 zhongshengping joined #puppet-openstack
09:42 openstackgerrit Tobias Urdin proposed openstack/puppet-neutron: Deprecate rabbitmq connection parameters  https://review.openstack.org/423147
09:59 owalsh joined #puppet-openstack
10:00 salmankhan joined #puppet-openstack
10:10 salmankhan1 joined #puppet-openstack
10:27 ansiwen_ joined #puppet-openstack
10:27 guimaluf joined #puppet-openstack
10:38 owalsh joined #puppet-openstack
11:02 openstackgerrit Merged openstack/puppet-horizon: Manage password_validator with puppet manifest  https://review.openstack.org/413653
11:13 richm joined #puppet-openstack
11:29 ducttape_ joined #puppet-openstack
11:47 yprokule joined #puppet-openstack
12:03 ansmith joined #puppet-openstack
12:08 cdelatte joined #puppet-openstack
12:08 openstackgerrit Merged openstack/puppet-gnocchi: Remove carbonara package reference  https://review.openstack.org/422918
12:18 nmathew joined #puppet-openstack
12:24 EmilienM hi
12:33 mkarpin Hi, guys, mwhahaha, EmilienM I saw the mail, thank you for your trust!
12:37 EmilienM mkarpin: thank *you* for your work!
12:49 openstackgerrit Brent Eagles proposed openstack/puppet-octavia: Move db include from ::octavia to ::octavia::api  https://review.openstack.org/422031
12:53 morazi joined #puppet-openstack
12:59 dprince joined #puppet-openstack
13:00 hjensas joined #puppet-openstack
13:04 zhongshengping joined #puppet-openstack
13:11 ducttape_ joined #puppet-openstack
13:11 morazi joined #puppet-openstack
13:15 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Allow configuring node_not_found_hook in inspector  https://review.openstack.org/423256
13:17 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:18 degorenko EmilienM, ah, right, approved
13:18 EmilienM thanks
13:24 ducttape_ joined #puppet-openstack
13:24 pradk can i get some reviews on https://review.openstack.org/#/c/420908/ please
13:25 EmilienM pradk: I just +2 it
13:25 EmilienM it was on my list
13:25 pradk ah thx :)
13:27 shardy joined #puppet-openstack
13:31 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Configure ironic-inspector parameters related to node discovery  https://review.openstack.org/423256
13:43 acormier joined #puppet-openstack
13:51 ansmith joined #puppet-openstack
13:56 openstackgerrit zhongshengping proposed openstack/puppet-nova: Fix api config  https://review.openstack.org/423279
13:57 acormier joined #puppet-openstack
14:00 openstackgerrit Merged openstack/puppet-nova: Allow python-memcache install from authtoken class  https://review.openstack.org/420127
14:05 cgfbee joined #puppet-openstack
14:11 openstackgerrit Brent Eagles proposed openstack/puppet-neutron: Remove several deprecated configuration items in Ocata  https://review.openstack.org/422755
14:36 dtantsur EmilienM, quick puppet question: does "pick" work with $::os_service_default?
14:36 dtantsur like in https://github.com/openstack/puppet-ironi​c/blob/master/manifests/conductor.pp#L161 ?
14:37 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:38 dtantsur the report from our QE: https://bugzilla.redhat.co​m/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:39 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:40 dtantsur another dumg puppet question: with "undef" values, should I use "if $value" or "if $value != undef"?
14:41 llua you can do if $value
14:41 openstackgerrit zhongshengping proposed openstack/puppet-nova: Fix api config  https://review.openstack.org/423279
14:41 dtantsur thanks llua
14:44 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:45 zhongshengping joined #puppet-openstack
14:49 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 openstackgerrit Merged openstack/puppet-openstack-integration: logs: add keystone-resources for debug  https://review.openstack.org/422302
14:51 iurygregory joined #puppet-openstack
14:53 ducttape_ joined #puppet-openstack
15:03 openstackgerrit Tobias Urdin proposed openstack/puppet-cinder: Allow blank rabbit password if using transport_url  https://review.openstack.org/423320
15:09 acormier joined #puppet-openstack
15:09 acormier joined #puppet-openstack
15:12 acormier joined #puppet-openstack
15:21 udesale joined #puppet-openstack
15:25 udesale joined #puppet-openstack
15:29 ducttape_ joined #puppet-openstack
15:39 jaosorior joined #puppet-openstack
15:45 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:46 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:49 shardy joined #puppet-openstack
15:55 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:57 mjblack zhongshengping: pki token might be
15:58 xarses joined #puppet-openstack
15:58 mjblack but the pki is still there for others
15:59 xarses_ joined #puppet-openstack
16:01 zhongshengping joined #puppet-openstack
16:01 zhongshengping What puppet modules? or What parameters?
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:03 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: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:06 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:07 openstackgerrit zhongshengping proposed openstack/puppet-nova: Fix api config  https://review.openstack.org/423279
16:08 mwhahaha well lets just revert the keystone change then
16:09 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Revert "Remove PKI signing related parameters"  https://review.openstack.org/423349
16:10 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Revert "Remove PKI signing related parameters"  https://review.openstack.org/423349
16:10 dfisher joined #puppet-openstack
16:43 paramite joined #puppet-openstack
16:49 jaosorior joined #puppet-openstack
16:59 _ducttape_ joined #puppet-openstack
17:05 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup  https://review.openstack.org/422248
17:06 mwhahaha owalsh: -^ reorganized some stuff and threw in an acceptance test so we'll see if it works
17:07 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: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:12 mkarpin mwhahaha: ok, thanks, I have the same opinion
17:17 agrebennikov_ joined #puppet-openstack
17:24 pradk mwhahaha, EmilienM, does this error ring any bells? "Error: No set of title patterns matched the title "@service". on node"
17:25 openstackgerrit Merged openstack/puppet-neutron: Add new MidoNet plugin package  https://review.openstack.org/421847
17:32 jaosorior joined #puppet-openstack
17:33 openstackgerrit Merged openstack/puppet-ceilometer: Configure compute params in config  https://review.openstack.org/420908
17:36 ducttape_ joined #puppet-openstack
17:45 iurygregory hello
17:56 mwhahaha pradk: sounds like a provider error
18:01 dprince joined #puppet-openstack
18:12 openstackgerrit Ricardo Noriega proposed openstack/puppet-neutron: Add support for BGPVPN plugin  https://review.openstack.org/422051
18:14 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
18:37 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup  https://review.openstack.org/422248
18:45 ccamacho joined #puppet-openstack
18:49 owalsh-brb joined #puppet-openstack
18:50 hecliunyx joined #puppet-openstack
19:27 nkinder joined #puppet-openstack
19:31 paramite 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:32 pradk amoralej, ^^
19:32 amoralej EmilienM, mwhahaha we need ^ for puppet promotion
19:34 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:42 dprince Hunner: hi again, getting back to these noop providers again https://review.openstack.org/#/c/209030/
19:43 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:47 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
19:52 jaosorior joined #puppet-openstack
19:54 openstackgerrit Pradeep Kilambi proposed openstack/puppet-gnocchi: Remove api-paste configuration  https://review.openstack.org/423504
20:02 openstackgerrit Merged openstack/puppet-vitrage: Fix typo in vitrage_api_paste_ini_spec.rb  https://review.openstack.org/423068
20:03 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
20:04 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup  https://review.openstack.org/422248
20:06 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
20:14 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:15 mwhahaha we use it instead of $::processorcount
20:15 pradk mwhahaha, where is the fact calculated?
20:15 mwhahaha https://github.com/openstack/puppet-opensta​cklib/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:16 mwhahaha dfisher: you coming out to the openstack meetup next week? :D
20:16 dfisher more info!
20:16 dfisher U SEND
20:17 dfisher because … things are changing 'round here.
20:17 dfisher :(
20:17 mwhahaha dfisher: https://www.meetup.com/OpenSta​ck-Colorado/events/236828606/
20:18 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:19 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:20 hecliunyx joined #puppet-openstack
20:22 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:23 dfisher vinsh:  Busaba on the map from the link
20:24 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:25 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:25 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:28 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:32 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:33 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:34 dfisher nope!
20:34 dmsimard and now it will never get merged
20:34 hecliunyx joined #puppet-openstack
20:36 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:39 dfisher well, if people want to know more about the Solaris stuff, lemme know.
21:42 rajinir joined #puppet-openstack
21:48 salmankhan joined #puppet-openstack
21:48 _ducttape_ joined #puppet-openstack
21:50 _ducttape_ joined #puppet-openstack
21:50 salmankhan joined #puppet-openstack
22:00 openstackgerrit David Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/420992
22:01 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup  https://review.openstack.org/422248
22:59 xarses_ joined #puppet-openstack
23:00 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Add python-gabbi for tempest on Ubuntu  https://review.openstack.org/423547
23:09 EmilienM mwhahaha: what is blocking promotion?
23:09 EmilienM for puppet CI
23:10 mwhahaha https://review.openstack.org/#/c/423497/
23:10 EmilienM mwhahaha: approved
23:11 EmilienM mwhahaha: is there any patch I need to look?
23:11 EmilienM I'm here ~30 min
23:11 mwhahaha no
23:12 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:13 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:14 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:15 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:16 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:17 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:19 EmilienM mwhahaha: I commented
23:20 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:21 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:22 EmilienM and no progress on https://bugs.launchpad.net/ubun​tu/+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 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:23 openstackgerrit Merged openstack/puppet-ironic: Fix deprecated cleaning_network_uuid and provisioning_network_uuid  https://review.openstack.org/423308
23:30 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:31 openstackgerrit Merged openstack/puppet-cinder: Support for ScaleIO Cinder backend  https://review.openstack.org/422778
23:34 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:35 EmilienM have rest and fun
23:41 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: catch SElinux alerts in permissive mode  https://review.openstack.org/423554
23:42 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:43 EmilienM dmsimard: ok if it pass CI i'll approve it over the weekend
23:44 EmilienM I'm off now
23:45 openstackgerrit Merged openstack/puppet-openstack-guide: Add meeting #104  https://review.openstack.org/421314
23:51 openstackgerrit Merged openstack/puppet-magnum: Add unit test for client.pp  https://review.openstack.org/423135

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