Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-09
17:18:09 bauzas dansmith: do you agree with not adding a DB API method just for a object method ?
17:18:21 bauzas dansmith: ie. just calling the DB directly within the object method ?
17:18:36 bauzas dansmith: ie. I need to add a ComputeNode.get_by_uuid method
17:18:51 bauzas dansmith: I just want to write the DB call within that method like we do for the API DB
17:18:58 dansmith I'm buried with other stuff and didn't really see what you asked before
17:19:08 dansmith but if you want a ComputeNode.get_by_uuid call that's probably fine
17:19:28 bauzas dansmith: no worries, nevermind my previous pings, just asking for writing this method
17:19:53 bauzas it seems to me not necessary to modify the DB API for something like that
17:49:52 openstackgerrit Matt Riedemann proposed openstack/python-novaclient: DNM: Test devstack revert for neutronclient functional test failures https://review.openstack.org/417645
17:52:32 sc68cal mriedem: so.... this OS_NOVACLIENT_NETWORK variable
17:53:05 sc68cal neutron bails out if there are multiple networks to choose from, and some code spelunking has led me to that env var
17:54:04 mriedem sc68cal: that's in novaclient itself?
17:54:54 sc68cal yeah I'm looking at this function pick_network
17:57:20 sc68cal mriedem: https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/functional/base.py#L232
17:59:57 sc68cal mriedem: basically, in the presence of multiple networks (which we've always had) you need to pick a network to spawn the instance on
18:00:29 sc68cal mriedem: the only time it's not the case, is if there is only one network available. So if you're a tenant that only has access to one network, it's all good
18:01:16 mriedem sc68cal: kevinbenton is also looking at this in -neutron
18:01:22 openstackgerrit Eric Brown proposed openstack/nova: Remove references to Python 3.4 https://review.openstack.org/417997
18:01:36 mriedem sc68cal: and yeah i know that's how it's always worked, but apparently something changed since 1/6 to make multiple networks available to the tenants in these tests
18:01:56 mriedem if it turns out that some bug was fixed and the novaclient tests were always just wrong and getting by, and we have to fix novaclient, then ok
18:04:35 mlavalle mriedem, jaypipes: this weekend I tested https://review.openstack.org/#/c/415031/ with https://review.openstack.org/#/c/358658/. It worked nicely. But I had to manuallybump https://github.com/openstack/python-novaclient/blob/master/novaclient/__init__.py#L28 to 2.41 in my test system Is someone goig to fix that soon?
18:05:06 mriedem mlavalle: i have a patch up for that
18:05:13 mriedem but it's blocked on this other thing blocking novaclient functional testse
18:05:15 mriedem ^
18:05:26 mriedem mlavalle: https://review.openstack.org/#/c/417265/
18:17:59 openstackgerrit Sarafraj Singh proposed openstack/nova: Add service_token for nova-cinder interaction https://review.openstack.org/397399
18:22:25 dansmith holy
18:22:26 dansmith shite
18:22:32 dansmith down to 4 failures
18:35:30 melwitt dansmith: good news :) bad news, it's not obvious what's wrong, at least to me
18:36:11 dansmith I'll take four unknown things over 35
18:36:19 melwitt heh
18:36:21 alaski I saw a highlight about me breaking something. Did I break something or just make life difficult? :)
18:36:34 dansmith alaski: you broke lots of stuff
18:36:37 dansmith too many to list
18:36:50 alaski I should narrow that down, did I break something recently
18:37:20 dansmith not recently
18:37:33 dansmith like, not this morning
18:37:34 dansmith but the day is young
18:37:46 alaski yeah, I'll step up my game
18:38:13 melwitt :)
18:38:52 dansmith melwitt: looking at the test, it kinda looks like leakage between two tests
18:39:05 dansmith I wonder if that's delay in replicating or something
18:39:13 dansmith the metadata issue, that is
18:39:13 melwitt hmm
18:41:33 dansmith the image one seems like a similar race kinda thing
18:41:42 dansmith and the other is complaining about being in snapshotting state, so that might be fallout
18:43:41 dansmith although that's a negative test
18:43:42 dansmith maybe unrelated
18:44:18 cdent bauzas: Was it you that was asking about running just one gabbi test recently. edleafe and I just figured out a way to make it happen using testtools.run and --load-list (which is needed for dynamic test discovery):
18:44:19 cdent .tox/functional/bin/python -m testtools.run --load-list <(echo nova.tests.functional.api.openstack.placement.test_placement_api.aggregate_clear_those_aggregates.test_request)
18:47:56 openstackgerrit Moshe Levi proposed openstack/os-vif: OVS acceleration https://review.openstack.org/398277
18:51:30 openstackgerrit Sarafraj Singh proposed openstack/nova: Add service_token for nova-neutron interaction https://review.openstack.org/410394
18:52:06 openstackgerrit John Garbutt proposed openstack/nova: Make allocate_for_instance take consistent args https://review.openstack.org/418024
18:52:06 openstackgerrit John Garbutt proposed openstack/nova: Add and use ProcessedNetworkRequest object https://review.openstack.org/418025
18:52:07 openstackgerrit John Garbutt proposed openstack/nova: Make _validate_requested_port_ids return objects https://review.openstack.org/418026
18:52:07 openstackgerrit John Garbutt proposed openstack/nova: Return objects from _validate_requested_network_ids https://review.openstack.org/418027
18:52:11 openstackgerrit Dan Smith proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962
18:52:11 openstackgerrit Dan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775
19:32:01 knikolla hi, would it be possible to get the fix for #1635008 backported to Mitaka?
19:32:10 knikolla https://bugs.launchpad.net/nova/+bug/1635008
19:32:10 openstack Launchpad bug 1635008 in OpenStack Compute (nova) "rbd credentials in nova.conf overwrite connection_info from cinder" [Low,Fix released] - Assigned to Matt Riedemann (mriedem)
19:36:11 diana_clarke knikolla: stable/mitaka is closed for backports unless it's security related
19:36:47 diana_clarke knikolla: https://releases.openstack.org/
19:36:56 knikolla diana_clarke: alright thank you!
19:39:00 stevemar where is your fearless leader?
19:39:17 stevemar mr<tab> is not bringing up mattyR
19:42:13 diana_clarke stevemar: he's on a plane to somewhere, dunno where
19:42:30 stevemar diana_clarke: bah humbug
19:44:48 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova: Refactor _post_live_migratiion https://review.openstack.org/413021
19:47:43 openstackgerrit Anusha Unnam proposed openstack/nova: Add zero down time upgrade steps https://review.openstack.org/393931
20:14:38 openstackgerrit Moshe Levi proposed openstack/os-vif: OVS acceleration https://review.openstack.org/398277
20:23:15 openstackgerrit Artom Lifshitz proposed openstack/nova: Plumbing for tagged nic attachment https://review.openstack.org/415622
20:23:15 openstackgerrit Artom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment https://review.openstack.org/391941
20:23:16 openstackgerrit Artom Lifshitz proposed openstack/nova: API support for tagged device attachment https://review.openstack.org/376021
20:23:16 openstackgerrit Artom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment https://review.openstack.org/415520
20:23:17 openstackgerrit Artom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment https://review.openstack.org/418100
20:36:06 openstackgerrit Sarafraj Singh proposed openstack/nova: Add service_token for nova-neutron interaction https://review.openstack.org/410394
20:37:47 openstackgerrit Julian proposed openstack/nova: Transform instance.add_fixed_ip notification https://review.openstack.org/332876
21:01:01 openstackgerrit Dan Smith proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962
21:01:01 openstackgerrit Dan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775
21:07:26 openstackgerrit Julian proposed openstack/nova: Transform keypair.create notification https://review.openstack.org/408689
21:23:32 openstackgerrit Sujitha proposed openstack/nova: Document testing process for zero downtime upgrade https://review.openstack.org/395269
21:52:52 morgan sc68cal: is https://review.openstack.org/#/c/410394/ the fix for the devstack neutron/nova interaction issue?
22:02:40 openstackgerrit Sujitha proposed openstack/nova: Document testing process for zero downtime upgrade https://review.openstack.org/395269
22:10:03 openstackgerrit Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961
22:10:04 openstackgerrit Sylvain Bauza proposed openstack/nova: WIP/DNM : Put all the magic in a PlacementClient https://review.openstack.org/417111
22:10:04 openstackgerrit Sylvain Bauza proposed openstack/nova: Add ComputeNode.get_by_uuid method https://review.openstack.org/418134
22:32:39 dansmith melwitt: I realized that just the devstack change might be breaking some of the cells stuff,
22:32:55 dansmith confirmed by depending on it from the metadata patch
22:33:10 dansmith so that narrows down the things changing in each a little
22:34:16 dansmith guess I need an empty one at the beginning to measure the change
22:34:44 melwitt dansmith: oh, I see. hm
22:37:11 openstackgerrit Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141
22:39:58 openstackgerrit Eric Fried proposed openstack/nova: WIP(3): Introduce base PowerVM driver (console) https://review.openstack.org/409402
22:40:54 melwitt dansmith: isn't the metadata thing the same thing we ran into with the other patch? the change from reading instance at the top to instance in the cell lacks a cell_name?
22:41:41 dansmith melwitt: I haven't looked at the current set of failures, maybe it is
22:41:50 melwitt that is, in nova/api/metadata/base.py it's changing to read the instance from a cell
22:41:53 melwitt okay
22:42:11 dansmith might need to pull your update thing out front

Earlier   Later