Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-08-09

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

All times shown according to UTC.

Time Nick Message
00:06 bodepd hogepodge: that module doesn't come from him
00:06 bodepd its from ekarlso
00:07 hogepodge Dan was the last person to touch it. I can contact ekarlso also. HIs own repository hasn't been touched in several months. Thanks for pointing that out.
00:08 hogepodge Regardless, we forked it so we could release the dependency.
00:09 hogepodge I'll probably have to do another quantum bug fix release again. We got the name wrong (puppet/quantum for the module name rather than puppetlabs/quantum. Minor, but still something to make consistent.)
00:13 openstackgerrit Connor Kaz proposed a change to stackforge/puppet-nova: Update Puppetfile,README,fixtures.yml for grizzly  https://review.openstack.org/40998
00:13 blkperl bodepd: first one ^
00:13 blkperl that was kinda painful :P
00:15 bodepd blkperl: why
00:15 bodepd ?
00:15 blkperl lets just say they learned a lot of new vim techniques :)
00:16 bodepd blkperl: by the way, you and your army of students workers are awesome :)
00:16 Hunner blkperl++
00:17 hogepodge The branch says master, but the topic says stable/grizzly.
00:30 blkperl wut
00:31 blkperl oh my bad... didn't know about that :)
00:31 hogepodge I ran into this, where I meant to target stable/grizzly, but instead targeted master. Dan pointed it out to me, and now I'm pointing the same symptom out to you.
00:31 hogepodge I didn't either. :-)
00:33 hogepodge The command 'git review stable/grizzly' is supposed to work, but now that you've submitted the review it may be problematic (that's what I experienced)
00:34 blkperl bodepd: so do you want Puppetfiles?
00:39 bodepd blkperl: I don't need them. I mainly care about the .fixtures.yaml file and want to ensure unti tests are passing
00:40 blkperl bodepd: alright
00:58 blkperl does openstackgerrit not publish for stable/grizzly?
01:10 mgagne I think you have to configure it to do so
01:10 mgagne I can look into it
01:15 mgagne https://review.openstack.org/41002
01:19 blkperl cool, ty :)
01:28 openstackgerrit A change was merged to stackforge/puppet-openstack: Proper selinux logic for Horizon  https://review.openstack.org/39931
01:31 openstackgerrit Connor Kaz proposed a change to stackforge/puppet-swift: Update fixtures.yml to use puppetlabs/xinetd  https://review.openstack.org/41003
01:57 openstackgerrit Connor Kaz proposed a change to stackforge/puppet-ceilometer: Update .fixtures.yml to use puppetlabs/inifile  https://review.openstack.org/41007
02:03 xingchao joined #puppet-openstack
02:07 openstackgerrit Maru Newby proposed a change to stackforge/puppet-tempest: Add provision config for 'service_available'.  https://review.openstack.org/41008
02:08 openstackgerrit Maru Newby proposed a change to stackforge/puppet-openstack: Add provision config for 'service_available'.  https://review.openstack.org/41009
02:35 otherwiseguy joined #puppet-openstack
02:52 xingchao_ joined #puppet-openstack
02:52 andi- joined #puppet-openstack
03:59 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Update .fixtures.yml to use puppetlabs/inifile  https://review.openstack.org/41007
04:34 otherwiseguy joined #puppet-openstack
05:07 otherwiseguy joined #puppet-openstack
05:27 marun joined #puppet-openstack
05:28 marun mgagne: ping
05:30 otherwiseguy joined #puppet-openstack
07:30 bodepd marun: can I help with anything?
07:52 marun bodepd: sorry to bug you at this late hour
07:52 marun i sent an email
07:53 marun I'll submit a new patch that does rest of the renam.e
07:56 fvollero Morning :)
08:01 openstackgerrit Maru Newby proposed a change to stackforge/puppet-openstack: Some more quantum -> neutron renaming.  https://review.openstack.org/41045
08:04 xingchao_ marun: is this patch about all quantum -> neutron renaming in puppet-openstack ?
08:04 marun xingchao_: yes, a few bits were missed by the major patch
08:06 xingchao marun: So if this patch is merged, then puppet-openstack could not work either for puppet-quantum or puppet-neutron
08:07 marun Is your concern for backwards compatibility?
08:08 marun (rather, I'm not sure I understand your question)
08:08 marun xingchao ^
08:08 marun I'll leave it to Terry to review.
08:08 marun He was the one submitted a renaming patch that missed these.  Maybe it was intentional.
08:09 xingchao marun: I think it's better to make all quantum -> neutron renaming in this patch, any thoughts?
08:09 marun xingchao: this is the remainder of it.
08:09 xingchao marun: Oh, I see
08:09 marun xingchao: most of the renaming was merged earlier
08:10 fvollero bodepd: if you've a slot can you give a review of https://review.openstack.org/#/c/40714/ ?
08:14 xingchao marun: I have find that patch, I miss it  :)
08:20 xingchao fvollero: good commit, and I leave some comments  :)
08:31 openstackgerrit Maru Newby proposed a change to stackforge/puppet-quantum: Fix router interface addition check.  https://review.openstack.org/40740
09:13 fvollero xingchao: Thanks a bunch for the review! I don't mind to do it in a separate change if is ok for you.
09:16 fvollero xingchao: btw, Ni hao :)
09:21 xingchao fvollero: ok, I would like to merge it.
09:21 xingchao fvollero: btw, Bonjour :)
09:25 openstackgerrit A change was merged to stackforge/puppet-heat: Added support for QPID in Heat  https://review.openstack.org/40714
09:32 fvollero xingchao: It's french but I understood and I appreciate it :)
09:33 fvollero xingchao: I'm slowly, learning chinese but it's complex and nice in the same way :) So crazy :)
09:36 xingchao fvollero: cool,  I'm expecting the day that we could talk in chinese :)
09:36 fvollero xingchao: hahahaah Hao ba
09:37 fvollero xingchao: the catch is my 'teacher' is from Taiwan
09:38 fvollero xingchao: and she is _lazy_ but is a friend and she's doing as exchange to learn italian :)
09:39 EmilienM fvollero: bonjour :)
09:39 * EmilienM is the french guy
09:39 fvollero EmilienM: Bonjour mon ami
09:39 xingchao fvollero: hmm, what does catch mean? Do you mean coach?
09:40 fvollero xingchao: no, catch as in tricky or problematic thing
09:41 xingchao fvollero: get it
09:42 fvollero xingchao: you are based in china? if so, which part?
09:43 xingchao fvollero: I live in Beijing now, I guess you know this place :)
09:43 fvollero xingchao: Yeah! For you the day is almost over :)
09:45 * fvollero have some colleague from Bejing as well
09:46 xingchao fvollero: yes, It's nealy 18:00 here
09:47 fvollero xingchao: \o/ and... weekend!
09:51 xingchao fvollero: I will keep on working on weekend, lots of things need to be done :)
09:54 fvollero xingchao: Gotcha... I hear you, but i'm going in PTO, I need to paint my house, what a vacation ah :)
09:56 * fvollero is thinking to do a quick trip to Paris as well...
09:56 * fvollero quick obviously does not mean cheap in Paris :)
10:16 xingchao fvollero:  Ah, better than my vacation. I will go to gym now, bye :)
10:23 * dachary following the https://github.com/stackforge/pupp​et-openstack/blob/master/README.md to install openstack on ubuntu 12.04
10:27 dachary it says : Puppet 2.7.12 or greater
10:28 dachary but 12.04 has 2.7.11-1ubuntu2.3
10:28 dachary hum
10:30 xingchao dachary: you can try this:
10:31 xingchao wget http://apt.puppetlabs.com/pu​ppetlabs-release-precise.deb
10:31 xingchao sudo dpkg -i puppetlabs-release-precise.deb
10:31 xingchao sudo apt-get update
10:31 xingchao apt-get install puppet-common=2.7.22-1puppetlabs1 puppet=2.7.22-1puppetlabs1
10:32 * dachary added
10:32 dachary deb http://ubuntu-cloud.archive.canonical.com/ubuntu precise-updates/grizzly main
10:32 dachary to the sources.list
10:32 dachary but no luck
10:33 dachary I'll take the risk and hope that 2.7.11 is enough
10:39 * dachary added deb http://apt.puppetlabs.com precise main
10:39 dachary to sources.list
10:39 dachary and installed
10:39 dachary apt-get install puppet=2.7.22-1puppetlabs1 puppet-common=2.7.22-1puppetlabs1 puppetmaster=2.7.22-1puppetlabs\
10:39 dachary 1 puppetmaster-common=2.7.22-1puppetlabs1
10:48 dmsimard joined #puppet-openstack
10:52 dmsimard joined #puppet-openstack
11:42 mxmln joined #puppet-openstack
11:56 dprince joined #puppet-openstack
12:32 dachary moving to git checkout stable/grizzly in https://github.com/stackforge/puppet-openstack/ to avoid trouble
12:41 EmilienM dachary: I had the same issue
12:41 EmilienM dachary: now, I use package from puppetlabs repo
12:42 EmilienM dachary: you should take care, since yesterday we have branches which respect openstack milestones
12:42 EmilienM master -> havana (neutron)
12:42 EmilienM stable/grizzly -> grizzly (quantum)
12:56 derekh joined #puppet-openstack
12:58 dmsimard joined #puppet-openstack
13:05 dprince joined #puppet-openstack
13:11 fvollero EmilienM: \o/
13:17 morazi joined #puppet-openstack
13:39 prad joined #puppet-openstack
13:46 mxmln has anyone used fuel for openstack?
13:47 mxmln I was just looking for openstack installation/orchestration...puppet,juju,fuel etc. collecting infos which way I go
13:48 dprince joined #puppet-openstack
14:03 dmsimard mxmln: you might have a better chance asking in #openstack
14:14 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41097
14:37 EmilienM fvollero: could you give me the name of Ceilometer central agent in Red Hat please ?
14:38 EmilienM oh I have it
14:38 EmilienM openstack-ceilometer-central
14:46 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
14:46 xingchao joined #puppet-openstack
15:00 fvollero EmilienM: :)
15:02 technolo-g joined #puppet-openstack
15:03 dachary joined #puppet-openstack
15:04 dachary EmilienM: this is all very confusing I must say :-) How would you recommend that I try to install grizzly ? Other than bumping into walls hoping to find the right path :-)
15:07 mjblack joined #puppet-openstack
15:09 fvollero EmilienM: I could not reply you because my internet was off, but... i did the review of your patch ;)
15:09 EmilienM fvollero: thx
15:09 EmilienM :)
15:09 EmilienM fvollero: I fail jenkins
15:09 EmilienM arf, I miss something
15:10 otherwiseguy joined #puppet-openstack
15:10 fvollero EmilienM: Yeah, but seems strange it worked to me
15:10 EmilienM fvollero: do you have any idea ?
15:10 EmilienM fvollero: logs are not explicit enough for me
15:11 fvollero EmilienM: Gonna fetch it and give back to you...
15:11 fvollero EmilienM: acutally seems you miss an end somewhere
15:11 fvollero http://logs.openstack.org/03/41103​/1/check/gate-puppet-ceilometer-pu​ppet-unit-2.7/da4381a/console.html
15:11 fvollero EmilienM: wait
15:11 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-openstack: Expose nova security_group_api parameter  https://review.openstack.org/41112
15:11 mjblack joined #puppet-openstack
15:12 EmilienM ceilometer_agent_central_spec.rb:82: syntax error, unexpected $end, expecting kEND
15:12 EmilienM my bad
15:13 EmilienM should be good now
15:13 fvollero :)
15:13 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
15:13 fvollero but still not understand why it worked to me
15:13 EmilienM \o/
15:13 fvollero whateva
15:18 dachary is there a way to report issues about https://github.com/stackforge/puppet-nova ?
15:18 fvollero dachary: there's a launchpad issues somewhere
15:19 fvollero EmilienM: the problem still there, seems
15:19 EmilienM dachary: https://bugs.launchpad.net/puppet-openstack
15:19 EmilienM and soon https://bugs.launchpad.net/puppet-nova
15:19 EmilienM mgagne: could I create bug trackers on ^
15:19 EmilienM seems not configured yet
15:22 dachary bodepd: hi, is there a tracker where you expect people to create issues ?
15:23 EmilienM dachary: I told you, it's https://bugs.launchpad.net/puppet-openstack
15:23 EmilienM :)
15:24 dachary oh, I clicked on the second link
15:24 dachary :-)
15:24 fvollero EmilienM: I found the bug.
15:24 dachary EmilienM: How would you recommend that I try to install grizzly ?
15:24 fvollero EmilienM: Your bug
15:25 EmilienM dachary: on Ubuntu ?
15:25 EmilienM fvollero: tell me :)
15:25 EmilienM dachary: Cloud Archives
15:25 EmilienM dachary: with stable/grizzly branches of puppet modules
15:25 dachary ok
15:25 dachary got all this
15:25 * dachary lucky
15:26 dachary what I'm unsure about is ... what to do next.
15:27 EmilienM dachary: ?
15:27 EmilienM write manifests
15:27 dachary that's the point
15:27 EmilienM and deploy OpenStack enjoying the life
15:27 fvollero EmilienM: I put a comment on the gerrit :)
15:28 dachary How can I figure out what manifests to write ?
15:28 EmilienM dachary: you have a doc here > https://github.com/stackforge/puppet-openstack
15:28 EmilienM but I'm not sure about it
15:28 EmilienM if it's enough
15:28 EmilienM fvollero: thx
15:28 dachary EmilienM: I'm quite sure it's not.
15:28 dachary because I tried it and failed in various ways.
15:29 dachary EmilienM: I'm looking for a simple  3 machine installation, nothing fancy.
15:29 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
15:32 fvollero EmilienM: My friend, run rake spec first :)
15:33 EmilienM fvollero: it also worked for you ? :P
15:33 otherwiseguy bah, more quantum references snuck into master in puppet-openstack: https://review.openstack.org/#/c/41045/
15:36 dachary EmilienM: in your expert opinion should I be able to deploy grizzly ( cinder / keystone / nova / quantum / glance ) with stable/grizzly ? Or should I expect trouble.
15:36 EmilienM dachary: you should be, of course
15:37 EmilienM dachary: even havana
15:38 otherwiseguy The fun of using submodules pointed to stackforge master for all dependencies in packstack and working on the bleeding edge.
15:38 * dachary studying https://github.com/stackfor​ge/puppet-openstack_dev_env
15:38 EmilienM otherwiseguy: I've updated all my submodules today \o/
15:38 EmilienM to use stable/grizzly
15:38 EmilienM fvollero: could you help me on http://logs.openstack.org/03/41103​/3/check/gate-puppet-ceilometer-pu​ppet-unit-2.7/f79a844/console.html ?
15:39 otherwiseguy EmilienM: As soon as those extra quantum references in puppet-openstack get approved, I can update the last one in packstack. Until then, no builds for me. :(
15:39 otherwiseguy EmilienM: thanks for the review, btw. :)
15:39 EmilienM my pleasure
15:39 EmilienM I've never used puppet-openstack btw
15:39 fvollero EmilienM: Does not work, you've 4 Failures...
15:39 * fvollero check now EmilienM link
15:39 EmilienM yes, but stil not understand
15:39 fvollero EmilienM: Yeah, give me a sec and i'll try to figure it out
15:40 EmilienM sure
15:40 fvollero otherwiseguy: I can't understand why i have to fight all the times with packstack :D
15:40 xingchao joined #puppet-openstack
15:40 otherwiseguy We use the new provisioning stuff to basically mirror how devstack sets up neutron networks, subnets, routers, etc. for doing a basic smoketest.
15:40 otherwiseguy fvollero: rapid state of flux. it drives me nuts.
15:40 EmilienM oh I know
15:40 EmilienM my bad
15:41 xingchao_ joined #puppet-openstack
15:41 EmilienM fvollero: ^
15:41 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
15:41 EmilienM fvollero: a typo in the manifest
15:41 EmilienM it's Friday
15:41 otherwiseguy fvollero: I don't know how many times I've said: THERE! I FIXED IT only to find out that something else has completely broken everything--or that it works on my machine, but nowhere else because I have a particular kernel version, or whatever.
15:42 EmilienM fvollero: http://goo.gl/GSaFHo
15:42 fvollero EmilienM: I was pointing you on that
15:43 fvollero otherwiseguy: i hear you :)
15:43 fvollero EmilienM: otherwiseguy : I am so focused to add heat and ceilo to packstack that I forgot about everything else
15:44 EmilienM \o/
15:44 EmilienM nova at least :P
15:47 fvollero EmilienM: imagine i could use again my ruby skills to detect your problem :)
15:47 EmilienM \o/
15:47 EmilienM fvollero: thx for review & helping :)
15:48 fvollero EmilienM: you're welcome
15:49 dachary are you using 'openstack::controller' or some lower level classes in your manifests ?
16:06 EmilienM dachary: no, I use nova::api, etc
16:09 otherwiseguy both https://review.openstack.org/#/c/40740/ and https://review.openstack.org/#/c/41045/ are really simple reviews, I don't suppose there are any new core reviewers available who would love to approve those. :p
16:10 * otherwiseguy tries to get his packstack build done and take the rest of the day off. :)
16:10 EmilienM ok
16:10 fvollero otherwiseguy: Oh man... don't tell me
16:10 dachary EmilienM: you would not be willing to share your manifests by any chance ?
16:10 EmilienM I think we should rename the channel to #packstack
16:10 EmilienM \o/
16:10 * dachary contemplating a long bumpy road to get manifests actually working
16:10 EmilienM dachary: I have not, but I can write with you
16:11 dachary EmilienM: how do you mean ?
16:11 EmilienM I don't have manifests for dev & testing for 3 nodes
16:11 technolo-g good morning :)
16:11 technolo-g I was wondering if anyone else was having an issue with puppet-librarian and the latest puppetlabs/openstack Puppetfile
16:11 technolo-g v2.1.0
16:15 dachary EmilienM: how long do you figure it would take you to write a manifest to deploy three nodes ?
16:15 dachary not that I'll ask you to do it ;-) Just to get an idea.
16:15 EmilienM dachary: a sec, I'm looking at one
16:15 dachary From scratch I mean.
16:16 EmilienM dachary: it could take 2 or 3 hours
16:16 EmilienM the time to debug
16:18 dachary If you help me with this I'll write the matching HOWTO for others to use. Are you game ?
16:18 hogepodge joined #puppet-openstack
16:18 fvollero joined #puppet-openstack
16:19 dachary EmilienM: ^
16:19 EmilienM dachary: why not, but I have to figure out the time :)
16:20 EmilienM should be funny
16:21 dachary and *extremely* useful. At this point in time, it is just impossible for anyone to install openstack with puppet without investing days of work.
16:22 dachary Either we do this together and it will take a few hours of our cumulated time.
16:22 dachary Or I do this by myself and it will take me a few days.
16:22 fvollero dachary: Well, there's an option :) Is called packstack :)
16:22 fvollero otherwiseguy: ^^^
16:22 fvollero dachary: or... using the foreman with puppet :)
16:25 Hunner technolo-g: What sort of issues?
16:25 dachary fvollero: I seriously considered switching to fedora for this reason. packstack still is tested only with fedora/redhat, right ?
16:26 fvollero dachary: centos as well
16:26 technolo-g Hunner: its giving me a  Malformed version number string 0.1.0-pl1 which appears to be the vswitch repo. I've tried both the one originally in the Puppetfile as well as puppetlabs/vswitch to no avail
16:26 dachary that's a big leap for me, because I'm mostly a .deb type of guy ;-)
16:26 Hunner Arg. Why does it think that is malformed? It fits semver 1.0.0 just fine...
16:27 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-cinder: Fixes some puppet lint warnings  https://review.openstack.org/41138
16:27 fvollero dachary: or...or... if you're brave enough, you can use The Foreman (http://theforeman.org/)
16:27 EmilienM dachary: let's do it next week
16:27 Hunner Oh, librarian-puppet expects x.y.z and isn't checking against semver at all
16:27 * EmilienM accepts the challenge :)
16:27 technolo-g i see that there is a pull request on the librarian-puppet project to try and fix these types of failures but neither "fixes" the problem
16:27 EmilienM it's just a matter of finding the time
16:27 dprince joined #puppet-openstack
16:27 dachary EmilienM: deal. An evening might be better to not be distracted by work :-)
16:28 Hunner technolo-g: I'll do a 0.1.1 release of vswitch I guess...
16:28 EmilienM dachary: hum. not sure I'm available next week on the evening, it's summer you know ;)
16:28 technolo-g Hunner: that would be awesome
16:28 EmilienM Hunner: +1
16:28 fvollero EmilienM: Uhm... Summer? In France?
16:29 EmilienM fvollero: \o/
16:29 dachary EmilienM: tuesday 9:00 am ?
16:29 mgagne EmilienM: thanks for configuring bug tracker and all. was on my TODO for today
16:30 mgagne EmilienM: we should enforce creation of bugs for better traceability IMO
16:30 openstackgerrit A change was merged to stackforge/puppet-quantum: Fix router interface addition check.  https://review.openstack.org/40740
16:30 EmilienM mgagne: sorry for breaking your backlog
16:31 EmilienM mgagne: and I +1 your proposition
16:31 mgagne EmilienM: np, isn't complicated anyway =)
16:31 EmilienM mgagne: one thing I'm not sure, it's the puppet-openstack group. I let you manage this part
16:31 EmilienM but we will see if it works
16:31 EmilienM maybe will we have some bugs at the beginning
16:31 mgagne EmilienM: I can see that people prefers openstack-puppet-modules, is it correct?
16:31 EmilienM mgagne: yeah
16:31 openstackgerrit A change was merged to stackforge/puppet-openstack: Some more quantum -> neutron renaming.  https://review.openstack.org/41045
16:32 EmilienM mgagne: I'll let you this part ;)
16:32 EmilienM since I'm in week-end in a few time
16:32 mgagne =)
16:32 EmilienM dachary: not sure I can
16:32 mgagne EmilienM: sprint review and retro today ;)
16:32 EmilienM I'll let you know
16:32 EmilienM ahah
16:32 EmilienM mgagne: tomorrow, sprint planning ?
16:32 mgagne EmilienM: we don't work on Saturday here ;)
16:33 EmilienM mgagne: TBH, we should make meetings on #openstack-meeting-alt once a week
16:33 dachary EmilienM: well, get back to me with a time frame. I'll make myself available, except monday.
16:33 EmilienM dachary: ack
16:33 mgagne EmilienM: yes, I really wish to establish a better frame to the project
16:34 EmilienM even if we don't have a "PTL", we should at least discuss
16:34 EmilienM sometimes it fix a lot of problems
16:34 mgagne EmilienM: yes
16:35 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-quantum: Fix some puppet lint warnings.  https://review.openstack.org/41139
16:36 Hunner technolo-g: Also, I'm really bugged that rodjek hasn't fixed librarian-puppet to use Semver. At least r10k does (a l-p clone with more features)
16:37 Hunner This comes up all the time...
16:38 technolo-g Yeah I can see that. I tried a few other puppetfiles just to see whats up and half failed with the same error on different modules
16:38 mgagne rodjek has been inactive for quite a while IMO...
16:40 Hunner Yeah. I think he got buzy
16:40 Hunner probably just time to write a PR for him
16:40 Hunner I mean, it's fixed in the maestrodev fork of librarian-puppet
16:41 Hunner technolo-g: I think librarian-puppet is actually more or less abandoned. I think r10k is the tool that's currently under development that does the same thing
16:42 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-nova: Fix some puppet lint warnings.  https://review.openstack.org/41140
16:45 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-glance: Fix some puppet lint warnings.  https://review.openstack.org/41142
16:47 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-keystone: Fix some puppet lint errors.  https://review.openstack.org/41143
16:48 technolo-g Thanks Hunner: lemme give that a whirl
16:48 Hunner technolo-g: Not forged yet... just a sec
16:49 * Hunner needs to get auto-forge-release on these repos
16:50 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-quantum: Fix some puppet lint warnings.  https://review.openstack.org/41139
16:50 mgagne Hunner: tell me about it
16:50 Hunner Oh, ashp++ uploaded it.
16:51 mgagne EmilienM: https://answers.launchpad.ne​t/launchpad/+question/233792
16:52 EmilienM mgagne: did you forgot to add subprojects ? or we will be able to do it ?
16:53 mgagne EmilienM: FAQ didn't say about subprojects. Hopefully we will be able to add them later on.
16:53 EmilienM mgagne: cool thx for pushing it :)
16:53 openstackgerrit A change was merged to stackforge/puppet-quantum: Fix some puppet lint warnings.  https://review.openstack.org/41139
16:53 mgagne EmilienM: trying to fire everywhere at the same time :O
16:54 mgagne can someone approve https://review.openstack.org/#/c/40786/ ?
16:54 EmilienM ahah
16:54 mgagne it's puppet-neutron btw: https://launchpad.net/puppet-neutron
16:55 EmilienM mgagne: I should amend my patch on infra, then
16:55 mgagne EmilienM: and read my comment ;)
16:55 hogepodge mgagne: eucatools still aren't available for rhel?
16:55 mgagne hogepodge: I don't know?
16:56 mgagne hogepodge: my change is targeted at removing the nested definition
16:56 hogepodge mgagne: in my experience, from some time ago, they weren't. Just asking out of curiosity.
16:56 openstackgerrit A change was merged to stackforge/puppet-nova: Use ensure_packages to install utilities  https://review.openstack.org/40786
16:56 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-keystone: Adds Support to Manage SSL  https://review.openstack.org/39205
16:56 EmilienM mgagne: oh so I need to delete the lines, except for quantum, right ?
16:56 mgagne EmilienM: yes
16:57 technolo-g Hunner: the Puppetfile pulls from the git repo so it is indeed working now
16:57 technolo-g i was just able to build the openstack module :)
16:57 openstackgerrit A change was merged to stackforge/puppet-cinder: Fixes some puppet lint warnings  https://review.openstack.org/41138
16:58 EmilienM mgagne: nice catch
16:58 EmilienM mgagne: new patchset : https://review.openstack.org/#/c/41128/2
16:58 openstackgerrit A change was merged to stackforge/puppet-keystone: Fix some puppet lint errors.  https://review.openstack.org/41143
16:58 openstackgerrit A change was merged to stackforge/puppet-glance: Fix some puppet lint warnings.  https://review.openstack.org/41142
16:59 openstackgerrit A change was merged to stackforge/puppet-nova: Fix some puppet lint warnings.  https://review.openstack.org/41140
16:59 mgagne EmilienM: I did the initial mapping so I'm know a bit about its reason =)
16:59 EmilienM \o/ indeed
17:01 Hunner technolo-g: Thanks! Let me know if you run into other snags... I haven't personally done a build off of the latest release. I gotta find where the tests are run
17:07 dprince joined #puppet-openstack
17:09 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-tempest: Add Modulefile  https://review.openstack.org/41145
17:11 marun mgagne, bodepd: Would be great if https://review.openstack.org/#/c/41008 and https://review.openstack.org/#/c/41009 could go in today if you have the time.  The changes are pretty trivial, but are required for puppet-tempest to be able to configure tempest's master branch.
17:12 mgagne marun: was checking atm
17:12 marun Once those are in, I'll start work on making sure puppet-* stable/grizzly can also provision and configure tempest.
17:13 mgagne marun: swift is false on purpose right?
17:14 mgagne marun: it defers from default value in tempest. is it because puppet-openstack does not support/provision swift yet?
17:17 openstackgerrit A change was merged to stackforge/puppet-swift: Update fixtures.yml to use puppetlabs/xinetd  https://review.openstack.org/41003
17:20 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Improve Neutron support in networking  https://review.openstack.org/41147
17:22 marun mgagne: Pretty much, yeah.
17:22 marun mgagne: I have no firm opinions on the defaults, though, and the fact that the values are parameters means anybody is free to override them.
17:22 mgagne EmilienM: you can run rspec tests locally with bundle install && bundle exec rake spec ;)
17:22 EmilienM sorry
17:23 mgagne marun: sure, unless we have a good reason (like in your case), we should aim to use OpenStack default values.
17:23 fvollero mgagne: rake spec will not always do the magic? You've to add bundle exec all the time?
17:24 fvollero EmilienM: You should run rake spec before doing git review :) maybe a git hook will help you on this :
17:24 fvollero :(
17:24 fvollero grrr :)
17:24 EmilienM ahah
17:24 mgagne fvollero: it should work if you installed puppet-rspec on your system
17:28 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Improve Neutron support in networking  https://review.openstack.org/41147
17:28 EmilienM mgagne: enjoy the week end guys :)
17:28 EmilienM fvollero: ^
17:29 EmilienM hum
17:29 EmilienM I did the tests and I have a failure :/
17:29 bodepd fvollero: bundler is used to install the gem deps for tests
17:29 bodepd fvollero: if you want to use that environment, then you need to run bundle exec
17:30 bodepd fvollero: so it depedns on how your environment is setup
17:30 bodepd EmilienM: thanks for all of your hard work this week
17:30 EmilienM bodepd: it's actually my job, but thank's, I appreciate the community
17:30 fvollero bodepd: well, i know what's bundler, but i use rbenv or rvm to test the gems :)
17:31 fvollero bodepd: sorry, the puppet stuff.
17:31 fvollero bodepd: you know, you can create and delete them :)
17:31 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Improve Neutron support in networking  https://review.openstack.org/41147
17:32 EmilienM bodepd: the test in nova is named quantum, I did not change it. Should I ?
17:32 EmilienM (master branchà
17:32 EmilienM (master branch)
17:34 EmilienM got my +1, I'm allowed to leave in WE :-). Cheers!
17:35 bodepd EmilienM: I'm still appreciative :) even better if you get paid for it
17:35 bodepd fvollero: I must have misunderstood your question then
17:36 bodepd EmilienM: for the master branch, every reference needs to be changed
17:36 bodepd EmilienM: AFAIK
17:55 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-openstack: Expose nova security_group_api parameter  https://review.openstack.org/41112
18:11 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Update NEUTRON_PLUGIN_CONFIG for cisco plugin  https://review.openstack.org/40491
18:36 otherwiseguy joined #puppet-openstack
18:53 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Install bridge-utils only if needed  https://review.openstack.org/40122
18:57 openstackgerrit A change was merged to stackforge/puppet-quantum: Update NEUTRON_PLUGIN_CONFIG for cisco plugin  https://review.openstack.org/40491
18:57 openstackgerrit A change was merged to stackforge/puppet-tempest: Add provision config for 'service_available'.  https://review.openstack.org/41008
19:01 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41097
19:07 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-nova: Remove nova_volume  https://review.openstack.org/41170
19:09 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-nova: Remove nova_volume  https://review.openstack.org/41170
19:18 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
19:18 mjblack resubmitted with the changes based on feedback
19:21 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
19:22 fvollero mgagne: merci, yeah, i'm using rbenv as i said above, so i run just once to install the needed packages... Years of ruby and rails development helped to be better on that
19:23 mgagne fvollero: unless we require specific version of gems, you should be safe installing puppet-rspec system wide.
19:24 fvollero mgagne: yes but i worked with fragile systems before, so I prefer, to use a specific gemset for every project i'm playing with. Mostly because people usually don't care about dependencies
19:25 fvollero mgagne: anyway i'm a newbie in puppet specific world, so i'm still adapting to the change :)
19:25 mgagne =)
19:26 fvollero :)
19:26 fvollero Just a question, we're sure that the puppet-lint is working? For some reason, is returning SUCCESS for every change
19:27 mgagne fvollero: what are you suspecting as not working?
19:27 mgagne fvollero: we fixed a lot of lint errors during the last days
19:28 mgagne fvollero: please note that lint won't fail build on warnings (yet)
19:31 fvollero mgagne: aha... I said it just because few days before it was all the time failing and now is going to return SUCCESS
19:31 fvollero mgagne: i replied to your comment on the None value
19:33 mgagne fvollero: my concern is that we never used the "None" string before, we use undef instead.
19:33 mgagne fvollero: my wish is to be consistent across manifests/modules
19:34 fvollero mgagne: yeah, but also we used string 'true' instead of booleans sometimes, so I completely understand you :)
19:34 mgagne fvollero: false, I changed them to puppet boolean recently ;)
19:34 fvollero mgagne: All the modules?
19:35 mgagne fvollero: if there are still some boolean values, I missed them or they are in new modules such as tempest or heat
19:36 fvollero mgagne: Ops, it may be possible also in ceilometer :)
19:36 fvollero mgagne: i'm already projected to my PTO... 2 weeks without having to deal with python (packstack) and ruby (everything else)
19:37 mgagne =)
19:37 dmsimard joined #puppet-openstack
19:38 fvollero yeah, it's waiting for me _paiting_ the house
19:38 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-nova: Remove nova_volume  https://review.openstack.org/41170
19:40 dmsimard1 joined #puppet-openstack
19:42 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
19:42 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-heat: Use Puppet boolean for better consistency  https://review.openstack.org/41178
19:43 fvollero mgagne: lol! I knew you was going to double check it :)
19:43 mgagne fvollero: my OCD just kicked in, thank you
19:43 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-openstack: Removes un-needed cinder parameter  https://review.openstack.org/41180
19:43 fvollero mgagne: OCD ?
19:44 mgagne obsessive compulsive disorder
19:45 fvollero mgagne: hahahahahahaah :D
19:46 fvollero mgagne: Sorry for that, anyway, maybe we can also add validate_boolean for those? :D
19:46 * fvollero run
19:46 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-openstack: Removes un-needed cinder parameter  https://review.openstack.org/41180
19:46 mgagne fvollero: if there wasn't 95 parameters per class, maybe. ;)
19:47 fvollero LOL :)
19:47 mgagne fvollero: I tried to add puppetdoc to all openstack classes, it was awful. 3-4 pages of doc before you can start to see the actual class content.
19:48 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Improve Neutron support in networking  https://review.openstack.org/41147
19:49 fvollero Yeah, I understand, that for sure it's a pain...This morning I was talking with a friend and I asked: I miss some params for qpid in my puppet module? he reply: No. 5 sec later he commented: 'hey qpid support HA'
19:49 fvollero o_O
19:49 mgagne =)
19:50 fvollero but, in my defense, no where else is used. So I have work for the week i come back from PTO :)
19:50 mgagne fvollero: it starts to bother me how classes are organized ATM where the init class has 78 parameters to support all RPC backends ^^'
19:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Use Puppet boolean for better consistency  https://review.openstack.org/41184
19:51 fvollero mgagne: yeah, it's kind of messy but anyway, i was thinking to put them in params and later maybe in the installation include the custom class, that would be a solution?
19:52 mgagne fvollero: until we figure out a better way, it's the way to do it. we had a discussion about it a couple of months ago.
19:53 fvollero ack
19:54 mgagne bodepd and I had some ideas but didn't have time to propose something yet
19:54 mjblack mgagne: in regards to your comment in https://review.openstack.org/41097, I didnt see it in time before I pushed my fixes. Should I just cancel it out and start a new review?
19:54 mgagne ?
19:55 mjblack sorry it was actually https://review.openstack.org/41173
19:55 mgagne mjblack: just amend your commit and re-execute git review. Make sure to not modify the Change-Id and it will update your review. You will have to restore it first if you can.
19:55 mjblack I tried to amend the git commit to do that but it complained that nothing was changed
19:56 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Improve Neutron support in networking  https://review.openstack.org/41147
19:56 mgagne mjblack: should not be the case :-/ git commit --amend ?
19:57 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
19:57 mjblack lol been a long day, forgot I abandoned the amend
20:01 EmilienM mgagne: both "undef" stuff > done
20:01 mgagne EmilienM: thanks
20:01 EmilienM thx for review ;)
20:01 bodepd mgagne: are we bashing puppet-openstack again?
20:02 mgagne bodepd: contemplating the situation =)
20:02 bodepd mgagne: I started drafting a draft of a blueprint
20:02 mgagne bodepd: have a draft of the draft? ;)
20:03 bodepd mgagne: https://github.com/CiscoSystems/ope​nstack-cosi-blueprints/blob/master/​blueprints/better_puppet_module.md
20:03 bodepd mgagne: feel free to submit a PR
20:03 bodepd mgagne: I'm trying to document the requirements
20:03 bodepd mgagne: I know what most of it looks like already in my head
20:04 mgagne bodepd: cool, just hard to find time for it ^^'
20:04 bodepd mgagne: yeah. my priority is CI first, then this once I can get tests running reasonably with what I have
20:04 bodepd mgagne: I hacked the shit out of that vagrant providers
20:04 mgagne nice
20:04 bodepd mgagne: patches are coming later
20:05 bodepd mgagne: we're evaluating vagrant vs. heat
20:05 mgagne bodepd: if you think it worked out of the box for us, you are wrong ;)
20:05 bodepd mgagne: I still have to make more changes. I kind of want every job to create its own tenant and networks
20:06 bodepd mgagne: I patched it so that I can assign private and public ips
20:06 mgagne bodepd: yes, we cannot run multiple integration tests, boxes are shared atm
20:06 bodepd mgagne: and I had to make more patches that I think are related to the fact I am using quantum
20:06 mgagne bodepd: ... with ovs
20:06 bodepd Hunner: we're talking about the vagrant openstack provider :)
20:07 bodepd mgagne: the way it was finding hte public ip to login with did not work for me
20:07 bodepd I think it's b/c you have a public netowrk and are not using floatin ips?
20:07 mgagne bodepd: that is correct
20:08 bodepd hmmm. I kind of like that better
20:08 bodepd I patched it to use floating ips
20:08 bodepd which is complicated
20:08 bodepd b/c they have to be pre-allocated to projects
20:09 bodepd fvollero: you were trying to get somethign merged before you left?
20:09 bodepd fvollero: pretty sure I dropped the ball when my hair set on fire
20:12 technolo-g bodepd: draft of draft looks great ;)
20:13 mjblack how do I force jenkins to re-run? looks like github was down
20:14 mgagne recheck no bug
20:14 openstackgerrit Robert Werfelmann proposed a change to stackforge/puppet-swift: Fixed some spelling errors  https://review.openstack.org/41191
20:14 mjblack do I put it in the commit message?
20:15 mgagne just add a comment
20:16 bodepd mjblack: for the cisco ci tests?
20:16 bodepd ah
20:16 bodepd recheck no bug
20:16 bodepd technolo-g: thanks. I keep trying to find time to work on it. I wrote a prototype of it that I was not happy with
20:17 bodepd technolo-g: I really like florians's project kickstart
20:17 bodepd technolo-g: but I don't like the way it handles data. I need to be able to pre-specify all the data for the deployment
20:17 mjblack bodepd: it was with the openstack-ci, jenkins flag the change as -1
20:17 blkperl bodepd: is it time to remove create_resources from fixture files? 2.6 has been EOL for a while now :)
20:17 technolo-g kickstack maybe?
20:17 bodepd technolo-g: yeah :_
20:17 bodepd :)
20:17 bodepd mjblack: I'm working that right now
20:18 bodepd blkperl: yeah.
20:18 bodepd blkperl: impressive you even know what that bit of cruft is there
20:18 technolo-g bodepd: have you tried kickstack on rhel?
20:18 bodepd mjblack: those tests are a bit fragile at the moment, and not working with havana
20:18 bodepd technolo-g: no, I just had a long discussion with florian about it
20:19 bodepd technolo-g: I want to make some pretty major changes to it
20:19 bodepd technolo-g: the thing I like about it is the level of granualarity of the components
20:19 bodepd technolo-g: and the fact all data lookup is external
20:19 bodepd technolo-g: but I want to refactor it to use hiera from class parameters
20:20 technolo-g that would be great
20:20 bodepd technolo-g: are you with RH?
20:20 bodepd technolo-g: I also want to look at the role granularity of packstack
20:20 technolo-g no, im with a company called beatport
20:20 bodepd ah, I guess I just assumed b/c you seemed to jump on the bandwagon at the same time of lots of redhat folks
20:21 bodepd mjblack: which job?
20:21 technolo-g nope, but we are a SL shop so we're definitely keeping an eye on progress on the rhel side
20:22 bodepd technolo-g: CERN uses the modules with SL, so it should 'just work' :)
20:22 mjblack bodepd: https://review.openstack.org/41173
20:22 technolo-g indeed they do. just got puppetlabs/openstack up with quantum this morning
20:22 technolo-g up == clean puppet runs :D
20:23 bodepd mjblack: you made the right comment. It should fire off again
20:25 blkperl bodepd: yeah we were using create resources when we using 2.6 :)
20:26 mjblack_ joined #puppet-openstack
20:26 openstackgerrit A change was merged to stackforge/puppet-swift: Fixed some spelling errors  https://review.openstack.org/41191
20:34 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
20:35 dmsimard joined #puppet-openstack
20:37 mgagne build will fail =)
20:39 openstackgerrit Robert Werfelmann proposed a change to stackforge/puppet-glance: Update fixtures.yml for grizzly  https://review.openstack.org/41196
20:43 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-nova: Remove nova_volume  https://review.openstack.org/41170
20:45 mgagne thanks for update
20:46 mgagne we should find a better/elegant way to handle deprecated parameters =)
20:46 bodepd deprecation warnings?
20:47 bodepd or explicit errors?
20:47 bodepd there will be backwards compat breaking changes between releases
20:47 bodepd b/c of the nature of openstack
20:48 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
20:51 blkperl sigh... github is down
20:52 technolo-g argh. i was blaming r10k lol
20:55 hogepodge joined #puppet-openstack
20:56 dmsimard joined #puppet-openstack
21:03 mjblack_ bodepd: you mentioned about cherry picking my changes for the grizzly branch, would that be a new gerrit change?
21:05 mgagne mjblack_: yes, it would result in a new gerrit change but targeted at stable/grizzly branch
21:05 mjblack_ ah ok, I'll do that now
21:09 bodepd mgagne: yes
21:09 bodepd mjblack_: https://wiki.openstack.org/wiki/​Puppet-openstack#stable_branches
21:09 bodepd mjblack_: feel free to update that if it does not provide enough info
21:10 mgagne bodepd: sorry, forgot context. well I think we need to figure out when and how can backward compatibility be broken.
21:10 bodepd backwards compat is completely broken between grizzly and havana
21:10 bodepd and it happened this week
21:10 bodepd I know what you mean though, b/c some people want master to be stable
21:11 bodepd I think the time for backwards compat changes is now!!!!
21:11 bodepd I still like the idea of having warnings
21:11 bodepd and such if we can
21:11 bodepd b/c backwards compat breakage sucks :(
21:12 mgagne bodepd: you would be the first guy to cry if someone at puppetlabs removed a parameter without warning in master ;)
21:12 mgagne bodepd: just trying to give a chance for people to upgrade their manifests
21:38 marun mgagne, bodepd: one more patch to approve and tempest can be properly configured! https://review.openstack.org/#/c/41009/
21:48 mjblack_ hmmm
21:48 mgagne yha
21:48 mjblack_ github is really down now
21:50 mgagne I guess that as soon as git.openstack.org is officially available, we should move fixture locations to git.o.o. =) (for our projects at least)
22:07 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
22:19 blkperl bodepd: is there a reason why the swift module needs to manage a bunch of directories (/var/{cache,lib}/swift, /etc/swift, /home/swift) that should be manged by the package?
22:44 hogepodge_ joined #puppet-openstack
22:51 bodepd mgagne: software is hard :)
22:51 mgagne bodepd: how? =)
22:51 bodepd blkperl: it's possible that it is for historical reasons and needs cleanup
22:51 mgagne blkperl: not sure why they are managed, could be for dependency reasons
22:52 bodepd blkperl: remember though that there are 4 sets of packages
22:52 bodepd blkperl: before you change, check Debian, Fedora, RHEL, and Ubuntu :)
22:52 bodepd blkperl: it could be that it is needed on one of those platforms
22:52 blkperl sadness
23:01 technolo-g have a good weekend guys :) thanks for the help thus far
23:02 mgagne thanks!
23:10 mjblack joined #puppet-openstack
23:17 mjblack_ joined #puppet-openstack
23:21 fvollero guys have great 2 weeks of work :)
23:22 mjblack_ you too
23:22 fvollero|PTO mjblack_: i'll try :)
23:37 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
23:47 hogepodge joined #puppet-openstack
23:54 openstackgerrit Donovan Finch proposed a change to stackforge/puppet-swift: Remove puppetlabs-create_resources dependency.  https://review.openstack.org/41220

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