Monday, 2012-01-23

*** rsampaio has quit IRC00:05
*** rsampaio has joined #openstack-dev00:13
*** pixelbeat has joined #openstack-dev00:15
*** sleepsonthefloo has joined #openstack-dev00:23
*** eglynn__ has joined #openstack-dev00:28
*** eglynn_ has quit IRC00:30
*** pixelbeat has quit IRC00:38
*** reed has joined #openstack-dev00:50
*** Leseb has quit IRC00:52
*** eglynn has joined #openstack-dev00:58
*** eglynn__ has quit IRC01:01
*** rsampaio has quit IRC01:04
openstackgerritCole Robinson proposed a change to openstack/nova: usage: Fix time filtering  https://review.openstack.org/328401:05
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add a 'usage' module and 'usage-list' cli command  https://review.openstack.org/328501:17
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: shell: Hook --debug up to more stuff  https://review.openstack.org/328601:23
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Return a useful message if passed API version is unknown  https://review.openstack.org/328701:23
*** rsampaio has joined #openstack-dev01:59
*** sdake has quit IRC02:15
*** sdake has joined #openstack-dev02:19
*** rsampaio has quit IRC02:22
*** rsampaio has joined #openstack-dev02:35
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: shell: Default --version to 1.1 if not specified  https://review.openstack.org/328802:40
*** rods has quit IRC02:43
*** dprince has joined #openstack-dev02:44
*** sleepsonthefloo has quit IRC02:45
*** dprince has quit IRC02:48
*** Ryan_Lane has quit IRC03:11
*** andrewsmedina has quit IRC03:13
*** utlemming has joined #openstack-dev03:16
*** utlemming_afk has quit IRC03:19
*** utlemming_afk has joined #openstack-dev03:23
*** utlemming has quit IRC03:24
*** dtroyer has quit IRC03:36
*** rsampaio has quit IRC03:44
*** bencherian has joined #openstack-dev03:50
*** sdake has quit IRC04:03
*** mjfork has quit IRC04:19
*** sdake has joined #openstack-dev04:23
*** rsampaio has joined #openstack-dev04:28
*** rsampaio has quit IRC04:36
*** rsampaio_ has joined #openstack-dev04:36
*** zns has joined #openstack-dev04:58
openstackgerritdan wendlandt proposed a change to openstack/quantum: bug 920299: remove duplicate + outdate README  https://review.openstack.org/328905:07
uvirtbotLaunchpad bug 920299 in quantum "quantum has outdated + duplicate README" [Medium,In progress] https://launchpad.net/bugs/92029905:07
*** rsampaio_ has quit IRC05:11
openstackgerritdan wendlandt proposed a change to openstack/python-quantumclient: move batch_config.py to client library.  https://review.openstack.org/329005:22
openstackgerritdan wendlandt proposed a change to openstack/python-quantumclient: move batch_config.py to client library.  https://review.openstack.org/329005:33
openstackgerritdan wendlandt proposed a change to openstack/quantum: moving batch config out of quantum-server repo.  https://review.openstack.org/329105:43
*** bencherian has quit IRC06:06
openstackgerritRajaram Mallya proposed a change to openstack/melange: fixes bug 919158 , default ip blocks are now optional  https://review.openstack.org/329206:10
openstackgerritRajaram Mallya proposed a change to openstack/melange: fixes bug 919155 by tracking deallocated macs in the allocatable_macs table  https://review.openstack.org/322306:10
uvirtbotLaunchpad bug 919158 in melange "creating default ipblocks should be optional" [Medium,New] https://launchpad.net/bugs/91915806:10
uvirtbotLaunchpad bug 919155 in melange "mac addressess are not being reallocated" [Medium,In progress] https://launchpad.net/bugs/91915506:10
*** sdake has quit IRC06:28
*** bepernoot has joined #openstack-dev06:29
*** sdake has joined #openstack-dev06:29
*** reed has quit IRC06:34
*** bepernoot has quit IRC06:47
*** mikal has quit IRC07:02
*** mikal has joined #openstack-dev07:13
*** zigo has joined #openstack-dev07:22
*** bepernoot has joined #openstack-dev07:47
*** rbasak has joined #openstack-dev07:50
openstackgerritkavan-patil proposed a change to openstack/tempest: Added Keypair extension (os-keypairs) client and tests LP#900139  https://review.openstack.org/317907:51
*** Ryan_Lane has joined #openstack-dev08:09
*** rods has joined #openstack-dev08:12
*** reidrac has joined #openstack-dev08:16
openstackgerritMark McLoughlin proposed a change to openstack/glance: glance-control need not locate a server's config file (#919520)  https://review.openstack.org/329308:30
openstackgerritMark McLoughlin proposed a change to openstack/glance: glance-control need not locate a server's config file (#919520)  https://review.openstack.org/329308:33
*** Remco_ has joined #openstack-dev08:39
*** jeffjapan has quit IRC08:43
*** kyriakos has joined #openstack-dev09:17
*** ncode has joined #openstack-dev09:18
*** ncode has joined #openstack-dev09:18
*** derekh has joined #openstack-dev09:23
*** Ryan_Lane has quit IRC09:31
*** bepernoot has quit IRC09:32
*** bepernoot has joined #openstack-dev09:32
*** pixelbeat has joined #openstack-dev09:47
openstackgerritdan wendlandt proposed a change to openstack-dev/devstack: bug 920364: clean-up ovs_quantum DB between runs.  https://review.openstack.org/329409:56
uvirtbotLaunchpad bug 920364 in quantum "quantum doesn't clean-up DB between devstack runs" [High,In progress] https://launchpad.net/bugs/92036409:56
*** apevec has joined #openstack-dev10:08
*** darraghb has joined #openstack-dev10:12
*** Mandell has quit IRC10:15
openstackgerritThierry Carrez proposed a change to openstack/nova: Cleanups for rootwrap module  https://review.openstack.org/329510:20
openstackgerritThierry Carrez proposed a change to openstack/nova: Fix environment passing in DnsmasqFilter  https://review.openstack.org/329610:20
openstackgerriteglynn proposed a change to openstack/glance: glance-api fails fast if default store unsupported  https://review.openstack.org/329710:24
openstackgerritThierry Carrez proposed a change to openstack/nova: Fix environment passing in DnsmasqFilter  https://review.openstack.org/329610:24
openstackgerriteglynn proposed a change to openstack/glance: glance-api fails fast if default store unsupported  https://review.openstack.org/329710:25
*** vinayp has joined #openstack-dev10:29
*** ncode has quit IRC10:29
*** vinayp has quit IRC10:29
*** journeeman has joined #openstack-dev10:32
*** journeeman is now known as Guest6325410:33
*** Guest63254 has left #openstack-dev10:33
*** Leseb has joined #openstack-dev10:34
openstackgerritMark McLoughlin proposed a change to openstack/openstack-common: Add support to cfg for disabling interspersed args  https://review.openstack.org/329810:45
openstackgerritThierry Carrez proposed a change to openstack/nova: Add a specific filter for kill commands  https://review.openstack.org/329911:03
openstackgerritKei Masumoto proposed a change to openstack/nova: Adding live migration server actions  https://review.openstack.org/231511:22
*** andrewsmedina has joined #openstack-dev11:29
*** Leseb has quit IRC11:29
*** andrewsmedina has quit IRC11:33
*** Leseb has joined #openstack-dev11:54
*** bagit has quit IRC11:56
*** bagit has joined #openstack-dev11:56
*** Leseb has quit IRC11:57
*** andrewsmedina has joined #openstack-dev12:00
*** mjfork has joined #openstack-dev12:00
*** Leseb has joined #openstack-dev12:04
*** bsza has joined #openstack-dev12:04
openstackgerritRajaram Mallya proposed a change to openstack/melange: Removes client and associated functional test code  https://review.openstack.org/330012:07
*** rkukura has left #openstack-dev12:08
*** Leseb has joined #openstack-dev12:08
*** rods has left #openstack-dev12:10
*** markvoelker has joined #openstack-dev12:14
*** vinilios has joined #openstack-dev12:26
*** rods has joined #openstack-dev12:32
openstackgerritArmando Migliaccio proposed a change to openstack/nova: blueprint host-aggregates  https://review.openstack.org/310912:36
openstackgerritThierry Carrez proposed a change to openstack/nova: Add a specific filter for kill commands  https://review.openstack.org/329913:02
openstackgerritThierry Carrez proposed a change to openstack/nova: Fix environment passing in DnsmasqFilter  https://review.openstack.org/329613:02
*** kyriakos has left #openstack-dev13:11
*** kyriakos has joined #openstack-dev13:19
Kialldevcamcar: ping, was hoping to get some advice re https://review.openstack.org/326813:21
*** kyriakos has quit IRC13:24
*** stuntmachine has joined #openstack-dev13:26
*** crobinso has joined #openstack-dev13:36
openstackgerritGhe Rivero proposed a change to openstack/quantum: Improved VlanMap.  https://review.openstack.org/230213:40
*** dtroyer has joined #openstack-dev13:42
*** rajaram has joined #openstack-dev13:51
*** kyriakos has joined #openstack-dev13:52
*** ayoung has quit IRC14:02
*** ayoung has joined #openstack-dev14:07
*** LinuxJedi has quit IRC14:08
*** lts has joined #openstack-dev14:09
openstackgerritAnne Gentle proposed a change to openstack/openstack-manuals: Adds install manual, fixes 920456  https://review.openstack.org/330114:12
*** markmc has joined #openstack-dev14:14
crobinsohi all, anyone know if Kevin Mitchell is around? I don't see an obvious nick (maybe he's hiding behind a funny one)14:19
pvocrobinso: his nick is Vek, but he probably won't be online for a bit.14:19
crobinsopvo: thanks14:20
*** bsza has quit IRC14:23
crobinsoVek: whenever you're around (or maybe someone else can help) I'm having some difficulties getting my flavors-manage nova extension to actually 'extend' the existing flavors functionality. tried to follow some other examples but i'm not sure how it's all supposed to work14:23
*** derekh has quit IRC14:27
*** bsza has joined #openstack-dev14:27
Kiallcrobinso: the disk_config extension is probably the simplest example to follow14:28
KiallOh right, sorry, your review adds additional actions rather than additional properties.. might not be the best example so ;)14:29
crobinsoKiall: so the canonical images.py functionality provides a get() api with GET /images/<image uuid> . disk_config.py extension 'extends' the get() api. does that mean if disk_config is enabled, doing GET /images/<image uuid> should invoke the method in disk_config.py instead of images.py? or is it additive? or is it only exposed via a totally different api entry point?14:37
KiallIts additive, the first property here is what it adds: http://paste.openstack.org/show/4480/14:39
Kiall"OS-DCF:diskConfig" is injected into GET servers/$id via that extension..14:39
KiallThe method at L120 shows where it actually does the injection..14:41
*** dprince has joined #openstack-dev14:42
crobinsoKiall: okay, thanks for the info, I'll poke around some more14:42
Kiallcrobinso: Also - It seems I "lied" about which extension was the simplest, deferred_delete.py is cleaner and does something very similar to what you are trying ...14:43
Kiallcrobinso: BTW, You're not the same Cole working for RedHat I met recently in Dublin?14:45
openstackgerritArmando Migliaccio proposed a change to openstack/nova: blueprint host-aggregates  https://review.openstack.org/318414:46
*** LinuxJedi has joined #openstack-dev14:47
*** zns has joined #openstack-dev14:54
*** derekh has joined #openstack-dev14:54
openstackgerritSandy Walsh proposed a change to openstack/nova: ComputeNode Capacity support  https://review.openstack.org/246014:55
crobinsoKiall: yeah that's me! (as we just discussed in /msg :) )14:59
Kialllol - didnt want to seem like you we're ignoring me?  ;)15:00
*** zns has quit IRC15:01
*** zns has joined #openstack-dev15:02
*** tryggvil__ has quit IRC15:06
*** rkukura has joined #openstack-dev15:07
*** dtroyer has quit IRC15:14
*** kbringard has joined #openstack-dev15:16
openstackgerritMark McLoughlin proposed a change to openstack/openstack-common: Add cfg test case for recursive substitution  https://review.openstack.org/330215:24
*** maploin has joined #openstack-dev15:27
*** maploin has quit IRC15:27
*** maploin has joined #openstack-dev15:27
*** david-kranz has joined #openstack-dev15:27
*** Gordonz has joined #openstack-dev15:31
*** deshantm has joined #openstack-dev15:33
openstackjenkinsProject nova-docs build #2004: SUCCESS in 2 min 53 sec: https://jenkins.openstack.org/job/nova-docs/2004/15:33
openstackjenkinsdan.prince: Fix 'nova-manage config list'15:33
*** LinuxJedi has quit IRC15:34
crobinsoKiall: hmm after playing some more I think what I'm trying to do just can't be done with current extension code, whether intentionally or just oversight. extensions can register brand new 'actions', and extend any existing api functionality, but unfortunately create and delete are special cases that don't fall under 'actions' so they can only be extended.15:38
crobinsoKiall: and the existing flavors api doesn't have anything to extend in that case, and stubbing them out probably isn't acceptable.15:39
KiallHumm, Sounds like a oversight all right..15:39
KiallStubbing it out would be acceptable IMHO..  just "raise webob.exc.HTTPNotFound()"15:40
crobinsoKiall: yeah I tried that, but then I have to 'extend', and extending calls the original function so that error is raised to the caller15:42
crobinsoKiall: I'll just tweak the wsgi.py code to do what I want and see what the devs say (it's a small change)15:42
KiallYea - Probably a better fix anyway15:43
*** LinuxJedi has joined #openstack-dev15:43
openstackgerritSandy Walsh proposed a change to openstack/nova: ComputeNode Capacity support  https://review.openstack.org/246015:46
*** zigo-_- has joined #openstack-dev15:47
*** Remco_ has quit IRC15:48
*** zigo has quit IRC15:48
*** dtroyer has joined #openstack-dev15:49
*** zzed has joined #openstack-dev15:49
*** rajaram has quit IRC15:50
maploinis it normal that about half of the quantum unittests are failing? Is this page still being maintained? http://wiki.openstack.org/QuantumUnitTestStatus (found it in the TESTING file)15:50
*** GheRivero_ has joined #openstack-dev15:50
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Merged work-in-progress for the configuration changes. Tests are still failing on the D5 tests.  https://review.openstack.org/327015:50
*** zigo has joined #openstack-dev15:55
*** reed has joined #openstack-dev15:55
*** zigo-_- has quit IRC15:56
*** guaqua has quit IRC16:00
*** cp16net has joined #openstack-dev16:00
openstackgerritVerification of a change to openstack/keystone failed: Implement Secure Token Auth  https://review.openstack.org/288916:02
openstackgerritThierry Carrez proposed a change to openstack/swift: Drop eval-based json parser fallback  https://review.openstack.org/330416:03
*** gyee has joined #openstack-dev16:05
*** bencherian has joined #openstack-dev16:08
*** rnirmal has joined #openstack-dev16:09
Vekcan I get mtaylor or jeblair to look at https://review.openstack.org/#change,3190 please?  devstack keeps failing to start nova-api, but I can't replicate and need more information.16:11
Vek(smokestack passes it, btw)16:11
openstackgerritSapan Kona proposed a change to openstack/tempest: Wrote testcases for List Floating IP Operations  https://review.openstack.org/313616:11
*** adjohn has joined #openstack-dev16:12
*** LinuxJedi has quit IRC16:12
*** LinuxJedi has joined #openstack-dev16:12
openstackgerritZhongyue Luo proposed a change to openstack/nova: nova flags list tool  https://review.openstack.org/321016:13
*** deshantm_ has joined #openstack-dev16:14
openstackgerritThierry Carrez proposed a change to openstack/swift: Drop eval-based json parser fallback  https://review.openstack.org/330416:15
openstackgerritp-draigbrady proposed a change to openstack/nova: API version check cleanups  https://review.openstack.org/330516:16
*** deshantm has quit IRC16:18
openstackgerritJohn Griffith proposed a change to openstack/nova: Jays recommendations  https://review.openstack.org/330616:26
openstackgerritJohn Griffith proposed a change to openstack/nova: blueprint solidfire-san-iscsidriver  https://review.openstack.org/257516:26
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288916:27
*** bencherian has quit IRC16:27
*** adjohn has quit IRC16:28
adam_gjeblair: around?16:28
*** adjohn has joined #openstack-dev16:28
openstackgerriteglynn proposed a change to openstack/glance: Client.add_image() accepts image data as iterable.  https://review.openstack.org/330716:29
*** bepernoot has quit IRC16:30
*** utlemming_afk is now known as utlemming16:31
*** Tribaal has quit IRC16:33
*** ewindisch has quit IRC16:33
*** reidrac has quit IRC16:36
*** Leseb has quit IRC16:36
*** cdub has quit IRC16:36
openstackgerritVerification of a change to openstack/keystone failed: Implement Secure Token Auth  https://review.openstack.org/288916:37
*** zaitcev has joined #openstack-dev16:39
*** Tribaal has joined #openstack-dev16:45
openstackgerritCole Robinson proposed a change to openstack/nova: extensions: Allow registering actions for create + delete  https://review.openstack.org/330816:46
openstackgerritCole Robinson proposed a change to openstack/nova: Add an API extension for creating+deleting flavors  https://review.openstack.org/319716:46
openstackgerritTrey Morris proposed a change to openstack/nova: get_instance_nw_info() now returns model, virt shim and translation in place for virts to get at the old stuff  https://review.openstack.org/330916:48
Kiallcrobinso: something looks very wrong with that patchset ;)16:48
*** ncode has joined #openstack-dev16:49
*** ncode has joined #openstack-dev16:49
crobinsoKiall: hmm where is the wrongness?16:50
Kiall+2610, -2138 lines of code?16:50
Kialloh16:51
Kiallwhat the16:51
Kiallits a gerrit bug16:51
Kialltry adjust the "old version history" box to Patch Set 1..16:51
mtaylorKiall: point me at bug?16:51
KiallI wanted to see the differences16:51
Kiallmtaylor: on https://review.openstack.org/319716:51
Kiallsee it?16:52
crobinsoKiall: ahh, probably showing the diff between head when I submitted the first patch and current head16:52
*** _adjohn has joined #openstack-dev16:53
KiallAFAIK, That should show the diff between PS1 and PS216:53
*** lloydde has joined #openstack-dev16:53
*** adjohn has quit IRC16:53
*** _adjohn is now known as adjohn16:53
KiallLike it does here: https://review.openstack.org/#change,319516:53
*** lloydde has quit IRC16:53
*** rnirmal has quit IRC16:54
openstackgerritJohn Griffith proposed a change to openstack/nova: blueprint solidfire-san-iscsidriver  https://review.openstack.org/257516:54
crobinso...maybe I'm blind, but how do I submitted responses to review comments? I can't seem to get them out of 'draft'16:54
Kiallhit "review" on the main patchset page and it'll publish them16:55
crobinsoKiall: thanks, that did it16:57
*** rkukura has quit IRC16:57
*** maploin has quit IRC16:57
Kiallmtaylor: see what happens with that review?16:57
* crobinso is a bit more accustomed to patch-review-on-mailing-list :)16:57
Kiallcrobinso: np16:57
*** lloydde has joined #openstack-dev16:57
mtaylorKiall: I don't? where should I be looking?16:58
Kiallon https://review.openstack.org/#change,3197, change the "Old Version History" to Patch Set 1, I end up getting about 2k lines of code changed that didnt change16:59
Kialleg this: http://dl.dropbox.com/u/1400487/gerrit.png17:00
mtayloroh, wow!17:00
mtaylorthat's exciting17:00
* mtaylor will file that as a bug17:00
openstackgerritAndrew Hale proposed a change to openstack/swift: Fix inconsistencies in swift-init reload and restart  https://review.openstack.org/331017:01
*** rkukura has joined #openstack-dev17:03
mtaylorKiall: https://bugs.launchpad.net/openstack-ci/+bug/92054717:03
uvirtbotLaunchpad bug 920547 in openstack-ci "gerrit displays crazy amount of changes" [Undecided,New]17:03
*** pixelbeat has quit IRC17:04
*** ewindisch has joined #openstack-dev17:04
*** rnirmal has joined #openstack-dev17:05
Kiallcrobinso: is the @wsgi.action() decorator not intended for use with the resources actions endpoint? eg POST /servers/$id/action ?17:06
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288917:07
*** mattray has joined #openstack-dev17:07
KiallIt seems wrong to give that decorator a double meaning depending on the parameter..17:07
*** mattray has quit IRC17:07
*** mattray has joined #openstack-dev17:07
crobinsoKiall: really not sure tbh, the distinction wasn't clear to me17:08
Kiall@wsgi.action()'s doc block suggests so: "Mark a function as an action. The given name will be taken as the action key in the body."17:08
larissaKiall: Error: "wsgi.action()'s" is not a valid command.17:08
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288917:09
*** reed has quit IRC17:10
crobinsoKiall: hmmm didn't see that. I guess it's an unresolved question then how to do what I want to do (allow an extension to add a create or delete operation). or whether that's supposed to be allowed...17:11
*** rnirmal has quit IRC17:11
*** rnirmal has joined #openstack-dev17:11
KiallI can only imagine it is supposed to be allowed, I guess you'll need to find someone more familiar with the OS extension mechanism to get an answer on that..17:12
crobinsoKiall: hmm well Vek just ack'd both patches :)  Vek: no problems with overloading @wsgi.action like that?17:15
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288917:15
Kiallcrobinso: At the very least a comment needs updating ;)17:15
*** troytoman-away is now known as troytoman17:16
smoseris there an expected time to wait after signing contributors agreement and before being able to 'git review'17:16
Kiallsmoser: did you join the launchpad group?17:17
smoserno17:17
smoseri see that now17:17
crobinsoKiall: agreed17:17
openstackgerritVerification of a change to openstack/quantum failed: bug 920299: remove duplicate + outdate README  https://review.openstack.org/328917:17
uvirtbotLaunchpad bug 920299 in quantum "quantum has outdated + duplicate README" [Medium,In progress] https://launchpad.net/bugs/92029917:17
*** troytoman has quit IRC17:21
*** troytoman-away has joined #openstack-dev17:21
*** lloydde has quit IRC17:21
*** pvo has quit IRC17:22
*** pvo has joined #openstack-dev17:22
*** ChanServ sets mode: +v pvo17:22
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add flavor create/delete support  https://review.openstack.org/331117:23
smoserok. i've requested to be in the launchpad group. hopefully someone can ack me17:24
*** Leseb has joined #openstack-dev17:26
*** Leseb_ has joined #openstack-dev17:27
openstackgerritVerification of a change to openstack/keystone failed: Implement Secure Token Auth  https://review.openstack.org/288917:27
*** rbasak has quit IRC17:29
*** rbasak has joined #openstack-dev17:29
*** Leseb has quit IRC17:30
*** Leseb_ is now known as Leseb17:30
smoseri guess anyone how is nova, glance, swift core can do it17:30
*** sleepsonthefloo has joined #openstack-dev17:32
sleepsonthefloocrobinso - cool if I start moving the horizon usage ui over to novaclient, based off your branch?17:34
*** LinuxJedi has quit IRC17:36
*** LinuxJedi has joined #openstack-dev17:37
Kiallsleepsonthefloo: horizon dev?17:37
*** Mandell has joined #openstack-dev17:38
Kiallany suggestions for the final piece of this? https://review.openstack.org/326817:38
sleepsontheflooKiall - yes, crobinso proposed a novaclient usage api for simple-usage that I expect will land today.  I want to have the associated horizon changes ready to go so we can kick out openstackx by e317:39
*** guaqua has joined #openstack-dev17:40
KiallNot quite what I was getting at ;) I have a Horizon review pending with a question .. ;)17:40
*** GheRivero_ has quit IRC17:41
Kiallmind taking a look for me?17:41
sleepsontheflooKiall - sure17:41
KiallIts #3268 -  https://review.openstack.org/326817:42
KiallBasically, I haven't noticed anywhere in Horizon that does form level, rather than field level validation errors..17:42
sleepsontheflooKiall - have you thought about different-tenant source group rules?17:42
Kiallnot yet, I figured with the e3 feature freeze being so soon same tenant was better than nothing..17:43
KiallAlso, I'm not sure novaclient support different-tenant source group rules, although I could be mistaken..17:44
sleepsontheflooKiall - no, it doesn't, and I'm not sure that we can even do it with the existing nova api.  we may need a get_by_name security group call, or some such17:45
mtaylorsmoser: I've approved you - it'll take up to 15 minutes for that to sync to gerrit, but then you should be good to go17:45
Kiallsleepsonthefloo: right, hence I went with same-tenant source group rules :)17:46
Kiallanyway, any suggestions for how the validation should be handled and displayed?17:46
sleepsontheflooKiall: :) - just trying to imagine how this ui would extend when we add that17:46
KiallYea, I couldnt find anywhere else that did form level rather than field level errors17:47
*** Leseb has left #openstack-dev17:49
*** Leseb has joined #openstack-dev17:50
*** cdub has joined #openstack-dev17:51
*** maplebed has joined #openstack-dev17:53
*** bencherian has joined #openstack-dev17:55
*** bencherian has quit IRC17:56
*** bencherian has joined #openstack-dev17:57
*** rkukura has quit IRC17:57
*** pixelbeat has joined #openstack-dev17:59
*** jdg has joined #openstack-dev18:01
*** jog0 has joined #openstack-dev18:04
*** rkukura has joined #openstack-dev18:08
*** bencherian has quit IRC18:09
openstackgerritScott Moser proposed a change to openstack/python-novaclient: do not require NOVA_VERSION in env, default to 1.1  https://review.openstack.org/331218:10
dprincemtaylor: Are we still planning on automatically marking stale merge props as abandoned? Any news on this front?18:12
* dprince dprince tires of seeing the same old red -2's every day18:13
*** derekh has quit IRC18:13
dprincemtaylor: something like if a branch has -2's on it and its been sitting there 2-3 weeks we auto abandon it?18:14
*** Ryan_Lane has joined #openstack-dev18:16
mtaylordprince: yes. we've got it written - have just been testing it18:17
mtaylordprince: and then we didn't want to roll anything new out milestone week (in the past, that's led to pain)18:17
*** bepernoot has joined #openstack-dev18:17
dprincedprince: awesome. sure. sounds good.18:18
dprincemtaylor: ^^18:18
openstackgerritKiall Mac Innes proposed a change to openstack/horizon: Add support for same tenant source-group security rules. Fixes bug #860780.  https://review.openstack.org/326818:18
uvirtbotLaunchpad bug 860780 in horizon "security groups should allow control via source groups" [High,In progress] https://launchpad.net/bugs/86078018:18
mtaylordprince: the initial scheme is going to be something with negative reviews which is untouched for 1 week, or something with no activity for 2 weeks gets auto-abandoned18:18
mtaylordprince: configurable, of course, once we get feedback from folks on taht18:18
openstackjenkinsProject nova-docs build #2005: SUCCESS in 2 min 59 sec: https://jenkins.openstack.org/job/nova-docs/2005/18:18
openstackjenkinsThierry Carrez: Cleanups for rootwrap module18:18
dprincemtaylor: sure. That sounds fine to me.18:18
Kiallsleepsonthefloo: just updated that review a bit, it all works.. but the form-level validation error messages are in the wrong place..18:19
dprincemtaylor: Nova has 95 reviews now so it'd be nice to pull the trigger on it soon. You have my vote anyway.18:19
dprincemtaylor: maybe this would get rid of 30-40 of them.18:20
mtaylordprince: I'm pretty sure it will!18:20
*** jdurgin has joined #openstack-dev18:22
*** bencherian has joined #openstack-dev18:27
openstackgerritScott Moser proposed a change to openstack/python-novaclient: do not require NOVA_VERSION in env, default to 1.1  https://review.openstack.org/331218:32
openstackjenkinsProject nova-docs build #2006: SUCCESS in 3 min 6 sec: https://jenkins.openstack.org/job/nova-docs/2006/18:33
openstackjenkinsThierry Carrez: Fix environment passing in DnsmasqFilter18:33
crobinsosleepsonthefloo: sorry missed your ping. please go ahead with horizon changes WRT usage18:34
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288918:35
sleepsonthefloocrobinso.  roger - I'll get it done asap18:36
sleepsontheflooKiall - added some feedback18:36
*** flaviamissi has joined #openstack-dev18:36
Kiallgot the email, thanks..18:36
*** bencherian has quit IRC18:37
*** eglynn has quit IRC18:38
*** eglynn has joined #openstack-dev18:39
*** bencherian has joined #openstack-dev18:42
*** darraghb has quit IRC18:45
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add a 'usage' module and 'usage-list' cli command  https://review.openstack.org/328518:47
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add flavor create/delete support  https://review.openstack.org/331118:47
openstackjenkinsProject nova-docs build #2007: SUCCESS in 3 min 7 sec: https://jenkins.openstack.org/job/nova-docs/2007/18:48
openstackjenkinsThierry Carrez: Add a specific filter for kill commands18:48
openstackgerriteglynn proposed a change to openstack/glance: glance-api fails fast if default store unsupported  https://review.openstack.org/329718:49
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add flavor create/delete support  https://review.openstack.org/331118:50
*** novas0x2a|laptop has joined #openstack-dev18:58
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288918:58
vishy_0x44: ping18:59
*** bencherian has quit IRC18:59
crobinsoVek, bcwaldon: ping re https://review.openstack.org/#change,330819:00
*** kyriakos has left #openstack-dev19:01
bcwaldoncrobinso: what up19:01
openstackgerritA change to openstack/nova has been rejected: get_instance_nw_info() now returns model, virt shim and translation in place for virts to get at the old stuff  https://review.openstack.org/330919:01
crobinsoVek, bcwaldon: you guys okay with overloading wsgi.action to work with create + delete ops, or would it be better if I added another decorator like wsgi.replace like Kiall suggested?19:01
*** bencherian has joined #openstack-dev19:01
KiallIts also probably worth mentioning that that change prevents any use of "create" or "delete" as actual actions, rather than HTTP POST/DELETE19:02
KiallI'm not sure if that matters, considering the semantics of it, but worth mentioning..19:03
*** zns1 has joined #openstack-dev19:03
*** zns has quit IRC19:04
crobinsoKiall: yeah more proof that my patch is a hack :) I think your decorator suggestion is a good one, just want to make sure maintainers are fine with it19:04
KiallYea.. wsgi.replace() sounds right to me.. but with that name, it would (functionally) need to ensure that the original (if any) was not called.. But, not up to me etc etc ;)19:06
*** zns1 has quit IRC19:07
*** martine has joined #openstack-dev19:09
*** zns has joined #openstack-dev19:09
*** vinilios has quit IRC19:11
crobinsodprince: ping, re the review you just did. any opinion on the back chat ^^19:13
*** GheRivero_ has joined #openstack-dev19:13
crobinsodprince: adding a new wsgi decorator rather than overloading wsgi.action to work with create + delete19:13
dprincecrobinso: reading...19:14
*** lloydde has joined #openstack-dev19:16
*** adjohn has quit IRC19:16
*** darthflatus has joined #openstack-dev19:17
crobinsodprince: and per your question in the review, I have a flavor related extension also proposed that makes use of the patch you commented on19:17
*** zns has quit IRC19:17
bcwaldoncrobinso: will the 'wsgi.extends' decorator not work for your create/delete problem?19:17
openstackjenkinsProject nova-docs build #2008: SUCCESS in 3 min 15 sec: https://jenkins.openstack.org/job/nova-docs/2008/19:19
openstackjenkinsjustin: Support filter based on CPU core (over)allocation19:19
crobinsobcwaldon: no, it's a limitation of the current code (to the best of my understanding)19:19
crobinsobcwaldon: 'extends' can be used for brand new 'actions' (which have a specific definition that does not include create or delete)19:19
crobinsobcwaldon: or it can be used to extend an operation that already exists in the resource being extended19:20
crobinsobcwaldon: which is additive, not replacing, so the original method can't just be a NotImplemented stub or the code chokes19:20
bcwaldoncrobinso: and the case you have is a completely new method, create and delete?19:20
Kiallcrobinso: really? I thought @wsgi.action('action') == new action, @wsgi.extends('action') == extend action, and @wsgi.action() == extend a normal method19:21
crobinsoKiall: hmm, lemme check19:21
crobinsoKiall: ah yes, sorry I wasn't clear. you can use 'extends' to extend an action19:23
vishyVek: ^^19:23
Kiallright - so I'm not going mad (yet)19:23
crobinsoKiall: 'extends' can extend anything, and it can also be used to patch in brand new actions19:23
crobinsoKiall: problem is there is no infrastructure to patch in brand new non-'actions'19:24
KiallYea, I get that.. :)19:24
crobinsoKiall: (right, sorry, I obv don't need to explain this to you :) )19:25
dprincecrobinso: You can add new subresources and do that right? Essentially just add a new resource extension and have at it right?19:25
crobinsodprince: if I follow, you mean just create a whole new api namespace like /os-flavor-manage and add commands there? that does work, but reviews on the first round of my submission seemed to suggest I look into plugging my functionality into the existing /flavors entry point. that's were the infrastructure is lacking AIUI19:27
bcwaldoncrobinso: yes, keeping it on /flavors is preferred19:27
dprincecrobinso: ah. Gotcha. I was missing context. Thanks.19:27
vishyi guess Vek already +2ed so he must be ok with the approach19:28
*** martine has quit IRC19:29
dprincecrobinso: Either approach (decorator or not) looks like it might work to me.19:29
openstackgerritRick Harris proposed a change to openstack/python-novaclient: Print adminPass when rescuing an instance.  https://review.openstack.org/331319:30
*** rkukura has quit IRC19:32
*** rkukura has joined #openstack-dev19:32
*** deshantm__ has joined #openstack-dev19:32
*** MrBIOS has joined #openstack-dev19:34
*** maplebed_ has joined #openstack-dev19:34
jeblairadam_g: i am now19:34
*** maplebed_ is now known as Guest5025619:34
*** Guest50256 has quit IRC19:35
*** deshantm_ has quit IRC19:36
*** rkukura has quit IRC19:37
*** maplebed has quit IRC19:37
*** zns has joined #openstack-dev19:41
vishydprince: ping19:41
*** rkukura has joined #openstack-dev19:45
*** bepernoot has quit IRC19:47
openstackjenkinsProject nova-docs build #2009: SUCCESS in 3 min 31 sec: https://jenkins.openstack.org/job/nova-docs/2009/19:49
openstackjenkinsVishvananda Ishaya: Add context and request spec to filter_properties19:49
*** kbringard has quit IRC19:52
*** kbringard has joined #openstack-dev19:53
openstackgerritJay Pipes proposed a change to openstack/glance: Bug #909574: Glance does not sanity-check given image size on upload  https://review.openstack.org/272719:53
uvirtbotLaunchpad bug 909574 in glance "Glance does not sanity-check given image size on upload" [Medium,In progress] https://launchpad.net/bugs/90957419:53
openstackgerritDan Prince proposed a change to openstack/nova: Fixes nova-manage fixed list.  https://review.openstack.org/331419:54
openstackgerritDan Prince proposed a change to openstack/nova: Fixes nova-manage fixed list.  https://review.openstack.org/331419:55
jaypipesblamar, bcwaldon: if you get a moment... https://review.openstack.org/#change,272719:55
crobinsosleepsonthefloo: in that usage_list vs. usage_show comment, you meant for a cli command?19:55
bcwaldonjaypipes: will get to it today19:55
sleepsonthefloocrobinso - yes, for cli19:55
jaypipesbcwaldon: cheers19:56
Vekcan I get mtaylor or jeblair to look at https://review.openstack.org/#change,3190 please?  devstack keeps failing to start nova-api, but I can't replicate and need more information.  (Smokestack passes it, btw...)19:59
*** bepernoot has joined #openstack-dev19:59
*** zns has quit IRC20:00
sleepsonthefloocrobinso - looks like nova's usage db query may be off a bit20:01
sleepsonthefloonova.db.api.sqlalchemy.instance_get_active_by_window may need some tweaking, I'll see if I can track down the issue20:01
openstackgerritVerification of a change to openstack/nova failed: Adds availability zone filter  https://review.openstack.org/325120:03
jeblairVek: did you see the last two lines of the syslog?20:03
*** sleepsonthefloo has quit IRC20:03
jeblairhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/671/artifact/logs/syslog.txt20:03
*** zns has joined #openstack-dev20:04
Vekjeblair: no, I did not.  I looked only at the console output.20:05
*** bpg has quit IRC20:05
jeblairso the devstack jobs capture syslog and keep it as a "build artifact" in jenkins20:05
jaypipesmtaylor: heya, what's the deal with this bug? https://bugs.launchpad.net/glance/+bug/81993820:05
uvirtbotLaunchpad bug 819938 in glance "setup.py needs to define dependencies" [Medium,Triaged]20:05
jeblairhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/671/20:05
jaypipesmtaylor: are we going the install_requires route still? If not, could you mark that bug Invalid?20:06
Vekjeblair: OK.  So, now the error is revealed--the devstack job is running with a really old api-paste.ini20:06
jeblairif you look at that page, you'll see "syslog.txt" under "Build Artifacts" just under the build number in the center20:06
Vekand my patch updates api-paste.ini yet again.20:06
Vekany chance I can get you to work around that for me?20:06
jeblairVek: I thought a patch went in recently that caused devstack to use api-paste.ine from nova?20:07
jaypipesVek: mind taking another look at https://bugs.launchpad.net/glance/+bug/910233 and letting me know if that is still relevant?20:07
uvirtbotLaunchpad bug 910233 in glance "Keystone now returning UUID tenant IDs, resulting in test failures" [Medium,Confirmed]20:07
Vekwell, it wasn't there on Friday, which is when this patch got approved :)20:07
jaypipesbcwaldon: for https://blueprints.launchpad.net/glance/+spec/interim-glance-authz-service, is the only remaining task documenting how the policy stuff works and can be configured?20:08
Vekcould also be a problem with nova.conf calling for the wrong thing...20:09
*** bencherian has quit IRC20:09
Vekjaypipes: last time I ran the test suite, I did not see that error20:09
Vekunless it's been @skip'd in the interrim, it's probably no longer an issue.20:10
*** bencherian has joined #openstack-dev20:11
jaypipesVek: k, thx20:11
jeblairVek: I can't work around this, i can help out if the job isn't running correctly, but you should have all the tools needed to fix it (by running and patching devstack if needed)20:12
Vekjeblair: I think I see the problem now.20:13
* Vek hrms20:13
Vekor maybe not...20:13
Vekyeah, I was right, and it is devstack's fault20:14
openstackgerritVerification of a change to openstack/tempest failed: Added flavor filter tests * Resolved lp bug#899979, bug#899980, and bug#899982 * Modified flavor service to return results in line with other services * Added bug flag to tests that are failing due to known issues  https://review.openstack.org/295520:14
Vekdevstack is trying to set up to use Keystone20:14
Vekbut my patch changes the pipeline to remove the serialize and extension middleware20:14
Vekso devstack replaces the pipeline with a Keystone-compliant pipeline which resurrects the serialize middleware20:15
Vekwhich no longer exists20:15
Vekand so it fails.20:15
openstackgerritVerification of a change to openstack/tempest failed: Added flavor filter tests * Resolved lp bug#899979, bug#899980, and bug#899982 * Modified flavor service to return results in line with other services * Added bug flag to tests that are failing due to known issues  https://review.openstack.org/295520:16
Vekso, in order to fix this, I have to patch devstack?20:17
jeblairVek: likely so.  there was a patch recently to devstack that used the paste config from nova, but then added keystone stuff to it.  i guess that's the bit that you're talking about now.20:18
openstackgerritCole Robinson proposed a change to openstack/nova: usage: Fix time filtering  https://review.openstack.org/328420:19
openstackgerritCole Robinson proposed a change to openstack/nova: Add an API extension for creating+deleting flavors  https://review.openstack.org/319720:19
openstackgerritCole Robinson proposed a change to openstack/nova: extensions: Allow registering actions for create + delete  https://review.openstack.org/330820:19
Vekit would seem so.20:19
openstackjenkinsProject nova-docs build #2010: SUCCESS in 3 min 40 sec: https://jenkins.openstack.org/job/nova-docs/2010/20:19
openstackjenkinsVishvananda Ishaya: Adds isolated hosts filter20:19
jeblairsince these have to go in in sequence, it would be great if devstack could be changed so that it didn't add anything to the paste config20:19
Veknow, the problem is...where do I find devstack to apply a change to it?20:19
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288920:21
Vekjeblair: Where do I find devstack so I can fork it and propose a patch?  I don't see it under github/openstack, which means either it's somewhere else or has a different name than I'm expecting.20:22
*** eglynn has quit IRC20:23
jeblairhttps://github.com/openstack-dev/devstack20:23
jeblairVek: you may want to chat with vishy or sleepsonthefloor20:23
jeblairgetting that change past the integration test could be tricky20:24
Vek*nod*20:25
*** eglynn has joined #openstack-dev20:26
Vekand I've found the relevant section of stack.sh that's to blame.20:26
Vekvishy or sleepsonthefloor, are either of you present?20:27
*** sleepsonthefloo has joined #openstack-dev20:29
*** sleepsonthefloo has quit IRC20:30
jeblairthat was unfortunate :(20:30
openstackgerritDean Troyer proposed a change to openstack/nova: Handle error in associate floating IP (bug 845507)  https://review.openstack.org/331520:30
uvirtbotLaunchpad bug 845507 in nova "No error is returned if public interface doesn't exist" [Low,In progress] https://launchpad.net/bugs/84550720:30
*** sleepsonthefloo has joined #openstack-dev20:30
jeblairsleepsonthefloo: Vek has a question about a devstack/nova change20:31
sleepsonthefloojeblair, Vek - hey20:31
VekHaving a problem with https://review.openstack.org/#change,3190 merging due to devstack20:32
openstackgerritVerification of a change to openstack/keystone failed: Implement Secure Token Auth  https://review.openstack.org/288920:32
Vekthe problem is, that patch modifies the pipeline, but devstack doesn't know that.20:32
Vekand since I removed the old filters from the api-paste.ini, when it tries to add them back, it bombs.20:33
Vekat this point, it would appear I have two options--revert my changes to api-paste.ini or update the pipelines in devstack20:33
vishyVek: don't modify the pipeline!20:33
vishy:)20:33
crobinsosleepsonthefloo: that usage problem you mention in nova, I submitted a patch for it yesterday (and did a v2 today). so ack's appreciated :)20:34
Vekvishy: I updated the pipeline to remove deprecated and now-do-nothing middleware.20:34
openstackgerritA change to openstack/nova has been rejected: Add an API extension for creating+deleting flavors  https://review.openstack.org/319720:34
vishyVek: the middleware currently does nothing?20:34
vishyVek: so we can remove it before your patch lands?20:34
Vekvishy: with my patch, the middleware does nothing.20:35
vishyVek: so i think you probably need to leave it in as a stub20:35
Vekbefore my patch, the middleware also does nothing, but the code is still there and could potentially be used.20:35
vishyoh20:35
sleepsonthefloocrobinso - coolio, looking20:35
VekI have in the patch; marked with @utils.deprecated()20:35
vishyVek: if tests will pass without it20:35
Vekunit tests will20:36
Vekdevstack can't, because it restores the middlewares to the pipeline, but their defining sections are gone.20:36
vishyVek: will devstack run without the middlewares in the pipeline before your patch lands?20:36
Vekit *should*20:36
vishyVek: if so, then just propose a patch to devstack20:36
vishyVek: removing the middlewares20:36
*** eglynn has quit IRC20:37
VekOK.20:37
jeblairVek: devstack is in gerrit, so it's the same process20:37
Vekjeblair: yeah, I looked that up; I just needed to know the difference between openstack and openstack-dev :)20:37
vishypipeline changes are still a little painful :)20:37
Vekvishy: ya think?  :)20:37
vishyVek: at least I minimized the pain to just the pipeline20:38
Vektrue :)20:38
vishyVek: it used to be any change to paste config blew up20:38
openstackgerritphilip-knouff proposed a change to openstack/nova: Fixed limiting for flavors  https://review.openstack.org/331620:38
*** stuntmachine has quit IRC20:38
Vekyeah, I know.20:39
johan_-_is anyone from the CI team around? i need to get a small change made to the integration test setup20:39
jeblairjohan_-_: hi20:39
johan_-_hi jeblair20:39
johan_-_a merge last week because of a change to the unit tests20:40
bcwaldonjaypipes:20:40
johan_-_https://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/592/console20:40
johan_-_the migrations are dependent on the connection_type in the branch20:40
jaypipesbcwaldon: ?20:40
bcwaldonjaypipes: that image size branch20:40
bcwaldonjaypipes: is it ready to go in again?20:40
johan_-_and the integration tests don't specify a connection_type20:41
jaypipesbcwaldon: this one? https://review.openstack.org/#change,272720:41
bcwaldonthats the one20:41
jaypipesbcwaldon: yep, should be.20:41
jaypipesbcwaldon: feel free to pull the trigger.20:41
openstackgerritKevin L. Mitchell proposed a change to openstack-dev/devstack: Update pipelines  https://review.openstack.org/331720:41
Vekwas about to paste when openstackgerrit beat me to it :)  Mind taking a look at the patch?20:42
bcwaldonjaypipes: kk, curious why we use 1 << 50 instead of just typing the value out20:42
jaypipesbcwaldon: meh...20:42
jeblairjohan_-_: that's probably something that can be fixed in devstack20:42
*** jakedahn has joined #openstack-dev20:43
johan_-_jeblair: so get a change to devstack merged first and then merge my branch?20:43
jeblairjohan_-_: the job uses devstack to set up the environment20:43
jeblairhttps://github.com/openstack-dev/devstack20:43
jeblairjohan_-_: yes20:43
johan_-_jeblair: does it automatically use the latest devstack?20:43
johan_-_s/does it/does jenkins/20:43
jeblairjohan_-_: yes, they are all tied together -- your change to devstack will need to work with the latest nova first.  once that's in, you can re-approve the change to nova, and it will use the latest devstack.20:44
johan_-_ok, i can do that. thanks20:44
*** dprince has quit IRC20:44
blamarbcwaldon: do you know offhand that 1125899906842624 is 1 PiB? :)20:45
bcwaldonblamar: don't get sassy with me20:45
Vekheh20:45
bcwaldonblamar: I also don't know 1 << 50 is a PiB20:45
blamarbcwaldon: noob!20:45
crobinsobcwaldon: regarding your comment about policy.json and the like, does that also mean dropping context.is_admin checks in the flavorsmanager extension?20:46
bcwaldonyes sir20:46
*** bepernoot has quit IRC20:47
bcwaldonwe're on a road to more automated authorization of extensions20:47
bcwaldonmost of the way there20:47
blamarbcwaldon: are we there yet?20:48
openstackgerritVish Ishaya proposed a change to openstack/nova: Adds availability zone filter  https://review.openstack.org/325120:49
bcwaldonblamar: don't make me turn this car around, mister20:49
*** rkukura has quit IRC20:49
mtaylorjaypipes: no, we are not going the install_requires route still for the main server projects, because we aren't really expecting them to be installed via pipi20:52
mtaylorjaypipes: pip, that is20:52
jaypipesmtaylor: coolio. so, can I mark that as Invalid?20:52
mtaylorjaypipes: we _are_ expecting client libs to have proper install_requires20:52
jaypipesk20:52
mtaylorjaypipes: yeah - the problem should be taken care of when we do client split for glance20:52
*** Remco_ has joined #openstack-dev20:52
mtaylorjaypipes: I just did the split for quantum, so I think i might be able to make you a good patch20:53
mtaylorwell, I did the first pass at the split for quantum - haven't gotten the final patched merged into quantum itself yet...20:53
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Release Notes for E3  https://review.openstack.org/331820:54
jaypipesmtaylor: cheers, I'll update the bug.20:55
*** markwash has joined #openstack-dev20:57
*** eglynn has joined #openstack-dev21:00
*** glenc_ has joined #openstack-dev21:01
*** glenc has quit IRC21:04
*** glenc_ is now known as glenc21:04
crobinsobcwaldon: okay... so the flavormanage module uses 2 main apis: instance_types.delete and instance_types.create. where do I add the policy checks? nova/compute/instance_types.py? some wrapper in nova/compute/api.py? just open code them in the flavormanage module?21:05
crobinsobcwaldon: and in the actual policy.json file, will the access checks have an instance_type: prefix or a compute: prefix?21:06
bcwaldoncrobinso: actually, for now what I would prefer you do is just add the extension-level rules21:06
bcwaldonwe will need authz at the lower levels but that needs to be a different patch21:06
crobinsobcwaldon: hmm what does that mean exactly? :)21:07
bcwaldonlook at nova/api/openstack/compute/contrib/admin_actions.py21:07
openstackgerritDean Troyer proposed a change to openstack/nova: Handle error in associate floating IP (bug 845507)  https://review.openstack.org/331521:08
uvirtbotLaunchpad bug 845507 in nova "No error is returned if public interface doesn't exist" [Low,In progress] https://launchpad.net/bugs/84550721:08
bcwaldoncrobinso: hmm, hold on a sec21:08
bcwaldoncrobinso: I'm sorry, I thought my patch had gotten in21:08
bcwaldonfeel free to ignore me for now and I'll do it in my follow-up review21:08
bcwaldoncrobinso: your authz stuff is fine for now21:09
crobinsobcwaldon: okay, sounds good. so what changes do I need for this current iteration, just dropping admin_only ?21:09
bcwaldonno, you don't need to do anything extra21:10
*** glenc_ has joined #openstack-dev21:10
*** markmc has quit IRC21:10
bcwaldonI remove it in one of my patches, and I thought that had made it into trunk21:10
crobinsobcwaldon: ah I see, sounds good. I think my patch needs the -1 undone then :)21:12
bcwaldonyes, good point21:12
*** glenc has quit IRC21:12
kbringardanyone know off the top of their head if the keystone admin api can be used to manage (create, delete, etc) tenants and users, or if it's just for tokens and endpoints and stuff?21:14
Kiallkbringard, i assume it can since horizon can...21:14
kbringardhmmm, good point21:15
vishykbringard: yes it can21:15
vishykbringard: although it is through an extension21:15
*** bepernoot has joined #openstack-dev21:15
kbringardvishy, Kiall: awesome… I'll check out the dashboard code and see how it's done there as a place to start21:16
kbringardthanks!21:16
openstackgerritphilip-knouff proposed a change to openstack/nova: Fixed limiting for flavors  https://review.openstack.org/331621:17
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add a 'usage' module and 'usage-list' cli command  https://review.openstack.org/328521:18
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add flavor create/delete support  https://review.openstack.org/331121:18
openstackjenkinsProject nova-docs build #2011: SUCCESS in 2 min 44 sec: https://jenkins.openstack.org/job/nova-docs/2011/21:18
openstackjenkinsVishvananda Ishaya: Adds availability zone filter21:18
*** bepernoot has quit IRC21:19
openstackgerritBrian Waldon proposed a change to openstack/nova: Remove admin_only ext attr in favor of authz  https://review.openstack.org/323621:22
*** flaviamissi has quit IRC21:22
openstackgerritTrey Morris proposed a change to openstack/nova: get_instance_nw_info() now returns model, virt shim and translation in place for virts to get at the old stuff  https://review.openstack.org/330921:24
znskbringard: the extension for keystone to manage users & tenants is documented in the extension registry (look for OS-KSADM here http://docs.rackspace.com/openstack-extensions/) with examples here http://keystone.openstack.org/adminAPI_curl_examples.html21:24
*** Leseb has quit IRC21:24
kbringardzns: w00t, thanks mate21:24
*** GheRivero_ has quit IRC21:25
*** lts has quit IRC21:25
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth  https://review.openstack.org/288921:26
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add a 'usage' module and 'usage-list' cli command  https://review.openstack.org/328521:29
openstackgerritCole Robinson proposed a change to openstack/python-novaclient: Add flavor create/delete support  https://review.openstack.org/331121:29
openstackgerritAlex Meade proposed a change to openstack/nova: Instances to be created with a bookmark link  https://review.openstack.org/332021:30
vishycomstud: pingzors21:34
vishymtaylor, jeblair: do we have a process for adding members to devstack-core?21:37
vishyor is it still the wild west :)21:37
openstackgerriteglynn proposed a change to openstack/glance: glance-api fails fast if default store unsupported  https://review.openstack.org/329721:40
bcwaldonVek: https://review.openstack.org/#change,3308 << +2 if you're good21:40
openstackgerritphilip-knouff proposed a change to openstack/nova: Fixed limiting for flavors  https://review.openstack.org/331621:42
bcwaldonpknouff: ping21:42
*** andrewsmedina has quit IRC21:42
bcwaldonblamar: beep21:43
blamarbcwaldon: sup21:45
bcwaldonblamar: is pknouff around?21:46
bcwaldonblamar: need to hash out this flavor marker stuff21:46
openstackgerritJohn Dickinson proposed a change to openstack/swift: removed tools/rfc.sh  https://review.openstack.org/332121:47
openstackjenkinsProject nova-docs build #2012: SUCCESS in 2 min 58 sec: https://jenkins.openstack.org/job/nova-docs/2012/21:48
openstackjenkinsSandy Walsh: ComputeNode Capacity support21:48
*** zns has quit IRC22:02
openstackgerritTres Henry proposed a change to openstack/horizon: Fixing default for boot from volume (from /dev/vda to vda)  https://review.openstack.org/332222:04
*** andrewsmedina has joined #openstack-dev22:07
*** prectechCharlie has joined #openstack-dev22:07
*** prectechCharlie has left #openstack-dev22:07
johan_-_is github.com/cloudbuilders/devstack canonical or is github.com/openstack-dev/devstack canonical?22:07
*** glenc_ has quit IRC22:07
*** glenc has joined #openstack-dev22:08
*** glenc has quit IRC22:09
*** zns has joined #openstack-dev22:09
openstackgerritphilip-knouff proposed a change to openstack/nova: Fixed limiting for flavors  https://review.openstack.org/331622:09
jeblairjohan_-_: github.com/openstack-dev/devstack is canonical (and is what jenkins uses)22:09
openstackgerritRick Harris proposed a change to openstack/nova: Error out instance on set password failure.  https://review.openstack.org/332322:10
johan_-_jeblair: thanks!22:10
*** rkukura has joined #openstack-dev22:11
notmynamejeblair: mtaylor: can one of you comment on https://review.openstack.org/#change,3321 (removing rfc.sh)?22:16
*** zigo has quit IRC22:17
*** maplebed has joined #openstack-dev22:18
notmynamejeblair: what's the difference in github.com/openstack and github.com/openstack-dev?22:18
openstackjenkinsProject nova-docs build #2013: SUCCESS in 3 min 21 sec: https://jenkins.openstack.org/job/nova-docs/2013/22:19
openstackjenkinsKevin L. Mitchell: Remove deprecated extension code.22:19
adam_gjeblair: sorry, was out. i'm still having LP/CLA issues but i think mtaylor was looking into it for me. i noticed your reply bug #881019 today. if theres anyhing related to my LP account you want me to check/test, lemme know22:21
uvirtbotLaunchpad bug 881019 in launchpad "Lp login is broken after account merge" [Critical,Triaged] https://launchpad.net/bugs/88101922:21
comstudvishy: pong22:24
vishycomstud: didn't want to distract you to excessively22:24
comstudi'm at castle22:25
mtaylornotmyname: looking right now22:25
comstudalready have huge distractions22:25
comstud:)22:25
vishycomstud: just curious if you had an ide of LoE to get the distributed_scheduler working with volume22:25
comstud1 or 2 more is no big deal22:25
mtaylorvishy: it's still the wild west22:25
vishymtaylor: k figured22:25
openstackgerritGabriel Hurley proposed a change to openstack/horizon: Swift exception handling improvements.  https://review.openstack.org/332422:25
mtaylorvishy: actually, all core teams are technically just managed by the core team itself - nova's core team just happens to have a process :)22:25
comstudvishy: i don't think it's much trouble if we ignore zones22:26
comstudvishy: which are going to go away anyway22:26
comstudin the scheduler22:26
comstudneed to add some stuff to the host manager for volumes22:26
*** bencherian has quit IRC22:28
mtaylorjeblair: haha. jinx. I may have stream of consciousnessed a littel more22:28
notmynamemtaylor: thanks. I don't know if there is some process for deprecating rfc.sh or not. I think we'd be safe to remove it from swift and rely on git-review, but if there is some global, coordinated process to follow, I guess we can do that22:28
comstudvishy: if i were doing it, perhaps a day of work... with a couple basic filters.. but not all of them22:28
vishycomstud: i don't think volume does much filtering22:29
vishyaz filter which should still work22:29
notmynamemtaylor: ah. cool. just saw you comment22:29
jeblairnotmyname: the idea was to try to reserve openstack/ for the actual product itself and keep meta-projects out of that org.  if it's causing people to have trouble finding important projet though, perhaps that outweight organizational purity.22:29
vishyand a max_gb filter22:29
*** Remco_ has quit IRC22:29
comstudvishy: yeah.. i guess i was thinking about vsa22:29
vishycomstud: oh…yeah that is a little tougher22:30
notmynamejeblair: but there are still meta projects in openstack/? (at least there appear to be)22:30
vishycomstud: i want to kill simple_scheduler22:30
comstudi'm guessing to duplicate simple scheduler functionality would be a day or so22:30
comstudvsa inherits from simple right now22:30
notmynamejeblair: isn't openstack-common the definition of a metaproject? ;-)22:30
mtaylornotmyname: we've been intending on moving them ... it just means coordinating a move on all of the jenkins slaves22:30
comstudso there is just some consideration there22:30
mtaylornotmyname: hehe. openstack-common will actually produce an installable lib that other projects depend on ... so it's a real code thing22:31
notmynamemtaylor: just curious since I hadn't really noticed the -dev before22:31
mtaylornotmyname: openstack-ci and openstack-ci-puppet are the ones that are in there that don't make organizational sense22:31
notmynamebut it's code about code :-)22:31
mtaylorindeed.22:31
jeblairmeta may have been the wrong word22:31
notmynamemtaylor: so what's the difference in openstack/openstack-ci and openstack-ci? ;-)22:31
mtaylornotmyname: hysterical raisins22:31
notmynamemy favorite kind22:32
mtaylornotmyname: openstack/openstack-ci is the repo with the scripts we use in jenkins and else where ... it should really live in openstack-ci22:32
*** apevec has quit IRC22:32
*** cp16net has quit IRC22:32
mtaylornotmyname: but it's checked out on all of the jenkins build slaves atm, so we'll have to do a transition, and that seems low priority given other needs :)22:32
notmynameseems like you could take that logic pretty far and have nova/ swift/ etc22:33
*** Ryan_Lane has quit IRC22:33
jeblairadam_g: it's likely that the lp folks deleted the openid other than the one we were expecting.  what "account id" is listed on this page: https://review.openstack.org/#settings22:33
*** markvoelker has quit IRC22:34
jeblairnotmyname: the logic is to try to keep the openstack project together, but separate it from potentially transient development infrastructure.22:34
vishymtaylor: ubuntu-vpc packaging branch is failing to patch22:35
vishymtaylor: I am kind of a quilt-n00b22:35
vishymtaylor: any tips?22:35
mtaylorvishy: uhm. - what's ubuntu-vpc? link?22:36
*** Ryan_Lane has joined #openstack-dev22:36
vishymtaylor: oh actually i think just need to remove the patch22:36
vishy:)22:36
notmynamejeblair: mtaylor: meh, I'm mostly doing the "hey a hornets' nest. let's poke it with a stick" thing. I'll stop now ;-)22:36
vishymtaylor: it is for smokestack lp:~dan-prince/nova/ubuntu-vpc22:37
mtaylornotmyname: it's always a fun day when you poke at the hornet's nests! :)22:37
notmynameI try to keep you on your toes22:37
*** zns1 has joined #openstack-dev22:38
notmynamemtaylor: just getting practice for the design summits ;-)22:39
vishytr3buchet: pingity22:40
vishymtaylor: trying with the patch removed.  I will let you know if i need quilt help, but I think that should do it22:40
*** zns has quit IRC22:40
mtaylorvishy: cool. you removed it from debian/patches/series too, yeah/22:41
mtaylor?22:41
mtaylornotmyname: AWESOME22:41
vishymtaylor: yup22:41
*** zns1 has quit IRC22:41
*** apevec has joined #openstack-dev22:41
openstackgerritJohannes Erdfelt proposed a change to openstack-dev/devstack: Explicitly set connection_type to libvirt  https://review.openstack.org/332522:47
openstackgerritDolph Mathews proposed a change to openstack/keystone: Fixed: Inserting URLs into endpoint version attr  https://review.openstack.org/332622:48
openstackjenkinsProject nova-docs build #2014: SUCCESS in 3 min 11 sec: https://jenkins.openstack.org/job/nova-docs/2014/22:49
openstackjenkinsRick Harris: Error out instance on set password failure.22:49
novas0x2a|laptophm. is there an easy way to load keystone as a wsgi app without launching it as a demon and without it depending on external things? (e.g. so i can feed it as a wsgi app to webtest in unit tests without actually launching it as a daemon)22:50
novas0x2a|laptop(also, i propose spelling it as dæmon everywhere)22:51
*** PaulM has quit IRC22:52
novas0x2a|laptopit kinda looks like no, but i was hoping...22:52
novas0x2a|laptopheh22:52
*** maplebed has quit IRC22:53
*** Ryan_Lane has quit IRC22:53
vishyany stable-maintainers around?22:53
vishythat could kick off https://review.openstack.org/#change,328222:54
*** Ryan_Lane has joined #openstack-dev22:54
vishyfollowed by: https://review.openstack.org/#change,302822:54
adam_gjeblair: account # 142022:54
*** LinuxJedi has quit IRC22:54
*** LinuxJedi has joined #openstack-dev22:55
jeblairadam_g: thanks, and what's your lp id?22:56
adam_gjeblair: gandelman-a22:56
*** ttx has quit IRC22:57
jeblairadam_g: the next time our gerrit/lp sync script runs (should be within 20 mins) you should be all set.22:58
openstackgerritA change to openstack-dev/devstack has been rejected: Explicitly set connection_type to libvirt  https://review.openstack.org/332522:59
adam_gjeblair: great!22:59
*** glenc has joined #openstack-dev23:00
*** glenc has quit IRC23:00
openstackgerritJohannes Erdfelt proposed a change to openstack-dev/devstack: Explicitly set connection_type to libvirt  https://review.openstack.org/332523:01
*** pixelbeat has quit IRC23:02
openstackgerritJohannes Erdfelt proposed a change to openstack-dev/devstack: Explicitly set connection_type to libvirt  https://review.openstack.org/332523:03
*** dolphm has joined #openstack-dev23:03
*** bencherian has joined #openstack-dev23:04
*** bencherian has quit IRC23:04
*** adjohn has joined #openstack-dev23:05
openstackgerritDolph Mathews proposed a change to openstack/keystone: Suppressed backtraces in tests causes sweaty eyes  https://review.openstack.org/332723:05
openstackgerritJoe Gordon proposed a change to openstack/nova: Add HACKING compliance testing to run_test.sh  https://review.openstack.org/321923:06
openstackgerritRick Harris proposed a change to openstack/nova: Remove sensitive info from rpc logging.  https://review.openstack.org/332823:07
*** Gordonz has quit IRC23:08
vishyjeblair, Daviey, zul, mtaylor: ^^ see my above question about horizon reviews23:08
*** zns has joined #openstack-dev23:09
*** mattray has quit IRC23:10
openstackgerritA change to openstack/nova has been rejected: Remove sensitive info from rpc logging.  https://review.openstack.org/332823:11
*** kbringard has quit IRC23:11
zulvishy: what was the question?23:14
*** maplebed has joined #openstack-dev23:14
vishyzul: horizon has a couple of branches that need to go in for stable/diablo23:15
zulvishy: okie dokie23:15
vishyzul: but no one can push the button but stable-maint23:15
zulurl?23:15
vishyhttps://review.openstack.org/#change,3282  then https://review.openstack.org/#change,302823:15
Kialljeblair, Daviey, jaypipes, soren: can one of you guys +2 a couple of stable branch reviews for me? 2631, 3028, 328223:15
Kialloh.. lol23:15
Kialltalk about timing23:16
Kiallzul, https://review.openstack.org/#change,2631 aswell.. although not as important ;)23:16
*** ncode has quit IRC23:18
openstackgerritRick Harris proposed a change to openstack/nova: Remove sensitive info from rpc logging.  https://review.openstack.org/332823:19
jeblairzul: if you want that change to go in, you also need to leave a "+1 Approved" review23:20
zulgah23:20
Kiallzul, also - I think you need to +1 approve them too before jenkins will wake up?23:20
Kiall#2631 would be nice to get rid of too ;)23:21
jeblairzul: but if you're wating for someone else to +2, then that's the right thing.  not sure what process stable-maint uses.  :)23:21
Kiallzul, thanks :)23:23
*** zns has quit IRC23:24
*** flaviamissi has joined #openstack-dev23:25
KiallWhile someone with stable branch rights is at it .. I have another 2 that should be good to go.. https://review.openstack.org/#change,3232 and https://review.openstack.org/#change,323923:26
*** dolphm has quit IRC23:28
*** zns has joined #openstack-dev23:28
tr3buchetvishy: back, what's up23:30
vishytr3buchet: just curious about the big patch you submitted23:30
vishytr3buchet: is it just refactoring/cleanup?23:31
vishytr3buchet: or was there a specific problem it is trying to solve23:31
tr3buchetvishy: it's to tie quantum, nova cache and the nova network models toegther23:31
tr3buchetnext step is to get virt using the new network model, and then it's refactor time23:32
*** bencherian has joined #openstack-dev23:33
vishytr3buchet: get virt using it?23:33
tr3buchetvishy: yes, this branch puts a shim in place, so network and compute can use it, and it's converted back to legacy before it hands it over to virt23:34
tr3bucheti'm going to update xenapi after this branch23:35
openstackgerritBrian Waldon proposed a change to openstack/nova: Remove admin_only ext attr in favor of authz  https://review.openstack.org/323623:36
openstackgerritJoe Gordon proposed a change to openstack/nova: Add HACKING compliance testing to run_test.sh  https://review.openstack.org/321923:47
openstackjenkinsProject nova-docs build #2015: SUCCESS in 2 min 50 sec: https://jenkins.openstack.org/job/nova-docs/2015/23:48
openstackjenkinsRick Harris: Remove sensitive info from rpc logging.23:48
openstackgerritA change to openstack/horizon has been rejected: horizon: instances: Gracefully error if can't talk to glance  https://review.openstack.org/290823:51
openstackgerritJake Dahn proposed a change to openstack/horizon: Implementing multi-region support.  https://review.openstack.org/332923:52
*** vikram has joined #openstack-dev23:52
*** vikram has left #openstack-dev23:52
bcwaldonjk0: you cool with this? https://review.openstack.org/#change,321323:54
*** _adjohn has joined #openstack-dev23:56
*** adjohn has quit IRC23:56
*** _adjohn is now known as adjohn23:56
*** creiht has joined #openstack-dev23:57
*** ChanServ sets mode: +v creiht23:57
*** creiht has left #openstack-dev23:57
*** adjohn has quit IRC23:58
*** zzed has quit IRC23:59

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