| *** achanda has joined #heat | 00:00 | |
| asalkeld | stevebaker: before you go, just send me an email with anything you want done whilst you are away (things need doing, concerns, etc...) | 00:02 |
|---|---|---|
| stevebaker | asalkeld: ok, I've beaten up the l-3 milestone page. https://launchpad.net/heat/+milestone/liberty-3 | 00:03 |
| stevebaker | asalkeld: I've left bugs that have some hope of landing, but I'm not aware of any blockers | 00:03 |
| asalkeld | ok, what date are we shooting for? | 00:04 |
| stevebaker | asalkeld: I've put some blueprints into rc1, we can discuss at the meeting tonight whether they are worth a FFE https://launchpad.net/heat/+milestone/liberty-rc1 | 00:04 |
| *** achanda has quit IRC | 00:04 | |
| stevebaker | asalkeld: their thursday (our friday morning?) | 00:04 |
| asalkeld | ok, do we do 1-1 meetings anymore - anything i need to attend? | 00:05 |
| stevebaker | asalkeld: there is a free-form sync in #openstack-relmgr-office every morning and evening. You should join so you can be pinged, and you'll eventually be asked for a git hash to branch l-3 on | 00:07 |
| asalkeld | ok | 00:07 |
| *** Qiming has quit IRC | 00:07 | |
| stevebaker | asalkeld: thanks. I'll try and get online tomorrow evening | 00:09 |
| asalkeld | np | 00:09 |
| stevebaker | (from the Methvan pub after snowboarding ;) | 00:09 |
| asalkeld | i have added a convergence-bugs tag: https://bugs.launchpad.net/heat/+bugs?field.tag=convergence-bugs | 00:09 |
| asalkeld | stevebaker: sounds super productive ;) | 00:10 |
| *** achanda has joined #heat | 00:12 | |
| *** dims__ has quit IRC | 00:16 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #heat | 00:23 | |
| *** pm90_ has quit IRC | 00:27 | |
| *** zhenguo has joined #heat | 00:30 | |
| asalkeld | stevebaker: 2*+2 need approve https://review.openstack.org/#/c/215489/7 | 00:30 |
| stevebaker | asalkeld: done, also I'll approve once Pavlo's suggestion is done https://review.openstack.org/#/c/218695/1/heat/engine/resources/openstack/heat/test_resource.py,cm | 00:34 |
| asalkeld | stevebaker: yip, done that locally | 00:34 |
| asalkeld | just wanted the first one in | 00:34 |
| *** dims__ has joined #heat | 00:51 | |
| *** dims__ has quit IRC | 00:56 | |
| *** sabeen1 has quit IRC | 00:56 | |
| *** junhongl has joined #heat | 01:05 | |
| *** Qiming has joined #heat | 01:06 | |
| *** hdd has joined #heat | 01:06 | |
| *** achanda has quit IRC | 01:09 | |
| *** jaime49 has joined #heat | 01:13 | |
| *** erkules_ has joined #heat | 01:21 | |
| *** zeih has joined #heat | 01:21 | |
| *** erkules has quit IRC | 01:24 | |
| asalkeld | this has been up for months and on L3 -> https://review.openstack.org/#/c/197402/ | 01:25 |
| *** zeih has quit IRC | 01:26 | |
| *** kebray has quit IRC | 01:30 | |
| asalkeld | this gate must be really slow | 01:37 |
| *** ricolin has joined #heat | 01:39 | |
| stevebaker | ricolin: does the gate fail with this every time? https://bugs.launchpad.net/heat/+bug/1491199 | 01:42 |
| openstack | Launchpad bug 1491199 in heat "Image fedora-heat-test-image could not be found" [Undecided,New] | 01:42 |
| ricolin | stevebaker: we have few reviews hit the same issue, and it happens every time. | 01:46 |
| stevebaker | ricolin: I think python-glanceclient 1.0.0 has some changed command arguments | 01:47 |
| ricolin | stevebaker: or just network unstable issue | 01:48 |
| stevebaker | ricolin: http://logs.openstack.org/68/219368/1/check/gate-heat-dsvm-functional-orig-mysql/49b0e66/console.html#_2015-09-01_23_34_05_042 | 01:48 |
| asalkeld | https://github.com/openstack/python-glanceclient/commit/181131ef2ea153a3cb768bd224ddcde4a2b42665 | 01:51 |
| asalkeld | i wonder if switching to v2 api had anything to do with it | 01:51 |
| asalkeld | (by default) | 01:51 |
| stevebaker | --is-public and --location are still documented in glance help image-create | 01:52 |
| ricolin | yes | 01:52 |
| stevebaker | we could just switch to openstack client to get it working again | 01:54 |
| stevebaker | openstack image create --disk-format qcow2 --container-format bare --public --location http://tarballs.openstack.org/heat-test-image/fedora-heat-test-image.qcow2 fedora-heat-test-image | 01:55 |
| *** jdandrea has quit IRC | 01:55 | |
| ricolin | I will try it | 01:55 |
| *** Yanyanhu has joined #heat | 01:56 | |
| ricolin | maybe all our client should move to openstack client | 01:56 |
| stevebaker | ricolin: yes, I don't see why not. The others are starting to be considered legacy | 01:57 |
| asalkeld | you see glance now gets attributes from the server as schema, so you just don't know from looking at the client code | 01:57 |
| asalkeld | and the server is smoke and mirrors | 01:58 |
| *** hdd has quit IRC | 01:58 | |
| asalkeld | this is the closest i can come to: https://github.com/openstack/glance/blob/master/glance/api/v2/images.py#L832 | 01:59 |
| asalkeld | seeing if v2 supports is-public | 01:59 |
| asalkeld | but maybe it's now visibility=public | 02:00 |
| ricolin | if you grep is-public it only shows up in v1 | 02:01 |
| ricolin | Agree with you | 02:01 |
| asalkeld | https://bugs.launchpad.net/python-glanceclient lots of bugs | 02:02 |
| ricolin | visibility=public should be the new schema | 02:02 |
| asalkeld | related to v2 / v1 | 02:02 |
| *** liusheng has joined #heat | 02:02 | |
| asalkeld | glance --os-image-api-version 2 help image-create | 02:04 |
| asalkeld | --visibility <VISIBILITY> | 02:04 |
| *** achanda has joined #heat | 02:04 | |
| asalkeld | Scope of image accessibility Valid values: public, | 02:04 |
| asalkeld | private | 02:04 |
| asalkeld | default help is v1, but default action is v2 :-O | 02:04 |
| asalkeld | genius | 02:05 |
| ricolin | it is... | 02:07 |
| stevebaker | asalkeld: is there a replacement for --location? piping from curl seems to be working | 02:07 |
| asalkeld | only file | 02:08 |
| stevebaker | this works for me url http://tarballs.openstack.org/heat-test-image/fedora-heat-test-image.qcow2 | glance image-create --name fedora-heat-test-image --disk-format qcow2 --container-format bare --visibility public | 02:08 |
| asalkeld | --file <FILE> Local file that contains disk image to be uploaded | 02:08 |
| asalkeld | during creation. Must be present if images are not | 02:08 |
| asalkeld | passed to the client via stdin. | 02:08 |
| stevebaker | curl http://tarballs.openstack.org/heat-test-image/fedora-heat-test-image.qcow2 | glance image-create --name fedora-heat-test-image --disk-format qcow2 --container-format bare --visibility public | 02:08 |
| asalkeld | so stdin looks like the way to go | 02:09 |
| asalkeld | +1 go with that | 02:09 |
| asalkeld | always nice to have these issues before milestones | 02:10 |
| stevebaker | pff | 02:12 |
| ricolin | +1 | 02:13 |
| stevebaker | ricolin, asalkeld: who wants to propose the fix? | 02:13 |
| *** achanda has quit IRC | 02:14 | |
| *** kencjohn_ has joined #heat | 02:15 | |
| *** kencjohn_ has quit IRC | 02:15 | |
| asalkeld | stevebaker: is that to devstack? | 02:15 |
| *** pm90_ has joined #heat | 02:16 | |
| stevebaker | asalkeld: heat_integrationtests/prepare_test_env.sh | 02:16 |
| *** kencjohn_ has joined #heat | 02:16 | |
| asalkeld | ok, i can do it | 02:16 |
| ricolin | cool | 02:16 |
| *** Yanyanhu has quit IRC | 02:16 | |
| stevebaker | ta | 02:16 |
| *** Yanyanhu has joined #heat | 02:17 | |
| ricolin | remember add bug tag 1491199 | 02:18 |
| openstackgerrit | Angus Salkeld proposed openstack/heat: Use the new (and default) glance v2 cli options https://review.openstack.org/219533 | 02:18 |
| asalkeld | doh, i'll do that now | 02:19 |
| ricolin | :) | 02:19 |
| openstackgerrit | Angus Salkeld proposed openstack/heat: Use the new (and default) glance v2 cli options https://review.openstack.org/219533 | 02:20 |
| *** kebray has joined #heat | 02:20 | |
| *** zeih has joined #heat | 02:22 | |
| *** achanda has joined #heat | 02:23 | |
| *** steveg_afk has joined #heat | 02:24 | |
| *** elynn has joined #heat | 02:26 | |
| *** zeih has quit IRC | 02:26 | |
| *** ricolin has quit IRC | 02:32 | |
| elynn | morning :) | 02:32 |
| *** steveg_afk has quit IRC | 02:34 | |
| *** steveg_afk has joined #heat | 02:36 | |
| *** vijayagurug has joined #heat | 02:37 | |
| *** jaime49 has quit IRC | 02:42 | |
| *** ricolin has joined #heat | 02:44 | |
| *** ananta_ has joined #heat | 02:45 | |
| *** achanda has quit IRC | 02:49 | |
| *** pm90_ has quit IRC | 02:51 | |
| openstackgerrit | Qiming Teng proposed openstack/heat: Split engine service test cases (9) https://review.openstack.org/216130 | 02:51 |
| asalkeld | bbl | 02:53 |
| cmyster | morning | 02:56 |
| *** lkarm has joined #heat | 02:58 | |
| *** ricolin has quit IRC | 03:01 | |
| *** lkarm has quit IRC | 03:02 | |
| *** achanda has joined #heat | 03:02 | |
| *** achanda has quit IRC | 03:08 | |
| *** sabeen1 has joined #heat | 03:15 | |
| *** kencjohn_ has quit IRC | 03:18 | |
| *** zeih has joined #heat | 03:23 | |
| *** zeih has quit IRC | 03:28 | |
| *** ananta_ has quit IRC | 03:30 | |
| sirushti | morning all | 03:38 |
| *** steveg_afk has quit IRC | 03:40 | |
| openstackgerrit | Sirushti Murugesan proposed openstack/heat: Add console_scripts entry points for all heat services https://review.openstack.org/192334 | 03:43 |
| *** hdd has joined #heat | 03:43 | |
| *** links has joined #heat | 03:48 | |
| *** ricolin has joined #heat | 03:52 | |
| *** dims__ has joined #heat | 04:03 | |
| *** ishant has joined #heat | 04:08 | |
| *** jrist is now known as jrist-afk | 04:17 | |
| *** dims__ is now known as dims | 04:17 | |
| openstackgerrit | Hidekazu Nakamura proposed openstack/heat-templates: Add test code for software config hook for ansible https://review.openstack.org/219554 | 04:17 |
| *** zeih has joined #heat | 04:23 | |
| *** dims has quit IRC | 04:24 | |
| *** vijayagurug has quit IRC | 04:24 | |
| *** zeih has quit IRC | 04:28 | |
| *** sabeen1 has quit IRC | 04:33 | |
| *** ananta_ has joined #heat | 04:41 | |
| *** achanda has joined #heat | 04:42 | |
| *** getvasanth has joined #heat | 04:48 | |
| *** ananta_ has quit IRC | 04:49 | |
| *** jcoufal has joined #heat | 04:52 | |
| *** mtreinish has quit IRC | 04:57 | |
| openstackgerrit | Sirushti Murugesan proposed openstack/heat: Add console_scripts entry points for all heat services https://review.openstack.org/192334 | 04:57 |
| *** tochi has quit IRC | 04:57 | |
| *** mtreinish has joined #heat | 05:04 | |
| *** hdd has quit IRC | 05:09 | |
| *** tochi has joined #heat | 05:12 | |
| *** kebray has quit IRC | 05:13 | |
| *** zeih has joined #heat | 05:24 | |
| openstackgerrit | Rico Lin proposed openstack/heat-specs: switch opensack client https://review.openstack.org/219559 | 05:25 |
| *** inc0 has joined #heat | 05:27 | |
| *** zeih has quit IRC | 05:28 | |
| *** vijayagurug has joined #heat | 05:38 | |
| *** lkarm has joined #heat | 05:40 | |
| *** lkarm has quit IRC | 05:45 | |
| *** jcoufal has quit IRC | 05:50 | |
| *** jtomasek has joined #heat | 06:09 | |
| openstackgerrit | Rico Lin proposed openstack/heat-specs: switch opensack client https://review.openstack.org/219559 | 06:11 |
| *** Qiming_ has joined #heat | 06:12 | |
| *** KanagarajM has joined #heat | 06:17 | |
| KanagarajM | good morning all ! | 06:17 |
| *** rakesh_hs_ has joined #heat | 06:21 | |
| *** KanagarajM has quit IRC | 06:22 | |
| *** KanagarajM has joined #heat | 06:22 | |
| *** KanagarajM has quit IRC | 06:24 | |
| *** sdake has joined #heat | 06:24 | |
| *** KanagarajM has joined #heat | 06:24 | |
| *** zeih has joined #heat | 06:25 | |
| *** zeih has quit IRC | 06:29 | |
| ricolin | morning | 06:30 |
| *** pas-ha has joined #heat | 06:33 | |
| *** jprovazn has joined #heat | 06:44 | |
| *** KanagarajM_ has joined #heat | 06:45 | |
| *** KanagarajM has quit IRC | 06:46 | |
| *** achanda has quit IRC | 06:47 | |
| *** dims has joined #heat | 06:50 | |
| *** dims has quit IRC | 06:54 | |
| *** tlashchova has joined #heat | 06:55 | |
| *** Drago1 has joined #heat | 07:00 | |
| asalkeld | stevebaker: meeting? | 07:00 |
| *** KanagarajM_ has quit IRC | 07:04 | |
| *** zeih has joined #heat | 07:08 | |
| *** inc0 has quit IRC | 07:14 | |
| *** dkusidlo has joined #heat | 07:14 | |
| *** KanagarajM has joined #heat | 07:16 | |
| *** coolsvap_ has joined #heat | 07:21 | |
| *** achanda has joined #heat | 07:24 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/heat: Register all objects https://review.openstack.org/190183 | 07:24 |
| *** shardy has joined #heat | 07:25 | |
| *** coolsvap_ has quit IRC | 07:26 | |
| *** pbrooko has joined #heat | 07:28 | |
| *** zeih has quit IRC | 07:28 | |
| *** ananta_ has joined #heat | 07:33 | |
| *** dkusidlo has quit IRC | 07:33 | |
| *** dkusidlo has joined #heat | 07:34 | |
| *** shardy_ has joined #heat | 07:36 | |
| *** dkusidlo has quit IRC | 07:37 | |
| *** shardy has quit IRC | 07:37 | |
| *** KanagarajM_ has joined #heat | 07:39 | |
| *** dkusidlo has joined #heat | 07:41 | |
| *** shardy_ has quit IRC | 07:41 | |
| *** shardy has joined #heat | 07:42 | |
| *** zeih_ has joined #heat | 07:42 | |
| *** KanagarajM has quit IRC | 07:42 | |
| *** KanagarajM__ has joined #heat | 07:42 | |
| *** dims has joined #heat | 07:44 | |
| *** zengyingzhe_ has quit IRC | 07:44 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/heat: Connect object serializer https://review.openstack.org/199452 | 07:45 |
| *** KanagarajM_ has quit IRC | 07:46 | |
| *** ananta_ has quit IRC | 07:48 | |
| *** dims has quit IRC | 07:49 | |
| *** achanda has quit IRC | 07:51 | |
| ricolin | guys I need more comment for this new bp | 07:52 |
| ricolin | https://review.openstack.org/#/c/219559/ | 07:52 |
| *** lkarm has joined #heat | 07:52 | |
| shardy | ricolin: have you seen that stevebaker has already proposed adding an openstackclient plugin to the python-heatclient tree? | 07:53 |
| stevebaker | ricolin: are you keen to help? because I've started https://review.openstack.org/#/c/195867/ | 07:54 |
| shardy | IMO that's the way to go, because we control that repo and landing code in osc is notoriously hard/impossible | 07:54 |
| shardy | AFAIK they don't want projects that aren't "core" enough ;) | 07:54 |
| stevebaker | ...and they want new commands to be plugins anyway | 07:54 |
| shardy | stevebaker: sure, I was just commenting on the history, it's like grenade, where they blocked us for a year before finally declaring they didn't want the code and that it had to be a plugin | 07:55 |
| stevebaker | ricolin: I mean ryansb started, and I brought his work over to heatclient and change a bunch of stuff | 07:55 |
| ricolin | stevebaker: okey~ | 07:56 |
| shardy | ricolin: FWIW I'm +1 on having a fully functional osc plugin, but I'm strongly -1 on making it *the* heat interface, at least in the forseeable/near future | 07:56 |
| *** lkarm has quit IRC | 07:56 | |
| shardy | IMHO keystone made a mistake by having a very short deprecation period and it was not friendly to users/operators at all | 07:57 |
| stevebaker | shardy: for us it will be all heatclient anyway, so neither will be going away | 07:57 |
| ricolin | agree | 07:57 |
| stevebaker | we may have a period of mandatory feature parity, when any new argument needs to be applied to both | 07:58 |
| shardy | stevebaker: cool, as long as we maintain both for now that sounds good to me | 07:58 |
| *** Drago1 has quit IRC | 07:58 | |
| shardy | I guess the question will come if/when we do a v2 API - that was the point where keystone did a flag-day migration, but it was still an unfriendly way to do it IMO | 07:59 |
| shardy | hopefully keeping the code in the same repo will minimse the expense of maintaining the two shell interfaces | 07:59 |
| shardy | (and probably keep us honest wrt pushing logic behind the python APIs) | 08:00 |
| stevebaker | shardy: yes, and maybe someone will be motivated to convert the legacy commands to cliff, so there will be even less overhead | 08:00 |
| shardy | stevebaker: yup, something like that would be great | 08:01 |
| *** e0ne has joined #heat | 08:01 | |
| *** KanagarajM_ has joined #heat | 08:03 | |
| *** KanagarajM__ has quit IRC | 08:06 | |
| *** inc0 has joined #heat | 08:10 | |
| *** jistr has joined #heat | 08:11 | |
| *** yassine_ has joined #heat | 08:11 | |
| *** e0ne has quit IRC | 08:13 | |
| *** zeih_ has quit IRC | 08:19 | |
| *** zeih has joined #heat | 08:19 | |
| *** Drago1 has joined #heat | 08:21 | |
| skraynev | KamagarajM_: could you please clarify: what do you mean by duplication code in patch https://review.openstack.org/#/c/207118/6 | 08:23 |
| *** KanagarajM_ has quit IRC | 08:28 | |
| skraynev | asalkeld: could you post comment about your patch for this bug https://bugs.launchpad.net/heat/+bug/1491199 | 08:36 |
| openstack | Launchpad bug 1491199 in heat "glance: error: unrecognized arguments: --is-public True --location..." [Critical,In progress] - Assigned to Angus Salkeld (asalkeld) | 08:36 |
| skraynev | looks like it was not added automatically | 08:37 |
| *** Beetnik has joined #heat | 08:37 | |
| *** dims has joined #heat | 08:38 | |
| *** dims has quit IRC | 08:43 | |
| *** zeih has quit IRC | 08:43 | |
| *** Marga_ has joined #heat | 08:44 | |
| *** zeih has joined #heat | 08:47 | |
| *** Marga_ has quit IRC | 08:48 | |
| openstackgerrit | Tetiana Lashchova proposed openstack/heat: Support use_autoconfig in sahara cluster templates https://review.openstack.org/219602 | 08:49 |
| *** Marga_ has joined #heat | 08:49 | |
| *** derekh has joined #heat | 08:50 | |
| asalkeld | skraynev: therve just did | 08:57 |
| skraynev | asalkeld: ok. cool. thx | 08:57 |
| therve | Yeah sorry for not mentioning it | 08:57 |
| asalkeld | not sure why it didn't do that normally | 08:58 |
| asalkeld | (bot is exhausted?) | 08:58 |
| openstackgerrit | Rico Lin proposed openstack/python-heatclient: OpenStackClient plugin for stack show https://review.openstack.org/195867 | 09:00 |
| *** tochi has quit IRC | 09:01 | |
| *** tochi has joined #heat | 09:01 | |
| ricolin | shardy: do you have time to review https://review.openstack.org/#/c/209439/ ? | 09:04 |
| *** Marga_ has quit IRC | 09:05 | |
| shardy | ricolin: thanks, I'll try to take a look today | 09:07 |
| ricolin | shardy: thx, just need to make sure that's what you mean | 09:09 |
| shardy | ricolin: sure, will do, sorry there's been so much confusion, I'm not sure why - perhaps my initial bug report wasn't clear enough | 09:09 |
| *** KanagarajM has joined #heat | 09:09 | |
| skraynev | stevebaker: regarding this one https://review.openstack.org/#/c/217753/8. Do we really want to support case with only provided subnet without network? it's more tricky and has difference with neutron port interface | 09:10 |
| KanagarajM | skraynev: role assignment as added as part of User and Group resource plugins, as this patch introduce new resource plugins for the same, suggested to deprecate the properties in User and Group for role_assignments. | 09:12 |
| ricolin | shardy: I think those confusion caused by my wrong implementation | 09:13 |
| ricolin | shardy: we just find two different possible anyway : -) | 09:13 |
| skraynev | so my main point, that it's just a difference between port API and properties for server | 09:14 |
| openstackgerrit | Tetiana Lashchova proposed openstack/heat: Support use_autoconfig in sahara clusters https://review.openstack.org/219614 | 09:14 |
| shardy | ricolin: heh, okay, well thanks for picking it up anyway, I'll try to take a look later :) | 09:14 |
| ricolin | : -) | 09:16 |
| skraynev | KanagarajM: hm... you want to deprecate "user" and "group" properties ? but mixn class has not such propeties | 09:18 |
| KanagarajM | skraynev: no, in User and Group resource plugins, deprecate roles properties. | 09:19 |
| KanagarajM | as it will be take care by the new keystone plugins | 09:20 |
| skraynev | KanagarajM: I am puzzled... could you give a link on resource, where you want to deprecate property | 09:21 |
| skraynev | ? | 09:21 |
| KanagarajM | skraynev: 1sec | 09:21 |
| *** elynn has quit IRC | 09:22 | |
| *** Yanyanhu has quit IRC | 09:23 | |
| KanagarajM | skraynev: https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/keystone/role_assignments.py#L72 | 09:24 |
| skraynev | KanagarajM: it's mixn class and used in some other classes, so Roles property need for them (e.g. https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/keystone/user.py) | 09:27 |
| *** amaksimov is now known as maximov | 09:29 | |
| KanagarajM | skraynev: but UserRoleAssignment and GroupRoleAssignment will do the same functionality as roles property in user.py and group.py | 09:29 |
| *** KanagarajM_ has joined #heat | 09:31 | |
| KanagarajM_ | skraynev: but UserRoleAssignment and GroupRoleAssignment will do the same functionality as roles property in user.py and group.py | 09:31 |
| *** e0ne has joined #heat | 09:32 | |
| KanagarajM_ | skraynev: so in tempalte, for example, we use OS::Keystone::UserRoleAssignment in place of OS::Keystone::User's property 'roles' | 09:32 |
| *** dims has joined #heat | 09:32 | |
| skraynev | KanagarajM_: but in this case you should create several OS::Keystone::UserRoleAssignment, if you want to create several relationships. In the same time Roles allows you to specify just a list | 09:34 |
| *** KanagarajM has quit IRC | 09:34 | |
| openstackgerrit | Rico Lin proposed openstack/heat-specs: switch opensack client https://review.openstack.org/219559 | 09:36 |
| skraynev | KanagarajM_: hang on.. or you suggest deprecate Roles property only for resources in user.py and group.py and save them for userassignmenta and roleassignments | 09:36 |
| skraynev | ? | 09:36 |
| *** dims has quit IRC | 09:37 | |
| *** KanagarajM_ has quit IRC | 09:37 | |
| *** KanagarajM has joined #heat | 09:41 | |
| *** KanagarajM has quit IRC | 09:41 | |
| *** KanagarajM has joined #heat | 09:42 | |
| openstackgerrit | Rico Lin proposed openstack/heat-specs: switch opensack client https://review.openstack.org/219559 | 09:42 |
| KanagarajM | skraynev: yes, i was asking for it | 09:44 |
| *** e0ne has quit IRC | 09:46 | |
| *** zeih has quit IRC | 09:50 | |
| openstackgerrit | Rico Lin proposed openstack/heat-specs: switch opensack client https://review.openstack.org/219559 | 09:52 |
| *** e0ne has joined #heat | 09:55 | |
| *** ochuprykov has joined #heat | 09:56 | |
| *** ochuprykov has quit IRC | 09:56 | |
| *** ochuprykov has joined #heat | 09:59 | |
| *** ochuprykov has quit IRC | 10:02 | |
| *** tochi has quit IRC | 10:04 | |
| *** dkusidlo has quit IRC | 10:14 | |
| *** blinky_ghost has joined #heat | 10:14 | |
| *** dims has joined #heat | 10:26 | |
| skraynev | KanagarajM: hm... I am not sure, that we should do it: 1. currently I don't know how to deprecate these properties only in some child classes (if we deprecate it in mixin - it will be deprecated everywhere in child resources.) | 10:29 |
| skraynev | 2. I am not sure, that we should deprecate it, because it gives additional way to define role assignments | 10:29 |
| skraynev | like we may define FIPAssociation in FIP resource | 10:30 |
| KanagarajM | skraynev: only concern here is, repeating the functionality across more than one resource plugin. | 10:30 |
| *** dims has quit IRC | 10:31 | |
| skraynev | KanagarajM: I don't think, that it's really repeating. | 10:31 |
| skraynev | it use same mixn class as parent | 10:31 |
| openstackgerrit | Rico Lin proposed openstack/heat-specs: switch openstack client https://review.openstack.org/219559 | 10:32 |
| skraynev | also we allows to define role assignment in several resources | 10:32 |
| skraynev | as user wants | 10:32 |
| skraynev | do we have any bad sides of existing "repetition" ? | 10:32 |
| *** ricolin has quit IRC | 10:36 | |
| *** chmouel has quit IRC | 10:40 | |
| *** chmouel has joined #heat | 10:41 | |
| *** akuznetsova has quit IRC | 10:41 | |
| *** akuznetsova has joined #heat | 10:41 | |
| KanagarajM | skraynev: i was in meeting ..... 1. is it ok to file a defect to deprecate it and go ahead with this patch 2. more than once resource plugins for same purpose would bring us the maintanence over head | 10:51 |
| KanagarajM | skraynev: bad side , i don't see any, other than repetition of it. | 10:53 |
| skraynev | KanagarajM: 1. let's file a bug and discuss it separately from patch ;) 2. I am not sure, that we maintenance a lot of resources. it's just a mixin class, which spread some functionality to other resources. | 10:54 |
| *** Qiming has quit IRC | 10:54 | |
| *** Qiming_ is now known as Qiming | 10:54 | |
| *** KanagarajM_ has joined #heat | 10:54 | |
| skraynev | user may use UserRoleAssigment to show, that it's separate thing, nut if he want he may define assignments inside of User resource (without user Role assignment resource) | 10:55 |
| *** KanagarajM has quit IRC | 10:58 | |
| *** dkusidlo has joined #heat | 10:59 | |
| KanagarajM_ | skraynev: to show, if i understood right, user should create the roleassignment using UserRoleAssignment, but he can't show those role assignment created via user, so even when we do _show_resource in User or Group, we report only User object or Group object | 11:01 |
| *** rakesh_hs__ has joined #heat | 11:02 | |
| *** rakesh_hs_ has quit IRC | 11:02 | |
| skraynev | KanagarajM_: yes, you can not see separate resource, but you can find this information in corresponding service, like we do it for volumes (volume attachments) and FIP (fip associations) | 11:03 |
| *** dkusidlo has quit IRC | 11:03 | |
| skraynev | so I don't think, that it's something special. | 11:03 |
| KanagarajM_ | skraynev: i found a code, which could do depreacation by manipulating 'properties_schema[role_assignments.KeystoneRoleAssignment.properties_schema.ROLES].support_status' in user and group | 11:05 |
| KanagarajM_ | skraynev: will that help to set the deprecation ? | 11:05 |
| skraynev | KanagarajM_: probably - yes, but I still thing, that it's not good idea | 11:05 |
| KanagarajM_ | skraynev: kindly let me know why ? | 11:12 |
| skraynev | KanagarajM_: because as I said before - we have the same situation for volumes and FIP, i.e. you may create association and you may define this association inside some other resource | 11:13 |
| skraynev | and it looks logically | 11:13 |
| KanagarajM_ | skraynev: ok. | 11:15 |
| *** junhongl_ has joined #heat | 11:16 | |
| *** junhongl has quit IRC | 11:19 | |
| *** dims has joined #heat | 11:21 | |
| *** dims has quit IRC | 11:26 | |
| *** zeih has joined #heat | 11:29 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add floating_ip property to Nova::Server nics https://review.openstack.org/217754 | 11:30 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add intrinsic creating port for server https://review.openstack.org/217753 | 11:30 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add port_extra_properties to Nova::Server nics https://review.openstack.org/218257 | 11:30 |
| *** KanagarajM_ has quit IRC | 11:30 | |
| *** pbrooko has quit IRC | 11:33 | |
| *** dkusidlo has joined #heat | 11:33 | |
| openstackgerrit | Limor Stotland proposed openstack/heat: fix : removing policies from mistral types https://review.openstack.org/218633 | 11:34 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Check number of deployments after stack creation https://review.openstack.org/218627 | 11:34 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Add batching create to ResourceGroup https://review.openstack.org/185385 | 11:34 |
| openstackgerrit | Limor Stotland proposed openstack/heat: fix : removing policies from mistral types https://review.openstack.org/218633 | 11:35 |
| *** pbrooko has joined #heat | 11:36 | |
| *** junhongl__ has joined #heat | 11:37 | |
| *** junhongl_ has quit IRC | 11:40 | |
| *** pbrooko has quit IRC | 11:41 | |
| *** bapalm has quit IRC | 11:43 | |
| *** bapalm has joined #heat | 11:49 | |
| *** Qiming_ has joined #heat | 11:49 | |
| *** zeih has quit IRC | 12:02 | |
| zaneb | gate is broken with "Error validating value 'fedora-heat-test-image': The Image (fedora-heat-test-image) could not be found." in functional tests. any ideas? | 12:03 |
| *** steveg_afk has joined #heat | 12:05 | |
| therve | zaneb, Almost fixed | 12:08 |
| therve | When the queue is getting empty, that is | 12:08 |
| therve | It only got there 8h ago... | 12:09 |
| *** jcoufal has joined #heat | 12:11 | |
| *** spzala has joined #heat | 12:12 | |
| *** dkusidlo has quit IRC | 12:16 | |
| *** dkusidlo has joined #heat | 12:19 | |
| *** junhongl__ has quit IRC | 12:20 | |
| *** jcoufal has quit IRC | 12:22 | |
| therve | Is there a benefit for us to be in the integrated queue? | 12:22 |
| therve | Projects break us every other day anyway, and it increases landing time uselessly | 12:23 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (1) https://review.openstack.org/219669 | 12:23 |
| skraynev | zaneb: we have patch with fix, but it is in queue... | 12:23 |
| skraynev | zaneb: https://review.openstack.org/#/c/219533/ | 12:23 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (1) https://review.openstack.org/219669 | 12:26 |
| *** lkarm has joined #heat | 12:27 | |
| *** ricolin has joined #heat | 12:27 | |
| *** david-lyle has quit IRC | 12:27 | |
| *** l0r3zz_ has quit IRC | 12:28 | |
| *** l0r3zz_ has joined #heat | 12:28 | |
| *** david-lyle has joined #heat | 12:28 | |
| *** inc0_ has joined #heat | 12:31 | |
| *** inc0 has quit IRC | 12:32 | |
| *** jaime49 has joined #heat | 12:34 | |
| *** jaime49 has quit IRC | 12:37 | |
| *** inc0_ has quit IRC | 12:38 | |
| *** jprovazn has quit IRC | 12:38 | |
| *** liusheng has quit IRC | 12:42 | |
| jdob | shardy: thanks for the guidance on the list_stacks show_nested pattern, that'll speed that along to have something to follow | 12:46 |
| shardy | jdob: np, thanks for working on it! | 12:46 |
| shardy | I'm a bit snowed under with reviews today, but I'll try to pull it later and offer some more suggestions | 12:47 |
| jdob | cool, I'm hoping to have another patch before normal end of your day so i can incorporate anything in my afternoon. code freeze is EOD (relative term, I know) tomorrow right? | 12:47 |
| shardy | jdob: if it helps, just wire in a new show_nested as True in your patch, and I can help wire it in via the API and heatclient later | 12:48 |
| *** dims has joined #heat | 12:48 | |
| jdob | ah cool, thanks. i'll see how far I get | 12:48 |
| shardy | jdob: Yeah, but I think stevebaker has said asking for a FFE may be possible, which will give us a little more time | 12:48 |
| jdob | whats convention around here, should the client change be the same commit or a different one? | 12:48 |
| jdob | whats FFE? feature freeze extension? | 12:49 |
| jdob | i saw him make a comment in the blueprint but I didnt get to ask what it meant | 12:49 |
| ryansb | feature freeze exception, I believe | 12:49 |
| shardy | jdob: Yeah, feature freeze exception | 12:50 |
| shardy | jdob: if we can show it's shaping up reasonably well by tomorrow, I think asking for one should be OK | 12:50 |
| jdob | gotcha, and cool, that'd make our lives (well, tuskar UI) easier if we can get this in | 12:50 |
| shardy | let's sync up tomorrow and see how you feel about it :) | 12:50 |
| jdob | perfect, I'll coffee up and get started on the flag | 12:50 |
| jdob | though let's quickly talk about the recursion/validate comment | 12:50 |
| jdob | is that something you think needs to be addressed in this patch? | 12:51 |
| shardy | jdob: you mean the files comment? | 12:51 |
| jdob | this one: "My concern also is that the validate code-paths for create/update (e.g stack.Stack.validate) have already diverged from the template-validate code, we do run the risk of making that worse if we end up with a fully separate recursive validate? " | 12:51 |
| shardy | jdob: Yeah, I think that is a risk, but we can probably refactor after, if you're comfortable with the current approach | 12:52 |
| jdob | ok, cool, thats how I'd like to handle it too | 12:52 |
| shardy | jdob: I was hoping we could kill two birds with one stone and just align the code-paths while implementing this initially tho :) | 12:52 |
| jdob | i'll get some thoughts together on how I think we could extract them out | 12:52 |
| shardy | jdob: if we go for a FFE then it's probably got to be low-risk and self-contained, so not messing with the create validation is probably best I guess | 12:53 |
| jdob | thats what i'd be more comfortable with, assuming we're cool with that divergence lasting a bit longer | 12:53 |
| shardy | I'd like to get some ideas developing about how we fix the divergence long term tho, even if that waits until M | 12:53 |
| *** e0ne has quit IRC | 12:53 | |
| jdob | ok, cool, i'd like to be part of that too | 12:53 |
| shardy | jdob: yup, cool, lets just look out for ways to minimise it where possible for now | 12:53 |
| shardy | jdob: would it help if I did some patches which wired in files, and show_nested, including the heatclient changes? | 12:55 |
| shardy | jdob: then you can focus on the core validation stuff and I'll rebase your patch on top? | 12:56 |
| jdob | if you have time, help certainly wouldn't hurt. if not, i'm fine with trying to figure it out. i'd like to at least do the API change so i can start to familiarize with that code | 12:56 |
| jdob | the heatclient changes i'm less on top of; still not entirely sure of what the files change is yet | 12:57 |
| shardy | jdob: Ok, if I get time later I'll do a couple of WIP patches we can discuss | 12:57 |
| jdob | cool | 12:58 |
| shardy | the files change is because in the resource_registry we may have e.g OS:Foo: foo.yaml | 12:58 |
| *** blomquisg has quit IRC | 12:58 | |
| shardy | but atm template_validate doesn't have the foo.yaml body | 12:58 |
| shardy | so you can't recurse in to it | 12:58 |
| shardy | it's the TemplateResource thing, you can't instantiate it without the template | 12:58 |
| shardy | https://github.com/openstack/python-heatclient/blob/master/heatclient/v1/shell.py#L726 | 13:00 |
| shardy | I think we actually already pass it in from the client | 13:00 |
| shardy | but it's ignored in the API: | 13:00 |
| shardy | https://github.com/openstack/heat/blob/master/heat/api/openstack/v1/stacks.py#L513 | 13:00 |
| shardy | So we need to add data.files() there, and the related files=None to the RPC client/service | 13:01 |
| shardy | https://github.com/openstack/heat/blob/master/heat/rpc/client.py#L294 | 13:01 |
| *** jcoufal has joined #heat | 13:02 | |
| *** inc0 has joined #heat | 13:03 | |
| *** blomquisg has joined #heat | 13:12 | |
| *** jprovazn has joined #heat | 13:13 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (2) https://review.openstack.org/219694 | 13:13 |
| *** jdandrea has joined #heat | 13:13 | |
| *** zeih has joined #heat | 13:14 | |
| jdob | ahhh, ok, that's the part I didn't get, I figured it was sent by the client, but you saying its ignored in the API makes more sense now | 13:17 |
| *** ishant has quit IRC | 13:17 | |
| *** links has quit IRC | 13:17 | |
| *** dims_ has joined #heat | 13:17 | |
| *** kencjohn_ has joined #heat | 13:19 | |
| *** dims has quit IRC | 13:21 | |
| *** inc0 has quit IRC | 13:31 | |
| *** ricolin_ has joined #heat | 13:32 | |
| *** jaime49 has joined #heat | 13:32 | |
| *** ricolin has quit IRC | 13:34 | |
| *** jaime49 has quit IRC | 13:36 | |
| *** jaime49 has joined #heat | 13:37 | |
| *** jasond has joined #heat | 13:40 | |
| jdob | shardy: ping again | 13:42 |
| *** dkusidlo has quit IRC | 13:44 | |
| openstackgerrit | Rakesh H S proposed openstack/heat: Convergence: Fix failing scenario tests https://review.openstack.org/219699 | 13:45 |
| shardy | jdob: hey | 13:45 |
| jdob | hey, got a few more seconds to talk about this data.files() thing? | 13:45 |
| shardy | jdob: sure | 13:45 |
| jdob | ok, so, data.files() will return the dict with all files, including the parent template and environment? | 13:46 |
| jdob | whats confusing me is a call in teh API that does: | 13:46 |
| jdob | data.template(), | 13:46 |
| jdob | data.environment(), | 13:46 |
| jdob | data.files(), | 13:46 |
| jdob | feels redundant | 13:46 |
| *** blinky_ghost_ has joined #heat | 13:46 | |
| *** blinky_ghost_ has quit IRC | 13:47 | |
| shardy | jdob: so template() is the parent template, environment is the environment (resource_registry, parameters and parameter_defaults), and files() is all the nested templates and other file references, e.g stuff like scripts and puppet manifests | 13:47 |
| shardy | arguably it could be rationalized, but that's the way it works atm | 13:47 |
| *** dkusidlo has joined #heat | 13:47 | |
| jdob | ah, ok, so files is smart enough to rip out the parent and env | 13:47 |
| shardy | jdob: yup, in heatclient we walk the templates looking for e.g type: foo.yaml, or resource_registry entries which reference templates | 13:48 |
| jdob | ok, so the call into service.py:validate_template is going to change to accept that file dict as well, and then somehow jam that into the loading mechanism that validate calls | 13:48 |
| shardy | or get_file references | 13:48 |
| shardy | then we bundle the contents of all into files() | 13:48 |
| jdob | i always thought files() also contained the parent and env, gotcha | 13:49 |
| shardy | which is a map of e.g "foo.yaml": "<body of foo.yaml" | 13:49 |
| *** blinky_ghost has quit IRC | 13:49 | |
| jdob | ok, something else that's confusing me | 13:50 |
| jdob | my code works right now | 13:50 |
| jdob | without that change | 13:50 |
| jdob | like, from a live REST call to running server, not just a unit test | 13:50 |
| shardy | jdob: hmm | 13:50 |
| shardy | I may need to make coffee and consider that one ;) | 13:50 |
| jdob | haha | 13:50 |
| shardy | give me 5, I'll need to take a look at the code and/or test it | 13:51 |
| shardy | I'm not actually sure how that is possible, so you must be doing something special! :D | 13:51 |
| *** spzala has quit IRC | 13:51 | |
| jdob | ok cool, thanks. reason I'm asking is because my test cases write out temp files with the nested templates and pass in file:// urls in the env, so i'm wondering if there's a way i can accomplish that entirely in memory | 13:51 |
| *** spzala_ has joined #heat | 13:51 | |
| jdob | and if the validate_template call is going to change to accept a dict, that'll clean up the unit tests, but then i'm not sure of how the loading will work | 13:52 |
| *** spzala_ is now known as spzala | 13:52 | |
| shardy | jdob: possibly the file:/// URLs are resolving locally? | 13:52 |
| jdob | on my live call? | 13:52 |
| shardy | that is explicitly forbidden in the actual stack-create (e.g in environment.py) for security reasons | 13:52 |
| jdob | on the live REST call, they don't use file urls, just relative paths | 13:52 |
| jdob | e.g. nested-1.yaml | 13:53 |
| shardy | jdob: yeah, are the files on the same machine as the heat-engine? | 13:53 |
| jdob | and i assumed the heat client read all of that and sent it through rest | 13:53 |
| shardy | jdob: you can do heat --debug template-validate ... | 13:53 |
| shardy | then you'll see what is passed | 13:53 |
| jdob | oh, lemme look at that | 13:53 |
| shardy | I just thought the files map was then dropped/ignored in the API atm | 13:53 |
| * shardy wonders off to make coffee | 13:54 | |
| *** spzala has quit IRC | 13:54 | |
| jdob | ah, looks like the client is adding the file URL | 13:54 |
| jdob | {"files": {"file:///home/jdob/code/nested-1a2.yaml": | 13:54 |
| jdob | the env just says nested-1a2.yaml | 13:54 |
| jdob | well, shit :) | 13:59 |
| *** pm90_ has joined #heat | 14:02 | |
| openstackgerrit | Jay Dobies proposed openstack/heat: Resolve parameter list including nested stacks https://review.openstack.org/219016 | 14:03 |
| *** pm90__ has joined #heat | 14:05 | |
| *** inc0 has joined #heat | 14:06 | |
| openstackgerrit | Rakesh H S proposed openstack/heat: convergence scenario tests https://review.openstack.org/203661 | 14:07 |
| openstackgerrit | Rakesh H S proposed openstack/heat: Convergence: Fix failing scenario tests https://review.openstack.org/219699 | 14:07 |
| *** jprovazn has quit IRC | 14:07 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Add batching create to ResourceGroup https://review.openstack.org/185385 | 14:07 |
| *** pm90_ has quit IRC | 14:09 | |
| *** rakesh_hs__ has quit IRC | 14:09 | |
| *** zeih has quit IRC | 14:09 | |
| *** jprovazn has joined #heat | 14:11 | |
| *** getvasanth has quit IRC | 14:12 | |
| *** getvasanth has joined #heat | 14:12 | |
| *** dkusidlo has quit IRC | 14:13 | |
| shardy | jdob: aha, yeah that shouldn't/won't work when you start instantiating TemplateResource objects: | 14:13 |
| shardy | https://github.com/openstack/heat/blob/master/heat/engine/resources/template_resource.py#L181 | 14:13 |
| *** Piet has quit IRC | 14:13 | |
| shardy | we look in the stack.t.files (which is from that files map passed in to service.py) | 14:13 |
| shardy | and if not found then get_template_file attempts to download, but file:/// isn't allowed for user templates | 14:13 |
| shardy | (vs those defined globally e.g in /etc/heat/environment.d | 14:14 |
| jdob | well, not encouraged, but it works :) | 14:14 |
| shardy | lol | 14:14 |
| jdob | though i might be kinda stuck | 14:14 |
| jdob | because generate_class in template_resource calls urlfetch | 14:14 |
| shardy | it works, it's just a security hole :D | 14:14 |
| jdob | there doesnt seem to be a way to pass the contents in | 14:14 |
| shardy | jdob: the line I just linked | 14:15 |
| jdob | 181? | 14:15 |
| shardy | That's why the files map is needed | 14:15 |
| shardy | then you can pass it in when creating the stack object, and the stack.t.files will find the file | 14:15 |
| jdob | oh oh, ok, and stack.t, I'm guessing that's a Template object? | 14:15 |
| jdob | which has a files parameter | 14:15 |
| shardy | then the urlfetch won't happen :) | 14:15 |
| *** zeih has joined #heat | 14:15 | |
| shardy | jdob: yup | 14:15 |
| jdob | ok cool, lemme see about wiring that up. so i'll make the change to the API and engine method to pass files too, you dont have to worry about that | 14:16 |
| *** zeih has quit IRC | 14:16 | |
| shardy | This is kinda why I was mentioning the stack-preview approach, because it already should handle all this | 14:16 |
| jdob | i'll do that part now, since i need it to keep going | 14:16 |
| jdob | i'm looking at that now | 14:16 |
| *** crose has joined #heat | 14:16 | |
| shardy | (although my testing shows at least the output is broken atm..) | 14:16 |
| jdob | ya, I didnt even look since you mentioned that, though in hindsight I should have | 14:17 |
| shardy | It's all good, there are pros/cons to both so it's good to investigate | 14:17 |
| jdob | i'm not actually instantiating a Stack object anywhere, but this stuff is | 14:17 |
| shardy | the main thing IME is stack-preview is a huge memory hog | 14:17 |
| *** zeih has joined #heat | 14:17 | |
| shardy | so we might have to look at ways to improve that if we went that path | 14:18 |
| jdob | any idea why? even with the THT templates, that's not a ton of data to turn into an object model | 14:18 |
| shardy | jdob: not exactly sure, but with THT I OOM'd my undercloud heat trying stack-preview | 14:18 |
| jdob | yikes | 14:18 |
| jdob | where were you running the undercloud, on your phone? :) | 14:18 |
| shardy | haha | 14:18 |
| jdob | actually, no, screw that, a phone should stil be able to handle that much text | 14:19 |
| shardy | it works on my laptop but not in an instack VM w/4G ram | 14:19 |
| jdob | ok, lemme get back to jamming the files stuff in | 14:19 |
| jdob | this helped clear some stuff up, thanks! | 14:19 |
| shardy | np! | 14:19 |
| *** rpothier has joined #heat | 14:21 | |
| *** zeih has quit IRC | 14:22 | |
| *** jaime49 has quit IRC | 14:24 | |
| *** vijendar has joined #heat | 14:24 | |
| *** zhenguo has quit IRC | 14:24 | |
| *** getvasanth has quit IRC | 14:25 | |
| *** getvasanth has joined #heat | 14:27 | |
| *** zeih has joined #heat | 14:27 | |
| *** sdake has quit IRC | 14:28 | |
| openstackgerrit | Merged openstack/heat: Use the new (and default) glance v2 cli options https://review.openstack.org/219533 | 14:30 |
| *** sdake has joined #heat | 14:31 | |
| *** kencjohn_ has quit IRC | 14:32 | |
| *** hongbin has joined #heat | 14:32 | |
| *** inc0 has quit IRC | 14:32 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (3) https://review.openstack.org/219717 | 14:33 |
| *** hongbin has left #heat | 14:34 | |
| *** zeih has quit IRC | 14:36 | |
| *** pm90__ has quit IRC | 14:36 | |
| *** pm90_ has joined #heat | 14:37 | |
| *** pm90__ has joined #heat | 14:38 | |
| *** kencjohn_ has joined #heat | 14:38 | |
| *** e0ne has joined #heat | 14:39 | |
| *** kencjoh__ has joined #heat | 14:41 | |
| *** alexpilotti has joined #heat | 14:42 | |
| *** pm90_ has quit IRC | 14:42 | |
| *** kencjohn_ has quit IRC | 14:43 | |
| *** jaime49 has joined #heat | 14:45 | |
| *** jaime49 has quit IRC | 14:45 | |
| *** spzala has joined #heat | 14:45 | |
| *** jaime49 has joined #heat | 14:45 | |
| *** hdd has joined #heat | 14:48 | |
| skraynev | Guys! Good news! fix was merged, so gate is unblocked | 14:50 |
| skraynev | zaneb: I reverify your patches | 14:50 |
| *** hdd has quit IRC | 14:50 | |
| zaneb | skraynev: \o/ thank you | 14:50 |
| *** hdd has joined #heat | 14:50 | |
| skraynev | Hope, that we will merge patches which were approved | 14:51 |
| skraynev | zaneb: np. Hope, that now it lands as expected ;) | 14:51 |
| *** sabeen has joined #heat | 14:57 | |
| *** sabeen2 has joined #heat | 14:59 | |
| *** sabeen has quit IRC | 15:01 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (4) https://review.openstack.org/219729 | 15:01 |
| openstackgerrit | Limor Stotland proposed openstack/heat: fix : removing policies from mistral types https://review.openstack.org/218633 | 15:01 |
| jdob | shardy: fyi, for some reason, template-validate translates the registry urls to file:// but stack-create doesn't, so i'm looking into the client now. while i'm there, i'll add the show_nested, so dont worry about working on that | 15:03 |
| shardy | jdob: Hmm, ok that's weird, they should work the same IIRC | 15:04 |
| jdob | i tried from a different box. smae templates with both template-validate and stack-create. the stack-create succeeds, but the template-validate bombs out saying it can't get to file://<my local stuff> | 15:05 |
| jdob | surprised me too | 15:05 |
| shardy | jdob: it's possible we have a bug in the client, given that the files from template-validate have never been used | 15:06 |
| jdob | gotcha, i'll see what I find | 15:08 |
| * shardy can't see any differences in the client | 15:08 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (5) https://review.openstack.org/219734 | 15:12 |
| *** dims_ has quit IRC | 15:12 | |
| *** jistr is now known as jistr|call | 15:12 | |
| *** Qiming_ has quit IRC | 15:13 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Move Resource exceptions to common module (5) https://review.openstack.org/219734 | 15:13 |
| *** Drago has joined #heat | 15:17 | |
| *** Drago has quit IRC | 15:17 | |
| *** Drago has joined #heat | 15:17 | |
| *** blomquisg has quit IRC | 15:21 | |
| *** ricolin_ has quit IRC | 15:25 | |
| *** yassine_ has quit IRC | 15:31 | |
| *** blomquisg has joined #heat | 15:35 | |
| *** jistr|call is now known as jistr | 15:38 | |
| *** dims has joined #heat | 15:51 | |
| *** inc0 has joined #heat | 16:02 | |
| *** dims has quit IRC | 16:02 | |
| *** dims has joined #heat | 16:03 | |
| *** LimorStotland has quit IRC | 16:03 | |
| *** dims_ has joined #heat | 16:04 | |
| *** kebray has joined #heat | 16:05 | |
| openstackgerrit | Merged openstack/heat: Removes unused posix-ipc requirement https://review.openstack.org/218947 | 16:05 |
| openstackgerrit | Merged openstack/heat: enabling the stevedore.sphinxext extension. https://review.openstack.org/217401 | 16:05 |
| *** kebray has quit IRC | 16:05 | |
| openstackgerrit | Merged openstack/heat: Removing unused dependency: discover https://review.openstack.org/218287 | 16:05 |
| openstackgerrit | Merged openstack/heat: Refactor _get_network_matches method https://review.openstack.org/209437 | 16:05 |
| *** kebray has joined #heat | 16:06 | |
| openstackgerrit | Merged openstack/heat: Use _get_network_id in other methods of mixin class https://review.openstack.org/217284 | 16:06 |
| openstackgerrit | Merged openstack/heat: Add subnet property to Nova::Server nic https://review.openstack.org/217752 | 16:06 |
| *** dims has quit IRC | 16:07 | |
| *** spzala has quit IRC | 16:07 | |
| *** devlaps has joined #heat | 16:16 | |
| *** alexpilotti has quit IRC | 16:17 | |
| shardy | zaneb: Hey I put some questions about reusing prev_raw_template_id on https://review.openstack.org/#/c/205754/ | 16:18 |
| *** Nepoc has quit IRC | 16:20 | |
| shardy | It's a little confusing as AFAICT if we do it to be consistent with convergence we won't use the last template the user passed in, but the last one which didn't cause a failed update | 16:20 |
| *** Nepoc has joined #heat | 16:20 | |
| shardy | which makes sense I guess, unless it was an error in the environment which caused the failure | 16:20 |
| *** neelashah has joined #heat | 16:21 | |
| *** jtomasek has quit IRC | 16:23 | |
| *** pm90__ has quit IRC | 16:24 | |
| *** spzala has joined #heat | 16:25 | |
| *** ricolin has joined #heat | 16:26 | |
| *** Piet has joined #heat | 16:28 | |
| *** getvasanth has quit IRC | 16:30 | |
| *** sergmelikyan has joined #heat | 16:30 | |
| *** sergmelikyan has quit IRC | 16:30 | |
| *** kebray has quit IRC | 16:32 | |
| *** sergmelikyan has joined #heat | 16:32 | |
| *** sergmelikyan has quit IRC | 16:34 | |
| zaneb | shardy: I have a meeting right now but I will take a look after lunch. it doesn't sound that inconsistent though | 16:34 |
| shardy | zaneb: np, just wanted a sanity check before I attempt another revision of the patch | 16:35 |
| shardy | zaneb: thanks | 16:35 |
| *** vijayagurug has quit IRC | 16:35 | |
| *** dyasny has quit IRC | 16:36 | |
| *** kebray has joined #heat | 16:40 | |
| *** lkarm has quit IRC | 16:41 | |
| *** lkarm has joined #heat | 16:42 | |
| *** sergmelikyan has joined #heat | 16:42 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/heat: Run py34 first in default tox run https://review.openstack.org/219781 | 16:43 |
| *** jistr has quit IRC | 16:44 | |
| *** lkarm has quit IRC | 16:44 | |
| *** lkarm has joined #heat | 16:44 | |
| *** sergmelikyan has quit IRC | 16:45 | |
| *** sergmelikyan has joined #heat | 16:46 | |
| *** jprovazn has quit IRC | 16:47 | |
| *** jtomasek has joined #heat | 16:50 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-heatclient: Add py34 env to default tox run https://review.openstack.org/219787 | 16:57 |
| *** derekh has quit IRC | 16:57 | |
| *** lkarm has quit IRC | 17:00 | |
| *** pm90_ has joined #heat | 17:02 | |
| zaneb | shardy: commented | 17:04 |
| *** kebray has quit IRC | 17:07 | |
| shardy | zaneb: thanks | 17:12 |
| *** e0ne has quit IRC | 17:12 | |
| zaneb | shardy: does that make sense? | 17:12 |
| zaneb | btw do we have any docs at all about what the format of an environment file should look like? | 17:13 |
| *** jtomasek has quit IRC | 17:15 | |
| shardy | zaneb: Yes I think so, thanks for the pseudo-code, that helps clarify | 17:15 |
| shardy | zaneb: http://docs.openstack.org/developer/heat/template_guide/environment.html ? | 17:15 |
| zaneb | shardy: thanks. is there ever a time where we put a whole parameter schema definition into the env? users seem to think there is, but it's not documented there | 17:17 |
| shardy | zaneb: No, you can only specify the parameter values AFAIK | 17:18 |
| zaneb | ok, users are crazy then :) | 17:18 |
| * shardy is shocked ;) | 17:18 | |
| *** kencjoh__ has quit IRC | 17:19 | |
| *** vijayagurug has joined #heat | 17:19 | |
| shardy | zaneb: that said, we're currently doing a pretty poor job of validating the environment syntax and returning helpful errors | 17:20 |
| shardy | ref bug #1490913 | 17:20 |
| openstack | bug 1490913 in python-heatclient "Malformed environment syntax not clearly reported" [Undecided,New] https://launchpad.net/bugs/1490913 | 17:20 |
| shardy | Like, we should at least say which environment file, so you stand some chance of figuring it out when you pass a dozen environments into a TripleO stack create :) | 17:22 |
| zaneb | heh :) | 17:23 |
| *** achanda has joined #heat | 17:24 | |
| *** inc0 has quit IRC | 17:25 | |
| *** Drago has quit IRC | 17:27 | |
| *** steveg_afk has quit IRC | 17:31 | |
| *** dims_ has quit IRC | 17:33 | |
| pas-ha | exit | 17:33 |
| * pas-ha fails | 17:33 | |
| *** pas-ha has quit IRC | 17:33 | |
| *** dims has joined #heat | 17:35 | |
| *** dyasny has joined #heat | 17:37 | |
| *** lkarm has joined #heat | 17:37 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Merge Server os-collect-config metadata on update https://review.openstack.org/219449 | 17:38 |
| openstackgerrit | Steven Hardy proposed openstack/heat: Add server validation check for deployments metadata https://review.openstack.org/219799 | 17:38 |
| larsks | Has anyone seen a situation in which "heat stack-list" shows a stack (in DELETE_FAILED state), but "heat show <name>" and "heat show <uuid>" both return "stack not found"? | 17:41 |
| *** ricolin has quit IRC | 17:44 | |
| shardy | larsks: does heat stack-show "<name>/<ID>" work? | 17:44 |
| larsks | shardy: Actually, s/show/resource-list/, and yes, stack-show works. | 17:45 |
| larsks | Sorry for the typo confusing things. | 17:45 |
| shardy | larsks: that's expected I think, depending on when the delete failed | 17:45 |
| shardy | we delete all the resource records on delete, but only soft-delete the stack record, and retain the events | 17:46 |
| larsks | Okay. I was trying to figure out which resources had failed. | 17:46 |
| shardy | IIRC | 17:46 |
| larsks | Oddly, the delete failed because "No valid host was found. There are not enough hosts available", which is why the stack failed to deploy in the first place. | 17:46 |
| *** jaime49 has quit IRC | 17:47 | |
| shardy | larsks: hmm, sounds like a bad error path with nova 500ing us? | 17:47 |
| larsks | It looks like a second stack-delete has succeeded. Yeah, something is up with nova. I'm experimenting with the 'rdo director' installer, and things are wonky. | 17:48 |
| shardy | larsks: it'd be interesting to see the backtrace near the DELETE_FAILED in the heat-engine log | 17:48 |
| shardy | larsks: ah, that's a known bug I think | 17:48 |
| larsks | Doesn't look like it produced a backtrace. | 17:50 |
| shardy | larsks: IIRC the issue is/was that on delete if a server is in ERROR state, we'd race after deleting it and mis-detect the ERROR as a delete error, vs something which already happened | 17:52 |
| shardy | I think it's fixed now, I'm just failing to find the bug reference | 17:53 |
| larsks | Interesting. I'll watch for updates... | 17:54 |
| *** lkarm has quit IRC | 17:55 | |
| *** blomquisg has quit IRC | 17:55 | |
| shardy | larsks: https://bugs.launchpad.net/heat/+bug/1463716 | 17:56 |
| openstack | Launchpad bug 1463716 in heat "Can not delete stack with nova server resource" [High,Fix released] - Assigned to huangtianhua (huangtianhua) | 17:56 |
| larsks | shardy: Thanks! | 17:58 |
| *** jaime49 has joined #heat | 17:58 | |
| *** jaime491 has joined #heat | 18:00 | |
| *** Drago has joined #heat | 18:01 | |
| *** jaime49 has quit IRC | 18:02 | |
| *** Drago has quit IRC | 18:04 | |
| *** Drago has joined #heat | 18:04 | |
| *** lkarm has joined #heat | 18:06 | |
| *** blomquisg has joined #heat | 18:11 | |
| *** sdake_ has joined #heat | 18:13 | |
| *** vijayagurug has left #heat | 18:16 | |
| *** kebray has joined #heat | 18:16 | |
| *** kencjohn_ has joined #heat | 18:16 | |
| *** sdake has quit IRC | 18:17 | |
| *** sdake has joined #heat | 18:21 | |
| *** sdake_ has quit IRC | 18:25 | |
| *** steveg_afk has joined #heat | 18:32 | |
| *** lkarm has quit IRC | 18:34 | |
| *** lkarm has joined #heat | 18:35 | |
| jaime491 | shardy,sabeen2: ping - Steven are you around? | 18:36 |
| *** sdake_ has joined #heat | 18:41 | |
| *** e0ne has joined #heat | 18:42 | |
| *** sdake has quit IRC | 18:45 | |
| *** jecarey has joined #heat | 18:46 | |
| *** dkusidlo has joined #heat | 18:47 | |
| *** harlowja has quit IRC | 18:54 | |
| *** harlowja has joined #heat | 18:58 | |
| *** dkusidlo has quit IRC | 18:58 | |
| *** Nepoc has quit IRC | 18:58 | |
| Drago | jaime491: are you looking for shardy? | 19:03 |
| jaime491 | Drago: yep | 19:04 |
| Drago | jaime491: k, that's his nic | 19:04 |
| Drago | *handle | 19:05 |
| jaime491 | Drago: wanted some feedback on where we might put our api tests | 19:05 |
| *** coolsvap_ has joined #heat | 19:09 | |
| *** e0ne has quit IRC | 19:12 | |
| *** kebray has quit IRC | 19:18 | |
| *** Nepoc has joined #heat | 19:21 | |
| *** dyasny has quit IRC | 19:22 | |
| *** kebray has joined #heat | 19:28 | |
| *** tlashchova has quit IRC | 19:31 | |
| *** metral is now known as metral_zzz | 19:31 | |
| *** metral_zzz is now known as metral | 19:33 | |
| shardy | jaime491: hi | 19:33 |
| shardy | sorry been afk for a bit | 19:34 |
| *** blomquisg has quit IRC | 19:38 | |
| *** coolsvap_ has quit IRC | 19:45 | |
| *** blomquisg has joined #heat | 19:51 | |
| *** hdd has quit IRC | 19:53 | |
| *** hdd has joined #heat | 19:54 | |
| *** shardy has quit IRC | 20:00 | |
| *** blomquisg has quit IRC | 20:06 | |
| *** crose has quit IRC | 20:07 | |
| *** jtomasek has joined #heat | 20:13 | |
| *** sdake_ is now known as sdake | 20:15 | |
| *** Marga_ has joined #heat | 20:20 | |
| *** blomquisg has joined #heat | 20:21 | |
| *** lkarm has quit IRC | 20:29 | |
| *** pm90__ has joined #heat | 20:35 | |
| *** e0ne has joined #heat | 20:35 | |
| *** pm90_ has quit IRC | 20:38 | |
| *** wirehead_1 is now known as wirehead_ | 20:39 | |
| *** pm90__ has quit IRC | 20:39 | |
| *** vahidh has joined #heat | 20:40 | |
| *** lkarm has joined #heat | 20:40 | |
| *** Piet has quit IRC | 20:45 | |
| *** harlowja has quit IRC | 20:45 | |
| *** signed8bit has joined #heat | 20:47 | |
| *** jaime491 has quit IRC | 20:50 | |
| openstackgerrit | Jay Dobies proposed openstack/heat: Resolve parameter list including nested stacks https://review.openstack.org/219016 | 21:01 |
| *** pm90_ has joined #heat | 21:02 | |
| *** pm90__ has joined #heat | 21:03 | |
| *** pm90_ has quit IRC | 21:07 | |
| *** jaime49 has joined #heat | 21:08 | |
| *** jaime49 has quit IRC | 21:08 | |
| *** achanda has quit IRC | 21:08 | |
| *** jaime49 has joined #heat | 21:09 | |
| *** Marga_ has quit IRC | 21:15 | |
| *** jtomasek has quit IRC | 21:16 | |
| *** Marga_ has joined #heat | 21:16 | |
| *** jdob has quit IRC | 21:19 | |
| *** dims has quit IRC | 21:19 | |
| *** dims has joined #heat | 21:19 | |
| *** harlowja has joined #heat | 21:21 | |
| *** dims has quit IRC | 21:24 | |
| *** lkarm has quit IRC | 21:25 | |
| *** rpothier has quit IRC | 21:26 | |
| *** e0ne has quit IRC | 21:32 | |
| *** e0ne has joined #heat | 21:33 | |
| *** derekh has joined #heat | 21:33 | |
| *** achanda has joined #heat | 21:37 | |
| *** e0ne has quit IRC | 21:41 | |
| *** kebray has quit IRC | 21:42 | |
| *** e0ne has joined #heat | 21:43 | |
| *** achanda has quit IRC | 21:47 | |
| *** kencjohn_ has quit IRC | 21:47 | |
| *** e0ne has quit IRC | 21:52 | |
| *** achanda has joined #heat | 21:54 | |
| *** kebray has joined #heat | 22:04 | |
| *** blomquisg has quit IRC | 22:05 | |
| *** signed8bit has quit IRC | 22:08 | |
| *** jasond has quit IRC | 22:09 | |
| *** bleepbloop has joined #heat | 22:12 | |
| openstackgerrit | Merged openstack/heat: Correct http method to 'POST' for stack-preview test https://review.openstack.org/216134 | 22:13 |
| openstackgerrit | Merged openstack/heat: Support project access manager for volume type https://review.openstack.org/194478 | 22:13 |
| *** jcoufal has quit IRC | 22:14 | |
| *** Beetnik has quit IRC | 22:15 | |
| *** hdd has quit IRC | 22:19 | |
| *** sabeen2 has quit IRC | 22:23 | |
| *** achanda has quit IRC | 22:24 | |
| *** achanda_ has joined #heat | 22:24 | |
| *** jecarey has quit IRC | 22:25 | |
| *** jecarey has joined #heat | 22:25 | |
| *** vijendar has quit IRC | 22:26 | |
| *** Marga_ has quit IRC | 22:27 | |
| *** steveg_afk has quit IRC | 22:30 | |
| openstackgerrit | Merged openstack/heat: Support is_proxy_gateway in sahara node group templates https://review.openstack.org/219319 | 22:43 |
| openstackgerrit | Merged openstack/heat: install.sh: stop using deprecated option group for rabbit https://review.openstack.org/219356 | 22:45 |
| *** pm90__ has quit IRC | 22:47 | |
| *** derekh has quit IRC | 22:50 | |
| openstackgerrit | Merged openstack/heat: Removes dead code from context.request_context https://review.openstack.org/191040 | 22:59 |
| *** kebray has quit IRC | 23:03 | |
| openstackgerrit | Merged openstack/heat: Validate support_status while searching resource registry https://review.openstack.org/213107 | 23:03 |
| openstackgerrit | Merged openstack/heat: Search based on the resource type name https://review.openstack.org/213108 | 23:04 |
| *** achanda_ has quit IRC | 23:12 | |
| *** sdake has quit IRC | 23:15 | |
| *** Piet has joined #heat | 23:17 | |
| *** achanda has joined #heat | 23:21 | |
| *** blomquisg has joined #heat | 23:23 | |
| *** Qiming_ has joined #heat | 23:26 | |
| *** sdake has joined #heat | 23:27 | |
| *** elynn has joined #heat | 23:35 | |
| *** elynn has quit IRC | 23:35 | |
| *** harlowja has quit IRC | 23:42 | |
| *** tochi has joined #heat | 23:42 | |
| *** harlowja has joined #heat | 23:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/219492 | 23:45 |
| *** achanda has quit IRC | 23:52 | |
| *** sergmelikyan has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!