*** sjmc7 has quit IRC | 00:02 | |
SpamapS | stevebaker: cores can resurrect abandoned changes IIRC | 00:03 |
---|---|---|
stevebaker | oh, I see a restore button | 00:04 |
stevebaker | SpamapS: I can replicate the lock issue locally, bisecting now | 00:04 |
*** bgorski has quit IRC | 00:08 | |
SpamapS | stevebaker: woot | 00:08 |
SpamapS | stevebaker: you know, thinking more about it, I think if we succeed in setting user_creds_id = NULL when the user_creds are gone, that will actually fix the gate problem. | 00:09 |
SpamapS | stevebaker: but the bigger problem is that we are likely interrupting deletes wrong. | 00:09 |
stevebaker | SpamapS: I'll test that once I'm done bisecting | 00:09 |
*** jreypo has quit IRC | 00:10 | |
stevebaker | apparently this is the culprit https://review.openstack.org/#/c/97569/ | 00:10 |
*** dims__ has quit IRC | 00:11 | |
*** dims has joined #heat | 00:11 | |
SpamapS | stevebaker: that is.. odd | 00:14 |
SpamapS | stevebaker: is it possible that just changed the race winner? | 00:15 |
*** Qiming has joined #heat | 00:16 | |
stevebaker | SpamapS: seems likely, doing an unconditional authenticate would hit keystone, which would lead to an eventlet yield | 00:16 |
stevebaker | SpamapS: your patch doesn't help much, but reverting that change does. I suggest we revert urgently | 00:17 |
stevebaker | zaneb: ^ | 00:18 |
*** dims has quit IRC | 00:21 | |
*** bgorski has joined #heat | 00:22 | |
SpamapS | stevebaker: indeed! | 00:22 |
SpamapS | stevebaker: and I'm not surprised that the None check doesn't help.. it is just a symptom of the failed race. | 00:23 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions" https://review.openstack.org/99517 | 00:23 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions" https://review.openstack.org/99517 | 00:23 |
SpamapS | stevebaker: can you get rid of that bare space too? | 00:24 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions" https://review.openstack.org/99517 | 00:24 |
stevebaker | jogo: https://review.openstack.org/#/c/99517/ is the revert which should fix bug 1306029 | 00:24 |
uvirtbot | Launchpad bug 1306029 in heat "tempest.api.orchestration.stacks.test_stacks times out during tearDownClass" [Critical,Triaged] https://launchpad.net/bugs/1306029 | 00:24 |
*** zhiyan_ is now known as zhiyan | 00:27 | |
SpamapS | stevebaker: IMO, it will _hide_ bug 1306029 .. but yes. :) | 00:27 |
uvirtbot | Launchpad bug 1306029 in heat "tempest.api.orchestration.stacks.test_stacks times out during tearDownClass" [Critical,Triaged] https://launchpad.net/bugs/1306029 | 00:27 |
*** matsuhashi has joined #heat | 00:27 | |
*** dims has joined #heat | 00:27 | |
*** nosnos has joined #heat | 00:32 | |
stevebaker | SpamapS: yeah, it hides the bug, but we know about it now | 00:37 |
stevebaker | its still Critical | 00:38 |
*** randallburt has joined #heat | 00:42 | |
*** randallburt has quit IRC | 00:43 | |
*** Qiming has quit IRC | 00:43 | |
*** randallburt has joined #heat | 00:43 | |
*** m_22 has joined #heat | 00:46 | |
*** liusheng has quit IRC | 00:51 | |
*** liusheng has joined #heat | 00:52 | |
*** yanyan has joined #heat | 00:52 | |
zaneb | stevebaker: what was the test you used to bisect? | 00:57 |
stevebaker | zaneb: tempest test_stacks | 00:58 |
stevebaker | zaneb: testr run --until-failure test_stacks | 00:58 |
zaneb | you just ran it multiple times? | 00:58 |
zaneb | for each patch, I mean | 00:58 |
stevebaker | zaneb: the test doesn't actually fail. I guess the check job fails because of the ERROR entries in the heat-engine log | 01:01 |
zaneb | oh | 01:01 |
zaneb | but there is a bug where it is actually timing out, right? | 01:01 |
zaneb | do we think that's a different cause? | 01:01 |
stevebaker | zaneb: actually I lie, I modified test_stack_crud_no_resources to do 4 deletes in succession. Anything less than 3 doesn't trigger it | 01:04 |
jogo | stevebaker: sweet | 01:05 |
stevebaker | zaneb: Yeah, I'm not sure what the test timeout failure is, but I'd wager 500ml beer its the same problem | 01:06 |
*** randallburt has quit IRC | 01:14 | |
*** gokrokve has joined #heat | 01:14 | |
cyli | stevebaker: so therve's sync with oslo-incubator also obviates this one, right? https://review.openstack.org/#/c/77472/ (which just seems to be a sync of the config module) | 01:15 |
stevebaker | cyli: I'm guessing so. You should sync with him when he is awake | 01:15 |
cyli | stevebaker: ok. all the other active reviews depend on the previous ones (e.g. https://review.openstack.org/#/c/72798/ depends on https://review.openstack.org/#/c/77472/) - if indeed https://review.openstack.org/#/c/77472/ should be abandoned, do I need to do anything to change the dependencies? | 01:17 |
cyli | (or will it automatically change if I keep the change ID but rebase https://review.openstack.org/#/c/72798 so it doesn't have the parent commit from https://review.openstack.org/#/c/77472/ ?) | 01:20 |
stevebaker | cyli: you should pull down the whole series and do an interactive rebase to remove the abandoned ones | 01:20 |
cyli | stevebaker: oh, I see, great. :) thank you so much for yours and zaneb's patience explaining everything! | 01:21 |
zaneb | cyli: what stevebaker said, or just cherry-pick the ones you need onto master | 01:21 |
zaneb | the latter might be easier, because there will no doubt be heaps of conflicts | 01:22 |
zaneb | git fetch https://review.openstack.org/openstack/heat refs/changes/<random_number>/<review_number>/<patchset_number> && git cherry-pick FETCH_HEAD | 01:23 |
cyli | zaneb: thanks! | 01:23 |
zaneb | except don't actually use a random number, get it from the Dowload section on the review ;) | 01:24 |
zaneb | Download* | 01:24 |
cyli | :) ok, thanks! | 01:25 |
zaneb | if you click on 'cherry-pick' it gives you the command you can copy | 01:25 |
*** tiantian has joined #heat | 01:25 | |
cyli | oh nice | 01:25 |
*** yanyan has quit IRC | 01:27 | |
zaneb | oh, the random number is the last two digits of the review id | 01:29 |
zaneb | I guess that makes sense for distributing them evenly | 01:29 |
*** gokrokve has quit IRC | 01:30 | |
*** Qiming has joined #heat | 01:30 | |
*** nati_ueno has quit IRC | 01:30 | |
openstackgerrit | Ying Li proposed a change to openstack/heat: This is part of the refactoring to fix the inheritance hierarchy of AutoScaleResourceGroup. In this case, attempting to move some common functionality out of the inheritance hierarchy entirely (perhaps to the heat/scaling module). https://review.openstack.org/99470 | 01:30 |
*** gokrokve has joined #heat | 01:30 | |
*** randallburt has joined #heat | 01:31 | |
*** Qiming has quit IRC | 01:32 | |
zaneb | cyli: btw that ^ commit is missing a headline, which is why pep8 is failing it. https://wiki.openstack.org/wiki/GitCommitMessages is a really helpful page to read | 01:33 |
zaneb | everyone involved in OpenStack should read that wiki page once a month imo ;D | 01:34 |
*** gokrokve has quit IRC | 01:35 | |
*** m_22 has quit IRC | 01:35 | |
cyli | zaneb: oh ok! I thought it was just that the commit message line length was too long (as per the pep8 failure). Thanks for the link! | 01:36 |
*** ericgoncz has joined #heat | 01:38 | |
*** alexheneveld has quit IRC | 01:39 | |
*** zns has quit IRC | 01:39 | |
*** DandyPandy has joined #heat | 01:39 | |
*** ericgoncz_ has joined #heat | 01:40 | |
*** DandyPandy has quit IRC | 01:40 | |
*** DandyPandy has joined #heat | 01:41 | |
*** rwsu has quit IRC | 01:42 | |
*** ericgoncz has quit IRC | 01:44 | |
*** ericgoncz_ is now known as ericgoncz | 01:44 | |
*** bgorski has quit IRC | 01:44 | |
openstackgerrit | Ying Li proposed a change to openstack/heat: Move calculating new capacity to a function https://review.openstack.org/99470 | 01:46 |
*** alexheneveld has joined #heat | 01:48 | |
*** rpothier has left #heat | 01:51 | |
*** alexheneveld has quit IRC | 02:02 | |
*** achampion has joined #heat | 02:05 | |
*** rpothier has joined #heat | 02:06 | |
*** aweiteka has quit IRC | 02:08 | |
elynn_ | morning all! | 02:15 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add quotas module with driver and resources https://review.openstack.org/96716 | 02:16 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add db APIs to access quota https://review.openstack.org/96697 | 02:16 |
*** rpothier_ has joined #heat | 02:18 | |
*** ericgoncz has quit IRC | 02:19 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add RPC methods for quota actions https://review.openstack.org/97001 | 02:19 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add APIs for quota https://review.openstack.org/98026 | 02:20 |
*** dims has quit IRC | 02:40 | |
*** ramishra has joined #heat | 02:56 | |
*** gokrokve has joined #heat | 03:05 | |
*** dims_ has joined #heat | 03:07 | |
openstackgerrit | A change was merged to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions" https://review.openstack.org/99517 | 03:08 |
*** dims_ has quit IRC | 03:12 | |
stevebaker | that was quick | 03:13 |
SpamapS | stevebaker: cause it fixed the gate ;) | 03:14 |
SpamapS | stevebaker: now to find the race | 03:15 |
* SpamapS goes to eat dinner | 03:15 | |
*** piyush has joined #heat | 03:15 | |
*** zaneb has quit IRC | 03:16 | |
*** rpothier has quit IRC | 03:17 | |
*** piyush has quit IRC | 03:19 | |
*** cmyster has joined #heat | 03:25 | |
*** cmyster has joined #heat | 03:25 | |
cmyster | morning | 03:26 |
*** sgordon has joined #heat | 03:29 | |
*** zhiyan is now known as zhiyan_ | 03:30 | |
*** akuznetsov has joined #heat | 03:30 | |
*** zaneb has joined #heat | 03:31 | |
*** ramishra has quit IRC | 03:33 | |
*** arbylee has quit IRC | 03:37 | |
*** zhiyan_ is now known as zhiyan | 03:41 | |
*** nosnos has quit IRC | 03:44 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement NovaFloatingIpAssociation res updatable https://review.openstack.org/93389 | 03:47 |
*** ramishra has joined #heat | 03:49 | |
*** zns has joined #heat | 03:51 | |
*** achampio1 has joined #heat | 03:51 | |
zaneb | SpamapS: still about? | 03:53 |
*** achampion has quit IRC | 03:54 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Add parameter type validation before stack-create https://review.openstack.org/99561 | 03:55 |
elynn_ | hi stevebaker , could you review this patch again? I just add some tests for it. https://review.openstack.org/#/c/98580/ | 03:56 |
*** elynn_ has quit IRC | 03:57 | |
*** elynn has joined #heat | 03:58 | |
*** zns has quit IRC | 03:59 | |
*** zns has joined #heat | 04:00 | |
*** dstufft has joined #heat | 04:02 | |
dstufft | Hey, is there any way to have an incrementing value in a hot parameter? specifically the servername so it can be foo1, foo2, foo3, etc. | 04:03 |
*** arbylee has joined #heat | 04:04 | |
stevebaker | dstufft: currently no | 04:05 |
dstufft | ok | 04:05 |
dstufft | thanks | 04:06 |
stevebaker | dstufft: can you just use the default server names? | 04:07 |
dstufft | stevebaker: maybe, will have to poke a bit :) | 04:07 |
stevebaker | dstufft: they attempt to be a reasonable combination of readable and unique | 04:08 |
dstufft | ok | 04:08 |
*** akuznetsov has quit IRC | 04:09 | |
*** elynn has quit IRC | 04:09 | |
*** elynn has joined #heat | 04:10 | |
*** matsuhashi has quit IRC | 04:11 | |
randallburt | stevebaker, dstufft there's this in review for ResourceGroup that would allow you to do something like that: https://review.openstack.org/#/c/88636/ | 04:11 |
*** DandyPandy has quit IRC | 04:12 | |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Fix race condition deleting in-progress stack https://review.openstack.org/99562 | 04:13 |
*** lazy_prince has joined #heat | 04:13 | |
dstufft | stevebaker: kinda ugly, but it works well enough I think. thanks | 04:13 |
dstufft | randallburt: cool! | 04:13 |
*** KanagarajM has joined #heat | 04:15 | |
tiantian | <elynn_>: See my comments:) | 04:15 |
*** harlowja is now known as harlowja_away | 04:16 | |
zaneb | stevebaker: I may have a culprit on the race condition front, see above ^^^ | 04:17 |
*** andersonvom has joined #heat | 04:17 | |
*** arbylee has quit IRC | 04:18 | |
stevebaker | zaneb: cool, I'll thrash it locally soon | 04:19 |
zaneb | eeexcellent | 04:19 |
*** akuznetsov has joined #heat | 04:20 | |
stevebaker | zaneb: is eventlet.sleep OK to do inside a task? | 04:22 |
zaneb | by task you mean heat.engine.scheduler task? | 04:22 |
zaneb | it's... OK, but rarely what you want | 04:22 |
zaneb | it will allow other requests to run, but not other things in the same request that should be happening in parallel | 04:23 |
*** randallburt has quit IRC | 04:23 | |
stevebaker | zaneb: yes, this is for exponential backoff of action retries | 04:24 |
zaneb | yeah, that would be bad | 04:24 |
zaneb | don't do that ;) | 04:24 |
stevebaker | any other ways I could sleep? | 04:25 |
zaneb | instead of "sleep(n)", do "for i in xrange(n): yield" | 04:25 |
stevebaker | does a yield imply 1 second? | 04:26 |
zaneb | yes | 04:26 |
zaneb | well, sort of | 04:26 |
stevebaker | yeah, close enough | 04:27 |
zaneb | the top level thing runs until _all_ of the tasks have yielded, then sleeps for 1s | 04:27 |
zaneb | so it depends how much time other tasks take up | 04:27 |
zaneb | but for your purposes, close enough | 04:27 |
*** dstufft has left #heat | 04:30 | |
zaneb | ok, half past midnight. time for bed | 04:36 |
*** zns has quit IRC | 04:36 | |
*** zns has joined #heat | 04:39 | |
*** andersonvom has quit IRC | 04:45 | |
*** saju_m has joined #heat | 04:47 | |
*** matsuhashi has joined #heat | 04:48 | |
*** nosnos has joined #heat | 04:48 | |
*** gokrokve has quit IRC | 04:53 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/96507 | 04:54 |
*** achampion has joined #heat | 05:08 | |
*** achampio1 has quit IRC | 05:10 | |
elynn | Hi tiantian | 05:16 |
*** zns has quit IRC | 05:16 | |
*** zns has joined #heat | 05:16 | |
elynn | I think you misunderstand my logic in this patch https://review.openstack.org/#/c/98580/ | 05:16 |
*** saju_m has quit IRC | 05:18 | |
*** daneyon_ has quit IRC | 05:29 | |
*** sgordon has quit IRC | 05:36 | |
*** akuznetsov has quit IRC | 05:40 | |
*** mandira has joined #heat | 05:45 | |
*** nkhare has joined #heat | 05:47 | |
mandira | Hello all............ | 05:48 |
mandira | <therve>hi | 05:49 |
mandira | <therve>i want to attach volume to this web server instance in this template ....http://paste.openstack.org/show/83751/.....can you explain me how can i do this ? | 05:54 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Add parameter type validation before stack-create https://review.openstack.org/99561 | 05:59 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/89750 | 06:02 |
sirushti | mandira, Have you looked at the AWS::EC2::VolumeAttachment resource? | 06:08 |
mandira | <sirushti>i have seen that and added AWS::EC2::VolumeAttachment this to my template but got error | 06:10 |
*** akuznetsov has joined #heat | 06:11 | |
sirushti | mandira, what's the error? | 06:12 |
tiantian | <elynn>: hi lauch time was | 06:14 |
elynn | tiantian, I reply your comment. : ) | 06:15 |
mandira | <sirushti>property error availability referance attribute is incorrect | 06:15 |
mandira | <sirushti>i found that in my "launchconfig" resources ,availability zone property is not there in aws template format.... | 06:17 |
sirushti | mandira, could you post the template and the exact error on paste.openstack.org and post the corresponding link here? | 06:21 |
mandira | <sirushti>ok | 06:22 |
*** jprovazn has joined #heat | 06:24 | |
*** akuznetsov has quit IRC | 06:25 | |
mandira | <sirushti>this is my template "http://paste.openstack.org/show/83753/" | 06:27 |
mandira | and my error is "Create_Failed: Resource CREATE failed: StackValidationFailed: Property error : DataVolume: AvailabilityZone The Referenced Attribute (LaunchConfig AvailabilityZone) is incorrect." | 06:27 |
sirushti | mandira, could you try "Fn::GetAZs" : "" instead of what you've entered for your DataVolume resource? | 06:39 |
mandira | <sirushti>i am trying that | 06:45 |
*** jamiehannaford has joined #heat | 06:46 | |
therve | Good morning! | 06:46 |
cyli | therve: morning! | 06:46 |
therve | cyli, Hi! | 06:47 |
mandira | <sirushti> not working.. :( getting error Error: ERROR: Property error : DataVolume: AvailabilityZone Value must be a string | 06:47 |
*** akuznetsov has joined #heat | 06:48 | |
therve | mandira, if you want to use volumes with AWS::AutoScaling::LaunchConfiguration you need to use BlockDeviceMappings I think | 06:49 |
mandira | <therve> i tried BlockDeviceMapping..it throws error that it is yet to be implemented | 06:50 |
*** alexknith has joined #heat | 06:50 | |
therve | Oh yeah :) | 06:50 |
therve | mandira, So you can't use that | 06:50 |
therve | You need OS::Heat::AutoScalingGroup | 06:50 |
alexknith | shardy: Hi ! | 06:51 |
mandira | <therve> means insted of AWS::AutoScaling::LaunchConfiguration, i have to use OS::Heat::AutoScalingGroup ? | 06:53 |
shardy | morning all | 06:53 |
*** cody-somerville has quit IRC | 06:53 | |
therve | mandira, And no AWS::AutoScaling::AutoScalingGroup as well | 06:53 |
mandira | <therve>ok..i am trying that | 06:56 |
mandira | thanks :) | 06:56 |
*** jstrachan has joined #heat | 06:57 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Remove unused generate_request_id https://review.openstack.org/99583 | 07:00 |
*** julienvey has joined #heat | 07:01 | |
*** zns_ has joined #heat | 07:02 | |
*** akuznetsov has quit IRC | 07:02 | |
*** zns has quit IRC | 07:03 | |
*** mandira has quit IRC | 07:04 | |
*** mandira has joined #heat | 07:05 | |
*** cody-somerville has joined #heat | 07:05 | |
*** jcoufal has joined #heat | 07:13 | |
*** akuznetsov has joined #heat | 07:15 | |
*** mandira has quit IRC | 07:24 | |
*** e0ne has joined #heat | 07:24 | |
*** tomek_adamczewsk has joined #heat | 07:27 | |
*** zns_ has quit IRC | 07:29 | |
*** Yanyan has joined #heat | 07:33 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add RPC methods for quota actions https://review.openstack.org/97001 | 07:33 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add quotas module with driver and resources https://review.openstack.org/96716 | 07:33 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add quota model and migration https://review.openstack.org/96696 | 07:33 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add db APIs to access quota https://review.openstack.org/96697 | 07:33 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add APIs for quota https://review.openstack.org/98026 | 07:33 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Expose stack limits API https://review.openstack.org/99589 | 07:33 |
*** jcoufal has quit IRC | 07:33 | |
*** tomek_adamczewsk has quit IRC | 07:36 | |
*** alexheneveld has joined #heat | 07:36 | |
*** cody-somerville has quit IRC | 07:36 | |
*** tomek_adamczewsk has joined #heat | 07:36 | |
tiantian | <shardy>hi | 07:42 |
*** TonyBurn has joined #heat | 07:43 | |
tiantian | <shardy>: I reply your comment about the floatingip updatable patch: https://review.openstack.org/#/c/93389/ | 07:43 |
*** jcoufal has joined #heat | 07:43 | |
*** vpanizo_ has joined #heat | 07:44 | |
*** e0ne has quit IRC | 07:44 | |
*** e0ne has joined #heat | 07:45 | |
*** vpanizo has quit IRC | 07:46 | |
*** tomek_adamczewsk has quit IRC | 07:47 | |
*** tomek_adamczewsk has joined #heat | 07:49 | |
*** e0ne has quit IRC | 07:49 | |
*** cody-somerville has joined #heat | 07:50 | |
*** jcoufal has quit IRC | 07:51 | |
pas-ha | morning all | 07:58 |
*** jcoufal has joined #heat | 08:00 | |
shardy | tiantian: I'm still not clear on why doing delete/create in handle_update is better than raising UpdateReplace | 08:00 |
*** tomek_adamczewsk has quit IRC | 08:01 | |
shardy | tiantian: Is it to avoid the update replacing the dependent server? | 08:01 |
shardy | tiantian: basically, your handle_update *is* replacing the resource, because you delete the underlying resource and change the resource_id | 08:01 |
*** jistr has joined #heat | 08:02 | |
*** tomek_adamczewsk has joined #heat | 08:04 | |
*** tomek_adamczewsk has quit IRC | 08:06 | |
*** tomek_adamczewsk has joined #heat | 08:06 | |
tiantian | shardy>: The UpdateReplace flow is to create a new and then to delete the old one, isn't? | 08:08 |
tiantian | <shardy>: for current code, the 'server' and 'floatingip' are all not updatable, when stack-update with a new value either one of the two, it will go to the UpdateReplace flow, isn't? | 08:11 |
*** derekh_ has joined #heat | 08:11 | |
*** Fayablazer has joined #heat | 08:12 | |
*** fbo_away is now known as fbo | 08:14 | |
*** alexheneveld has quit IRC | 08:14 | |
*** jcoufal has quit IRC | 08:14 | |
*** Qiming has joined #heat | 08:15 | |
*** plomakin_ has quit IRC | 08:16 | |
elynn | hi shardy, do you have time to review my patches? https://review.openstack.org/#/c/99561/ https://review.openstack.org/#/c/98580/ | 08:16 |
*** skraynev has quit IRC | 08:16 | |
*** tnurlygayanov has quit IRC | 08:17 | |
shardy | tiantian: ah, that's what I was missing - you want to reverse the flow so delete->create instead of create->delete via UpdateReplace? | 08:19 |
tiantian | <shardy>: yes, if the properties are updatable, then it will delete and then create | 08:21 |
tiantian | <shardy>: in handle_update() | 08:21 |
tiantian | <shardy>: and if change the server_id, there is no need to delete | 08:22 |
tiantian | <shardy>: just to create the association, the same as the neutron floatingip do, that patch has already merged:) | 08:23 |
pas-ha | shardy, yes, even when needed to recreate a resource a correct way is to delete then create (as VolumeAttachment for example), as there can be only one "thing", and UpdateReplace first creates two | 08:23 |
pas-ha | sometimes that is | 08:23 |
*** tomek_adamczewsk has quit IRC | 08:24 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Enable SubnetId update for EC2::Instance https://review.openstack.org/98042 | 08:24 |
tiantian | right:) | 08:25 |
shardy | tiantian: If the server_id changes, don't you need to deallocate the floating IP from the old server_id? | 08:25 |
tiantian | <shardy>: yes, I test it | 08:25 |
shardy | pas-ha, tiantian: Thanks for the clarification, I understand the handle_delete path now | 08:26 |
* shardy probably just needs more coffee | 08:26 | |
*** tomek_adamczewsk has joined #heat | 08:27 | |
tiantian | <shardy>: :) | 08:27 |
shardy | tiantian: What I mean is why isn't there a remove_floating_ip in the handle_update, when server_id has changed? | 08:27 |
shardy | Does nova automatically deallocate the IP from the existing server when you do add_floating_ip? | 08:27 |
tiantian | <shardy>: because it will success, yes nova will do this | 08:28 |
tiantian | <shardy>: I test this situation just in nova | 08:29 |
tiantian | <shardy>: would you please to review the patch about the quota and absolute limits:) | 08:34 |
shardy | tiantian: I started looking at it but you've not addressed my comment re the quota_update DB API | 08:34 |
*** julienvey has quit IRC | 08:35 | |
tiantian | <shardy>: about this, I want to discuss with you:) | 08:35 |
shardy | I don't really want to -1 it three times for the same reason, so I'll wait for feedback from other heat-core folks to see if they agree with me | 08:35 |
*** julienvey has joined #heat | 08:35 | |
*** tomek_adamczewsk has quit IRC | 08:36 | |
tiantian | <shardy>: the quota_update is to update the limit of a quota_resource, such as stacks | 08:36 |
tiantian | <shardy>: We can't create a new quota, just to update the limit of the quota_resources which pre-defined | 08:37 |
tiantian | <shardy>: If we update a quota_resource limit, then we record this new limit of the quota_resource in DB | 08:37 |
tiantian | <shardy>: It's the implement of the quota_update, nova/cinder/neutron do the same | 08:38 |
*** cody-somerville has quit IRC | 08:39 | |
*** jcoufal has joined #heat | 08:43 | |
tiantian | <shardy>: and for the first time of update a limit of a resource, we need to create a new record, and also there is not quota_create api in nova/neutron/cinder | 08:46 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Implement PrivateIpAddress property for EC2::Instance https://review.openstack.org/98580 | 08:46 |
shardy | tiantian: I guess that's my argument, in your current implementation there's no point having quota_create | 08:47 |
shardy | and the pattern you're adopting, while it may be consistent with nova/cinder/neutron, is completely inconsistent with everything else in our DB API | 08:47 |
alexknith | shardy: I had another look at the devstack conf for both keystone & heat, I've followed that but to no avail. Same config files keystone+heat on devstack-ubuntu and on centos-rdo but on my centos-rdo the auth between heat and keystone is still broken. Can you have a look at the logs,configs ? Many thanks !! the paste in a sec | 08:47 |
shardy | alexknith: I'd try re-running packstack with the answer file it generated on first install | 08:48 |
shardy | I don't really have the time to get into more debugging this morning, sorry | 08:48 |
alexknith | shardy: ok, no worries | 08:49 |
*** dims_ has joined #heat | 08:49 | |
alexknith | shardy: maybe someone else ? | 08:49 |
alexknith | cmyster: Hi ! | 08:50 |
cmyster | Hi alexknith | 08:50 |
alexknith | cmyster: Hi ! can you help with a bit of debugging ? | 08:51 |
cmyster | depends :) | 08:51 |
*** cody-somerville has joined #heat | 08:53 | |
*** e0ne has joined #heat | 08:53 | |
cmyster | alexknith: off to eat in 10 | 08:53 |
*** dims_ has quit IRC | 08:54 | |
tiantian | <shardy>: the quota implement in nova/cinder/neutron are not the same | 08:55 |
*** tomek_adamczewsk has joined #heat | 08:56 | |
cmyster | right | 08:57 |
cmyster | well, I'm off to eat alexknith, see you when I get back | 08:57 |
pas-ha | shardy, what's the local workflow when I need to add something to heat.conf.sample? any extra steps? | 08:58 |
shardy | pas-ha: tools/config/generate_sample.sh | 08:59 |
pas-ha | ok | 08:59 |
pas-ha | thanks | 08:59 |
openstackgerrit | Yaoguo Jiang proposed a change to openstack/heat: Add boolean type parameter process for template resource https://review.openstack.org/99603 | 09:00 |
hoangdo | hello guys, mind if I ask a question. I obtained alarm_url from ScalingPolicy (value: { get_attr : [web_server_scaleup_policy, alarm_url] }). Then what data I should post to the url to trigger alarm? I tried to post some random data and got 404. | 09:01 |
*** tspatzier has joined #heat | 09:01 | |
tiantian | OK, hope more people to review the patches | 09:01 |
alexknith | cmyster: enjoy :) | 09:08 |
*** ramishra has quit IRC | 09:11 | |
*** sorantis has joined #heat | 09:17 | |
*** mandira has joined #heat | 09:21 | |
*** zhiyan is now known as zhiyan_ | 09:23 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement NovaFloatingIpAssociation res updatable https://review.openstack.org/93389 | 09:30 |
openstackgerrit | Chen Xiao proposed a change to openstack/heat: Fix DB2 column 'id' autoincrement issue https://review.openstack.org/92073 | 09:35 |
tiantian | <shardy>: I have add the comments for the floating ip updatable patch. review agian:) thanks | 09:37 |
*** jprovazn has quit IRC | 09:38 | |
*** Yanyan has quit IRC | 09:39 | |
*** julienve_ has joined #heat | 09:39 | |
shardy | tiantian: +2, thanks! :) | 09:40 |
*** julienvey has quit IRC | 09:43 | |
*** e0ne_ has joined #heat | 09:44 | |
*** e0ne has quit IRC | 09:48 | |
*** jprovazn has joined #heat | 10:08 | |
*** mandira has quit IRC | 10:10 | |
*** saju_m has joined #heat | 10:10 | |
*** tspatzier has quit IRC | 10:12 | |
*** tspatzier has joined #heat | 10:13 | |
*** skraynev has joined #heat | 10:14 | |
*** Qiming has quit IRC | 10:17 | |
*** tspatzier has quit IRC | 10:19 | |
*** matsuhashi has quit IRC | 10:20 | |
*** matsuhashi has joined #heat | 10:20 | |
*** akuznetsov has quit IRC | 10:21 | |
*** dims_ has joined #heat | 10:23 | |
*** matsuhashi has quit IRC | 10:25 | |
*** CarstenD has joined #heat | 10:26 | |
CarstenD | hello | 10:26 |
pas-ha | CarstenD, hi | 10:26 |
*** blues-man has joined #heat | 10:28 | |
blues-man | hello | 10:28 |
CarstenD | I’m trying to figure out, how to pass a yml file as user-data inside a hot template. the manual command looks something like this „nova boot --user-data ./cloud-config.yaml --image …“ | 10:28 |
CarstenD | I’ve found a few examples … but I can’t get it to work. it creates the stack without using the user-data | 10:30 |
shardy | CarstenD: you need user_data_format=RAW to pass the data directly to cloud-init | 10:31 |
shardy | CarstenD: unfortunately, if you're running Havana, that won't exist as it was added during Icehouse | 10:32 |
CarstenD | umpf … ok :-) | 10:33 |
shardy | https://github.com/openstack/heat-templates/blob/master/hot/software-config/example-templates/example-cloud-init.yaml#L99 | 10:33 |
CarstenD | then I’ve found another reason to update ... | 10:33 |
shardy | CarstenD: wel also added some other resources, e.g OS::Heat::CloudConfig in that example, which you may find useful | 10:34 |
*** denis_makogon has joined #heat | 10:34 | |
CarstenD | yes, thanx | 10:34 |
*** e0ne_ has quit IRC | 10:37 | |
*** e0ne has joined #heat | 10:37 | |
openstackgerrit | A change was merged to openstack/heat: Use ResourceDefinition when updating Load Balancer https://review.openstack.org/96003 | 10:39 |
*** e0ne has quit IRC | 10:41 | |
*** tomek_adamczewsk has quit IRC | 10:43 | |
*** tiantian has quit IRC | 10:43 | |
*** tomek_adamczewsk has joined #heat | 10:44 | |
*** ericgoncz has joined #heat | 10:48 | |
*** mkollaro has joined #heat | 10:52 | |
*** tomek_adamczewsk has quit IRC | 10:57 | |
*** saju_m has quit IRC | 10:58 | |
*** tomek_adamczewsk has joined #heat | 10:58 | |
*** vpanizo_ has quit IRC | 11:04 | |
*** Qiming has joined #heat | 11:06 | |
*** saju_m has joined #heat | 11:10 | |
*** nosnos has quit IRC | 11:13 | |
*** ericgoncz has quit IRC | 11:15 | |
*** e0ne has joined #heat | 11:15 | |
*** e0ne has quit IRC | 11:17 | |
*** e0ne has joined #heat | 11:17 | |
*** e0ne has quit IRC | 11:22 | |
*** shakayumi has joined #heat | 11:23 | |
*** shakayumi has quit IRC | 11:23 | |
*** shakayumi has joined #heat | 11:24 | |
*** nkhare has quit IRC | 11:26 | |
*** plomakin has joined #heat | 11:29 | |
*** sorantis has quit IRC | 11:30 | |
*** julienve_ has quit IRC | 11:35 | |
*** julienvey has joined #heat | 11:36 | |
*** blues-man has quit IRC | 11:37 | |
*** sorantis has joined #heat | 11:37 | |
*** julienve_ has joined #heat | 11:38 | |
afaranha | What's the deal with Heat Templates? Can I use 'tab' to indenting? And how many white spaces do I need to use? | 11:38 |
*** jamiehan_ has joined #heat | 11:39 | |
CarstenD | for the yaml files, use 2 white spaces to indent one level | 11:39 |
CarstenD | best is to use an editor which can display whitespaces, that helps a lot | 11:40 |
*** jamiehannaford has quit IRC | 11:41 | |
*** julienvey has quit IRC | 11:41 | |
afaranha | I'm using it, but I having "<unicode string>" errors, I tried using 'tab', 1 white space, 2, and 4 | 11:41 |
afaranha | and all of them returns error :/ | 11:42 |
cmyster | about unicode string ? | 11:43 |
cmyster | and your editor won't tell you anything about them ? | 11:43 |
cmyster | like, IDK ^M at the end of a line or something like that | 11:44 |
*** e0ne has joined #heat | 11:45 | |
*** jamiehan_ is now known as jamiehannaford | 11:45 | |
afaranha | no, I'm using Sublime. I don't see it. Also, when I'm using 4 white spaces the error change to: ERROR: u'u\'"str_replace" syntax should be str_replace:\\n | 11:46 |
* cmyster uses elvis but thats just him | 11:47 | |
afaranha | never heard about it | 11:49 |
*** shakayumi has quit IRC | 11:50 | |
cmyster | afaranha: its a VI (not VIM) clone | 11:51 |
CarstenD | as far as i understand, I could use OS::Heat::CloudConfig to include my cloud init yaml inside a hot file … is it possible to do something like this? http://paste.openstack.org/show/83797/ | 11:52 |
CarstenD | especially the line 19 to 21 … | 11:52 |
*** rpothier_ has quit IRC | 11:53 | |
CarstenD | the problem is, I need to get a token url from etcd.io for core os. but the token must be the same for each instance inside the ResourceGroup | 11:54 |
CarstenD | I’m not sure if the command is only executed once for the group or once for each count inside the group | 11:55 |
*** achampion has quit IRC | 12:00 | |
*** aniket has joined #heat | 12:01 | |
*** aniket has quit IRC | 12:01 | |
*** aniket has joined #heat | 12:02 | |
*** cmyster has quit IRC | 12:02 | |
aniket | hi all | 12:03 |
CarstenD | hi aniket | 12:04 |
aniket | <CarstenD>hello | 12:04 |
aniket | <CarstenD>for volume scaling,i am using OS::Heat::AutoScalingGroup....do you have any idea which alarm should i use to moniter volume usage? | 12:08 |
CarstenD | I’m sorry … I’m new to heat … just here to ask questions for my own :-) | 12:09 |
afaranha | How can I supply user_data in heat templates? Accordingly to this guide http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#str-replace , after "str_replace" I need to indent to start "template: |", but accordingly to my Heat I cannot do this, str_replace and template: should be in the same level, and now my heat it's complaining that the str_replace content must be a string: "ERROR: Property error : my_instance: user_data Value mus | 12:09 |
aniket | <CarstenD>haha..welcome | 12:09 |
afaranha | This is my template file: http://paste.openstack.org/show/83798/ | 12:10 |
*** KanagarajM has quit IRC | 12:10 | |
aniket | <elynn> hi | 12:12 |
*** jamiehannaford has quit IRC | 12:14 | |
afaranha | therve: Hi | 12:14 |
therve | Hello | 12:14 |
*** jamiehannaford has joined #heat | 12:15 | |
afaranha | do you know about heat templates? | 12:15 |
therve | Yeah | 12:17 |
*** jdob has joined #heat | 12:17 | |
afaranha | therve: I'm having some issues with it, could you help? | 12:18 |
afaranha | I'm following this template guide: http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#str-replace but it's not working | 12:18 |
*** rbuilta has joined #heat | 12:18 | |
therve | You're missing an indentation before the template key | 12:19 |
therve | Also you don't have keys so maybe don't use str_replace | 12:19 |
afaranha | which keys? | 12:19 |
afaranha | This is my template so far: http://paste.openstack.org/show/83798/ | 12:20 |
therve | Well params | 12:20 |
afaranha | therve: Do you know why this is not working? | 12:22 |
*** dims_ has quit IRC | 12:22 | |
therve | afaranha, Yes because you're missing an indentation before the template key | 12:22 |
*** dims_ has joined #heat | 12:23 | |
*** jamiehan_ has joined #heat | 12:23 | |
afaranha | therve: if i indent it it will raise another error, let mw show | 12:23 |
therve | afaranha, Why do you use str_replace if you don't replace anything? | 12:23 |
*** jamiehannaford has quit IRC | 12:24 | |
*** e0ne_ has joined #heat | 12:24 | |
afaranha | therve: Here is template with the error raised: http://paste.openstack.org/show/83799/ | 12:24 |
therve | afaranha, Yeah you don't have params | 12:25 |
therve | Not the bestest error message | 12:25 |
*** dims_ is now known as dims | 12:25 | |
*** john-n-seattle has joined #heat | 12:26 | |
CarstenD | http://paste.openstack.org/show/83800/ | 12:26 |
CarstenD | maybe you want something like this? | 12:27 |
afaranha | I didn't understand what you mean by params, which param? | 12:27 |
afaranha | CarstenD: ah, ok, gonna try that, thanks | 12:27 |
CarstenD | str replace is for replacing a string(needle = param) in a haystack (template) | 12:27 |
*** john-n-seattle has quit IRC | 12:28 | |
*** e0ne has quit IRC | 12:28 | |
CarstenD | in line 24 get_attr must be get_param | 12:28 |
CarstenD | http://paste.openstack.org/show/83801/ | 12:29 |
*** tomek_adamczewsk has quit IRC | 12:30 | |
afaranha | therve CarstenD: Thank you, it worked fine :D | 12:30 |
CarstenD | you’re welcome … just learning heat templates myself | 12:31 |
*** tomek_adamczewsk has joined #heat | 12:33 | |
*** TonyBurn has quit IRC | 12:37 | |
*** akuznetsov has joined #heat | 12:38 | |
*** chandan_kumar has joined #heat | 12:41 | |
*** pafuent has joined #heat | 12:42 | |
*** chandan_kumar has quit IRC | 12:44 | |
*** tomek_adamczewsk has quit IRC | 12:45 | |
*** openstackgerrit has quit IRC | 12:46 | |
*** tomek_adamczewsk has joined #heat | 12:47 | |
*** openstackgerrit has joined #heat | 12:48 | |
openstackgerrit | Thomas Herve proposed a change to openstack/heat: Port to oslo.messaging https://review.openstack.org/99648 | 12:48 |
*** sgordon has joined #heat | 12:49 | |
*** chandankumar has joined #heat | 12:50 | |
*** julienve_ has quit IRC | 12:55 | |
*** dims_ has joined #heat | 12:55 | |
*** julienvey has joined #heat | 12:56 | |
*** tomek_adamczewsk has quit IRC | 12:56 | |
*** dims has quit IRC | 12:56 | |
*** tomek_adamczewsk has joined #heat | 12:57 | |
*** achampion has joined #heat | 12:58 | |
aniket | <therve> hi..is there any heat version equivalent to AWS::AutoScaling::LaunchConfiguration? any idea? | 13:01 |
therve | aniket, You mean for OS::Heat::AutoScalingGroup ? | 13:02 |
shardy | aniket: http://docs.openstack.org/developer/heat/template_guide/index.html | 13:02 |
aniket | <therve>yes | 13:02 |
therve | aniket, You shouldn't need one | 13:03 |
*** jamiehannaford has joined #heat | 13:03 | |
shardy | aniket: You can just reference a nested stack, like in this example for ResourceGroup | 13:04 |
shardy | https://github.com/hardys/demo_templates/blob/master/juno_summit_intro_to_heat/example3_server_with_volume_group/server_with_volume_group.yaml | 13:04 |
shardy | I think you need "resource" instead of "resource_def" for AutoScalingGroup | 13:05 |
shardy | also see https://github.com/openstack/heat-templates/blob/master/hot/autoscaling.yaml | 13:06 |
*** jamiehan_ has quit IRC | 13:06 | |
aniket | <shardy> ok...thanks..i am going through that | 13:08 |
*** tomek_adamczewsk has quit IRC | 13:08 | |
*** tomek_adamczewsk has joined #heat | 13:11 | |
*** vijendar has joined #heat | 13:14 | |
*** sorantis has quit IRC | 13:17 | |
*** mkollaro has quit IRC | 13:22 | |
*** sgordon has quit IRC | 13:23 | |
*** ericgoncz has joined #heat | 13:26 | |
aniket | <shardy>are you there? | 13:27 |
*** blomquisg has joined #heat | 13:28 | |
aniket | <shardy>http://paste.openstack.org/show/83805/ this is my template...as per http://docs.openstack.org/developer/heat/template_guide/index.html,in OS::Heat::AutoScalingGroup,cooldown property is optional..but this template is throwing error | 13:29 |
*** tomek_adamczewsk has quit IRC | 13:30 | |
therve | aniket, Which error? | 13:30 |
aniket | <therve>cooldown:Unknown Property | 13:31 |
*** jergerber has joined #heat | 13:32 | |
therve | aniket, isn't it "Cooldown: Unknown property" ? | 13:32 |
aniket | <therve>yes | 13:33 |
therve | aniket, Case matters | 13:33 |
aniket | <therve>yup..got it :P | 13:33 |
shardy | aniket: please refer to the autoscaling.yaml example I linked, then you can see how to use these resources | 13:33 |
shardy | "resource": { "Ref" : "EC2Instance" } won't work, as resource needs to be a nested stack definition, not a resource ID | 13:34 |
*** andersonvom has joined #heat | 13:34 | |
*** julienve_ has joined #heat | 13:36 | |
*** julienvey has quit IRC | 13:38 | |
*** pas-ha has quit IRC | 13:40 | |
*** mkollaro has joined #heat | 13:40 | |
*** pas-ha has joined #heat | 13:42 | |
*** andersonvom has quit IRC | 13:42 | |
*** sileht has quit IRC | 13:43 | |
aniket | <shardy>https://github.com/openstack/heat-templates/blob/master/hot/autoscaling.yaml in this..type: lb_server.yaml..can you please give some information on this..? | 13:44 |
aniket | <shardy>should i need to write a template with ld_server.yaml ? | 13:45 |
aniket | *like | 13:45 |
shardy | aniket: yes, it's specifying another heat template so you can scale nested stacks | 13:46 |
shardy | aniket: You can just specify e.g OS::Nova::Server if you want to scale instances | 13:46 |
*** Tross has quit IRC | 13:48 | |
*** DandyPandy has joined #heat | 13:48 | |
*** afaranha has quit IRC | 13:49 | |
aniket | <shardy> i want to scale volume..for that if i refer https://github.com/openstack/heat-templates/blob/master/hot/F18/NovaInstanceWithCinderVolume_Native.yaml this template as resource,will it be working? | 13:54 |
*** beekneemech is now known as bnemec | 13:54 | |
shardy | aniket: provided that template works for you, yes it should | 13:55 |
*** zz_gondoi is now known as gondoi | 13:57 | |
aniket | <shardy> so,this https://github.com/openstack/heat-templates/blob/master/hot/F18/NovaInstanceWithCinderVolume_Native.yaml template should be in working condition on my openstack,to launch my volume scaling stack? | 13:57 |
*** sgordon has joined #heat | 13:58 | |
*** jistr has quit IRC | 14:00 | |
*** CoilDomain has joined #heat | 14:01 | |
CoilDomain | How would you end up passing an ip address from one resource to another in a template? | 14:01 |
*** sileht has joined #heat | 14:02 | |
CoilDomain | I wanted to input that into a set of commands for the next resource in line | 14:02 |
*** jistr has joined #heat | 14:02 | |
*** randallburt has joined #heat | 14:05 | |
*** randallburt has quit IRC | 14:05 | |
*** randallburt has joined #heat | 14:05 | |
therve | Hum there is an issue with the test runner, it doesn't load all the tests | 14:06 |
*** ericgoncz_ has joined #heat | 14:06 | |
*** ericgoncz has quit IRC | 14:08 | |
*** ericgoncz_ is now known as ericgoncz | 14:08 | |
therve | Good lord, it doesn't fail anymore when there is an import error | 14:08 |
*** asalkeld has quit IRC | 14:12 | |
*** sabeen has joined #heat | 14:15 | |
*** arbylee has joined #heat | 14:18 | |
*** tomek_adamczewsk has joined #heat | 14:20 | |
*** arbylee has quit IRC | 14:21 | |
*** arbylee has joined #heat | 14:22 | |
*** sjmc7 has joined #heat | 14:25 | |
*** jcoufal has quit IRC | 14:27 | |
*** CarstenD has quit IRC | 14:34 | |
*** radez_g0n3 is now known as radez | 14:34 | |
*** piyush has joined #heat | 14:36 | |
*** piyush has quit IRC | 14:36 | |
*** piyush has joined #heat | 14:37 | |
*** lazy_prince has quit IRC | 14:37 | |
*** alexheneveld has joined #heat | 14:38 | |
*** kgriffs|afk is now known as kgriffs | 14:40 | |
*** mkerrin has quit IRC | 14:44 | |
*** Meeral has joined #heat | 14:45 | |
*** Meeral has quit IRC | 14:49 | |
*** tomek_adamczewsk has quit IRC | 14:50 | |
*** tomek_adamczewsk has joined #heat | 14:50 | |
*** Qiming has quit IRC | 14:54 | |
*** jistr has quit IRC | 14:56 | |
*** jistr has joined #heat | 14:56 | |
*** tomek_adamczewsk has quit IRC | 14:57 | |
*** Fayablazer has quit IRC | 14:58 | |
*** tomek_adamczewsk has joined #heat | 14:59 | |
*** alexheneveld has quit IRC | 15:02 | |
*** jprovazn is now known as jprovazn_afk | 15:04 | |
*** Fayablazer has joined #heat | 15:06 | |
*** Michalik- has quit IRC | 15:11 | |
*** akuznetsov has quit IRC | 15:11 | |
*** tomek_adamczewsk has quit IRC | 15:14 | |
*** andreaf has quit IRC | 15:16 | |
*** daneyon has joined #heat | 15:18 | |
*** akuznetsov has joined #heat | 15:19 | |
*** denis_makogon has quit IRC | 15:19 | |
*** blues-man has joined #heat | 15:21 | |
*** bandarji has joined #heat | 15:24 | |
*** bandarji has quit IRC | 15:24 | |
*** alexpilotti has joined #heat | 15:26 | |
*** kgriffs is now known as kgriffs|afk | 15:30 | |
*** kgriffs|afk is now known as kgriffs | 15:33 | |
*** jamiehannaford has quit IRC | 15:36 | |
*** jamiehannaford has joined #heat | 15:37 | |
*** jamiehannaford has quit IRC | 15:38 | |
*** jamiehannaford has joined #heat | 15:40 | |
*** saju_m has quit IRC | 15:41 | |
*** Michalik- has joined #heat | 15:43 | |
*** samstav has joined #heat | 15:44 | |
*** zns has joined #heat | 15:49 | |
*** daneyon has quit IRC | 15:50 | |
*** daneyon has joined #heat | 15:51 | |
*** robsparker has joined #heat | 15:52 | |
*** blues-man has quit IRC | 15:53 | |
*** gokrokve has joined #heat | 15:54 | |
*** blues-man has joined #heat | 15:54 | |
*** daneyon_ has joined #heat | 15:55 | |
*** daneyon has quit IRC | 15:55 | |
*** bandarji has joined #heat | 15:59 | |
*** e0ne_ has quit IRC | 15:59 | |
*** e0ne has joined #heat | 16:00 | |
*** akuznetsov has quit IRC | 16:03 | |
*** mestery has quit IRC | 16:03 | |
*** mestery has joined #heat | 16:04 | |
*** e0ne has quit IRC | 16:04 | |
*** alexheneveld has joined #heat | 16:07 | |
*** tango|2 has joined #heat | 16:09 | |
*** jistr has quit IRC | 16:19 | |
*** alexknith has quit IRC | 16:22 | |
*** blues-man has quit IRC | 16:28 | |
*** bandarji has quit IRC | 16:32 | |
*** alexheneveld has quit IRC | 16:33 | |
*** gokrokve has quit IRC | 16:33 | |
*** akuznetsov has joined #heat | 16:34 | |
*** openstackgerrit has quit IRC | 16:38 | |
*** bandarji has joined #heat | 16:38 | |
*** Fayablazer has quit IRC | 16:45 | |
*** Fayablazer has joined #heat | 16:52 | |
*** mkerrin has joined #heat | 16:53 | |
*** harlowja_away is now known as harlowja | 16:54 | |
*** erecio has quit IRC | 16:58 | |
*** skraynev has quit IRC | 16:58 | |
*** jcoufal has joined #heat | 16:58 | |
*** skraynev has joined #heat | 16:59 | |
*** erecio has joined #heat | 16:59 | |
*** rwsu has joined #heat | 17:00 | |
*** radez is now known as radez_g0n3 | 17:00 | |
*** alexheneveld has joined #heat | 17:01 | |
*** samstav_ has joined #heat | 17:03 | |
*** randallburt has quit IRC | 17:03 | |
*** zns_ has joined #heat | 17:04 | |
*** erecio has quit IRC | 17:06 | |
*** nati_ueno has joined #heat | 17:06 | |
*** zns has quit IRC | 17:07 | |
*** samstav_ has quit IRC | 17:07 | |
*** samstav has quit IRC | 17:08 | |
*** samstav has joined #heat | 17:08 | |
*** erecio has joined #heat | 17:09 | |
*** zns_ has quit IRC | 17:09 | |
*** zns has joined #heat | 17:10 | |
*** samstav_ has joined #heat | 17:10 | |
*** samstav has quit IRC | 17:10 | |
SpamapS | zaneb: 99562 looks pretty interesting | 17:13 |
zaneb | SpamapS: I'm just about to post a fixed version too | 17:14 |
SpamapS | zaneb: I kind of don't want you distracted from update-failure-recovery .. because the more I'm getting into the initial POC forays into convergence, the more I'm seeing that I basically have to build on your template-snippet-saving to do it right. | 17:14 |
*** derekh_ has quit IRC | 17:14 | |
zaneb | too late, I've been distracted from that for like 2 weeks :( | 17:15 |
*** tspatzier has joined #heat | 17:18 | |
*** tspatzier has quit IRC | 17:21 | |
*** gokrokve has joined #heat | 17:22 | |
*** saju_m has joined #heat | 17:26 | |
*** DandyPandy has quit IRC | 17:28 | |
*** jstrachan has quit IRC | 17:29 | |
*** samstav_ has quit IRC | 17:29 | |
*** rwsu has quit IRC | 17:30 | |
*** ericgoncz has quit IRC | 17:31 | |
*** rwsu has joined #heat | 17:32 | |
*** jprovazn_afk is now known as jprovazn | 17:41 | |
*** mkollaro has quit IRC | 17:47 | |
*** andersonvom has joined #heat | 17:49 | |
*** zaneb has quit IRC | 17:52 | |
*** alexheneveld has quit IRC | 17:53 | |
*** mkollaro has joined #heat | 17:54 | |
*** daneyon_ has quit IRC | 17:55 | |
*** andersonvom_ has joined #heat | 17:59 | |
*** mspreitz has joined #heat | 17:59 | |
*** edmund has joined #heat | 18:00 | |
*** andersonvom has quit IRC | 18:02 | |
*** ericgoncz has joined #heat | 18:04 | |
*** alexheneveld has joined #heat | 18:06 | |
*** zaneb has joined #heat | 18:06 | |
*** piyush has quit IRC | 18:07 | |
*** jdob has quit IRC | 18:12 | |
*** jdob has joined #heat | 18:13 | |
*** openstackgerrit has joined #heat | 18:13 | |
openstackgerrit | Bartosz Górski proposed a change to openstack/heat: Adds multi-cloud to OS::Heat::Stack resource https://review.openstack.org/98533 | 18:14 |
openstackgerrit | Bartosz Górski proposed a change to openstack/heat: Adds multi-region support for stack resource https://review.openstack.org/53313 | 18:14 |
*** ericgoncz_ has joined #heat | 18:14 | |
*** DandyPandy has joined #heat | 18:17 | |
*** jcoufal has quit IRC | 18:17 | |
*** ericgoncz has quit IRC | 18:18 | |
*** ericgoncz_ is now known as ericgoncz | 18:18 | |
*** DandyPandy has quit IRC | 18:18 | |
*** DandyPandy has joined #heat | 18:19 | |
*** e0ne has joined #heat | 18:21 | |
*** jprovazn is now known as jprovazn_afk | 18:22 | |
*** zns has quit IRC | 18:23 | |
*** mspreitz has quit IRC | 18:25 | |
*** jdob has quit IRC | 18:25 | |
*** jdob has joined #heat | 18:25 | |
*** piyush has joined #heat | 18:25 | |
*** randallburt has joined #heat | 18:26 | |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Fix race condition deleting in-progress stack https://review.openstack.org/99562 | 18:27 |
*** piyush1 has joined #heat | 18:27 | |
zaneb | SpamapS, stevebaker: I'm reasonably confident that ^ should fix our race | 18:27 |
*** kgriffs is now known as kgriffs|afk | 18:28 | |
*** samstav has joined #heat | 18:28 | |
*** samstav has quit IRC | 18:29 | |
*** samstav has joined #heat | 18:30 | |
*** piyush has quit IRC | 18:30 | |
*** zns has joined #heat | 18:30 | |
*** DandyPandy has quit IRC | 18:32 | |
*** DandyPandy has joined #heat | 18:32 | |
*** piyush1 has quit IRC | 18:33 | |
*** piyush has joined #heat | 18:33 | |
*** erecio has quit IRC | 18:34 | |
*** erecio has joined #heat | 18:34 | |
*** DandyPandy_ has joined #heat | 18:35 | |
*** bandarji has quit IRC | 18:35 | |
*** DandyPandy_ has joined #heat | 18:37 | |
*** DandyPandy has quit IRC | 18:37 | |
*** DandyPandy_ has quit IRC | 18:37 | |
*** DandyPandy has joined #heat | 18:38 | |
*** DandyPandy has quit IRC | 18:40 | |
*** Fayablazer has quit IRC | 18:41 | |
*** rbuilta has quit IRC | 18:43 | |
openstackgerrit | Ton Ngo proposed a change to openstack/heat-specs: Stack breakpoint specification https://review.openstack.org/99734 | 18:46 |
*** DandyPandy has joined #heat | 18:48 | |
*** DandyPandy_ has joined #heat | 18:51 | |
*** DandyPandy_ has joined #heat | 18:52 | |
*** erecio has quit IRC | 18:53 | |
*** randallburt has quit IRC | 18:54 | |
*** DandyPan_ has joined #heat | 18:54 | |
*** randallburt has joined #heat | 18:54 | |
*** DandyPandy has quit IRC | 18:55 | |
*** randallburt has quit IRC | 18:55 | |
*** DandyPandy_ has quit IRC | 18:56 | |
*** randallburt has joined #heat | 18:57 | |
*** kgriffs|afk is now known as kgriffs | 18:58 | |
*** samstav_ has joined #heat | 19:00 | |
*** samstav has quit IRC | 19:02 | |
*** hoangdo has quit IRC | 19:02 | |
*** funzo has quit IRC | 19:02 | |
*** erecio has joined #heat | 19:03 | |
*** funzo has joined #heat | 19:03 | |
*** hoangdo has joined #heat | 19:03 | |
*** tango|3 has joined #heat | 19:05 | |
*** radez_g0n3 is now known as radez | 19:05 | |
*** Tross has joined #heat | 19:07 | |
*** tango|2 has quit IRC | 19:07 | |
*** kgriffs is now known as kgriffs|afk | 19:09 | |
*** kgriffs|afk is now known as kgriffs | 19:10 | |
*** Tross has left #heat | 19:13 | |
*** e0ne has quit IRC | 19:13 | |
*** e0ne has joined #heat | 19:14 | |
*** sabeen1 has joined #heat | 19:16 | |
*** e0ne has quit IRC | 19:16 | |
*** sabeen has quit IRC | 19:17 | |
*** sabeen has joined #heat | 19:19 | |
*** sabeen1 has quit IRC | 19:19 | |
*** sabeen has quit IRC | 19:20 | |
*** sabeen has joined #heat | 19:20 | |
*** alexheneveld has quit IRC | 19:33 | |
*** saju_m has quit IRC | 19:34 | |
*** saju_m has joined #heat | 19:34 | |
*** bandarji has joined #heat | 19:37 | |
*** aweiteka has joined #heat | 19:41 | |
*** saju_m has quit IRC | 19:46 | |
*** nati_ueno has quit IRC | 19:47 | |
*** saju_m has joined #heat | 19:47 | |
*** e0ne has joined #heat | 19:48 | |
*** nati_ueno has joined #heat | 19:56 | |
*** akuznetsov has quit IRC | 19:56 | |
*** e0ne has quit IRC | 19:59 | |
*** e0ne has joined #heat | 20:00 | |
*** e0ne has quit IRC | 20:02 | |
*** arbylee has quit IRC | 20:03 | |
*** arbylee has joined #heat | 20:06 | |
*** zns has quit IRC | 20:07 | |
*** radez is now known as radez_g0n3 | 20:07 | |
*** zns has joined #heat | 20:07 | |
*** julienve_ has quit IRC | 20:08 | |
*** ericgoncz has quit IRC | 20:08 | |
*** tomek_adamczewsk has joined #heat | 20:14 | |
*** DandyPan_ has quit IRC | 20:17 | |
*** morganfainberg has quit IRC | 20:18 | |
*** morganfainberg has joined #heat | 20:18 | |
*** noTHD has joined #heat | 20:24 | |
*** nati_ueno has quit IRC | 20:28 | |
*** TravT has joined #heat | 20:29 | |
*** radez_g0n3 is now known as radez | 20:33 | |
*** alexheneveld has joined #heat | 20:34 | |
*** openstackgerrit has quit IRC | 20:35 | |
*** openstackgerrit has joined #heat | 20:36 | |
*** jprovazn_afk has quit IRC | 20:37 | |
*** julienvey has joined #heat | 20:38 | |
*** julienvey has quit IRC | 20:40 | |
*** julienvey has joined #heat | 20:40 | |
SpamapS | Ok this has sat a long time | 20:40 |
SpamapS | is nobody doing reviews on python-heatclient? https://review.openstack.org/#/c/93834/ | 20:40 |
*** nati_ueno has joined #heat | 20:44 | |
*** julienvey has quit IRC | 20:45 | |
*** dims_ has quit IRC | 20:47 | |
stevebaker | morning | 20:49 |
openstackgerrit | David J Hu proposed a change to openstack/python-heatclient: This is a Keystone v3 integration POC. https://review.openstack.org/92728 | 20:49 |
*** erecio has quit IRC | 20:53 | |
*** jamiehannaford has quit IRC | 20:53 | |
*** achampion has quit IRC | 20:56 | |
*** e0ne has joined #heat | 20:57 | |
*** asalkeld has joined #heat | 20:58 | |
*** jdob has quit IRC | 20:59 | |
openstackgerrit | A change was merged to openstack/heat: Handle missing user_creds in stack.delete https://review.openstack.org/99503 | 21:00 |
*** randallburt has quit IRC | 21:01 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/96507 | 21:02 |
*** gokrokve_ has joined #heat | 21:02 | |
*** alexheneveld has quit IRC | 21:03 | |
*** pafuent has left #heat | 21:04 | |
SpamapS | stevebaker: so have you tried reproducing the race with https://review.openstack.org/#/c/99562/ and unreverting the revert we did yesterday? | 21:04 |
*** gokrokve has quit IRC | 21:05 | |
stevebaker | SpamapS: I'll try it now. I was having too much fun with my resiliant-heat-with-transient-failures patch | 21:05 |
*** nati_ueno has quit IRC | 21:07 | |
cyli | stevebaker: zaneb: just as an update, synced with therve yesterday - he has a lot of that work done already, going to try to help with any remaining pieces :) | 21:08 |
cyli | stevebaker: zaneb: wrt to oslo.messaging blueprint | 21:09 |
zaneb | cyli: cool, thanks for the update :) | 21:09 |
stevebaker | cyli: cool, thanks. has he just been doing config fixes, or oslo.messaging too? | 21:09 |
cyli | stevebaker: oslo.messaging | 21:09 |
cyli | stevebaker: it looks like he has a lot of the updates from those patches already, but less so the tests. | 21:10 |
stevebaker | ok | 21:10 |
*** vpanizo has joined #heat | 21:14 | |
*** bgorski has joined #heat | 21:15 | |
*** DandyPandy has joined #heat | 21:15 | |
*** julienvey has joined #heat | 21:15 | |
*** DandyPandy has quit IRC | 21:16 | |
*** DandyPandy has joined #heat | 21:16 | |
*** randallburt has joined #heat | 21:20 | |
*** piyush has quit IRC | 21:21 | |
*** e0ne has quit IRC | 21:26 | |
*** e0ne has joined #heat | 21:26 | |
*** petertoft has joined #heat | 21:28 | |
*** TravT has quit IRC | 21:31 | |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Fix race condition deleting in-progress stack https://review.openstack.org/99562 | 21:31 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Use auth_token_info to initialize heat_keystoneclient sessions https://review.openstack.org/99769 | 21:31 |
*** e0ne has quit IRC | 21:31 | |
stevebaker | SpamapS, zaneb, I still get a racey looking error http://paste.openstack.org/show/83866/ | 21:32 |
zaneb | doh | 21:32 |
stevebaker | see my review comment https://review.openstack.org/#/c/99562/ | 21:32 |
SpamapS | races are like that | 21:38 |
SpamapS | you find all the easy ones when you go looking | 21:38 |
zaneb | stevebaker: fwiw I based patch set 2 on the patch *before* the revert specifically so you wouldn't have to revert the revert on top of it :P | 21:38 |
stevebaker | zaneb: oh, I didn't notice that. no wonder I was getting cherry-pick weirdness | 21:39 |
*** julienvey has quit IRC | 21:40 | |
zaneb | stevebaker: why the -1, btw? | 21:43 |
stevebaker | because it didn't fix the race? | 21:43 |
zaneb | but it fixes *a* race | 21:44 |
zaneb | though clearly not the one we are looking for :( | 21:45 |
zaneb | and will get that 'lock already deleted' crap out of our logs | 21:45 |
*** jamiehannaford has joined #heat | 21:46 | |
*** arbylee has quit IRC | 21:47 | |
stevebaker | zaneb: ok, since its an improvement I'll approve | 21:48 |
*** kgriffs is now known as kgriffs|afk | 21:49 | |
stevebaker | a lock testing tempest api test would be pretty trivial, I wonder why we don't have one | 21:49 |
zaneb | stevebaker: thanks. no hurry to approve, but definitely no reason to not commit it eventually imo | 21:50 |
*** radez is now known as radez_g0n3 | 21:50 | |
*** ccrouch has quit IRC | 21:51 | |
*** sgordon has quit IRC | 21:53 | |
*** mcwoods has quit IRC | 21:55 | |
*** mcwoods has joined #heat | 21:56 | |
*** dims_ has joined #heat | 21:57 | |
*** aniket has quit IRC | 21:58 | |
*** arbylee has joined #heat | 22:03 | |
*** nati_ueno has joined #heat | 22:03 | |
*** ccrouch has joined #heat | 22:04 | |
*** ccrouch has quit IRC | 22:04 | |
*** julienvey has joined #heat | 22:04 | |
*** piyush has joined #heat | 22:11 | |
*** ccrouch has joined #heat | 22:12 | |
*** jergerber has quit IRC | 22:12 | |
openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Encrypt heat template parameters https://review.openstack.org/97900 | 22:13 |
*** piyush has quit IRC | 22:13 | |
*** vijendar has quit IRC | 22:14 | |
*** alexheneveld has joined #heat | 22:15 | |
*** gondoi is now known as zz_gondoi | 22:17 | |
*** julienvey has quit IRC | 22:18 | |
*** blomquisg has quit IRC | 22:18 | |
*** andreaf has joined #heat | 22:20 | |
*** petertoft has quit IRC | 22:26 | |
SpamapS | Hrm.. do we expect templates that do things like {get_attr: [ Res1, {get_param: some_parameter_name} ]} to validate? | 22:29 |
SpamapS | seems like validation should only stick to schemas and function names.. not anything that can't be resolved statically. | 22:30 |
*** gokrokve_ has quit IRC | 22:32 | |
*** sabeen has quit IRC | 22:32 | |
openstackgerrit | Ton Ngo proposed a change to openstack/heat-specs: Stack breakpoint specification https://review.openstack.org/99734 | 22:33 |
*** bandarji has quit IRC | 22:35 | |
stevebaker | SpamapS: it would be nice if that would work | 22:36 |
*** andreaf has quit IRC | 22:36 | |
*** arbylee has quit IRC | 22:36 | |
SpamapS | stevebaker: I think we just need to think carefully about validation as a subset of parsing. | 22:37 |
*** achampion has joined #heat | 22:41 | |
*** jogo has left #heat | 22:42 | |
*** gokrokve has joined #heat | 22:46 | |
*** sjmc7 has quit IRC | 22:50 | |
*** randallburt has quit IRC | 22:52 | |
*** alexheneveld has quit IRC | 23:00 | |
*** dims_ has quit IRC | 23:01 | |
*** bandarji has joined #heat | 23:03 | |
*** bandarji1 has joined #heat | 23:09 | |
*** bandarji has quit IRC | 23:09 | |
*** alexheneveld has joined #heat | 23:12 | |
therve | cyli, Hi! I posted a first patch FWIW | 23:19 |
therve | Fixing the tests now | 23:19 |
cyli | therve: oh wow, you are awake! | 23:20 |
cyli | therve: thanks! | 23:20 |
therve | That is true | 23:20 |
cyli | therve: heh, ok, so I guess I should stop trying to do that too :D | 23:21 |
cyli | therve: oh wait, did you mean adding tests? | 23:23 |
*** andersonvom_ has quit IRC | 23:23 | |
therve | cyli, No making the current ones pass | 23:23 |
cyli | therve: oh ok | 23:24 |
cyli | therve: can I assign this to you then? https://blueprints.launchpad.net/heat/+spec/oslo-messaging | 23:25 |
therve | SpamapS, We don't want i18n in debug, we actually just removed some | 23:25 |
cyli | therve: (since you've basically done it now :)) | 23:25 |
therve | cyli, Sure! | 23:25 |
cyli | (all I've done so far is ask a bunch of questions about how to get started :)) | 23:26 |
therve | cyli, You'll see in my next patch that I'm actually mocking private APIs, so suggestions welcome | 23:26 |
cyli | therve: ok :) | 23:26 |
*** randallburt has joined #heat | 23:28 | |
stevebaker | we may have a fix for most heat-slow failures https://review.openstack.org/#/c/99694/ | 23:34 |
*** jamiehannaford has quit IRC | 23:35 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Refactor OS::Nova::Server delete handling https://review.openstack.org/99800 | 23:36 |
*** gokrokve has quit IRC | 23:39 | |
therve | stevebaker, Using external DNS servers feels strange | 23:40 |
stevebaker | therve: I think the test already did | 23:40 |
therve | Still :) | 23:41 |
*** edmund has quit IRC | 23:42 | |
openstackgerrit | Thomas Herve proposed a change to openstack/heat: Port to oslo.messaging https://review.openstack.org/99648 | 23:44 |
*** ccrouch has quit IRC | 23:44 | |
*** Qiming has joined #heat | 23:45 | |
*** andersonvom has joined #heat | 23:51 | |
stevebaker | therve: btw, I'm thinking if there is an API call failure in a custom constraint then the validation should pass (with a logged warning) | 23:52 |
therve | stevebaker, Hum really? What kind of failure? | 23:53 |
stevebaker | therve: transient network failures, or nova-api having kittens | 23:53 |
therve | stevebaker, How you used HP cloud? :) | 23:53 |
therve | That feels a bit weird though | 23:54 |
stevebaker | better than failing the stack though | 23:54 |
stevebaker | longer term there could be some retry logic | 23:54 |
therve | But if it fails at validation wouldn't it fail just after at creation? | 23:55 |
stevebaker | therve: I'm working on making the actions resilient to transient failures, so it can retry until success. But that is all for nothing if validation fails | 23:57 |
therve | stevebaker, Presumably it should use the same mechanism | 23:58 |
stevebaker | therve: yes, that has just occured to me | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!