Thursday, 2014-01-02

*** sdake has quit IRC00:30
*** funzo has joined #heat00:35
*** funzo has quit IRC00:39
*** funzo has joined #heat00:39
*** rongze has joined #heat00:42
*** funzo has quit IRC00:42
*** rongze has quit IRC00:46
*** liang has quit IRC00:49
*** rongze has joined #heat01:15
*** gokrokve has joined #heat01:28
*** rongze has quit IRC01:34
*** funzo has joined #heat01:43
*** alexheneveld has quit IRC01:44
*** funzo has quit IRC01:48
*** rongze has joined #heat01:49
openstackgerritJun Jie Nan proposed a change to openstack/heat: Deleted TEMPDIR on exit  https://review.openstack.org/6450602:03
*** liang has joined #heat02:03
*** funzo has joined #heat02:05
openstackgerritJun Jie Nan proposed a change to openstack/heat: Deleted TEMPDIR on exit  https://review.openstack.org/6450602:11
*** erkules_ has joined #heat02:24
AlienyygHi guys ,anyone here ?02:24
*** erkules has quit IRC02:26
Alienyygnanjj: I create a new cfn image with diskimage-builder and got the same problem : http://paste.openstack.org/show/59482/  anyhelp?02:29
*** funzo has quit IRC02:32
*** funzo has joined #heat02:32
*** funzo has quit IRC02:37
*** rpothier__ has quit IRC02:45
*** rpothier_ has joined #heat02:45
Alienyyganyone can help with this ?02:51
*** erkules_ has quit IRC02:52
openstackgerrithuangtianhua proposed a change to openstack/heat: "mountpoint" should not be required parameter  https://review.openstack.org/6463603:06
*** funzo has joined #heat03:32
*** funzo has quit IRC03:35
*** funzo has joined #heat03:35
openstackgerritJun Jie Nan proposed a change to openstack/heat: Removed unused variables  https://review.openstack.org/6282703:39
*** funzo has quit IRC03:40
*** nkhare has joined #heat03:41
*** nati_ueno has joined #heat04:04
*** nkhare has quit IRC04:14
*** nati_ueno has quit IRC04:16
*** Alienyyg_ has joined #heat04:33
*** funzo has joined #heat04:36
*** Alienyyg has quit IRC04:36
*** funzo has quit IRC04:41
*** rongze has quit IRC04:47
*** rongze has joined #heat04:48
*** erkules_ has joined #heat04:51
*** rongze has quit IRC04:52
*** alienyyg has joined #heat04:58
*** urulama has joined #heat04:59
openstackgerritJun Jie Nan proposed a change to openstack/heat: Sort the output of config/generator.py by group name  https://review.openstack.org/6450005:14
*** nkhare has joined #heat05:18
*** rongze has joined #heat05:18
openstackgerritJun Jie Nan proposed a change to openstack/heat: Added heat url in config  https://review.openstack.org/6451805:18
*** rongze has quit IRC05:20
*** rongze_ has joined #heat05:20
openstackgerritJun Jie Nan proposed a change to openstack/python-heatclient: Fixed typo error OS_IMAGE_URL  https://review.openstack.org/6366105:21
openstackgerritJun Jie Nan proposed a change to openstack/python-heatclient: Wrapped some long lines  https://review.openstack.org/6366205:23
*** jay-lau-513 has quit IRC05:23
*** jay-lau-513 has joined #heat05:24
*** rongze_ has quit IRC05:25
openstackgerritJun Jie Nan proposed a change to openstack/python-heatclient: Added API reference document  https://review.openstack.org/6366505:26
*** jay-lau-513 has quit IRC05:27
*** jay-lau-513 has joined #heat05:28
*** funzo has joined #heat05:37
*** funzo has quit IRC05:41
*** chandankumar has joined #heat05:48
openstackgerritJenkins proposed a change to openstack/heat: Imported Translations from Transifex  https://review.openstack.org/6450406:05
openstackgerritJun Jie Nan proposed a change to openstack/heat: Added heat url in config for heat standalone mode  https://review.openstack.org/6451806:09
*** rongze has joined #heat06:18
*** rongze has quit IRC06:23
*** rongze has joined #heat06:28
*** funzo has joined #heat06:37
*** funzo_ has joined #heat06:39
*** funzo has quit IRC06:39
*** funzo_ has quit IRC06:44
*** zigo has joined #heat06:56
openstackgerritLiang Chen proposed a change to openstack/heat: Marconi message queue resource implementation  https://review.openstack.org/6103607:06
openstackgerritLiang Chen proposed a change to openstack/heat: Add marconi client  https://review.openstack.org/6103507:06
openstackgerritA change was merged to openstack/heat: Empty files shouldn't contain copyright nor license  https://review.openstack.org/6391507:11
*** rushiagr has joined #heat07:19
Alienyyg_hi guys, after I rebuild my own fedora-cfn-image with diskimage-builde, I launch a autoscaing stack with aws loadbalancer, and load balancer failed to create, because the waitcondition resource wait for the success signal, and in the loadbalancer ,I got heat-provision.log :http://paste.openstack.org/show/59482/07:39
Alienyyg_so , can anyone figure where went wrong?07:40
*** funzo has joined #heat07:40
Alienyyg_the cf-signal.log http://paste.openstack.org/show/59538/07:42
*** gokrokve has quit IRC07:43
*** ifarkas has joined #heat07:44
*** funzo has quit IRC07:45
*** rongze has quit IRC07:45
*** nkhare has quit IRC07:47
*** nkhare has joined #heat07:49
*** rongze has joined #heat07:51
*** gokrokve has joined #heat08:14
*** gokrokve_ has joined #heat08:16
*** gokrokve has quit IRC08:19
*** gokrokve_ has quit IRC08:20
*** pasquier-s has joined #heat08:21
*** jprovazn has joined #heat08:24
*** gokrokve has joined #heat08:32
Alienyyg_anyone can take a look at this error ?08:34
*** gokrokve has quit IRC08:37
*** funzo has joined #heat08:41
*** rushiagr is now known as rushiagr_away08:41
*** funzo has quit IRC08:46
*** lsmola_ has joined #heat09:10
*** derekh has joined #heat09:10
*** liang has quit IRC09:19
shardyAlienyyg_: If you install nmap in the instance, and nmap 7.7.7.7, do you see port 8000 open?09:20
shardyIt looks like cfn-signal is working, so maybe you're missing some firewall rules?09:20
Alienyyg_shardy: the 8000 is open09:21
Alienyyg_shardy: I can get it with netsate -an | grep 800009:21
shardyAlienyyg_: Next step is to tail /var/log/heat/api-cfn.log on the box running heat-api-cfn, looking for errors09:22
Alienyyg_no errors09:22
shardyAlienyyg_: You can re-run cfn-signal yourself inside the instance09:22
Alienyyg_the last line is start listen 0.0.0.0:800009:22
shardyAlienyyg_: You can kill heat-api-cfn and run it on the CLI in debug mode (-d), or set the loglevel to debug in /etc/heat/heat.conf09:23
openstackgerritThomas Herve proposed a change to openstack/heat: Remove usage of mox class  https://review.openstack.org/6416909:23
*** rushiagr_away is now known as rushiagr09:24
shardyThen you should see the request getting processed in the API logs09:24
*** jistr has joined #heat09:24
Alienyyg_shard: yes, I can turn on the debug level, I forgot it...09:25
*** giulivo has joined #heat09:25
shardyAlienyyg_: In your pastes earlier this week you had some issues with very slow connectivity to the internet, has that been solved?09:25
*** bgorski has joined #heat09:26
Alienyyg_shary: It is Ok now, the administrator set some rules09:26
*** gokrokve has joined #heat09:33
thervechmouel, "ERROR: (ProgrammingError) column "id" cannot be cast to type integer" from the devstack migrate step09:34
chmoueltherve: are you speaking about https://review.openstack.org/#/c/63839/ ?09:34
thervechmouel, Yeah09:34
therveThe error is buried in http://logs.openstack.org/39/63839/6/check/check-tempest-dsvm-postgres-full/756394c/logs/devstacklog.txt.gz09:35
chmoueltherve: oh, its weird that i don't see this error on my vm09:35
chmoueltherve: just saw it in the log09:35
thervechmouel, You're using postgres locally?09:36
chmoueltherve: ah good point09:36
therve:)09:36
chmoueltherve: tks :)09:36
thervechmouel, Isn't there a problem with that migration anyway?09:37
therveCasting a string to an integer isn't going to work09:37
chmouelwell yeah but this doesnt seem to show under myself09:38
chmouelmysql09:38
thervechmouel, You should probably rename the id column instead.09:38
therveI'm not sure how to create an autoincrement column with proper values though09:39
chmoueltherve: proper? you mean default values?09:40
thervechmouel, Well values so that the rows are ordered09:40
chmoueltherve: well let me come to the office so you can explain me :)10:01
*** Alienyyg_ has quit IRC10:19
*** Alienyyg_ has joined #heat10:19
*** nkhare has quit IRC10:31
*** funzo has joined #heat10:42
*** rongze_ has joined #heat10:43
*** nkhare has joined #heat10:44
*** rongze has quit IRC10:46
*** funzo has quit IRC10:47
*** rongze_ has quit IRC10:48
*** rongze has joined #heat11:14
*** rongze_ has joined #heat11:18
*** rongze has quit IRC11:19
openstackgerritA change was merged to openstack/heat: Add test for AutoScalingGroup resize and metadata  https://review.openstack.org/6406011:21
*** rongze_ has quit IRC11:22
Alienyyg_shardy: can you take a look at the firewallr? I know little about  iptables:( ,  http://paste.openstack.org/show/59577/11:39
shardyAlienyyg_: I don't see anything opening port 8000, but I thought you said nmap showed it open from the VM?11:41
Alienyyg_shardy: the 8000 is listen on the machine heat-api-cfn runs11:42
shardyAlienyyg_: You need to check from inside the instance11:43
*** liang has joined #heat11:43
*** funzo has joined #heat11:43
shardyyum -y install nmap && nmap 7.7.7.711:43
Alienyyg_shardy: It seems Ok ,http://paste.openstack.org/show/59593/11:45
shardySo you need to check the api-cfn.logs and see if you can see the request hitting the API11:47
shardyPresumably there will be an error in there, unless the timeout is actually being exceeded11:48
*** funzo has quit IRC11:48
*** Alienyyg__ has joined #heat11:58
*** rpothier__ has joined #heat11:58
*** Alienyyg_ has quit IRC11:59
*** rushiagr is now known as rushiagr_away11:59
*** rpothier_ has quit IRC12:01
*** rongze has joined #heat12:02
Alienyyg__shardy: got it ,I am doing this12:04
*** rushiagr_away is now known as rushiagr12:10
openstackgerritA change was merged to openstack/heat: Use WithScenarios base class instead of load_tests  https://review.openstack.org/6414512:13
Alienyyg__shardy: the loadbalancer now successfully created, I delete two reject rules in iptables before I pasted them to  paste.o.o, so the stack is created ,but in the webserver instance ,the heat-init is failed, the template is for F17,but I am now using F19.cat heat-provision.log12:13
Alienyyg__Provision began: 2014-01-02 12:08:30.14334812:13
Alienyyg__/var/lib/heat-cfntools/cfn-userdata12:13
Alienyyg__#!/bin/bash -v12:13
Alienyyg__/opt/aws/bin/cfn-signal -e 1 -r \"$1\" /opt/aws/bin/cfn-init -s g -r LaunchConfig  --region ap-southeast-112:13
Alienyyg__usage: cfn-signal [-h] [-s SUCCESS] [-r REASON] [-d DATA] [-i UNIQUE_ID]12:13
Alienyyg__                  [-e EXIT_CODE] [--exit EXIT]12:13
Alienyyg__                  url12:13
Alienyyg__cfn-signal: error: unrecognized arguments: --region ap-southeast-112:13
Alienyyg__# Setup MySQL root password and create a user12:13
Alienyyg__mysqladmin -u root password 'admin'12:13
Alienyyg__/var/lib/heat-cfntools/cfn-userdata: line 4: mysqladmin: command not found12:13
Alienyyg__mysql -u root --password='admin' < /tmp/setup.mysql12:13
Alienyyg__/var/lib/heat-cfntools/cfn-userdata: line 5: /tmp/setup.mysql: No such file or directory12:13
Alienyyg__sed -i "/Deny from All/d" /etc/httpd/conf.d/wordpress.conf12:13
Alienyyg__sed: can't read /etc/httpd/conf.d/wordpress.conf: No such file or directory12:14
Alienyyg__sed --in-place --e s/database_name_here/wordpress/ --e s/username_here/admin/ --e s/password_here/admin/ /usr/share/wordpress/wp-config.php12:14
Alienyyg__sed: can't read /usr/share/wordpress/wp-config.php: No such file or directory12:14
Alienyyg__systemctl restart httpd.service12:14
Alienyyg__Failed to issue method call: Unit httpd.service failed to load: No such file or directory. See system logs and 'systemctl status httpd.service' for details.12:14
Alienyyg__# install crontab12:14
Alienyyg__crontab /tmp/stats-crontab.txt12:14
Alienyyg__/tmp/stats-crontab.txt: No such file or directory12:14
therveUrg :/12:14
Alienyyg__Provision done: 2014-01-02 12:08:30.54481512:14
Alienyyg__Provision failed with exit code 112:14
Alienyyg__really sorry for this12:14
therveAlienyyg__, please use a pastebin12:14
Alienyyg__my mistake12:14
shardyAlienyyg__: paste.openstack.org, please.12:14
Alienyyg__OK ,I just enter ctrl12:14
Alienyyg__really sorry for that12:15
shardyAlienyyg__: Some of the templates have not been tested on F19, it may be that some modifications are required, in particular IIRC the httpd config is slightly different12:15
shardycompare the F17 and F19 templates in heat-templates to see what I mean12:16
Alienyyg__shardy: OK ,I will try it. and sorry for my misoperation again12:17
shardyhttps://github.com/openstack/heat-templates/blob/master/cfn/F19/WordPress_NoKey.yaml12:18
shardySee that example, the sedding of /etc/httpd/conf.d/wordpress.conf is different from F1712:18
Alienyyg__shardy: but it get s signal error :http://paste.openstack.org/show/59599/12:21
shardy"error: unrecognized arguments: --region ap-southeast-1"12:22
Alienyyg__shadry: and it result in all the rest errors12:22
shardyWhat template are you using?12:22
Alienyyg__shardy: https://github.com/openstack/heat-templates/blob/master/cfn/F17/AutoScalingMultiAZSample.yaml12:23
Alienyyg__shardy: I had a mistake12:25
Alienyyg__shardy : I did not check the template , I forgot I have modified it12:26
Alienyyg__shardy: sorry for this :(12:26
*** rongze has quit IRC12:36
*** rongze has joined #heat12:37
*** jistr has quit IRC12:39
*** liang has quit IRC12:42
*** rongze has quit IRC12:42
*** funzo has joined #heat12:44
*** nkhare has quit IRC12:47
*** funzo has quit IRC12:49
*** rpothier__ has quit IRC13:02
*** rongze has joined #heat13:03
therveskraynev, Hi, around?13:05
*** jistr has joined #heat13:07
*** aweiteka has joined #heat13:22
openstackgerritA change was merged to openstack/heat: Remove usage of mox class  https://review.openstack.org/6416913:28
*** sgordon has joined #heat13:35
*** sgordon has quit IRC13:35
*** sgordon has joined #heat13:35
*** funzo has joined #heat13:45
*** jcru has joined #heat13:46
*** funzo has quit IRC13:49
*** bvandenh has joined #heat13:57
*** vijendar has joined #heat13:57
therveshardy, Unfortunately it still doesn't work with nose :/13:57
shardytherve: :(13:57
*** rpothier__ has joined #heat13:59
bgorskitherve, What does not work with nose?13:59
shardybgorski: Any test using magic like testscenarios, which makes it a lot less convenient to e.g put a print in the code to see what's going on14:01
*** pafuent has joined #heat14:04
*** therve_ has joined #heat14:05
*** therve_ has quit IRC14:05
*** jprovazn has quit IRC14:05
*** urulama has quit IRC14:14
*** erkules_ is now known as erkules14:26
*** jprovazn has joined #heat14:28
*** sabeen has joined #heat14:31
*** jasond` has quit IRC14:34
openstackgerritA change was merged to openstack/heat: Remove redundant word from OS::Nova::Server  https://review.openstack.org/6456614:38
*** ruhe has joined #heat14:42
*** funzo has joined #heat14:45
*** funzo has quit IRC14:49
*** andersonvom has joined #heat14:52
*** ruhe has quit IRC14:55
*** jasond` has joined #heat15:04
*** sjmc has joined #heat15:07
*** sabeen has quit IRC15:09
*** sabeen has joined #heat15:10
andersonvomMorning, folks! Happy 2014!15:14
*** jpeeler has joined #heat15:16
*** jpeeler has quit IRC15:16
*** jpeeler has joined #heat15:16
*** zaneb has joined #heat15:16
*** tomek_adamczewsk has joined #heat15:16
*** spzala has joined #heat15:17
openstackgerritA change was merged to openstack/heat: Enhance message for watch rule  https://review.openstack.org/6449515:18
zanebhappy new year y'all15:21
shardyHi zaneb & andersonvom, happy new year :)15:21
shardyHey zaneb, I assigned bug #1265209 to you as it seemed related to the Schema rework, hope that's OK15:22
uvirtbotLaunchpad bug 1265209 in heat "heat resource-template fails for most resources" [High,Triaged] https://launchpad.net/bugs/126520915:22
zanebhey shardy :)15:22
shardyI had a look but it wasn't that clear to me what the best fix was15:22
spzalaHappy New Year zaneb and all15:22
zaneblooking...15:22
bgorskithx and Happy New Year for you too15:23
zanebshardy: so generating the template fails? or trying to use it for something fails?15:24
shardyzaneb: Generating the template fails15:24
zanebok thanks, I'll look into it15:24
spzalabgorski thanks :)15:24
shardyzaneb: Thanks, seems like we need to somhow abstract away the legacy keynames, as various places seem to be trying to use the SCHEMA_KEYS keys to look up things in the Schema object15:31
shardymostly seems to be in _generate_input but I only had a quick look15:32
zanebyeah, we just need to switch it over to reading the Schema objects15:32
*** tomek_adamczewsk has quit IRC15:32
zanebI forgot that hadn't been done already15:32
zanebI did it everywhere else (e.g. in generating docs)15:32
zanebbefore we started creating resources with the new schema15:32
zanebbut missed this one15:33
shardyzaneb: Cool, I figured you'd probably know what needed doing, I'm not that familiar with that code yet15:33
*** rongze has quit IRC15:34
*** bvandenh has quit IRC15:36
*** gokrokve has quit IRC15:41
*** gokrokve has joined #heat15:41
*** funzo has joined #heat15:44
*** rongze has joined #heat15:44
andersonvomshardy: just looking at your comments now (https://review.openstack.org/#/c/63041/). do you have a min?15:47
shardyandersonvom: sure15:48
andersonvomshardy: so, I thought of adding the tenant scoping at the database level, but it felt slightly weird for the database to know about the current user (i.e. is the user a cloud admin?). and it felt slightly better to wrap the engine methods with the appropriate filtering, though still annoying.15:51
andersonvomshardy: what do you think?15:51
shardyWell my thinking is the context contains everything we need to scope the query at the DB API level15:51
*** sdake has joined #heat15:52
*** sdake has quit IRC15:52
*** sdake has joined #heat15:52
shardyso doing it in one place seems potentially less error prone than expecting every place which queries stuff to be appropriately decorated15:52
shardyandersonvom: If you look at the sqlalchemy in e.g nova, they do quite a bit of context specific filtering in the model_query15:53
sdakemorning15:53
shardyHi sdake15:53
sdakehey shardy15:53
bgorskisdake, Hi, Happy New Year15:54
shardyandersonvom: So my thinking is we should set all the data we need to appropriately scope the data returned in the context, then enforce that scope in the DB API15:54
*** lsmola_ has quit IRC15:56
andersonvomshardy: that was precisely my thinking to put it there too, but it still feels like we're "leaking" information to the database15:57
shardyandersonvom: how?15:57
shardyThe DB API isn't "the database", it's just our interface for querying it15:58
*** alexheneveld has joined #heat15:59
andersonvomshardy: in the sense that the context, as is it today, can almost be removed from the db_api without losing anything (just session caching), so it's as if it wasn't even "supposed" to be there16:01
*** Linz has quit IRC16:03
thervezaneb, Hi! Could we get your feedback on https://bugs.launchpad.net/heat/+bug/1264933 ?16:03
uvirtbotLaunchpad bug 1264933 in heat "The result is inconsistent on identify_stack()" [Undecided,In progress]16:03
andersonvomshardy: I'm really playing the devil's advocate here, since I'm leaning towards agreeing with you.  This is just me thinking out loud =P16:03
*** Linz has joined #heat16:04
zanebtherve: you're correct, it's intentional16:04
thervezaneb, OK cool. Maybe we should document it?16:04
therveI think it makes sense, but the question will reappear :)16:05
shardyandersonvom: well there are arguments either way I guess, but my feeling atm is that pushing the context-scoping into the lowest level (ie the DB API) makes sense, because then every layer above doesnt have to care about requesting stuff scoped according to what is already known in the context16:05
shardye.g we can kill all of that get_foo_by_tenant stuff16:06
andersonvomshardy: true! I think I can agree with that16:06
shardyandersonvom: That seems to fit the pattern of what I've seen in some other projects, but I'm happy to hear other ideas :)16:06
shardyandersonvom: Also, I'm still uncomfortable with the whole idea of API requests which can defeat tenant scoping, so I think making everything tenant-scoped by default at the lowest level somewhat minimises the risk of security disasters ;)16:10
*** ifarkas has quit IRC16:11
andersonvomshardy: that is indeed a very compelling argument to keep it there.16:11
*** rushiagr has quit IRC16:12
*** Linz has quit IRC16:16
openstackgerritDennyZhang proposed a change to openstack/heat: Remove vim header  https://review.openstack.org/6469116:17
SpamapSshardy: regarding OS::Heat::UpdateWaitConditionHandle vs. a new native one.. can we still just subclass AWS::CloudFormation::WaitConditionHandle until we need a different implementation? I really quite like the current approach for simple things and would rather we keep it around for the mere fact that I think it is the most primitive way to do what is being done...16:17
zanebshardy: there was a point at which we needed it (when e.g. waitcondition requests were not authed with keystone). It definitely does increase the risk of disaster, though, so +1 to pushing it down to the db level if we can now do that16:18
openstackgerritDennyZhang proposed a change to openstack/heat: quit when error happens in cinder-keystone-setup  https://review.openstack.org/6469216:18
*** jprovazn has quit IRC16:19
therveSpamapS, You mean subclass for code reuse purpose?16:20
SpamapStherve: well and also "works like X except Y" in documentation :)16:21
SpamapShttps://review.openstack.org/#/c/63245/ for context16:21
zanebbgorski: congrats on making core! just saw the thread... late to the party but +1 :)16:21
therveYeah I saw the review16:21
therveSpamapS, My only concern would be to have a "native" resource using the CFN API16:22
bgorskizaneb, thx16:22
therveBut otherwise It seems ok16:22
shardySpamapS: I think we needa native resource which uses the native API16:22
shardybut obviously we're still working out the auth for that, ref instance-users which I've been looking at16:23
*** blamar has joined #heat16:23
*** jergerber has joined #heat16:23
SpamapSWhat is the danger of letting native things exist in the cfn API? We have.. a lot of that already.16:23
shardySpamapS: What i'd prefer to avoid is having some short-term-hack derived class, then another native solution16:24
SpamapSwell the 'then another' is only a danger if somebody decides to do that and more people decide to +2 it...16:24
therveSpamapS, *only* existing there though16:24
shardySpamapS: One of the main issues is environments where they don't want to use cfn compatible API extensions, like ec2tokens, and heat-api-cfn I guess16:25
shardys/they/users I've spoken to16:25
SpamapSshardy: those environments will get zero in-instance capability right now.. so.. until there is a native in-instance API.. I'm inclined to tell them to step up and help or punt.16:25
shardySpamapS: well that's the problem I'm trying to solve with instance-users16:26
SpamapSI really dislike "we can't have this functionality because some day very soon we will have something else" unless that some day very soon is "when this patch topic lands"16:26
zanebshardy: both native and AWS waitconditions will need to use pre-signed URLs though... won't they?16:27
shardyzaneb: No, I thought we were going with the domain-user idea, where we create users (as the heat service user) in a heat specific domain, with a project per stack16:27
openstackgerritDennyZhang proposed a change to openstack/heat: Improve tools/uninstall-heat  https://review.openstack.org/6469416:27
shardyzaneb: then we deploy a randomly generated username/password in the instance and use normal user/pass auth16:28
SpamapSshardy: you understand my frustration right? I'm carrying this patch now.. understanding fully well that it may not be the end solution.. but if the end solution is a whole new wait condition mechanism, I have to just carry this patch for a lot longer.16:28
zanebshardy: so the userdata script will have to use some in-instance tool, not just curl to a URL?16:28
openstackgerritDennyZhang proposed a change to openstack/heat: Improve tools/uninstall-heat  https://review.openstack.org/6469416:28
SpamapSzaneb: curl can provide the username/pass16:29
SpamapSor is it going to have to be multi-step with tokens and stuff?16:29
therveSpamapS, Maybe just add a property to WaitConditionHandle?16:29
shardyzaneb: No, we'll just have e.g an attribute of a native waitcondition handle which generates a curl call containing the appropriate headers16:29
SpamapStherve: thats basically what I did, but I made it native because I don't want people using pure CFN to think that will port back to amazon.16:29
therveYeah pick your poison...16:30
therveSpamapS, We have extensions already though...16:30
zanebshardy: right, so a native WaitCondition and an AWS WaitCondition should ultimately have the same implementation? the only work to be done is changing what the URL looks like?16:30
SpamapStherve: right, and I think those were bad ideas in hindsight.16:30
*** jprovazn has joined #heat16:30
zanebif that is the case, I see no reason not to implement a native one now16:30
shardySpamapS: I understand your position, maybe we should just merge your patch, but what do we do when we create something which supersedes it, we have to support three different resources16:31
therveSpamapS, I agree it's bad, but so is having a native resource with awsism16:31
* zaneb goes to find SpamapS's patch...16:31
thervezaneb, Well to implement a total native one we need a new API16:32
*** jistr has quit IRC16:32
shardyzaneb: Yeah, I think we'll probably want both to use the same method, although I was thinking initially we'd leave the AWS one alone, and just create the pre-signed URL using the new domain users and ec2-keypair16:32
therveAnd we need working credential mechanism :)16:32
zanebtherve: but that's an implementation detail as far as the template author is concerned16:32
shardytherve: well if we use username/password, then we have one :)16:32
thervezaneb, Fair enough16:32
therveWell is it though?16:33
zanebshardy: either way, it's the same thing in the template16:33
zanebjust a different URL comes out16:33
thervezaneb, Template authors need a way to talk to it though in the userdata or something16:33
shardyzaneb: I was thinking that currently all users are probably using cfn-signal not curl directly, so we could just modify that to look for the user/pass in nova metadata16:33
shardyAnd have a native version which works with a curl string generated via a resource attribute16:34
SpamapSshardy: what to do later, we deprecate it and move on. :)16:34
shardySpamapS: I guess that's my main issue, we've not yet worked out how to deprecate resources, and your implemenation doesn't look like a long term thing16:35
shardySo if, as I anicipate, we work out the instance-users and native wait handle thing for icehouse GA, we ship duplicate functionality16:35
SpamapSshardy: it is surely worth keeping long term if wait conditions are worth keeping long term.16:35
zanebSpamapS: I feel your pain, but deprecating resources also sucks for users. And is a PITA for us, for that matter.16:36
SpamapSshardy: heh, duplicate functionality.. it adds 11 lines including whitespace (and 45 lines of tests...)16:36
shardymeh16:36
*** funzo has quit IRC16:37
SpamapSalso it is a natural need for users who will likely be using the AWS waitconditions for as long as Heat will support them doing so.16:37
*** funzo has joined #heat16:37
shardySpamapS: So what will be the barrier to you using a native alternative?16:38
SpamapSLike right now people aren't using AWS waitconditions so they can go back to EC2.. they're using it because nothing else exists in Heat.16:38
SpamapSshardy: time16:38
SpamapSshardy: also not spending time on this frees me up to focus on retries and rolling updates.16:38
SpamapSwhich I need even more desperately :)16:38
SpamapSI want to be supportive and use the new stuff.16:39
SpamapSI just don't know what the state is or when it will be ready. Meanwhile I'm trying to get CD deploying of OpenStack using Heat working.16:40
zanebSpamapS: does your patch even do anything? surely the update code will not call handle_update() if nothing has changed (and there aren't even any properties *to* change). The existing WaitConditionHandle already raises UpdateReplace if it were to actually get updated16:40
*** wirehead_1 is now known as wirehead_16:41
SpamapSzaneb: I read the code as handle_update would be called with no diff, but I may have been hasty in my reading of it.16:42
jasond`would someone mind giving me a general thumbs up/down on the approach here (before i start on unit tests)? https://review.openstack.org/#/c/63002/16:42
*** funzo has quit IRC16:42
zanebSpamapS: https://github.com/openstack/heat/blob/master/heat/engine/resource.py#L47916:42
SpamapSzaneb: so is there no way to have a resource updated on every update?16:43
zanebSpamapS: I confess the algorithm was somewhat clearer before this: https://github.com/openstack/heat/commit/b46a4ad44222a284d372b491c7f63f0c39126d7616:43
zanebSpamapS: well the good news is that that ^ patch means you can override update() to do that16:44
zaneb(some refactoring advised)16:47
*** pscheie_ is now known as pscheie16:47
*** therve_ has joined #heat16:47
*** therve_ has quit IRC16:47
zanebhmm, ChangeIds in review comments don't turn into links any more :( what gives?16:48
*** Linz has joined #heat16:48
*** rongze has quit IRC16:48
*** radez_g0` is now known as radez16:48
SpamapSzaneb: so you're saying I should be overriding update instead of handle_update16:49
*** rongze has joined #heat16:49
SpamapSI admit, I am running that patch, but 5 or 6 other things have been exploding so I haven't actually been able to verify it does wait through the update16:49
SpamapSdurn holidays making it hard to get patches in. :)16:49
zanebSpamapS: yes... come to think of it, just override update to raise UpdateReplace and you are done16:49
SpamapSzaneb: that actually sounds more like what I wanted anyway. :)16:50
zaneb:)16:51
openstackgerritDennyZhang proposed a change to openstack/heat: Remove vim header  https://review.openstack.org/6469116:54
openstackgerritDennyZhang proposed a change to openstack/heat: log warning, when $SERVICE_HOST is localhost  https://review.openstack.org/6469716:59
*** funzo has joined #heat17:00
*** funzo has quit IRC17:00
*** funzo_ has joined #heat17:00
*** lsmola_ has joined #heat17:01
*** Linz has quit IRC17:09
*** Linz has joined #heat17:13
*** Linz has quit IRC17:14
*** Linz has joined #heat17:15
*** rongze has quit IRC17:17
SpamapSshardy: ok so I just confirmed zaneb's analysis that my patch actually is doing nothing for my CD cloud at the moment. ;)17:18
*** rongze has joined #heat17:18
SpamapSshardy: sorry for being difficult, but I really do need this "soon". :-P17:18
zanebSpamapS: I know you're going to hate this suggestion, but you could always carry that patch as a plugin17:19
* zaneb ducks17:19
SpamapSshardy: it sounds like there is an ordering problem for the long term viability which is that we need native in-instance stuff, and then most likely stevebaker's hot-software-config wait condition enhancements will encompass this need.17:19
SpamapSzaneb: no, I can't.17:19
*** rongze has joined #heat17:19
SpamapSzaneb: Official deployment program.. use openstack.. etc.17:19
zanebyeah, fair enough I guess17:20
SpamapSif the patch is really blocked then I have to spin-wait on external polling of services17:20
SpamapSwhich I actually do already.. but I have to grow the timeout _a lot_17:20
zanebalthough, we do provide a plug-in architecture for a reason17:20
shardySpamapS: Sure, I'm also not trying to be difficult, I'm just trying to look towards what we're planning to deliver for Icehouse, to make sure it makes sense, is maintainable, and won't just confuse users17:21
therve*cough* What about contrib?17:21
zanebtherve: ++17:22
therveI don't know what we said about supportability of that17:22
shardyI, personally, would rather not be answering a million "which wait handle resource" do I use questions from uysers ad-ininitum17:22
shardySpamapS: But if there is general agreement it should go in, I won't block your patch :)17:23
zanebSpamapS: I have reservations about this approach too... basically waitconditions suck on update, because they have no way of knowing which resources that reference them are being replaced. And this doesn't do anything to fix that, the user still has to figure out how many servers are going to be replaced and handle it manually17:23
zanebso other than increasing the number of possible ways of doing things that Don't Work, I'm not sure what it achieves17:24
SpamapSzaneb: that assumes that the user doesn't set things up to just refresh wait condition handles whenever anything changes17:25
SpamapSzaneb: this handle assumes that the user has set things up to do just that.17:26
zanebSpamapS: you're assuming a count of 1?17:26
SpamapSzaneb: since the count is "number of unique ids" .. it should work out just fine as long as the user has set their in-instance bits up to always refresh, not just on first boot17:27
*** rwsu has joined #heat17:27
SpamapSzaneb: I'm assuming in-instance bits know when the wait condition handle has changed and will refresh it.17:28
zanebSpamapS: thinking more about this... maybe raising UpdateReplace is not what you want, because that will change the URL and therefore force all of the servers using it to be replaced17:28
SpamapSno17:28
SpamapSbecause I don't put the handle in the server userdata17:28
SpamapSit is in Metadata17:28
SpamapSwhich does not force replacement on change17:29
SpamapS(and that is the whole point :)17:29
zanebso all of the instances re-signal when they get new metadata17:30
zanebthat seems like it solves the problem quite neatly17:31
zanebit also sounds really hard to explain to a user :/17:32
*** funzo_ has quit IRC17:32
openstackgerritDennyZhang proposed a change to openstack/heat: fix typo in heat-api  https://review.openstack.org/6469917:36
*** gokrokve has quit IRC17:44
*** gokrokve has joined #heat17:44
*** rongze has quit IRC17:45
*** gokrokve_ has joined #heat17:45
*** rongze has joined #heat17:47
*** gokrokve has quit IRC17:48
*** harlowja has joined #heat17:49
SpamapSzaneb: I don't think it is all that hard to explain to a user. "All consumers of the wait condition handle must signal success before the wait condition can continue on create or update." Did I miss something?17:52
SpamapSzaneb: how to achieve that is also pretty simple if the wait condition url changes on update.17:53
*** derekh has quit IRC17:53
SpamapSsince they will intrinsically see a new handle that they have not signalled yet.17:53
SpamapSthough TripleO just signals every time any metadata changes for an even simpler implementation17:54
zanebSpamapS: well, you have to explain that this resource only works if you reference the waitconditionhandle in the metadata, not in the user data, and that you need to have something set up on the instance that resends the data whenever the metadata is updated, and BTW that's all completely different to how this other waitconditionhandle over here works17:54
SpamapSzaneb: I don't really have to explain that for just this. I have to explain that for any case that wants to do things using updates without replacing servers.17:56
SpamapSits not really "completely" different. The other one just drops the "or update" from my original sentence.17:56
*** tomek_adamczewsk has joined #heat18:03
*** rongze has quit IRC18:09
*** tomek_adamczewsk has quit IRC18:13
*** alexheneveld has quit IRC18:14
*** chandankumar has quit IRC18:38
*** nati_ueno has joined #heat18:39
*** rongze has joined #heat18:40
*** yogesh has joined #heat18:41
*** rongze has quit IRC18:45
*** nati_ueno has quit IRC18:47
*** nati_ueno has joined #heat18:49
*** lsmola_ has quit IRC18:52
*** tomek_adamczewsk has joined #heat18:54
*** nati_ueno has quit IRC18:54
*** gokrokve_ has quit IRC18:59
*** rpothier__ has quit IRC18:59
*** sileht has quit IRC18:59
*** pafuent1 has joined #heat19:01
*** swygue has quit IRC19:01
*** swygue has joined #heat19:02
*** rongze has joined #heat19:03
*** pafuent has quit IRC19:03
*** gokrokve_ has joined #heat19:05
*** rpothier__ has joined #heat19:05
*** sileht has joined #heat19:05
*** jasond` has quit IRC19:10
*** sgordon has quit IRC19:10
*** jasond` has joined #heat19:12
*** rongze has quit IRC19:13
*** spzala has quit IRC19:24
*** sgordon has joined #heat19:25
*** larsks has quit IRC19:35
*** tomek_adamczewsk has quit IRC19:38
*** sabeen has quit IRC19:46
*** spzala has joined #heat20:07
*** bgorski has quit IRC20:11
SpamapSzaneb: https://bugs.launchpad.net/bugs/1264933 <-- is it possible we're missing some documentation there?20:14
uvirtbotLaunchpad bug 1264933 in heat "The result is inconsistent on identify_stack()" [Undecided,Invalid]20:14
*** jprovazn has quit IRC20:14
*** gokrokve_ has quit IRC20:26
*** pafuent1 has quit IRC20:27
openstackgerritVijendar Komalla proposed a change to openstack/heat: Implement adopt-stack for nested stacks  https://review.openstack.org/6472020:28
openstackgerritVijendar Komalla proposed a change to openstack/heat: Implement adopt-stack  https://review.openstack.org/6273020:29
*** gokrokve has joined #heat20:35
*** pafuent has joined #heat20:39
openstackgerritVijendar Komalla proposed a change to openstack/heat: Add _() to rackspace resource strings  https://review.openstack.org/6292220:39
*** gokrokve has quit IRC20:40
zanebSpamapS: yes, entirely possible :)20:45
*** spzala has quit IRC20:49
*** radez is now known as radez_g0n320:51
*** gokrokve has joined #heat20:57
*** gokrokve_ has joined #heat20:59
*** gokrokve has quit IRC21:01
*** gokrokve_ has quit IRC21:03
*** rongze has joined #heat21:12
openstackgerritVijendar Komalla proposed a change to openstack/python-heatclient: python-heatclient stack-abandon support  https://review.openstack.org/6292521:12
openstackgerritVijendar Komalla proposed a change to openstack/heat: update rackspace clients.py  https://review.openstack.org/6248321:13
*** scollier has quit IRC21:14
*** gokrokve has joined #heat21:14
*** pafuent has left #heat21:16
*** rongze has quit IRC21:17
sdakegot record player for chrismas21:19
sdakevinyl ftw!21:19
sdakeswapping records kind of a hassle21:20
sdakebut we really gave up something when we went to all digital music..21:20
openstackgerritA change was merged to openstack/heat: Rename scaleout_apis.rst -> scale_deployment.rst  https://review.openstack.org/6396221:22
*** scollier has joined #heat21:26
*** sabeen has joined #heat21:35
asalkeld_morning21:40
*** aweiteka has quit IRC22:02
*** spzala has joined #heat22:03
andersonvomasalkeld_: morning! happy new year!22:06
*** rongze has joined #heat22:12
*** rongze has quit IRC22:17
asalkeld_you too andersonvom22:24
*** vijendar has quit IRC22:24
*** DaveJ_ has quit IRC22:32
*** rwsu has quit IRC22:33
*** rwsu has joined #heat22:40
sdakestevebaker pingola22:43
*** jcru has quit IRC22:47
*** Linz has quit IRC22:49
*** Linz has joined #heat22:50
*** alexheneveld has joined #heat22:50
*** rpothier__ has quit IRC22:52
*** Linz has quit IRC22:53
*** Linz has joined #heat22:53
*** jasond` has quit IRC22:56
*** rwsu has quit IRC22:59
*** jasond` has joined #heat23:12
*** rwsu has joined #heat23:13
*** rongze has joined #heat23:13
*** rongze has quit IRC23:18
*** sjmc has quit IRC23:20
*** Linz has quit IRC23:21
*** Linz has joined #heat23:21
*** sgordon has quit IRC23:22
*** gokrokve has quit IRC23:27
*** alexheneveld has quit IRC23:30
*** gokrokve has joined #heat23:34
*** sgordon has joined #heat23:35
*** Linz has quit IRC23:39
*** Linz has joined #heat23:39
*** Linz_ has joined #heat23:40
*** Linz has quit IRC23:43
*** rpothier__ has joined #heat23:50
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Unscoped List Stacks  https://review.openstack.org/6304123:52
openstackgerritAngus Salkeld proposed a change to openstack/heat: Use oslo db.migration script  https://review.openstack.org/6475023:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!