*** lindsayk1 has quit IRC | 00:03 | |
*** IlyaE has quit IRC | 00:09 | |
*** gokrokve has quit IRC | 00:15 | |
*** tango has quit IRC | 00:15 | |
*** kebray has quit IRC | 00:25 | |
openstackgerrit | A change was merged to openstack/heat: Catch NotFound for every stack delete db operation https://review.openstack.org/103715 | 00:26 |
---|---|---|
*** lindsayk1 has joined #heat | 00:31 | |
*** lindsayk1 has quit IRC | 00:31 | |
*** lindsayk1 has joined #heat | 00:31 | |
*** packet has quit IRC | 00:33 | |
*** IlyaE has joined #heat | 00:38 | |
*** metral is now known as metral_zzz | 00:43 | |
*** randallburt has quit IRC | 00:50 | |
*** Yanyan has joined #heat | 00:51 | |
*** randallburt has joined #heat | 00:53 | |
*** gokrokve has joined #heat | 00:55 | |
*** IlyaE has quit IRC | 00:58 | |
*** ccrouch has joined #heat | 00:59 | |
*** liusheng-leave is now known as liusheng | 01:00 | |
*** kebray has joined #heat | 01:01 | |
*** aweiteka has joined #heat | 01:02 | |
*** Qiming has joined #heat | 01:03 | |
*** tiantian has joined #heat | 01:10 | |
*** nosnos has joined #heat | 01:13 | |
*** Qiming has quit IRC | 01:15 | |
*** nosnos has quit IRC | 01:20 | |
*** randallburt has quit IRC | 01:22 | |
*** aweiteka has quit IRC | 01:26 | |
*** IlyaE has joined #heat | 01:27 | |
*** IlyaE has quit IRC | 01:28 | |
*** sorantis has joined #heat | 01:34 | |
*** gokrokve has quit IRC | 01:35 | |
*** gokrokve has joined #heat | 01:43 | |
*** Qiming has joined #heat | 01:55 | |
*** blamar has joined #heat | 02:01 | |
*** daneyon has quit IRC | 02:02 | |
*** sorantis has quit IRC | 02:02 | |
*** swygue has joined #heat | 02:04 | |
*** gokrokve has quit IRC | 02:11 | |
*** gokrokve has joined #heat | 02:11 | |
*** shakamunyi has joined #heat | 02:14 | |
*** m_22 has quit IRC | 02:23 | |
*** Dev_Jin has joined #heat | 02:25 | |
*** Dev_Jin has quit IRC | 02:25 | |
*** killer_prince has quit IRC | 02:26 | |
*** alexheneveld has quit IRC | 02:27 | |
*** alexheneveld has joined #heat | 02:27 | |
elynn | morning all :) | 02:37 |
*** ramishra has joined #heat | 02:37 | |
*** lindsayk1 has quit IRC | 02:49 | |
*** lindsayk1 has joined #heat | 02:50 | |
*** lindsayk1 has quit IRC | 02:54 | |
openstackgerrit | liusheng proposed a change to openstack/python-heatclient: Use None instead of mutables in method params defaults https://review.openstack.org/104423 | 02:54 |
*** arbylee has joined #heat | 02:57 | |
ramishra | stevebaker: Hi | 03:01 |
stevebaker | ramishra: hi | 03:02 |
ramishra | stevebaker: I have a query on heatclient patches breaking tempest tests check-tempest-dsvm-full. | 03:05 |
*** JAGDISH__ has quit IRC | 03:05 | |
stevebaker | that would be bad | 03:05 |
*** JAGDISH__ has joined #heat | 03:05 | |
*** JAGDISH__ has quit IRC | 03:06 | |
ramishra | stevebaker: I can create a tempest patch. those are simple cli tests. how they would be merged.. | 03:06 |
stevebaker | ramishra: you mean heatclient as a cli or heatclient as a lib? | 03:07 |
ramishra | stevebaker: as cli, if you can have a quick look at https://review.openstack.org/#/c/104118/ | 03:07 |
ramishra | stevebaker: this breaks SimpleReadOnlyHeatClientTest, I have the fix for that but not sure how they would be merged.. in what order | 03:08 |
*** killer_prince has joined #heat | 03:10 | |
*** killer_prince is now known as lazy_prince | 03:10 | |
stevebaker | ramishra: by deprecating resource-template instead of deleting it, then the tempest test can switch over at leisure | 03:10 |
stevebaker | ramishra: we still have deprecated commands in heatclient since forever, and my muscle-memory still types them ;) | 03:11 |
ramishra | stevebaker: ok:) I got it. thanks.. will do that.. | 03:12 |
ramishra | stevebaker: Did u manage to have a look at https://review.openstack.org/#/c/102616/? Any more feedback on it? | 03:14 |
stevebaker | ramishra: I'll take a look. I'm currently playing with a docker hook | 03:14 |
ramishra | stevebaker: gr8. thanks.. | 03:15 |
stevebaker | ramishra: I'm wondering what is the best way to get some unit testing on those hooks. | 03:15 |
ramishra | stevebaker: I did think about it while writing it. but did not spend time on it;) | 03:17 |
stevebaker | We could consider doctest | 03:18 |
ramishra | ok | 03:22 |
*** dims has quit IRC | 03:32 | |
*** mohits has joined #heat | 03:32 | |
cmyster | morning | 03:33 |
*** ramishra has quit IRC | 03:33 | |
*** arbylee has quit IRC | 03:35 | |
*** avquadri has joined #heat | 03:39 | |
*** ramishra has joined #heat | 03:40 | |
*** zhiyan_ is now known as zhiyan | 03:47 | |
*** nosnos has joined #heat | 03:52 | |
*** ramishra has quit IRC | 03:52 | |
*** ramishra has joined #heat | 03:53 | |
*** arbylee has joined #heat | 03:55 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement events pagination, sorting and filtering https://review.openstack.org/104439 | 04:09 |
*** ajc_ has joined #heat | 04:17 | |
*** avquadri has quit IRC | 04:17 | |
*** avquadri has joined #heat | 04:17 | |
*** rushiagr has joined #heat | 04:20 | |
*** sab has joined #heat | 04:21 | |
*** avquadri has quit IRC | 04:22 | |
*** shakamunyi has quit IRC | 04:23 | |
*** shakamunyi has joined #heat | 04:24 | |
*** cmyster has quit IRC | 04:24 | |
*** rushiagr_ has joined #heat | 04:26 | |
*** blamar has quit IRC | 04:28 | |
*** rushiagr has quit IRC | 04:28 | |
*** cmyster has joined #heat | 04:30 | |
*** avquadri has joined #heat | 04:32 | |
*** bvivek has joined #heat | 04:43 | |
*** bvandenh has joined #heat | 04:45 | |
*** rakesh_hs has joined #heat | 04:46 | |
*** sab has quit IRC | 04:48 | |
*** blamar has joined #heat | 04:50 | |
*** ramishra_ has joined #heat | 04:50 | |
*** sab has joined #heat | 04:51 | |
*** ramishr__ has joined #heat | 04:51 | |
*** ramishra has quit IRC | 04:53 | |
*** ramishra_ has quit IRC | 04:55 | |
*** mohits has quit IRC | 04:55 | |
*** mohits has joined #heat | 04:58 | |
*** Dev_Jin has joined #heat | 05:00 | |
*** nati_ueno has joined #heat | 05:02 | |
*** blamar has quit IRC | 05:05 | |
*** ramishr__ has quit IRC | 05:08 | |
*** ramishra has joined #heat | 05:08 | |
*** IlyaE has joined #heat | 05:13 | |
*** loki184 has joined #heat | 05:14 | |
*** tiantian has quit IRC | 05:18 | |
*** achampion has joined #heat | 05:26 | |
*** sabeen has joined #heat | 05:27 | |
*** sabeen has quit IRC | 05:27 | |
*** unmeshg has joined #heat | 05:29 | |
*** gokrokve has quit IRC | 05:30 | |
*** robsparker has quit IRC | 05:31 | |
*** PragadeeswaranS has joined #heat | 05:33 | |
*** shakamunyi has quit IRC | 05:33 | |
*** mohits_ has joined #heat | 05:33 | |
*** mohits has quit IRC | 05:35 | |
*** saju_m has joined #heat | 05:37 | |
*** nkhare has joined #heat | 05:38 | |
*** IlyaE has quit IRC | 05:40 | |
*** robsparker has joined #heat | 05:46 | |
*** tomek_adamczewsk has joined #heat | 05:47 | |
*** harlowja is now known as harlowja_away | 05:47 | |
*** ishant has joined #heat | 05:49 | |
openstackgerrit | Rakesh H S proposed a change to openstack/heat: Fixed stack never changed from DELETE_IN_PROGRESS https://review.openstack.org/104454 | 05:50 |
*** tomek_adamczewsk has quit IRC | 05:53 | |
*** tomek_adamczewsk has joined #heat | 05:54 | |
*** gokrokve has joined #heat | 06:00 | |
*** mohits_ is now known as mohits | 06:02 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/89750 | 06:02 |
*** Qiming_ has joined #heat | 06:02 | |
*** Qiming has quit IRC | 06:05 | |
*** mohits_ has joined #heat | 06:10 | |
*** mohits has quit IRC | 06:11 | |
*** jstrachan has joined #heat | 06:12 | |
*** IlyaE has joined #heat | 06:12 | |
*** gokrokve has quit IRC | 06:16 | |
*** mohits_ is now known as mohits | 06:20 | |
*** nati_ueno has quit IRC | 06:20 | |
*** mohits has quit IRC | 06:20 | |
*** mohits has joined #heat | 06:20 | |
*** tiantian has joined #heat | 06:20 | |
*** chandan_kumar has joined #heat | 06:25 | |
*** lazy_prince is now known as killer_prince | 06:28 | |
*** IlyaE has quit IRC | 06:30 | |
*** jcoufal has joined #heat | 06:31 | |
*** jamiehannaford has joined #heat | 06:33 | |
*** Qiming__ has joined #heat | 06:34 | |
*** Qiming_ has quit IRC | 06:37 | |
shardy | morning all | 06:37 |
Qiming__ | morning, sir | 06:38 |
*** killer_prince is now known as lazy_prince | 06:39 | |
cmyster | morning | 06:46 |
cmyster | in early today eh? | 06:46 |
shardy | ha, yeah a bit early I guess ;) | 06:48 |
*** tspatzier has joined #heat | 06:51 | |
openstackgerrit | lawrancejing proposed a change to openstack/heat: Fix the section name in CONTRIBUTING.rst https://review.openstack.org/103767 | 06:51 |
openstackgerrit | Rabi Mishra proposed a change to openstack/python-heatclient: Alter misleading cli command 'resource-template' https://review.openstack.org/104118 | 06:52 |
*** jprovazn has joined #heat | 06:53 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Fix failed to delete the EIP resource https://review.openstack.org/102758 | 06:53 |
*** mohits has quit IRC | 06:55 | |
*** arbylee has quit IRC | 06:56 | |
*** pasquier-s has joined #heat | 07:02 | |
*** pasquier-s_ has joined #heat | 07:07 | |
*** Qiming__ is now known as Qiming | 07:09 | |
*** julienvey has joined #heat | 07:10 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement events pagination, sorting and filtering https://review.openstack.org/104439 | 07:11 |
*** pasquier-s has quit IRC | 07:12 | |
Qiming | SpamapS: hi | 07:12 |
*** gokrokve has joined #heat | 07:12 | |
Qiming | shardy: ? | 07:14 |
*** sgordon` has quit IRC | 07:15 | |
*** sgordon` has joined #heat | 07:15 | |
shardy | Qiming: Hi | 07:16 |
Qiming | shardy, since it's late night for SpamapS, maybe you can help point me to an architecture overview diagram of convergence + observer design? | 07:17 |
*** gokrokve has quit IRC | 07:17 | |
shardy | Qiming: I don't think there is one, although I did request one in my review of one of the specs | 07:17 |
Qiming | it is a little bit hard for me to picture what the whole framework will look like from the specs | 07:18 |
* shardy goes to look at the latest revision | 07:18 | |
skraynev | Morning | 07:18 |
shardy | I agree, which is why I asked SpamapS to add a diagram to the spec | 07:18 |
Qiming | shardy: :) One digram is more than a thousand words | 07:18 |
shardy | Qiming: there are a lot more words in the summit etherpad, if that helps: | 07:18 |
shardy | https://etherpad.openstack.org/p/heat-workflow-vs-convergence | 07:18 |
shardy | Qiming: Yes, I agree ;) | 07:18 |
Qiming | shardy: the problem is indeed too many words without a picture, :P | 07:19 |
Qiming | shardy: thanks, rereading the etherpad text now | 07:20 |
*** tomek_adamczewsk has quit IRC | 07:20 | |
*** nati_ueno has joined #heat | 07:20 | |
*** tomek_adamczewsk has joined #heat | 07:23 | |
*** lsmola__ has joined #heat | 07:24 | |
EmilienM | shardy: could you confirm that heat-engine should be run on a single instance in Icehouse? | 07:26 |
shardy | EmilienM: in Icehouse you can run multiple engine processes | 07:26 |
shardy | in Havana, you could only run one | 07:26 |
EmilienM | shardy: gosh, I did not know | 07:26 |
*** ifarkas has joined #heat | 07:26 | |
EmilienM | shardy: thanks ! | 07:27 |
shardy | EmilienM: note there is one limitiation, which is you must use ceilometer alarms, not the CWLite internal alarm resource | 07:27 |
*** tomek_adamczewsk has quit IRC | 07:28 | |
EmilienM | shardy: I'm good with that. Most of our deployments use Ceilometer Alarms. | 07:28 |
shardy | EmilienM: hey, a while ago you mentioned help with the grenade integration for heat | 07:28 |
shardy | is that something you're interested in helping with? | 07:28 |
EmilienM | shardy: oh yeah, definitly | 07:28 |
shardy | I've been meaning to get back to it but not yet had time | 07:28 |
EmilienM | shardy: I'll have a look | 07:29 |
shardy | EmilienM: Ok, any help would be awesome :) | 07:29 |
shardy | My exising WIP patch: | 07:29 |
EmilienM | shardy: i'll also see the javelin2 integration | 07:29 |
shardy | https://review.openstack.org/#/c/86978/ | 07:29 |
EmilienM | yup | 07:29 |
shardy | The main issue I had was I was unable to get a local development environment working from grenade master | 07:30 |
shardy | which basically stalled progress :( | 07:30 |
*** nati_ueno has quit IRC | 07:33 | |
EmilienM | shardy: no worries, I'll help this out later today | 07:34 |
*** saju_m has quit IRC | 07:35 | |
*** rushiagr_ has quit IRC | 07:35 | |
*** pas-ha has joined #heat | 07:37 | |
therve | 'morning | 07:38 |
shardy | EmilienM: sounds great, thanks! :) | 07:39 |
shardy | Hey therve | 07:39 |
*** bvivek has quit IRC | 07:47 | |
pas-ha | morning :) | 07:50 |
*** tomek_adamczewsk has joined #heat | 07:51 | |
ramishra | shardy: Hi .. Morning | 07:58 |
shardy | ramishra: Hi! | 07:58 |
ramishra | shardy: you have few mins to talk about https://review.openstack.org/104118 | 07:59 |
shardy | sure | 07:59 |
shardy | ramishra: so is the issue that you want to repurpose resource-template in future to return a resource object snippet? | 08:00 |
ramishra | shardy: yes.. so in this case I have moved some code so that they are in the right class resource_types.py and then added tests there were no tests earlier | 08:01 |
ramishra | shardy: and then deprecated resource-template | 08:01 |
shardy | ramishra: but quite soon you want to move resource-template from deprecated to doing a different thing though, right? | 08:02 |
ramishra | shardy: there were some tempest cli tests failing, so I could not remove resource-template | 08:03 |
ramishra | sahrdy: I'll put a tempest patch | 08:03 |
shardy | ramishra: yes, well we can't just remove stuff which is currently released, folks may be using it | 08:03 |
ramishra | shardy: yes, so I've deprecated it.. | 08:04 |
shardy | ramishra: Yeah, my question is for how long | 08:04 |
shardy | it will be really misleading if folks go from one heatclient relase to another and resource-template suddenly does a completely different thing IMO | 08:04 |
*** nosnos has quit IRC | 08:05 | |
*** nosnos has joined #heat | 08:05 | |
*** derekh_ has joined #heat | 08:05 | |
ramishra | shardy: probably have different command named 'resource-template-show' or something like that to start with. does that sound ok? and then decide when to change this. | 08:06 |
shardy | ramishra: I guess I don't really understand why we need this | 08:06 |
shardy | ramishra: can't we just add an option to template-show, so you either pass a stack, or a stack and a resource? | 08:06 |
shardy | and leave all the existing stuff alone? | 08:06 |
ramishra | shardy: Isn't 'resource-template' misleading and generate_template stuffed in 'resources.py' | 08:07 |
openstackgerrit | unmesh-gurjar proposed a change to openstack/heat: Fixed wait skipped after 1st step of task https://review.openstack.org/104471 | 08:08 |
shardy | ramishra: I suppose. If we go with this, I propose that we leave it deprecated for a long time, and use template-show for your new resource template stuff | 08:09 |
ramishra | shardy: I agree we can add an option in template-show. | 08:10 |
shardy | ramishra: Ok cool, as long as we don't immediately repurpose resource-show then I'm ok with this | 08:10 |
ramishra | shardy: I was thinking as it's cli, it should not be major issue.. however, I accept your point.. | 08:11 |
pas-ha | guys, question - at what point custom constraints get validated? | 08:13 |
*** gokrokve has joined #heat | 08:13 | |
pas-ha | say I have a resource with neutron.network constraint. and I want to create the said network in the same template and pass it via get_resource. Would it validate? | 08:14 |
*** rushiagr_ has joined #heat | 08:16 | |
*** gokrokve has quit IRC | 08:18 | |
*** bvivek has joined #heat | 08:19 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Fix failed to delete the EIP resource https://review.openstack.org/102758 | 08:20 |
*** saju_m has joined #heat | 08:24 | |
*** gokrokve has joined #heat | 08:31 | |
*** loki184 has quit IRC | 08:32 | |
*** gokrokve has quit IRC | 08:35 | |
*** swygue has quit IRC | 08:36 | |
*** nkhare has quit IRC | 08:38 | |
*** ckmvishnu has joined #heat | 08:41 | |
*** jcoufal has quit IRC | 08:41 | |
*** jcoufal has joined #heat | 08:44 | |
cmyster | shardy: there ? | 08:51 |
shardy | cmyster: I am, hi | 08:51 |
cmyster | some tempest stuff, | 08:52 |
openstackgerrit | liusheng proposed a change to openstack/heat: Fix H201 violations and re-enable gating https://review.openstack.org/102778 | 08:52 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource https://review.openstack.org/72336 | 08:52 |
cmyster | I talked with one of the bigger contributors here, and I think we need to make some work on the basic stuff in tempest api | 08:52 |
cmyster | if you check .../api/networking/base.py I think we need one of those | 08:53 |
cmyster | sorry its network/admin/base.py | 08:53 |
cmyster | so I think I'll start working on that with my patch. I am unable to find a simpler solution to make it in a 'tempest' way | 08:55 |
shardy | cmyster: I'm confused, we already have BaseOrchestrationTest for the API tests? | 08:56 |
shardy | tempest/api/orchestration/base.py | 08:56 |
cmyster | we do, but at the moment it serves as a direct resp client, no setting up of stuff, you just give it complete jobs to do. but if you have a need to do something more complex where there are tiers of resources and a dependency between each tier, it would be unaesthetic to have it all in one bulky test (like what I was told). | 08:59 |
openstackgerrit | liusheng proposed a change to openstack/heat: Fix H201 violations and re-enable gating https://review.openstack.org/102778 | 09:00 |
shardy | cmyster: Yeah, that's the same with all tests though, IMO you just need to refactor your test a bit | 09:01 |
shardy | It's OK to have some utility functions which create stuff in several tests | 09:01 |
*** bvivek has quit IRC | 09:01 | |
cmyster | also, I can't just split my patch to smaller tests since there is no way to guaranty that things will happen in the same way they are written. | 09:01 |
shardy | e.g if you need to set up the same stuff in several tests, that's perfectly OK IMO, provided the time penalty isn't too extreme | 09:01 |
shardy | cmyster: I don't really understand that last comment tbh | 09:02 |
*** loki184 has joined #heat | 09:03 | |
cmyster | sec lemme paste something quick it will be easier that way | 09:03 |
* cmyster is a kind of guy that needs to draw a picture | 09:03 | |
shardy | cmyster: IMO all you need is to break it into two test cases, each containing a test per API action | 09:04 |
*** loki184 has quit IRC | 09:04 | |
shardy | TestSoftwareConfig and TestSoftwareDeployments, with the crud tested for each | 09:04 |
cmyster | I hear you, but I think that what is really needed in this case is a different class that can create resources for me, where the test will only do validation on things. reason being is that even if I split the test into two, one of them (one that tests the deployment metadata) will need all resources created, so why bother with the first? | 09:07 |
shardy | cmyster: So have a base-class with a _create_config helper function or something | 09:08 |
shardy | If you think there's a better way though, feel free | 09:09 |
shardy | I'm just thinking of the simplest possible way to get this patch landed ;) | 09:09 |
cmyster | just wanted to hear your idea about it | 09:09 |
cmyster | simpler, yes, but no room to expand | 09:09 |
shardy | cmyster: If I were you, I'd do what's needed to land this, then post a followup patch with the refactoring improvements you're describing | 09:10 |
shardy | entirely up to you though ;) | 09:10 |
cmyster | I am not sure which will take longer :) | 09:10 |
*** sab has quit IRC | 09:10 | |
cmyster | which is btw very close to what I was trying to do at first if you see my earlier patch sets... | 09:11 |
shardy | cmyster: Yeah, putting the helpers for the specific resource into the base class makes no sense to me tbh | 09:12 |
shardy | but feel free to revisit that approach if you wish | 09:13 |
shardy | it would not be my preference, so you should solicit feedback from tempest-core before spending too much time on it | 09:13 |
shardy | FWIW I don't understand the problem with a quick cut/paste refactor of the existing test | 09:14 |
cmyster | cause if you split it to two, then the second one dealing with deployments, still needs to do exactly what the first one does | 09:14 |
cmyster | less verification on its part but still, | 09:14 |
shardy | So you create a few extra config resources during the test, so be it IMO | 09:15 |
cmyster | you have to anyway | 09:15 |
shardy | create a base class with a _create_config helper and you're done | 09:15 |
shardy | It's not like these are super-expensive resources to create or anything | 09:16 |
*** loki184 has joined #heat | 09:17 | |
cmyster | know what, lemme figure something that you would find a sense in :) | 09:18 |
shardy | cmyster: I guess I'm just frustrated - IMO your patch was ready, so if a tempest core wants it breaking up a bit, lets just do it and get it in | 09:19 |
shardy | how many months do these new tests have to eat resources? | 09:20 |
shardy | It's very frustrating (I'm talking about my stuff as well as yours here) | 09:20 |
*** TonyBurn_ has joined #heat | 09:20 | |
cmyster | shardy: true, and its not like I have the time to spend on it... | 09:20 |
shardy | cmyster: That's why I'm suggesting a simple cut/paste fix rather than refactoring all-the-things | 09:21 |
cmyster | I hear you | 09:21 |
*** loki184 has quit IRC | 09:22 | |
cmyster | btw, I broke it too much that last time ? | 09:22 |
shardy | cmyster: Yeah, and all the test cases are named the same | 09:22 |
cmyster | that was just a silly paste error | 09:23 |
shardy | You just need two files, test_software_config.py, and test_software_deployment.py | 09:23 |
cmyster | ok, since its launch soon you convinced me :) | 09:23 |
shardy | Have one testcase in each TestSoftwareConfig and TestSoftwareDeployment | 09:23 |
shardy | Then have the crud tests for each API in those | 09:23 |
cmyster | I'd still need crud for configuration inside deployment | 09:24 |
cmyster | well, cd | 09:24 |
shardy | Just create a base class with a helper which creates a config, and inherit from that for each test | 09:25 |
cmyster | thats what I said $lines ago :} | 09:25 |
* shardy sighs | 09:25 | |
shardy | That's what *I* just said too :( | 09:26 |
shardy | *a* base class, not *the* API test base class | 09:26 |
cmyster | at times like these I hate myself for quitting smoking. | 09:26 |
* cmyster remains silent | 09:27 | |
*** gokrokve has joined #heat | 09:31 | |
*** gokrokve has quit IRC | 09:36 | |
*** dmakogon_ is now known as denis_makogon | 09:38 | |
*** rakesh_hs has quit IRC | 09:38 | |
*** rakesh_hs has joined #heat | 09:39 | |
ishant | hi shardy | 09:39 |
ishant | shardy: Thanks for the review. | 09:40 |
*** lazy_prince is now known as killer_prince | 09:40 | |
ishant | shardy: I have addressed the review comments and submitted a new patch https://review.openstack.org/#/c/103198 | 09:41 |
*** sorantis has joined #heat | 09:42 | |
shardy | ishant: OK, thanks | 09:46 |
*** ramishra has quit IRC | 09:47 | |
*** killer_prince is now known as lazy_prince | 09:48 | |
*** bvivek has joined #heat | 09:52 | |
*** ramishra has joined #heat | 09:55 | |
*** Yanyan has quit IRC | 09:58 | |
*** tomek_adamczewsk has quit IRC | 10:01 | |
*** Qiming has quit IRC | 10:01 | |
*** avquadri has quit IRC | 10:03 | |
*** tomek_adamczewsk has joined #heat | 10:05 | |
*** ishant has quit IRC | 10:07 | |
*** avquadri has joined #heat | 10:11 | |
*** rakesh_hs has quit IRC | 10:14 | |
*** rakesh_hs has joined #heat | 10:15 | |
*** ishant has joined #heat | 10:16 | |
*** avquadri has quit IRC | 10:17 | |
*** vigneshvar has joined #heat | 10:27 | |
vigneshvar | hi, i am using a template which uses completion_signal and it waits for the signal | 10:28 |
vigneshvar | i tried using cfn-signal | 10:28 |
vigneshvar | with url generated from cfn metadata | 10:29 |
vigneshvar | in heat-api-cfn logs i find No route matched for PUT | 10:29 |
vigneshvar | can anyone help | 10:30 |
*** gokrokve has joined #heat | 10:32 | |
*** gokrokve has quit IRC | 10:37 | |
*** sorantis has quit IRC | 10:37 | |
openstackgerrit | A change was merged to openstack/heat: Deprecate direct access to clients methods https://review.openstack.org/97973 | 10:37 |
*** pasquier-s_ has quit IRC | 10:38 | |
*** sorantis has joined #heat | 10:52 | |
*** lazy_prince has quit IRC | 10:55 | |
*** lazy_prince has joined #heat | 10:56 | |
*** ramishra has quit IRC | 10:57 | |
*** jprovazn has quit IRC | 10:57 | |
*** jpeeler has quit IRC | 10:57 | |
*** ramishra has joined #heat | 10:57 | |
*** saju_m has quit IRC | 10:58 | |
*** saju_m has joined #heat | 10:59 | |
*** derekh_ has quit IRC | 10:59 | |
*** jprovazn has joined #heat | 11:00 | |
*** derekh_ has joined #heat | 11:00 | |
*** tomek_adamczewsk has quit IRC | 11:05 | |
*** Qiming has joined #heat | 11:06 | |
*** tomek_adamczewsk has joined #heat | 11:08 | |
*** avquadri has joined #heat | 11:09 | |
*** tiantian has quit IRC | 11:16 | |
*** PragadeeswaranS has quit IRC | 11:21 | |
*** jcoufal has quit IRC | 11:30 | |
*** jcoufal has joined #heat | 11:30 | |
*** sorantis has quit IRC | 11:31 | |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Fix test for stack delete on same active engine https://review.openstack.org/104525 | 11:32 |
*** gokrokve has joined #heat | 11:33 | |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement CFN CancelUpdateStack action https://review.openstack.org/102852 | 11:34 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update https://review.openstack.org/102851 | 11:34 |
*** gokrokve has quit IRC | 11:37 | |
*** sorantis has joined #heat | 11:40 | |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource https://review.openstack.org/100288 | 11:41 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin https://review.openstack.org/100588 | 11:41 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource https://review.openstack.org/72336 | 11:41 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::ClusterTemplate resource https://review.openstack.org/101509 | 11:41 |
*** radez_g0n3 is now known as radez | 11:43 | |
*** tspatzier has quit IRC | 11:43 | |
*** ckmvishnu has quit IRC | 11:43 | |
*** tspatzier has joined #heat | 11:44 | |
tspatzier | therve: can it sometimes happen jenkins/gerrit does automatically start merging a change when a dependent change got merged? | 11:44 |
therve | tspatzier, If it got approved you mean? | 11:45 |
tspatzier | I'm looking at https://review.openstack.org/#/c/97974 which you approved this morning and was wondering why there is no update even though the change it is dependent on got merged | 11:45 |
tspatzier | therve: should we just approve again to trigger the merge process? | 11:45 |
therve | That's weird | 11:46 |
tspatzier | therve: I added my +2 and wf+1 - let's see if this helps. | 11:48 |
therve | tspatzier, Cool | 11:49 |
therve | The gate queue is particularly empty, maybe there is an issue | 11:49 |
tspatzier | ok | 11:49 |
tspatzier | let's see how long it will take overall to get this huge patch chain in ;) | 11:49 |
*** pasquier-s has joined #heat | 11:50 | |
therve | Seems like it triggered something | 11:50 |
*** mkerrin has quit IRC | 11:51 | |
*** tomek_adamczewsk has quit IRC | 11:52 | |
*** nosnos has quit IRC | 11:55 | |
*** tomek_adamczewsk has joined #heat | 11:55 | |
ishant | hi shardy | 11:56 |
openstackgerrit | Tomas Sedovic proposed a change to openstack/heat: Remove instance_user and admin_user options https://review.openstack.org/103928 | 11:57 |
*** pasquier-s_ has joined #heat | 11:57 | |
*** pasquier-s has quit IRC | 11:57 | |
*** nosnos_ has joined #heat | 11:58 | |
*** mkerrin has joined #heat | 11:59 | |
*** jcoufal has quit IRC | 12:01 | |
*** ramishra has quit IRC | 12:03 | |
*** ramishra has joined #heat | 12:04 | |
*** jcoufal has joined #heat | 12:04 | |
*** dims has joined #heat | 12:04 | |
*** pasquier-s_ has quit IRC | 12:06 | |
*** pasquier-s has joined #heat | 12:07 | |
pas-ha | Qiming, here? | 12:08 |
*** sorantis has quit IRC | 12:08 | |
*** ramishra has quit IRC | 12:08 | |
*** bvandenh has quit IRC | 12:09 | |
pas-ha | Qiming, could you check https://review.openstack.org/#/c/100995 and see if bug #1324499 is something that can affect that resource as well? | 12:10 |
uvirtbot | Launchpad bug 1324499 in heat "physical_resource_name of Nova server should not be overriden" [High,Fix released] https://launchpad.net/bugs/1324499 | 12:10 |
cmyster | bbl | 12:11 |
*** cmyster has quit IRC | 12:12 | |
*** ramishra has joined #heat | 12:15 | |
*** tomek_adamczewsk has quit IRC | 12:16 | |
*** sgordon has joined #heat | 12:19 | |
*** rbuilta has joined #heat | 12:20 | |
*** swygue has joined #heat | 12:20 | |
*** unmeshg has quit IRC | 12:23 | |
*** jcoufal has quit IRC | 12:24 | |
*** jcoufal has joined #heat | 12:24 | |
*** tomek_adamczewsk has joined #heat | 12:24 | |
*** mkollaro has joined #heat | 12:28 | |
*** ajc_ has quit IRC | 12:30 | |
*** gokrokve has joined #heat | 12:33 | |
*** bvivek has quit IRC | 12:35 | |
*** zigo has quit IRC | 12:37 | |
*** zigo has joined #heat | 12:37 | |
*** gokrokve has quit IRC | 12:38 | |
*** rbuilta has quit IRC | 12:39 | |
*** zigo has quit IRC | 12:41 | |
*** zigo has joined #heat | 12:42 | |
EmilienM | shardy: looking at heat now | 12:43 |
EmilienM | (grenade) | 12:43 |
Qiming | pas-ha, checking | 12:45 |
*** zigo has quit IRC | 12:45 | |
*** mtreinish has quit IRC | 12:45 | |
*** pasquier-s has quit IRC | 12:46 | |
EmilienM | shardy: looking at http://logs.openstack.org/78/86978/6/check/check-grenade-dsvm/cdeb66d/logs/grenade.sh.txt.gz, the problem seems pretty obvious | 12:46 |
EmilienM | http://paste.openstack.org/show/B9uyVWpiygImjyYYLNF3/ | 12:46 |
therve | EmilienM, We shouldn't require qpid | 12:49 |
EmilienM | therve: the upgrade fails when running "heat-manage db_sync" | 12:50 |
*** pasquier-s has joined #heat | 12:50 | |
*** ckmvishnu has joined #heat | 12:52 | |
Qiming | pas-ha: my understanding is that the 1324499 bug is only relevant to subclasses of StackUser at the moment | 12:52 |
pas-ha | ok, so could you leave a comment so that the patch can be approved? | 12:53 |
Qiming | pas-ha, okay | 12:54 |
*** rbuilta has joined #heat | 12:54 | |
Qiming | pas-ha: though it won't cause problems today, I would still recommend changing the code to avoid potential name conflicts | 12:55 |
*** sorantis has joined #heat | 12:57 | |
*** dims_ has joined #heat | 12:58 | |
*** ramishra has quit IRC | 12:59 | |
*** achampio1 has joined #heat | 13:01 | |
*** dims has quit IRC | 13:02 | |
*** achampion has quit IRC | 13:03 | |
ckmvishnu | Hi all, I'm working on a database migration. Is it enough to make the script compatible with mysql and postgrsql? | 13:13 |
*** pasquier-s_ has joined #heat | 13:15 | |
*** pasquier-s has quit IRC | 13:15 | |
*** chandan_kumar is now known as chandankumar | 13:18 | |
*** alexpilotti has joined #heat | 13:20 | |
*** arbylee has joined #heat | 13:21 | |
*** DandyPandy has joined #heat | 13:23 | |
*** pasquier-s has joined #heat | 13:24 | |
*** pasquier-s_ has quit IRC | 13:25 | |
*** kebray- has joined #heat | 13:26 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert WaitConditionHandle to use handle_signal https://review.openstack.org/101351 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Update waitcondition API to use signal RPC interface https://review.openstack.org/101352 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing https://review.openstack.org/101353 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource https://review.openstack.org/101354 | 13:27 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 13:27 |
*** aweiteka has joined #heat | 13:28 | |
*** zz_gondoi is now known as gondoi | 13:31 | |
*** gokrokve has joined #heat | 13:34 | |
*** m_22 has joined #heat | 13:35 | |
*** achampion has joined #heat | 13:36 | |
*** jdob has joined #heat | 13:36 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Port resource https://review.openstack.org/104224 | 13:36 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Node resource https://review.openstack.org/104223 | 13:36 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add Ironic resource base class https://review.openstack.org/104222 | 13:37 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add OS::Ironic::Chassis resource https://review.openstack.org/103133 | 13:37 |
*** arbylee has quit IRC | 13:37 | |
*** arbylee has joined #heat | 13:37 | |
*** nosnos_ has quit IRC | 13:37 | |
*** achampio1 has quit IRC | 13:38 | |
*** DandyPandy has quit IRC | 13:38 | |
*** gokrokve has quit IRC | 13:39 | |
*** rakesh_hs has quit IRC | 13:42 | |
*** DandyPandy has joined #heat | 13:42 | |
Qiming | ckmvishnu, what do you mean by database migration? | 13:42 |
*** kebray- has quit IRC | 13:43 | |
*** pasquier-s has quit IRC | 13:46 | |
saju_m | hi | 13:47 |
*** nosnos has joined #heat | 13:49 | |
*** pasquier-s has joined #heat | 13:49 | |
saju_m | stack-list command not working with --heat-url | 13:50 |
saju_m | without --heat-url : http://dpaste.com/1HGK5DA | 13:50 |
saju_m | with --heat-url : http://dpaste.com/3BV6FVD | 13:51 |
*** jstrachan has quit IRC | 13:53 | |
*** tango has joined #heat | 13:54 | |
*** rushiagr_ has quit IRC | 13:55 | |
ishant | saju_m : your heat url is incorrect | 13:57 |
ishant | it should be http://192.168.0.131:8004/v1/<tenant-id> | 13:57 |
saju_m | ishant, : thanks, let me try | 13:58 |
*** zigo has joined #heat | 13:58 | |
*** nosnos has quit IRC | 13:59 | |
*** kebray has quit IRC | 13:59 | |
*** tspatzier has quit IRC | 14:00 | |
*** aweiteka has quit IRC | 14:03 | |
pas-ha | therve, ping | 14:04 |
Qiming | http://192.168.0.131:8004/v1/%(tenant_id)s | 14:06 |
*** daneyon has joined #heat | 14:07 | |
*** daneyon has quit IRC | 14:07 | |
*** alexpilotti_ has joined #heat | 14:08 | |
*** daneyon has joined #heat | 14:08 | |
*** alexpilotti has quit IRC | 14:09 | |
*** alexpilotti_ is now known as alexpilotti | 14:09 | |
therve | pas-ha, Hi | 14:10 |
pas-ha | therve, I am playing with custom constraints and get strange (for me) behaviour | 14:11 |
pas-ha | at what place do they actually validate? | 14:11 |
*** daneyon has quit IRC | 14:11 | |
pas-ha | can I generally use a parameter that has a custom constranint in the resource on it together with get_resource in the template? | 14:12 |
*** shakamunyi has joined #heat | 14:13 | |
therve | It depends | 14:13 |
therve | get_resource with a parameter? | 14:13 |
pas-ha | what I tried now is using neutron.network constraint on Nova::Server and Neutroon::Port resources, and create a network in the same template and pass it to the property as get_resource | 14:14 |
pas-ha | property (silly me) | 14:14 |
*** ramishra has joined #heat | 14:14 | |
pas-ha | the strange thing is that Neutron::Port creates just fine but Nova::Server fails on the stack validation stage | 14:15 |
*** sjmc7 has joined #heat | 14:15 | |
*** aweiteka has joined #heat | 14:15 | |
pas-ha | exactly fails on self.properties.validate() | 14:16 |
*** dims_ has quit IRC | 14:16 | |
*** ramishra has quit IRC | 14:18 | |
*** gokrokve has joined #heat | 14:19 | |
*** andersonvom has joined #heat | 14:21 | |
*** alexpilotti has quit IRC | 14:23 | |
pas-ha | nope, probably some problem with my templates, rechecked and the Neutron::Port is also not creating | 14:23 |
*** ramishra has joined #heat | 14:24 | |
pas-ha | so generally the answer to my question seems to be "No"... | 14:24 |
saju_m | <ishant>, Qiming : I installed heat in a separate machine. I am trying to execute this command from the machine i installed heat. I added tenant id , but it failing again | 14:26 |
saju_m | <ishant>, Qiming : see http://dpaste.com/37BQ6B0 | 14:27 |
Qiming | saju_m: use the exact string: http://192.168.0.131:8004/v1/%(tenant_id)s | 14:27 |
saju_m | Qiming, let me try | 14:29 |
saju_m | <ishant>, Qiming : i haven't registered new heat setup in keystone, is that the issue ? http://dpaste.com/347SMXB | 14:29 |
openstackgerrit | Thomas Herve proposed a change to openstack/heat: Migrate to oslo.db https://review.openstack.org/104566 | 14:29 |
Qiming | seems you are trying to 127.0.0.1 ? | 14:29 |
*** m_22 has quit IRC | 14:30 | |
saju_m | Qiming, yes, I am trying to execute this from heat machine | 14:30 |
therve | pas-ha, Yeah I don't know. Maybe paste the template? | 14:31 |
pas-ha | just a sec | 14:31 |
pas-ha | https://github.com/pshchelo/stackdev/blob/master/templates/nets/portnet.yaml | 14:31 |
saju_m | Qiming, not taking that format http://dpaste.com/1GV1TQD | 14:32 |
pas-ha | and the change in resource is https://github.com/pshchelo/heat/compare/test-custom-constraint?expand=1 | 14:32 |
*** gokrokve has quit IRC | 14:32 | |
Qiming | saju_m: you have to register it in keystone | 14:32 |
saju_m | <ishant>, Qiming : i can able to execute this command successfully if i try from the machine where i installed devstack | 14:32 |
saju_m | <ishant>, Qiming : Let me explain about my setup | 14:33 |
*** ishant has quit IRC | 14:33 | |
pas-ha | actually no resource is using neutron.network constraint still... | 14:33 |
Qiming | saju_m: quote the string? | 14:33 |
*** kebray has joined #heat | 14:33 | |
*** ericgoncz has joined #heat | 14:34 | |
*** kebray has quit IRC | 14:34 | |
saju_m | I have Virtual machine where i installed Openstack with heat using devstack. Then i am trying to install heat using puppet in a separate machine and point to running devstack setup. point to | 14:35 |
*** kebray has joined #heat | 14:35 | |
*** pasquier-s_ has joined #heat | 14:35 | |
*** pasquier-s has quit IRC | 14:37 | |
*** chandankumar has quit IRC | 14:38 | |
saju_m | Qiming, i added quote, again error:heatclient.exc.HTTPUnauthorized: ERROR: Authentication required | 14:38 |
saju_m | Qiming, let me make entry in keystone and try | 14:39 |
Qiming | saju_m: your setup is weird, but anyway, I am trying it on my devstack env | 14:39 |
*** gokrokve has joined #heat | 14:40 | |
*** alexpilotti has joined #heat | 14:42 | |
*** ramishra has quit IRC | 14:43 | |
saju_m | <ishant>, Qiming : Here is the result from Devstack VM, http://dpaste.com/2D2W9J8 | 14:44 |
saju_m | Qiming, '%(tenant_id)s' format not working | 14:45 |
Qiming | saju_m: notice that as well | 14:46 |
shardy | saju_m: If you're passing os-auth-url, why do you need to pass the heat endpoint? | 14:46 |
shardy | can't you just add the new heat endpoint to the keystone service catalog? | 14:47 |
shardy | Oh wait, are you running two keystone services? | 14:47 |
saju_m | result from my heat VM: http://dpaste.com/0XFZB6D , I can not use 192.168.0.131 in --heat-url since that is the heat service ruiing in my devstack VM. | 14:49 |
saju_m | shardy, i dont have keystone service running in my Heat VM. Keystine running only in devstack VM 192.168.0.131. | 14:51 |
shardy | But you're passing 192.168.0.131 as the heat URL, but you just said it was running on a different VM? | 14:52 |
shardy | So you're running heat in both vm's right? | 14:53 |
*** ramishra has joined #heat | 14:53 | |
*** pasquier-s has joined #heat | 14:54 | |
shardy | saju_m: Maybe what you want is the non-devstack heat running in standalone mode: | 14:56 |
Qiming | shardy: in my environment, I can reproduce saju_m's problem if I have "%(tenant_id)s" in the --heat-url option | 14:56 |
shardy | http://docs.openstack.org/developer/heat/getting_started/standalone.html | 14:56 |
shardy | Qiming: It needs to be the actual tenant ID | 14:56 |
Qiming | yes, that is what I have just learned | 14:56 |
Qiming | but the doc and help didn't say that explicitly | 14:56 |
EmilienM | shardy: I'm debugging the heat/grenade stuff. It miss python-qpid after the upgrade from havana to icehouse. This is weird, I'm running some tests atm | 14:57 |
shardy | That format string is the endpoint format in the catalog, which keystone substitutes with the actual tenant from the context | 14:57 |
*** gokrokve has quit IRC | 14:57 | |
saju_m | shardy, I am not using non-devstack heat. I installed heat using puppet in separate machine | 14:57 |
shardy | EmilienM: Yeah, I saw that, odd - we have that (possibly incorrectly) in requirements.txt, so I'm not sure why it's missing? | 14:58 |
saju_m | shardy, i will share the puppet code | 14:58 |
*** IlyaE has joined #heat | 14:58 | |
*** pasquier-s_ has quit IRC | 14:58 | |
EmilienM | shardy: that's what I'm figuring out | 14:59 |
saju_m | shardy, puppet code http://dpaste.com/3JACTMV Here 192.168.0.131 is the IP of devstack VM. | 14:59 |
*** lazy_prince is now known as killer_prince | 14:59 | |
*** tspatzier has joined #heat | 15:00 | |
shardy | saju_m: Like I just said, unless you're going to configure the devstack keystone to know about the puppet installed heat, you will have to use heat in standalone mode against the remote devstack | 15:00 |
jasond | zaneb: do you know where i can find issue #133 mentioned here? https://github.com/openstack/heat/blob/master/heat/tests/test_parser.py#L455-L456 | 15:01 |
saju_m | shardy, got it, let me check http://docs.openstack.org/developer/heat/getting_started/standalone.html | 15:02 |
saju_m | shardy, thanks | 15:02 |
shardy | jasond: it was probably lost when the heat-api org got deleted from github, unfortunately | 15:02 |
jasond | shardy: ok, thanks | 15:03 |
shardy | IIRC the issue was the base64 encoded userdata didn't work with cloud-init | 15:03 |
shardy | It was a long time ago though, so I'm a little fuzzy on the details ;) | 15:03 |
jasond | ah, that helps, thanks | 15:03 |
*** dims_ has joined #heat | 15:04 | |
*** shakamunyi has quit IRC | 15:05 | |
sirushti | Hi all, I have a Heat-Infra question. Any ideas on this? http://logs.openstack.org/68/104068/2/check/gate-heat-python26/bf0c28b/console.html . It says tox not found. Or must I go to #openstack-infra and post this? PS: I did a `recheck no bug` too | 15:05 |
*** ramishra has quit IRC | 15:08 | |
*** dims_ has quit IRC | 15:08 | |
*** pasquier-s has quit IRC | 15:14 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 15:15 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 15:15 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 15:15 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource https://review.openstack.org/101354 | 15:15 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 15:15 |
EmilienM | shardy: so the patch is now green: https://review.openstack.org/#/c/86978/ | 15:16 |
shardy | EmilienM: awesome, thanks! :) | 15:16 |
*** sabeen has joined #heat | 15:17 | |
EmilienM | shardy: the issue is not fixed though, I'm looking for the place to fix the real issue | 15:17 |
*** radez is now known as radez_g0n3 | 15:20 | |
*** ifarkas has quit IRC | 15:22 | |
*** jpeeler has joined #heat | 15:23 | |
*** ifarkas has joined #heat | 15:24 | |
*** gokrokve has joined #heat | 15:24 | |
*** saju_m has quit IRC | 15:26 | |
*** david-ly_ has joined #heat | 15:27 | |
*** Qiming has quit IRC | 15:29 | |
*** pafuent has joined #heat | 15:29 | |
*** avquadri has quit IRC | 15:30 | |
*** jcoufal has quit IRC | 15:31 | |
SpamapS | shardy: I have some diagrams showing the difference in how convergence will handle failure versus current engine. | 15:32 |
SpamapS | shardy: but I can make a real architecture diagram. Drawing pictures is good for the soul. ;) | 15:33 |
shardy | SpamapS: sounds good ;) | 15:33 |
*** TonyBurn_ has quit IRC | 15:44 | |
*** jamiehannaford has quit IRC | 15:46 | |
*** shakamunyi has joined #heat | 15:46 | |
*** pasquier-s has joined #heat | 15:48 | |
*** jamiehannaford has joined #heat | 15:49 | |
*** sorantis has quit IRC | 15:50 | |
*** david-ly_ is now known as david-lyle | 15:54 | |
*** flyemsafe has joined #heat | 15:54 | |
*** nati_ueno has joined #heat | 15:56 | |
*** swygue has quit IRC | 15:57 | |
*** randallburt has joined #heat | 15:58 | |
*** mtreinish has joined #heat | 16:00 | |
openstackgerrit | A change was merged to openstack/heat: Convert WaitConditionHandle to use handle_signal https://review.openstack.org/101351 | 16:03 |
openstackgerrit | A change was merged to openstack/heat: Update waitcondition API to use signal RPC interface https://review.openstack.org/101352 | 16:04 |
*** mtreinish has quit IRC | 16:04 | |
*** mtreinish has joined #heat | 16:04 | |
*** dims_ has joined #heat | 16:05 | |
openstackgerrit | A change was merged to openstack/heat: Make heat.engine.clients a package https://review.openstack.org/97974 | 16:06 |
*** ramishra has joined #heat | 16:09 | |
*** dims_ has quit IRC | 16:09 | |
*** alexheneveld has quit IRC | 16:10 | |
*** kebray has quit IRC | 16:11 | |
*** ramishra has quit IRC | 16:13 | |
*** ckmvishnu has quit IRC | 16:14 | |
*** nati_ueno has quit IRC | 16:17 | |
*** nati_uen_ has joined #heat | 16:17 | |
*** dims_ has joined #heat | 16:21 | |
*** sorantis has joined #heat | 16:22 | |
*** alexheneveld has joined #heat | 16:25 | |
*** vigneshvar has quit IRC | 16:27 | |
*** bandarji has joined #heat | 16:27 | |
*** ifarkas has quit IRC | 16:27 | |
*** zhiyan is now known as zhiyan_ | 16:29 | |
*** pasquier-s has quit IRC | 16:30 | |
*** lsmola__ has quit IRC | 16:30 | |
*** arbylee1 has joined #heat | 16:33 | |
*** arbylee has quit IRC | 16:33 | |
*** aweiteka has quit IRC | 16:34 | |
*** sorantis has quit IRC | 16:36 | |
*** arbylee1 has quit IRC | 16:37 | |
*** arbylee has joined #heat | 16:37 | |
*** ckmvishnu has joined #heat | 16:42 | |
*** ericgoncz has quit IRC | 16:44 | |
*** aweiteka has joined #heat | 16:48 | |
*** ishant has joined #heat | 16:52 | |
andersonvom | hey folks, do we have code anywhere that, given a certain property (e.g. LIST, MAP, STRING) will return a default sane value (e.g. [], {}, '')? | 16:53 |
shardy | andersonvom: no, it always returns None, unless the schema specifies a default | 16:53 |
*** shakamunyi has quit IRC | 16:54 | |
shardy | Changing it has been discussed before, but it carries a high risk of regressions, so we have not done it | 16:54 |
andersonvom | shardy: that's what I thought! :( It would come in really handy now, though | 16:54 |
*** saju_m has joined #heat | 16:56 | |
*** tomek_adamczewsk has quit IRC | 16:57 | |
*** jstrachan has joined #heat | 16:58 | |
*** derekh_ has quit IRC | 16:58 | |
shardy | andersonvom: maybe we could add a default_for_schema method to the Property class, but doing property.get(thing) or [] is what's used in various places | 17:01 |
shardy | obviously that requires you to keep the code in sync with the resource schema, but presumably you have to do that anyway | 17:01 |
*** Dev_Jin has quit IRC | 17:03 | |
*** shakamunyi has joined #heat | 17:03 | |
*** bandarji has quit IRC | 17:05 | |
*** ramishra has joined #heat | 17:10 | |
*** tango has quit IRC | 17:10 | |
*** rushiagr_ has joined #heat | 17:10 | |
*** pafuent has quit IRC | 17:12 | |
*** ramishra has quit IRC | 17:14 | |
*** bnemec is now known as bnemec_appt | 17:15 | |
*** arbylee has quit IRC | 17:16 | |
*** arbylee has joined #heat | 17:16 | |
*** denis_makogon has quit IRC | 17:16 | |
*** blamar has joined #heat | 17:20 | |
*** nati_uen_ has quit IRC | 17:31 | |
*** nati_ueno has joined #heat | 17:31 | |
*** radez_g0n3 is now known as radez | 17:33 | |
*** cmyster has joined #heat | 17:36 | |
*** cmyster has joined #heat | 17:36 | |
*** blamar has quit IRC | 17:39 | |
*** rushiagr_ has quit IRC | 17:42 | |
*** ishant has quit IRC | 17:44 | |
*** tomek_adamczewsk has joined #heat | 17:46 | |
*** cmyster has quit IRC | 17:48 | |
*** tango has joined #heat | 17:51 | |
*** rdo has quit IRC | 17:52 | |
*** rdo has joined #heat | 17:53 | |
*** randallburt has quit IRC | 17:53 | |
*** tomek_adamczewsk has quit IRC | 17:57 | |
*** tomek_adamczewsk has joined #heat | 17:58 | |
*** shakamunyi has quit IRC | 18:07 | |
*** shakamunyi has joined #heat | 18:09 | |
*** liusheng has quit IRC | 18:10 | |
*** tomek_adamczewsk has quit IRC | 18:10 | |
*** liusheng has joined #heat | 18:11 | |
*** BillArnold has joined #heat | 18:14 | |
*** julienvey has quit IRC | 18:16 | |
openstackgerrit | A change was merged to openstack/heat: Add a OS::Nova::ServerGroup resource. https://review.openstack.org/100995 | 18:25 |
*** IlyaE has quit IRC | 18:28 | |
*** tomek_adamczewsk has joined #heat | 18:29 | |
*** gondoi is now known as zz_gondoi | 18:31 | |
*** nati_ueno has quit IRC | 18:32 | |
*** thomas_ustudio has joined #heat | 18:34 | |
*** zz_gondoi is now known as gondoi | 18:34 | |
thomas_ustudio | I’m trying to get the list of internal network addresses from all the nodes in a group as an output of a stack, but it’s not working. The value for the output is: value: { get_attr: [api_nodes, networks, private, 0]} but it just gives me null back | 18:35 |
thomas_ustudio | however, if I just make it { get_attr: [api_nodes, networks] there *is* a network named “private” in the map | 18:36 |
thomas_ustudio | do I need to quote “private”? or do something else? | 18:36 |
*** ericgoncz has joined #heat | 18:36 | |
*** shakamunyi has quit IRC | 18:38 | |
*** shakamunyi has joined #heat | 18:38 | |
*** shakamunyi has quit IRC | 18:38 | |
*** jstrachan has quit IRC | 18:38 | |
*** shakamunyi has joined #heat | 18:39 | |
*** shakamunyi has quit IRC | 18:39 | |
*** ramishra has joined #heat | 18:39 | |
*** shakamunyi has joined #heat | 18:40 | |
*** tomek_adamczewsk has quit IRC | 18:42 | |
*** ericgoncz_ has joined #heat | 18:43 | |
*** tomek_adamczewsk has joined #heat | 18:44 | |
*** ramishra has quit IRC | 18:44 | |
*** ericgoncz has quit IRC | 18:45 | |
*** ericgoncz_ is now known as ericgoncz | 18:45 | |
SpamapS | thomas_ustudio: that sounds like a bug | 18:46 |
SpamapS | thomas_ustudio: if you see a map with {"private": [1.2.3.4]} .. your get_attr should work. | 18:47 |
thomas_ustudio | well, since it’s a server group, what I see is: [{“private”: […]}, {“private”: […]}] (where it’s an array of mappings for each server in the resource group) | 18:48 |
thomas_ustudio | could the outer array be confusing things? | 18:49 |
*** IlyaE has joined #heat | 18:49 | |
openstackgerrit | A change was merged to openstack/heat: Updated from global requirements https://review.openstack.org/96507 | 18:50 |
*** DandyPandy has quit IRC | 18:50 | |
*** DandyPandy has joined #heat | 18:50 | |
*** lindsayk1 has joined #heat | 18:50 | |
zaneb | thomas_ustudio: { get_attr: [api_nodes, networks, 0, private, 0]} | 18:51 |
zaneb | try that ^ | 18:51 |
thomas_ustudio | ok, will do | 18:51 |
thomas_ustudio | it takes a while to test, since the servers take a while to stand up | 18:52 |
thomas_ustudio | that will give me the IPs for all the nodes, right? not just one of them? | 18:52 |
*** rbuilta has quit IRC | 18:53 | |
openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add resource properties to stack-preview https://review.openstack.org/94479 | 18:53 |
*** tomek_adamczewsk has quit IRC | 18:53 | |
*** tspatzier has quit IRC | 18:54 | |
*** DandyPandy has quit IRC | 18:54 | |
*** DandyPandy has joined #heat | 18:55 | |
*** kebray has joined #heat | 18:55 | |
*** tomek_adamczewsk has joined #heat | 18:56 | |
*** IlyaE has quit IRC | 18:57 | |
*** ccrouch has quit IRC | 19:03 | |
*** alexheneveld has quit IRC | 19:03 | |
thomas_ustudio | zaneb: that just gave me the IP from one node, not all of them | 19:04 |
zaneb | thomas_ustudio: I don't think Heat's intrinsic functions can do what you want then | 19:05 |
thomas_ustudio | can I use Fn::Select? It seems like, if I can do this for public IP addresses, I should be able to do it for private IP addresses | 19:06 |
zaneb | well, want you want to do is effective for-each, but Heat doesn't include any looping constructs | 19:07 |
zaneb | *effectively | 19:08 |
*** kebray has quit IRC | 19:08 | |
*** kebray has joined #heat | 19:08 | |
*** dims_ has quit IRC | 19:09 | |
*** tomek_adamczewsk has quit IRC | 19:09 | |
*** tomek_adamczewsk has joined #heat | 19:10 | |
*** tango has quit IRC | 19:10 | |
thomas_ustudio | why does { get_attr: [api_nodes, accessIPv4]} work, then? It gives me an array of IPs for every server in the api_nodes resource group | 19:11 |
*** ericgoncz has quit IRC | 19:11 | |
*** tomek_adamczewsk has quit IRC | 19:12 | |
*** tomek_adamczewsk has joined #heat | 19:12 | |
*** kebray has quit IRC | 19:13 | |
*** kebray has joined #heat | 19:13 | |
*** tomek_adamczewsk has quit IRC | 19:19 | |
*** tomek_adamczewsk has joined #heat | 19:20 | |
*** jamiehannaford has quit IRC | 19:20 | |
*** tomek_adamczewsk has quit IRC | 19:22 | |
*** tomek_adamczewsk has joined #heat | 19:22 | |
*** robsparker has quit IRC | 19:26 | |
*** jamiehannaford has joined #heat | 19:35 | |
*** robsparker has joined #heat | 19:37 | |
*** julienvey has joined #heat | 19:37 | |
*** shakamunyi has quit IRC | 19:38 | |
*** shakamunyi has joined #heat | 19:39 | |
*** shakamunyi has quit IRC | 19:39 | |
*** m_22 has joined #heat | 19:39 | |
*** ramishra has joined #heat | 19:40 | |
*** tomek_adamczewsk has quit IRC | 19:42 | |
*** ramishra has quit IRC | 19:44 | |
*** tomek_adamczewsk has joined #heat | 19:45 | |
*** DandyPandy has quit IRC | 19:47 | |
*** shakamunyi has joined #heat | 19:47 | |
*** tomek_adamczewsk has quit IRC | 19:51 | |
*** blamar has joined #heat | 19:51 | |
*** IlyaE has joined #heat | 19:51 | |
*** tomek_adamczewsk has joined #heat | 19:52 | |
*** dims_ has joined #heat | 19:53 | |
*** tomek_adamczewsk has quit IRC | 19:58 | |
*** tomek_adamczewsk has joined #heat | 19:59 | |
*** gondoi is now known as zz_gondoi | 20:03 | |
*** IlyaE has quit IRC | 20:03 | |
*** blamar has quit IRC | 20:03 | |
*** sjmc7 has quit IRC | 20:05 | |
*** IlyaE has joined #heat | 20:05 | |
*** tango has joined #heat | 20:07 | |
*** zz_gondoi is now known as gondoi | 20:08 | |
*** tomek_adamczewsk has quit IRC | 20:12 | |
*** randallburt has joined #heat | 20:13 | |
*** tomek_adamczewsk has joined #heat | 20:17 | |
*** jamiehannaford has quit IRC | 20:17 | |
*** jamiehannaford has joined #heat | 20:20 | |
*** aweiteka has quit IRC | 20:23 | |
*** shakamunyi has quit IRC | 20:23 | |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Refactor handling of exceptions in Resource actions https://review.openstack.org/104676 | 20:24 |
*** IlyaE has quit IRC | 20:24 | |
*** ccrouch has joined #heat | 20:25 | |
*** m_221 has joined #heat | 20:25 | |
*** jprovazn has quit IRC | 20:25 | |
*** pafuent has joined #heat | 20:26 | |
*** m_221 has quit IRC | 20:26 | |
*** m_22 has quit IRC | 20:26 | |
*** m_22 has joined #heat | 20:26 | |
*** m_22 has quit IRC | 20:27 | |
*** bnemec_appt is now known as bnemec | 20:27 | |
*** lindsayk1 has quit IRC | 20:27 | |
*** tomek_adamczewsk has quit IRC | 20:27 | |
*** tomek_adamczewsk has joined #heat | 20:29 | |
*** tomek_adamczews1 has joined #heat | 20:31 | |
*** tomek_adamczewsk has quit IRC | 20:33 | |
*** daneyon has joined #heat | 20:34 | |
*** jamiehannaford has quit IRC | 20:36 | |
*** andersonvom has quit IRC | 20:37 | |
*** julienvey has quit IRC | 20:40 | |
*** ramishra has joined #heat | 20:41 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Add resource properties to stack-preview https://review.openstack.org/94479 | 20:42 |
*** daneyon_ has joined #heat | 20:42 | |
*** daneyon has quit IRC | 20:43 | |
*** ramishra has quit IRC | 20:46 | |
stevebaker | morning | 20:47 |
*** IlyaE has joined #heat | 20:48 | |
*** jdob has quit IRC | 20:55 | |
*** lindsayk1 has joined #heat | 21:05 | |
*** pafuent has left #heat | 21:15 | |
*** arbylee has quit IRC | 21:15 | |
*** flyemsafe has quit IRC | 21:16 | |
*** radez is now known as radez_g0n3 | 21:16 | |
*** shakamunyi has joined #heat | 21:20 | |
*** arbylee has joined #heat | 21:25 | |
*** sgordon has quit IRC | 21:28 | |
*** arbylee has quit IRC | 21:30 | |
*** arbylee has joined #heat | 21:33 | |
*** lindsayk1 has quit IRC | 21:33 | |
*** tomek_adamczews1 has quit IRC | 21:35 | |
*** saju_m has quit IRC | 21:35 | |
*** saju_m has joined #heat | 21:36 | |
*** shakamunyi has quit IRC | 21:39 | |
*** lindsayk1 has joined #heat | 21:40 | |
*** tomek_adamczewsk has joined #heat | 21:43 | |
*** alexheneveld has joined #heat | 21:44 | |
*** arbylee has quit IRC | 21:51 | |
*** shakamunyi has joined #heat | 21:55 | |
*** ccrouch has quit IRC | 21:56 | |
*** alexheneveld has quit IRC | 21:56 | |
*** shakamunyi has quit IRC | 21:57 | |
*** gondoi is now known as zz_gondoi | 22:01 | |
*** tomek_adamczewsk has quit IRC | 22:05 | |
*** ccrouch has joined #heat | 22:07 | |
*** IlyaE has quit IRC | 22:13 | |
*** daneyon_ has quit IRC | 22:13 | |
*** dims__ has joined #heat | 22:13 | |
*** dims_ has quit IRC | 22:15 | |
*** thomas_ustudio has quit IRC | 22:17 | |
*** IlyaE has joined #heat | 22:25 | |
*** vijendar has joined #heat | 22:25 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Update Rackspace clients again https://review.openstack.org/104712 | 22:57 |
*** david-lyle has quit IRC | 22:57 | |
*** vijendar has quit IRC | 22:58 | |
*** david-ly_ has joined #heat | 22:58 | |
*** lindsayk1 has quit IRC | 23:02 | |
*** david-ly_ has quit IRC | 23:03 | |
*** IlyaE has quit IRC | 23:05 | |
*** lindsayk1 has joined #heat | 23:12 | |
*** gokrokve has quit IRC | 23:12 | |
*** nijaba has quit IRC | 23:12 | |
*** nijaba has joined #heat | 23:13 | |
*** nijaba has quit IRC | 23:13 | |
*** nijaba has joined #heat | 23:13 | |
*** lindsayk1 has quit IRC | 23:16 | |
*** lindsayk1 has joined #heat | 23:17 | |
*** alexpilotti has quit IRC | 23:19 | |
*** ccrouch has quit IRC | 23:25 | |
*** lindsayk2 has joined #heat | 23:29 | |
*** lindsayk1 has quit IRC | 23:31 | |
*** mtreinish has quit IRC | 23:31 | |
openstackgerrit | David J Hu proposed a change to openstack/python-heatclient: Add keystone v3 auth support https://review.openstack.org/92728 | 23:35 |
*** mtreinish has joined #heat | 23:39 | |
*** lindsayk2 has quit IRC | 23:42 | |
*** lindsayk1 has joined #heat | 23:42 | |
*** ramishra has joined #heat | 23:43 | |
*** randallburt has quit IRC | 23:47 | |
*** ramishra has quit IRC | 23:48 | |
*** saju_m has quit IRC | 23:48 | |
*** derekh_ has joined #heat | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!