Wednesday, 2014-12-03

« Tuesday, 2014-12-02 Index Thursday, 2014-12-04 »
*** unicell1 has quit IRC00:00
*** jwang_ has joined #openstack-containers00:14
*** yuanying has quit IRC00:23
*** yuanying has joined #openstack-containers00:23
*** yuanying has quit IRC00:24
*** EricGonczer_ has quit IRC00:26
*** daneyon has quit IRC00:26
*** yuanying has joined #openstack-containers00:27
*** kebray has joined #openstack-containers00:28
*** unicell has joined #openstack-containers00:31
*** dims_ has joined #openstack-containers00:31
*** dims has quit IRC00:34
*** martyfalatic has quit IRC01:05
*** yuanying has quit IRC01:08
*** yuanying has joined #openstack-containers01:09
*** yuanying has quit IRC01:15
*** yuanying has joined #openstack-containers01:16
*** yuanying has quit IRC01:17
*** daneyon has joined #openstack-containers01:18
*** yuanying has joined #openstack-containers01:19
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Add object.pod  https://review.openstack.org/13835101:24
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Add object.service  https://review.openstack.org/13835201:26
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828401:26
*** EricGonczer_ has joined #openstack-containers01:32
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832901:32
*** yuanying has quit IRC01:33
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832301:33
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Add an object.container  https://review.openstack.org/13835001:35
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Add object.pod  https://review.openstack.org/13835101:37
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828401:37
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832901:37
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Add object.service  https://review.openstack.org/13835201:37
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832301:37
*** suro_ has quit IRC01:37
*** yuanying has joined #openstack-containers01:38
*** yuanying has quit IRC01:40
*** EricGonczer_ has quit IRC01:47
*** EricGonczer_ has joined #openstack-containers01:49
*** yuanying has joined #openstack-containers01:50
*** daneyon has quit IRC01:54
*** EricGonczer_ has quit IRC02:08
*** EricGonczer_ has joined #openstack-containers02:11
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828402:16
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832902:17
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832302:17
*** EricGonczer_ has quit IRC02:22
*** erkules_ has joined #openstack-containers02:23
*** marcoemorais has quit IRC02:24
*** erkules has quit IRC02:25
dims_sdake: ok, last 3 of the series needs work02:26
dims_sdake: got pep8 working and added jsonpatch into requirements.txt02:27
*** takadayuiko has joined #openstack-containers02:30
*** jwang_ has quit IRC02:34
*** dims_ has quit IRC02:36
*** erkules_ has quit IRC02:49
*** erkules_ has joined #openstack-containers02:50
*** hblixt has quit IRC03:17
*** dims has joined #openstack-containers03:36
*** kebray has quit IRC03:42
*** dims has quit IRC03:42
*** achanda has joined #openstack-containers03:44
*** harlowja_ is now known as harlowja_away03:47
*** yuanying_ has joined #openstack-containers04:14
*** adrian_otto has joined #openstack-containers04:15
*** yuanying has quit IRC04:17
*** jeckersb is now known as jeckersb_gone04:28
*** achanda has quit IRC04:43
*** achanda has joined #openstack-containers04:43
*** achanda has quit IRC04:46
*** achanda_ has joined #openstack-containers04:46
*** achanda_ has quit IRC05:23
*** unicell has quit IRC05:47
*** achanda has joined #openstack-containers05:51
*** achanda has quit IRC05:55
*** ParsectiX has quit IRC06:08
*** nshaikh has joined #openstack-containers06:56
sdakedims_ sorry just got back07:13
*** suro_ has joined #openstack-containers07:22
*** openstackgerrit has quit IRC07:50
*** openstackgerrit has joined #openstack-containers07:50
*** nimissa has quit IRC08:10
*** nimissa has joined #openstack-containers08:11
*** suro_ has quit IRC08:36
*** erkules_ is now known as erkules09:00
*** dims has joined #openstack-containers09:03
*** dims has quit IRC09:08
openstackgerritSteven Dake proposed stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828410:04
openstackgerritSteven Dake proposed stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832910:04
openstackgerritSteven Dake proposed stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832310:04
*** takadayuiko has quit IRC10:21
*** adrian_otto has quit IRC10:55
*** dims has joined #openstack-containers11:30
*** nshaikh has quit IRC11:31
*** nimissa has quit IRC11:35
*** nimissa has joined #openstack-containers11:52
*** nimissa has quit IRC11:56
*** nimissa has joined #openstack-containers12:06
dimssdake: good morning12:08
*** julim has joined #openstack-containers12:18
*** funzo has quit IRC12:20
*** funzo has joined #openstack-containers12:40
*** EricGonczer_ has joined #openstack-containers12:41
*** stannie has joined #openstack-containers12:48
*** unicell has joined #openstack-containers12:53
*** kitch has joined #openstack-containers13:30
*** EricGonc_ has joined #openstack-containers13:33
*** EricGonc_ has quit IRC13:33
*** unicell has quit IRC13:41
*** adrian_otto has joined #openstack-containers13:55
*** thomasem has joined #openstack-containers13:58
*** thomasem has quit IRC13:58
*** thomasem has joined #openstack-containers13:59
*** jeckersb_gone is now known as jeckersb14:05
*** EricGonczer_ has quit IRC14:08
*** fawadkhaliq has joined #openstack-containers14:27
*** EricGonczer_ has joined #openstack-containers14:56
*** dims has quit IRC15:04
*** dims has joined #openstack-containers15:04
*** EricGonczer_ has quit IRC15:15
*** EricGonczer_ has joined #openstack-containers15:20
*** kitch has quit IRC15:32
*** kitch has joined #openstack-containers15:47
*** EricGonczer_ has quit IRC15:52
*** kebray has joined #openstack-containers16:05
*** kebray has quit IRC16:13
*** fawadkha_ has joined #openstack-containers16:35
*** fawadkhaliq has quit IRC16:35
*** EricGonczer_ has joined #openstack-containers16:37
*** daneyon has joined #openstack-containers16:38
*** fawadkha_ has quit IRC16:41
*** EricGonczer_ has quit IRC16:41
*** daneyon_ has joined #openstack-containers16:44
*** daneyon has quit IRC16:44
*** daneyon_ has quit IRC16:46
*** daneyon has joined #openstack-containers16:46
*** EricGonczer_ has joined #openstack-containers16:47
*** prad has joined #openstack-containers16:51
*** EricGonc_ has joined #openstack-containers16:52
*** achanda has joined #openstack-containers16:54
*** EricGonczer_ has quit IRC16:56
*** diga has joined #openstack-containers16:57
digasdake: Hi16:59
*** achanda has quit IRC17:02
*** kebray has joined #openstack-containers17:07
*** achanda has joined #openstack-containers17:10
*** marcoemorais has joined #openstack-containers17:13
sdakemorning17:13
sdakehey diga17:14
digaHi17:14
digaThis is regarding https://blueprints.launchpad.net/magnum/+spec/magnum-api-service17:14
sdakeshoot17:14
digaAPI blueprint17:14
*** marcoemorais1 has joined #openstack-containers17:15
digaI am not able to understand what checkins should I do for this ?17:15
digabecause another blueprint has changes the existing code17:15
*** marcoemorais1 has quit IRC17:16
digaChanged***17:16
*** marcoemorais1 has joined #openstack-containers17:17
*** marcoemorais has quit IRC17:17
digaI think we need to define the use cases for these same api blueprints17:21
*** ParsectiX has joined #openstack-containers17:22
digaMy thinking is not to repeate the code for the same functionality17:22
sdakehow do you mean17:31
sdakerather where do you want to do it17:31
*** stannie has quit IRC17:33
digaNo, my intension is specify different tasks for these blueprint17:35
*** suro_ has joined #openstack-containers17:36
digacan you update tasks for these blueprints17:36
digarather tell me, I'll update those tasks so there will be no confusion here while implementing the code17:38
sdakesure, do you need instructions on updating you mean?17:39
sdakethe individual data I don't think can be changed17:39
sdakethis is one problem with launchpad17:39
digahmm17:40
digaBetter I'll define my tasks for magnum-api-service, then you can verify those17:41
digawill this be fine for you ?17:41
sdakesure whatever is easiest17:43
sdakealthough magnum-api-service is almonst done with the patches in the review queue17:43
sdakewe just need to define the objects contents at this point17:44
digaYes, Agreed17:46
*** nshaikh has joined #openstack-containers17:46
sdakediga are youy a core reviewer by chance?17:47
sdakehttps://review.openstack.org/#/c/138352/17:48
sdakethis change and all its dependencies need a review^17:48
digaI was at sometime for gantt17:48
digaok17:48
digalet me check it17:48
digasdake: Tomorrow I won't be available after 6PM india17:51
digawill be travelling17:52
sdakediga are you in india17:57
digayes18:00
digaI m from Pune India18:01
*** jwang_ has joined #openstack-containers18:02
*** harlowja_away is now known as harlowja_18:04
digaI am in India18:05
digaits late here18:05
digasdake: I'll ping you tomorrow18:05
digaThanks18:05
sdakecool18:06
*** achanda has quit IRC18:20
*** achanda has joined #openstack-containers18:21
*** diga has quit IRC18:29
*** ParsectiX has quit IRC18:38
openstackgerritPradeep Kilambi proposed stackforge/magnum: WIP: Implement python interface to execute k8s CLI services  https://review.openstack.org/13881718:43
dimssdake: the last 3 in the patch series are still failing tests...18:45
dimssdake: do we fix the tests?18:46
sdakeya sorry i have had arough start this morning18:47
sdakeneed more minerals18:47
dims:) one of those days!18:47
sdakeprad nice work on that interface18:47
sdakeya just a bad day18:47
sdakealready noon and feel like I have done nothing18:48
pradsdake, thx yea its still very early version.. i just whipped something to run it by you and make sure i'm doing whats needed for our use case18:48
sdakelooks good18:49
sdakei think it should be in a different file though18:49
pradsdake, currently as you see i just execute the command and return the output as is.. if we have an specifics on how i should return lemme know18:49
sdakeas far as data output return, best to return something in yaml or json for the objects18:49
sdakeeg, parse the output a bit18:49
sdakeprobably need a return code as well in the result18:50
pradsdake, sure i can return a dict of some sort18:50
sdakeyup tha will do it18:50
pradsdake, so regarding different file .. trying to understand why? isn't this k8s.py suppose to be the interface?18:50
sdakethe file rather then being in k8s should be something like k8s_cli.py18:51
sdakek8s.py is going to be the implementation of the code that uses k8s_cli.py I suspect18:51
sdakejust makes more tidy code base I think18:51
sdakeup to you really18:51
sdakewe can always move i t around later18:51
*** achanda has quit IRC18:51
sdakeyou said you tested on fedora 20, was that with larsks heat-kube repo?18:51
pradah ok .. sure, based on the amount of processing we do with this data, it dint seem like a separate file was needed.18:52
pradi'm cool either way18:52
*** achanda has joined #openstack-containers18:52
*** achanda has quit IRC18:52
sdakeyup up to you18:52
*** achanda has joined #openstack-containers18:52
sdakeI'm a one file for one class type of guy :)18:52
pradsdake, yea i have a env running using lars's heat templete18:52
sdakecool18:52
sdakeanyway looks like a rockin start18:53
sdakethe next part is the actual k8s management which is mostly just calling those functions and returning the right hing to the backend18:53
sdakethe problem we have now, is the backend isn't implemented :(18:53
pradcool and in case of an error, i assume we just log it and return the error code in the json with msg18:54
pradagreed18:54
sdakewfm18:55
sdakemaybe make it part of the dict18:55
pradonce i have the code ironed out, will add tests for this18:55
pradsure18:55
sdakeeg, result or something18:55
pradyea18:55
sdakealso note418:56
*** sdake has quit IRC18:56
*** sdake has joined #openstack-containers18:57
sdakeprad I just noticed18:57
sdakeyou are using the rpc interfaces for the function calls18:57
sdakeI'd suggest putting them in separate functions18:57
sdakethat way the cli interface is possibly a different set of objects from the rpc objects18:57
sdakemake sense?18:57
pradsdake, which part are you referring to? the execute?18:58
sdakeoh nm, apparently my brain is busted18:59
sdakejust ignore that18:59
sdakeguess I Should go back to sleep :(18:59
pradheh np18:59
*** achanda has quit IRC19:41
*** achanda has joined #openstack-containers19:42
*** adrian_otto has quit IRC19:46
*** achanda has quit IRC19:46
*** nshaikh has left #openstack-containers20:09
*** kebray has quit IRC20:14
*** kebray has joined #openstack-containers20:17
*** ycombinator_ has joined #openstack-containers20:27
*** daneyon has quit IRC20:49
*** daneyon has joined #openstack-containers20:57
*** ycombinator_ has quit IRC21:01
*** adrian_otto has joined #openstack-containers21:20
dimssdake: not seeing anyone doing reviews...sigh21:55
* sdake sadfaces21:55
sdakeI think I have the patch stream ready to go as well21:55
sdakeif I can fix this one last t est case21:55
sdakeboy these tests were a mess21:55
sdakewith the db integration etc21:56
sdakemade it painful21:56
dimsi see21:56
*** ycombinator_ has joined #openstack-containers21:57
*** kitch has quit IRC21:57
*** prad has quit IRC22:00
openstackgerritSteven Dake proposed stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828422:08
openstackgerritSteven Dake proposed stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832922:08
openstackgerritSteven Dake proposed stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832322:08
sdakeboy that took forever22:09
sdaketime for a break bbiaf22:09
*** ycombinator_ has quit IRC22:10
dimssdake: i'll fix some pep8 issues22:18
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828422:21
*** jeckersb is now known as jeckersb_gone22:22
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832922:22
openstackgerritDavanum Srinivas (dims) proposed stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832322:22
*** EricGonc_ has quit IRC22:24
dimsthe whole series is good to go22:24
sdakethe test cases aren't fixed up for containers and pods and services22:24
sdakebut I guess we can do that later22:24
dimsright22:24
sdakeI had intended to do that in the stream, but git pull too hard ;-)22:25
sdakedims do you run pep8 locally?22:25
dimsyes22:25
sdakemine spits out different errors, i assume because different versions22:25
sdakeadrian_otto can you review that patch stream real quick like so we can get cracking on the rpc server22:26
dimsi periodically run "sudo pip install -U -r requirements.txt -r test-requirements.txt" from nova directory, that usually helps me get back in sync with versions of hacking/tox/flake8 etc22:26
sdakegot it22:27
sdakeneed one more core reviewer I think22:28
sdake4 people around all the time for reviews usually gets the job done ;-)22:29
*** EricGonczer_ has joined #openstack-containers22:29
dimsyep. my feeling is we are not yet ready to move to 2 +2's, but we gotta try22:30
sdakedims that is actually my largest patch stream I have ever written22:30
sdakeI think it was 15 patches22:30
sdakelongest maybe not largest ;-)22:30
dimshehe22:31
sdakepretty cool all that worked22:32
sdakeI think containers work as well22:32
sdakeneed action on this now22:39
sdakehttps://blueprints.launchpad.net/magnum/+spec/implement-backend-rpc22:39
*** jwang_ has quit IRC22:40
sdakedims maybe we should just go back to 1 core reviewer for now until we can build out the cor eteam more22:42
sdakeotherwise our dev will lag22:42
dims+122:42
dimssdake: does BaysController's patch method work?22:43
sdakewhich patch method22:43
sdakelink to the review or code?22:43
dimslatest in the patch series22:44
sdakethe last one is remove cruft?22:44
sdakeor the one before that22:44
sdakewhat is the commit topic22:44
sdakecommit sbuject I mean22:45
dimsline 262 - https://review.openstack.org/#/c/138284/23/magnum/api/controllers/v1/bay.py,unified22:45
dimsthere was a put method before22:45
sdakeso what I did with taht f ile22:46
sdakeis I copied ironic/api/v1/sensor.py22:46
sdakeand then tailored it22:46
sdakeso I doubt that bay_uuid works properly22:46
sdakeor the patch method for that matter22:46
dimsk now i know where to look thanks22:46
sdakeusually bay_uuid is deleted22:46
sdake(when I made the patches)22:46
dimsk22:47
sdakebay_uuid there is meant to reference a different uuid for a different object type22:47
dimsright22:47
sdakerather sensor/port I think22:48
sdakesensor was some other codebase ;_)22:48
sdakelots of c&P in this one22:48
dimsy. am going to pull the trigger now22:48
*** EricGonczer_ has quit IRC22:49
openstackgerritMerged stackforge/magnum: Add versioned object dependency modules  https://review.openstack.org/13827822:52
dimssdake: what do you want to do with https://review.openstack.org/#/c/138229/ and https://review.openstack.org/#/c/138105/?22:52
*** kebray has quit IRC22:52
openstackgerritMerged stackforge/magnum: Add some common code copied from Ironic  https://review.openstack.org/13827922:52
*** kebray has joined #openstack-containers22:53
openstackgerritMerged stackforge/magnum: Copy Ironic's database model codebase  https://review.openstack.org/13828022:53
openstackgerritMerged stackforge/magnum: Add objects/utils.py copied from Ironic  https://review.openstack.org/13828122:54
openstackgerritMerged stackforge/magnum: Set max limit as required by versioned objects  https://review.openstack.org/13828222:54
openstackgerritMerged stackforge/magnum: Remove unnecessary model init call  https://review.openstack.org/13832822:54
openstackgerritMerged stackforge/magnum: Modify the object registry to support ver objects  https://review.openstack.org/13828322:55
openstackgerritMerged stackforge/magnum: Add an object.container  https://review.openstack.org/13835022:56
openstackgerritMerged stackforge/magnum: Add object.pod  https://review.openstack.org/13835122:56
openstackgerritMerged stackforge/magnum: Add object.service  https://review.openstack.org/13835222:56
openstackgerritMerged stackforge/magnum: Use versioned objects for bays  https://review.openstack.org/13828422:56
openstackgerritMerged stackforge/magnum: Use versioned objects for Container objects  https://review.openstack.org/13832922:56
openstackgerritMerged stackforge/magnum: Remove cruft in api tree  https://review.openstack.org/13832322:57
sdakefirst one needs to be merged for the code to work I think, second one is a bad link?22:57
sdakeyou always know when an engine blows, a man can't win em all23:00
*** jwang_ has joined #openstack-containers23:02
sdakeactually dims that first patch may not be necessary23:03
sdakelets just hold off on it for a bit23:03
sdakeif master seems to work without it no sense adding it23:04
*** julim has quit IRC23:27
*** jwang__ has joined #openstack-containers23:55
*** jwang_ has quit IRC23:59
« Tuesday, 2014-12-02 Index Thursday, 2014-12-04 »

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