Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-19
18:35:14 efried lmk when you want to start talking impl
18:35:50 dustinc oh good!
18:36:09 dustinc I am back in office today and want to get my freshly failing tests fixed on other, let's start Monday morning?
18:36:11 efried artom: I mean all three of them come from invoking driver.get_host_ip_addr
18:36:17 efried dustinc: sure
18:36:22 artom efried, honestly, if it makes your life easier, drop my patch entirely
18:36:29 dustinc efried: great, ttyt
18:36:55 efried artom: I'm not looking for an easy life. I want danger. I want thrills.
18:37:27 artom Have you tried insulting Putin?
18:38:08 artom It's just... I won't know for sure what needs fixing until I have your full patch in front of me
18:38:15 efried artom: Here's yours:
18:38:15 efried http://logs.openstack.org/71/671471/5/check/openstack-tox-py27/0282bb0/testr_results.html.gz
18:38:15 efried http://logs.openstack.org/71/671471/5/check/nova-tox-functional/318a334/testr_results.html.gz
18:39:22 artom I dunno, your call
18:39:31 efried the only overlap is the functional one, test_cold_migrate_with_physnet, which fails in yours because of init_host and in mine because of get_ip. The fix there might actually be the same.
18:39:32 efried so
18:39:34 efried let's do this
18:39:35 artom Given that this is latent, we could indeed merge my fix first
18:39:48 efried fix yours til it passes on top of mine
18:39:53 efried then we'll swap them
18:39:57 efried which should actually make mine pass.
18:40:12 efried dig?
18:41:10 artom efried, right, I think I get you
18:41:15 efried It's not exactly latent when your patch goes from 3 netifaces calls to 44 :)
18:41:33 artom Yours breaks because of driver.get_host_ip_addr()
18:41:37 artom Mine will fix that ^^
18:41:43 efried correct
18:41:46 artom But it will break others things in the process
18:41:50 efried correct
18:41:52 efried until it doesn't.
18:42:09 artom Yes, which is why I'm paid the big bucks, apparently
18:42:36 efried gotta do something to afford your meth habit
18:43:31 artom "Meth. Why?" is indeed a good anti-drugs slogan
19:23:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: libvirt: move checking CONF.my_ip to init_host() https://review.opendev.org/671471
19:23:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: Poison netifaces.interfaces() in tests https://review.opendev.org/671773
19:24:09 artom efried, let's wait for Zuul, but I think I got them all
19:24:26 efried artom: cool. did you reverse them as well?
19:24:30 artom Aye
19:24:42 efried okay. Thanks for sticking with this.
19:24:51 artom Np :)
19:27:44 redkrieg hello all, I'm trying to use rescue mode to install windows, but the official iso (which works in a computer when burned) says no bootable device found in rescue mode. an ubuntu iso works fine on the same host. has anyone else ever hit this issue?
19:38:48 efried redkrieg: That's... not exactly a nova issue, is it?
19:40:46 redkrieg I don't know if it's something specific to rescue mode's way of mounting isos. I've been able to use the iso with libvirt in the past. it's fine, I'll find a more appropriate venue
19:41:30 efried redkrieg: Sorry, yeah, #openstack is more appropriate for support. This channel is for dev.
19:42:24 redkrieg thanks, I'll ask there
19:44:54 artom efried, hrmm, one thing I forgot to consider was the backport situation
19:45:02 artom That poison patch isn't going back
19:45:17 efried artom: that's fine, then, because you switched em.
19:45:21 efried You don't rely on the poison patch
19:45:29 efried and your swizzle of the mocks is appropriate to backport.
19:45:48 artom Oh yeah
19:45:58 efried and I'm not too worried about whether stable branches have other violations we'll be missing.
19:46:17 efried though we can always backport the poison patch to find out, then abandon it.
19:46:25 efried but it seems pretty unlikely.
19:46:38 artom Yeah
19:46:47 artom Thanks for your help in all this, btw :)
19:47:22 efried for sure
21:02:42 efried artom: +2
21:03:47 artom efried, \o/
21:04:23 efried dansmith: we got this one all ready for you https://review.opendev.org/#/c/671471/
21:23:05 openstackgerrit Eric Fried proposed openstack/nova master: Remove @safe_connect from _delete_provider https://review.opendev.org/671866
21:41:11 openstackgerrit Eric Fried proposed openstack/nova master: Remove @safe_connect from _delete_provider https://review.opendev.org/671866
22:03:40 openstackgerrit Eric Fried proposed openstack/nova master: Disambiguate logs in delete_allocation_for_instance https://review.opendev.org/671869
22:03:54 efried o/
22:46:11 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for node.list https://review.opendev.org/656027
22:46:12 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for validating instance and node https://review.opendev.org/656028
22:46:12 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690
22:46:13 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691
22:46:13 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213
#openstack-nova - 2019-07-20
02:17:34 openstackgerrit Merged openstack/nova master: Optimize SchedulerReportClient.delete_resource_provider https://review.opendev.org/657070
03:26:43 openstackgerrit Merged openstack/nova master: Exit 1 when db sync runs before api_db sync https://review.opendev.org/665626
07:21:08 openstackgerrit zhangyangyang proposed openstack/python-novaclient master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/671887
17:42:14 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Run 'tempest-ipv6-only' job in gate https://review.opendev.org/671900
#openstack-nova - 2019-07-21
01:40:40 openstackgerrit Merged openstack/python-novaclient master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/671887
13:52:24 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Rename 'nova.objects.instance_numa_topology' https://review.opendev.org/671789
13:52:25 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove unnecessary try-catch around 'getCPUMap' https://review.opendev.org/671790
13:52:25 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_from_dict' functions https://review.opendev.org/537414
13:52:26 openstackgerrit Stephen Finucane proposed openstack/nova master: claims: Remove useless caching https://review.opendev.org/671791
13:52:26 openstackgerrit Stephen Finucane proposed openstack/nova master: Add '[compute] cpu_dedicated_set' option https://review.opendev.org/671792
13:52:27 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
13:52:27 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting 'pcpus' to the resource tracker https://review.opendev.org/671794
13:52:28 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Rename exception argument https://review.opendev.org/671795
13:52:28 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Remove unused function parameter https://review.opendev.org/671796
13:52:29 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'hardware.get_host_numa_usage_from_instance' https://review.opendev.org/671797
13:52:29 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'hardware.host_topology_and_format_from_host' https://review.opendev.org/671798
13:52:30 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'hardware.instance_topology_from_instance' https://review.opendev.org/671799
13:52:30 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
13:52:31 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support translating CPU policy extra specs, image meta https://review.opendev.org/671801
#openstack-nova - 2019-07-22
01:31:50 brinzhang lyarwood: After you review this spec[1] 5/20, now this spec wait for your update, when do you have time to do it? [1]https://review.opendev.org/#/c/580336/
05:47:52 alex_xu efried: bauzas Luyao found a problem https://review.opendev.org/#/c/671222/4/nova/virt/libvirt/device.py@119, I guess we should use the migration as the consumer when migrate, just like we did for the placement
07:32:57 openstackgerrit wangwei1 proposed openstack/nova master: fix spelling error in nova/api/validation/__init__.py https://review.opendev.org/669244
08:03:05 bauzas alex_xu: ack, I'll need to look
09:00:57 alex_xu bauzas: thanks, after take a look the detail, I don't think the migration uuid works. in the end, I'm thinking passing the flavor id to the claim/unclaim_for_instance. Then claim/unclaim based on the (instance_uuid, flavor_id)
09:26:00 openstackgerrit **** proposed openstack/nova master: Nova: node should be deleted when last service is deleted https://review.opendev.org/671731
11:29:23 sean-k-mooney stephenfin: when you get back form lunch we have killed the cacheing schduler so maybe we should delete the core, ram and disk filters then nuke the filed for the resouce tracker?
11:30:20 sean-k-mooney it would be a change to the hyperviors api but you could always proxy the data from placement.
11:37:25 openstackgerrit **** proposed openstack/nova master: Nova: node should be deleted when last service is deleted https://review.opendev.org/671731
11:41:04 sean-k-mooney johnthetubaguy: lyarwood can ye take a look at this stable backport https://review.opendev.org/#/c/671532/ if ye have time
11:57:38 openstackgerrit Alex Xu proposed openstack/nova master: Add the virt driver interface for claim and unclaim the devices https://review.opendev.org/670782
11:57:39 openstackgerrit Alex Xu proposed openstack/nova master: Moves the allocation retrieving early https://review.opendev.org/670783

Earlier   Later