Thursday, 2015-09-24

openstackgerritHongbin Lu proposed openstack/magnum: Implement bay monitoring and notifications  https://review.openstack.org/21533500:03
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Make bay.api_address contains protocol  https://review.openstack.org/22302500:03
*** sirushti has quit IRC00:07
*** mfalatic has quit IRC00:09
*** devkulkarni has quit IRC00:12
openstackgerritMerged openstack/magnum: doc8 all documentation and fix doc style  https://review.openstack.org/22417100:15
*** suro-patz has quit IRC00:22
*** sthillma has quit IRC00:25
*** achanda has joined #openstack-containers00:33
*** achanda has quit IRC00:38
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Enable barbican cert manager in devstack  https://review.openstack.org/22706600:52
*** annegentle has joined #openstack-containers00:54
*** annegentle has quit IRC00:59
vilobhmm11hongbin : ping01:01
*** sthillma has joined #openstack-containers01:03
*** sdake has joined #openstack-containers01:03
*** sthillma_ has joined #openstack-containers01:03
*** gangil has quit IRC01:04
*** sthillma_ has quit IRC01:07
*** sthillma has quit IRC01:07
*** sthillma has joined #openstack-containers01:10
*** hongbin has joined #openstack-containers01:14
*** eghobo has joined #openstack-containers01:17
*** devkulkarni has joined #openstack-containers01:23
*** sthillma has quit IRC01:28
*** Marga__ has joined #openstack-containers01:29
*** Marga_ has quit IRC01:32
*** Marga__ has quit IRC01:33
*** suro-patz has joined #openstack-containers01:33
*** vilobhmm11 has quit IRC01:34
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Enable barbican cert manager in devstack  https://review.openstack.org/22706601:34
*** sthillma has joined #openstack-containers01:36
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Make bay.api_address contains protocol  https://review.openstack.org/22302501:37
*** zz_dimtruck is now known as dimtruck01:37
*** eghobo_ has joined #openstack-containers01:40
*** devkulkarni has quit IRC01:41
*** zhenguo has joined #openstack-containers01:43
*** eghobo has quit IRC01:44
*** harshs has quit IRC01:46
*** suro-patz has quit IRC01:50
*** julim has joined #openstack-containers01:51
*** suro-patz has joined #openstack-containers01:51
*** sthillma has quit IRC01:53
*** erkules_ has joined #openstack-containers01:55
*** erkules has quit IRC01:58
*** achanda has joined #openstack-containers02:01
*** muralia1 has joined #openstack-containers02:06
*** muralia has quit IRC02:06
*** sdake has quit IRC02:06
*** unicell has quit IRC02:23
*** annegentle has joined #openstack-containers02:43
*** ganeshna has quit IRC02:45
*** annegentle has quit IRC02:48
openstackgerritMerged openstack/magnum: Add registry to template  https://review.openstack.org/22352602:48
*** eghobo_ has quit IRC02:56
*** harshs has joined #openstack-containers03:01
*** eghobo has joined #openstack-containers03:10
*** dimtruck is now known as zz_dimtruck03:21
*** deepthi has joined #openstack-containers03:26
*** sirushti has joined #openstack-containers03:27
*** suro-patz has quit IRC03:33
*** shuquan has joined #openstack-containers03:34
*** sirushti has quit IRC03:35
*** sirushti has joined #openstack-containers03:35
*** julim has quit IRC03:41
*** fawadkhaliq has joined #openstack-containers03:41
openstackgerritMerged openstack/magnum: Enable barbican cert manager in devstack  https://review.openstack.org/22706603:43
*** diga has joined #openstack-containers03:45
*** harshs has quit IRC03:49
*** shuquan has quit IRC03:51
*** diga has quit IRC03:51
*** ganeshna has joined #openstack-containers03:53
*** achanda has quit IRC03:55
*** shuquan has joined #openstack-containers03:56
*** sthillma has joined #openstack-containers04:01
*** muralia1 has quit IRC04:08
*** muralia has joined #openstack-containers04:08
*** diga has joined #openstack-containers04:09
*** harshs has joined #openstack-containers04:11
*** VikasC has joined #openstack-containers04:16
*** Drago has quit IRC04:21
*** jwcroppe has joined #openstack-containers04:26
*** annegentle has joined #openstack-containers04:32
*** annegentle has quit IRC04:36
*** chandankumar has joined #openstack-containers04:39
*** Marga_ has joined #openstack-containers04:42
*** Marga_ has quit IRC04:43
*** Marga_ has joined #openstack-containers04:43
*** eghobo has quit IRC04:43
*** sdake has joined #openstack-containers04:54
*** achanda has joined #openstack-containers04:55
*** ganeshna has quit IRC04:58
*** shuquan has quit IRC05:03
*** shuquan has joined #openstack-containers05:03
*** tbh has joined #openstack-containers05:08
*** fawadkhaliq has quit IRC05:10
*** ganeshna has joined #openstack-containers05:15
openstackgerritMerged openstack/python-magnumclient: Add certificates operations  https://review.openstack.org/22431005:16
*** vimal has joined #openstack-containers05:18
*** shuquan has quit IRC05:18
*** puranamr has joined #openstack-containers05:22
*** BhavyaM has joined #openstack-containers05:25
*** puranamr has quit IRC05:26
*** puranamr has joined #openstack-containers05:27
*** diga has quit IRC05:37
*** sdake has quit IRC05:40
*** Tango|2 has joined #openstack-containers05:48
*** puranamr has quit IRC05:52
*** vimal has quit IRC05:58
*** jwcroppe has quit IRC05:59
*** jwcroppe has joined #openstack-containers06:00
eliqiaohello, anyone tested swarm bay with atomic-5 image?06:02
*** jwcroppe has quit IRC06:04
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: [WIP] Add TLS support in heat kubernetes template  https://review.openstack.org/20287306:05
*** eghobo has joined #openstack-containers06:06
openstackgerritwangqun proposed openstack/magnum: Fix the problem that using unsupport coe can create the bay model Fixs Bug1497134  https://review.openstack.org/22712806:06
Tango|2Hi eliqiao, are you seeing some problems?06:14
*** Tango|2 is now known as Tango06:14
Tangoeliqiao: We did upgrade from Docker 1.3.2 to 1.7.106:16
*** vimal has joined #openstack-containers06:20
*** annegentle has joined #openstack-containers06:20
*** madhuri has joined #openstack-containers06:21
madhuriyuanying: Hi06:21
*** sthillma has quit IRC06:22
yuanyingmadhuri: hi06:22
*** chandankumar has quit IRC06:24
*** annegentle has quit IRC06:25
*** j___ has joined #openstack-containers06:26
*** suro-patz has joined #openstack-containers06:33
*** chandankumar has joined #openstack-containers06:34
*** irenab has quit IRC06:35
*** harshs has quit IRC06:42
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Move security group setting to kubecluster.yaml  https://review.openstack.org/22714506:51
*** suro-patz has quit IRC06:52
*** ganeshna has quit IRC06:52
*** Tango has quit IRC06:58
*** eghobo_ has joined #openstack-containers06:59
*** daneyon has quit IRC07:00
*** daneyon has joined #openstack-containers07:01
*** eghobo has quit IRC07:02
*** BertrandN has joined #openstack-containers07:02
*** irenab has joined #openstack-containers07:04
*** diga has joined #openstack-containers07:06
*** eghobo has joined #openstack-containers07:12
eliqiaohi Tango:  I just tested it, docker deamon can not start07:13
*** eghobo_ has quit IRC07:14
*** BertrandN has quit IRC07:16
*** alex_klimov has joined #openstack-containers07:22
*** jwcroppe has joined #openstack-containers07:28
*** achanda has quit IRC07:28
openstackgerritwangqun proposed openstack/magnum: Fix the problem that using unsupport coe can create the bay model Fixs Bug1497134 Change-Id: Icae270fef12db6912c8fb7d825872dd502462055 Closes-Bug: #1497134  https://review.openstack.org/22712807:31
openstackbug 1497134 in Magnum "Using unsupport coe can create the bay model" [Undecided,New] https://launchpad.net/bugs/1497134 - Assigned to wangqun (bjwqun)07:31
*** BertrandN has joined #openstack-containers07:33
*** eghobo has quit IRC07:34
*** xiaolidi has joined #openstack-containers07:46
eliqiaohello , anyone  can help ? I can not start docker deamon on atomic-5 image.07:48
*** ganeshna has joined #openstack-containers07:50
yuanyingAre you using swarm?07:50
openstackgerritwangqun proposed openstack/magnum: Fix the problem that using unsupport coe can create the bay model when we create the baymodel using the unsupport parameter of  coe, it can create one baymodel. Fixs Bug1497134 Change-Id: Icae270fef12db6912c8fb7d825872dd502462055 Closes-Bug: #1497134  https://review.openstack.org/22712807:51
openstackbug 1497134 in Magnum "Using unsupport coe can create the bay model" [Undecided,In progress] https://launchpad.net/bugs/1497134 - Assigned to wangqun (bjwqun)07:51
yuanyingI checked docker daemon at kubernetes bay just now, it worked07:51
eliqiaoyuanying:  yes07:54
eliqiaoyuanying:  do you konw where can I find some debuging message?07:55
eliqiaoyuanying:  #link http://paste.openstack.org/show/473833/07:55
eliqiaohi wangqun07:55
eliqiaohi yuanying: I got this error "Error starting daemon: error initializing graphdriver: Unknown option dm.fs"08:03
*** annegentle has joined #openstack-containers08:09
*** annegentle has quit IRC08:14
*** diga has quit IRC08:14
yuanyinghongbin: around?08:47
*** diga has joined #openstack-containers08:48
*** alex_klimov has quit IRC09:02
*** daneyon has quit IRC09:06
*** daneyon has joined #openstack-containers09:07
*** ganeshna has quit IRC09:18
*** ganeshna has joined #openstack-containers09:20
*** diga has quit IRC09:36
*** alex_klimov has joined #openstack-containers09:40
*** lan has joined #openstack-containers09:40
*** VikasC has quit IRC09:57
*** madhuri has quit IRC09:57
*** annegentle has joined #openstack-containers09:58
*** annegentle has quit IRC10:03
openstackgerritEli Qiao proposed openstack/magnum: Swarm: Add configure-swarm.sh to configure docker-storage  https://review.openstack.org/22721010:17
openstackgerritEli Qiao proposed openstack/magnum: [WIP Dont merge]Swarm: Add configure-swarm.sh to configure docker-storage  https://review.openstack.org/22721010:18
*** ganeshna has quit IRC10:19
*** diga has joined #openstack-containers10:34
*** ganeshna has joined #openstack-containers10:35
*** irenab has quit IRC10:37
*** erkules_ has quit IRC10:38
*** zhenguo has quit IRC10:42
*** lan has quit IRC10:49
*** julim has joined #openstack-containers10:58
*** irenab has joined #openstack-containers11:03
*** chandankumar has quit IRC11:28
*** achanda has joined #openstack-containers11:29
*** devkulkarni has joined #openstack-containers11:33
*** BhavyaM has quit IRC11:34
*** achanda has quit IRC11:41
*** ganeshna has quit IRC11:41
*** ganeshna has joined #openstack-containers11:42
*** annegentle has joined #openstack-containers11:47
*** annegentle has quit IRC11:52
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Make bay.api_address contains protocol  https://review.openstack.org/22302511:58
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: [WIP] Add TLS support in heat kubernetes template  https://review.openstack.org/20287311:58
*** ganeshna has quit IRC12:05
*** junhongl has quit IRC12:06
*** chandankumar has joined #openstack-containers12:07
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: [WIP] Add TLS support in heat kubernetes template  https://review.openstack.org/20287312:07
*** dims__ has quit IRC12:19
*** dims_ has joined #openstack-containers12:20
*** annegentle has joined #openstack-containers12:22
*** alex_klimov has quit IRC12:24
*** annegentle has quit IRC12:34
openstackgerritKevin Lefevre proposed openstack/magnum: Fix baymodel documentation  https://review.openstack.org/22727012:42
*** dane_leblanc has joined #openstack-containers12:43
*** dane_leblanc has quit IRC12:50
*** alex_klimov has joined #openstack-containers12:53
openstackgerritKevin Lefevre proposed openstack/magnum: Fix baymodel documentation  https://review.openstack.org/22727012:54
*** rpothier has joined #openstack-containers12:56
*** rlrossit has joined #openstack-containers12:56
*** dane_leblanc has joined #openstack-containers12:58
*** devkulkarni has quit IRC13:00
*** larsks has quit IRC13:07
*** diga has quit IRC13:07
*** deepthi has quit IRC13:14
*** alejandrito has joined #openstack-containers13:14
*** banix has joined #openstack-containers13:17
*** tbh has quit IRC13:26
*** banix has quit IRC13:26
*** erkules_ has joined #openstack-containers13:28
*** devkulkarni has joined #openstack-containers13:29
*** banix has joined #openstack-containers13:35
*** banix has quit IRC13:42
*** yuanying_ has joined #openstack-containers13:43
*** harshs has joined #openstack-containers13:43
*** devkulkarni has quit IRC13:49
*** zz_dimtruck is now known as dimtruck13:52
*** chandankumar has quit IRC13:55
*** achanda has joined #openstack-containers13:57
*** julim has quit IRC13:58
*** achanda has quit IRC13:59
*** vimal has quit IRC14:07
*** muralia has quit IRC14:08
*** muralia has joined #openstack-containers14:08
*** dims__ has joined #openstack-containers14:11
*** banix has joined #openstack-containers14:11
*** j___ has quit IRC14:13
*** dims_ has quit IRC14:14
*** hongbin_ has joined #openstack-containers14:20
*** Tango has joined #openstack-containers14:22
hongbin_yuanying: yuanying_ I saw you ping me before. What's up14:22
*** manjeets has joined #openstack-containers14:27
*** jwcroppe has quit IRC14:28
*** ganeshna has joined #openstack-containers14:29
*** ganeshna has quit IRC14:29
*** dane_leblanc has quit IRC14:29
*** dane_leblanc has joined #openstack-containers14:30
*** yuanying_ has quit IRC14:36
*** yuanying_ has joined #openstack-containers14:36
*** julim has joined #openstack-containers14:38
*** vahidh has quit IRC14:39
*** fawadkhaliq has joined #openstack-containers14:42
*** yuanying_ has quit IRC14:42
*** devkulkarni has joined #openstack-containers14:42
*** kebray has joined #openstack-containers14:43
*** kebray has quit IRC14:43
*** Tango|2 has joined #openstack-containers14:43
*** hongbin_ has quit IRC14:43
*** banix has quit IRC14:44
*** Tango has quit IRC14:46
*** devkulkarni has quit IRC14:46
*** dane_leblanc has quit IRC14:47
*** kebray has joined #openstack-containers14:47
*** dane_leblanc has joined #openstack-containers14:48
muraliahongbin_: the magnum api has a baymodel update method, but the CLI does not support update. do you know why? was that intentional or did we just not get around to implementing it?14:48
ArchiFleKsHi, how does bay-model create knows which heat-template to use ? is it the os_distro property ? where is it implemented ?14:50
*** klindgren has quit IRC14:50
*** klindgren has joined #openstack-containers14:51
*** larsks has joined #openstack-containers14:57
*** annegentle has joined #openstack-containers15:01
openstackgerritKevin Lefevre proposed openstack/magnum: Fix baymodel documentation  https://review.openstack.org/22727015:06
*** devkulkarni has joined #openstack-containers15:08
*** annegentle has quit IRC15:09
*** harshs has quit IRC15:09
*** annegentle has joined #openstack-containers15:09
*** hongbin_ has joined #openstack-containers15:15
*** adrian_otto has joined #openstack-containers15:16
hongbin_muralia: Yes, the baymodel-update is not implemented, I guess it is just because there is no use case for that15:19
hongbin_muralia: but I think there will be use cases to update the baymodel in future.15:21
muraliathanks. i think i need it and will work on a patch for it15:21
hongbin_ArchiFleKs: os_distro is used to select the heat template15:23
hongbin_ArchiFleKs: since there are different templates for different os (one for fedora, one for coreos)15:23
ArchiFleKsok and what about the unified templates ? (kubecluster, kubemaster, kubeminion) ?15:24
*** Drago has joined #openstack-containers15:24
hongbin_ArchiFleKs: those are for fedora-atomic15:25
hongbin_ArchiFleKs: so it is not unified. There are another set of templates for coreos15:25
*** Drago has quit IRC15:25
*** Drago has joined #openstack-containers15:25
ArchiFleKsok so kubecluster-coreos is for the master ? and kubeminion-coreos for the minion ?15:26
hongbin_yes15:26
ArchiFleKsok thanks15:27
hongbin_welcome15:27
*** harshs has joined #openstack-containers15:31
openstackgerritJames E. Blair proposed openstack/magnum: Remove ERROR_ON_CLONE references  https://review.openstack.org/22700015:32
*** suro-patz has joined #openstack-containers15:34
*** larsks has quit IRC15:37
*** larsks has joined #openstack-containers15:38
*** suro-patz has quit IRC15:42
*** puranamr has joined #openstack-containers15:43
*** harshs has quit IRC15:45
*** puranamr has quit IRC15:48
*** muralia1 has joined #openstack-containers15:51
*** muralia has quit IRC15:54
*** fawadkhaliq has quit IRC16:01
*** fawadkhaliq has joined #openstack-containers16:02
*** puranamr has joined #openstack-containers16:04
*** fawadkhaliq has quit IRC16:05
*** eghobo has joined #openstack-containers16:05
*** puranamr has quit IRC16:08
*** mfalatic has joined #openstack-containers16:09
*** kbyrne has quit IRC16:24
*** afranc202 has quit IRC16:24
*** iraw- has joined #openstack-containers16:24
*** unicell has joined #openstack-containers16:24
*** gangil has joined #openstack-containers16:24
*** kbyrne has joined #openstack-containers16:24
*** puranamr has joined #openstack-containers16:29
*** eghobo has quit IRC16:30
*** unicell has quit IRC16:35
*** eghobo has joined #openstack-containers16:35
*** eghobo has quit IRC16:37
*** banix has joined #openstack-containers16:40
*** alex_klimov has quit IRC16:47
*** eghobo has joined #openstack-containers16:49
*** suro-patz has joined #openstack-containers16:49
*** BertrandN has quit IRC16:49
*** dane_leblanc has quit IRC16:50
*** gangil1 has joined #openstack-containers16:52
*** gangil has quit IRC16:52
Tango|2daneyon: ping16:54
*** Tango|2 is now known as Tango16:54
*** achanda has joined #openstack-containers17:00
*** puranamr has quit IRC17:01
daneyonTango|2 hey17:01
TangoSo I am using the new support for labels17:02
daneyonok17:02
Tangodaneyon: I did a baymodel-create with:17:02
Tangodaneyon: magnum baymodel-create --name k8smodelv1 --image-id fedora-21-atomic-6 --keypair-id time4funkey --external-network-id $NIC_ID --dns-nameserver 8.8.8.8 --flavor-id m1.small --docker-volume-size 5 --coe kubernetes --labels network_driver=flannel17:03
Tangodaneyon: And I get the error:17:03
daneyonb/c that's an invalid label17:03
Tangodaneyon: ERROR: Cannot fulfill request with a None network_driver, expecting a flannel network_driver. (HTTP 400)17:03
daneyon1 sec17:03
daneyonthese are the only supported labels: https://github.com/openstack/magnum/blob/master/magnum/conductor/template_definition.py#L410-L41117:04
daneyonyou wanted to do this --network-driver flannel --labels flannel_network_cidr=10.1.0.0/16,flannel_network_subnetlen=26,flannel_use_vxlan=true17:05
daneyonreplace my label values with whatever you want17:05
TangoAh ok17:05
TangoI got confused then17:06
TangoSo one thing I ran into was that when I did a db upgrade to add the new column, the labels of the existing baymodel is None17:07
TangoIt should be {}17:07
TangoWhen I try to create a bay using this baymodel, I would get an error17:08
daneyonahh..17:08
Tangoin template_definition.py:42517:08
TangoProbaby some code in alembic17:09
daneyonTango so as part of the db upgrade script, do we need a func that checks for existing baymodels and adds the {}?17:09
*** unicell has joined #openstack-containers17:10
TangoI guess when it adds the new columns for labels, it should fill in as {} instead of None17:10
daneyonTango when u have a moment, do you mind filing a bug on the issue?17:10
Tangofor existing baymodel17:10
daneyonright17:11
TangoSure, good idea17:11
daneyondef did not cross my mind to check for existing baymodels17:11
daneyonbut who cares about upgrading anyways... lol!!!17:11
TangoMoving fast :)17:12
*** eghobo has quit IRC17:12
*** devkulkarni has quit IRC17:15
*** larsks has quit IRC17:18
*** devkulkarni has joined #openstack-containers17:19
*** j___ has joined #openstack-containers17:21
*** puranamr has joined #openstack-containers17:22
*** eghobo has joined #openstack-containers17:26
*** puranamr has quit IRC17:26
*** eghobo_ has joined #openstack-containers17:28
*** harshs has joined #openstack-containers17:30
*** eghobo has quit IRC17:32
*** kebray has quit IRC17:34
*** achanda has quit IRC17:36
*** adrian_otto has quit IRC17:39
*** adrian_otto has joined #openstack-containers17:39
*** vilobhmm11 has joined #openstack-containers17:41
*** puranamr has joined #openstack-containers17:43
*** puranamr has quit IRC17:47
*** eghobo has joined #openstack-containers17:47
*** eghobo has quit IRC17:49
*** adrian_otto has quit IRC17:49
*** eghobo_ has quit IRC17:49
openstackgerritDaneyon Hansen proposed openstack/magnum: Updates Swarm Heat Templates for Container Network Model  https://review.openstack.org/22436717:50
*** Marga_ has quit IRC17:50
*** adrian_otto has joined #openstack-containers17:50
*** achanda has joined #openstack-containers17:51
*** puranamr has joined #openstack-containers17:51
*** devkulkarni has quit IRC17:51
*** devkulkarni has joined #openstack-containers17:52
*** puranamr has quit IRC17:55
*** devkulkarni has quit IRC17:55
*** puranamr has joined #openstack-containers17:55
*** devkulkarni has joined #openstack-containers17:55
*** kebray has joined #openstack-containers17:56
*** annegentle has quit IRC17:57
*** eghobo has joined #openstack-containers17:57
openstackgerritTom Cammann proposed openstack/magnum: Eggnore the .egg directory  https://review.openstack.org/22744617:59
*** puranamr has quit IRC18:01
*** devkulkarni1 has joined #openstack-containers18:02
*** devkulkarni has quit IRC18:03
*** rlrossit has quit IRC18:10
*** devkulkarni1 has quit IRC18:11
openstackgerritTom Cammann proposed openstack/magnum: Create master tmptls for k8s ironic/coreos bay  https://review.openstack.org/21016918:12
*** manjeets_ has joined #openstack-containers18:13
*** exploreshaifali has joined #openstack-containers18:14
*** manjeets has quit IRC18:16
*** dane_leblanc has joined #openstack-containers18:18
*** rlrossit has joined #openstack-containers18:18
*** dane_leblanc has quit IRC18:22
openstackgerritMerged openstack/magnum: Fix H405 and E131 ignored pep8 rules  https://review.openstack.org/22418318:28
*** achanda has quit IRC18:29
*** dane_leblanc has joined #openstack-containers18:30
openstackgerritMerged openstack/magnum: Use --max-complexity flake8 rule  https://review.openstack.org/22418418:30
*** Marga_ has joined #openstack-containers18:31
*** achanda has joined #openstack-containers18:31
*** Marga_ has quit IRC18:40
*** Marga_ has joined #openstack-containers18:40
openstackgerritAndrew Melton proposed openstack/magnum: Add TLS to Docker-Swarm Template  https://review.openstack.org/21259818:41
*** adrian_otto has quit IRC18:47
*** eghobo has quit IRC18:54
openstackgerritVilobh Meshram proposed openstack/magnum: WIP : Objects from Bay - Replication Controller  https://review.openstack.org/21336819:02
openstackgerritVilobh Meshram proposed openstack/magnum: Objects from Bay : Retrieve k8s API client  https://review.openstack.org/22034419:02
vilobhmm11hongbin : ping19:03
hongbin_vilobhmm11: yo19:03
vilobhmm11need some help19:03
vilobhmm11have identified19:03
vilobhmm11problems with this patch19:03
vilobhmm11 https://review.openstack.org/21336819:03
vilobhmm11#1. get/READ calls are very slow19:04
vilobhmm11it takes 160 sec to complete even a get_one test19:04
hongbin_.........19:04
hongbin_160 sec?19:04
vilobhmm11yes the call get blocked…wanted to check19:05
hongbin_on unit test?19:05
vilobhmm11yes …whether is it because of the _call method here https://review.openstack.org/#/c/213368/10/magnum/conductor/api.py #115, #10919:06
vilobhmm11looks like its a blocking call19:06
hongbin_Have you tried to mock the slow call accordingly19:06
vilobhmm11I did try mocking rpcapi.rc_show and rpcapi.rc_list and return results accordingly if thats what you mean to say19:07
hongbin_OK, I will try the patch locally19:08
*** gangil1 has quit IRC19:08
vilobhmm11If you can try magnum.tests.unit.api.controllers.v1.test_replicationcontroller.TestListRC.test_detail_with_pagination_marker by checking out https://review.openstack.org/#/c/213368/10 we can reporduce the problem19:08
hongbin_k19:08
vilobhmm11hongbin : thanks!19:08
hongbin_vilobhmm11: Here is the error I got http://paste.openstack.org/show/473942/19:12
*** kebray has quit IRC19:12
hongbin_It failed right away19:12
vilobhmm11hmmm..something wrong with my env then….how about this magnum.tests.unit.api.controllers.v1.test_replicationcontroller.TestListRC.test_get_all_with_pagination_marker19:13
vilobhmm11I started it at 12:12:23 its still running19:14
vilobhmm11the one which  you tried (magnum.tests.unit.api.controllers.v1.test_replicationcontroller.TestListRC.test_detail_with_pagination_marker)19:14
vilobhmm11hongbin : in my case i am not getting "circular ref detected" http://paste.openstack.org/show/473943/19:18
hongbin_vilobhmm11: try this: tox -epy27 --recreate  ....19:20
vilobhmm11ok19:23
*** j___ has quit IRC19:24
*** kebray has joined #openstack-containers19:29
*** kebray has quit IRC19:31
*** kebray has joined #openstack-containers19:32
vilobhmm11hongbin : also problem #2. is with fetching the manifest…irrespective of doing this https://review.openstack.org/#/c/213368/10/magnum/api/controllers/v1/replicationcontroller.py #319 the manifest is empty…need to associated the in-memory rc obj with the manifest…parse_manifest just parses the manifest…problem : http://paste.openstack.org/show/473944/19:34
*** kebray has quit IRC19:35
hongbin_\me is looking into the error19:37
vilobhmm11ok19:38
*** banix has quit IRC19:39
hongbin_vilobhmm11: I guess the reason is that you are using a None manifest19:39
hongbin_vilobhmm11: try another manifest (e.g. {'foo': 'bar'}) see if it works19:40
vilobhmm11hongbin : would like to understand it better….http://paste.openstack.org/show/473944/ if you see #2019:40
hongbin_I see #1319:41
vilobhmm11true19:41
vilobhmm11but will the rc['manifest'] will get populated by #18-#1919:41
vilobhmm11call19:41
vilobhmm11It didn't seem so19:41
vilobhmm11let me try adding a break point there or dumping it19:42
hongbin_#18 - #19 doesn't populate the manifest19:42
vilobhmm11so we are on same page19:43
hongbin_since your 'patch' doesn't have manifest19:43
hongbin_k19:43
vilobhmm11ok…so the test explicitly needs to pass manifest19:43
vilobhmm11as you suggested19:43
vilobhmm11so that patch has manifest19:43
vilobhmm11let me try that quickly19:43
hongbin_vilobhmm11: actually, I think you can remove that test19:45
hongbin_since the test are replacing the image19:45
hongbin_which is not supported in your implementaiton19:45
hongbin_so it doesn't make sense to test replacing image19:45
vilobhmm11hongbin : but if we see at the replace_namespaced_replication_controller api won't images be just another attribute which should be tested to check whether it can be replaced or not and hence i thought this test should be there19:49
vilobhmm11but if you are saying so…will remove it…just because of above thought i didn't remove it19:49
hongbin_vilobhmm11: yes, just remove it. If users want to replace image, they can do it through a manifest19:51
hongbin_I believe there are many tests that you can remove19:52
vilobhmm11hongbin : yes i did sent out an e-mail here are the details of all the test that need change https://etherpad.openstack.org/p/objects-from-bay-tests19:52
vilobhmm11will remove few of them that don't satisfy19:53
vilobhmm11thanks!19:53
*** alejandrito has quit IRC19:53
hongbin_vilobhmm11: if you don't want to remove, you can replace the patch to point to manifest (instead of image)19:54
vilobhmm11hongbin_ : would prefer that than removing19:54
*** adrian_otto has joined #openstack-containers19:54
hongbin_k19:55
vilobhmm11will point the patch to manifest..19:55
vilobhmm11hongbin : so for ofb quick update on getting the unit test to work so for now #1. delete tests r working #2. with this fix update/patch should work #3. read path test are taking time and broken for now will focus on #219:57
vilobhmm11can apply the same model used for rc obj to service/pods19:57
*** kebray has joined #openstack-containers19:57
hongbin_ack19:58
manjeets_._.20:00
hongbin_vilobhmm11: Actually, I have one thing for you. I guess you are not working on this bug https://bugs.launchpad.net/magnum/+bug/1487224 . I can take it back if you don't mnd20:00
openstackLaunchpad bug 1487224 in Magnum "Support specifying memory on container-create" [High,Confirmed] - Assigned to Vilobh Meshram (vilobhmm)20:00
vilobhmm11hongbin : sure20:02
hongbin_thx20:02
vilobhmm11np20:02
*** jwcroppe has joined #openstack-containers20:05
*** absubram has joined #openstack-containers20:08
openstackgerritAndrew Melton proposed openstack/python-magnumclient: Add insecure flag to baymodel  https://review.openstack.org/22752920:08
*** alex_klimov has joined #openstack-containers20:09
openstackgerritAndrew Melton proposed openstack/python-magnumclient: Add insecure flag to baymodel  https://review.openstack.org/22752920:13
*** gangil has joined #openstack-containers20:19
*** eghobo has joined #openstack-containers20:28
*** eghobo has quit IRC20:29
*** achanda has quit IRC20:30
*** exploreshaifali has quit IRC20:41
*** eghobo has joined #openstack-containers20:47
*** banix has joined #openstack-containers20:53
openstackgerritHongbin Lu proposed openstack/magnum: Fix double-wrapping of exception in conductor  https://review.openstack.org/22755220:58
*** daneyon has quit IRC21:04
*** achanda has joined #openstack-containers21:05
*** daneyon has joined #openstack-containers21:05
*** achanda has quit IRC21:09
*** rpothier has quit IRC21:10
*** eghobo has quit IRC21:11
openstackgerritTon Ngo proposed openstack/magnum: Configure CoreOS for Kubernetes load balancer feature  https://review.openstack.org/19242521:12
openstackgerritTon Ngo proposed openstack/magnum: Configure Ironic for Kubernetes load balancer feature  https://review.openstack.org/19242621:12
openstackgerritTon Ngo proposed openstack/magnum: Configure Fedora Atomic for Kubernetes load balancer feature  https://review.openstack.org/19187821:12
*** manjeets_ has quit IRC21:14
*** Drago has quit IRC21:18
*** Drago has joined #openstack-containers21:20
*** eghobo has joined #openstack-containers21:22
*** absubram is now known as absubram_afk21:23
*** dims__ has quit IRC21:29
*** rlrossit has left #openstack-containers21:34
*** iraw- has quit IRC21:41
*** kebray has quit IRC21:41
*** alejandrito has joined #openstack-containers21:41
*** iraw- has joined #openstack-containers21:42
*** daneyon has quit IRC21:49
*** daneyon has joined #openstack-containers21:50
*** jlvillal has quit IRC21:51
*** Drago has quit IRC21:52
*** Drago has joined #openstack-containers21:52
*** dims_ has joined #openstack-containers21:52
*** adrian_otto has quit IRC21:56
*** jlvillal has joined #openstack-containers21:57
*** achanda has joined #openstack-containers22:06
*** jwcroppe has quit IRC22:07
*** jwcroppe has joined #openstack-containers22:07
*** eghobo has quit IRC22:10
*** jwcroppe has quit IRC22:12
vilobhmm11hongbin : for this part of code https://github.com/openstack/magnum/blob/master/magnum/tests/unit/api/controllers/v1/test_replicationcontroller.py#L258-L26822:15
vilobhmm11how will we combine manifest and images together ?22:17
vilobhmm11as part of the patch request22:17
*** kebray has joined #openstack-containers22:17
*** kebray has quit IRC22:17
hongbin_vilobhmm11: manifest basically is a file that specify the resources22:18
hongbin_users can change image by edit the manifest22:19
hongbin_so no need to combine them22:19
vilobhmm11true …we can get it readily in code by "rc_dict = apiutils.rc_post_data() "22:19
hongbin_just manifest should be enough22:19
vilobhmm11hongbin : specifically concered about this part in code https://github.com/openstack/magnum/blob/master/magnum/tests/unit/api/controllers/v1/test_replicationcontroller.py#L260-L263 …are you thinking along these lines http://paste.openstack.org/show/473957/22:23
hongbin_don't get your question22:24
*** banix has quit IRC22:24
*** kebray has joined #openstack-containers22:24
hongbin_vilobhmm11: could you elaborate the question?22:25
vilobhmm11what parameter patch_json should expect…since when I debug more it will call http://python-json-patch.readthedocs.org/en/latest/mod-jsonpatch.html but I still see manifest as empty….so what parameters should patch_json take apart from uuid, bay_uuid22:25
vilobhmm11'/rcs/%s/%s' % (self.rc.uuid, self.rc.bay_uuid),   will help it to construct request22:26
vilobhmm11https://github.com/openstack/magnum/blob/master/magnum/tests/unit/api/controllers/v1/test_replicationcontroller.py#L260-L263 here path: '/images/0', looks like that be changed to something specific to manifest22:27
vilobhmm11is that clear now ?22:27
vilobhmm11hongbin : ^^22:28
hongbin_I think your parameter is correct22:29
hongbin_the thing is the operation 'add'22:29
*** banix has joined #openstack-containers22:29
hongbin_if you change 'add' to 'replace', does the manifest still empty?22:29
hongbin_I think onbody is using add right now22:30
hongbin_The only one that is used in 'replace'22:31
vilobhmm11ok22:31
*** absubram_afk has quit IRC22:32
hongbin_frankly, I don't see any value for testing the 'add' operation right now22:33
hongbin_if the 'replace' operation works, magnum is working22:34
vilobhmm11ok22:35
vilobhmm11manifest till comes empty22:35
hongbin_...........22:35
hongbin_vilobhmm11: I need to reproduce the error22:37
hongbin_cannot figure out why by just looking22:37
hongbin_vilobhmm11: Could you upload what you have to gerrit. I will download your patch and try the tests tomorrow22:39
vilobhmm11I have uploaded here https://review.openstack.org/#/c/213368/22:40
hongbin_k22:41
*** eghobo has joined #openstack-containers23:00
*** alejandrito has quit IRC23:03
*** alex_klimov has quit IRC23:04
*** dims_ has quit IRC23:05
suro-patzhongbin_: Whenever you have time, please review https://review.openstack.org/#/c/220702/ - this is ready now, after addressing the review comments/renaming23:16
hongbin_suro-patz: will do that23:17
suro-patzhongbin_: thnx! appreciate it!23:17
hongbin_np23:19
*** hongbin_ has quit IRC23:19
yuanyinghongbin: around?23:25
*** dimtruck is now known as zz_dimtruck23:25
yuanyinga bit late23:35
*** sthillma has joined #openstack-containers23:38
*** agireud has quit IRC23:42
openstackgerritMotohiro/Yuanying Otsuka proposed openstack/magnum: Move security group setting to kubecluster.yaml  https://review.openstack.org/22714523:43
openstackgerritMerged openstack/magnum: Add TLS to Docker-Swarm Template  https://review.openstack.org/21259823:46
*** harlowja has quit IRC23:56
*** gus has quit IRC23:57
*** harlowja has joined #openstack-containers23:57
*** eghobo has quit IRC23:58
*** gus has joined #openstack-containers23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!