Wednesday, 2018-12-19

*** jmlowe has quit IRC00:07
*** fragatina has joined #heat00:40
*** maddtux has joined #heat00:52
*** hjensas has joined #heat01:59
*** ricolin has joined #heat02:20
openstackgerritRico Lin proposed openstack/heat-templates master: Remove deprecated hooks  https://review.openstack.org/58477703:18
*** hjensas has quit IRC03:25
openstackgerritMerged openstack/python-heatclient master: Set X-Region-Name in header when using SessionClient  https://review.openstack.org/60357403:34
*** hongbin has joined #heat03:51
openstackgerritRabi Mishra proposed openstack/heat master: Bump minimal_image_ref  https://review.openstack.org/62591204:37
*** vabada has quit IRC05:05
*** hongbin has quit IRC05:40
*** hjensas has joined #heat06:26
*** hjensas has quit IRC06:27
*** fragatina has quit IRC06:27
*** fragatina has joined #heat06:28
*** hjensas has joined #heat06:29
*** fragatina has quit IRC06:35
*** fragatina has joined #heat06:36
*** fragatina has quit IRC07:01
*** fragatina has joined #heat07:01
openstackgerritRico Lin proposed openstack/heat master: Allow provide auth to context  https://review.openstack.org/58055007:13
openstackgerritRico Lin proposed openstack/heat master: Add tool for parse auth from Secret  https://review.openstack.org/58311207:13
openstackgerritRico Lin proposed openstack/heat master: Add tools for parse credential from Secret  https://review.openstack.org/60802907:13
openstackgerritRico Lin proposed openstack/heat master: Support remote stack with another OpenStack provider  https://review.openstack.org/57839307:13
openstackgerritRico Lin proposed openstack/heat master: Override ssl options for heatclient in RemoteStack  https://review.openstack.org/48092307:13
openstackgerritRico Lin proposed openstack/heat master: Add document for Multi OpenStack Cloud support  https://review.openstack.org/57839407:13
*** vabada has joined #heat07:27
openstackgerritRico Lin proposed openstack/heat master: Add release note for 1649759  https://review.openstack.org/62610707:31
ricolinHi all, let's start a meeting in 7mins07:54
ricolin#startmeeting heat08:01
openstackMeeting started Wed Dec 19 08:01:10 2018 UTC and is due to finish in 60 minutes.  The chair is ricolin. Information about MeetBot at http://wiki.debian.org/MeetBot.08:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.08:01
*** openstack changes topic to " (Meeting topic: heat)"08:01
openstackThe meeting name has been set to 'heat'08:01
ricolin#topic roll call08:01
*** openstack changes topic to "roll call (Meeting topic: heat)"08:01
zanebo/08:01
ricolino/08:01
ricolinrabi seems not around:)08:01
ricolinzaneb, btw I update the temp file patch for multi cloud earlier08:03
zaneboh, I guess I have to review that series now then08:03
ricolinzaneb, that will be kind of you!08:06
ricolin#topic Cirros image is AWOL in the gate08:06
*** openstack changes topic to "Cirros image is AWOL in the gate (Meeting topic: heat)"08:06
zanebthe master gate is broken08:06
zanebRabi has been working on it08:06
ricolinso the gate is failing in that patch for now08:06
zaneb#link https://review.openstack.org/62591208:06
zanebyeah, strange stuff is happening08:07
zanebI don't understand why the new devstack is not downloading images08:07
ricolinthere must be some discussion that we're not aware about08:08
zanebwe're not going to solve it in this meeting, but it needs to get solved so I thought it was worth flagging on the agenda08:08
ricolinzaneb, I will try to dive in and find what's wrong, and I think Rabi will keep doing so too08:09
zanebcool. I put a suggestion in there but I don't know if it's crazy or not :)08:10
* zaneb is not a devstack user08:10
ricolindevstack is getting easier to use now08:10
ricolinat least you can use systemctl to control services08:11
ricolin#topic We need to use upper-constaints in heat-agents to unblock stable branches08:11
*** openstack changes topic to "We need to use upper-constaints in heat-agents to unblock stable branches (Meeting topic: heat)"08:11
*** gkadam has joined #heat08:11
*** jtomasek has joined #heat08:11
openstackgerritRabi Mishra proposed openstack/heat master: Bump minimal_image_ref  https://review.openstack.org/62591208:12
zanebthe stable branch gates for heat-agents are all blocked because we don't use upper-constraints08:12
zanebwe need reviews on the patches to unblock them08:12
zaneb#link https://review.openstack.org/#/q/Ib2ef3b3d0d0733842abad97898114c3dbdab3b7a08:12
zanebshouldn't take more than 3 minutes08:13
zaneball the major revisions of paunch have caused us no end of trouble, but once those merge we should finally have proper dependency management for heat-agents08:13
zanebwhich leads nicely into our next topic...08:14
ricolinapproved in master08:14
zanebricolin: thanks. I will approve the stable ones with your +108:15
zanebonce that merges08:15
ricolintalk about stable branch08:15
ricolinis there chances we can land these https://review.openstack.org/#/c/617593/08:16
ricolinthe missing of region in os-collect config for pike and ocata08:16
ricolinzaneb, do you happen to be the core of os-collect-config?:)08:17
zanebwe already closed stable/pike for heat-agents, so I don't know that we should merge anything to ocata/pike08:17
zanebI am not stable-core on that08:17
zanebso it's TripleO people that would have to merge it08:18
ricolinIt's weird to fix it, and also weird to just leave it08:18
ricolins/and/but/08:19
ricolin#topic It's time to remove the deprecated copy of heat-agents from heat-templates08:20
*** openstack changes topic to "It's time to remove the deprecated copy of heat-agents from heat-templates (Meeting topic: heat)"08:20
zanebricolin: thanks for updating the patch08:20
ricolinI should done that for long time:(08:21
zaneb#link https://review.openstack.org/58477708:21
zanebI +2d it already, so unless somebody objects very soon I'm gonna approve it08:21
ricolinzaneb, cool08:21
openstackgerritMerged openstack/heat-agents master: Use upper-constraints in tox tests  https://review.openstack.org/62336308:21
* zaneb is sick of having to -2 patches to those agents08:21
zanebthat ^ was quick08:22
ricolinI'm also concern about the python versions too08:22
ricolinare we going to keep update the python language version for heat-templates?08:22
zanebis there any python code left in that repo after this?08:23
ricolinzaneb, well, welcome back to Asia time zone08:23
zaneblol08:23
zanebI have been pushing patches at times when I would have guessed for sure there'd be nothing running in the gate, and it has still taken ages08:24
ricolin./setup.py08:25
ricolin./tests/software_config/test_hook_cfn_init.py08:25
ricolin./tests/software_config/common.py08:25
ricolin./tests/software_config/test_heat_config.py08:25
ricolin./tests/software_config/test_hook_json_file.py08:25
ricolin./tests/software_config/test_hook_apply_config.py08:25
ricolin./tests/software_config/test_hook_kubelet.py08:25
ricolin./tests/software_config/test_heat_config_notify.py08:25
ricolin./tests/software_config/test_hook_salt.py08:25
ricolin./tests/software_config/test_heat_config_docker_compose.py08:25
*** shardy has joined #heat08:25
ricolin./tests/software_config/heat_config_notify.py08:25
ricolin./tests/software_config/test_hook_script.py08:25
ricolin./tests/software_config/test_hook_puppet.py08:25
ricolin./tests/software_config/hook-fake.py08:25
ricolin./tests/software_config/hook_kubelet.py08:25
ricolin./tests/software_config/config-tool-fake.py08:25
ricolin./tests/software_config/test_hook_atomic.py08:25
ricolin./tests/software_config/test_hook_chef.py08:25
ricolin./tests/software_config/test_heat_config_kubelet.py08:25
ricolin./tests/software_config/test_hook_docker_compose.py08:25
ricolin./tests/software_config/test_hook_ansible.py08:25
ricolin./tests/software_config/__init__.py08:25
ricolin./tests/software_config/test_hook_docker_cmd.py08:25
ricolin./tests/software_config/test_hook_hiera.py08:25
ricolin./tests/__init__.py08:25
ricolin./doc/source/conf.py08:25
ricolinsorry, I should use paste08:25
ricolinAnyway, these are all python files left08:25
ricolinin heat-templates08:25
therveThose are the tests for agents08:25
zanebyeah, those should be deleted along with the agents08:26
ricolinI guess I should remove them all in same patch08:27
zanebricolin: I promise to re-review it promptly :)08:28
ricolinthanks08:28
ricolin#topic WTF is the name of the service(s) project in RDO???08:29
*** openstack changes topic to "WTF is the name of the service(s) project in RDO??? (Meeting topic: heat)"08:29
zaneb#link https://review.openstack.org/62523908:29
ricolinI don't know:(08:30
zanebso it's either "service" or "services"08:30
zanebit has flipped back and forth like 4 times now08:30
zanebit would be great to know the answer?08:30
zanebbut failing that we should at least say we have no idea and if one doesn't work for you try the other08:31
ricolin#link http://git.openstack.org/cgit/openstack/glance/tree/doc/source/install/install-rdo.rst08:31
ricolinmaybe this will help you08:31
ricolinI see most of other project use `service`08:32
ricolinwhy we're services?08:32
zanebbecause somebody said it was 'services'08:32
ricolin#link http://git.openstack.org/cgit/openstack/cinder/tree/doc/source/install/cinder-controller-install-rdo.rst08:32
ricolincinder ^^^08:32
zaneb#link https://review.openstack.org/53111608:33
zanebricolin: you and I reviewed it ^ ;)08:33
ricolinlol08:33
zanebthere were 2 +1s but maybe they were just trying to pad their stats08:34
ricolinlet's secretly move it back:)08:34
zanebthe thing is, RDO definitely does (or did) use a different name for that project than the official install guide08:34
zanebtherve: do you remember the details?08:34
ricolintherve, good morning:)08:35
therveNot precisely. I think it's some puppet BS at the origin08:35
zanebI'd vote for change it back but add a note saying if that doesn't work then try 'services'08:35
thervepuppet uses services, so tripleo uses services, so rdo uses services?08:36
therveOr something like that08:36
zanebtherve: or the other way around? :D08:37
therveExactly08:37
ricolinso rdo use puppet?08:37
* ricolin never play with rdo installation08:37
zaneb#action zaneb to propose a new patchset08:37
ricolin#topic Tap-as-a-Service patches are ready for merging IMHO08:39
*** openstack changes topic to "Tap-as-a-Service patches are ready for merging IMHO (Meeting topic: heat)"08:39
ricolin#link https://review.openstack.org/58923808:39
ricolin#link https://review.openstack.org/62521008:39
zanebI believe I reviewed all of the pending new resource type patches this week08:39
zanebL2Gateway and Blazar need some minor tweaks08:40
ricolinVitrage one?08:40
zanebbut I believe the Tap-as-a-Service is ready to merge with one more reviewer08:40
zanebricolin: I did not review Vitrage08:41
ricolinzaneb, I can review it right after meeting08:42
zanebricolin: is Vitrage ready for Stein?08:42
zanebI thought you mentioned at the last meeting that they were WIP still08:42
zanebalso they weren't in my queue08:42
ricolinzaneb, No, if you mean the resource type, but she needs more feedback for sure08:42
ricolinAnd yes it's WIP08:43
zanebok, added it to my queue08:43
ricolinbut not going to go anyway, if we provide no feedback08:43
ricolinzaneb, thanks:)08:44
zanebbut my top priority for now is stuff that is merge-able in Stein08:44
ricolinzaneb, agreed08:44
thervezaneb: The glance image stuff maybe?08:44
ricolintherve, glance V2 support?08:45
thervericolin: web-download support08:45
ricolintherve, yes08:45
thervehttps://review.openstack.org/61791508:45
zanebtherve: is the 'WIP' in the headline still accurate?08:45
thervezaneb: Somewhat08:46
therveProbably need a story, and also if we care about graceful failure08:46
zanebwe don't have a mechanism in general for hiding resources based on optional features, do we08:47
ricolintherve, maybe we should just change Image  to use webdownload?08:47
zanebwhich is something openstack in general is somewhat weak in: https://review.openstack.org/#/c/621516/08:47
thervezaneb: We do, but not that the level that glances exposes it08:47
*** e0ne has joined #heat08:47
therveWe do for service stuff like load balancing08:48
zanebright, yeah we can hide it if there's no service endpoint in the catalog08:48
zanebbut within a service, we can't do that kind of introspection08:48
therveWell there is service extension too08:49
zaneboh, ok, it's fancier than I thought08:49
zanebbut this is less of an extension and more of a configuration option?08:50
zanebis it tied to policy at all, or just an option?08:50
therveI think it's just an option08:50
therveIt probably could be exposed as an extension but I don't believe that's the case08:51
*** rcernin has quit IRC08:51
zanebtherve: I'd be inclined to go with 'meh, just ship it' then08:54
therveOK :)08:54
therveIt can definitely be fixed easily after08:54
zanebusers want it, and if it doesn't work they'll get an error _fairly_ early08:54
zanebtherve: I'm still not convinced we need to change the type name though08:55
zanebthe new resource can still delete images created by the old one, right? what else is there to do to maintain compatibility?08:55
therveThe properties are not the exact same08:56
zanebah, ok08:56
zanebI hate naming resources with verbs is all08:57
ricolinIIRC, it's an option, and we need to give declare to say we only support web download if we like to mark substitute_class for Image?08:57
zanebah, you put the substitute_class thing in, that's cool08:57
therveWe can replace the existing, but I'm not going to test the behavior :)08:58
*** gfidente has joined #heat08:59
zanebit should work08:59
therveYep I don't mind. Rabi suggested the new name in the first place09:00
ricolinmaybe we can bump to v2 in following patch:)09:00
zanebactually, I think the properties have to be compatible, so I'll check that09:00
zanebwhat about OS::Glance::WebImage?09:00
ricolinzaneb, yes, it have to be cover all the old properties IIRC09:01
therveWe can keep the props and ignore the irrelevant ones09:01
ricolintherve, +109:01
zanebI didn't actually know the props were different until just now09:02
therveis_public, extra_properties are out09:02
therveos_version and visibility are in09:02
ricolinOkay let's review that patch as following actions09:05
ricolinAlso tap as a service too09:05
ricolin#topic Open discussion09:05
*** openstack changes topic to "Open discussion (Meeting topic: heat)"09:05
zanebtherve: so I think you just need to add translation rules for those to make it compatible09:05
therve"just" :D09:06
zanebSMOP09:06
ricolinIf you're using translate rule that means you have to also define them in new resource type too09:07
zanebhttp://www.catb.org/jargon/html/S/SMOP.html <- reference09:07
ricolinthat's not ideal IMO09:07
zanebugh, do you09:07
zanebmaybe just skip the substitute class thing then09:08
zanebGlanceV1 has been dead for long enough that there's probably not many users affected09:08
ricolinmaybe:)09:10
ricolinAnything else for topic?:)09:10
*** skramaja has joined #heat09:10
zanebif we have to add the old properties with translations then there's no point doing the new class. I thought the translations ran first, but if not I don't think it's worth the effort09:11
* zaneb has nothing else09:11
zanebthis felt productive though!09:12
zaneblet's do it again. but not next week because it's christmas09:12
ricolinzaneb, we need to trace more to be productive09:12
ricolinzaneb, agree, let's skip the next one09:13
zanebricolin: yeah, we do need to make sure we follow up09:13
zanebbut at least we talked about stuff09:13
ricolinzaneb, yep!09:13
zanebit's very quiet out here in +13 TZ09:13
ricolinmost of the people are sleeping while you're awake:)09:15
ricolin#endmeeting09:15
*** openstack changes topic to "OpenStack Heat (Logs: http://eavesdrop.openstack.org/irclogs/%23heat/)"09:15
openstackMeeting ended Wed Dec 19 09:15:21 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)09:15
openstackMinutes:        http://eavesdrop.openstack.org/meetings/heat/2018/heat.2018-12-19-08.01.html09:15
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/heat/2018/heat.2018-12-19-08.01.txt09:15
openstackLog:            http://eavesdrop.openstack.org/meetings/heat/2018/heat.2018-12-19-08.01.log.html09:15
*** k_mouza has joined #heat09:19
*** k_mouza has quit IRC09:26
*** k_mouza has joined #heat09:27
openstackgerritEthan Lynn proposed openstack/heat-tempest-plugin master: [Do Not Merge] Test for softwaredeployment on delete  https://review.openstack.org/62612509:27
*** e0ne has quit IRC09:28
zanebtherve: I attempted to summarise our discussions from the meeting in the review comments09:29
thervezaneb: Thanks!09:29
zanebnp09:29
zanebricolin, therve: the install guide has 'service':  https://docs.openstack.org/keystone/rocky/install/keystone-users-obs.html which makes me think 'services' is actually correct for RDO because I remember it being different09:35
ricolinzaneb, that means we should notify other project about this:)09:40
*** e0ne has joined #heat09:50
*** ricolin has quit IRC09:56
*** ttsiouts has joined #heat09:58
openstackgerritZane Bitter proposed openstack/heat master: Incorrect project name in installation instructions for CentOS  https://review.openstack.org/62523910:05
zanebtherve: see if you think that ^ looks right10:06
openstackgerritRabi Mishra proposed openstack/heat master: Bump minimal_image_ref  https://review.openstack.org/62591210:10
openstackgerritSurya Prakash (spsurya) proposed openstack/heat-templates master: dict_object.keys() is not required for *in* operator  https://review.openstack.org/62613610:12
*** ttsiouts has quit IRC10:16
*** ttsiouts has joined #heat10:16
*** ttsiouts has quit IRC10:21
*** ttsiouts has joined #heat10:25
*** maddtux has quit IRC10:29
*** ssbarnea|rover has quit IRC10:34
*** ssbarnea|rover has joined #heat10:36
*** pbourke has quit IRC10:37
*** pbourke has joined #heat10:39
*** e0ne has quit IRC11:09
*** ttsiouts has quit IRC11:27
*** jtomasek has quit IRC11:37
*** jtomasek has joined #heat11:44
openstackgerritSurya Prakash (spsurya) proposed openstack/heat-agents master: dict_object.keys() is not required for *in* operator  https://review.openstack.org/62616011:48
*** e0ne has joined #heat12:01
*** jtomasek has quit IRC12:11
*** ttsiouts has joined #heat12:20
e0nehi team. is anybody familiar around with heat-dashboard?12:30
*** gfidente has quit IRC12:32
*** gfidente has joined #heat12:33
*** ttsiouts has quit IRC12:54
*** jtomasek has joined #heat12:57
*** ttsiouts has joined #heat12:57
*** jtomasek has quit IRC13:13
*** jtomasek has joined #heat13:13
*** jcoufal has joined #heat13:33
*** e0ne has quit IRC13:43
*** e0ne has joined #heat14:02
*** skramaja has quit IRC14:13
*** ssbarnea|rover has quit IRC14:40
*** ssbarnea has joined #heat14:41
openstackgerritLudovic Logiou proposed openstack/heat master: Rename tenant_id to project_id in octavia loadbalancer properties  https://review.openstack.org/62559714:42
*** fragatina has quit IRC15:41
*** jmlowe has joined #heat15:47
*** fragatina has joined #heat16:01
*** gfidente is now known as gfidente|afk16:24
*** jtomasek has quit IRC16:39
*** jistr has quit IRC16:41
*** jistr has joined #heat16:42
*** jistr_ has joined #heat16:46
*** jistr has quit IRC16:47
*** jistr_ has quit IRC16:51
*** jistr has joined #heat16:52
*** e0ne has quit IRC17:05
*** ttsiouts has quit IRC17:14
*** e0ne has joined #heat17:19
*** e0ne has quit IRC17:24
*** fragatina has quit IRC17:39
*** gkadam has quit IRC17:46
*** k_mouza_ has joined #heat17:52
*** k_mouza has quit IRC17:55
*** k_mouza_ has quit IRC17:56
*** k_mouza has joined #heat18:27
*** k_mouza has quit IRC18:32
*** shardy has quit IRC18:32
*** e0ne has joined #heat18:33
*** e0ne has quit IRC18:40
*** sshnaidm|off has quit IRC19:24
*** sshnaidm has joined #heat19:27
*** e0ne has joined #heat19:40
*** k_mouza has joined #heat20:10
*** jcoufal has quit IRC20:12
*** brtknr has quit IRC20:13
*** e0ne has quit IRC20:23
*** jmlowe has quit IRC20:31
*** ssbarnea has quit IRC21:00
*** k_mouza has quit IRC21:01
*** k_mouza has joined #heat21:29
*** k_mouza has quit IRC21:46
*** rcernin has joined #heat22:19
*** rcernin has quit IRC22:36
*** rcernin has joined #heat22:38
*** rcernin has quit IRC22:57
*** rcernin has joined #heat22:58
*** rcernin has quit IRC22:59
*** rcernin has joined #heat23:07
*** k_mouza has joined #heat23:10
*** jmlowe has joined #heat23:16
*** mchlumsky has quit IRC23:18
*** k_mouza has quit IRC23:26
*** k_mouza has joined #heat23:27
*** k_mouza has quit IRC23:28
*** k_mouza has joined #heat23:28
*** rcernin has quit IRC23:38
*** fragatina has joined #heat23:38
*** rcernin has joined #heat23:41
*** k_mouza has quit IRC23:43

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