*** Liang__ has joined #openstack-glance | 00:44 | |
*** Liang__ is now known as LiangFang | 00:50 | |
*** irclogbot_1 has quit IRC | 00:55 | |
*** cburgess has quit IRC | 02:22 | |
*** trident has quit IRC | 02:25 | |
*** cburgess has joined #openstack-glance | 02:26 | |
*** trident has joined #openstack-glance | 02:30 | |
*** edmondsw has quit IRC | 02:56 | |
*** ratailor has joined #openstack-glance | 03:03 | |
*** udesale has joined #openstack-glance | 04:21 | |
*** udesale has quit IRC | 04:21 | |
*** udesale has joined #openstack-glance | 04:37 | |
*** abhishekk has joined #openstack-glance | 05:07 | |
*** jmlowe has quit IRC | 05:41 | |
LiangFang | abhishekk: are you around? | 06:21 |
---|---|---|
LiangFang | I take a look of the code regarding extra_properties check | 06:23 |
abhishekk | LiangFang, yep | 06:24 |
LiangFang | currently oslo.policy don't support operators such as '<' '>' | 06:24 |
LiangFang | only support == | 06:24 |
abhishekk | yes | 06:24 |
abhishekk | == and != i guess | 06:25 |
LiangFang | yes | 06:25 |
LiangFang | so if I want to check an extra_property aaa, want check aaa>100, currently cannot do this | 06:25 |
abhishekk | right | 06:26 |
abhishekk | I guess you can override policy rule somehow | 06:27 |
LiangFang | do you mean in oslo.policy? | 06:27 |
abhishekk | not sure if it is a good idea | 06:28 |
LiangFang | for sure this is the right way to solve this issue | 06:29 |
abhishekk | I guess you can subclass a BaseCheck class and write your own rule | 06:29 |
LiangFang | currently oslo.policy is not strong enough | 06:29 |
abhishekk | or you need to add this support in oslo.policy first | 06:29 |
LiangFang | do you mean add a subclass in glance? I don't know if Jokke and rosmaita will accept this or not | 06:31 |
abhishekk | LiangFang, that is really a bad idea ( I am also doubtful over that) | 06:32 |
LiangFang | add in oslo.policy may be better | 06:32 |
abhishekk | yes | 06:32 |
abhishekk | also, you need to submit a spec for that in oslo.policy i guess | 06:33 |
LiangFang | I think so | 06:33 |
*** mosulica has joined #openstack-glance | 06:36 | |
*** ratailor_ has joined #openstack-glance | 06:39 | |
*** ratailor has quit IRC | 06:41 | |
LiangFang | abhishekk: glance do nothing upon extra_properties, just save them to DB, right? | 06:46 |
LiangFang | so different projects may save different project specific extra_property to glance, right? | 06:47 |
abhishekk | yes | 06:49 |
LiangFang | https://review.openstack.org/#/c/616873/ | 06:49 |
abhishekk | for example nova or cinder might use these properties for scheduling the instance or volume on specific nodes | 06:49 |
openstackgerrit | Merged openstack/glance master: Embed validation data in locations https://review.openstack.org/597368 | 06:50 |
LiangFang | so, glance really don't care about the validation of extra_property | 06:51 |
openstackgerrit | Abhishek Kekane proposed openstack/glance master: Add location to default backend if backend is not provided https://review.openstack.org/617229 | 06:52 |
LiangFang | it's property owner's responsiblity to valid extra_property | 06:52 |
abhishekk | as of now it just uses policies for this purpose | 06:52 |
LiangFang | ok | 06:53 |
LiangFang | so I may need to abandon this patch: https://review.openstack.org/#/c/616873/ | 06:55 |
LiangFang | could you please give comment there? | 06:55 |
LiangFang | thanks | 06:55 |
LiangFang | abhishekk: background, I'm now trying to upstream patches which contributed by our partners | 06:58 |
abhishekk | LiangFang, I will in some time | 07:02 |
*** hoonetorg has quit IRC | 07:03 | |
LiangFang | abhishekk: thanks | 07:04 |
LiangFang | this is about the topic we just discussed | 07:05 |
abhishekk | LiangFang, ack and done | 07:06 |
LiangFang | abhishekk: thanks | 07:13 |
abhishekk | LiangFang, always welcome | 07:13 |
*** LiangFang has quit IRC | 07:16 | |
*** Liang__ has joined #openstack-glance | 07:20 | |
*** Luzi has joined #openstack-glance | 07:20 | |
*** hoonetorg has joined #openstack-glance | 07:20 | |
*** Liang__ is now known as LiangFang | 07:21 | |
*** mosulica has quit IRC | 08:10 | |
*** lbragstad has joined #openstack-glance | 08:18 | |
*** mosulica has joined #openstack-glance | 08:35 | |
*** hoonetorg has quit IRC | 08:38 | |
*** pcaruana has quit IRC | 08:39 | |
*** ratailor_ has quit IRC | 08:43 | |
*** LiangFang has quit IRC | 09:48 | |
*** udesale has quit IRC | 10:16 | |
*** udesale has joined #openstack-glance | 10:32 | |
*** abhishekk has quit IRC | 10:36 | |
*** poojajadhav has joined #openstack-glance | 10:45 | |
*** udesale has quit IRC | 10:54 | |
*** lbragstad has quit IRC | 11:09 | |
*** poojajadhav has quit IRC | 11:36 | |
*** mosulica has quit IRC | 11:51 | |
openstackgerrit | guotao proposed openstack/glance-specs master: Update http to https https://review.openstack.org/617299 | 11:53 |
*** mosulica has joined #openstack-glance | 11:59 | |
*** lbragstad has joined #openstack-glance | 12:07 | |
*** mosulica has quit IRC | 12:13 | |
openstackgerrit | rongyaozu proposed openstack/python-glanceclient master: update the links to https https://review.openstack.org/617303 | 12:15 |
openstackgerrit | rongyaozu proposed openstack/glance-specs master: update the links to https https://review.openstack.org/617304 | 12:22 |
*** mvkr has quit IRC | 12:54 | |
*** mosulica has joined #openstack-glance | 13:10 | |
*** rosmaita has joined #openstack-glance | 13:10 | |
*** rosmaita has quit IRC | 13:19 | |
*** rosmaita has joined #openstack-glance | 13:31 | |
*** mvkr has joined #openstack-glance | 13:38 | |
*** mhen has joined #openstack-glance | 13:48 | |
*** mosulica has quit IRC | 13:52 | |
*** mosulica has joined #openstack-glance | 13:55 | |
*** mosulica has quit IRC | 13:56 | |
*** MattMan_ has quit IRC | 14:04 | |
*** MattMan_ has joined #openstack-glance | 14:04 | |
*** ThomasWhite has joined #openstack-glance | 14:28 | |
*** Luzi has quit IRC | 14:53 | |
*** mhen has quit IRC | 15:05 | |
*** jmlowe has joined #openstack-glance | 15:37 | |
*** jmlowe has quit IRC | 15:42 | |
*** Liang__ has joined #openstack-glance | 15:49 | |
*** lbragstad has quit IRC | 16:07 | |
*** Liang__ has quit IRC | 16:35 | |
*** gyee has joined #openstack-glance | 16:40 | |
*** lbragstad has joined #openstack-glance | 16:58 | |
*** pcaruana has joined #openstack-glance | 17:27 | |
*** lbragstad has quit IRC | 17:36 | |
*** _alastor_ has joined #openstack-glance | 18:22 | |
*** jmlowe has joined #openstack-glance | 18:32 | |
*** jmlowe has quit IRC | 18:55 | |
*** jmlowe has joined #openstack-glance | 18:57 | |
*** jmlowe has quit IRC | 19:04 | |
*** zul has joined #openstack-glance | 19:38 | |
*** mvkr has quit IRC | 19:53 | |
*** hoonetorg has joined #openstack-glance | 20:27 | |
*** mvkr has joined #openstack-glance | 20:28 | |
*** mattoliverau has quit IRC | 21:16 | |
*** lbragstad has joined #openstack-glance | 21:19 | |
*** itlinux has joined #openstack-glance | 21:31 | |
*** itlinux has quit IRC | 22:12 | |
*** mattoliverau has joined #openstack-glance | 22:26 | |
*** imacdonn_ has quit IRC | 22:40 | |
*** imacdonn_ has joined #openstack-glance | 22:40 | |
*** _alastor_ has quit IRC | 22:49 | |
*** lbragstad has quit IRC | 22:57 | |
*** pcaruana has quit IRC | 22:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!