Tuesday, 2020-09-08

*** zzzeek has quit IRC00:06
*** zzzeek has joined #heat00:08
*** zzzeek has quit IRC00:24
*** zzzeek has joined #heat00:29
*** zzzeek has quit IRC00:34
*** zzzeek has joined #heat00:35
*** elod has quit IRC00:35
*** elod has joined #heat00:36
*** hongbin has joined #heat01:28
*** hongbin_ has joined #heat01:46
*** dopplerg- has joined #heat01:47
*** dopplergange has quit IRC01:47
*** hoonetorg has quit IRC01:47
*** hongbin has quit IRC01:48
*** hoonetorg has joined #heat01:49
*** mikecmpbll has quit IRC02:37
*** mikecmpbll has joined #heat02:43
*** zzzeek has quit IRC02:47
*** zzzeek has joined #heat02:51
*** rcernin has quit IRC02:59
*** rcernin has joined #heat03:14
*** hongbin_ has quit IRC03:15
*** hongbin has joined #heat03:16
*** zzzeek has quit IRC03:19
*** zzzeek has joined #heat03:21
*** simondodsley has quit IRC03:22
*** simondodsley has joined #heat03:26
*** vishalmanchanda has joined #heat03:52
openstackgerritwu.shiming proposed openstack/python-heatclient master: Remove translation sections from setup.cfg  https://review.opendev.org/75027203:57
*** zzzeek has quit IRC04:03
*** zzzeek has joined #heat04:05
*** ramishra has quit IRC04:31
*** ramishra has joined #heat04:31
*** mikecmpbll has quit IRC04:41
*** mikecmpbll has joined #heat04:46
*** hongbin has quit IRC05:12
*** zzzeek has quit IRC06:02
*** zzzeek has joined #heat06:03
*** zzzeek has quit IRC06:22
*** zzzeek has joined #heat06:24
*** gfidente has joined #heat06:57
*** jtomasek has joined #heat06:58
*** tosky has joined #heat07:35
*** rcernin has quit IRC08:00
*** tosky has quit IRC08:05
*** tosky has joined #heat08:10
*** tosky has quit IRC08:14
*** tosky has joined #heat08:14
*** apetrich has joined #heat08:14
*** zzzeek has quit IRC08:16
*** zzzeek has joined #heat08:17
*** k_mouza has joined #heat08:26
*** zzzeek has quit IRC08:47
*** zzzeek has joined #heat08:50
openstackgerritwu.shiming proposed openstack/python-heatclient master: Remove six.PY2  https://review.opendev.org/75030109:35
*** tosky has quit IRC09:39
*** k_mouza has quit IRC10:16
*** zzzeek has quit IRC10:48
*** zzzeek has joined #heat10:50
*** k_mouza has joined #heat11:08
*** k_mouza has quit IRC11:14
*** k_mouza has joined #heat11:18
*** k_mouza has quit IRC11:18
openstackgerritZihao Wang proposed openstack/heat master: Remove babel.cfg  https://review.opendev.org/75035613:00
*** cliffparsons has joined #heat13:01
*** k_mouza has joined #heat13:09
*** gfidente has quit IRC13:12
openstackgerritZihao Wang proposed openstack/heat master: Remove babel.cfg  https://review.opendev.org/75035613:20
*** beekneemech is now known as bnemec13:34
*** ekultails has joined #heat13:37
*** gfidente has joined #heat13:37
openstackgerritLuke Short proposed openstack/heat stable/ussuri: Lazily cache parsed value of list/json parameters  https://review.opendev.org/75036313:40
zanebgregwork: your template looks ok to me, I'd have expected that to work14:22
gregworkits coming through the other end as standalone-ansible-zzz132451/Stanadlone  with this14:24
gregwork# [{"network_config": [{"addresses": [{"ip_netmask": {"list_join": ["/", [{"get_param": "ControlPlaneIp"}, {"get_param": "ControlPlaneSubnetCidr"}]]}}], "dns_servers": {"ge14:24
gregworkt_param": "DnsServers"}, "domain": {"get_param": "DnsSearchDomains"}, "members": [{"mtu": {"get_param": "InterfaceLocalMtu"}, "name": "interface_name", "primary": true, "ty14:24
gregworkpe": "interface"}, {"mtu": {"get_param": "InterfaceLocalMtu"}, "type": "vlan", "vlan_id": {"get_param": "AioManagementNetworkVlanID"}}], "mtu": {"get_param": "InterfaceLoca14:24
gregworklMtu"}, "name": "br-ctlplane", "ovs_extra": ["br-set-external-id br-ctlplane bridge-id br-ctlplane"], "routes": {"get_param": "ControlPlaneStaticRoutes"}, "type": "ovs_brid14:24
gregworkge", "use_dhcp": false}]}] : the json serialized os-net-config config to apply14:24
gregwork#14:24
gregworkthose should be resolved when it ends up here14:25
gregworkim going to try adding the block to the net-config-standalone.j2.yaml directly .. this should work, but it kind of makes me wonder how anyone was going to use the override14:26
*** k_mouza has quit IRC14:27
zanebgregwork: wait, are you saying that the parameter value contains intrinsic functions like "list_join" &c.?14:28
gregworkok so let me start from the beginning .. during my stack deploy i am passing an environment file that contains StandaloneNetConfigOverride and  AioManagementNetworkVlanID14:29
gregworkthe former contains that big block and the latter is a simple integer14:29
gregworknet-config-standalone.yaml has a if statement that says if standalonenetconfigoveride is defined, then use that, else use what is hardcoded in the template14:30
gregworkmy environment file does define it successfully, but the content of that override is not being interpolated14:30
gregworkand its being written literally14:30
gregworkso "get_param" link list all of that14:31
gregworkhttp://paste.openstack.org/show/797589/14:33
gregworkthose are more or less the moving parts14:34
gregworkim guessing i cant pass StandaloneNetConfigOvrride from that high up ?14:34
gregworkor maybe i cant pass dynamic variables down14:36
gregworkor blocks containing strings that should be interpreted dynamically when read14:36
gregworkmaybe that last one is it .. but i yield to the masters for clarity14:37
gregwork:P14:37
zanebgregwork: ok, now I understand. the result of get_param is always a literal value. there's no way to force it to be interpreted as if it were part of the template14:41
gregworkok so this is just not a thing, but based on the comments from that template... how is this ever going to be used14:42
gregworkhttps://github.com/openstack/tripleo-heat-templates/blob/master/net-config-standalone.j2.yaml#L77-L8314:42
gregworklike its a great idea ..14:42
*** tkajinam has quit IRC14:43
gregworkhowever outside of editing the template directly with my block i dont think this can be used as intended14:43
gregworkand editing the template defeats the purpose because i could just edit the content of the if statement14:43
gregworkand im trying to not modify the sources14:43
zanebif you want a custom config you just pass it14:43
gregworkbut how do i pass data to StandaloneNetConfigOverride14:44
gregworkwhere things are interpreted14:44
gregworkor are we just confirming that is not possible and the only network config i can pass is one that doesnt need to be looked up/resolved14:46
zanebyou don't, you have to calculate the config you want. does it depend on data produced by a higher-level stack? in that case I can see what you mean about it being pretty useless14:46
gregworkzaneb: right it does depend on things set at the higher level14:47
gregworkfor my use case i can write it out as http://paste.openstack.org/show/797591/14:49
gregworkbut thats crap14:49
gregworksince all of that was defined in other templates above it14:49
zanebwhich part are you trying to override?14:49
gregworkwell the whole default block in the https://github.com/openstack/tripleo-heat-templates/blob/master/net-config-standalone.j2.yaml#L115-L15614:50
zaneboh, it's the extra vlan member?14:51
gregworkyeah im trying to use vlans in my config14:51
zanebI don't think there's any way to do what you want :(14:51
*** k_mouza has joined #heat14:51
gregworkyou agree i could write out resolved block and use that though14:52
gregworki just cant use anything that needs to be resolved/looked up as the intrinsic would be passed literally14:52
zanebyes, that would work but obviously could get out of sync with any IP addresses that are generated inside the Heat templates14:53
zanebcorrect14:53
gregworkyeah its less than ideal14:53
gregworkis this kind of a use case rfe worthy ?14:53
gregworkeg. some mechanism to pass blocks which contain intrinsics which get resolved14:54
zanebit'd be worth raising this with the tripleo team. if they know what customisations people are needing, they might be willing to add them to the templates14:54
gregworkwell that would be fixing the template, im talking about enhancing heat with new capability :)14:55
zaneblike an exec function? it's an interesting idea14:55
zanebnot sure it's actually possible, I'd have to think about it a bit more14:55
gregworkyeah14:59
zanebit'd be interesting if it could help out with the case where you want to pass a calculation with %index% as a parameter to a member of a ResourceGroup. I don't think a straight exec would be enough to solve that though15:00
gregworkso we could: This: { interpret: { get_param: that } }15:00
zanebdoesn't hurt to raise an issue15:01
gregworkon the main heat project ?15:02
*** k_mouza has quit IRC15:03
zanebyep15:05
*** k_mouza has joined #heat15:10
*** mailingsam has joined #heat15:16
*** k_mouza has quit IRC15:29
gmannzaneb: ricolin can you review this, needed for Focal migration and to avoid break gate when base tox jobs will be switched to Focal  https://review.opendev.org/#/c/750080/15:29
*** jmlowe has quit IRC15:30
*** jmlowe has joined #heat15:32
*** k_mouza has joined #heat15:41
zanebgmann: approved16:01
gmannthanks16:01
*** k_mouza has quit IRC16:06
*** k_mouza has joined #heat16:08
*** k_mouza has quit IRC16:18
*** vishalmanchanda has quit IRC16:21
*** k_mouza has joined #heat16:24
*** gfidente has quit IRC16:25
*** k_mouza has quit IRC16:30
openstackgerritMerged openstack/python-heatclient master: [goal] Migrate testing to ubuntu focal  https://review.opendev.org/75008016:33
*** bnemec has quit IRC16:52
*** bnemec has joined #heat17:02
*** jtomasek has quit IRC17:04
openstackgerritLuke Short proposed openstack/heat stable/train: Lazily cache parsed value of list/json parameters  https://review.opendev.org/75046917:58
*** mailingsam has quit IRC18:15
*** apetrich has quit IRC19:00
*** cliffparsons has quit IRC19:06
*** _erlon_ has joined #heat19:31
*** cliffparsons has joined #heat19:32
openstackgerritSam Kumar proposed openstack/heat master: Adds backups and backups_gigabytes parameters  https://review.opendev.org/75048119:51
*** zzzeek has quit IRC20:16
*** zzzeek has joined #heat20:17
*** mikecmpbll has quit IRC20:54
*** ekultails has quit IRC21:12
*** _erlon_ has quit IRC22:56
*** tkajinam has joined #heat22:57
*** tkajinam has quit IRC22:57
*** tkajinam has joined #heat22:58
*** rcernin has joined #heat23:02
openstackgerritGhanshyam Mann proposed openstack/heat-tempest-plugin master: [goal] Migrate heat-tempest-plugin jobs to focal  https://review.opendev.org/74310623:15
openstackgerritGhanshyam Mann proposed openstack/heat-tempest-plugin master: [goal] Migrate tox based testing to Focal  https://review.opendev.org/74310623:20
*** zzzeek has quit IRC23:33
*** zzzeek has joined #heat23:35

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!