| adam_g | any reason why devstack deploys diablo branch of keystone? | 00:03 |
|---|---|---|
| openstackgerrit | Paul McMillan proposed a change to openstack/horizon: Update Actions to require class-level name property. https://review.openstack.org/3253 | 00:03 |
| andrewbogott | Hm... I have a record in the nova db which is 'deleted' but causing a conflict when I try to recreate a record with the same key. | 00:05 |
| andrewbogott | How do I delete it for real so that I can move on? | 00:05 |
| openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Implement Secure Token Auth https://review.openstack.org/2889 | 00:05 |
| comstud | what is the 'key' that is the same? | 00:05 |
| openstackgerrit | Paul McMillan proposed a change to openstack/horizon: Update Actions to require class-level name property. https://review.openstack.org/3253 | 00:06 |
| comstud | (in what table) | 00:06 |
| andrewbogott | comstud: It's in the dns_entry table. | 00:06 |
| comstud | ahh | 00:06 |
| andrewbogott | I'm getting a ref, calling ref.delete(session) | 00:06 |
| andrewbogott | But... maybe that's not really how you delete things? | 00:07 |
| andrewbogott | um, my mistake, the table is called dns_domains | 00:07 |
| andrewbogott | The key is just the name of a domain. So it's not weird to want to reuse an old key. | 00:07 |
| comstud | yeah i see | 00:07 |
| comstud | it's implemented slightly differently than other tables | 00:08 |
| comstud | but | 00:08 |
| comstud | ref.delete() should be fine | 00:08 |
| comstud | to delete it | 00:08 |
| andrewbogott | I implemented it, so if it's slightly different from other tables that's maybe a mistake. | 00:08 |
| andrewbogott | s/maybe/probably/ | 00:08 |
| comstud | i think it's okay | 00:08 |
| comstud | it's just that most other tables use a unique key of an incrementing id | 00:09 |
| comstud | and we just mark 'deleted' instead of removing the entry | 00:09 |
| * andrewbogott nods | 00:09 | |
| comstud | but there are other tables that require a real delete like yours.. | 00:09 |
| comstud | it's fine | 00:09 |
| comstud | ref.delete() should work | 00:09 |
| comstud | it is not deleting? | 00:09 |
| *** maplebed_ has joined #openstack-dev | 00:10 | |
| *** andrewbogott_ has joined #openstack-dev | 00:10 | |
| *** andrewbogott_ has joined #openstack-dev | 00:10 | |
| *** maplebed_ is now known as Guest54362 | 00:10 | |
| comstud | the certificates table is an example you could look at | 00:10 |
| comstud | like certificate_destroy() | 00:10 |
| *** Guest54362 has quit IRC | 00:10 | |
| *** Guest54362 has joined #openstack-dev | 00:10 | |
| andrewbogott_ | I'll look -- thank you. | 00:11 |
| comstud | it grabs the ref and just does a .delete() | 00:11 |
| *** maplebed is now known as Guest89327 | 00:11 | |
| *** Guest54362 is now known as maplebed | 00:11 | |
| comstud | if that doesn't work, there's something else weird going on ;) | 00:11 |
| *** jog0 has left #openstack-dev | 00:11 | |
| *** Ryan_Lane1 has joined #openstack-dev | 00:11 | |
| andrewbogott_ | Wait, I think that what certificate_destroy() is doing is what I'm already doing. | 00:12 |
| andrewbogott_ | domain_ref.delete(session=session) | 00:12 |
| comstud | it sounded like it | 00:12 |
| comstud | so, what happens for you ? | 00:12 |
| *** Ryan_Lane has quit IRC | 00:13 | |
| comstud | you say you get a conflict with domain already existing | 00:13 |
| comstud | did it not actually remove the old entry? | 00:13 |
| andrewbogott_ | The first delete appears to work; if I repeat the delete I get exception.NotFound() | 00:13 |
| *** dtroyer has quit IRC | 00:14 | |
| *** Guest89327 has quit IRC | 00:14 | |
| comstud | ok, sounds like it did delete then | 00:14 |
| andrewbogott_ | I don't think I know how to look at the db by hand, though. | 00:14 |
| *** andrewbogott has quit IRC | 00:14 | |
| *** andrewbogott_ is now known as andrewbogott | 00:14 | |
| andrewbogott | When I try to recreate, I get an integrity error: Duplicate entry 'private.domain' for key 'PRIMARY' | 00:14 |
| openstackgerrit | Verification of a change to openstack/nova failed: pass filter_properties into scheduling requests for run_instance https://review.openstack.org/3241 | 00:14 |
| *** adjohn has quit IRC | 00:14 | |
| *** pixelbeat has joined #openstack-dev | 00:14 | |
| *** _adjohn has joined #openstack-dev | 00:14 | |
| andrewbogott | ('private.domain' being my key, weirdly.) | 00:14 |
| comstud | echo "select * from dns_domains" | mysql -u <username> -h <mysql_host> -p nova | 00:15 |
| comstud | if your mysql server is on localhost, ignore the -h option | 00:15 |
| andrewbogott | Hm... I wonder what username and password devstack is using... | 00:16 |
| comstud | i haven't looked at devstack yet | 00:17 |
| comstud | so I dunno | 00:17 |
| comstud | but if the 2nd delete failed with NotFound, then it sounds like it was surely removed | 00:18 |
| comstud | then you say trying to add the entry back fails with duplicate key error? | 00:18 |
| andrewbogott | ok, looks to me like the record is there, and it has a 'deleted at' timestamp. | 00:18 |
| comstud | that must be a different table than dns_domains | 00:19 |
| comstud | which table? | 00:19 |
| andrewbogott | big paste: | 00:19 |
| andrewbogott | $ echo "select * from dns_domains" | mysql -u root -p nova | 00:19 |
| andrewbogott | Enter password: | 00:19 |
| andrewbogott | created_atupdated_atdeleted_atdeleteddomainscopeavailability_zoneproject_id | 00:19 |
| andrewbogott | 2012-01-20 23:47:592012-01-20 23:48:102012-01-20 23:48:101private.domainprivateNULLNULL | 00:19 |
| vishy | mtaylor, jeblair: this has been happening infrequently: https://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/685/console | 00:19 |
| comstud | oh | 00:19 |
| comstud | andrewbogott: It looks to me that nova/db/sqlalchemy/models.py the DNSDomain class is not in sync with the real DB table. | 00:20 |
| comstud | andrewbogott: that is likely the problem | 00:20 |
| vishy | mtaylor, jeblair: looks like it is just failing to download stuff from review, perhaps we could get a retry in there? | 00:20 |
| andrewbogott | Pretty sure I've done a 'nova-manage db sync' once already in this session, but will do it again to make sure. | 00:20 |
| comstud | andrewbogott: or maybe not.. there's just a few columns missing i guess | 00:21 |
| andrewbogott | Yeah, I think the output is just badly formatted -- tabs vs. spaces or something. | 00:21 |
| comstud | well, i just notice 'created_at', 'deleted_at', etc is missing from models.py | 00:21 |
| comstud | but I guess it shouldn't matter, if you don't use them | 00:21 |
| comstud | ie, 'deleted' is missing from the model | 00:21 |
| *** Ryan_Lane1 is now known as Ryan_Lane | 00:21 | |
| *** Ryan_Lane has joined #openstack-dev | 00:21 | |
| andrewbogott | Right -- they're in the table, I left them out because other models seem to leave them out too. | 00:21 |
| comstud | hmm.. well somehow, something is setting 'deleted' to True in your DB entry | 00:22 |
| comstud | I wonder how | 00:22 |
| vishy | andrewbogott: if you are the stack user you can mysql without password | 00:22 |
| comstud | if it's not in the models | 00:22 |
| comstud | oooooooh | 00:22 |
| openstackgerrit | Rick Harris proposed a change to openstack/nova: ERROR out instance if unrescue fails. https://review.openstack.org/3248 | 00:22 |
| comstud | neermind | 00:23 |
| comstud | there's a base class | 00:23 |
| vishy | andrewbogott: otherwise it asks you for a MYSQL_PASSWORD when you run stack | 00:23 |
| comstud | ref.delete() definitely just marks the entry as deleted | 00:23 |
| bcwaldon | comstud, vishy: having trouble getting resize to work in the SchedulerHints extension, gonna prop with just create support | 00:23 |
| openstackgerrit | Verification of a change to openstack/keystone failed: Implement Secure Token Auth https://review.openstack.org/2889 | 00:23 |
| vishy | bcwaldon: that seems reasonable for now | 00:23 |
| comstud | andrewbogott: Ok. I think what you want to do is not use ref.delete()... and instead use .delete() on your query | 00:24 |
| zns | mtaylor: is this an environment failure? https://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/686/console - something about gapi did not start? Or is it our code? | 00:24 |
| andrewbogott | comstud: ok, trying... | 00:24 |
| comstud | andrewboggot: ie: | 00:25 |
| comstud | model_query(context, models.DNSDomain, session=session, read_deleted="no").\ filter_by(domain=fqdomain).\ delete() | 00:25 |
| andrewbogott | vishy: thanks; happens that devstack set up 'root' as the mysql user, so don't really want to run as that. | 00:25 |
| vishy | andrewbogott: ah | 00:25 |
| andrewbogott | (but I only figured that about about 2 minutes ago, after I asked my original question) | 00:25 |
| comstud | do we know what the problem is with 'git fetch' ? | 00:27 |
| comstud | it seems like the script should abort earlier | 00:27 |
| comstud | or retry | 00:27 |
| *** zns has quit IRC | 00:29 | |
| *** rods has joined #openstack-dev | 00:29 | |
| *** jakedahn_ has joined #openstack-dev | 00:30 | |
| *** lloydde has quit IRC | 00:33 | |
| comstud | gotta run out for a bit | 00:33 |
| * comstud & | 00:33 | |
| *** jakedahn has quit IRC | 00:34 | |
| *** jakedahn_ is now known as jakedahn | 00:34 | |
| bcwaldon | comstud: you jerk, assertIn is only in 2.7! | 00:41 |
| bcwaldon | comstud: no no, I'll fix it for you | 00:41 |
| openstackgerrit | Brian Waldon proposed a change to openstack/nova: Add SchedulerHints compute extension https://review.openstack.org/3254 | 00:41 |
| openstackgerrit | Brian Waldon proposed a change to openstack/nova: Add SchedulerHints compute extension https://review.openstack.org/3254 | 00:42 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Add option to force hosts to scheduler https://review.openstack.org/3249 | 00:42 |
| comstud | bcwaldon: It's in nova/test.py | 00:42 |
| comstud | bcwaldon: You jerk | 00:43 |
| bcwaldon | I don't even care | 00:43 |
| comstud | (I added it a while back) | 00:43 |
| bcwaldon | I was working off your branch on gh befor eyou fixed those two tests | 00:43 |
| bcwaldon | I was so onfused why the tests were failing | 00:43 |
| bcwaldon | turns out you dont like to run tests beofre handing off code! | 00:43 |
| openstackgerrit | Verification of a change to openstack/keystone failed: Implement Secure Token Auth https://review.openstack.org/2889 | 00:44 |
| comstud | i wasn't done with the branch | 00:45 |
| comstud | it was a WIP | 00:45 |
| *** mnewby has joined #openstack-dev | 00:45 | |
| bcwaldon | now your story changes | 00:45 |
| openstackgerrit | Brian Waldon proposed a change to openstack/nova: Add SchedulerHints compute extension https://review.openstack.org/3254 | 00:46 |
| bcwaldon | I'm just going to keep 'git review'ing this | 00:46 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Add context and request spec to filter_properties https://review.openstack.org/3250 | 00:46 |
| vishy | bcwaldon: i think we might conflict on this one https://review.openstack.org/3249 | 00:48 |
| mnewby | I have ported nova's test runner to swift - we needed CI here at internap and couldn't wait. | 00:48 |
| bcwaldon | vishy: I can rebase, no biggie | 00:48 |
| mnewby | Fixes this bug: https://bugs.launchpad.net/swift/+bug/909177 | 00:48 |
| uvirtbot | Launchpad bug 909177 in swift "swift is missing a venv-based test run" [Undecided,New] | 00:48 |
| mnewby | Should I put it up for review in gerrit, or is mtaylor working on a different solution? | 00:48 |
| openstackjenkins | Project nova-docs build #1992: SUCCESS in 3 min 0 sec: https://jenkins.openstack.org/job/nova-docs/1992/ | 00:48 |
| openstackjenkins | Chris Behrens: pass filter_properties into scheduling requests for run_instance | 00:48 |
| vishy | bcwaldon: we both added filter properties, but you stuck them up a level | 00:49 |
| vishy | so mine will have to change to an update i guess | 00:49 |
| bcwaldon | would it make sense to colocate the force_hosts/ignore_hosts stuff? | 00:50 |
| vishy | bcwaldon: I think we should probably use a special key for hints | 00:51 |
| vishy | bcwaldon: just so we don't accidentally allow users to specify filter_properties that we are using internally | 00:51 |
| bcwaldon | ok, that's a good idea | 00:51 |
| bcwaldon | call it user_hints? | 00:51 |
| vishy | sure | 00:51 |
| bcwaldon | kk | 00:51 |
| comstud | vishy: comments on yours | 00:52 |
| vishy | bcwaldon: ignore_hosts is only set in resize | 00:52 |
| *** _adjohn has quit IRC | 00:52 | |
| vishy | bcwaldon: so not sure what you mean by colocate? | 00:52 |
| adam_g | q! | 00:53 |
| bcwaldon | ok, I mean ignore what I just said | 00:53 |
| *** adjohn has joined #openstack-dev | 00:53 | |
| vishy | comstud: I originally was pulling az out of instance opts, i can switch back to that | 00:55 |
| vishy | comstud: not sure about your second comment | 00:55 |
| comstud | ah | 00:55 |
| comstud | what is desireable with force_hosts, i guess? | 00:55 |
| *** rods has quit IRC | 00:56 | |
| comstud | maybe it needs a better name | 00:56 |
| comstud | but i assumed it meant only include the listed hosts | 00:56 |
| vishy | comstud: ah right because it is should filter out all of the other hosts | 00:56 |
| comstud | yeah | 00:56 |
| vishy | comstud: updating | 00:56 |
| comstud | if you're going to change availability_zone to not be in the Instance table for some reason, then i'm okay with your change | 00:57 |
| comstud | otherwise instance_properties is just a copy of the instance ref | 00:57 |
| comstud | essentially | 00:57 |
| comstud | if that's not changing, I don't see the need for the extra arg | 00:58 |
| vishy | comstud: no that isn't it | 00:59 |
| vishy | comstud: I also need a return true | 00:59 |
| *** Ryan_Lane has quit IRC | 01:01 | |
| openstackgerrit | Jake Dahn proposed a change to openstack/python-novaclient: Implementing client for new x509 support in nova. https://review.openstack.org/3246 | 01:01 |
| vishy | comstud: the idea being that if force_host skips all the other filters | 01:01 |
| jakedahn | vishy: check https://review.openstack.org/#change,3246 for cli usage | 01:01 |
| comstud | so: return self.host in force_hosts if force_hosts | 01:01 |
| comstud | it sounds like | 01:01 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Add option to force hosts to scheduler https://review.openstack.org/3249 | 01:02 |
| jakedahn | vishy: probably need to change it to set proper permissions on created files? | 01:02 |
| openstackgerrit | Tres Henry proposed a change to openstack/horizon: VNC no longer loads by default on instance detail page. https://review.openstack.org/3255 | 01:02 |
| openstackgerrit | Brian Waldon proposed a change to openstack/nova: Handle FlavorNotFound on server list w/ filter https://review.openstack.org/3256 | 01:03 |
| vishy | jakedahn: nice. Maybe a message at the end saying "Cert written to xxx" "Pk written to xxx" so that it doesn't just return blank | 01:03 |
| jakedahn | k - what do file permissions need to be? | 01:03 |
| andrewbogott | comstud: Your suggestion seems to work great. Thank you! | 01:04 |
| comstud | andrewbogott: no prob! | 01:04 |
| *** heckj has quit IRC | 01:06 | |
| comstud | ok, dreaded run to costco | 01:06 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Adds availability zone filter https://review.openstack.org/3251 | 01:06 |
| comstud | will try to avoid killing people in my way | 01:06 |
| vishy | comstud: updated them | 01:07 |
| openstackgerrit | andrewbogott proposed a change to openstack/nova: Change the logic for deleting a record dns_domains. https://review.openstack.org/3257 | 01:08 |
| *** bencherian has quit IRC | 01:11 | |
| comstud | you could have fixed the tests without having to mox.IgnoreArg() | 01:11 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Add option to force hosts to scheduler https://review.openstack.org/3249 | 01:11 |
| comstud | but I'll ignore the less strict testing ;p | 01:11 |
| *** mnewby has quit IRC | 01:12 | |
| bcwaldon | vishy: I cherry-picked your force_host commit, so my branch will just be dependent on yours | 01:12 |
| comstud | bcwaldon: i just approved them all | 01:14 |
| comstud | er | 01:14 |
| comstud | +2'd them all | 01:14 |
| bcwaldon | yeah, saw | 01:14 |
| comstud | ok | 01:14 |
| comstud | i'm out for real now | 01:14 |
| comstud | good work, dudes | 01:15 |
| bcwaldon | kk, bye christopher | 01:15 |
| * comstud & poof | 01:15 | |
| vishy | bcwaldon: cool after your patch is in we can redo _0x44's patch and see if it works | 01:15 |
| bcwaldon | kk | 01:15 |
| vishy | bcwaldon: we will need novaclient changes as well | 01:15 |
| bcwaldon | vishy: there's a bug in this -> https://review.openstack.org/#patch,unified,3249,4,nova/compute/api.py | 01:17 |
| bcwaldon | NameError on 295 if not availability_zone | 01:17 |
| bcwaldon | host won't be defined | 01:17 |
| bcwaldon | I fixed it in my dep branch by moving that if context... branch into the if av_zone block | 01:18 |
| *** bencherian has joined #openstack-dev | 01:20 | |
| *** andrewbogott has quit IRC | 01:22 | |
| vishy | bcwaldon: do tell | 01:27 |
| vishy | bcwaldon: why did you approve it if there is a bug? | 01:28 |
| bcwaldon | well it was caught by a test in my branch, so maybe I caused it here? | 01:31 |
| vishy | bcwaldon: no it was my bad | 01:32 |
| vishy | i removed the line | 01:32 |
| bcwaldon | ok, well the tests will fail on this build | 01:33 |
| bcwaldon | push and we can reapprove | 01:33 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Add option to force hosts to scheduler https://review.openstack.org/3249 | 01:34 |
| openstackgerrit | Verification of a change to openstack/nova failed: Add option to force hosts to scheduler https://review.openstack.org/3249 | 01:34 |
| vishy | bcwaldon: pushed | 01:34 |
| vishy | got that annoying RPC error anyway | 01:34 |
| *** bencherian has quit IRC | 01:34 | |
| bcwaldon | whew, nobody will ever know! | 01:35 |
| vishy | bcwaldon: want to retrigger it? | 01:35 |
| bcwaldon | yeah | 01:35 |
| bcwaldon | is this causing the devstack failures: | 01:38 |
| bcwaldon | nova [scheduler-extension] % git fetch https://review.openstack.org/p/openstack/nova refs/changes/49/3249/5 && git checkout FETCH_HEAD | 01:38 |
| bcwaldon | error: RPC failed; result=22, HTTP code = 503 | 01:38 |
| bcwaldon | fatal: The remote end hung up unexpectedly | 01:38 |
| bcwaldon | I can't pull from gerrit | 01:38 |
| vishy | and mtaylor and jeblair seem to be away | 01:40 |
| bcwaldon | the problem was just temporary for me | 01:40 |
| bcwaldon | but I got it 4 or 5 times in a row | 01:40 |
| *** bencherian has joined #openstack-dev | 01:45 | |
| *** bencherian has quit IRC | 01:45 | |
| openstackgerrit | Dave Lapsley proposed a change to openstack/nova: bug 917397 https://review.openstack.org/3245 | 01:47 |
| uvirtbot | Launchpad bug 917397 in nova/essex "Quantum Manager configuration sanity checks" [Undecided,In progress] https://launchpad.net/bugs/917397 | 01:47 |
| *** jog0 has joined #openstack-dev | 01:48 | |
| *** _adjohn has joined #openstack-dev | 01:48 | |
| *** adjohn has quit IRC | 01:48 | |
| *** _adjohn has quit IRC | 01:48 | |
| openstackgerrit | justinsb proposed a change to openstack/nova: Support filter based on CPU core (over)allocation https://review.openstack.org/3209 | 01:48 |
| *** adjohn has joined #openstack-dev | 01:49 | |
| openstackgerrit | Jake Dahn proposed a change to openstack/python-novaclient: Implementing client for new x509 support in nova. https://review.openstack.org/3246 | 01:51 |
| *** Ryan_Lane has joined #openstack-dev | 01:53 | |
| openstackgerrit | Brian Waldon proposed a change to openstack/nova: Add SchedulerHints compute extension https://review.openstack.org/3254 | 02:01 |
| openstackjenkins | Project nova-docs build #1993: SUCCESS in 2 min 32 sec: https://jenkins.openstack.org/job/nova-docs/1993/ | 02:03 |
| openstackjenkins | Vishvananda Ishaya: Add option to force hosts to scheduler | 02:03 |
| openstackgerrit | Akira YOSHIYAMA proposed a change to openstack/keystone: Follows recent ec2token changes. https://review.openstack.org/3258 | 02:03 |
| openstackgerrit | Akira YOSHIYAMA proposed a change to openstack/keystone: Adds keystone auth-n/auth-z for Swift S3 API. https://review.openstack.org/3075 | 02:03 |
| *** rods has joined #openstack-dev | 02:08 | |
| *** pixelbeat has quit IRC | 02:09 | |
| *** msam has joined #openstack-dev | 02:11 | |
| msam | i get "warning: remote HEAD refers to nonexistent ref, unable to checkout" when i try to do git clone nova.git through a proxy. what's wrong? | 02:13 |
| *** tryggvil__ has joined #openstack-dev | 02:15 | |
| *** tryggvil_ has quit IRC | 02:15 | |
| *** jdurgin has quit IRC | 02:15 | |
| bcwaldon | are you literally typing 'git clone nova.git' ? | 02:16 |
| openstackgerrit | Akira YOSHIYAMA proposed a change to openstack/keystone: Adds keystone auth-n/auth-z for Swift S3 API. https://review.openstack.org/3075 | 02:16 |
| openstackgerrit | Brad Hall proposed a change to openstack/nova: Add floating IP support to Quantum Manager https://review.openstack.org/3259 | 02:16 |
| openstackgerrit | Joe Gordon proposed a change to openstack/nova: Add HACKING compliance testing to run_test.sh https://review.openstack.org/3219 | 02:16 |
| msam | no | 02:16 |
| bcwaldon | ok, just wanted to make sure :) | 02:17 |
| msam | git clone https://github.com/openstack/nova.git | 02:17 |
| bcwaldon | I'm not sure how to help, sorry | 02:17 |
| *** jog0 has left #openstack-dev | 02:17 | |
| msam | np | 02:17 |
| *** Ryan_Lane has quit IRC | 02:19 | |
| *** jog0 has joined #openstack-dev | 02:20 | |
| *** adjohn has quit IRC | 02:21 | |
| *** dtroyer has joined #openstack-dev | 02:22 | |
| *** maplebed has quit IRC | 02:23 | |
| novas0x2a|laptop | i've seen problems with github https through some types of proxy | 02:35 |
| novas0x2a|laptop | maybe try the git scheme? | 02:35 |
| novas0x2a|laptop | (^--- msam) | 02:35 |
| *** jog0 has quit IRC | 02:46 | |
| *** novas0x2a|laptop has quit IRC | 02:48 | |
| _0x44 | vishy: What will need to change in my patch post bcwaldon's? | 02:49 |
| *** stacker443 has joined #openstack-dev | 03:00 | |
| stacker443 | Question: How does openstack utilize eucatools, for instance "uec-publish-tarball ubuntu-10.04-server-uec-amd64.tar.gz mybucket" | 03:00 |
| *** gyee has quit IRC | 03:03 | |
| *** bencherian has joined #openstack-dev | 03:17 | |
| comstud | _0x44: I believe walrus needs to pass this scheduler_hints in filter_properties. Then you can check filter_properties['scheduler_hints']['whatever'] -- he made it arbitrary so you could define what you need in a filter | 03:26 |
| comstud | so 'whatever' is up to you | 03:26 |
| _0x44 | comstud: Ah, cool. | 03:26 |
| _0x44 | comstud: Grazie | 03:26 |
| _0x44 | comstud: Also, walrus? | 03:26 |
| comstud | so, it sounds like you could do things like "near_instance": instance_id | 03:27 |
| comstud | though i'd use uuid | 03:27 |
| comstud | sorry, waldon | 03:27 |
| _0x44 | I've already patched it locally to use uuid instead of id | 03:29 |
| comstud | nod | 03:30 |
| zaitcev | Blasted Glance refuses to start registry with index out range in do_start(). Do you guys actually try to run the git HEAD or just ./run_test.sh and declare victory? | 03:31 |
| zaitcev | Or, another hypothesis: nobody ever uses glance-control, everyone just runs glance by hand, or using distro scripts. | 03:33 |
| *** rods has quit IRC | 03:38 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Handle FlavorNotFound on server list w/ filter https://review.openstack.org/3256 | 03:41 |
| bcwaldon | comstud: whoops, thanks for pointing that out | 03:43 |
| comstud | np | 03:47 |
| comstud | i have something i need you to review as well that I forgot | 03:47 |
| openstackgerrit | Brian Waldon proposed a change to openstack/nova: Add SchedulerHints compute extension https://review.openstack.org/3254 | 03:47 |
| bcwaldon | jesus christ | 03:48 |
| bcwaldon | what | 03:48 |
| comstud | forgot to update chance scheduler for avoid host change | 03:48 |
| comstud | when i added filter_properties passing | 03:48 |
| comstud | tests passed because there's no integration test | 03:48 |
| bcwaldon | while you're thinking about it, can you kick this off again: https://review.openstack.org/#change,3256 | 03:49 |
| bcwaldon | ok, so would you like me to add something to my patch? | 03:49 |
| comstud | i'll take a look | 03:49 |
| comstud | while you look at: | 03:49 |
| openstackgerrit | Chris Behrens proposed a change to openstack/nova: Forgot to update chance scheduler for ignore_hosts change https://review.openstack.org/3260 | 03:49 |
| bcwaldon | it failed to get the code from gerrit | 03:49 |
| comstud | https://review.openstack.org/3260 | 03:49 |
| bcwaldon | ok it | 03:50 |
| bcwaldon | on* | 03:50 |
| comstud | yours too | 03:51 |
| comstud | eh i didn't mean to rename one of my tests.. o well | 03:52 |
| comstud | it's fine | 03:52 |
| bcwaldon | whatever | 03:53 |
| bcwaldon | just a test | 03:53 |
| comstud | yea | 03:53 |
| comstud | bah | 04:03 |
| comstud | you missed something in my branch | 04:03 |
| comstud | so did I | 04:03 |
| bcwaldon | maybe I shouldnt be doing revies any more | 04:05 |
| openstackgerrit | Chris Behrens proposed a change to openstack/nova: Forgot to update chance scheduler for ignore_hosts change https://review.openstack.org/3260 | 04:05 |
| comstud | yeah, don't do revies.. just do reviews | 04:05 |
| comstud | plz | 04:05 |
| bcwaldon | well I'm done for today | 04:05 |
| bcwaldon | apparently | 04:05 |
| comstud | come on | 04:05 |
| openstackgerrit | Verification of a change to openstack/nova failed: Handle FlavorNotFound on server list w/ filter https://review.openstack.org/3256 | 04:10 |
| *** jakedahn has quit IRC | 04:13 | |
| *** Ryan_Lane has joined #openstack-dev | 04:18 | |
| openstackgerrit | Chris Behrens proposed a change to openstack/nova: Refactor test_scheduler into unit tests https://review.openstack.org/3263 | 04:20 |
| zaitcev | I think Glance bugs out this way only glance/common/cfg.py:find_config_files() farts it | 04:28 |
| openstackgerrit | Verification of a change to openstack/nova failed: Handle FlavorNotFound on server list w/ filter https://review.openstack.org/3256 | 04:34 |
| zaitcev | Which "obviously" can never work because a) ConfigOpts can only find a configuration if either prog= is set or program name is one of known ones, but b) glance-control constructs GlanceConfigOpts before it knows the server name, so duh, of course it tracebacks. | 04:34 |
| openstackgerrit | Chris Behrens proposed a change to openstack/nova: Refactor test_scheduler into unit tests https://review.openstack.org/3263 | 04:35 |
| *** Ryan_Lane has quit IRC | 04:46 | |
| comstud | this devstack gate issue is getting rather annoying | 04:47 |
| vishy | comstud: is it dying again? | 04:53 |
| comstud | yeah | 04:53 |
| vishy | gerrit server seems to be hosed in some way | 04:53 |
| comstud | i've been sending in one of waldon branches repeatedly | 04:54 |
| comstud | it's still spinning on it | 04:54 |
| openstackgerrit | Verification of a change to openstack/nova failed: Handle FlavorNotFound on server list w/ filter https://review.openstack.org/3256 | 04:54 |
| comstud | ^^ | 04:54 |
| uvirtbot | comstud: Error: "^" is not a valid command. | 04:54 |
| comstud | hehe | 04:54 |
| vishy | comstud: fyi, you can see if it is failing and manually cancel and restart it | 04:54 |
| comstud | ahh, good to know | 04:55 |
| comstud | that'll speed up retries | 04:55 |
| vishy | if you click on the red dot to get console output and see the 503 | 04:55 |
| comstud | ok | 04:55 |
| vishy | just hit the x in the top right | 04:55 |
| vishy | a | 04:55 |
| vishy | and then click retrigger | 04:55 |
| comstud | vishy: and I could use a review on: https://review.openstack.org/#change,3263 | 04:55 |
| comstud | vishy: but it's kinda long... ;) | 04:56 |
| *** Ryan_Lane has joined #openstack-dev | 04:56 | |
| comstud | it drops test runs on scheduler from 40s to 4s on one of my boxes | 04:56 |
| vishy | oh i just approved the depending patch | 04:56 |
| comstud | yep! | 04:56 |
| comstud | thnx | 04:56 |
| vishy | nice | 04:56 |
| comstud | i've had that one in queue for over a month | 04:56 |
| comstud | resolved all of the conflicts the last couple days | 04:56 |
| vishy | comstud: I guess I need to get over my dislike of mox | 04:58 |
| bcwaldon | mox is awesome! | 04:58 |
| comstud | i know some people don't like it | 04:58 |
| comstud | i like that it has strict argument checking | 04:58 |
| comstud | vs having to manually verify args if you use stubs | 04:58 |
| *** andrewsmedina has quit IRC | 04:58 | |
| comstud | i know the .AndReturn() can be harder to see and read though | 04:59 |
| vishy | comstud: I just find the ReplayAll stuff non-intuitive | 04:59 |
| comstud | yeah, i had trouble at first | 04:59 |
| comstud | now i've gotten used to it | 04:59 |
| vishy | comstud: test_start_instance_exception_puts_instance_in_error_state | 05:00 |
| vishy | comstud: I still don't know why we are using the scheduler for start_instance | 05:00 |
| comstud | why we *aren't* you mean? | 05:00 |
| comstud | err | 05:00 |
| comstud | are we | 05:00 |
| comstud | ii keep getting confused | 05:00 |
| comstud | lol | 05:00 |
| *** Ryan_Lane has quit IRC | 05:00 | |
| *** bencherian has quit IRC | 05:00 | |
| comstud | ok, right, we are | 05:01 |
| comstud | i think we can switch that | 05:01 |
| comstud | intance['host'] should still be assigned | 05:01 |
| comstud | so we should be able to just cast right to the compute manager | 05:01 |
| vishy | exactly | 05:01 |
| comstud | i can propose that in a follow up | 05:01 |
| vishy | comstud: there is a bug because some things aren't implementing schedule_start_instance | 05:02 |
| comstud | yeah, dist scheduler does not | 05:02 |
| comstud | although it would be easy to | 05:02 |
| comstud | but i'd rather it not go through scheduler | 05:02 |
| comstud | i don't see the need | 05:02 |
| comstud | it was scheduled already once | 05:02 |
| comstud | and its resources are still accounted for | 05:02 |
| comstud | even if its destroyed in the virt layer | 05:03 |
| vishy | exactly | 05:03 |
| comstud | i can prop that tomorrow | 05:03 |
| comstud | depending on how hung over i am | 05:03 |
| comstud | looks like devstack is working okay on this current job | 05:04 |
| vishy | comstud: https://bugs.launchpad.net/nova/+bug/918615 | 05:06 |
| uvirtbot | Launchpad bug 918615 in nova "compute_api.start() crashes at schedule_start_instance()" [Low,Triaged] | 05:06 |
| comstud | hm | 05:06 |
| comstud | ok | 05:07 |
| comstud | yeah | 05:07 |
| comstud | i'll take it | 05:07 |
| vishy | comstud: you didn't need to requeue that with an approve | 05:07 |
| vishy | comstud: if you hit retrigger on the job in jenkins it will go again | 05:07 |
| vishy | (which I did) | 05:07 |
| comstud | yeah, i had the review open | 05:08 |
| comstud | was easier | 05:08 |
| comstud | ahh | 05:08 |
| comstud | ok :) | 05:08 |
| comstud | the current one failed | 05:08 |
| *** andrewsmedina has joined #openstack-dev | 05:08 | |
| *** eglynn_ has joined #openstack-dev | 05:08 | |
| vishy | yup just canceled and retriggerred | 05:08 |
| vishy | that one is working | 05:09 |
| comstud | ok | 05:09 |
| comstud | i wasn't seeing the 'x' to cancel | 05:09 |
| comstud | though we might have had a race condition | 05:09 |
| *** eglynn has quit IRC | 05:12 | |
| *** andrewsmedina has quit IRC | 05:15 | |
| comstud | oh my | 05:15 |
| comstud | we're passing instance_uuid into _cast_compute_message in compute api | 05:16 |
| comstud | and it sometimes pulls the instance record again | 05:16 |
| comstud | when the caller typically already has it | 05:16 |
| *** zaitcev has quit IRC | 05:16 | |
| comstud | hm, i guess only if host= is not passed | 05:16 |
| comstud | which is rare | 05:16 |
| comstud | but that could use a clean up | 05:17 |
| openstackjenkins | Project nova-docs build #1994: SUCCESS in 2 min 39 sec: https://jenkins.openstack.org/job/nova-docs/1994/ | 05:18 |
| openstackjenkins | Brian Waldon: Add SchedulerHints compute extension | 05:18 |
| vishy | lots of cleanup! | 05:21 |
| comstud | yeah | 05:21 |
| vishy | at least we have two full milestones to do it! | 05:21 |
| comstud | yeah, i kinda want to discuss this zones stuff again ;) | 05:22 |
| comstud | because I can get the core in next week and we'll be working full time on stabilizing and enhancing | 05:23 |
| comstud | if i wasn't working on this stuff today, i could have cleaned up what i had and propped it | 05:23 |
| comstud | hehe | 05:23 |
| vishy | maybe we can do a FFE for it | 05:25 |
| comstud | i'll git some code up somewhere and drop you a note | 05:26 |
| comstud | with what I'm thinking | 05:26 |
| comstud | the new zones service is pluggable and I think I can even keep the current API communication work, it seems | 05:27 |
| vishy | git some code? | 05:27 |
| vishy | definitely too much coding today! | 05:27 |
| comstud | and it would allow us to remove the zones shit from dist scheduler to clean that up | 05:27 |
| comstud | i think that would be really nice for essex | 05:27 |
| comstud | github.com/comstud/nova zone_rpc branch | 05:28 |
| comstud | it's not quite right yet.. i have some local changes not pushed up | 05:28 |
| comstud | with some of the pluggable stuff in the zones service | 05:28 |
| comstud | well, i get pulled in many directions, so that code is from last sunday night | 05:29 |
| comstud | heh. | 05:29 |
| openstackjenkins | Project nova-docs build #1995: SUCCESS in 2 min 42 sec: https://jenkins.openstack.org/job/nova-docs/1995/ | 05:33 |
| openstackjenkins | Brian Waldon: Handle FlavorNotFound on server list w/ filter | 05:33 |
| *** bencherian has joined #openstack-dev | 05:42 | |
| *** cdub has quit IRC | 05:45 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Forgot to update chance scheduler for ignore_hosts change https://review.openstack.org/3260 | 05:46 |
| *** cdub has joined #openstack-dev | 05:46 | |
| openstackjenkins | Project nova-docs build #1996: SUCCESS in 2 min 45 sec: https://jenkins.openstack.org/job/nova-docs/1996/ | 05:48 |
| openstackjenkins | * Chris Behrens: Forgot to update chance scheduler for ignore_hosts change | 05:48 |
| openstackjenkins | * Chris Behrens: Refactor test_scheduler into unit tests | 05:48 |
| comstud | awesome | 05:48 |
| comstud | ty guys | 05:48 |
| openstackgerrit | Verification of a change to openstack/nova failed: Forgot to update chance scheduler for ignore_hosts change https://review.openstack.org/3260 | 06:17 |
| *** mjfork has quit IRC | 06:17 | |
| *** reed has quit IRC | 06:23 | |
| *** zigo has joined #openstack-dev | 06:26 | |
| *** adjohn has joined #openstack-dev | 06:45 | |
| *** adjohn has quit IRC | 06:46 | |
| comstud | hm | 06:46 |
| *** Ryan_Lane has joined #openstack-dev | 06:54 | |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Adds isolated hosts filter https://review.openstack.org/3264 | 07:17 |
| vishy | comstud: looks like that failure is something to do with ordering of dependent branch causing a tarball build failure | 07:18 |
| vishy | the dependant branch tarball build went first i think | 07:19 |
| comstud | interesting | 07:19 |
| vishy | comstud: I think if we merge all of the filter branches in we have everything from simple scheduler | 07:19 |
| comstud | well, mine all went in | 07:19 |
| vishy | comstud: yeah | 07:19 |
| comstud | the last failure was apparently a repeated job somehow | 07:20 |
| comstud | i'm finishing refactoring compute api's compute manager messaging right now | 07:20 |
| vishy | comstud: joe gordon was working on some patches to split compute filter into a few different filters | 07:20 |
| comstud | ok | 07:20 |
| comstud | i saw one | 07:20 |
| comstud | i forgot to go back and look again | 07:20 |
| vishy | comstud: once all that gets finished we can switch the default to distributed scheduler | 07:21 |
| comstud | there's some functionality that will be missing until you write some weighting methods | 07:22 |
| comstud | i think | 07:22 |
| comstud | this isolated stuff is missing the cores check | 07:23 |
| comstud | which might be interesting | 07:23 |
| comstud | to implement | 07:23 |
| vishy | comstud: yeah I don't know how important that is | 07:23 |
| comstud | the logic there in simple scheduler is kinda funky | 07:23 |
| comstud | yea | 07:23 |
| comstud | cool, tests pass with my refactoring here | 07:24 |
| comstud | just pep8 | 07:24 |
| vishy | comstud: it would have to disable an another filter | 07:24 |
| comstud | yeah, exactly | 07:24 |
| comstud | that's what i was wondering | 07:24 |
| comstud | that's where things get interesting | 07:24 |
| vishy | comstud: I'm thinking that the right way to do that | 07:24 |
| vishy | comstud: is to put allocable cores into the db | 07:24 |
| vishy | comstud: so you can have different max cores per host | 07:25 |
| vishy | comstud: the hacky way would be a flag where you could override the number of cores based on host name | 07:25 |
| comstud | hm | 07:25 |
| comstud | can't you do this today by setting up the service table properly? | 07:25 |
| comstud | or does this need to be an addition to service table | 07:25 |
| vishy | comstud: well it depends on where you pull vcpus from | 07:25 |
| vishy | comstud: I think justin's patch pulls it from actual cores | 07:26 |
| comstud | ah | 07:28 |
| comstud | i see his | 07:28 |
| comstud | i don't think his does what he wants it to do | 07:28 |
| comstud | he's checking the wrong thing in his filter | 07:29 |
| comstud | afaict | 07:29 |
| comstud | adding a comment in his review | 07:29 |
| comstud | ah shit | 07:29 |
| comstud | maybe it's right | 07:29 |
| comstud | doh | 07:29 |
| comstud | ok it's right | 07:30 |
| vishy | vcpus is reported from the virtualization layer | 07:30 |
| comstud | yeah | 07:30 |
| comstud | s/service table/compute node table/ above | 07:30 |
| vishy | i suppose we could have a flag to override it | 07:30 |
| comstud | which is updated from virt layer, yeah | 07:30 |
| vishy | in libvirt we could make it return FLAGS.cpu_override if FLAGS.cpu_override is not None else multiprocessing.cpu_count() | 07:33 |
| vishy | so you could fake a bunch more cores | 07:33 |
| vishy | I think that would solve the same problem | 07:33 |
| vishy | really it seems like there should be a field in the hosts table though | 07:33 |
| vishy | that could be updated by an admin | 07:33 |
| comstud | i think that makes sense | 07:34 |
| comstud | what's your thoughts on use of 'assert' in main code? | 07:35 |
| vishy | comstud: hmm, don't really like it | 07:36 |
| comstud | yeah, me neither | 07:36 |
| comstud | ok | 07:36 |
| comstud | i'd added one, but wasn't happy with it | 07:36 |
| comstud | only added it because i saw a few in instance_types.py i think it was | 07:36 |
| comstud | Ran 2529 tests in 188.660s | 07:52 |
| comstud | this is getting much better | 07:53 |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/horizon: Remove services openstackx dependency https://review.openstack.org/3265 | 08:01 |
| openstackgerrit | Chris Behrens proposed a change to openstack/nova: Refactor compute api messaging calls to compute manager https://review.openstack.org/3266 | 08:11 |
| openstackgerrit | Chris Behrens proposed a change to openstack/nova: Make start_instance cast directly to compute host https://review.openstack.org/3267 | 08:11 |
| comstud | vishy: ^ | 08:11 |
| comstud | i think i've done enough for today | 08:11 |
| comstud | beer pong time | 08:11 |
| * comstud & | 08:11 | |
| openstackgerrit | Vish Ishaya proposed a change to openstack/nova: Adds isolated hosts filter https://review.openstack.org/3264 | 08:21 |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/horizon: Remove services openstackx dependency https://review.openstack.org/3265 | 08:24 |
| *** sleepsonthefloo has quit IRC | 08:24 | |
| *** zigo has quit IRC | 09:41 | |
| *** pixelbeat has joined #openstack-dev | 09:46 | |
| *** Ryan_Lane has quit IRC | 09:59 | |
| *** bencherian has quit IRC | 10:17 | |
| *** pixelbeat has quit IRC | 10:35 | |
| *** Mandell has quit IRC | 10:40 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Add support for Qpid to nova.rpc. https://review.openstack.org/2982 | 10:51 |
| *** andrewsmedina has joined #openstack-dev | 11:11 | |
| *** andrewsmedina has quit IRC | 11:21 | |
| *** zigo has joined #openstack-dev | 11:41 | |
| *** bepernoot has joined #openstack-dev | 12:01 | |
| *** dtroyer has quit IRC | 12:07 | |
| *** dtroyer has joined #openstack-dev | 12:08 | |
| *** pixelbeat has joined #openstack-dev | 12:33 | |
| *** troytoman-away is now known as troytoman | 13:51 | |
| Kiall | Anyone around who can help me identify why these tests pass locally for me, but fail on jenkins? https://review.openstack.org/3028 | 13:52 |
| *** bepernoot has quit IRC | 13:54 | |
| *** stacker443 has quit IRC | 14:00 | |
| *** troytoman is now known as troytoman-away | 14:02 | |
| *** bepernoot has joined #openstack-dev | 14:40 | |
| *** bepernoot has quit IRC | 15:04 | |
| *** bepernoot has joined #openstack-dev | 16:00 | |
| *** pixelbeat has quit IRC | 16:05 | |
| *** andrewsmedina has joined #openstack-dev | 16:06 | |
| *** bencherian has joined #openstack-dev | 16:12 | |
| *** andrewsmedina has quit IRC | 16:31 | |
| *** bepernoot has quit IRC | 16:40 | |
| *** bepernoot has joined #openstack-dev | 16:40 | |
| *** bepernoot has quit IRC | 16:45 | |
| *** bepernoot has joined #openstack-dev | 16:46 | |
| openstackgerrit | Kiall Mac Innes proposed a change to openstack/horizon: Add support for same tenant source-group security rules. Fixes bug #860780. https://review.openstack.org/3268 | 16:46 |
| uvirtbot | Launchpad bug 860780 in horizon "security groups should allow control via source groups" [High,Confirmed] https://launchpad.net/bugs/860780 | 16:46 |
| Kiall | devcamcar: ping? | 16:46 |
| openstackgerrit | Kiall Mac Innes proposed a change to openstack/horizon: Add support for same tenant source-group security rules. Fixes bug #860780. https://review.openstack.org/3268 | 17:03 |
| uvirtbot | Launchpad bug 860780 in horizon "security groups should allow control via source groups" [High,Confirmed] https://launchpad.net/bugs/860780 | 17:03 |
| *** bepernoot has quit IRC | 17:11 | |
| *** bepernoot has joined #openstack-dev | 17:15 | |
| *** Gordonz has joined #openstack-dev | 17:19 | |
| *** bencherian has quit IRC | 17:24 | |
| *** bepernoot has quit IRC | 17:37 | |
| *** Gordonz has quit IRC | 17:46 | |
| openstackgerrit | Russell Bryant proposed a change to openstack/nova: Add support for Qpid to nova.rpc. https://review.openstack.org/2982 | 17:50 |
| *** bepernoot has joined #openstack-dev | 18:01 | |
| *** bepernoot has quit IRC | 18:10 | |
| *** zns has joined #openstack-dev | 18:13 | |
| *** bencherian has joined #openstack-dev | 18:18 | |
| *** bencherian has quit IRC | 18:19 | |
| *** bencherian has joined #openstack-dev | 18:19 | |
| *** zns has quit IRC | 18:19 | |
| *** bepernoot has joined #openstack-dev | 18:20 | |
| *** bepernoot has quit IRC | 18:29 | |
| *** eglynn__ has joined #openstack-dev | 18:30 | |
| *** eglynn_ has quit IRC | 18:33 | |
| *** deshantm has joined #openstack-dev | 18:40 | |
| *** RobertLaptop has joined #openstack-dev | 18:44 | |
| *** RobertLaptop has left #openstack-dev | 18:44 | |
| *** zns has joined #openstack-dev | 18:50 | |
| *** Ryan_Lane has joined #openstack-dev | 18:52 | |
| *** bepernoot has joined #openstack-dev | 18:52 | |
| *** bepernoot has quit IRC | 18:55 | |
| *** bencherian has quit IRC | 19:03 | |
| *** deshantm has quit IRC | 19:05 | |
| *** bepernoot has joined #openstack-dev | 19:05 | |
| *** bepernoot has quit IRC | 19:14 | |
| Kiall | ttx: the 26th is the deadline for feature additions in essex right? | 19:17 |
| *** deshantm has joined #openstack-dev | 19:18 | |
| dtroyer | Anyone around who can re-kick https://review.openstack.org/3227? Jenkins failed with the git fetch 503 error... | 19:35 |
| Kiall | dtroyer: I've no clue why I of all people can.. but done ;) | 19:37 |
| dtroyer | kiall: Thanks. I don't know the criteria either but I can't… | 19:38 |
| openstackgerrit | Verification of a change to openstack/keystone failed: Implement Secure Token Auth https://review.openstack.org/2889 | 19:38 |
| Kiall | It looks like launchpad tells gerrit i'm part of this team.. https://launchpad.net/~nova | 19:39 |
| Kiall | It must be that | 19:39 |
| openstackgerrit | Tres Henry proposed a change to openstack/horizon: Hide boot from volume UI when no volumes are available. https://review.openstack.org/3269 | 19:44 |
| Kiall | dtroyer: lol.. hard luck | 19:47 |
| Kiall | failed again | 19:47 |
| Kiall | Oh .. wait, looks like that error was ignored | 19:48 |
| dtroyer | kiall: yup, it's in. thanks again | 19:49 |
| *** bepernoot has joined #openstack-dev | 19:49 | |
| *** bepernoot has quit IRC | 20:00 | |
| openstackjenkins | Project nova-docs build #1997: SUCCESS in 2 min 59 sec: https://jenkins.openstack.org/job/nova-docs/1997/ | 20:03 |
| openstackjenkins | Dean Troyer: Fix nova-manage floating list (fixes bug 918804) | 20:03 |
| uvirtbot | Launchpad bug 918804 in nova "'nova-manage floating list' calls undefined method 'floating_ip_get_all'" [Medium,Fix committed] https://launchpad.net/bugs/918804 | 20:03 |
| *** zigo has quit IRC | 20:10 | |
| vishy | Kiall e-3 branch is the 24th | 21:37 |
| vishy | shipped on the 26th | 21:38 |
| Kiall | vishy: thanks.. and if a review doesnt make it in time? (for horizon in case that matters..) | 21:38 |
| Kiall | (I'm specifically thinking of this.. https://review.openstack.org/3268 ) | 21:39 |
| openstackgerrit | Verification of a change to openstack/nova failed: Add support for Qpid to nova.rpc. https://review.openstack.org/2982 | 21:57 |
| vishy | Kiall: it will be up to the PTl | 22:08 |
| vishy | * PTL | 22:08 |
| vishy | I will be doing feature freeze exceptions in certain cases | 22:08 |
| Kiall | vishy: right, makes sense.. guess I'll have to figure out how horizon does validation pretty quick so ;) | 22:08 |
| vishy | and cleanup/refactoring + admin apis + non-breaking extensions probably will be fine for e-4 | 22:08 |
| vishy | Kiall: ask devcamcar | 22:09 |
| Kiall | I ping'ed him earlier, guess he doesnt work weekends ;) Will try catch him tomorrow.. | 22:09 |
| Kiall | thanks... | 22:09 |
| vishy | he doesn't work weekends? | 22:10 |
| vishy | amateur! | 22:10 |
| comstud | i agree | 22:10 |
| Kiall | lol .. I would call it lazy, but whatever ;) | 22:10 |
| vishy | comstud: just sent you an email | 22:12 |
| vishy | comstud: My thought is that they are using diablo and those bugs have been fixed | 22:12 |
| vishy | comstud: but I figured you would know :) | 22:13 |
| comstud | #2 isn't fixed... except for notifications | 22:14 |
| comstud | It's something I think about now and then.. but never thought it to be a huge issue | 22:14 |
| comstud | But... I didn't thinka bout rpc.calls! | 22:14 |
| comstud | his message does say "diablo-based code" | 22:16 |
| comstud | I think it's probably a very rare condition | 22:19 |
| comstud | you'd have to have deleted all rabbit queues are just started services for the first time | 22:20 |
| comstud | i think | 22:20 |
| vishy | comstud: ha, apparently I'm not so good at actually reading emails | 22:20 |
| comstud | are/or | 22:20 |
| openstackgerrit | Ed Leafe 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/3270 | 22:23 |
| comstud | i filed a bug 919835 | 22:24 |
| uvirtbot | Launchpad bug 919835 in nova "rpc cast/call should declare the queue" [Undecided,New] https://launchpad.net/bugs/919835 | 22:24 |
| comstud | pizza time | 22:24 |
| openstackgerrit | Dan Prince proposed a change to openstack/nova: Allow compute manager prep_resize to accept kwargs https://review.openstack.org/3271 | 22:34 |
| *** Ryan_Lane has quit IRC | 22:56 | |
| *** Ryan_Lane has joined #openstack-dev | 23:07 | |
| *** Ryan_Lane has joined #openstack-dev | 23:08 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Allow compute manager prep_resize to accept kwargs https://review.openstack.org/3271 | 23:18 |
| *** andrewsmedina has joined #openstack-dev | 23:18 | |
| vishy | retriggered the above ^^ if anyone is watching | 23:19 |
| comstud | yeah | 23:24 |
| openstackgerrit | eglynn proposed a change to openstack/glance: More flexible specification of auth credentials. https://review.openstack.org/3272 | 23:32 |
| openstackjenkins | Project nova-docs build #1998: SUCCESS in 2 min 29 sec: https://jenkins.openstack.org/job/nova-docs/1998/ | 23:33 |
| openstackjenkins | dan.prince: Allow compute manager prep_resize to accept kwargs | 23:33 |
| *** msam has quit IRC | 23:47 | |
| *** Mandell has joined #openstack-dev | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!