*** samstav has quit IRC | 00:02 | |
*** maxskew has quit IRC | 00:02 | |
*** samstav has joined #heat | 00:03 | |
*** maxskew has joined #heat | 00:03 | |
*** jcoufal has quit IRC | 00:06 | |
*** andersonvom has joined #heat | 00:06 | |
*** asalkeld has quit IRC | 00:06 | |
*** samstav has quit IRC | 00:07 | |
*** andersonvom has quit IRC | 00:11 | |
*** Tross has joined #heat | 00:15 | |
*** Qiming has joined #heat | 00:19 | |
*** asalkeld has joined #heat | 00:20 | |
*** gokrokve has quit IRC | 00:23 | |
*** gokrokve has joined #heat | 00:25 | |
*** Qiming has quit IRC | 00:26 | |
*** gokrokve has quit IRC | 00:26 | |
*** randallburt has quit IRC | 00:26 | |
*** dims_ has joined #heat | 00:27 | |
*** tspatzier has joined #heat | 00:28 | |
*** joeroyall has joined #heat | 00:31 | |
*** Tross has quit IRC | 00:31 | |
*** Tross has joined #heat | 00:32 | |
*** daneyon has quit IRC | 00:38 | |
*** daneyon has joined #heat | 00:41 | |
*** metral is now known as metral_zzz | 00:47 | |
*** daneyon has quit IRC | 00:49 | |
*** tspatzier has quit IRC | 00:53 | |
*** tspatzier has joined #heat | 01:00 | |
openstackgerrit | Winnie Tsang proposed a change to openstack/python-heatclient: Add stack-update argument for patching parameters https://review.openstack.org/109105 | 01:03 |
---|---|---|
openstackgerrit | Winnie Tsang proposed a change to openstack/python-heatclient: Add stack-update argument for reset parameters https://review.openstack.org/109465 | 01:03 |
*** piyush has joined #heat | 01:05 | |
*** nanjj has joined #heat | 01:18 | |
*** tiantian has joined #heat | 01:19 | |
*** liusheng has joined #heat | 01:20 | |
*** Qiming has joined #heat | 01:24 | |
openstackgerrit | Steve Martinelli proposed a change to openstack/python-heatclient: Update theme for docs https://review.openstack.org/109434 | 01:24 |
openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Import the stack module directly https://review.openstack.org/107292 | 01:28 |
openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Use the template module to access the Template class https://review.openstack.org/107291 | 01:28 |
*** ajc_ has joined #heat | 01:31 | |
liusheng | asalkeld: Hi Angus Salkeld, could you please approve the spec: https://review.openstack.org/#/c/105952/ ? if you think it valuable :) | 01:31 |
asalkeld | looking liusheng | 01:32 |
liusheng | asalkeld: if so, I will upload patchesets for that | 01:32 |
liusheng | asalkeld: thanks :) | 01:32 |
*** nanjj has quit IRC | 01:33 | |
*** nanjj has joined #heat | 01:33 | |
asalkeld | stevebaker, what's our "policy" on approving specs? | 01:33 |
asalkeld | how many +2's do we need? does ptl need to approve? - I seem to have missed the memo | 01:34 |
tiantian | <asalkeld>: they said stevebaker in UK this week | 01:35 |
asalkeld | o, ok - thanks | 01:35 |
asalkeld | them openstack people really seem to travel :-O | 01:36 |
*** metral_zzz is now known as metral | 01:36 | |
tiantian | <asalkeld>: may be zaneb is around :) | 01:36 |
* zaneb hides | 01:37 | |
asalkeld | zaneb, just qu about approving specs | 01:37 |
asalkeld | do you have to approve, how many +2's etc... | 01:37 |
zaneb | I don't want to the PTL to be a single point of failure | 01:37 |
asalkeld | you mean single point of success? | 01:38 |
asalkeld | :) | 01:38 |
zaneb | aim for consensus... more controversial changes require more consensus | 01:38 |
zaneb | no hard and fast rule | 01:38 |
asalkeld | dang | 01:38 |
asalkeld | that's the worst | 01:38 |
zaneb | asalkeld: well, if I don't see it and nothing gets approved as a result, then it's a single point of failure | 01:38 |
*** nosnos has joined #heat | 01:38 | |
asalkeld | o, totally - but could be hard and fast 3 * +2 | 01:39 |
asalkeld | ? | 01:39 |
zaneb | there is too much stuff in OpenStack that relies on the PTL as a SPOF | 01:39 |
asalkeld | if 3 cores agree then really should be ok | 01:39 |
zaneb | asalkeld: add it to the meeting agenda, we can discuss a hard-an-fast rule | 01:39 |
zaneb | in almost all cases yes | 01:39 |
asalkeld | zaneb, it just makes it more likely to impove spec approval rate | 01:40 |
zaneb | for wide sweeping changes, I can imagine wanting more to look at it | 01:40 |
asalkeld | zaneb, sure | 01:40 |
asalkeld | aka convergance | 01:40 |
zaneb | exactly | 01:41 |
asalkeld | so tiantian I'll leave your spec at a +2 | 01:41 |
zaneb | it's a tradeoff | 01:41 |
zaneb | I don't want to put anyone in a difficult position in terms of worrying that they may have approved too early | 01:41 |
zaneb | but I trust everyone's judgement in the core team | 01:42 |
asalkeld | zaneb, maybe is you are the third core you *can* approve if confident | 01:42 |
zaneb | maybe if we said that it's OK but not compulsory to approve after 3 | 01:42 |
asalkeld | but if not just +1 | 01:42 |
zaneb | yeah, exactly | 01:42 |
asalkeld | exactly | 01:42 |
asalkeld | zaneb, do you mind if i delete some old agenda's | 01:44 |
zaneb | nah, go ahead | 01:44 |
asalkeld | got about a dozen there | 01:44 |
zaneb | I just added that to next week's agenda though | 01:44 |
zaneb | so check for conflicts | 01:44 |
asalkeld | yeah i see it | 01:45 |
asalkeld | cool thanks | 01:45 |
*** metral is now known as metral_zzz | 01:45 | |
*** tiantian has quit IRC | 01:46 | |
Qiming | hi, your comments/suggestions to this bug are appreciated: https://bugs.launchpad.net/heat/+bug/1343873 | 01:46 |
uvirtbot | Launchpad bug 1343873 in heat "'rolling_update' property should be treated as implicitly specified" [Undecided,New] | 01:46 |
*** tiantian has joined #heat | 01:46 | |
zaneb | tiantian: I approved your spec :) | 01:46 |
tiantian | <asalkeld>: correct, it's not my spec | 01:47 |
asalkeld | tiantian, https://review.openstack.org/#/c/108917/ | 01:48 |
asalkeld | need some specialist areas | 01:48 |
openstackgerrit | A change was merged to openstack/heat-specs: Add log translation hints for Heat https://review.openstack.org/105952 | 01:49 |
tiantian | <zaneb>: in fact I have a spec https://review.openstack.org/#/c/105644/ hope your review/approve :) tks tks | 01:49 |
*** liusheng has quit IRC | 01:50 | |
*** liusheng has joined #heat | 01:51 | |
asalkeld | seems like we need a 'team' interested in aws compat. | 01:52 |
zaneb | tiantian: +2 | 01:52 |
asalkeld | seems to be a lot of interest | 01:52 |
zaneb | what is the opposite of "to volunteer"? | 01:53 |
asalkeld | to hide in the corner? | 01:54 |
asalkeld | :) | 01:55 |
*** Yanyanhu has joined #heat | 01:55 | |
openstackgerrit | A change was merged to openstack/heat-specs: Add specification to implement LaunchConfiguration bdm https://review.openstack.org/105644 | 01:55 |
asalkeld | Coercion? | 01:56 |
asalkeld | conscripted | 01:57 |
Qiming | seduction | 01:58 |
tiantian | <asalkeld>: first volunteer | 01:58 |
asalkeld | zaneb this might need you eyes https://review.openstack.org/#/c/105742/ | 01:58 |
asalkeld | tiantian not sure, haven't used it in a while | 01:59 |
asalkeld | wow tspatzier is verbose, phew now I have a heap to read | 02:00 |
nanjj | asalkeld: :) | 02:02 |
*** Yanyanhu has quit IRC | 02:02 | |
tiantian | <asalkeld>: happy when need | 02:02 |
*** Yanyanhu has joined #heat | 02:02 | |
*** liusheng has quit IRC | 02:03 | |
nanjj | asalkeld: https://review.openstack.org/#/c/108949/ needs your comments | 02:03 |
tspatzier | hi asalkeld, what are you referring to? | 02:03 |
asalkeld | hi tspatzier | 02:03 |
asalkeld | the action spec | 02:04 |
*** liusheng has joined #heat | 02:04 | |
asalkeld | it's all good | 02:04 |
asalkeld | tspatzier, a question while you are here | 02:05 |
tspatzier | ah, I see ;) It actually did not really change since you gave a +2 some time back. More minor changes in response to comments IIRC. | 02:05 |
asalkeld | configs:168 | 02:05 |
asalkeld | - actions: [ CREATE ] | 02:05 |
tspatzier | asalkeld: sure | 02:05 |
nanjj | tspatzier: where you are now? | 02:05 |
asalkeld | but you say one action | 02:05 |
asalkeld | why the list | 02:05 |
tspatzier | nanjj: I am in Austin this week. 9pm here. | 02:05 |
*** samstav has joined #heat | 02:05 | |
tspatzier | asalkeld: stevebaker suggested we make this a list, so you could re-use a config for multiple actions if desired. E.g. if you have a script or chef recipe that can be used for both CREATE and UPDATE, you can specify this. And then only have something else for e.g. SUSPEND and RESUME. | 02:06 |
asalkeld | I see, does the action get passed into the script? | 02:07 |
asalkeld | can the script know that's it an update or create | 02:07 |
tspatzier | Yeah, we decided that the in-instance hook would get all the config hooks for all the actions. And then we would also pass the current action, so the in-instance hook can select the right one execute. | 02:08 |
asalkeld | ok | 02:08 |
asalkeld | seems to be counter to: | 02:08 |
asalkeld | Use of scripts that conditionally handle actions267 | 02:09 |
asalkeld | It would be possible to provide scripts that get invoked for all of a268 | 02:09 |
asalkeld | resource's lifecycle actions. Those scripts would have to include a lot of269 | 02:09 |
asalkeld | conditional logic, which would make them very complicated. | 02:09 |
tspatzier | Alternative would have been to let the SoftwareDeployment resource only pass the one config for the current action down. | 02:09 |
asalkeld | anyways that is been super critical | 02:09 |
asalkeld | could be argued either way | 02:09 |
asalkeld | +2'd | 02:10 |
tspatzier | agree, either way would have worked. The only important point IMO is that you really don't have to code the conditions in the script, but either the generic in-instance hook logic or the SoftwareDeployment resource logic does it. | 02:10 |
tspatzier | thanks asalkeld | 02:10 |
asalkeld | why is this even a spec: https://review.openstack.org/#/c/108949/5/specs/vnc-console.rst | 02:11 |
asalkeld | seems like a bug/ very minor feature | 02:11 |
asalkeld | nanjj, ^ | 02:11 |
asalkeld | why not just post a patch | 02:11 |
asalkeld | o, well approved | 02:13 |
asalkeld | lunch bbl | 02:14 |
openstackgerrit | A change was merged to openstack/heat-specs: VNC console attribute support https://review.openstack.org/108949 | 02:14 |
nanjj | asalkeld: yes, tiny | 02:14 |
tspatzier | asalkeld: btw, maybe we could also approve https://review.openstack.org/#/c/105742 - 5 +2's already, not sure if we have any particular threshold ... | 02:15 |
elynn | morning all :) | 02:19 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Match tenant_id when name ambiguity in get_secgroup_uuids() https://review.openstack.org/109222 | 02:24 |
*** achampion has joined #heat | 02:26 | |
*** metral_zzz is now known as metral | 02:27 | |
*** samstav has quit IRC | 02:34 | |
*** metral is now known as metral_zzz | 02:37 | |
*** harlowja is now known as harlowja_away | 02:37 | |
openstackgerrit | A change was merged to openstack/heat-specs: Implement AWS "Updates are not supported" https://review.openstack.org/105742 | 02:38 |
tiantian | <zaneb><asalkeld>: hi, about the quota-api | 02:38 |
tiantian | <zaneb><asalkeld> https://review.openstack.org/#/c/96696/, is it valuable? | 02:40 |
*** metral_zzz is now known as metral | 02:46 | |
*** tspatzier has quit IRC | 02:46 | |
*** dims_ has quit IRC | 02:48 | |
*** zackf has joined #heat | 02:52 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Move UpdateReplace exception to exception.py https://review.openstack.org/109194 | 03:02 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Enable lazy translation on heat https://review.openstack.org/108917 | 03:04 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Import oslo.i18n and port heat bins to use oslo.i18n https://review.openstack.org/108924 | 03:06 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Switch heat api to oslo i18n https://review.openstack.org/109479 | 03:08 |
*** zackf has quit IRC | 03:10 | |
*** Yanyan has joined #heat | 03:20 | |
*** Yanyanhu has quit IRC | 03:20 | |
asalkeld | tiantian, looking | 03:22 |
*** mkerrin has joined #heat | 03:25 | |
tiantian | <asalkeld>: this blueprint has been approved by shardy, and the code is finished, seens Clint 'SpamapS' Byrum disagree, I and shardy reply the ml, then no reply, so may be this should be discuss? | 03:28 |
asalkeld | tiantian, the ml thread suggests most cores are thinking it's not needed | 03:30 |
asalkeld | so we should use the quotas exposed via other services | 03:30 |
asalkeld | and there is not much to quota | 03:31 |
asalkeld | SpamapS, maybe one thing that could have a quota is convergance worker jobs | 03:31 |
asalkeld | (so one stack donesn't flood the message queues) | 03:32 |
asalkeld | tiantian, that is a way off tho' | 03:32 |
SpamapS | that is an operator concern | 03:33 |
asalkeld | ok, so then no need for quota api | 03:34 |
SpamapS | I'm still not convinced that Heat, even though it ends up being heavy now because "reasons"... needs a per-user quota system. | 03:34 |
SpamapS | But I could be wrong. It's just my opinion. | 03:35 |
SpamapS | And I don't really run a massive Heat install. I think the people who would be most interesting to ask would be those at Rackspace running their beta Heat service. | 03:35 |
*** kopparam has joined #heat | 03:35 | |
tiantian | <asalkeld>I hear the rally needs this | 03:35 |
asalkeld | tiantian, what for ? | 03:36 |
*** kopparam has quit IRC | 03:36 | |
tiantian | <asalkeld>:Specific I don't know, may be Sergey Kraynev know this, or rally need absolutely-limit to show the pie | 03:38 |
asalkeld | I see, maybe this is just and admin/operator api then? | 03:39 |
asalkeld | (readonly) | 03:39 |
asalkeld | tho' adding an api purely for a test tool seems a little weird | 03:39 |
tiantian | <asalkeld>: another blueprint about the absolutely-limit depends on this blueprint: https://review.openstack.org/#/c/99589/ | 03:40 |
*** cyen has quit IRC | 03:40 | |
asalkeld | tiantian, so this is just a "show" - readonly | 03:41 |
asalkeld | https://review.openstack.org/#/c/99589/5/heat/engine/service.py | 03:42 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Move UpdateReplace exception to exception.py https://review.openstack.org/109194 | 03:42 |
*** ajc_ has quit IRC | 03:43 | |
tiantian | <asalkeld>: for absolutely-limit, it is | 03:43 |
asalkeld | maybe just a show would be ok, people are unsure of the use of crud api for quotas | 03:43 |
asalkeld | tiantian, does rally want to modify quota's | 03:43 |
asalkeld | or just read what they are | 03:43 |
asalkeld | I think exposing the limits is reasonable | 03:44 |
asalkeld | to view | 03:44 |
tiantian | asalkeld: I don't know much, I will ask Sergey Kraynev later, about the rally whether is need the quota-api | 03:45 |
asalkeld | ok, cool | 03:45 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Enable lazy translation on heat https://review.openstack.org/108917 | 03:46 |
tiantian | <asalkeld>: if the quota-api isn't need, the implemention of absolutely-limit need modify | 03:47 |
SpamapS | tiantian: I don't mean to be a party-pooper. I am merely concerned that doing quotas "just because somebody might want it" is a large undertaking without a large use case. | 03:47 |
SpamapS | tiantian: as I said, I am not running a large heat. My biggest heat has 1 user, 2 stacks and just manages 40 baremetal servers. So it's possible I have no clue whatsoever how expensive it is to let Heat manage hundreds of users and thousands of stacks. | 03:48 |
asalkeld | I am all for starting small, maybe just view what limits we have is useful for a user | 03:48 |
asalkeld | like the recursion limits | 03:48 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Import oslo.i18n and port heat bins to use oslo.i18n https://review.openstack.org/108924 | 03:49 |
SpamapS | asalkeld: I do wonder if we could eliminate the recursion limits by implementing a "root stack resource counter". | 03:50 |
SpamapS | I was hesitant to do that when I was putting in the limits | 03:50 |
asalkeld | shouldn't be hard | 03:50 |
SpamapS | But I actually think convergence will give us a good opportunity to do it. | 03:51 |
tiantian | SpamapS: Of course not,you are pretty:) | 03:51 |
SpamapS | Since we're not going to be parsing whole stacks all the time, we can just maintain it in the resource create/delete methods. | 03:51 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Switch heat api to oslo i18n https://review.openstack.org/109479 | 03:51 |
SpamapS | Anyway, I'm jet lagged and grumpy.. time to hit the sack. | 03:52 |
SpamapS | tiantian: aw thanks. ;) | 03:52 |
asalkeld | har | 03:53 |
*** nosnos has quit IRC | 03:55 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.contrib https://review.openstack.org/109484 | 04:00 |
tiantian | asalkeld: https://review.openstack.org/#/c/103725/ has modified according your comments, hope you review it again, tks | 04:03 |
*** nanjj has quit IRC | 04:03 | |
asalkeld | looking now | 04:06 |
asalkeld | getting review weary ... | 04:09 |
tiantian | <asalkeld>: tks :) | 04:14 |
asalkeld | np | 04:14 |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.api https://review.openstack.org/109487 | 04:17 |
*** piyush has quit IRC | 04:18 | |
*** cody-somerville has joined #heat | 04:28 | |
*** piyush has joined #heat | 04:32 | |
*** piyush has quit IRC | 04:32 | |
*** piyush has joined #heat | 04:33 | |
*** rushiagr has joined #heat | 04:36 | |
*** ramishra has joined #heat | 04:38 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Switch heat engine to oslo.i18n https://review.openstack.org/109488 | 04:39 |
asalkeld | elynn, are you and liusheng working together? | 04:40 |
elynn | asalkeld, no... | 04:40 |
*** piyush has quit IRC | 04:41 | |
asalkeld | ok, as long as what both of you are doing is heading in a sensible direction | 04:41 |
*** unmeshg has joined #heat | 04:41 | |
elynn | he has a plan to switch to oslo.i18n? | 04:41 |
asalkeld | Log translation hint for Heat.api | 04:41 |
asalkeld | brb: picking kids up from school | 04:43 |
elynn | ok, I will not do that part when switching to oslo.i18n, if his patches landed first, I will do rebase work. | 04:43 |
*** piyush has joined #heat | 04:44 | |
*** bmahalakshmi has joined #heat | 04:44 | |
*** nosnos has joined #heat | 04:51 | |
*** kopparam has joined #heat | 04:53 | |
*** ajc_ has joined #heat | 04:54 | |
*** piyush has quit IRC | 04:54 | |
*** bvivek has joined #heat | 04:57 | |
*** mkerrin has quit IRC | 04:59 | |
*** cody-somerville has quit IRC | 05:01 | |
*** bandarji has joined #heat | 05:11 | |
*** kopparam has quit IRC | 05:14 | |
*** kopparam has joined #heat | 05:16 | |
*** tiantian has quit IRC | 05:21 | |
*** Dev_Jin has joined #heat | 05:21 | |
*** killer_prince is now known as lazy_prince | 05:22 | |
*** akuznetsov has joined #heat | 05:22 | |
*** rakesh_hs has joined #heat | 05:28 | |
*** bandarji has quit IRC | 05:33 | |
*** k4n0 has joined #heat | 05:56 | |
*** chandankumar has joined #heat | 05:57 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/104975 | 06:01 |
*** ramishra has quit IRC | 06:11 | |
*** ramishra has joined #heat | 06:12 | |
rakesh_hs | Hi guys.. I have proposed an patch for defect fix (bug#: 1290565).. https://review.openstack.org/#/c/107551/ | 06:16 |
rakesh_hs | can you guys please review it | 06:16 |
Dev_Jin | In HEAT::AutoScalingGroup how to give unique names to instances getting launched? By default it gives same name (from properties) | 06:16 |
*** ramishra has quit IRC | 06:16 | |
Qiming | Dev_Jin, what do you mean by 'same name'? | 06:18 |
Dev_Jin | Qiming .. e.g. type: OS::Nova::Server | 06:19 |
Dev_Jin | properties: | 06:19 |
Dev_Jin | name: Worker | 06:19 |
Dev_Jin | all the instances come up with Woker.. how to make them say Worker1 Worker2… | 06:20 |
Qiming | Dev_Jin, I see what you are talking about | 06:20 |
*** tiantian has joined #heat | 06:22 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.contrib https://review.openstack.org/109484 | 06:22 |
Qiming | Dev_Jin, when I ignore the name property, Heat will automatically give them one | 06:22 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Replace str(err) with six.text_type in tests https://review.openstack.org/109502 | 06:23 |
Qiming | Dev_Jin, tbh, I didn't try what will happen if 'name' is explicitly specified | 06:23 |
Dev_Jin | Qiming is there anyway to have more control? | 06:24 |
Qiming | Dev_Jin, not that I know of | 06:24 |
*** cyen has joined #heat | 06:25 | |
Dev_Jin | Qiming, thanks | 06:26 |
Qiming | Dev_Jin, it sounds like something backed by a use case? Why are you referencing an individual member of the group? | 06:27 |
Dev_Jin | Qiming.. well what’s happening is s/w configuration management identifies individual clients by hostnames.. but if all of them have same then it treats them as 1 | 06:28 |
Qiming | Dev_Jin, if I'm understanding this correctly, each instance will have a dedicated SoftwareDepolyment resource. | 06:31 |
Qiming | Qiming, unless you are referring to some other out-of-band configuration management tools? | 06:32 |
Qiming | Dev_Jin, as I said, I have no idea what will happen if you have instance name explicitly specified. By default, each instance WILL get a different hostname. | 06:33 |
Dev_Jin | Yes Qiming I am using out of band config management tool (like chef/puppet) … Anyways.. for now I can work with not specifying name in the property and let the openstack ensures uniquness | 06:33 |
*** ramishra has joined #heat | 06:34 | |
Qiming | Dev_Jin, ahh, now I understand your question... | 06:34 |
elynn | Hi guys, could anyone can review this patch and help it merge? https://review.openstack.org/#/c/105101/ | 06:38 |
Dev_Jin | Qiming.. some unrelated question.. I am bit strugling with how call heat stack create from python client (or curl) .. my real struggle is how to form payload of stack create call (which consists of template file + stack name info and parameters…) | 06:39 |
Dev_Jin | is there something simple which I am missing ( I dont want to reinvent the wheel ) it works well with heat stack-create command.. but I want to do it programatically using python | 06:39 |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.common https://review.openstack.org/109506 | 06:39 |
Qiming | Dev_Jin, and environments ... | 06:40 |
Dev_Jin | Quming.. yes.. that’s correct | 06:40 |
Dev_Jin | Qiming — typo | 06:40 |
Qiming | Dev_Jin, my feeling is that you have quite some work to do before invoking the ReST API if you decide to do that, maybe I'm missing something as well | 06:44 |
Dev_Jin | Qiming.. my experience tells the same.. I have spent good amount of time trying to read heat template file convert it into json format etc etc.. | 06:45 |
Qiming | Dev_Jin, I hope you really have fun doing that | 06:46 |
Dev_Jin | Qiming.. You bet [:) | 06:46 |
Qiming | Dev_Jin, horizon file openstack_dashboard/dashboards/project/stacks/forms.py could be a good reference, :) | 06:59 |
*** noTHD has quit IRC | 07:00 | |
*** ramishra has quit IRC | 07:01 | |
*** Dev_Jin has quit IRC | 07:01 | |
*** pasquier-s has joined #heat | 07:04 | |
openstackgerrit | lvdongbing proposed a change to openstack/python-heatclient: Don't expose X-Auth-Token in heat CLI https://review.openstack.org/109234 | 07:06 |
*** rushiagr has quit IRC | 07:08 | |
*** Dev_Jin has joined #heat | 07:08 | |
Dev_Jin | Qiming.. I just had a quick pass.. it might be helpful.. let me dig dip now | 07:08 |
*** kopparam has quit IRC | 07:08 | |
*** rushiagr has joined #heat | 07:09 | |
*** kopparam has joined #heat | 07:09 | |
Qiming | Dev_Jin, at first glance, it is must simpler than Heat shell, because it is dealing with lesser options | 07:09 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Switch heat api to oslo i18n https://review.openstack.org/109479 | 07:12 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Import oslo.i18n and port heat bins to use oslo.i18n https://review.openstack.org/108924 | 07:12 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Switch heat engine to oslo.i18n https://review.openstack.org/109488 | 07:12 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Enable lazy translation on heat https://review.openstack.org/108917 | 07:12 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Move UpdateReplace exception to exception.py https://review.openstack.org/109194 | 07:12 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Switch heat resources to oslo.i18n https://review.openstack.org/109510 | 07:12 |
*** kopparam has quit IRC | 07:14 | |
*** lvdongbing has joined #heat | 07:14 | |
*** asalkeld has quit IRC | 07:15 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.engine https://review.openstack.org/109512 | 07:16 |
*** ramishra has joined #heat | 07:30 | |
*** jstrachan has joined #heat | 07:33 | |
skraynev | Good Morning | 07:37 |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.tests https://review.openstack.org/109514 | 07:37 |
tiantian | <skraynev>:hi | 07:40 |
skraynev | hello tiantian | 07:40 |
*** rushiagr_ has joined #heat | 07:40 | |
*** rushiagr has quit IRC | 07:40 | |
tiantian | <skraynev>: this morning we talked about whether quota-api blueprint is needed, see the history :) I remember you said it's usesful for rally?for what? I don't know much about this, or I remember wrong | 07:44 |
skraynev | tiantian: you mean IRC discussion or where? | 07:44 |
skraynev | tiantian: It's really need for rally - you remember it correct :) It should be used for test case, when we try to create a lot of stacks for one tenant. | 07:46 |
tiantian | <skraynev>: on july, in IRC | 07:47 |
skraynev | tiantian: thx, I will look it later ;) | 07:48 |
tiantian | <skraynev>: about the quota-api blueprint, no reviewer now, https://review.openstack.org/#/c/96696/ and I don't know whether it's needed. It seems most core member thinking it's no need | 07:51 |
*** kopparam has joined #heat | 07:52 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Remove gettextutils from heat https://review.openstack.org/109517 | 07:52 |
*** lsmola has joined #heat | 07:54 | |
elynn | Hi skraynev , do you have time to review this patch? https://review.openstack.org/#/c/109502/ | 07:56 |
skraynev | tiantian: I understand your worries. Unfortunately, I had not time before to speak about it with other reviewers. | 07:56 |
*** bvivek has quit IRC | 07:57 | |
skraynev | tiantian: I will ask zaneb about it. Let's see what he will say. | 07:57 |
tiantian | <skraynev>: thanks very much:) | 07:57 |
skraynev | tiantian: np | 07:58 |
skraynev | elynn: thx, as soon as possibly. | 07:58 |
elynn | skraynev, :) | 07:58 |
therve | elynn, I have some troubles with your switch to oslo.i18n | 08:00 |
therve | elynn, Why do you think the 2 libraries can coexist? | 08:01 |
elynn | hi therve | 08:01 |
*** tomek_adamczewsk has joined #heat | 08:02 | |
elynn | therve, I just wanna to make sure that every patch will not break unittests. | 08:02 |
elynn | and I remove gettextuitls in patch https://review.openstack.org/#/c/109517/ | 08:03 |
therve | Mokay | 08:04 |
elynn | Split this patch into several patches just to benefit me, so I won't miss any place :) | 08:05 |
elynn | therve, sorry I have to leave for a while, I have a meeting... | 08:06 |
therve | Well it's nice to have smaller patches for sure | 08:06 |
*** tomek_adamczewsk has quit IRC | 08:08 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.api https://review.openstack.org/109487 | 08:09 |
*** cdent has joined #heat | 08:12 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.contrib https://review.openstack.org/109484 | 08:17 |
*** bvivek has joined #heat | 08:20 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.common https://review.openstack.org/109506 | 08:25 |
*** rushiagr_ has quit IRC | 08:27 | |
*** rushiagr_ has joined #heat | 08:28 | |
*** lvdongbing has quit IRC | 08:32 | |
*** lvdongbing has joined #heat | 08:33 | |
*** lsmola has quit IRC | 08:36 | |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.engine https://review.openstack.org/109512 | 08:38 |
openstackgerrit | liusheng proposed a change to openstack/heat: Log translation hint for Heat.tests https://review.openstack.org/109514 | 08:41 |
*** lvdongbing has quit IRC | 08:46 | |
*** lvdongbing has joined #heat | 08:48 | |
*** tomek_adamczewsk has joined #heat | 08:48 | |
*** nikunj2512 has joined #heat | 08:49 | |
*** ramishra has quit IRC | 08:50 | |
*** ramishra has joined #heat | 08:50 | |
*** Yanyan has quit IRC | 08:53 | |
*** ramishra has quit IRC | 08:54 | |
*** rushiagr_ has quit IRC | 09:01 | |
*** bvivek has quit IRC | 09:07 | |
*** bvivek has joined #heat | 09:09 | |
*** lvdongbing has quit IRC | 09:11 | |
openstackgerrit | xu-haiwei proposed a change to openstack/heat: Show correct error messages https://review.openstack.org/108603 | 09:11 |
*** lvdongbing has joined #heat | 09:12 | |
*** saju_m has joined #heat | 09:12 | |
*** zslot has joined #heat | 09:17 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement BlockDeviceMappings for AWS::EC2::Instance https://review.openstack.org/103725 | 09:22 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement BlockDeviceMappings for LaunchConfiguration https://review.openstack.org/105647 | 09:22 |
*** kopparam has quit IRC | 09:26 | |
*** zslot has quit IRC | 09:29 | |
*** zslot has joined #heat | 09:30 | |
*** akuznetsov has quit IRC | 09:30 | |
*** kopparam has joined #heat | 09:33 | |
openstackgerrit | A change was merged to openstack/heat: Remove deprecated tools/README instructions https://review.openstack.org/109404 | 09:35 |
*** lsmola has joined #heat | 09:39 | |
*** akuznetsov has joined #heat | 09:42 | |
*** saju_m has quit IRC | 09:47 | |
*** saju_m has joined #heat | 09:50 | |
*** ramishra has joined #heat | 09:54 | |
*** bvandenh has joined #heat | 09:55 | |
openstackgerrit | Alexey Ivanitskiy proposed a change to openstack/heat: Adding log messages to improve error handling https://review.openstack.org/109333 | 09:55 |
*** bmahalakshmi2 has joined #heat | 10:03 | |
*** Qiming has quit IRC | 10:03 | |
*** bmahalakshmi has quit IRC | 10:03 | |
*** lvdongbing has quit IRC | 10:12 | |
*** nosnos has quit IRC | 10:15 | |
*** bvivek2 has joined #heat | 10:17 | |
*** bvivek has quit IRC | 10:17 | |
*** blinky_ghost has joined #heat | 10:18 | |
*** Dev_Jin has quit IRC | 10:19 | |
*** pasquier-s has quit IRC | 10:24 | |
*** dims has joined #heat | 10:24 | |
*** Dev_Jin has joined #heat | 10:25 | |
*** saju_m has quit IRC | 10:40 | |
*** denis_makogon has joined #heat | 10:41 | |
*** tspatzier has joined #heat | 10:43 | |
*** tiantian has quit IRC | 10:45 | |
*** rushiagr_ has joined #heat | 10:47 | |
*** blues-man has joined #heat | 10:48 | |
blinky_ghost | hi all, I'm testing OS::Heat::SoftwareDeployment function in a fedora instance I've created following this procedure: https://github.com/openstack/heat-templates/tree/master/hot/software-config/elements. However when I launch the instance I get this error os-collect-config: (heat-config) [ERROR] No config file /var/run/heat-config/heat-config | 10:48 |
blinky_ghost | [2014-07-25 10:49:53,861] (os-refresh-config) [ERROR] during configure phase. [Command '['dib-run-parts', '/opt/stack/os-config-refresh/configure.d']' returned non-zero exit status 1] | 10:48 |
blinky_ghost | What am I missing? thanks | 10:49 |
*** mkerrin has joined #heat | 10:52 | |
*** mkerrin1 has joined #heat | 10:54 | |
*** mkerrin has quit IRC | 10:56 | |
*** Qiming has joined #heat | 10:57 | |
*** saju_m has joined #heat | 10:57 | |
*** piyush has joined #heat | 11:05 | |
*** abramley has quit IRC | 11:06 | |
openstackgerrit | Alexey Ivanitskiy proposed a change to openstack/heat: Adding log messages to improve error handling https://review.openstack.org/109333 | 11:07 |
*** abramley has joined #heat | 11:09 | |
*** vpanizo has quit IRC | 11:14 | |
*** ramishra_ has joined #heat | 11:18 | |
*** ramishra has quit IRC | 11:19 | |
therve | blinky_ghost, https://review.openstack.org/#/c/108330/ I'd say | 11:19 |
openstackgerrit | A change was merged to openstack/heat-templates: Rename os-config-applier to os-apply-config https://review.openstack.org/108330 | 11:19 |
*** achampio1 has joined #heat | 11:23 | |
*** ramishra_ has quit IRC | 11:24 | |
*** ramishra has joined #heat | 11:25 | |
*** achampion has quit IRC | 11:26 | |
blinky_ghost | therve: can you tell me if this patch is in merged in RDO already? | 11:26 |
*** alexpilotti has joined #heat | 11:27 | |
therve | blinky_ghost, It's in heat-templates, so I don't think it gets in RDO at all | 11:27 |
therve | You need to rebuild your image with those elements AFAIU | 11:27 |
blinky_ghost | therve: but I've build my image yesterday, I've built it from a CentOS box. | 11:29 |
*** ramishra has quit IRC | 11:29 | |
therve | blinky_ghost, The patch got merged 10 minutes ago | 11:30 |
blinky_ghost | therve: :P | 11:30 |
*** kopparam has quit IRC | 11:30 | |
blinky_ghost | therve: Sorry, I'm a moron :P | 11:30 |
*** kopparam has joined #heat | 11:30 | |
*** ramishra has joined #heat | 11:33 | |
blinky_ghost | therve: can you tell me if that elements will be available soon for Centos 6? | 11:33 |
therve | blinky_ghost, No idea. Why is not available now? | 11:33 |
*** ramishra_ has joined #heat | 11:34 | |
blinky_ghost | therve: stevebaker told me yesterday here that it was only available for Fedora | 11:34 |
*** rushiagr_ has quit IRC | 11:35 | |
therve | He would know better than me | 11:35 |
*** rushiagr_ has joined #heat | 11:35 | |
*** joeroyall has quit IRC | 11:36 | |
blinky_ghost | therve: ok thanks I'll rebuild the fedora image and try again :) | 11:36 |
*** ramishra has quit IRC | 11:38 | |
*** k4n0 has quit IRC | 11:38 | |
*** saju_m has quit IRC | 11:39 | |
*** blues-man has quit IRC | 11:42 | |
unmeshg | therve: can you please revisit https://review.openstack.org/104471 | 11:50 |
*** andersonvom has joined #heat | 11:50 | |
*** andersonvom has quit IRC | 11:51 | |
*** andersonvom has joined #heat | 11:51 | |
*** blomquisg has quit IRC | 11:53 | |
*** lsmola has quit IRC | 11:55 | |
*** rbuilta has joined #heat | 11:57 | |
*** pasquier-s has joined #heat | 11:57 | |
*** jcoufal has joined #heat | 11:59 | |
*** jcoufal has quit IRC | 11:59 | |
*** jcoufal has joined #heat | 12:06 | |
*** chandan_kumar has joined #heat | 12:09 | |
*** chandan_kumar has quit IRC | 12:09 | |
*** joeroyall has joined #heat | 12:10 | |
*** bmahalakshmi has joined #heat | 12:13 | |
*** achampio1 has quit IRC | 12:14 | |
*** unmeshg has left #heat | 12:14 | |
*** bmahalakshmi2 has quit IRC | 12:16 | |
*** tspatzier has quit IRC | 12:22 | |
*** dims has quit IRC | 12:23 | |
*** dims has joined #heat | 12:25 | |
*** tonisbones has joined #heat | 12:31 | |
*** jcoufal has quit IRC | 12:32 | |
*** bvandenh has quit IRC | 12:33 | |
*** ajc_ has quit IRC | 12:38 | |
*** jcoufal has joined #heat | 12:40 | |
*** jcoufal has quit IRC | 12:43 | |
*** Qiming has quit IRC | 12:47 | |
*** russellb is now known as rustlebee | 12:50 | |
*** kopparam has quit IRC | 12:51 | |
*** kopparam has joined #heat | 12:52 | |
*** che-arne|2 is now known as che-arne | 12:54 | |
*** mkerrin1 has quit IRC | 12:55 | |
*** rwsu has joined #heat | 12:56 | |
*** kopparam has quit IRC | 12:56 | |
*** jcoufal has joined #heat | 12:59 | |
*** jcoufal has quit IRC | 13:00 | |
*** jcoufal has joined #heat | 13:00 | |
*** ccrouch has joined #heat | 13:01 | |
*** bmahalakshmi2 has joined #heat | 13:04 | |
*** nikunj2512 has quit IRC | 13:04 | |
*** blues-man has joined #heat | 13:04 | |
*** bnemec has joined #heat | 13:04 | |
*** bmahalakshmi has quit IRC | 13:05 | |
*** rushiagr_ has quit IRC | 13:05 | |
*** bvandenh has joined #heat | 13:06 | |
*** jdob has joined #heat | 13:09 | |
*** rakesh_hs has quit IRC | 13:09 | |
*** aweiteka has joined #heat | 13:09 | |
*** jstrachan has quit IRC | 13:10 | |
*** jstrachan has joined #heat | 13:10 | |
*** lazy_prince has quit IRC | 13:11 | |
*** viktors|afk is now known as viktors | 13:11 | |
*** radez_g0n3 is now known as radez | 13:12 | |
*** derekh_ has joined #heat | 13:13 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: SoftwareDeployments move signal reason into resource plugin https://review.openstack.org/106825 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Appropriate exception for signal handling failure https://review.openstack.org/109096 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert CFN waitcondition API to resource_signal https://review.openstack.org/106824 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Allow plugins to specify signal event reason https://review.openstack.org/106821 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Update test_signal to use stub_keystoneclient https://review.openstack.org/106820 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Return metadata from resource_signal RPC interface https://review.openstack.org/106823 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: WaitConditionHandle resources, create events with reason data https://review.openstack.org/106822 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 13:14 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Mark metadata_update RPC call deprecated https://review.openstack.org/106985 | 13:14 |
*** jdob has quit IRC | 13:15 | |
*** jdob has joined #heat | 13:16 | |
*** achampion has joined #heat | 13:20 | |
*** mkerrin has joined #heat | 13:20 | |
*** mkerrin1 has joined #heat | 13:24 | |
*** Tross has quit IRC | 13:24 | |
*** blomquisg has joined #heat | 13:25 | |
*** mkerrin has quit IRC | 13:26 | |
*** tspatzier has joined #heat | 13:29 | |
*** bnemec is now known as beekneemech | 13:34 | |
openstackgerrit | A change was merged to openstack/heat: Add template version for the provider template https://review.openstack.org/96999 | 13:34 |
*** lsmola has joined #heat | 13:40 | |
*** bvandenh has quit IRC | 13:43 | |
*** sjmc7 has joined #heat | 13:50 | |
*** hipokrit has joined #heat | 13:54 | |
*** sjmc7_ has joined #heat | 13:55 | |
*** richa has joined #heat | 13:57 | |
*** samstav has joined #heat | 13:58 | |
*** sjmc7 has quit IRC | 13:59 | |
*** sergmelikyan has quit IRC | 14:00 | |
*** gokrokve has joined #heat | 14:01 | |
*** jasondotstar has joined #heat | 14:02 | |
*** sjmc7_ is now known as sjmc7 | 14:05 | |
*** chandan_kumar has joined #heat | 14:07 | |
*** chandan_kumar has quit IRC | 14:10 | |
*** bvandenh has joined #heat | 14:10 | |
*** ramishra_ has quit IRC | 14:12 | |
*** bvandenh has quit IRC | 14:15 | |
*** jstrachan has quit IRC | 14:17 | |
*** david-lyle has joined #heat | 14:18 | |
*** jstrachan has joined #heat | 14:18 | |
*** jasond` is now known as jasond | 14:18 | |
*** ramishra has joined #heat | 14:19 | |
*** vijendar has joined #heat | 14:20 | |
*** vijendar has quit IRC | 14:20 | |
*** vijendar has joined #heat | 14:20 | |
*** piyush has quit IRC | 14:21 | |
*** chandankumar has quit IRC | 14:21 | |
*** Dev_Jin has quit IRC | 14:23 | |
*** viktors has left #heat | 14:30 | |
*** ramishra has quit IRC | 14:33 | |
*** zackf has joined #heat | 14:44 | |
*** piyush has joined #heat | 14:46 | |
*** metral is now known as metral_zzz | 14:46 | |
*** tspatzier has quit IRC | 14:51 | |
*** jstrachan has quit IRC | 14:56 | |
*** david-ly_ has joined #heat | 14:58 | |
*** david-lyle has quit IRC | 14:59 | |
*** piyush1 has joined #heat | 15:00 | |
*** piyush has quit IRC | 15:01 | |
therve | shardy, Hi, around? I have a question about native wait condition | 15:10 |
shardy | therve: Hi | 15:11 |
therve | shardy, So I wonder what's that token storage about :) | 15:12 |
therve | Was it present in the original patch? | 15:12 |
*** gokrokve_ has joined #heat | 15:12 | |
shardy | therve: It's so we avoid requesting a new token every time the attribute is referenced | 15:12 |
therve | shardy, Do we really want to do that? I thought the point of the stack user was being able to use regular authentication | 15:13 |
therve | It seems using a token would have several shortcomings, expiration being the main one | 15:13 |
shardy | therve: well the point is to avoid having a password in the instance, although we could optionally allow that later | 15:14 |
shardy | therve: That's true, but it's designed for use-once signalling, where expiration is not really an issue | 15:14 |
*** bandarji has joined #heat | 15:14 | |
shardy | for more complex scenarios, SoftwareDeployments should probably be used instead | 15:14 |
shardy | happy to hear other ideas though ;) | 15:14 |
*** gokrokve has quit IRC | 15:14 | |
therve | Hum | 15:15 |
shardy | I wanted to avoid the instance needing to talk to keystone to get a token, e.g so one-line signalling from cirros is possible | 15:15 |
shardy | An alternative would be to enable signalling via python-heatclient using username/password, but that won't work for cirros | 15:15 |
shardy | And cirros was my main use-case for this :) | 15:16 |
*** BillArnold has joined #heat | 15:16 | |
*** Meeral has joined #heat | 15:17 | |
therve | Well requestion a token from keystone is just an additional request | 15:18 |
therve | I guess you don't have replay problems with wait conditions, so that's one thing | 15:18 |
therve | I can understand the need of convenience, I wish we'd be able to have the ability to use regular auth, which seems impossible with current implementation | 15:21 |
*** imcsk8 has quit IRC | 15:22 | |
*** imcsk8 has joined #heat | 15:22 | |
shardy | therve: what do you mean by "regular auth"? | 15:22 |
shardy | It's using regular auth via a token via the native rest API? | 15:22 |
therve | shardy, As the server stack user, signal the wait condition | 15:22 |
*** chandankumar has joined #heat | 15:22 | |
therve | Or does it work regardless? | 15:23 |
shardy | That's exactly what the patch does | 15:23 |
shardy | only using a stack-domain user identity | 15:23 |
*** radez is now known as radez_g0n3 | 15:23 | |
shardy | as you don't want to put the actual stack owner's token in the instance | 15:23 |
therve | Yeah but it's creating a new one? | 15:23 |
shardy | They can dignal using their credentials via heat resource-signal too if they wish | 15:24 |
shardy | Yeah, like all SignalResponder resources do | 15:24 |
shardy | We could optimise in future to only have one per stack I guess | 15:24 |
stevebaker | zaneb: hey, you about? | 15:24 |
therve | shardy, I guess my question is: Can I signal that resource using another stack user | 15:24 |
shardy | but eg. SoftwareDeployments (and the AWS WaitConditionHandle) do the exact same thing, just using different credentials | 15:25 |
shardy | therve: Yes | 15:25 |
therve | Okay that's what I misunderstand then :) | 15:25 |
zaneb | stevebaker: hey | 15:25 |
shardy | therve: Ok, cool :) | 15:25 |
therve | Presumably we could have a property to control whether creating a user or not | 15:26 |
shardy | therve: Yeah, we could add that later if folks need it, but then where would the user get created? | 15:26 |
shardy | Have a OS::Heat::StackUser resource? | 15:26 |
therve | shardy, In the server that uses it? | 15:26 |
shardy | therve: Ok, so like SoftwareDeployments work for polling | 15:27 |
shardy | therve: sure, we can do that later if needed | 15:27 |
therve | It's just a small optimization I suppose | 15:27 |
* shardy is in a meeting and needs to go for a bit | 15:27 | |
therve | Cheers, thanks for the answers | 15:27 |
stevebaker | zaneb: I'm rejigging create-retry on your refactor, but I have some inline comments in https://review.openstack.org/#/c/102397/ | 15:28 |
stevebaker | zaneb: I'm heading to the airport soon, we may have time to discuss the options | 15:29 |
zaneb | ok, looking | 15:29 |
*** jasondotstar has quit IRC | 15:30 | |
*** randallburt has joined #heat | 15:32 | |
zaneb | stevebaker: I see what you mean about creating events for the failing create/delete | 15:33 |
*** chandankumar has quit IRC | 15:33 | |
stevebaker | yup | 15:33 |
zaneb | that does make things tricky if we want to do that :/ | 15:33 |
*** erecio has joined #heat | 15:34 | |
stevebaker | I can just put with self._action_recorder(action): inside the loop? | 15:34 |
zaneb | the other option is to move the retry loop outside of _do_action altogether | 15:34 |
*** randallburt has quit IRC | 15:35 | |
*** randallburt has joined #heat | 15:35 | |
zaneb | you could, but as soon as the loop goes outside action_recorder, it changes your exception handling | 15:35 |
stevebaker | right now this only seems to make sense for create | 15:36 |
zaneb | you'd have to look at the ResourceException to see what type of exception triggered it | 15:36 |
*** bvivek2 has quit IRC | 15:36 | |
stevebaker | maybe I should just move this to create() | 15:36 |
zaneb | yeah, I would support that | 15:37 |
zaneb | even better would be to spin it out as a separate context manager or something, but that will be quite intricate work and YAGNI for now | 15:37 |
stevebaker | don't make me think, its been a big week ;) | 15:38 |
zaneb | lol | 15:38 |
zaneb | put it in create() | 15:38 |
*** erecio has quit IRC | 15:38 | |
stevebaker | zaneb: and how about my point about the backoff applying to deletes too? | 15:39 |
zaneb | I think that resolves the dependency issue between our patches too :) | 15:39 |
stevebaker | \o/ | 15:39 |
zaneb | I think in my implementation the backoff increases, but only if the delete fails | 15:40 |
zaneb | I believe that's what we want? | 15:40 |
stevebaker | that seems appropriate | 15:40 |
zaneb | arguably we should reign in the backoff (but keep the jitter) when a delete succeeds | 15:42 |
*** EricGonczer_ has joined #heat | 15:42 | |
zaneb | rein | 15:42 |
*** EricGonczer_ has quit IRC | 15:46 | |
*** kebray has joined #heat | 15:47 | |
*** kebray has quit IRC | 15:47 | |
*** kebray has joined #heat | 15:47 | |
*** jasondotstar has joined #heat | 15:48 | |
*** blomquisg has quit IRC | 15:48 | |
*** rbuilta has quit IRC | 15:52 | |
*** sergmelikyan has joined #heat | 15:54 | |
*** dims has quit IRC | 15:56 | |
*** jasondotstar has quit IRC | 15:57 | |
*** jasondotstar has joined #heat | 15:57 | |
*** jdob has quit IRC | 15:57 | |
*** Qiming has joined #heat | 15:58 | |
*** derekh_ has quit IRC | 15:58 | |
stevebaker | zaneb: ok, I've got something to hack on the plane now | 16:01 |
SpamapS | stevebaker: hacks on a plane | 16:01 |
zaneb | lol | 16:01 |
SpamapS | I'm tired of these monkey fightin hacks, on this monday friday plane! | 16:01 |
stevebaker | yus | 16:02 |
*** Meeral has quit IRC | 16:02 | |
zaneb | it's times like these I wish we had an IRC greatest hits wiki pages of quotes | 16:02 |
*** jcoufal has quit IRC | 16:03 | |
randallburt | I'm sick of all these m-effing patches on this m-effing plane!!! | 16:03 |
*** tomek_adamczewsk has quit IRC | 16:03 | |
*** metral_zzz is now known as metral | 16:03 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Implement barbican client plugin https://review.openstack.org/98684 | 16:04 |
*** ccrouch has quit IRC | 16:05 | |
*** dims has joined #heat | 16:07 | |
*** ccrouch has joined #heat | 16:13 | |
blinky_ghost | therve: I've rebuilt my fedora image and now I'm getting this message: WARNING os_collect_config.heat [-] No auth_url configured. WARNING os-collect-config [-] Source [heat] Unavailable. WARNING os_collect_config.cfn [-] 500 Server Error: Internal Server Error | 16:14 |
blinky_ghost | any hint? | 16:14 |
*** pasquier-s has quit IRC | 16:19 | |
ryansb | can someone take a peek at https://review.openstack.org/#/c/108399/ for me? It's a pretty trivial delete-old-stuff change. | 16:21 |
*** bandarji has quit IRC | 16:26 | |
*** joeroyall has quit IRC | 16:27 | |
*** blues-man has quit IRC | 16:30 | |
zaneb | ryansb: found a typo | 16:31 |
*** radez_g0n3 is now known as radez | 16:31 | |
*** richa has quit IRC | 16:32 | |
*** ramishra has joined #heat | 16:34 | |
ryansb | thanks zaneb, will re-submit | 16:36 |
*** ramishra has quit IRC | 16:37 | |
*** ramishra has joined #heat | 16:38 | |
openstackgerrit | Ryan Brown proposed a change to openstack/heat: Remove rst2script.sed and integration.sh https://review.openstack.org/108399 | 16:40 |
ryansb | zaneb: fixed ^ | 16:40 |
zaneb | +2 | 16:41 |
*** ramishra has quit IRC | 16:42 | |
*** lsmola has quit IRC | 16:43 | |
*** zslot has quit IRC | 16:43 | |
*** sgordon has joined #heat | 16:45 | |
*** sgordon has quit IRC | 16:45 | |
*** sgordon has joined #heat | 16:45 | |
*** bmahalakshmi2 has quit IRC | 16:48 | |
*** blinky_ghost has quit IRC | 16:50 | |
*** bmahalakshmi has joined #heat | 16:53 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Only support client plugins https://review.openstack.org/98688 | 16:54 |
*** iyozhikov is now known as IgorYozhikov | 16:56 | |
*** andrew_plunk has joined #heat | 16:56 | |
*** andrew_plunk has left #heat | 16:56 | |
*** gokrokve has joined #heat | 16:59 | |
*** Qiming has quit IRC | 17:00 | |
openstackgerrit | Victor Sergeyev proposed a change to openstack/heat: WIP: fix migrations https://review.openstack.org/109658 | 17:00 |
*** gokrokve_ has quit IRC | 17:02 | |
*** gokrokve has quit IRC | 17:03 | |
*** cdent has quit IRC | 17:04 | |
stevebaker | so we appear to have come full circle, and heat functional tests are heading back to the heat tree | 17:13 |
SpamapS | wtf? | 17:14 |
SpamapS | stevebaker: how so? | 17:14 |
stevebaker | SpamapS: see thread [openstack-dev] Thoughts on the patch test failure rate and moving forward | 17:15 |
stevebaker | SpamapS: I'm going to assume it won't be too hard to forklift the tempest orchestration scenario tests into heat and remove the tempest dependencies | 17:15 |
*** harlowja_away is now known as harlowja | 17:17 | |
SpamapS | stevebaker: Oh I didn't see that was how things were going. I think it's good actually. Heat's problems shouldn't be Cinder's problem. | 17:19 |
stevebaker | SpamapS: yeah, the extra work is a pain but it will be good | 17:20 |
*** bmahalakshmi2 has joined #heat | 17:22 | |
*** andersonvom has quit IRC | 17:23 | |
*** jcoufal has joined #heat | 17:23 | |
*** bmahalakshmi has quit IRC | 17:23 | |
*** randallburt has quit IRC | 17:26 | |
*** bmahalakshmi2 has quit IRC | 17:29 | |
*** ramishra has joined #heat | 17:32 | |
*** piyush1 has quit IRC | 17:33 | |
*** ramishra has quit IRC | 17:36 | |
*** rwsu has quit IRC | 17:41 | |
*** cody-somerville has joined #heat | 17:41 | |
*** andersonvom has joined #heat | 17:42 | |
*** gokrokve has joined #heat | 17:45 | |
*** joeroyall has joined #heat | 17:45 | |
*** Tross has joined #heat | 17:46 | |
*** blomquisg has joined #heat | 17:54 | |
*** piyush has joined #heat | 17:56 | |
*** piyush1 has joined #heat | 17:58 | |
*** piyush has quit IRC | 18:00 | |
*** andersonvom has quit IRC | 18:04 | |
*** andersonvom has joined #heat | 18:05 | |
openstackgerrit | William C. Arnold proposed a change to openstack/heat: Implements stack lifecycle plugpoints https://review.openstack.org/89363 | 18:11 |
*** killer_prince has joined #heat | 18:17 | |
*** killer_prince is now known as lazy_prince | 18:17 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Port resource https://review.openstack.org/104224 | 18:26 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Node resource https://review.openstack.org/104223 | 18:26 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add Ironic resource base class https://review.openstack.org/104222 | 18:26 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Chassis resource https://review.openstack.org/103133 | 18:26 |
*** joeroyall has quit IRC | 18:26 | |
jpeeler | zaneb: i guess you pretty much did this last week? https://bugs.launchpad.net/heat/+bug/1336072 | 18:27 |
uvirtbot | Launchpad bug 1336072 in heat "Purge remaining CFN functions in HOT" [Undecided,New] | 18:27 |
zaneb | I guess I did :) | 18:27 |
zaneb | except for Fn::Select | 18:27 |
zaneb | maybe leave a comment to that effect and target it to j-3 with high priority | 18:29 |
jpeeler | i'll take it if that's cool | 18:29 |
zaneb | that's fine, but to help out TripleO we don't want to remove Fn::Select until late in the Juno cycle | 18:30 |
*** randallburt has joined #heat | 18:30 | |
jpeeler | ah well okay | 18:31 |
zaneb | that's the only reason I left it in for now | 18:31 |
zaneb | you can submit a patch and -1 the workflow (i.e. make it WIP) if you want | 18:32 |
*** ramishra has joined #heat | 18:33 | |
*** mkerrin1 has quit IRC | 18:36 | |
*** ramishra has quit IRC | 18:37 | |
*** tonisbones has quit IRC | 18:43 | |
*** randallburt has quit IRC | 18:48 | |
*** randallburt has joined #heat | 18:48 | |
*** ccrouch has quit IRC | 18:51 | |
*** morganfainberg is now known as morganfainberg_Z | 18:53 | |
*** cody-somerville has quit IRC | 18:53 | |
*** zackf has quit IRC | 19:03 | |
*** zackf has joined #heat | 19:03 | |
*** hipokrit has quit IRC | 19:04 | |
*** cody-somerville has joined #heat | 19:10 | |
jpeeler | cody-somerville: whatever happened with this? https://bugs.launchpad.net/heat/+bug/1226287 | 19:18 |
uvirtbot | Launchpad bug 1226287 in oslo "Unable to prevent heat from logging to stdout without all heat proccesses logging to the same file" [Medium,Fix released] | 19:18 |
*** zackf has quit IRC | 19:19 | |
*** zackf has joined #heat | 19:27 | |
*** randallburt has quit IRC | 19:28 | |
*** gokrokve has quit IRC | 19:28 | |
*** jcoufal has quit IRC | 19:29 | |
*** ramishra has joined #heat | 19:33 | |
*** pasquier-s has joined #heat | 19:33 | |
*** ramishra has quit IRC | 19:38 | |
*** akuznetsov has quit IRC | 19:46 | |
*** pasquier-s has quit IRC | 19:52 | |
stevebaker | zaneb: should I be decorating create() with @scheduler.wrappertask? | 19:57 |
zaneb | stevebaker: yes, you'll probably have to | 19:57 |
stevebaker | zaneb: and yeild _do_action instead of return its result? | 19:58 |
zaneb | exactly | 19:58 |
stevebaker | k, thx | 19:58 |
*** david-ly_ has quit IRC | 20:02 | |
*** piyush1 has quit IRC | 20:08 | |
*** tspatzier has joined #heat | 20:11 | |
*** hipokrit has joined #heat | 20:15 | |
*** gokrokve has joined #heat | 20:19 | |
*** randallburt has joined #heat | 20:19 | |
*** randallburt has quit IRC | 20:20 | |
*** randallburt has joined #heat | 20:20 | |
*** dims has quit IRC | 20:20 | |
*** dims has joined #heat | 20:21 | |
*** tspatzier has quit IRC | 20:27 | |
openstackgerrit | A change was merged to openstack/heat: timeutils function retry_backoff_delay https://review.openstack.org/108477 | 20:30 |
*** ramishra has joined #heat | 20:34 | |
*** ccrouch has joined #heat | 20:36 | |
*** ramishra has quit IRC | 20:39 | |
*** beekneemech has quit IRC | 20:50 | |
*** achampion has quit IRC | 20:58 | |
*** piyush has joined #heat | 20:59 | |
*** dims has quit IRC | 21:03 | |
*** BillArnold has quit IRC | 21:03 | |
*** aweiteka has quit IRC | 21:18 | |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Add OS::Swift::Signal resource https://review.openstack.org/96947 | 21:27 |
*** vpanizo has joined #heat | 21:28 | |
*** vpanizo has quit IRC | 21:28 | |
*** gokrokve has quit IRC | 21:38 | |
*** hipokrit has quit IRC | 21:44 | |
*** ccrouch has quit IRC | 21:45 | |
*** jasondotstar has quit IRC | 21:52 | |
*** gokrokve has joined #heat | 21:54 | |
*** achampion has joined #heat | 21:55 | |
*** morganfainberg_Z is now known as morganfainberg | 21:55 | |
*** gokrokve has quit IRC | 21:55 | |
*** gokrokve has joined #heat | 21:56 | |
*** achampion has quit IRC | 21:59 | |
*** gokrokve has quit IRC | 22:01 | |
*** gokrokve has joined #heat | 22:03 | |
*** piyush has quit IRC | 22:07 | |
*** randallburt has quit IRC | 22:09 | |
openstackgerrit | Clark Boylan proposed a change to openstack/heat: Set python hash seed to 0 in tox.ini https://review.openstack.org/109756 | 22:11 |
*** ramishra has joined #heat | 22:24 | |
*** vijendar has quit IRC | 22:25 | |
*** ramishra has quit IRC | 22:29 | |
uberj | stevebaker: the SoftwareConfig/SoftwareDeploy pattern is quite elegant! | 22:30 |
uberj | thanks for mentioning it | 22:31 |
uberj | I SoftwareConfig resource needs to define its type (i.e. OS::Heat::SoftwareConfig::Chef) and a SoftwareDeploy resource needs to also define its type. If a SD is responsible for simply instantiating a SC instance (who already has an underlying type), why does a SD need a type? | 22:53 |
uberj | !s/^I/If/ | 22:53 |
openstack | uberj: Error: "s/^I/If/" is not a valid command. | 22:53 |
uberj | openstack: worst.sed.bot.ever | 22:53 |
*** morganfainberg is now known as morganfainberg_Z | 22:55 | |
*** gokrokve has quit IRC | 22:55 | |
*** harlowja is now known as harlowja_away | 22:58 | |
*** harlowja_away is now known as harlowja | 23:06 | |
*** piyush has joined #heat | 23:08 | |
*** morganfainberg_Z is now known as morganfainberg | 23:08 | |
*** randallburt has joined #heat | 23:12 | |
*** randallburt has quit IRC | 23:13 | |
*** sjmc7 has quit IRC | 23:14 | |
*** piyush has quit IRC | 23:15 | |
*** gokrokve has joined #heat | 23:18 | |
*** andersonvom_ has joined #heat | 23:19 | |
*** jdandrea_ has joined #heat | 23:20 | |
*** Chinorro_ has joined #heat | 23:21 | |
*** Tross1 has joined #heat | 23:21 | |
*** gpocente1 has joined #heat | 23:24 | |
*** shuffleb1t has joined #heat | 23:24 | |
*** vpanizo has joined #heat | 23:25 | |
*** ramishra has joined #heat | 23:25 | |
*** andersonvom has quit IRC | 23:29 | |
*** Tross has quit IRC | 23:29 | |
*** denis_makogon has quit IRC | 23:29 | |
*** gpocentek has quit IRC | 23:29 | |
*** Chinorro has quit IRC | 23:29 | |
*** ekarlso has quit IRC | 23:29 | |
*** shufflebot has quit IRC | 23:29 | |
*** jdandrea has quit IRC | 23:29 | |
*** boris-42 has quit IRC | 23:29 | |
*** ramishra has quit IRC | 23:30 | |
*** jdandrea_ is now known as jdandrea | 23:30 | |
*** boris-42 has joined #heat | 23:31 | |
*** denis_makogon has joined #heat | 23:31 | |
*** ekarlso has joined #heat | 23:32 | |
*** mtreinish has quit IRC | 23:33 | |
*** zackf has quit IRC | 23:43 | |
*** mtreinish has joined #heat | 23:46 | |
*** morganfainberg is now known as morganfainberg_Z | 23:51 | |
*** gokrokve has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!