| *** hongbin has quit IRC | 00:10 | |
| *** catintheroof has quit IRC | 00:14 | |
| *** catintheroof has joined #openstack-containers | 00:15 | |
| *** EricGonczer_ has quit IRC | 00:16 | |
| *** EricGonczer_ has joined #openstack-containers | 00:18 | |
| *** absubram has quit IRC | 00:18 | |
| *** catintheroof has quit IRC | 00:19 | |
| *** FengShengqin has joined #openstack-containers | 00:26 | |
| *** FengShengqin has quit IRC | 00:26 | |
| *** jwcroppe has joined #openstack-containers | 00:29 | |
| *** jwcroppe has quit IRC | 00:30 | |
| *** jwcroppe has joined #openstack-containers | 00:30 | |
| *** chetna has quit IRC | 00:36 | |
| *** hieulq has quit IRC | 00:49 | |
| *** hieulq has joined #openstack-containers | 00:52 | |
| *** syed_ has quit IRC | 01:15 | |
| *** vijendar has joined #openstack-containers | 01:36 | |
| *** EricGonczer_ has quit IRC | 01:40 | |
| *** chetna has joined #openstack-containers | 01:56 | |
| *** chetna has quit IRC | 01:57 | |
| *** tovin07_ has joined #openstack-containers | 02:06 | |
| *** tovin07 has joined #openstack-containers | 02:15 | |
| *** david-lyle has joined #openstack-containers | 02:22 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 02:24 | |
| *** chetna has joined #openstack-containers | 02:26 | |
| *** chetna has quit IRC | 02:28 | |
| *** Drago has quit IRC | 02:30 | |
| *** EricGonczer_ has joined #openstack-containers | 02:31 | |
| *** tkarke has quit IRC | 02:38 | |
| *** chetna has joined #openstack-containers | 02:45 | |
| *** vijendar has quit IRC | 02:54 | |
| *** EricGonczer_ has quit IRC | 03:27 | |
| *** adrian_otto has joined #openstack-containers | 03:32 | |
| *** EricGonczer_ has joined #openstack-containers | 03:32 | |
| *** EricGonczer_ has quit IRC | 03:35 | |
| *** dave-mccowan has quit IRC | 03:38 | |
| *** hongbin has joined #openstack-containers | 03:41 | |
| *** houming has joined #openstack-containers | 03:41 | |
| *** adrian_otto has quit IRC | 03:46 | |
| *** hongbin has quit IRC | 03:51 | |
| *** adisky has joined #openstack-containers | 03:52 | |
| *** vmud213 has joined #openstack-containers | 04:10 | |
| *** fragatin_ has joined #openstack-containers | 04:26 | |
| *** bks has joined #openstack-containers | 04:28 | |
| *** houming has quit IRC | 04:29 | |
| *** fragatina has quit IRC | 04:29 | |
| *** GB21 has joined #openstack-containers | 04:30 | |
| *** fragatin_ has quit IRC | 04:31 | |
| *** houming has joined #openstack-containers | 04:33 | |
| *** mkrai has joined #openstack-containers | 04:37 | |
| *** adrian_otto has joined #openstack-containers | 04:44 | |
| *** chhavi has joined #openstack-containers | 04:46 | |
| *** GB21 has quit IRC | 04:49 | |
| *** GB21 has joined #openstack-containers | 05:01 | |
| openstackgerrit | yatin proposed openstack/python-magnumclient: Fix: Creation of cluster without floating ip https://review.openstack.org/395400 | 05:02 |
|---|---|---|
| *** fragatina has joined #openstack-containers | 05:03 | |
| *** adrian_otto has quit IRC | 05:06 | |
| *** adrian_otto has joined #openstack-containers | 05:07 | |
| *** fragatina has quit IRC | 05:08 | |
| *** tkarke has joined #openstack-containers | 05:08 | |
| *** adrian_otto has quit IRC | 05:12 | |
| *** adrian_otto has joined #openstack-containers | 05:15 | |
| *** snarwade has joined #openstack-containers | 05:16 | |
| *** houming has quit IRC | 05:20 | |
| *** houming has joined #openstack-containers | 05:21 | |
| *** david-lyle has quit IRC | 05:22 | |
| *** fragatina has joined #openstack-containers | 05:22 | |
| *** david-lyle has joined #openstack-containers | 05:23 | |
| *** fragatina has quit IRC | 05:27 | |
| *** adrian_otto has quit IRC | 05:29 | |
| *** adrian_otto has joined #openstack-containers | 05:33 | |
| *** adrian_otto has quit IRC | 05:35 | |
| *** adrian_otto has joined #openstack-containers | 05:36 | |
| *** fengbeihong has joined #openstack-containers | 05:41 | |
| *** GB21 has quit IRC | 05:48 | |
| *** tkarke has quit IRC | 06:00 | |
| *** GB21 has joined #openstack-containers | 06:01 | |
| *** vimal has joined #openstack-containers | 06:03 | |
| *** vimal has quit IRC | 06:05 | |
| *** vimal has joined #openstack-containers | 06:05 | |
| *** GB21 has quit IRC | 06:06 | |
| *** GB21 has joined #openstack-containers | 06:06 | |
| *** GB21 has quit IRC | 06:07 | |
| *** GB21 has joined #openstack-containers | 06:07 | |
| *** adrian_otto has quit IRC | 06:11 | |
| *** houming has quit IRC | 06:14 | |
| *** manikanta_tadi has quit IRC | 06:20 | |
| *** houming has joined #openstack-containers | 06:21 | |
| *** fengbeihong_ has joined #openstack-containers | 06:42 | |
| *** fengbeihong has quit IRC | 06:42 | |
| *** fengbeihong_ is now known as fengbeihong | 06:42 | |
| *** houming has quit IRC | 06:47 | |
| *** houming has joined #openstack-containers | 06:48 | |
| *** rcernin has joined #openstack-containers | 06:54 | |
| *** zenoway has joined #openstack-containers | 07:05 | |
| *** Jeffrey4l has joined #openstack-containers | 07:09 | |
| *** fengbeihong has quit IRC | 07:18 | |
| *** fengbeihong has joined #openstack-containers | 07:24 | |
| openstackgerrit | fengbeihong proposed openstack/magnum-ui: [WIP] Add dcos support for magnum-ui https://review.openstack.org/395437 | 07:34 |
| openstackgerrit | zhongshengping proposed openstack/magnum: Change cfg.set_defaults into cors.set_defaults https://review.openstack.org/395438 | 07:35 |
| openstackgerrit | Merged openstack/magnum-ui: Updated from global requirements https://review.openstack.org/395325 | 07:40 |
| *** belmoreira has joined #openstack-containers | 07:45 | |
| *** chetna has quit IRC | 08:02 | |
| *** dcwangmit01_ has joined #openstack-containers | 08:03 | |
| *** dcwangmit01 has quit IRC | 08:05 | |
| *** pcastets has joined #openstack-containers | 08:05 | |
| *** tkarke has joined #openstack-containers | 08:11 | |
| *** tkarke has quit IRC | 08:12 | |
| *** houming has quit IRC | 08:27 | |
| *** houming has joined #openstack-containers | 08:30 | |
| *** chetna has joined #openstack-containers | 08:32 | |
| *** chetna has quit IRC | 08:33 | |
| *** fengbeihong has quit IRC | 08:40 | |
| *** tkarke has joined #openstack-containers | 08:42 | |
| *** tkarke has quit IRC | 08:43 | |
| *** chhavi has quit IRC | 08:46 | |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** openstackgerrit has joined #openstack-containers | 08:48 | |
| *** chhavi has joined #openstack-containers | 09:02 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 09:12 | |
| *** chetna has joined #openstack-containers | 09:18 | |
| *** chetna has quit IRC | 09:19 | |
| *** dimtruck is now known as zz_dimtruck | 09:25 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 09:30 |
| *** chhavi has quit IRC | 09:39 | |
| *** GB21 has quit IRC | 09:42 | |
| yatin | strigazi, ping | 09:45 |
| strigazi | Hi yatin | 09:45 |
| yatin | strigazi: still not working | 09:46 |
| yatin | strigazi: cinder optional, have you checked creating cluster with docker_volume_size > 0 | 09:46 |
| strigazi | yatin: yes I did with 5 | 09:46 |
| yatin | strigazi: functionality wise or just cluster_complete status | 09:47 |
| strigazi | yatin, logi in the node and cat /etc/sysconfig/heat-params | 09:47 |
| yatin | pvdisplay | 09:48 |
| strigazi | with which docker_storage_driver? | 09:48 |
| yatin | devicemapper | 09:48 |
| strigazi | I did with overlay | 09:48 |
| strigazi | will do devicemapper | 09:48 |
| yatin | Ok | 09:49 |
| yatin | have you checked /etc/fstab with overlay | 09:49 |
| strigazi | have you checked /etc/sysconfig/heat-params ? Is the number passed in the node? | 09:53 |
| yatin | after the revised patch | 09:53 |
| strigazi | No, I haven't checked /etc/fstab | 09:54 |
| strigazi | yatin: have you checked /etc/sysconfig/heat-params ? Is the number passed in the node? | 09:54 |
| yatin | Checking | 09:56 |
| yatin | i deleted the cluster, creating again | 09:57 |
| strigazi | yatin: pvdisplay looks fine | 09:58 |
| yatin | Ok, i will confirm | 09:59 |
| strigazi | ytain: http://paste.openstack.org/show/588524/ | 10:02 |
| strigazi | yatin: http://paste.openstack.org/show/588524/ | 10:02 |
| *** chhavi has joined #openstack-containers | 10:02 | |
| yatin | strigazi: Yes, its created this time | 10:07 |
| strigazi | yatin: cool | 10:08 |
| *** tovin07_ has quit IRC | 10:10 | |
| *** fengbeihong has joined #openstack-containers | 10:16 | |
| *** chetna has joined #openstack-containers | 10:18 | |
| *** chetna has quit IRC | 10:19 | |
| *** vimal has quit IRC | 10:21 | |
| *** tkarke has joined #openstack-containers | 10:23 | |
| *** hvlad has joined #openstack-containers | 10:25 | |
| *** tkarke has quit IRC | 10:26 | |
| *** vimal has joined #openstack-containers | 10:28 | |
| strigazi | yatin, Is there anything else you want to test on this? | 10:29 |
| strigazi | yatin, we can do it in parallel to double check | 10:29 |
| strigazi | yatin, otherwise you can +1 :) and next week +2 :) | 10:29 |
| yatin | strigazi: currently going through the patch | 10:30 |
| yatin | :) | 10:30 |
| strigazi | yatin, Thanks | 10:30 |
| *** askb has quit IRC | 10:31 | |
| *** tonanhngo has quit IRC | 10:31 | |
| *** fengbeihong has quit IRC | 10:35 | |
| *** tkarke has joined #openstack-containers | 10:40 | |
| *** fengbeihong has joined #openstack-containers | 10:41 | |
| *** tkarke has quit IRC | 10:42 | |
| *** chetna has joined #openstack-containers | 10:49 | |
| *** chetna has quit IRC | 10:50 | |
| *** fengbeihong has quit IRC | 10:51 | |
| *** snarwade has quit IRC | 10:55 | |
| *** DanyC has joined #openstack-containers | 11:02 | |
| *** jperry has joined #openstack-containers | 11:21 | |
| *** tonanhngo has joined #openstack-containers | 11:38 | |
| *** tonanhngo has quit IRC | 11:40 | |
| *** snarwade has joined #openstack-containers | 11:41 | |
| *** hvlad has quit IRC | 11:46 | |
| *** jvgrant_ has joined #openstack-containers | 11:50 | |
| *** wangbo has joined #openstack-containers | 11:52 | |
| *** jvgrant__ has quit IRC | 11:53 | |
| wangbo | hello, is there some configure like api-workers to run multiple magnum-api processes? thx | 11:57 |
| *** chhavi has quit IRC | 12:09 | |
| *** DanyC has left #openstack-containers | 12:09 | |
| *** hvlad has joined #openstack-containers | 12:10 | |
| *** catintheroof has joined #openstack-containers | 12:16 | |
| *** mkrai has quit IRC | 12:20 | |
| *** catinthe_ has joined #openstack-containers | 12:20 | |
| *** catintheroof has quit IRC | 12:21 | |
| *** chhavi has joined #openstack-containers | 12:24 | |
| *** EricGonczer_ has joined #openstack-containers | 12:34 | |
| *** tkarke has joined #openstack-containers | 12:51 | |
| *** tkarke has quit IRC | 12:54 | |
| *** GB21 has joined #openstack-containers | 12:55 | |
| *** EricGonczer_ has quit IRC | 12:59 | |
| *** chhavi has quit IRC | 13:03 | |
| *** tkarke has joined #openstack-containers | 13:08 | |
| *** tkarke has quit IRC | 13:09 | |
| *** vmud213 has quit IRC | 13:12 | |
| *** jwcroppe has quit IRC | 13:16 | |
| yatin | wangbo: As far as i know there is no such config, can anyone else confirm. Can you share the use case where you want to add this or you faced some issue. | 13:17 |
| *** chetna has joined #openstack-containers | 13:19 | |
| *** chhavi has joined #openstack-containers | 13:20 | |
| *** chetna has quit IRC | 13:21 | |
| *** houming has quit IRC | 13:27 | |
| *** bks has quit IRC | 13:28 | |
| wangbo | yatin, we tried to integrate harbor into magnum. there are some synchronize calls between magnum-conductor and harbor server and these api calls take long time. | 13:29 |
| wangbo | these calls occupied one magnum-conductor fully | 13:30 |
| wangbo | such as image-upload | 13:30 |
| *** EricGonczer_ has joined #openstack-containers | 13:31 | |
| wangbo | so I want to find way to run multiple processes. | 13:31 |
| *** EricGonczer_ has quit IRC | 13:32 | |
| *** dave-mccowan has joined #openstack-containers | 13:34 | |
| *** tonanhngo has joined #openstack-containers | 13:35 | |
| *** tonanhngo has quit IRC | 13:36 | |
| wangbo | yatin: another problem is that all magnum-api rpc to magnum-conductor use synchronize calls. so we need multiple magnum-api processes. | 13:37 |
| *** tonanhngo has joined #openstack-containers | 13:39 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 13:47 |
| *** catintheroof has joined #openstack-containers | 13:47 | |
| *** catinthe_ has quit IRC | 13:50 | |
| *** jwcroppe has joined #openstack-containers | 13:51 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: typo: Fix in docker storage configuration https://review.openstack.org/395605 | 13:54 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 13:56 |
| *** _mbelanger has joined #openstack-containers | 13:58 | |
| *** GB21 has quit IRC | 13:59 | |
| *** jperry has quit IRC | 14:03 | |
| yatin | wangbo: If this is the case, this can be discussed with the Team | 14:05 |
| *** jperry has joined #openstack-containers | 14:10 | |
| yatin | wangbo: can you share some data and scenario(your use case) via bug/etherpad that specifies long delays so it can be better discussed within Team and solution can be proposed or may be there exist some alternative to this that can be used. | 14:10 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 14:21 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 14:21 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Convert floating_ip_enabled from boolean to enum https://review.openstack.org/395095 | 14:26 |
| *** tkarke has joined #openstack-containers | 14:29 | |
| *** tkarke has quit IRC | 14:30 | |
| *** kaliya has joined #openstack-containers | 14:31 | |
| *** jvgrant__ has joined #openstack-containers | 14:31 | |
| *** yuanying_ has joined #openstack-containers | 14:32 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Convert floating_ip_enabled from boolean to enum https://review.openstack.org/395095 | 14:33 |
| *** jvgrant_ has quit IRC | 14:34 | |
| *** jasond has joined #openstack-containers | 14:34 | |
| *** _mbelanger has quit IRC | 14:38 | |
| *** Drago has joined #openstack-containers | 14:40 | |
| *** chhavi has quit IRC | 14:43 | |
| *** Drago has quit IRC | 14:45 | |
| *** Drago has joined #openstack-containers | 14:49 | |
| *** yuanying_ has quit IRC | 14:51 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Convert floating_ip_enabled from boolean to enum https://review.openstack.org/395095 | 14:52 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Convert floating_ip_enabled from boolean to enum https://review.openstack.org/395095 | 14:53 |
| *** hongbin has joined #openstack-containers | 15:01 | |
| *** adrian_otto has joined #openstack-containers | 15:13 | |
| *** vimal has quit IRC | 15:19 | |
| *** syed__ has joined #openstack-containers | 15:24 | |
| *** randallburt has joined #openstack-containers | 15:25 | |
| *** randallburt has quit IRC | 15:30 | |
| *** pcastets has quit IRC | 15:32 | |
| *** _mbelanger has joined #openstack-containers | 15:32 | |
| *** chris_hultin|AWA is now known as chris_hultin | 15:32 | |
| *** zz_dimtruck is now known as dimtruck | 15:33 | |
| *** absubram has joined #openstack-containers | 15:33 | |
| *** absubram_ has joined #openstack-containers | 15:34 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum-specs: Initial commit https://review.openstack.org/395673 | 15:34 |
| *** chetna has joined #openstack-containers | 15:35 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum-specs: Import all implemented specs from magnum repo https://review.openstack.org/395674 | 15:35 |
| *** chetna has quit IRC | 15:36 | |
| *** randallburt has joined #openstack-containers | 15:36 | |
| *** absubram has quit IRC | 15:38 | |
| *** absubram_ is now known as absubram | 15:38 | |
| *** vimal has joined #openstack-containers | 15:50 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 15:50 |
| *** tonanhngo_ has joined #openstack-containers | 15:51 | |
| *** tonanhngo has quit IRC | 15:51 | |
| *** hvlad has quit IRC | 15:54 | |
| *** pcastets has joined #openstack-containers | 15:55 | |
| *** tonanhngo_ has quit IRC | 15:55 | |
| *** vimal has quit IRC | 15:57 | |
| *** jerrygb has joined #openstack-containers | 15:57 | |
| *** vimal has joined #openstack-containers | 16:01 | |
| *** vimal has quit IRC | 16:04 | |
| openstackgerrit | Randall Burt proposed openstack/magnum: Add cluster driver encapsulation spec https://review.openstack.org/389835 | 16:04 |
| *** vijendar_ has joined #openstack-containers | 16:06 | |
| *** belmoreira has quit IRC | 16:11 | |
| *** jasond has quit IRC | 16:13 | |
| *** adrian_otto has quit IRC | 16:13 | |
| *** jasond has joined #openstack-containers | 16:13 | |
| *** rcernin has quit IRC | 16:13 | |
| *** absubram has quit IRC | 16:17 | |
| *** chhavi has joined #openstack-containers | 16:18 | |
| *** ramishra has quit IRC | 16:18 | |
| *** ramishra has joined #openstack-containers | 16:19 | |
| *** chetna has joined #openstack-containers | 16:29 | |
| *** vmud213 has joined #openstack-containers | 16:32 | |
| *** EricGonczer_ has joined #openstack-containers | 16:34 | |
| *** adrian_otto has joined #openstack-containers | 16:36 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Convert floating_ip_enabled from boolean to enum https://review.openstack.org/395095 | 16:38 |
| *** strigazi is now known as strigazi_AFK | 16:39 | |
| *** EricGonczer_ has quit IRC | 16:39 | |
| *** EricGonczer_ has joined #openstack-containers | 16:40 | |
| Drago | hongbin: ping | 16:40 |
| *** fragatina has joined #openstack-containers | 16:42 | |
| hongbin | Drago: sup | 16:43 |
| Drago | hongbin: Is it only from the name and docstring that sync_state sounds like polling? | 16:44 |
| hongbin | Drago: yes | 16:44 |
| Drago | hongbin: So if it was only named/described differently (without changing the interface!), you would be in favor of it? | 16:45 |
| hongbin | Drago: well, i am in favor of #1 | 16:46 |
| hongbin | Drago: but if there is a better name & doc-string, i am open to consider that. | 16:46 |
| Drago | hongbin: I am just talking about the naming of that one method. We can talk about the other comment after | 16:46 |
| hongbin | Drago: do you have a better name? | 16:47 |
| Drago | hongbin: Well, I personally don't think that "synchronize" has anything to do with push or poll | 16:48 |
| Drago | Synchronize via push, or synchronize via poll are both valid | 16:48 |
| hongbin | i don't think it is intuitive to synchronize via push | 16:49 |
| hongbin | synchronize and push sounds opposite | 16:49 |
| Drago | Not to me | 16:49 |
| hongbin | Then, we are in a tie | 16:49 |
| Drago | hongbin: You can search on the internet and see many other services that synchronize via push | 16:51 |
| hongbin | link? | 16:52 |
| hongbin | Drago: you don't like the "wait_for_*" approach? | 16:53 |
| Drago | hongbin: The wait_for_* is something we should probably look into once we do our initial refactoring, but it is a much bigger change to do in the first iteration | 16:55 |
| hongbin | Drago: this is a spec, but an implementation. that means you can change the implenetation step-by-step, but the spec should be a final version | 16:56 |
| hongbin | or what we expect to be the final so far | 16:56 |
| hongbin | in addition, i don't think the change will be huge | 16:56 |
| hongbin | it possibly can be done with a few tweaks. | 16:57 |
| Drago | hongbin: Additionally, it should probably be called "wait_for" and you specify what state you want. Right now, HeatPoller just looks for a stable state and then quits | 16:57 |
| hongbin | then, how about naming it to "wait_for_[stable_state]" | 16:57 |
| *** vmud213 has quit IRC | 16:58 | |
| hongbin | i like that because this is the common logic in nova/heat/.... | 16:58 |
| hongbin | it looks quite easy to implement and being understand | 16:58 |
| Drago | Well, that's fine with me, because then we're back to using one method to synchronize state | 16:58 |
| Drago | hongbin: How about this: keep calling it sync_state, and in the docstring specify that it should synchronize the state until it becomes stable, whether that's polling or pushing | 16:59 |
| hongbin | i don't want to limit ourselves to what we have right now | 16:59 |
| hongbin | Drago: the name of the method is very important | 17:00 |
| hongbin | if the name is wrong, it causes a lot of confusion in the implementation. | 17:00 |
| Drago | hongbin: I argue that it does not matter what state you're waiting for, because you can always tell when you reach a stable state, and whether that state is the state you wanted is not the method's concern | 17:00 |
| Drago | hongbin: I think that wait_for sounds like it is assuming push | 17:01 |
| hongbin | then, come up with a better name that doesn't sound like pushing or polling | 17:02 |
| Drago | hongbin: Many email clients rely on push to synchronize mail https://en.wikipedia.org/wiki/Push_email | 17:02 |
| hongbin | Drago: but in nova, the implement "wait_for" by using polling..... | 17:02 |
| *** yuanying_ has joined #openstack-containers | 17:03 | |
| hongbin | Drago: but if they implement the interface in this way? | 17:03 |
| Drago | hongbin: So a developer that is familiar with nova would see wait_for and think they should implement polling? | 17:03 |
| randallburt | lol | 17:03 |
| hongbin | Drago: yes | 17:03 |
| *** zenoway has quit IRC | 17:04 | |
| Drago | hongbin: So wait_for is also a bad name. Can you come up with a better name? :) | 17:04 |
| *** zenoway has joined #openstack-containers | 17:04 | |
| hongbin | Drago: i already wrote down what I can think of in the spec | 17:04 |
| hongbin | Drago: if you have a better suggestion, please propose it. | 17:04 |
| Drago | hongbin: Thesaurus to the rescue | 17:05 |
| Drago | harmonize_state | 17:05 |
| Drago | coordinate_state | 17:05 |
| Drago | unify_state | 17:05 |
| Drago | reconcile_state | 17:05 |
| *** tonanhngo has joined #openstack-containers | 17:05 | |
| hongbin | Drago: just let other team members to comment | 17:05 |
| Drago | correlate_state | 17:05 |
| hongbin | i am going to leave the spec for a while for further inputs | 17:06 |
| hongbin | will leave for lunch for a while | 17:06 |
| Drago | hongbin: Okay, talk to you later | 17:06 |
| *** yuanying_ has quit IRC | 17:07 | |
| Drago | hongbin: Push sync in AWS http://docs.aws.amazon.com/cognito/latest/developerguide/push-sync.html | 17:07 |
| *** zenoway has quit IRC | 17:08 | |
| *** tonanhngo has quit IRC | 17:09 | |
| *** tonanhngo has joined #openstack-containers | 17:10 | |
| Drago | hongbin: How to synchronize data between front and backend? Top answer: push. http://stackoverflow.com/questions/1945305/synchronize-data-between-frontend-and-backend | 17:13 |
| *** vijendar has joined #openstack-containers | 17:14 | |
| *** chetna has quit IRC | 17:17 | |
| *** chetna has joined #openstack-containers | 17:17 | |
| Drago | hongbin: http://blog.cloud-elements.com/webhooks-vs.-polling-sync-your-apps | 17:20 |
| Drago | hongbin: "Synchronize" is not strongly correlated with push or poll, from what I can find. I think it is a fitting word. Any lingering confusion can be resolved by having a clear docstring. | 17:22 |
| *** EricGonczer_ has quit IRC | 17:47 | |
| hongbin | Drago: it is hard to see what their interfaces looks like | 17:47 |
| hongbin | Drago: it seems AWS is putting "onEven" into their interface | 17:48 |
| hongbin | Drago: then, another alternative is naming that method as "on_cluster_active" | 17:49 |
| hongbin | Drago: but i guess this is not what you like? | 17:50 |
| Drago | hongbin: We are not talking about interfaces though, only whether "synchronize" implies "poll" | 17:51 |
| hongbin | Drago: then, i will make a similar arguement about "wait_for" implies "push" | 17:52 |
| *** chhavi has quit IRC | 17:52 | |
| hongbin | Drago: so it won't lead to a conclusion to keeping arguing that | 17:52 |
| hongbin | Drago: as i said, just wait for others to chime in and comments on what they like. | 17:53 |
| *** jperry has quit IRC | 17:57 | |
| *** jperry has joined #openstack-containers | 17:58 | |
| Drago | hongbin: What is the purpose of splitting the methods up in #1? | 17:59 |
| *** tkarke has joined #openstack-containers | 18:00 | |
| hongbin | Drago: there are several advantages | 18:02 |
| hongbin | 1. the interface looks more straight forward, because the meaning of each method is well-defined by its name. | 18:03 |
| *** fragatin_ has joined #openstack-containers | 18:04 | |
| hongbin | 2. It allows the flexibility to wait for different things by using different logic | 18:04 |
| *** fragatina has quit IRC | 18:07 | |
| Drago | hongbin: I think that makes sense | 18:14 |
| Drago | hongbin: Let me ask you something slightly different | 18:14 |
| Drago | hongbin: If the spec was accepted exactly as it is right now, would there be anything preventing another spec being proposed with #1 and accepted and implemented as well? | 18:16 |
| hongbin | Drago: not sure, it might or might not | 18:17 |
| randallburt | Drago: no | 18:18 |
| Drago | hongbin: My point is, if the current spec is a step in the right direction, there's nothing stopping us from further improving the interface. | 18:18 |
| hongbin | Drago: i disagree. the current spec is not totally on the right direction | 18:18 |
| Drago | hongbin: Please explain | 18:19 |
| randallburt | hongbin: even when I made the changes you explicitly requested? | 18:19 |
| hongbin | Drago: as i said, the name of method is confusing | 18:19 |
| randallburt | hongbin: please tell me what name you'd approve of and I'll change it. | 18:19 |
| Drago | hongbin: Okay, point taken, I am more concerned with the 3 proposals you outlined | 18:20 |
| hongbin | randallburt: i listed 3 proposals i can think of in the spec | 18:20 |
| hongbin | randallburt: please wait for the feedback from community to evualate which proposal is the best, it might be your proposal | 18:21 |
| randallburt | hongbin: so you want me to totally re-write my spec based on one or all of your suggestions? | 18:21 |
| Drago | hongbin: Ignoring concerns about naming, are there other things in the spec that are not a step in the right direction? | 18:21 |
| randallburt | hongbin: with due respect, if you want to propose a competing spec, please do and cross link as alternatives | 18:21 |
| Drago | hongbin, randallburt: I am trying to see if there's anything in the current spec that would prevent later specs from refining the interface | 18:22 |
| hongbin | Drago: maybe not just the name, but the entire method: def sync_state(self, cluster) | 18:23 |
| hongbin | Drago: besides that method, it looks good. | 18:23 |
| Drago | hongbin: Okay, that is more substantive (in my opinion). What about the method would prevent further refinement in the future? | 18:23 |
| hongbin | Drago: not sure exactly | 18:25 |
| hongbin | Drago: what do you get by rushing that method into the interface? | 18:25 |
| Drago | hongbin: If you can think on that, and let me know, I would appreciate it. | 18:25 |
| Drago | hongbin: The overall goal of the spec is to refactor drivers enough to decouple Magnum from Heat. It's intent is not to come up with a final version of the driver interface | 18:26 |
| Drago | *Its | 18:26 |
| *** tkarke has quit IRC | 18:26 | |
| hongbin | ok, then you can write the spec in a different way and taking out the driver interface entirely | 18:27 |
| Drago | hongbin: Why should the driver interface be taken out of the spec? | 18:27 |
| hongbin | Drago: because you claimed what you wanted is a refactoring, then it has nothing to do with defining a driver interface | 18:28 |
| Drago | hongbin: How would you decouple from Heat without changing the driver interface? | 18:28 |
| hongbin | Drago: you can change the interface in a implementation, but it has nothing to do by defining a interface in a spec | 18:29 |
| Drago | hongbin: I think we would be having the same conversation during the implementation though | 18:31 |
| Drago | hongbin: About the changes to the interface | 18:31 |
| hongbin | Drago: that depends on specific implementation | 18:32 |
| randallburt | sorry, had to take the dog out | 18:32 |
| *** tkarke has joined #openstack-containers | 18:33 | |
| randallburt | hongbin: what we get by taking our best stab at it is progress. | 18:33 |
| randallburt | hongbin: the refactoring involves moving code around in Magnum and into the current driver interface | 18:33 |
| hongbin | randallburt: removing the method from the interface :) | 18:34 |
| hongbin | randallburt: you can add the method later if you want | 18:34 |
| randallburt | hongbin: and we save discussion on patches if we can agree on a general approach rather than going round and round forever and surprising you later | 18:34 |
| randallburt | hongbin: done | 18:35 |
| hongbin | option 1: avoid defining a driver interface in the spec | 18:35 |
| openstackgerrit | Randall Burt proposed openstack/magnum: Add cluster driver encapsulation spec https://review.openstack.org/389835 | 18:35 |
| hongbin | option 2: removing the "sync_state" method from the spec, and adding it in a separated spec | 18:36 |
| randallburt | hongbin: option 1 just leads to us having this discussion all over again when patches start being submitted | 18:36 |
| Drago | hongbin: So when you see this method introduced in a patch, because *something* is necessary to decouple from Heat, you will be… okay with it?? | 18:36 |
| hongbin | option 3: wait for the the community input to define the best driver interface. | 18:36 |
| randallburt | Drago: whatever it takes to move forward, man. rather argue later about one method than stop any work | 18:36 |
| hongbin | Drago: again, depending on what the patch looks like | 18:38 |
| hongbin | Drago: it is hard to comment on a patch that is not there | 18:38 |
| Drago | hongbin: I agree with you on that :) | 18:38 |
| randallburt | hongbin: out of curiosity, who else are we waiting on to provide input? I can add them to the review and hopefully get that going | 18:38 |
| randallburt | hongbin: or you can of course if there are specific people you're wanting to hear form | 18:38 |
| randallburt | s/form/from | 18:39 |
| hongbin | randallburt: if you want, ping all the core reviewers | 18:39 |
| randallburt | hongbin: what's the group name for that | 18:39 |
| hongbin | randallburt: and sent a ML to ask | 18:39 |
| randallburt | ? | 18:39 |
| hongbin | ?? | 18:39 |
| hongbin | frankly, i don't know why you guys are rushing this spec | 18:40 |
| randallburt | hongbin: what's the name for the magnum core reviewers? In heat, for example, its heat-core. If I add that as a reviewer, all the cores get added | 18:40 |
| randallburt | hongbin: because its a priority for us | 18:40 |
| hongbin | magnum-core | 18:40 |
| randallburt | hongbin: thanks! | 18:40 |
| *** adisky has quit IRC | 18:43 | |
| Drago | hongbin: We are trying to propose a spec that keeps disruption to a minimum. Part of that is limiting its scope. None of us think this will declare a "final" interface of the drivers. Instead, it will be one of many incremental improvements, and we expect future specs to build on top of this work. | 18:46 |
| Drago | hongbin: I think that splitting the methods as you propose could be one of those improvements | 18:47 |
| *** pcastets has quit IRC | 18:47 | |
| *** fragatin_ has quit IRC | 18:47 | |
| hongbin | Drago: ok, i see the sync_state method was removed, then it looks good to me. | 18:47 |
| Drago | Thank you, hongbin | 18:48 |
| *** fragatina has joined #openstack-containers | 18:48 | |
| *** harlowja has quit IRC | 18:59 | |
| *** rcernin has joined #openstack-containers | 19:00 | |
| *** dcwangmit01 has joined #openstack-containers | 19:08 | |
| vijendar | adrian_otto: when you get a chance, please take a look at https://review.openstack.org/#/c/395209/ | 19:08 |
| *** catinthe_ has joined #openstack-containers | 19:10 | |
| *** dcwangmit01_ has quit IRC | 19:10 | |
| *** catintheroof has quit IRC | 19:12 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: Make cinder volume optional https://review.openstack.org/391830 | 19:16 |
| *** dave-mccowan has quit IRC | 19:25 | |
| *** dave-mccowan has joined #openstack-containers | 19:26 | |
| *** adrian_otto has quit IRC | 19:31 | |
| *** dave-mccowan has quit IRC | 19:36 | |
| *** rcernin has quit IRC | 19:55 | |
| *** EricGonczer_ has joined #openstack-containers | 19:58 | |
| *** yuanying_ has joined #openstack-containers | 20:03 | |
| *** yuanying_ has quit IRC | 20:08 | |
| *** swatson_ has joined #openstack-containers | 20:12 | |
| *** chetna has quit IRC | 20:21 | |
| *** chetna has joined #openstack-containers | 20:23 | |
| *** chetna has quit IRC | 20:33 | |
| *** dave-mccowan has joined #openstack-containers | 20:38 | |
| *** vijendar has quit IRC | 20:50 | |
| *** vijendar_ has quit IRC | 20:50 | |
| *** vijendar_ has joined #openstack-containers | 20:53 | |
| *** vijendar has joined #openstack-containers | 20:53 | |
| openstackgerrit | Paul Czarkowski proposed openstack/magnum: remove extra bracket from script in docs. https://review.openstack.org/393313 | 20:56 |
| *** slagle has quit IRC | 20:57 | |
| *** adrian_otto has joined #openstack-containers | 20:58 | |
| *** askb has joined #openstack-containers | 21:03 | |
| *** EricGonczer_ has quit IRC | 21:03 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: [WIP] Convert floating_ip_enabled from boolean to enum https://review.openstack.org/395095 | 21:12 |
| *** vijendar_ has quit IRC | 21:14 | |
| *** vijendar has quit IRC | 21:14 | |
| *** randallburt has quit IRC | 21:24 | |
| *** slagle has joined #openstack-containers | 21:31 | |
| *** _mbelanger has quit IRC | 21:31 | |
| *** Jeffrey4l has quit IRC | 21:34 | |
| *** chetna has joined #openstack-containers | 21:39 | |
| *** harlowja has joined #openstack-containers | 21:49 | |
| *** kaliya has quit IRC | 21:54 | |
| *** kaliya has joined #openstack-containers | 21:55 | |
| *** jwcroppe has quit IRC | 21:57 | |
| *** jwcroppe has joined #openstack-containers | 21:58 | |
| *** jwcroppe has quit IRC | 22:03 | |
| *** yuanying_ has joined #openstack-containers | 22:46 | |
| *** yuanying_ has quit IRC | 22:50 | |
| *** jperry has quit IRC | 22:56 | |
| *** chetna has quit IRC | 22:56 | |
| *** tonanhngo has quit IRC | 22:57 | |
| *** kaliya has quit IRC | 22:57 | |
| *** adrian_otto has quit IRC | 22:57 | |
| *** chetnakh has joined #openstack-containers | 22:57 | |
| *** jasond has quit IRC | 23:02 | |
| *** catinthe_ has quit IRC | 23:04 | |
| *** harlowja_ has joined #openstack-containers | 23:08 | |
| *** harlowja has quit IRC | 23:12 | |
| *** jerrygb has quit IRC | 23:16 | |
| *** vijendar has joined #openstack-containers | 23:17 | |
| *** tonanhngo has joined #openstack-containers | 23:17 | |
| *** tonanhngo has quit IRC | 23:22 | |
| *** jerrygb has joined #openstack-containers | 23:23 | |
| *** jwcroppe has joined #openstack-containers | 23:24 | |
| *** jwcroppe has quit IRC | 23:25 | |
| *** jwcroppe has joined #openstack-containers | 23:25 | |
| *** jwcroppe has quit IRC | 23:30 | |
| *** Jeffrey4l has joined #openstack-containers | 23:31 | |
| *** Jeffrey4l has quit IRC | 23:32 | |
| *** Jeffrey4l has joined #openstack-containers | 23:32 | |
| *** Jeffrey4l has quit IRC | 23:33 | |
| *** Jeffrey4l has joined #openstack-containers | 23:34 | |
| *** chetnakh has quit IRC | 23:34 | |
| *** vijendar has quit IRC | 23:37 | |
| *** vijendar has joined #openstack-containers | 23:37 | |
| *** vijendar_ has joined #openstack-containers | 23:38 | |
| *** tonanhngo has joined #openstack-containers | 23:41 | |
| *** tonanhngo has quit IRC | 23:50 | |
| *** tonanhngo has joined #openstack-containers | 23:55 | |
| *** dfflanders has joined #openstack-containers | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!