| *** matsuhashi has joined #heat | 00:09 | |
| *** andersonvom has joined #heat | 00:14 | |
| *** mnaser has quit IRC | 00:16 | |
| openstackgerrit | A change was merged to openstack/heat: Update olso gettextutils https://review.openstack.org/67758 | 00:17 |
|---|---|---|
| openstackgerrit | A change was merged to openstack/heat: Fix order of arguments in assertEqual (patch 1/2) https://review.openstack.org/61457 | 00:17 |
| *** agileclipse has joined #heat | 00:17 | |
| *** agileclipse has quit IRC | 00:20 | |
| *** agileclipse has joined #heat | 00:21 | |
| *** andersonvom has quit IRC | 00:21 | |
| openstackgerrit | A change was merged to openstack/python-heatclient: Refactor environment functions in preparation for get_file https://review.openstack.org/66229 | 00:22 |
| *** matsuhashi has quit IRC | 00:33 | |
| *** agileclipse has quit IRC | 00:37 | |
| *** IlyaE has joined #heat | 00:41 | |
| *** matsuhashi has joined #heat | 00:48 | |
| *** yogesh has joined #heat | 00:48 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat-cfntools: Log curl stderr on non-zero exit status https://review.openstack.org/69245 | 00:54 |
| *** nosnos has joined #heat | 00:58 | |
| *** IlyaE has quit IRC | 00:59 | |
| *** IlyaE has joined #heat | 01:03 | |
| *** faramir has joined #heat | 01:05 | |
| *** nati_ueno has joined #heat | 01:06 | |
| *** agileclipse has joined #heat | 01:08 | |
| *** gokrokve has joined #heat | 01:19 | |
| *** nati_ueno has quit IRC | 01:23 | |
| *** gokrokve has quit IRC | 01:23 | |
| openstackgerrit | liusheng proposed a change to openstack/heat: fix some flaws in heat documents https://review.openstack.org/69248 | 01:30 |
| *** yogesh has quit IRC | 01:31 | |
| *** tsmadds has joined #heat | 01:35 | |
| *** tsmadds has quit IRC | 01:40 | |
| *** larsks has quit IRC | 01:50 | |
| *** samkottler has quit IRC | 01:51 | |
| *** nosnos_ has joined #heat | 01:53 | |
| *** nosnos has quit IRC | 01:54 | |
| *** larsks has joined #heat | 01:56 | |
| *** tsmadds has joined #heat | 01:59 | |
| *** gokrokve has joined #heat | 01:59 | |
| *** tsmadds has quit IRC | 02:04 | |
| *** gokrokve has quit IRC | 02:04 | |
| *** gokrokve has joined #heat | 02:06 | |
| *** JAC has joined #heat | 02:08 | |
| *** gokrokve_ has joined #heat | 02:08 | |
| *** JAC is now known as Guest42717 | 02:08 | |
| *** gokrokve has quit IRC | 02:11 | |
| *** gokrokv__ has joined #heat | 02:11 | |
| *** gokrokve_ has quit IRC | 02:13 | |
| *** gokrokv__ has quit IRC | 02:16 | |
| *** SilverSurfer972 has joined #heat | 02:16 | |
| *** andersonvom has joined #heat | 02:18 | |
| *** matsuhashi has quit IRC | 02:26 | |
| *** andersonvom has quit IRC | 02:27 | |
| *** matsuhashi has joined #heat | 02:32 | |
| *** erkules_ has joined #heat | 02:46 | |
| *** erkules has quit IRC | 02:46 | |
| *** dims has quit IRC | 02:50 | |
| *** sdake has joined #heat | 02:53 | |
| *** sdake has quit IRC | 02:53 | |
| *** sdake has joined #heat | 02:53 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement HOT intrinsic function get_file https://review.openstack.org/66160 | 02:58 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Store files in the raw_template table https://review.openstack.org/67617 | 02:58 |
| *** samkottler has joined #heat | 02:59 | |
| *** tsmadds has joined #heat | 03:00 | |
| *** tsmadds has quit IRC | 03:03 | |
| *** matsuhashi has quit IRC | 03:18 | |
| *** SilverSurfer972 has quit IRC | 03:30 | |
| *** SilverSurfer972 has joined #heat | 03:30 | |
| *** gokrokve has joined #heat | 03:39 | |
| *** gokrokve has quit IRC | 03:44 | |
| *** david-lyle has joined #heat | 03:51 | |
| *** tsmadds has joined #heat | 04:00 | |
| *** tsmadds has quit IRC | 04:05 | |
| *** agileclipse has quit IRC | 04:08 | |
| *** chandankumar_ has joined #heat | 04:10 | |
| *** chandankumar_ has quit IRC | 04:24 | |
| *** ramishra has joined #heat | 04:28 | |
| *** chandankumar_ has joined #heat | 04:29 | |
| *** coolsvap has joined #heat | 04:38 | |
| *** gokrokve has joined #heat | 04:38 | |
| *** ramishra has quit IRC | 04:39 | |
| *** matsuhashi has joined #heat | 04:40 | |
| lifeless | stevebaker: still around? | 04:42 |
| *** gokrokve has quit IRC | 04:43 | |
| lifeless | stevebaker: when you are, can we chat qpid ? | 04:45 |
| *** nkhare has joined #heat | 04:46 | |
| *** chandankumar_ has quit IRC | 04:53 | |
| *** akuznetsov has joined #heat | 04:59 | |
| *** rpothier_ has joined #heat | 05:00 | |
| *** IlyaE has quit IRC | 05:02 | |
| *** rpothier has quit IRC | 05:04 | |
| *** ramishra has joined #heat | 05:05 | |
| stevebaker | lifeless: sure. But I don't run qpid currently | 05:14 |
| *** silversurfer has joined #heat | 05:14 | |
| *** silversurfer is now known as Guest66818 | 05:15 | |
| *** SilverSurfer972 has quit IRC | 05:17 | |
| *** chandankumar_ has joined #heat | 05:23 | |
| lifeless | stevebaker: ok so | 05:28 |
| lifeless | stevebaker: https://review.openstack.org/68785 | 05:30 |
| lifeless | stevebaker: and https://bugs.launchpad.net/heat/+bug/1225191 | 05:30 |
| stevebaker | ok | 05:30 |
| lifeless | stevebaker: if the argument is 'rabbit users don't want the qpid library installed', then clearly the reverse is also an argument | 05:31 |
| lifeless | stevebaker: and we should remove both as dependencies | 05:31 |
| stevebaker | personally I have no problem including both | 05:33 |
| lifeless | ok, simples then :) | 05:33 |
| lifeless | can you say so on the bug ? | 05:33 |
| stevebaker | are any projects including psycopg2? That would be a precedent | 05:33 |
| lifeless | we don't have anyone working on deployment upstream doing pg deploys yet | 05:34 |
| lifeless | RH are working on deployment upstream (yay) but want to deploy with qpid because they support that on RHEL | 05:35 |
| stevebaker | is qipd-python pure python? | 05:35 |
| lifeless | so I think we basically need to decide of qpid is supported or not by us | 05:35 |
| *** tspatzier has joined #heat | 05:36 | |
| stevebaker | yep, I have no problem with that. This will all go away in our glorious AMQP 1.0 future | 05:36 |
| *** tspatzier has quit IRC | 05:38 | |
| lifeless | afaik it is | 05:39 |
| stevebaker | commented | 05:41 |
| *** erkules_ is now known as erkules | 05:41 | |
| *** renlt has joined #heat | 05:44 | |
| skraynev | Morning guys | 05:45 |
| *** ramishra has quit IRC | 05:51 | |
| *** chandankumar_ has quit IRC | 05:52 | |
| *** ramishra has joined #heat | 05:57 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 06:04 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/64504 | 06:05 |
| *** gokrokve has joined #heat | 06:08 | |
| stevebaker | shardy: it looks like all gate heat unit tests are failing - I can't reproduce it locally though http://logs.openstack.org/38/69238/1/check/gate-heat-python27/9899837/testr_results.html.gz | 06:10 |
| *** gokrokve has quit IRC | 06:13 | |
| *** werebutt has joined #heat | 06:20 | |
| *** werebutt has left #heat | 06:20 | |
| *** andersonvom has joined #heat | 06:23 | |
| *** andersonvom has quit IRC | 06:27 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 06:31 | |
| *** IlyaE has joined #heat | 06:35 | |
| *** chandankumar_ has joined #heat | 06:39 | |
| *** chandankumar_ has quit IRC | 06:46 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 06:57 | |
| *** ramishra has quit IRC | 07:00 | |
| *** renlt has quit IRC | 07:00 | |
| *** nati_ueno has joined #heat | 07:06 | |
| *** gokrokve has joined #heat | 07:08 | |
| *** e0ne has joined #heat | 07:09 | |
| *** gokrokve has quit IRC | 07:13 | |
| *** nati_ueno has quit IRC | 07:13 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 07:20 | |
| *** lsmola_ has joined #heat | 07:31 | |
| *** silversurfer has joined #heat | 07:33 | |
| *** Guest66818 has quit IRC | 07:33 | |
| *** silversurfer is now known as Guest89391 | 07:34 | |
| *** coolsvap has quit IRC | 07:39 | |
| *** al-maisan has quit IRC | 07:40 | |
| *** liang has joined #heat | 07:42 | |
| *** renlt has joined #heat | 07:45 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 07:48 | |
| *** tspatzier has joined #heat | 07:49 | |
| *** jprovazn has quit IRC | 07:52 | |
| *** jprovazn has joined #heat | 07:53 | |
| *** IlyaE has quit IRC | 07:59 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 08:03 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 08:03 | |
| *** gokrokve has joined #heat | 08:08 | |
| *** e0ne has quit IRC | 08:12 | |
| *** tiantian has joined #heat | 08:12 | |
| *** gokrokve has quit IRC | 08:12 | |
| shardy | morning | 08:13 |
| therve | Good morning! | 08:13 |
| pshchelo | morning | 08:15 |
| *** Guest89391 has quit IRC | 08:16 | |
| *** renlt has quit IRC | 08:16 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 08:17 | |
| *** coolsvap has joined #heat | 08:21 | |
| *** Guest89391 has joined #heat | 08:21 | |
| *** andersonvom has joined #heat | 08:23 | |
| *** tspatzier has quit IRC | 08:24 | |
| *** andersonvom has quit IRC | 08:27 | |
| therve | stevebaker, I'm not seeing this issue anywhere else? | 08:32 |
| shardy | therve: I also can't reproduce it locally | 08:36 |
| *** matsuhashi has quit IRC | 08:36 | |
| therve | shardy, I'd blame oslo.config | 08:37 |
| therve | We're mocking the global cfg config | 08:37 |
| shardy | I'm not sure how my patches got through the gate if there's a problem, but it would appear it's related to the lazy import which registers the option group | 08:37 |
| shardy | Ah | 08:37 |
| therve | Presumably there is a race condition when you run the tests in parallel | 08:37 |
| *** matsuhashi has joined #heat | 08:37 | |
| *** che-arne has quit IRC | 08:40 | |
| *** ifarkas has joined #heat | 08:40 | |
| *** cmyster has joined #heat | 08:41 | |
| *** tiantian has quit IRC | 08:42 | |
| shardy | therve: The "Update base oslo modules" commit to heat introduces a new config-override fixture, maybe we need to use that | 08:46 |
| therve | shardy, Maybe. As long as we have a global object I'm not sure if it can be fixed though | 08:46 |
| *** jistr has joined #heat | 08:48 | |
| chmouel | global object should be killed IMHO | 08:49 |
| shardy | Yeah, agreed, I'm just confused as to why this has suddenly started failing - that set_override stuff has been in out tests for months | 08:50 |
| *** tsmadds has joined #heat | 08:51 | |
| shardy | s/out/our | 08:51 |
| cmyster | morning | 08:52 |
| therve | shardy, Presumably, you added new tests to that file, increasing the change of it appearing | 08:52 |
| shardy | therve: I guess. Maybe we can mock the calls to cfg.CONF.keystone_authtoken in heat_keystoneclient to work around it | 08:54 |
| therve | Ah yeah maybe | 08:55 |
| therve | I'd rather pass explicit objects though | 08:57 |
| shardy | therve, stevebaker: Raised bug #1273126 | 09:01 |
| shardy | therve: Sorry, I don't quite follow your explicit objects comment, can you explain? | 09:01 |
| therve | shardy, Like, passing a dict to KeystoneClient.__init__ containing the information | 09:02 |
| therve | So that it doesn't have to know about cfg.CONF | 09:02 |
| shardy | therve: Hmm, that just makes using the class less convenient though, as then the caller has to know all the stuff which needs to be looked up and create the dict | 09:04 |
| *** giulivo has joined #heat | 09:04 | |
| therve | Maybe yeah | 09:04 |
| shardy | we use cfg.CONF in lots of places, so I'd rather just figure out a better mocking stategy for the tests, rather than compromising the usability of the code itself | 09:05 |
| therve | Well mocking has the same problems as the current strategy afaict | 09:05 |
| therve | You can scope to a module maybe? | 09:06 |
| shardy | Maybe create a class variable which points to cfg.CONF which can be easily overridden with a mock config object | 09:07 |
| skraynev | therve, excuse me. Could you approve this patch https://review.openstack.org/#/c/68240/ ? | 09:08 |
| *** gokrokve has joined #heat | 09:08 | |
| therve | Sure | 09:08 |
| therve | shardy, Sounds good | 09:08 |
| skraynev | therve, thx ) | 09:09 |
| cmyster | guys, I need an opinion here please, I own bug https://bugzilla.redhat.com/show_bug.cgi?id=1050216 and trying to verify the fix in Grizzly is, well for the lack of other definition a PITA. | 09:09 |
| shardy | cmyster: Can you just verify by inspection, to prove that the required patch exists in the version being verified? | 09:11 |
| cmyster | I could do that, | 09:11 |
| *** gokrokve has quit IRC | 09:13 | |
| *** e0ne has joined #heat | 09:15 | |
| *** al-maisan has joined #heat | 09:16 | |
| *** derekh has joined #heat | 09:18 | |
| therve | Gah we really need https://review.openstack.org/#/c/67844/ in | 09:24 |
| cmyster | shardy: the changes are in place but what if I want to see if they were modified in /test/unit/* ? I need an extra package I suppose ? | 09:24 |
| cmyster | implemented* | 09:25 |
| *** Guest89391 has quit IRC | 09:25 | |
| shardy | cmyster: You mean you want to run the tests under heat/tests? | 09:25 |
| *** Guest89391 has joined #heat | 09:25 | |
| therve | shardy, In https://review.openstack.org/#/c/67531/, aren't you missing VerifyAll calls? | 09:26 |
| cmyster | at least to see that the tests were changed according to whats specified in https://review.openstack.org/#/c/32235/13 | 09:26 |
| openstackgerrit | huangtianhua proposed a change to openstack/heat: I18N support for log message and attributes schema https://review.openstack.org/69111 | 09:26 |
| *** tiantian has joined #heat | 09:26 | |
| shardy | therve: It's added via addCleanup | 09:26 |
| *** pshchelo has quit IRC | 09:26 | |
| therve | Oh cool | 09:27 |
| therve | I'm glad I asked here :) | 09:27 |
| shardy | cmyster: Those are oslo tests, which don't get copied into the heat tree when we sync oslo modules into heat/openstack/common | 09:28 |
| shardy | therve: It is confusing as some tests do it via addCleanup and others repeat the VerifyAll throughout the tests | 09:28 |
| cmyster | I guessed as much, question is, is verifying that the changes in the link ^ in openstack/heat suffice | 09:28 |
| shardy | cmyster: Given the nature of the bug it may be impractical to do anything else, I have no idea how you can provoke next_receiver() to raise the required exception | 09:32 |
| shardy | cmyster: I guess you could ask russellb since he reported the bug | 09:33 |
| *** pshchelo has joined #heat | 09:34 | |
| openstackgerrit | Ryo Miki proposed a change to openstack/heat: Add handle_update for OS::Neutron::RouterL3Agent https://review.openstack.org/69303 | 09:34 |
| cmyster | shardy: the issue is not with provoking it, I guess I can try and reach that area of code or write a small unit test around it. thing is even deploying heat on grizzly is a pitta and I am stuck at that point | 09:34 |
| *** _ruhe is now known as ruhe | 09:35 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/python-heatclient: Fix order of arguments in assertEqual https://review.openstack.org/64039 | 09:36 |
| skraynev | shardy: should we support I18 for attribute schema ? | 09:39 |
| skraynev | shardy: I have thought, that it makes sense only for error and log messages. | 09:40 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/python-heatclient: Add `update_type` as parameter to stack-update https://review.openstack.org/64274 | 09:45 |
| *** tiantian has quit IRC | 09:45 | |
| *** tnurlygayanov_ has quit IRC | 09:49 | |
| *** tnurlygayanov has joined #heat | 09:50 | |
| *** al-maisan has quit IRC | 09:57 | |
| openstackgerrit | Ryo Miki proposed a change to openstack/heat: Fix handle_create of NetDHCPAgent for updating https://review.openstack.org/69311 | 09:58 |
| *** al-maisan has joined #heat | 10:02 | |
| shardy | cmyster: Can you explain why deploying heat is problematic on grizzly? | 10:03 |
| shardy | cmyster: There are a few manual steps but it should be pretty straight forward | 10:03 |
| shardy | skraynev: AFAIK there are no plans for making attribute/properties schema translatable atm | 10:04 |
| *** tspatzier has joined #heat | 10:04 | |
| cmyster | maybe its me, but I am still working on it. on two different machines I followed http://openstack.redhat.com/Deploy_Heat_and_launch_your_first_Application, followed it to the letter and failed on different steps. | 10:04 |
| cmyster | still, it could be me, I'm retrying on a re-re-re-provisioned server | 10:05 |
| shardy | cmyster: If you provide details of the problems I'm sure we can help get things working | 10:05 |
| skraynev | shardy, yeap. I have thought so. | 10:05 |
| skraynev | shardy: thanks | 10:05 |
| cmyster | shardy: working on it. I want to be able to troubleshoot it alone. best way to learn anything. | 10:05 |
| *** gokrokve has joined #heat | 10:08 | |
| *** gokrokve has quit IRC | 10:13 | |
| *** e0ne_ has joined #heat | 10:13 | |
| *** faramir has quit IRC | 10:13 | |
| *** chandankumar_ has joined #heat | 10:13 | |
| *** che-arne has joined #heat | 10:14 | |
| *** e0ne has quit IRC | 10:16 | |
| *** rurul has joined #heat | 10:16 | |
| rurul | hi | 10:17 |
| *** andersonvom has joined #heat | 10:23 | |
| openstackgerrit | Thomas Spatzier proposed a change to openstack/heat: Refactor Parameters Schema based on common Schema https://review.openstack.org/67183 | 10:24 |
| openstackgerrit | Thomas Spatzier proposed a change to openstack/heat: Move param format for template-validate to API https://review.openstack.org/67171 | 10:24 |
| *** andersonvom has quit IRC | 10:28 | |
| *** chandankumar_ has quit IRC | 10:37 | |
| *** pshchelo has quit IRC | 10:39 | |
| *** pshchelo has joined #heat | 10:39 | |
| *** alexpilotti has joined #heat | 10:58 | |
| *** alexpilotti has quit IRC | 11:03 | |
| *** chandankumar_ has joined #heat | 11:03 | |
| *** coolsvap_away has joined #heat | 11:05 | |
| *** coolsvap has quit IRC | 11:06 | |
| *** aignatov_ is now known as aignatov | 11:07 | |
| *** gokrokve has joined #heat | 11:08 | |
| *** chandankumar_ has quit IRC | 11:08 | |
| *** ruhe is now known as _ruhe | 11:12 | |
| *** gokrokve has quit IRC | 11:13 | |
| *** chandankumar_ has joined #heat | 11:15 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 11:17 | |
| *** alexpilotti has joined #heat | 11:25 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 11:33 | |
| *** coolsvap_away is now known as coolsvap | 11:36 | |
| *** chandankumar_ has quit IRC | 11:39 | |
| *** _ruhe is now known as ruhe | 11:39 | |
| *** e0ne has joined #heat | 11:39 | |
| *** matsuhashi has quit IRC | 11:40 | |
| *** e0ne_ has quit IRC | 11:42 | |
| *** matsuhashi has joined #heat | 11:45 | |
| *** agileclipse has joined #heat | 11:49 | |
| *** asalkeld has quit IRC | 11:49 | |
| openstackgerrit | Swann Croiset proposed a change to openstack/heat: Add autoscaling notifications https://review.openstack.org/62087 | 11:57 |
| *** gokrokve has joined #heat | 11:59 | |
| *** nkhare has quit IRC | 12:00 | |
| *** faramir has joined #heat | 12:01 | |
| *** chandankumar_ has joined #heat | 12:05 | |
| *** chandankumar_ has quit IRC | 12:08 | |
| *** matsuhashi has quit IRC | 12:12 | |
| *** matsuhashi has joined #heat | 12:13 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stub cfg.CONF in heat_keystoneclient https://review.openstack.org/69339 | 12:14 |
| *** e0ne_ has joined #heat | 12:16 | |
| *** e0ne has quit IRC | 12:20 | |
| *** dims has joined #heat | 12:23 | |
| *** andersonvom has joined #heat | 12:24 | |
| *** andersonvom has quit IRC | 12:28 | |
| *** asalkeld has joined #heat | 12:30 | |
| *** rpothier_ has quit IRC | 12:33 | |
| *** coolsvap has quit IRC | 12:36 | |
| cmyster | heh | 12:40 |
| cmyster | https://access.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/3/html/Release_Notes/ch02s02.html | 12:40 |
| cmyster | guess I need to re-re-re-re-re-provision :) | 12:40 |
| *** ruhe is now known as _ruhe | 12:52 | |
| *** chandankumar_ has joined #heat | 12:54 | |
| *** _ruhe is now known as ruhe | 12:54 | |
| *** Tross1 has quit IRC | 12:58 | |
| *** andersonvom has joined #heat | 13:01 | |
| *** andersonvom has quit IRC | 13:02 | |
| *** andersonvom has joined #heat | 13:02 | |
| *** david-lyle has quit IRC | 13:05 | |
| *** aignatov is now known as aignatov_ | 13:06 | |
| *** chandankumar_ has quit IRC | 13:22 | |
| *** chandankumar_ has joined #heat | 13:23 | |
| *** Guest89391 has quit IRC | 13:25 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 13:31 | |
| *** rpothier has joined #heat | 13:34 | |
| *** aignatov_ is now known as aignatov | 13:38 | |
| *** aweiteka has joined #heat | 13:38 | |
| *** sgordon has joined #heat | 13:44 | |
| *** faramir has quit IRC | 13:44 | |
| *** chandankumar_ has quit IRC | 13:44 | |
| *** pafuent has joined #heat | 13:47 | |
| *** gokrokve has quit IRC | 13:50 | |
| *** gokrokve has joined #heat | 13:55 | |
| *** randallburt has joined #heat | 13:58 | |
| *** rurul has quit IRC | 13:58 | |
| *** rcleere has quit IRC | 13:59 | |
| *** skraynev has quit IRC | 14:00 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stub cfg.CONF in heat_keystoneclient https://review.openstack.org/69339 | 14:00 |
| *** skraynev has joined #heat | 14:02 | |
| *** blomquisg has joined #heat | 14:07 | |
| *** rustlebee is now known as russellb | 14:12 | |
| *** dims has quit IRC | 14:13 | |
| *** Tross1 has joined #heat | 14:14 | |
| *** Tross1 has left #heat | 14:14 | |
| rpothier | Hi All, I have an issue with a new install of heat on Havana. | 14:14 |
| rpothier | I’m getting 403 error with ‘heat list’, however, when I run heat –debug list, and then run the curl | 14:14 |
| rpothier | command manually, the curl command succeeds . Has anyone seen this? | 14:14 |
| *** dims has joined #heat | 14:15 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/heat: Use a HeatIdentifier instance to create Parameters https://review.openstack.org/68257 | 14:16 |
| *** jcru has joined #heat | 14:21 | |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/heat: RPC service for software config/deployment https://review.openstack.org/58877 | 14:21 |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/heat: REST API for software config/deployment https://review.openstack.org/66514 | 14:21 |
| *** chandankumar_ has joined #heat | 14:24 | |
| *** achampion has quit IRC | 14:24 | |
| randallburt | rpothier: are you on a mac? | 14:26 |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/heat: Native Pseudo Parameters https://review.openstack.org/66640 | 14:27 |
| *** chandankumar_ has quit IRC | 14:27 | |
| rpothier | no, this is a multinode system with ubuntu servers | 14:27 |
| openstackgerrit | Swann Croiset proposed a change to openstack/heat: Fix event_type names for stack CRUD notifications https://review.openstack.org/69372 | 14:28 |
| randallburt | rpothier: hrm. sorry then. thought https://bugs.launchpad.net/python-heatclient/+bug/1266581 could be related | 14:28 |
| *** chandankumar_ has joined #heat | 14:29 | |
| rpothier | randallburt: thanks, looks similar, but I get "code": 403, "type": "HTTPForbidden"} | 14:33 |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/heat: Prevent access Parameters key in template dict https://review.openstack.org/68312 | 14:33 |
| *** chandankumar_ has quit IRC | 14:33 | |
| randallburt | rpothier: odd. might be a keystone config issue? | 14:34 |
| *** rcleere has joined #heat | 14:34 | |
| *** rcleere has quit IRC | 14:34 | |
| *** chandankumar_ has joined #heat | 14:35 | |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/heat: Resource type for software configuration https://review.openstack.org/67621 | 14:35 |
| *** radez_g0n3 is now known as radez | 14:38 | |
| *** vijendar has joined #heat | 14:40 | |
| shardy | rpothier: Can you check the API and engine logs and paste any errors into paste.openstack.org? | 14:44 |
| *** chandankumar_ has quit IRC | 14:44 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Allow force registration of Rackspace resources https://review.openstack.org/69374 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add Docker resources to docs https://review.openstack.org/68753 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Rename rackspace.rst to contrib.rst https://review.openstack.org/68752 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add Rackspace resources summary to docs https://review.openstack.org/68793 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add force_resource_mapping option https://review.openstack.org/68746 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Restructure Rackspace resources https://review.openstack.org/68747 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Reorganize docker-plugin for consistency https://review.openstack.org/68748 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Allow force registration of docker resource https://review.openstack.org/68749 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Change Docker resource mapping name https://review.openstack.org/68750 | 14:45 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Generate docs for contrib plugins https://review.openstack.org/68751 | 14:45 |
| rpothier | shardy: thanks - http://paste.openstack.org/show/61939/ | 14:50 |
| rpothier | shardy: nothing with todays date in the heat-engine.log | 14:50 |
| shardy | rpothier: can you set debug=true in /etc/heat/heat.conf, restart the API and engine services, then try again please? | 14:51 |
| *** ativelkov has left #heat | 14:52 | |
| *** aignatov is now known as aignatov_ | 14:52 | |
| *** che-arne has quit IRC | 14:56 | |
| *** jdob has joined #heat | 14:56 | |
| *** chandankumar_ has joined #heat | 14:57 | |
| *** agileclipse has quit IRC | 14:57 | |
| *** nkhare has joined #heat | 14:59 | |
| *** che-arne has joined #heat | 14:59 | |
| rpothier | shardy: I am not getting any more information, I do see an error with /etc/heat/environment.d on restart, could that be related? | 14:59 |
| shardy | rpothier: probably not, but please paste it anyway | 15:00 |
| openstackgerrit | A change was merged to openstack/heat: Updated from global requirements https://review.openstack.org/68240 | 15:00 |
| rpothier | shardy: http://paste.openstack.org/show/61942/ | 15:02 |
| shardy | rpothier: How did you install heat? It looks like the contents of /etc/heat/ is wrong, there should be an environment.d directory, containing the default environment | 15:05 |
| shardy | https://github.com/openstack/heat/tree/stable/havana/etc/heat | 15:06 |
| *** chandankumar_ has quit IRC | 15:06 | |
| *** rcleere has joined #heat | 15:07 | |
| *** e0ne has joined #heat | 15:11 | |
| *** e0ne_ has quit IRC | 15:11 | |
| *** tims has quit IRC | 15:11 | |
| sdake | morning | 15:11 |
| rpothier | shardy: installed through an installer we package here, looks like environment.d is missing, I will update this and recheck, thanks! | 15:12 |
| *** ruhe is now known as _ruhe | 15:12 | |
| *** achampion has joined #heat | 15:13 | |
| *** rwsu has joined #heat | 15:16 | |
| *** cody-somerville has joined #heat | 15:17 | |
| shardy | rpothier: That probably is unrelated to your issue, but a good thing to fix anyway | 15:18 |
| shardy | rpothier: If the curl request is working, and the heat CLI isn't, I guess the next step is to check the traffic (e.g with ngrep) to see what's different | 15:19 |
| shardy | rpothier: also please confirm what version of python-heatclient you're using | 15:19 |
| *** sgran has quit IRC | 15:20 | |
| andersonvom | morning | 15:23 |
| shardy | Hi andersonvom | 15:23 |
| rpothier | shardy: I tried to switch to the stable/havana version of python-heatclient by downloading and changing my pythonpath | 15:23 |
| rpothier | shardy: but I don't see a stable/havana version? | 15:23 |
| shardy | rpothier: there is no stable branch for python-heatclient, it should work with master and havana/grizzly | 15:24 |
| *** scollier has quit IRC | 15:24 | |
| andersonvom | hey shardy! just getting ready for another day of unscoped stacks here ;) | 15:24 |
| shardy | the clients are supposed to be backwards compatible and have an independent release cadence from the openstack services | 15:24 |
| rpothier | shardy: I will try that and check traffic, thanks! | 15:25 |
| shardy | andersonvom: Hey, yeah I added a comment to the review - have we reached consensus re the interfaces now? | 15:25 |
| shardy | zaneb: ^^ | 15:26 |
| * zaneb goes back to look | 15:26 | |
| shardy | https://review.openstack.org/#/c/63041/ | 15:26 |
| zaneb | thanks :) | 15:26 |
| *** sabeen has joined #heat | 15:27 | |
| therve | Hum that cfg bug happened again | 15:28 |
| therve | zaneb, https://review.openstack.org/#/c/69339/ too if you can | 15:28 |
| cmyster | this is the one with the race conditions ? | 15:29 |
| therve | Yeah | 15:29 |
| shardy | therve: Maybe we should merge my "fix" and I'll rebase the remaining v3-only patches on that | 15:30 |
| zaneb | shardy: comment added | 15:30 |
| therve | shardy, Your call. You can wait otherwise :) | 15:31 |
| therve | I guess it does modify the same test file | 15:31 |
| *** e0ne has quit IRC | 15:31 | |
| shardy | therve: I'm going to have to rebase whichever way we do it ;) | 15:31 |
| *** e0ne has joined #heat | 15:31 | |
| therve | Right | 15:31 |
| therve | Well the bug blocked your other branches, so that makes sense | 15:31 |
| andersonvom | shardy: I think so. I like the general idea so for, and I think the query param option works and solves our problem as well... so I'm gonna go with that until told otherwise | 15:32 |
| shardy | therve: The fix is pretty ugly but I couldn't think of another way to do it | 15:32 |
| therve | You think so? That ain't so bad | 15:32 |
| *** _ruhe is now known as ruhe | 15:33 | |
| therve | It's hard to build something nice on top of oslo.config | 15:33 |
| *** aignatov_ is now known as aignatov | 15:33 | |
| zaneb | I believe olso.config is eventually going to allow a non-global config object | 15:34 |
| shardy | andersonvom: Ok, sounds good - I think it should be fairly simple, new parameter to the engine list_stacks API, API query parameter and a little bit of refactoring to the API policy_enforce so you can call the policy enforcer inside the controller function not just as a decorator | 15:35 |
| zaneb | it's moderately ugly, but I think it will do. approved | 15:35 |
| shardy | zaneb: thanks | 15:35 |
| *** scollier has joined #heat | 15:36 | |
| andersonvom | shardy: yeah... hopefully that'll be it! but since we're on the subject... do we expect any particular behavior from the timestamp stuff? since I'll have to change that, I just want to make sure I'll keep whatever behavior is expected from it | 15:37 |
| therve | zaneb, That'd be nice. https://wiki.openstack.org/wiki/Oslo#Why_does_oslo.config_have_a_CONF_object.3F_Global_object_SUCK.21 always made me moderatively angry | 15:37 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 15:38 | |
| zaneb | therve: hmm, it's possible my information is out of date then. I thought I had heard talk of it more recently than that | 15:38 |
| therve | This wiki page is very old | 15:39 |
| zaneb | seems odd, because it shouldn't be that hard to create a transition path | 15:39 |
| *** coolsvap has joined #heat | 15:40 | |
| *** IlyaE has joined #heat | 15:40 | |
| shardy | andersonvom: If you go with the approach described, will you still have to change the timestamp stuff? | 15:41 |
| shardy | andersonvom: I was thinking you'd just need a parameter to the list_stacks which calls stack_get_all instead of get_all_by_tenant | 15:42 |
| *** david-lyle has joined #heat | 15:42 | |
| shardy | andersonvom: and you won't need any of the context stuff at all | 15:42 |
| *** metabro has quit IRC | 15:44 | |
| andersonvom | shardy: the thing is that everytime we fetch a bunch of stacks from the database, they get parsed by parser.Stack(), which implies calling stack_get to get the timestamp again. so I'm pretty sure the answer is yes. | 15:47 |
| *** tims has joined #heat | 15:48 | |
| *** tims1 has joined #heat | 15:48 | |
| *** tims has quit IRC | 15:48 | |
| andersonvom | shardy: so if we want to prevent access to all stacks, we have to change the timestamp, cause that's what's really causing the problem. | 15:48 |
| *** tims1 has quit IRC | 15:50 | |
| zaneb | shardy, andersonvom: tbh I think enforcing policies based on which database call you make is probably a mistake anyway | 15:52 |
| zaneb | you should choose a policy based on the request, and just enforce that policy in all database calls | 15:52 |
| *** nkhare has quit IRC | 15:52 | |
| *** sdake has quit IRC | 15:52 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Move X-Auth-Url logic to auth_url middleware https://review.openstack.org/68800 | 15:53 |
| zaneb | the problem with the current patch is that it's not granular enough in selecting a policy | 15:53 |
| shardy | zaneb: well, true, but there's two aspects, whether we process the call, and the scope of the data returned/accessible | 15:53 |
| shardy | I agree the policy part of it should be granular enough to enforce stuff per API action | 15:53 |
| zaneb | shardy: agreed, but the scope accessible can be encoded into the context | 15:53 |
| zaneb | as it is in andersonvom's current patch | 15:53 |
| shardy | zaneb: Ok, so you want to keep the is_service_admin stuff, and add the global_index as well? | 15:54 |
| shardy | I was suggesting we should just have the global_index policy rule, and have a specific exception to normal request scoping just for list_stacks | 15:55 |
| zaneb | shardy: actually I hate is_service_admin | 15:55 |
| *** jprovazn has quit IRC | 15:56 | |
| zaneb | I'd prefer tenant_id == GLOBAL_TENANT or something | 15:56 |
| shardy | zaneb: but what sets that - tenant_id is always set based on what keystone gives us from the token? | 15:56 |
| *** metabro has joined #heat | 15:56 | |
| zaneb | right, but we can change it | 15:57 |
| zaneb | another flag would be OK too | 15:57 |
| zaneb | just not 'really_admin' | 15:57 |
| shardy | Ok, but where do we do that, is it derived from policy or just a hack-override in list_stacks? | 15:58 |
| zaneb | or it could even be just a parameter to list_stacks | 15:58 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 15:58 | |
| shardy | zaneb: I was suggesting an additional parameter to EngineService::ListStacks, global_scope or something | 15:58 |
| *** randallburt has quit IRC | 15:58 | |
| *** jistr has quit IRC | 15:58 | |
| shardy | Or tenant_safe I guess since we use that elsewhere already | 15:59 |
| zaneb | but at the end of the day, I think it's somewhat naive to think that one API call will always map to exactly one db call | 15:59 |
| zaneb | even if we clean up the timeout stuff | 15:59 |
| shardy | FWIW there is a precedent for a super-really-admin policy rule: | 16:00 |
| shardy | https://github.com/openstack/keystone/blob/master/etc/policy.v3cloudsample.json#L3 | 16:00 |
| *** tims has joined #heat | 16:01 | |
| shardy | but I agree using it as a global switch is a bad idea, it's repeating the is_admin mistake | 16:01 |
| *** randallburt has joined #heat | 16:01 | |
| *** tspatzier has quit IRC | 16:10 | |
| *** nosnos_ has quit IRC | 16:14 | |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Provide region_name to trove-client https://review.openstack.org/69397 | 16:15 |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Delete rackspace database resource https://review.openstack.org/68144 | 16:15 |
| *** sdake has joined #heat | 16:15 | |
| *** sdake has quit IRC | 16:15 | |
| *** sdake has joined #heat | 16:15 | |
| *** matsuhashi has quit IRC | 16:15 | |
| *** aignatov is now known as aignatov_ | 16:16 | |
| jpeeler | shardy: i assume now that https://review.openstack.org/#/c/59685/ has been merged, that this patch can be dropped: https://github.com/redhat-openstack/heat/commit/5a86c19a2b5a5b13bd680a3ec912ee064588cd18 | 16:20 |
| *** gokrokve has quit IRC | 16:21 | |
| shardy | jpeeler: won't you need it for backwards compatibility, e.g to decrypt existing credentials in the DB after upgrade? | 16:21 |
| *** jergerber has joined #heat | 16:22 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Add personality files property to Server resource https://review.openstack.org/68685 | 16:23 |
| *** e0ne_ has joined #heat | 16:24 | |
| *** scollier has quit IRC | 16:25 | |
| jpeeler | shardy: guess so, so should it be modified to use the new crypto or just continue to patch it the same way? | 16:25 |
| *** ruhe is now known as _ruhe | 16:25 | |
| *** e0ne has quit IRC | 16:26 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 16:30 | |
| shardy | jpeeler: Honestly I don't know - oslo is still using Crypto.Cipher | 16:30 |
| shardy | jpeeler: Who made the call to carry that patch initially? | 16:31 |
| *** vijendar has quit IRC | 16:31 | |
| sdake | I made the call because rhel was broken without it | 16:31 |
| shardy | sdake: Hey, do we know if that issue still exists on el6.5? | 16:31 |
| sdake | iirc the dependency was entirely broken re crypto | 16:31 |
| sdake | shardy no clue | 16:31 |
| jpeeler | i may just have to leave it for now and migrate it later if that's decided to be best | 16:32 |
| sdake | if its not broken we should get rid of special case patches | 16:32 |
| shardy | sdake: Ok, cool, I guess that's the first thing to find out | 16:32 |
| shardy | sdake: +1 | 16:32 |
| sdake | but I expect rhel 6.5 is still broken since there was no bug filed | 16:32 |
| aweiteka | i've got a multi-instance heat template for openshift where the instances reboot after deploy. i need to run some additional 1-time config (bash) when it comes back up. where can i put my script so it runs once? cloud-init dir? | 16:33 |
| jpeeler | sdake: is it ok to do as i proposed? that is, just leave it for now | 16:33 |
| sdake | jpeeler status quo orks for me | 16:34 |
| *** jistr has joined #heat | 16:34 | |
| sdake | at some point it would be nice to dig into that special case | 16:34 |
| sdake | but I think that would make more sense around icehouse rather then mid cycle | 16:34 |
| shardy | I'm not sure I understand, why can't we fix it in RHEL, why isn't there a bug? | 16:34 |
| sdake | around icehosue rrelease that is | 16:34 |
| sdake | there should be a bug | 16:34 |
| sdake | I'm not sure what the actual bug is, and nobody ever filed one iirc | 16:35 |
| sdake | one should be filed | 16:35 |
| sdake | I would file one myself but I'm not quite sure what the exact issue is | 16:35 |
| shardy | jpeeler: Are you in a position to test with/without the patch on el6.5 and raise a bug if needed? | 16:35 |
| sdake | does anyone know what the real issue is? | 16:35 |
| jpeeler | shardy: eventually... but the package needs to be done today | 16:36 |
| jpeeler | sdake: i thought the main issue was that pycrypto wasn't blessed crypto in RHEL | 16:37 |
| sdake | well lets not rush a breaking change :) | 16:37 |
| sdake | jpeeler the other crypto lib doesn't wokr at all | 16:37 |
| *** gokrokve has joined #heat | 16:38 | |
| therve | sdake, Perfect security then! | 16:38 |
| sdake | pycrypto is the blessed one yes, but the one we have in the code base as a dependency is totally broken in rhel | 16:38 |
| sdake | therve ;=) | 16:38 |
| sdake | jpeeler this is for rhos 3.0? | 16:38 |
| sdake | rhos 3.0 = rhel 6.4 | 16:39 |
| jpeeler | sdake: no icehouse-2 packages | 16:39 |
| sdake | jpeeler take a look before i3 then and see if it can be removed | 16:39 |
| sdake | I suspect oslo's usage of that dependency is also broken in rhel all across openstack ;) | 16:39 |
| *** randallburt has quit IRC | 16:40 | |
| sdake | which would put a bit more pressure on platform to fix it :) | 16:40 |
| jpeeler | well at the least it'll have to remain forever for upgrades sounds like | 16:40 |
| sdake | we could add a special migration in the db perhaps | 16:40 |
| * sdake ughs - this is why upstream first is mandatory :) | 16:40 | |
| sdake | thankfully its only one patch | 16:41 |
| sdake | in retrospect the proper solution would have been to fix the dependent crypto package to work correctly | 16:41 |
| sdake | my galaxy note phone rocks | 16:43 |
| sdake | wireless charging = winning | 16:43 |
| therve | jpeeler, We still import pycrypto fwiw, you probably need to update that patch | 16:46 |
| jpeeler | yes, the patch definitely needs updating | 16:46 |
| therve | pycrypto has been maintained again fwiw, maybe the decision to boot it off rhel can be changed | 16:47 |
| *** randallburt has joined #heat | 16:49 | |
| *** vijendar has joined #heat | 16:50 | |
| sdake | therve I think the issue was it has a bunch of security vulnerabilities that our security people don't want to deal with | 16:50 |
| sdake | they want everyone to use 1 lib :) | 16:50 |
| sdake | (good luck) | 16:50 |
| therve | sdake, Right, but I thought those had been fixed | 16:50 |
| sdake | not clear - I know enough about crypto to use it, but not analyze an implementation for faults | 16:51 |
| sdake | implemented crypto+signing for corosync some years back | 16:51 |
| sdake | was intersting learning how crypto works | 16:51 |
| sdake | semi-expert :) | 16:51 |
| *** lsmola_ has quit IRC | 16:53 | |
| *** pablosan has joined #heat | 16:54 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 16:54 | |
| *** e0ne_ has quit IRC | 16:54 | |
| andersonvom | shardy, zaneb: just double checking... is this what we want? | 16:58 |
| andersonvom | 1) add another rule for stacks:global_index (defaults to nobody); | 16:58 |
| andersonvom | 3) add a flag called global_tenant (default to False) to the context; | 16:58 |
| andersonvom | 2) add a query param to be able to switch from regular index to global_index and enforce the policy; | 16:58 |
| andersonvom | 2.1) if global_index, set global_tenant to True | 16:58 |
| andersonvom | 4) check for global_tenant at DB level and scope by tenant if False | 16:58 |
| *** tspatzier has joined #heat | 16:59 | |
| *** gokrokve has quit IRC | 17:01 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 17:03 | |
| shardy | andersonvom: basically, although I was questioning the need for the global_tenant flag in the context | 17:03 |
| andersonvom | shardy: what would we use intead? stick to is_service_admin? | 17:05 |
| zaneb | andersonvom: ++ | 17:05 |
| * zaneb has been on the phone; goes to read scrollback | 17:05 | |
| shardy | andersonvom: Are you proposing setting that flag via a policy rule or by copying the current context and forcing that flag in EngineService::list_stacks()? | 17:05 |
| *** gokrokve has joined #heat | 17:08 | |
| andersonvom | shardy: what if you check for the query param + policy rule and set it in the context? | 17:09 |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Provide region_name to trove-client https://review.openstack.org/69413 | 17:10 |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Delete rackspace database resource https://review.openstack.org/69414 | 17:10 |
| andersonvom | shardy: something like this: https://github.com/openstack/heat/blob/master/heat/common/context.py#L70 | 17:10 |
| andersonvom | shardy: hummm... we don't have the request there. never mind | 17:12 |
| shardy | andersonvom: So is the only reason we can't just use stack_get_all in list_stacks the timestamp stuff? | 17:12 |
| andersonvom | shardy: yup. | 17:12 |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Delete rackspace database resource https://review.openstack.org/68144 | 17:13 |
| shardy | andersonvom: So maybe fixing that is preferable to keeping the global flag in the context | 17:13 |
| shardy | andersonvom: basically having seen the impact of the global flag in the latest patch, I'm thinking we really need a way to keep the scope override local to the code which handles the list_stacks request | 17:14 |
| shardy | zaneb: Can you comment on why the timestamp code works like that? If we pass in a DB object to the parser.Stack.load, why do the timestamps need to do another stack_get? | 17:16 |
| shardy | I'm not sure I fully understand what the functools.partial magic is doing there | 17:16 |
| zaneb | it seemed at the time like we would always want the most up-to-date timestamp | 17:16 |
| zaneb | as it turns out, we | 17:16 |
| zaneb | we're storing the completely wrong thing in the update_timestamp field anyway | 17:17 |
| zaneb | so not only was it not that important, if we were doing it right it wouldn't even be an issue | 17:17 |
| *** gokrokve has quit IRC | 17:18 | |
| zaneb | https://bugs.launchpad.net/heat/+bug/1193269 | 17:18 |
| zaneb | I think I will reassign that to nobody | 17:19 |
| *** gokrokve_ has joined #heat | 17:20 | |
| *** cmyster has quit IRC | 17:23 | |
| andersonvom | shardy: since v2 will hopefully solve the scoping problem anyway, shouldn't we keep it the context and figure out whether to filter by tenant there? | 17:24 |
| *** gokrokve_ has quit IRC | 17:25 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 17:26 | |
| shardy | andersonvom: and do what, set it in StackController.index? | 17:27 |
| andersonvom | shardy: hummm true... I'll just pass the "global" query param down as tenant_safe, since like you said, we're already using it elsewhere, and use that to choose to filter by tenant or not | 17:30 |
| shardy | andersonvom: +1 | 17:35 |
| andersonvom | o/ | 17:35 |
| *** pvaneck has joined #heat | 17:36 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 17:41 | |
| pafuent | SpamapS: Are you arround? | 17:46 |
| SpamapS | pafuent: I am. how's the migration going? | 17:46 |
| pafuent | SpamapS: So, so | 17:46 |
| pafuent | SpamapS: I managed to get it working for mysql | 17:46 |
| *** gokrokve has joined #heat | 17:47 | |
| pafuent | SpamapS: But when I tried to run tox, to see if everything is Ok, a lot of test failed due to how my migration runs for sqlite | 17:47 |
| pafuent | SpamapS: In sqlite seems to be 'impossible' to drop primary key columns | 17:48 |
| SpamapS | pafuent: you may have to create a new table, copy, and rename. | 17:49 |
| SpamapS | which, IMO, is a bug in sqlite | 17:49 |
| pafuent | SpamapS: I read that. It will be fine if I do that for sqlite only? | 17:50 |
| SpamapS | pafuent: totally | 17:50 |
| SpamapS | pafuent: if you can test on pgsql, I think handling all 3 of those is important. | 17:50 |
| pafuent | SpamapS: I'll try. I never use pgsql and I hope use it with devstack is easy | 17:51 |
| shardy | Hmm, has anyone else found run_test.sh fails all the contrib tests? | 17:53 |
| sdake | 5000 unread mails in openstack-dev | 17:54 |
| * sdake ughs | 17:54 | |
| pafuent | SpamapS: BTW, What should I do? Upload my patch using the same Change-ID of Chmouel? I need to write something special in the comment telling that I'm continuing with this patch? | 17:54 |
| *** derekh has quit IRC | 17:55 | |
| shardy | nvm it's my environment | 17:55 |
| SpamapS | pafuent: yes. If you are just tweaking it, add Co-Authored-By: you .. if you are doing most of the work, switch him to Co-Author and take the Author piece. | 17:55 |
| pafuent | SpamapS: Ok, thanks | 17:56 |
| *** akuznetsov has quit IRC | 18:00 | |
| radix | was the Rackspace Cloud DB resource in the Havana release? | 18:03 |
| *** Tross has joined #heat | 18:04 | |
| shardy | radix: https://github.com/openstack/heat/tree/stable/havana/contrib/rackspace/heat/engine/plugins | 18:04 |
| radix | hmm, guess it was | 18:04 |
| radix | so is there a compatibility concern? I guess contrib has no guarantees | 18:04 |
| randallburt | radix: contrib means never having to say you're sorry | 18:05 |
| radix | :) | 18:05 |
| *** akuznetsov has joined #heat | 18:05 | |
| radix | +1 on delete then! | 18:05 |
| *** nati_ueno has joined #heat | 18:05 | |
| *** harlowja_away is now known as harlowja | 18:05 | |
| randallburt | I suppose we could do a deprecation cycle, but it doesn't really seem worth it for the Rackspace:: resources IMO. | 18:06 |
| *** david-lyle has quit IRC | 18:06 | |
| SpamapS | deprecate them | 18:07 |
| *** arbylee has joined #heat | 18:07 | |
| SpamapS | it's hurting nobody to leave them there, functioning, for 3 more months | 18:07 |
| SpamapS | that reminds me we should have a way to warn users they're using deprecated features | 18:08 |
| radix | SpamapS: we need a way to warn users about all sorts of things | 18:08 |
| SpamapS | Yeah, we need the concept of warnings in the response. | 18:08 |
| radix | I guess we could use the event system, but it'd be a lot better to have them in the response | 18:09 |
| SpamapS | events would be the wrong place IMO | 18:09 |
| radix | there are a ton of log statements in the heat codebase that are wishfully aimed at the end user | 18:09 |
| radix | like "oh, maybe a sysadmin will send an email to the user with this log message" | 18:10 |
| SpamapS | We had at least in principle decided not to log such things. | 18:12 |
| SpamapS | the logs need to be informing the sysadmin of whether or not _HEAT_ is working right | 18:12 |
| SpamapS | the stack is the user's problem :) | 18:12 |
| radix | right | 18:17 |
| randallburt | You can specify the support status of a resource now, so that's an option. | 18:19 |
| *** vt102 has joined #heat | 18:20 | |
| *** tsmadds has quit IRC | 18:20 | |
| *** david-lyle has joined #heat | 18:21 | |
| *** gokrokve has quit IRC | 18:21 | |
| pafuent | SpamapS: It's ok if I import the actual model and call .create() in the migration script? | 18:22 |
| pafuent | SpamapS: It's for the sqlite thing | 18:23 |
| SpamapS | pafuent: I believe so yes. | 18:23 |
| pafuent | SpamapS: Ok | 18:23 |
| SpamapS | pafuent: migrations aren't expected to necessarily work "online". | 18:23 |
| pafuent | SpamapS: Ok, thanks. | 18:24 |
| *** gokrokve_ has joined #heat | 18:24 | |
| *** akuznetsov has quit IRC | 18:25 | |
| *** nati_ueno has quit IRC | 18:25 | |
| *** nati_ueno has joined #heat | 18:26 | |
| *** nati_ueno has quit IRC | 18:26 | |
| *** nati_ueno has joined #heat | 18:27 | |
| aweiteka | sdake, is there a CI system to support heat-template validation? i'm thinking of support matrix issues | 18:28 |
| SpamapS | I'd love to see CI jobs on heat-templates turn into that. | 18:30 |
| *** jistr has quit IRC | 18:32 | |
| *** _ruhe is now known as ruhe | 18:32 | |
| sdake | aweiteka no | 18:34 |
| sdake | aweiteka but would be nice ;) | 18:35 |
| aweiteka | sdake, i'm looking at support in jenkins job builder | 18:35 |
| sdake | aweiteka we are currently trying to get tempest to do the same sorts of things, but not for heat-templates | 18:36 |
| sdake | aweiteka atm, our functional testing is pretty weak in heat - very basic | 18:37 |
| sdake | we need full functional coverage - have testcase implementation day on 27th IIRC | 18:37 |
| *** yogesh has joined #heat | 18:40 | |
| *** jprovazn has joined #heat | 18:44 | |
| sdake | aweiteka - I think in general a heat-templates verification is a bit horse before the cart, since we don't have heat itself being functionallity verified | 18:46 |
| *** gokrokve_ has quit IRC | 18:46 | |
| *** tsmadds has joined #heat | 18:46 | |
| sdake | but if it gets done, that would be great ;) | 18:47 |
| *** pafuent has left #heat | 18:47 | |
| funzo | sdake: hey sorry re the meeting, was on a call with a customer and didn't even look away | 18:47 |
| sdake | funzo no sweat | 18:47 |
| funzo | was just talking with jpeeler about autoscalinggroup support with native resources | 18:48 |
| aweiteka | sdake, understood. we'll see :) | 18:49 |
| *** tsmadds has quit IRC | 18:51 | |
| *** yogesh has quit IRC | 18:51 | |
| *** yogesh has joined #heat | 18:52 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 18:53 | |
| *** yogesh has quit IRC | 18:55 | |
| *** che-arne has quit IRC | 18:56 | |
| *** yogesh has joined #heat | 18:56 | |
| radix | jasond: so I notice that personality_files doesn't match the name of either the REST API or the python client API | 19:01 |
| *** yogesh has quit IRC | 19:02 | |
| jasond | radix: do you think it should just be files? | 19:02 |
| radix | well | 19:02 |
| radix | Server makes it a point to match the REST API in particular | 19:03 |
| radix | using "diskConfig" because that's what the REST API calls it, even though the python API calls it disk_conig | 19:03 |
| jasond | good point, i can update it | 19:03 |
| radix | er, disk_config | 19:03 |
| radix | so that would be "personality" to match the API | 19:03 |
| radix | jasond: ok, I'll throw up a review to formalize it :) | 19:04 |
| jasond | do you think it should match the API or client? the client uses "files" | 19:04 |
| radix | the API, I think, since that's what Server already does | 19:05 |
| radix | I would prefer it to match all one or the other | 19:05 |
| jasond | ok, thanks for the feedback | 19:05 |
| *** tspatzier has quit IRC | 19:08 | |
| *** yogesh has joined #heat | 19:11 | |
| *** ruhe is now known as _ruhe | 19:15 | |
| *** tango has joined #heat | 19:18 | |
| *** gokrokve has joined #heat | 19:31 | |
| *** coolsvap has quit IRC | 19:33 | |
| *** jprovazn has quit IRC | 19:34 | |
| *** gokrokve has quit IRC | 19:36 | |
| *** tsmadds has joined #heat | 19:37 | |
| *** tsmadds has quit IRC | 19:43 | |
| *** gokrokve has joined #heat | 19:47 | |
| *** yogesh has quit IRC | 19:59 | |
| *** yogesh has joined #heat | 20:03 | |
| *** openstackgerrit has quit IRC | 20:06 | |
| *** openstackgerrit has joined #heat | 20:06 | |
| *** _ruhe is now known as ruhe | 20:06 | |
| *** jcru_ has joined #heat | 20:06 | |
| *** jcru has quit IRC | 20:08 | |
| *** nati_uen_ has joined #heat | 20:08 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make Server compatible with Nova Key Pair resource https://review.openstack.org/68230 | 20:09 |
| *** nati_ueno has quit IRC | 20:09 | |
| *** radez is now known as radez_g0n3 | 20:14 | |
| *** gokrokve has quit IRC | 20:19 | |
| *** jmckind has joined #heat | 20:25 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Add personality files property to Server resource https://review.openstack.org/68685 | 20:26 |
| *** tspatzier has joined #heat | 20:32 | |
| *** gokrokve has joined #heat | 20:35 | |
| *** nati_uen_ has quit IRC | 20:37 | |
| *** nati_ueno has joined #heat | 20:38 | |
| *** gokrokve has quit IRC | 20:39 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 20:44 | |
| *** ruhe is now known as _ruhe | 20:46 | |
| *** asalkeld_ has joined #heat | 20:51 | |
| *** asalkeld has quit IRC | 20:55 | |
| *** _ruhe is now known as ruhe | 20:56 | |
| *** gokrokve has joined #heat | 21:00 | |
| *** sabeen1 has joined #heat | 21:00 | |
| *** jcru_ has quit IRC | 21:01 | |
| *** jcru has joined #heat | 21:01 | |
| *** tspatzier has quit IRC | 21:02 | |
| *** sabeen has quit IRC | 21:04 | |
| openstackgerrit | A change was merged to openstack/heat: Move param format for template-validate to API https://review.openstack.org/67171 | 21:04 |
| *** sabeen1 has quit IRC | 21:04 | |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack/python-heatclient: Add HTTP proxy support to heat client https://review.openstack.org/67153 | 21:05 |
| *** e0ne has joined #heat | 21:07 | |
| *** sabeen has joined #heat | 21:11 | |
| *** radez_g0n3 is now known as radez | 21:19 | |
| *** slong has joined #heat | 21:21 | |
| *** gokrokve has quit IRC | 21:28 | |
| *** slong has quit IRC | 21:31 | |
| *** gokrokve_ has joined #heat | 21:32 | |
| openstackgerrit | A change was merged to openstack/heat: stub cfg.CONF in heat_keystoneclient https://review.openstack.org/69339 | 21:33 |
| *** slong has joined #heat | 21:33 | |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack/python-heatclient: Add HTTP proxy support to heat client https://review.openstack.org/67153 | 21:40 |
| *** topol has joined #heat | 21:40 | |
| *** sabeen has quit IRC | 21:46 | |
| *** sabeen has joined #heat | 21:47 | |
| *** alexpilotti has quit IRC | 21:47 | |
| *** alexpilotti_ has joined #heat | 21:48 | |
| *** ruhe is now known as _ruhe | 21:48 | |
| *** cadenzajon has joined #heat | 21:56 | |
| *** randallburt has quit IRC | 22:00 | |
| *** dims has quit IRC | 22:00 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make Server compatible with Nova Key Pair resource https://review.openstack.org/68230 | 22:01 |
| *** dims has joined #heat | 22:02 | |
| *** gokrokve_ has quit IRC | 22:03 | |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Delete rackspace database resource https://review.openstack.org/68144 | 22:03 |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Provide region_name to trove-client https://review.openstack.org/69413 | 22:03 |
| *** sgordon has quit IRC | 22:06 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Native Nova Server compatibility for Cloud Server https://review.openstack.org/58043 | 22:16 |
| *** e0ne has quit IRC | 22:18 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make Server compatible with Nova Key Pair resource https://review.openstack.org/68230 | 22:21 |
| *** aweiteka has quit IRC | 22:23 | |
| *** achampion has quit IRC | 22:27 | |
| *** alexheneveld has joined #heat | 22:29 | |
| *** rpothier has quit IRC | 22:29 | |
| *** giulivo has quit IRC | 22:30 | |
| *** _ruhe is now known as ruhe | 22:30 | |
| *** alexheneveld has quit IRC | 22:33 | |
| *** ruhe is now known as _ruhe | 22:38 | |
| *** jdob has quit IRC | 22:42 | |
| *** vijendar has quit IRC | 22:47 | |
| *** topol has quit IRC | 22:52 | |
| *** andersonvom1 has joined #heat | 22:54 | |
| *** andersonvom has quit IRC | 22:55 | |
| *** e0ne has joined #heat | 23:05 | |
| *** e0ne has quit IRC | 23:10 | |
| *** jergerber has quit IRC | 23:16 | |
| *** jmckind has quit IRC | 23:18 | |
| *** yogesh has quit IRC | 23:21 | |
| *** _ruhe is now known as ruhe | 23:22 | |
| *** FL1SK has joined #heat | 23:23 | |
| FL1SK | hi | 23:23 |
| FL1SK | are there any SUSE templates | 23:24 |
| *** IgorYozhikov has quit IRC | 23:24 | |
| *** IgorYozhikov has joined #heat | 23:24 | |
| *** jpeeler has quit IRC | 23:29 | |
| *** jpeeler has joined #heat | 23:30 | |
| *** jpeeler has quit IRC | 23:30 | |
| *** jpeeler has joined #heat | 23:30 | |
| *** dims has quit IRC | 23:33 | |
| *** achampion has joined #heat | 23:38 | |
| *** david-lyle has quit IRC | 23:40 | |
| *** blomquisg has quit IRC | 23:41 | |
| *** tims has quit IRC | 23:43 | |
| *** ruhe is now known as _ruhe | 23:44 | |
| *** dmakogon_ has joined #heat | 23:46 | |
| *** sabeen has quit IRC | 23:46 | |
| *** rbrady1 has joined #heat | 23:48 | |
| *** DandyPandy_ has joined #heat | 23:48 | |
| *** dmueller has joined #heat | 23:48 | |
| *** denis_makogon has quit IRC | 23:48 | |
| *** vt102 has quit IRC | 23:48 | |
| *** DandyPandy has quit IRC | 23:48 | |
| *** clarkb has quit IRC | 23:48 | |
| *** rbrady has quit IRC | 23:48 | |
| *** dims has joined #heat | 23:48 | |
| *** vt102 has joined #heat | 23:49 | |
| *** rcleere has quit IRC | 23:49 | |
| *** clarkb has joined #heat | 23:49 | |
| *** radez is now known as radez_g0n3 | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!