Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-20
16:31:59 aspiers but B and C are independent
16:32:15 efried It's possible that Depends-On would work in-repo
16:32:20 efried but I'm not sure about that.
16:32:26 efried And frankly it's not worth it in this case.
16:32:37 aspiers I don't think it would
16:32:51 aspiers a merge commit would be required
16:33:08 efried yeah, based on my vague notions of how depends-on works, I think you're right that it wouldn't work.
16:33:13 openstackgerrit Kashyap Chamarthy proposed openstack/os-traits master: hw: cpu: Add missing certain missing traits https://review.opendev.org/660168
16:33:18 efried so anyway, moving on...
16:33:23 aspiers before submitting to Gerrit, the only way to get the local tree in the right shape would be to merge B and C, then base A on top of that merge commit
16:33:43 aspiers which is actually how I do it in git-explode
16:33:47 kashyap aspiers: efried: First pass --^. Surely I got some things wrong; but thought I'll "put it out there" :D
16:34:07 kashyap Err, what a grammatical blunder in the summary
16:34:27 openstackgerrit Kashyap Chamarthy proposed openstack/os-traits master: hw: cpu: Add certain missing traits https://review.opendev.org/660168
16:36:04 kashyap [NB: This is WIP, as the commit message notes. If the layout is amenable to those who care, then I can split it, if need be.]
16:38:31 openstackgerrit Adam Spiers proposed openstack/nova master: Extract provider tree functional tests into new file. https://review.opendev.org/660157
16:39:16 aspiers efried: I've had a better idea - rebased this refactor ^^^ on the highest review in the dependency tree I can get away with
16:39:38 aspiers efried: thereby avoiding any CI churn on earlier changes
16:39:57 efried yup, that works too.
17:14:10 adrianc efried: was held up in some b2b meeting will upload PS in a few min.
17:15:19 adrianc also saw that no need to use save_and_reraise exception ctx
17:17:46 efried adrianc: I would feel better if you did that anyway :)
17:18:29 adrianc sure
17:18:30 efried until we have hard evidence that it's really not needed, since it doesn't hurt anything...
17:19:01 efried back in a reboot...
17:36:19 openstackgerrit Adrian Chiris proposed openstack/nova master: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/643024
18:02:52 openstackgerrit Kashyap Chamarthy proposed openstack/os-traits master: hw: cpu: Add certain missing traits https://review.opendev.org/660168
19:07:17 efried kashyap, mriedem_away, bauzas: Would y'all please look at https://review.opendev.org/#/c/651681/ some time soon?
19:09:15 mriedem_away dansmith has my power of attorney while i'm in dev jail
19:09:26 mriedem_away /dev/jail
19:09:32 efried nice
19:09:42 dansmith heh
20:34:38 openstackgerrit Merged openstack/nova master: Enable n-novnc in nova-multi-cell job https://review.opendev.org/655711
#openstack-nova - 2019-05-21
01:19:44 tonyb I have a couple of ironic nodes in my OpenStack, how can I see which traits they provide ... and if they don't provide useful ones add them?
06:32:49 openstackgerrit Wangliangyu proposed openstack/nova master: Replace colon with comma in route comment https://review.opendev.org/660256
07:30:51 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
07:47:26 kashyap efried: Will look. Actually, Ya Wang did ping me on a few things in that PostCopy / Auto-Converge spec
07:50:35 kashyap yaawang: Hi, what do you mean "applicatoins sensitive to performance reduced"?
07:53:33 openstackgerrit ya.wang proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.opendev.org/660271
08:05:36 yaawang kashyap: Hi, for example: some scientific computing applications may be more sensitive to memory access latency.
08:23:18 kashyap yaawang: Ah, noted.
08:23:35 kashyap efried: yaawang: I just reviewed it. The idea itself is sound. I had some small comments on phrasing -- they can be addressed as a follow-on, or in this if there's a strong need to respin.
08:24:47 openstackgerrit ya.wang proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.opendev.org/660271
08:25:31 openstackgerrit Alexandre arents proposed openstack/nova master: Fix live-migration when glance image deleted https://review.opendev.org/659054
08:28:41 openstackgerrit Merged openstack/nova master: Fix failure to boot instances with qcow2 format images https://review.opendev.org/640271
08:28:51 openstackgerrit Merged openstack/nova stable/ocata: Do not dump all instances in the scheduler https://review.opendev.org/659833
08:53:52 openstackgerrit Hamdy Khader proposed openstack/nova master: OVS DPDK port representors support https://review.opendev.org/658785
09:19:39 mdbooth johnthetubaguy: If you're around, any chance you might prod this over the line? https://review.opendev.org/#/c/659317/
09:24:15 openstackgerrit Lee Yarwood proposed openstack/nova master: fixtures: Return a mocked class instead of method within fake_imagebackend https://review.opendev.org/619804
09:25:41 lyarwood stephenfin: https://review.opendev.org/#/c/551349/ - would you mind taking a look at this if you have time today
09:26:12 lyarwood mdbooth: ^ there's additional stuff on top of that I could use your take on btw - https://review.opendev.org/#/q/topic:live-migration-volume-bdm-rollback-rework+(status:open+OR+status:merged)
09:26:24 stephenfin lyarwood: Sure. Fixing some issues with pbr/taskflow at the moment but it's next in the queue after that
09:26:29 lyarwood thanks
09:28:10 lyarwood stephenfin / mdbooth ; https://review.opendev.org/#/c/572790/ - could also use reviews if you have time.
09:34:03 mdbooth lyarwood: Ack
09:53:54 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Stop logging traceback when skipping quiesce https://review.opendev.org/660325
09:54:17 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Stop logging traceback when skipping quiesce https://review.opendev.org/660327
09:54:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: Stop logging traceback when skipping quiesce https://review.opendev.org/660328
09:55:28 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Fix target used in nova.policy.check_is_admin https://review.opendev.org/660330
10:06:02 stephenfin lyarwood: Can I get a reno for https://review.opendev.org/#/c/572790/ please?
10:06:31 stephenfin Actually, that's mriedem's. nvm. He'll see it soon enough
10:19:46 lyarwood stephenfin: I will if he can't
10:33:14 cdent bauzas: Have you got a new version of https://review.opendev.org/#/c/552924/ coming?
10:45:02 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: refactor volume bdm rollback error handling https://review.opendev.org/656500
10:55:08 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova-consoleauth https://review.opendev.org/652969
10:55:09 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove ConsoleAuthToken.to_dict https://review.opendev.org/652970
10:56:10 openstackgerrit Merged openstack/nova master: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/571410
10:58:45 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove ConsoleAuthToken.to_dict https://review.opendev.org/652970
11:05:15 openstackgerrit Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786
11:24:41 openstackgerrit Hamdy Khader proposed openstack/nova master: OVS DPDK port representors support https://review.opendev.org/658785
11:26:57 openstackgerrit Merged openstack/nova master: Fix guestfs.set_backend_settings call https://review.opendev.org/524748
11:42:57 openstackgerrit Hamdy Khader proposed openstack/nova master: OVS DPDK port representors support https://review.opendev.org/658785
11:44:25 openstackgerrit Merged openstack/nova master: Replace colon with comma in route comment https://review.opendev.org/660256
11:46:55 openstackgerrit Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786
12:07:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/660361
12:07:57 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/660362
12:08:16 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/660363
12:27:38 openstackgerrit Merged openstack/nova master: Restore connection_info after live migration rollback https://review.opendev.org/551349
12:36:44 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Restore connection_info after live migration rollback https://review.opendev.org/660370
12:44:33 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Restore connection_info after live migration rollback https://review.opendev.org/660375
12:46:36 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Add uuid filter to nova list https://review.opendev.org/660376
13:05:02 alex_xu sean-k-mooney: good morning, do you want to have look at this https://review.opendev.org/#/c/658716/1/specs/train/approved/show-server-numa-topology.rst@110 ?
13:05:30 alex_xu i guess core and threads should be removed also.
13:06:08 efried thanks for that, kashyap
13:06:46 alex_xu since numa_cell.cpu_topology only valid for dedicated cpu policy, and I guess cores and threads is empty all the time
13:07:28 sean-k-mooney alex_xu im just waiting for someome form the gas company to call but i can take a look in an hour or so
13:07:58 alex_xu sean-k-mooney: thanks, not hurry
13:11:20 sean-k-mooney alex_xu: im ok with the api reporting the number of sockets, cores per socket and threads per core.
13:11:35 sean-k-mooney im not sure it shoudl be reported at the numa node level however
13:11:43 sean-k-mooney given that its vm wide
13:11:55 alex_xu sean-k-mooney: yea, at least not in the numa cell
13:12:59 sean-k-mooney i would add cpu_topology:{sockets:1,cores:10, threads:2} as a sibling but not childe of nodes
13:13:12 alex_xu whether we have separated file for sockets, cores and threads, that should be a question for yonglihe and chris
13:13:27 sean-k-mooney that said i have not really given that much tought
13:14:40 alex_xu sean-k-mooney: I will tell yonglihe, let him check with starlingx whether need sockets, cores and threads
13:15:07 openstackgerrit Merged openstack/nova master: Add infrastructure for invoking libvirt's getDomainCapabilities API https://review.opendev.org/655268
13:16:34 alex_xu sean-k-mooney: actually, we won't have different hugepage size for cells
13:17:06 alex_xu i'm not sure this one also
13:18:23 sean-k-mooney correct we wont
13:18:43 sean-k-mooney we store the info in the db percell

Earlier   Later