miarmak | Qiming, zaneb, stevebaker so, thank you guys! | 00:01 |
---|---|---|
*** Qiming has quit IRC | 00:01 | |
asalkeld | zaneb, this is create_with_template ... https://review.openstack.org/#/c/86228 | 00:01 |
asalkeld | ok - don't worry | 00:02 |
asalkeld | I understand your confusion;) | 00:03 |
*** IlyaE has joined #heat | 00:05 | |
openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Pass the parent's registry into child stacks https://review.openstack.org/86228 | 00:11 |
*** randallburt has joined #heat | 00:13 | |
*** rpothier has joined #heat | 00:15 | |
zaneb | asalkeld: I suspect that's a bug in preview though | 00:18 |
asalkeld | yeah - I wasn't aware of preview, it's a fancier "validate"? | 00:20 |
asalkeld | not sure why we don't just improve validate | 00:20 |
asalkeld | all these actions | 00:20 |
*** matsuhashi has joined #heat | 00:22 | |
asalkeld | stevebaker, are you busy hacking up clients.py? should I ditch my review? | 00:22 |
asalkeld | I don't have a strong feeling about this patch just trying to get rid of that bug | 00:23 |
stevebaker | asalkeld: its next on my TODO when I have a decent uninterrupted block | 00:28 |
asalkeld | k | 00:31 |
*** gokrokve has joined #heat | 00:38 | |
*** gokrokve has quit IRC | 00:42 | |
*** ramishra has joined #heat | 00:45 | |
*** abramley has joined #heat | 00:48 | |
*** nati_ueno has quit IRC | 00:50 | |
*** randallburt has quit IRC | 00:51 | |
*** zz_pablosan is now known as pablosan | 00:55 | |
*** piyush has joined #heat | 00:56 | |
*** TravT has quit IRC | 00:56 | |
*** Qiming has joined #heat | 00:59 | |
*** miarmak has quit IRC | 00:59 | |
*** Qiming_ has joined #heat | 01:00 | |
*** Qiming has quit IRC | 01:04 | |
*** pablosan has quit IRC | 01:04 | |
*** pablosan has joined #heat | 01:11 | |
*** ericgoncz has joined #heat | 01:11 | |
*** shakamunyi has quit IRC | 01:16 | |
*** vpanizo has quit IRC | 01:18 | |
*** nati_ueno has joined #heat | 01:21 | |
tiantian | morning all:) | 01:22 |
*** randallburt has joined #heat | 01:23 | |
*** randallburt has quit IRC | 01:23 | |
*** randallburt has joined #heat | 01:23 | |
mattoliverau | moring tiantian | 01:24 |
*** piyush has quit IRC | 01:26 | |
*** piyush has joined #heat | 01:28 | |
*** ramishra has quit IRC | 01:28 | |
*** piyush has left #heat | 01:29 | |
*** nati_ueno has quit IRC | 01:32 | |
*** asalkeld has quit IRC | 01:38 | |
openstackgerrit | A change was merged to openstack/heat: Add Boolean parameter type https://review.openstack.org/92920 | 01:39 |
*** vpanizo has joined #heat | 01:41 | |
*** aweiteka has joined #heat | 01:41 | |
*** nati_ueno has joined #heat | 01:42 | |
*** nati_ueno has quit IRC | 01:42 | |
*** nosnos has joined #heat | 01:45 | |
*** zz_gondoi is now known as gondoi | 01:45 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add database model to store quotas https://review.openstack.org/95633 | 01:47 |
*** ramishra has joined #heat | 01:48 | |
*** gokrokve has joined #heat | 01:51 | |
*** pablosan is now known as zz_pablosan | 02:06 | |
*** asalkeld has joined #heat | 02:07 | |
*** andersonvom has joined #heat | 02:23 | |
*** dims has quit IRC | 02:29 | |
*** aweiteka has quit IRC | 02:30 | |
*** zhiyan_ is now known as zhiyan | 02:34 | |
openstackgerrit | William C. Arnold proposed a change to openstack/heat: Implements stack lifecycle plugpoints https://review.openstack.org/89363 | 02:37 |
*** piyush has joined #heat | 02:39 | |
elynn | morning all! | 02:41 |
*** gondoi is now known as zz_gondoi | 02:41 | |
*** gokrokve_ has joined #heat | 02:42 | |
*** ericgoncz has quit IRC | 02:43 | |
*** gokrokve has quit IRC | 02:45 | |
*** IlyaE has quit IRC | 02:48 | |
*** edmund has joined #heat | 02:51 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Account for differences in Rackspace Cloud Glance https://review.openstack.org/96337 | 02:52 |
*** matsuhashi has quit IRC | 03:00 | |
*** matsuhashi has joined #heat | 03:04 | |
*** dockthis has joined #heat | 03:05 | |
*** IlyaE has joined #heat | 03:05 | |
*** ramishra has quit IRC | 03:12 | |
*** piyush has quit IRC | 03:20 | |
*** gokrokve_ has quit IRC | 03:25 | |
*** nosnos has quit IRC | 03:33 | |
*** matsuhashi has quit IRC | 03:34 | |
*** matsuhashi has joined #heat | 03:35 | |
*** daneyon has joined #heat | 03:36 | |
*** matsuhashi has quit IRC | 03:39 | |
*** daneyon has quit IRC | 03:41 | |
*** dockthis has quit IRC | 03:42 | |
*** asalkeld has quit IRC | 03:47 | |
*** zhiyan is now known as zhiyan_ | 03:47 | |
*** asalkeld has joined #heat | 03:48 | |
*** shakamunyi has joined #heat | 03:49 | |
*** SnowDust has quit IRC | 03:53 | |
*** lazy_prince has joined #heat | 03:54 | |
*** edmund has quit IRC | 03:55 | |
*** matsuhashi has joined #heat | 03:56 | |
*** daneyon has joined #heat | 03:56 | |
*** daneyon has quit IRC | 04:02 | |
*** shakamunyi has quit IRC | 04:05 | |
*** SnowDust has joined #heat | 04:05 | |
*** SnowDust has quit IRC | 04:08 | |
*** zhiyan_ is now known as zhiyan | 04:10 | |
*** akuznetsov has joined #heat | 04:11 | |
*** renlt has joined #heat | 04:18 | |
*** Qiming_ has quit IRC | 04:23 | |
*** asalkeld has quit IRC | 04:30 | |
*** lazy_prince has quit IRC | 04:32 | |
*** ramishra has joined #heat | 04:36 | |
*** harlowja_ is now known as harlowja_away | 04:38 | |
*** Qiming has joined #heat | 04:43 | |
*** nosnos has joined #heat | 04:45 | |
*** sab has joined #heat | 04:52 | |
*** zhiyan is now known as zhiyan_ | 04:52 | |
*** andersonvom has quit IRC | 04:53 | |
*** nkhare has joined #heat | 04:57 | |
*** xiej has joined #heat | 04:57 | |
*** nkhare_ has joined #heat | 04:58 | |
*** xiej has quit IRC | 04:58 | |
*** sab_ has joined #heat | 05:01 | |
*** shakamunyi has joined #heat | 05:03 | |
*** nkhare_ has quit IRC | 05:04 | |
*** sab has quit IRC | 05:04 | |
*** lazy_prince has joined #heat | 05:05 | |
elynn | Hi all | 05:05 |
elynn | I have a question about disable_rollback option. | 05:05 |
elynn | How can I pass this disable_rollback option to stack? | 05:06 |
*** ramishra has quit IRC | 05:10 | |
*** ramishra has joined #heat | 05:12 | |
*** shakamunyi has quit IRC | 05:14 | |
*** randallburt has quit IRC | 05:14 | |
SpamapS | elynn: it is disabled by default | 05:14 |
SpamapS | elynn: you can enable rollback with the cli option --enable-rollback | 05:15 |
elynn | SpamapS, AFAIK heat stack-create with an option "-r" can enable it. | 05:15 |
elynn | SpamapS, But when I update this stack, it turns to false again and can't rollback when update fail. | 05:16 |
elynn | SpamapS, How can I enable it on update stack ? | 05:16 |
*** ramishra has quit IRC | 05:16 | |
*** asalkeld has joined #heat | 05:18 | |
*** IlyaE has quit IRC | 05:20 | |
elynn | Do we have a section in template to set disable_rollback to false? | 05:20 |
*** tiantian has quit IRC | 05:21 | |
SpamapS | elynn: no, if you didn't enable it at the time of creation, you won't get rollback | 05:21 |
elynn | I'm confuse with this patch https://review.openstack.org/#/c/79721 | 05:22 |
elynn | when I create a stack with disable_rollback set to false. | 05:22 |
elynn | But if I update this stack, disable_rollback will set to true again, right? | 05:23 |
elynn | So when this update is failed, then there is no rollback action, right? | 05:23 |
SpamapS | right | 05:23 |
elynn | since cli stack-update has no option --enable-rollback. | 05:24 |
elynn | Is it a bug or design to it? | 05:24 |
SpamapS | elynn: probably a bug | 05:26 |
SpamapS | elynn: it probably exists in the API, just not in the CLI | 05:26 |
*** yogeshmehra has joined #heat | 05:27 | |
elynn | thx SpamapS ! I think I should investigate this deeper :) | 05:27 |
*** IlyaE has joined #heat | 05:27 | |
SpamapS | elynn: looks like it is only available in the CFN API.. weird. | 05:28 |
*** zhiyan_ is now known as zhiyan | 05:29 | |
mattoliverau | Can you send in disable_rollback as a parameter when using the cli (heat stack-update -P disable_rollback=true) or false or what ever? | 05:30 |
SpamapS | mattoliverau: no | 05:30 |
elynn | I saw that it's implemented by shardy ,maybe he can help? | 05:30 |
elynn | SpamapS, I think disable_rollback should inherit from oldstack if we don't specify it when stack-update | 05:33 |
SpamapS | elynn: makes sense to me. | 05:34 |
SpamapS | elynn: report this as a bug. I think it needs better documentation as well. | 05:34 |
elynn | SpamapS, ok, reporting now. | 05:35 |
*** shakamunyi has joined #heat | 05:41 | |
*** SnowDust has joined #heat | 05:46 | |
*** shakamunyi has quit IRC | 05:50 | |
elynn | SpamapS, feel free to discuss it in https://bugs.launchpad.net/heat/+bug/1324366 | 05:53 |
uvirtbot | Launchpad bug 1324366 in heat "disable_rollback will change to true although we don't specify it when stack-update" [Undecided,New] | 05:53 |
SpamapS | elynn: thanks! | 05:54 |
elynn | : ) | 05:56 |
*** yogeshmehra has quit IRC | 06:01 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/89750 | 06:02 |
*** Zhang_ has quit IRC | 06:05 | |
*** yogeshmehra has joined #heat | 06:16 | |
Qiming | requirements.txt file says heat requires python-heatclient>0.2.3, isn't this too low a version now? | 06:19 |
Qiming | to support SoftwareConfig/SoftwareDeployment, we may need at least 0.2.9? | 06:19 |
*** jprovazn has joined #heat | 06:22 | |
*** SushillKM has joined #heat | 06:23 | |
*** SushillKM has left #heat | 06:23 | |
*** tiantian has joined #heat | 06:28 | |
*** divya has joined #heat | 06:33 | |
*** divya is now known as Guest55370 | 06:34 | |
*** Guest55370 has quit IRC | 06:35 | |
*** yogeshmehra has quit IRC | 06:36 | |
*** bappa has joined #heat | 06:37 | |
skraynev | Morning all! | 06:37 |
bappa | goodmorning!!! | 06:37 |
*** lazy_prince has quit IRC | 06:38 | |
*** lazy_prince has joined #heat | 06:40 | |
*** SnowDust has quit IRC | 06:43 | |
*** yogeshmehra has joined #heat | 06:46 | |
*** shakamunyi has joined #heat | 06:47 | |
mattoliverau | Morning skraynev and bappa | 06:48 |
skraynev | hi mattoliverau | 06:49 |
openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Adding error message for output description https://review.openstack.org/85687 | 06:52 |
*** lokesh184 has joined #heat | 06:53 | |
shardy | morning all | 06:53 |
asalkeld | hi shardy | 06:55 |
shardy | asalkeld: I spent most of yesterday looking at trusts and oauth | 06:55 |
asalkeld | power on dude! | 06:55 |
shardy | asalkeld: It actually looks like we may be able to fix bug #1317293 without moving to oauth, but I'm still working out the code | 06:56 |
uvirtbot | Launchpad bug 1317293 in solum "Heat stack update fails due to Authorization error in deployer" [High,Confirmed] https://launchpad.net/bugs/1317293 | 06:56 |
shardy | asalkeld: my assumption that you can't create a trust with a trust-scoped token was incorrect, you just can't create another token | 06:56 |
asalkeld | I see | 06:57 |
shardy | I'm just waiting on confirmation that is not a keystone bug before proposing code ;) | 06:57 |
asalkeld | so as long as you use the admin to generate it's ok? | 06:57 |
shardy | No, my tests indicate you can chain trusts as non-admin users, so they could then be consumed in both solum and heat | 06:58 |
asalkeld | shardy, maybe respond to that thread on the ml | 06:58 |
shardy | It's not the way I thought trusts were supposed to work tbh, waiting on more info from ayoung | 06:58 |
*** shakamunyi has quit IRC | 06:59 | |
shardy | asalkeld: I'm planning to but really wanted a sanity check from ayoung first | 06:59 |
asalkeld | ok | 06:59 |
asalkeld | so many terms floating around | 06:59 |
* asalkeld is getting confused | 06:59 | |
shardy | The oauth stuff is more complicated - I'm not yet sure how to get an oauth credential for delegation which doesn't expire | 06:59 |
shardy | The example from stevemar doesn't really demonstrate that, so I'll ping him later asking for help working up another example | 07:00 |
asalkeld | aparently you have to configure keystone | 07:00 |
asalkeld | to never expire them' | 07:00 |
shardy | Ugh | 07:00 |
asalkeld | which sucks | 07:00 |
shardy | I thought from my conversations at summit it was a per-access-token setting :( | 07:00 |
shardy | Also there is the issue it's disable by default in keystone, wherease trusts is enabled | 07:01 |
shardy | disabled even | 07:01 |
asalkeld | from stevemar: | 07:01 |
asalkeld | Note: Access Tokens can be used to get Keystone Tokens for as long as they live | 07:01 |
asalkeld | (which is defined in keystone.conf - able to set it to never expire) | 07:01 |
*** achampion has quit IRC | 07:02 | |
shardy | The problem with oauth is it seems completely geared around providing temporary access to things, rather than permanently delegating roles between users | 07:04 |
asalkeld | yip | 07:05 |
shardy | and I would imagine many deployers would consider non-expiring bearer tokens a security problem ;) | 07:05 |
asalkeld | that's what a trust is? | 07:05 |
shardy | asalkeld: Ok, I'll keep working on the trusts solution today, hopefully will have something posted by EOD | 07:06 |
asalkeld | cool | 07:06 |
shardy | asalkeld: no! A trust is not a bearer token | 07:06 |
shardy | It's just a relationship between users | 07:06 |
shardy | you still need credentials to consume it | 07:06 |
shardy | wherease with an oauth accesskey (and an ec2 keypair btw) you can consume them without any other credential | 07:07 |
asalkeld | i see | 07:07 |
*** bappa has quit IRC | 07:07 | |
shardy | asalkeld: Sorry it's taken a while to get started on this, been a lot of distractions with summit etc lately | 07:07 |
asalkeld | sure | 07:07 |
asalkeld | It took me a week to get over jet lag and summit'itus | 07:08 |
shardy | Yeah me too | 07:08 |
asalkeld | well it would be a win if we didn't have do oauth - less work | 07:09 |
shardy | Yup - I'm still interested in further investigation of oauth, but it would be nice to deprioritize it a bit, if we can get the existing (more well proven) trusts stuff to work | 07:10 |
*** IlyaE has quit IRC | 07:10 | |
*** vpanizo has quit IRC | 07:11 | |
*** asalkeld is now known as asalkeld_cooking | 07:12 | |
*** vpanizo has joined #heat | 07:15 | |
SpamapS | https://bugs.launchpad.net/heat/+bugs?field.tag=tempest | 07:18 |
Qiming | shardy: a quick question? | 07:25 |
shardy | Qiming: sure | 07:25 |
Qiming | requirements.txt file says heat requires python-heatclient>0.2.3, isn't this too low a version now? | 07:25 |
SpamapS | yes | 07:25 |
Qiming | any reason we still leave it as 0.2.3 there? | 07:25 |
SpamapS | 0.2.9 is needed for software config | 07:25 |
*** shakamunyi has joined #heat | 07:26 | |
shardy | Qiming: Someone forgot to update it, and the gate doesn't catch it because it always uses the latest version | 07:26 |
Qiming | I found this because our production team is packaging RPMs based on the requirements.txt file | 07:27 |
shardy | Qiming: It's a mistake, I'll propose a change now which fixes it | 07:27 |
shardy | First we'll need to get a bump into the global requirements | 07:27 |
Qiming | yep. | 07:27 |
Qiming | thanks for taking care of this | 07:28 |
*** ifarkas has joined #heat | 07:31 | |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: bump heatclient to latest version in requirements https://review.openstack.org/96380 | 07:32 |
shardy | Qiming: ^^ | 07:32 |
shardy | That references the requirements patch in Depends-On | 07:33 |
* shardy wonders if the cross-project dependency will work | 07:33 | |
*** jistr has joined #heat | 07:34 | |
*** shakamunyi has quit IRC | 07:36 | |
*** shivamshukla_ has joined #heat | 07:39 | |
mattoliverau | I'm calling it a day, night all! | 07:44 |
*** sab_ has quit IRC | 07:48 | |
*** shivamshukla_ has quit IRC | 07:48 | |
*** lazy_prince has quit IRC | 07:50 | |
*** sab has joined #heat | 07:51 | |
*** lazy_prince has joined #heat | 07:53 | |
*** asalkeld_cooking is now known as asalkeld_afk | 07:56 | |
*** matsuhashi has quit IRC | 07:58 | |
*** matsuhashi has joined #heat | 07:59 | |
*** mkollaro has joined #heat | 08:00 | |
*** matsuhas_ has joined #heat | 08:02 | |
*** shakamunyi has joined #heat | 08:03 | |
*** matsuhashi has quit IRC | 08:04 | |
*** e0ne has joined #heat | 08:12 | |
*** cdent has joined #heat | 08:12 | |
*** shakamunyi has quit IRC | 08:14 | |
openstackgerrit | xiejunan proposed a change to openstack/heat: validate a invalid number_typed parameter https://review.openstack.org/96390 | 08:14 |
*** cdent has quit IRC | 08:16 | |
*** yogeshmehra has quit IRC | 08:18 | |
openstackgerrit | liusheng proposed a change to openstack/python-heatclient: Don't allow "propagate"of heatclient logger https://review.openstack.org/96391 | 08:18 |
*** cdent has joined #heat | 08:23 | |
*** tomek_adamczewsk has joined #heat | 08:24 | |
*** derekh_ has joined #heat | 08:25 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/heat-specs: Convergence Observer Specification https://review.openstack.org/96394 | 08:29 |
*** sab has quit IRC | 08:33 | |
*** sab has joined #heat | 08:33 | |
skraynev | shardy, SpamapS, therve: could you give feedback about this https://bugs.launchpad.net/heat/+bug/1324431 | 08:33 |
uvirtbot | Launchpad bug 1324431 in heat "Parameters should validate values according to their data_type" [Undecided,New] | 08:33 |
openstackgerrit | A change was merged to openstack/heat: Fix broken link to API reference https://review.openstack.org/94238 | 08:34 |
openstackgerrit | Gauvain Pocentek proposed a change to openstack/heat: doc: generate a valid xml tree https://review.openstack.org/96398 | 08:39 |
*** shakamunyi has joined #heat | 08:41 | |
elynn | Hi SpamapS, Could you please review this patch: https://review.openstack.org/#/c/93270/ | 08:49 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add RPC methods for quota actions https://review.openstack.org/95710 | 08:52 |
*** shakamunyi has quit IRC | 08:52 | |
*** matsuhas_ has quit IRC | 08:53 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat-cfntools: Add sort_keys=True to json.dumps() https://review.openstack.org/96401 | 08:56 |
*** matsuhashi has joined #heat | 08:58 | |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat-cfntools: Add sort_keys=True to json.dumps() https://review.openstack.org/96401 | 09:01 |
openstackgerrit | Christian Berendt proposed a change to openstack/heat: all non debug log messages should be translated https://review.openstack.org/94217 | 09:06 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/heat-specs: Convergence Engine Specification https://review.openstack.org/96404 | 09:06 |
openstackgerrit | Christian Berendt proposed a change to openstack/heat: add hacking checks for translation support https://review.openstack.org/94255 | 09:09 |
openstackgerrit | Christian Berendt proposed a change to openstack/heat: all non debug log messages should be translated https://review.openstack.org/94217 | 09:09 |
*** sab_ has joined #heat | 09:09 | |
*** aniket has joined #heat | 09:10 | |
openstackgerrit | Christian Berendt proposed a change to openstack/heat: all non debug log messages should be translated https://review.openstack.org/94217 | 09:11 |
*** tspatzier has joined #heat | 09:11 | |
pas-ha | elynn, here? | 09:11 |
elynn | Hi pas-ha | 09:11 |
*** renlt has quit IRC | 09:12 | |
*** sab has quit IRC | 09:12 | |
pas-ha | was just browsing the backlog and saw your question on rollbacks and updates | 09:12 |
pas-ha | there is a patch in the review adding this switch to the cli client | 09:12 |
pas-ha | https://review.openstack.org/#/c/94004 | 09:12 |
openstackgerrit | Christian Berendt proposed a change to openstack/heat: add hacking checks for translation support https://review.openstack.org/94255 | 09:12 |
elynn | pas-ha, Thx! | 09:14 |
pas-ha | but I think you are right that we should honor the previous state of disable_rollback when updating, and use the switch only to override/ensure the value of disable_rollback | 09:17 |
pas-ha | but having it set by default in cli kind of complicates it | 09:19 |
*** shakamunyi has joined #heat | 09:19 | |
*** e0ne has quit IRC | 09:20 | |
pas-ha | on the other hand, than it is user's responsibility to realize what specific behavior (s)he wants during the action, and form the request accordingly | 09:20 |
*** e0ne has joined #heat | 09:21 | |
pas-ha | so I it would seem that simply adding this switch to update cli call is enough | 09:21 |
elynn | pas-ha, at lease we have a workaround for it. Honor previous state of disable_rollback when updating may not be easy. | 09:22 |
pas-ha | elynn, exactly | 09:22 |
elynn | : ) | 09:23 |
pas-ha | mostly because we don't have two swithches, and can not send None as disable_rollback value (== use whatever was before) | 09:23 |
pas-ha | although that might be doable... | 09:24 |
*** zhiyan is now known as zhiyan_ | 09:30 | |
*** sergmelikyan has quit IRC | 09:31 | |
*** sergmelikyan has joined #heat | 09:31 | |
*** shakamunyi has quit IRC | 09:31 | |
aniket | hello all :) | 09:32 |
*** matsuhashi has quit IRC | 09:33 | |
*** matsuhashi has joined #heat | 09:33 | |
shardy | pas-ha: default it to None and check for True/False | 09:34 |
shardy | Then if it is not None, override/update the stored value | 09:35 |
*** matsuhashi has quit IRC | 09:35 | |
*** matsuhashi has joined #heat | 09:36 | |
pas-ha | shardy, our cli code will allways provide true or false, as we have a default and a switch t use "not default" | 09:36 |
shardy | pas-ha: we should probably modify the client to pass None by default | 09:36 |
pas-ha | yes, that would be the way. and add a second swithc --disable-rollback then | 09:37 |
shardy | Or not use the getopt default and actually check if the option was passed | 09:37 |
shardy | Yeah either two switches, or one which takes an argument | 09:37 |
pas-ha | yep | 09:37 |
pas-ha | seems does not worth a bp though, right? | 09:38 |
shardy | There's a bug already isn't there? | 09:38 |
pas-ha | yes | 09:39 |
shardy | I'd just do the change as part of the bugfix | 09:39 |
pas-ha | ok | 09:39 |
shardy | and the bug can refer to both heat and python-heatclient if needed | 09:39 |
shardy | click "also affects project" | 09:39 |
openstackgerrit | Ethan Lynn proposed a change to openstack/heat-cfntools: Update sysvinit function to support ubuntu https://review.openstack.org/93270 | 09:40 |
*** Qiming has quit IRC | 09:57 | |
*** shakamunyi has joined #heat | 09:59 | |
*** nosnos has quit IRC | 10:02 | |
*** nosnos has joined #heat | 10:05 | |
*** shakamunyi has quit IRC | 10:05 | |
*** sab has joined #heat | 10:16 | |
*** sab_ has quit IRC | 10:16 | |
*** aniket has quit IRC | 10:21 | |
*** aniket has joined #heat | 10:22 | |
*** lokesh184 has quit IRC | 10:28 | |
openstackgerrit | Christian Berendt proposed a change to openstack/heat-specs: cleaning up index.rst file https://review.openstack.org/96416 | 10:29 |
*** Marek1211 has joined #heat | 10:33 | |
*** e0ne has quit IRC | 10:34 | |
*** e0ne has joined #heat | 10:34 | |
*** e0ne has quit IRC | 10:39 | |
*** matsuhashi has quit IRC | 10:41 | |
*** nosnos has quit IRC | 10:41 | |
*** vpanizo has quit IRC | 10:45 | |
*** nosnos has joined #heat | 10:45 | |
*** lokesh184 has joined #heat | 10:46 | |
*** jamie_h has joined #heat | 10:46 | |
*** nosnos has quit IRC | 10:46 | |
sab | Hi all, how can I know from a 'Instance' which compute node its running on? | 10:47 |
*** nosnos has joined #heat | 10:47 | |
*** nosnos has quit IRC | 10:48 | |
*** aniket has quit IRC | 10:49 | |
*** cdent_ has joined #heat | 10:50 | |
*** cdent has quit IRC | 10:50 | |
*** cdent_ is now known as cdent | 10:50 | |
*** nkhare has quit IRC | 10:52 | |
SpamapS | sab: as a non-admin, you can't. | 10:53 |
SpamapS | sab: as an admin you will be shown the host in 'nova show uuid' | 10:53 |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Add RPC methods for quota actions https://review.openstack.org/95710 | 10:53 |
*** sergmelikyan has quit IRC | 10:55 | |
*** sergmelikyan has joined #heat | 10:56 | |
sab | SpamapS, is there property in the Instance class which has the host name of compute node? | 10:56 |
*** e0ne has joined #heat | 11:02 | |
*** Qiming has joined #heat | 11:04 | |
*** alexheneveld has joined #heat | 11:04 | |
SpamapS | sab: it is an extension | 11:07 |
SpamapS | sab: you will only see it if you are an admin. | 11:07 |
SpamapS | sab: nova hides it from users because it is not user-consumable information. | 11:07 |
Qiming | umh ... just came across another interesting scenario about stack_user_domain | 11:14 |
SpamapS | do tell | 11:14 |
* shardy listens | 11:15 | |
Qiming | I have two stacks created one after another | 11:15 |
SpamapS | I keep forgetting to fix it so we can use the name. | 11:15 |
Qiming | each stack contains a single OS::Nova::Server, having user_data_format set to SOFTWARE_CONFIG | 11:15 |
Qiming | this will trigger two stack user creation operation | 11:15 |
Qiming | However, I have names for the two Nova servers assigned the same value | 11:16 |
*** dims has joined #heat | 11:16 | |
Qiming | then I got a keystone complain | 11:16 |
SpamapS | Qiming: interesting | 11:17 |
SpamapS | Qiming: this is suggesting that the short ID is not unique enough | 11:17 |
shardy | That's a bug, we should be sing the physical_resource_name, which includes a random suffix | 11:17 |
Qiming | because Keystone is using (stack_user_domain, resource_name) as a unique key | 11:17 |
SpamapS | ohh | 11:17 |
SpamapS | yeah physical, not resource_name | 11:17 |
Qiming | right. If I'm not assigning a name property to the Nova server resource, it will get a random string. | 11:17 |
shardy | Qiming: thanks for the info, please raise a bug an we'll look at a fix asap | 11:18 |
Qiming | If I do assign a 'name' value for the server, I have to make sure they are unique across the stack_user_domain. | 11:18 |
shardy | The name property should only affect the name passed to nova, not the internal naming used by other resources | 11:18 |
Qiming | A simple fix would be include the stack name into the user name for the server | 11:18 |
shardy | If we use physical_resource_name it will include the stack name, resource name and random suffix | 11:20 |
shardy | But probably we need to fix the way the name property is handled to not override the resource name, which should be respected from whatever the template key is | 11:20 |
shardy | either way it should be a fairly simple fix | 11:21 |
shardy | thanks for spotting it! :) | 11:21 |
Qiming | judging from stack_user.py (_create_user), we are using physical_resource_name() | 11:22 |
Qiming | however, the physical_resource_name() turned out to be the 'name' I assigned to a Nova server | 11:22 |
shardy | The server resource overrides physical_resource_name | 11:23 |
sab | SpamapS, Okay.. thank you :) | 11:23 |
shardy | that is probably the bug | 11:23 |
SpamapS | ah so it only happens with the property set. | 11:23 |
Qiming | SpamapS: right. it is a corner case. | 11:24 |
shardy | https://github.com/openstack/heat/blob/master/heat/engine/resources/server.py#L355 | 11:24 |
shardy | That looks wrong, I'm not sure why we're doing that | 11:24 |
*** piyush has joined #heat | 11:24 | |
SpamapS | shardy: agreed | 11:25 |
shardy | We probably need a new server_name function, which does a similar thing to shorten the name to the 53 character limit, without overriding the physical_resource_name function | 11:25 |
SpamapS | cd5d149a6bf9773b865ead267ddbccbe91b8dc54 | 11:26 |
*** SushillKM has joined #heat | 11:32 | |
SushillKM | hello ... | 11:33 |
SushillKM | needed help regarding how can i replace AWS::CloudFormation::Init in OS::Nova::Server | 11:33 |
SushillKM | is there anything usable in OpenStack Resource Types to be used in templates | 11:34 |
SpamapS | SushillKM: AWS::CloudFormation::Init is not a resource type | 11:35 |
SpamapS | SushillKM: it will work fine in OS::Nova::Server as long as your image has heat-cfntools installed | 11:36 |
SushillKM | agree ... | 11:36 |
SushillKM | but i do not want to use any aws specifications | 11:36 |
SushillKM | and i want to create a file in a OS::Nova::Server | 11:36 |
*** alexheneveld has quit IRC | 11:36 | |
SpamapS | SushillKM: you might just want to use OS::Heat::CloudConfig | 11:37 |
*** lokesh184 has quit IRC | 11:38 | |
SushillKM | any help ... except for http://docs.openstack.org/developer/heat/template_guide/openstack.html | 11:38 |
shardy | SushillKM: there are three ways, simple script, cloud-config passed to cloud-init, or SoftwareConfig/SoftwareDeployments | 11:38 |
SushillKM | because there are no examples | 11:38 |
shardy | https://github.com/openstack/heat-templates/blob/master/hot/F20/WordPress_Native.yaml | 11:38 |
shardy | ^^ simple script | 11:38 |
shardy | https://github.com/openstack/heat-templates/blob/master/hot/software-config/example-templates/example-cloud-init.yaml | 11:38 |
shardy | cloud config | 11:38 |
SushillKM | i dont want the simple script way also .... | 11:39 |
shardy | https://github.com/openstack/heat-templates/blob/master/hot/software-config/example-templates/example-script-template.yaml | 11:39 |
SushillKM | let me chk other one | 11:39 |
shardy | software config | 11:39 |
SushillKM | it seems cloud config thing is ok ... let me test that | 11:39 |
SushillKM | many thanks for help shardy | 11:39 |
shardy | there are various other examples in heat-templates which may also help | 11:39 |
SushillKM | :) | 11:39 |
shardy | np | 11:39 |
SushillKM | ok cool | 11:39 |
SushillKM | i wud look over them | 11:39 |
SushillKM | thanks for linking me to help bank ;) | 11:40 |
shardy | Note if you use SoftwareDeployments, you have to build an image with the correct agent/hook-script | 11:41 |
shardy | https://github.com/openstack/heat-templates/tree/master/hot/software-config/elements | 11:41 |
shardy | The CloudConfig stuff should work with just cloud-init though | 11:42 |
SushillKM | ok i chkd them ..... | 11:42 |
SushillKM | moreover it seems to be they all are helping create user-data | 11:42 |
SushillKM | is there anyways using metadata ... | 11:42 |
shardy | SoftwareDeployments use metadata, and can re-apply config on stack updates | 11:43 |
shardy | The other methods can't | 11:43 |
SushillKM | like it does for AWS::CloudFormation::Init | 11:43 |
*** rpothier has quit IRC | 11:46 | |
*** jcoufal has joined #heat | 11:50 | |
SushillKM | which one did u say about metadata @shardy | 11:52 |
shardy | SushillKM: SoftwareDeployment resources | 11:53 |
SushillKM | thank u .... | 11:54 |
*** dims has quit IRC | 12:00 | |
*** tiantian has quit IRC | 12:01 | |
*** sgordon has joined #heat | 12:05 | |
*** sgordon has quit IRC | 12:05 | |
*** sgordon has joined #heat | 12:05 | |
*** aweiteka has joined #heat | 12:14 | |
*** edmund1 has joined #heat | 12:21 | |
*** piyush has quit IRC | 12:24 | |
*** piyush has joined #heat | 12:26 | |
*** sab has quit IRC | 12:26 | |
*** rpothier has joined #heat | 12:34 | |
*** nkhare has joined #heat | 12:34 | |
tspatzier | skraynev: you're around? | 12:38 |
*** cdent has quit IRC | 12:39 | |
*** alexheneveld has joined #heat | 12:42 | |
*** blomquisg has joined #heat | 12:43 | |
*** piyush has quit IRC | 12:50 | |
*** cdent has joined #heat | 12:51 | |
skraynev | tspatzier: now - yes. | 12:52 |
tspatzier | hi skraynev | 12:53 |
skraynev | tspatzier: hi ;) | 12:53 |
tspatzier | so I am look into https://review.openstack.org/#/c/94641/5. The whole rounding stuff with numbers should work now. I only have a question again on your last comment. | 12:54 |
skraynev | tspatzier; ok. rounding works fine | 12:55 |
tspatzier | So you basically want to ensure that there can only be string values for an allowed_values constraint of Schema type String, right? E.g. ['1', 'two'] would be ok but not [1, 2] | 12:55 |
skraynev | yeap | 12:56 |
skraynev | it will be more consistent with properties behabiuor | 12:56 |
skraynev | s/behabiuor/behavior | 12:56 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient: Fix issues with token-only auth https://review.openstack.org/96452 | 12:57 |
skraynev | do not you agree? | 12:57 |
tspatzier | got it. I tried to add a check with isinstance(value, basestring) and found that this raises a couple of errors with some cloud_watch and firewall resource class | 12:57 |
shardy | asalkeld_afk, julienvey: ^^ FYI I think that should fix your Solum issues | 12:57 |
tspatzier | They include None as allowed_value which I find a bit strange. | 12:57 |
skraynev | tspatzier: wow. could you give example? | 12:58 |
skraynev | link please | 12:58 |
tspatzier | one sec | 12:58 |
tspatzier | https://github.com/openstack/heat/blob/master/heat/engine/resources/cloud_watch.py#L126 | 12:58 |
tspatzier | I would have thought that a property can be optional and ONLY if the value is set it would be validated against constraints | 12:59 |
tspatzier | seems strange to me to explicitly allow None in allowed_values | 12:59 |
skraynev | hm.. really strange | 12:59 |
tspatzier | I mean, I can work around it in the current patch, but it just feels strange | 12:59 |
skraynev | I think we should compare this with AWS yo understand where is mistake | 13:00 |
*** alexheneveld has quit IRC | 13:00 | |
skraynev | have you tried to create this resource with None value? | 13:01 |
skraynev | I just curious: does it work.. | 13:02 |
skraynev | tspatzier: have we any same resources else? | 13:04 |
tspatzier | https://github.com/openstack/heat/blob/master/heat/engine/resources/neutron/firewall.py#L272 | 13:06 |
tspatzier | I have to check if this is really necessary | 13:07 |
*** e0ne_ has joined #heat | 13:08 | |
tspatzier | skraynev: Besides that, I wonder if that additional check in the AllowedValues constraint is a bit too strict. In case of YAML, the user does not have to quote strings like in JSON, but you can write e.g. [one, two, three]. Now if you have a mix of strings and numbers, it might be overkill to require to explicitly quote numbers. | 13:10 |
*** dims has joined #heat | 13:10 | |
*** e0ne has quit IRC | 13:10 | |
*** mwheckmann has joined #heat | 13:11 | |
*** jdob has joined #heat | 13:11 | |
*** zhiyan_ is now known as zhiyan | 13:11 | |
*** SushillKM has quit IRC | 13:13 | |
*** lazy_prince has quit IRC | 13:14 | |
*** pafuent has joined #heat | 13:14 | |
tspatzier | skraynev: on the None value if FireWallRule and CloudWatch Alarm I think the property definitions should actually remove None from allow_values and set the required flag of the property to False. What do you think? | 13:16 |
*** shakamunyi has joined #heat | 13:16 | |
*** alexheneveld has joined #heat | 13:17 | |
*** achampion has joined #heat | 13:18 | |
*** shakayumi has joined #heat | 13:19 | |
*** shakamunyi has quit IRC | 13:21 | |
*** vijendar has joined #heat | 13:21 | |
skraynev | tspatzier: possibly yes it's difficult to operate allowed_values, that contains number and strings, but I have not seen such combination (or may be I forgot :)) | 13:25 |
tspatzier | skraynev: So I can add the check for basestring. I am currently trying with fixed property definitions for those two resource types I found. Their properties definition is just broken if you ask me. | 13:26 |
skraynev | tspatzier: about second.. | 13:27 |
skraynev | tspatzier: these properties are not required. | 13:28 |
skraynev | tspatzier: STOP | 13:28 |
skraynev | I was wrong | 13:28 |
tspatzier | Required defaults to False in the constructor of properties.Schema. Therefore I think that explicitly specifying None as allowed_value is not necessary. | 13:29 |
skraynev | look here https://github.com/openstack/heat/blob/master/heat/engine/properties.py#L213 | 13:29 |
*** ramishra has joined #heat | 13:29 | |
skraynev | tspatzier: yes, possibly it's bug (I mean using None in allow values), but it may be backward compatible important. | 13:31 |
skraynev | may be some folks have templates with None value (and in this case we will get error) | 13:32 |
skraynev | about validation. | 13:32 |
tspatzier | So if I fix those two property definitions (CloudWatch and FirewallRule) all tests works. But for any user templates we do not know, it might be good to be tolerant. | 13:33 |
skraynev | I suppose, that more reasonable make "soft" validation for string in such case, that user can pass None value for string format parameter | 13:33 |
*** shakayumi has quit IRC | 13:34 | |
skraynev | tspatzier: I think it should be different patch (https://review.openstack.org/#/c/94641/5 should rebased on it) | 13:34 |
skraynev | tspatzier: I am not sure what reaction will be from core team. may be allowed values should include None variable. | 13:35 |
tspatzier | I'll make the check tolerant and include None. I'll post an updated patch set soon. | 13:36 |
skraynev | tspatzier: Currently I should go home. I think we could discuss this question later with zaneb | 13:36 |
tspatzier | ... did not expect this fix to cause so much discussion ;-) | 13:36 |
tspatzier | skraynev: ok, CU later. | 13:36 |
skraynev | tspatzier: I am sorry ;) | 13:36 |
tspatzier | skraynev: np, good discussion! | 13:37 |
skraynev | tspatzier: but I think it was useful | 13:37 |
tspatzier | agree | 13:37 |
skraynev | tspatzier: yeap :) bye | 13:37 |
*** jamie_h has quit IRC | 13:38 | |
*** Qiming has quit IRC | 13:42 | |
*** zz_pablosan is now known as pablosan | 13:44 | |
*** pablosan has quit IRC | 13:44 | |
*** pablosan has joined #heat | 13:45 | |
*** gokrokve has joined #heat | 13:46 | |
*** shakayumi has joined #heat | 13:48 | |
*** IlyaE has joined #heat | 13:55 | |
*** piyush has joined #heat | 13:59 | |
*** piyush has quit IRC | 14:00 | |
openstackgerrit | Thomas Spatzier proposed a change to openstack/heat: Respect schema data type in constraint validation https://review.openstack.org/94641 | 14:01 |
*** andersonvom has joined #heat | 14:04 | |
*** aweiteka has quit IRC | 14:08 | |
openstackgerrit | lvdongbing proposed a change to openstack/heat: Fix rollback failure when updating stack https://review.openstack.org/88306 | 14:11 |
*** gokrokve has quit IRC | 14:14 | |
*** sjmc7 has joined #heat | 14:15 | |
*** ericgoncz has joined #heat | 14:21 | |
*** aweiteka has joined #heat | 14:22 | |
*** tnurlygayanov has quit IRC | 14:24 | |
*** sabeen1 has joined #heat | 14:26 | |
*** tnurlygayanov has joined #heat | 14:26 | |
openstackgerrit | Jason Dunsmore proposed a change to openstack/python-heatclient: Add option to show soft-deleted stacks in stack-list https://review.openstack.org/84520 | 14:26 |
*** ramishra has quit IRC | 14:27 | |
*** jprovazn has quit IRC | 14:28 | |
*** david-lyle has joined #heat | 14:28 | |
*** dims has quit IRC | 14:33 | |
*** ericgoncz has quit IRC | 14:35 | |
*** nati_ueno has joined #heat | 14:37 | |
*** jamie_h has joined #heat | 14:38 | |
*** piyush has joined #heat | 14:38 | |
*** ericgoncz has joined #heat | 14:40 | |
*** randallburt has joined #heat | 14:45 | |
*** andrew_plunk has joined #heat | 14:46 | |
*** gokrokve has joined #heat | 14:48 | |
*** ericgoncz has quit IRC | 14:51 | |
*** zhiyan is now known as zhiyan_ | 14:52 | |
*** gokrokve has quit IRC | 14:57 | |
*** gokrokve has joined #heat | 14:58 | |
*** ericgoncz has joined #heat | 14:59 | |
*** gokrokve has quit IRC | 15:02 | |
*** arbylee has joined #heat | 15:10 | |
*** jmckind has joined #heat | 15:11 | |
*** piyush has quit IRC | 15:12 | |
*** alexheneveld has quit IRC | 15:13 | |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Use stack lock context manager in abandon & delete https://review.openstack.org/95839 | 15:14 |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Implement context manager for stack lock https://review.openstack.org/95838 | 15:14 |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Use stack lock context manager in update_stack https://review.openstack.org/89409 | 15:14 |
*** alexheneveld has joined #heat | 15:18 | |
*** gokrokve has joined #heat | 15:19 | |
*** jergerber has joined #heat | 15:23 | |
*** nati_ueno has quit IRC | 15:27 | |
*** piyush has joined #heat | 15:28 | |
*** blamar has quit IRC | 15:29 | |
*** sjmc7_ has joined #heat | 15:31 | |
*** akuznetsov has quit IRC | 15:33 | |
*** sjmc7 has quit IRC | 15:35 | |
*** nkhare has quit IRC | 15:35 | |
*** ramishra has joined #heat | 15:38 | |
*** ramishra has quit IRC | 15:43 | |
*** piyush has quit IRC | 15:43 | |
*** piyush has joined #heat | 15:44 | |
openstackgerrit | A change was merged to openstack/heat-templates: Include selinux-permissive element for fedora https://review.openstack.org/94758 | 15:46 |
zaneb | tspatzier, skraynev: I suspect that None is in the list of allowed values because you could have a function there (e.g. get_attr or something) that returns None | 15:46 |
*** tomek_adamczewsk has quit IRC | 15:47 | |
zaneb | tspatzier, skraynev: so the correct fix is probably to always allow None when the field is optional and there is no default | 15:47 |
zaneb | and not have to list it explicitly | 15:47 |
*** zhiyan_ is now known as zhiyan | 15:47 | |
*** randallburt has quit IRC | 15:48 | |
*** ericgoncz has quit IRC | 15:49 | |
*** jcoufal has quit IRC | 15:50 | |
*** tomek_adamczewsk has joined #heat | 15:51 | |
*** kgriffs|afk is now known as kgriffs | 15:54 | |
*** gokrokve has quit IRC | 15:54 | |
*** gokrokve has joined #heat | 15:54 | |
*** gokrokve has quit IRC | 15:55 | |
*** ericgoncz has joined #heat | 15:55 | |
*** zhiyan is now known as zhiyan_ | 15:57 | |
*** ericgoncz has quit IRC | 15:57 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/96507 | 15:59 |
*** shakayumi is now known as shakamunyi | 15:59 | |
*** dmueller_ has joined #heat | 16:02 | |
*** mkollaro has quit IRC | 16:03 | |
tspatzier | hi zaneb | 16:03 |
tspatzier | so you mean, None should be allowed in an allowed_values list then? | 16:04 |
zaneb | tspatzier: I mean it should always be allowed in that case, and you shouldn't have to specify it in the list | 16:04 |
tspatzier | ... that's the way I have it in my latest patch set at https://review.openstack.org/#/c/94641/ | 16:04 |
zaneb | ok cool, I haven't actually read the comments yet | 16:05 |
zaneb | just saw the IRC scrollback | 16:05 |
*** stannie has quit IRC | 16:05 | |
tspatzier | zaneb: so that means it would be good to not have to specify it in the list, but if the value is None AND the property is not required, then just validate | 16:06 |
*** sjmc7_ is now known as sjmc7 | 16:07 | |
zaneb | if the value is None *and there is no default* and the property is not required, then just call it valid | 16:07 |
tspatzier | zaneb: yep, that sounds like it. I think the current patch should be ok and fix the bug it is supposed to fix. But it might make sense to look into some cleanup in general. | 16:08 |
*** mkollaro has joined #heat | 16:11 | |
*** kgriffs is now known as kgriffs|afk | 16:11 | |
*** nati_ueno has joined #heat | 16:12 | |
*** zhiyan_ is now known as zhiyan | 16:12 | |
*** aweiteka has quit IRC | 16:13 | |
*** dmueller_ has quit IRC | 16:15 | |
*** blamar has joined #heat | 16:16 | |
*** dmueller_ has joined #heat | 16:19 | |
*** stannie has joined #heat | 16:20 | |
*** zhiyan is now known as zhiyan_ | 16:21 | |
*** sjmc7_ has joined #heat | 16:22 | |
andrew_plunk | hey zaneb: Sorry I missed you yesterday, I am still trying to get our third party setup working. I ended up having to create a whole new testing environment which is why it has been taking so long. Hopefully I can finish by today, but I will update you on the status again tomorrow. | 16:24 |
zaneb | hey, np | 16:24 |
zaneb | cool. we had a meeting, so I am obliged to follow up on action items because protocol ;) | 16:25 |
andrew_plunk | yeah my original quote was based on not having to secure a second heat environment haha | 16:25 |
andrew_plunk | sure thing | 16:25 |
zaneb | ok, no hurry :) | 16:25 |
andrew_plunk | thanks! | 16:25 |
zaneb | just need something to report at the next meeting ;) | 16:25 |
zaneb | even if it's that there is nothing to report yet | 16:26 |
*** sjmc7 has quit IRC | 16:26 | |
andrew_plunk | haha, well I am glad to help then | 16:26 |
*** nati_ueno has quit IRC | 16:26 | |
*** aweiteka has joined #heat | 16:26 | |
*** SushillKM has joined #heat | 16:29 | |
*** dmueller_ has quit IRC | 16:29 | |
*** dmueller_ has joined #heat | 16:29 | |
*** e0ne_ has quit IRC | 16:30 | |
*** e0ne has joined #heat | 16:31 | |
*** e0ne has quit IRC | 16:35 | |
*** yogeshmehra has joined #heat | 16:40 | |
*** tomek_adamczewsk has quit IRC | 16:41 | |
*** gokrokve has joined #heat | 16:42 | |
*** dmueller_ has quit IRC | 16:45 | |
*** TravT has joined #heat | 16:50 | |
*** jistr is now known as jistr|afk | 16:56 | |
*** IlyaE has quit IRC | 16:56 | |
*** ericgoncz has joined #heat | 16:57 | |
*** shardy is now known as shardy_afk | 17:01 | |
*** harlowja_away is now known as harlowja_ | 17:02 | |
*** gokrokve_ has joined #heat | 17:07 | |
*** alexheneveld has quit IRC | 17:09 | |
*** gokrokve has quit IRC | 17:12 | |
openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Check that personality is provided before iterating https://review.openstack.org/96523 | 17:12 |
*** zhiyan_ is now known as zhiyan | 17:12 | |
*** jistr|afk has quit IRC | 17:16 | |
*** alexheneveld has joined #heat | 17:16 | |
*** derekh_ has quit IRC | 17:18 | |
*** zhiyan is now known as zhiyan_ | 17:22 | |
*** IlyaE has joined #heat | 17:22 | |
SushillKM | I am trying to use https://github.com/openstack/heat-templates/blob/master/hot/software-config/example-templates/example-cloud-init.yaml#L26-L32 as user data script | 17:23 |
SushillKM | with the help of | 17:23 |
SushillKM | user_data_format: RAW | 17:23 |
SushillKM | user_data: | 17:23 |
SushillKM | get_resource: one_init | 17:23 |
*** jergerber has quit IRC | 17:23 | |
SushillKM | but it gives me error | 17:23 |
SushillKM | and says | 17:23 |
SushillKM | user_data Value must be a string | 17:23 |
SushillKM | any suggestions | 17:24 |
*** piyush has quit IRC | 17:27 | |
*** rpothier_ has joined #heat | 17:28 | |
*** cdent has quit IRC | 17:30 | |
*** SushillKM has quit IRC | 17:30 | |
*** rpothier has quit IRC | 17:31 | |
*** swygue_ has quit IRC | 17:36 | |
*** ericgoncz_ has joined #heat | 17:36 | |
*** ericgoncz has quit IRC | 17:36 | |
*** ericgoncz_ is now known as ericgoncz | 17:36 | |
*** nati_ueno has joined #heat | 17:37 | |
openstackgerrit | A change was merged to openstack/heat: Use Emacs-friendly file variable to set file encoding https://review.openstack.org/95864 | 17:37 |
*** akuznetsov has joined #heat | 17:47 | |
*** alexheneveld has quit IRC | 17:51 | |
*** randallburt has joined #heat | 17:52 | |
jasond | can someone review this? it's been sitting around for a while. LGTM https://review.openstack.org/#/c/88636/5 | 17:52 |
*** lekha has joined #heat | 17:54 | |
*** arbylee1 has joined #heat | 17:57 | |
*** openstackgerrit has quit IRC | 17:57 | |
*** arbylee has quit IRC | 18:00 | |
*** edmund1 has quit IRC | 18:01 | |
*** edmund has joined #heat | 18:01 | |
*** edmund has quit IRC | 18:10 | |
*** zhiyan_ is now known as zhiyan | 18:13 | |
*** alexheneveld has joined #heat | 18:19 | |
*** swygue has joined #heat | 18:23 | |
*** sab has joined #heat | 18:23 | |
*** zhiyan is now known as zhiyan_ | 18:23 | |
*** IlyaE has quit IRC | 18:25 | |
*** sab has quit IRC | 18:28 | |
*** piyush has joined #heat | 18:28 | |
*** piyush1 has joined #heat | 18:30 | |
*** radez_g0n3 is now known as radez | 18:32 | |
*** piyush has quit IRC | 18:33 | |
*** IlyaE has joined #heat | 18:37 | |
*** alexheneveld has quit IRC | 18:38 | |
*** akuznetsov has quit IRC | 18:38 | |
*** akuznetsov has joined #heat | 18:42 | |
*** IlyaE has quit IRC | 18:44 | |
*** IlyaE has joined #heat | 18:44 | |
*** tspatzier has quit IRC | 18:46 | |
*** arbylee1 has quit IRC | 18:50 | |
*** arbylee has joined #heat | 18:50 | |
*** IlyaE has quit IRC | 18:50 | |
*** IlyaE has joined #heat | 18:51 | |
skraynev | zaneb: are you here? | 18:56 |
zaneb | I sure am | 18:56 |
skraynev | I wanted do discuss couple questions | 18:57 |
skraynev | 1. https://bugs.launchpad.net/heat/+bug/1324431 | 18:57 |
uvirtbot | Launchpad bug 1324431 in heat "Parameters should validate values according to their data_type" [Undecided,New] | 18:57 |
skraynev | How do you think it makes sense? | 18:58 |
skraynev | In my opinion it will useful to validate parameters data_type | 18:58 |
skraynev | zaneb: ^ | 18:59 |
zaneb | tbh I thought we already were | 18:59 |
*** akuznetsov has quit IRC | 19:00 | |
jasond | skraynev: i think that'd be good, but wouldn't we have to maintain backwards compatibility somehow? right now parameters are being cast to the correct type | 19:01 |
skraynev | zaneb: oh. ok:) I will try to move it to your expectations :) | 19:01 |
zaneb | +1 :) | 19:01 |
jasond | i' | 19:02 |
skraynev | jasond: so my vision is using it similar on properties | 19:02 |
zaneb | possibly we used to and it fell through the cracks? | 19:02 |
zaneb | I know tspatzier and I did a lot of refactoring during Icehouse | 19:02 |
skraynev | I think that all used parameters were corresponding properties types | 19:03 |
skraynev | zaneb: yeap. I think that tspatzier's work give me this idea, because he wants to fix validation for parameter constraints :) | 19:04 |
*** SushillKM has joined #heat | 19:04 | |
skraynev | 2. and second question about this: https://bugs.launchpad.net/heat/+bug/1324493 | 19:05 |
uvirtbot | Launchpad bug 1324493 in heat "preview for autoscaling group is failed, because reference is None" [Undecided,New] | 19:05 |
skraynev | zaneb: current get_resourse_by_ferid cann't work during preview :( | 19:06 |
skraynev | because several resources return None for this method and also all resources in state INIT COMPLETE. | 19:07 |
*** e0ne has joined #heat | 19:07 | |
skraynev | so it looks impossible to choose correct resource. | 19:07 |
*** openstackgerrit has joined #heat | 19:07 | |
zaneb | wth is get_resourse_by_ferid? | 19:08 |
skraynev | zaneb: https://github.com/openstack/heat/blob/master/heat/engine/parser.py#L311 | 19:09 |
zaneb | oh, refid :) | 19:09 |
skraynev | wrong name, sorry | 19:09 |
*** randallburt has quit IRC | 19:09 | |
skraynev | yeap :) | 19:09 |
zaneb | resource_by_refid is a hack and must die | 19:10 |
skraynev | he-he ;) | 19:10 |
skraynev | could you give some advices for replacing it. I may to implement it. | 19:11 |
zaneb | although we could easily solve that by returning a unique placeholder for the refid instead of None | 19:11 |
*** SushillKM has quit IRC | 19:11 | |
skraynev | for each resource? | 19:11 |
zaneb | yes | 19:12 |
*** SushillKM has joined #heat | 19:12 | |
zaneb | if self.resource_id is None: return ResourceReference(self); | 19:12 |
zaneb | or something | 19:12 |
*** akuznetsov has joined #heat | 19:13 | |
zaneb | as far as replacing it goes... every resource that uses it does so in a different, though equally broken, way | 19:13 |
*** openstackgerrit has quit IRC | 19:13 | |
zaneb | starting points: https://etherpad.openstack.org/p/icehouse-summit-heat-exorcism | 19:14 |
*** zhiyan_ is now known as zhiyan | 19:14 | |
*** SushillKM has quit IRC | 19:16 | |
skraynev | Oh. Is it all should be done? | 19:16 |
*** nati_ueno has quit IRC | 19:16 | |
skraynev | Or some parts are already resolved? | 19:18 |
zaneb | deprecate RouterGateway resource, implement router gateway propert(y|ies) <- that was done | 19:19 |
zaneb | the rest remains unresolved afaik | 19:19 |
*** gokrokve_ has quit IRC | 19:20 | |
*** akuznetsov has quit IRC | 19:21 | |
*** dims has joined #heat | 19:21 | |
skraynev | zaneb: anyway, thanks. Now I have good development plans, in short periods between reviewing :) | 19:22 |
*** zhiyan is now known as zhiyan_ | 19:24 | |
*** gokrokve has joined #heat | 19:25 | |
*** derekh_ has joined #heat | 19:27 | |
*** nati_ueno has joined #heat | 19:28 | |
*** jrist has quit IRC | 19:30 | |
zaneb | stevebaker: about? | 19:34 |
*** openstackgerrit has joined #heat | 19:35 | |
*** nati_ueno has quit IRC | 19:36 | |
*** nati_ueno has joined #heat | 19:36 | |
*** gokrokve has quit IRC | 19:40 | |
*** gokrokve has joined #heat | 19:41 | |
*** gokrokve has quit IRC | 19:45 | |
*** jrist has joined #heat | 19:45 | |
*** ifarkas has quit IRC | 19:49 | |
jpeeler | zaneb: does there exist a way to show all the dependencies each resource has? | 19:51 |
zaneb | jpeeler: I believe there is an API for that now | 19:51 |
zaneb | the horizon visualisation stuff uses it | 19:52 |
*** randallburt has joined #heat | 19:52 | |
jpeeler | hrm, okay. that would be useful to have in the client i would think as well | 19:53 |
*** nati_ueno has quit IRC | 19:54 | |
zaneb | I'm looking for the api and I can't find it | 19:54 |
zaneb | so it may turn out to have been a figment of my imagination | 19:54 |
jpeeler | ah that's too bad | 19:54 |
*** ericgoncz has quit IRC | 19:55 | |
*** andrew_plunk has quit IRC | 19:56 | |
*** nati_ueno has joined #heat | 19:57 | |
*** gokrokve has joined #heat | 20:00 | |
*** nati_ueno has quit IRC | 20:01 | |
*** nati_ueno has joined #heat | 20:03 | |
*** andrew_plunk has joined #heat | 20:06 | |
*** tomek_adamczewsk has joined #heat | 20:11 | |
*** achampio1 has joined #heat | 20:13 | |
*** sgordon_ has joined #heat | 20:14 | |
*** zhiyan_ is now known as zhiyan | 20:15 | |
*** nati_ueno has quit IRC | 20:15 | |
*** achampion has quit IRC | 20:16 | |
*** sgordon has quit IRC | 20:17 | |
*** IlyaE has quit IRC | 20:17 | |
*** IlyaE has joined #heat | 20:18 | |
-openstackstatus- NOTICE: Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC. | 20:18 | |
*** ChanServ changes topic to "Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC." | 20:18 | |
*** derekh_ has quit IRC | 20:19 | |
*** kgriffs|afk is now known as kgriffs | 20:20 | |
*** jergerber has joined #heat | 20:22 | |
*** tomek_adamczewsk has quit IRC | 20:24 | |
*** tomek_adamczewsk has joined #heat | 20:24 | |
*** edmund has joined #heat | 20:25 | |
*** openstackgerrit has quit IRC | 20:25 | |
*** Marek1211 has quit IRC | 20:25 | |
*** zhiyan is now known as zhiyan_ | 20:26 | |
*** blomquisg has quit IRC | 20:32 | |
*** gokrokve has quit IRC | 20:35 | |
*** ChanServ changes topic to "support @ https://ask.openstack.org | developer wiki @ https://wiki.openstack.org/wiki/Heat | development @ https://launchpad.net/heat | logged @ http://eavesdrop.openstack.org/irclogs/%23heat/" | 20:37 | |
-openstackstatus- NOTICE: Gerrit is back online | 20:37 | |
*** ericgoncz has joined #heat | 20:39 | |
*** radez is now known as radez_g0n3 | 20:39 | |
*** tomek_adamczewsk has quit IRC | 20:41 | |
*** tomek_adamczewsk has joined #heat | 20:43 | |
*** Marek1211 has joined #heat | 20:45 | |
*** gokrokve has joined #heat | 20:47 | |
*** tomek_adamczewsk has quit IRC | 20:50 | |
*** gokrokve has quit IRC | 20:50 | |
*** edmund has quit IRC | 20:51 | |
*** edmund1 has joined #heat | 20:51 | |
*** shakamunyi has quit IRC | 20:51 | |
*** rpothier_ has quit IRC | 20:51 | |
*** jmckind has quit IRC | 20:52 | |
stevebaker | zaneb: hi | 20:53 |
*** shakamunyi has joined #heat | 20:53 | |
*** tomek_adamczewsk has joined #heat | 20:53 | |
zaneb | stevebaker: apparently the 2014.1.1 release is today | 20:53 |
zaneb | how many of those patches were you expecting to land? | 20:53 |
zaneb | there seems to be a lot | 20:53 |
*** jdob has quit IRC | 20:54 | |
zaneb | and most of them failed pep8 | 20:54 |
*** asalkeld has joined #heat | 20:54 | |
*** asalkeld_afk has quit IRC | 20:54 | |
*** gokrokve has joined #heat | 20:58 | |
stevebaker | zaneb: bug/1306743 is all performance optimisation, so they can wait. All the others are clean cherry-picks of bug fixes so it would be nice if they could land | 21:05 |
stevebaker | https://review.openstack.org/#/c/95924/ | 21:05 |
stevebaker | https://review.openstack.org/#/c/95923/ | 21:05 |
stevebaker | https://review.openstack.org/#/c/95922/ | 21:05 |
stevebaker | https://review.openstack.org/#/c/95921/ | 21:05 |
stevebaker | https://review.openstack.org/#/c/95919/ | 21:05 |
*** denis_makogon has joined #heat | 21:06 | |
*** gokrokve has quit IRC | 21:06 | |
*** Marek1211 has quit IRC | 21:13 | |
*** shakamunyi has quit IRC | 21:14 | |
*** denis_makogon has left #heat | 21:17 | |
*** zhiyan_ is now known as zhiyan | 21:18 | |
zaneb | stevebaker: ok, I +2/+A'd a bunch, and +2'd the rest | 21:18 |
zaneb | what's the appropriate time delay to apply to the guy who is treating my email as his own personal template typo checker? | 21:21 |
stevebaker | zaneb: send him to the openstack list | 21:22 |
*** vpanizo has joined #heat | 21:22 | |
asalkeld | ask.openstack | 21:25 |
*** zhiyan is now known as zhiyan_ | 21:27 | |
*** pafuent has left #heat | 21:29 | |
*** nati_ueno has joined #heat | 21:31 | |
*** openstackgerrit has joined #heat | 21:34 | |
*** gokrokve has joined #heat | 21:37 | |
*** kgriffs is now known as kgriffs|afk | 21:37 | |
*** tomek_adamczewsk has quit IRC | 21:38 | |
*** gokrokve_ has joined #heat | 21:38 | |
*** piyush1 has quit IRC | 21:40 | |
*** gokrokve has quit IRC | 21:41 | |
*** gokrokve_ has quit IRC | 21:43 | |
*** arbylee1 has joined #heat | 21:46 | |
*** aweiteka has quit IRC | 21:47 | |
*** openstackgerrit has quit IRC | 21:47 | |
*** vijendar has quit IRC | 21:49 | |
*** arbylee has quit IRC | 21:49 | |
zaneb | stevebaker, asalkeld: "You're likely to get a faster response in a public forum. $YOUR_COMPANY made $28.6B last year, and if y'all need my services as a typo-checker then I want a cut :)" | 21:52 |
asalkeld | owch | 21:52 |
zaneb | too much? | 21:52 |
asalkeld | it should suffice ;) | 21:53 |
zaneb | this is about the 5th time he has emailed me privately with questions | 21:53 |
*** jamie_h has quit IRC | 21:53 | |
*** sjmc7_ is now known as sjmc7 | 21:53 | |
sjmc7 | name and shame! | 21:53 |
zaneb | sjmc7: lmgtfy | 21:54 |
asalkeld | zaneb, you see there is a usecase for a template builder | 21:54 |
asalkeld | :-O | 21:54 |
zaneb | asalkeld: that use case could be covered equally well be a template visualiser | 21:55 |
zaneb | or just a template validator | 21:55 |
zaneb | or, in this case, learning to read | 21:55 |
asalkeld | lol | 21:56 |
*** IlyaE has quit IRC | 21:57 | |
*** arbylee has joined #heat | 21:58 | |
*** IlyaE has joined #heat | 21:58 | |
*** dims has quit IRC | 21:58 | |
*** arbylee1 has quit IRC | 21:58 | |
*** gokrokve has joined #heat | 22:01 | |
*** jergerber has quit IRC | 22:02 | |
*** rpothier has joined #heat | 22:04 | |
lifeless | zaneb: 'Some asshole is sending mail from your address and being a real leech. Attached is the original to help you track them down. Kind regards, Zane.' | 22:04 |
zaneb | lifeless: nice, but too late. I already sent the one above ;) | 22:05 |
*** gokrokve has quit IRC | 22:06 | |
*** andersonvom has quit IRC | 22:07 | |
mattoliverau | Morning all | 22:11 |
*** shakamunyi has joined #heat | 22:11 | |
lifeless | zaneb: heh, yeah I just read the browns letter :0 - it was fresh in mind | 22:15 |
*** andrew_plunk has quit IRC | 22:18 | |
*** IlyaE has quit IRC | 22:18 | |
*** IlyaE has joined #heat | 22:18 | |
*** zhiyan_ is now known as zhiyan | 22:18 | |
*** lubuntu_Marek has joined #heat | 22:20 | |
*** pablosan is now known as zz_pablosan | 22:20 | |
*** Marek1211 has joined #heat | 22:21 | |
*** shakamunyi has quit IRC | 22:22 | |
*** nati_ueno has quit IRC | 22:22 | |
*** shakayumi has joined #heat | 22:22 | |
*** piyush has joined #heat | 22:23 | |
*** randallburt has quit IRC | 22:23 | |
*** piyush has quit IRC | 22:23 | |
*** dims has joined #heat | 22:24 | |
*** ericgoncz has quit IRC | 22:24 | |
*** zhiyan is now known as zhiyan_ | 22:27 | |
*** rbuilta has joined #heat | 22:31 | |
*** gokrokve has joined #heat | 22:35 | |
*** sjmc7_ has joined #heat | 22:38 | |
lifeless | stevebaker: is your api memoisation stuff in trunk ? | 22:38 |
*** Qiming has joined #heat | 22:40 | |
*** sjmc7 has quit IRC | 22:41 | |
*** arbylee has quit IRC | 22:43 | |
*** rbuilta has quit IRC | 22:44 | |
*** mwheckmann has quit IRC | 22:47 | |
*** IlyaE has quit IRC | 22:48 | |
*** arbylee has joined #heat | 22:53 | |
*** IlyaE has joined #heat | 22:54 | |
*** alexheneveld has joined #heat | 22:55 | |
*** nati_ueno has joined #heat | 22:56 | |
*** zz_pablosan is now known as pablosan | 22:59 | |
*** achampio1 has quit IRC | 23:09 | |
*** derekh_ has joined #heat | 23:09 | |
*** alexheneveld has quit IRC | 23:16 | |
*** yogeshmehra has quit IRC | 23:17 | |
*** yogeshmehra has joined #heat | 23:17 | |
*** david-lyle has quit IRC | 23:17 | |
*** zhiyan_ is now known as zhiyan | 23:19 | |
*** yogeshmehra has quit IRC | 23:22 | |
*** mkollaro has quit IRC | 23:28 | |
*** zhiyan is now known as zhiyan_ | 23:28 | |
*** arbylee has quit IRC | 23:29 | |
*** asalkeld has quit IRC | 23:30 | |
*** alexheneveld has joined #heat | 23:41 | |
*** asalkeld has joined #heat | 23:42 | |
*** derekh_ has quit IRC | 23:44 | |
*** miarmak has joined #heat | 23:52 | |
*** gokrokve has quit IRC | 23:54 | |
*** gokrokve has joined #heat | 23:55 | |
*** sjmc7_ has quit IRC | 23:55 | |
*** gokrokve has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!