*** alexpilotti has quit IRC | 00:01 | |
*** jrist has joined #heat | 00:02 | |
*** alexpilotti has joined #heat | 00:02 | |
*** alexpilotti has quit IRC | 00:03 | |
*** sdake_ has joined #heat | 00:04 | |
*** alexpilotti has joined #heat | 00:04 | |
*** alexpilotti has quit IRC | 00:06 | |
*** sdake has quit IRC | 00:08 | |
*** jamielennox is now known as jamielennox|away | 00:09 | |
*** liusheng has quit IRC | 00:10 | |
*** liusheng has joined #heat | 00:10 | |
*** dims__ is now known as dims | 00:13 | |
*** sdake has joined #heat | 00:13 | |
*** achanda has quit IRC | 00:15 | |
*** sdake_ has quit IRC | 00:17 | |
*** jamielennox|away is now known as jamielennox | 00:23 | |
*** __TheDodd__ has quit IRC | 00:32 | |
*** Tango|2 has quit IRC | 00:35 | |
openstackgerrit | Merged openstack/heat-templates: Update example template for 'docker-compose' hook https://review.openstack.org/165044 | 00:43 |
---|---|---|
*** cdent has quit IRC | 00:48 | |
openstackgerrit | Steve Baker proposed openstack/heat: Func tests use fedora-heat-test-image https://review.openstack.org/163287 | 00:50 |
openstackgerrit | Steve Baker proposed openstack/heat: WIP Enable software config integration test https://review.openstack.org/165287 | 00:50 |
openstackgerrit | Steve Baker proposed openstack/heat: iniset heat_integrationtests.conf https://review.openstack.org/165286 | 00:50 |
asalkeld | randall's keystone v2 patch broke my standalone:( | 00:57 |
stevebaker | asalkeld: I run standalone with v3 now | 01:11 |
stevebaker | asalkeld: this just landed https://review.openstack.org/#/c/141910/ | 01:11 |
asalkeld | stevebaker: ok - how you do that? | 01:11 |
asalkeld | stevebaker: so - just update my devstack? | 01:12 |
stevebaker | I'm seeing if I can do all my work with a devstack heat standalone pointing at a juno rdo cloud | 01:12 |
stevebaker | yep | 01:12 |
asalkeld | no special setup? | 01:12 |
asalkeld | awesome! | 01:12 |
asalkeld | nice work | 01:12 |
stevebaker | seems to work | 01:12 |
asalkeld | stevebaker: yip i have a mini desktop that is my "stable" devstack | 01:12 |
asalkeld | and laptop as heat devstack | 01:13 |
asalkeld | works really well | 01:13 |
stevebaker | right, sounds very similar | 01:13 |
stevebaker | I have mini dell running packstack | 01:13 |
asalkeld | nice, I have one of these: http://shopap.lenovo.com/au/en/desktops/thinkcentre/tinys/ | 01:14 |
asalkeld | same size as my router :-O | 01:14 |
*** achanda has joined #heat | 01:15 | |
stevebaker | nifty | 01:16 |
*** LiJiansheng has joined #heat | 01:19 | |
*** achanda has quit IRC | 01:21 | |
openstackgerrit | Merged openstack/heat: Fix config drive not created when boot from volume https://review.openstack.org/153454 | 01:26 |
*** dgonzalez has quit IRC | 01:26 | |
huangtianhua | morning all:) | 01:27 |
*** smc7 has joined #heat | 01:28 | |
*** dgonzalez has joined #heat | 01:30 | |
asalkeld | hi huangtianhua | 01:32 |
*** smc7 has quit IRC | 01:33 | |
*** aweiteka has joined #heat | 01:41 | |
*** Marga_ has quit IRC | 01:43 | |
*** spzala has quit IRC | 01:46 | |
*** Kennan has joined #heat | 01:48 | |
Kennan | :sdake there? | 01:50 |
sdake | kennan shoot | 01:50 |
Kennan | could you tell me if heat autoscale used ceilometer alram or not? | 01:51 |
*** liuh has joined #heat | 01:51 | |
Kennan | :sdake I am not sure if ceilometer is the only choice for heat autoscale or not | 01:52 |
sdake | i think so, but shardy or asalkeld would be your go to guy there | 01:53 |
sdake | i haven't worked on heat for over a year | 01:53 |
asalkeld | Kennan: it can | 01:54 |
asalkeld | Kennan: more correctly, yes you should use it | 01:54 |
Kennan | :asalkeld so heat used ceilometer alarm to trigger autoscale ? | 01:55 |
asalkeld | yes | 01:55 |
asalkeld | http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Ceilometer::Alarm | 01:55 |
Kennan | ok. if without ceilometer alarm is it possible to trigger heat autoscale ? | 01:55 |
asalkeld | Kennan: sure, you just need to trigger the webhook | 01:56 |
asalkeld | https://github.com/openstack/heat-templates/blob/master/hot/asg_of_stacks.yaml#L83 | 01:57 |
*** Qiming has joined #heat | 01:57 | |
asalkeld | that will give you the url's to post to - to scale up and down | 01:57 |
Kennan | :asalkeld what do you think the better choice for heat autoscale ? | 01:58 |
Kennan | with ceilometer or not ? | 01:58 |
asalkeld | Kennan: well ceilometer gives us alarming functionality | 01:58 |
asalkeld | you could use what you want to do that | 01:58 |
asalkeld | but we test with ceilometer | 01:58 |
asalkeld | so the scaling alarm/policy and group all work together | 01:59 |
asalkeld | we should really integrate with monasca at some point | 01:59 |
Kennan | ok. asalkeld: so heat receieve ceilometer alarm event and then trigger related autoscale, is it the right logic ? | 01:59 |
asalkeld | yes | 01:59 |
asalkeld | all ceilometer does is say "in alarm state, POST to configured webhook" | 02:00 |
asalkeld | heat gets the webhook and actions the scaling | 02:00 |
Kennan | :asalkeld so when you talked about Monasca | 02:00 |
asalkeld | so monasca has alarms too | 02:01 |
asalkeld | and we should have alarm heat resources | 02:01 |
asalkeld | to offer similar functionality | 02:01 |
Kennan | what's the differnet with ceilometer and monasca, I think ceilometer did monitoring function | 02:01 |
Kennan | ? | 02:01 |
asalkeld | as far as heat is concerned nothing | 02:01 |
Kennan | like instances images, cpus | 02:01 |
*** Yanyanhu has joined #heat | 02:02 | |
asalkeld | but there is a lot of differences | 02:02 |
asalkeld | the period of evalution | 02:02 |
asalkeld | and scaling concerns | 02:02 |
Kennan | OK. so you used monasca for another autoscale cases compared with ceilometer | 02:02 |
Kennan | rigjht ? | 02:02 |
asalkeld | i have never used monasca | 02:02 |
asalkeld | but am suggestion that we *should* integrate with it | 02:03 |
Kennan | ok. Thanks for info | 02:03 |
Kennan | Good to know that | 02:03 |
asalkeld | no problem | 02:03 |
Qiming | asalkeld, previously, team had some discussions about 'heatr' | 02:10 |
asalkeld | Qiming: what's that? | 02:10 |
Qiming | a repo for heat templates? | 02:11 |
asalkeld | o, that's right | 02:11 |
Qiming | several months ago | 02:11 |
asalkeld | yeah, just needs people to do it | 02:11 |
Qiming | I was wondering if we should provide such interfaces on Heat, instead of relying on other projects? | 02:11 |
asalkeld | Qiming: i would have thought glance is the place | 02:12 |
Qiming | we are saving all artifacts related to templates in db already | 02:12 |
asalkeld | or a dedicated website | 02:12 |
Qiming | those are all options | 02:12 |
Qiming | I agree | 02:12 |
asalkeld | Qiming: you want global site? or something in horizon? | 02:12 |
Qiming | no, I was just thinking about providing a repo of templates within Heat | 02:13 |
asalkeld | mmm, not a big fan of that | 02:13 |
asalkeld | i'd rather heat understands glance | 02:14 |
*** harlowja_ is now known as harlowja_away | 02:14 | |
asalkeld | and get_file can look in glance too | 02:14 |
asalkeld | but i understand that the artifact thingy is struggling to get in | 02:14 |
Qiming | but so far no progress on artifact storage in glance yet .. | 02:15 |
asalkeld | https://review.openstack.org/#/q/status:open+project:openstack/glance+branch:master+topic:bp/artifact-repository,n,z | 02:15 |
asalkeld | still in review | 02:15 |
asalkeld | patch set 104 | 02:15 |
Qiming | okay, that is some real progress, cool | 02:16 |
Qiming | though still in review | 02:16 |
asalkeld | can't see it going in for k | 02:16 |
asalkeld | must be some real apathy from glance devs | 02:17 |
Qiming | right, a pity is it | 02:17 |
asalkeld | Qiming: i think projects should be focused, heat is not really a file store | 02:17 |
Qiming | ... the artifact plugins loader -- 101 patchset? wow ... | 02:18 |
Qiming | asalkeld, I understand that | 02:18 |
asalkeld | the other issue is how to share | 02:18 |
asalkeld | do we want upstream templates (more formal version of heat-templates) | 02:19 |
Qiming | I was just thinking if that would be a convenience for template composition | 02:19 |
asalkeld | ok, like a std lib | 02:19 |
asalkeld | kinda | 02:19 |
asalkeld | i think glance is the way to go for that | 02:20 |
Qiming | right, I didn't mean a repo | 02:20 |
openstackgerrit | Merged openstack/heat: Move test_notifications to functional https://review.openstack.org/152573 | 02:20 |
openstackgerrit | Merged openstack/heat: Docker plugin add restart policies and capability properties https://review.openstack.org/163277 | 02:20 |
*** reed has quit IRC | 02:20 | |
asalkeld | just really disapointed with the progress | 02:20 |
*** erkules_ has joined #heat | 02:20 | |
openstackgerrit | Merged openstack/heat: Fixes StackWatchService to use objects in place of db https://review.openstack.org/165349 | 02:20 |
Qiming | agreed, if glance has it, we don't reinvent them | 02:20 |
Qiming | anyway, 101 patchsets was .. a long march | 02:21 |
*** erkules has quit IRC | 02:23 | |
asalkeld | yeah, happy we don't block like that | 02:24 |
*** achanda has joined #heat | 02:27 | |
*** dims has quit IRC | 02:28 | |
*** aweiteka has quit IRC | 02:29 | |
asalkeld | wow found in embarrassing bug in the engine listener code | 02:30 |
*** randallburt has joined #heat | 02:45 | |
*** randallburt has quit IRC | 02:45 | |
*** randallburt has joined #heat | 02:46 | |
*** strictlyb has quit IRC | 02:55 | |
*** Yanyan has joined #heat | 02:56 | |
asalkeld | who added the "heat-manage service list" ? | 02:57 |
*** Yanyanhu has quit IRC | 03:00 | |
*** spzala has joined #heat | 03:01 | |
*** coolsvap|afk is now known as coolsvap | 03:04 | |
*** GonZo2K has quit IRC | 03:07 | |
*** spzala has quit IRC | 03:11 | |
*** vijayagurug has joined #heat | 03:15 | |
*** Marga_ has joined #heat | 03:21 | |
*** Marga_ has quit IRC | 03:21 | |
*** Marga_ has joined #heat | 03:22 | |
*** sdake_ has joined #heat | 03:22 | |
openstackgerrit | Randall Burt proposed openstack/heat: Ignore migrating environments for deleted stacks https://review.openstack.org/164856 | 03:23 |
*** sdake has quit IRC | 03:26 | |
*** Yanyanhu has joined #heat | 03:26 | |
*** Yanyan has quit IRC | 03:29 | |
*** coolsvap is now known as coolsvap|afk | 03:32 | |
Kennan | :asalkeld there? | 03:36 |
asalkeld | yip | 03:36 |
Kennan | I followed heat developer guide and enable osprofiler | 03:36 |
Kennan | ubuntu@test-alarm:~$ heat --profile SECRET_KEY stack-list +----+------------+--------------+---------------+ | id | stack_name | stack_status | creation_time | +----+------------+--------------+---------------+ +----+------------+--------------+---------------+ Trace ID: 8a410e34-550f-4ac0-aabc-175c2e0a9384 To display trace use next command: osprofiler trace show --html 8a410e34-550f-4ac0-aabc-175c2e0a9384 | 03:36 |
Kennan | But I used command following | 03:37 |
Kennan | I found | 03:37 |
Kennan | ubuntu@test-alarm:~$ osprofiler trace show --html 8a410e34-550f-4ac0-aabc-175c2e0a9384 Trace with UUID 8a410e34-550f-4ac0-aabc-175c2e0a9384 not found. There are 2 possible reasons: 1) You are using not admin credentials 2) You specified wrong trace id | 03:37 |
Kennan | :asalkeld do you know the issue ? | 03:37 |
elynn | Morning | 03:37 |
asalkeld | Kennan: i haven't tried for ages | 03:37 |
asalkeld | boris-42: ^ | 03:37 |
asalkeld | Kennan: try on #openstack-rally | 03:38 |
Kennan | ok. thanks | 03:39 |
asalkeld | sorry it's been a couple of months since i tried | 03:39 |
*** coolsvap|afk is now known as coolsvap | 03:39 | |
asalkeld | my head a bit out of ti | 03:39 |
*** achanda has quit IRC | 03:42 | |
*** greghaynes has quit IRC | 03:43 | |
*** achanda has joined #heat | 03:44 | |
zaneb | w00t | 03:46 |
zaneb | tests are passing | 03:46 |
*** kebray has joined #heat | 03:48 | |
*** kebray has quit IRC | 03:48 | |
elynn | Hi I have a question, how to get nested stack resource attribute? | 03:53 |
elynn | {get_attr: [neststack, resource.randstr, value]} | 03:54 |
elynn | seems not work | 03:54 |
elynn | output value is still null | 03:54 |
asalkeld | elynn: check your template version | 03:54 |
asalkeld | try making it as new as possible | 03:54 |
*** sdake has joined #heat | 03:55 | |
elynn | I'm using 2013-05-23 | 03:55 |
elynn | If I using a juno version | 03:55 |
asalkeld | elynn: it's new'ish | 03:55 |
elynn | is there any possible to get nested stack resource attribute? | 03:56 |
elynn | or get nested stack outputs. | 03:56 |
asalkeld | https://github.com/openstack/heat/commit/dacc3cc35fb48f7df1380323a9a800e510e048cc | 03:56 |
*** Yanyan has joined #heat | 03:57 | |
elynn | I follow this link http://docs.openstack.org/developer/heat/template_guide/composition.html but it doesn't work. | 03:57 |
asalkeld | elynn: the "old" way it just to use outputs in the nested stack | 03:57 |
elynn | so i can use a template like {get_attr: [neststack, outputs, key1]} to get outputs? | 03:58 |
*** sdake_ has quit IRC | 03:59 | |
asalkeld | elynn: {get_attr: [neststack, key1]} | 03:59 |
asalkeld | the output is converted to an attribute | 03:59 |
elynn | Cool, will have a try. | 04:00 |
*** Yanyanhu has quit IRC | 04:01 | |
elynn | In master branch, I upgrade heat_template_version from 2013-05-23 to 2014-10-16, and get resource attributes worked. | 04:01 |
*** cmyster has quit IRC | 04:01 | |
*** Yanyanhu has joined #heat | 04:01 | |
asalkeld | nice | 04:01 |
*** Yanyan has quit IRC | 04:02 | |
*** cmyster has joined #heat | 04:02 | |
*** vijayagurug has left #heat | 04:04 | |
*** sdake has quit IRC | 04:04 | |
*** achanda has quit IRC | 04:05 | |
*** jamielennox is now known as jamielennox|away | 04:06 | |
*** Yanyan has joined #heat | 04:09 | |
cmyster | morning | 04:13 |
cmyster | stevebaker: still around ? | 04:13 |
*** Yanyanhu has quit IRC | 04:13 | |
*** kebray has joined #heat | 04:14 | |
openstackgerrit | Zane Bitter proposed openstack/heat: Add a functional test for hooks/breakpoints https://review.openstack.org/164652 | 04:17 |
openstackgerrit | Zane Bitter proposed openstack/heat: Add breakpoint support https://review.openstack.org/146123 | 04:17 |
openstackgerrit | Zane Bitter proposed openstack/heat: Store Resources before they are created https://review.openstack.org/165686 | 04:17 |
*** Marga_ has quit IRC | 04:18 | |
zaneb | stevebaker, asalkeld: could y'all keep an eye on those ^ and see if the tests pass? | 04:18 |
asalkeld | ok | 04:18 |
cmyster | morning zaneb | 04:18 |
zaneb | Ich gehe ins Bett | 04:18 |
cmyster | and nn | 04:18 |
*** dims has joined #heat | 04:19 | |
zaneb | cmyster: you're right, it is morning :( | 04:25 |
cmyster | heh | 04:25 |
cmyster | tough night I see | 04:25 |
zaneb | yeah | 04:26 |
zaneb | but I think the hooks/breakpoints patch is going to work now | 04:26 |
cmyster | don't worry zaneb, its always darkest before the dawn... | 04:26 |
*** Yanyanhu has joined #heat | 04:26 | |
zaneb | so \o/ | 04:26 |
cmyster | indeed | 04:26 |
*** erkules_ is now known as erkules | 04:26 | |
*** erkules has joined #heat | 04:26 | |
cmyster | I need to add more and more tests... damn this 24h... | 04:26 |
cmyster | zaneb: could you please look at a link for a minute ? | 04:27 |
*** dims has quit IRC | 04:27 | |
zaneb | sure | 04:27 |
cmyster | sec | 04:27 |
zaneb | ugh, 7 hours 45 minutes until next meeting :/ | 04:28 |
*** Yanyan has quit IRC | 04:28 | |
asalkeld | har | 04:29 |
asalkeld | slave to the meeting | 04:29 |
zaneb | g'night y'all \o | 04:34 |
cmyster | nn | 04:34 |
*** achanda has joined #heat | 04:37 | |
*** sdake has joined #heat | 04:38 | |
*** achanda has quit IRC | 04:41 | |
*** Raj1 has joined #heat | 04:45 | |
asalkeld | brb, picking kids up | 04:45 |
*** Raj1 has quit IRC | 04:46 | |
*** Marga_ has joined #heat | 04:47 | |
*** harlowja_at_home has joined #heat | 04:47 | |
*** smc7 has joined #heat | 05:06 | |
*** achanda has joined #heat | 05:08 | |
*** smc7 has quit IRC | 05:11 | |
*** sdake has quit IRC | 05:12 | |
*** Raj1 has joined #heat | 05:15 | |
*** Yanyan has joined #heat | 05:16 | |
*** Yanyanhu has quit IRC | 05:20 | |
*** liusheng has quit IRC | 05:26 | |
*** liusheng has joined #heat | 05:26 | |
*** rakesh_hs has joined #heat | 05:34 | |
*** greghaynes has joined #heat | 05:39 | |
*** Yanyanhu has joined #heat | 05:39 | |
*** harlowja_at_home has quit IRC | 05:40 | |
*** coolsvap is now known as coolsvap|afk | 05:41 | |
*** Yanyan has quit IRC | 05:42 | |
*** coolsvap|afk is now known as coolsvap | 05:48 | |
*** KanagarajM has joined #heat | 05:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Imported Translations from Transifex https://review.openstack.org/165701 | 06:02 |
*** sarob has quit IRC | 06:05 | |
*** liuh_ has joined #heat | 06:07 | |
*** Tango has joined #heat | 06:09 | |
*** liuh has quit IRC | 06:10 | |
*** liuh has joined #heat | 06:11 | |
*** dims has joined #heat | 06:13 | |
*** liuh_ has quit IRC | 06:14 | |
*** Yanyan has joined #heat | 06:15 | |
*** Yanyanhu has quit IRC | 06:18 | |
*** dims has quit IRC | 06:19 | |
*** Tango has quit IRC | 06:21 | |
*** Yanyanhu has joined #heat | 06:23 | |
*** Yanyan has quit IRC | 06:26 | |
*** achanda has quit IRC | 06:28 | |
*** Yanyan has joined #heat | 06:31 | |
*** Yanyanhu has quit IRC | 06:35 | |
*** kebray has quit IRC | 06:38 | |
*** ishant has joined #heat | 06:43 | |
*** Qiming has quit IRC | 06:47 | |
*** achanda has joined #heat | 06:52 | |
*** jcoufal has joined #heat | 06:53 | |
*** smc7 has joined #heat | 06:55 | |
*** jtomasek has joined #heat | 06:57 | |
*** liuh_ has joined #heat | 06:59 | |
*** achanda has quit IRC | 07:00 | |
openstackgerrit | Angus Salkeld proposed openstack/heat: Set num_engine_workers=2 for gate functional tests https://review.openstack.org/165396 | 07:00 |
openstackgerrit | Angus Salkeld proposed openstack/heat: Fix the topic and sever that the listener client uses https://review.openstack.org/165712 | 07:00 |
openstackgerrit | Angus Salkeld proposed openstack/heat: Don't assume that there is one service per host https://review.openstack.org/165713 | 07:00 |
*** smc7 has quit IRC | 07:00 | |
*** liuh has quit IRC | 07:01 | |
*** achanda has joined #heat | 07:04 | |
*** ifarkas has joined #heat | 07:05 | |
*** sarob has joined #heat | 07:05 | |
*** Marga_ has quit IRC | 07:09 | |
*** sarob has quit IRC | 07:11 | |
skraynev | morning guys | 07:16 |
asalkeld | hi skraynev | 07:19 |
skraynev | hi asalkeld | 07:19 |
shadower | morning | 07:23 |
*** ParsectiX has joined #heat | 07:23 | |
*** Yanyan has quit IRC | 07:24 | |
*** Qiming has joined #heat | 07:28 | |
*** randallburt has quit IRC | 07:28 | |
skraynev | shadower: yeah, sunny morning :) | 07:29 |
shadower | skraynev: omg, right? | 07:29 |
skraynev | shadower: yes:) spring comes to Russia. | 07:30 |
asalkeld | autumn here and 36 degrees, waiting for it to finally cool down | 07:30 |
shadower | wow, that's really hot | 07:31 |
shadower | we only get that in the summer | 07:32 |
*** jtomasek has quit IRC | 07:36 | |
*** achanda has quit IRC | 07:37 | |
*** jprovazn has joined #heat | 07:44 | |
*** chlong has quit IRC | 07:47 | |
*** nkhare has joined #heat | 07:48 | |
*** pitr-ch has joined #heat | 07:49 | |
*** Raj1 has quit IRC | 07:51 | |
*** Qiming_ has joined #heat | 08:03 | |
*** inc0 has joined #heat | 08:03 | |
*** dims has joined #heat | 08:04 | |
*** Qiming has quit IRC | 08:06 | |
*** dims has quit IRC | 08:10 | |
asalkeld | skraynev: https://review.openstack.org/#/c/165686/ | 08:12 |
asalkeld | that also *really* needs reviews .. | 08:12 |
skraynev | asalkeld: ok. got it. | 08:12 |
asalkeld | shadower: you need to hunt down cores to review that stuff | 08:14 |
shadower | ah okay | 08:15 |
* shadower didn't want to bother people | 08:16 | |
asalkeld | today is the last day to get it in | 08:16 |
asalkeld | (unless we get a FFE) | 08:16 |
asalkeld | shadower: do lots of bothering | 08:16 |
asalkeld | shadower: i'll get on it in a couple of mins - just fixin suppper for the kids | 08:17 |
shadower | ok thanks | 08:17 |
*** jtomasek has joined #heat | 08:33 | |
*** jistr has joined #heat | 08:37 | |
*** Qiming_ is now known as Qiming | 08:41 | |
*** Yanyan has joined #heat | 08:41 | |
*** BManojlovic has joined #heat | 08:45 | |
*** Yanyanhu has joined #heat | 08:49 | |
*** Yanyan has quit IRC | 08:51 | |
shardy_z | morning all | 08:55 |
*** shardy_z is now known as shardy | 08:55 | |
shardy | shadower: hooray, I see zaneb came up with another INIT_COMPLETE patch :) | 08:59 |
* shardy reviews | 08:59 | |
*** inc0_ has joined #heat | 09:00 | |
*** inc0 has quit IRC | 09:01 | |
*** dulek has joined #heat | 09:03 | |
*** randallburt has joined #heat | 09:03 | |
*** dulek has quit IRC | 09:04 | |
shadower | shardy: yeah, and this one seems to be working :-) | 09:04 |
shadower | I think the breakpoint test failure is transient but the two functional ones are not -- looking at them now | 09:04 |
*** dulek has joined #heat | 09:04 | |
shardy | shadower: ah, OK I just rechecked against the timeout | 09:05 |
*** Yanyan has joined #heat | 09:10 | |
*** sorantis has joined #heat | 09:11 | |
*** kairat_kushaev has quit IRC | 09:14 | |
asalkeld | shardy: can you approve this https://review.openstack.org/#/c/161306/ | 09:14 |
asalkeld | shardy: i have a FFE on breakpoints btw | 09:14 |
shardy | asalkeld: looking | 09:14 |
*** Yanyanhu has quit IRC | 09:15 | |
shardy | asalkeld: great, thanks - if we can fix the functional tests it may land today anyway :) | 09:15 |
asalkeld | (don't stress, just do it well) | 09:15 |
shardy | asalkeld: ack | 09:15 |
*** rdo has quit IRC | 09:16 | |
*** dulek has quit IRC | 09:16 | |
*** inc0_ has quit IRC | 09:16 | |
shadower | shardy: any idea why this is failing? http://logs.openstack.org/52/164652/4/check/check-heat-dsvm-functional-mysql/b94230a/console.html#_2015-03-19_05_31_00_256 | 09:17 |
shadower | related code: https://review.openstack.org/#/c/164652/4/heat_integrationtests/functional/test_hooks.py (line 218) | 09:17 |
shadower | apparently the json for the "nested" resource we get back has "rel" = "self" instead of "rel" = "nested" | 09:18 |
shadower | so the assertion fails | 09:18 |
shadower | but it is a nested stack, is it not? | 09:18 |
shardy | shadower: it should have rel self and rel nested, if it's a nested stack | 09:19 |
shardy | shadower: let me finish a couple of reviews then I'll pull it locally and reproduce | 09:19 |
shadower | thanks | 09:19 |
shardy | it does look like it's failing to find the nested stack, but I'm not sure why atm | 09:19 |
*** kairat_kushaev has joined #heat | 09:19 | |
shadower | I'll keep looking | 09:19 |
*** dims has joined #heat | 09:22 | |
*** logan2 has quit IRC | 09:22 | |
shardy | skraynev: what are your thoughts re the comments in https://review.openstack.org/#/c/161306/6 ? | 09:22 |
*** logan2 has joined #heat | 09:22 | |
asalkeld | shadower: ttx what's to cut k3 at the latest at 21utc | 09:23 |
asalkeld | shardy: .. | 09:23 |
shardy | IMO the set rework isn't worth blocking for, but is it worth fixing the misleading docstrings then we can quickly approve? | 09:23 |
shardy | followup patch is also fine | 09:23 |
*** inc0 has joined #heat | 09:24 | |
shardy | asalkeld: I'm fine with approving it now, I just thought if skraynev has 5 mins, we could fix the most obvious issues and not forget about them | 09:24 |
skraynev | shardy: Oh I forgot to give answer. IMO, it's not major cases | 09:24 |
asalkeld | skraynev: the doc strings are a bit messed up | 09:24 |
asalkeld | shardy: and i are here, we can approve fast | 09:24 |
skraynev | Sure will fix it! | 09:24 |
shardy | yeah that's all I really care about, a couple of docstrings are obviously wrong | 09:24 |
shardy | if we don't do it now, we'll forget and it will confuse folks later | 09:25 |
skraynev | sure guys give me a couple minutes to fix it :) | 09:25 |
asalkeld | skraynev: just ping me, i am watching tv | 09:25 |
skraynev | asalkeld: :) sure | 09:25 |
asalkeld | shardy: fun things happen when we turn engine_workers > 1 | 09:26 |
asalkeld | https://review.openstack.org/#/c/165712/ | 09:26 |
shardy | asalkeld: Hmm, yeah I saw your bug | 09:26 |
*** dims has quit IRC | 09:26 | |
asalkeld | seems like it needs to be tests | 09:26 |
shardy | I was testing TripleO with multiple workers last week and I didn't see any issues | 09:26 |
shardy | yeah, historically it's not been well tested | 09:27 |
asalkeld | that's cancelling an action | 09:27 |
shardy | do we know if the gate VMs have more than one core? | 09:27 |
asalkeld | shardy: honestly not sure | 09:27 |
asalkeld | i am not sure if it will actually make it faster | 09:27 |
asalkeld | but seems to test more | 09:27 |
shardy | I'm assuming we'll only see performance benefits with more than one vCPU, although my local testing has been inconclusive so far | 09:28 |
*** pas-ha has joined #heat | 09:28 | |
asalkeld | yeah, tho' normally the 1 cpu is stuck at 100% | 09:28 |
asalkeld | multi workers you have 3 at 50-60% | 09:28 |
shardy | Yeah, probably we need to scale out over more real hardware to see significant benefits | 09:29 |
*** dulek has joined #heat | 09:29 | |
Qiming | http://eavesdrop.openstack.org/irclogs/%23heat/%23heat.2015-03-19.log | 09:32 |
pas-ha | morning all | 09:34 |
*** derekh has joined #heat | 09:37 | |
shardy | shadower: hey, I noticed a minor error in the docs for https://review.openstack.org/#/c/146123/ | 09:37 |
shardy | otherwise it looks good to me now | 09:38 |
*** liusheng has quit IRC | 09:41 | |
*** Yanyanhu has joined #heat | 09:41 | |
shadower | shardy: thanks, I'll fix that now | 09:42 |
*** liusheng has joined #heat | 09:42 | |
shadower | shardy: in the meantime, setting "expected_status" to CREATE_COMPLETE seems to fix the assert_resource_is_a_nested_stack issue | 09:43 |
* shadower is baffled | 09:43 | |
shadower | are we hitting some sort of a race? | 09:43 |
shadower | e.g. the nested stack has not been created yet when we try to look at it | 09:43 |
shardy | shadower: well that does make sense, we can't get the nested ID until the stack create is finished | 09:44 |
*** Yanyan has quit IRC | 09:44 | |
shardy | shadower: FWIW I'd like to add recursive listing of events, which will make this easier | 09:44 |
shadower | yea | 09:44 |
* shardy pulls the patch and tries to reproduce | 09:44 | |
*** dims has joined #heat | 09:45 | |
shardy | shadower: Oh, I added one more nit re the commit message, if you're reworking those docs anyway | 09:47 |
shadower | shardy: I just hit git review but on a machine which doesn't have my gerrit info :-) | 09:49 |
shardy | shadower: you're right, the self.stack_create returns before CREATE_COMPLETE< so the nested_identifier = self.assert_resource_is_a_stack needs to wait for the resource status | 09:50 |
*** Trozz has quit IRC | 09:50 | |
shardy | shadower: probably we need a call to _wait_for_resource_status before that line | 09:51 |
* shardy tests | 09:51 | |
shadower | shardy: yea but this seems to show that the breakpoint isn't being set in the nested stack, right? | 09:52 |
openstackgerrit | Tomas Sedovic proposed openstack/heat: Add breakpoint support https://review.openstack.org/146123 | 09:52 |
shadower | shardy: that should have the doc & commit fixes ^ | 09:53 |
*** Trozz has joined #heat | 09:53 | |
shardy | shadower: Hmm, aren't we just trying to get the nested ID of something before it's started being created? | 09:53 |
shardy | I'm trying making _wait_for_resource_status wait on "nested" to go CREATE_IN_PROGRESS before trying to get the nested ID | 09:54 |
shardy | maybe even that is racy, given that we set the id after the state change | 09:55 |
shardy | but we can't wait for CREATE_COMPLETE due to the hook | 09:55 |
*** Trozz is now known as Trozz_ | 09:55 | |
*** Trozz_ is now known as Trozz | 09:55 | |
shardy | maybe we need to poll for the nested id | 09:55 |
shadower | shardy: right. But it does seem to succeed | 09:55 |
shadower | (waiting for CREATE_COMPLETE) | 09:55 |
shadower | which means something is wrong | 09:55 |
*** tspatzier has joined #heat | 09:55 | |
shardy | well the hook will stop it (the parent "nested") during create_in_progress won't it? | 09:56 |
shardy | then we need to poll for the nested ID, and inspect the events for the nested stack | 09:56 |
shardy | you're right though, it still doesn't work for me | 09:56 |
shadower | shardy: it is supposed to. But apparently it's not being set on nested stacks for whatever reason | 09:56 |
shadower | or something | 09:56 |
shadower | shardy: when I just changed CREATE_IN_PROGRESS to CREATE_COMPLETE in the "stack_create" call, it succeede | 09:57 |
shadower | which it shouldn't have | 09:57 |
shardy | Hmm, I've only teted this with provider resources, not type: foo.yaml | 09:57 |
*** sorantis has quit IRC | 09:57 | |
shadower | I've tested it that way | 09:58 |
shadower | but not recently | 09:58 |
shadower | could be we've broken something in the last few patches | 09:58 |
shardy | yeah, possibly, I've not re-tested with tripleo for a couple of days | 09:58 |
shardy | Oh well, lets figure out what broke ;) | 09:59 |
*** inc0 has quit IRC | 09:59 | |
openstackgerrit | Sergey Kraynev proposed openstack/heat: Add way to collect map of needed attributes https://review.openstack.org/161306 | 10:00 |
*** sorantis has joined #heat | 10:01 | |
skraynev | shardy, asalkeld: new patchset uploaded. wait a Jenkins :) | 10:02 |
*** dulek has quit IRC | 10:05 | |
*** dulek has joined #heat | 10:06 | |
shadower | shardy: yeah so hooks on nested resources don't appear to be set anymore | 10:08 |
*** Yanyan has joined #heat | 10:09 | |
asalkeld | skraynev: looking | 10:09 |
skraynev | asalkeld: suggestions about correct doc are appreciated | 10:10 |
*** dulek_ has joined #heat | 10:10 | |
asalkeld | still says "return a dictionary" | 10:11 |
asalkeld | https://review.openstack.org/#/c/161306/7/heat/engine/function.py | 10:11 |
asalkeld | skraynev: returns a list? | 10:11 |
*** dulek has quit IRC | 10:11 | |
*** shaohe_robot has joined #heat | 10:11 | |
*** alex_xu has quit IRC | 10:12 | |
skraynev | right | 10:12 |
skraynev | asalkeld: yes | 10:12 |
*** shaohe_feng has quit IRC | 10:12 | |
asalkeld | are you going to do another quick patch? | 10:12 |
skraynev | asalkeld: yes, I can (if it necessary ) | 10:13 |
*** Yanyanhu has quit IRC | 10:13 | |
skraynev | *is's necessary | 10:13 |
asalkeld | sure may as well make those correct | 10:13 |
skraynev | *it's | 10:13 |
asalkeld | easy | 10:13 |
asalkeld | brb | 10:13 |
*** alex_xu has joined #heat | 10:14 | |
*** Yanyanhu has joined #heat | 10:14 | |
*** Yanyan has quit IRC | 10:14 | |
*** Yanyanhu has quit IRC | 10:18 | |
*** vijayagurug has joined #heat | 10:21 | |
openstackgerrit | Peter Razumovsky proposed openstack/heat-specs: Improvements in deprecation process https://review.openstack.org/153235 | 10:27 |
*** inc0 has joined #heat | 10:30 | |
*** sorantis has quit IRC | 10:32 | |
*** tspatzier has quit IRC | 10:32 | |
*** smc7 has joined #heat | 10:32 | |
*** GonZo2K has joined #heat | 10:34 | |
openstackgerrit | Sergey Kraynev proposed openstack/heat: Add way to collect map of needed attributes https://review.openstack.org/161306 | 10:36 |
skraynev | asalkeld: done. | 10:36 |
asalkeld | thanks | 10:37 |
*** smc7 has quit IRC | 10:37 | |
shadower | shardy: so apparently, the resource_registry gets whittled down as it's being passed to nested stacks, but then when we check if a resource has a hook, its parent stack has the full registry again | 10:38 |
asalkeld | skraynev: +2'd | 10:38 |
shadower | maybe the env changes are not being persisted? | 10:39 |
shadower | shardy: do you think zane's patch may have caused that? | 10:39 |
skraynev | asalkeld: thx for the pointing on missed place | 10:39 |
asalkeld | np | 10:39 |
*** coolsvap is now known as coolsvap|afk | 10:41 | |
*** liuh_ has quit IRC | 10:42 | |
*** tspatzier has joined #heat | 10:43 | |
*** tspatzier has quit IRC | 10:55 | |
shardy | shadower: hmm, it looks like we're not storing the whittled down version from remove_resources_except | 10:56 |
shardy | because matches_hook in the nested template sees the parent resource registry | 10:56 |
shadower | shardy: that's what I said | 10:56 |
shardy | e.g {u'nested': {u'foo_step2': {u'hooks': u'pre-create'}}} | 10:56 |
shadower | (or at least that's what I meant) | 10:56 |
shadower | yeah so I got to the same conclusion | 10:57 |
shardy | shadower: Yeah, I guess I'm just confirming I see the same thing ;) | 10:57 |
shadower | shardy: any ideas why it might be? It used to work before... | 10:57 |
shadower | yeah, sorry :-) | 10:57 |
shardy | not sure yet, trying to figure it out | 10:57 |
* shardy may need more coffee | 10:57 | |
openstackgerrit | Merged openstack/heat: Store Resources before they are created https://review.openstack.org/165686 | 10:58 |
* shadower just brewed a nice cup of earl grey | 10:58 | |
shardy | shadower: Ooh, we're not storing the stack after the call to environment.get_child_environment | 11:01 |
shardy | which was fine before, but now the create/update happens via RPC, it won't work | 11:02 |
shardy | Hmm, although we're passing child_env via the RPC call... | 11:02 |
*** jaypipes has joined #heat | 11:04 | |
*** nkhare has quit IRC | 11:05 | |
openstackgerrit | Peter Razumovsky proposed openstack/heat-specs: Improvements in deprecation process https://review.openstack.org/153235 | 11:08 |
*** prazumovsky has joined #heat | 11:10 | |
*** GonZo2K has quit IRC | 11:19 | |
shardy | shadower: I think we're forgetting to pass child_resource_name into environment.get_child_environment in StackResource.create_with_template | 11:21 |
shardy | https://review.openstack.org/#/c/146123/22/heat/engine/environment.py | 11:22 |
shardy | so we never hit the if child_resource_name on create, so the remove_resources_except is never called | 11:22 |
*** GonZo2K has joined #heat | 11:27 | |
shadower | oh | 11:29 |
shadower | shardy: I'll try it out, then | 11:29 |
shardy | shadower: I needed to make some changes to the tests too, just re-testing then I can push what I have if you like? | 11:30 |
shadower | shardy: that would be great | 11:31 |
shardy | Ran 6 tests in 90.847s OK | 11:31 |
shardy | \o/ | 11:31 |
shardy | Ok, let me remove all my debug logging :) | 11:31 |
*** nkhare has joined #heat | 11:32 | |
*** coolsvap|afk is now known as coolsvap | 11:36 | |
*** GonZo2K has quit IRC | 11:42 | |
asalkeld | any last review requests before i head to bed? | 11:43 |
shardy | asalkeld: I'm about to push a new breakpoint patch, but given the FFE I guess that can wait | 11:45 |
asalkeld | shardy: if it get's in that's fine too | 11:45 |
*** sorantis has joined #heat | 11:47 | |
*** DaveJ__ has joined #heat | 11:48 | |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Add OS::Mistral::Workflow for Mistral workflows https://review.openstack.org/147645 | 11:49 |
*** prazumovsky has quit IRC | 11:50 | |
*** nkhare has quit IRC | 11:51 | |
*** dulek_ has quit IRC | 11:51 | |
openstackgerrit | Steven Hardy proposed openstack/heat: Add a functional test for hooks/breakpoints https://review.openstack.org/164652 | 11:52 |
openstackgerrit | Steven Hardy proposed openstack/heat: Add breakpoint support https://review.openstack.org/146123 | 11:52 |
shardy | shadower: ^^ | 11:52 |
asalkeld | shardy: watching TV some brittish guys doing "extreme ironing" :-O | 11:54 |
shardy | asalkeld: doesn't everyone do that? :D | 11:55 |
asalkeld | yes, ironing clothes on cliffs | 11:55 |
asalkeld | mad | 11:55 |
shardy | heh, sounds it | 11:55 |
cmyster | shardy: breakpoint tests seems fine, I need to try it myself though | 11:57 |
*** cdent has joined #heat | 11:59 | |
* cmyster wonders if this patch will land... | 12:01 | |
*** noakoffman has joined #heat | 12:04 | |
shadower | shardy: thanks! Looking at it | 12:06 |
shadower | shardy: by the way, regarding the heatclient interface, I was thinking we keep breakpoints as they are now and then add a more generic hook functionality | 12:09 |
shadower | both are useful depending on what you want to do imho | 12:09 |
shardy | shadower: Hmm, ok, I'd assumed we'd align the terminology with the environment | 12:10 |
*** noakoffman has quit IRC | 12:10 | |
*** blinky_ghost has joined #heat | 12:11 | |
shardy | For me personally, just passing an additional environment file is probably good enough tbh | 12:11 |
shadower | shardy: yeah, I was thinking more about the clearing command | 12:11 |
shadower | ftr, I do want a proper hook-based interface. But they're conceptually different from breakpoints | 12:12 |
shardy | yeah, I'd assumed we'd want hook-clear --pre-create mystack myresource | 12:12 |
openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Versioned-object StackLock https://review.openstack.org/165363 | 12:12 |
shardy | or something, to avoid always sending a signal for every hook | 12:12 |
shadower | shardy: hm, okay, thad't work | 12:12 |
*** noakoffman has joined #heat | 12:12 | |
shardy | shadower: Yeah, I guess I'm just questioning exposing the *breakpoint stuff at all in the client, given that it's now inconsistent with the resource registry interface | 12:13 |
shadower | hm, that's true, too | 12:13 |
shardy | having two names for the same thing is invariably confusing ;) | 12:13 |
shardy | I don't feel that strongly about it though, lets see what other folks think | 12:13 |
shardy | at least the client isn't constrained by FF, so we can just press ahead with the heat part for now | 12:14 |
*** chlong has joined #heat | 12:14 | |
shadower | shardy: I'll look more into the client once the server stuff is done. Slowly coming over to your side on naming :-) | 12:15 |
*** noakoffman has quit IRC | 12:16 | |
*** noakoffman has joined #heat | 12:16 | |
*** sasuke_at_hp has joined #heat | 12:19 | |
*** vijayagurug has left #heat | 12:21 | |
*** KanagarajM has quit IRC | 12:21 | |
*** sasuke_at_hp has quit IRC | 12:21 | |
*** noakoffman has quit IRC | 12:22 | |
*** noakoffman has joined #heat | 12:23 | |
*** noakoffman has quit IRC | 12:24 | |
*** sdake has joined #heat | 12:24 | |
*** sdake_ has joined #heat | 12:25 | |
*** Marga_ has joined #heat | 12:26 | |
*** chlong has quit IRC | 12:26 | |
*** lkarm has joined #heat | 12:26 | |
*** blomquisg has quit IRC | 12:27 | |
asalkeld | shardy: can you keep an eye on this: https://review.openstack.org/#/c/161306/ | 12:29 |
*** sdake has quit IRC | 12:29 | |
asalkeld | and approve when it passes tests? | 12:29 |
asalkeld | it's the last blueprint for k3 | 12:30 |
asalkeld | also mark the bp as done please | 12:30 |
*** rdo has joined #heat | 12:30 | |
* asalkeld heads to bed - night all | 12:30 | |
*** asalkeld has quit IRC | 12:30 | |
*** aweiteka has joined #heat | 12:31 | |
*** htruta has quit IRC | 12:31 | |
*** htruta has joined #heat | 12:32 | |
*** noakoffman has joined #heat | 12:36 | |
*** serg_melikyan has joined #heat | 12:37 | |
*** noakoffman has quit IRC | 12:40 | |
*** noakoffman has joined #heat | 12:41 | |
*** achanda has joined #heat | 12:42 | |
*** chlong has joined #heat | 12:43 | |
*** dulek has joined #heat | 12:44 | |
*** achanda has quit IRC | 12:47 | |
*** blinky_ghost has left #heat | 12:48 | |
shadower | shardy: the patch survived everything I threw at it including nested stacks, wildcard hook on all ResourceGroup members and on a specifig member, too | 12:53 |
shadower | shardy: so the patch has now been authored by four people, what's the protocol here? (assuming the tests pass) | 12:54 |
openstackgerrit | Rakesh H S proposed openstack/heat: Convergence prepare traversal https://review.openstack.org/159444 | 12:54 |
*** noakoffman has quit IRC | 12:54 | |
*** sasuke_at_hp has joined #heat | 12:58 | |
*** noakoffman has joined #heat | 12:58 | |
*** ParsectiX has quit IRC | 12:59 | |
*** noakoffman has quit IRC | 13:00 | |
*** noakoffman has joined #heat | 13:01 | |
*** dims has quit IRC | 13:01 | |
*** sdake_ has quit IRC | 13:01 | |
*** dims has joined #heat | 13:02 | |
*** noakoffman has quit IRC | 13:03 | |
*** noakoffman has joined #heat | 13:03 | |
*** noakoffman has quit IRC | 13:04 | |
pas-ha | shadower, just add them all as Co-Authored-By: Name <email> to the commit message | 13:05 |
pas-ha | that will be counted as commit by e.g. Stackalytics | 13:06 |
*** noakoffman has joined #heat | 13:06 | |
pas-ha | https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references last paragraph | 13:07 |
*** jistr has quit IRC | 13:08 | |
*** noakoffman has quit IRC | 13:08 | |
*** jistr has joined #heat | 13:09 | |
*** tspatzier has joined #heat | 13:09 | |
*** sdake has joined #heat | 13:09 | |
shadower | pas-ha: I was thinking more about the reviews | 13:09 |
shadower | those same people are pretty much the ones who would +2/A it... | 13:09 |
pas-ha | oh, I see it.. | 13:10 |
*** noakoffman has joined #heat | 13:10 | |
shardy | shadower: FWIW I wasn't expecting any Co-Authored by attribution, my changes were pretty minor | 13:11 |
*** ParsectiX has joined #heat | 13:12 | |
shardy | normally if I collaborate on a patch and make significant changes, I'd add one myself, but it didn't seem justified in this case | 13:12 |
*** sdake_ has joined #heat | 13:12 | |
shardy | I'll review now we have something confirmed as working :) | 13:12 |
shadower | right, okay | 13:12 |
shadower | :-) | 13:12 |
*** kaufer has joined #heat | 13:13 | |
*** ParsectiX has quit IRC | 13:14 | |
*** ParsectiX has joined #heat | 13:15 | |
*** sdake has quit IRC | 13:16 | |
*** sasuke_at_hp has quit IRC | 13:18 | |
*** Qiming has quit IRC | 13:19 | |
*** pm90_ has joined #heat | 13:19 | |
*** Qiming has joined #heat | 13:19 | |
*** pm90_ has quit IRC | 13:19 | |
*** tspatzier has quit IRC | 13:19 | |
*** pm90_ has joined #heat | 13:20 | |
*** chlong has quit IRC | 13:21 | |
*** tspatzier has joined #heat | 13:23 | |
*** sdake has joined #heat | 13:25 | |
*** pm90__ has joined #heat | 13:26 | |
*** sasuke_at_hp has joined #heat | 13:27 | |
*** sasuke_at_hp has quit IRC | 13:27 | |
*** sasuke_at_hp has joined #heat | 13:28 | |
*** sdake_ has quit IRC | 13:28 | |
*** pm90_ has quit IRC | 13:29 | |
zaneb | shardy, shadower: just finished reading scrollback. so we think we're in good shape? | 13:33 |
*** Marga_ has quit IRC | 13:35 | |
shardy | zaneb: I believe so, local functional tests pass, waiting for gate to confirm | 13:36 |
zaneb | \o/ | 13:37 |
shardy | zaneb: thanks for your help getting the init_complete part sorted | 13:37 |
shadower | zaneb: yep! | 13:37 |
zaneb | np, thanks for figuring out where all the subtleties were | 13:37 |
zaneb | that was a big help | 13:37 |
shardy | heh, I think I just reached the point where I'd looked at it for too long, glad we have it sorted now :) | 13:38 |
zaneb | lol, yeah, been there | 13:40 |
*** blomquisg has joined #heat | 13:40 | |
*** ttx has joined #heat | 13:40 | |
ttx | ohai, priority review on https://review.openstack.org/#/c/161306/, since it's blocking k3 | 13:40 |
zaneb | ttx: IMHO we should not block for that | 13:41 |
ttx | zaneb: asalkeld told me to, though | 13:42 |
zaneb | it's a building block for L, at the moment it's just dead code that is not called anywhere except for in unit tests | 13:42 |
zaneb | so I'm 100% comfortable landing it during feature freeze | 13:42 |
zaneb | or any other type of freeze | 13:42 |
ttx | zaneb: noted :) I'll wait a bit more and cut without it if needed | 13:43 |
zaneb | kk | 13:43 |
*** sarob has joined #heat | 13:43 | |
*** jprovazn has quit IRC | 13:43 | |
zaneb | (I will try to review it soon though) | 13:43 |
skraynev | btw, shardy: could you look at https://review.openstack.org/#/c/161306/ | 13:43 |
skraynev | mentioned above :) I have fixed docs | 13:44 |
skraynev | or may be it's not necessary due to zaneb initiative :) | 13:44 |
*** ishant has quit IRC | 13:47 | |
shardy | ttx: I'll review that now, as I already reviewed it a few hours ago | 13:47 |
*** sarob has quit IRC | 13:48 | |
ttx | col, thx | 13:49 |
*** ParsectiX has quit IRC | 13:50 | |
*** rakesh_hs has quit IRC | 13:51 | |
*** inc0 has quit IRC | 13:56 | |
*** serg_melikyan has quit IRC | 13:59 | |
*** spzala has joined #heat | 13:59 | |
*** InblEric has joined #heat | 14:00 | |
*** ParsectiX has joined #heat | 14:01 | |
*** smc7 has joined #heat | 14:07 | |
*** radez_g0n3 is now known as radez | 14:09 | |
*** serg_melikyan has joined #heat | 14:09 | |
*** tspatzier has quit IRC | 14:12 | |
*** randallburt has quit IRC | 14:14 | |
*** dhellmann has quit IRC | 14:19 | |
*** dhellmann has joined #heat | 14:19 | |
openstackgerrit | Merged openstack/heat: Fixes SoftwareConfigService to use Resource object https://review.openstack.org/165342 | 14:20 |
*** KarolynChambers has left #heat | 14:21 | |
*** vijendar has joined #heat | 14:26 | |
*** jprovazn has joined #heat | 14:27 | |
*** jcoufal_ has joined #heat | 14:29 | |
*** sdake_ has joined #heat | 14:32 | |
*** sdake_ has quit IRC | 14:32 | |
*** jcoufal has quit IRC | 14:32 | |
*** rpothier has joined #heat | 14:32 | |
*** sdake__ has joined #heat | 14:33 | |
*** sdake has quit IRC | 14:34 | |
*** achanda has joined #heat | 14:35 | |
*** tspatzier has joined #heat | 14:39 | |
zaneb | shardy: I just reviewed skraynev's patch... IMO it's not worth rushing it to get in before FF, but I guess we can always just fix it later | 14:42 |
* zaneb shrugs | 14:42 | |
*** tspatzier has quit IRC | 14:42 | |
openstackgerrit | Sergey Kraynev proposed openstack/heat: Fix missed docstring https://review.openstack.org/165855 | 14:42 |
*** spzala has quit IRC | 14:43 | |
*** achanda has quit IRC | 14:43 | |
skraynev | zaneb, shardy: sorry for the missed place ^ | 14:43 |
*** sarob has joined #heat | 14:44 | |
skraynev | zaneb: heh. now I am confused: should I fix all your comment in current patch or do another one? | 14:46 |
zaneb | skraynev: I'm not sure if anyone other than shardy can stop the current one from merging at this point | 14:48 |
*** InblEric1 has joined #heat | 14:48 | |
skraynev | A may try... | 14:48 |
zaneb | so I'd say start working on a separate patch, and if we end up not merging the current patch set you can squash them using git rebase -i | 14:48 |
skraynev | zaneb: yeah. sure | 14:49 |
*** kebray has joined #heat | 14:49 | |
*** sarob has quit IRC | 14:49 | |
*** timbyr_ has joined #heat | 14:49 | |
*** timbyr_ has quit IRC | 14:49 | |
*** sasuke_at_hp has quit IRC | 14:50 | |
*** InblEric has quit IRC | 14:50 | |
*** jdob has quit IRC | 14:51 | |
*** pm90__ has quit IRC | 14:51 | |
*** jdob has joined #heat | 14:53 | |
*** sarob has joined #heat | 14:56 | |
*** kebray has quit IRC | 15:00 | |
*** Raj2 has joined #heat | 15:01 | |
*** jdob has quit IRC | 15:02 | |
*** Raj2 has quit IRC | 15:03 | |
*** Raj2 has joined #heat | 15:03 | |
*** Drago has joined #heat | 15:05 | |
*** Drago has quit IRC | 15:06 | |
*** Drago has joined #heat | 15:06 | |
*** ParsectiX has quit IRC | 15:07 | |
*** sdake has joined #heat | 15:07 | |
*** alexheneveld has joined #heat | 15:09 | |
*** sasuke_at_hp has joined #heat | 15:09 | |
openstackgerrit | Sergey Kraynev proposed openstack/heat: Add functional test for AWS loadbalancer https://review.openstack.org/153192 | 15:10 |
*** spzala has joined #heat | 15:11 | |
*** Marga_ has joined #heat | 15:11 | |
*** sdake__ has quit IRC | 15:11 | |
*** kebray has joined #heat | 15:14 | |
*** serg_melikyan has quit IRC | 15:14 | |
shadower | shardy, zaneb: both patches failed with check-heat-dsvm-functional-mysql timing out again :-( | 15:16 |
zaneb | :( | 15:16 |
shadower | it does look like https://bugs.launchpad.net/heat/+bug/1433555 to me | 15:16 |
openstack | Launchpad bug 1433555 in heat "functional tests are taking too long and cause false failures" [High,In progress] - Assigned to Angus Salkeld (asalkeld) | 15:16 |
shadower | but maybe not? | 15:16 |
shadower | I didn't notice anything interesting in the logs | 15:16 |
shadower | would you guys check if something jumps out at you? | 15:17 |
*** Qiming_ has joined #heat | 15:18 | |
zaneb | shadower: I wasn't aware of that bug, but it does look very similar | 15:18 |
zaneb | can we just increase the timeout? | 15:18 |
*** sasuke_at_hp has quit IRC | 15:20 | |
shardy | shadower: Yeah, I took a look at the logs, for some reason all the *group tests are taking forever on replacement updates | 15:20 |
shardy | we're not using actual nova resources in them are we? | 15:20 |
* shardy checks | 15:20 | |
*** Qiming has quit IRC | 15:21 | |
*** sasuke_at_hp has joined #heat | 15:22 | |
*** pm90_ has joined #heat | 15:27 | |
*** sorantis has quit IRC | 15:28 | |
*** LiJiansheng has quit IRC | 15:28 | |
shardy | Looks like we're overriding it with a provider resource, but the test still takes a really long time, even running locally on bare-metal | 15:30 |
*** sdake_ has joined #heat | 15:30 | |
*** Qiming_ has quit IRC | 15:31 | |
*** sdcf has joined #heat | 15:31 | |
*** sorantis has joined #heat | 15:32 | |
zaneb | shardy: logs show that tests were continuing to pass even 2 minutes before the timeout hit | 15:33 |
*** sdake has quit IRC | 15:34 | |
zaneb | if the tests are just slow we should increase the timeout and try to improve them later imo | 15:34 |
zaneb | (of course if there are tests that never complete then we have to fix that, but I'd guess that is not the case here) | 15:34 |
zaneb | test_update_nochange_resource_needs_update [1049.335623s] | 15:36 |
openstackgerrit | Sergey Kraynev proposed openstack/heat: Use outputs section for getting ip addresses https://review.openstack.org/165882 | 15:36 |
zaneb | ouch | 15:36 |
zaneb | AutoScalingSignalTest.test_signal_with_policy_update [832.644550s] | 15:36 |
zaneb | all of AutoscalingGroupUpdatePolicyTest | 15:37 |
*** reed has joined #heat | 15:37 | |
*** sasuke_at_hp has quit IRC | 15:37 | |
zaneb | test_instance_group_update_no_replace [698.874717s] | 15:37 |
zaneb | test_instance_group_update_replace_huge_min_in_service [508.550756s] | 15:38 |
*** Tango has joined #heat | 15:38 | |
zaneb | those appear to be the worst offenders | 15:38 |
*** serg_melikyan has joined #heat | 15:39 | |
shardy | zaneb: Yeah, increasing the timeout seems a reasonable stopgap until we work out how to make things run faster | 15:42 |
pas-ha | I am rewriting neutron lb test to use cirros images, may be will become faster | 15:43 |
zaneb | hmm, I wonder if https://review.openstack.org/#/c/116336/ is slowing them down | 15:43 |
*** serg_melikyan has quit IRC | 15:44 | |
zaneb | suspicious that the bug asalkeld reported was from a test on that patch | 15:44 |
*** KanagarajM has joined #heat | 15:46 | |
*** Trozz has quit IRC | 15:46 | |
shardy | Hmm, it would indeed be interesting to see before/after numbers | 15:46 |
*** Trozz has joined #heat | 15:47 | |
shadower | so I suppose we should just recheck in the meantime? (for the hook patches) | 15:47 |
shardy | Although that enables better scalability, there's obviously some additional overhead in the single-node single-cpu case | 15:47 |
shardy | I wouldn't have thought it was *that* much though.. | 15:47 |
shardy | shadower: +1 | 15:47 |
openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Versioned-object StackLock https://review.openstack.org/165363 | 15:53 |
cmyster | zaneb: welcome back, | 16:02 |
zaneb | :) | 16:02 |
cmyster | zaneb: I'm confused do we have a meeting today ? | 16:02 |
cmyster | \or, now ? | 16:03 |
zaneb | we did but it was cancelled | 16:03 |
cmyster | @ home so the calendar items are usless | 16:05 |
*** Raj2 has quit IRC | 16:05 | |
*** Raj1 has joined #heat | 16:05 | |
cmyster | bbl | 16:06 |
*** jcoufal_ has quit IRC | 16:08 | |
*** Raj2 has joined #heat | 16:08 | |
*** Raj2 has quit IRC | 16:09 | |
*** Raj1 has quit IRC | 16:09 | |
*** Raj2 has joined #heat | 16:09 | |
*** sorantis has quit IRC | 16:10 | |
*** vijendar has quit IRC | 16:13 | |
*** vijendar has joined #heat | 16:14 | |
*** vijendar has quit IRC | 16:14 | |
*** InblEric has joined #heat | 16:15 | |
*** vijendar has joined #heat | 16:15 | |
*** InblEric1 has quit IRC | 16:17 | |
*** vijendar has quit IRC | 16:17 | |
*** ttx has left #heat | 16:21 | |
*** vijendar has joined #heat | 16:22 | |
*** InblEric1 has joined #heat | 16:23 | |
*** elynn_ has joined #heat | 16:23 | |
*** Marga_ has quit IRC | 16:23 | |
*** radez is now known as radez_g0n3 | 16:25 | |
*** InblEric has quit IRC | 16:26 | |
*** Raj2 has quit IRC | 16:26 | |
*** pitr-ch has quit IRC | 16:30 | |
*** sarob has quit IRC | 16:34 | |
*** ifarkas has quit IRC | 16:35 | |
*** BManojlovic has quit IRC | 16:44 | |
Tango | shadower: Hi Tomas | 16:47 |
*** kaufer has quit IRC | 16:49 | |
shadower | Tango: hey Ton :-) | 16:49 |
*** ttx has joined #heat | 16:49 | |
*** kaufer has joined #heat | 16:49 | |
*** hdd has joined #heat | 16:49 | |
ttx | zaneb: that patch I was waiting on failed testing, so I think I'll tag without it. You said it's not critical anyway | 16:49 |
Tango | I am trying out the hook patch (sorry I fell out of the loop for awhile). Is the resource_status_reason supposed to show something when a resource is paused? | 16:50 |
shadower | Tango: yeah | 16:50 |
shadower | well | 16:50 |
shadower | no, I don't think so | 16:50 |
shadower | sorry | 16:50 |
Tango | shadower: it's showing blank | 16:51 |
shadower | it used to, but now the code changed not to change the resource status at all | 16:51 |
shadower | Tango: it emits an event, but it doesn't change the status at any point | 16:51 |
Tango | shadower: ah I see, so I should look at the event list | 16:51 |
shadower | Tango: check out _break_if_required in resource.py | 16:51 |
shadower | ya | 16:51 |
shadower | Tango: I'm thinking we want to have a programatic way of detecting breakpoints, but that can be a followup | 16:52 |
shadower | right now I'm just trying to get this merged ;-) | 16:52 |
Tango | shadower: yep, agree | 16:52 |
Tango | shadower: So with this, we can start to write a simple gdb-like debugger | 16:54 |
shadower | yeah | 16:54 |
Tango | shadower: The actual interfact to heat can change, but the user interface can stay the same | 16:54 |
shadower | yep, that's the plan :-) | 16:54 |
shadower | so the tripleo check succeeds an hour before the normal tests do. And they still may time out | 16:56 |
shadower | super scary ^ | 16:56 |
shadower | I was thinking "so how about we increase the timeout to something crazy like 2 hours". Then I realised that was already the case | 16:58 |
*** Raj1 has joined #heat | 16:58 | |
*** Raj2 has joined #heat | 16:59 | |
Tango | shadower: So typically a debugger will keep track of the breakpoints it sets, so if we can query the event list and somehow figure out which breakpoints are hit, then it should be sufficient | 16:59 |
*** Raj2 has quit IRC | 16:59 | |
*** Raj1 has quit IRC | 16:59 | |
*** Raj1 has joined #heat | 16:59 | |
shadower | Tango: yeah, but I'd like something better then checking the event string. | 16:59 |
shadower | plus it's kind of hard because we don't have a recursive event list now | 16:59 |
shadower | so you have to check all nested stacks | 17:00 |
Tango | shadower: OK, that would certainly help | 17:00 |
*** derekh has quit IRC | 17:02 | |
*** jistr has quit IRC | 17:07 | |
*** harlowja_away is now known as harlowja_ | 17:09 | |
*** Marga_ has joined #heat | 17:13 | |
*** Marga_ has quit IRC | 17:17 | |
ttx | tagging heat kilo-3 | 17:18 |
*** dulek has quit IRC | 17:18 | |
*** pitr-ch has joined #heat | 17:24 | |
ttx | heat kilo-3 tagged: https://launchpad.net/heat/+milestone/kilo-3 | 17:26 |
*** achanda has joined #heat | 17:28 | |
*** achanda has quit IRC | 17:28 | |
*** achanda has joined #heat | 17:28 | |
ryansb | shadower: not sure adding a status is the way to go for that | 17:33 |
ryansb | I kind of prefer it to just be an event and use the existing statuses we have | 17:33 |
*** Raj1 has quit IRC | 17:33 | |
*** coolsvap is now known as coolsvap|afk | 17:33 | |
*** packet has joined #heat | 17:35 | |
shadower | ryansb: yeah, I didn't mean to say that adding a status is the way to go | 17:35 |
shadower | but it'd be better to make this data more readily available | 17:35 |
shadower | e.g. right now, if you want to check whether your stack is waiting on a breakpoint, you have to list events of the entire nested stack tree | 17:36 |
shadower | and parse a string that's potentialy translated and/or subject to change | 17:36 |
ryansb | true | 17:37 |
*** pm90_ has quit IRC | 17:43 | |
*** pm90_ has joined #heat | 17:44 | |
*** Marga_ has joined #heat | 17:44 | |
*** rakesh_hs has joined #heat | 17:45 | |
zaneb | ttx: ack, thanks | 17:47 |
*** Marga__ has joined #heat | 17:47 | |
*** sdcf has quit IRC | 17:48 | |
*** Marga_ has quit IRC | 17:48 | |
zaneb | so it looks like a high proportion of patches are going to be timing out on the functional test jobs, judging by the fact that https://review.openstack.org/161306 also failed | 17:48 |
zaneb | which makes me suspect even more strongly that the RPC nested stacks thing is the cause | 17:49 |
*** Raj1 has joined #heat | 17:52 | |
*** Raj2 has joined #heat | 17:52 | |
*** smc7 has quit IRC | 17:56 | |
*** Raj1 has quit IRC | 17:56 | |
*** alexheneveld has quit IRC | 17:57 | |
*** Raj2 has quit IRC | 17:57 | |
*** Raj1 has joined #heat | 17:58 | |
*** kaufer1 has joined #heat | 17:58 | |
*** kaufer has quit IRC | 17:59 | |
*** reed has quit IRC | 17:59 | |
*** boris-42 has quit IRC | 18:02 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/heat: Use CirrOS image in Neutron Loadbalancer test https://review.openstack.org/165944 | 18:03 |
*** GonZo2K has joined #heat | 18:09 | |
*** GonZo2K has quit IRC | 18:09 | |
*** GonZo2K has joined #heat | 18:09 | |
*** sdake_ has quit IRC | 18:09 | |
*** elynn_ has quit IRC | 18:10 | |
pas-ha | guys, how should I regenerate heat_integrationtest.conf.sample? | 18:10 |
*** sdake has joined #heat | 18:11 | |
*** coolsvap|afk has quit IRC | 18:11 | |
*** Marga__ has quit IRC | 18:11 | |
*** Marga_ has joined #heat | 18:12 | |
*** Marga_ has quit IRC | 18:12 | |
*** fd has joined #heat | 18:12 | |
*** Marga_ has joined #heat | 18:13 | |
pas-ha | nm, problem solved | 18:13 |
*** KanagarajM has quit IRC | 18:14 | |
*** Marga_ has quit IRC | 18:14 | |
*** Marga_ has joined #heat | 18:15 | |
*** KanagarajM has joined #heat | 18:15 | |
*** rakesh_hs has quit IRC | 18:19 | |
*** cdent has quit IRC | 18:19 | |
*** Drago1 has joined #heat | 18:21 | |
*** Drago1 has quit IRC | 18:22 | |
*** Drago1 has joined #heat | 18:22 | |
*** Drago has quit IRC | 18:23 | |
*** Drago1 has quit IRC | 18:24 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/heat: Use CirrOS image in Neutron Loadbalancer test https://review.openstack.org/165944 | 18:26 |
*** reed has joined #heat | 18:29 | |
*** spzala has quit IRC | 18:30 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/heat: Update integrations tests config sample https://review.openstack.org/165958 | 18:32 |
*** blomquisg has quit IRC | 18:33 | |
*** fd has quit IRC | 18:34 | |
*** pm90__ has joined #heat | 18:38 | |
*** blomquisg has joined #heat | 18:40 | |
*** Drago has joined #heat | 18:41 | |
*** pm90_ has quit IRC | 18:41 | |
*** hdd has quit IRC | 18:42 | |
*** Marga_ has quit IRC | 18:42 | |
*** jprovazn has quit IRC | 18:43 | |
*** kebray has quit IRC | 18:44 | |
*** Marga_ has joined #heat | 18:45 | |
*** JK__ has joined #heat | 18:46 | |
*** smc7 has joined #heat | 18:46 | |
*** pm90__ has quit IRC | 18:48 | |
*** pm90_ has joined #heat | 18:49 | |
*** spzala has joined #heat | 18:52 | |
*** radez_g0n3 is now known as radez | 18:52 | |
*** randallburt has joined #heat | 19:05 | |
*** randallburt has quit IRC | 19:06 | |
*** randallburt has joined #heat | 19:06 | |
*** Marga_ has quit IRC | 19:06 | |
*** JK__ has quit IRC | 19:08 | |
*** dsneddon is now known as dsneddon_lunch | 19:09 | |
*** elynn_ has joined #heat | 19:10 | |
*** elynn_ has quit IRC | 19:16 | |
pas-ha | g'night all | 19:16 |
*** pas-ha has quit IRC | 19:17 | |
*** ttx has left #heat | 19:22 | |
*** Marga_ has joined #heat | 19:24 | |
*** Marga_ has quit IRC | 19:25 | |
*** hdd has joined #heat | 19:26 | |
*** kaufer1 has quit IRC | 19:30 | |
*** kaufer has joined #heat | 19:30 | |
stevebaker | morning | 19:39 |
stevebaker | zaneb: I'm suspecting regression on the RPC patch too https://bugs.launchpad.net/heat/+bug/1433555/comments/3 | 19:40 |
openstack | Launchpad bug 1433555 in heat "functional tests are taking too long and cause false failures" [Critical,In progress] - Assigned to Angus Salkeld (asalkeld) | 19:40 |
zaneb | stevebaker: on a call atm | 19:40 |
stevebaker | kk | 19:40 |
*** dsneddon_lunch is now known as dsneddon | 19:43 | |
*** KanagarajM has quit IRC | 19:44 | |
*** sdake_ has joined #heat | 19:48 | |
*** metral is now known as metral_zzz | 19:50 | |
*** sdake has quit IRC | 19:51 | |
*** sasuke_at_hp has joined #heat | 19:51 | |
*** sdake has joined #heat | 19:53 | |
*** sdake_ has quit IRC | 19:55 | |
*** Marga_ has joined #heat | 19:55 | |
*** sasuke_at_hp has quit IRC | 19:56 | |
zaneb | stevebaker: back | 19:58 |
zaneb | oh great, Heisenbugs | 19:59 |
zaneb | that's all we need | 19:59 |
zaneb | hmm, apparently we also have the typical pre-release problem with slow nodes as well: https://bugs.launchpad.net/openstack-ci/+bug/1311066 | 20:06 |
openstack | Launchpad bug 1311066 in OpenStack-Gate "Some nodes allocated in node pool are very very slow" [High,Confirmed] | 20:06 |
*** shardy has quit IRC | 20:07 | |
*** Marga_ has quit IRC | 20:08 | |
*** kebray has joined #heat | 20:11 | |
*** elynn_ has joined #heat | 20:13 | |
*** elynn_ has quit IRC | 20:19 | |
stevebaker | ug, I really need a recursive heat event-list | 20:26 |
*** dims has quit IRC | 20:34 | |
*** Marga_ has joined #heat | 20:38 | |
*** randallburt1 has joined #heat | 20:41 | |
*** randallburt has quit IRC | 20:42 | |
*** radez is now known as radez_g0n3 | 20:43 | |
*** randallburt1 has quit IRC | 20:47 | |
*** jaypipes has quit IRC | 20:50 | |
*** Marga_ has quit IRC | 20:52 | |
*** Marga_ has joined #heat | 20:52 | |
*** sasuke_at_hp has joined #heat | 20:53 | |
*** Marga_ has quit IRC | 20:54 | |
*** dulek has joined #heat | 20:54 | |
*** Marga_ has joined #heat | 20:54 | |
*** sasuke_at_hp has quit IRC | 20:58 | |
*** dulek has quit IRC | 20:59 | |
*** blomquisg has quit IRC | 21:01 | |
shadower | zaneb, stevebaker: all tests on https://review.openstack.org/#/c/146123/ and https://review.openstack.org/#/c/164652/ have passed \o/ | 21:05 |
shadower | can I get a +A? | 21:05 |
zaneb | \o/ !!! | 21:05 |
stevebaker | approved | 21:06 |
shadower | thanks! | 21:07 |
*** pitr-ch has quit IRC | 21:07 | |
*** InblEric1 has quit IRC | 21:14 | |
*** Marga_ has quit IRC | 21:14 | |
*** InblEric1 has joined #heat | 21:15 | |
*** Marga_ has joined #heat | 21:15 | |
stevebaker | Could you please look at this? http://paste.openstack.org/show/193688/ | 21:15 |
*** elynn_ has joined #heat | 21:16 | |
stevebaker | weird delay after 07:37Z. All resources are UPDATE_COMPLETE in 6 seconds, but JobServerGroup doesn't go to UPDATE_COMPLETE until 09:16Z | 21:17 |
*** radez_g0n3 is now known as radez | 21:17 | |
*** elynn_ has quit IRC | 21:21 | |
*** aweiteka has quit IRC | 21:22 | |
*** achanda has quit IRC | 21:28 | |
Tango | shadower: Buy you a beer at Vancouver :) | 21:29 |
shadower | :-) | 21:30 |
shadower | wouldn't have done it without shardy and zaneb's help | 21:30 |
Tango | ah then beer for everyone :) | 21:34 |
*** asalkeld has joined #heat | 21:34 | |
asalkeld | morning | 21:34 |
shadower | evening :-) | 21:34 |
*** achanda has joined #heat | 21:37 | |
*** InblEric1 has quit IRC | 21:37 | |
*** alexpilotti has joined #heat | 21:39 | |
*** dims has joined #heat | 21:39 | |
*** alexheneveld has joined #heat | 21:41 | |
asalkeld | shadower: did you get your patch in? | 21:41 |
asalkeld | looks like no | 21:42 |
asalkeld | ok, coffee time | 21:43 |
shadower | asalkeld: they're approved, waiting for the gate jobs now | 21:45 |
asalkeld | ok | 21:45 |
*** Drago1 has joined #heat | 21:49 | |
*** Drago has quit IRC | 21:49 | |
*** Drago1 has quit IRC | 21:49 | |
Tango | asalkeld: Hi Angus | 21:49 |
asalkeld | hi Tango | 21:49 |
*** blues-man has joined #heat | 21:49 | |
asalkeld | i am just organising breakfast in-n-out | 21:50 |
Tango | asalkeld: when you have a chance, can you take another look at https://review.openstack.org/#/c/154618/ ? | 21:50 |
*** rpothier has quit IRC | 21:51 | |
asalkeld | ok | 21:52 |
*** achanda has quit IRC | 21:53 | |
Tango | asalkeld: Thanks! | 21:54 |
*** kaufer has quit IRC | 21:56 | |
*** hdd has quit IRC | 21:59 | |
*** achanda has joined #heat | 22:01 | |
*** lkarm has left #heat | 22:05 | |
*** dsneddon is now known as dsneddon_afk | 22:07 | |
*** jamielennox|away is now known as jamielennox | 22:09 | |
stevebaker | asalkeld: you could be right about refresh() not working, but I wonder if it is our object refresh() which is the issue rather than oslo | 22:11 |
asalkeld | yeah, might be | 22:11 |
asalkeld | stevebaker: also looking at that refresh, it returns the object but doesn't refresh the local object | 22:13 |
asalkeld | so if I obj.refresh(), then use obj, it will be the same obj | 22:14 |
asalkeld | weird | 22:14 |
stevebaker | right, so it needs to be obj = obj.refresh() | 22:14 |
*** packet has quit IRC | 22:15 | |
stevebaker | hmm, maybe not. I haven't really looked into this versioned object stuff yet | 22:16 |
*** elynn_ has joined #heat | 22:17 | |
*** dims has quit IRC | 22:19 | |
*** Marga_ has quit IRC | 22:21 | |
*** Marga_ has joined #heat | 22:21 | |
*** elynn_ has quit IRC | 22:21 | |
*** Raj1 has quit IRC | 22:23 | |
asalkeld | brb, school run | 22:27 |
stevebaker | i may have a fix when you're back | 22:29 |
*** Raj1 has joined #heat | 22:29 | |
*** boris-42 has joined #heat | 22:36 | |
*** sasuke_at_hp has joined #heat | 22:41 | |
openstackgerrit | Steve Baker proposed openstack/heat: Call db refresh in Stack object refresh https://review.openstack.org/166030 | 22:42 |
stevebaker | fix for critical bug ^ | 22:42 |
stevebaker | (well, an attempt at a fix, works for my sample size) | 22:43 |
*** radez is now known as radez_g0n3 | 22:44 | |
*** Raj1 has quit IRC | 22:45 | |
*** sasuke_at_hp has quit IRC | 22:46 | |
*** smc7 has quit IRC | 22:46 | |
*** Raj1 has joined #heat | 22:46 | |
asalkeld | ooo well done stevebaker | 22:48 |
asalkeld | let's watch the times | 22:48 |
*** vijendar has quit IRC | 22:49 | |
*** Raj2 has joined #heat | 22:50 | |
*** Raj1 has quit IRC | 22:51 | |
*** pm90_ has quit IRC | 22:53 | |
asalkeld | stevebaker: did you see zaneb zabana query | 22:53 |
asalkeld | they did seem to start with oslo reviews | 22:54 |
openstackgerrit | Merged openstack/heat: Add breakpoint support https://review.openstack.org/146123 | 22:54 |
asalkeld | stevebaker: resource also has a refresh | 22:58 |
openstackgerrit | Angus Salkeld proposed openstack/heat: Correct confusing "cls" argument on non-static method https://review.openstack.org/166036 | 23:10 |
*** Raj2 has quit IRC | 23:10 | |
*** Raj1 has joined #heat | 23:11 | |
*** blues-man has quit IRC | 23:16 | |
asalkeld | https://etherpad.openstack.org/p/heat-timeout-bug | 23:17 |
*** elynn_ has joined #heat | 23:18 | |
*** elynn_ has quit IRC | 23:22 | |
*** chlong has joined #heat | 23:27 | |
*** pm90_ has joined #heat | 23:31 | |
*** pm90_ has quit IRC | 23:31 | |
*** pm90_ has joined #heat | 23:31 | |
ramishra | stevebaker: Hi!.. thanks for the feedback on https://review.openstack.org/#/c/164572/4, I've reduced on the image size to 400+MB. I'll upload the uploaded patch. | 23:32 |
ramishra | stevebaker: could not test it yesterday as I had to rebuild my setup. | 23:33 |
*** Marga_ has quit IRC | 23:36 | |
*** Marga_ has joined #heat | 23:36 | |
*** Drago has joined #heat | 23:39 | |
*** Drago has quit IRC | 23:40 | |
*** Drago has joined #heat | 23:41 | |
*** tochi has joined #heat | 23:42 | |
*** Drago has quit IRC | 23:42 | |
*** Drago has joined #heat | 23:42 | |
*** kebray has quit IRC | 23:43 | |
*** dsneddon_afk is now known as dsneddon | 23:45 | |
*** radez_g0n3 is now known as radez | 23:45 | |
*** Raj1 has quit IRC | 23:46 | |
*** derekh has joined #heat | 23:50 | |
*** derekh has quit IRC | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!