*** gokrokve_ has quit IRC | 00:07 | |
*** gokrokve has joined #heat | 00:08 | |
*** arbylee has joined #heat | 00:10 | |
*** arbylee1 has joined #heat | 00:11 | |
*** arbylee has quit IRC | 00:11 | |
*** dims has quit IRC | 00:11 | |
*** gokrokve has quit IRC | 00:12 | |
*** yogesh has joined #heat | 00:19 | |
*** rcleere has quit IRC | 00:22 | |
*** matsuhashi has joined #heat | 00:22 | |
*** cadenzajon_ has quit IRC | 00:23 | |
*** dims has joined #heat | 00:27 | |
*** arbylee1 has quit IRC | 00:29 | |
*** yogesh has quit IRC | 00:30 | |
*** yogesh has joined #heat | 00:31 | |
*** nati_ueno has joined #heat | 00:32 | |
*** matsuhashi has quit IRC | 00:32 | |
*** matsuhas_ has joined #heat | 00:34 | |
*** nati_uen_ has quit IRC | 00:35 | |
openstackgerrit | Cyril Roelandt proposed a change to openstack/python-heatclient: Python3: fix bytes/str issues https://review.openstack.org/73470 | 00:44 |
---|---|---|
*** alexheneveld has quit IRC | 00:47 | |
*** topol has quit IRC | 00:55 | |
sdake | zaneb thanks for the mock help, my understanding has increased significantly :) | 00:56 |
zaneb | :) | 00:56 |
sdake | down to 67 unit test failures which are all unique :) | 00:56 |
zaneb | rofl | 00:57 |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Use olso.messaging https://review.openstack.org/72798 | 00:57 |
sdake | calling it a day | 00:57 |
zaneb | welcome to my last week | 00:57 |
sdake | I've been jerking around with unit tests for atleast 5 days | 00:57 |
*** david-lyle has quit IRC | 00:57 | |
sdake | after I finish the rpc api, I fear I will have to rework the rpc server part of things | 00:58 |
* sdake groans | 00:58 | |
* zaneb has fingers crossed for patch set 5 of https://review.openstack.org/#/c/73822/ | 00:58 | |
sdake | zaneb https://review.openstack.org/#/c/71459/ could use some love before it expires ;) | 00:59 |
zaneb | why not | 01:00 |
zaneb | approved | 01:00 |
sdake | tx | 01:01 |
sdake | I saw a cat talk about CI | 01:01 |
sdake | he said if you aren't committing to the repo once a day your doing it wrong | 01:02 |
sdake | clearly he has never seen a project with 80 developers ;) | 01:02 |
*** yogesh has quit IRC | 01:04 | |
*** russellb has quit IRC | 01:05 | |
*** pvaneck has quit IRC | 01:09 | |
*** pablosan has quit IRC | 01:15 | |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Use olso.messaging https://review.openstack.org/72798 | 01:18 |
*** pablosan has joined #heat | 01:18 | |
*** jergerber has joined #heat | 01:20 | |
*** nati_uen_ has joined #heat | 01:21 | |
*** nati_ueno has quit IRC | 01:23 | |
*** mestery has quit IRC | 01:24 | |
*** derekh has quit IRC | 01:24 | |
*** harlowja_away has quit IRC | 01:25 | |
*** mestery has joined #heat | 01:30 | |
*** spzala has joined #heat | 01:32 | |
*** nosnos has joined #heat | 01:36 | |
openstackgerrit | A change was merged to openstack/heat-templates: Add a F20 WordPress_Native template https://review.openstack.org/71459 | 01:40 |
*** FL1SK has quit IRC | 01:42 | |
*** FL1SK has joined #heat | 01:42 | |
*** arbylee has joined #heat | 01:47 | |
*** kfox1111_ has quit IRC | 01:49 | |
*** ramishra has joined #heat | 01:53 | |
ramishra | stevebaker: Hi Morning | 01:58 |
stevebaker | ramishra: hi | 01:58 |
ramishra | stevebaker: I have changed the patch.. https://review.openstack.org/#/c/72454/, had to override both handle_create/handle_update. | 02:00 |
ramishra | stevebaker: my logic of looping through the keys, is to make sure if the key is present and then update it.. as some keys like segmentation_id may not be present | 02:02 |
ramishra | stevebaker: do you have few mins to check if it looks ok now? | 02:02 |
stevebaker | ramishra: looking now | 02:03 |
ramishra | stevebaker: also wondering why the jenkins check job is not verifying it.. I am new to the gate process.. | 02:04 |
stevebaker | ramishra: I think it is clogged up due to everyone getting their changes in before the proposal freeze | 02:05 |
*** arbylee has quit IRC | 02:08 | |
*** lazy_prince is now known as killer_prince | 02:08 | |
ramishra | ramishra: ahh.. Also can the 3 LBaaS bugs/changes assigned to me? I have started working on the changes.. Can you check if they can be approved too. | 02:09 |
*** nati_ueno has joined #heat | 02:09 | |
*** pvaneck has joined #heat | 02:11 | |
*** nati_uen_ has quit IRC | 02:12 | |
*** fandi has joined #heat | 02:13 | |
*** spzala has quit IRC | 02:16 | |
*** liang has joined #heat | 02:18 | |
openstackgerrit | A change was merged to openstack/python-heatclient: Adding stack-adopt support to python-heatclient https://review.openstack.org/70380 | 02:31 |
openstackgerrit | A change was merged to openstack/heat: Add StackResource specific preview behavior https://review.openstack.org/68407 | 02:31 |
openstackgerrit | A change was merged to openstack/heat: Add preview for NestedStack https://review.openstack.org/68408 | 02:31 |
openstackgerrit | A change was merged to openstack/heat: Add preview for ResourceGroup https://review.openstack.org/68409 | 02:32 |
openstackgerrit | A change was merged to openstack/heat: Add preview for InstanceGroup https://review.openstack.org/68410 | 02:32 |
openstackgerrit | A change was merged to openstack/heat: Add preview for TemplateResource https://review.openstack.org/68411 | 02:32 |
*** erkules_ has joined #heat | 02:32 | |
*** arbylee has joined #heat | 02:34 | |
*** erkules has quit IRC | 02:35 | |
*** gokrokve has joined #heat | 02:35 | |
*** asalkeld has quit IRC | 02:36 | |
*** arbylee has quit IRC | 02:40 | |
*** jergerber has quit IRC | 02:42 | |
*** asalkeld has joined #heat | 02:48 | |
openstackgerrit | Rabi Mishra proposed a change to openstack/heat: Add ability to create provider networks (vlan,flat). https://review.openstack.org/72454 | 02:49 |
ramishra | stevebaker: as suggested added a static method.. please have a look and review it once more:). thanks | 02:51 |
*** killer_prince is now known as lazy_prince | 02:52 | |
stevebaker | ramishra: I actually meant a method which prepends provider: to the key of a single property | 02:53 |
ramishra | stevebaker: yeah, thats what I have done, static method to prepend provider: | 02:54 |
stevebaker | ramishra: to be called 3 times, one for each property | 02:54 |
*** lindsayk has quit IRC | 02:55 | |
*** nati_uen_ has joined #heat | 02:57 | |
ramishra | stevabker: static method for string update, to prepend provider: to a key? | 03:00 |
*** nati_ueno has quit IRC | 03:00 | |
*** lazy_prince has quit IRC | 03:05 | |
*** gokrokve has quit IRC | 03:08 | |
*** gokrokve has joined #heat | 03:09 | |
*** killer_prince has joined #heat | 03:12 | |
*** gokrokve has quit IRC | 03:13 | |
*** gokrokve has joined #heat | 03:15 | |
ramishra | stevebaker: sorry, I think I got you now.. updated the patch... please review | 03:15 |
openstackgerrit | Rabi Mishra proposed a change to openstack/heat: Add ability to create provider networks (vlan,flat). https://review.openstack.org/72454 | 03:16 |
*** andersonvom has joined #heat | 03:23 | |
openstackgerrit | A change was merged to openstack/python-heatclient: get_file_contents: use six.itervalues() instead of dict.itervalues() https://review.openstack.org/72812 | 03:26 |
*** matsuhas_ has quit IRC | 03:29 | |
openstackgerrit | A change was merged to openstack/python-heatclient: Improve help strings https://review.openstack.org/72439 | 03:30 |
openstackgerrit | A change was merged to openstack/heat: Revert "Merge "Re-enable lazy translation"" https://review.openstack.org/74454 | 03:30 |
*** asalkeld has quit IRC | 03:37 | |
*** nati_ueno has joined #heat | 03:45 | |
*** nati_ueno has quit IRC | 03:46 | |
*** ramishra has quit IRC | 03:47 | |
*** nati_uen_ has quit IRC | 03:48 | |
*** asalkeld has joined #heat | 03:50 | |
*** ramishra has joined #heat | 03:51 | |
*** rcleere has joined #heat | 03:53 | |
*** ramishra has quit IRC | 04:03 | |
*** ramishra has joined #heat | 04:03 | |
*** pablosan has quit IRC | 04:09 | |
*** david-lyle has joined #heat | 04:13 | |
*** ramishra has joined #heat | 04:19 | |
*** Tross has quit IRC | 04:22 | |
*** Tross has joined #heat | 04:22 | |
*** Tross has left #heat | 04:23 | |
*** coolsvap has joined #heat | 04:25 | |
*** andersonvom has quit IRC | 04:25 | |
*** gokrokve has quit IRC | 04:27 | |
*** gokrokve has joined #heat | 04:28 | |
openstackgerrit | Steve Baker proposed a change to openstack/heat: REST deployment metadata method https://review.openstack.org/74203 | 04:29 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: RPC method to fetch deployments metadata https://review.openstack.org/74202 | 04:29 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: SignalResponder move signed URL deleting to its own method https://review.openstack.org/74205 | 04:32 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Resource type implementation for structured software config https://review.openstack.org/74206 | 04:32 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: OS::Nova::Server support for software config https://review.openstack.org/67625 | 04:32 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Resource type implementation for software deployment https://review.openstack.org/67624 | 04:32 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Nova server to ref cloud-config resources in user_data https://review.openstack.org/69238 | 04:32 |
*** gokrokve has quit IRC | 04:32 | |
*** matsuhashi has joined #heat | 04:35 | |
*** nkhare has joined #heat | 04:44 | |
*** nkhare has quit IRC | 04:44 | |
*** ramishra_ has joined #heat | 04:47 | |
*** nkhare_ has joined #heat | 04:50 | |
*** ramishra has quit IRC | 04:50 | |
*** akuznetsov has joined #heat | 04:56 | |
*** nkhare_ has quit IRC | 04:59 | |
*** nkhare has joined #heat | 05:00 | |
*** nkhare_ has joined #heat | 05:03 | |
*** nkhare_ has quit IRC | 05:05 | |
*** rcleere has quit IRC | 05:05 | |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Move HOT to a separate subpackage https://review.openstack.org/74584 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Provide access to the template version https://review.openstack.org/74585 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Disallow Heat-only functions in CloudFormation templates https://review.openstack.org/74586 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Move built-in functions to separate modules https://review.openstack.org/74587 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Remove Fn::FindInMap from HOT https://review.openstack.org/74588 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Give cfn a separate Template class https://review.openstack.org/74589 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Add a plugin_manager module https://review.openstack.org/74590 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Use PluginManager to load resources https://review.openstack.org/74591 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Load functions from plugins https://review.openstack.org/74592 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Replace function resolution with lazy evaluation https://review.openstack.org/73822 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: unit tests: Fix types in Fn::ResourceFacade tests https://review.openstack.org/73820 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: unit tests: Refactor to accomodate lazy evaluation https://review.openstack.org/73821 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: unit tests: Always resolve static functions https://review.openstack.org/73818 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Fix test_association_eip unit test https://review.openstack.org/73819 | 05:10 |
openstackgerrit | Zane Bitter proposed a change to openstack/heat: Evaluate lazy functions in autoscaling launch config https://review.openstack.org/73817 | 05:10 |
*** nkhare has quit IRC | 05:13 | |
*** nkhare has joined #heat | 05:13 | |
*** KiwiMartin has quit IRC | 05:13 | |
*** nkhare has quit IRC | 05:17 | |
*** zaneb has quit IRC | 05:18 | |
*** nkhare has joined #heat | 05:21 | |
*** nkhare has quit IRC | 05:26 | |
*** chandan_kumar has joined #heat | 05:26 | |
*** nkhare has joined #heat | 05:26 | |
*** nati_ueno has joined #heat | 05:27 | |
*** nati_ueno has quit IRC | 05:28 | |
*** nati_ueno has joined #heat | 05:29 | |
*** pvaneck has quit IRC | 05:31 | |
*** nati_uen_ has joined #heat | 05:37 | |
*** nati_ueno has quit IRC | 05:41 | |
openstackgerrit | A change was merged to openstack/heat: raw_template hot parameter type data migration https://review.openstack.org/70050 | 05:44 |
*** coolsvap1 has joined #heat | 05:59 | |
*** liang has quit IRC | 06:00 | |
*** gokrokve has joined #heat | 06:01 | |
*** e0ne has joined #heat | 06:02 | |
*** coolsvap has quit IRC | 06:03 | |
*** gokrokve has quit IRC | 06:06 | |
*** liang has joined #heat | 06:13 | |
*** cadenzajon has joined #heat | 06:20 | |
*** gokrokve has joined #heat | 06:24 | |
*** achampion has joined #heat | 06:26 | |
*** cadenzajon has quit IRC | 06:26 | |
*** e0ne has quit IRC | 06:28 | |
openstackgerrit | A change was merged to openstack/python-heatclient: Do not use the '+' operation dict_items() https://review.openstack.org/72988 | 06:30 |
openstackgerrit | A change was merged to openstack/python-heatclient: Only call decode() on bytes https://review.openstack.org/74123 | 06:30 |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Add an iteritems for RequestContext https://review.openstack.org/74606 | 06:32 |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Use olso.messaging https://review.openstack.org/72798 | 06:32 |
sdake | definately bedtime | 06:33 |
sdake | night | 06:33 |
*** matsuhashi has quit IRC | 06:35 | |
*** matsuhashi has joined #heat | 06:37 | |
*** gokrokve_ has joined #heat | 06:45 | |
*** gokrokve has quit IRC | 06:48 | |
*** gokrokve_ has quit IRC | 06:49 | |
*** e0ne has joined #heat | 06:59 | |
*** e0ne has quit IRC | 07:00 | |
*** e0ne has joined #heat | 07:01 | |
*** e0ne has quit IRC | 07:06 | |
*** sergmelikyan has quit IRC | 07:08 | |
*** zaneb has joined #heat | 07:10 | |
*** ramishra_ has quit IRC | 07:12 | |
*** ramishra has joined #heat | 07:13 | |
*** achampion has quit IRC | 07:17 | |
*** chandan_kumar has quit IRC | 07:18 | |
*** liang has quit IRC | 07:23 | |
*** tspatzier has joined #heat | 07:23 | |
*** e0ne has joined #heat | 07:24 | |
*** TonyBurn_ has joined #heat | 07:24 | |
*** chandan_kumar has joined #heat | 07:25 | |
*** ifarkas has joined #heat | 07:27 | |
*** e0ne has quit IRC | 07:28 | |
*** saju_m has joined #heat | 07:30 | |
*** erkules_ is now known as erkules | 07:34 | |
*** liang has joined #heat | 07:36 | |
openstackgerrit | Lee Li proposed a change to openstack/python-heatclient: Using common methods from oslo cliutils https://review.openstack.org/67120 | 07:40 |
skraynev_afk | morning | 07:41 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Ensure property 'name' can be updated https://review.openstack.org/74624 | 07:45 |
*** nati_uen_ has quit IRC | 07:49 | |
*** nati_ueno has joined #heat | 07:49 | |
*** e0ne has joined #heat | 07:56 | |
*** pshchelo has quit IRC | 07:57 | |
openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Validation for Outputs template section https://review.openstack.org/70358 | 07:58 |
openstackgerrit | A change was merged to openstack/heat: Add validation to KeyPair resource https://review.openstack.org/72443 | 07:59 |
*** e0ne has quit IRC | 08:00 | |
*** skraynev_afk has quit IRC | 08:01 | |
*** pshchelo has joined #heat | 08:03 | |
*** jprovazn has joined #heat | 08:07 | |
*** achampion has joined #heat | 08:12 | |
*** achampion has quit IRC | 08:17 | |
*** saju_m has quit IRC | 08:21 | |
*** ramishra has quit IRC | 08:24 | |
*** jistr has joined #heat | 08:29 | |
*** openstack has joined #heat | 08:42 | |
*** PhilK has joined #heat | 08:42 | |
*** gondoi has joined #heat | 08:43 | |
*** lekha has joined #heat | 08:43 | |
*** nati_uen_ has joined #heat | 08:43 | |
*** chandan_kumar has joined #heat | 08:44 | |
*** nati_ueno has quit IRC | 08:47 | |
pshchelo | morning :) | 08:48 |
cmyster | morning | 08:56 |
*** ramishra has joined #heat | 08:57 | |
*** nosnos_ has joined #heat | 08:59 | |
*** nosnos has quit IRC | 08:59 | |
*** e0ne has joined #heat | 09:04 | |
*** zaneb has quit IRC | 09:05 | |
*** coolsvap1 has quit IRC | 09:06 | |
*** alexheneveld has joined #heat | 09:08 | |
*** derekh has joined #heat | 09:13 | |
*** chandan_kumar has quit IRC | 09:23 | |
*** sgran has quit IRC | 09:26 | |
*** coolsvap has joined #heat | 09:34 | |
*** chandankumar_ has joined #heat | 09:42 | |
*** matsuhashi has quit IRC | 09:46 | |
*** nosnos_ has quit IRC | 09:46 | |
*** nosnos has joined #heat | 09:47 | |
*** matsuhashi has joined #heat | 09:47 | |
jprovazn | Hi, is there a way how to pass hostname on one instance to other instance in stack? IOW I have a stack with instance1 and instance2 and I'd like to know hostname of instance2 in instance1. I tried to pass something like this to instance1: "instance2: {Fn:GetAtt: [instance2, PrivateDnsName]}". But this didn't work | 09:47 |
*** jamieh has joined #heat | 09:57 | |
*** saju_m has joined #heat | 09:58 | |
*** tomek_adamczewsk has joined #heat | 09:59 | |
pshchelo | jprovazn: you could try using dependson attribute, that way the second instance will be created only after the first is finished creating, so the attribute will be available already | 09:59 |
pshchelo | well, not attribute, but "construct" | 10:00 |
therve | You have an implicit dependency when linking a reference | 10:00 |
therve | jprovazn, What doesn't workN | 10:00 |
therve | ? | 10:01 |
jprovazn | I was hitting "resource not avaialable" (or similar - can retry) | 10:01 |
jprovazn | therve: it will take a while, will ping you | 10:03 |
*** nati_uen_ has quit IRC | 10:07 | |
*** liang has quit IRC | 10:08 | |
*** jamieh has quit IRC | 10:17 | |
*** jamieh has joined #heat | 10:20 | |
ramishra | skraynev: Hi | 10:31 |
*** chandankumar_ has quit IRC | 10:32 | |
ramishra | skraynev: regarding https://review.openstack.org/#/c/72454/, stevebaker suggested to change the properties in the handle_create and handle_update for readbility. | 10:34 |
*** alexpilotti has joined #heat | 10:34 | |
*** TonyBurn_ has quit IRC | 10:36 | |
*** TonyBurn__ has joined #heat | 10:36 | |
*** aignatov_ is now known as aignatov | 10:40 | |
*** aignatov is now known as aignatov_ | 10:41 | |
*** TonyBurn__ has quit IRC | 10:45 | |
*** TonyBurn_ has joined #heat | 10:45 | |
*** chandankumar_ has joined #heat | 10:48 | |
*** nati_ueno has joined #heat | 10:49 | |
*** mkollaro has joined #heat | 10:49 | |
*** nati_ueno has quit IRC | 10:53 | |
*** che-arne has joined #heat | 10:55 | |
*** dmakogon_ is now known as denis_makogon | 10:57 | |
skraynev | hello, ramishra | 11:06 |
skraynev | I agree with him | 11:06 |
skraynev | I suggest move part of code in other function | 11:06 |
skraynev | other is ok | 11:06 |
ramishra | skraynev: Yes, that what is he suggested.. not to move them outside these functions | 11:07 |
skraynev | ramishra, sorry, that so late, I have a dinner | 11:07 |
ramishra | skyanev: you see the other patch...earlier one.. he suggested not to move them outside... | 11:08 |
skraynev | ramishra: Hm. let me look.. | 11:08 |
ramishra | check patch#17... I had earlier done that... He said not to.. so I changed it immediately | 11:09 |
ramishra | skraynev: then he reviewed it | 11:09 |
ramishra | skraynev: it's been hell of a job to make this small change... back and forth with minor changes.. | 11:12 |
skraynev | ramishra: I understand. don't worry | 11:14 |
*** achampion has joined #heat | 11:14 | |
*** aignatov_ is now known as aignatov | 11:14 | |
skraynev | you are right in IRC log he told about it... | 11:14 |
*** chandankumar_ has quit IRC | 11:14 | |
ramishra | yes | 11:15 |
*** chandan_kumar has joined #heat | 11:15 | |
skraynev | So I will discuss it with him | 11:15 |
skraynev | but as you can see my idea a little different | 11:15 |
skraynev | I don't offer remove current static method | 11:16 |
skraynev | it's ok | 11:16 |
skraynev | but will be good add one more where will be called your method with with prefix | 11:17 |
skraynev | Anyway Could you fix other comments and don't change it. | 11:18 |
ramishra | done and uploaded | 11:18 |
ramishra | already.. patch 19 | 11:18 |
ramishra | skraynev: r u talking about some other comments.. | 11:19 |
skraynev | great!) | 11:19 |
skraynev | no.. | 11:19 |
skraynev | I have not seen it yet | 11:19 |
skraynev | but now I start | 11:19 |
skraynev | only one question: why did you do PROVIDER_EXT - global ? | 11:20 |
skraynev | I thought that using it direct as ['provider' + key] - was normal | 11:20 |
ramishra | for easy change in the future... if they change the extension....as done in many other files.. | 11:21 |
skraynev | Could you give other example with such extension? | 11:22 |
skraynev | s/such extension/ such approach for neutron extension | 11:23 |
ramishra | it's just a const... heat.openstack.common.generator.py | 11:23 |
ramishra | I think its better to keep them at the top for changes in the future.. do not have to search for them... If the extension changes... | 11:24 |
ramishra | heat.openstack.common.config.generator.py | 11:24 |
skraynev | ramishra: IMO, it make sense if you have some bp or other changes which will be use it . | 11:25 |
ramishra | It's there in many neutron files | 11:26 |
skraynev | the reason : it will be used in future ... hm.. I guess, it's good for core team. | 11:26 |
*** jamieh has quit IRC | 11:27 | |
skraynev | ok, I will not add -1, but leave comments with question. | 11:28 |
*** matsuhashi has quit IRC | 11:28 | |
skraynev | and we will know what think about it other reviewers. I suppose it's better solution ;) | 11:29 |
ramishra | This may be required elsewhere... as this is part of the extension.. not sure if we see more type of provider extension in the future.. | 11:30 |
*** fandi has quit IRC | 11:30 | |
ramishra | also check neutron.plugins.ml2.plugin.py | 11:31 |
*** matsuhashi has joined #heat | 11:31 | |
ramishra | anyway.. let me change it and upload... don't want another round of back and forth:) | 11:33 |
skraynev | ))) | 11:33 |
skraynev | wow | 11:33 |
skraynev | wait | 11:33 |
*** jamieh has joined #heat | 11:33 | |
skraynev | let's ask other guys | 11:34 |
skraynev | shardy: are you here? | 11:34 |
jprovazn | therve: here is the template I use: https://gist.github.com/jprovaznik/9090247#file-gistfile1-txt-L733 and here is subset of errors from log: http://paste.openstack.org/show/67238/ (list of errors is not complete). If I try to replace PrivateDnsName with instance_name everywhere in the template, it starts as expected. So I guess I'm using PrivateDnsName in a wrong way. | 11:35 |
skraynev | ramishra: I guess will be easier to ask stevebaker and shardy about it and then upload a final (I hope) version ;) | 11:35 |
ramishra | skraynev: uploaded...if its required it would be changed later;) | 11:37 |
skraynev | ramishra: Oh. ok!) | 11:37 |
ramishra | skraynev: do you mind if I own the LBaaS resource fixes for VIP,Pool etc | 11:41 |
skraynev | ramishra: I have added comment about my idea. Other LGTM! thank you. | 11:41 |
*** ifarkas has quit IRC | 11:42 | |
skraynev | ramishra: are you about https://bugs.launchpad.net/heat/+bug/1258490 ? | 11:42 |
uvirtbot | Launchpad bug 1258490 in heat "Moving vip in separate resource or making it more clear." [Medium,New] | 11:42 |
ramishra | skraynev: I have three bugs for that file.. | 11:42 |
skraynev | Ou. Could you share links, please | 11:43 |
ramishra | skraynev: stevebaker suggested to create different bugs | 11:43 |
ramishra | https://bugs.launchpad.net/heat/+bug/1280161 | 11:44 |
uvirtbot | Launchpad bug 1280161 in heat "Neutron LBaaS-Modify Pool Resource" [Undecided,New] | 11:44 |
ramishra | https://bugs.launchpad.net/heat/+bug/1280167 | 11:45 |
uvirtbot | Launchpad bug 1280167 in heat "Neutron LBaaS - VIP Resource Modify create/delete/update methods" [Undecided,New] | 11:45 |
*** che-arne has quit IRC | 11:45 | |
ramishra | skraynev: stevebaker suggested to create multiple bugs.. I had created a blueprint to rewrite this file and its resources, but he suggested not to do it and create multiple bugs | 11:46 |
skraynev | I guess it related with backward compatibility | 11:48 |
ramishra | skyanev: the reason there is a chnage freeze for icehouse in a few days and we could not make it with the change.. | 11:48 |
ramishra | skryanev: bugs would be better to manage | 11:48 |
*** nkhare has quit IRC | 11:49 | |
skraynev | The main reason why a don't do any changes in this way was - a discussion with therve. | 11:50 |
ramishra | skraynev: I have rewritten the resources to some extent... I want to VIP one also assigned to me so that I can make all the changes together... | 11:50 |
therve | jprovazn, I think you have circular dependencies | 11:52 |
therve | jprovazn, Like notcomputeSlave0Config references itself in the metadata | 11:53 |
jprovazn | therve: aah, thanks, will try w/o selfref | 11:53 |
ramishra | therve: Hi | 11:55 |
therve | ramishra, FWIW, I have no idea what 1280167means | 11:55 |
therve | Hello :) | 11:55 |
skraynev | therve: As i remember we discussed it | 11:56 |
skraynev | And the result was: do touch it while | 11:56 |
skraynev | because currently neutron don't change architecture | 11:57 |
ramishra | therve: 1280167means: there was no VIP resource or LB resource in neutron API | 11:57 |
ramishra | therve: I mean earlier LBaaS implementation | 11:57 |
*** coolsvap has quit IRC | 11:57 | |
skraynev | but neutron team have some plans for modifying relationship between VIP and LB | 11:58 |
ramishra | therve: Its already done | 11:58 |
therve | ramishra, And there still is no such thing AFAICT | 11:58 |
ramishra | therve: in the V2 API | 11:58 |
ramishra | therve: http://docs.openstack.org/api/openstack-network/2.0/content/lbaas_ext_concepts.html | 11:59 |
therve | ramishra, That's havana code | 11:59 |
therve | Things like https://blueprints.launchpad.net/neutron/+spec/lbaas-multiple-vips-per-pool haven't landed AFAICS | 11:59 |
skraynev | therve: yeap | 12:00 |
skraynev | and I still wait these changes for implement it | 12:00 |
ramishra | therve: sure.. but our current resources are not inline with Havana code... | 12:00 |
therve | ramishra, And? Which feature isn't accessible to the user? | 12:01 |
skraynev | therve: As i remember it was dispute : what is major vip or loadbalancer | 12:02 |
therve | skraynev, Sorry please rephrase | 12:02 |
skraynev | therve: I mean, that I offer same changes in the past | 12:03 |
skraynev | therve: And we discussed it with shardy | 12:03 |
therve | Right :) | 12:03 |
therve | We recorded the discussion on https://bugs.launchpad.net/heat/+bug/1258490 | 12:03 |
uvirtbot | Launchpad bug 1258490 in heat "Moving vip in separate resource or making it more clear." [Medium,New] | 12:04 |
therve | So that we don't need to have it again :) | 12:04 |
skraynev | therve: he. You are right, I forgot about your comment for this bug. | 12:05 |
ramishra | therve: can we cerate multiple VIPs, In the design, one LB can have multiple VIPs..correct me If I am wrong here | 12:05 |
therve | ramishra, You're wrong | 12:05 |
skraynev | But I remembered , that changes in neutron should be merged first | 12:05 |
ramishra | therve: the rest url for VIP creation: /lb/vips | 12:06 |
therve | ramishra, We're waiting for that feature to be merged before doing the work | 12:06 |
therve | ramishra, See https://blueprints.launchpad.net/neutron/+spec/lbaas-multiple-vips-per-pool | 12:06 |
skraynev | ramishra: I discussed it with neutron team. and they didn't finish it still. So I wait their sign for implement it in heat and during this process moving vip in separate resource. | 12:07 |
skraynev | and I will do it ;) | 12:07 |
skraynev | therve: how about creating bp, for it in heat. I think it will useful, because bug isn't enough for describing implementation plan? | 12:09 |
skraynev | therve: I should target it on juno, i suppose. | 12:10 |
ramishra | therve: I think that's a single server pool mapping to multiple vips.. But in the current implementation has issues like. Current OS::Neutron::Pool has a VIP property a MAP that contains VIP Schema. However, in the new neutron Pool resource, the only reference to VIP is 'vip_id' | 12:10 |
therve | skraynev, Sure why not | 12:10 |
skraynev | ok, | 12:10 |
therve | ramishra, Heat model doesn't have to map exactly to Neutron model | 12:10 |
* skraynev do it... | 12:10 | |
therve | Many times it doesn't because neutron models are... not user-friendly to say the least | 12:10 |
ramishra | therve: ok..I had the impression that we can make it inline with neutron anyway, leave it to the groups better judgement | 12:14 |
*** ifarkas has joined #heat | 12:14 | |
skraynev | therve: done | 12:17 |
skraynev | Will add more details lately | 12:17 |
therve | ramishra, Ultimately we may want to, but there is no reason if it's just to map to neutron model | 12:17 |
skraynev | https://blueprints.launchpad.net/heat/+spec/multiple-vips-for-lbaas | 12:17 |
*** matsuhashi has quit IRC | 12:20 | |
*** sgran has joined #heat | 12:20 | |
ramishra | therve: IMO, its better if it is inline with neutron api... then it becomes easy for changes for future neutron api spec changes.. however, as I mentioned earlier that's my view | 12:20 |
*** aignatov is now known as aignatov_ | 12:21 | |
therve | ramishra, Except if you make the change too soon and the neutron API changes in a way that you didn't expect | 12:21 |
*** rpothier_ has quit IRC | 12:21 | |
ramishra | therve: I don't think its going to change much.. only multiple vip mapping for the same pool would be added to the existing v2 api as you pointed out... | 12:22 |
therve | ramishra, But you don't know :) | 12:23 |
*** matsuhashi has joined #heat | 12:25 | |
ramishra | therve: hmm..ok | 12:25 |
*** aignatov_ is now known as aignatov | 12:25 | |
therve | ramishra, From what I understood, 1280167 looks like a duplicate of 1258490 | 12:34 |
ramishra | therve: I had created a BP to rewrite all the changes... however, stevebaker asked to make these incremental changes and multiple bugs | 12:37 |
ramishra | therve: it's there in the IRC log | 12:37 |
ramishra | therve: probably because they can't get in the icehouse timeline as a change in a few days | 12:41 |
therve | ramishra, Mokay. The thing is that those bugs seems to say roughly the same thing, which is "Create a VIP resource" | 12:42 |
therve | I don't know what are "all the changes" | 12:42 |
ramishra | therve: one is change to Pool resource (Remove VIP map with the vip_id) | 12:44 |
therve | ramishra, And by remove you mean deprecate it? :) | 12:44 |
ramishra | therve: sorry, we can have both id and map coexist.. | 12:46 |
*** che-arne has joined #heat | 12:49 | |
ramishra | therve: let me work on the changes without breaking it and get back.. Can we have 2 implementations coexist? Looking for suggestions here.. | 12:51 |
therve | ramishra, Well I gave you my suggestions multiple times already, which is to wait for the neutron change | 12:51 |
skraynev | therve: but then I will do it ;) | 12:52 |
skraynev | ramishra: why is it so important for you implement now? | 12:53 |
skraynev | AFAIK, there are several bugs without developers.. | 12:53 |
*** tomek_adamczewsk has quit IRC | 12:54 | |
ramishra | skraynev: I don't think any more lbaas related changes would be in icehouse from neutron with the freeze in few days..so my thought is to be be api compliant in icehouse | 12:55 |
*** tomek_adamczewsk has joined #heat | 12:56 | |
*** nosnos has quit IRC | 12:56 | |
skraynev | ramishra: It's good, idea, but.. if something work and is checked a lot of users, make sense stay it without changes until new release is not opened. | 12:57 |
jprovazn | therve: I've updated the template to not reference itself - https://gist.github.com/jprovaznik/9091400 but I'm still getting the same error - Returning exception The Resource (notcomputeSlave0Config) is not available. to caller | 12:57 |
jprovazn | any idea? | 12:57 |
skraynev | I believe, that you can make it good and really care about relationships with neutron. but if it be bad, we will get unstable and not trusted resource near of release time. | 12:59 |
therve | jprovazn, I think you still have some. Like notcomputeConfig links to notcompute which links to notcomputeConfig again | 13:00 |
*** sgordon has joined #heat | 13:00 | |
ramishra | skraynev: do you have data to suggest that lbaas is used by many users in live deployments? | 13:00 |
*** rpothier_ has joined #heat | 13:00 | |
skraynev | Also I have had meeting with one of core in neutron and he told that they will do some architecture changes in next cycle (not only multi-vips) | 13:01 |
*** david-lyle has quit IRC | 13:01 | |
ramishra | skraynev: next cycle.. may be you are right... Are there any bps that you can point me to? | 13:02 |
jprovazn | therve: well, if I replace PrivateDnsName with instance_name, then the template works -> shouldn't be circular dep issue in such case too? | 13:03 |
therve | jprovazn, I don't know | 13:05 |
skraynev | ramishra: now only bp with multi-vips. what about others? they discuss new architecture design on summit and then I may share created bp. (unfortunately now the have not final conclusion) | 13:05 |
ramishra | ramishra: then IMO it's better to keep in scope what is in there in neutron icehouse scope and make changes. anyway, I leave it to the group to decide... | 13:08 |
*** matsuhashi has quit IRC | 13:08 | |
*** matsuhashi has joined #heat | 13:08 | |
*** matsuhashi has quit IRC | 13:08 | |
*** fandi has joined #heat | 13:14 | |
*** zhiyan_ is now known as zhiyan | 13:14 | |
chmouel | a/sen | 13:14 |
ramishra | skraynev: then IMO it's better to keep in scope what is in there in neutron icehouse scope and make changes. anyway, I leave it to the group to decide... | 13:16 |
skraynev | ramishra: I have read it already ;) | 13:16 |
skraynev | ramishra: I just tell you, that it old question and I asked about it at start of icehouse. | 13:17 |
*** aweiteka has quit IRC | 13:19 | |
*** andersonvom has joined #heat | 13:22 | |
*** jdob has joined #heat | 13:23 | |
*** achampion has quit IRC | 13:26 | |
*** pafuent has joined #heat | 13:29 | |
*** jistr is now known as jistr|english | 13:31 | |
*** ramishra has quit IRC | 13:36 | |
*** mojorison has quit IRC | 13:41 | |
SpamapS | hmmmmmmm | 13:55 |
SpamapS | have we broken Fn::Select or Fn::GetAtt? | 13:55 |
SpamapS | I'm getting a blank address when using this snippet: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/undercloud-source.yaml#n134 ... the address is coming out '' | 13:56 |
SpamapS | so I"m getting http://:8000 | 13:56 |
therve | The snippet doesn't use Select or GetAtt | 13:58 |
therve | I guess you mean ControllerAddress default? | 13:58 |
*** aweiteka has joined #heat | 14:00 | |
*** jamieh has quit IRC | 14:03 | |
*** ramishra has joined #heat | 14:04 | |
*** saju_m has quit IRC | 14:11 | |
*** achampion has joined #heat | 14:20 | |
*** e0ne has quit IRC | 14:23 | |
*** e0ne has joined #heat | 14:31 | |
*** dims has quit IRC | 14:32 | |
*** dims has joined #heat | 14:35 | |
*** jistr|english is now known as jistr | 14:36 | |
*** vijendar has joined #heat | 14:39 | |
*** jdob_ has joined #heat | 14:40 | |
*** jdob has quit IRC | 14:41 | |
*** zaneb has joined #heat | 14:47 | |
*** russellb has joined #heat | 14:48 | |
*** arbylee has joined #heat | 14:54 | |
*** sabeen has joined #heat | 14:57 | |
*** jdob has joined #heat | 14:58 | |
*** jdob_ has quit IRC | 14:58 | |
*** mkollaro has quit IRC | 15:02 | |
*** mkollaro1 has joined #heat | 15:02 | |
*** daneyon has joined #heat | 15:03 | |
*** blomquisg has joined #heat | 15:03 | |
*** e0ne_ has joined #heat | 15:03 | |
radix | good morning | 15:04 |
*** daneyon has quit IRC | 15:07 | |
*** e0ne has quit IRC | 15:07 | |
*** daneyon has joined #heat | 15:08 | |
andersonvom | radix: morning! | 15:09 |
*** openstackgerrit has joined #heat | 15:09 | |
*** tims has joined #heat | 15:10 | |
*** tims1 has joined #heat | 15:11 | |
*** tims has quit IRC | 15:14 | |
*** jergerber has joined #heat | 15:16 | |
*** Michalik has quit IRC | 15:16 | |
openstackgerrit | Richard Lee proposed a change to openstack/heat: Add admin_pass as attribute and property to server https://review.openstack.org/72745 | 15:18 |
*** e0ne_ has quit IRC | 15:19 | |
*** e0ne has joined #heat | 15:20 | |
*** radez_g0n3 is now known as radez | 15:21 | |
*** gokrokve has joined #heat | 15:22 | |
*** zhiyan is now known as zhiyan_ | 15:29 | |
*** topol has joined #heat | 15:29 | |
*** mkollaro has joined #heat | 15:32 | |
*** coolsvap has joined #heat | 15:33 | |
*** mkollaro1 has quit IRC | 15:34 | |
*** david-lyle has joined #heat | 15:39 | |
SpamapS | therve: yes I meant the ControllerAddress default | 15:39 |
SpamapS | therve: seemed terribly broken, even after updating with a specific ControllerAddress Metadata that referenced ControllerAddress was not updated. | 15:40 |
therve | SpamapS, It used to work? | 15:40 |
SpamapS | therve: definitely | 15:42 |
SpamapS | therve: seems to have broken some time in the last 24 hours | 15:43 |
*** Michalik has joined #heat | 15:43 | |
therve | SpamapS, Zane made some changes I don't understand. Worth investigating | 15:44 |
*** jamieh has joined #heat | 15:46 | |
SpamapS | therve: yeah I'm going to try to bisect things tonight | 15:47 |
therve | If you have a small reproducible example I can have a look | 15:47 |
*** rcleere has joined #heat | 15:49 | |
therve | SpamapS, What I don't understand is that you have a dependency cycle in the template, which could result in problems like that | 15:49 |
*** lon has joined #heat | 15:49 | |
lon | Warm in here. | 15:49 |
*** aignatov is now known as aignatov_ | 15:57 | |
*** TonyBurn_ has quit IRC | 15:57 | |
*** fandi has quit IRC | 16:04 | |
sdake | zaneb is this implemented properly | 16:04 |
sdake | https://review.openstack.org/#/c/74606/ | 16:04 |
*** andrew_plunk has joined #heat | 16:05 | |
*** jprovazn has quit IRC | 16:06 | |
*** edmund has joined #heat | 16:09 | |
SpamapS | therve: there's no circle in the template that I see. | 16:14 |
therve | SpamapS, ControllerAddress -> undercloud -> undercloudConfig -> ControllerAddress | 16:16 |
*** pablosan has joined #heat | 16:17 | |
sdake | radix what do you think the top 5 problems are with the current autoscaling implementation in heat | 16:18 |
therve | Sounds like an interview question | 16:18 |
sdake | shadower asked me and I came up with a list | 16:19 |
sdake | wanted to see if they matched :) | 16:19 |
radix | heh | 16:19 |
radix | well, the first is probably the lack of being able to scale arbitrary resources | 16:19 |
radix | (including composed ones) | 16:19 |
radix | there also probably needs to be more policy options | 16:20 |
radix | (replace oldest first, or newest first, for example) | 16:20 |
openstackgerrit | A change was merged to openstack/heat: Add Heat API reference to developer docs https://review.openstack.org/73011 | 16:20 |
radix | and also the ability to choose specific resources to destroy when scaling down | 16:20 |
radix | oh, and, core things in Heat like the lack of convergence | 16:21 |
radix | convergence and failure recovery are suuuper important for autoscaling (and also everything else) | 16:21 |
sdake | go into more detail on convergence please | 16:21 |
sdake | first I've heard of it | 16:21 |
radix | self-healing, being able to recreate or fix a resource that fell out of the expected state | 16:21 |
sdake | got it | 16:22 |
sdake | anything else? | 16:22 |
radix | the failure-recovery part is being able to recover when a resource creation spuriously fails | 16:22 |
sdake | I came up with | 16:22 |
sdake | 1) scaling and Heat tightly integrated into one blob | 16:23 |
sdake | 2) no native scaling resources | 16:23 |
sdake | 3) no clear documented way for applications to trigger scale events | 16:23 |
sdake | shadower ^^ there you go thats whats wrong :) | 16:24 |
radix | I'm working on #2, there's a patch up for review :) | 16:24 |
shadower | will have a look | 16:24 |
*** david_lyle_ has joined #heat | 16:24 | |
sdake | radix do they scale OS::Nova::Server resources? | 16:24 |
radix | sdake: arbitrary resources | 16:24 |
radix | you specify the type | 16:24 |
sdake | planned for i3? | 16:25 |
radix | the bp is target to i3, yeah :) | 16:25 |
sdake | cool | 16:25 |
radix | *ed | 16:25 |
sdake | getting alot of complaints about autoscaling not working for OS::Nova::Server | 16:25 |
therve | radix, Should we build native policy too? It should be fairly easy | 16:26 |
*** radez is now known as radez_g0n3 | 16:26 | |
radix | therve: what do you mean? SpamapS's proposal? | 16:26 |
therve | radix, I mean OS::Heat::ScalingPolicy | 16:27 |
radix | oh | 16:27 |
radix | therve: yeah sure, but I'm not really worried about it | 16:27 |
radix | it's a purely superficial thing | 16:27 |
radix | we need to get the actual functionality working | 16:27 |
therve | True | 16:27 |
radix | in fact it could be implemented as a template on AWS::AutoScaling::ScalingPolicy | 16:28 |
*** david-lyle has quit IRC | 16:28 | |
sdake | therve or shardy does https://review.openstack.org/#/c/74606/ look right? | 16:28 |
therve | Hum | 16:29 |
therve | sdake, What about return self.to_dict().iteritems() ? | 16:29 |
*** radez_g0n3 is now known as radez | 16:29 | |
shadower | that's what I'd do, too | 16:29 |
sdake | cool I'll try that | 16:29 |
therve | sdake, I'm a bit worried about doing the switch to oslo.message that late in the cycle fwiw | 16:30 |
therve | It doesn't seem to be working so well in nova yet | 16:30 |
sdake | re switching, we do have 1 month to fix any problems that crop up between i3 and release | 16:31 |
sdake | so far the api part has been not all that invasive | 16:31 |
sdake | (except the test cases) | 16:32 |
*** jprovazn has joined #heat | 16:32 | |
therve | Yeah I would prefer to fix real bugs though :) | 16:32 |
therve | But okay | 16:32 |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Use olso.messaging https://review.openstack.org/72798 | 16:34 |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Add an iteritems for RequestContext https://review.openstack.org/74606 | 16:34 |
*** arbylee1 has joined #heat | 16:35 | |
*** arbylee has quit IRC | 16:35 | |
*** gokrokve has quit IRC | 16:38 | |
shadower | I saw Python3-related fixes in python-heatclient. What's the story with py3k? | 16:38 |
*** gokrokve has joined #heat | 16:39 | |
shadower | is it heatclient-only or do we care about it in heat, too? | 16:39 |
therve | shadower, We need all clients to be ported. | 16:41 |
therve | So we don't care in the code yet, deps need to be done first | 16:41 |
shadower | thanks | 16:43 |
*** gokrokve has quit IRC | 16:43 | |
openstackgerrit | Jenkins proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/72566 | 16:44 |
*** tspatzier has quit IRC | 16:45 | |
*** spzala has joined #heat | 16:48 | |
*** jistr has quit IRC | 16:50 | |
*** aignatov_ is now known as aignatov | 16:52 | |
*** arbylee1 has quit IRC | 16:53 | |
*** arbylee has joined #heat | 16:53 | |
*** randallburt has joined #heat | 16:53 | |
*** tomek_adamczewsk has quit IRC | 16:56 | |
*** lazzurs has quit IRC | 16:58 | |
*** e0ne has quit IRC | 16:58 | |
*** derekh has quit IRC | 17:03 | |
*** aignatov is now known as aignatov_ | 17:04 | |
*** aignatov_ is now known as aignatov | 17:06 | |
daneyon | I am trying to use cfn-init in my template to ensure certain packages are installed. I am receiving a Exception: invalid metadata from the cfn-init command, however I am supplying the correct metadata. See pastebin for more details: http://pastebin.com/2hTGvH4V Any troubleshooting suggestions? | 17:08 |
*** cadenzajon has joined #heat | 17:13 | |
*** radez is now known as radez_g0n3 | 17:13 | |
*** radez_g0n3 is now known as radez | 17:14 | |
*** chandankumar_ has joined #heat | 17:24 | |
*** chandankumar_ has quit IRC | 17:27 | |
*** jamieh has quit IRC | 17:28 | |
*** topol has quit IRC | 17:31 | |
*** topol has joined #heat | 17:31 | |
*** gokrokve has joined #heat | 17:32 | |
*** jamieh has joined #heat | 17:34 | |
openstackgerrit | Andrew Plunk proposed a change to openstack/heat: Oslo config option changed by test causing problem https://review.openstack.org/74157 | 17:40 |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Rackspace Cloud Networks resource https://review.openstack.org/68790 | 17:41 |
*** pvaneck has joined #heat | 17:41 | |
*** ramishra has quit IRC | 17:44 | |
*** cmyster has quit IRC | 17:44 | |
*** ramishra_ has joined #heat | 17:47 | |
*** packet has joined #heat | 17:48 | |
*** ramishra_ has quit IRC | 17:51 | |
openstackgerrit | Richard Lee proposed a change to openstack/heat: Make Keystone client pluggable https://review.openstack.org/74190 | 18:00 |
openstackgerrit | Richard Lee proposed a change to openstack/heat: Add Keystone V2 plugin https://review.openstack.org/74191 | 18:00 |
*** lindsayk has joined #heat | 18:08 | |
*** david_lyle_ is now known as david_lyle | 18:08 | |
*** rbuilta has joined #heat | 18:08 | |
*** wchrisj has joined #heat | 18:09 | |
*** chandan_kumar has quit IRC | 18:12 | |
*** ramishra has joined #heat | 18:18 | |
*** rbuilta has quit IRC | 18:18 | |
*** nati_ueno has joined #heat | 18:21 | |
*** ramishra has quit IRC | 18:23 | |
*** kebray_ has joined #heat | 18:27 | |
*** randallburt has quit IRC | 18:30 | |
*** wchrisj has quit IRC | 18:32 | |
*** wchrisj_ has joined #heat | 18:32 | |
*** arbylee has quit IRC | 18:33 | |
*** dtalton has joined #heat | 18:42 | |
*** e0ne has joined #heat | 18:44 | |
*** spzala has quit IRC | 18:44 | |
*** spzala has joined #heat | 18:45 | |
*** tspatzier has joined #heat | 18:48 | |
*** radez is now known as radez_g0n3 | 18:48 | |
*** rpothier has joined #heat | 18:49 | |
*** jamieh has quit IRC | 18:51 | |
*** radez_g0n3 is now known as radez | 18:51 | |
*** rpothier_ has quit IRC | 18:52 | |
zaneb | stevebaker: any thoughts on https://blueprints.launchpad.net/heat/+spec/extraroute-as-contrib ? I think we should approve | 18:54 |
*** jamieh has joined #heat | 18:56 | |
*** yogesh has joined #heat | 18:59 | |
*** killer_prince is now known as lazy_prince | 18:59 | |
*** e0ne has quit IRC | 19:00 | |
*** lindsayk has quit IRC | 19:02 | |
*** lindsayk has joined #heat | 19:04 | |
*** jamieh has quit IRC | 19:10 | |
*** wchrisj_ has quit IRC | 19:11 | |
*** wchrisj has joined #heat | 19:12 | |
*** pshchelo_ has joined #heat | 19:14 | |
*** ramishra has joined #heat | 19:19 | |
*** e0ne has joined #heat | 19:21 | |
*** ramishra has quit IRC | 19:24 | |
*** nati_ueno has quit IRC | 19:24 | |
*** nati_ueno has joined #heat | 19:25 | |
*** randallburt1 has joined #heat | 19:29 | |
*** arbylee has joined #heat | 19:29 | |
*** rbuilta has joined #heat | 19:30 | |
*** kebray_ has quit IRC | 19:30 | |
*** yogesh has quit IRC | 19:31 | |
*** sballe has quit IRC | 19:31 | |
*** sballe has joined #heat | 19:32 | |
*** aignatov is now known as aignatov_ | 19:33 | |
*** sballe has quit IRC | 19:33 | |
skraynev | zaneb: Could you look https://review.openstack.org/#/c/70358/ in free time period? | 19:34 |
*** kebray_ has joined #heat | 19:34 | |
*** lindsayk has quit IRC | 19:34 | |
*** sballe has joined #heat | 19:34 | |
*** lindsayk has joined #heat | 19:35 | |
zaneb | haha "free time period". good one ;) | 19:35 |
*** jrist has quit IRC | 19:35 | |
*** arbylee has quit IRC | 19:37 | |
*** arbylee has joined #heat | 19:37 | |
*** sballe has quit IRC | 19:38 | |
sdake | zaneb got a noob question again for you | 19:39 |
sdake | can you help :) | 19:39 |
sdake | show_deleted = kwargs.get('show_deleted') or context.show_deleted | 19:39 |
zaneb | yep :) | 19:39 |
skraynev | zaneb: I am glad here, that you like it) | 19:39 |
sdake | when I print out context, there is a show_deleted : False in it | 19:39 |
sdake | wait different error | 19:40 |
sdake | return (context and context.session) or get_session() | 19:40 |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API limit exception in nova_utils.refresh_server https://review.openstack.org/71660 | 19:41 |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Move refresh_server() to nova_utils https://review.openstack.org/72008 | 19:41 |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make server resources use nova_utils.refresh_server() https://review.openstack.org/72009 | 19:41 |
sdake | zaneb, starting over with problem :) | 19:41 |
sdake | show_deleted = kwargs.get('show_deleted') or context.show_deleted | 19:42 |
sdake | when I print out context, a show_deleted : False is present in the dict | 19:42 |
sdake | 2014-02-19 12:41:38.403 TRACE oslo.messaging._executors.base AttributeError: 'di | 19:42 |
sdake | ct' object has no attribute 'show_deleted' | 19:42 |
sdake | error I get^ | 19:42 |
*** aignatov_ is now known as aignatov | 19:43 | |
therve | Well context is a dict and has no attribute show_deleted | 19:43 |
therve | :) | 19:43 |
sdake | RequestContext has a self.show_deleted | 19:44 |
zaneb | so it sounds like you're passing a dict where a RequestContext was expected? | 19:45 |
sdake | the code seems to think its a dict already | 19:45 |
therve | If it's the db API, it's a context object you need to pass, not a dict | 19:46 |
*** rbuilta has quit IRC | 19:47 | |
sdake | oslo.messaging is the one passing the context | 19:47 |
*** rbuilta has joined #heat | 19:48 | |
*** sballe has joined #heat | 19:48 | |
zaneb | sdake: so you may need to reconstruct the context by calling RequestContext.from_dict() after you get it from oslo.messaging | 19:49 |
*** tango has joined #heat | 19:49 | |
therve | sdake, You removed the request_context decorator maybe? | 19:50 |
sdake | therve that is probably it thanks | 19:51 |
*** pshchelo_ has quit IRC | 19:54 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/python-heatclient: Improve help strings https://review.openstack.org/74816 | 20:00 |
skraynev | Is alter-time meeting today? | 20:01 |
*** jrist has joined #heat | 20:01 | |
*** lindsayk has quit IRC | 20:09 | |
*** lindsayk has joined #heat | 20:11 | |
stevebaker | morning | 20:13 |
stevebaker | zaneb: +1 on extraroutes config | 20:13 |
zaneb | ok | 20:13 |
zaneb | ta | 20:13 |
*** radez is now known as radez_g0n3 | 20:15 | |
skraynev | stevebaker: hi, thank for the answer on review with provider network ;) | 20:16 |
stevebaker | is it still before the feature proposal freeze somewhere in the world? | 20:16 |
zaneb | stevebaker: technically... no | 20:18 |
stevebaker | zaneb: I'm just thinking we have a bunch of properties to deprecate as part of bug fixes, but no mechanism to marke them deprecated | 20:20 |
zaneb | stevebaker: I don't think anyone is going to call you out for implementing that | 20:21 |
stevebaker | zaneb: I'd like to add a schema field which takes a resource.SupportStatus | 20:21 |
*** radez_g0n3 is now known as radez | 20:21 | |
zaneb | sounds kinda heavyweight | 20:22 |
zaneb | in a way I think we'd be doing users a disservice by providing any annotations more complicated than "use this" and "don't use this" | 20:23 |
*** randallburt1 has quit IRC | 20:23 | |
stevebaker | zaneb: we do need at least a deprecation message, and I can see us using all of SupportStatus in the future for our documentation generation | 20:24 |
therve | stevebaker, Where would the deprecation message appear? | 20:25 |
zaneb | agree on the deprecation message, so you're probably right | 20:25 |
zaneb | therve: in the docs | 20:25 |
skraynev | zaneb: about validation of outputs. You offer make rebase on it, right? | 20:25 |
stevebaker | therve: in the resource reference section of the template guide | 20:25 |
therve | stevebaker, We may as well write some text then | 20:26 |
zaneb | skraynev: yes, I suggest you rebase on that patch series | 20:26 |
*** cadenzajon has quit IRC | 20:26 | |
skraynev | zaneb: good. I have had such idea, i guess it was one of the reason why I asked you about reviewing it. | 20:27 |
*** tspatzier has quit IRC | 20:28 | |
zaneb | ++ thanks for letting me know | 20:28 |
skraynev | zaneb: lol, np ;) | 20:28 |
*** cmyster has joined #heat | 20:30 | |
*** cmyster has quit IRC | 20:32 | |
*** rbuilta has quit IRC | 20:32 | |
*** cmyster has joined #heat | 20:33 | |
*** cmyster has quit IRC | 20:35 | |
skraynev | zaneb: and one more question about it: I am not sure, that we can check get_attr before creating resource, because it depend on resource status. | 20:36 |
zaneb | skraynev: right, it returns None if the resource is not in the right status | 20:36 |
zaneb | but that's not a problem | 20:37 |
skraynev | and when you mentioned about resource section, what did you mean? | 20:37 |
skraynev | validate method? | 20:37 |
zaneb | I mean we validate the resource section of the template too | 20:37 |
zaneb | and that has cross-references between resources with get_attr | 20:37 |
skraynev | oh. understand | 20:37 |
skraynev | so what about None? | 20:38 |
skraynev | why is not problem? | 20:38 |
skraynev | if we get None, we will not check this attribute it at all. | 20:39 |
skraynev | or I am missing something | 20:39 |
openstackgerrit | Daneyon Hansen proposed a change to openstack/heat-templates: Adds Support for OpenShift Origin v3.0 on Fedora 19 https://review.openstack.org/74450 | 20:45 |
zaneb | skraynev: true. it's fairly poor validation. but at least we check the resource exists | 20:47 |
*** vijendar has quit IRC | 20:50 | |
*** ramishra has joined #heat | 20:50 | |
*** vijendar has joined #heat | 20:50 | |
stevebaker | therve: hey, do the new runtime constraints allow the value to be replaced? (such as name->id?) | 20:56 |
*** randallburt has joined #heat | 20:58 | |
*** andersonvom has quit IRC | 21:00 | |
sdake | good news everybody, oslo.messaging seems to mostly work :) | 21:00 |
sdake | now to beat the test cases into submission | 21:00 |
skraynev | zaneb: but the main problem of my bug is checking attribute (not resource). Also could you please point a place in code with checking (I mean resource section) | 21:01 |
stevebaker | \o/ | 21:01 |
skraynev | sdake: really good news :) | 21:01 |
sdake | yes, although there are random exceptions popping up here and there | 21:01 |
sdake | need to sort those out | 21:01 |
sdake | but can create/list/describe/delete stacks :) | 21:01 |
* stevebaker looks for a property to deprecate, with prejudice | 21:02 | |
*** kebray_ has quit IRC | 21:02 | |
*** dtalton has left #heat | 21:03 | |
*** andrew_plunk has quit IRC | 21:05 | |
openstackgerrit | Rabi Mishra proposed a change to openstack/heat: Add ability to create provider networks (vlan,flat). https://review.openstack.org/72454 | 21:06 |
ramishra | stevebaker: Hello reg:https://review.openstack.org/#/c/72454/ after our irc yesterday, I had the impression that you did not suggest move the repeatable code to a static methods for readability... may be I understood you incorrectly | 21:09 |
ramishra | stevebaker: updated with a new patch | 21:09 |
*** topol has quit IRC | 21:10 | |
*** pafuent has left #heat | 21:10 | |
stevebaker | ramishra: looks fine to me | 21:10 |
*** jamieh has joined #heat | 21:11 | |
*** alexpilotti has quit IRC | 21:11 | |
*** alexpilotti_ has joined #heat | 21:11 | |
ramishra | stevabker: ok.thanx.. please review it.. | 21:12 |
*** cadenzajon has joined #heat | 21:13 | |
*** jamieh has quit IRC | 21:14 | |
*** andrew_plunk has joined #heat | 21:14 | |
ramishra | stevebaker: reg lbaas resource bugs..I had a chat with therve.. he is of the opinion that we should wait for any more neutron api changes before changing it...but I don't see many API changes | 21:14 |
stevebaker | ramishra: I am not a subject matter expert for lbaas. I would defer to therve's opinion | 21:15 |
*** jamieh has joined #heat | 21:16 | |
*** jamieh has quit IRC | 21:16 | |
ramishra | stevebaker: my view is the model is frozen in neutron at the moment inline with https://wiki.openstack.org/wiki/LBaaS/CoreResourceModel/proposal | 21:16 |
ramishra | stevebaker: ok... | 21:16 |
stevebaker | ramishra: convince therve then ;) | 21:16 |
ramishra | stevebaker: ok will try;) | 21:18 |
*** jamieh has joined #heat | 21:19 | |
*** arbylee1 has joined #heat | 21:20 | |
skraynev | stevebaker: about lbaas vip bugs, which were mentioned by ramishra. I not fully understand why we create two new bugs instead https://bugs.launchpad.net/heat/+bug/1258490 | 21:21 |
uvirtbot | Launchpad bug 1258490 in heat "Moving vip in separate resource or making it more clear." [Medium,New] | 21:21 |
*** arbylee has quit IRC | 21:21 | |
*** lindsayk has quit IRC | 21:23 | |
*** lindsayk has joined #heat | 21:23 | |
skraynev | I was surprised that we get two new bugs with same aim. Also therve mentioned the reason (which was in comments for this bug), why we have not done it yet (in icehouse). | 21:24 |
stevebaker | skraynev: what you describe appears to be some incremental changes to add missing properties and rename the resource. Is this correct? | 21:25 |
*** arbylee1 has quit IRC | 21:25 | |
*** arbylee has joined #heat | 21:25 | |
*** e0ne has quit IRC | 21:28 | |
openstackgerrit | Steven Dake proposed a change to openstack/heat: Use olso.messaging https://review.openstack.org/72798 | 21:30 |
*** randallburt has quit IRC | 21:31 | |
skraynev | stevebaker: not fully. at the first time the idea was such as said ramishra - move VIP to separate resource and make architecture of lbaas same with neutron's. | 21:32 |
skraynev | But when I failed this bug, I had had discussion with therve and shardy about reason. | 21:32 |
skraynev | so now I plan to wait changes in neutron (unfortunately it will be in juno) and then change current resource scheme | 21:34 |
*** shakayumi has joined #heat | 21:35 | |
skraynev | first step is (with bp) moving vip to separate resource for supporting multiple vips per pool (https://blueprints.launchpad.net/heat/+spec/multiple-vips-for-lbaas) | 21:35 |
stevebaker | skraynev: that looks like a reasonable plan. ramishra are you OK with halting your efforts for now? | 21:37 |
*** spzala has quit IRC | 21:37 | |
ramishra | stevebaker: IMO, the model is almost frozen for icehouse (I don't think multiple vip->pool bp would land in icehouse), so we should get the resource inline with the model...any changes for juno we should do in juno | 21:39 |
stevebaker | ramishra, skraynev, can you both attend today's 0000 UTC meeting to discuss this? | 21:40 |
ramishra | stevebaker: having said that I am ok if the group decides otherwise...leave it till juno... | 21:41 |
*** jprovazn has quit IRC | 21:41 | |
stevebaker | ramishra, skraynev, ideally I'd like you two to figure it out | 21:42 |
ramishra | stevebaker: In juno, there are additional l7 changes, https://wiki.openstack.org/wiki/Neutron/LBaaS/LoadbalancerInstance | 21:42 |
skraynev | stevebaker: I am not sure, that I could join to meeting today (4 Am in my time zone will be). But if it necessary and can not be moved to the next week meeting, I will try . | 21:44 |
stevebaker | skraynev: that seems unreasonably late. A thread on openstack-dev might be better | 21:45 |
*** blomquisg has quit IRC | 21:45 | |
skraynev | ramishra: about l7 and ssl supporting, could you look it: https://blueprints.launchpad.net/heat/+spec/lbaas-l7-rules-support , but it stiil wait juno release | 21:46 |
skraynev | s/stiil/too | 21:46 |
*** devincognito has joined #heat | 21:47 | |
*** radez is now known as radez_g0n3 | 21:47 | |
stevebaker | skraynev: sounds like there is no hurry then. It will be looked at | 21:48 |
skraynev | stevebaker: Is it possible to create such thread tomorrow? (I mean your early morning and my evening) | 21:48 |
stevebaker | skraynev: yes, post the message when you're ready | 21:49 |
skraynev | stevebaker: he-he. Neutron team has other problems in gate . I think it's more important. Also they wait summit to make design for new neutron architecture | 21:50 |
skraynev | stevebaker: just for clarification: the main questions for message: changes in neutron and how,when it will be implemented in heat, right? | 21:52 |
*** aweiteka has quit IRC | 21:52 | |
stevebaker | skraynev: yes, and whether we make some incremental improvements to our existing LB resource before the icehouse release, or do it all in juno | 21:53 |
skraynev | stevebaker: gotcha | 21:54 |
skraynev | I am happy, that we find conclusion ! | 21:54 |
skraynev | it's good reason go to sleep ;) | 21:54 |
skraynev | night | 21:55 |
*** skraynev is now known as skraynev_afk | 21:55 | |
*** jamieh has quit IRC | 21:55 | |
*** wusWWW has joined #heat | 21:57 | |
openstackgerrit | Richard Lee proposed a change to openstack/heat: Add admin_pass as attribute and property to server https://review.openstack.org/72745 | 21:57 |
*** jdob has quit IRC | 21:57 | |
*** wusWWW has quit IRC | 21:58 | |
*** ramishra has quit IRC | 21:58 | |
*** devincognito has quit IRC | 22:02 | |
*** vijendar has quit IRC | 22:03 | |
*** cadenzajon has quit IRC | 22:09 | |
*** cadenzajon has joined #heat | 22:13 | |
*** topol has joined #heat | 22:24 | |
sdake | zaneb would you mind taking this review https://bugzilla.redhat.com/show_bug.cgi?id=1067184 | 22:24 |
uvirtbot | sdake: Error: Could not parse XML returned by bugzilla.redhat.com: HTTP Error 404: Not Found | 22:24 |
zaneb | sure, can do | 22:25 |
sdake | thx | 22:25 |
zaneb | not that I have ever used/seen os-apply-config... | 22:25 |
sdake | its required for software config | 22:26 |
*** jomara has quit IRC | 22:29 | |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Refactor SupportStatus into its own module https://review.openstack.org/74853 | 22:31 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Add support_status attribute to properties schema https://review.openstack.org/74854 | 22:31 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Use support_status for existing deprecated properties https://review.openstack.org/74855 | 22:31 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Use property support_status in documentation generation https://review.openstack.org/74856 | 22:31 |
openstackgerrit | Steve Baker proposed a change to openstack/heat: Doc generation display message for deprecated resources https://review.openstack.org/74857 | 22:31 |
*** e0ne has joined #heat | 22:39 | |
*** rcleere has quit IRC | 22:39 | |
*** daneyon_ has joined #heat | 22:42 | |
*** e0ne has quit IRC | 22:42 | |
*** daneyon has quit IRC | 22:45 | |
*** aignatov is now known as aignatov_ | 22:47 | |
*** FL1SK has quit IRC | 22:52 | |
*** FL1SK has joined #heat | 22:53 | |
*** kebray_ has joined #heat | 22:54 | |
*** daneyon_ has quit IRC | 22:56 | |
openstackgerrit | Richard Lee proposed a change to openstack/heat: Allow update of admin_pass on server https://review.openstack.org/72746 | 23:02 |
*** rbuilta has joined #heat | 23:02 | |
sdake | zaneb can you take another review: https://bugzilla.redhat.com/show_bug.cgi?id=1067200 | 23:02 |
uvirtbot | sdake: Error: Could not parse XML returned by bugzilla.redhat.com: HTTP Error 404: Not Found | 23:02 |
zaneb | ok, should be fairly similar | 23:03 |
sdake | this one has a systemd file | 23:03 |
*** kebray_ has quit IRC | 23:03 | |
*** lindsayk has quit IRC | 23:07 | |
*** kebray_ has joined #heat | 23:07 | |
*** lindsayk has joined #heat | 23:08 | |
*** radez_g0n3 is now known as radez | 23:11 | |
*** kebray_ has quit IRC | 23:13 | |
openstackgerrit | A change was merged to openstack-dev/heat-cfnclient: Rename Openstack to OpenStack https://review.openstack.org/73175 | 23:17 |
openstackgerrit | A change was merged to openstack/heat: Add systemd notification support to heat https://review.openstack.org/74489 | 23:22 |
*** sabeen has quit IRC | 23:27 | |
*** wchrisj has quit IRC | 23:28 | |
*** radez is now known as radez_g0n3 | 23:29 | |
*** topol has quit IRC | 23:33 | |
*** openstack has joined #heat | 23:34 | |
*** andrew_plunk has quit IRC | 23:37 | |
sdake | zaneb: last one https://bugzilla.redhat.com/show_bug.cgi?id=1067216 | 23:41 |
uvirtbot | sdake: Error: Could not parse XML returned by bugzilla.redhat.com: HTTP Error 404: Not Found | 23:41 |
sdake | enjoy :) | 23:41 |
*** arbylee1 has joined #heat | 23:44 | |
*** arbylee has quit IRC | 23:44 | |
*** jergerber has quit IRC | 23:46 | |
*** randallburt has joined #heat | 23:47 | |
*** alexpilotti_ has quit IRC | 23:47 | |
*** mkollaro has quit IRC | 23:47 | |
*** arbylee1 has quit IRC | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!