Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-05-22

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

All times shown according to UTC.

Time Nick Message
00:00 gildub richm, this is a special case, token issue
00:00 richm gildub: not a special case - keystone_request
00:00 richm same with glance_request, cinder_request, nova_request, etc. etc.
00:00 gildub richm, this is why it needs to keystone_request
00:01 richm It's not that I am objecting to the specific case of password - I meant that as an example - I am objecting in general using ENV as a generic parameter passing mechanism between ruby functions
00:01 gildub richm, keystone_request was there before, there is not structural change from that perspective
00:02 richm gildub: but now the _implementation_ of keystone_request has changed - it now uses ENV to pass parameters to request
00:02 gildub richm, we're not using ENV, we're setting variables to be run using withenv, not the same
00:03 richm what does withenv do?
00:03 richm I guess the implementation of withenv could be changed to do something other than set ENV
00:04 gildub richm, we need to have: 1.save env, 2.fush env. 3.use temp env. 4. run op 5. restore env - Atomically
00:04 britthouser joined #puppet-openstack
00:04 gildub ^2.flush env
00:04 richm yes, absolutely, we need to have that in Puppet::Provider::Openstack.request
00:05 gildub richm, is not withenv doing that?....checking
00:05 richm no other way to pass parameters safely to the openstack client process, without polluting the puppet process
00:05 richm gildub: I'm saying that the only place withenv should be used is in Puppet::Provider::Openstack.request
00:06 richm or Puppet::Provider::Openstack::Auth.request
00:06 richm either way, it effectively hides the implementation from outside of Puppet::Provider::Openstack
00:06 gildub richm, ok
00:07 richm but if you and everyone else thinks its ok, then I won't object
00:08 richm that is, if everyone else thinks its ok to use withenv to pass parameters to Puppet::Provider::Openstack.request from other puppet/ruby code, I won't object
00:08 openstackgerrit Sebastien Badia proposed stackforge/puppet-modulesync-configs: Synchronize nodeset (split trusty and centos)  https://review.openstack.org/184352
00:10 gildub richm, got your point
00:12 gildub richm, the issue here is the way that has evolved, Puppet:::Provider::Openstack is the masterclass doing the OSC interface the 'real' request command, but not taking care of the credentials part, that was and is still done by subclass (now module interface and subclass itself). But I think that makes perfect sense to have the withen in the master class. Just a bit of change.
00:12 gildub richm, so basically, yes, I agree with you!
00:14 richm gildub: ok
00:15 gildub richm, I'm on it!!!
00:16 richm gildub: excellent
00:16 richm gildub: I will be signing off soon
00:17 gildub richm, so finally we're having the credentials back as parameters but not type parameters!
00:17 gildub richm, enjoy your time off
00:17 richm yes
00:18 openstackgerrit Sebastien Badia proposed stackforge/puppet-modulesync-configs: hook: Add the Gerrit commit message hook for msync  https://review.openstack.org/184921
00:18 richm gildub: Things will progress much more quickly next week with everyone back from the summit
00:19 gildub richm, yeah, but those reviews are going to miss you anyway :)
00:21 gildub richm, but don't worry about it, you're off!
00:27 ducttape_ joined #puppet-openstack
00:34 ferest joined #puppet-openstack
00:35 ferest joined #puppet-openstack
01:17 gildub joined #puppet-openstack
01:18 arnaud_orange joined #puppet-openstack
03:21 panda joined #puppet-openstack
04:23 sks joined #puppet-openstack
04:35 xavpaice joined #puppet-openstack
04:52 alex_bh joined #puppet-openstack
04:54 saneax joined #puppet-openstack
05:23 tfz joined #puppet-openstack
05:27 openstackgerrit Emilien Macchi proposed stackforge/puppet-monasca: First review  https://review.openstack.org/184942
05:42 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Refactorize blueprints repository  https://review.openstack.org/184943
05:50 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document core-reviewers  https://review.openstack.org/184945
05:55 gfidente joined #puppet-openstack
06:14 dgurtner joined #puppet-openstack
06:21 aimon joined #puppet-openstack
07:02 paramite joined #puppet-openstack
07:21 openstackgerrit Ivan Chavero proposed stackforge/puppet-neutron: Add support for keystone v3 API  https://review.openstack.org/184856
07:21 panda joined #puppet-openstack
07:28 jistr joined #puppet-openstack
07:50 jpena joined #puppet-openstack
08:05 openstackgerrit Martin Mágr proposed stackforge/puppet-openstacklib: Bring Redhat support to acceptance tests  https://review.openstack.org/184969
08:39 jtomasek joined #puppet-openstack
08:51 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
09:01 IBerezovskiy joined #puppet-openstack
09:10 openstackgerrit Martin Mágr proposed stackforge/puppet-openstacklib: Bring Redhat support to acceptance tests  https://review.openstack.org/184969
09:42 openstackgerrit Ivan Berezovskiy proposed stackforge/puppet-ceilometer: Fix dependency on nova-common package  https://review.openstack.org/184979
09:53 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
11:49 openstackgerrit Clayton O'Neill proposed stackforge/puppet-swift: Fix concat file mode  https://review.openstack.org/184994
11:53 openstackgerrit Clayton O'Neill proposed stackforge/puppet-swift: Fix memcache comparison  https://review.openstack.org/184995
12:00 openstackgerrit Clayton O'Neill proposed stackforge/puppet-swift: Fix concat file mode  https://review.openstack.org/184994
12:00 openstackgerrit Clayton O'Neill proposed stackforge/puppet-swift: Ignore lint quoted boolean warning  https://review.openstack.org/184997
12:01 openstackgerrit Clayton O'Neill proposed stackforge/puppet-swift: Fix memcache comparison  https://review.openstack.org/184995
12:11 openstackgerrit Merged stackforge/puppet-vswitch: make dkms on Debian/Ubuntu optional, add tests  https://review.openstack.org/151629
13:01 iurygregory joined #puppet-openstack
13:28 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
13:40 ducttape_ joined #puppet-openstack
14:29 kriskrishna__ joined #puppet-openstack
14:43 ducttape_ joined #puppet-openstack
14:57 openstackgerrit Martin Mágr proposed stackforge/puppet-openstacklib: Bring Redhat support to acceptance tests  https://review.openstack.org/184969
14:58 openstackgerrit Martin Mágr proposed stackforge/puppet-openstacklib: Bring Redhat support to acceptance tests  https://review.openstack.org/184969
15:02 paramite joined #puppet-openstack
15:22 panda joined #puppet-openstack
15:29 derekh joined #puppet-openstack
15:36 Fin1te joined #puppet-openstack
16:03 ducttape_ joined #puppet-openstack
16:08 dprince joined #puppet-openstack
16:12 dalgaaf joined #puppet-openstack
16:15 mdorman joined #puppet-openstack
16:17 cwolferh joined #puppet-openstack
16:23 openstackgerrit Merged stackforge/puppet-tempest: Bring Redhat support to acceptance tests  https://review.openstack.org/184242
16:44 simon-AS559 joined #puppet-openstack
16:52 openstackgerrit Emilien Macchi proposed stackforge/puppet-swift: mount.pp: fix lint issue  https://review.openstack.org/185069
16:52 openstackgerrit Emilien Macchi proposed stackforge/puppet-swift: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184016
16:58 openstackgerrit Emilien Macchi proposed stackforge/puppet-sahara: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184054
17:00 britthouser joined #puppet-openstack
17:00 openstackgerrit Emilien Macchi proposed stackforge/puppet-sahara: Bring Redhat support to acceptance tests  https://review.openstack.org/184055
17:01 openstackgerrit Emilien Macchi proposed stackforge/puppet-sahara: Fixed /etc/sahara handling  https://review.openstack.org/184261
17:02 britthou_ joined #puppet-openstack
17:13 openstackgerrit Emilien Macchi proposed stackforge/puppet-trove: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184246
17:15 openstackgerrit Emilien Macchi proposed stackforge/puppet-trove: Bring Redhat support to acceptance tests  https://review.openstack.org/184247
17:16 openstackgerrit Emilien Macchi proposed stackforge/puppet-trove: CentOS 7 compatibility  https://review.openstack.org/184464
17:24 openstackgerrit Merged stackforge/puppet-openstacklib: Bring Redhat support to acceptance tests  https://review.openstack.org/184969
17:26 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
17:27 openstackgerrit Merged stackforge/puppet-sahara: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184054
17:30 alex_bh joined #puppet-openstack
17:42 openstackgerrit Sebastien Badia proposed stackforge/puppet-ironic: db: Add postgresql support  https://review.openstack.org/185091
17:42 openstackgerrit Sebastien Badia proposed stackforge/puppet-ironic: auth: Fix Ironic service description  https://review.openstack.org/185092
17:56 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document core-reviewers  https://review.openstack.org/184945
17:57 Fin1te joined #puppet-openstack
18:02 ducttape_ joined #puppet-openstack
18:23 openstackgerrit Merged stackforge/puppet-trove: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184246
18:23 simon-AS559 joined #puppet-openstack
18:24 openstackgerrit Merged stackforge/puppet-trove: Bring Redhat support to acceptance tests  https://review.openstack.org/184247
18:25 ducttape_ joined #puppet-openstack
18:27 openstackgerrit Merged stackforge/puppet-sahara: Bring Redhat support to acceptance tests  https://review.openstack.org/184055
18:29 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Refactorize blueprints repository  https://review.openstack.org/184943
18:30 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document core-reviewers  https://review.openstack.org/184945
18:33 openstackgerrit Merged stackforge/puppet-sahara: Fixed /etc/sahara handling  https://review.openstack.org/184261
18:33 openstackgerrit Merged stackforge/puppet-trove: CentOS 7 compatibility  https://review.openstack.org/184464
18:34 mattymo EmilienM, ping
18:34 mattymo or bodepd crinkle mfisch ?
18:34 openstackgerrit Merged stackforge/puppet-ceilometer: Fix dependency on nova-common package  https://review.openstack.org/184979
18:35 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Refactorize blueprints repository  https://review.openstack.org/184943
18:38 Fin1te joined #puppet-openstack
18:38 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document core-reviewers  https://review.openstack.org/184945
18:41 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Refactorize blueprints repository  https://review.openstack.org/184943
18:46 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document core-reviewers  https://review.openstack.org/184945
18:47 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix concat file mode  https://review.openstack.org/184994
18:47 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: proxy/cache: Fix logic for swift::proxy::cache  https://review.openstack.org/184667
18:47 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
18:47 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Bring Redhat support to acceptance tests  https://review.openstack.org/182621
18:47 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: mount.pp: fix lint issue  https://review.openstack.org/185069
18:47 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184016
18:49 derekh joined #puppet-openstack
18:51 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: proxy/cache: Fix logic for swift::proxy::cache  https://review.openstack.org/184667
18:51 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
18:51 abramley joined #puppet-openstack
18:52 nigelk joined #puppet-openstack
18:57 mfisch mattymo: whats up
19:22 panda joined #puppet-openstack
19:38 simon-AS5591 joined #puppet-openstack
19:50 derekh joined #puppet-openstack
20:01 simon-AS559 joined #puppet-openstack
20:40 simon-AS559 joined #puppet-openstack
20:57 panda EmilienM: where should I put the spec ? on spec/kilo ?
20:57 EmilienM panda: yes Sir!
20:57 panda EmilienM: ok, waiting for merge, then I'll rebase and move.
20:58 EmilienM panda: yeah, it's not a problem
21:08 jtomasek joined #puppet-openstack
21:12 stevemar joined #puppet-openstack
21:12 cwolferh joined #puppet-openstack
21:42 alex_bh joined #puppet-openstack
21:51 openstackgerrit Colleen Murphy proposed stackforge/puppet-nova: Unpin beaker-rspec (do not merge)  https://review.openstack.org/185161
21:53 EmilienM crinkle: do you think we can send https://review.openstack.org/#/c/184943/ ?
21:54 EmilienM crinkle: and thanks for looking at the issue with newest version of beaker
21:54 crinkle EmilienM: i just cringe a little that ceilometer is the example
21:55 EmilienM crinkle: why?
21:56 crinkle EmilienM: could you use nova or keystone as the example? also if there's not a general standard across projects then i don't entirely understand the need for the change
21:56 EmilienM I can drop this line if you like
21:56 EmilienM crinkle: well, I just need sphinx structure here
21:56 EmilienM to have http://specs.openstack.org/
21:57 EmilienM crinkle: what is the diff between ceilometer & nova ? They are 2 official projects
21:58 crinkle EmilienM: i'm just saying "consistency with ceilometer" doesn't make any sense, "consistency with the other main openstack projects" makes more sense and nova and keystone are more central projects
21:58 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Refactorize blueprints repository  https://review.openstack.org/184943
21:58 EmilienM crinkle, sbadia ^
21:59 crinkle EmilienM: ty
21:59 EmilienM crinkle: cool!
22:03 sbadia haha ok :)
22:03 openstackgerrit Merged stackforge/puppet-openstack-specs: Refactorize blueprints repository  https://review.openstack.org/184943
22:04 sbadia ceilometer is a forbidden word :)
22:04 crinkle :P
22:05 EmilienM lol
22:05 EmilienM friday is troll day
22:06 EmilienM hopefully we have a good steak tonight
22:06 crinkle mmmsteak
22:11 nibalizer you should invite mmmpork
22:17 bkero mmmporkchops?
22:41 ddmitriev1 joined #puppet-openstack
22:45 saneax joined #puppet-openstack
23:25 EmilienM crinkle: I think I've missed something for http://specs.openstack.org/stackforge/puppet-openstack-specs - I'll look after my holidays if no change
23:30 EmilienM Hunner: https://goo.gl/sNGQR1
23:31 Hunner spredzy: https://www.youtube.com/watch?v=dQw4w9WgXcQ
23:55 logan2 joined #puppet-openstack
23:56 panda EmilienM: quick question before I faint on a bed, why are beaker test installing puppet from puppetlabs instead of using the default repo package, like what is done with git ?
23:57 EmilienM panda: no clue now
23:57 EmilienM Hunner: ^ ?

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