Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-12-17

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

All times shown according to UTC.

Time Nick Message
00:14 richm1 crinkle: What if the puppet-openstacklib openstack.rb class automatically converted the csv output to a list of hashes?  Otherwise, I can see that every puppet module is going to have to implement parsing logic - if it is contained in openstack.rb, we can easily change from csv to json
00:17 crinkle richm1: the return output of a request is unpredictable to openstacklib
00:17 crinkle it could be nothing or it could have any number of parameters
00:26 richm1 authenticate_request already differentiates between a list and something else
00:26 richm1 a list is always going to output a row of column headers followed by 0 or more rows of values
00:27 richm1 I'll take a look at this
00:28 crinkle okay, it could maybe use the headers as keys
01:22 gildub joined #puppet-openstack
01:23 vinsh joined #puppet-openstack
01:51 britthouser joined #puppet-openstack
01:54 openstackgerrit Colleen Murphy proposed stackforge/puppet-openstacklib: Swap aviator for openstackclient  https://review.openstack.org/134843
01:55 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
01:55 crinkle richm1: ^
01:55 crinkle that was a good idea
01:57 richm1 I have my moments :-)
02:02 richm1 crinkle: how about this?  https://gist.github.com/richm/66b6e2e0388284dee7d3
02:03 richm1 doesn't downcase the hash keys though
02:04 crinkle oh i didn't know there was a CSV.parse >.<
02:04 crinkle lol
02:05 richm1 ruby has everything . . .
02:06 xingchao joined #puppet-openstack
02:08 crinkle richm1: hmm I think this needs to work on ruby 1.8.7 and I don't think CSV works like that on 1.8.7 http://ruby-doc.org/stdlib-1.8.7/lib​doc/csv/rdoc/CSV.html#method-c-parse
02:09 crinkle well idunno, do we still need things to work on 1.8.7?
02:11 nibalizer well so there is this http://developerblog.redhat​.com/2013/09/12/rhscl1-ga/
02:11 nibalizer but looks like you need a redhat license
02:11 xarses joined #puppet-openstack
02:11 nibalizer to get 1.9.3 on el 6
02:12 nibalizer here is the centos version http://wiki.centos.org/Additio​nalResources/Repositories/SCL[6~
02:13 crinkle well i think we need this to work on unmodified el6
02:13 * richm1 sees if there is some way to install ruby1.8.7 on fedora 20 . . .
02:14 crinkle I don't think our unit tests are testing on 1.8.7, we could have tons of things that don't work
02:19 richm1 csv in ruby 1.8.7 does _not_ support header: :P
02:19 richm1 headers:
02:24 richm1 csv in ruby 1.9.3 works fine with headers:
02:24 richm1 so - is 1.8.7 support required?
02:26 crinkle i think we still want these to work on rhel 6
02:26 crinkle but i'm not sure
02:28 richm1 ok
02:28 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
02:29 openstackgerrit Colleen Murphy proposed stackforge/puppet-openstacklib: Swap aviator for openstackclient  https://review.openstack.org/134843
02:29 crinkle so this ^ uses CSV but not headers, so it's a little more condensed but should still work on 1.8.7
02:31 crinkle oh I guess I should fix the tests too...
02:40 gildub joined #puppet-openstack
02:41 badiane_ka joined #puppet-openstack
02:42 richm1 this works on 1.8.7 - https://gist.github.com/richm/66b6e2e0388284dee7d3
02:45 richm1 crinkle: Why do you need the response.to_s.gsub(/"/,'') ?
02:48 crinkle richm1: the to_s is because I was having a strange issue where the command was returning an object that wasn't a string, and the gsub is because otherwise the strings come out with double quotes in them
02:48 richm1 CSV.parse should remove the double quotes
02:49 crinkle oh maybe that was from before I was using that
03:01 openstackgerrit Colleen Murphy proposed stackforge/puppet-openstacklib: Swap aviator for openstackclient  https://review.openstack.org/134843
03:11 mdorman_ joined #puppet-openstack
03:40 aimon__ joined #puppet-openstack
03:43 aimon joined #puppet-openstack
03:43 aimon_ joined #puppet-openstack
03:44 aimon___ joined #puppet-openstack
04:03 badiane_ka joined #puppet-openstack
04:04 badiane_ka joined #puppet-openstack
04:13 vinsh joined #puppet-openstack
04:42 djm68 joined #puppet-openstack
04:49 sanjayu joined #puppet-openstack
04:55 Mosibi_ joined #puppet-openstack
04:56 sanjayu joined #puppet-openstack
05:07 gildub joined #puppet-openstack
05:24 ikkeT joined #puppet-openstack
06:45 djm68 joined #puppet-openstack
07:16 pcaruana joined #puppet-openstack
07:39 berendt joined #puppet-openstack
07:43 tfz joined #puppet-openstack
08:11 xingchao joined #puppet-openstack
08:24 dgurtner joined #puppet-openstack
08:24 dgurtner joined #puppet-openstack
08:27 arnaud_orange joined #puppet-openstack
08:30 arnaud_orange joined #puppet-openstack
09:13 mmagr joined #puppet-openstack
09:16 derekh joined #puppet-openstack
09:29 stamak joined #puppet-openstack
09:37 gildub joined #puppet-openstack
09:46 arnaud_orange joined #puppet-openstack
10:31 gildub joined #puppet-openstack
11:52 openstackgerrit Javier Peña proposed stackforge/puppet-trove: Fix trove-db-sync  https://review.openstack.org/142424
12:46 pasquier-s joined #puppet-openstack
12:52 dprince joined #puppet-openstack
13:05 imcsk8 joined #puppet-openstack
13:09 fandi joined #puppet-openstack
13:15 pasquier-s joined #puppet-openstack
13:21 pasquier-s joined #puppet-openstack
13:27 EmilienM hello from the other side
13:28 EmilienM crinkle: what's up?
13:37 pasquier-s joined #puppet-openstack
13:42 rcallawa joined #puppet-openstack
13:43 pasquier-s joined #puppet-openstack
14:01 rcallawa_ joined #puppet-openstack
14:03 djm68 joined #puppet-openstack
14:09 pasquier-s joined #puppet-openstack
14:14 pasquier-s joined #puppet-openstack
14:14 pasquier-s joined #puppet-openstack
14:15 pasquier-s joined #puppet-openstack
14:17 francois1 joined #puppet-openstack
14:24 ericpeterson joined #puppet-openstack
14:26 richm1 joined #puppet-openstack
14:26 richm1 left #puppet-openstack
14:28 richm1 joined #puppet-openstack
14:43 pasquier-s joined #puppet-openstack
14:49 pasquier-s joined #puppet-openstack
14:51 pasquier-s joined #puppet-openstack
15:01 pasquier-s joined #puppet-openstack
15:15 pasquier-s joined #puppet-openstack
15:28 djm68 joined #puppet-openstack
15:29 vinsh_ joined #puppet-openstack
15:35 pasquier-s joined #puppet-openstack
15:40 pasquier-s joined #puppet-openstack
15:48 vinsh joined #puppet-openstack
15:49 ericpeterson joined #puppet-openstack
15:52 mdorman joined #puppet-openstack
16:00 rcallawa joined #puppet-openstack
16:01 djm68 joined #puppet-openstack
16:02 pasquier-s joined #puppet-openstack
16:07 prad joined #puppet-openstack
16:15 pasquier-s joined #puppet-openstack
16:16 ericpeterson joined #puppet-openstack
16:17 stamak joined #puppet-openstack
16:18 mschmitt joined #puppet-openstack
16:20 richm1 crinkle: Do you mind if I update the puppet-keystone review with my fixes?  I'm not sure if it is "cool"  to update someone else's review.
16:29 EmilienM richm1: you have to pay her one beer for one patchset AFIK :-P
16:33 richm1 lol
16:34 openstackgerrit Merged stackforge/puppet-tempest: Allow to not manage Tempest code in Puppet  https://review.openstack.org/138838
16:34 EmilienM sbadia: I did the backport FYI ^
16:37 sbadia EmilienM: thanks!
16:38 sbadia I just wait jenkins, but the zuul job queue reach 20k :o
16:39 pasquier-s joined #puppet-openstack
16:39 EmilienM sbadia: I use to +2 without jenkins +1 (it's not risky for check since it's not gate)
16:44 stamak joined #puppet-openstack
16:58 rcallawa_ joined #puppet-openstack
17:01 EmilienM so with social we are discussing about create an array composed of hashes
17:01 openstackgerrit Ryan Bak proposed stackforge/puppet-monasca: Changed yaml files for checks to build with concat  https://review.openstack.org/142506
17:01 EmilienM like: [{"address":"*:0a:00.*","physical_network":"phys​net1"},{"vendor_id":"1137","product_id":"0071"}]
17:02 social hmm I have idea
17:02 EmilienM hehe
17:02 social actually you can
17:02 social with an help from inline template
17:02 social <.>
17:02 social EmilienM: pci_passthrough_whitelist = []
17:04 social ufg but the inline template would be quite ugly
17:04 social can I get variable content from template file?
17:05 social I mean fill in variable with generated stuff from template other way than inline template ?
17:07 pasquier-s joined #puppet-openstack
17:09 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
17:10 crinkle richm1: I've been putting a lot of effort into this and kind of want to keep owning it, if that's okay
17:10 crinkle the keystone_tenant tests won't pass in jenkins until the openstacklib patch is merged anyway
17:10 crinkle we can work on converting the other providers though
17:12 richm1 crinkle: ok - I'm working on keystone_user
17:15 EmilienM crinkle: you miss one review on https://review.openstack.org/#/c/139942/
17:17 crinkle EmilienM: I abandoned that patch, since we're going to openstackclient
17:18 EmilienM I missed an episode? And ubuntu ?
17:18 EmilienM without JSON
17:18 EmilienM right
17:18 crinkle EmilienM: we're going to use the old version
17:18 crinkle with csv
17:18 EmilienM sure
17:18 EmilienM crinkle: what can I review right now?
17:18 crinkle EmilienM: https://review.openstack.org/#/c/134843/
17:18 EmilienM crinkle: ack
17:20 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
17:23 EmilienM crinkle: wow, nice one. I'll take time tonight to read & test it.
17:23 EmilienM crinkle: expect a review tomorrow morning for you maybe or even tonight
17:23 crinkle EmilienM: ty
17:25 richm1 EmilienM: +1
17:31 rcallawa joined #puppet-openstack
17:34 crinkle dprince: how are you feeling about rabbitmq?
17:42 rmoe joined #puppet-openstack
17:43 EmilienM crinkle: why do you hardcode /usr/bin/openstack?
17:43 EmilienM on my setup it's in /usr/local/bin/openstack
17:44 crinkle EmilienM: oops I'll change that
17:44 EmilienM crinkle: don't worry, I put it in the review so you don't forget it
17:45 dprince crinkle: its good. I got it sorted out. Thanks for asking
17:48 EmilienM crinkle: in the provider (https://review.openstack.org/#/c/134​843/10/lib/puppet/provider/openstack.rb,cm) - we miss a lot of parameters for credentials, related to keystone v3 API. (domain-id, etc)
17:48 EmilienM is it a big deal now?
17:50 rcallawa joined #puppet-openstack
17:52 EmilienM richm1: if I understand correctly, the big deal with CSV is when we will use https://review.openstack.org/#/c/134843 to make API call, the exploitation of output would be tricky, right?
17:52 mmagr joined #puppet-openstack
17:54 tchaypo yo nibalizer
17:54 tchaypo I’d like to review 141947 but I don’t understand from the commit message what it’s trying to do
17:54 richm1 EmilienM: not sure I understand
17:55 EmilienM richm1: what's the difficult thing with CSV comparing to JSON,
17:55 EmilienM ?
17:55 tchaypo nibalizer: could you add some detail or a link to more info?
17:55 richm1 EmilienM: From a programming standpoint, CSV is only slightly more complicated, and that's only because we have to support ruby 1.8.7 CSV which doesn't know about headers
17:56 EmilienM richm1: ok
17:56 richm1 The goal is that $service 'list' commands should return a list of hashes, and the parsing of CSV/JSON/whatever is hidden in openstacklib openstack.rb provider
17:57 richm1 CSV is fine since it is supported by ruby - the tabular/sql style output is not ok
17:57 tfz joined #puppet-openstack
18:12 crinkle EmilienM: I didn't think about authing with v3
18:13 crinkle do you need domain to auth with v3? the wiki doesn't seem to think so https://wiki.openstack.org/wiki/Open​StackClient/Commands#Global_Options
18:14 nibalizer tchaypo: i think that can be abandoned
18:14 nibalizer because dan has pullet it into his patch
18:15 tchaypo excellent :)
18:15 tchaypo thanks :)
18:17 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
18:17 crinkle EmilienM: could you try testing on a system with openstackclient installed natively? We want to make sure it works with version 0.4.0-ish
18:18 openstackgerrit Colleen Murphy proposed stackforge/puppet-openstacklib: Swap aviator for openstackclient  https://review.openstack.org/134843
18:37 ericpeterson joined #puppet-openstack
18:37 ericpeterson joined #puppet-openstack
18:42 rmoe joined #puppet-openstack
18:53 xarses joined #puppet-openstack
18:54 openstackgerrit Solly Ross proposed stackforge/puppet-trove: Configure auth via conf file, not paste file  https://review.openstack.org/142539
18:54 openstackgerrit Solly Ross proposed stackforge/puppet-trove: Set control_exchange in the main config file  https://review.openstack.org/142540
18:54 openstackgerrit Solly Ross proposed stackforge/puppet-trove: Support Neutron  https://review.openstack.org/142541
19:01 rcallawa_ joined #puppet-openstack
19:04 stamak joined #puppet-openstack
19:25 bitblt joined #puppet-openstack
19:32 openstackgerrit Ryan Bak proposed stackforge/puppet-monasca: Changed yaml files for checks to build with concat  https://review.openstack.org/142506
19:42 rcallawa joined #puppet-openstack
19:53 ericpeterson joined #puppet-openstack
19:58 djm68 joined #puppet-openstack
20:27 openstackgerrit Merged stackforge/puppet-trove: Fix trove-db-sync  https://review.openstack.org/142424
20:31 EmilienM crinkle: ack
20:54 richm1 crinkle: some column names (keys) have spaces in them, which makes it problematic to convert them to symbols - in self.authenticate_request
20:54 richm1 Service Name and Service Type et. al.
20:55 richm1 key = 'Service Name'
20:55 richm1 key.downcase.to_sym
20:55 richm1 => :"service name"
20:56 richm1 so we'll have to use endpoint[:"service name"] to access
20:59 crinkle richm1: huh, what if i changed it to .gsub(/ /, '').to_sym or .gsub(/ /, '_').to_sym ?
21:03 richm1 crinkle: I like the latter
21:03 richm1 we'll just have to make sure provider developers know to sub '_' for ' '
21:11 pasquier-s joined #puppet-openstack
21:15 ericpeterson joined #puppet-openstack
21:25 ericpeterson joined #puppet-openstack
21:52 openstackgerrit Colleen Murphy proposed stackforge/puppet-openstacklib: Swap aviator for openstackclient  https://review.openstack.org/134843
22:01 pasquier-s joined #puppet-openstack
22:08 openstackgerrit Ryan Hallisey proposed stackforge/puppet-nova: Add Ironic support into nova puppet modules  https://review.openstack.org/132704
22:12 ericpeterson joined #puppet-openstack
22:18 pasquier-s joined #puppet-openstack
22:32 bitblt joined #puppet-openstack
22:33 mfisch richm1: crinkle python-openstackclient (1.0.1-0ubuntu1) vivid; urgency=medium
22:33 mfisch * New upstream release (LP: #1393873).
22:33 mfisch -- James Page <james.page@ubuntu.com>   Wed, 17 Dec 2014 21:57:18 +0000
22:33 mfisch ** Changed in: python-openstackclient (Ubuntu)
22:33 mfisch Status: Confirmed => Fix Released
22:33 mfisch moving the right direction
22:33 crinkle mfisch: yay!
22:37 richm1 mfisch: thanks
22:40 mfisch you can thanks med for some of that but he's not in this channel
22:40 openstackgerrit Ryan Hallisey proposed stackforge/puppet-ironic: Fix Ironic modules so services properly run in Juno  https://review.openstack.org/134048
22:43 rcallawa_ joined #puppet-openstack
22:45 openstackgerrit Ryan Hallisey proposed stackforge/puppet-glance: Add some config into Glance to support Ironic  https://review.openstack.org/141911
23:17 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
23:17 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Migrate keystone_service to use openstackclient  https://review.openstack.org/142608
23:32 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Migrate keystone_service to use openstackclient  https://review.openstack.org/142608
23:33 crinkle I've restored https://review.openstack.org/#/c/136186/ , I'd like to get it approved so we can link to the blueprint in the commits
23:35 pasquier-s joined #puppet-openstack
23:40 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: Use openstackclient for keystone_endpoint  https://review.openstack.org/142614
23:40 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: Use openstackclient for keystone_user  https://review.openstack.org/142615
23:47 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
23:47 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Migrate keystone_service to use openstackclient  https://review.openstack.org/142608

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