Wednesday, 2018-10-31

« Tuesday, 2018-10-30 Index Thursday, 2018-11-01 »
*** salmankhan has quit IRC00:11
openstackgerritMerged openstack/puppet-ironic master: Install python3-ironicclient in Fedora or RedHat > 7  https://review.openstack.org/61366100:37
*** markvoelker has quit IRC00:49
*** markvoelker has joined #puppet-openstack00:50
*** markvoelker has quit IRC00:55
*** chem has quit IRC02:11
*** ykarel has joined #puppet-openstack02:34
*** markvoelker has joined #puppet-openstack02:51
*** mschuppert has quit IRC02:56
openstackgerrityatin proposed openstack/puppet-openstacklib master: Revert "Install python3-openstackclient in Fedora or RedHat > 7"  https://review.openstack.org/61439103:04
*** markvoelker has quit IRC03:24
*** markvoelker has joined #puppet-openstack04:21
*** markvoelker has quit IRC04:54
*** ykarel has quit IRC05:21
*** ykarel has joined #puppet-openstack05:21
*** markvoelker has joined #puppet-openstack05:51
*** ykarel has quit IRC05:51
*** markvoelker has quit IRC06:26
*** dpawlik has joined #puppet-openstack06:27
*** dpawlik has quit IRC06:28
*** dpawlik has joined #puppet-openstack06:28
*** agurenko has joined #puppet-openstack06:33
openstackgerritMerged openstack/puppet-zaqar master: Install python3-zaqarclient in Fedora or RedHat > 7  https://review.openstack.org/61365807:02
*** pma78 has joined #puppet-openstack07:07
openstackgerritzhongshengping proposed openstack/puppet-heat master: Add manage_backend_package parameter  https://review.openstack.org/61441807:13
*** chkumar|off is now known as chandankumar07:13
openstackgerritzhongshengping proposed openstack/puppet-nova master: Add manage_backend_package parameter  https://review.openstack.org/61441907:23
*** markvoelker has joined #puppet-openstack07:24
*** ykarel has joined #puppet-openstack07:42
*** shardy has joined #puppet-openstack07:45
*** hjensas has quit IRC07:46
*** shardy_ has joined #puppet-openstack07:46
*** ykarel has quit IRC07:49
*** ykarel has joined #puppet-openstack07:50
*** ykarel has quit IRC07:56
*** markvoelker has quit IRC07:57
*** ykarel has joined #puppet-openstack08:02
*** ykarel has quit IRC08:10
*** jtomasek has joined #puppet-openstack08:18
*** ykarel has joined #puppet-openstack08:27
*** ykarel has quit IRC08:28
*** ykarel has joined #puppet-openstack08:29
*** ykarel_ has joined #puppet-openstack08:31
*** ykarel has quit IRC08:34
*** ykarel has joined #puppet-openstack08:34
*** ykarel_ has quit IRC08:38
*** ykarel has quit IRC08:40
*** derekh has joined #puppet-openstack08:51
*** jpena|off is now known as jpena08:52
*** markvoelker has joined #puppet-openstack08:54
*** chem has joined #puppet-openstack09:05
iurygregorygood morning09:11
*** markvoelker has quit IRC09:27
*** hjensas has joined #puppet-openstack09:29
*** gfidente has joined #puppet-openstack09:37
*** dpawlik_ has joined #puppet-openstack09:55
*** dpawlik_ has quit IRC09:56
*** dpawlik has quit IRC09:57
*** dpawlik has joined #puppet-openstack09:57
*** salmankhan has joined #puppet-openstack10:05
*** markvoelker has joined #puppet-openstack10:24
*** dtantsur|afk is now known as dtantsur10:49
*** markvoelker has quit IRC10:58
*** iurygregory is now known as iury|lunch11:00
*** jtomasek has quit IRC11:16
openstackgerritJanki Chhatbar proposed openstack/puppet-neutron master: IPv6 support for ODL-OVS deployment  https://review.openstack.org/58474011:33
*** jtomasek has joined #puppet-openstack11:38
*** morazi has joined #puppet-openstack11:53
*** markvoelker has joined #puppet-openstack11:54
*** iury|lunch is now known as iurygregory12:01
*** trown|outtypewww is now known as trown12:05
*** markvoelker has quit IRC12:13
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Use puppet4 functions-api  https://review.openstack.org/61422612:17
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api  https://review.openstack.org/61423112:21
derekhHi, if anybody has a couple of minutes, id appreciate a quick puppet-neutron review https://review.openstack.org/#/c/611297/12:22
openstackgerritTobias Urdin proposed openstack/puppet-murano master: Use puppet4 functions-api  https://review.openstack.org/61423812:27
*** jpena is now known as jpena|lunch12:46
iurygregoryderekh, doing now =)12:54
derekhiurygregory: thanks12:55
iurygregoryderekh, if the config its not present is the same as false right ? so you wouldnt be able to manage, (just thinking about the test13:01
*** agurenko has quit IRC13:08
*** agurenko has joined #puppet-openstack13:10
*** ade_lee has joined #puppet-openstack13:28
openstackgerritDaniel Abad proposed openstack/puppet-ironic master: Expose service_token_roles{,_required} params  https://review.openstack.org/61450513:41
*** jpena|lunch is now known as jpena13:48
derekhiurygregory: if the config isn't present then nothing gets put into the plugin config file and default (True) ends up getting used14:10
iurygregoryderekh, nice o/ tks14:11
iurygregoryi already +2 the patch14:11
*** dpawlik has quit IRC14:29
*** dpawlik has joined #puppet-openstack14:30
*** dpawlik has quit IRC14:34
openstackgerritDaniel Abad proposed openstack/puppet-ironic master: Expose service_token_roles{,_required} params  https://review.openstack.org/61450514:54
openstackgerritAlan Bishop proposed openstack/puppet-cinder master: Deprecate several iscsi_XXX parameters  https://review.openstack.org/61453614:56
*** jistr is now known as jistr|call15:05
zigomwhahaha: tobias-urdin: EmilienM: mnaser: How would I change clients_keystone/ca_file in heat.conf ? I tried this: http://paste.openstack.org/show/733705/ but it didn't work (it didn't do anything...)15:25
mwhahahayou can just use heat_config (not sure why you have resources)15:26
mwhahahaalternatively hieradata via ::heat::config15:26
zigomwhahaha: I'm not in the heat module, I'm in my own ...15:27
mwhahaharight15:27
zigoOk.15:28
zigoTrying.15:28
mwhahahazigo: there is ::heat::keystone::authtoken::cafile15:28
mwhahahais that not right?15:28
zigomwhahaha: That's unrelated.15:29
zigoUseful, but unrelated ! :)15:29
mwhahahatoo many config options15:29
zigoYeah.15:29
zigoThere's only 1200 options in Nova alone, you know ... :P15:29
mwhahahayou shouldn't need to do any relationship thing with the heat_config because ::heat::deps will automatically do all heat_config at the right time15:29
zigoOk.15:30
zigoIn my case, I get an SSL error if I do "openstack --debug stack list" and there's no ca_file in [clients_keystone]/ca_file.15:30
zigo(and it just then fails with a 500 error ...)15:30
tobias-urdinmwhahaha: were you able to determine if it was because it didn't use rspec-puppet-facts?15:31
tobias-urdinthe CI keeps on giving http://logs.openstack.org/35/613435/1/gate/puppet-openstack-libraries-puppet-lint-centos-7/3a53d7b/job-output.txt.gz#_2018-10-31_02_48_38_95549115:31
zigoOh, it worked, wonderful ! :)15:31
mwhahahatobias-urdin: that's unrelated15:31
tobias-urdinlibraries-lint job fails because of the new code in puppet-nova, the lint configs are probably different15:31
mwhahahayea i thought we ignored that error15:31
tobias-urdinwe do, but not when Gemfile in puppet-nova is installed it seems like15:32
tobias-urdinwe should probably change that so it uses the same as all other testing15:32
*** jistr|call is now known as jistr15:35
mwhahahatobias-urdin: and to answer your question, yes it's because the nova::metadata::novajoin::api does not use rspec-puppet-facts15:36
mwhahahawe'll need to convert that15:36
tobias-urdini'll get on it15:36
*** markvoelker has joined #puppet-openstack15:52
openstackgerritDaniel Abad proposed openstack/puppet-ironic master: Expose service_token_roles{,_required} params  https://review.openstack.org/61450516:05
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts  https://review.openstack.org/61457716:11
tobias-urdinmwhahaha: this is also weird, it didn't fail even though it got an error16:24
tobias-urdinhttp://logs.openstack.org/49/613449/3/check/puppet-openstack-lint/1cfcdd1/job-output.txt.gz#_2018-10-30_21_35_49_25411516:24
*** agurenko has quit IRC16:24
mwhahahahow did this stuff land :/16:25
tobias-urdinyeah, testing is pure wrong16:26
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts  https://review.openstack.org/61457716:29
tobias-urdinmwhahaha: for some reason, i don't understand, this call https://github.com/openstack/puppet-openstack_spec_helper/blob/master/run_lint_tests.sh#L22 doesn't honor https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/rake_tasks.rb#L2116:35
tobias-urdinthe other one is probably a lint bug, where it didn't exit 1 on lint ERROR:s16:35
mwhahahawould have to compare versions16:36
mwhahahalet me see16:36
mwhahahahrm same versions for the previous patch set16:39
mwhahahahttp://logs.openstack.org/49/613449/3/check/puppet-openstack-lint/1cfcdd1/job-output.txt.gz#_2018-10-30_21_35_33_855201 vs http://logs.openstack.org/49/613449/2/check/puppet-openstack-lint/c0ebfdd/job-output.txt.gz#_2018-10-26_03_35_29_16257316:39
mwhahahaso this change added it https://review.openstack.org/#/c/613510/, let me see16:40
*** trown is now known as trown|lunch16:40
mwhahahayea the error didn't stop it16:40
mwhahahahttp://logs.openstack.org/10/613510/1/check/puppet-openstack-lint/47125ea/job-output.txt.gz#_2018-10-28_10_00_48_81527916:40
*** gfidente is now known as gfidente|afk16:40
tobias-urdinmwhahaha: yeah, that's probably a puppet-lint bug right?16:42
tobias-urdiniirc if you specify ignores it shouldn't output them as well16:42
tobias-urdinso if you check the lint job here https://review.openstack.org/#/c/614083/16:42
tobias-urdinit outputs warnings and exits ok16:42
tobias-urdinchecking your comment here https://github.com/openstack/puppet-openstack_spec_helper/blob/master/puppet-openstack_spec_helper.gemspec#L2116:45
tobias-urdinwe pin 2.3.0 but that issue happens on both xenial and centos 7 (both puppet-lint 2.3.0)16:45
tobias-urdinthere are 2.3.6 available, perhaps we should try to release that pinning16:45
tobias-urdincant see anything relevant here though https://github.com/rodjek/puppet-lint/blob/master/CHANGELOG.md16:47
openstackgerritTobias Urdin proposed openstack/puppet-openstack_spec_helper master: Update puppet-lint pin to 2.3.6  https://review.openstack.org/61458916:49
* tobias-urdin signs out from offce -- will be online from home16:50
*** chandankumar is now known as chkumar|off16:53
*** shardy has quit IRC16:56
*** shardy_ has quit IRC16:56
openstackgerritTong Liu proposed openstack/puppet-neutron stable/queens: Adds new neutron nsx plugin parameters  https://review.openstack.org/61459417:20
*** hjensas has quit IRC17:22
openstackgerritCarol Bouchard proposed openstack/puppet-neutron master: CiscoNexus: Update and Remove deprecated vars  https://review.openstack.org/60695717:24
openstackgerritCarlos Goncalves proposed openstack/puppet-octavia master: WIP: Set sharable controller and DB configurations  https://review.openstack.org/61239517:31
*** derekh has quit IRC18:01
*** jpena is now known as jpena|off18:04
*** trown|lunch is now known as trown18:04
*** hjensas has joined #puppet-openstack18:09
*** dtantsur is now known as dtantsur|afk18:17
*** salmankhan has quit IRC19:07
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461819:14
openstackgerritAlan Bishop proposed openstack/puppet-cinder master: Deprecate several iscsi_XXX parameters  https://review.openstack.org/61453619:18
*** morazi has quit IRC19:24
openstackgerritAlan Bishop proposed openstack/puppet-cinder master: Deprecate several iscsi_XXX parameters  https://review.openstack.org/61453619:24
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461919:29
openstackgerritRajini Karthik proposed openstack/puppet-nova stable/rocky: Support for libvirt volume multipath  https://review.openstack.org/61462319:42
openstackgerritRajini Karthik proposed openstack/puppet-nova stable/queens: Support for libvirt volume multipath  https://review.openstack.org/61462519:48
openstackgerritAlex Schultz proposed openstack/puppet-nova master: Ignore 140 char lint issues  https://review.openstack.org/61462819:54
mwhahahatobias-urdin: -^19:54
tobias-urdinnice20:21
*** morazi has joined #puppet-openstack20:22
openstackgerritTobias Urdin proposed openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts  https://review.openstack.org/61457720:26
*** trown is now known as trown|outtypewww20:29
radezHey folks, anyone have a few mins to take a look at this review for us? https://review.openstack.org/#/c/611297/20:37
*** sai_p has joined #puppet-openstack20:40
*** salmankhan has joined #puppet-openstack20:45
*** salmankhan has quit IRC20:50
*** chem has quit IRC21:28
*** salmankhan has joined #puppet-openstack22:18
*** salmankhan has quit IRC22:25
*** gfidente|afk has quit IRC22:36
mwhahahameh will need to squash the 140 lint stuff with fixing up the unit tests22:57
mwhahahatobias-urdin: if you don't get to it, i'll take a stab tomorrow22:58
*** sai_p has quit IRC23:09
« Tuesday, 2018-10-30 Index Thursday, 2018-11-01 »

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