| *** piousbox has quit IRC | 00:00 | |
| *** lindsayk1 has joined #heat | 00:00 | |
| *** lindsayk has quit IRC | 00:01 | |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Return None when get_attr cannot resolve a value https://review.openstack.org/77713 | 00:02 |
|---|---|---|
| randallburt | jasond`: https://review.openstack.org/#/c/77713 if you're still about | 00:03 |
| stevebaker | SpamapS: afaik, to be ready for instance-users tripleo will need to do the equivalent of devstack/lib/heat create_heat_accounts() | 00:04 |
| stevebaker | SpamapS: nothing to do with trusts | 00:04 |
| *** zns has quit IRC | 00:04 | |
| *** arbylee has joined #heat | 00:10 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-heatclient: Python 3: Fix YamlEnvironmentTest tests https://review.openstack.org/77737 | 00:11 |
| stevebaker | I've disabled https://answers.launchpad.net/heat | 00:13 |
| randallburt | but how will I get answers about heat now!!?? | 00:14 |
| *** alexpilotti has quit IRC | 00:17 | |
| *** piousbox has joined #heat | 00:18 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-heatclient: Pass bytes to NamedTemporaryFile.write() https://review.openstack.org/77738 | 00:19 |
| *** rbuilta has joined #heat | 00:20 | |
| *** rwsu has quit IRC | 00:27 | |
| zaneb | stevebaker: is there any way to set it up to point to ask.openstack.org? We still seemed to be getting a lot of questions through there | 00:27 |
| stevebaker | zaneb: no, I've set it as External but launchpad didn't ask for a URL | 00:29 |
| stevebaker | zaneb: all answers entries have been imported to ask.openstack.org | 00:30 |
| stevebaker | https://ask.openstack.org/en/questions/scope:all/sort:activity-desc/page:1/query:heat%20migrated/ | 00:30 |
| *** rwsu has joined #heat | 00:30 | |
| *** kfox1111 has joined #heat | 00:33 | |
| *** jdob has quit IRC | 00:34 | |
| *** jdob has joined #heat | 00:36 | |
| *** radez_g0n3 is now known as radez | 00:38 | |
| *** jdob has quit IRC | 00:40 | |
| saurabhs | I need some help with understanding stack-update. I have created a stack with a template which has some parameters. now with stack-update command if I pass it a different value for that parameter, I don't see it getting updated in the DB in stack table for parameters column. is it suppose to get updated? | 00:40 |
| *** achampion has joined #heat | 00:42 | |
| *** openstackgerrit has quit IRC | 00:44 | |
| SpamapS | stevebaker: could you consider patching tripleo the same way? | 00:44 |
| *** openstackgerrit has joined #heat | 00:44 | |
| stevebaker | SpamapS: maybe shardy would have time | 00:45 |
| *** jprovazn has quit IRC | 00:45 | |
| *** rbuilta has quit IRC | 00:46 | |
| *** pvaneck has quit IRC | 00:46 | |
| *** nati_ueno has joined #heat | 00:46 | |
| *** nati_uen_ has quit IRC | 00:49 | |
| SpamapS | stevebaker: at the very least, we'd need a bug so we can prepare and hopefully land the fix in advance of the breakdage | 00:51 |
| *** nati_ueno has quit IRC | 00:51 | |
| stevebaker | SpamapS: raising one now | 00:52 |
| *** giulivo has quit IRC | 00:53 | |
| *** lindsayk1 has quit IRC | 00:53 | |
| *** dims has joined #heat | 00:54 | |
| *** nati_ueno has joined #heat | 00:57 | |
| stevebaker | SpamapS, lifeless https://bugs.launchpad.net/tripleo/+bug/1287453 | 00:58 |
| *** cody-somerville has quit IRC | 01:02 | |
| *** nati_ueno has quit IRC | 01:09 | |
| *** nati_ueno has joined #heat | 01:10 | |
| *** nati_ueno has quit IRC | 01:10 | |
| *** nati_ueno has joined #heat | 01:11 | |
| *** nati_ueno has quit IRC | 01:12 | |
| *** nati_ueno has joined #heat | 01:12 | |
| *** randallburt has quit IRC | 01:13 | |
| *** nati_ueno has quit IRC | 01:14 | |
| *** nati_ueno has joined #heat | 01:15 | |
| *** nati_ueno has quit IRC | 01:16 | |
| *** nati_ueno has joined #heat | 01:16 | |
| *** nati_ueno has quit IRC | 01:17 | |
| *** rwsu has quit IRC | 01:17 | |
| *** nati_ueno has joined #heat | 01:17 | |
| *** alexheneveld has joined #heat | 01:18 | |
| *** nati_ueno has quit IRC | 01:18 | |
| *** nati_ueno has joined #heat | 01:19 | |
| *** nati_ueno has quit IRC | 01:22 | |
| *** nati_ueno has joined #heat | 01:22 | |
| *** nati_ueno has quit IRC | 01:23 | |
| *** nati_ueno has joined #heat | 01:23 | |
| *** tango|2 has quit IRC | 01:24 | |
| *** nati_ueno has quit IRC | 01:24 | |
| *** nati_ueno has joined #heat | 01:25 | |
| *** nati_ueno has quit IRC | 01:26 | |
| *** nati_ueno has joined #heat | 01:26 | |
| *** nati_uen_ has joined #heat | 01:27 | |
| *** nati_ueno has quit IRC | 01:27 | |
| *** randallburt has joined #heat | 01:27 | |
| *** nati_uen_ has quit IRC | 01:28 | |
| *** nosnos has joined #heat | 01:29 | |
| *** nati_ueno has joined #heat | 01:29 | |
| *** nati_ueno has quit IRC | 01:30 | |
| *** derekh has quit IRC | 01:30 | |
| *** nati_ueno has joined #heat | 01:30 | |
| *** nati_ueno has quit IRC | 01:31 | |
| *** nati_ueno has joined #heat | 01:31 | |
| *** nati_ueno has quit IRC | 01:32 | |
| *** jprovazn has joined #heat | 01:33 | |
| *** rbuilta has joined #heat | 01:35 | |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Handle API limit exception in nova_utils.refresh_server https://review.openstack.org/71660 | 01:35 |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Make server resources use nova_utils.refresh_server() https://review.openstack.org/72009 | 01:35 |
| *** fandi has joined #heat | 01:38 | |
| *** ramishra has quit IRC | 01:51 | |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Move HOT version checking to HOTemplate class https://review.openstack.org/77750 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Calculate template version only once https://review.openstack.org/77751 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Move HOT parameters code to a separate module https://review.openstack.org/77752 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Move HOT template code to its own module https://review.openstack.org/77753 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Add an InvalidTemplateVersion exception https://review.openstack.org/77754 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Make template formats pluggable https://review.openstack.org/77755 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Docs: use the plugin manager to list resources https://review.openstack.org/76324 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Load functions from plugins https://review.openstack.org/74592 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Move the code that reads the global environment https://review.openstack.org/76323 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Give cfn a separate Template class https://review.openstack.org/74589 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Add a plugin_manager module https://review.openstack.org/74590 | 02:11 |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Use PluginManager to load resources https://review.openstack.org/74591 | 02:11 |
| *** erkules_ has joined #heat | 02:14 | |
| *** erkules has quit IRC | 02:16 | |
| *** alexheneveld has quit IRC | 02:22 | |
| *** ramishra has joined #heat | 02:26 | |
| *** gokrokve has quit IRC | 02:30 | |
| *** ramishra has quit IRC | 02:31 | |
| *** ramishra has joined #heat | 02:31 | |
| *** ramishra has quit IRC | 02:35 | |
| *** aweiteka has joined #heat | 02:42 | |
| saurabhs | Spamaps: can you please help me with the stack-update doubt | 02:43 |
| saurabhs | I have created a stack with a template which has some parameters. now with stack-update command if I pass it a different value for that parameter, I don't see it getting updated in the DB in stack table for parameters column. is it suppose to get updated? | 02:43 |
| saurabhs | How do I perform a metadata update with heat stack-update? | 02:43 |
| *** piousbox has quit IRC | 02:48 | |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/heat: Implement OS::Neutron::ExtraRoute as /contrib https://review.openstack.org/74899 | 02:51 |
| *** kfox1111 has quit IRC | 02:55 | |
| *** radez is now known as radez_g0n3 | 02:56 | |
| *** nati_ueno has joined #heat | 02:59 | |
| *** nati_ueno has quit IRC | 02:59 | |
| *** jprovazn has quit IRC | 03:02 | |
| *** ramishra has joined #heat | 03:11 | |
| *** ramishra_ has joined #heat | 03:17 | |
| openstackgerrit | huangtianhua proposed a change to openstack/heat: Allow Server name property to be updated https://review.openstack.org/74624 | 03:20 |
| *** ramishra has quit IRC | 03:20 | |
| *** spzala has quit IRC | 03:21 | |
| *** pablosan has quit IRC | 03:25 | |
| *** gokrokve has joined #heat | 03:27 | |
| *** pablosan has joined #heat | 03:27 | |
| *** akuznetsov has quit IRC | 03:36 | |
| *** chandan_kumar has joined #heat | 03:38 | |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Support specifying which cartridges are installed on node https://review.openstack.org/77772 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Convert all params from CamelCase to under_score for consistency https://review.openstack.org/77773 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Remove openshift-enterprise-yum-validator https://review.openstack.org/77774 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Add session_persistence to LBaaS pool https://review.openstack.org/77775 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Provide full support for all repository sources https://review.openstack.org/77776 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Delay creating mongo replica set until other broker hosts available https://review.openstack.org/77777 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Tighten up security group port settings for broker and node. https://review.openstack.org/77778 | 03:39 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: README documentation updates https://review.openstack.org/77779 | 03:39 |
| *** annegentle has quit IRC | 03:42 | |
| *** Tross has joined #heat | 03:46 | |
| *** aweiteka has quit IRC | 03:51 | |
| *** ramishra_ has quit IRC | 03:55 | |
| *** andersonvom has quit IRC | 03:55 | |
| *** andersonvom has joined #heat | 04:03 | |
| *** asalkeld has quit IRC | 04:20 | |
| *** ramishra has joined #heat | 04:21 | |
| *** daneyon has joined #heat | 04:26 | |
| *** akuznetsov has joined #heat | 04:26 | |
| *** wchrisj has quit IRC | 04:27 | |
| *** john-n-seattle has left #heat | 04:29 | |
| *** harlowja is now known as harlowja_away | 04:30 | |
| openstackgerrit | Zhang Yang proposed a change to openstack/python-heatclient: Modify timeout in client shell https://review.openstack.org/77784 | 04:33 |
| *** arbylee1 has joined #heat | 04:34 | |
| *** arbylee has quit IRC | 04:34 | |
| *** harlowja_away is now known as harlowja | 04:35 | |
| *** wchrisj has joined #heat | 04:36 | |
| *** asalkeld has joined #heat | 04:38 | |
| *** Linz has joined #heat | 04:40 | |
| *** Linz has quit IRC | 04:44 | |
| *** Linz has joined #heat | 04:44 | |
| *** akuznetsov has quit IRC | 04:45 | |
| *** nati_ueno has joined #heat | 04:45 | |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Migrate to oslo.messaging from oslo-incubator rpc https://review.openstack.org/72798 | 04:48 |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Update config generator for oslo.messaging https://review.openstack.org/77472 | 04:48 |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Always report the action in state_reason as engine encodes it https://review.openstack.org/77406 | 04:48 |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Convert empty filter dictionary to None https://review.openstack.org/77382 | 04:48 |
| *** akuznetsov has joined #heat | 04:51 | |
| *** chandan_kumar has quit IRC | 04:53 | |
| *** arbylee has joined #heat | 04:54 | |
| *** arbylee1 has quit IRC | 04:54 | |
| *** akuznetsov has quit IRC | 05:01 | |
| openstackgerrit | Liang Chen proposed a change to openstack/heat: Provide attribute schema for waitcondition resource https://review.openstack.org/76552 | 05:02 |
| *** lazy_prince has quit IRC | 05:12 | |
| *** randallburt has quit IRC | 05:13 | |
| *** pablosan has quit IRC | 05:14 | |
| *** akuznetsov has joined #heat | 05:15 | |
| *** lazy_prince has joined #heat | 05:16 | |
| *** wchrisj has quit IRC | 05:17 | |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/heat: Implement OS::Neutron::ExtraRoute as /contrib https://review.openstack.org/74899 | 05:22 |
| *** rmiki has joined #heat | 05:24 | |
| *** blomquisg has quit IRC | 05:27 | |
| *** akuznetsov has quit IRC | 05:29 | |
| *** shakayumi has joined #heat | 05:30 | |
| *** nkhare has joined #heat | 05:32 | |
| *** nkhare has quit IRC | 05:33 | |
| *** saurabhs has quit IRC | 05:36 | |
| *** jprovazn has joined #heat | 05:36 | |
| *** erkules_ is now known as erkules | 05:36 | |
| *** Linz has quit IRC | 05:38 | |
| *** gokrokve has quit IRC | 05:38 | |
| skraynev | Morning | 05:39 |
| *** gokrokve has joined #heat | 05:39 | |
| *** blomquisg has joined #heat | 05:41 | |
| *** gokrokve has quit IRC | 05:43 | |
| ramishra | morning skraynev | 05:43 |
| *** zhiyan_ is now known as zhiyan | 05:44 | |
| skraynev | hi, ramishra | 05:44 |
| *** liang has joined #heat | 05:47 | |
| *** nkhare has joined #heat | 05:49 | |
| *** bvandenh has joined #heat | 05:49 | |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Migrate to oslo.messaging from oslo-incubator rpc https://review.openstack.org/72798 | 05:54 |
| *** jprovazn has quit IRC | 05:59 | |
| *** topol has quit IRC | 06:01 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/72566 | 06:08 |
| *** bvandenh has quit IRC | 06:11 | |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Migrate to oslo.messaging from oslo-incubator rpc https://review.openstack.org/72798 | 06:15 |
| *** harlowja is now known as harlowja_away | 06:16 | |
| *** gokrokve has joined #heat | 06:27 | |
| *** gokrokve_ has joined #heat | 06:29 | |
| *** cfriesen_ has quit IRC | 06:30 | |
| *** gokrokve has quit IRC | 06:31 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Server property for software config transport https://review.openstack.org/77798 | 06:35 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Remove signal_id from deployments API and model https://review.openstack.org/77799 | 06:35 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Resource type implementations for structured software config https://review.openstack.org/74206 | 06:35 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: OS::Nova::Server support for software config https://review.openstack.org/67625 | 06:35 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Resource type implementation for software deployment https://review.openstack.org/67624 | 06:35 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Nova server to ref cloud-config resources in user_data https://review.openstack.org/69238 | 06:35 |
| stevebaker | Hey all, reviews on https://review.openstack.org/#/c/69238/ would close off blueprint cloud-init-resource | 06:38 |
| openstackgerrit | Ryo Miki proposed a change to openstack/heat: Add handle_update for OS::Neutron::RouterL3Agent https://review.openstack.org/69303 | 06:40 |
| *** amritanshu_RnD has joined #heat | 06:45 | |
| *** amritanshu_RnD is now known as Guest77723 | 06:45 | |
| *** arbylee has quit IRC | 06:45 | |
| *** yogesh has joined #heat | 06:47 | |
| *** tspatzier has joined #heat | 06:55 | |
| *** yogesh has quit IRC | 06:55 | |
| therve | 'morning | 07:05 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Allow getting attributes on suspended resources. https://review.openstack.org/77452 | 07:07 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Allow handle_signal to assert which actions are valid https://review.openstack.org/77453 | 07:07 |
| *** saju_m has joined #heat | 07:12 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Raise and catch a specific error during validation https://review.openstack.org/71287 | 07:18 |
| *** lsmola has joined #heat | 07:23 | |
| *** daneyon has quit IRC | 07:28 | |
| *** bvandenh has joined #heat | 07:30 | |
| *** nosnos has quit IRC | 07:31 | |
| *** nosnos_ has joined #heat | 07:31 | |
| *** nkhare has quit IRC | 07:34 | |
| *** nkhare has joined #heat | 07:35 | |
| *** bvandenh has quit IRC | 07:36 | |
| sdake_ | victory oslo.messaging passes the check after 48 patches | 07:37 |
| *** ramishra has quit IRC | 07:38 | |
| skraynev | sdake_: Thumbs up! | 07:39 |
| sdake_ | now to acutally beat the patch into submission | 07:42 |
| sdake_ | I have atleast 8 test cases commented out that i don't know how to mock | 07:42 |
| sdake_ | and I need to refunctional test multiengine | 07:42 |
| *** tspatzier has quit IRC | 07:44 | |
| sdake_ | plus i'm sure there will be lots of complaning aobut service.py :) | 07:44 |
| *** ramishra has joined #heat | 07:45 | |
| *** tspatzier has joined #heat | 07:45 | |
| *** arbylee has joined #heat | 07:46 | |
| *** yogesh has joined #heat | 07:52 | |
| *** arbylee has quit IRC | 07:55 | |
| openstackgerrit | A change was merged to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 08:03 |
| *** nosnos_ has quit IRC | 08:04 | |
| *** nosnos has joined #heat | 08:04 | |
| *** TonyBurn_ has joined #heat | 08:11 | |
| *** tomek_adamczewsk has joined #heat | 08:11 | |
| *** tomek_adamczewsk has quit IRC | 08:22 | |
| *** tomek_adamczewsk has joined #heat | 08:23 | |
| *** Guest77723 has quit IRC | 08:25 | |
| *** saju_m has quit IRC | 08:26 | |
| *** yogesh_ has joined #heat | 08:30 | |
| *** yogesh has quit IRC | 08:32 | |
| *** cmyster has joined #heat | 08:35 | |
| shardy | morning! | 08:37 |
| *** alexpilotti has joined #heat | 08:37 | |
| *** tomek_adamczewsk has quit IRC | 08:38 | |
| *** alexheneveld has joined #heat | 08:39 | |
| cmyster | morning | 08:39 |
| cmyster | which reminds me, shardy I was looking for you yesterday, have a few minutes for me ? | 08:40 |
| cmyster | but first, coffee | 08:40 |
| *** alexheneveld has quit IRC | 08:46 | |
| therve | shardy, I didn't expose the URL as an attribute, FWIW | 08:50 |
| shardy | therve: Hmm, so ceilometer can't signal the resource at all atm then? | 08:50 |
| therve | shardy, Right | 08:51 |
| shardy | Oh right for some reason I was thinking the attribute got inherited from the super-class | 08:51 |
| shardy | -ENOCOFFEE | 08:52 |
| shardy | cmyster: sure | 08:52 |
| therve | Hum | 08:52 |
| therve | Well the schema is | 08:52 |
| therve | Actually you may be right :) | 08:53 |
| shardy | therve: I think probably just exposing AlarmUrl is the easiest solution | 08:53 |
| shardy | As in the future we can leave AlarmUrl just as an unsigned URL, and add stuff to support native auth | 08:53 |
| therve | Fair enough | 08:54 |
| therve | I guess I'll just rename it to alarm_url and add some tests | 08:54 |
| *** Guest77723 has joined #heat | 08:54 | |
| shardy | Unless you have any better ideas ;) | 08:54 |
| shardy | Either way we definitely need this to work with ceilometer IMO | 08:54 |
| therve | Well obviously my idea was to support actions with auth in ceilometer | 08:55 |
| *** alexheneveld has joined #heat | 08:55 | |
| cmyster | ++ | 08:55 |
| therve | But that sounds tricky by icehouse | 08:55 |
| *** jistr has joined #heat | 08:55 | |
| cmyster | how come ? | 08:55 |
| cmyster | well I am trying to make it work myself so I know its hard, question is, can I ? :) | 08:56 |
| therve | Well the code is not there? And the freeze was like yesterday? | 08:56 |
| cmyster | right | 08:56 |
| cmyster | freeze in here == master will not change other then major+ bugs ? | 08:56 |
| shardy | therve: Yeah we definitely need to rework the ceilometer side of things, but that's gonna have to wait unfortunately | 08:57 |
| therve | cmyster, In theory yeah | 08:57 |
| shardy | cmyster: Yeah from pretty much now we move to bug-fixes only until the release | 08:57 |
| therve | I don't know how hard it would be to implement the feature | 08:57 |
| shardy | there is an exception process for remaining high priority blueprints but we need them in very soon | 08:57 |
| therve | It'd not even clear how that would work. Passing the password in clear in the request doesn't sound super awesome? Presumably we have no choice | 08:59 |
| shardy | therve: Not that hard but not trivial IMO - we either have to make the ScalingPolicy user created with a password, and pass the username/password to ceilometer, or (preferable) create a trust between the user and the ceilometer service user, and pass that to ceilometer when setting up the alarm, so ceilometer can impersonate the user when triggering the alarm | 08:59 |
| shardy | the cool thing about doing it with trusts is that we might not even need to store the trust anymore, as currently we proxy that impersonation (for deferred_auth_method=trusts) via the user_creds table | 09:00 |
| cmyster | well, in tempest there is a scenario that uses scalling (atm skipped due to a bug) how does that work then ? | 09:00 |
| shardy | cmyster: there is a non-voting heat-slow job which tests autoscaling | 09:01 |
| shardy | cmyster: we pass ceilometer an EC2 style pre-signed URL | 09:01 |
| therve | shardy, Ah, I see. I was only thinking about a user, like instances users. Trusts would be more work indeed. | 09:01 |
| shardy | which is non-ideal for native resources as some folks really really don't want to use the ec2tokens keystone extension | 09:01 |
| cmyster | so we want something more 'native' | 09:02 |
| shardy | and it's suceptible to replay attacks because the secret isn't shared with ceilometer | 09:02 |
| shardy | cmyster: exactly, so folks can, if they wish, deploy a fully functional heat without any of the AWS compatibility stuff enabled | 09:02 |
| cmyster | and thats not in Icehouse... | 09:02 |
| cmyster | OK | 09:03 |
| shardy | cmyster: Yeah the ceilometer side is not there, it can only post to a URL atm | 09:03 |
| cmyster | I have a task to start writing scenarios for tempest, so I am trying to find all the bits I can about it. Obviously I don't want to start working on something that uses that if were gonna have something that is native | 09:04 |
| shardy | therve: yeah, I'm thinking notification from ceilometer is a more trust-appropriate use-case, since it's a known and trusted service, not some random user instance | 09:04 |
| therve | Right | 09:04 |
| cmyster | still easy enough to break if your not at least behind https... | 09:05 |
| shardy | cmyster: care to expand on that? | 09:06 |
| shardy | cmyster: scenario tests should just use the heat resource interfaces and heatclient, so I don't think you need to be concerned about possible changes in the underlying implementation | 09:07 |
| shardy | In fact writing them now would be better, so we can verify the functionality isn't broken when we start hacking on it :) | 09:08 |
| *** sballe has quit IRC | 09:08 | |
| cmyster | shardy: true, but then again I'll have to change the templates I am useing | 09:08 |
| *** sballe has joined #heat | 09:08 | |
| shardy | cmyster: possibly but not necessarily | 09:08 |
| *** yogesh_ has quit IRC | 09:08 | |
| shardy | we always try to avoid template breakage, but sometimes it's unavoidable | 09:09 |
| cmyster | say I am using something simple like that F20 wordpress and I want on top of that add scaling, I am not sure if the scaling part will be the same | 09:09 |
| cmyster | should, but might change | 09:09 |
| * cmyster ratherr build tests for the ages | 09:09 | |
| cmyster | -r | 09:09 |
| shardy | cmyster: native autoscaling is the most bleeding of bleeding edge features, I suggest looking at other resources if you want absolute stability of interfaces ;) | 09:10 |
| *** e0ne has joined #heat | 09:10 | |
| cmyster | I see, | 09:11 |
| cmyster | hints? | 09:11 |
| cmyster | :) | 09:11 |
| shardy | https://bugs.launchpad.net/heat/+bugs?field.tag=tempest | 09:12 |
| cmyster | wonderful, thanks | 09:13 |
| shardy | Autoscaling is pretty much the only scenario which *is* tested in tempest atm, so you can choose anything | 09:13 |
| cmyster | if you are referring to the single scenario there, its skipped | 09:14 |
| cmyster | @s/skipped/being skipped/ | 09:14 |
| shardy | the most important gap to me is lack of neutron resource testing, but I see that's already assigned (no patches yet) | 09:14 |
| shardy | cmyster: it's not skipped in the gate, it's just non-voting | 09:14 |
| shardy | probably some missing config in your environment? | 09:15 |
| *** e0ne has quit IRC | 09:15 | |
| *** e0ne_ has joined #heat | 09:15 | |
| *** e0ne_ has quit IRC | 09:16 | |
| openstackgerrit | Liang Chen proposed a change to openstack/heat: Provide attribute schema for waitcondition resource https://review.openstack.org/76552 | 09:16 |
| *** e0ne has joined #heat | 09:16 | |
| *** sballe has quit IRC | 09:17 | |
| *** e0ne_ has joined #heat | 09:20 | |
| *** e0ne has quit IRC | 09:20 | |
| therve | So yeah I asked ceilometer, it's definitely too late for icehouse | 09:21 |
| therve | Let's make sure it's in Juno :) | 09:21 |
| *** rmiki has quit IRC | 09:24 | |
| *** tomek_adamczewsk has joined #heat | 09:24 | |
| shardy | therve: cool, sounds good | 09:25 |
| shardy | therve: hey I spoke to stevebaker last night and he's happy for us to get instance-users in | 09:26 |
| shardy | therve: There is some concern about fallout with tripleo but I think we've reached the point where we just have to merge it and deal with any bugs | 09:26 |
| shardy | therve: as you have familiarity with the series, any chance of some reviews sometime today? :) | 09:27 |
| shardy | stevebaker has +2'd them all | 09:27 |
| *** e0ne has joined #heat | 09:28 | |
| *** e0ne_ has quit IRC | 09:28 | |
| cmyster | to those not in the know, are these things documented ? | 09:28 |
| cmyster | or are the changes enough | 09:29 |
| shardy | cmyster: what things? | 09:29 |
| *** e0ne has quit IRC | 09:29 | |
| cmyster | instance-users for instance | 09:29 |
| sdake_ | documentd in launchpad | 09:29 |
| cmyster | k | 09:29 |
| shardy | cmyster: yes, to a point, via the blueprint | 09:29 |
| sdake_ | and on that note, i'm off to bed | 09:29 |
| sdake_ | enjoy :) | 09:29 |
| *** e0ne has joined #heat | 09:29 | |
| cmyster | nn sdake_ | 09:29 |
| shardy | sdake_: 2.30am!! o/ | 09:30 |
| shardy | cmyster: https://blueprints.launchpad.net/heat/+spec/instance-users | 09:31 |
| cmyster | thanks, was already in it :) | 09:31 |
| shardy | and there is a wiki page linked "Read the full specification" | 09:31 |
| sdake_ | shardy if you have an opportunity check out https://review.openstack.org/#/q/status:open+project:openstack/heat+branch:master+topic:bp/oslo-messaging,n,z | 09:31 |
| sdake_ | there are 3 very easy reviews | 09:31 |
| shardy | that's more of a discussion of ideas rather than a specification tho ;) | 09:31 |
| sdake_ | and 1 super onerous one | 09:31 |
| sdake_ | the 3 easy reviews will take less them 5 minutes | 09:31 |
| shardy | sdake_: Ok, will do - sorry I said that yesterday didn't I, on it now :) | 09:32 |
| sdake_ | its ok they needed to be rebased anyway | 09:32 |
| sdake_ | but before I have to further rebase, might as well get them in while they are already +2'ed :) | 09:32 |
| *** akuznetsov has joined #heat | 09:33 | |
| sdake_ | if you are up for a several hour long complaining session might have a look at https://review.openstack.org/#/c/72798/ | 09:33 |
| *** e0ne has quit IRC | 09:33 | |
| shardy | sdake_: +2574, -8370 ouch | 09:33 |
| sdake_ | sooner I know what people want fixed sooner I can fix em :) | 09:33 |
| sdake_ | nothing I can do to break the patch up unfortunately | 09:34 |
| sdake_ | the other oslo.messaging patches are similar in scope | 09:34 |
| sdake_ | its not like i'm doing it wrong :( | 09:34 |
| *** e0ne has joined #heat | 09:34 | |
| sdake_ | just a shitty amount of rework that has to go in one patch because the tests gate with the code | 09:34 |
| sdake_ | when I wake up in the morning I'm going to give it a review as well | 09:35 |
| sdake_ | this was my first patch that passed the gate, bu ti had to comment out about 7 patches | 09:35 |
| sdake_ | because I am not one with mock | 09:36 |
| therve | shardy, Definitely, will catch up on reviews | 09:36 |
| shardy | therve: awesome, thanks, feel free to ping me if you have any which are high priortity for i3 :) | 09:36 |
| *** giulivo has joined #heat | 09:38 | |
| *** e0ne_ has joined #heat | 09:38 | |
| shardy | Anyone know of a gerrit review interface which allows you to not highlight whitespace changes? | 09:38 |
| *** dmakogon_ is now known as denis_makogon | 09:40 | |
| *** e0ne has quit IRC | 09:42 | |
| *** saju_m has joined #heat | 09:42 | |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/python-heatclient: Encode non utf8 files in base64 https://review.openstack.org/76443 | 09:42 |
| sdake_ | i tried out tempest tonight | 09:43 |
| sdake_ | only 50 test cases for heat | 09:43 |
| * sdake_ sighs | 09:43 | |
| cmyster | on it | 09:43 |
| cmyster | sdake_: nice you can IRC while sleeping ;) | 09:44 |
| sdake_ | magic powers | 09:44 |
| cmyster | indeed | 09:44 |
| sdake_ | i can also levitate objects | 09:44 |
| *** liang has left #heat | 09:46 | |
| *** liang has joined #heat | 09:46 | |
| sdake_ | anyone remember bitkeeper | 09:46 |
| sdake_ | shardy can you give cmyster the link for the 162 tempest work, I don't have access to the vpn at the moment | 09:49 |
| sdake_ | on the ep | 09:49 |
| * cmyster wonders | 09:49 | |
| *** arbylee has joined #heat | 09:52 | |
| sdake_ | you know what I dont get about star trek | 10:00 |
| sdake_ | they have warp 9 | 10:00 |
| sdake_ | yet they go warp 2, warp 7, etc | 10:00 |
| *** arbylee has quit IRC | 10:00 | |
| sdake_ | why not always just go warp 9 | 10:00 |
| openstackgerrit | Ionut Artarisi proposed a change to openstack/heat: replace dict comprehension with dict constructor https://review.openstack.org/77834 | 10:02 |
| cmyster | sdake_: there was an episode where they explained that | 10:03 |
| cmyster | warp 9 will break space | 10:04 |
| sdake_ | i get why they dont go faster then warp 9 | 10:04 |
| cmyster | most of the times they limit it to no more then warp 5 | 10:04 |
| sdake_ | I just dont get why they dont roll at warp 9 all the time | 10:04 |
| cmyster | ^ | 10:04 |
| cmyster | will *break* space | 10:04 |
| sdake_ | warp 8.9? :) | 10:04 |
| cmyster | 5+ | 10:05 |
| sdake_ | anything over 5 breaks the system huh | 10:05 |
| sdake_ | well that explains it | 10:05 |
| sdake_ | I can sleep well tonight :) | 10:05 |
| cmyster | :) | 10:06 |
| shardy | They should just make it go to 11 ;) | 10:06 |
| sdake_ | anything over warp 10 causes time travel apparently | 10:06 |
| shardy | sdake_: reference to a cult movie called This is spinal tap | 10:07 |
| sdake_ | the only way to roll at warp 10 is to go around a star fast | 10:07 |
| *** shakayumi has quit IRC | 10:07 | |
| shardy | sdake_: they make their marshall guitar amps go to 11, because "it's one louder" ;D | 10:07 |
| shardy | https://www.youtube.com/watch?v=_sRhuh8Aphc | 10:08 |
| * shardy gets back to work.. | 10:08 | |
| *** aedocw has quit IRC | 10:09 | |
| shadower | would anyone from the core please have a look at this? https://bugs.launchpad.net/tripleo/+bug/1286307 | 10:10 |
| shadower | I think it's a high priority and should be fixed by icehouse | 10:11 |
| cmyster | shardy: 10 is max and was only achieved twice | 10:11 |
| shadower | but I'm biased :p | 10:11 |
| *** nati_ueno has quit IRC | 10:12 | |
| shardy | shadower: Thanks for the detailed report, I agree it looks like something we should try to fix, I'll try to reproduce when I've got through my morning reviews :) | 10:14 |
| shadower | shardy: thanks | 10:14 |
| openstackgerrit | Zhang Yang proposed a change to openstack/heat: Add doccument for firewall properties https://review.openstack.org/77840 | 10:15 |
| openstackgerrit | Vyacheslav Vakhlyuev proposed a change to openstack/heat: Fix comparison with singletons in unit tests https://review.openstack.org/76209 | 10:16 |
| sdake_ | shardy can you have a look at that comment on the -1 and make a quick recommendation so I can fix it up | 10:26 |
| *** mkollaro has joined #heat | 10:26 | |
| *** e0ne_ has quit IRC | 10:33 | |
| *** e0ne has joined #heat | 10:34 | |
| *** e0ne_ has joined #heat | 10:35 | |
| shardy | sdake_: done | 10:36 |
| *** e0ne has quit IRC | 10:38 | |
| sdake_ | shardy the only problem is the conf file just doesn't look right | 10:39 |
| sdake_ | also the clients section | 10:40 |
| sdake_ | I dont know why that happens | 10:40 |
| sdake_ | reading the code, its not immediately obvious | 10:40 |
| *** liang has quit IRC | 10:40 | |
| shardy | It could be a bug in the new version of the oslo generator stuff | 10:40 |
| asalkeld | if anyone isn't using synergy, you should give it a go: http://synergy-foss.org/ (software keyboard/mouse switching) | 10:40 |
| shardy | we definitely still have those options in common.config (register_clients_opts) | 10:40 |
| asalkeld | very nice | 10:41 |
| sdake_ | shardy that code looks evil | 10:41 |
| sdake_ | the register opts part | 10:41 |
| sdake_ | but it may be because files in the bin directory aren't being processed | 10:41 |
| sdake_ | notice host is deleted from each of the binaries | 10:43 |
| shardy | asalkeld: awesome, I've been looking for something like that, thanks | 10:43 |
| asalkeld | super cheap kvm switch | 10:44 |
| openstackgerrit | A change was merged to openstack/heat-templates: An example template demonstrating cloud-init config resources https://review.openstack.org/68549 | 10:44 |
| shardy | I actually have a KVM switch but the mouse gets broken when switching sometimes | 10:44 |
| asalkeld | k, this also allows some cool stuff like mapping keys on one machine back to the other | 10:45 |
| asalkeld | like the play/next button:) | 10:45 |
| shardy | it would be even better if there was some way to multiplex displays over the network, so you could scroll machines like desktop workspaces | 10:45 |
| asalkeld | that's frustrating | 10:45 |
| cmyster | sdake_: in https://bugs.launchpad.net/heat/+bug/1272002 that AllowedResources: is a template thing? if so I can't find anything about that in the usualy places | 10:45 |
| asalkeld | shardy, there might just be | 10:47 |
| *** fandi has quit IRC | 10:47 | |
| asalkeld | shardy, gnome-boxes? | 10:48 |
| shardy | asalkeld: Mostly xfce | 10:48 |
| asalkeld | haha | 10:49 |
| *** jamieh has joined #heat | 10:49 | |
| shardy | asalkeld: cool, that might work, thanks! | 10:50 |
| shardy | cmyster: http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Heat::AccessPolicy | 10:51 |
| shardy | cmyster: but the DescribeStackResource reference is legacy and wrong | 10:51 |
| shardy | that resource is to control any access to the resource from a particular (template defined) user | 10:52 |
| shardy | the API calls they can access are defined by the policy.json | 10:52 |
| sdake_ | shardy the way the client options are coplied in some odd way | 11:05 |
| sdake_ | anyway off to bed | 11:05 |
| *** blues-man has joined #heat | 11:06 | |
| *** cmyster has quit IRC | 11:24 | |
| *** nkhare has quit IRC | 11:32 | |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Migrate to oslo.messaging from oslo-incubator rpc https://review.openstack.org/72798 | 11:37 |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Update config generator for oslo.messaging https://review.openstack.org/77472 | 11:37 |
| *** mkollaro has quit IRC | 11:47 | |
| *** nkhare has joined #heat | 11:48 | |
| *** pshchelo has quit IRC | 11:50 | |
| *** cmyster has joined #heat | 11:52 | |
| *** pshchelo has joined #heat | 12:01 | |
| *** aedocw has joined #heat | 12:09 | |
| openstackgerrit | Thomas Spatzier proposed a change to openstack/heat: Fix HOT inconsistencies in resource sections https://review.openstack.org/73580 | 12:13 |
| *** ramishra has quit IRC | 12:23 | |
| *** arbylee has joined #heat | 12:28 | |
| *** rpothier has quit IRC | 12:38 | |
| *** david-lyle has quit IRC | 12:39 | |
| *** topol has joined #heat | 12:39 | |
| *** saju_m has quit IRC | 12:40 | |
| *** zhiyan has left #heat | 12:43 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Raise and catch a specific error during validation https://review.openstack.org/71287 | 12:43 |
| *** blues-man has quit IRC | 12:50 | |
| *** aweiteka has joined #heat | 12:50 | |
| *** saju_m has joined #heat | 12:52 | |
| *** achampion has quit IRC | 12:57 | |
| *** alexheneveld has quit IRC | 12:58 | |
| *** alexheneveld has joined #heat | 13:01 | |
| *** mkollaro has joined #heat | 13:01 | |
| *** rbuilta has joined #heat | 13:01 | |
| *** nkhare has quit IRC | 13:02 | |
| *** saju_m has quit IRC | 13:04 | |
| therve | shardy, There is a small problem with tests that I noticed on https://review.openstack.org/#/c/71930/ | 13:06 |
| *** bvandenh has joined #heat | 13:08 | |
| shardy | therve: oh, good catch, thanks! | 13:10 |
| *** ramishra has joined #heat | 13:12 | |
| *** sgordon has joined #heat | 13:16 | |
| openstackgerrit | A change was merged to openstack/heat: Convert empty filter dictionary to None https://review.openstack.org/77382 | 13:17 |
| *** faramir1 has joined #heat | 13:17 | |
| *** cmyster has quit IRC | 13:17 | |
| *** cmyster has joined #heat | 13:17 | |
| *** mkollaro has quit IRC | 13:18 | |
| *** mkollaro has joined #heat | 13:18 | |
| *** rbuilta has quit IRC | 13:18 | |
| *** rbuilta has joined #heat | 13:18 | |
| *** bvandenh has quit IRC | 13:18 | |
| *** bvandenh has joined #heat | 13:18 | |
| *** arbylee has quit IRC | 13:21 | |
| *** lsmola has quit IRC | 13:23 | |
| *** sballe has joined #heat | 13:33 | |
| openstackgerrit | A change was merged to openstack/heat: Add neutron network constraint https://review.openstack.org/73976 | 13:35 |
| *** rpothier has joined #heat | 13:35 | |
| *** lsmola has joined #heat | 13:36 | |
| *** rbuilta has quit IRC | 13:41 | |
| *** blomquisg has quit IRC | 13:43 | |
| *** nszceta has joined #heat | 13:43 | |
| *** alexheneveld has quit IRC | 13:45 | |
| *** mkollaro1 has joined #heat | 13:46 | |
| *** mkollaro has quit IRC | 13:46 | |
| *** arbylee has joined #heat | 13:52 | |
| *** nosnos has quit IRC | 13:56 | |
| *** blues-man has joined #heat | 13:57 | |
| *** tspatzier has quit IRC | 14:05 | |
| *** arbylee has quit IRC | 14:06 | |
| *** arbylee has joined #heat | 14:06 | |
| *** wchrisj has joined #heat | 14:08 | |
| *** annegentle has joined #heat | 14:09 | |
| *** achampion has joined #heat | 14:09 | |
| *** vijendar has joined #heat | 14:09 | |
| *** faramir1 has quit IRC | 14:12 | |
| *** tspatzier has joined #heat | 14:13 | |
| *** mkollaro has joined #heat | 14:15 | |
| *** mkollaro1 has quit IRC | 14:16 | |
| *** vijendar has quit IRC | 14:16 | |
| *** vijendar has joined #heat | 14:16 | |
| *** radez_g0n3 is now known as radez | 14:19 | |
| *** lsmola has quit IRC | 14:24 | |
| *** cfriesen_ has joined #heat | 14:24 | |
| *** bgorski has joined #heat | 14:30 | |
| *** blomquisg has joined #heat | 14:32 | |
| *** lsmola has joined #heat | 14:33 | |
| *** lsmola has quit IRC | 14:33 | |
| *** lsmola has joined #heat | 14:34 | |
| *** jasond has joined #heat | 14:35 | |
| therve | bgorski, I think https://review.openstack.org/#/c/76211/ may be related to the bug you just opened | 14:37 |
| *** blues-man has quit IRC | 14:38 | |
| *** blomquisg has quit IRC | 14:40 | |
| *** dims has quit IRC | 14:43 | |
| bgorski | therve, Thank you for the link. It is related but not the same | 14:43 |
| therve | Okay | 14:45 |
| *** dims has joined #heat | 14:46 | |
| *** jprovazn has joined #heat | 14:47 | |
| *** tomek_adamczewsk has quit IRC | 14:48 | |
| radix | hmm, the attribute | 14:48 |
| radix | therve: I forgot about that, shouldn't the attribute on ScalingPolicy also be renamed? | 14:48 |
| *** ramishra has quit IRC | 14:49 | |
| therve | radix, Yes I'll do that in a bit | 14:50 |
| radix | ok | 14:50 |
| *** lsmola has quit IRC | 14:50 | |
| radix | therve: and what do you think about the restriction of use only with the new AutoScalingGroup? | 14:50 |
| *** tspatzier has quit IRC | 14:50 | |
| therve | radix, Nothing. What about it? | 14:50 |
| radix | this is what I wrote: "Maybe we should restrict this resource to only being used with OS::Heat::AutoScalingGroup, since users would be able to use it with the AWS version now, but in the future that will probably be difficult to support." | 14:51 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Remove vim header from files. https://review.openstack.org/76378 | 14:52 |
| *** ramishra has joined #heat | 14:53 | |
| *** blomquisg has joined #heat | 14:55 | |
| therve | radix, Maybe add some documentation? | 14:56 |
| *** tspatzier has joined #heat | 14:56 | |
| therve | I'm not super fan of adding code to prevent it | 14:56 |
| radix | ok | 14:56 |
| radix | as long as we can not care about compatibility later | 14:56 |
| openstackgerrit | Bartosz Górski proposed a change to openstack/heat: Pass stack create/update timeout to scaling group https://review.openstack.org/77902 | 14:58 |
| aweiteka | tspatzier, thanks for the catch on the template review. since this was an existing problem should i add another commit to fix? https://review.openstack.org/#/c/77773/ | 14:58 |
| tspatzier | hi aweiteka | 14:59 |
| aweiteka | tspatzier, hello | 14:59 |
| tspatzier | Yes, I think this is something that already existed in the files. I saw that in an earlier review, but I was too late back then since I was on vacation, so it got merged. | 14:59 |
| *** zns has joined #heat | 14:59 | |
| tspatzier | So yes, maybe do all those syntax fixes in a different patch which we can then review. | 15:00 |
| aweiteka | tspatzier, sure, no problem. i was just about to test another run so the timing is good. | 15:00 |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Native ScalingPolicy resource https://review.openstack.org/77529 | 15:00 |
| therve | radix, I added a note | 15:00 |
| radix | ok | 15:00 |
| tspatzier | aweiteka: ok, sounds like a plan :-) | 15:01 |
| *** gokrokve_ has quit IRC | 15:01 | |
| *** gokrokve has joined #heat | 15:01 | |
| *** arbylee has quit IRC | 15:02 | |
| openstackgerrit | A change was merged to openstack/heat: Modify stack_user_domain config option to take domain ID https://review.openstack.org/73978 | 15:02 |
| openstackgerrit | A change was merged to openstack/heat: Move HOT version checking to HOTemplate class https://review.openstack.org/77750 | 15:02 |
| openstackgerrit | A change was merged to openstack/heat: Add config options to specify stack domain admin https://review.openstack.org/76035 | 15:03 |
| openstackgerrit | A change was merged to openstack/heat: Add a validation step to parameters schema https://review.openstack.org/69655 | 15:03 |
| openstackgerrit | Ziad Sawalha proposed a change to openstack/heat: Update api docstrings to match guidelines https://review.openstack.org/73515 | 15:03 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make the first line of every file consistent. https://review.openstack.org/76591 | 15:04 |
| *** Tross has quit IRC | 15:04 | |
| *** tspatzier has quit IRC | 15:04 | |
| *** mnaser has quit IRC | 15:08 | |
| *** Guest77723 has quit IRC | 15:08 | |
| *** alexheneveld has joined #heat | 15:08 | |
| openstackgerrit | A change was merged to openstack/heat: engine: allow stack_user_project users to retrieve stack https://review.openstack.org/71300 | 15:09 |
| openstackgerrit | A change was merged to openstack/heat: migrate StackUser base class to stack domain users https://review.openstack.org/71210 | 15:09 |
| openstackgerrit | A change was merged to openstack/heat: Implement OS::Neutron::ExtraRoute as /contrib https://review.openstack.org/74899 | 15:09 |
| *** mnaser has joined #heat | 15:09 | |
| *** jergerber has joined #heat | 15:12 | |
| *** mkollaro1 has joined #heat | 15:13 | |
| *** mkollaro has quit IRC | 15:13 | |
| *** lazy_prince is now known as killer_prince | 15:13 | |
| *** tspatzier has joined #heat | 15:14 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 15:16 |
| *** arbylee has joined #heat | 15:17 | |
| openstackgerrit | Alexander Ignatov proposed a change to openstack/heat: OS::Savanna::Cluster resource implementation https://review.openstack.org/72336 | 15:20 |
| *** sabeen has joined #heat | 15:21 | |
| radix | therve: typo in your docstring :P | 15:23 |
| therve | Damn | 15:23 |
| radix | hehe | 15:23 |
| therve | radix, Where? | 15:23 |
| radix | therve: you say AWS::AutoScaling::ScalingPolicy twice | 15:23 |
| *** sabeen has left #heat | 15:24 | |
| radix | oh wait | 15:24 |
| therve | No? :) | 15:24 |
| radix | no, I misread it :P | 15:24 |
| therve | Heh | 15:24 |
| radix | reading comprehension, I don't has it | 15:24 |
| *** chandan_kumar has joined #heat | 15:25 | |
| *** arbylee has quit IRC | 15:26 | |
| *** arbylee has joined #heat | 15:26 | |
| *** david-lyle has joined #heat | 15:28 | |
| sdake | morning | 15:29 |
| *** spzala has joined #heat | 15:30 | |
| *** lsmola has joined #heat | 15:32 | |
| andersonvom | morning | 15:33 |
| radix | is there a way to affect the parallelism of test runs by tox -epy27? | 15:35 |
| *** randallburt has joined #heat | 15:37 | |
| *** jmckind has joined #heat | 15:37 | |
| jasond | radix: tox -e py27 -- --concurrency=1 | 15:37 |
| *** bvandenh has quit IRC | 15:38 | |
| *** coolsvap has joined #heat | 15:38 | |
| radix | jasond: ok thanks | 15:40 |
| radix | jasond: do you know what it does by default? NUMCPU-1 or something? | 15:40 |
| jasond | radix: the default is --concurrency=0 (auto-detect) | 15:41 |
| * radix looks up testr docs | 15:41 | |
| jasond | seems to be the number of CPUs | 15:41 |
| *** bvandenh has joined #heat | 15:41 | |
| radix | oh ok, yeah, just saw that | 15:43 |
| therve | radix, SC_NPROCESSORS_ONLN! | 15:45 |
| radix | therve: "Did you mean: _SC_NPROCESSORS_ONLN" | 15:45 |
| radix | google is pretty cool | 15:45 |
| therve | :) | 15:45 |
| *** tomek_adamczewsk has joined #heat | 15:46 | |
| radix | also, anyone know if it's possible to specify user-local tox config options? | 15:46 |
| radix | I want to change something about tox but I don't want to maintain an indefinite patch against tox.ini... | 15:46 |
| *** nati_ueno has joined #heat | 15:57 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Fixup uuid stubbing in test_stack_user.py https://review.openstack.org/77920 | 15:57 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: migrate User/AccessKey resources to StackUser base class https://review.openstack.org/72763 | 15:57 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: StackUser add _delete_keypair function https://review.openstack.org/72762 | 15:57 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add test for StackUser._create_keypair https://review.openstack.org/72761 | 15:57 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: StackUser add suspend/resume support https://review.openstack.org/71930 | 15:57 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add delete_stack_domain_user_keypair https://review.openstack.org/71929 | 15:57 |
| *** coolsvap1 has joined #heat | 15:58 | |
| shardy | therve: hopefully that fixes the issue you spotted, although I couldn't reproduce the sqlalchemy error you referred to | 15:58 |
| therve | shardy, Just warning message | 15:58 |
| therve | shardy, You know, the "deprecated message" thing | 15:58 |
| sdake | isn't it reverify bug xyz when the gate fails to do the final commit (not the initial check's test of the patch) | 15:59 |
| shardy | therve: ah, right, I thought that was a different issue | 15:59 |
| therve | shardy, We ignore db error when creating events | 15:59 |
| shardy | therve: well I no longer stub uuid at all, so hopefully all good now :) | 16:00 |
| shardy | therve: umm, that sounds like a bug to me | 16:00 |
| therve | Maybe yeah | 16:00 |
| pscheie_ | I have a few stacks whose state is UPDATE_FAILED because of some missing parameters in the manifest when I tried to update the stack. | 16:00 |
| *** coolsvap has quit IRC | 16:00 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Don't include a default HOT version for unit tests https://review.openstack.org/77922 | 16:01 |
| pscheie_ | If I go into the heat DB and manually change that status to, say, UPDATE_COMPLETE, should I then be able to do updates on those stacks then? | 16:01 |
| shardy | pscheie_: anything could happen, because the partially updated stack is in an inconsistent state | 16:02 |
| *** tomek_adamczewsk has quit IRC | 16:02 | |
| shardy | pscheie_: it's a known issue, we're looking at enabling retry from failed updates, but right now there's no safe way to do it | 16:02 |
| *** rcleere has joined #heat | 16:02 | |
| pscheie_ | shardy, is there any way to manually go through and validate the state? | 16:03 |
| shardy | If it's a dev environment, you might get away with hacking the DB | 16:03 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Fix resolving for Ref function https://review.openstack.org/77923 | 16:03 |
| *** nszceta has quit IRC | 16:03 | |
| shardy | pscheie_: it really depends on the template and why the update failed | 16:03 |
| andersonvom | shardy: god morning! when you have a few mins to spare, could you take a look at this? https://review.openstack.org/#/c/74190/ | 16:05 |
| pscheie_ | shardy, in one case it was caused by a Resource (a vm) specifying an InstanceType that was not on the AllowedValues list | 16:05 |
| shardy | andersonvom: sure, I have already looked at it but was waiting for zaneb to take a look, as he did most of the heat-engine plugin implementation | 16:07 |
| *** nszceta has joined #heat | 16:09 | |
| andersonvom | shardy: thanks. I guess it makes sense to add him to the review as well. =] | 16:09 |
| *** nszceta has quit IRC | 16:09 | |
| shardy | andersonvom: I already did, last week | 16:09 |
| andersonvom | :P | 16:09 |
| openstackgerrit | Clint "SpamapS" Byrum proposed a change to openstack/heat: WIP: Record resource actions in a journal https://review.openstack.org/77374 | 16:10 |
| * zaneb has 1526 unread Gerrit emails atm :( | 16:11 | |
| SpamapS | zaneb: I have found gerrit emails to be basically useless at this point. :-/ | 16:13 |
| SpamapS | too much noise to signal | 16:13 |
| zaneb | I used to keep up with them religiously | 16:13 |
| cmyster | D$+$ ftw | 16:13 |
| zaneb | but yeah, all the rechecks and rebases have bumped up the noise ratio a lot | 16:14 |
| SpamapS | At this point I just try to make sure I respond to the reviews I've commented on, and drive queues toward 0 | 16:14 |
| zaneb | and as soon as you start ignoring them, then you get out of sync with the ones you have actually dealt with for some other reason (someone pinged you, or you saw it in Gerrit) | 16:14 |
| openstackgerrit | Victor Stinner proposed a change to openstack/heat: Replace ad-hoc coroutines with Trollius coroutines https://review.openstack.org/77925 | 16:15 |
| *** jamieh has quit IRC | 16:15 | |
| zaneb | and then scrubbing the mailbox involves figuring out which you still need to deal with and which you don't | 16:15 |
| SpamapS | zaneb: yeah, oddly enough I find that the main "my changes" page is good enough for keeping up with the ones you're commenting on actively.. | 16:15 |
| shardy | zaneb: I feel your pain, I just cleaned out my filtered gerrit mailbox, had over 29,000 emails, mostly unread | 16:16 |
| zaneb | SpamapS: not really for me. it only lists 13 items, and once you review them they don't make way for other ones | 16:16 |
| shardy | email seems like a suboptimal interface to prioritize reviews to me, so I gave up using them | 16:17 |
| zaneb | and with all the rebases and rechecks, it takes forever for patches to get off that list | 16:17 |
| *** bvandenh has quit IRC | 16:17 | |
| SpamapS | zaneb: I have about 30 on that list.. andonce I've reviewed all of them, I go back and see if I can help land the ones I've +2'd | 16:18 |
| *** rbuilta has joined #heat | 16:18 | |
| SpamapS | which is often a recheck or rebase ;) | 16:18 |
| shardy | SpamapS: did you see the remaining instance-users patches started landing? | 16:18 |
| shardy | SpamapS: there was some concern re impact on the tripleo ci stuff | 16:18 |
| zaneb | shardy: everything is suboptimal :( I like emails because I can mark them as read (dealt with) or not (not) independently of whatever action I take on them | 16:18 |
| radix | the *one* nice thing about the gerrit emails is that they include comments | 16:18 |
| shardy | as you'll need to create the domain, user and update the config | 16:19 |
| radix | or rather, include the inline comments | 16:19 |
| radix | not even the web view does that, which is hecka lame | 16:19 |
| *** rcleere has quit IRC | 16:19 | |
| sdake | andersonvom I wanted to confirm you saw my last comment in https://review.openstack.org/#/c/58641/29 re the use of type() | 16:19 |
| zaneb | radix: agree, it's a PITA. they could at least link to the files containing new comments, instead of making you go searching | 16:19 |
| shardy | zaneb: Yeah, I don't claim to have found an optimal solution, I use a variety of queries in the web UI instead | 16:19 |
| andersonvom | sdake: I did, and I added another comment there. actually, the implementation of Resource.type() actually returns the type used in the template, which is precisely what you wanted, right? | 16:20 |
| SpamapS | 1393949773.278 1225 192.168.122.1 TCP_MISS_ABORTED/000 0 POST http://server.test:5000/v3/auth/tokens - HIER_DIRECT/1.0.0.1 - | 16:20 |
| sdake | cool | 16:20 |
| sdake | I didn't know that :) | 16:20 |
| SpamapS | still stuff not properly mocked | 16:20 |
| SpamapS | shardy: right I asked stevebaker to open a bug and we'll make sure we test/respond this week as we have all the devs here in sunnyvale | 16:21 |
| sdake | well anyway I gave it a +2 \o/ :) | 16:21 |
| andersonvom | sdake: I had to rebase the patch, though. | 16:21 |
| zaneb | sdake: self.type() is an instance method, type(self) is a built-in function that returns the class | 16:21 |
| sdake | yes I saw that | 16:21 |
| jasond | i wish gerrit would include me in the To: header of the email when someone replies to a comment i made | 16:21 |
| shardy | SpamapS: Ok, cool, well feel free to raise a bug with details on the mocking issues, and let me know if you need any assistance with the transition to domain users | 16:22 |
| shardy | SpamapS: we reached the point where we had to get it in, hopefully any transitional disruption will be minimal :) | 16:22 |
| *** andersonvom has quit IRC | 16:22 | |
| zaneb | radix, jasond: BTW Gerrit is Open Source. Just sayin' ;) | 16:23 |
| sdake | zaneb ;) | 16:23 |
| *** andersonvom has joined #heat | 16:23 | |
| jasond | zaneb: good point :) | 16:23 |
| sdake | the neutron jobs are voting on the gate | 16:23 |
| sdake | and non-voting on the check | 16:23 |
| SpamapS | shardy: if it is disruptive to tripleo, it will be disruptive to all deployers.. so -> release notes ;) | 16:23 |
| sdake | which is annoying,because they fail all the time! | 16:23 |
| *** jprovazn has quit IRC | 16:23 | |
| shardy | SpamapS: absolutely, we will document in the release notes, but tripleo is the pre-release canary ;) | 16:24 |
| pscheie_ | shardy, is there a way to do a "test run" of an update, so one could find errors in the template *before* potentially putting a stack in an UPDATE_FAILED state? | 16:24 |
| * SpamapS wonders if shardy just called TripleO cute | 16:24 | |
| shardy | basically just creating a domain and a user, and putting the details in heat.conf, hopefully not too painful :) | 16:24 |
| pscheie_ | I know there is template-validate, but that just checks for syntax, I think. | 16:24 |
| sdake | watching zuul is like watching grass grow | 16:25 |
| SpamapS | oh evil | 16:25 |
| SpamapS | I'm on an evil hotel network which responds to requests for serve.test with 1.0.0.1 | 16:26 |
| SpamapS | which is their evil "pay for access" thing | 16:26 |
| sdake_ | are there any low or medium priority blueprints folks would like reviews on? | 16:26 |
| SpamapS | harlowja_away: see you soon eh? ;) | 16:26 |
| * SpamapS heads to Yahoo! | 16:26 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update https://review.openstack.org/63779 | 16:26 |
| *** tnurlygayanov has joined #heat | 16:26 | |
| *** e0ne has joined #heat | 16:27 | |
| sdake_ | (targeted for i3) | 16:27 |
| SpamapS | shardy: aaaand with that, we have fail | 16:27 |
| SpamapS | "Anyone seen this: ERROR: Property error : undercloud: key_name stack_user_domain ID not set in heat.conf" | 16:27 |
| shardy | SpamapS: that's what I was just talking about :) | 16:27 |
| SpamapS | shardy: can you suggest a fix? | 16:28 |
| *** pablosan has joined #heat | 16:28 | |
| jasond | would someone mind reviewing this? https://review.openstack.org/#/c/72009/ (its child has already been approved) | 16:28 |
| *** tomek_adamczewsk has joined #heat | 16:28 | |
| SpamapS | shardy: also.. doesn't that break upgrades in an unacceptable way? | 16:28 |
| *** pablosan is now known as ZZpablosan | 16:28 | |
| shardy | SpamapS: It's in https://review.openstack.org/#/c/73978/ | 16:29 |
| shardy | The commit message describes what needs to be done (as does the error message) | 16:29 |
| *** e0ne_ has quit IRC | 16:29 | |
| shardy | SpamapS: for havana->icehouse upgrades, whatever tooling performs the upgrade will need to update heat.conf as part of the process | 16:29 |
| sdake_ | shardy I think people magically expect old config files to work :) | 16:30 |
| *** rollerj has joined #heat | 16:30 | |
| shardy | SpamapS: Also see https://review.openstack.org/#/c/76035/ where there are two more things which need setting in heat.conf | 16:30 |
| SpamapS | shardy: that sort of requirement has been rejected for cinder recently | 16:30 |
| shardy | SpamapS: So how do you propose we handle it? | 16:30 |
| SpamapS | shardy: I think it is fine, but just letting you know that cinder wasn't able to require a new setting ... | 16:31 |
| SpamapS | shardy: I think we're handling it fine. Just pointing out it is perhaps a bit different than other projects' policies. | 16:31 |
| zaneb | jasond: done | 16:31 |
| shardy | SpamapS: who rejected it? | 16:31 |
| *** jistr has quit IRC | 16:31 | |
| jasond | zaneb: thanks | 16:32 |
| sdake_ | shardy the general openstack theory is you should be be able to roll from release to release without changing anything | 16:32 |
| sdake_ | the reality is the db updates and rpc updates aren't always kosher | 16:32 |
| sdake_ | (changing anything but the binaries) | 16:32 |
| *** shakayumi has joined #heat | 16:32 | |
| shardy | SpamapS: Ok, cool, well like I said, tripleo is the early adopter here by default so we can take feedback from you if you think there's some better, less disruptive way | 16:32 |
| shardy | maybe we set the heat domain id to "default" and use the service admin creds by default if they are not set | 16:33 |
| SpamapS | shardy: sdague rejected the cinder change way back when because it would break grenade to require the config file to be updated. | 16:33 |
| SpamapS | shardy: we wanted to require setting a lock dir | 16:33 |
| shardy | SpamapS: Ok, well if we have to I may be able to provide a patch which makes the new code do the same thing as before if no config data is found, but if possible I'd prefer not to as the user side of it would be a bit messy | 16:35 |
| SpamapS | shardy: what about "sane defaults" ? | 16:35 |
| shardy | SpamapS: devstack is already updated due to the gate, and I was just planning to document the new requirements in the release notes | 16:36 |
| shardy | SpamapS: what is a sane default for keystone_authtoken admin_password? | 16:36 |
| shardy | It is a required config input, just like these new values | 16:36 |
| shardy | That is the cost of fixing the admin requirement for WaitConditions &c. | 16:37 |
| *** TonyBurn_ has quit IRC | 16:38 | |
| shardy | We *could* default the stack_user_domain to "default" and use the context credentials if stack_domain_admin* aren't set | 16:38 |
| shardy | that would closely approximate the previous behavior I guess | 16:39 |
| shardy | but then folks will complain that they still require admin and that we didn't magic them a new config file | 16:39 |
| *** harlowja_away is now known as harlowja | 16:40 | |
| *** harlowja has quit IRC | 16:41 | |
| *** ZZpablosan has quit IRC | 16:43 | |
| *** pablosan has joined #heat | 16:44 | |
| *** arbylee has quit IRC | 16:45 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-heatclient: Python3: fix a bytes/str issue https://review.openstack.org/77938 | 16:53 |
| *** nszceta has joined #heat | 16:56 | |
| *** harlowja has joined #heat | 16:56 | |
| *** cmyster has quit IRC | 16:56 | |
| *** samstav has joined #heat | 16:58 | |
| *** ramishra has quit IRC | 16:58 | |
| *** ramishra has joined #heat | 16:59 | |
| *** blues-man has joined #heat | 16:59 | |
| *** vijendar1 has joined #heat | 17:00 | |
| *** samstav_ has joined #heat | 17:01 | |
| *** vijendar1 has quit IRC | 17:01 | |
| *** vijendar1 has joined #heat | 17:01 | |
| *** randallburt has quit IRC | 17:01 | |
| *** sabeen has joined #heat | 17:01 | |
| *** sabeen has left #heat | 17:01 | |
| *** vijendar has quit IRC | 17:02 | |
| *** ramishra has quit IRC | 17:03 | |
| *** rcleere has joined #heat | 17:03 | |
| *** vijendar has joined #heat | 17:04 | |
| *** samstav has quit IRC | 17:04 | |
| *** alexheneveld has quit IRC | 17:04 | |
| *** jdob has joined #heat | 17:06 | |
| *** alexheneveld has joined #heat | 17:06 | |
| *** vijendar1 has quit IRC | 17:06 | |
| *** arbylee has joined #heat | 17:08 | |
| *** jprovazn has joined #heat | 17:09 | |
| *** pablosan has quit IRC | 17:11 | |
| *** pablosan has joined #heat | 17:12 | |
| *** topol has quit IRC | 17:12 | |
| jasond | shardy: would you mind taking a look at this? https://review.openstack.org/#/c/74190/ | 17:13 |
| shardy | jasond: sure | 17:13 |
| *** nati_ueno has quit IRC | 17:13 | |
| jasond | thanks | 17:13 |
| *** rwsu has joined #heat | 17:14 | |
| *** Tross has joined #heat | 17:14 | |
| shardy | Oh that's the one andersonvom was asking about earlier | 17:15 |
| andersonvom | popular patch ;) | 17:15 |
| jasond | oh, i didn't realize | 17:15 |
| shardy | zaneb: did you want to take a look or shall I go ahead and approve? | 17:16 |
| zaneb | looking... | 17:16 |
| zaneb | lgtm | 17:17 |
| andersonvom | o/ | 17:17 |
| *** e0ne has quit IRC | 17:17 | |
| shardy | cool, thanks | 17:18 |
| andersonvom | shardy, zaneb: thanks | 17:18 |
| *** samstav_ has quit IRC | 17:18 | |
| *** samstav has joined #heat | 17:19 | |
| *** lindsayk has joined #heat | 17:19 | |
| sdake_ | when working with mock, what does this mean UnexpectedMethodCallError: Unexpected method call start.__call__() -> None | 17:20 |
| sdake_ | could someone translate that into engrish for me pls :) | 17:21 |
| arbylee | sdake_: that looks like a mox error rather than a mock one to me | 17:23 |
| *** derekh has joined #heat | 17:23 | |
| *** derekh has quit IRC | 17:23 | |
| arbylee | sdake_: you have some mox object setup that didn't set up all the expected calls before verifying | 17:23 |
| *** alexheneveld has quit IRC | 17:24 | |
| *** tomek_adamczewsk has quit IRC | 17:25 | |
| *** rbuilta has quit IRC | 17:27 | |
| *** pablosan is now known as ZZpablosan | 17:29 | |
| *** ZZpablosan is now known as pablosan | 17:29 | |
| *** pablosan is now known as ZZpablosan | 17:29 | |
| andersonvom | sdake_: mock will usually tell you what was expected and what the actual call was | 17:29 |
| *** ZZpablosan is now known as pablosan | 17:30 | |
| *** pablosan is now known as ZZpablosan | 17:30 | |
| sdake_ | andersonvom http://paste.fedoraproject.org/82247/93954237 | 17:30 |
| sdake_ | what I dont get is why mock and mox are used in the same test case | 17:31 |
| *** nszceta has quit IRC | 17:31 | |
| *** ZZpablosan is now known as pablosan | 17:31 | |
| *** varora has joined #heat | 17:32 | |
| *** derekh has joined #heat | 17:32 | |
| *** aweiteka has quit IRC | 17:32 | |
| arbylee | sdake_: it looks like your listener object (self.listener on line 299) is a mox object that wasn't expecting the #start call. | 17:33 |
| *** cody-somerville has joined #heat | 17:33 | |
| andersonvom | oooh, my test! | 17:34 |
| andersonvom | sdake_: yeah, if you look at line 1321, EngineListener is being stubbed with mox | 17:35 |
| sdake_ | andersonvom I need to stub out oslo.messaging.server.MessagingHandlingServer.start so it doesn't actually do anything | 17:37 |
| andersonvom | sdake_: oh, I see... on EngineService.__init__, start is already being called, but it's also being called on line 1187 in your code | 17:37 |
| *** cadenzajon has joined #heat | 17:38 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Add IP address to error message of failed stack https://review.openstack.org/77231 | 17:39 |
| sdake_ | andersonvom nosetests heat.tests.test_engine_service:StackServiceTest.test_start_gets_all_stacks | 17:40 |
| sdake_ | andersonvom http://paste.fedoraproject.org/82252/39395481 | 17:40 |
| sdake_ | this is the code I am working with, the line 3's you mentioned don't match up | 17:40 |
| shardy | SpamapS: FYI, started a thread on openstack-dev re upgrades | 17:40 |
| shardy | SpamapS: a similar argument will apply to flipping trusts to be default I guess | 17:41 |
| SpamapS | shardy: many thanks for driving that. :) | 17:41 |
| shardy | although that is a more transparent change, provided the deployment has the trusts extension is enabled in the keystone used by heat | 17:42 |
| shardy | SpamapS: np, happy to get feedback so we can get it right before the release :) | 17:42 |
| *** Tross has quit IRC | 17:42 | |
| *** rbuilta has joined #heat | 17:43 | |
| sdake_ | andersonvom line 1451 | 17:43 |
| andersonvom | sdake_: it looks like you're stubbing MessagingHandlingServer twice. once with mock and another one with mox. I'd get rid of lines 1456, 1458, 1462, since you've already mocked it with the decorator | 17:46 |
| andersonvom | sdake_: then, add another line similar to 1440 | 17:47 |
| andersonvom | sdake_: this is based on lines 298,299 of your stack trace (http://paste.fedoraproject.org/82247/93954237/) | 17:48 |
| *** blues-man has left #heat | 17:50 | |
| sdake_ | andersonvom yes but for adding a line similiar to 1440, I want to mock the super start, not the eng.start | 17:50 |
| *** topol has joined #heat | 17:51 | |
| andersonvom | sdake_: oh, so you want Engine.start() to be run? | 17:52 |
| sdake_ | yes! | 17:52 |
| sdake_ | I just don't want the super of Engine.start to run | 17:52 |
| andersonvom | sdake_: rather, EngineListener.start? | 17:52 |
| sdake_ | hence the reason for @mock.patch.object(service.messaging.MessageHandlingServer, 'start') | 17:52 |
| *** chandan_kumar has quit IRC | 17:53 | |
| *** zns has quit IRC | 17:53 | |
| *** che-arne has quit IRC | 17:53 | |
| sdake_ | in my reorg of service.py, EngineListener is a self of EngineService | 17:53 |
| *** pablosan is now known as ZZpablosan | 17:54 | |
| sdake_ | service.py http://paste.fedoraproject.org/82259/55654139 | 17:54 |
| *** topol has quit IRC | 17:55 | |
| *** gokrokve has quit IRC | 17:55 | |
| *** gokrokve has joined #heat | 17:56 | |
| andersonvom | sdake_: oh... I see | 17:56 |
| *** randallburt has joined #heat | 17:56 | |
| *** randallburt has quit IRC | 17:56 | |
| andersonvom | sdake_: so, if you look at lines 1178 and 1187, self.listener.start() is being called twice | 17:56 |
| *** randallburt has joined #heat | 17:57 | |
| andersonvom | sdake_: but on line 1439 of your test (http://paste.fedoraproject.org/82252/39395481/), it's being "moxed" only once. hence mox complaining about it not being expected | 17:58 |
| andersonvom | sdake_: if you refactor it to be mocked with mock instead, this error should go away. | 17:58 |
| *** alexheneveld has joined #heat | 17:58 | |
| sdake_ | woha | 17:58 |
| sdake_ | bad | 17:58 |
| sdake_ | thanks | 17:58 |
| sdake_ | I would have never figured that out :) | 17:58 |
| *** topol has joined #heat | 17:59 | |
| andersonvom | np! :D | 17:59 |
| *** Michalik has quit IRC | 17:59 | |
| *** pmathews has joined #heat | 17:59 | |
| *** tspatzier has quit IRC | 17:59 | |
| *** gokrokve has quit IRC | 18:00 | |
| *** gokrokve has joined #heat | 18:01 | |
| andersonvom | our for lunch, kids. brb | 18:01 |
| andersonvom | s/our/out/ | 18:01 |
| *** rbuilta has quit IRC | 18:02 | |
| *** Linz has joined #heat | 18:02 | |
| *** Linz has quit IRC | 18:02 | |
| *** Linz has joined #heat | 18:03 | |
| sdake_ | andersonvom thanks :) | 18:03 |
| *** Michalik has joined #heat | 18:04 | |
| *** daneyon has joined #heat | 18:04 | |
| *** lindsayk has quit IRC | 18:05 | |
| * sdake_ groans - 61 test failures after that change to service.py | 18:06 | |
| *** samstav has quit IRC | 18:07 | |
| *** samstav has joined #heat | 18:08 | |
| *** arbylee has quit IRC | 18:09 | |
| *** che-arne has joined #heat | 18:10 | |
| *** pvaneck has joined #heat | 18:10 | |
| *** aweiteka has joined #heat | 18:13 | |
| *** yogesh has joined #heat | 18:18 | |
| *** nszceta has joined #heat | 18:19 | |
| *** nati_ueno has joined #heat | 18:22 | |
| *** nati_ueno has quit IRC | 18:22 | |
| *** nati_ueno has joined #heat | 18:23 | |
| openstackgerrit | Alexander Ignatov proposed a change to openstack/heat: OS::Savanna::Cluster resource implementation https://review.openstack.org/72336 | 18:23 |
| *** gokrokve has quit IRC | 18:25 | |
| *** gokrokve has joined #heat | 18:25 | |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Return None when get_attr cannot resolve a value https://review.openstack.org/77713 | 18:27 |
| randallburt | jasond`, andersonvom ^^ at your earliest, thanks! | 18:28 |
| *** gokrokve has quit IRC | 18:29 | |
| openstackgerrit | A change was merged to openstack/heat: Allow Server name property to be updated https://review.openstack.org/74624 | 18:31 |
| *** arbylee has joined #heat | 18:33 | |
| *** saurabhs has joined #heat | 18:37 | |
| *** e0ne has joined #heat | 18:37 | |
| openstackgerrit | Paul Van Eck proposed a change to openstack/heat: Allow proper instance with volume suspension https://review.openstack.org/76126 | 18:39 |
| openstackgerrit | A change was merged to openstack/heat: Nova keypair validation breaks client plugins https://review.openstack.org/73441 | 18:39 |
| *** e0ne has quit IRC | 18:39 | |
| *** achampion has quit IRC | 18:42 | |
| *** pmathews has left #heat | 18:42 | |
| *** pmathews has joined #heat | 18:42 | |
| *** rbuilta has joined #heat | 18:43 | |
| openstackgerrit | A change was merged to openstack/heat: Make server resources use nova_utils.refresh_server() https://review.openstack.org/72009 | 18:43 |
| *** bgorski has quit IRC | 18:44 | |
| *** achampion has joined #heat | 18:49 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Keep trying to get SSH transport for up to 30 sec https://review.openstack.org/77677 | 18:50 |
| *** gokrokve has joined #heat | 18:50 | |
| kevinbenton | zaneb: ping | 18:52 |
| *** Linz has quit IRC | 18:53 | |
| kevinbenton | stevebaker: ping | 18:54 |
| *** piousbox has joined #heat | 18:54 | |
| *** jistr has joined #heat | 18:57 | |
| *** e0ne has joined #heat | 18:57 | |
| *** jmckind has quit IRC | 18:57 | |
| *** Linz has joined #heat | 19:01 | |
| *** alexheneveld has quit IRC | 19:05 | |
| *** zns has joined #heat | 19:06 | |
| *** sballe has quit IRC | 19:07 | |
| zaneb | kevinbenton: pong | 19:08 |
| stevebaker | kevinbenton: pung | 19:08 |
| kevinbenton | stevebaker, zaneb: can we do a quick post-mortem on https://review.openstack.org/#/c/41044/ ? | 19:10 |
| *** pmathews has quit IRC | 19:10 | |
| kevinbenton | I wasn't really clear on what pushed it into contrib | 19:10 |
| kevinbenton | if it was the risk of race conditions or the use of CIDRs | 19:10 |
| zaneb | the use case that was proffered (by Kanabuchi-san) for it was for for network admins only | 19:11 |
| zaneb | that was why it was pushed into contrib, because that use case doesn't apply to the general Heat user | 19:12 |
| stevebaker | kevinbenton: the neutron API didn't allow for your use case, so until the neutron API is changed the only way of giving you what you need is by putting the resource somewhere which indicates that it is not entirely endorsed by Heat | 19:12 |
| *** ZZpablosan is now known as pablosan | 19:12 | |
| kevinbenton | zaneb: his use case was not the same as mine | 19:13 |
| zaneb | understood | 19:13 |
| kevinbenton | stevebaker: so a change to allow safe updates would be okay? | 19:13 |
| zaneb | so IMO I don't think modelling that particular neutron API directly is ever going to yield a good result | 19:13 |
| stevebaker | kevinbenton: a dedicated neutron API for route entries would be ideal | 19:14 |
| zaneb | I think we need to break it down into individual use cases, with individual solutions, that may use that API on the back end but not be modelled on it | 19:14 |
| *** e0ne has quit IRC | 19:16 | |
| *** e0ne has joined #heat | 19:16 | |
| kevinbenton | so there are slightly conflicting messages that I'm interpreting. zaneb: it seems like you are mainly against users being allowed to reference arbitrary CIDRs, right? stevebaker: I understand your POV and will look into making changes on neutron depending on zaneb's | 19:16 |
| *** gokrokve_ has joined #heat | 19:17 | |
| zaneb | so +1 to fixing the Neutron API if we can | 19:17 |
| zaneb | there are a couple of problems that I see with the current one | 19:18 |
| zaneb | #1 is that the API is inherently race-y :( | 19:18 |
| kevinbenton | zaneb: i completely understand that concern | 19:19 |
| *** wusWWW has joined #heat | 19:19 | |
| zaneb | #2 is that the resource doesn't take any inputs that allow Heat to infer the order in which it needs to be called | 19:19 |
| kevinbenton | zaneb: right, so would a reasonable compromise be to add the egress router interface for next-hop as part of the input? | 19:21 |
| *** gokrokve has quit IRC | 19:21 | |
| kevinbenton | zaneb: so then users can still specify an arbitrary next-hop and CIDR, but heat can know to wait until the router interface is active | 19:22 |
| zaneb | kevinbenton: I believe so. if that guarantees the correct order then yes | 19:22 |
| zaneb | +1 | 19:22 |
| *** piousbox has quit IRC | 19:23 | |
| kevinbenton | ok, i'm going to look into what it would take to fix extra routes then | 19:23 |
| kevinbenton | on the neutron side | 19:23 |
| kevinbenton | to support more goodness and less badness :-) | 19:24 |
| zaneb | excellent :) | 19:24 |
| kevinbenton | zaneb, stevebaker: thanks for your time | 19:24 |
| sdake_ | gate sluggish today | 19:26 |
| sdake_ | wonder what it could possibly be :) | 19:26 |
| *** jamieh has joined #heat | 19:26 | |
| * sdake_ fires gate-tempest-dvsm-neutron | 19:27 | |
| *** bada has joined #heat | 19:29 | |
| *** tango|2 has joined #heat | 19:32 | |
| *** tomek_adamczewsk has joined #heat | 19:33 | |
| *** lsmola has quit IRC | 19:35 | |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Return None when get_attr cannot resolve a value https://review.openstack.org/77713 | 19:35 |
| *** piousbox has joined #heat | 19:35 | |
| *** bada has quit IRC | 19:39 | |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Refactor CLB to work with groups https://review.openstack.org/65586 | 19:39 |
| *** topol has quit IRC | 19:43 | |
| *** wchrisj has quit IRC | 19:47 | |
| *** nszceta has quit IRC | 19:47 | |
| *** DaveJ__ has quit IRC | 19:47 | |
| *** wchrisj has joined #heat | 19:49 | |
| *** sabeen has joined #heat | 19:54 | |
| *** topol has joined #heat | 19:54 | |
| *** kebray has joined #heat | 19:55 | |
| *** andersonvom has quit IRC | 19:56 | |
| stevebaker | zaneb: could you take a look at this bug fix? https://review.openstack.org/#/c/77923/1 | 19:56 |
| stevebaker | skraynev: I'm surprised ^ fixed the port update issue | 19:56 |
| zaneb | stevebaker: will do, thanks | 19:56 |
| *** cadenzajon has quit IRC | 19:57 | |
| stevebaker | zaneb: TC meeting | 19:59 |
| zaneb | ta | 19:59 |
| *** IlyaE has joined #heat | 20:02 | |
| *** kebray_ has joined #heat | 20:02 | |
| *** kebray_ has quit IRC | 20:04 | |
| *** kebray has quit IRC | 20:05 | |
| *** kebray has joined #heat | 20:05 | |
| *** denis_makogon has quit IRC | 20:09 | |
| *** dmakogon_ has joined #heat | 20:09 | |
| randallburt | stevebaker: FWIW, we agreed at the glance mid-summit that application catalog stuff belongs in glance, even for Murano | 20:10 |
| openstackgerrit | A change was merged to openstack/heat: Make Keystone client pluggable https://review.openstack.org/74190 | 20:13 |
| *** shakayumi has quit IRC | 20:17 | |
| jasond | jpeeler: would you mind re-reviewing this? https://review.openstack.org/#/c/77713/ | 20:17 |
| *** shakayumi has joined #heat | 20:19 | |
| *** jistr has quit IRC | 20:20 | |
| sdake | 3 hour check pipeline | 20:22 |
| * sdake ughs | 20:22 | |
| randallburt | its that time of year... | 20:22 |
| sdake | seems to be | 20:22 |
| sdake | jam it all in at the last minute | 20:22 |
| sdake | waiting for my gate check to be merged before submitting my new oslo changeset | 20:22 |
| sdake | been waiting for 3 hours | 20:22 |
| * sdake ughs | 20:22 | |
| sdake | v2 keystone plugin next in line | 20:23 |
| sdake | v2 plugin just failed | 20:25 |
| openstackgerrit | A change was merged to openstack/heat: Add Keystone V2 plugin https://review.openstack.org/74191 | 20:26 |
| openstackgerrit | A change was merged to openstack/heat: Always report the action in state_reason as engine encodes it https://review.openstack.org/77406 | 20:27 |
| sdake | i guess log errors are ok | 20:27 |
| sdake | who would have known :) | 20:27 |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Migrate to oslo.messaging from oslo-incubator rpc https://review.openstack.org/72798 | 20:27 |
| openstackgerrit | Steven Dake proposed a change to openstack/heat: Update config generator for oslo.messaging https://review.openstack.org/77472 | 20:27 |
| *** kebray_ has joined #heat | 20:30 | |
| *** sgordon has quit IRC | 20:33 | |
| *** sgordon has joined #heat | 20:33 | |
| *** kebray has quit IRC | 20:33 | |
| *** topol has quit IRC | 20:36 | |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 20:43 |
| randallburt | arbylee: thanks! will +2 once jenkins does its thing | 20:44 |
| arbylee | randallburt: \o/ | 20:45 |
| stevebaker | sdake: you know you can tail the check logs as they are running? | 20:47 |
| *** cody-somerville has quit IRC | 20:50 | |
| sdake | yup | 20:53 |
| sdake | console in jenkins | 20:54 |
| *** rbuilta has quit IRC | 20:55 | |
| *** asalkeld has quit IRC | 20:55 | |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Allow update of admin_pass on server https://review.openstack.org/72746 | 20:57 |
| *** jmckind has joined #heat | 20:58 | |
| *** cadenzajon has joined #heat | 21:06 | |
| *** kebray_ has quit IRC | 21:08 | |
| *** asalkeld has joined #heat | 21:08 | |
| *** jamieh has quit IRC | 21:09 | |
| sdake_ | jasond is https://blueprints.launchpad.net/heat/+spec/improve-api-polling implemented now that your patches are merged? | 21:09 |
| openstackgerrit | A change was merged to openstack/heat: replace dict comprehension with dict constructor https://review.openstack.org/77834 | 21:12 |
| *** tspatzier has joined #heat | 21:16 | |
| *** bada has joined #heat | 21:17 | |
| *** IlyaE has quit IRC | 21:21 | |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Fix parameter constraints syntax https://review.openstack.org/78005 | 21:23 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: README documentation updates https://review.openstack.org/77779 | 21:23 |
| openstackgerrit | Aaron Weitekamp proposed a change to openstack/heat-templates: Tighten up security group port settings for broker and node. https://review.openstack.org/77778 | 21:23 |
| *** larsks has quit IRC | 21:23 | |
| *** randallburt has quit IRC | 21:25 | |
| *** sabeen has quit IRC | 21:30 | |
| jasond | sdake_: yep, i just marked it implemented. thanks | 21:35 |
| * stevebaker emerges from the meeting tunnel, blinks | 21:40 | |
| stevebaker | will andersonvom be around today? | 21:41 |
| *** randallburt has joined #heat | 21:43 | |
| *** randallburt has quit IRC | 21:43 | |
| *** randallburt has joined #heat | 21:43 | |
| *** saurabhs has quit IRC | 21:43 | |
| *** saurabhs has joined #heat | 21:43 | |
| arbylee | stevebaker: just pinged him for you | 21:44 |
| *** andersonvom has joined #heat | 21:45 | |
| saurabhs | stevebaker: I am trying to use stack-update command to update metadata associated with the instance. | 21:45 |
| saurabhs | I have created a stack with a template which has some parameters. now with stack-update command if I pass it a different value for that parameter, I don't see it getting updated in the DB in stack table for parameters column. is it suppose to get updated? | 21:45 |
| arbylee | stevebaker: he's in the airport. anything I can help with? | 21:45 |
| andersonvom | stevebaker: hey | 21:45 |
| saurabhs | How do I perform a metadata update with heat stack-update? | 21:45 |
| *** randallburt1 has joined #heat | 21:45 | |
| *** randallburt1 has quit IRC | 21:45 | |
| *** randallburt1 has joined #heat | 21:46 | |
| stevebaker | saurabhs: instead of looking in the DB you should state your problem in terms of whether the stack is doing what you want/expect. | 21:46 |
| stevebaker | andersonvom: I'm just thinking if we deferred stack-check now that would free up some reviewing resource for the remaining blueprints. How would you feel about that | 21:47 |
| stevebaker | ? | 21:47 |
| *** achampion has quit IRC | 21:47 | |
| *** randallburt has quit IRC | 21:47 | |
| randallburt1 | andersonvom: may be in the air right now | 21:47 |
| *** randallburt1 is now known as randallburt | 21:47 | |
| randallburt | stevebaker: ^^ | 21:48 |
| andersonvom | stevebaker: I was actually wondering if it would make it today so that we can start implementing the other resources. =P | 21:48 |
| randallburt | andersonvom: wow, internets on airplanes? | 21:48 |
| stevebaker | andersonvom: ok, lets see how it goes | 21:48 |
| randallburt | what a time to be alive | 21:48 |
| stevebaker | feature freeze is in ~12 hours | 21:48 |
| saurabhs | stevebaker: so I have built image using os-apply-config and os-refresh-config, when I use stack-update command with different setup of parameter, which internally are reffered in metadata in the template, I don't see those values getting updated into the instance in my config. | 21:48 |
| randallburt | stevebaker: that means landed, not just in-flight? | 21:49 |
| andersonvom | randallburt: still at the airport, waiting to board. (did not expect port 7000 to be open in the airport network, though :P) | 21:49 |
| stevebaker | saurabhs: could you paste.openstack.org your template? | 21:49 |
| andersonvom | randallburt: you may be the best person to opine on not getting stack-check in. | 21:50 |
| randallburt | tbh, I'd certainly prefer it be in, but I appreciate stevebaker's situation | 21:50 |
| andersonvom | stevebaker: sdake_ and other folks have already given their thumbs up in the past, it's just all the rebasing that's been keeping it unmerged | 21:50 |
| randallburt | andersonvom: the problem is that if its not implemented in the other resources, that patch doesn't do much for me on its own | 21:51 |
| stevebaker | either way, it will get deferred if its not in by FF, since its Medium priority. I'll take a look at it today | 21:51 |
| stevebaker | randallburt, andersonvom, and support for other resource can't land until feature freeze is lifted | 21:51 |
| saurabhs | stevebaker: this is is the template http://paste.openstack.org/show/72326/ and the stack-create command I used was | 21:51 |
| *** radez is now known as radez_g0n3 | 21:51 | |
| saurabhs | heat stack-create Trove-API-2 -f trove.yaml -P "Trove-Image-ID=a8e065cd-5c18-4de7-84c5-c4977138dc68;Trove-Debug=False" | 21:51 |
| andersonvom | =( | 21:52 |
| saurabhs | heat stack-update Trove-API-2 -f trove.yaml -P "Trove-Image-ID=a8e065cd-5c18-4de7-84c5-c4977138dc68;Trove-Debug=True" | 21:52 |
| *** zns has quit IRC | 21:52 | |
| saurabhs | stevebaker: I expected the updated value of Trove-Debug to be reflected in the instance | 21:53 |
| randallburt | stevebaker: support for other resources can still land as part of Icehouse or will it have to wait until Juno opens up? | 21:53 |
| saurabhs | but since I didn't see that being updated I went in the table to check if at least its getting updated in there, but I didin't see that its getting update in the database table (stack) either | 21:54 |
| *** topol has joined #heat | 21:54 | |
| andersonvom | technically they wouldnt be new features, since it's "stack-check not working for resource X", right? | 21:54 |
| stevebaker | saurabhs: you're missing os-collect-config config and the associated user/policy resources in that template. Until SoftwareDeployment resource lands you're only option is to do tripleo-heat-templates style | 21:54 |
| stevebaker | randallburt: smells like a feature | 21:54 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Retry logic for SSH connection in Cloud Server resource https://review.openstack.org/77677 | 21:55 |
| andersonvom | looks like a feature, smells like a feature, acts like a feature? it's a BUG! ;) | 21:55 |
| randallburt | stevebaker: I don't disagree, just wondering if its worth pushing the existing patch without the other resource support. I certainly don't want to be in a position to backport before the actual release. | 21:55 |
| stevebaker | randallburt: so maybe its best to defer now, and feel free to work on the resources during FF | 21:56 |
| randallburt | randallburt: so, my position is this: I'd really like to have stack-check in Icehouse, but would rather it work for every resource rather than just one. So I'm ok with that stevebaker. | 21:57 |
| saurabhs | stevebaker: what is intended use of stack-update, in its current form, is it not supposed to be used for updating metadata? | 21:57 |
| stevebaker | saurabhs: here is everything you would need to get os-collect-config working http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/block-storage.yaml (as well as os-collect-config on your image) | 21:57 |
| randallburt | andersonvom: sound ok? | 21:58 |
| stevebaker | saurabhs: stack-update would work for this, but your template appears to have no mechanism for the server to poll for metadata changes. That is what os-collect-config does | 21:58 |
| andersonvom | randallburt: yeah, I guess you guys are right. we can start working on the other resources, then | 21:59 |
| stevebaker | saurabhs: with blueprint hot-software-config this is all much easier to do, but I'm not sure if you're in the position to wait for that to land | 21:59 |
| shardy | saurabhs: Note you can use heat resource-metadata <stack> <resource> on the CLI to see the metadata update without fishing in the DB | 21:59 |
| randallburt | cool | 22:00 |
| randallburt | thanks andersonvom and stevebaker | 22:00 |
| stevebaker | randallburt, andersonvom, ok, I'll defer and -2 the change to freeze it | 22:00 |
| stevebaker | pscheie_: are you about? | 22:03 |
| pscheie_ | I am, for a couple more minutes | 22:03 |
| saurabhs | stevebaker: I understand the change that you are suggested, I will try that out (by not specifying os-refresh-config I am missing the polling for those metadata field). But just so that I understand this better, I think stack-update did do the metadata update, but where does the updated values gets saved, since I didn't see them in DB? are they directly pushed to the metadata server? | 22:03 |
| *** zns has joined #heat | 22:04 | |
| stevebaker | saurabhs: forget about the DB, try shardy's suggestion to see if the metadata has been updated in heat (heat resource-metadata ...) | 22:05 |
| saurabhs | stevebaker: I tried resource-metadata, I still see the value of Debug parameter to be false, Its not reflecting the updated value for it | 22:07 |
| stevebaker | pscheie_: how would you feel about deferring cancel-update-stack until Juno. It would free up some reviewing resource for the feature freeze | 22:08 |
| *** blomquisg has quit IRC | 22:08 | |
| stevebaker | pscheie_: its hard to tell, but it looks like cancel-update-stack depends on stack-check anyway, which has just been deferred | 22:09 |
| pscheie_ | stevebaker, I'd already mentally assumed it would get pushed to juno, so it's okay by me. | 22:10 |
| pscheie_ | must run. be back tomorrow. | 22:11 |
| stevebaker | pscheie_: heh, I see it has had zero reviewer attention, our bad :{ | 22:11 |
| stevebaker | pscheie_: ok, I'll defer. Thanks for understanding | 22:11 |
| *** rpothier has quit IRC | 22:11 | |
| *** achampion has joined #heat | 22:11 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Resource type implementations for structured software config https://review.openstack.org/74206 | 22:12 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Server property for software config transport https://review.openstack.org/77798 | 22:12 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Remove signal_id from deployments API and model https://review.openstack.org/77799 | 22:12 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: OS::Nova::Server support for software config https://review.openstack.org/67625 | 22:12 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Resource type implementation for software deployment https://review.openstack.org/67624 | 22:12 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Nova server to ref cloud-config resources in user_data https://review.openstack.org/69238 | 22:12 |
| saurabhs | stevebaker shardy: Here is the output of resource-metadata http://paste.openstack.org/show/72342/. This the template http://paste.openstack.org/show/72326/ | 22:13 |
| *** andersonvom has quit IRC | 22:14 | |
| *** IlyaE has joined #heat | 22:14 | |
| *** pablosan is now known as ZZpablosan | 22:15 | |
| stevebaker | saurabhs: OK, I would expect that to work. Could you please raise a bug? | 22:15 |
| stevebaker | saurabhs: and attach all the things | 22:15 |
| saurabhs | stevebaker: ok. Will do. One more question about updating the image on an heat stack. if lets say I want to update the image ID for one or more resources in my heat-stack. should I be using stack-update for that too? and once the update is done, what should I be doing to rebuild those resources using updated image-id | 22:17 |
| stevebaker | saurabhs: changing the image ID in an update will result in the servers being deleted and created with the new ID | 22:18 |
| stevebaker | saurabhs: actually you can set the server property image_update_policy to controll whether to do a rebuild or replace | 22:19 |
| stevebaker | saurabhs: tripleo uses that | 22:19 |
| saurabhs | hmm so that will also result in updated IP of the instance right? do you know how tripleo does this? I thought they do nova reset/rebuild | 22:20 |
| saurabhs | stevebaker: thanks I think you answered that. I will look at image_update_policy and how to set it | 22:20 |
| *** ZZpablosan is now known as pablosan | 22:21 | |
| *** blomquisg has joined #heat | 22:22 | |
| SpamapS | shardy: around? | 22:22 |
| shardy | SpamapS: yup, hi! | 22:23 |
| SpamapS | shardy: so, keystoneclient doesn't have the 'domain create' command | 22:23 |
| shardy | SpamapS: you have to use python-openstackclient | 22:23 |
| SpamapS | shardy: ... that seems a bit unreasonable. :-/ | 22:23 |
| SpamapS | or has keystoneclient been deprecated? | 22:24 |
| shardy | SpamapS: I agree, but the keystone devs are deprecating the keystoneclient CLI | 22:24 |
| stevebaker | SpamapS: keystoneclient doesn't do v3 | 22:24 |
| shardy | personally I think it's a bad decision, and have questioned it, but that is the direction they are headed <shrug> | 22:24 |
| SpamapS | Why isn't this being screamed about whenever I run keystoneclient cli commands? | 22:25 |
| shardy | I have provided the openstackclient commands in the commit messages | 22:25 |
| SpamapS | "WARNING: This is deprecated. You can't use it anymore." | 22:25 |
| shardy | SpamapS: It's all over the keystone logs | 22:25 |
| SpamapS | shardy: yeah we don't have openstackclient in TripleO ... gah | 22:25 |
| stevebaker | messengers! run away! | 22:25 |
| shardy | lol | 22:25 |
| SpamapS | shardy: yes, but that isn't visible to me, the user, running 'keystone xxx' | 22:25 |
| shardy | SpamapS: agree, noisy client warnings would be good | 22:26 |
| shardy | my guess is they plan to add them sometime soon seeing as v2 is deprecated for Icehouse | 22:26 |
| shardy | jamielennox would be the person to ask | 22:26 |
| SpamapS | shardy: many many stalled developers right now because Heat is broken. | 22:27 |
| shardy | SpamapS: I'm not sure what you're saying, how is heat broken? | 22:27 |
| shardy | clearly we need a way for you to install openstackclient | 22:28 |
| shardy | Or a tiny bit of python calling keystoneclient to create the domain and user if that is impossible | 22:28 |
| SpamapS | shardy: heat is requiring config and users and openstackclient , it did not require such things before. | 22:29 |
| SpamapS | shardy: we're up to 4 patches now | 22:29 |
| shardy | SpamapS: Ok, so lets work together to find a solution, this is valuable feedback and I want us to fix it | 22:29 |
| SpamapS | I'm trying hard not to say it should be reverted. I know that is painful.. but if there are any CD Heat shops.. they're broken. | 22:30 |
| *** e0ne has quit IRC | 22:30 | |
| shardy | SpamapS: but these patches have been around for months, along with considerable discussion, and I have been trying to engage you for feedback on impact for tripleo | 22:30 |
| SpamapS | shardy: yes, hence me trying hard to not assign blame as it is just as easily assignable to me as anyone else. :p | 22:31 |
| SpamapS | shardy: But the reality is.. we shouldn't be landing patches that break deployers without a deprecation warning. | 22:32 |
| shardy | SpamapS: So can you explain the barrier to just installing openstackclient, creating the stuff and updating the config? | 22:32 |
| *** lindsayk has joined #heat | 22:32 | |
| SpamapS | shardy: Firedrill vs. "TODO" | 22:32 |
| shardy | SpamapS: this is the bleeding edge of a development branch not a stable release | 22:32 |
| SpamapS | shardy: 2 of us are dropping everything to do the patches. Plus two reviewers having to push these reviews to the top of the stack. | 22:33 |
| SpamapS | shardy: vs. "oh we've got a deprecation warning we need to fix that some time in the next 6 months" | 22:33 |
| *** larsks has joined #heat | 22:33 | |
| SpamapS | shardy: OpenStack supports continuous deployers. We don't throw them under the bus. | 22:33 |
| shardy | SpamapS: I accept tripleo is in a uniquely pain-inflicted position here, you are almost like an extension of the gate, only we have no control or feedback, other than you complaining of course ;D | 22:34 |
| SpamapS | shardy: and even with the stable release scenario, Havana users should be able to upgrade and then deal with deprecation warnings. | 22:34 |
| SpamapS | Otherwise we make the upgrade orchestration more painful, making it harder to stay current. Icehouse in particular is the first release where we're trying hard to guarantee that. | 22:34 |
| shardy | stevebaker: what's your take here - do we revert and I'll come up with some way to gracefully fall back to the previous behavior? | 22:34 |
| SpamapS | shardy: I think at this point, no you don't have to revert. | 22:35 |
| SpamapS | But this must be avoided in the future. | 22:35 |
| SpamapS | And perhaps you can also merge a patch to gracefully fall back before Icehouse release. | 22:36 |
| shardy | SpamapS: Ok, point taken, I'll raise a critical bug and propose a patch tomorrow which will fail back to the previous behaviour if the config options arent there (with warnings in the logs) | 22:36 |
| shardy | would that be OK? | 22:36 |
| shardy | SpamapS: what would it take to provide gate feedback from the tripleo CI infra? | 22:37 |
| stevebaker | I'm ok with reverting if its necessary, but it sounds like its not | 22:37 |
| shardy | SpamapS: do you already have a tripleo bug I can tag onto? | 22:38 |
| stevebaker | shardy: I raised one yesterday | 22:39 |
| *** achampion has quit IRC | 22:39 | |
| SpamapS | shardy: We are on the path of getting tripleo CI into the gate | 22:39 |
| stevebaker | shardy: https://bugs.launchpad.net/bugs/1287453 | 22:39 |
| SpamapS | shardy: we have had reliability issues that we're just now solving.. thus making infra hesitant to depend on us. :) | 22:39 |
| stevebaker | oh, you mean for gating | 22:39 |
| SpamapS | shardy: yes bug 1287453 | 22:40 |
| shardy | SpamapS: even non voting check feedback would be hugely useful in situations like this | 22:40 |
| SpamapS | shardy: and thanks for the quick response. I think that would be fantastic and I really appreciate your position on this. | 22:40 |
| shardy | SpamapS: np, I'm sorry for the inconvenience | 22:40 |
| SpamapS | shardy: right, we are just now able to test our own things non-voting.. then we'll gate our own things and non-vote on OpenStack .. then gate OpenStack.. :-/ | 22:41 |
| stevebaker | I wonder if there is a better way of reconciling the pathologies of time-based releases vs CI | 22:41 |
| *** samstav has quit IRC | 22:41 | |
| *** samstav has joined #heat | 22:42 | |
| shardy | stevebaker: +1 | 22:42 |
| shardy | CI is always going to be a challenge unless you gate on everything | 22:43 |
| SpamapS | shardy: agreed, and we do gate on _most_ things. | 22:43 |
| SpamapS | and realistically, a CD shop will always have their own gate | 22:43 |
| SpamapS | HP Cloud runs a week or two behind master for most services.. | 22:44 |
| SpamapS | but that is because it is Continuous Delivery, not Continuous Deployment .. meaning each change is always _available_ .. but you still only deploy when you want/need to. | 22:44 |
| *** Linz_ has joined #heat | 22:48 | |
| *** giulivo has quit IRC | 22:50 | |
| *** tspatzier has quit IRC | 22:50 | |
| stevebaker | shardy: maybe heat-manage should depend on openstackclient and have a command which checks for values in heat.conf and runs keystone commands to create any missing values. Then it could print out the required heat.conf changes | 22:51 |
| *** Linz has quit IRC | 22:51 | |
| *** aweiteka has quit IRC | 22:52 | |
| *** dims has quit IRC | 22:52 | |
| shardy | stevebaker: well heat-manage would need keystone credentials and would basically do the same job openstackclient already does (which is to talk to the keystoneclient python bindings) | 22:52 |
| stevebaker | shardy: yes, but it could do it with a little more handholding than running openstackclient manually | 22:53 |
| stevebaker | shardy: if it means openstackclient doesn't need to be installed, even better | 22:54 |
| *** nati_ueno has quit IRC | 22:54 | |
| shardy | we could add it to bin/heat-keystone-setup | 22:54 |
| *** mkollaro1 has quit IRC | 22:54 | |
| stevebaker | shardy: only if tripleo already uses bin/heat-keystone-setup. | 22:54 |
| shardy | stevebaker: looks like they have their own pile of shell script: | 22:55 |
| shardy | https://review.openstack.org/#/c/78020/5/scripts/init-keystone | 22:55 |
| stevebaker | shardy: something python based which only does the domain stuff would be useful, since it wouldn't need openstackclient | 22:56 |
| *** blamar has joined #heat | 22:56 | |
| shardy | stevebaker: I can really easily provide a tools/domain_setup.py | 22:56 |
| shardy | which would only depend on keystoneclient | 22:56 |
| *** nati_ueno has joined #heat | 22:56 | |
| shardy | SpamapS: ^^ would that be easier for you guys than installing openstackclient? | 22:57 |
| shardy | (as well as the config fall-back) | 22:57 |
| *** cfriesen_ has quit IRC | 22:58 | |
| *** jasond has quit IRC | 22:59 | |
| *** yogesh has quit IRC | 23:00 | |
| *** rollerj has quit IRC | 23:00 | |
| *** rpothier has joined #heat | 23:01 | |
| SpamapS | shardy: yes that would actually be better | 23:06 |
| *** slagle has joined #heat | 23:06 | |
| *** Michalik has quit IRC | 23:08 | |
| *** dims has joined #heat | 23:08 | |
| *** shakayumi has quit IRC | 23:14 | |
| *** blomquisg has quit IRC | 23:15 | |
| *** zns has quit IRC | 23:19 | |
| *** pablosan is now known as ZZpablosan | 23:23 | |
| *** Tross has joined #heat | 23:24 | |
| *** Michalik has joined #heat | 23:25 | |
| *** jergerber has quit IRC | 23:25 | |
| *** rcleere has quit IRC | 23:25 | |
| *** lindsayk has quit IRC | 23:28 | |
| *** blomquisg has joined #heat | 23:29 | |
| *** randallburt has quit IRC | 23:30 | |
| *** samstav has quit IRC | 23:31 | |
| *** samstav has joined #heat | 23:31 | |
| *** radez_g0n3 is now known as radez | 23:33 | |
| *** david-lyle has quit IRC | 23:34 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Allow getting attributes on suspended resources. https://review.openstack.org/77452 | 23:37 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Allow handle_signal to assert which actions are valid https://review.openstack.org/77453 | 23:37 |
| *** ZZpablosan is now known as pablosan | 23:38 | |
| openstackgerrit | A change was merged to openstack/heat: Give cfn a separate Template class https://review.openstack.org/74589 | 23:40 |
| openstackgerrit | A change was merged to openstack/heat: Calculate template version only once https://review.openstack.org/77751 | 23:40 |
| *** vijendar has quit IRC | 23:41 | |
| *** samstav has quit IRC | 23:41 | |
| *** kebray has joined #heat | 23:45 | |
| openstackgerrit | A change was merged to openstack/heat: Add a plugin_manager module https://review.openstack.org/74590 | 23:46 |
| openstackgerrit | A change was merged to openstack/heat: Move the code that reads the global environment https://review.openstack.org/76323 | 23:46 |
| openstackgerrit | A change was merged to openstack/heat: Docs: use the plugin manager to list resources https://review.openstack.org/76324 | 23:46 |
| *** zns has joined #heat | 23:47 | |
| *** blamar has quit IRC | 23:48 | |
| *** blamar has joined #heat | 23:49 | |
| *** kebray has quit IRC | 23:57 | |
| *** IlyaE has quit IRC | 23:58 | |
| *** tomek_adamczewsk has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!