Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:03 acormier joined #puppet-openstack
00:04 ducttape_ joined #puppet-openstack
00:05 ntpttr joined #puppet-openstack
00:11 ntpttr joined #puppet-openstack
00:19 ntpttr joined #puppet-openstack
00:25 ntpttr joined #puppet-openstack
00:28 owalsh_ joined #puppet-openstack
00:31 ntpttr joined #puppet-openstack
00:34 ducttape_ joined #puppet-openstack
00:37 ntpttr joined #puppet-openstack
00:39 acormier joined #puppet-openstack
00:40 acormier joined #puppet-openstack
00:43 ntpttr joined #puppet-openstack
00:49 ntpttr joined #puppet-openstack
00:55 ntpttr joined #puppet-openstack
01:05 ntpttr joined #puppet-openstack
01:13 ntpttr joined #puppet-openstack
01:19 ntpttr joined #puppet-openstack
01:24 openstackgerrit Merged openstack/puppet-ceilometer master: Fix dispatcher params  https://review.openstack.org/453318
01:25 ntpttr joined #puppet-openstack
01:31 ntpttr joined #puppet-openstack
01:37 ntpttr joined #puppet-openstack
01:46 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:49 ntpttr joined #puppet-openstack
01:57 ntpttr joined #puppet-openstack
01:58 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh stable/newton: Add hooks for external install & svc management  https://review.openstack.org/453431
02:05 ntpttr joined #puppet-openstack
02:13 ntpttr joined #puppet-openstack
02:14 ducttape_ joined #puppet-openstack
02:19 ntpttr joined #puppet-openstack
02:24 jaganathan joined #puppet-openstack
02:25 ntpttr joined #puppet-openstack
02:32 acormier joined #puppet-openstack
02:33 ntpttr joined #puppet-openstack
02:39 ntpttr joined #puppet-openstack
02:45 ntpttr joined #puppet-openstack
02:51 ntpttr joined #puppet-openstack
03:03 ntpttr joined #puppet-openstack
03:09 ntpttr joined #puppet-openstack
03:10 acormier joined #puppet-openstack
03:11 vnogin joined #puppet-openstack
03:19 ntpttr joined #puppet-openstack
03:25 ntpttr joined #puppet-openstack
03:31 ntpttr joined #puppet-openstack
03:39 ntpttr joined #puppet-openstack
03:47 ducttape_ joined #puppet-openstack
03:53 ntpttr joined #puppet-openstack
04:01 ntpttr joined #puppet-openstack
04:07 rcernin joined #puppet-openstack
04:07 ntpttr joined #puppet-openstack
04:13 ntpttr joined #puppet-openstack
04:19 ntpttr joined #puppet-openstack
04:25 ntpttr joined #puppet-openstack
04:31 ntpttr joined #puppet-openstack
04:37 ntpttr joined #puppet-openstack
04:41 richm joined #puppet-openstack
04:43 ntpttr joined #puppet-openstack
04:49 ntpttr joined #puppet-openstack
04:55 ntpttr joined #puppet-openstack
05:00 yprokule joined #puppet-openstack
05:01 ntpttr joined #puppet-openstack
05:07 ntpttr joined #puppet-openstack
05:08 udesale joined #puppet-openstack
05:09 nmathew joined #puppet-openstack
05:13 ntpttr joined #puppet-openstack
05:13 ducttape_ joined #puppet-openstack
05:14 skramaja joined #puppet-openstack
05:19 ntpttr joined #puppet-openstack
05:29 ntpttr joined #puppet-openstack
05:37 ntpttr joined #puppet-openstack
05:43 ntpttr joined #puppet-openstack
05:49 ntpttr joined #puppet-openstack
05:57 ntpttr joined #puppet-openstack
06:03 ntpttr joined #puppet-openstack
06:09 jaganathan joined #puppet-openstack
06:11 ntpttr joined #puppet-openstack
06:19 ntpttr joined #puppet-openstack
06:25 ntpttr joined #puppet-openstack
06:31 rcernin joined #puppet-openstack
06:31 ntpttr joined #puppet-openstack
06:37 ntpttr joined #puppet-openstack
06:44 ducttape_ joined #puppet-openstack
06:47 ntpttr joined #puppet-openstack
06:52 tesseract joined #puppet-openstack
06:53 ntpttr joined #puppet-openstack
06:59 ntpttr joined #puppet-openstack
07:04 chem joined #puppet-openstack
07:06 ntpttr joined #puppet-openstack
07:17 ntpttr joined #puppet-openstack
07:22 shardy joined #puppet-openstack
07:25 ntpttr joined #puppet-openstack
07:35 hjensas joined #puppet-openstack
07:35 hjensas joined #puppet-openstack
07:36 ntpttr joined #puppet-openstack
07:41 ntpttr joined #puppet-openstack
07:48 ntpttr joined #puppet-openstack
07:55 vnogin joined #puppet-openstack
07:56 ntpttr joined #puppet-openstack
07:58 jpich joined #puppet-openstack
08:00 chem karthiks: hi, I know why the test are failing on the backport, give me a bit more time and I will comment on the review.
08:04 ntpttr joined #puppet-openstack
08:05 flwang1 joined #puppet-openstack
08:05 valery joined #puppet-openstack
08:08 openstackgerrit Mykyta Karpin proposed openstack/puppet-zaqar master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/453506
08:12 ntpttr joined #puppet-openstack
08:14 ducttape_ joined #puppet-openstack
08:17 openstackgerrit Christian Schwede proposed openstack/puppet-swift master: Add missing Swift services  https://review.openstack.org/445998
08:18 ntpttr joined #puppet-openstack
08:18 chem karthiks: reviewed, if you could have a look, as it may be some kind of unexpected behavior ..
08:20 zioproto good morning
08:20 zioproto I am upgrading cinder from Mitaka to Newton. I have the feeling that default setting for heartbeat_timeout_threshold changes in puppet-cinder but I cant find anything in https://docs.openstack.org/releasenotes/puppet-cinder/newton.html
08:23 zioproto in Mitaka heartbeat_timeout_threshold was explicitly set to 0 in cinder.conf and now this value is gone
08:26 ntpttr joined #puppet-openstack
08:27 zioproto bingo, look at tag 1b14df8e1591c40203d6b57f8a3fad2b917c8e1a
08:27 zioproto sorry I mean commit 1b14df8e1591c40203d6b57f8a3fad2b917c8e1a
08:27 zioproto this was 0 and now is $::os_service_default
08:27 zioproto because 0 was not the default upstream in openstack, I think this should go into the release notes
08:28 zioproto EmilienM: should I propose a patch for the release notes ? never done it before
08:32 ntpttr joined #puppet-openstack
08:39 bogdando joined #puppet-openstack
08:39 ntpttr joined #puppet-openstack
08:41 salmankhan joined #puppet-openstack
08:43 derekh joined #puppet-openstack
08:46 ntpttr joined #puppet-openstack
08:54 ntpttr joined #puppet-openstack
08:54 karthiks chem,  looking
08:55 jtomasek joined #puppet-openstack
08:55 chem karthiks: ack, there may be a problem with the master version as empty string may lead to problem
08:59 gfidente joined #puppet-openstack
08:59 gfidente joined #puppet-openstack
08:59 karthiks chem,  the same test case in master passed. Am i missing something ?
09:00 chem karthiks: it's in the review, puppet4 changed the empty string evaluation in conditional context from false to true
09:01 chem karthiks: relative to puppet3
09:02 karthiks chem,  ack
09:02 ntpttr joined #puppet-openstack
09:02 chem karthiks: then I think that master will have problem with empty string ...
09:06 karthiks chem,  should we check for if host_core_list and !empty(host_core_list) { core_list_string = host_core_list } else if core_list and !empty (core_list) {core_list_string = core_list} else {fail}
09:06 karthiks to avoid the confusion
09:08 ntpttr joined #puppet-openstack
09:13 karthiks chem,  sorry just saw your comments.  Will Fix it
09:16 ntpttr joined #puppet-openstack
09:23 ntpttr joined #puppet-openstack
09:24 rnoriega mkarpin, ping
09:32 ntpttr joined #puppet-openstack
09:38 ntpttr joined #puppet-openstack
09:45 ducttape_ joined #puppet-openstack
09:46 ntpttr joined #puppet-openstack
09:52 ntpttr joined #puppet-openstack
09:56 ntpttr joined #puppet-openstack
10:04 ntpttr joined #puppet-openstack
10:09 openstackgerrit Mykyta Karpin proposed openstack/puppet-barbican master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/448213
10:10 mkarpin rnoriega: hey
10:10 ntpttr joined #puppet-openstack
10:13 richm joined #puppet-openstack
10:15 xarses_ joined #puppet-openstack
10:18 ntpttr joined #puppet-openstack
10:18 chem karthiks: do you need further help for the patch ?
10:19 karthiks chem,  adding a check =='' is also throwing errors
10:20 chem karthiks: ah,  ... oki, let me check
10:24 ntpttr joined #puppet-openstack
10:30 ntpttr joined #puppet-openstack
10:34 ntpttr joined #puppet-openstack
10:34 chem karthiks: well, I see it's failing on other check ... and it kind of making sense
10:40 ntpttr joined #puppet-openstack
10:44 chem joined #puppet-openstack
10:48 chem` joined #puppet-openstack
10:50 rmart04 joined #puppet-openstack
10:51 chem` karthiks: sorry got connection problems.
10:51 chem` karthiks: so if you look at this in master https://github.com/openstack/puppet-vswitch/blob/master/manifests/dpdk.pp#L63-L68
10:52 ntpttr joined #puppet-openstack
10:52 chem` karthiks: when $socket_mem is empty, it will be true in puppet4 and then it will create the string '$socket_string = ""'
10:52 chem` karthiks: I don't think it's the intend here, WDYT
10:53 chem` karthiks: my point being that master is wrong
10:54 chem` karthiks: because it assume that empty string is false (like it was in puppet3)
10:58 ntpttr joined #puppet-openstack
11:04 ntpttr joined #puppet-openstack
11:08 rnoriega mkarpin, nevermind! sorry! :-)
11:08 mkarpin np)
11:11 chem` karthiks: ?
11:12 udesale joined #puppet-openstack
11:12 ntpttr joined #puppet-openstack
11:20 udesale joined #puppet-openstack
11:27 dixiaoli joined #puppet-openstack
11:27 EmilienM hello
11:27 dixiaoli joined #puppet-openstack
11:28 ntpttr joined #puppet-openstack
11:28 morazi joined #puppet-openstack
11:34 Arrfab joined #puppet-openstack
11:34 ntpttr joined #puppet-openstack
11:35 Arrfab hi guys .. based on https://docs.openstack.org/developer/puppet-openstack-guide/supported-platforms.html , can I understand that those puppet modules are supposed to work on for example puppet 3.6.x ?
11:36 EmilienM Arrfab: if you deploy stable/ocata or before (newton, mitaka), then yes you can use Puppet 3.6
11:37 EmilienM but starting from Pike (this cycle), we don't support Puppet 3.6
11:37 Arrfab EmilienM: thanks .. dmsimard told me that I'd have to update, but as our puppetmasterd is still 3.6.x (and so our agents) I'd like to use it
11:38 Arrfab but the issue I have is that I'll have conflict with puppet pkgs from rdo repositories, that will then automatically update my puppet pkgs from our infra repo
11:40 ntpttr joined #puppet-openstack
11:41 EmilienM Arrfab: deploy puppet frm rdo
11:41 EmilienM Arrfab: and make sure you don't have epel
11:42 Arrfab EmilienM: that's what I don't want, as puppet from rdo is higher than what I have in infra
11:42 Arrfab so I want to stick with 3.6.x
11:42 Arrfab it seems I'll have to modify .repo to put exclude, but still I hope that there is no Requires: that would need higher puppet :-(
11:43 Arrfab reason why I was asking about puppet modules themselves not needing a specific puppet version
11:44 Arrfab would that work ?
11:47 Arrfab it seems the only pkg depending on puppet is packstack, so I guess that excluding packstack and puppet* from centos-openstack-$release would probably do it (and then importing all the mandatory puppet-$modulename on puppetmasterd, but not through rpm pkgs)
11:48 EmilienM Arrfab: that's what we use in our CI: http://logs.openstack.org/30/452630/4/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/3c9feb5/logs/repolist.txt.gz
11:50 Arrfab EmilienM: oh, so http://yum.puppetlabs.com/el/7/PC1/x86_64/ .. it's way much higher that I'd want to go
11:50 ntpttr joined #puppet-openstack
11:51 Arrfab EmilienM: thanks for the info, so it seems that I can just continue using puppet 3.6 and cherry-pick puppet openstack modules I want to deploy newton/ocata cloud in our infra .. now time to find the good doc about how to use those
11:56 ntpttr joined #puppet-openstack
11:59 dprince joined #puppet-openstack
11:59 EmilienM Arrfab: that's how we deploy ocata in our CI: https://github.com/openstack/puppet-openstack-integration/tree/stable/ocata
11:59 EmilienM and newton https://github.com/openstack/puppet-openstack-integration/tree/stable/newton
12:00 EmilienM we have 4 scenarios, in fixtures directory
12:00 EmilienM hope it helps
12:05 openstackgerrit Ricardo Noriega proposed openstack/puppet-neutron master: Adding database sync for L2 Gateway service plugin  https://review.openstack.org/451752
12:07 acormier joined #puppet-openstack
12:10 ntpttr joined #puppet-openstack
12:14 ntpttr joined #puppet-openstack
12:26 ntpttr joined #puppet-openstack
12:26 ducttape_ joined #puppet-openstack
12:34 ansmith joined #puppet-openstack
12:36 ntpttr joined #puppet-openstack
12:41 iurygregory morning
12:42 ntpttr joined #puppet-openstack
12:42 jaganathan joined #puppet-openstack
12:45 ducttape_ joined #puppet-openstack
12:50 ntpttr joined #puppet-openstack
12:52 openstackstatus NOTICE: The Gerrit service on http://review.openstack.org is being restarted to address hung remote replication tasks, and should return to an operable state momentarily
12:55 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-vswitch stable/ocata: OVS2.6 changes for DPDK  https://review.openstack.org/453197
12:55 markvoelker joined #puppet-openstack
12:59 openstackgerrit Pradeep Kilambi proposed openstack/puppet-ceilometer stable/ocata: Fix dispatcher params  https://review.openstack.org/453625
13:06 ntpttr joined #puppet-openstack
13:09 openstackgerrit Mykyta Karpin proposed openstack/puppet-glare master: Manage backend section for Glare  https://review.openstack.org/444814
13:13 chem` karthiks: so I went ahead and made a review that make the rspec passes for puppet3 and puppet4, looks at the comment and tell me if that makes sense :)
13:14 chem` karthiks: https://review.openstack.org/#/c/453197/
13:18 ntpttr joined #puppet-openstack
13:28 mkoderer joined #puppet-openstack
13:28 ntpttr joined #puppet-openstack
13:35 rmart04 joined #puppet-openstack
13:36 ducttape_ joined #puppet-openstack
13:36 ntpttr joined #puppet-openstack
13:42 ntpttr joined #puppet-openstack
13:46 agrebennikov joined #puppet-openstack
13:52 ntpttr joined #puppet-openstack
13:52 agurenko joined #puppet-openstack
13:55 rmart04 joined #puppet-openstack
13:58 ntpttr joined #puppet-openstack
14:01 rmart04 joined #puppet-openstack
14:12 udesale joined #puppet-openstack
14:18 jaganathan joined #puppet-openstack
14:31 dmsimard EmilienM, mwhahaha: have you seen that before ? http://paste.openstack.org/show/605515/
14:31 dmsimard errors everywhere in https://logs.rdoproject.org/weirdo-generic-puppet-openstack-scenario001/1406/weirdo-project/logs/nova/nova-api.txt.gz
14:31 mwhahaha dmsimard: nope
14:31 EmilienM no
14:31 dmsimard what's weirdest is that is ran from current-passed-ci ...
14:32 mwhahaha i'm sure there's a new db manage command that needs to be run or something
14:32 * mwhahaha pokes owalsh
14:32 derekh joined #puppet-openstack
14:32 dmsimard mwhahaha: but it wouldn't have passed CI in the first place I guess ?
14:32 dmsimard I'll dig some more ..
14:32 * owalsh wakes up confused
14:32 mwhahaha is it not actually causing a problem perhaps?
14:33 dmsimard mwhahaha: everything is failing, including packstack as well
14:33 mwhahaha weird
14:33 dmsimard mwhahaha: It might be my fault somehow, we're trying to move workloads to "the cloud"
14:34 dprince joined #puppet-openstack
14:34 owalsh mwhahaha: looks like the db schema doesn't match the version of the code
14:36 dmsimard I'm going to try again with puppet-passed-ci out of curiosity
14:37 owalsh dmsimard: services.force_down is not add by a db migration... suggests it's hitting the wrong db, or db has never been synced
14:37 owalsh dmsimard: i.e it's not on the wrong db version, db just doen't have the tables
14:38 dmsimard owalsh: yeah looks like a failure occured fairly early on
14:39 agurenko joined #puppet-openstack
14:40 acormier joined #puppet-openstack
14:46 ntpttr__ joined #puppet-openstack
14:48 ntpttr_laptop joined #puppet-openstack
14:49 jaganathan joined #puppet-openstack
14:50 dmsimard mwhahaha: seeing this as well btw https://bugzilla.redhat.com/show_bug.cgi?id=1408314
14:50 openstack bugzilla.redhat.com bug 1408314 in memcached "memcached.service results in Unknown lvalue 'MemoryDenyWriteExecute' in section 'Service'" [Low,Assigned] - Assigned to lhh
14:51 mwhahaha dmsimard: that bug doesn't affect functionality
14:51 dmsimard that's my understanding as well, just generally digging :)
15:02 xarses joined #puppet-openstack
15:03 andreww joined #puppet-openstack
15:07 ntpttr_laptop joined #puppet-openstack
15:08 ducttape_ joined #puppet-openstack
15:13 EmilienM flwang: re: puppet module. Not sure what you want to merge. What is missing in puppet-zaqar that you have in your version?
15:15 karthiks chem, sorry, I was away for a personal work.
15:16 ntpttr joined #puppet-openstack
15:16 chem karthiks: he man, np.  Have a look when you can, I made some corrections and opened a lp bug for the empty string
15:16 chem karthiks: hope you like it :)
15:17 chem karthiks: all in all it's just about what happen when people passes empty string so it may not be a big deal
15:17 karthiks chem,  Yes. I have 1 comment.  I didn't understand how 2 times !empty () check helps
15:18 chem karthiks: looking I may have made a mistake
15:18 dgurtner joined #puppet-openstack
15:19 openstackgerrit Merged openstack/puppet-zaqar master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/453506
15:19 chem karthiks: a big fat wrong copy/paste ...
15:20 chem karthiks: correcting
15:23 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-vswitch stable/ocata: OVS2.6 changes for DPDK  https://review.openstack.org/453197
15:24 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Allow use of device aliases for OSDs  https://review.openstack.org/451826
15:24 ntpttr joined #puppet-openstack
15:25 openstackgerrit Merged openstack/puppet-barbican master: oslo log: check puppet resource instead of actual config in spec  https://review.openstack.org/448213
15:26 chem karthiks: oki, I've updated it.  The logic is still the same though, so the tests are still passing for puppet3 and puppet4
15:26 karthiks chem,  Thanks
15:27 chem karthiks: the last thing I tried is to test with 'undef' in the puppet rspec, but it's supported (giving undef as params) from puppet-rspec 2.3 and we use 2.2 ... so no luck here
15:29 karthiks chem,  ack.
15:30 ntpttr joined #puppet-openstack
15:30 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Allow use of device aliases for OSDs  https://review.openstack.org/451826
15:36 ntpttr joined #puppet-openstack
15:37 rnoriega oh man... any charitable puppet expert who could take a look to an issue I'm having??
15:37 rnoriega :-)
15:43 mwhahaha rnoriega: what ar eyou running into?
15:45 rnoriega mwhahaha, :-D
15:45 rnoriega mwhahaha, this stupid patch: https://review.openstack.org/#/c/451752/
15:46 rnoriega mwhahaha, sorry, stupid me...not the patch
15:46 mwhahaha rnoriega: sure gimme a sec, at first glance it s eems that the conditions for calling that exec are wrong in either the test or the manifest
15:47 mwhahaha rnoriega: since it's not finding it in the catalog
15:47 rnoriega mwhahaha, it fails saying that l2gw-db-sync is not in the catalog, and it is! my latest change was to set the sync_db to true
15:47 jaganathan joined #puppet-openstack
15:48 xb_ joined #puppet-openstack
15:48 mwhahaha rnoriega: it's the way the test is structured, it isn't
15:48 ntpttr joined #puppet-openstack
15:48 mwhahaha rnoriega: so updating the default_params doesn't actually set it to true
15:48 mwhahaha rnoriega: that block is only used to check the default params were set
15:49 mwhahaha rnoriega: i don't like the way this test is structured, so let me comment on it
15:49 rnoriega mwhahaha, oh man... ruby is black magic to me xD
15:49 rnoriega mwhahaha, sure! any comment is welcome!
15:49 mwhahaha rnoriega: it's not even ruby, it's rspec which is worse :D
15:49 EmilienM HEH
15:49 rnoriega mwhahaha, indeed... whatever!! xD
15:50 rnoriega mwhahaha, I feel like this: http://andrewgarrison.com/wp-content/uploads/2012/10/CodeMonkey-68762_960x3601.jpg
15:50 mwhahaha pretty much
15:50 mwhahaha that's rspec
15:51 rnoriega hahah
15:51 rnoriega that's just me... I'm sure you guys can handle it much better!
15:51 mwhahaha rnoriega: so in rspec just think of context -> your test case.
15:51 mwhahaha rnoriega: so in that test the 'default' test case has sync_db disabled because it's not overriding an y params
15:52 rnoriega mwhahaha, I thought that was the it 'xxx' do
15:52 mwhahaha rnoriega: there's another test case in there that overrides stuff, so you'd want to add your test into that
15:52 mwhahaha rnoriega: no it '...' is your check in the test case
15:52 rnoriega mwhahaha, ok! let me see what I can do! thanks man
15:54 ntpttr joined #puppet-openstack
15:59 skramaja joined #puppet-openstack
15:59 ducttape_ joined #puppet-openstack
16:00 openstackgerrit Ricardo Noriega proposed openstack/puppet-neutron master: Adding database sync for L2 Gateway service plugin  https://review.openstack.org/451752
16:01 rnoriega mwhahaha, so the difference between the first test case "with default params" and "with multople service providers" is that you merge more parameters in the second one??
16:01 rnoriega mwhahaha, that's why we add the sync_db in the second case, because it will be added to params?
16:01 mwhahaha rnoriega: the first case has nothing overriding params
16:02 mwhahaha rnoriega: that's what the before :each block is doing
16:02 mwhahaha rnoriega: since the default in the manifest is false for sync_db, you need to enable it to test it
16:02 mwhahaha rnoriega: thats why if you update the before :each to include the sync_db => true, then you can test that it actually would call the exec
16:02 mwhahaha rnoriega: like i said i don't like the structure of this test (it's very old) and not clear on what it's actually testing
16:03 rnoriega mwhahaha, I thought that this: default_params.merge(params) would actually get the values into the context
16:04 mwhahaha rnoriega: no, in the tests for puppet, let (:params) {{ }} is the only thing that actually changes the params
16:04 mwhahaha rnoriega: so if it's not in the params for the scope of the context, it didn't set it
16:04 rnoriega mwhahaha, and how would a newer test look like?? I could refactor them in the near-future
16:04 skramaja joined #puppet-openstack
16:04 mwhahaha rnoriega: so the before each is operating on the params for the scope to override params
16:05 rnoriega mwhahaha, alright! Master class! thanks
16:05 mwhahaha rnoriega: https://github.com/openstack/puppet-aodh/blob/master/spec/classes/aodh_evaluator_spec.rb#L32-L91
16:05 mwhahaha something more like that where we do a let (:params) {  ... } in the context
16:05 mwhahaha rnoriega: it's more clear as to what is going on there as opposed to using the before :each
16:06 rnoriega mwhahaha, ok! got it
16:08 ntpttr joined #puppet-openstack
16:14 ntpttr joined #puppet-openstack
16:20 ntpttr joined #puppet-openstack
16:23 nmathew joined #puppet-openstack
16:26 ntpttr joined #puppet-openstack
16:27 morazi joined #puppet-openstack
16:32 ntpttr joined #puppet-openstack
16:38 ntpttr joined #puppet-openstack
16:43 * mwhahaha appoligizes for the spam
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-aodh master: Drop -dev for release prep  https://review.openstack.org/453750
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-barbican master: Drop -dev for release prep  https://review.openstack.org/453751
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-ceilometer master: Drop -dev for release prep  https://review.openstack.org/453752
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-cinder master: Drop -dev for release prep  https://review.openstack.org/453753
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-congress master: Drop -dev for release prep  https://review.openstack.org/453754
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-designate master: Drop -dev for release prep  https://review.openstack.org/453755
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-ec2api master: Drop -dev for release prep  https://review.openstack.org/453756
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-glance master: Drop -dev for release prep  https://review.openstack.org/453757
16:44 EmilienM oh noes
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-gnocchi master: Drop -dev for release prep  https://review.openstack.org/453758
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-heat master: Drop -dev for release prep  https://review.openstack.org/453759
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-horizon master: Drop -dev for release prep  https://review.openstack.org/453760
16:44 openstackgerrit Alex Schultz proposed openstack/puppet-ironic master: Drop -dev for release prep  https://review.openstack.org/453761
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-keystone master: Drop -dev for release prep  https://review.openstack.org/453762
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-magnum master: Drop -dev for release prep  https://review.openstack.org/453763
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-manila master: Drop -dev for release prep  https://review.openstack.org/453764
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-mistral master: Drop -dev for release prep  https://review.openstack.org/453766
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-murano master: Drop -dev for release prep  https://review.openstack.org/453767
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-neutron master: Drop -dev for release prep  https://review.openstack.org/453768
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-nova master: Drop -dev for release prep  https://review.openstack.org/453769
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-octavia master: Drop -dev for release prep  https://review.openstack.org/453770
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_extras master: Drop -dev for release prep  https://review.openstack.org/453771
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-openstacklib master: Drop -dev for release prep  https://review.openstack.org/453772
16:45 openstackgerrit Alex Schultz proposed openstack/puppet-oslo master: Drop -dev for release prep  https://review.openstack.org/453773
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-ovn master: Drop -dev for release prep  https://review.openstack.org/453774
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-panko master: Drop -dev for release prep  https://review.openstack.org/453775
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-sahara master: Drop -dev for release prep  https://review.openstack.org/453776
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-swift master: Drop -dev for release prep  https://review.openstack.org/453777
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-tacker master: Drop -dev for release prep  https://review.openstack.org/453778
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-tempest master: Drop -dev for release prep  https://review.openstack.org/453779
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-trove master: Drop -dev for release prep  https://review.openstack.org/453780
16:46 ntpttr joined #puppet-openstack
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-vswitch master: Drop -dev for release prep  https://review.openstack.org/453781
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-watcher master: Drop -dev for release prep  https://review.openstack.org/453782
16:46 openstackgerrit Alex Schultz proposed openstack/puppet-zaqar master: Drop -dev for release prep  https://review.openstack.org/453783
16:51 chem hehe
16:52 ntpttr joined #puppet-openstack
16:58 ntpttr joined #puppet-openstack
16:58 rnoriega mwhahaha, that was it man! thanks :)
16:58 mwhahaha no prob
16:59 _ducttape_ joined #puppet-openstack
17:00 EmilienM it sounds like tempest tests are failing on telemetry on scenario001 / newton
17:00 EmilienM http://logs.openstack.org/31/453431/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/2aaf132/console.html#_2017-04-05_12_11_38_371988
17:01 mwhahaha http://logs.openstack.org/31/453431/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/2aaf132/console.html#_2017-04-05_12_11_38_372743
17:01 mwhahaha did someone backport something that used the new _error_checker?
17:03 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph master: Allow use of device aliases for OSDs  https://review.openstack.org/451826
17:03 EmilienM mwhahaha: no idea
17:04 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration stable/newton: CI test on newton  https://review.openstack.org/453791
17:04 ntpttr joined #puppet-openstack
17:08 openstackgerrit Merged openstack/puppet-ceilometer stable/ocata: Fix dispatcher params  https://review.openstack.org/453625
17:10 ntpttr joined #puppet-openstack
17:11 saneax joined #puppet-openstack
17:16 ntpttr joined #puppet-openstack
17:28 ntpttr joined #puppet-openstack
17:36 ntpttr joined #puppet-openstack
17:44 ntpttr joined #puppet-openstack
17:48 agrebennikov joined #puppet-openstack
17:50 ntpttr joined #puppet-openstack
18:00 ntpttr joined #puppet-openstack
18:01 chem_gone EmilienM: hi, I know it's a bit rude to ask directly, but I'll give it a try anyway.  If you could push this one in the obvlivion of the W+1 that would be super giga ultra cool : https://review.openstack.org/#/c/453197/
18:01 chem_gone EmilienM: whatever you decide, thanks :)
18:06 dprince joined #puppet-openstack
18:06 ntpttr joined #puppet-openstack
18:07 EmilienM chem_gone: hey
18:07 EmilienM chem_gone: nothing rude no worries I'll review it
18:07 EmilienM chem_gone: approved
18:08 ducttape_ joined #puppet-openstack
18:10 openstackgerrit Alex Schultz proposed openstack/puppet-ceph master: Require ceph::profile::client instead of ::base in mds.pp and rgw.pp  https://review.openstack.org/446227
18:12 ntpttr joined #puppet-openstack
18:18 ntpttr joined #puppet-openstack
18:20 jaganathan joined #puppet-openstack
18:24 ntpttr joined #puppet-openstack
18:30 ntpttr joined #puppet-openstack
18:38 ntpttr joined #puppet-openstack
18:40 ntpttr_ joined #puppet-openstack
18:47 hjensas joined #puppet-openstack
18:48 ntpttr joined #puppet-openstack
18:57 vnogin joined #puppet-openstack
18:58 ntpttr joined #puppet-openstack
19:04 ntpttr joined #puppet-openstack
19:10 ntpttr joined #puppet-openstack
19:12 ducttape_ joined #puppet-openstack
19:18 ntpttr joined #puppet-openstack
19:23 salmankhan joined #puppet-openstack
19:24 ntpttr joined #puppet-openstack
19:26 salmankhan joined #puppet-openstack
19:30 ntpttr joined #puppet-openstack
19:36 ntpttr joined #puppet-openstack
19:42 ntpttr joined #puppet-openstack
19:46 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration stable/newton: ignore test_check_glance_v1_notifications tests on Ubuntu  https://review.openstack.org/453843
19:46 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration stable/newton: ignore test_check_glance_v1_notifications tests on Ubuntu  https://review.openstack.org/453843
19:46 EmilienM mwhahaha: is it ok? ^ I never know how to do that
19:48 ntpttr joined #puppet-openstack
19:49 dgurtner joined #puppet-openstack
19:49 dgurtner joined #puppet-openstack
19:49 EmilienM oh I think I need to set an empty value on centos
19:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration stable/newton: ignore test_check_glance_v1_notifications tests on Ubuntu  https://review.openstack.org/453843
19:54 ntpttr joined #puppet-openstack
19:56 mwhahaha EmilienM: yea i think that's it
19:56 EmilienM wouhouuu
19:56 EmilienM mwhahaha: do we want to ignore and do my patch or investigate and fix it somewhere?
19:57 EmilienM tbh I don't think we have time for that now
19:57 mwhahaha other than filing a bug in upstream ubuntu for it lets just skip it
19:57 mwhahaha it seems  someone backported something and didn't test it
20:00 ntpttr joined #puppet-openstack
20:06 ntpttr joined #puppet-openstack
20:14 ntpttr joined #puppet-openstack
20:20 ntpttr joined #puppet-openstack
20:26 ntpttr joined #puppet-openstack
20:32 ntpttr joined #puppet-openstack
20:38 ntpttr joined #puppet-openstack
20:43 ducttape_ joined #puppet-openstack
20:46 ntpttr joined #puppet-openstack
20:52 ntpttr joined #puppet-openstack
20:57 EmilienM dmsimard, mwhahaha : https://review.openstack.org/453843 please
20:58 ntpttr joined #puppet-openstack
21:04 ntpttr joined #puppet-openstack
21:12 ntpttr joined #puppet-openstack
21:18 ntpttr joined #puppet-openstack
21:24 ntpttr joined #puppet-openstack
21:30 ntpttr joined #puppet-openstack
21:32 xarses joined #puppet-openstack
21:36 ntpttr joined #puppet-openstack
21:39 openstackgerrit Merged openstack/puppet-openstack-integration stable/newton: ignore test_check_glance_v1_notifications tests on Ubuntu  https://review.openstack.org/453843
21:42 ntpttr joined #puppet-openstack
21:48 ntpttr joined #puppet-openstack
21:54 ntpttr joined #puppet-openstack
22:00 ntpttr joined #puppet-openstack
22:06 ntpttr joined #puppet-openstack
22:08 ansmith joined #puppet-openstack
22:14 ducttape_ joined #puppet-openstack
22:14 ntpttr joined #puppet-openstack
22:20 ntpttr joined #puppet-openstack
22:28 ntpttr joined #puppet-openstack
22:32 rwsu joined #puppet-openstack
22:35 ntpttr joined #puppet-openstack
22:42 ntpttr joined #puppet-openstack
22:48 ntpttr joined #puppet-openstack
22:54 ntpttr joined #puppet-openstack
23:04 ntpttr joined #puppet-openstack
23:10 ntpttr joined #puppet-openstack
23:16 ntpttr joined #puppet-openstack
23:22 ntpttr joined #puppet-openstack
23:27 acormier joined #puppet-openstack
23:28 ducttape_ joined #puppet-openstack
23:30 ntpttr joined #puppet-openstack
23:36 ntpttr joined #puppet-openstack
23:39 acormier joined #puppet-openstack
23:40 acormier joined #puppet-openstack
23:40 ducttape_ joined #puppet-openstack
23:42 ntpttr joined #puppet-openstack
23:48 ntpttr joined #puppet-openstack
23:56 ntpttr joined #puppet-openstack

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