| *** alexheneveld has quit IRC | 00:00 | |
| *** blamar has quit IRC | 00:02 | |
| *** alexheneveld has joined #heat | 00:10 | |
| *** alexheneveld has quit IRC | 00:12 | |
| *** ccs has quit IRC | 00:15 | |
| *** alexheneveld has joined #heat | 00:16 | |
| *** gokrokve has joined #heat | 00:21 | |
| *** alexheneveld has quit IRC | 00:23 | |
| *** asalkeld has joined #heat | 00:25 | |
| *** gokrokve has quit IRC | 00:26 | |
| *** gokrokve has joined #heat | 01:22 | |
| *** gokrokve has quit IRC | 01:27 | |
| *** harlowja has joined #heat | 01:55 | |
| *** guohliu has joined #heat | 01:57 | |
| *** blamar has joined #heat | 01:59 | |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/python-heatclient: Fixed typo error OS_IMAGE_URL https://review.openstack.org/63661 | 02:07 |
|---|---|---|
| *** rpothier_ has joined #heat | 02:25 | |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/python-heatclient: Wrapped some long lines https://review.openstack.org/63662 | 02:25 |
| *** rpothier__ has quit IRC | 02:28 | |
| sdake | evening | 02:32 |
| *** erkules_ has joined #heat | 02:35 | |
| *** erkules has quit IRC | 02:38 | |
| *** vkozhukalov has joined #heat | 02:54 | |
| *** harlowja is now known as harlowja_away | 02:55 | |
| sdake | bedtime, ttyl :) | 03:07 |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/python-heatclient: Added API reference document https://review.openstack.org/63665 | 03:17 |
| *** nanjj has joined #heat | 03:17 | |
| *** aignatov has joined #heat | 03:19 | |
| *** blamar has quit IRC | 03:21 | |
| *** aignatov has quit IRC | 03:21 | |
| *** aignatov has joined #heat | 03:23 | |
| openstackgerrit | Jun Jie Nan proposed a change to openstack/python-heatclient: Added API reference document https://review.openstack.org/63665 | 03:34 |
| *** nanjj has quit IRC | 03:37 | |
| *** nanjj has joined #heat | 03:37 | |
| *** ccs has joined #heat | 03:55 | |
| *** guohliu has quit IRC | 04:03 | |
| *** guohliu has joined #heat | 04:04 | |
| *** nkhare has joined #heat | 04:25 | |
| *** harlowja_away is now known as harlowja | 04:31 | |
| *** harlowja has quit IRC | 04:35 | |
| *** chandankumar has joined #heat | 04:44 | |
| openstackgerrit | DennyZhang proposed a change to openstack/heat: Change loglevel for certain failures https://review.openstack.org/63633 | 04:44 |
| *** akuznetsov has joined #heat | 04:47 | |
| *** aignatov has quit IRC | 04:48 | |
| openstackgerrit | DennyZhang proposed a change to openstack/heat: Change loglevel for certain failures https://review.openstack.org/63669 | 04:59 |
| *** gokrokve has joined #heat | 05:00 | |
| *** gokrokve has quit IRC | 05:05 | |
| *** nanjj has quit IRC | 05:31 | |
| *** ccs has quit IRC | 05:50 | |
| *** gokrokve has joined #heat | 06:00 | |
| *** gokrokve has quit IRC | 06:05 | |
| *** gokrokve has joined #heat | 06:05 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/62551 | 06:06 |
| *** gokrokve has quit IRC | 06:10 | |
| *** gokrokve has joined #heat | 06:17 | |
| skraynev | morning | 06:34 |
| *** guohliu has quit IRC | 06:44 | |
| *** vkozhukalov has quit IRC | 06:50 | |
| *** gokrokve has quit IRC | 07:07 | |
| *** erkules_ is now known as erkules | 07:09 | |
| sergmelikyan | \o/ Good morning :) | 07:22 |
| sergmelikyan | http://paste.openstack.org/show/56285/ - do I need explicit return here? I believe - no, but I am not sure. | 07:24 |
| sergmelikyan | Because exception going to be re-raised right after exit from ContextManager: https://github.com/openstack/heat/blob/master/heat/openstack/common/excutils.py#L60 | 07:26 |
| *** ifarkas has joined #heat | 07:26 | |
| *** gokrokve has joined #heat | 07:37 | |
| *** Alienyyg_ has joined #heat | 07:39 | |
| Alienyyg_ | morning guys | 07:39 |
| *** gokrokve_ has joined #heat | 07:39 | |
| *** gokrokve has quit IRC | 07:42 | |
| *** gokrokve_ has quit IRC | 07:44 | |
| *** akuznetsov has quit IRC | 07:47 | |
| *** tomek_adamczewsk has joined #heat | 08:01 | |
| *** bvandenh has quit IRC | 08:03 | |
| *** ccs has joined #heat | 08:06 | |
| therve | Good morning! | 08:07 |
| *** ccs has quit IRC | 08:09 | |
| *** ccs_ has joined #heat | 08:09 | |
| *** tomek_adamczewsk has quit IRC | 08:14 | |
| *** tomek_adamczewsk has joined #heat | 08:16 | |
| *** tomek_adamczewsk has quit IRC | 08:25 | |
| *** tomek_adamczewsk has joined #heat | 08:26 | |
| *** vkozhukalov has joined #heat | 08:29 | |
| *** gokrokve has joined #heat | 08:30 | |
| *** ccs_ has quit IRC | 08:32 | |
| *** gokrokve has quit IRC | 08:35 | |
| *** tomek_adamczewsk has quit IRC | 08:38 | |
| *** tomek_adamczewsk has joined #heat | 08:40 | |
| *** akuznetsov has joined #heat | 08:42 | |
| *** aignatov has joined #heat | 08:44 | |
| *** aignatov has quit IRC | 08:56 | |
| *** SergeyLukjanov has joined #heat | 09:02 | |
| *** ccs has joined #heat | 09:08 | |
| *** ccs has quit IRC | 09:24 | |
| *** shardy_afk is now known as shardy | 09:32 | |
| *** aignatov has joined #heat | 09:32 | |
| shardy | morning | 09:32 |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Add new Ceilometer combination alarm resource https://review.openstack.org/62672 | 09:38 |
| *** bgorski has joined #heat | 09:48 | |
| asalkeld | hi shardy | 09:51 |
| *** pshchelo has joined #heat | 09:54 | |
| therve | skraynev, Why would I add tests? | 09:55 |
| *** chandankumar has quit IRC | 09:56 | |
| skraynev | therve: Hm. It was only offer. I think, that each resource has own tests. Also this resource has different option are not similar on standard ceilometer alarm. | 09:57 |
| therve | skraynev, I have there are tests already. What do you see missing tests? | 09:58 |
| *** chandankumar has joined #heat | 09:59 | |
| *** gokrokve has joined #heat | 10:00 | |
| skraynev | therve: doh. Sorry, my bad. When I look at the changes between 2 and 3 patch set, I change old version to 2. Of course, in that case added test was not shown. | 10:00 |
| skraynev | therve: All is ok ;) | 10:01 |
| *** derekh has joined #heat | 10:01 | |
| therve | Cool :) | 10:02 |
| openstackgerrit | A change was merged to openstack/heat: Add a schema for the AWS::EC2::Instance Volumes property https://review.openstack.org/62434 | 10:02 |
| *** SergeyLukjanov has quit IRC | 10:03 | |
| *** SergeyLukjanov has joined #heat | 10:04 | |
| openstackgerrit | A change was merged to openstack/heat: Use property name constants in CinderVolume https://review.openstack.org/62449 | 10:04 |
| *** aignatov has quit IRC | 10:04 | |
| *** aignatov has joined #heat | 10:04 | |
| *** gokrokve has quit IRC | 10:05 | |
| *** akuznetsov has quit IRC | 10:06 | |
| *** chandankumar has quit IRC | 10:06 | |
| *** uvirtbot has quit IRC | 10:07 | |
| *** akuznetsov has joined #heat | 10:08 | |
| *** uvirtbot has joined #heat | 10:08 | |
| *** chandankumar has joined #heat | 10:09 | |
| *** akuznetsov has quit IRC | 10:25 | |
| *** akuznetsov has joined #heat | 10:28 | |
| *** akuznetsov has quit IRC | 10:41 | |
| *** akuznetsov has joined #heat | 10:44 | |
| *** akuznetsov has quit IRC | 10:45 | |
| *** alexheneveld has joined #heat | 10:56 | |
| *** bvandenh has joined #heat | 10:58 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/63709 | 10:59 |
| *** bvandenh has quit IRC | 11:09 | |
| openstackgerrit | A change was merged to openstack/heat: Add support for multiple encryption methods https://review.openstack.org/62295 | 11:22 |
| *** nkhare has quit IRC | 11:23 | |
| *** akuznetsov has joined #heat | 11:25 | |
| *** akuznetsov has quit IRC | 11:30 | |
| *** akuznetsov has joined #heat | 11:32 | |
| *** akuznetsov has quit IRC | 11:45 | |
| *** akuznetsov has joined #heat | 11:46 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 11:48 | |
| skraynev | shardy: do you know what is the problem with commits https://review.openstack.org/#/q/status:open+project:openstack/heat+branch:stable/havana,n,z ? May be it related with [outdated] patch in https://review.openstack.org/#/c/56829/2 . It's not very urgently, but it's looks strange. | 11:48 |
| *** _SergeyLukjanov has quit IRC | 11:48 | |
| shardy | skraynev: Yeah, looks like the queue of patches needs a rebase | 11:50 |
| *** akuznetsov has quit IRC | 11:50 | |
| shardy | stevebaker: ^^ | 11:50 |
| skraynev | shardy: ok, thanks ;) | 11:51 |
| *** akuznetsov has joined #heat | 11:52 | |
| *** ifarkas has quit IRC | 11:57 | |
| *** gokrokve has joined #heat | 12:00 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Implement neutron metering resources https://review.openstack.org/62378 | 12:01 |
| *** gokrokve has quit IRC | 12:04 | |
| *** SergeyLukjanov has joined #heat | 12:30 | |
| *** agileclipse has joined #heat | 12:31 | |
| *** che-arne has quit IRC | 12:37 | |
| openstackgerrit | Serg Melikyan proposed a change to openstack/heat: Implements Nova FloatingIP resources https://review.openstack.org/62836 | 12:43 |
| *** gokrokve has joined #heat | 13:00 | |
| *** rpothier_ has quit IRC | 13:02 | |
| *** gokrokve has quit IRC | 13:05 | |
| openstackgerrit | A change was merged to openstack/heat: SignalResponder store access/secret in resource data https://review.openstack.org/62440 | 13:20 |
| *** Tross has joined #heat | 13:25 | |
| openstackgerrit | A change was merged to openstack/heat: Imported Translations from Transifex https://review.openstack.org/62551 | 13:27 |
| openstackgerrit | A change was merged to openstack/heat: Remove unnecessary setUp call https://review.openstack.org/63225 | 13:27 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Implement neutron metering resources https://review.openstack.org/62378 | 13:38 |
| openstackgerrit | A change was merged to openstack/heat: Updated from global requirements https://review.openstack.org/63709 | 13:39 |
| openstackgerrit | A change was merged to openstack/heat: Change loglevel for certain failures https://review.openstack.org/63669 | 13:41 |
| *** rpothier has joined #heat | 13:49 | |
| sergmelikyan | skraynev, I had updated link that I was mentioned in the review :) | 13:49 |
| skraynev | sergmelikyan: ok, I will look) | 13:50 |
| openstackgerrit | Serg Melikyan proposed a change to openstack/heat: Added session_persistence property to VIP https://review.openstack.org/61074 | 13:51 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Implement neutron metering resources https://review.openstack.org/62378 | 13:55 |
| *** gokrokve has joined #heat | 14:00 | |
| *** gokrokve has quit IRC | 14:05 | |
| *** pafuent has joined #heat | 14:07 | |
| *** Tross has quit IRC | 14:12 | |
| openstackgerrit | Dmitry Shulyak proposed a change to openstack/heat: Transition to alembic migration with oslo migration tool https://review.openstack.org/63767 | 14:24 |
| *** AndrewMichalik has joined #heat | 14:24 | |
| *** andersonvom has joined #heat | 14:26 | |
| *** pafuent1 has joined #heat | 14:31 | |
| *** pafuent has quit IRC | 14:35 | |
| *** jasond` has joined #heat | 14:39 | |
| *** bvandenh has joined #heat | 14:46 | |
| *** nijaba has quit IRC | 14:50 | |
| openstackgerrit | A change was merged to openstack/heat: Let users specify metadatas on swift accounts. https://review.openstack.org/63624 | 14:53 |
| *** gokrokve_ has joined #heat | 14:56 | |
| *** nijaba has joined #heat | 14:57 | |
| *** nijaba has quit IRC | 14:57 | |
| *** nijaba has joined #heat | 14:57 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 14:58 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Add way to group exceptions in DependencyTaskGroup https://review.openstack.org/62479 | 14:58 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Refactor Instance#_check_active https://review.openstack.org/62478 | 14:58 |
| *** rwsu has joined #heat | 14:59 | |
| *** rustlebee is now known as russellb | 14:59 | |
| andersonvom | how did _that_ happen? | 15:01 |
| *** nijaba has quit IRC | 15:01 | |
| *** jpeeler has joined #heat | 15:02 | |
| *** jpeeler has quit IRC | 15:02 | |
| *** jpeeler has joined #heat | 15:02 | |
| pshchelo | are you talking about my commits? was trying to publish a change dependent on https://review.openstack.org/#/c/58641/14 | 15:02 |
| *** nijaba has joined #heat | 15:04 | |
| pshchelo | I'm really sorry. was following https://wiki.openstack.org/wiki/GerritWorkflow#Add_dependency | 15:04 |
| sergmelikyan | This issue may be caused by outdated git-review, | 15:04 |
| pshchelo | I had an outdeted verison, yes. It was not allowing me to commit the review at all. Update it just now. Then it worked in this misterious way | 15:05 |
| *** tims has joined #heat | 15:05 | |
| sergmelikyan | andersonvom, this changes are draft-one, can you delete patch-set | 15:06 |
| andersonvom | sergmelikyan: I don't think I have permissions to do that | 15:06 |
| andersonvom | I just noticed all the commits became drafts | 15:07 |
| *** tims1 has joined #heat | 15:07 | |
| andersonvom | pshchelo: if you change it to WIP it may solve the issue | 15:08 |
| *** tims has quit IRC | 15:10 | |
| pshchelo | I've "published" my change, but it still DRAFT in those patches I've submitted to your and Robert's changes | 15:11 |
| pshchelo | And I'm still *really* interested in understanding what actually happened to avoid it in the future | 15:12 |
| pshchelo | If someone could help me with this it would be awesome.. | 15:13 |
| pshchelo | s/Ropert's/Richard's | 15:14 |
| andersonvom | pshchelo: submitting draft patches is usually not a good idea. you have to explicitly give permissions for people to see your patch and it makes everything it depends on drafts as well | 15:15 |
| jasond` | pshchelo: you probably checked out https://review.openstack.org/62478 for the dependency, rebased against upstream master at some point, and submitted your patch with "git review --draft" | 15:16 |
| pshchelo | yes, now I see it :( But even without drafts, my patch would've been submitted to other people's branches, right? | 15:16 |
| jasond` | pshchelo: yes. it might be better to not rebase someone else's patch | 15:17 |
| andersonvom | ^ +1 | 15:18 |
| pshchelo | and that (implicit rebase) in turn was probably done by the older version of git-review :( | 15:18 |
| jasond` | pshchelo: there's a --no-rebase option | 15:20 |
| pshchelo | I'll try to describe my steps again, so that I have a clearer picture. I had (as I know now) git-review v1.12. I followed "Gerrit Workflow" page and issued "git review -d XXXXX", "git checkout -b mytopic", inmtroduced my changes, ... | 15:22 |
| pshchelo | that "git commit -a", "git review -D" | 15:22 |
| jasond` | pshchelo: did it ask you to resolve a merge conflict? | 15:23 |
| pshchelo | at that point git review was producing an error. I updated git-review, again did "git review -D" and now we have what we have | 15:23 |
| pshchelo | jasond`: no, it did not | 15:23 |
| jasond` | was was the error? | 15:24 |
| jasond` | *what was | 15:24 |
| pshchelo | remote rejected] HEAD -> refs/draft/master/bp/cancel-update-stack (can not create new references | 15:24 |
| *** rcleere has joined #heat | 15:24 | |
| *** sergmelikyan has left #heat | 15:24 | |
| *** sergmelikyan has joined #heat | 15:25 | |
| pshchelo | that was with older git review, and probably as described in https://wiki.openstack.org/wiki/GerritWorkflow#Drafts | 15:25 |
| jasond` | pshchelo: it must auto-rebase by default. i'd use "--no-rebase" if your patch is based on someone else's change | 15:26 |
| pshchelo | jasond`: ok, very appreciate your help. the only question remains what to do with the mess I created? :( | 15:27 |
| openstackgerrit | Dmitry Shulyak proposed a change to openstack/heat: Transition to alembic migration with oslo migration tool https://review.openstack.org/63767 | 15:29 |
| jasond` | pshchelo: it might not be an inconvenience for richard (especially since his patch was in "ready for review"). i'd just let him know | 15:30 |
| andersonvom | pshchelo, jasond`: yeah, we may have to resubmit those again, since now all the reviews have been cleared | 15:31 |
| pshchelo | andersonvom: can you remove DRAFT form the last patch-set I've so dumbly introduced? | 15:31 |
| pshchelo | s/form/from/ | 15:31 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to HOT spec and parameter schema https://review.openstack.org/62418 | 15:31 |
| *** topol has joined #heat | 15:35 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add policy enforcement to ReST API https://review.openstack.org/63185 | 15:37 |
| pshchelo | andersonvom: or even better erase the whole patchset? | 15:38 |
| andersonvom | pshchelo: I don't have permissions to do that, however, I'm trying to change something in my patch to resubmit again. | 15:39 |
| andersonvom | pshchelo: if that works, you can just leave your patch as is (maybe comment on it saying it should be disregarded) and it will eventually be abandoned by gerrit | 15:39 |
| andersonvom | pshchelo: or maybe shardy or stevebaker will know a better way of getting rid of it | 15:40 |
| andersonvom | ;) | 15:40 |
| pshchelo | andersonvom: strangely neither I have such permissions (on your code branch) ... who should have them then?.. | 15:40 |
| therve | There is an "Abandon change" button in gerrit | 15:41 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 15:42 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 15:42 | |
| *** _SergeyLukjanov has quit IRC | 15:43 | |
| pshchelo | I'm writing a very self-deprecating letter to Richard now.. sigh... | 15:43 |
| andersonvom | pshchelo: you can try using what therve suggested and see if it will fix things | 15:44 |
| therve | I'm not sure what problems you're trying to fix | 15:46 |
| *** aignatov has quit IRC | 15:46 | |
| pshchelo | nope, it does not change anything in other people branches | 15:47 |
| andersonvom | therve: pshchelo submitted a draft based on someone else's patch, which in turn made all dependent patches drafts as well | 15:47 |
| pshchelo | yes, that | 15:48 |
| therve | Ah, that's unfortunate | 15:48 |
| therve | pshchelo, Maybe retry submitting the patch not drafted | 15:48 |
| andersonvom | pshchelo: I was able to resubmit mine and get it back to its review state. Couldn't do anything about richard's though. he's gonna have to do it himself next year | 15:50 |
| *** pafuent1 has quit IRC | 15:50 | |
| pshchelo | andersonvom: if I try to do what therve says, would it affect yours as well? again? | 15:51 |
| pshchelo | i'm really nervous now.. | 15:52 |
| andersonvom | pshchelo: it probably will, since I just noticed it also changed the topic branches as well... I can't get any worse, though, so you might as well try | 15:54 |
| andersonvom | pshchelo: no worries, it happens to the best of us! ;) | 15:55 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update https://review.openstack.org/63779 | 15:55 |
| pshchelo | ^done | 15:55 |
| pshchelo | let's see | 15:55 |
| *** vkozhukalov has quit IRC | 15:57 | |
| andersonvom | pshchelo: yeah... it didn't change a thing =\ | 16:01 |
| *** topol has quit IRC | 16:01 | |
| pshchelo | may be because they diverged already due to your last patch-set, cause in depends-on mine says [OUTDATED] | 16:01 |
| andersonvom | try clicking rebase from your patch | 16:02 |
| *** SergeyLukjanov has joined #heat | 16:03 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update https://review.openstack.org/63779 | 16:04 |
| *** tomek_adamczewsk has quit IRC | 16:08 | |
| *** tomek_adamczewsk has joined #heat | 16:09 | |
| andersonvom | oh well... I guess richard is going to have to resubmit his as well. | 16:11 |
| pshchelo | andersonvom, it made no difference. can I technically checkout richard's patches, make some cosmetic changes myself and submit it back to this change-id? | 16:12 |
| pshchelo | just to remove the draft status.. | 16:12 |
| *** ccs has joined #heat | 16:13 | |
| jasond` | pshchelo: i think you can do that | 16:13 |
| *** pfreund has joined #heat | 16:17 | |
| andersonvom | pshchelo: other than rebasing, you shouldn't have permissions to change someone else's patch, but you can try... | 16:17 |
| andersonvom | since we're on the subject, jasond` or therve, does any of you know if there's a way of making a change depend on two different patches? | 16:19 |
| shardy | andersonvom: I don't think that's true, you can push patches to any review by including the relevant change id | 16:19 |
| shardy | andersonvom: If someone does that by mistake, the original owner will have to resubmit | 16:19 |
| jasond` | andersonvom: re: depending on two patches. that would be the equivalent of a merge commit | 16:20 |
| andersonvom | shardy: really? huh... I think I thought that because I have gotten a permission error in the past. must've been something else | 16:20 |
| shardy | andersonvom: I don't think you can depend on two patches, you should structure your changes so they can be submitted in order | 16:20 |
| andersonvom | shardy: trying to organize things here to submit another one for the management api based on your patch ;) | 16:22 |
| andersonvom | thanks | 16:22 |
| jasond` | andersonvom: can you make shardy's patch the first dependency in the series? | 16:23 |
| andersonvom | jasond`: yeah, that's what I'll do | 16:24 |
| shardy | andersonvom: I've just reworked the tests based on the review comments so hopefully should be ready to merge soon :) | 16:26 |
| andersonvom | \o/ | 16:26 |
| shardy | andersonvom: Are you going with a new policy rule which sets a tenant override flag in the context? | 16:26 |
| shardy | that was the approach I had in mind | 16:27 |
| *** akuznetsov has quit IRC | 16:27 | |
| *** pafuent has joined #heat | 16:27 | |
| *** SergeyLukjanov has quit IRC | 16:28 | |
| andersonvom | shardy: yeah, that's what I understood from your patch: policy rule that checks for role and tenant. then on the new wrapper, check for that policy rule, if it passes, set the tenant_safe flag to False and decorate engine methods with a @tenant_safe that will look for tenant_safe and set a tenant_id filter appropriately | 16:29 |
| shardy | andersonvom: Ok, cool, sounds good | 16:30 |
| andersonvom | shardy: question, though, should we be using that "context_is_admin" rule for this, or should I create a whole different rule for checking role & tenant? | 16:31 |
| shardy | andersonvom: definitely not context_is_admin IMO, as that is for tenant admins | 16:32 |
| shardy | andersonvom: we need a completely separate rule and flag, which allows a small subset of actions to defeat tenant scoping | 16:33 |
| andersonvom | shardy: oh, ok... I wasn't sure what that was for. thanks | 16:33 |
| shardy | andersonvom: Currently, context_is_admin isn't used, but in all other projects, it's used for tenant-scoped admin tasks | 16:33 |
| *** swygue has quit IRC | 16:33 | |
| shardy | and users/deployers might want to define a custom policy which did things like e.g allow only tenant admins to create stacks or something | 16:34 |
| shardy | we want the policy implementation to be flexible but non prescriptive IMO (although with some sane defaults in our provided policy) | 16:35 |
| *** swygue has joined #heat | 16:35 | |
| *** pafuent has left #heat | 16:36 | |
| *** pafuent has joined #heat | 16:37 | |
| andersonvom | shardy: sounds good to me. | 16:37 |
| *** pafuent has quit IRC | 16:48 | |
| *** IlyaE has joined #heat | 16:48 | |
| *** marouni has joined #heat | 16:50 | |
| marouni | Hello guys | 16:51 |
| marouni | I'm getting : Unknown resource Type : OS::Heat::ResourceGroup | 16:52 |
| marouni | with heat OpenStack Havana | 16:52 |
| marouni | Is it supported in Havana ? | 16:52 |
| shardy | marouni: No, it's a new feature added after Havana was released | 16:53 |
| marouni | So no way to create resource groups in heat Havana ? | 16:54 |
| *** pafuent has joined #heat | 16:54 | |
| shardy | marouni: Not via a resource definition, but you have the option of generating a template which contains multiple resource definitions | 16:55 |
| marouni | I see :) | 16:55 |
| *** IlyaE has quit IRC | 16:55 | |
| shardy | marouni: The workaround some folks are using is simply a wrapper script which mechanically generates the template (ie a for loop around the resource you want a group of) | 16:56 |
| shardy | marouni: If it's a fixed number of resources you need, you could also define a provider (nested stack) template, and map that to OS::Heat::ResourceGroup via the environment file | 16:56 |
| shardy | (providers/environments are supported in Havana) | 16:57 |
| marouni | shardy: I check that | 16:57 |
| marouni | shardy: Thanks for help | 16:59 |
| *** marouni has left #heat | 17:00 | |
| shardy | marouni: np, see this blog post if you need an introduction to providers/environments (also links most of the related docs): | 17:00 |
| shardy | http://hardysteven.blogspot.co.uk/2013/10/heat-providersenvironments-101-ive.html | 17:00 |
| *** dmueller_ has joined #heat | 17:00 | |
| *** Alienyyg_ has quit IRC | 17:00 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Add way to group exceptions in DependencyTaskGroup https://review.openstack.org/62479 | 17:01 |
| pshchelo | andersonvom: looks like I was able to remove the DRAFT status from first of Richard's changes, but you have to rebase now | 17:05 |
| *** pfreund has quit IRC | 17:07 | |
| *** amarouni has joined #heat | 17:07 | |
| *** amarouni has left #heat | 17:08 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 17:08 |
| andersonvom | pshchelo: good. thanks | 17:08 |
| *** tomek_adamczewsk has quit IRC | 17:11 | |
| *** tomek_adamczewsk has joined #heat | 17:12 | |
| *** derekh has quit IRC | 17:16 | |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Add way to group exceptions in DependencyTaskGroup https://review.openstack.org/62479 | 17:16 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Refactor Instance#_check_active https://review.openstack.org/62478 | 17:16 |
| *** AndrewMichalik has quit IRC | 17:18 | |
| *** bvandenh has quit IRC | 17:18 | |
| *** dmueller_ has quit IRC | 17:21 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 17:26 |
| *** skraynev has quit IRC | 17:27 | |
| *** metabro has quit IRC | 17:27 | |
| *** obondarev has quit IRC | 17:27 | |
| *** skraynev has joined #heat | 17:28 | |
| *** metabro has joined #heat | 17:28 | |
| pshchelo | so, looks the issue I've introduced is resolved (at least partially). Anderson, Richard, again my deep apologies. | 17:28 |
| *** obondarev has joined #heat | 17:30 | |
| *** shakayumi has joined #heat | 17:30 | |
| andersonvom | pshchelo: cool. no worries. =) | 17:33 |
| *** IlyaE has joined #heat | 17:33 | |
| pshchelo | andersonvom: :) | 17:34 |
| *** agileclipse has quit IRC | 17:37 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update https://review.openstack.org/63779 | 17:39 |
| *** akuznetsov has joined #heat | 17:39 | |
| *** amrit has quit IRC | 17:43 | |
| *** aignatov has joined #heat | 17:44 | |
| *** aignatov has quit IRC | 17:44 | |
| *** harlowja has joined #heat | 17:50 | |
| *** SergeyLukjanov has joined #heat | 17:52 | |
| *** AndrewMichalik has joined #heat | 17:57 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Fix comparison with singletons https://review.openstack.org/62852 | 18:05 |
| *** aignatov has joined #heat | 18:05 | |
| *** AndrewMichalik has quit IRC | 18:08 | |
| *** shakayumi has quit IRC | 18:09 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Fix comparison with singletons https://review.openstack.org/62852 | 18:11 |
| *** shakayumi has joined #heat | 18:12 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Fix comparison with singletons https://review.openstack.org/62852 | 18:14 |
| *** vkozhukalov has joined #heat | 18:15 | |
| *** shakayumi has quit IRC | 18:16 | |
| *** AndrewMichalik has joined #heat | 18:22 | |
| *** ccs has quit IRC | 18:27 | |
| *** shakayumi has joined #heat | 18:30 | |
| *** aignatov has quit IRC | 18:31 | |
| *** gokrokve_ has quit IRC | 18:41 | |
| *** gokrokve has joined #heat | 18:42 | |
| *** akuznetsov has quit IRC | 18:42 | |
| *** pshchelo has quit IRC | 18:44 | |
| openstackgerrit | A change was merged to openstack/heat: rackspace: Convert resources to new Schema format https://review.openstack.org/61834 | 18:46 |
| *** gokrokve has quit IRC | 18:46 | |
| openstackgerrit | A change was merged to openstack/heat: rackspace: Convert CloudLoadBalancer to new Schema format https://review.openstack.org/61828 | 18:46 |
| *** dmueller_ has joined #heat | 19:09 | |
| *** IlyaE has quit IRC | 19:18 | |
| *** DaveJ__ has joined #heat | 19:19 | |
| *** pscheie_ has joined #heat | 19:20 | |
| *** ccs has joined #heat | 19:22 | |
| *** swygue has quit IRC | 19:26 | |
| *** jpeeler has quit IRC | 19:26 | |
| *** erkules has quit IRC | 19:26 | |
| *** DaveJ_ has quit IRC | 19:26 | |
| *** brint has quit IRC | 19:26 | |
| *** pscheie has quit IRC | 19:26 | |
| *** xuan has quit IRC | 19:26 | |
| *** dmueller_ is now known as dmueller | 19:26 | |
| *** DaveJ__ is now known as DaveJ_ | 19:26 | |
| *** brint has joined #heat | 19:27 | |
| *** andersonvom has quit IRC | 19:32 | |
| *** xuan has joined #heat | 19:32 | |
| *** erkules has joined #heat | 19:33 | |
| *** andersonvom has joined #heat | 19:33 | |
| *** swygue has joined #heat | 19:33 | |
| *** jpeeler has joined #heat | 19:34 | |
| *** jpeeler has quit IRC | 19:34 | |
| *** jpeeler has joined #heat | 19:34 | |
| pafuent | Hi everyone. Can somebody take a look at these patches? | 19:37 |
| pafuent | https://review.openstack.org/#/c/60607/ | 19:37 |
| pafuent | https://review.openstack.org/#/c/62446/ | 19:37 |
| pafuent | https://review.openstack.org/#/c/62752/ | 19:37 |
| *** IlyaE has joined #heat | 19:38 | |
| *** AndrewMichalik has quit IRC | 19:41 | |
| *** alexheneveld has quit IRC | 19:43 | |
| *** dmueller has quit IRC | 19:44 | |
| *** rcleere has quit IRC | 20:04 | |
| *** ccs has quit IRC | 20:11 | |
| *** ccs has joined #heat | 20:11 | |
| *** ccs has quit IRC | 20:11 | |
| *** ccs has joined #heat | 20:12 | |
| *** ccs has quit IRC | 20:13 | |
| *** ccs has joined #heat | 20:13 | |
| *** ccs has quit IRC | 20:13 | |
| *** ccs has joined #heat | 20:14 | |
| *** ccs has quit IRC | 20:14 | |
| *** bgorski has quit IRC | 20:14 | |
| *** ccs has joined #heat | 20:15 | |
| *** ccs has quit IRC | 20:15 | |
| *** pafuent has quit IRC | 20:44 | |
| *** pafuent has joined #heat | 20:57 | |
| *** asalkeld has quit IRC | 21:03 | |
| *** pafuent has left #heat | 21:06 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Store AccessKey secret_key in resource data https://review.openstack.org/63829 | 21:12 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient revise get_ec2_keypair https://review.openstack.org/62441 | 21:12 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Fix SignalResponder signature response when deleted https://review.openstack.org/62987 | 21:14 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add test coverage for SignalResponder delete https://review.openstack.org/62988 | 21:14 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Fix SignalResponder signature response when deleted https://review.openstack.org/62987 | 21:18 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add test coverage for SignalResponder delete https://review.openstack.org/62988 | 21:18 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Don't provide default values to properties.get() https://review.openstack.org/62760 | 21:25 |
| *** vkozhukalov has quit IRC | 21:35 | |
| *** mestery has joined #heat | 21:50 | |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack/heat: Rename id to event_uuid and id as auto_incremented https://review.openstack.org/63839 | 21:58 |
| *** shardy is now known as shardy_afk | 22:04 | |
| *** rpothier has quit IRC | 22:05 | |
| *** shakayumi has quit IRC | 22:11 | |
| *** che-arne has joined #heat | 22:24 | |
| *** jasond` has quit IRC | 22:35 | |
| chmouel | config/check_uptodate.sh not working on macosx is a bit annoying :( | 22:43 |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack/heat: Rename id to event_uuid and id as auto_incremented https://review.openstack.org/63839 | 22:43 |
| *** jasond` has joined #heat | 22:51 | |
| *** AndrewMichalik has joined #heat | 22:58 | |
| *** alexheneveld has joined #heat | 23:01 | |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack/heat: Rename id to event_uuid and id as auto_incremented https://review.openstack.org/63839 | 23:03 |
| *** alexheneveld has quit IRC | 23:10 | |
| *** tomek_adamczewsk has quit IRC | 23:11 | |
| *** asalkeld has joined #heat | 23:16 | |
| *** andersonvom1 has joined #heat | 23:18 | |
| *** andersonvom is now known as Guest30544 | 23:18 | |
| *** andersonvom1 is now known as andersonvom | 23:18 | |
| *** swygue has quit IRC | 23:20 | |
| *** AndrewMichalik has quit IRC | 23:20 | |
| *** swygue has joined #heat | 23:21 | |
| *** Guest30544 has quit IRC | 23:21 | |
| *** harlowja_ has joined #heat | 23:29 | |
| *** harlowja has quit IRC | 23:29 | |
| *** rpothier has joined #heat | 23:30 | |
| *** nijaba has quit IRC | 23:45 | |
| *** nijaba has joined #heat | 23:45 | |
| *** nijaba has quit IRC | 23:45 | |
| *** nijaba has joined #heat | 23:45 | |
| *** SergeyLukjanov has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!