Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-04-06

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

All times shown according to UTC.

Time Nick Message
00:04 ntpttr joined #puppet-openstack
00:05 openstackgerrit Merged openstack/puppet-vswitch stable/ocata: OVS2.6 changes for DPDK  https://review.openstack.org/453197
00:10 ntpttr joined #puppet-openstack
00:16 ntpttr joined #puppet-openstack
00:22 ntpttr joined #puppet-openstack
00:30 ntpttr joined #puppet-openstack
00:34 ducttape_ joined #puppet-openstack
00:36 ntpttr joined #puppet-openstack
00:42 ntpttr joined #puppet-openstack
00:45 guoguo joined #puppet-openstack
00:50 ntpttr joined #puppet-openstack
00:55 ducttape_ joined #puppet-openstack
00:56 ntpttr joined #puppet-openstack
01:04 ntpttr joined #puppet-openstack
01:09 ducttape_ joined #puppet-openstack
01:12 ntpttr joined #puppet-openstack
01:18 ntpttr joined #puppet-openstack
01:24 ntpttr joined #puppet-openstack
01:32 ntpttr joined #puppet-openstack
01:34 ducttape_ joined #puppet-openstack
01:42 ntpttr joined #puppet-openstack
01:48 ilbot3 joined #puppet-openstack
01:48 Topic for #puppet-openstack is now CI Status: GREEN(ish) | Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
01:48 ntpttr joined #puppet-openstack
01:49 jaganathan joined #puppet-openstack
01:56 ntpttr joined #puppet-openstack
02:02 ducttape_ joined #puppet-openstack
02:02 ntpttr joined #puppet-openstack
02:04 acormier joined #puppet-openstack
02:08 ducttape_ joined #puppet-openstack
02:08 ntpttr joined #puppet-openstack
02:20 acormier joined #puppet-openstack
02:30 ntpttr joined #puppet-openstack
03:05 acormier_ joined #puppet-openstack
03:11 ducttape_ joined #puppet-openstack
03:29 jaganathan joined #puppet-openstack
03:31 ntpttr joined #puppet-openstack
03:35 ntpttr joined #puppet-openstack
03:41 ntpttr joined #puppet-openstack
03:45 ntpttr joined #puppet-openstack
03:51 ntpttr joined #puppet-openstack
03:57 ntpttr joined #puppet-openstack
04:03 ntpttr joined #puppet-openstack
04:09 ntpttr joined #puppet-openstack
04:09 udesale joined #puppet-openstack
04:15 ntpttr joined #puppet-openstack
04:16 paramite joined #puppet-openstack
04:21 ntpttr joined #puppet-openstack
04:26 ntpttr joined #puppet-openstack
04:28 ducttape_ joined #puppet-openstack
04:32 ntpttr joined #puppet-openstack
04:39 ntpttr joined #puppet-openstack
04:40 xarses joined #puppet-openstack
04:41 andreww joined #puppet-openstack
04:44 ntpttr joined #puppet-openstack
04:53 ntpttr joined #puppet-openstack
04:53 skramaja joined #puppet-openstack
04:59 ntpttr joined #puppet-openstack
05:05 ntpttr joined #puppet-openstack
05:11 ntpttr joined #puppet-openstack
05:17 ntpttr joined #puppet-openstack
05:22 ntpttr joined #puppet-openstack
05:29 ntpttr joined #puppet-openstack
05:30 rwsu joined #puppet-openstack
05:35 ntpttr joined #puppet-openstack
05:44 ntpttr joined #puppet-openstack
05:58 ducttape_ joined #puppet-openstack
05:59 ntpttr joined #puppet-openstack
06:07 openstackgerrit Merged openstack/puppet-openstack-guide master: Add meeting #109  https://review.openstack.org/453225
06:08 makowals joined #puppet-openstack
06:11 ntpttr joined #puppet-openstack
06:19 jtomasek joined #puppet-openstack
06:20 ntpttr joined #puppet-openstack
06:25 openstackgerrit Merged openstack/puppet-aodh master: Drop -dev for release prep  https://review.openstack.org/453750
06:27 jaganathan joined #puppet-openstack
06:28 openstackgerrit Merged openstack/puppet-oslo master: Drop -dev for release prep  https://review.openstack.org/453773
06:29 ntpttr joined #puppet-openstack
06:29 openstackgerrit Merged openstack/puppet-swift master: Drop -dev for release prep  https://review.openstack.org/453777
06:31 openstackgerrit Merged openstack/puppet-nova master: Drop -dev for release prep  https://review.openstack.org/453769
06:31 openstackgerrit Merged openstack/puppet-zaqar master: Drop -dev for release prep  https://review.openstack.org/453783
06:32 openstackgerrit Merged openstack/puppet-watcher master: Drop -dev for release prep  https://review.openstack.org/453782
06:32 openstackgerrit Merged openstack/puppet-vswitch master: Drop -dev for release prep  https://review.openstack.org/453781
06:32 openstackgerrit Merged openstack/puppet-trove master: Drop -dev for release prep  https://review.openstack.org/453780
06:33 openstackgerrit Merged openstack/puppet-barbican master: Drop -dev for release prep  https://review.openstack.org/453751
06:34 ntpttr joined #puppet-openstack
06:34 openstackgerrit Merged openstack/puppet-octavia master: Drop -dev for release prep  https://review.openstack.org/453770
06:35 openstackgerrit Merged openstack/puppet-tempest master: Drop -dev for release prep  https://review.openstack.org/453779
06:36 openstackgerrit Merged openstack/puppet-tacker master: Drop -dev for release prep  https://review.openstack.org/453778
06:37 openstackgerrit Merged openstack/puppet-sahara master: Drop -dev for release prep  https://review.openstack.org/453776
06:37 openstackgerrit Merged openstack/puppet-panko master: Drop -dev for release prep  https://review.openstack.org/453775
06:37 openstackgerrit Merged openstack/puppet-openstacklib master: Drop -dev for release prep  https://review.openstack.org/453772
06:38 openstackgerrit Merged openstack/puppet-openstack_extras master: Drop -dev for release prep  https://review.openstack.org/453771
06:38 openstackgerrit Merged openstack/puppet-neutron master: Drop -dev for release prep  https://review.openstack.org/453768
06:39 openstackgerrit Merged openstack/puppet-ovn master: Drop -dev for release prep  https://review.openstack.org/453774
06:39 openstackgerrit Merged openstack/puppet-murano master: Drop -dev for release prep  https://review.openstack.org/453767
06:40 openstackgerrit Merged openstack/puppet-mistral master: Drop -dev for release prep  https://review.openstack.org/453766
06:40 ntpttr joined #puppet-openstack
06:41 openstackgerrit Merged openstack/puppet-manila master: Drop -dev for release prep  https://review.openstack.org/453764
06:41 openstackgerrit Merged openstack/puppet-magnum master: Drop -dev for release prep  https://review.openstack.org/453763
06:42 openstackgerrit Merged openstack/puppet-keystone master: Drop -dev for release prep  https://review.openstack.org/453762
06:42 openstackgerrit Merged openstack/puppet-ironic master: Drop -dev for release prep  https://review.openstack.org/453761
06:42 openstackgerrit Merged openstack/puppet-horizon master: Drop -dev for release prep  https://review.openstack.org/453760
06:43 openstackgerrit Merged openstack/puppet-heat master: Drop -dev for release prep  https://review.openstack.org/453759
06:43 openstackgerrit Merged openstack/puppet-gnocchi master: Drop -dev for release prep  https://review.openstack.org/453758
06:44 skramaja_ joined #puppet-openstack
06:45 openstackgerrit Merged openstack/puppet-glance master: Drop -dev for release prep  https://review.openstack.org/453757
06:45 openstackgerrit Merged openstack/puppet-ec2api master: Drop -dev for release prep  https://review.openstack.org/453756
06:46 openstackgerrit Merged openstack/puppet-designate master: Drop -dev for release prep  https://review.openstack.org/453755
06:46 openstackgerrit Merged openstack/puppet-congress master: Drop -dev for release prep  https://review.openstack.org/453754
06:46 openstackgerrit Merged openstack/puppet-cinder master: Drop -dev for release prep  https://review.openstack.org/453753
06:46 openstackgerrit Merged openstack/puppet-ceilometer master: Drop -dev for release prep  https://review.openstack.org/453752
06:46 yprokule joined #puppet-openstack
06:51 skramaja_ joined #puppet-openstack
06:53 ntpttr joined #puppet-openstack
06:58 ntpttr joined #puppet-openstack
07:01 jpich joined #puppet-openstack
07:04 tesseract joined #puppet-openstack
07:05 ntpttr joined #puppet-openstack
07:10 rcernin joined #puppet-openstack
07:10 rcernin joined #puppet-openstack
07:11 ntpttr joined #puppet-openstack
07:21 ntpttr joined #puppet-openstack
07:27 ntpttr joined #puppet-openstack
07:35 ntpttr joined #puppet-openstack
07:43 rmart04 joined #puppet-openstack
07:43 ntpttr joined #puppet-openstack
07:49 ntpttr joined #puppet-openstack
07:50 karthiks joined #puppet-openstack
07:52 yprokule_ joined #puppet-openstack
07:55 ntpttr joined #puppet-openstack
07:55 ccamacho joined #puppet-openstack
07:56 jaosorior mkarpin: are you around?
07:56 mkarpin jaosorior: Hi yeah
07:56 skramaja joined #puppet-openstack
07:58 jaosorior mkarpin: I'm thinking of adding the domain creation in the domain-specific ldap backends bit of puppet-keystone https://github.com/openstack/puppet-keystone/blob/master/manifests/ldap_backend.pp
07:59 jaosorior mkarpin: however, that requires that after a domain creation (via the keystome_domain resource) the keystone service be refreshed
07:59 jaosorior mkarpin: any idea on how to do that? I think if I just add a notify, there will be a cycle
07:59 jaosorior * dependency cycle
08:01 ntpttr joined #puppet-openstack
08:05 mkarpin jaosorior: hmm I see, but I think we can refresh keystone service after ldap config was created but before actually keystone_domain resource is applied, refresh is needed just in order to read ldap domain config for keystone
08:07 ntpttr joined #puppet-openstack
08:07 jaosorior mkarpin: right, but that's not the case for domain-specific ldap backends. One needs to create the domain in keystone e.g. openstack domain create mydomain.  And afterwards one needs to restart httpd, that way keystone will read the configuration from  /etc/keystone/domains/keystone.mydomain.conf
08:12 ntpttr joined #puppet-openstack
08:17 mkarpin jaosorior: but this keystone.mydomain.conf is generated by keystone_domain_config resource in that ldap_backend define, from my POV we can do it like this keystone_domain_config refreshes service, after this we are creating needed domain in keystone using this command, and it will be fine, I think it's not required to refresh service on create domain command
08:17 mkarpin we need refresh service only when we are changing its config
08:18 mkarpin files
08:19 ntpttr joined #puppet-openstack
08:19 shardy joined #puppet-openstack
08:20 yprokule_ joined #puppet-openstack
08:22 jaosorior mkarpin: unfortunately, if you don't do a "domain create" keystone won't take into account the configuration
08:23 jaosorior mkarpin: this is addressed by configuring domains via the API. But there is no support in the OpenStack CLI yet.
08:28 mkarpin jaosorior: hmm it's strange, in Fuel we have ldap plugin which refreshes service only on keystone_config resource and it works somehow)
08:28 ntpttr joined #puppet-openstack
08:28 jaosorior mkarpin: is it multiple ldap backends? or just one?
08:29 yprokule_ joined #puppet-openstack
08:30 jaosorior mkarpin: if it's a single ldap backend then it works and that's expected. But this is for domain-specific ldap backends
08:33 mkarpin jaosorior: yes it can setup multiple domains
08:35 ntpttr joined #puppet-openstack
08:38 mkarpin jaosorior: a actually we refresh service also on some other resources, so maybe that's why it works https://github.com/openstack/fuel-plugin-ldap/blob/master/deployment_scripts/puppet/modules/plugin_ldap/manifests/controller.pp#L174-L176
08:39 jaosorior mkarpin: right, that makes a lot of sense
08:40 jaosorior mkarpin: so I was thinking it would make sense to add that refresh to puppet-keystone
08:41 jaosorior mkarpin: what do you think?
08:45 derekh joined #puppet-openstack
08:47 openstackgerrit Ricardo Noriega proposed openstack/puppet-neutron master: Adding database sync for L2 Gateway service plugin  https://review.openstack.org/451752
08:47 mkarpin jaosorior: I understand, but not sure it can be easily done considering how our deps are written currently... need to think about it
08:49 jaosorior mkarpin: I can also just push it to puppet-tripleo if it messes up something for fuel
08:49 ntpttr joined #puppet-openstack
08:49 jaosorior wouldn't want to break stuff
08:49 jtomasek_ joined #puppet-openstack
08:54 yprokule_ joined #puppet-openstack
08:57 ntpttr joined #puppet-openstack
08:58 salmankhan joined #puppet-openstack
09:00 mkarpin jaosorior: honestly I don't think it breaks something for Fuel, I just think it is more operator's thing, it may bring unneeded complexity to the module itself, so maybe indeed we should keep it out of our modules, but lets also discuss this with mwhahaha and EmilienM, maybe I am missing something.
09:03 ntpttr joined #puppet-openstack
09:09 ntpttr joined #puppet-openstack
09:10 jaosorior mkarpin: sure, that sounds fair
09:13 vnogin joined #puppet-openstack
09:15 ntpttr joined #puppet-openstack
09:17 hjensas joined #puppet-openstack
09:17 hjensas joined #puppet-openstack
09:20 nmathew joined #puppet-openstack
09:34 ntpttr joined #puppet-openstack
09:43 ntpttr joined #puppet-openstack
09:49 ntpttr joined #puppet-openstack
09:53 karthiks joined #puppet-openstack
09:59 ntpttr joined #puppet-openstack
10:00 gfidente joined #puppet-openstack
10:00 gfidente joined #puppet-openstack
10:05 ntpttr joined #puppet-openstack
10:08 nmathew- joined #puppet-openstack
10:11 ntpttr joined #puppet-openstack
10:13 richm joined #puppet-openstack
10:17 ntpttr joined #puppet-openstack
10:27 markvoelker joined #puppet-openstack
10:29 ntpttr joined #puppet-openstack
10:30 karthiks joined #puppet-openstack
10:36 ntpttr joined #puppet-openstack
10:41 ntpttr joined #puppet-openstack
10:51 ntpttr joined #puppet-openstack
10:55 openstackgerrit Merged openstack/puppet-aodh stable/newton: Add hooks for external install & svc management  https://review.openstack.org/453431
10:56 karthiks joined #puppet-openstack
10:56 ntpttr joined #puppet-openstack
11:03 ntpttr joined #puppet-openstack
11:04 jtomasek joined #puppet-openstack
11:13 ntpttr joined #puppet-openstack
11:23 ntpttr joined #puppet-openstack
11:28 ntpttr joined #puppet-openstack
11:29 EmilienM hello
11:31 openstackgerrit Mykyta Karpin proposed openstack/puppet-glare master: Update glare package name for Debian  https://review.openstack.org/454114
11:31 mkarpin EmilienM: hey
11:31 EmilienM hey oh
11:32 mkarpin we with jaosorior: have a question
11:33 mkarpin about creation of domains for ldap backends for keystone
11:33 jaosorior EmilienM: right, so currently there is an ldap_backend resource in puppet-keystone. Which creates the configurations for ldap backends per domain
11:33 jaosorior EmilienM: however, there are extra steps needed for that to actually do its job
11:34 jaosorior 1. actually create the domain in keystone (which we can do via keystone_domain)
11:34 jaosorior 2. after the domain has been created and the configuration is stored, we need to restart the keystone server (httpd) so it reads the configuration for that domain
11:35 jaosorior EmilienM: there are two options: * Add that to puppet-keystone  * keep that in the deployment-specific resources (e.g. in fuel or in puppet-tripleo)
11:35 ntpttr joined #puppet-openstack
11:35 jaosorior what's your take on this?
11:36 EmilienM gimme a sec
11:37 mkarpin but in puppet-keystone we create domain only after we are sure service is running https://github.com/openstack/puppet-keystone/blob/master/manifests/deps.pp#L65 so it's a kind of loop
11:37 jaosorior right
11:39 EmilienM restart keystone? wow
11:39 EmilienM that sucks
11:40 EmilienM I would try to solve it in puppet-keystone and make it optional
11:40 EmilienM with a boolean
11:40 EmilienM so if fuel or/and tripleo want to use it, good
11:40 EmilienM but do not do it in puppet-tripleo please
11:40 karthiks joined #puppet-openstack
11:41 jaosorior EmilienM: why not?
11:42 jaosorior EmilienM: yeah, gotta restart keystone :/, this is a limitation in the file-based backend. It's not a problem with the API based backend, but there is no support for this yet via the OpenStack CLI, and after support comes from there, then there's gonna be a provider needed too. So this is what we have at the moment :/
11:45 mkarpin jaosorior: are you sure that keystone_domain_config (file edition) -> service refresh -> domain create sequence won't work?
11:45 EmilienM jaosorior: why not? Because we try to not carry any logic in our module, and move everything to the openstack/puppet-* modules to fuel and other users can use it
11:45 EmilienM this use case is not only for tripleo, so we want this code in puppet-keystone
11:47 jaosorior mkarpin: pretty sure, it was verified by a colleague and they even mention it in this book:  http://shop.oreilly.com/product/0636920045960.do
11:47 jaosorior mkarpin: either way, had a colleague try it out in a deployment. We do need that refresh
11:49 ntpttr joined #puppet-openstack
11:50 mkarpin ok, ok than lets do it with boolean as EmilienM suggested
11:50 jaosorior mkarpin, EmilienM: what do you suggest for the refresh? since we would end up with a dependency cycle
11:51 EmilienM jaosorior: there is already a restart in init.pp at the end
11:51 EmilienM for the default domain I think
11:51 EmilienM maybe we could do it before
11:51 jaosorior EmilienM: awesome, thanks
11:51 EmilienM I think
11:51 EmilienM let me look code
11:52 EmilienM here https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L1246-L1250
11:52 EmilienM jaosorior: add something like https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L1253-L1260
11:52 EmilienM but rename default domain to something like ldap domain or whatever
11:55 ntpttr joined #puppet-openstack
11:55 jaosorior EmilienM: damn... it gets a bit tricky with TripleO, since we have a "bootstrap node" that would do the domain creation, but a refresh is needed on all nodes.
11:56 jaosorior EmilienM: or wait... are all the other nodes also started on step 3?>
11:56 gfidente joined #puppet-openstack
11:56 EmilienM mwhahaha and I made httpd start at step3 and it shouldn't restart at step 4
11:57 jaosorior great!
11:57 jaosorior so then we're all good
11:57 xb_ joined #puppet-openstack
12:00 ducttape_ joined #puppet-openstack
12:02 ducttape_ joined #puppet-openstack
12:03 acormier joined #puppet-openstack
12:04 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
12:04 jaosorior EmilienM, mkarpin what about this? ^^
12:05 ntpttr joined #puppet-openstack
12:08 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
12:10 jaosorior EmilienM, mkarpin: If that looks like a reasonable approach I'll add the spec tests and release note, and then that'll be tested on an LDAP enabled deployment. Let me know what you think.
12:10 EmilienM gimme a sec
12:12 EmilienM jaosorior: perfect to me
12:12 EmilienM go for it
12:13 mkarpin jaosorior: added some small comments
12:15 ntpttr joined #puppet-openstack
12:15 jaosorior mkarpin: thanks, commented.
12:17 ansmith joined #puppet-openstack
12:17 jaosorior mkarpin: addressed the comments, thanks
12:17 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
12:19 mkarpin jaosorior: thanks)
12:21 EmilienM mkarpin: nice catch on the collector
12:21 EmilienM I would have been ok with a collector fwiw
12:26 mkarpin EmilienM: yeah not a big difference, I just don't see a strict need in it here,
12:27 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
12:27 jaosorior lets see if I didn't mess up the spec test
12:27 ntpttr joined #puppet-openstack
12:28 markvoelker joined #puppet-openstack
12:28 dprince joined #puppet-openstack
12:35 ntpttr joined #puppet-openstack
12:37 rpittau joined #puppet-openstack
12:39 ducttape_ joined #puppet-openstack
12:41 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
12:51 ntpttr joined #puppet-openstack
12:53 jaosorior joined #puppet-openstack
12:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
13:03 morazi joined #puppet-openstack
13:03 ntpttr joined #puppet-openstack
13:09 ntpttr joined #puppet-openstack
13:15 ntpttr joined #puppet-openstack
13:21 ntpttr joined #puppet-openstack
13:27 ntpttr joined #puppet-openstack
13:30 nmathew joined #puppet-openstack
13:33 indistylo joined #puppet-openstack
13:33 ntpttr joined #puppet-openstack
13:36 ducttape_ joined #puppet-openstack
13:43 ntpttr joined #puppet-openstack
13:49 ntpttr joined #puppet-openstack
13:56 ntpttr joined #puppet-openstack
14:03 ducttape_ joined #puppet-openstack
14:05 pradk joined #puppet-openstack
14:05 ducttape_ joined #puppet-openstack
14:05 ntpttr joined #puppet-openstack
14:06 ducttape_ joined #puppet-openstack
14:06 agrebennikov joined #puppet-openstack
14:07 ducttape_ joined #puppet-openstack
14:09 acormier joined #puppet-openstack
14:14 paramite joined #puppet-openstack
14:15 ntpttr joined #puppet-openstack
14:16 patrickeast joined #puppet-openstack
14:21 ntpttr joined #puppet-openstack
14:22 hjensas joined #puppet-openstack
14:29 markvoelker joined #puppet-openstack
14:29 ntpttr joined #puppet-openstack
14:35 ntpttr joined #puppet-openstack
14:38 udesale joined #puppet-openstack
14:41 jaganathan joined #puppet-openstack
14:41 ducttape_ joined #puppet-openstack
14:53 openstackgerrit Oliver Walsh proposed openstack/puppet-nova master: Use a resource provider for cell_v2 cells  https://review.openstack.org/439915
14:58 andreww joined #puppet-openstack
14:59 markvoelker joined #puppet-openstack
14:59 xarses joined #puppet-openstack
15:09 dprince joined #puppet-openstack
15:12 ducttape_ joined #puppet-openstack
15:30 indistylo joined #puppet-openstack
15:31 openstackgerrit Mykyta Karpin proposed openstack/puppet-glare master: Add sync_db option to glare init class  https://review.openstack.org/454235
15:37 ntpttr joined #puppet-openstack
15:44 openstackgerrit Mykyta Karpin proposed openstack/puppet-glare master: Add sync_db option to glare init class  https://review.openstack.org/454235
15:54 mkarpin jaosorior: EmilienM: looks like resource collector is needed, sorry that I confused you...
15:55 openstackgerrit Sven Anderson proposed openstack/puppet-tempest master: Enable more tempest tests for ec2-api  https://review.openstack.org/454247
15:59 openstackgerrit Oliver Walsh proposed openstack/puppet-nova master: Use a resource provider for cell_v2 cells  https://review.openstack.org/439915
15:59 alee_ joined #puppet-openstack
16:05 ntpttr joined #puppet-openstack
16:13 EmilienM mkarpin: ah-ah!!!
16:15 jaosorior joined #puppet-openstack
16:15 ntpttr joined #puppet-openstack
16:17 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Enable creating the domain for LDAP backends  https://review.openstack.org/454138
16:17 jaosorior mwhahaha: what do you think? ^^
16:19 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Allow use of device aliases for OSDs  https://review.openstack.org/451826
16:21 ntpttr joined #puppet-openstack
16:29 ntpttr joined #puppet-openstack
16:35 ntpttr joined #puppet-openstack
16:36 mwhahaha gfidente: you need to update the puppet file to stop escaping \$(...)
16:37 gfidente the actual manifest/osd.pp you mean
16:37 mwhahaha gfidente: yea
16:37 gfidente ack
16:43 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Allow use of device aliases for OSDs  https://review.openstack.org/451826
16:45 ntpttr joined #puppet-openstack
16:47 gfidente mwhahaha tests pass locally, should be all sorted
16:47 mwhahaha gfidente: ok
16:47 gfidente let's see how the actual integration and tripleo jobs do
16:50 mwhahaha gfidente: why the change to $journal?
16:51 gfidente mwhahaha because line 133 is always executing readlink
16:51 gfidente and it fails when it's undef
16:52 gfidente returns empty otherwise, as it used to do before
16:52 mwhahaha gfidente: i'm not sure setting it to '' is right
16:53 mwhahaha gfidente: based on the param comments it seems like if $journal is not defined it should default to $title
16:53 ntpttr joined #puppet-openstack
16:54 gfidente mwhahaha no it's actually ceph-disk defaulting to colocate it
16:54 rmart04 joined #puppet-openstack
16:54 gfidente if a path is not given
16:54 mwhahaha hmm ok i'll have to take a closer look
16:59 salmankhan joined #puppet-openstack
17:05 ntpttr joined #puppet-openstack
17:11 ntpttr joined #puppet-openstack
17:19 ntpttr joined #puppet-openstack
17:25 ntpttr joined #puppet-openstack
17:31 ntpttr joined #puppet-openstack
17:37 ntpttr joined #puppet-openstack
17:38 jtomasek joined #puppet-openstack
17:43 ntpttr joined #puppet-openstack
17:45 danpawlik joined #puppet-openstack
17:47 vnogin joined #puppet-openstack
17:49 ntpttr joined #puppet-openstack
17:54 EmilienM owalsh: hey
17:54 EmilienM owalsh: I'm going to land your patch for the provider
17:54 EmilienM so we can make progress
17:54 EmilienM owalsh: do you need a backport?
17:54 EmilienM because if that's the case, it would have been great to have a release note
17:55 ntpttr joined #puppet-openstack
17:56 EmilienM owalsh: also, can we deprecate nova::cell_v2::cell class?
17:57 EmilienM or use the provider in this define
17:57 sdatko joined #puppet-openstack
18:01 ntpttr joined #puppet-openstack
18:02 xarses joined #puppet-openstack
18:02 danpawlik joined #puppet-openstack
18:03 andreww joined #puppet-openstack
18:03 openstackgerrit Merged openstack/puppet-glare master: Update glare package name for Debian  https://review.openstack.org/454114
18:04 andreww joined #puppet-openstack
18:07 ntpttr joined #puppet-openstack
18:13 ntpttr joined #puppet-openstack
18:19 ntpttr joined #puppet-openstack
18:25 ntpttr joined #puppet-openstack
18:31 ntpttr joined #puppet-openstack
18:33 jaganathan joined #puppet-openstack
18:37 ntpttr joined #puppet-openstack
18:43 ntpttr joined #puppet-openstack
18:49 ntpttr joined #puppet-openstack
18:49 openstackgerrit Merged openstack/puppet-neutron master: Adding database sync for L2 Gateway service plugin  https://review.openstack.org/451752
18:50 openstackgerrit Merged openstack/puppet-nova master: Use a resource provider for cell_v2 cells  https://review.openstack.org/439915
18:57 ntpttr joined #puppet-openstack
19:05 ntpttr joined #puppet-openstack
19:11 ntpttr joined #puppet-openstack
19:17 ntpttr joined #puppet-openstack
19:23 ntpttr joined #puppet-openstack
19:25 salmankhan joined #puppet-openstack
19:29 ntpttr joined #puppet-openstack
19:35 ntpttr joined #puppet-openstack
19:45 ntpttr joined #puppet-openstack
19:47 openstack joined #puppet-openstack
20:00 owalsh EmilienM: hey, cool. I'll follow up with a reno and deprecate nova::cell_v2::cell
20:00 EmilienM cool
20:21 imcsk8 joined #puppet-openstack
20:21 zerick joined #puppet-openstack
21:02 openstackstatus joined #puppet-openstack
21:35 openstackgerrit joined #puppet-openstack
21:35 openstackgerrit Pradeep Kilambi proposed openstack/puppet-ceilometer master: Add support to configure publisher in pipeline.yaml  https://review.openstack.org/446801
21:48 openstackgerrit Pradeep Kilambi proposed openstack/puppet-ceilometer master: Add support to configure publisher in pipeline.yaml  https://review.openstack.org/446801
21:54 openstackgerrit Pradeep Kilambi proposed openstack/puppet-ceilometer master: Update default event publisher  https://review.openstack.org/454363
22:02 openstack joined #puppet-openstack
22:55 ansmith joined #puppet-openstack
23:24 xarses joined #puppet-openstack
23:34 xarses joined #puppet-openstack
23:47 imcsk8 joined #puppet-openstack

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