| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/105166 | 00:06 |
|---|---|---|
| lifeless | yes | 00:09 |
| *** achampion has joined #heat | 00:11 | |
| *** wirehead_ has joined #heat | 00:12 | |
| zaneb | lifeless: so in the former case you pass in a fixed IP for the port as a parameter, and in the latter case you leave it out | 00:20 |
| zaneb | as long as the address range for the dynamic IPs doesn't overlap the static IPs, there's no race afaict | 00:21 |
| zaneb | is it a question of how you can either specify a static IP or not specify a static IP using a single template? | 00:22 |
| lifeless | zaneb: I believe you are describing the 'use two neutron networks' approach I mentioned above | 00:22 |
| lifeless | zaneb: because to neutron there is no such thing as a static ip range | 00:22 |
| lifeless | zaneb: remember it doesn't do true DHCP - it has no wildcard facility | 00:23 |
| lifeless | its static-but-managed-via-API | 00:23 |
| *** piyush has quit IRC | 00:27 | |
| zaneb | there's no such thing as a static IP range, but there are ranges in the subnet that are not covered by a dynamic pool | 00:28 |
| zaneb | and you can assign a fixed IP in those ranges without risking a race with a dynamic assignment | 00:28 |
| zaneb | obviously there can still be a race between two people assigning the same fixed IP | 00:29 |
| zaneb | but you said there were other processes in place (i.e. spreadsheet) to prevent that | 00:29 |
| *** harlowja_away is now known as harlowja | 00:30 | |
| lifeless | does neutron permit that, huh | 00:30 |
| lifeless | I shall have to experiment | 00:30 |
| zaneb | so I don't think you _need_ a second subnet, although a second subnet would work just as well if you can handle the IPs being in different cidrs | 00:30 |
| lifeless | silly me thought that the allocation pool would be binding | 00:31 |
| lifeless | second subnet *object*, not second cidr | 00:31 |
| lifeless | neutron has no concept that network A and network B with subnet A' and B' on the same CIDR is problem | 00:31 |
| lifeless | because namespacing | 00:31 |
| zaneb | lifeless: the allocation pool is binding, but it doesn't have to cover the whole cidr of the subnet afaik | 00:31 |
| lifeless | it'll whinge like mad if you hook up a router thats incompatible though | 00:32 |
| lifeless | zaneb: ah so if the allocation pool is binding | 00:32 |
| lifeless | zaneb: then what you're saying makes no sense to me | 00:32 |
| zaneb | ok, wait | 00:32 |
| zaneb | what does binding mean? | 00:32 |
| lifeless | zaneb: the dynamically allocated addresses come from the allocation pool | 00:32 |
| zaneb | correct | 00:32 |
| lifeless | zaneb: restricts creation of ports to addresses in the allocation pool, whether manually specified or not. | 00:33 |
| zaneb | OK, so your thesis is that Neutron would not allow you to use a Fixed IP that is not in an allocation pool | 00:34 |
| zaneb | which _may_ be correct (I haven't tried) | 00:34 |
| zaneb | but if it is correct then Neutron is craaaazy | 00:34 |
| zaneb | (which would not be inconsistent with my past experience of it ;) | 00:35 |
| lifeless | well | 00:37 |
| lifeless | it depends what the intent of the allocation pool is | 00:37 |
| lifeless | well | 00:38 |
| lifeless | it permits it, so thats easy. | 00:38 |
| zaneb | right, if the pool is "addresses which we will serve up" then your interpretation is correct. If it means "addresses from which we'll select dynamic IPs" then my interpretation is correct | 00:38 |
| lifeless | zaneb: thank you | 00:38 |
| zaneb | sweet | 00:38 |
| zaneb | lifeless: btw, I'm curious which part of the VRRP spec is ambiguous? | 00:40 |
| * zaneb used to work on such things | 00:40 | |
| lifeless | so, it may not be, I haven't gotten all the way down to the bottom yet | 00:40 |
| lifeless | in the examples they give | 00:40 |
| lifeless | they have router 1 with IP A and router two with IP B, and 2 configured to take over A if 1 goes awol | 00:41 |
| lifeless | its not clear if that depends on router 2 *having IP B at all*. | 00:41 |
| lifeless | e.g. can one run two routers in a VRRP group with only multicast, sharing a single IP between them and no other IP host addresses on their interfaces | 00:42 |
| zaneb | hmm, memory is a bit hazy | 00:42 |
| zaneb | quite commonly you assign a secondary, fixed IP to *both* routers | 00:43 |
| lifeless | yeah | 00:46 |
| lifeless | fixed ips are expensive :) | 00:46 |
| lifeless | this will get much more fun btw | 00:47 |
| lifeless | when we get vlan support into ironic | 00:47 |
| lifeless | actually no, a related thing gets more fun :) | 00:47 |
| lifeless | but that a discussion for a different day | 00:47 |
| * zaneb can't wait :D | 00:49 | |
| *** piyush has joined #heat | 00:50 | |
| *** dims__ has joined #heat | 00:58 | |
| zaneb | lifeless: I took a look at our old docs, and all of the examples show either the router owning the VR's IP or having it's own separate IP, like you said above | 01:01 |
| zaneb | lifeless: however the VRRP protocol itself relies only on IP multicast | 01:02 |
| zaneb | so there's no reason in principle that should be required | 01:02 |
| zaneb | (though there might be other reasons for it to be useful) | 01:02 |
| *** dims__ has quit IRC | 01:03 | |
| lifeless | zaneb: thanks for digging! | 01:05 |
| lifeless | hmm | 01:07 |
| lifeless | totally unrelated thing | 01:07 |
| lifeless | well not totally | 01:07 |
| lifeless | passing fixed_ips in | 01:07 |
| lifeless | we've set the type of the parameter to Json | 01:08 |
| zaneb | lifeless: actually a closer reading of the RFC suggests that the Source IP of the advertisements has to be "the primary IPv4 address of the interface the packet is being sent from." | 01:08 |
| lifeless | is there a type where we can just use regular yaml in the env file rather than serailising it | 01:08 |
| lifeless | e.g. this | 01:09 |
| lifeless | { "parameters": { "PublicVirtualFixedIPs": [{"ip_address": "138.35.77.6"}], } } | 01:09 |
| lifeless | rather than | 01:09 |
| zaneb | lifeless: good question. try it and see? | 01:09 |
| lifeless | { "parameters": { "PublicVirtualFixedIPs": [{"ip_address": "138.35.77.6"}], } } | 01:09 |
| lifeless | bad | 01:09 |
| lifeless | { "parameters": { "PublicVirtualFixedIPs": "[{\"ip_address\": \"138.35.77.6\"}]", } } | 01:09 |
| lifeless | zaneb: I know it doesn't work, I was wondering if there was a parameter type I was missing where it would work | 01:11 |
| zaneb | lifeless: it should work passing an actual JSON object, not just JSON serialised as a string | 01:11 |
| zaneb | http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/parameters.py#n368 | 01:11 |
| zaneb | aha, but only if it's an object, not a list :/ | 01:11 |
| lifeless | za+ heat stack-create -e /home/shared/overcloud-env.json -t 360 -f /home/shared/tripleo-heat-templates/overcloud.yaml -P ExtraConfig= ci-overcloud | 01:12 |
| lifeless | ERROR: Value must be valid JSON: expected string or buffer | 01:12 |
| lifeless | + RESULT=1 | 01:12 |
| lifeless | want a bug for that ? | 01:12 |
| zaneb | yeah, I think we may even have one | 01:13 |
| zaneb | the initial idea was that the comma-separated-list type would do for lists and Json would do objects only | 01:13 |
| zaneb | that didn't work out | 01:13 |
| zaneb | but the changes to correct that mistake are unfortunately quite deep | 01:14 |
| *** piyush has quit IRC | 01:17 | |
| lifeless | we can't just accept anything made up only of basic types? | 01:22 |
| lifeless | e.g. string-> decode, list|dict -> object? | 01:22 |
| lifeless | anyhow let me see about a bug | 01:22 |
| *** pmyers has quit IRC | 01:27 | |
| *** kebray has joined #heat | 01:43 | |
| openstackgerrit | Takashi NATSUME proposed a change to openstack/heat: Add log output about HTTP calls in cinderclient https://review.openstack.org/104745 | 01:47 |
| tiantian | <zaneb>hi, would you please to review https://review.openstack.org/#/c/104439/, if you have time, tks | 01:55 |
| *** nosnos has joined #heat | 01:58 | |
| *** gokrokve has joined #heat | 02:01 | |
| *** Yanyanhu has joined #heat | 02:08 | |
| elynn | morning all :0 | 02:28 |
| *** piyush has joined #heat | 02:38 | |
| *** elynn has quit IRC | 02:42 | |
| lifeless | zaneb: couldn't see one so filed https://bugs.launchpad.net/heat/+bug/1342487 | 02:42 |
| *** piyush has quit IRC | 02:42 | |
| uvirtbot | Launchpad bug 1342487 in heat "json parameters require json strings rather than accepting arbitrary json structures" [Undecided,New] | 02:42 |
| *** gokrokve has quit IRC | 02:49 | |
| *** dims__ has joined #heat | 02:52 | |
| *** gokrokve has joined #heat | 02:56 | |
| *** nati_ueno has quit IRC | 02:57 | |
| *** ramishra has joined #heat | 02:59 | |
| *** KanagarajM has joined #heat | 03:00 | |
| *** daneyon has joined #heat | 03:00 | |
| *** daneyon has quit IRC | 03:00 | |
| *** daneyon has joined #heat | 03:01 | |
| ramishra | morning all! | 03:02 |
| *** cmyster has joined #heat | 03:10 | |
| cmyster | morning | 03:14 |
| *** harlowja is now known as harlowja_away | 03:16 | |
| Yanyanhu | morning | 03:27 |
| *** Yanyan has joined #heat | 03:29 | |
| *** Yanyanhu has quit IRC | 03:29 | |
| *** gokrokve has quit IRC | 03:30 | |
| *** piyush has joined #heat | 03:31 | |
| *** chandankumar has joined #heat | 03:35 | |
| *** saju_m has joined #heat | 03:38 | |
| *** nosnos has quit IRC | 03:43 | |
| *** bmahalakshmi has joined #heat | 04:01 | |
| *** EricGonczer_ has joined #heat | 04:01 | |
| *** EricGonczer_ has quit IRC | 04:07 | |
| *** dims__ has quit IRC | 04:10 | |
| *** kevinbenton has joined #heat | 04:13 | |
| *** Yanyan has quit IRC | 04:18 | |
| *** Yanyanhu has joined #heat | 04:28 | |
| *** killer_prince is now known as lazy_prince | 04:31 | |
| *** nosnos has joined #heat | 04:32 | |
| *** slok184 has joined #heat | 04:33 | |
| openstackgerrit | A change was merged to openstack/heat: Fix error count for stack-list while show deleted https://review.openstack.org/101078 | 04:38 |
| *** saju_m has quit IRC | 04:41 | |
| *** Yanyan has joined #heat | 04:43 | |
| *** ramishra has quit IRC | 04:43 | |
| *** ramishra_ has joined #heat | 04:44 | |
| *** kopparam has joined #heat | 04:46 | |
| *** Yanyanhu has quit IRC | 04:46 | |
| *** lindsayk has joined #heat | 04:47 | |
| *** lindsayk has quit IRC | 04:58 | |
| *** piyush has quit IRC | 05:02 | |
| *** bnemec has quit IRC | 05:03 | |
| *** sab has joined #heat | 05:04 | |
| openstackgerrit | Jeff Peeler proposed a change to openstack/heat-templates: Add HA templates to deploy OpenShift on CentOS https://review.openstack.org/106159 | 05:05 |
| *** nkhare has joined #heat | 05:06 | |
| *** chandankumar has quit IRC | 05:16 | |
| *** rakesh_hs has joined #heat | 05:20 | |
| *** jreypo has quit IRC | 05:33 | |
| *** rushiagr has joined #heat | 05:37 | |
| *** andreaf has quit IRC | 05:37 | |
| *** cmyster has quit IRC | 05:38 | |
| *** dims__ has joined #heat | 05:39 | |
| *** cmyster has joined #heat | 05:41 | |
| *** dims__ has quit IRC | 05:44 | |
| *** akuznetsov has quit IRC | 05:47 | |
| *** samstav_ has quit IRC | 05:47 | |
| *** Qiming has joined #heat | 05:48 | |
| *** slok184 has quit IRC | 05:54 | |
| *** chandankumar has joined #heat | 05:58 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/104975 | 06:00 |
| *** bvivek has joined #heat | 06:03 | |
| *** akuznetsov has joined #heat | 06:07 | |
| *** jyoti-ranjan has joined #heat | 06:08 | |
| *** unmeshg has joined #heat | 06:25 | |
| *** nati_ueno has joined #heat | 06:26 | |
| *** avquadri has joined #heat | 06:26 | |
| unmeshg | Hi, reviews required for https://review.openstack.org/#/c/104471/ | 06:27 |
| *** tomek_adamczewsk has joined #heat | 06:37 | |
| skraynev | good morning | 06:37 |
| cmyster | morning | 06:37 |
| *** jcoufal has joined #heat | 06:38 | |
| *** tspatzier has joined #heat | 06:39 | |
| *** dims__ has joined #heat | 06:40 | |
| *** dims__ has quit IRC | 06:45 | |
| *** jstrachan has joined #heat | 06:54 | |
| *** jreypo has joined #heat | 06:57 | |
| *** jreypo has quit IRC | 06:58 | |
| *** pasquier-s has joined #heat | 07:02 | |
| *** d0ugal has quit IRC | 07:07 | |
| *** d0ugal has joined #heat | 07:07 | |
| *** elynn has joined #heat | 07:09 | |
| Qiming | to whomever may concerns, good morning | 07:11 |
| cmyster | indeed | 07:12 |
| *** zslot has joined #heat | 07:18 | |
| therve | Good morning! | 07:20 |
| Qiming | skraynev, a quick question | 07:26 |
| skraynev | Qiming: I am ready :) | 07:27 |
| Qiming | I just found that you proposed a change to attributes.py back in April | 07:27 |
| Qiming | skraynev, it was about adding Schema class to Attribute | 07:27 |
| Qiming | however, I cannot find a record when this patch landed here: https://github.com/openstack/heat/commits/stable/icehouse | 07:28 |
| skraynev | Qiming: Hm. May it was merged later? | 07:30 |
| Qiming | I'm confused about how patches are merged into stable/icehouse ... do you have any hints on that? | 07:31 |
| skraynev | Qiming: according to https://review.openstack.org/#/c/89331/ it was merged at the end of May | 07:32 |
| therve | Qiming, https://wiki.openstack.org/wiki/StableBranch may explain a bit | 07:33 |
| Qiming | skraynev, does that mean it missed the Icehouse release? | 07:33 |
| *** jistr has joined #heat | 07:33 | |
| skraynev | Qiming: About patches in stable/icehouse, AFAIK, related bug should be marked as backport potential and then you may cherry-pick you patch from master to needed stable branch | 07:34 |
| skraynev | Qiming: I suppose - yes. | 07:34 |
| Qiming | skraynev, so ... if I'm gonna use attribute schema, I can only try Juno versions instead of the official icehouse release, | 07:35 |
| Qiming | okay, that solves my problem, thx | 07:35 |
| *** cody-somerville has joined #heat | 07:36 | |
| skraynev | Qiming: a little note, if you want to use attribute schema in the CURRENT state. The old attribute schema (just with description) is still available in icehouse | 07:36 |
| Qiming | skraynev, okay. I may try some workarounds if needed, though I prefer the current implementation. | 07:37 |
| skraynev | Qiming: sure, it's your choice :) | 07:38 |
| *** kebray has quit IRC | 07:38 | |
| Qiming | skraynev, according to https://review.openstack.org/#/c/86525/, the Schema class was merged in late April | 07:39 |
| Qiming | ah, it also missed April 17th, the deadline of Icehouse. | 07:40 |
| *** bvandenh has quit IRC | 07:40 | |
| skraynev | Qiming: right | 07:40 |
| *** ramishra_ has quit IRC | 07:42 | |
| *** ckmvishnu has joined #heat | 07:44 | |
| *** bvandenh has joined #heat | 07:53 | |
| *** ChanServ changes topic to "support @ https://ask.openstack.org | developer wiki @ https://wiki.openstack.org/wiki/Heat | development @ https://launchpad.net/heat | logged @ http://eavesdrop.openstack.org/irclogs/%23heat/" | 07:53 | |
| *** nati_ueno has quit IRC | 07:56 | |
| *** achampion has quit IRC | 08:06 | |
| *** derekh_ has joined #heat | 08:10 | |
| *** alexpilotti has joined #heat | 08:13 | |
| shardy | morning all | 08:29 |
| shardy | asalkeld: around? | 08:29 |
| *** pas-ha has joined #heat | 08:32 | |
| *** achampion has joined #heat | 08:32 | |
| therve | shardy, Hi, I'd like to chat about https://review.openstack.org/#/c/104745/ when you have 5 | 08:37 |
| shardy | therve: Hi, sure | 08:38 |
| therve | shardy, I feel that adding options for this is a waste. We could just reuse the debug flag | 08:38 |
| therve | Apparently we already started adding those though | 08:39 |
| *** zslot1 has joined #heat | 08:40 | |
| shardy | therve: I don't have a strong opinion on it tbh - do we want every client logging verbose request information when the global debug flag is on? | 08:47 |
| therve | I'd want that | 08:48 |
| shardy | Having some control to selectively enable verbose information for a particular client seems like it could be useful | 08:48 |
| shardy | I wonder if oslo logging allows log-levels instead of booleans? | 08:48 |
| shardy | That sounds like the best solution | 08:48 |
| therve | I'm not sure why you'd want to log cinder requests, but not say glance requests | 08:49 |
| shardy | therve: well maybe you're debugging a specific regression in cinder? | 08:49 |
| therve | But fair enough | 08:49 |
| therve | I'd just want to be conservative with new options, we have tons that nobody use | 08:50 |
| shardy | I guess there's also the log santisation aspect - we probably already log some sensitive data at debug level, but if we start logging every raw request, then we *definitely* will | 08:50 |
| therve | Also if I need to debug a cinder issue I can set a flag in code instead of changing configuration | 08:51 |
| therve | That's what debug is :) | 08:51 |
| shardy | Not if you're debugging a platform on a customer site in their datacentre ;) | 08:51 |
| *** zslot1 has quit IRC | 08:53 | |
| *** ishant has joined #heat | 08:59 | |
| *** cody-somerville has quit IRC | 08:59 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Refactor service readiness notification https://review.openstack.org/92456 | 09:03 |
| *** bvivek has quit IRC | 09:05 | |
| ishant | Hi, Can you please have a look at the review https://review.openstack.org/#/c/105408/ | 09:05 |
| *** bvivek has joined #heat | 09:06 | |
| *** achampion has quit IRC | 09:08 | |
| pas-ha | morning all | 09:09 |
| *** bvivek has quit IRC | 09:10 | |
| *** bvivek has joined #heat | 09:12 | |
| therve | ishant, Shouldn't it be named "subnets" if it's a list? | 09:12 |
| asalkeld | hi sha | 09:14 |
| asalkeld | hi shardy | 09:14 |
| shardy | asalkeld: Hey | 09:15 |
| shardy | asalkeld: I was just pinging you re your mail about updates/trusts | 09:15 |
| asalkeld | you get my email? | 09:15 |
| asalkeld | yeah | 09:15 |
| asalkeld | that's a pain | 09:15 |
| shardy | I've replied now, will look at it today | 09:15 |
| ishant | therve, what should be named as "subnets"? | 09:15 |
| asalkeld | thanks shardy | 09:16 |
| therve | ishant, The new attribute | 09:16 |
| therve | subnet implies that it's one subnet | 09:16 |
| shardy | asalkeld: we shouldn't really be creating multiple trusts like that, I was aware of the issue if the update created a new nested stack, but I didn't realize it also happened for backup stack | 09:16 |
| therve | If it's a list it should be named "subnets" | 09:16 |
| ishant | yeah, i will change that | 09:16 |
| shardy | asalkeld: The other thing was the keystone chained delegation thing, do you have any bandwidth to help with that? | 09:17 |
| asalkeld | shardy, mmmm - not sure | 09:17 |
| shardy | asalkeld: due to other pressures, I've had to deprioritize it a bit | 09:17 |
| asalkeld | depends on the time it would take | 09:17 |
| asalkeld | shardy, it's ok if the workaround can function | 09:17 |
| shardy | asalkeld: Does the create empty stack workaround solve your immediate problem (if we fix this update issue)? | 09:17 |
| asalkeld | shardy, yip | 09:18 |
| *** bvandenh has quit IRC | 09:18 | |
| *** sab has quit IRC | 09:18 | |
| shardy | asalkeld: Ok cool, it's still something I'm interested in doing for Juno, just under pressure do deliver some other stuff first ;) | 09:18 |
| shardy | s/do/to | 09:18 |
| asalkeld | shardy, ok | 09:18 |
| *** sab has joined #heat | 09:19 | |
| *** andreaf has joined #heat | 09:19 | |
| asalkeld | shardy, are you getting sucked into the convergence stuff? | 09:19 |
| shardy | asalkeld: possibly, initially I'm going to spend some time on bugs and autoscaling features which are important to TripleO and Tuskar | 09:20 |
| shadower | \o/ | 09:20 |
| asalkeld | ok | 09:20 |
| unmeshg | therve: regarding the wait being skipped between first and second step (review https://review.openstack.org/#/c/104471/) | 09:24 |
| unmeshg | please let me know if it sounds correct, else I'l submit a patch with the sleep at the beginning of the run_to_completion method | 09:26 |
| *** achampion has joined #heat | 09:27 | |
| *** andreaf has quit IRC | 09:27 | |
| *** andreaf has joined #heat | 09:28 | |
| therve | unmeshg, It's technically correct. | 09:28 |
| therve | The best kind of correct. | 09:28 |
| *** dims__ has joined #heat | 09:29 | |
| unmeshg | therve: thanks | 09:29 |
| *** Yanyan has quit IRC | 09:30 | |
| *** achampion has quit IRC | 09:32 | |
| *** sab has quit IRC | 09:33 | |
| *** dims__ has quit IRC | 09:33 | |
| *** rushiagr_ has joined #heat | 09:33 | |
| *** rushiagr has quit IRC | 09:33 | |
| *** cdent has joined #heat | 09:36 | |
| *** Qiming has quit IRC | 09:40 | |
| *** cody-somerville has joined #heat | 09:46 | |
| *** dims__ has joined #heat | 09:48 | |
| openstackgerrit | unmesh-gurjar proposed a change to openstack/heat: Fixed wait skipped after 1st step of task https://review.openstack.org/104471 | 09:57 |
| *** rushiagr_ has quit IRC | 09:58 | |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Move the Stack class into stack.py https://review.openstack.org/107290 | 09:59 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Use the template module to access the Template class https://review.openstack.org/107291 | 09:59 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Import the stack module directly https://review.openstack.org/107292 | 09:59 |
| unmeshg | shardy, therve: I've just submitted new patch with updated commit message to https://review.openstack.org/#/c/104471/ | 09:59 |
| therve | Thanks | 10:00 |
| *** kopparam has quit IRC | 10:01 | |
| *** unmeshg_ has joined #heat | 10:04 | |
| *** unmeshg has quit IRC | 10:04 | |
| *** Dev_Jin has joined #heat | 10:07 | |
| *** alexpilotti has quit IRC | 10:09 | |
| Dev_Jin | I am trying to get servers up using heat.. I have a requirement where I need to put one of IP assigned to a server into other server’s particular file (say /etc/hosts) .. this IP addresses is assigned by dhcp which is changing for every boot.. how do I refer to current IP address assigned to a server in user_data of another server property ? | 10:10 |
| therve | Dev_Jin, Something like that: https://github.com/openstack/heat-templates/blob/master/hot/servers_in_existing_neutron_net.yaml#L91 | 10:12 |
| therve | You can retrieve the proper attribute and then use it in your userdata | 10:13 |
| Dev_Jin | therve: can’t I refer the attribute directly in user_data using getattr:: ? | 10:14 |
| therve | Sure you can | 10:15 |
| shardy | tiantian: Hey, I just tested with an empty string keystone password, and via python-openstackclient and the v3 keystone API it doesn't work | 10:16 |
| shardy | So I stand by my comment that we should not allow folks to do insane things like set passwords to "" | 10:16 |
| shardy | It's not even possible in the current code, unless someone introduces a horrible regression in future | 10:17 |
| *** sab has joined #heat | 10:17 | |
| shardy | maybe I'll add a comment to the code explaining why the test is not is not None | 10:18 |
| Dev_Jin | therve: What’s happening is I am doing echo getatrr append it to file.. well it appending getattr as text and not the actual Ip that I am interested in? | 10:18 |
| *** ishant has quit IRC | 10:20 | |
| therve | Dev_Jin, Can you show your template? | 10:20 |
| *** lazy_prince is now known as killer_prince | 10:20 | |
| Dev_Jin | user_data_format: RAW | 10:26 |
| Dev_Jin | user_data: | | 10:26 |
| Dev_Jin | #!/bin/bash | 10:26 |
| Dev_Jin | sudo sh -c 'echo {get_attr: [init_vm,first_address]}" salt" >> /etc/hosts' | 10:26 |
| therve | Dev_Jin, That's not how templating works | 10:26 |
| therve | Dev_Jin, You need something like that: https://github.com/openstack/heat-templates/blob/master/hot/autoscaling.yaml#L72 | 10:27 |
| *** achampion has joined #heat | 10:28 | |
| Dev_Jin | therve : Ok I got it what I need to do.. thanks a lot.. | 10:29 |
| therve | np! | 10:29 |
| *** Qiming has joined #heat | 10:31 | |
| *** killer_prince is now known as lazy_prince | 10:32 | |
| *** dsneddon has quit IRC | 10:32 | |
| *** achampion has quit IRC | 10:32 | |
| *** andreaf has quit IRC | 10:32 | |
| *** cody-somerville has quit IRC | 10:33 | |
| *** andreaf has joined #heat | 10:34 | |
| *** nkhare has quit IRC | 10:36 | |
| *** akuznetsov has quit IRC | 10:40 | |
| *** nkhare has joined #heat | 10:40 | |
| cmyster | shardy: hey, here? | 10:43 |
| cmyster | or at least, there? | 10:44 |
| shardy | cmyster: Yup | 10:44 |
| cmyster | I want your perspective on something since I know you prefer to illuminate code duplication... | 10:46 |
| *** kopparam has joined #heat | 10:46 | |
| *** nkhare has quit IRC | 10:47 | |
| cmyster | I have those delete_config and deploy and I am moving the 204 test to the client, and that got me thinking, I can just move them both to a single _delete_resource | 10:47 |
| cmyster | but then I'd need to start passing parameters | 10:47 |
| *** pmyers has joined #heat | 10:47 | |
| cmyster | so will two diufferent functions with less parameters or a single one but then I need to make sure I pass the right thing | 10:48 |
| shardy | cmyster: IMO it's better to stick to functions per resource type, because the assertions and return codes expected are not always the same | 10:48 |
| Dev_Jin | I am seeing some weired behavior for some time.. I am associating key with a server instance.. if I don’t specify user_data_format : RAW then login doesn’t work.. but I specify user_data_format:RAW then same key works.. (i am only specfying data_format , user_data is empty) | 10:48 |
| cmyster | shardy: when does a 204 bad for a normal delete ? | 10:49 |
| shardy | cmyster: bug #1298518 | 10:50 |
| uvirtbot | Launchpad bug 1298518 in heat "Abandon and Delete API status codes are wrong" [Medium,Triaged] https://launchpad.net/bugs/1298518 | 10:50 |
| cmyster | cor! | 10:50 |
| cmyster | note to self, clicking on 'links' that start with a # is silly | 10:50 |
| cmyster | Hmmm | 10:52 |
| cmyster | thanks shardy | 10:52 |
| *** akuznetsov has joined #heat | 10:56 | |
| *** piyush has joined #heat | 11:05 | |
| *** piyush has quit IRC | 11:05 | |
| *** bmahalakshmi has quit IRC | 11:08 | |
| *** nkhare has joined #heat | 11:08 | |
| *** nkhare_ has joined #heat | 11:09 | |
| *** piyush has joined #heat | 11:09 | |
| *** dims__ has quit IRC | 11:10 | |
| *** dims__ has joined #heat | 11:13 | |
| *** akuznetsov has quit IRC | 11:20 | |
| openstackgerrit | A change was merged to openstack/heat: Add missing DeprecationWarning category to warning https://review.openstack.org/106990 | 11:23 |
| *** akuznetsov has joined #heat | 11:23 | |
| *** bvivek has quit IRC | 11:26 | |
| *** robsparker has quit IRC | 11:27 | |
| *** cody-somerville has joined #heat | 11:28 | |
| *** achampion has joined #heat | 11:28 | |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Import the stack module directly https://review.openstack.org/107292 | 11:32 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Move the Stack class into stack.py https://review.openstack.org/107290 | 11:32 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Use the template module to access the Template class https://review.openstack.org/107291 | 11:32 |
| *** KanagarajM has quit IRC | 11:32 | |
| *** achampion has quit IRC | 11:34 | |
| *** rushiagr_ has joined #heat | 11:43 | |
| *** robsparker has joined #heat | 11:43 | |
| *** kopparam has quit IRC | 11:43 | |
| *** kopparam has joined #heat | 11:44 | |
| *** rushiagr_ has quit IRC | 11:48 | |
| *** rushiagr_ has joined #heat | 11:52 | |
| *** stannie has joined #heat | 11:55 | |
| *** Yanyan has joined #heat | 11:59 | |
| *** sab has quit IRC | 12:02 | |
| *** achampion has joined #heat | 12:03 | |
| *** cmyster has quit IRC | 12:06 | |
| *** unmeshg_ has quit IRC | 12:07 | |
| *** achampion has quit IRC | 12:08 | |
| *** bvivek has joined #heat | 12:08 | |
| *** jdob has joined #heat | 12:09 | |
| *** sgordon has joined #heat | 12:14 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: SoftwareDeployments move signal reason into resource plugin https://review.openstack.org/106825 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert CFN waitcondition API to resource_signal https://review.openstack.org/106824 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Allow plugins to specify signal event reason https://review.openstack.org/106821 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Update test_signal to use stub_keystoneclient https://review.openstack.org/106820 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Return metadata from resource_signal RPC interface https://review.openstack.org/106823 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: WaitConditionHandle resources, create events with reason data https://review.openstack.org/106822 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 12:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Mark metadata_update RPC call deprecated https://review.openstack.org/106985 | 12:16 |
| *** rushiagr__ has joined #heat | 12:18 | |
| *** rushiagr_ has quit IRC | 12:18 | |
| *** che-arne has joined #heat | 12:19 | |
| *** EricGonczer_ has joined #heat | 12:19 | |
| *** rushiagr has joined #heat | 12:24 | |
| *** tonisbones has joined #heat | 12:24 | |
| *** rushiagr__ has quit IRC | 12:25 | |
| *** EricGonczer_ has quit IRC | 12:25 | |
| *** ramishra has joined #heat | 12:27 | |
| *** dims__ has quit IRC | 12:28 | |
| *** kopparam has quit IRC | 12:29 | |
| *** sergmelikyan has quit IRC | 12:29 | |
| *** dims__ has joined #heat | 12:30 | |
| *** kopparam has joined #heat | 12:33 | |
| *** bvandenh has joined #heat | 12:34 | |
| *** zslot has left #heat | 12:44 | |
| *** sballe has joined #heat | 12:44 | |
| *** lipinski1 has joined #heat | 12:44 | |
| *** nosnos has quit IRC | 12:49 | |
| *** asalkeld has quit IRC | 12:50 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement update of AWS::EC2::SecurityGroup https://review.openstack.org/88548 | 12:51 |
| *** cdent_ has joined #heat | 12:54 | |
| *** cdent has quit IRC | 12:55 | |
| *** cdent_ is now known as cdent | 12:55 | |
| *** plomakin has quit IRC | 12:56 | |
| *** kopparam has quit IRC | 12:57 | |
| *** rushiagr has quit IRC | 12:59 | |
| *** rushiagr has joined #heat | 12:59 | |
| *** lindsayk has joined #heat | 13:01 | |
| *** radez_g0n3 is now known as radez | 13:02 | |
| *** kopparam has joined #heat | 13:06 | |
| *** lindsayk has quit IRC | 13:06 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Move size update to Cinder volume https://review.openstack.org/106622 | 13:10 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Move VolumeAttachment updates to Cinder resource https://review.openstack.org/106623 | 13:10 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Unittests: Refactor Volume resources tests https://review.openstack.org/107362 | 13:10 |
| *** nkhare has quit IRC | 13:12 | |
| *** nkhare_ has quit IRC | 13:12 | |
| *** ckmvishnu has quit IRC | 13:13 | |
| *** rushiagr_ has joined #heat | 13:16 | |
| *** lipinski1 has quit IRC | 13:18 | |
| *** rushiagr has quit IRC | 13:20 | |
| *** nkhare has joined #heat | 13:26 | |
| *** achampion has joined #heat | 13:31 | |
| *** lazy_prince is now known as killer_prince | 13:40 | |
| *** zslot has joined #heat | 13:40 | |
| *** nkhare has quit IRC | 13:41 | |
| pas-ha | all, I would like to hear some suggestion re my last comment on bug #1340065 | 13:41 |
| uvirtbot | Launchpad bug 1340065 in heat "AWS instance must not support NICS in-place update" [Undecided,In progress] https://launchpad.net/bugs/1340065 | 13:41 |
| *** bvivek has quit IRC | 13:43 | |
| *** killer_prince is now known as lazy_prince | 13:43 | |
| *** aweiteka has joined #heat | 13:47 | |
| *** jcoufal has quit IRC | 13:50 | |
| *** piyush has quit IRC | 13:50 | |
| *** jcoufal has joined #heat | 13:51 | |
| *** plomakin has joined #heat | 13:54 | |
| *** kopparam has quit IRC | 13:56 | |
| *** andersonvom has joined #heat | 13:56 | |
| *** kopparam has joined #heat | 13:56 | |
| *** jyoti-ranjan has quit IRC | 14:00 | |
| *** nati_ueno has joined #heat | 14:00 | |
| *** jistr has quit IRC | 14:02 | |
| *** lindsayk has joined #heat | 14:02 | |
| *** jistr has joined #heat | 14:03 | |
| *** bnemec has joined #heat | 14:03 | |
| *** aweiteka has quit IRC | 14:05 | |
| jdandrea | Can anyone advise on what kind of OS Image was used when this template was written? (Fedora, Cirros, CentOS, Ubuntu ... something else?) https://github.com/openstack/heat-templates/blob/master/hot/autoscaling.yaml | 14:06 |
| *** lindsayk has quit IRC | 14:06 | |
| Qiming | jdandrea, I was using Fedora | 14:08 |
| *** Yanyan has quit IRC | 14:08 | |
| *** lazy_prince is now known as killer_prince | 14:09 | |
| *** dmakogon_ is now known as denis_makogon | 14:10 | |
| Qiming | jdandrea, I think you have to use Fedora or CentOS for this template, Cirros or Ubuntu won't work due to the scripts in user_data | 14:12 |
| jdandrea | Qiming: Exactly my thinking ... except it wasn't working for me in CentOS (no mariadb or wordpress downloads found by yum). | 14:13 |
| jdandrea | I could also modify the user_data scripts, but then things get interesting. (Switching to mysql is fine, but then I'm installing apache2 and getting WP latest.zip and by then it's not even close to the same thing.) | 14:14 |
| jdandrea | Ah, Fedora, thanks. I will try that next. :) | 14:14 |
| Qiming | jdandrea, Fedora would be my recommendation to try these templates | 14:15 |
| jdandrea | Excellent. Thank you so much! | 14:15 |
| Qiming | jdandrea, welcome | 14:15 |
| Qiming | has anyone by chance hit the mysql problem: ERROR 1040 (HY000): Too many connections ? | 14:17 |
| *** ramishra has quit IRC | 14:18 | |
| *** aweiteka has joined #heat | 14:18 | |
| *** piyush has joined #heat | 14:23 | |
| *** ramishra has joined #heat | 14:28 | |
| shadower | zaneb: I tried removing `unicode` from here: https://github.com/openstack/heat/blob/2c90937e630067b8074abadc0e90edd784683b01/heat/engine/stack_resource.py#L350 | 14:29 |
| shadower | which seems to work fine | 14:29 |
| shadower | will assign myself to the bug, write tests and send it | 14:29 |
| *** hipokrit has joined #heat | 14:31 | |
| *** jistr has quit IRC | 14:32 | |
| *** vijendar has joined #heat | 14:33 | |
| *** jcoufal has quit IRC | 14:35 | |
| *** Dev_Jin has quit IRC | 14:36 | |
| *** sjmc7 has joined #heat | 14:37 | |
| *** cdent_ has joined #heat | 14:38 | |
| *** kopparam has quit IRC | 14:38 | |
| *** cdent has quit IRC | 14:40 | |
| *** cdent_ is now known as cdent | 14:40 | |
| *** nati_ueno has quit IRC | 14:45 | |
| *** samstav has joined #heat | 14:46 | |
| *** unmeshg has joined #heat | 14:49 | |
| *** bvandenh has quit IRC | 14:53 | |
| *** piyush has quit IRC | 14:57 | |
| *** kebray has joined #heat | 14:58 | |
| *** samstav has quit IRC | 14:58 | |
| *** ramishra has quit IRC | 14:58 | |
| *** kebray has joined #heat | 14:59 | |
| *** samstav has joined #heat | 14:59 | |
| *** lindsayk has joined #heat | 15:03 | |
| *** piyush has joined #heat | 15:06 | |
| *** lindsayk has quit IRC | 15:07 | |
| *** akuznetsov has quit IRC | 15:07 | |
| *** akuznetsov has joined #heat | 15:08 | |
| *** rbuilta has joined #heat | 15:08 | |
| *** ifarkas has quit IRC | 15:15 | |
| *** avquadri has quit IRC | 15:15 | |
| *** aweiteka has quit IRC | 15:16 | |
| *** shakamunyi has joined #heat | 15:17 | |
| *** shakamunyi has quit IRC | 15:17 | |
| *** gokrokve has joined #heat | 15:18 | |
| *** pas-ha has quit IRC | 15:20 | |
| *** david-lyle has joined #heat | 15:20 | |
| *** piyush1 has joined #heat | 15:21 | |
| zaneb | shadower: sounds perfect, thanks! | 15:21 |
| *** jistr has joined #heat | 15:21 | |
| *** daneyon_ has joined #heat | 15:22 | |
| *** shakamunyi has joined #heat | 15:24 | |
| *** shakamunyi has quit IRC | 15:24 | |
| *** piyush has quit IRC | 15:24 | |
| *** shakamunyi has joined #heat | 15:24 | |
| *** shakamunyi has quit IRC | 15:24 | |
| *** daneyon has quit IRC | 15:25 | |
| *** Qiming has quit IRC | 15:26 | |
| *** nati_ueno has joined #heat | 15:28 | |
| *** aweiteka has joined #heat | 15:29 | |
| *** denis_makogon has quit IRC | 15:31 | |
| *** vijendar has quit IRC | 15:32 | |
| *** pscheie has joined #heat | 15:34 | |
| pscheie | Is there a limit to the length of the hostname heat assigns to instances? | 15:35 |
| *** nati_ueno has quit IRC | 15:35 | |
| *** ramishra has joined #heat | 15:36 | |
| pscheie | I just re-spun a stack where we're using multiple instances via InstanceGroups and the Size parameter | 15:36 |
| zaneb | pscheie: yes. https://github.com/openstack/heat/blob/master/heat/engine/resources/server.py#L351 | 15:36 |
| zaneb | https://github.com/openstack/heat/blob/master/heat/engine/resources/instance.py#L335 | 15:37 |
| *** nati_ueno has joined #heat | 15:38 | |
| pscheie | Is there yet a way to control for the format of such names? | 15:38 |
| pscheie | Some multiple instance nodes are ending up with names that are 53 characters long and even at that, it's kinda mangling the name and breaking our naming convention. | 15:39 |
| pscheie | Ah, comment in line 349 says limit is 53 characters. | 15:40 |
| *** rakesh_hs has quit IRC | 15:42 | |
| pscheie | What controls the format of the name? | 15:42 |
| zaneb | pscheie: with an OS::Nova::Server you can specify the name; with AWS::EC2::Instance you're stuck with the one we give you | 15:43 |
| zaneb | we're aware that the system kind of sucks for scaling groups (including InstanceGroup) | 15:43 |
| *** nati_ueno has quit IRC | 15:43 | |
| *** nati_ueno has joined #heat | 15:43 | |
| zaneb | there's too many random parts | 15:43 |
| pscheie | Yeah, it's mangling the first part of the name which is where we store the stack name. | 15:44 |
| pscheie | It's truncating those parts. so the search tools we have aren't finding them. | 15:44 |
| zaneb | fixes welcome, but probably needs a blueprint | 15:45 |
| *** gokrokve has quit IRC | 15:45 | |
| zaneb | pscheie: yeah, it's done that way to ensure the names remain unique | 15:45 |
| pscheie | zaneb, yeah, I can see that, and that is important. | 15:47 |
| zaneb | so the key to success is to eliminate extra random stuff from the middle | 15:48 |
| pscheie | Repeating the instance name--that is, the name taken from the stanza in the heat template--probably isn't necessary. | 15:48 |
| *** zslot has quit IRC | 15:50 | |
| pscheie | So, that's two parts in the middle--the name and the random string--that could tossed out and still maintain unique names. | 15:50 |
| pscheie | Ideally, I suppose, having it be configurable would be best. | 15:51 |
| *** cody-somerville has quit IRC | 15:52 | |
| *** ramishra has quit IRC | 15:53 | |
| *** vijendar has joined #heat | 15:54 | |
| *** mspreitz has joined #heat | 15:54 | |
| mspreitz | Have we made the switch to Stevedore? | 15:55 |
| *** daneyon_ has quit IRC | 15:56 | |
| *** daneyon has joined #heat | 15:57 | |
| *** cody-somerville has joined #heat | 15:59 | |
| shardy | mspreitz: it's in progress, for client plugins at least | 16:00 |
| shardy | https://review.openstack.org/#/q/status:open+project:openstack/heat+branch:master+topic:bp/client-plugins,n,z | 16:00 |
| pscheie | zaneb, can I use OS::Nova::Server when doing multiple instances? | 16:00 |
| mspreitz | OK, that's consistent with what I noticed | 16:00 |
| therve | We started using it for some stuff | 16:01 |
| mspreitz | shardy: so no Stevedore for heat engine plugins now, right? | 16:01 |
| zaneb | pscheie: yes, if you use OS::Heat::AutoscalingGroup instead of InstanceGroup | 16:01 |
| zaneb | pscheie: note that you can't specify a pattern for the hostname though, just a hostname. so they'll all get the same one | 16:01 |
| pscheie | But won't a random string be added at the end of the name? | 16:03 |
| mspreitz | I am trying to figure out why Angus suggested Stevedore already in his latest comment on https://review.openstack.org/#/c/89363/ | 16:04 |
| *** lindsayk has joined #heat | 16:04 | |
| *** daneyon has quit IRC | 16:04 | |
| *** tspatzier has quit IRC | 16:04 | |
| shardy | mspreitz: probably because he's proposed a patch which implements loading resources via stevedore: | 16:04 |
| shardy | https://review.openstack.org/#/c/103044/ | 16:04 |
| shardy | I'm not sure we really reached consensus on that on the ML, my understanding was that we probably were sticking with the existing plugin model for resources though | 16:05 |
| shardy | zaneb: ^^ | 16:05 |
| zaneb | yeah, I'm not sure we reached consensus either | 16:06 |
| zaneb | (I'm still convinced I'm right ;) | 16:06 |
| shardy | FWIW I'm in the "if it ain't broke don't fix it" (at the moment) camp ;) | 16:06 |
| *** unmeshg has left #heat | 16:07 | |
| *** daneyon has joined #heat | 16:07 | |
| pscheie | zaneb, is there an example of OS::Heat::AutoscalingGroup somewhere? | 16:07 |
| mspreitz | One quick question about Stevedore hooks: if there are multiple things registered to be called at the same point, does Stevedore have a way for those things to influence the order in which they are called? | 16:08 |
| *** lindsayk has quit IRC | 16:08 | |
| mspreitz | pscheie: simple examples in https://review.openstack.org/#/c/97366/ | 16:08 |
| *** nati_ueno has quit IRC | 16:09 | |
| *** stannie has quit IRC | 16:10 | |
| *** daneyon_ has joined #heat | 16:10 | |
| zaneb | shardy: I actually thought a compromise consensus was forming around using stevedore, but Hooks instead of Extensions... but then it looked maybe asalkeld was backing away from that, so I don't know what's happening | 16:11 |
| shardy | zaneb: Ah, OK, sounds like a topic for the meeting agenda ;) | 16:12 |
| zaneb | oh, that'll be a fun meeting | 16:13 |
| *** chandankumar has quit IRC | 16:13 | |
| shardy | lol | 16:13 |
| zaneb | does anybody know why I have to log in again *every* !@#$%^& time I edit the wiki? | 16:13 |
| mspreitz | I'm going to push back on Angus' comment, saying that's not a settled matter and let's not hold up this patch for that. | 16:14 |
| *** daneyon has quit IRC | 16:14 | |
| zaneb | mspreitz: I disagree with that | 16:15 |
| mspreitz | zaneb: what do you recommend? | 16:15 |
| zaneb | mspreitz: resource plugins are still up for debate, but stevedore is the future | 16:15 |
| mspreitz | ok.. | 16:15 |
| zaneb | and we have it already, for the client plugins, so the hard work is done | 16:16 |
| mspreitz | Is it true in Stevedore that to register a plugin you do not simply drop it in a directory but you also have to edit an entry_points.txt file? | 16:16 |
| zaneb | stevedore is a good fit for most things, but I haven't looked at your patch yet | 16:16 |
| *** cody-somerville has quit IRC | 16:17 | |
| zaneb | mspreitz: in general it seems that you package your plugin in a Python package, and install the package | 16:17 |
| *** andreaf has quit IRC | 16:17 | |
| zaneb | so no editing required, but it's not as simple as dropping in a directory; you need to create the Python package | 16:17 |
| mspreitz | And in the case of hooks, if multiple are registered at the same plug point, is there any control over their order? | 16:17 |
| zaneb | that's a good question, I don't know | 16:18 |
| zaneb | I suspect not, unless you define it inside the plugin | 16:18 |
| *** tomek_adamczewsk has quit IRC | 16:18 | |
| zaneb | e.g. have a priority() method or something | 16:18 |
| mspreitz | ah, right | 16:18 |
| mspreitz | thanks | 16:18 |
| mspreitz | wait | 16:19 |
| mspreitz | are we planning to use Stevedore to invoke the set of hooks, or does Stevedore merely supply the list and we write code to iterate over it? | 16:19 |
| therve | stevedore just supply the list | 16:21 |
| mspreitz | therve: thanks | 16:21 |
| therve | mspreitz, Basically in the patch, you remove the environment changes, and adapt get_plug_point_classes | 16:22 |
| mspreitz | thanks | 16:22 |
| therve | You can also use your custom priority code at that point to order the list | 16:22 |
| * mspreitz goes AFK | 16:24 | |
| *** rbuilta has quit IRC | 16:24 | |
| *** derekh_ has quit IRC | 16:27 | |
| *** rakesh_hs has joined #heat | 16:29 | |
| *** jistr has quit IRC | 16:30 | |
| *** pasquier-s has quit IRC | 16:36 | |
| *** gokrokve has joined #heat | 16:41 | |
| *** bvivek has joined #heat | 16:42 | |
| *** tspatzier has joined #heat | 16:45 | |
| *** che-arne|2 has joined #heat | 16:54 | |
| *** che-arne has quit IRC | 16:55 | |
| *** cody-somerville has joined #heat | 17:00 | |
| *** andreaf has joined #heat | 17:01 | |
| openstackgerrit | A change was merged to openstack/heat: Remove redundant method handle_get_attributes https://review.openstack.org/106734 | 17:02 |
| *** cdent_ has joined #heat | 17:06 | |
| *** achampion has quit IRC | 17:08 | |
| *** achampion has joined #heat | 17:08 | |
| *** tonisboner has joined #heat | 17:08 | |
| *** mspreitz has quit IRC | 17:08 | |
| *** cdent_ has quit IRC | 17:08 | |
| *** cdent has quit IRC | 17:10 | |
| *** andreaf has quit IRC | 17:10 | |
| *** nati_ueno has joined #heat | 17:11 | |
| *** cdent has joined #heat | 17:11 | |
| *** tonisbones has quit IRC | 17:11 | |
| *** zz_gondoi is now known as gondoi | 17:13 | |
| *** rbuilta has joined #heat | 17:16 | |
| *** openstackgerrit has quit IRC | 17:16 | |
| *** openstackgerrit has joined #heat | 17:17 | |
| *** tonisbones_ has joined #heat | 17:17 | |
| *** mspreitz has joined #heat | 17:17 | |
| *** harlowja_away is now known as harlowja | 17:18 | |
| *** achampion has quit IRC | 17:18 | |
| *** harlowja is now known as harlowja_away | 17:19 | |
| *** harlowja_away is now known as harlowja | 17:19 | |
| *** tonisboner has quit IRC | 17:19 | |
| *** shakamunyi has joined #heat | 17:20 | |
| *** shakamunyi has quit IRC | 17:20 | |
| *** achampion has joined #heat | 17:20 | |
| *** new2heat has joined #heat | 17:20 | |
| new2heat | hi, I have a question regarding "promotional code" for Openstack summit. Who are eligible to get the promotional code and what is the deadline for it? | 17:21 |
| *** shakamunyi has joined #heat | 17:21 | |
| *** shakamunyi has quit IRC | 17:21 | |
| *** andreaf has joined #heat | 17:25 | |
| *** sjmc7_ has joined #heat | 17:31 | |
| ruhe | new2heat: you have to be eligible as an ATC. see https://wiki.openstack.org/wiki/Governance/Foundation/TechnicalCommittee paragraph 'Voters for TC seats ("ATC")' | 17:33 |
| *** piyush1 has quit IRC | 17:33 | |
| *** sjmc7 has quit IRC | 17:34 | |
| new2heat | ruhe: so anyone who contributes code or review code is an ATC rt? as long as u sign the OS IC agreement.... | 17:37 |
| ruhe | new2heat: i'm not sure about reviews. but a code contribution to an official openstack projects makes you an ATC | 17:38 |
| new2heat | got it. thanks | 17:39 |
| *** shakamunyi has joined #heat | 17:39 | |
| *** shakamunyi has quit IRC | 17:39 | |
| *** shakamunyi has joined #heat | 17:41 | |
| mspreitz | I am curious about the thinking about health checking for scaling groups. To achieve parity with AWS functionality goes beyond what Convergence is about. | 17:41 |
| mspreitz | Is the thinking to indeed go beyond, or settle only for Convergence? | 17:41 |
| *** shakamunyi has quit IRC | 17:42 | |
| *** rushiagr_ has quit IRC | 17:42 | |
| *** ifarkas has joined #heat | 17:42 | |
| *** cdent_ has joined #heat | 17:44 | |
| *** saurabhs has joined #heat | 17:45 | |
| *** cdent has quit IRC | 17:45 | |
| *** cdent_ is now known as cdent | 17:45 | |
| zaneb | new2heat: yes, you need to get a patch merged before 21 August | 17:52 |
| new2heat | zaneb: thanks | 17:53 |
| *** cdent has quit IRC | 17:56 | |
| *** tonisboner has joined #heat | 17:59 | |
| *** achampio1 has joined #heat | 18:00 | |
| *** achampion has quit IRC | 18:02 | |
| *** tonisbones_ has quit IRC | 18:03 | |
| *** tonisbones_ has joined #heat | 18:03 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Pass user_creds_id when creating backup stack https://review.openstack.org/107455 | 18:03 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Pass user_creds_id into nested stacks https://review.openstack.org/107456 | 18:03 |
| *** tonisbones_ has quit IRC | 18:04 | |
| *** bvandenh has joined #heat | 18:05 | |
| *** tonisbones has joined #heat | 18:05 | |
| *** achampion has joined #heat | 18:05 | |
| *** achampio1 has quit IRC | 18:05 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Policy Enforcer, pass entire context dict https://review.openstack.org/107458 | 18:06 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Remove unused functions from policy enforcer https://review.openstack.org/107459 | 18:06 |
| *** tonisboner has quit IRC | 18:06 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Newline cleanups in db API https://review.openstack.org/107460 | 18:07 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 18:11 |
| *** dims__ has quit IRC | 18:11 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: SoftwareDeployments move signal reason into resource plugin https://review.openstack.org/106825 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert CFN waitcondition API to resource_signal https://review.openstack.org/106824 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Allow plugins to specify signal event reason https://review.openstack.org/106821 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Update test_signal to use stub_keystoneclient https://review.openstack.org/106820 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Return metadata from resource_signal RPC interface https://review.openstack.org/106823 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: WaitConditionHandle resources, create events with reason data https://review.openstack.org/106822 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 18:11 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Mark metadata_update RPC call deprecated https://review.openstack.org/106985 | 18:11 |
| *** killer_prince is now known as lazy_prince | 18:12 | |
| *** achampio1 has joined #heat | 18:12 | |
| *** tonisbones has quit IRC | 18:13 | |
| *** achampion has quit IRC | 18:13 | |
| *** tonisbones has joined #heat | 18:13 | |
| *** new2heat has quit IRC | 18:13 | |
| *** piyush has joined #heat | 18:18 | |
| *** piyush1 has joined #heat | 18:19 | |
| *** tonisbones has quit IRC | 18:19 | |
| *** achampio1 has quit IRC | 18:19 | |
| *** bvandenh has quit IRC | 18:21 | |
| *** piyush has quit IRC | 18:22 | |
| *** achampion has joined #heat | 18:24 | |
| *** lindsayk has joined #heat | 18:27 | |
| *** sgordon has quit IRC | 18:28 | |
| *** dims__ has joined #heat | 18:35 | |
| *** jstrachan has quit IRC | 18:38 | |
| *** andreaf has quit IRC | 18:38 | |
| *** Akshik has joined #heat | 18:41 | |
| pscheie | zaneb, looking at http://docs.openstack.org/developer/heat/template_guide/openstack.html I see the OS::Heat::AutoScalingGroup section, but no mention of what syntax to use to specify a name (?) | 18:44 |
| mspreitz | pscheie: what do you mean by "syntax to use to specify a name" ? | 18:45 |
| mspreitz | pscheie: I have submitted a change that includes examples; do they answer your question? | 18:46 |
| zaneb | pscheie: it's just a property of OS::Nova::Server | 18:46 |
| pscheie | Do you mean https://review.openstack.org/#/c/97366/ ? | 18:47 |
| mspreitz | pscheie: yes, that's the change | 18:47 |
| zaneb | pscheie: so inside the "resource" property's "properties" section you add the key "name" | 18:48 |
| pscheie | zaneb, perhaps I'm misunderstanding, but I'm not using OS::Nova::Server, I'm using OS::Heat::AutoscalingGroup. | 18:48 |
| zaneb | pscheie: OS::Heat::AutoscalingGroup creates a scaling group of any other type of resource | 18:49 |
| mspreitz | OS::Heat::AutoScalingGroup does not have a property "name" | 18:49 |
| zaneb | in this case, you select OS::Nova::Server as the type you want to scale | 18:49 |
| mspreitz | pscheie: what exactly do you want to name? | 18:50 |
| zaneb | so inside the "resource" property, you specify "type: OS::Nova::Server" | 18:50 |
| zaneb | like in https://review.openstack.org/#/c/97366/11/hot/asg_of_servers.yaml | 18:51 |
| pscheie | mspreitz, I'm trying to use multiple instances (resources, I guess) but the host names it is producing are way too long, exceding 53 characters, so heat/nova is truncating the first parts of the name which is breaking some of the tools we have. | 18:52 |
| pscheie | zaneb, ah, ok. I've been using Type: AWS::AutoScaling::LaunchConfiguration | 18:53 |
| pscheie | zaneb, sounds like I need to change that. | 18:53 |
| zaneb | yes, creating a whole group of launch configurations won't be much help to you ;) | 18:54 |
| pscheie | heh | 18:54 |
| *** mspreitz has quit IRC | 18:54 | |
| *** bvivek has quit IRC | 18:55 | |
| *** lazy_prince is now known as killer_prince | 18:56 | |
| *** achampion has quit IRC | 18:59 | |
| *** dsneddon has joined #heat | 19:02 | |
| pscheie | Hmm, heat is saying 'ERROR: Unknown resource Type : OS::Heat::AutoscalingGroup' | 19:02 |
| *** achampion has joined #heat | 19:03 | |
| lifeless | try a capital S ? | 19:03 |
| lifeless | OS::Heat::AutoScalingGroup from http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Heat::ResourceGroup | 19:04 |
| pscheie | lifeless, tried that, same result. | 19:04 |
| lifeless | bah, bad url again, but its on that page ;> | 19:04 |
| ryansb | http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Heat::AutoScalingGroup | 19:05 |
| pscheie | Does this require Icehouse? We're only on Havana. | 19:06 |
| ryansb | pscheie: I'll check | 19:07 |
| *** dims__ has quit IRC | 19:08 | |
| *** jmckind has joined #heat | 19:09 | |
| *** tonisboner has joined #heat | 19:10 | |
| ryansb | no, it does not exist in havana http://docs.openstack.org/developer/heat/havana/template_guide/index.html | 19:10 |
| *** tonisboner has quit IRC | 19:10 | |
| ryansb | only cloudformation autoscaling is available in havana http://docs.openstack.org/developer/heat/havana/template_guide/cfn.html#AWS::AutoScaling::AutoScalingGroup | 19:11 |
| *** tonisbones has joined #heat | 19:11 | |
| zaneb | pscheie: yep, requires Icehouse | 19:11 |
| pscheie | ryansb, ok, yeah, that's what I've been using. | 19:11 |
| *** Akshik has quit IRC | 19:12 | |
| pscheie | AWS::AutoScaling::AutoScalingGroup, I mean | 19:12 |
| ryansb | pscheie: yeah, until you upgrade to icehouse that's all you get | 19:12 |
| pscheie | Can I get away with just upgrading heat to icehouse? Or do I have to up nova as well? | 19:13 |
| pscheie | We did that for grizzly: upgraded it to havana ahead of upgrading everything else because there was something in havana we needed (I forget what). | 19:14 |
| zaneb | I don't think Heat depends on anything in Icehouse nova | 19:18 |
| zaneb | obviously I would test it first ;) | 19:18 |
| pscheie | zaneb, pfft, test. Where's the adventure in that? | 19:20 |
| zaneb | if you want to be really adventurous, just drop the plugins from Icehouse into the Havana version of Heat | 19:21 |
| pscheie | Well, we already have a mirror of the icehouse repo setup in spacewalk, so I just have to point the boxes at that. | 19:24 |
| *** tango has joined #heat | 19:33 | |
| *** andrearosa_home has joined #heat | 19:36 | |
| *** noTHD has quit IRC | 19:40 | |
| *** lindsayk has quit IRC | 19:44 | |
| pscheie | I upgrade heat to icehouse (0.2.9 from package python-heatclient-0.2.9-1.el6.noarch0) but I'm still getting the same error | 19:47 |
| pscheie | ERROR: Unknown resource Type : OS::Heat::AutoScalingGroup | 19:47 |
| *** sgordon has joined #heat | 19:49 | |
| *** noTHD has joined #heat | 19:49 | |
| zaneb | pscheie: python-heatclient isn't Heat. you need to upgrade heat-engine | 19:51 |
| pscheie | zaneb, I didn't have heat-engine installed on my heat client machine (where I build my templates and run the heat client) nor on the nova host running heat-api. | 19:55 |
| pscheie | zaneb, and OS has been working swimmingly for us for the past year. | 19:55 |
| pscheie | Did something change in icehouse such that heat-engine is now required? | 19:55 |
| zaneb | it's installed somewhere, otherwise it wouldn't be working | 19:55 |
| zaneb | heat-engine does literally everything in Heat, and always has | 19:55 |
| pscheie | oops, you're right, I forgot a box. | 19:59 |
| zaneb | meeting time! | 20:00 |
| *** noTHD has quit IRC | 20:05 | |
| *** bgorski has joined #heat | 20:06 | |
| *** piyush1 has quit IRC | 20:09 | |
| *** lindsayk has joined #heat | 20:09 | |
| *** tonisbones has quit IRC | 20:11 | |
| *** rakesh_hs has quit IRC | 20:13 | |
| *** dims has joined #heat | 20:14 | |
| *** ifarkas has quit IRC | 20:15 | |
| *** akuznetsov has quit IRC | 20:20 | |
| *** noTHD has joined #heat | 20:21 | |
| *** piyush has joined #heat | 20:27 | |
| *** piyush has quit IRC | 20:36 | |
| openstackgerrit | A change was merged to openstack/heat: Make sure we can create an empty template https://review.openstack.org/105894 | 20:38 |
| *** sgordon has quit IRC | 20:39 | |
| *** asalkeld has joined #heat | 20:53 | |
| *** andreaf has joined #heat | 20:54 | |
| *** ccrouch has quit IRC | 20:57 | |
| *** lindsayk has quit IRC | 21:04 | |
| *** lindsayk has joined #heat | 21:05 | |
| *** aweiteka has quit IRC | 21:06 | |
| *** jdob has quit IRC | 21:07 | |
| *** tspatzier has quit IRC | 21:08 | |
| *** noTHD has quit IRC | 21:08 | |
| shardy | stevebaker: thanks for the docs tip, any idea how you do run all tests for a particular contrib plugin tree? | 21:08 |
| shardy | the gate seems to do it a convoluted way via /usr/local/jenkins/slave_scripts/run-unittests.sh | 21:09 |
| *** piyush has joined #heat | 21:09 | |
| *** ccrouch has joined #heat | 21:09 | |
| stevebaker | shardy: testr run contrib.foo | 21:09 |
| stevebaker | shardy: might do it | 21:09 |
| *** lindsayk has quit IRC | 21:10 | |
| *** lindsayk has joined #heat | 21:10 | |
| *** andrearosa_home has quit IRC | 21:10 | |
| shardy | stevebaker: Hmm, keeps telling me PASSED but I think it's lying.. ;) | 21:12 |
| jdandrea | Trying software config/deployment for the first time (yay me!), but I'm getting "heatclient.exc.HTTPBadRequest: ERROR: 'NoneType' object has no attribute 'auth_token'" ... unsure what "NoneType" means. Anyone run into this? | 21:12 |
| stevebaker | shardy: probably because no tests ran | 21:12 |
| shardy | Yeah, it's confusing when it does that though | 21:13 |
| stevebaker | jdandrea: there is a bug with heat standalone mode and software config. Are you running standalone? | 21:13 |
| jdandrea | stevebaker: Suspecting yes. :) | 21:13 |
| jdandrea | ... although this is a standard-issue Icehouse install with all the trimmings. Perhaps it isn't standalone. | 21:15 |
| jdandrea | Pardon the naive question: Is there a way I can check? | 21:15 |
| *** ccrouch1 has joined #heat | 21:16 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Port resource https://review.openstack.org/104224 | 21:17 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Node resource https://review.openstack.org/104223 | 21:17 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add Ironic resource base class https://review.openstack.org/104222 | 21:17 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Chassis resource https://review.openstack.org/103133 | 21:17 |
| stevebaker | jdandrea: it would only be standalone if there is no orchestration endpoint in your keystone catalog | 21:17 |
| *** ccrouch has quit IRC | 21:18 | |
| shardy | night all | 21:20 |
| jdandrea | stevebaker: I see nine endpoints but I am not sure which would be for orchestration. | 21:20 |
| jdandrea | (If at all.) | 21:20 |
| *** aweiteka has joined #heat | 21:20 | |
| *** shardy is now known as shardy_afk | 21:20 | |
| stevebaker | jdandrea: because it is called orchestration, and may mention heat | 21:20 |
| jdandrea | stevebaker: That's what I was hoping ... but none of the endpoints shown via 'keystone endpoint-list' mention anything at all. Hmm. | 21:21 |
| stevebaker | jdandrea: how about "keystone catalog" | 21:21 |
| *** jmckind has quit IRC | 21:21 | |
| jdandrea | stevebaker: Ah! Much better, thanks. (TIL: keystone *catalog*) | 21:22 |
| jdandrea | There is orchestration in there. | 21:22 |
| jdandrea | Compute, Network. Cloud Formation, the whole kit and caboodle. | 21:22 |
| jdandrea | It seems like it isn't standalone then. :-o | 21:25 |
| *** samstav has quit IRC | 21:27 | |
| *** tomek_adamczewsk has joined #heat | 21:28 | |
| *** aweiteka has quit IRC | 21:30 | |
| *** radez is now known as radez_g0n3 | 21:31 | |
| *** sgordon has joined #heat | 21:31 | |
| *** kebray has quit IRC | 21:32 | |
| stevebaker | jdandrea: hmm, you may need to raise a bug with the details | 21:32 |
| *** sgordon has quit IRC | 21:36 | |
| asalkeld | hi stevebaker, enjoy your break? | 21:36 |
| * asalkeld gettting coffee | 21:37 | |
| *** tomek_adamczewsk has quit IRC | 21:37 | |
| *** tomek_adamczewsk has joined #heat | 21:39 | |
| *** noTHD has joined #heat | 21:40 | |
| openstackgerrit | A change was merged to openstack/heat: Fix H201 violations and re-enable gating https://review.openstack.org/102778 | 21:47 |
| *** achampion has quit IRC | 21:52 | |
| zaneb | jpeeler: I just +2'd your OpenShift templates :) | 21:53 |
| jpeeler | thanks, just need shardy maybe to do the same and my life will be complete | 21:54 |
| *** vijendar has quit IRC | 21:56 | |
| stevebaker | asalkeld: it was great, very relaxing. and warm | 21:58 |
| *** david-lyle has quit IRC | 22:00 | |
| asalkeld | nice | 22:01 |
| asalkeld | we escaped to hawaii - brisbane too cold | 22:01 |
| asalkeld | it's all relative I guess | 22:01 |
| *** david-lyle has joined #heat | 22:01 | |
| *** kebray has joined #heat | 22:07 | |
| *** tomek_adamczewsk has quit IRC | 22:07 | |
| *** tomek_adamczewsk has joined #heat | 22:08 | |
| stevebaker | its a bit tragic that brisbane winter ~= wellington summer | 22:09 |
| *** gondoi is now known as zz_gondoi | 22:11 | |
| *** cody-somerville has quit IRC | 22:15 | |
| *** hipokrit has quit IRC | 22:24 | |
| *** bgorski has quit IRC | 22:25 | |
| *** piyush has quit IRC | 22:26 | |
| asalkeld | stevebaker, we lived in titahi bay for a while - I enjoyed that | 22:34 |
| *** dims_ has joined #heat | 22:34 | |
| asalkeld | (maybe cos we moved from marsterton - brr) | 22:34 |
| stevebaker | even that far up the coast is warmer than here, northern current on west coast vs south coast | 22:35 |
| asalkeld | we stayed here: http://homepages.paradise.net.nz/margandj/tiro-moana/index.html | 22:36 |
| *** dims has quit IRC | 22:38 | |
| stevebaker | nice | 22:42 |
| *** tomek_adamczewsk has quit IRC | 22:46 | |
| *** tomek_adamczewsk has joined #heat | 22:47 | |
| openstackgerrit | Ton Ngo proposed a change to openstack/heat: Add RPC call to retrieve the parameters for a stack https://review.openstack.org/107524 | 22:48 |
| *** tomek_adamczewsk has quit IRC | 22:49 | |
| *** tomek_adamczewsk has joined #heat | 22:49 | |
| *** lindsayk has quit IRC | 23:00 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/heat-specs: Convergence Specification https://review.openstack.org/95907 | 23:03 |
| *** gokrokve has quit IRC | 23:04 | |
| *** gokrokve has joined #heat | 23:05 | |
| *** cody-somerville has joined #heat | 23:08 | |
| *** gokrokve has quit IRC | 23:10 | |
| *** sjmc7_ has quit IRC | 23:12 | |
| *** tomek_adamczewsk has quit IRC | 23:19 | |
| *** tomek_adamczewsk has joined #heat | 23:19 | |
| *** david-lyle has quit IRC | 23:19 | |
| *** kebray has quit IRC | 23:21 | |
| *** kebray has joined #heat | 23:22 | |
| asalkeld | any review requests? | 23:23 |
| asalkeld | stevebaker, ^ | 23:23 |
| asalkeld | not a bizilion, but could do a handful | 23:24 |
| stevebaker | asalkeld: I'll re-propose the remaining client-plugins soon | 23:24 |
| asalkeld | ok | 23:24 |
| zaneb | asalkeld: jpeeler would love you to do https://review.openstack.org/#/c/106159/ ;) | 23:24 |
| asalkeld | shardy_afk, has a bunch | 23:24 |
| asalkeld | zaneb, I'll sort it out | 23:25 |
| asalkeld | zaneb, did we come to some sort of conclusion re: plugins? | 23:25 |
| zaneb | cheers. just keep in mind that it's only using features from havana | 23:25 |
| zaneb | asalkeld: yeah, I was going to ask you the same ;) we discussed it briefly in the meeting today | 23:26 |
| asalkeld | I still don't get the objection to setup.cfg | 23:26 |
| asalkeld | shrug | 23:26 |
| zaneb | I am happy with the Hooks approach | 23:26 |
| asalkeld | zaneb, it doesn't really help | 23:26 |
| asalkeld | hooks still needs a name | 23:26 |
| asalkeld | hooks would be good for the pluggable action things | 23:27 |
| zaneb | only a name for the hook, I thought? | 23:27 |
| zaneb | like resource_plugin_loader | 23:27 |
| asalkeld | zaneb, no - each class still has a name | 23:27 |
| asalkeld | but they can be some in common | 23:27 |
| stevebaker | one question, how are we expecting deployers to deploy contrib plugins if they are installing from python sdist archives? Should we completely exclude contrib from our sdist since each will have its own setup.cfg? | 23:28 |
| zaneb | asalkeld: the class name, but not the name of the resource type, right? | 23:28 |
| zaneb | asalkeld: it occurs to me that we could define the type name inside the class | 23:28 |
| asalkeld | zaneb, the format is the same | 23:28 |
| zaneb | orly? | 23:29 |
| asalkeld | (setup.cfg format) | 23:29 |
| *** ccrouch1 has quit IRC | 23:29 | |
| asalkeld | zaneb, to me it's perfect for https://review.openstack.org/#/c/89363/ | 23:29 |
| zaneb | ...given that the environment is the place to define aliases now, we'll never need to register a single class with multiple names | 23:29 |
| asalkeld | as the hooks can be post_create = class | 23:29 |
| zaneb | agreed | 23:30 |
| asalkeld | zaneb, extension looks ok to me | 23:30 |
| asalkeld | (for resources) | 23:30 |
| asalkeld | it's the most generic form of stevedore plugin | 23:30 |
| asalkeld | zaneb, so agree re: environment | 23:31 |
| zaneb | do you have an example of the setup.cfg format for hooks handy? | 23:31 |
| zaneb | I'm trying to visualise it | 23:31 |
| asalkeld | not handy, but it's exactly the same as for extensions | 23:32 |
| asalkeld | <name> = class | 23:32 |
| asalkeld | the difference is in how you retrieve the extension | 23:32 |
| asalkeld | manager[name] returns a list | 23:32 |
| zaneb | what is the name used for? | 23:32 |
| asalkeld | not just one class | 23:33 |
| asalkeld | so get me all the plugins for "post_create_hook" | 23:33 |
| zaneb | hooks are advertised as "Single Name" | 23:33 |
| zaneb | I want my money back | 23:33 |
| asalkeld | :-) | 23:33 |
| *** rbuilta has quit IRC | 23:33 | |
| asalkeld | i think the format in setup.cfg has more to do with how namespaces are done | 23:34 |
| asalkeld | not really much to do with stevedore | 23:34 |
| asalkeld | (entry_points) | 23:34 |
| *** lindsayk has joined #heat | 23:35 | |
| asalkeld | stevebaker, I see in MANIFEST graft contrib | 23:36 |
| zaneb | looking at the current setup.cfg I see http://paste.openstack.org/show/86824/ | 23:37 |
| stevebaker | asalkeld: that is a bit messy, I wonder if we should exclude contrib, and insist that contrib plugins are sourced from git | 23:37 |
| asalkeld | stevebaker, maybe | 23:37 |
| asalkeld | zaneb, I am just thinking that is going to be more confusing to an operator | 23:38 |
| *** lindsayk1 has joined #heat | 23:39 | |
| asalkeld | zaneb, setup.cfg is probably better for a user (all resources in one place) | 23:39 |
| asalkeld | maybe a bit worse for devs | 23:39 |
| asalkeld | (tho' it makes no difference to me) | 23:39 |
| *** lindsayk has quit IRC | 23:39 | |
| asalkeld | I quite like seeing all resources listed out (without having to start heat to look at the log file) | 23:40 |
| zaneb | how I was hoping hooks would work was like this: http://paste.openstack.org/show/86825/ | 23:40 |
| asalkeld | zaneb, one option | 23:41 |
| asalkeld | heat.resources = | 23:41 |
| asalkeld | resource = heat.engine.resources.myresource.MyResource | 23:41 |
| asalkeld | (fudge it) | 23:42 |
| zaneb | yeah | 23:42 |
| asalkeld | I can change to that | 23:42 |
| asalkeld | you cool with that? | 23:42 |
| zaneb | seems hacky, but I am not against it :) | 23:42 |
| asalkeld | k | 23:42 |
| *** tomek_adamczewsk has quit IRC | 23:42 | |
| asalkeld | you see the parser rename? | 23:43 |
| asalkeld | https://review.openstack.org/#/c/107290/ | 23:43 |
| zaneb | I think it will be good to keep the resource name as close as possible to the definition. it's too far as it is, so if we can make this change and move it even closer I see that as a win | 23:43 |
| *** tomek_adamczewsk has joined #heat | 23:43 | |
| asalkeld | k - zaneb sounds good | 23:44 |
| zaneb | I didn't see it yet, but I've been wanting to do that since forever :D | 23:44 |
| zaneb | will review | 23:44 |
| asalkeld | should be backwards compatible | 23:45 |
| zaneb | asalkeld: reading comments... are you suggesting we do things just because they make sense?!? | 23:47 |
| asalkeld | well blow me down | 23:48 |
| zaneb | we need a procedure for that | 23:48 |
| asalkeld | lol | 23:48 |
| *** che-arne|2 has quit IRC | 23:48 | |
| stevebaker | asalkeld: did you find that "stack" was used as a variable name in lots of places, so parser rename leads to masking? | 23:48 |
| asalkeld | yeah | 23:48 |
| *** che-arne has joined #heat | 23:49 | |
| asalkeld | stack and template *everywhere* | 23:49 |
| zaneb | that's pretty much the only reason I hadn't done it already | 23:49 |
| stevebaker | I actually considered renaming parser to stackz, bonus of zero pep8 fixes ;) | 23:49 |
| asalkeld | i started with "from heat.engine import stack as stackm" | 23:49 |
| asalkeld | but huge change set | 23:50 |
| asalkeld | will lead to rebase madness | 23:50 |
| asalkeld | tho' could be done later | 23:51 |
| *** tomek_adamczewsk has quit IRC | 23:52 | |
| *** tomek_adamczewsk has joined #heat | 23:54 | |
| *** m_22 has joined #heat | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!