Earlier  
Posted Nick Remark
#openstack-nova - 2018-05-07
21:56:57 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add Project, User and Consumer objects https://review.openstack.org/565403
21:56:58 openstackgerrit Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404
21:56:59 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
21:57:00 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
21:57:01 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
22:23:37 openstackgerrit Merged openstack/nova stable/pike: Handle @safe_connect returns None side effect in _ensure_resource_provider https://review.openstack.org/566164
22:40:09 openstackgerrit melanie witt proposed openstack/nova master: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/555945
23:35:40 arvindn05 quick testing question, how can we mock 2 methods on the same instance object with mock.patch.object?
23:36:05 arvindn05 return_value=fake_rp_tree)
23:36:05 arvindn05 'get_provider_tree_and_ensure_root',
23:36:05 arvindn05 mock.patch.object(self.conductor_manager.report_client,
23:36:05 arvindn05 return_value=fake_instance_alloc),
23:36:05 arvindn05 'get_allocations_for_consumer',
23:36:17 arvindn05 should the above work?
23:36:36 melwitt that should work
23:37:06 arvindn05 melwitt: thanks. hmm...ok..something else is wrong i guess
23:37:51 arvindn05 the second mock is not working as expected....so i suspected something else is wrong
#openstack-nova - 2018-05-08
01:13:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (1) https://review.openstack.org/564639
01:27:20 openstackgerrit Merged openstack/nova master: Proxy is_volume through DriverBlockDevice https://review.openstack.org/564017
02:52:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2) https://review.openstack.org/564643
02:53:39 openstackgerrit karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620
04:21:43 openstackgerrit Merged openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367
04:53:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645
05:17:32 openstackgerrit Merged openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346
05:24:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658
05:25:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659
05:46:17 openstackgerrit jichenjc proposed openstack/nova master: [doc] merge quota document https://review.openstack.org/566783
05:54:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528
06:13:23 openstackgerrit Surya Seetharaman proposed openstack/nova master: [WIP] Add queued for delete to instance_mappings table. https://review.openstack.org/566788
07:06:49 openstackgerrit Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795
07:14:03 openstackgerrit Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795
07:35:21 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
07:35:22 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514
07:35:23 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842
07:35:24 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045
07:35:26 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554
07:35:26 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
07:35:27 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487
07:58:14 kashyap dansmith: Sorry, had to be AFK a bit early last evening. Just catching up with the scroll
08:02:14 kashyap dansmith: melwitt: Okido, let's go with the exhibit (a) which is: "deprecate in Rocky and hard-fail in Stein"
08:29:48 openstackgerrit Surya Seetharaman proposed openstack/nova master: [WIP/POC] Updating queued_for_delete from instance_destroy() https://review.openstack.org/566813
08:45:58 kashyap bauzas: When you are about, can you merge this simple Ocata backport? Already has a +2 from Sean McGinnis: https://review.openstack.org/#/c/565672/
08:46:09 kashyap bauzas: (Pinging you, as you ACKed the other branch backports)
08:46:21 kashyap Or even lyarwood ^
08:46:38 kashyap That's the outstanding backport for that bug.
08:56:57 lyarwood kashyap: looking
08:57:46 lyarwood kashyap: done
08:57:57 kashyap lyarwood: Splendid!
08:58:01 kashyap Thank you
09:13:32 openstackgerrit Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212
09:16:35 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
09:16:36 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514
09:16:37 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842
09:16:38 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045
09:16:39 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
09:16:40 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554
09:16:41 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487
09:23:34 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
09:44:26 jmccarthy mriedem: jgwentworth: Thanks for your help :) I tested the review, and in my setup it fixed the issue - https://review.openstack.org/#/c/566367
10:17:38 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add tests for sharing rps in test_common_rc() https://review.openstack.org/566842
10:19:25 tetsuro efried, jaypipes, bauzas: ^ found a new bug in sharing case. might be related to https://review.openstack.org/#/c/560459/3?
10:40:50 openstackgerrit Merged openstack/nova stable/ocata: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565672
10:50:14 openstackgerrit Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694
10:51:26 openstackgerrit Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694
10:52:40 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.openstack.org/456571
10:52:41 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572
10:52:42 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573
10:52:43 stephenfin owalsh: ^
10:53:14 owalsh stephenfin: ack, thanks
10:59:09 openstackgerrit Theodoros Tsioutsias proposed openstack/nova master: Add scheduling notification https://review.openstack.org/566470
11:02:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unused methods in nova/compute/utils.py https://review.openstack.org/566855
11:06:23 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Use new-style choice values https://review.openstack.org/530924
11:30:00 tetsuro Hmm... this makes me change my mind that this is a good opportunity to rework the allocation candidates logic in a totally different way. So it is fair that you ask that if we can amend nested alloc cands series without considering the existing path for sharing providers.
11:43:42 artom sean-k-m1, sean-k-mooney[m], one of you around? Have some questions about the Intel NFV CI
11:50:50 sean-k-mooney artom: hi
11:51:35 artom sean-k-mooney, heya - live migration with NUMA (either pinning or actual NUMA topologies) can be tested in the Intel NFV CI, right?
11:51:42 sean-k-mooney sorry i have been on vacation since last tuseday. my mum ha a minor operation so i was being a good son and makeing her tea and mining the house all week
11:51:52 artom sean-k-mooney, awww :)
11:51:57 sean-k-mooney artom: so yes nfv ci
11:52:11 artom Hope she's doing well
11:53:03 sean-k-mooney artom: she is doing fine. am yes you could test numa + livemigration in the intel nfv ci but i think the livemigration tests are not in cluded in teh tempest regex in the multinode job
11:53:49 theanalyst t
11:54:19 artom sean-k-mooney, could you point me to the code?
11:55:04 artom https://github.com/openstack/intel-nfv-ci-tests I assume
11:55:12 sean-k-mooney artom: the ci is currently configured using an internal git repo but im going to take a look at the tempest.conf
11:55:25 sean-k-mooney artom: that is a part of it but not the ci config
11:55:41 sean-k-mooney artom: that host our tempest plugin
11:56:28 sean-k-mooney artom: tempest-dsvm-intel-nfv-xenial/ uses that plugin but its not multinode
11:56:47 artom sean-k-mooney, does it go through gerrit? My plan is to write a live migration NUMA test (unless those already exist? A quick look at the code doesn't turn anything up), and make it Depends-On: my own patches for that
11:57:49 sean-k-mooney artom: the statdard livemigration test would partly test this. the tempest-dsvm-ovsdpdk-nfv-networking-xenial replaces the default flvors with ones that use cpu pinning + hugepages + 2 numa nodes
11:58:27 sean-k-mooney artom: this is one of the tempest.confs from a run on the ci http://52.27.155.124/portland/2018-04-27/553072/6/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/ff9bb78/logs/tempest_conf.txt.gz
11:59:00 sean-k-mooney so yes live_migration = False is set
11:59:45 sean-k-mooney we could modify that via a patch to networking-ovs-dpdk to enable livemigration via its devstack plugin.
11:59:51 artom sean-k-mooney, so at least as a first step I could need to get live_migration set to True?
12:00:04 artom *would need
12:00:39 sean-k-mooney artom: yep, we could also do that via changing the internal repo but i dont have access to it because i dont know where it is now
12:01:04 artom sean-k-mooney, heh, are the contacts listed on https://wiki.openstack.org/wiki/ThirdPartySystems/Intel_NFV_CI still good?
12:01:22 sean-k-mooney networking-ovs-dpdk however i have full contol over. yes the contacts are correct
12:02:10 artom sean-k-mooney, wait, so which one do you *not* have access to?
12:03:48 sean-k-mooney artom: i do not have acess to the zuul 2.5 configure repo that stores the job config. i am the maintainer of networking-ovs-dpdk and have full aceess to modify the job via that repo using the devstack plugin

Earlier   Later