| 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/puppet-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/puppetlabs-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-puppet-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/stackforge/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-puppet-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.net/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/openstack-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 |
