*** gridinv has joined #heat | 00:09 | |
*** pmannidi_2 has joined #heat | 00:10 | |
*** SmearedBeard has quit IRC | 00:14 | |
*** liyi has quit IRC | 00:16 | |
*** liyi has joined #heat | 00:17 | |
*** hieulq has quit IRC | 00:30 | |
*** hieulq has joined #heat | 00:31 | |
*** gouthamr has quit IRC | 00:31 | |
*** yuanying has joined #heat | 00:32 | |
*** blomquisg is now known as blomquisg_gone | 00:32 | |
*** yangyapeng has quit IRC | 00:37 | |
*** askb has quit IRC | 00:46 | |
*** askb has joined #heat | 00:49 | |
*** liyi has quit IRC | 00:51 | |
*** liyi has joined #heat | 00:52 | |
*** askb has quit IRC | 00:53 | |
*** askb has joined #heat | 00:54 | |
*** gouthamr has joined #heat | 01:10 | |
*** liyi has quit IRC | 01:15 | |
*** liyi has joined #heat | 01:15 | |
*** liyi_ has joined #heat | 01:26 | |
*** liyi has quit IRC | 01:30 | |
*** hongbin has quit IRC | 01:35 | |
openstackgerrit | Merged openstack/heat master: Remove non-updatable attrs from SFC update test https://review.openstack.org/517649 | 01:46 |
---|---|---|
*** wangbo has joined #heat | 02:01 | |
*** kiennt26 has joined #heat | 02:09 | |
*** yangyapeng has joined #heat | 02:24 | |
openstackgerrit | Merged openstack/python-heatclient master: Add --no-rollback option for stack cancel https://review.openstack.org/497011 | 02:29 |
openstackgerrit | Kien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job https://review.openstack.org/510400 | 02:29 |
*** fragatina has quit IRC | 02:30 | |
*** ricolin has joined #heat | 02:35 | |
*** ramishra has joined #heat | 02:39 | |
*** namnh has joined #heat | 02:45 | |
*** liyi_ has quit IRC | 02:53 | |
*** kazsh has left #heat | 03:00 | |
*** liyi has joined #heat | 03:02 | |
*** liyi has quit IRC | 03:04 | |
*** links has joined #heat | 03:04 | |
*** liyi has joined #heat | 03:04 | |
*** liyi has quit IRC | 03:05 | |
*** fragatina has joined #heat | 03:21 | |
*** fragatina has quit IRC | 03:25 | |
*** jrist has quit IRC | 03:29 | |
*** om2 has joined #heat | 03:45 | |
*** jrist has joined #heat | 03:53 | |
*** wangbo has quit IRC | 04:10 | |
*** wangbo has joined #heat | 04:12 | |
*** dbecker has quit IRC | 04:23 | |
openstackgerrit | Merged openstack/python-heatclient master: Allow cancelling create_in_progress stacks with --no-rollback https://review.openstack.org/497112 | 04:31 |
*** dbecker has joined #heat | 04:38 | |
*** hieulq has quit IRC | 04:42 | |
openstackgerrit | Merged openstack/heat master: Cleanup ROLLBACK_COMPLETE stack after the test finishes https://review.openstack.org/515300 | 04:42 |
*** hieulq has joined #heat | 04:42 | |
openstackgerrit | Rico Lin proposed openstack/heat master: Remove setting of version/release from releasenotes https://review.openstack.org/520885 | 04:52 |
*** wangbo has quit IRC | 04:53 | |
openstackgerrit | Rico Lin proposed openstack/heat-dashboard master: Remove setting of version/release from releasenotes https://review.openstack.org/520889 | 04:57 |
*** kiennt26 has quit IRC | 05:01 | |
*** fragatina has joined #heat | 05:12 | |
*** fragatina has quit IRC | 05:14 | |
*** fragatina has joined #heat | 05:15 | |
*** wangbo has joined #heat | 05:16 | |
*** yangyapeng has quit IRC | 05:24 | |
*** gouthamr has quit IRC | 05:34 | |
*** skramaja has joined #heat | 05:36 | |
openstackgerrit | Merged openstack/heat master: Add functional tests for stack cancel https://review.openstack.org/499550 | 05:38 |
*** yangyapeng has joined #heat | 05:52 | |
*** yangyapeng has quit IRC | 05:56 | |
*** wangbo has quit IRC | 06:05 | |
*** yangyapeng has joined #heat | 06:10 | |
*** kiennt26 has joined #heat | 06:14 | |
*** yangyapeng has quit IRC | 06:15 | |
*** armaan has joined #heat | 06:29 | |
*** yangyapeng has joined #heat | 06:36 | |
*** yangyapeng has quit IRC | 06:41 | |
*** maestropandy has joined #heat | 06:42 | |
*** maestropandy has quit IRC | 06:45 | |
*** pmannidi_2 has quit IRC | 06:47 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/heat master: Updated from global requirements https://review.openstack.org/520926 | 06:47 |
*** yangyapeng has joined #heat | 06:51 | |
*** armaan has quit IRC | 07:01 | |
*** armaan has joined #heat | 07:03 | |
*** armaan_ has joined #heat | 07:06 | |
*** armaan has quit IRC | 07:09 | |
openstackgerrit | Rabi Mishra proposed openstack/heat master: Refactor CooldownMixin https://review.openstack.org/459595 | 07:13 |
openstackgerrit | Rabi Mishra proposed openstack/heat master: Remove use of CooldownMixin with scaling policy https://review.openstack.org/294023 | 07:13 |
openstackgerrit | Rabi Mishra proposed openstack/heat master: Move CooldownMixin tests to group test modules https://review.openstack.org/459596 | 07:13 |
*** maestropandy has joined #heat | 07:16 | |
*** rcernin has quit IRC | 07:18 | |
*** namnh has quit IRC | 07:27 | |
*** ioggstream has joined #heat | 07:28 | |
*** maestropandy has quit IRC | 07:34 | |
*** maestropandy has joined #heat | 07:39 | |
openstackgerrit | Rabi Mishra proposed openstack/heat master: Remove ceilometer from the dsvm jobs https://review.openstack.org/520993 | 07:42 |
*** hjensas has quit IRC | 07:55 | |
*** AlexeyAbashkin has joined #heat | 08:01 | |
*** ioggstream has quit IRC | 08:07 | |
*** maestropandy has quit IRC | 08:11 | |
*** SmearedBeard has joined #heat | 08:15 | |
*** Smeared_Beard has joined #heat | 08:20 | |
*** SmearedBeard has quit IRC | 08:21 | |
*** wangbo has joined #heat | 08:22 | |
*** SmearedBeard has joined #heat | 08:23 | |
*** Smeared_Beard has quit IRC | 08:27 | |
*** armaan has joined #heat | 08:27 | |
*** amoralej|off is now known as amoralej | 08:27 | |
*** jmlowe has quit IRC | 08:27 | |
*** SmearedBeard has quit IRC | 08:29 | |
*** armaan_ has quit IRC | 08:30 | |
*** SmearedBeard has joined #heat | 08:36 | |
*** SmearedBeard has quit IRC | 08:43 | |
*** ccamacho has joined #heat | 08:49 | |
*** maestropandy has joined #heat | 08:53 | |
*** ricolin has quit IRC | 08:54 | |
*** hjensas has joined #heat | 09:02 | |
openstackgerrit | Rabi Mishra proposed openstack/heat-tempest-plugin master: Return Environment from validate_template https://review.openstack.org/521004 | 09:03 |
openstackgerrit | Rabi Mishra proposed openstack/heat-tempest-plugin master: Cleanup ROLLBACK_COMPLETE stack after the test finishes https://review.openstack.org/521005 | 09:03 |
openstackgerrit | Rabi Mishra proposed openstack/heat-tempest-plugin master: Add functional tests for stack cancel https://review.openstack.org/521006 | 09:03 |
chandankumar | ramishra: thanks for doing the heat tempest plugin work :-) | 09:04 |
ramishra | chandankumar: np:) we've to sync the repos before using the plugin heat dsvm jobs | 09:05 |
chandankumar | ramishra: we need one more help, please package it in RDO, https://review.rdoproject.org/r/#/q/topic:add-zaqar-tempest-plugin so that we can consume it in CI | 09:06 |
chandankumar | ramishra: feel free to add me as a Reviewer it will get it done asap as it lands in RDO | 09:06 |
ramishra | chandankumar: OK | 09:07 |
ltomasbo | Hi guys, I'm having problems with the 'repeat - for_each' at a heat template | 09:10 |
ltomasbo | when not using permutations | 09:10 |
ltomasbo | is there any known limitation with that? | 09:10 |
ltomasbo | this is the error I'm seeing: ERROR: Property error: : resources.trunk_port.properties.sub_ports: : For repeat, the length of for_each values should be equal if no nested loop. | 09:10 |
*** dbecker has quit IRC | 09:10 | |
*** shardy has joined #heat | 09:11 | |
ltomasbo | but if I remove the permutations:false, I see the output of both variables here http://paste.openstack.org/show/626583/ being of the same length | 09:11 |
ltomasbo | any clue/tip? | 09:11 |
*** wangbo has quit IRC | 09:18 | |
*** derekh has joined #heat | 09:20 | |
*** wangbo has joined #heat | 09:22 | |
*** wangbo has quit IRC | 09:24 | |
therve | ramishra, So still looking at https://bugs.launchpad.net/tripleo/+bug/1731032 | 09:27 |
openstack | Launchpad bug 1731032 in tripleo "CI: Deployment fails on controller timeout with task creation in heat timing out" [Critical,Triaged] | 09:27 |
therve | ramishra, Now pretty convinced the issue is in https://github.com/openstack/heat/blob/master/heat/engine/service_software_config.py#L100 | 09:27 |
therve | I don't know how yet, though | 09:27 |
therve | I wonder about https://review.openstack.org/#/c/321783/ | 09:30 |
ramishra | therve: that's merged 7 months back, you mean the issue comes with convergence being used? | 09:31 |
therve | I don't know | 09:32 |
*** gkadam has joined #heat | 09:32 | |
therve | I'm wondering about that session.refresh, it's unclear what it does | 09:32 |
therve | Maybe we miss it on other things, like expiring the deployments | 09:35 |
*** kiennt26 has quit IRC | 09:39 | |
therve | ramishra, We do a subtransaction in resource_update, but we never expire the session in retry_on_conflict | 09:43 |
therve | It's possible that we get stale objects there, no? | 09:43 |
ramishra | therve: AFAIK we don't have to expire session when using transactions, it happens with the transaction | 09:44 |
therve | ramishra, Right, but we do a new transaction? | 09:45 |
ramishra | therve: which part of the code you mean? | 09:45 |
therve | ramishra, In _push_metadata_software_deployments. It gets called a second time after a conflict. Will it retrieve the "new" software deployments properly? | 09:46 |
therve | The fact that we have to explicitly call refresh on the resource objects makes me pause | 09:47 |
ramishra | therve: refresh also commits everything and refreshes the orm layer, no? | 09:48 |
therve | ramishra, AFAIK it just does the second part | 09:48 |
ltomasbo | q | 09:49 |
ramishra | therve: possible there is an issue, but if we can reproduce it in some way | 09:51 |
therve | In theory we don't need a real server | 09:52 |
therve | Just a test that hammers metadata | 09:52 |
therve | Though that's what ParallelDeploymentsTest is supposed to do I think | 09:53 |
*** wangbo has joined #heat | 09:55 | |
*** maestropandy has quit IRC | 09:57 | |
therve | I wonder if the depends_on make a difference | 09:57 |
*** SmearedBeard has joined #heat | 09:59 | |
*** maestropandy has joined #heat | 10:18 | |
*** ricolin has joined #heat | 10:23 | |
*** fragatin_ has joined #heat | 10:25 | |
*** fragatina has quit IRC | 10:27 | |
*** wangbo has quit IRC | 10:28 | |
*** ricolin has quit IRC | 10:31 | |
openstackgerrit | Rabi Mishra proposed openstack/heat master: Use heat-tempest-plugin for integration tests https://review.openstack.org/508112 | 10:31 |
*** ricolin has joined #heat | 10:39 | |
*** wangbo has joined #heat | 10:48 | |
*** dbecker has joined #heat | 10:54 | |
ramishra | therve: looking at it again, Isn't that refresh only for the resource row(for the concurrent transactions)? why do youu think deployments would be missed? | 11:01 |
therve | ramishra, Well because of that, just the resource row is refreshed | 11:11 |
therve | If we have deployments not known for some reasons, they are not refreshed | 11:12 |
ramishra | therve: Isn't sd create and update happening in transactions? ex. https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/api.py#L1180 | 11:18 |
therve | ramishra, Sure | 11:20 |
*** maestropandy has quit IRC | 11:20 | |
therve | Not sure how's that related | 11:20 |
ramishra | therve: I mean we do a list fo deployments in the retry https://github.com/openstack/heat/blob/master/heat/engine/service_software_config.py#L108 | 11:22 |
ramishra | may be I'm missing your point | 11:22 |
therve | If I knew the issue I'd fix it :) | 11:23 |
therve | From what I saw, we do 2 swift PUT in a row, and the second one contains less deployments than the first one | 11:24 |
ramishra | therve: Oh! that's strange then | 11:24 |
therve | I don't see how it can only come from the listing being different | 11:25 |
therve | Hum, rather, "it can only come from the listing being different" | 11:26 |
therve | We fetch all of them each time | 11:26 |
ramishra | therve: yeah | 11:26 |
ramishra | I'm not sure how the list can be different for retry | 11:27 |
therve | Me neither, but there is a lot about sqlalchemy I don't understand | 11:28 |
ramishra | unless they are commited | 11:28 |
therve | Maybe we can ask zzzeek for some help | 11:30 |
ramishra | therve: yep, sounds good:) our db code is not great I think | 11:31 |
therve | No, no it's not... | 11:31 |
therve | ramishra, We could do something like filtering on atomic_key in software_deployment_get_all | 11:32 |
therve | With a join on resource | 11:32 |
ramishra | yeah probably worth try, increase the complexity though:) | 11:33 |
therve | Yeah... | 11:34 |
therve | Until we have a reproducer it's probably not worth it | 11:35 |
*** armaan has quit IRC | 11:39 | |
openstackgerrit | Thomas Herve proposed openstack/heat master: Add some debug info in sd metadata setting https://review.openstack.org/521064 | 11:41 |
*** hjensas has quit IRC | 11:55 | |
*** wangbo has quit IRC | 11:58 | |
*** wangbo has joined #heat | 11:58 | |
*** ramishra has quit IRC | 12:08 | |
*** dbecker has quit IRC | 12:08 | |
*** pbourke has quit IRC | 12:10 | |
*** aluria has joined #heat | 12:10 | |
*** pbourke has joined #heat | 12:11 | |
*** shardy is now known as shardy_lunch | 12:22 | |
*** ricolin has quit IRC | 12:31 | |
*** wangbo has quit IRC | 12:43 | |
*** clayton has quit IRC | 12:44 | |
*** yangyapeng has quit IRC | 12:45 | |
*** yangyapeng has joined #heat | 12:46 | |
*** yangyapeng has quit IRC | 12:48 | |
*** yangyapeng has joined #heat | 12:49 | |
*** yangyapeng has quit IRC | 12:53 | |
openstackgerrit | Thomas Herve proposed openstack/heat master: Add some debug info in sd metadata setting https://review.openstack.org/521064 | 12:56 |
*** om2 has quit IRC | 12:58 | |
*** kazsh has joined #heat | 12:59 | |
*** ramishra has joined #heat | 13:00 | |
*** skramaja has quit IRC | 13:09 | |
*** dprince has joined #heat | 13:10 | |
*** ramishra has quit IRC | 13:17 | |
*** shardy_lunch is now known as shardy | 13:18 | |
*** links has quit IRC | 13:19 | |
*** ramishra has joined #heat | 13:22 | |
*** hieulq has quit IRC | 13:28 | |
*** clayton has joined #heat | 13:29 | |
*** clayton has quit IRC | 13:38 | |
*** clayton has joined #heat | 13:39 | |
*** ioggstream has joined #heat | 14:03 | |
*** maestropandy has joined #heat | 14:07 | |
*** maestropandy has quit IRC | 14:07 | |
*** apuimedo has joined #heat | 14:07 | |
*** dulek has joined #heat | 14:09 | |
*** dulek has left #heat | 14:09 | |
*** dbecker has joined #heat | 14:10 | |
*** gouthamr has joined #heat | 14:10 | |
therve | ramishra, http://logs.openstack.org/64/521064/2/check/heat-functional-convg-mysql-lbaasv2/4b57e71/logs/screen-h-eng.txt.gz shows some weird stuff | 14:12 |
*** chandankumar is now known as chkumar|somewher | 14:13 | |
therve | It goes 7, 8, 10 deployments, and then 8 again | 14:13 |
therve | Though I don't know if it worked or not, I should probably move the debug | 14:13 |
openstackgerrit | Thomas Herve proposed openstack/heat master: Add some debug info in sd metadata setting https://review.openstack.org/521064 | 14:17 |
ltomasbo | ping zaneb, I'm having some problems with some heat templates: http://paste.openstack.org/show/626617/ | 14:21 |
ltomasbo | zaneb, do you know why get_attr is being parse before the port (subports) are created? | 14:22 |
zaneb | because that's how functions work | 14:23 |
ltomasbo | should wait (like depends on) when you have get_attr? | 14:23 |
ltomasbo | zaneb, ^^ | 14:23 |
ltomasbo | zaneb, funny thing is that (for 1 port and seg_id) it works if I remove the permutations: false, but it does not when it is there | 14:24 |
ltomasbo | as the first time it checks both variables have the same length, it fails | 14:24 |
ltomasbo | zaneb, is there a way of waiting until the subports ids are ready? | 14:25 |
zaneb | oh wait, sorry, I misunderstood the question | 14:26 |
zaneb | the trunk_port won't be created until after the subports, and it's properties won't be resolved until that time | 14:27 |
ltomasbo | zaneb, btw, do you live in Umea? | 14:27 |
zaneb | no, US | 14:27 |
ltomasbo | ahh, ok, I saw you are connected from there, and I was living there for 4 years! | 14:28 |
ltomasbo | anyway, getting back to the trunks, that is not what I'm getting | 14:28 |
ltomasbo | I'm getting this error: | 14:29 |
ltomasbo | ERROR: Property error: : resources.trunk_port.properties.sub_ports: : For repeat, the length of for_each values should be equal if no nested loop. | 14:29 |
ltomasbo | and I see it comes from here: https://github.com/openstack/heat/blob/master/heat/engine/hot/functions.py#L945-L959 | 14:30 |
ltomasbo | zaneb, ^ | 14:30 |
zaneb | I wonder if seg_ids is not being treated as a list | 14:31 |
zaneb | I would suggest changing the type of seg_ids to json | 14:31 |
ltomasbo | zaneb, the problem is with the port_ids | 14:31 |
ltomasbo | I printed the values at that checking | 14:31 |
ltomasbo | and I got this: | 14:31 |
ltomasbo | [[u'101'], []] | 14:32 |
ltomasbo | so, the seg_ids are there, the problem is the missing port_ids | 14:32 |
ltomasbo | actually, if I remove the raise exception at that point, I see there is a re-try and it ends up getting the port_ids properly | 14:33 |
*** ramishra has quit IRC | 14:34 | |
zaneb | is the subports group getting created? | 14:34 |
zaneb | I wonder if it's failing during validation and not actually starting to create the stack | 14:34 |
ltomasbo | well, if I don't remove the raise exception, the stack is not even created | 14:35 |
ltomasbo | so, yes, it is validation probably | 14:35 |
zaneb | ok, so it is failing during validation | 14:35 |
ltomasbo | if I remove the permutations, then it works (as it pass that validation) | 14:35 |
*** ramishra has joined #heat | 14:35 | |
ltomasbo | zaneb, great, do you know where I can look at that? (I'm no expect on heat) | 14:36 |
ltomasbo | zaneb, I mean, is there an easy way of skipping validations? | 14:36 |
zaneb | there is not | 14:36 |
ltomasbo | ok | 14:37 |
zaneb | # use empty list for references(None) else validation will fail | 14:37 |
zaneb | so that ^ was an incomplete bug fix | 14:37 |
zaneb | because if some of the lists are none and some are not, it will fail with the unequal lengths thing | 14:38 |
ltomasbo | yep, but how does it work in general for that type of validations? | 14:39 |
ltomasbo | or is this something just affecting Repeat actions? | 14:39 |
zaneb | it's specific to Repeat | 14:39 |
ltomasbo | I guess there are many templates with get_attr that gets their values on runtime | 14:39 |
zaneb | in general get_attr returns None during validation and every function has to handle it correctly | 14:39 |
zaneb | needless to say there are numerous bugs | 14:40 |
zaneb | we do have a long-term plan to improve it | 14:40 |
zaneb | but for now it's whack-a-mole | 14:40 |
ltomasbo | ok, one solution could be to skip length validation if get_attr is used (for repeat)? | 14:41 |
zaneb | so here is one thing you can try right now... | 14:41 |
ltomasbo | or use two get_attr... | 14:41 |
zaneb | create an OS::Heat::Value resource that does {get_param: seg_ids} and then use get_attr on that resource to get the seg_ids in your trunk port | 14:42 |
zaneb | in theory that way both values should be None during validation and it will work | 14:42 |
zaneb | obviously we should also raise a bug | 14:44 |
ltomasbo | ohh, nice trick! | 14:45 |
ltomasbo | thanks! | 14:45 |
*** serlex has joined #heat | 14:46 | |
*** serlex has left #heat | 14:46 | |
*** hongbin has joined #heat | 14:48 | |
*** ricolin has joined #heat | 14:50 | |
*** dbecker has quit IRC | 15:10 | |
*** dbecker has joined #heat | 15:10 | |
apuimedo | ltomasbo: did it work? | 15:16 |
ltomasbo | apuimedo, not yet: ERROR: resources.trunk_port.properties.sub_ports.repeat.for_each.<%seg_id%>.get_attr: : Argument to "get_attr" must be a list | 15:17 |
ltomasbo | but that must be me not setting it properly... | 15:17 |
apuimedo | :-) | 15:17 |
zaneb | ltomasbo: {getattr: [value_resource, value]} | 15:19 |
ltomasbo | ohh, right, I forgot the [] | 15:19 |
zaneb | ltomasbo: https://bugs.launchpad.net/heat/+bug/1732934 | 15:28 |
openstack | Launchpad bug 1732934 in OpenStack Heat "Validation of non-permuted repeat function fails with mixed data sources" [Medium,Triaged] | 15:28 |
*** ramishra has quit IRC | 15:28 | |
zaneb | patches welcome ;) | 15:28 |
ltomasbo | zaneb, thanks! | 15:28 |
ltomasbo | umm, tested it, and it passed the validation, but failed the deployment | 15:29 |
zaneb | doh! | 15:30 |
zaneb | what was the error? | 15:30 |
ltomasbo | TypeError: The values of the "for_each" argument to "repeat" must be lists | 15:30 |
zaneb | ltomasbo: try setting the 'type' property of the value resource to comma_delimited_list or json https://docs.openstack.org/heat/latest/template_guide/openstack.html#OS::Heat::Value-prop-type | 15:32 |
ltomasbo | zaneb, I have this: | 15:33 |
ltomasbo | http://paste.openstack.org/show/626624/ | 15:34 |
ltomasbo | umm, it is getting the wrong value: output_value: test_trunks-segmentation-kzu32upe2kre-segmentation_ids-6i2fc37hqpl2 | 15:36 |
ltomasbo | I did something wrong | 15:36 |
zaneb | output value should be {get_attr: [segmentation_ids, value]} not get_resource | 15:36 |
ltomasbo | yep, that should be it | 15:37 |
*** ramishra has joined #heat | 15:38 | |
ltomasbo | zaneb, apuimedo \o/ | 15:39 |
ltomasbo | that was it! thanks! | 15:39 |
apuimedo | I've been summoned | 15:40 |
*** derekh has quit IRC | 15:40 | |
apuimedo | what? | 15:40 |
apuimedo | it works now? | 15:40 |
*** ramishra has quit IRC | 15:43 | |
ltomasbo | apuimedo, yes! | 15:44 |
*** om2 has joined #heat | 15:45 | |
*** ramishra has joined #heat | 15:45 | |
apuimedo | zaneb: ltomasbo owes you a beer for sure :P | 15:45 |
apuimedo | great! | 15:45 |
ltomasbo | yep! I do! | 15:46 |
*** ayoung has quit IRC | 15:48 | |
*** ayoung has joined #heat | 15:49 | |
zaneb | apuimedo: I think we owe y'all a beer for letting that bug creep in there ;) | 15:55 |
apuimedo | :-) | 15:55 |
*** crose has joined #heat | 15:58 | |
ricolin | therve, can you help to review https://review.openstack.org/#/q/topic:bp/policy-in-code+project:openstack/heat if you have time:) | 16:05 |
*** AlexeyAbashkin has quit IRC | 16:05 | |
therve | ricolin, I might :) | 16:06 |
ricolin | therve, might it good enough for me:) | 16:06 |
zaneb | therve: I really thought we had resolved all the races with metadata updates for sw deployments :/ | 16:11 |
therve | zaneb, Yeah wishful thinking apparently | 16:11 |
ltomasbo | zaneb, all, is there a way to generate a list of numbers? for instance, I want to pass value 5, and get a list = [1,2,3,4,5]? | 16:15 |
ltomasbo | I've tried with index and count, but I need the index to not start on 0, is that also possible? | 16:16 |
zaneb | there's no non-horrible way to do generate a list like that | 16:16 |
ltomasbo | :( | 16:17 |
ltomasbo | do you have a link to one of such horrible ways :) | 16:17 |
zaneb | the horrible way would be to use a resource group of OS::Heat::Value resources and index substitution | 16:17 |
ltomasbo | I imagine it would be with HeatRandom or nested templates? | 16:17 |
zaneb | you can make it 1-based instead of 0-based by blacklisting index 0 | 16:18 |
ricolin | ltomasbo, another way is like this example https://github.com/openstack/heat-templates/blob/master/hot/resource_group/resource_group_index_lookup.yaml | 16:19 |
ltomasbo | ricolin, but then you need to have the random_lengths defined, right? | 16:19 |
ltomasbo | in my case if to asign the vlan ids for the subports of the trunk, and the values are between 1 and 4095 | 16:20 |
ltomasbo | so, I would like to avoid having a default: [1,2,3,4,5,....4094,4095] | 16:20 |
ltomasbo | ricolin, zaneb: blacklisting index 0 is something like: removal_policies: [ { "resource_list": ["0"] } ] | 16:21 |
zaneb | yes | 16:21 |
ricolin | ltomasbo, yes, like zaneb said, there's no non-horrible way for now | 16:21 |
zaneb | something like that :) | 16:21 |
ltomasbo | ok, thanks! going to try! | 16:21 |
therve | ltomasbo, To generate a list of numbers, you can use yaql | 16:22 |
therve | Not sure that answers your question? | 16:22 |
ltomasbo | umm, that sounds interesting | 16:22 |
*** ccamacho has quit IRC | 16:23 | |
ltomasbo | not sure how that works, but going to take a look! | 16:23 |
ltomasbo | that could be useful! thanks therve | 16:23 |
therve | It can only be 200 items in default config though | 16:23 |
ltomasbo | :( | 16:23 |
therve | So no 4096 lengths elements | 16:23 |
ltomasbo | why 200 limitation? | 16:24 |
therve | Because memory abuse | 16:24 |
ltomasbo | ok | 16:24 |
zaneb | oh yeah, I always forget about yaql | 16:24 |
ltomasbo | so, perhaps it is easier then to just skip the initial one ( and increase the number by 1) | 16:24 |
ltomasbo | though, for now, 200 per trunk should be enough for me | 16:25 |
*** f13o has joined #heat | 16:28 | |
*** shardy has quit IRC | 16:38 | |
*** jistr has quit IRC | 16:39 | |
*** shardy has joined #heat | 16:39 | |
*** gkadam has quit IRC | 16:41 | |
*** gkadam has joined #heat | 16:41 | |
*** jistr has joined #heat | 16:43 | |
*** Guest95810 has quit IRC | 16:44 | |
therve | zaneb, Any idea otherwise what might be the problem? | 16:53 |
zaneb | therve: not really. the atomic key + retry stuff looks bulletproof to me | 16:54 |
therve | zaneb, Do you remembe why the session.refresh was necessary? | 16:54 |
zaneb | and I know Crag tested it pretty had because we had problems before | 16:54 |
zaneb | yeah, it turned out there was stale data in the session cache iirc | 16:55 |
therve | So I wonder if that can happen with deployment objects | 16:56 |
zaneb | tests were failing without that: https://review.openstack.org/#/c/321783/4..5/heat/engine/service_software_config.py | 16:58 |
*** f13o has quit IRC | 17:01 | |
*** gmoro has quit IRC | 17:03 | |
*** gmoro_ has joined #heat | 17:03 | |
*** gridinv has quit IRC | 17:06 | |
*** gkadam has quit IRC | 17:08 | |
*** ioggstream has quit IRC | 17:13 | |
*** om2 has quit IRC | 17:20 | |
*** shardy has quit IRC | 17:23 | |
*** shardy has joined #heat | 17:24 | |
*** shardy has quit IRC | 17:25 | |
*** dbecker has quit IRC | 17:26 | |
*** ricolin has quit IRC | 18:01 | |
*** mugsie has joined #heat | 18:07 | |
*** ramishra has quit IRC | 18:20 | |
*** hjensas has joined #heat | 18:59 | |
*** fragatin_ has quit IRC | 19:22 | |
openstackgerrit | Zane Bitter proposed openstack/heat master: Retry on DB deadlock when updating resource https://review.openstack.org/521170 | 19:24 |
*** amoralej is now known as amoralej|off | 19:34 | |
*** gouthamr has quit IRC | 19:35 | |
*** dprince has quit IRC | 19:48 | |
*** fragatina has joined #heat | 19:54 | |
*** dprince has joined #heat | 20:41 | |
zaneb | therve: I think we might have been looking in the wrong place: https://bugs.launchpad.net/heat/+bug/1731032/comments/15 | 20:48 |
openstack | Launchpad bug 1731032 in tripleo "CI: Deployment fails on controller timeout with task creation in heat timing out" [Critical,Triaged] | 20:48 |
*** SmearedBeard has quit IRC | 20:49 | |
*** jmlowe has joined #heat | 20:55 | |
*** jmlowe has quit IRC | 21:01 | |
*** Hazelesque_ is now known as Hazelesque | 21:03 | |
*** AlexeyAbashkin has joined #heat | 21:19 | |
*** crose has quit IRC | 21:22 | |
*** AlexeyAbashkin has quit IRC | 21:24 | |
*** jtomasek has quit IRC | 22:02 | |
*** fragatina has quit IRC | 22:03 | |
*** fragatina has joined #heat | 22:05 | |
openstackgerrit | Zane Bitter proposed openstack/heat master: Resolve race in providing deployment data to Swift https://review.openstack.org/521204 | 22:14 |
*** hoonetorg has quit IRC | 22:28 | |
*** dprince has quit IRC | 22:33 | |
*** ayoung has quit IRC | 23:24 | |
*** gouthamr_ has joined #heat | 23:36 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!