Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-30
21:03:57 cdent zzzeek: the in series is what caused the need for the complicated reset business. the situation has probably changed somewhat now that https://review.openstack.org/#/c/619121/ has merged, which changes the Database handling-in-tests
21:04:27 zzzeek cdent: the reset handling here doesn't look that complicated. nova had soemthig much more exotic
21:04:53 cdent i meant the stuff where _started is being manipulated
21:04:58 mriedem zzzeek: unrelated, do you know much about group by and distinct clauses in postgresql?
21:05:06 zzzeek mriedem: sure
21:05:17 mriedem zzzeek: any sort of knowledge on https://review.openstack.org/#/c/619061/ for that would be great
21:05:30 zzzeek mriedem: i would say, it's unusual to be mixing those two things
21:05:35 mriedem my fix is basically based on internet forums...
21:05:43 zzzeek mriedem: also....i dont follow that often, are we supporting postgresql again in openstack ?
21:05:57 mriedem i believe the official stance is "meh"
21:06:04 zzzeek mriedem: OK so no change
21:06:06 mriedem we don't turn away fixes
21:06:16 mriedem but we don't bend over backwards to test and support it either
21:06:23 cdent my understanding is that every now and again mriedem and I try to make it happen
21:06:40 mriedem some neutron and manila people apparently pay attention to pg also
21:07:00 zzzeek mriedem: um, distinct(col_expr) doesnt work on mysql or sqlite unless I'm deeply confused
21:07:05 zzzeek mriedem: how is that working?
21:07:24 mriedem i assumed the underlying sqla plugins were just nooping or something, idk
21:07:30 mriedem i threw up a fix and it made the pg job work again
21:07:35 zzzeek mriedem: also...i dont see why this query uses group_by
21:07:43 cdent zzzeek: it's late and friday for me, so I'm going to watch bad tv and fall asleep. If you've got thoughts on that placement alembic stuff, please let me know, either on that review or on some wip that you invite me to look at
21:07:52 mriedem as noted in my comments, it might not be the best query to start with
21:07:57 cdent everyone have a good weekend
21:07:58 mriedem and simply rewriting the query could fix it for both
21:08:07 mriedem o/
21:10:09 zzzeek mriedem: does InstanceMapping have any column_property() that points to like func.max() or func.count() or an aggregate of some kind ?
21:10:35 zzzeek cdent: my plan is to make this work and put up a review
21:10:41 zzzeek cdent: or if oslo_db is broke, do taht first
21:11:06 cdent zzzeek: great, whichever it is point it out to me, will be very curious
21:12:55 mriedem zzzeek: this is the model https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L131
21:13:43 mriedem so i think the answer is "no"
21:14:20 zzzeek mriedem: sure, it would be extremely weird to map an aggregate
21:15:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert "Add regression test for bug 1550919" https://review.openstack.org/621289
21:15:36 openstack bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)
21:50:44 mriedem SteelyDan: leakypipes: there is some sage advice on here that i wanted to share with you https://www.siouxfallsshoppingnews.com/publishers-message.html
21:51:06 mriedem "Life is simpler when you plow around the stump."
21:51:13 mriedem i can tell you privately what i think that means
21:56:53 SteelyDan um.
21:57:38 openstackgerrit Merged openstack/nova stable/rocky: Don't use private interface in oslo.service https://review.openstack.org/619360
22:20:54 openstackgerrit Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
22:20:54 openstackgerrit Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
22:20:55 openstackgerrit Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
22:20:55 openstackgerrit Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042
22:20:56 openstackgerrit Eric Fried proposed openstack/nova master: Rip out the SchedulerClient https://review.openstack.org/617049
22:20:56 openstackgerrit Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711
22:27:24 openstackgerrit Merged openstack/nova stable/queens: Handle missing marker during online data migration https://review.openstack.org/610974
22:51:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
22:51:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.openstack.org/614035
22:51:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
22:51:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
22:51:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353
22:51:36 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
22:51:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Isolate cell-targeting code in MigrationTask https://review.openstack.org/621310
22:51:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Extract compute API _create_image to compute.utils https://review.openstack.org/621311
22:51:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Extract shelve API logic to compute.utils https://review.openstack.org/621312
22:51:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313
22:52:49 mriedem oooo under 2K LOC now
22:52:54 mriedem winning
#openstack-nova - 2018-12-01
05:07:20 openstackgerrit Merged openstack/nova master: Fix misuse of assertTrue https://review.openstack.org/620861
05:23:46 openstackgerrit Merged openstack/nova master: Revert "Add regression test for bug 1550919" https://review.openstack.org/621289
05:23:46 openstack bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)
10:38:40 openstackgerrit Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941
10:38:40 openstackgerrit Chris Dent proposed openstack/nova master: Delete the placement code https://review.openstack.org/618215
11:16:08 openstackgerrit Michael Still proposed openstack/nova master: Remove utils.execute() calls from xenapi. https://review.openstack.org/619700
11:16:09 openstackgerrit Michael Still proposed openstack/nova master: Remove utils.execute() from libvirt remotefs calls. https://review.openstack.org/619701
11:16:09 openstackgerrit Michael Still proposed openstack/nova master: Remove utils.execute() from quobyte libvirt storage driver. https://review.openstack.org/619702
11:16:10 openstackgerrit Michael Still proposed openstack/nova master: Move nova.libvirt.utils away from using nova.utils.execute(). https://review.openstack.org/619703
11:16:10 openstackgerrit Michael Still proposed openstack/nova master: Imagebackend should call processutils.execute directly. https://review.openstack.org/619704
11:16:11 openstackgerrit Michael Still proposed openstack/nova master: Remove final users of utils.execute() in libvirt. https://review.openstack.org/619705
11:16:11 openstackgerrit Michael Still proposed openstack/nova master: Remove the final user of utils.execute() from virt.images https://review.openstack.org/620007
11:16:12 openstackgerrit Michael Still proposed openstack/nova master: Remove utils.execute() from the hyperv driver. https://review.openstack.org/620008
11:16:12 openstackgerrit Michael Still proposed openstack/nova master: Remove utils.execute() from virt.disk.api. https://review.openstack.org/620009
11:16:13 openstackgerrit Michael Still proposed openstack/nova master: Move a generic bridge helper to a linux_net privsep file. https://review.openstack.org/620010
11:16:13 openstackgerrit Michael Still proposed openstack/nova master: Move bridge creation to privsep. https://review.openstack.org/620180
11:16:14 openstackgerrit Michael Still proposed openstack/nova master: Fix sloppy initialization of the new disk ops semaphore. https://review.openstack.org/621346
#openstack-nova - 2018-12-02
08:02:36 openstackgerrit Adrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115
10:15:03 openstackgerrit wangyue proposed openstack/nova master: Set usb controller model of server which os_type is windows https://review.openstack.org/621397
10:21:48 openstackgerrit Michael Still proposed openstack/nova master: Move some linux network helpers to use privsep. https://review.openstack.org/621398
13:52:50 openstackgerrit Alex Xu proposed openstack/nova master: Parse the numa topology from the new flavor when resize instance https://review.openstack.org/620874
13:52:50 openstackgerrit Alex Xu proposed openstack/nova master: Parse the new PCI request from flavor extra specs when resize https://review.openstack.org/621077
16:00:22 openstackgerrit Adrian Chiris proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929
16:00:23 openstackgerrit Adrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115
#openstack-nova - 2018-12-03
02:05:35 openstackgerrit Alex Xu proposed openstack/nova master: Parse the numa topology from the new flavor when resize instance https://review.openstack.org/620874
02:05:36 openstackgerrit Alex Xu proposed openstack/nova master: Parse the new PCI request from flavor extra specs when resize https://review.openstack.org/621077
02:28:59 openstackgerrit Corey Bryant proposed openstack/nova stable/pike: Revert "Backport tox.ini to switch to stestr" https://review.openstack.org/621227
04:10:04 openstackgerrit Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499
05:28:36 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
05:28:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
05:29:15 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628
05:29:36 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix flavor keyerror when nova boot vm https://review.openstack.org/582147
05:30:07 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125
05:30:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add descriptions about microversions https://review.openstack.org/619144
05:30:36 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Add description of custom resource classes https://review.openstack.org/619122
05:54:02 openstackgerrit Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474
05:57:08 openstackgerrit Yongli He proposed openstack/nova master: Add numa topology and pci devices into server detail API https://review.openstack.org/621476
07:37:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add descriptions of numbered resource classes and traits https://review.openstack.org/621494
07:38:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
07:39:16 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect https://review.openstack.org/607400
07:45:11 openstackgerrit Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499
07:45:11 openstackgerrit Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995

Earlier   Later