Wednesday, 2012-02-22

jeblairah00:03
openstackgerritVerification of a change to openstack/nova failed: Scheduler notifications added.  https://review.openstack.org/419400:03
jeblairokay so the unit test job builds the venv from the bundle00:03
jeblairthe bundle is only built with   pip bundle .cache.bundle -r tools/pip-requires00:04
jeblairso it lacks anything specified in test-requires00:04
jeblairi'll see about adding that to jenkins' venv build script00:05
sandywalshcomstud, what's the branch du jour for your zones stuff?00:09
openstackgerritVerification of a change to openstack/nova failed: xenapi: nova-volume support for multiple luns  https://review.openstack.org/426700:10
comstudsandywalsh: I need to throw it up00:10
sandywalsh*hairball00:10
comstudyeah00:10
comstudwas thinking something similar00:10
comstudsec00:10
comstudi can push something up real quick i think00:11
sandywalshk, I'll keep an eye out00:11
jeblairbcwaldon: i've changed the script, but it'll take a while for all the changes to propogate through and the venv to be rebuilt00:13
jeblairmtaylor: ^ fyi00:13
openstackgerritVerification of a change to openstack/nova failed: Fix error that causes 400 in flavor create.  https://review.openstack.org/437000:13
bcwaldonjeblair: thanks!00:13
*** dneary has quit IRC00:15
openstackgerritVerification of a change to openstack/nova failed: Fix error that causes 400 in flavor create.  https://review.openstack.org/437000:17
comstudsandywalsh: https://review.openstack.org/437800:18
comstudi gotta bolt00:21
*** sleepsonthefloo has quit IRC00:22
* comstud &00:22
sandywalshcomstud, thanks ... later00:23
*** utlemming has quit IRC00:27
*** rkukura has quit IRC00:30
*** utlemming has joined #openstack-dev00:31
*** zns has quit IRC00:31
*** heckj has joined #openstack-dev00:38
*** apevec has quit IRC00:43
mikalvishy: are you stilla round?00:45
*** dtroyer has joined #openstack-dev00:45
mikalOr in fact, is there anyone around who cares about the nova virtualization driver interface?00:46
vishymikal: hi00:47
mikalHeya00:47
mikalSo, I was adding instances to some of the log messages that lxc produces. This lead me up the call chain, to discover that lots of nova/virt/driver.py has calls where the call takes an instance_name to describe the instance to operate on.00:48
mikalThere are few issues there -- the comments imply that instance name should be an id, which I think is confused.00:48
mikalAlso, surely we should be using uuids to describe instances in that interface?00:48
mikalI'm wondering if I'd be going too far on a tangent if I tweaked those calls to always take an instance instead of just an instance name?00:49
vishyit is legacy00:50
vishydue to the fact that we didn't have uuids before00:50
vishyand there was discussion about how the virt driver should refer to an instance00:51
mikalSo you wouldn't hate a patch which moved to passing around full instances?00:51
vishyso we decided that instance_name was the right way to do it00:51
*** RobertLaptop has joined #openstack-dev00:51
vishy(which is a method on the model that we could change to some other representation later)00:51
*** RobertLaptop has left #openstack-dev00:51
*** bhall has quit IRC00:51
vishypassing the instance object makes sense in most cases yes00:51
vishyalthough there are times where that would require an extra lookup00:52
vishyperhaps00:52
vishyin that case we should move towards uuids00:52
*** littleidea has quit IRC00:52
mikalYeah, there is code in connection.py which only knows the uuid as well00:52
mikalSo that's consistent00:53
mikalI have a completely unrelated question as well.00:55
mikalI am unsure how to proceed on https://bugs.launchpad.net/nova/+bug/93446400:56
uvirtbot`Launchpad bug 934464 in nova "New instance imagecache doesn't work with keystone: get_admin_context() doesn't set a valid token/strategy" [Undecided,Confirmed]00:56
mikalBasically, is there an example of how to create an admin context that with will work keystone somewhere?00:56
mikals/work/work with/00:56
*** jakedahn has quit IRC00:57
*** jakedahn has joined #openstack-dev00:57
*** troytoman is now known as troytoman-away00:59
openstackgerritVerification of a change to openstack/keystone failed: Removing unused code from keystone.cli  https://review.openstack.org/427500:59
anotherjessemikal: do you have the admin token ?01:02
anotherjessemikal or admin user/password?01:02
mikalHmmm. I have whatever is in nova.conf I suppose.01:02
mikalThis is a periodic task of the compute manager01:02
mikalNo humans interact with it01:02
*** bsza has quit IRC01:05
spinningcogkeystone seems to default to having a database called bla.db, but I can't find that file anywhere, anyone know what's going on? keystone-manage db_sync doesn't seem to do anything (if the atime changed on a file I'ld be happy)01:10
*** zns has joined #openstack-dev01:11
dolphmspinningcog: for testing, that appears in tests/bla.db01:11
jdganotherjesse: I figured out the issue...01:11
jdgThere was a change made in /nova/volume/driver.py, the default value assigned for target_lun is '1', this is very bad for SolidFire01:13
spinningcogIf I issue ./bin/keystone-all -d it says 2012-02-21 17:01:01    DEBUG [keystone-all] sql.connection                 = sqlite:///bla.db01:13
spinningcogbut I've moved tests/bla.db to tests/bla.db.backup01:14
spinningcogand it doesn't seem to have created a new database01:14
*** davlap has joined #openstack-dev01:16
*** sleepsonthefloo has joined #openstack-dev01:18
*** sandywalsh has quit IRC01:20
*** pixelbeat has quit IRC01:22
*** ayoung has joined #openstack-dev01:27
*** dolphm has quit IRC01:32
*** maplebed has quit IRC01:33
*** sandywalsh has joined #openstack-dev01:37
*** jdg has quit IRC01:40
*** sandywalsh has quit IRC01:43
*** anotherjesse has quit IRC01:44
*** tomoe_ has quit IRC01:50
*** jakedahn_ has joined #openstack-dev01:50
*** jakedahn has quit IRC01:53
*** jakedahn_ is now known as jakedahn01:53
*** sandywalsh has joined #openstack-dev01:57
*** sandywalsh has quit IRC02:03
*** shevek_ has joined #openstack-dev02:03
*** jakedahn has quit IRC02:07
*** shevek_ has quit IRC02:10
*** zykes- has quit IRC02:15
*** sandywalsh has joined #openstack-dev02:18
*** anotherjesse has joined #openstack-dev02:20
*** dweimer has quit IRC02:21
*** mikeyp is now known as mikeyp_away02:22
*** dweimer has joined #openstack-dev02:24
*** dprince has joined #openstack-dev02:25
*** tomoe_ has joined #openstack-dev02:28
*** cp16net has joined #openstack-dev02:29
*** anotherjesse has quit IRC02:30
*** cp16net has quit IRC02:33
*** jdurgin has quit IRC02:36
*** jog0 has left #openstack-dev02:37
*** jog0_ has joined #openstack-dev02:37
*** jog0_ has quit IRC02:37
*** dayou has joined #openstack-dev02:39
*** ches has quit IRC02:41
*** ches has joined #openstack-dev02:41
*** dprince has quit IRC02:44
jeblairbcwaldon: i believe the problem is fixed; i retriggered (i think) all of the affected changes02:48
*** gyee has quit IRC02:49
*** Ryan_Lane has quit IRC02:51
*** nati2_ has joined #openstack-dev02:51
*** nati2 has quit IRC02:51
*** bengrue has quit IRC02:51
*** bengrue has joined #openstack-dev02:55
*** LanceHaigMob has joined #openstack-dev02:57
*** dayou has quit IRC02:57
*** LanceHaigMob has quit IRC02:58
*** dayou has joined #openstack-dev02:59
*** nati2 has joined #openstack-dev03:05
*** nati2_ has quit IRC03:05
*** hugokuo has joined #openstack-dev03:06
*** mikeyp_away has quit IRC03:20
*** hugokuo has quit IRC03:23
*** anotherjesse has joined #openstack-dev03:30
*** andrewbogott is now known as andrewbogott_afk03:31
bcwaldonjeblair: thanks for staying on top of it :)03:45
*** anotherjesse has quit IRC03:51
openstackgerritVerification of a change to openstack/nova failed: blueprint host-aggregates: improvements and clean-up  https://review.openstack.org/424403:52
*** mdomsch has joined #openstack-dev03:57
openstackgerritVerification of a change to openstack/nova failed: blueprint host-aggregates: host maintenance - xenapi implementation  https://review.openstack.org/432204:03
*** kaz____ is now known as kaz04:04
*** stuntmachine has quit IRC04:07
*** tomoe_ has quit IRC04:08
*** zns has quit IRC04:10
*** jog0_ has joined #openstack-dev04:20
*** jog0_ has left #openstack-dev04:22
*** dweimer has quit IRC04:23
*** dweimer has joined #openstack-dev04:23
*** littleidea has joined #openstack-dev04:37
*** mnewby has quit IRC04:47
*** jdg has joined #openstack-dev04:52
spinningcogHow do I add a keystone role without having a keystone admin token first04:55
spinningcogwhen you just run the keystone service it doesn't have an admin token, and since it doesn't have one it seems impossible to use the REST API to add one.04:56
*** ohnoimdead has quit IRC05:06
vishyspinningcog: there is a special token in the conf file05:07
vishythat you can use05:08
spinningcogvishy: ahh, the ADMIN_TOKEN = directive05:08
vishyyeah05:09
vishyhttps://github.com/openstack-dev/devstack/blob/master/files/keystone.conf#L405:09
vishyso you set a special token there05:09
vishythen you can use the keystone client using the token/endpoint syntax instead of user/pass syntax05:09
vishyhttps://github.com/openstack-dev/devstack/blob/master/files/keystone_data.sh#L305:09
jdgvishy: Sorry for the false alarm on the volume driver, and thanks for the clarification05:10
vishyjdg: no problem05:10
vishyjdg: fyi that bug you referenced has to do with guest configuration really05:11
vishyjdg: i don't think there is a way to fix it from the host side05:11
jdgvishy: Yeah, I really had a difficult time even trying to reproduce it.05:11
*** jdg has quit IRC05:14
spinningcogvishy: Thanks05:16
*** mdomsch has quit IRC05:22
*** mnewby has joined #openstack-dev05:26
spinningcogWith the default keystone configuration, the file bla.db is not created, and seems to exist only in memory (closing and restarting the service loses all data. How do I make this database persistent?05:28
*** danwent has quit IRC05:30
spinningcogThe admin router no longer routes '/' requests so the 'curl http://0.0.0.0:35357/v2.0/' requests don't work the way they did in pre-rebase version05:37
*** gabrielhurley has quit IRC05:45
*** dtroyer has quit IRC05:46
*** Mandell has quit IRC05:50
*** Mandell has joined #openstack-dev05:54
*** journeeman has joined #openstack-dev05:59
*** dubsquared has quit IRC06:01
*** deshantm has quit IRC06:01
*** sleepsonthefloo has quit IRC06:07
*** bengrue has quit IRC06:17
*** nati2 has quit IRC06:30
*** gabrielhurley has joined #openstack-dev06:31
*** bepernoot has joined #openstack-dev07:03
*** RobertLaptop has joined #openstack-dev07:04
*** bepernoo1 has joined #openstack-dev07:09
*** bepernoot has quit IRC07:11
*** bepernoo1 has quit IRC07:11
*** mnewby_ has joined #openstack-dev07:13
*** mnewby has quit IRC07:16
*** mnewby_ is now known as mnewby07:16
*** dayou has quit IRC07:27
*** dayou has joined #openstack-dev07:31
*** jeroenhn has joined #openstack-dev07:34
*** jeroenhn has joined #openstack-dev07:35
*** littleidea has quit IRC07:41
*** AlanClark has joined #openstack-dev07:41
*** shevek_ has joined #openstack-dev07:55
*** hashar has joined #openstack-dev07:56
*** AlanClark has quit IRC07:58
*** bepernoot has joined #openstack-dev08:00
*** eglynn has quit IRC08:16
*** berendt has joined #openstack-dev08:18
*** hashar has quit IRC08:19
*** apevec has joined #openstack-dev08:37
*** hashar has joined #openstack-dev08:42
*** eglynn has joined #openstack-dev08:45
*** derekh has joined #openstack-dev08:45
*** darraghb has joined #openstack-dev08:48
*** Remco_ has joined #openstack-dev08:55
*** sniperd has quit IRC08:58
*** sniperd has joined #openstack-dev08:59
*** corXi has joined #openstack-dev09:02
*** shang has joined #openstack-dev09:12
*** sniperd has quit IRC09:18
*** sniperd has joined #openstack-dev09:19
*** shang has quit IRC09:20
*** sniperd has quit IRC09:21
*** sniperd has joined #openstack-dev09:21
*** sniperd has quit IRC09:23
*** darraghb has quit IRC09:24
*** sniperd has joined #openstack-dev09:24
*** darraghb has joined #openstack-dev09:27
*** sniperd has quit IRC09:28
*** sniperd has joined #openstack-dev09:28
*** gabrielhurley has quit IRC09:31
*** sniperd has quit IRC09:32
*** sniperd has joined #openstack-dev09:32
*** Remco_ has quit IRC09:33
*** pixelbeat has joined #openstack-dev09:34
*** eglynn has quit IRC09:36
*** sniperd has quit IRC09:36
*** shang has joined #openstack-dev09:37
*** zigo has joined #openstack-dev09:37
*** Mkenneth has joined #openstack-dev09:38
*** sniperd has joined #openstack-dev09:39
*** sandywalsh has quit IRC09:40
*** shang has quit IRC09:43
*** shang has joined #openstack-dev09:44
*** paulormg has joined #openstack-dev09:50
*** eglynn has joined #openstack-dev09:51
*** Mkenneth has quit IRC09:56
*** sandywalsh has joined #openstack-dev10:05
*** AlanClark has joined #openstack-dev10:06
*** Mkenneth has joined #openstack-dev10:06
*** derekh has quit IRC10:12
*** eglynn has quit IRC10:15
*** hashar is now known as hasharAW10:16
*** sniperd has quit IRC10:20
*** zigo has quit IRC10:24
*** sniperd has joined #openstack-dev10:27
*** hasharAW has quit IRC10:33
*** dneary has joined #openstack-dev10:34
*** dneary has quit IRC10:34
*** dneary has joined #openstack-dev10:34
*** maploin has joined #openstack-dev10:35
*** maploin has quit IRC10:35
*** maploin has joined #openstack-dev10:35
*** zykes has joined #openstack-dev10:49
zykesso, reviving my DNSaaS idea, would it be smart to do something like CouchBase with Memcache + powerdns with a python proxy inbetween (powerdns doesn't speak couch natively) or should one use zookeeper instead ? if anyone has opinions10:50
*** rods has joined #openstack-dev10:50
Kiallzykes: sounds like a lot of dependencies..11:02
zykesKiall: well PowerDNS doesn't talk to anything other then *SQL atm and membase11:03
zykesKiall: i need to option for storing "zone" configurations11:03
KiallIt it were me, I would just use PowerDNS with MySQL/PostgreSQL/Whatever nova is using..11:03
KiallThen, You can go ahead and stick bind or w/e as the public facing servers (to remove all those MySQL queries)11:04
zykesbind what ?11:04
zykesKiall: yeah, but how can one replicate that across mysql ?11:04
*** markmc has joined #openstack-dev11:04
*** eglynn has joined #openstack-dev11:04
zykesacross availability zones and so on11:04
KiallWell, DNSaaS would be its own service, completely separate from the exiting zones etc IMO11:05
zykeswhen i mean DNSaaS I mean the ability to spawn independant DNS as well (private dns for a company for example)11:05
KiallSo, you mean private DNS instances, or allowing custom domains on the shared pool of DB servers>11:06
KiallSo, you mean private DNS instances, or allowing custom domains on the shared pool of DB servers?*11:06
zykesnot sure what the ideal would be ;)11:07
mikalAny nova core reviewers want to take a look at a couple of code reviews for me? https://review.openstack.org/#change,4300 fixes checksums for the libvirt image cache manager, and https://review.openstack.org/#change,4335 is some unit test coverage improvements.11:07
KiallIMO, Dedicated DNS instances are pointless - virtually nobody gets enough DNS traffic to warrent even an m1.small ;)11:07
zykesbut shouldn't configuration be spread across "zones" ?11:08
zykesi mean to utilize nova for example for deploying utility vm's11:08
zykeswould wonder what redhat does today :p11:08
zykeshttp://etherpad.openstack.org/HkEvt4crw911:08
KiallWell, I think there are 2 parts to that..11:08
KiallSupporting configuration from multiple regions, and interacting with nova..11:09
KiallSupporting multiple region config could use master<-->master MySQL, since the level of updates would be rather low the latency shouldnt be such and issue..,11:10
zykeswhy not use something like zookeeper og couchbase ?11:10
KiallZooKeeper and CouchBase are just another beast to manage, everyone has a SQL store for OpenStack already.11:11
zykesmaybe11:11
KiallThat existing knowledge and infrastructure should be taken advantage of, unless there is a really compelling reason to use ZK or CB11:12
zykeswell i guess replication11:12
zykeswould be one11:12
KiallAnyway .. If I was writing DNSaaS for OpenStack, I would limit it's scope to an API with SQL backed PowerDNS "hidden" masters11:12
zykeshaving dns in say europe, us and others11:12
zykeshidden masters ?11:13
Kiallstandard DNS replication can be used to provide global read redundancy, using the DNS server of the deployers choice...11:13
zykeswhy do that if you could have the backend do it if it's faster ?11:14
KiallSo "hidden masters" - Consider PowerDNS with MySQL as a public facing server, the MySQL traffic would be A) huge and B) the bottleneck11:14
KiallSo, Setup PowerDNS as the master - but dont list it in the SOA or NS records, instead you list a bunch of secondary servers running BIND or something not hitting SQL11:14
zykesthat was the reason for using a python proxy between the powerdns and zc / couchbase11:15
zykesuse memcache inbetween the masters11:15
KiallSounds like a lot of complexity for something standard DNS servers handle really well already :)11:16
KiallAnyway - There are, as always, 100 ways to skin a cat, this is just how I would do it ;)11:16
Kiallzykes: also, have you seen the DNS stuff that landed in essex?11:16
zykesnop11:17
Kiallhttps://github.com/openstack/nova/blob/master/nova/network/minidns.py etc11:18
KiallI think there is a review for a PowerDNS driver in there...11:18
Kialland .. https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/floating_ip_dns.py#L13111:18
zykesok, so i can drop that ball then ;p11:19
KiallWell, I don't think this is a full DNSaaS ...11:19
KiallPretty sure it's limited to a fixed set of domains and floating IPs11:19
Kiallwould be worth asking andrewbogott_afk for the details...11:20
zykesi might as well drop it, probably just a matter of time before http://docs.rackspace.com/cdns/api/v1.0/cdns-devguide/content/index.html gets released11:21
zykesdon't you think ? ;)11:21
KiallPossibly... Then again, it could be a year or two ;)11:22
zykesother ideas ?11:22
KiallAsk them what the plan is? (Pretty sure I suggested this a while back ;))11:22
zykes: o11:23
zykestried already, but dunno *who* in rs to go to11:23
*** jeremy_ has quit IRC11:27
*** jeremy has joined #openstack-dev11:27
*** zykes has left #openstack-dev11:28
*** zykes has joined #openstack-dev11:28
*** kaz has quit IRC11:36
*** Remco_ has joined #openstack-dev11:36
*** kaz___ has joined #openstack-dev11:36
Kiallzykes: ask one of the RS guys here to find out maybe?11:37
Kiallzykes: ask one of the RS guys here to find out who to talk to maybe?*11:37
*** zigo has joined #openstack-dev11:45
*** pixelbeat has quit IRC12:17
*** pixelbeat has joined #openstack-dev12:18
*** zul has quit IRC12:27
journeemanHello. I am installing devstack. But `git clone' is giving me problems. Says, `warning: remote HEAD refers to nonexistent ref, unable to checkout.'. I am using an http/https proxy. I am not well-versed with git. What does this error mean?12:28
*** zul has joined #openstack-dev12:31
Kialljourneeman: that sounds like the proxy is broken, or, the devstack repo is broken (very unlikely)12:33
KiallAlso - I've seen many people try, and fail, to get devstack working with a proxy..12:34
journeemanKiall: I have set $http_proxy and $https_proxy vars correctly. They work for all other programs. Maybe the problem is this (problem with Ubuntu's packages)? -> http://goo.gl/qU74A12:36
KiallThat might be the issue causing the inital clone to fail, but even if you get that workint... devstack runs piles of other commands and the proxy information doesn't seem to pass through.12:37
Kiall(Thats based on the 2 or 3 others I've seen attempt devstack from behind a proxy)12:37
journeemanMan.. :/12:38
KiallIn the end - they all gave up and found a way to not need the proxy12:38
KiallAnyway - My point is not that devstack behind a proxy is impossible, just that it doesnt seem to work out of the box and is for some reason harder than you might think to fix ;)12:39
journeemanOkay..12:39
journeemanI can't avoid the proxy where I am. So,..12:40
*** sandywalsh has quit IRC12:40
journeemanAnyway, thanks Kiall :)12:41
KiallGood luck - and if you manage to get it working, send the changes in for others ;)12:41
journeemanSure :)12:41
*** markvoelker has joined #openstack-dev12:41
*** shang has quit IRC12:46
*** shang has joined #openstack-dev12:46
journeemanKiall: My friend is able to pull it fine on Fedora and Mint. He faced problems with Ubuntu too.12:47
journeemanHe needs to use the proxy too, of course12:48
*** naehring has joined #openstack-dev12:48
KiallSure - The issue with the inital clone is likely those packages, but once you get that, devstack runs all sorts of things which dont expect a proxy/dont pass the proxy vars through/something that breaks with a proxy12:49
*** naehring has quit IRC12:49
*** naehring has joined #openstack-dev12:49
journeemanOkay12:51
*** zigo has quit IRC12:54
*** naehring has quit IRC12:54
*** naehring has joined #openstack-dev12:55
*** bsza has joined #openstack-dev12:57
*** sandywalsh has joined #openstack-dev12:58
journeemanKiall: I won't be able to bypass the proxy. So, I am going for a manual install. That's what my friend did too apparently. Like you said Devstack was giving him problems later.13:00
KiallThought so :)13:02
*** davidkranz has joined #openstack-dev13:08
*** journeeman has quit IRC13:10
*** dprince has joined #openstack-dev13:12
*** shang has quit IRC13:15
*** stuntmachine has joined #openstack-dev13:18
*** stuntmachine has quit IRC13:23
*** zns has joined #openstack-dev13:27
*** zns has quit IRC13:28
*** shang has joined #openstack-dev13:29
*** stuntmachine has joined #openstack-dev13:30
*** zul has quit IRC13:42
*** zul has joined #openstack-dev13:43
jaypipesdavidkranz: ping13:56
*** pr0gg3d has joined #openstack-dev13:56
*** mattray has joined #openstack-dev14:04
davidkranzjaypipes: Here14:15
jaypipesdavidkranz: heya, morning. just wanted to let you know I should be getting to all the Tempest reviews today..14:16
davidkranzOK, great. I just tried to resubmit the stress test but got this, after rebasing to master and squashing.14:17
davidkranzremote: Resolving deltas:  63% (7/11) To ssh://david-kranz@review.openstack.org:29418/openstack/tempest.git  ! [remote rejected] HEAD -> refs/for/master/stress-test (change 3800 closed) error: failed to push some refs to 'ssh://david-kranz@review.openstack.org:29418/openstack/tempest.git'14:17
*** deshantm has joined #openstack-dev14:17
jaypipesdavidkranz: yeah, I'm not quite sure what has happened with that branch :) we may need to just start a fresh branch and pull in the changes... :(14:18
davidkranzHow do I do that?14:18
*** dtroyer has joined #openstack-dev14:23
jaypipesdavidkranz: just create a new branch from master and then pull in your changes... you could do something like git reset HEAD^ && git stash && git checkout master && git pull && git checkout -b new-stress && git stash pop14:25
davidkranzWill do14:25
jaypipesdavidkranz: coolio.14:25
*** berendt has quit IRC14:29
openstackgerritVerification of a change to openstack/glance failed: Create sorting/pagination helper function  https://review.openstack.org/420614:31
*** dtroyer has quit IRC14:32
*** crobinso has joined #openstack-dev14:38
jaypipesdavidkranz: looks like that worked ok ;)14:40
*** kbringard has joined #openstack-dev14:41
davidkranzWell, it didn't quite because the code was already checked in so stash did nothing. Since I was just adding a whole directory I just copied it in and re-added to the new branch.14:42
*** lts has joined #openstack-dev14:42
davidkranzgit is a tough nut to crack14:42
*** mdomsch has joined #openstack-dev14:45
*** dtroyer has joined #openstack-dev14:45
*** Mkenneth has quit IRC14:46
*** littleidea has joined #openstack-dev14:46
*** danwent has joined #openstack-dev14:47
*** RobertLaptop has left #openstack-dev14:50
chmouelanybody can help with launchpad and pushing change to packaging branch?14:54
chmouelsoren, mtaylor: ^^14:54
*** danwent has quit IRC14:54
sorenchmouel: Sure.14:59
chmouelsoren: tks, so I have made some changes to swift package and i'd like to push it for code review15:01
sorenchmouel: Ok.15:01
chmouelsoren: but I am not sure to where this should go as this is failing, bzr push lp:~chmouel/openstack-ubuntu-packagers/lp66793515:02
chmouelsoren: or should it go to ~chmouel/+junk/lpblah15:03
sorenchmouel: How is it failing?15:03
sorenchmouel: Oh.15:03
*** PotHix has joined #openstack-dev15:03
sorenchmouel: openstack-ubuntu-packagers is not a project.15:04
sorenchmouel: It's ~owner/project/branch_name15:04
chmouelsoren: yeah sorry it's a team so I need to joni that team or push to somewhere else?15:04
sorenNo, just put a project name there instead of a team name.15:05
sorenLike, say, "swift".15:05
soren~chmoul/swift/whatever.15:05
sorenOr ~some_team_that_you_think_should_own_the_branch/the_project_it_Belongs_to/the_name_of_the_branch15:06
chmouelcool so I thin that should be ~openstack-ubuntu-packagers/swift/lp667935 but I can't find a link how to join that ubuntu packager team (and you are the owner btw)15:10
sorenYou don't need to.15:10
sorenJust put it under ~chmoul.15:10
chmouelhttp://pastie.org/private/skmcibgnqpimkkmgor9w15:11
sorenSorry, keep mistyping your name.15:11
chmouelokay wil do thanks15:11
chmouelno worries :)15:11
sorenI guess you know what I meant :)15:11
sorenchmouel: You only put it under a team if several people are expected to work on the same branch.15:12
sorenchmouel: In this case, you just need a place to put it so that we can merge it into another branch.15:12
chmouelsoren: okay so I have uploaded it to https://code.launchpad.net/~chmouel/swift/lp667935 now I need to a merge proposal right ? how do I do so that the ubuntu-packagers can review it (and not lp:swift)15:13
openstackgerritVerification of a change to openstack/nova failed: Don't allow EC2 removal of security group in use.  https://review.openstack.org/415415:15
sorenchmouel: When you file the merge proposal, one of the things you need to fill in is the target branch.15:16
chmouelsoren: something like that https://code.launchpad.net/~chmouel/swift/lp667935/+merge/94190 ?15:16
sorenchmouel: Exactly like that.15:17
chmouelsoren: awesome, thanks :)15:17
sorenchmouel: You'll see you have conflicts, though.15:17
chmouelsoren: nice, another thing to figure out for today :) (as how to resolve conflict in bzr)15:18
sorenchmouel: You merge the two branches, and bzr will tell you how to fix it.15:20
*** dolphm has joined #openstack-dev15:20
jaypipesdavidkranz: if you have a moment... https://review.openstack.org/#change,3882 needs a second review.15:23
chmouelsoren: that looked easy but I ended make a new merge which seems to have no conflicts https://code.launchpad.net/~chmouel/swift/lp667935/+merge/9419915:30
*** jdg has joined #openstack-dev15:31
davidkranzjaypipes: I'm on it15:32
jaypipesdavidkranz: cheers15:33
*** naehring has quit IRC15:34
*** dolphm has quit IRC15:35
*** zzed has joined #openstack-dev15:36
*** derekh has joined #openstack-dev15:38
sorenchmouel: Er... That was exactly what you were supposed to do, wasn't it?15:38
chmouelsoren: ah ok, I was too much in gerrit mind where you update your merge req directly, cool as long this is ok :)15:41
davidkranzjaypipes: from my review: I think this change introduces a requirement that image_ref and image_ref_alt both exist and are different. A default devstack install would not satisfy that. Do we want to require this?15:57
*** Gordonz has joined #openstack-dev16:02
*** jeroenhn has quit IRC16:03
*** dubsquared has joined #openstack-dev16:12
*** eglynn has quit IRC16:20
*** Mkenneth has joined #openstack-dev16:22
*** eglynn has joined #openstack-dev16:23
*** vinilios has joined #openstack-dev16:28
*** bepernoot has quit IRC16:28
*** ayoung has quit IRC16:34
*** gyee has joined #openstack-dev16:36
*** ayoung has joined #openstack-dev16:36
*** cdub has quit IRC16:38
*** eglynn has quit IRC16:38
*** eglynn has joined #openstack-dev16:44
openstackgerritVerification of a change to openstack/glance failed: Avoid TestClient error on missing '__mro__' attr  https://review.openstack.org/439616:53
*** maploin has quit IRC16:56
*** eglynn has quit IRC16:59
*** eglynn has joined #openstack-dev16:59
*** maplebed has joined #openstack-dev16:59
*** nati2 has joined #openstack-dev17:00
*** eglynn has quit IRC17:00
*** zzed has quit IRC17:04
*** stuntmachine has quit IRC17:05
*** cp16net has joined #openstack-dev17:07
*** zzed has joined #openstack-dev17:08
*** novas0x2a|laptop has quit IRC17:09
*** hhoover has joined #openstack-dev17:10
*** rickfoosusa has joined #openstack-dev17:10
rickfoosusairc://irc.oftc.net17:16
*** sleepsonthefloo has joined #openstack-dev17:17
*** rickfoosusa has quit IRC17:19
*** rickfoosusa has joined #openstack-dev17:19
*** eglynn has joined #openstack-dev17:20
*** adjohn has joined #openstack-dev17:21
*** nati2 has quit IRC17:29
openstackgerritVerification of a change to openstack/nova failed: blueprint host-aggregates: host maintenance - xenapi implementation  https://review.openstack.org/432217:30
*** bepernoot has joined #openstack-dev17:32
*** dwalleck_nova has joined #openstack-dev17:32
*** anotherjesse has joined #openstack-dev17:34
ayounggyee, regarding LDAP support...17:34
*** eglynn has quit IRC17:34
*** nati2 has joined #openstack-dev17:35
*** davidkranz has quit IRC17:37
*** jdurgin has joined #openstack-dev17:40
*** AndrewWeiss has joined #openstack-dev17:43
*** AndrewWeiss has left #openstack-dev17:44
*** ohnoimdead has joined #openstack-dev17:45
*** shang has quit IRC17:47
*** aweiss has joined #openstack-dev17:48
*** aweiss has left #openstack-dev17:51
*** vincentricci has joined #openstack-dev17:51
*** danwent has joined #openstack-dev17:51
*** eglynn has joined #openstack-dev17:52
*** aweiss has joined #openstack-dev17:54
*** aweiss has quit IRC17:54
*** mnewby has quit IRC17:55
YorikSarvishy: Can you help me with this i18n?17:55
*** mnewby has joined #openstack-dev17:56
YorikSarvishy: I don't get it, what does test_localization wants...17:56
*** aweiss has joined #openstack-dev17:57
*** mnewby has quit IRC17:59
*** vincent has joined #openstack-dev18:00
*** zzed has quit IRC18:00
*** vincent is now known as Guest6436918:01
*** bepernoot has quit IRC18:02
*** Remco_ has quit IRC18:02
*** vincentricci has left #openstack-dev18:04
*** zzed has joined #openstack-dev18:04
*** Guest64369 has left #openstack-dev18:05
*** vanchester12416 has joined #openstack-dev18:05
*** vricci has joined #openstack-dev18:09
*** cdub has joined #openstack-dev18:09
*** dolphm has joined #openstack-dev18:09
*** nati2_ has joined #openstack-dev18:11
*** Ryan_Lane has joined #openstack-dev18:11
*** nati2 has quit IRC18:12
YorikSarvishy: Nevermind, managed to understand that. Why don't we use ast to look for i18n strings?18:12
*** jdg has quit IRC18:12
mtaylorYorikSar: instead of gettext?18:13
*** bepernoot has joined #openstack-dev18:13
*** jdg has joined #openstack-dev18:14
mtaylorYorikSar: ah, nevermind- I read the test18:15
*** dwalleck_nova has quit IRC18:15
*** darraghb has quit IRC18:16
*** torgomatic has joined #openstack-dev18:16
*** vanchester12416 has left #openstack-dev18:17
jdgAnybody able to give me advice on the process to make a fix to my code?18:18
jdgI'm assuming I need to open a bug in launchpad assign it to myself18:18
jdgThen I can create the review in Gerritt no?18:18
dolphmjdg: yep!18:19
jdgdolphm: Thanks!  Thought so, but sometimes it's better to ask the stupid question first  :)18:19
*** zykes has quit IRC18:19
dolphmGoogle for 'openstack git gerrit workflow' if you want a ton of detail18:20
*** shevek_ has quit IRC18:21
*** novas0x2a|laptop has joined #openstack-dev18:21
openstackgerritVerification of a change to openstack/glance failed: Create sorting/pagination helper function  https://review.openstack.org/420618:22
YorikSarmtaylor: Yes, the test parses .py files in some tricky way instead of using the ast module.18:22
*** aweiss has quit IRC18:23
YorikSarWho can approve changes in Gerrit? How exactly is it different from review?18:26
*** zykes has joined #openstack-dev18:27
*** cp16net has quit IRC18:28
*** cp16net has joined #openstack-dev18:28
*** bengrue has joined #openstack-dev18:28
mtaylorYorikSar: core team members can approve18:30
mtaylorYorikSar: for a change to be merged, it must both be positively reviewed, and it must be Approved by a core member18:30
*** jdg has quit IRC18:32
YorikSarmtaylor: Oh... So I need only another core member to make it happen18:33
*** jdg has joined #openstack-dev18:33
*** eglynn has quit IRC18:33
mtaylorYorikSar: yes18:33
*** vricci has quit IRC18:34
*** gabrielhurley has joined #openstack-dev18:35
openstackgerritVerification of a change to openstack/glance failed: blueprint progressbar-upload-image  https://review.openstack.org/365418:36
*** shang_ has joined #openstack-dev18:37
*** kyriakos has quit IRC18:37
*** vincentricci has joined #openstack-dev18:38
*** vinilios has quit IRC18:39
russellbdo bugs in launchpad get automatically targeted at a milestone?  i've been setting the milestone on mine after a patch goes in, but wasn't sure if i'm wasting my time doing that18:42
gyeeayoung, u there? sorry I was away from my desk18:44
*** dolphm has quit IRC18:45
*** vincentricci has left #openstack-dev18:45
*** aweiss has joined #openstack-dev18:48
*** bepernoot has quit IRC18:50
*** vricci has joined #openstack-dev18:50
*** vricci has quit IRC18:50
*** ncode has joined #openstack-dev18:50
openstackgerritVerification of a change to openstack/glance failed: Correct kernel/ramdisk example in docs  https://review.openstack.org/439218:52
heckjtermie: ping18:54
*** eglynn has joined #openstack-dev18:56
termieheckj: heyoo!18:57
termieheckj: just getting in, wassup? was about to start reviewing ayoung's ldap18:57
heckjtermie: morning! Have you seen an error where keystone-manage db_sync throws a traceback during __import__()?18:57
*** AlanClark has quit IRC18:57
termieheckj: no, though i don't know much context on that18:58
*** jakedahn has joined #openstack-dev18:58
*** shang_ has quit IRC18:59
heckjtermie: http://paste.openstack.org/show/4941/ <-- just poking at it now and reading through the code18:59
*** shang_ has joined #openstack-dev18:59
termieheckj: ah, looks like it isn't picking up a configuration18:59
termieheckj: is this after installation?18:59
termieheckj: right now i think it has to be run as ./bin/keystone-manage19:00
*** jakedahn has quit IRC19:00
heckjtermie - yeah, after a package based install (helping debug that for Ubuntu guys). I think I might have borked the config -looking19:00
termieheckj: so that it can find the etc stuff, i think the conf file isn't being installed in the right place by setup.py or something19:00
heckjoh yeah, definitely fucked up the config19:00
termieheckj: it needs to go wherever the default place cfg.py looks for config files, if i recall19:00
*** jakedahn has joined #openstack-dev19:00
heckjpackaging error - I'll backtrack to find out what I didn't do :-)19:01
*** eglynn has quit IRC19:02
openstackgerritVerification of a change to openstack/glance failed: Support new image copied from external storage.  https://review.openstack.org/409619:03
*** dwalleck_nova has joined #openstack-dev19:04
*** dwallec__ has joined #openstack-dev19:05
*** dwalleck_nova has quit IRC19:06
markmcrussellb, there are scripts ttx uses for setting the milestone on FixCommitted at release time19:12
*** anotherjesse1 has joined #openstack-dev19:16
*** anotherjesse has quit IRC19:16
*** hub_cap has joined #openstack-dev19:17
*** vincentricci has joined #openstack-dev19:23
*** vincentricci has left #openstack-dev19:23
*** mnewby has joined #openstack-dev19:23
*** mnewby has joined #openstack-dev19:24
openstackgerritVerification of a change to openstack/nova failed: Don't store connection pool in RpcContext.  https://review.openstack.org/435619:27
*** martine has joined #openstack-dev19:29
*** mattray1 has joined #openstack-dev19:30
*** mattray has quit IRC19:31
*** cp16net_ has joined #openstack-dev19:32
*** zzed has quit IRC19:33
russellbmarkmc: thanks19:33
*** ncode has quit IRC19:33
*** cp16net has quit IRC19:34
*** cp16net_ is now known as cp16net19:34
*** dolphm has joined #openstack-dev19:34
*** bepernoot has joined #openstack-dev19:36
openstackgerritVerification of a change to openstack-dev/devstack failed: Add exercise that boots an instance from a volume.  https://review.openstack.org/404419:36
*** hhoover has quit IRC19:41
*** davidkranz has joined #openstack-dev19:41
*** hhoover has joined #openstack-dev19:41
*** gabrielhurley has quit IRC19:42
*** jog0 has joined #openstack-dev19:42
aweissanyone know about any bugs associated with glance and [paste_deploy] section causing glance to fail to start?19:42
*** gabrielhurley has joined #openstack-dev19:42
aweissusing the following glance package on precise: 2012.1~e4~20120217.1275-0ubuntu119:42
*** bepernoot has quit IRC19:42
spinningcogIt doesn't apear to me that the wsgi.Middleware passes the local config into __init__ as kwargs, as is described in the documentation, can anyone verify if this is working correctly?19:43
*** zzed has joined #openstack-dev19:44
termiespinningcog: which project?19:44
spinningcogkeystone19:45
termiespinningcog: we don't use the paste configs, we get all our config from the common cfg stuff19:45
termiespinningcog: paste is just used to initialize the pipelines19:45
spinningcogHow do I get the config variables specified in the keystone.conf file into my middleware layer?19:46
spinningcogI'm inheriting from keystone.common.wsgi.Middleware19:47
markmcspinningcog, config from the DEFAULT section or the specific middleware section?19:47
termiemarkmc: nay19:47
spinningcogspecific middleware section19:48
termiespinningcog: use CONF19:48
termiespinningcog: finding you a code example19:48
markmcspinningcog, yeah, keystone code doesn't pass local_conf19:48
* markmc just happens to be hacking on this code :)19:48
termiespinningcog: https://github.com/openstack/keystone/blob/master/keystone/middleware/core.py#L5919:48
*** mattray1 is now known as mattray19:49
*** mattray has joined #openstack-dev19:49
termiespinningcog: admin_token in this case is defined in the [DEFAULT] section, however if you are doing custome middleware you can make a new group and use that as well19:49
markmcyeah, we should move the PasteDeploy stuff into a separate file anyway19:50
termiemarkmc: disagree19:50
termiemarkmc: it can all live together fine19:50
markmctermie, PasteDeploy stuff is more code than config19:51
termiemarkmc: fewer config files is better19:51
markmctermie, e.g. on RPM systems, if you modify keystone.conf and update the RPM, you get left with an out of date keystone.conf19:51
spinningcogso I'm working on a custom middleware that has variables defined in it's own config so do I access them with CONF['appname'].variable or something?19:51
markmctermie, i.e. if there were pipeline changes in that RPM update19:51
termiespinningcog: CONF.appname.variable19:51
spinningcoggreat thanks19:52
termiespinningcog: if you have an [appname] section19:52
*** eglynn has joined #openstack-dev19:52
markmctermie, nova and glance have gone for the separate file approach19:52
spinningcogyup19:52
*** pr0gg3d has quit IRC19:53
termiemarkmc: nova and glance don't do plenty of things correctly, i'd rather hear your arguments for it19:53
markmctermie, I've just made my main argument19:53
termiemarkmc: why are configuration files not handled correctly in an rpm system?19:53
*** mattray has quit IRC19:53
termiemarkmc: my recollection from last time i dealt with config files on a dpkg system was that you had the option to merge in updates19:53
markmctermie, we don't try and do unpredictable 3 way merges of updates with user config19:54
termiemarkmc: the merge was manual19:54
markmctermie, concept doesn't exist on RPM systems19:54
*** mattray has joined #openstack-dev19:54
termiemarkmc: seems like that is still an option, and paste _is_ still config, you'll be shoving extensions in there19:54
markmctermie, my favorite type of default config file from a package is nothing but comments19:54
sleepsonthefloodanwent - q-svc does not need python-quantumclient, does it?  https://review.openstack.org/#patch,sidebyside,4405,1,stack.sh19:55
termiemarkmc: mine too, our config options haven't solidified enough for that yet though19:55
danwentsleepsonthefloor: it is useful to have if someone wants to inspect the current state of quantum19:56
*** bepernoot has joined #openstack-dev19:57
termiemarkmc: anyway, i'm not radically convinced it is worth it to separate them out, but you can work on heckj, he's my more reasonable twin19:57
markmctermie, thanks for the tip :)19:57
sleepsonthefloodanwent - ok cool, thanks19:57
markmctermie, glance bug on this: https://bugs.launchpad.net/glance/+bug/81520819:58
uvirtbot`Launchpad bug 815208 in glance "remove paste configuration from the config files" [Low,Fix released]19:58
heckjmarkmc: huh?19:58
spinningcogOdd, I'm getting 'NameError: global name 'CONF' is not definied'19:58
markmctermie, not a huge amount of discussion on it there, I grant you19:58
* heckj reads back19:59
markmcspinningcog, CONF = config.CONF19:59
termiespinningcog: look at the top of the file19:59
spinningcogshoot yeah19:59
spinningcogsorry19:59
*** mattray has quit IRC19:59
* markmc wanders off for dinner20:01
*** Ryan_Lane is now known as Ryan_Lane|away20:01
*** ncode has joined #openstack-dev20:03
*** ncode has joined #openstack-dev20:03
spinningcogmy section is actually a filter and not an app, so the section header looks like [filter:foo_bar].    When I try and do CONF.foo_bar.var_name it says NoSuchOptError: no such option foo_bar20:05
*** armaan has joined #openstack-dev20:06
*** armaan has left #openstack-dev20:06
*** armaan has joined #openstack-dev20:06
spinningcogthere are register_str directives in config.py... so I'm curious as to how this would work.20:07
dprinceblamar: I see your smokin' keystone!20:07
blamardprince: yeah, I'm guessing you're not using openstack-chef anymore for fedora?20:07
armaanIs gabrielhurley around  ??20:09
dprinceblamar: I'm mainting it... for now. I'm always happy to jump in on Chef things.20:09
dprinceblamar: keystone doesn't work since the KSL merge though.20:09
*** dneary has quit IRC20:09
blamardprince: gotcha20:09
dprinceblamar: If you select the XenServer 5.6 configuration without keystone it will work fine.20:10
annegentlemtaylor: or jeblair anyone ever see the Email Address in Gerrit Settings > Profile just blank? Trying to onboard a contributor and he's getting "Invalid committer" when he tries to do git review20:10
mtaylorannegentle: he may need to add his email address by hand in gerrit if his email address in launchpad was not public20:10
openstackgerritVerification of a change to openstack/nova failed: Add 'nova-manage export auth'  https://review.openstack.org/433420:10
dprinceblamar: And we should have keystone in the mix soon enough.20:10
annegentlemtaylor: ok, did that too.20:12
dprinceblamar: and to your initial point we are using Puppet for Fedora. At least for now. There aren't Chef packages in Fedora yet.20:12
annegentlemtaylor: I watched as he added the email and clicked confirmation link from Gerrit20:13
annegentlemtaylor: glad you're feeling better by the way!20:14
*** Mkenneth has quit IRC20:14
mtaylorannegentle: thanks!20:17
*** dneary has joined #openstack-dev20:18
*** cp16net has quit IRC20:18
annegentlemtaylor: looks like he's got a second email address in Launchpad that Gerrit thinks is him.20:20
*** aweiss has quit IRC20:21
*** aweiss has joined #openstack-dev20:22
*** nati2 has joined #openstack-dev20:26
*** Mkenneth has joined #openstack-dev20:26
*** nati2_ has quit IRC20:27
*** armaan has left #openstack-dev20:28
*** sandywalsh has quit IRC20:29
*** andrewbogott_afk is now known as andrewbogott20:29
*** aweiss has quit IRC20:29
mtaylorannegentle: does that mean that he has a gerrit username that is not the same as his launchpad username?20:29
annegentlemtaylor: so we're getting closer… it seems like he's not an invalid committer anymore. But, he's got about 10 commits he needs to squash.20:31
annegentlemtaylor: got him into an interactive rebase20:31
mtaylorannegentle: wot20:31
mtaylorw00t, I mean20:31
*** ohnoimdead has quit IRC20:32
annegentlemtaylor: but… now I'm stuck :)20:32
*** derekh has quit IRC20:32
annegentlemtaylor: ok, what's throwing me off is the order of the commits seems in reverse. He put squash on 10 commits and pick on the last commit.20:32
*** Mkenneth has quit IRC20:32
mtayloryou want to do that the opposite direction20:33
*** sandywalsh has joined #openstack-dev20:33
mtaylorpick on the earliest commit, squash on the remaining20:33
annegentlemtaylor: ok, so how do we get back into interactive rebase?20:33
mtaylorannegentle: git rebase --abort ; git rebase -i origin/master20:34
annegentlemtaylor: on the branch he has been working in?20:34
mtayloryes20:35
annegentleok20:35
mikalCan a nova core reviewer take a look at https://review.openstack.org/#change,4335 when they have a moment please?20:39
*** aweiss has joined #openstack-dev20:47
dprincemikal: +2 from me20:48
mikalTa20:49
mikalWanna take a look at another? https://review.openstack.org/#change,4379 vishy and I were talking about how the driver interface takes instance_names in a lot of places and how that's basically and ID. I had a go at moving one call to using a full instance, and I'd be interested in people's thoughts on it if they have the time.20:50
*** Ryan_Lane|away is now known as Ryan_Lane20:54
*** dwallec__ has quit IRC20:58
*** ewindisch has joined #openstack-dev21:00
*** dneary has quit IRC21:06
*** sannes has joined #openstack-dev21:07
*** ohnoimdead has joined #openstack-dev21:18
*** ncode has quit IRC21:18
*** Remco_ has joined #openstack-dev21:20
*** n0ano has quit IRC21:20
*** dolphm has quit IRC21:22
*** dolphm has joined #openstack-dev21:28
*** dprince has quit IRC21:34
*** jdg has quit IRC21:36
*** jdg has joined #openstack-dev21:37
anotherjesse1gabrielhurley: want to re +2 and approve https://review.openstack.org/#change,4336 - had dean update the commit message to link to the bugs21:40
gabrielhurleyanotherjesse1: lemme look21:41
*** anotherjesse1 is now known as anotherjesse21:41
*** markmc has left #openstack-dev21:41
gabrielhurleyanotherjesse: done and done21:42
sleepsonthefloojeblair - external networking does not work in ci tests for instances using floating ips right now21:43
sleepsonthefloojeblair: https://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/1724/console this is failing because the instance can't communicate with the internet21:44
*** aweiss has quit IRC21:45
openstackgerritVerification of a change to openstack/python-keystoneclient failed: Help output tweaks, Vol I  https://review.openstack.org/433621:45
*** aweiss has joined #openstack-dev21:45
*** andrewsmedina has joined #openstack-dev21:46
jeblairsleepsonthefloo: that should all be configured/configurable by devstack;  they're fairly vanilla rs public cloud servers before devstack is run on them21:55
jeblairsleepsonthefloo: want me to put your key on that host so you can poke around?21:58
*** heckj has quit IRC21:58
sleepsonthefloojeblair: yes please - I'll take a look and suggest something21:59
*** dwalleck_nova has joined #openstack-dev21:59
jeblairsleepsonthefloo: https://launchpad.net/~sleepsonthefloor/+sshkeys21:59
jeblairthat work?21:59
jeblairi mean, is your current key in there?22:00
sleepsonthefloosent on pm - thanks, I'll take a look22:02
*** deshantm_ has joined #openstack-dev22:03
*** bengrue has quit IRC22:04
*** hub_cap has quit IRC22:05
*** apevec has quit IRC22:05
*** deshantm has quit IRC22:05
*** aweiss has quit IRC22:06
*** zzed has quit IRC22:07
*** ncode has joined #openstack-dev22:08
*** ncode has joined #openstack-dev22:08
*** zzed has joined #openstack-dev22:09
*** bengrue has joined #openstack-dev22:10
*** martine has quit IRC22:12
*** hub_cap has joined #openstack-dev22:13
mikalIf anyone wants to do a code review... https://review.openstack.org/#change,4432 pushes the libvirt cache manager code to 100% unit test coverage.22:14
*** andrewsben has joined #openstack-dev22:15
vishycomstud: ping22:16
*** zzed has quit IRC22:16
*** hub_cap has quit IRC22:17
*** mattray has joined #openstack-dev22:19
*** andrewsben has quit IRC22:20
termieayoung: did a pass at your review, there is quite a lot of stuff to clean up per the style guide, do you think you could make a pass at fixing a bunch of those?22:20
ayoungtermie, will do.  Some of that is my fault...some is inherited22:20
ayounggyee, did a review too,  good feedbacl22:20
termieayoung: aye, there are a bunch of inherited bits, but it'll all need to be cleaned up before we can merge it22:21
termieayoung: also, you have two reviews open22:21
ayoungtermie, yes I do22:21
termieif you could abandon the one that is not being update22:21
ayoungI realized after I submitted that I didn;t wrap the ldap call ina thread pool22:21
termieayoung: if you do git review it should maintain the same change id22:21
ayounghowdo I update a review?  when I ran git review again, it submitted a new one22:21
ayounghmmm22:21
*** dwallec__ has joined #openstack-dev22:22
*** stuntmachine has joined #openstack-dev22:22
termielooks like you need to remove some stuff from your commit message22:22
ayoungtermie, should I rebase -i changes in, or should I submit it as an additional patch22:22
ayoungah...due to the rebase22:22
*** dwallec__ has quit IRC22:22
termieyou need to drop one of the change-id's from there22:22
termiebut yeah, rebase and squash your stuff, generally, before updating22:23
ayoungso if I keep the same change Id,  git review submits it as an update?22:23
ayoungOK. I'll drop the second review22:23
termieayoung: yeah22:23
*** PotHix has quit IRC22:24
ayoungtermie, ... and of course there are comments on both reviews.  I'll keep yours, as it is the more recent.22:25
*** dwalleck_nova has quit IRC22:25
mikalayoung: git commit -a --amend to update the previous diff, and then rerun git review22:25
termieayoung: lulz, sorry, i did mine on the one that looked most recent22:26
mikalhttp://wiki.openstack.org/GerritWorkflow has some helpful pointers22:26
chmouelsomebody knows what the  OS-KSVALIDATE is used for in keystone ? it's in legacy but not in current keystone22:26
ayoungtermie, NP22:26
*** ayoung is now known as ayoung-afk22:27
termiechmouel: i do not know, but i usually make anotherjesse look up legacy stuff22:27
termieanotherjesse: ^^22:27
anotherjesseheh - chmouel looking now22:27
*** dwalleck_nova has joined #openstack-dev22:28
comstudvishy: pong22:28
vishycomstud: looking at your periodic task for network cache22:29
comstudvishy: ok22:29
vishycomstud: I don't understand the timing checks22:29
comstudlooking22:29
vishycomstud: are you trying to make it run less often than periodic interval?22:30
vishyif so, why not use (ticks_between_runs=10) ?22:30
comstudi'd like to refactor that tick stuff22:30
comstudi mean22:31
comstudticks depdends on the setting of poll_interval22:31
*** anotherjesse has quit IRC22:31
comstudor whatever it is22:31
*** anotherjesse has joined #openstack-dev22:31
comstudi think there are other cases left that also don't use ticks22:31
comstudlike instance_reclaim_interval22:31
comstudand things like that22:31
comstudticks has to be hardcoded is my issue22:32
chmoueltermie, anotherjesse: I have just updated https://bugs.launchpad.net/keystone/+bug/938253 with my comments on the diff between legacy and current ks, let me know what do you think22:32
uvirtbot`Launchpad bug 938253 in keystone "need to update auth_token to most recent version" [High,Confirmed]22:32
zykesanotherjesse, did you ever prototype the dns stuff ?22:32
comstudif i raise periodic_poll_interval, then it modifies a number of things22:32
*** jeremy has quit IRC22:33
comstudvishy: know what I mean?22:33
comstudvishy: when F opens up, I'd like to redo a bit with the periodic tasks.. essentially maybe: @manager.periodic_task(interval=FLAGS.foo)22:34
comstudvs ticks22:34
comstudinterval being seconds22:34
*** nati2 has quit IRC22:37
chmouelthere is a lot of questions on #openstack about keystone and swift it would be nice if this can be reviewed https://review.openstack.org/#change,4233 so I can point people to the documentation22:42
*** markvoelker has quit IRC22:43
anotherjessechmouel: did a +2 to that22:44
annegentlechmouel: nice doc22:44
*** dwalleck_nova has quit IRC22:45
*** dwalleck_nova has joined #openstack-dev22:45
vishycomstud: so the periodic task code would just run every second?22:45
vishycomstud: or are you thinking a separate greenthread for each task?22:46
vishytr3buchet: ping22:46
comstudvishy: The periodic task code can be smarter.. single greenthread... but sleeps only for time until the next periodic task should run22:46
tr3buchethey man22:46
anotherjessechmouel: I think OS-KSVALIDATE was the idea where you could send a token in as a header22:47
comstudvishy: We can also add "enabled=FLAGS.foo" to @manager.periodic_task() such that it won't even try calling methods that are disabled22:47
vishytr3buchet: https://bugs.launchpad.net/nova/+bug/93906022:47
uvirtbot`Launchpad bug 939060 in nova "live migrations do not update dnsmasq entries or setup networking on destination node when using multi_host" [Undecided,New]22:47
vishythat is from the email discussion we had22:47
vishytr3buchet: I'm wondering if you think it is possible/useful to do this in essex?22:48
*** jeremy has joined #openstack-dev22:48
*** zykes has quit IRC22:49
*** lts has quit IRC22:49
tr3buchetvishy: i think it's possible, and useful, but you know more of the use case22:49
vishytr3buchet: The guys who came up with the existing patch really don't have the knowledge to do the "correct version"22:50
chmouelanotherjesse: would that need to be merged in current keystone (ie: is people using this?)?22:50
vishytr3buchet: so someone with a lot of knowledge of nova networking (+ quantum implications is going to have to do it)22:50
vishyer misplaced end paren22:51
anotherjessechmouel: https://github.com/openstack/keystone/blob/milestone-proposed/keystone/content/admin/OS-KSVALIDATE-admin.wadl is the WADL definition22:51
*** hhoover has left #openstack-dev22:51
vishytr3buchet: so I'm not sure I'm going to have time and i think we need code in asap.  Any ideas on who might be able to tackle it?22:51
vishycomstud: ok I'm content with letting you keep your way for now.22:52
tr3buchetvishy: let me dig in a bit and see how large it would actually be22:52
comstudvishy: If you really want ticks, I don't mind changing it... i just don't like having to do math to figure out time_i_want/time_periodic_interval_is_set_to22:54
comstudand if someone bumps periodic_interval, it changes 'x' things22:54
vishycomstud: yeah it is nasty22:54
vishycomstud: so bug, and fix in folsom?22:54
comstudSure!22:55
comstudI do really like this decorator, though... s1rp didn't have a lot of time to take it further at the time22:55
sleepsonthefloocomstud - I'm trying your nw cache branch.  I see that this is returned from the osapi for server.addresses: {u'private': [{u'version': 4, u'addr': u'10.0.0.2'}, {u'version': 4, u'addr': u'172.24.4.225'}]} - does that look right to you?  Notice the floating ip is listed in 'private'22:59
anotherjessechmouel: I would guess no one does yet - but we don't want a situation where the docs don't match reality22:59
chmouelanotherjesse: ok will make a bug for updating the doc23:00
comstudsleepsonthefloor: It looks right if you have both of those networks in the fixed_ips table labeled as 'private' i guess?23:01
*** spinningcog has quit IRC23:01
openstackgerritVerification of a change to openstack/horizon failed: Help texts and dynamic label change for entering security group rules. ICMP rules have different meanings for the from_port and to_port fields.  https://review.openstack.org/434623:01
comstudsleepsonthefloor: I didn't change any of that code.. I just removed a fallback to querying network API.  I suspect you'd get the same response even if you used master?23:01
*** spinningcog has joined #openstack-dev23:01
comstudsleepsonthefloor: (Or are you talking about trey and jason's branch that hit master a while ago that added this cache?)23:02
sleepsonthefloocomstud, I pulled from 4269.  Atm, no floating ips are returned for servers.list in master actually.  This is the data I have: http://paste.openstack.org/raw/4951/23:03
openstackgerritVerification of a change to openstack/python-keystoneclient failed: Move --version to --identity_api_version  https://review.openstack.org/430523:04
vishytr3buchet: can you let me know? Or if you have an initial implementation that you can throw together maybe I can find someone to clean it up and get it merged.23:04
tr3buchetvishy: yeah i'll let you know23:05
vishytr3buchet: assigned you the bug for now.  Thx23:06
tr3buchetvishy: sure no problem23:06
*** stuntmachine has quit IRC23:07
comstudsleepsonthefloor: can you dump the 'instance_info_caches' table for your instance in question?23:07
comstudsleepsonthefloor: and see what it looks like?  If it's busted, there's a bug in master23:07
*** littleidea has quit IRC23:08
*** dubsquared has quit IRC23:08
*** littleidea has joined #openstack-dev23:09
sleepsonthefloocomstud - this is what I have now: http://paste.openstack.org/show/4952/  I'll start over and see if I can repro the case where no ips are returned23:09
comstudsleepsonthefloor: although that looks okay... you only have 1 network, 'private'.  so I think your expected return above is correct.  both ips are on the private network23:09
comstudsleepsonthefloor: that looks correct23:09
comstudOS API combines the fixed ips and floating ips23:09
comstud(the old v1.0 didn't do that)23:10
sleepsonthefloocomstud - ok, sounds good - it just looks a bit funny, and happened just after was struggling trying to get it to return anything at all.  I'll file a bug if I can get it to misbehave again.23:11
comstudI don't think OS API has a way to tell you the difference between floating and fixed23:11
comstud(current OS API version)23:11
comstudok cool23:11
comstudi can say that the 4269 branch wouldn't affect what you're seeing, though23:12
comstudsince the info cache entry is there23:12
comstudthat would have been used in master as well23:12
comstudvishy: I created bug 93908723:15
uvirtbot`Launchpad bug 939087 in nova "periodic tasks could use a refactoring" [Low,Triaged] https://launchpad.net/bugs/93908723:15
vishyword23:15
comstudas 'low'23:15
comstudapparently i can't target for Folsom23:15
comstudvishy: this means you'll approve my branch?:)23:16
comstudneed that to go in asap23:16
vishycomstud: just waiting on anthony to see if he can repro an issue23:17
comstudalrighty23:17
*** jakedahn has quit IRC23:17
vishycomstud: he's having trouble running the branch, but I guess jenkins will reject it if so, so we'll see23:18
comstudit's got +1s from smokestack23:19
comstudthe removal was fairly trivial23:19
comstudin most cases it's not used, so there shouldn't be an issue23:19
comstudor well, i guess the fallback was hit right after a build23:19
comstudi've been running something similar locally for zones23:20
comstud(this fix is also required for zones.. as much as it is an optimization and removing network manager issues from affecting the API)23:20
comstudvishy: you want to make the executive decision on the zones naming, so I don't have to create a poll?  'cells' it is?23:21
comstud:)23:21
vishycomstud: cells!23:24
comstudbueno23:24
*** kbringard has quit IRC23:24
zaitcevahale: Sorry I had to push back on 4428. Not sure what to do, maybe put example into a plain comment that is not a docstring, for ease of copy-paste. I'm a Python novice, but it looked clearly wrong. Also, not the only place where docstrings have escapes.23:24
sleepsonthefloocomstud: ok, I can repro the "no floating ips returned through osapi servers.list" - both against your branch and master23:25
sleepsontheflooI'll file a bug once I can extract the relevant minimal repro steps from my scripts23:25
comstudsleepsonthefloor: ok.. how are you expecting to see floating ips show up?23:27
openstackgerritVerification of a change to openstack/nova failed: Remove network_api fallback for info_cache from APIs  https://review.openstack.org/426923:27
comstudsleeponsthefloor: I don't think there's anything in the OS API spec... or are you using ec2?23:28
comstudsleepsonthefloor: or is this an extension?23:28
vishycomstud: needs a rebase23:29
*** rods has quit IRC23:29
comstudvishy: hrm, i just checked that 10 min ago23:30
comstudyeah23:30
comstudtest_compute23:31
sleepsonthefloocomstud - they should be returned in server.addresses through the osapi23:31
comstudsleepsonthefloor: the view builder only returns 'addresses' though... so it combines fixed and floating into a single list.  it appears that's what you showed in your example above..  ?23:32
*** novas0x2a|laptop has quit IRC23:32
*** novas0x2a|laptop has joined #openstack-dev23:33
dolphmis there a base class for int, float, long? e.g. isinstance(value, number)23:33
*** jdg has joined #openstack-dev23:33
anotherjessedolphm: http://docs.python.org/library/numbers.html ?23:34
sleepsonthefloocomstud - that's right.  The state of my current system is that I've associated a floating ip, and but only the fixed is returned.  Could it be that the cache is slow to update?  so If I associate a floating ip, I may have to wait till that info is displayed?23:34
anotherjesseI doubt you are thinking about complex numbers23:35
comstudsleepsonthefloor: should be there immediately23:35
comstudsleepsonthefloor: is it missing from the instance_info_caches table?23:35
comstudsleepsonthefloor: (immedately after adding)23:35
dolphmanotherjesse: yeah, i just want primitives23:35
anotherjessedolphm: maybe termie or vishy knows23:37
dolphmtype(value) in [int, float, long] works well enough as far as i can tell, unless someone has a better way23:38
dolphmguess i should include complex in that list, for completeness23:39
*** hhoover has joined #openstack-dev23:40
sleepsonthefloocomstud: the floating ip is missing in the cache table, but eventually populates, it would seem23:40
*** dwalleck_nova has quit IRC23:40
*** Ryan_Lane has quit IRC23:41
*** jakedahn has joined #openstack-dev23:41
*** Ryan_Lane has joined #openstack-dev23:41
comstudsleepsonthefloo: in my branch, it'll populate.. at some point if it's not there initially23:42
comstudsleepsonthefloo: i suppose the same would happen in master as well... at a random time something might query the network info23:42
comstudsleepsonthefloo: that would be a bug, though, if it's not updated immediately.23:42
comstudsleepsonthefloo: unless.. the assignment happens in the background anyway?23:42
comstudsleepsonthefloo: i'm unfortunately not familiar with floating ip stuff23:44
sleepsonthefloocomstud - ok - I'll post a bug.  there may be some relation to the specific order of calls I'm doing, so I'll make it detailed.23:45
*** dwalleck_nova has joined #openstack-dev23:45
comstudsleepsonthefloo: what's the call you make to add a floating ip?23:46
*** dolphm has quit IRC23:46
bcwaldonzul: what do you think about the response to your injection flag MP?23:46
comstudsleepsonthefloo: (just curious what that is)23:46
*** sleepson- has joined #openstack-dev23:46
zulbcwaldon: i havent seen it yet23:46
*** dolphm has joined #openstack-dev23:47
bcwaldonzul: ok, I'll be here if you want to discuss23:47
sleepsonthefloocomstud: http://paste.openstack.org/raw/4954/23:47
*** mattray has quit IRC23:47
*** andrewsben has joined #openstack-dev23:47
sleepsontheflooafter that, for the next minute or two, nova list does not show the floating ip23:47
*** guaqua_ has joined #openstack-dev23:48
*** bodepd_ has joined #openstack-dev23:48
*** jeblair_ has joined #openstack-dev23:49
*** mortman_ has joined #openstack-dev23:50
*** jeremyb_ has joined #openstack-dev23:51
openstackgerritVerification of a change to openstack/keystone failed: Removing broken & redundant code (bug 933555)  https://review.openstack.org/440623:51
uvirtbot`Launchpad bug 933555 in keystone "GET /tokens/{token_id}/endpoints response does not match spec" [Medium,In progress] https://launchpad.net/bugs/93355523:51
*** ironcame12 has joined #openstack-dev23:51
comstudsleepsonfloor: ah, it's a non-blocking call23:51
*** soren_ has joined #openstack-dev23:51
*** TREllis_ has joined #openstack-dev23:52
*** rods has joined #openstack-dev23:52
*** dtroyer has quit IRC23:52
comstudsleepsonthefloor: oops, i lied23:52
*** wwkeyboard2 has joined #openstack-dev23:53
comstudsleepsonthefloor: yeah, i don't see any updating of the cache in associate_floating_ip or _associate_floating_ip in network manager.23:53
*** sleepsonthefloor has quit IRC23:54
*** bodepd has quit IRC23:54
*** guaqua has quit IRC23:54
*** jeblair has quit IRC23:54
*** soren has quit IRC23:54
*** jeremyb has quit IRC23:54
*** mortman has quit IRC23:54
*** ironcamel2 has quit IRC23:54
*** TREllis has quit IRC23:54
*** akscram has quit IRC23:54
*** wwkeyboard has quit IRC23:54
*** jeblair_ is now known as jeblair23:58
openstackgerritVerification of a change to openstack/keystone failed: Move cfg to keystone.openstack.common  https://review.openstack.org/442123:59
*** jakedahn has quit IRC23:59

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