Wednesday, 2015-04-22

*** otter768 has quit IRC00:00
*** emagana has quit IRC00:36
*** zhiwei has joined #openstack-chef00:39
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341500:39
*** neelashah has joined #openstack-chef00:42
*** kaseyalusi has joined #openstack-chef01:01
*** seizadi has quit IRC01:06
*** jmickle has quit IRC01:11
openstackgerritMa Wen Cheng proposed stackforge/cookbook-openstack-network: Update the deprecated group/name to Kilo release  https://review.openstack.org/17281001:20
*** ncerny has joined #openstack-chef01:21
*** ncerny_ has joined #openstack-chef01:23
*** wenchma has joined #openstack-chef01:25
*** ncerny has quit IRC01:26
*** _code_ has quit IRC01:28
*** kaseyalusi has quit IRC01:31
openstackgerritZhiwei Chen proposed stackforge/cookbook-openstack-common: Use partial templates to create configuration files  https://review.openstack.org/17615601:45
*** ncerny_ has quit IRC01:47
*** ncerny has joined #openstack-chef01:55
*** otter768 has joined #openstack-chef01:56
*** stevemar has joined #openstack-chef01:56
*** otter768 has quit IRC02:01
*** harlowja is now known as harlowja_away02:03
openstackgerritMa Wen Cheng proposed stackforge/cookbook-openstack-network: Make the metadata_workers attribute in metadata_agent.ini configurable.  https://review.openstack.org/17541202:04
*** kaseyalusi has joined #openstack-chef02:27
*** kaseyalusi has quit IRC02:32
*** neelashah has quit IRC02:36
openstackgerritMa Wen Cheng proposed stackforge/cookbook-openstack-network: neutron nova_admin_auth_url requires a protocol version  https://review.openstack.org/17586102:53
a13x2121running into the following berks warning: In your Berksfile, you have: cookbook 'openstack-client' But that cookbook is actually named 'openstack_client'03:06
*** kaseyalusi has joined #openstack-chef03:07
a13x2121https://github.com/stackforge/cookbook-openstack-client/blob/master/metadata.rb#L203:09
wenchmayeah03:23
wenchma@a13x2121  doesn't it make sense ?03:23
sc`that does break my pattern, damn. and i had it working. i'd posit that the metadata needs to be fixed to match the repo name :)03:38
*** kaseyalusi has quit IRC03:38
*** otter768 has joined #openstack-chef03:57
*** ncerny has quit IRC03:59
*** otter768 has quit IRC04:02
*** carlp has quit IRC04:06
*** carlp has joined #openstack-chef04:08
*** ChanServ sets mode: +o carlp04:08
*** pradipta has joined #openstack-chef04:31
*** nacer has quit IRC04:39
*** seizadi has joined #openstack-chef04:42
*** nacer has joined #openstack-chef04:44
*** seizadi has quit IRC05:15
*** nkrinner has joined #openstack-chef05:20
*** emagana has joined #openstack-chef05:20
openstackgerritEthan Lynn proposed stackforge/cookbook-openstack-orchestration: Set default value for log_dir in heat.conf  https://review.openstack.org/17617305:45
*** otter768 has joined #openstack-chef05:58
*** kaseyalusi has joined #openstack-chef05:58
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341506:01
*** otter768 has quit IRC06:03
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341506:03
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341506:08
openstackgerritEthan Lynn proposed stackforge/cookbook-openstack-orchestration: Set default value for log_dir in heat.conf  https://review.openstack.org/17617306:18
*** stevemar has quit IRC06:31
*** kaseyalusi has quit IRC06:34
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341506:37
*** wojdev has joined #openstack-chef06:51
*** emagana has quit IRC06:53
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341506:57
*** mrodden has quit IRC07:06
*** mrodden has joined #openstack-chef07:07
*** os-chef-bot has quit IRC07:10
*** os-chef-bot has joined #openstack-chef07:11
*** wojdev_ has joined #openstack-chef07:17
*** wojdev has quit IRC07:19
*** wojdev_ is now known as wojdev07:19
*** mrodden has quit IRC07:20
*** mrodden has joined #openstack-chef07:21
*** PierreRambaud has quit IRC07:48
*** otter768 has joined #openstack-chef07:59
*** PierreRambaud has joined #openstack-chef08:00
*** otter768 has quit IRC08:04
*** carlp has quit IRC08:29
*** carlp has joined #openstack-chef08:35
*** ChanServ sets mode: +o carlp08:35
*** wojdev has quit IRC08:47
*** wojdev has joined #openstack-chef08:52
*** jklare has quit IRC08:53
*** jklare has joined #openstack-chef08:55
*** nkrinner has quit IRC09:07
*** nkrinner has joined #openstack-chef09:09
*** wojdev has quit IRC09:11
openstackgerritMerged stackforge/cookbook-openstack-network: neutron nova_admin_auth_url requires a protocol version  https://review.openstack.org/17586109:42
openstackgerritMerged stackforge/cookbook-openstack-network: Make the metadata_workers attribute in metadata_agent.ini configurable.  https://review.openstack.org/17541209:46
*** wojdev has joined #openstack-chef09:51
openstackgerritMa Wen Cheng proposed stackforge/cookbook-openstack-network: Update the deprecated group/name to Kilo release  https://review.openstack.org/17281009:53
*** wenchma has quit IRC10:01
*** wojdev has quit IRC10:50
*** zhiwei has quit IRC10:56
*** pradipta has quit IRC11:29
*** wojdev has joined #openstack-chef11:32
*** DaveIshmael has joined #openstack-chef11:45
*** grealish has quit IRC11:51
*** grealish has joined #openstack-chef11:56
*** otter768 has joined #openstack-chef12:00
*** otter768 has quit IRC12:05
*** neelashah has joined #openstack-chef12:29
*** lan has joined #openstack-chef12:54
*** stevemar has joined #openstack-chef13:29
*** stevemar has quit IRC13:37
*** lan__ has joined #openstack-chef13:59
*** nkrinner has quit IRC14:01
*** otter768 has joined #openstack-chef14:01
*** lan has quit IRC14:03
*** zhiwei has joined #openstack-chef14:05
*** otter768 has quit IRC14:06
*** stevemar has joined #openstack-chef14:07
sc`j^2: we broke berks :(14:08
*** seizadi has joined #openstack-chef14:09
*** petertr7 has joined #openstack-chef14:12
*** seizadi has quit IRC14:13
sc`i don't want to go bushwacking because i'm sure there's a good reason for the name change14:13
zhiweithe openstack-client cookbook?14:13
sc`yeah14:13
*** ncerny has joined #openstack-chef14:14
zhiweiyou can upload a patch to make it correct.14:14
sc`i could, but against what i do not know14:14
sc`i could solve it in the berksfile by undoing my change, or change the cookbook name itself. being that it was done for some reason, i'd rather know the reason before i go changing it in code14:15
zhiwei?14:16
sc`i don't want to go changing all the references to openstack_client if there's a good reason not to14:17
sc`i'll submit a patch anyway14:18
cmlucianowhat’s breaking exactly?14:18
zhiweithe cookbook in Berksfile is openstack-client14:19
sc`berkshelf wants 'openstack-client' instead of openstack_client, which was fine up until not too long ago14:19
zhiweibut in metadata.rb, it is openstack_client14:19
sc`the most recent change to the client cookbook broke the pattern in openstack-chef-repo14:20
j^2Yeah the lwrp didn't work with the - only with _14:20
sc`yup. i knew there was a good reason for the name change14:20
zhiweij^2: I don't think so.14:21
zhiweithe openstack-image cookbook also have a LWRP14:21
zhiweiand also the identify cookbook14:22
j^2True but they don't have the same naming convention14:22
j^2The - in the name is a known but with chef iirc14:23
zhiweiI can't understand what you said.14:24
*** smccully has quit IRC14:24
*** smccully has joined #openstack-chef14:24
j^2The dash in the name doesn't work with lwrps14:26
j^2Hence the underscore instead. Why don't we just update all the berksfiles?14:26
*** stevemar has quit IRC14:26
j^2sc`: ^14:27
sc`this particular berksfile is just my change in openstack-chef-repo. probably a one-line fix14:27
j^2yep.14:27
sc`i can break client out of the loop14:27
sc`didn't mean to alarm, just wasn't informed enough. now i am14:28
j^2:)14:29
zhiweihttps://review.openstack.org/#/c/175100/3/Berksfile14:29
sc`yup14:29
sc`that's one side of it14:30
sc`that's where i'll fix it14:30
zhiweiok14:30
sc`i think that's the only berksfile that cares14:30
zhiweiChef does not allow using methods in attribute file? https://review.openstack.org/#/c/176156/1/attributes/database.rb14:30
sc`everything else is just metadata14:30
j^2zhiwei: it should14:34
zhiweiok, I will check the reason. Thanks.14:35
cmlucianozhiwei:  which methods are you referring to?14:36
cmlucianocase is definitely allows14:37
cmlucianoallowed*14:37
zhiweiplease check the code, I used get_password14:37
*** wojdev has quit IRC14:37
zhiweiI refrenced a method in common library.14:39
zhiweiclass ::Chef::Recipe # rubocop:disable Documentation14:39
zhiwei  include ::Openstack14:39
zhiweiend14:39
zhiweiusing this in attribute file.14:39
j^2Is common before on the run_list ?14:40
zhiweioh, no.14:40
j^2There us go :)14:40
zhiweithis is an attribute file in common method.14:40
j^2You*14:40
zhiweicommon cookbook14:40
openstackgerritSamuel Cassiba proposed stackforge/openstack-chef-repo: Remove client cookbook from array  https://review.openstack.org/17633614:41
zhiweiAre there any solutions to reference a method from cookbook's attribute in cookbook's library?14:42
j^2None that I can think of at this moment14:46
jklarezhiwei: j^2: i do not think that the run_list has any influence on methods that can be used or not, this is done via dependencies in the cookbook14:53
zhiweiThe get_password method is defined in common cookbook, and I want to call this method in attribute file that belongs to common cookbook.14:54
jklarezhiwei: j^2: the issue with using the method 'get_password' in an attributes file is, that this method is not defined for the Chef::Attributes class but the Chef::Recipe class14:55
zhiweiAttributes are loaded before libraries, so I cannot do this.14:55
zhiweioh, can I define a method both for Recipe and Attribute?14:56
jklareruby is anarchy, so basically you can do whatever you like, the question is if this is really needed and a good idea in this context14:57
zhiweihttps://review.openstack.org/#/c/176156/1/attributes/database.rb I just want to make all config options are directly assigned by attribute, so we can override them easily.14:58
zhiwei-class ::Chef::Recipe # rubocop:disable Documentation14:59
zhiwei+class ::Chef::Attribute # rubocop:disable Documentation14:59
zhiweitesting this now14:59
jklarenot sure if the ::Chef::Attribute class exists15:00
jklarei think its probably ::Chef::Node::Attirbutes15:00
jklarebut you should check the code15:00
zhiweiok15:01
jklareand ofc you would need to define the method again for this class15:02
zhiweiI knew Chef::Provide can work.15:02
zhiweiI think there is no way to do this, I need to add variables instead.15:08
jklarei think there is a way, but i do not think that this is desireable15:11
jklareusually you try to avoid logic in your attributes file15:12
jklareand if you need logic, you should probably move to the recipe15:12
jklareat least that is how i understand the chef way right now15:12
zhiweiyes.15:15
markvan_zhiwei: yeah, we went down this road last cycle, and can to the conclusion that while it seems logical, it just makes the end results more complex to maintain.15:16
zhiweioh..15:16
zhiweiI only understood 30% of your words in last meeting, pool English.15:17
zhiweibut make the service db attributes under a loop is still needed.15:19
zhiweiNeed to go to bed now.15:21
zhiweiHave nice day guys.15:21
openstackgerritMerged stackforge/cookbook-openstack-orchestration: Set default value for log_dir in heat.conf  https://review.openstack.org/17617315:26
openstackgerritMerged stackforge/cookbook-openstack-integration-test: This creates the role it doesn't by default  https://review.openstack.org/16485515:29
*** lan__ has quit IRC15:30
zhiweimarkvan_: Please check the defect about the pre-setup recipe, and chat with Richard about this if you have time. Thanks.15:30
openstackgerritMark Vanderwiel proposed stackforge/cookbook-openstack-dashboard: Prevent html page caching by default  https://review.openstack.org/17593415:31
markvan_zhiwei: k, will try, but swamped with a few issues right now15:31
*** zhiwei has left #openstack-chef15:33
markvan_cmluciano: j^2  my rabbitmq PR failed travis, but does not seem related to my change, is there a way to run travis against master to verify that?  https://github.com/jjasghar/rabbitmq/pull/25515:37
openstackgerritMark Vanderwiel proposed stackforge/cookbook-openstack-integration-test: Move to rhel 7.1  https://review.openstack.org/16949315:41
openstackgerritYuki Nishiwaki proposed stackforge/openstack-chef-repo: Conform to update of the cookbook-openstack-client  https://review.openstack.org/17636615:44
j^2markvan_: i’d go ahead an remove that test that’s failing16:01
j^2being the erlang cookbook should deal with the issue now16:01
*** otter768 has joined #openstack-chef16:02
markvan_yup, that's sounds right, thx.    btw, is there a way to run a travis against master?   or just put a dummy PR in?16:02
j^2i can re-run it16:02
j^2but you removing the test and pushing the code should kick it off again16:03
markvan_yup, I can easily add that to my PR16:03
openstackgerritMerged stackforge/cookbook-openstack-network: Update the deprecated group/name to Kilo release  https://review.openstack.org/17281016:04
*** mattray has joined #openstack-chef16:04
*** ChanServ sets mode: +o mattray16:04
markvan_j^2: new PR is pushed...and running in travis now...16:05
j^2k16:05
*** otter768 has quit IRC16:07
*** jmickle has joined #openstack-chef16:14
markvan_j^2: cmluciano PR   https://github.com/jjasghar/rabbitmq/pull/255  is passing travis now, thx.   That one along with https://github.com/jjasghar/rabbitmq/pull/247 are what I'm looking for this week.16:59
j^2k16:59
*** harlowja_away is now known as harlowja16:59
*** thumpba has joined #openstack-chef17:01
*** seizadi has joined #openstack-chef17:01
*** thumpba_ has joined #openstack-chef17:02
cmlucianoya j^2 that failing is related to the cleanup done in the erlang package resource17:04
cmlucianoit worked prior so I left it17:04
*** os-chef-bot1 has joined #openstack-chef17:04
j^2cool cool no worries, i have the rabbitmq cookbook release slated for tomorrow17:05
*** thumpba has quit IRC17:05
*** os-chef-bot has quit IRC17:11
*** mattray has quit IRC17:27
-openstackstatus- NOTICE: gerrit is restarting to clear hung stream-events tasks. any review events between 16:48 and 17:32 utc will need to be rechecked or have their approval votes reapplied to trigger testing in zuul17:31
openstackgerritMark Vanderwiel proposed stackforge/cookbook-openstack-dashboard: Allow non default db port for horizon  https://review.openstack.org/17510117:40
openstackgerritMark Vanderwiel proposed stackforge/cookbook-openstack-dashboard: Prevent html page caching by default  https://review.openstack.org/17593417:40
*** seizadi has quit IRC17:58
*** otter768 has joined #openstack-chef18:03
*** seizadi has joined #openstack-chef18:05
*** mattray has joined #openstack-chef18:06
*** ChanServ sets mode: +o mattray18:06
*** otter768 has quit IRC18:08
markvan_j^2: cmluciano looks lke both the libvirt and the django ubuntu bugs may have been fixed, I'm spinning up a ubuntu aio-nova + https://review.openstack.org/#/c/176366/ to see if we're whole again.18:23
markvan_Hah, misspoke...tried to use my script to run this:  "chef exec ruby test_patch -p 176366"  bit it failed, my script only handles patches from the cookbooks right now, not the repo.  Thoughts on if I should enhance it to support repo patches?  Is that really needed?18:25
cmlucianoeh18:31
cmlucianoI guess it depends18:31
cmlucianoI guess it would be helpful just for a dry run18:33
cmlucianofingers crossed on the libvirt issues18:33
*** neelashah has quit IRC18:50
*** stevemar has joined #openstack-chef18:50
*** a13x2121 has quit IRC18:57
*** neelashah has joined #openstack-chef19:06
*** neelashah1 has joined #openstack-chef19:07
*** a13x212 has joined #openstack-chef19:07
*** neelashah has quit IRC19:10
*** seizadi has quit IRC19:15
*** stevemar has quit IRC19:15
openstackgerritMark Vanderwiel proposed stackforge/openstack-chef-repo: Conform to update of the cookbook-openstack-client  https://review.openstack.org/17636619:18
*** seizadi has joined #openstack-chef19:20
*** nkrinner has joined #openstack-chef19:37
*** stevemar has joined #openstack-chef19:44
*** seizadi has quit IRC19:47
*** seizadi has joined #openstack-chef19:52
openstackgerritMerged stackforge/cookbook-openstack-identity: Allow mq setup for keystone  https://review.openstack.org/16988419:52
openstackgerritMerged stackforge/cookbook-openstack-compute: Use identity_uri_transform for identiry_uri support  https://review.openstack.org/16808519:53
*** openstackgerrit has quit IRC19:54
*** openstackgerrit has joined #openstack-chef19:55
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341520:01
openstackgerritImtiaz Chowdhury proposed stackforge/cookbook-openstack-identity: Make SSL settings configurable for Keystone  https://review.openstack.org/17341520:03
*** seizadi has quit IRC20:04
*** otter768 has joined #openstack-chef20:04
*** otter768 has quit IRC20:09
*** seizadi has joined #openstack-chef20:10
*** Guest23 has joined #openstack-chef20:21
*** Guest23 is now known as libsysguy20:21
openstackgerritMark Vanderwiel proposed stackforge/openstack-chef-repo: New test_patch helper script  https://review.openstack.org/16488420:31
*** seizadi has quit IRC20:34
*** mattray has quit IRC20:35
markvan_@core Good news, with this repo patch, https://review.openstack.org/#/c/176366/, aio_nova on ubuntu will converge and run the basic queries (see test_patch script).   Bad news, the libvirt qemu issue is still there.  There's a couple more repo patches  after this one goes.20:36
os-chef-bot1@j^2 @markvan @mattray @wenchma @jklare @cmluciano Good news, with this repo patch, https://review.openstack.org/#/c/176366/, aio_nova on ubuntu will converge and run the basic queries (see test_patch script).   Bad news, the libvirt qemu issue is still there.  There's a couple more repo patches  after this one goes.20:36
cmluciano:(20:36
j^2soooo close20:36
j^2i want to get buliding again!20:36
libsysguyme too20:37
j^2libsysguy: :-O20:37
j^2the prodigal son returns!20:37
libsysguysnuck in the back door20:37
markvan_j^2: btw need to add  Zhiwei Chen to core list in bot20:37
j^2ah yeah20:37
*** seizadi has joined #openstack-chef20:37
markvan_also, this patch is failing the new bootstrap https://review.openstack.org/#/c/165737/, so will need to get that fixed up before we put in the infra changes to start using Rake jobs20:38
j^2i’m actually planning on convert os-chef-bot1 to lita on some off time20:39
j^2i just need to do it ;)20:39
*** stevemar has quit IRC20:41
openstackgerritMark Vanderwiel proposed stackforge/cookbook-openstack-common: Update yum to 3.5.4  https://review.openstack.org/17647720:42
*** stevemar has joined #openstack-chef20:45
markvan_so, my test_patch repo script does not handle updating the repo itself, so getting that repo patch in will help remove the manual step for that.20:46
*** neelashah1 has quit IRC20:56
*** libsysguy has quit IRC20:57
*** nkrinner has quit IRC21:00
openstackgerritMerged stackforge/cookbook-openstack-dashboard: Allow non default db port for horizon  https://review.openstack.org/17510121:04
*** Guest23 has joined #openstack-chef21:10
*** jmickle has quit IRC21:10
*** stevemar has quit IRC21:13
openstackgerritMark Vanderwiel proposed stackforge/openstack-chef-repo: New test_patch helper script  https://review.openstack.org/16488421:14
*** openstackgerrit has quit IRC21:29
*** openstackgerrit has joined #openstack-chef21:30
*** neelashah has joined #openstack-chef21:36
*** seizadi has quit IRC21:46
*** seizadi has joined #openstack-chef21:48
*** Guest23 has quit IRC22:00
*** otter768 has joined #openstack-chef22:05
*** otter768 has quit IRC22:09
*** openstackgerrit has quit IRC22:11
*** openstackgerrit has joined #openstack-chef22:11
*** neelashah has quit IRC22:23
*** seizadi has quit IRC22:24
*** seizadi has joined #openstack-chef22:31
*** libsysguy has joined #openstack-chef22:36
*** libsysguy has quit IRC22:38
*** ncerny has quit IRC22:53
*** GonZo2K has joined #openstack-chef22:53
*** stevemar has joined #openstack-chef23:01
*** petertr7 has quit IRC23:31
*** thumpba_ has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!