Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-20
16:25:04 aspiers Now I need https://review.opendev.org/#/c/638680/ to *also* depend on this test_servers.py refactoring
16:25:30 efried oh, I see, you want to make a downward-facing tree
16:25:32 aspiers sean-k-mooney: like I said there are tons of trailing "." already in the history. That was just a joke ...
16:25:36 efried yeah, don't bother with that
16:25:42 efried just put the refactor on the bottom of the pile.
16:25:51 aspiers The top of the pile?
16:26:20 aspiers 655268 already has lots of + votes
16:26:21 efried "bottom" from the perspective of the 'Related changes' list on the top right of the gerrit panel.
16:26:44 aspiers Is that list ordered?
16:26:45 efried right, and if you rebase that onto the refactor, you'll preserve those votes.
16:26:54 efried aspiers: Yes, "first" is at the bottom
16:27:08 efried etc
16:28:27 aspiers Hrm
16:28:42 aspiers Yeah, this is *definitely* a flaw in the Gerrit model if it doesn't support doing this properly
16:29:04 aspiers A good model should definitely not force artificial dependencies between changes like this
16:30:19 aspiers We're assuming here that Gerrit can only handle a linear sequence of dependencies
16:30:30 aspiers Or maybe you know that for a fact
16:30:39 aspiers Either way, it should be able to handle trees
16:30:54 efried say wha?
16:30:57 efried oh
16:31:12 efried you can do upward-facing trees (again, upward from the perspective of Related Changes)
16:31:28 efried That is, you can have both B and C depend on A but not on each other.
16:31:38 efried gerrit will show B and C in arbitrary order, but both will always be above A
16:31:46 aspiers Yes, but this is the opposite way up
16:31:54 aspiers A depending on both B and C
16:31:58 efried Right, that's what you can't have.
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

Earlier   Later