Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-28
20:25:51 openstackgerrit Carlos Goncalves proposed openstack/os-vif master: Introduce network namespace aware plugging https://review.openstack.org/546961
20:31:17 melwitt zzzeek: ok, so the old way was context_manager.get_legacy_facade().get_engine() but when I do context_manager.get_engine() it does ValueError("mode should be WRITER or READER")'
20:31:38 zzzeek melwitt: yes, try context_manager.reader.get_engine()
20:32:22 melwitt zzzeek: ok, it's just that these get_engine methods we have were not calling out whether they were reader or writer
20:32:31 zzzeek melwitt: the reader is the "slave"
20:32:37 zzzeek you want the "slave" right
20:32:57 melwitt zzzeek: not always, only if use_slave=True was passed
20:32:59 zzzeek melwitt: if you want the non-slave hten ctx.writer.get_engine()
20:33:52 melwitt zzzeek: ok. what about this one? is this one implying that it is a writer then? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L140
20:34:16 zzzeek that sohuld be the same as the "writer" yes
20:34:39 melwitt ok, thank you. I didn't realize that's what they were underneath
20:44:33 melwitt ah, yup I see it here now https://github.com/openstack/oslo.db/blob/dff11337f55b7a9a28d7c30658df51b7b0b443eb/oslo_db/sqlalchemy/enginefacade.py#L1290
20:46:24 openstackgerrit Merged openstack/nova stable/ocata: Cleanup the caches when deleting a resource provider https://review.openstack.org/604021
20:46:30 openstackgerrit Merged openstack/nova stable/queens: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/603737
21:14:39 openstackgerrit Merged openstack/nova master: nova-manage - fix online_data_migrations counts https://review.openstack.org/605329
21:15:24 openstackgerrit Merged openstack/nova stable/queens: cover migration cases with functional tests https://review.openstack.org/602408
22:10:29 openstackgerrit melanie witt proposed openstack/nova master: Replace usage of get_legacy_facade() with get_engine() https://review.openstack.org/606213
22:38:26 openstackgerrit melanie witt proposed openstack/nova master: Use an alternative class 'MoxStubout' https://review.openstack.org/579482
#openstack-nova - 2018-09-29
00:15:42 openstackgerrit Merged openstack/nova master: Follow up for Iba230201803ef3d33bccaaf83eb10453eea43f20 https://review.openstack.org/605653
01:26:38 openstackgerrit Merged openstack/nova master: Follow up for Ie991d4b53e9bb5e7ec26da99219178ab7695abf6 https://review.openstack.org/605658
01:51:33 openstackgerrit Merged openstack/nova stable/rocky: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/605260
02:13:40 openstackgerrit Merged openstack/nova stable/rocky: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/605261
03:20:24 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
06:49:45 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
07:54:22 openstackgerrit Merged openstack/nova stable/queens: Update RequestSpec.flavor on resize_revert https://review.openstack.org/602409
08:47:45 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support ``volume_type`` https://review.openstack.org/605573
09:52:02 openstackgerrit Filippo Inzaghi proposed openstack/os-traits master: Removed older version of python added 3.5 https://review.openstack.org/606370
11:43:29 openstackgerrit Brin Zhang proposed openstack/nova master: Add compute API version for when a ``volume_type`` is requested https://review.openstack.org/605573
12:08:20 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
12:17:38 openstackgerrit Brin Zhang proposed openstack/nova master: Add compute API version for when a ``volume_type`` is requested https://review.openstack.org/605573
12:27:41 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
20:23:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix neutron-tempest-linuxbridge irrelevant-files https://review.openstack.org/606616
#openstack-nova - 2018-09-30
00:12:00 openstackgerrit Merged openstack/nova master: Fix stacktraces with redis caching backend https://review.openstack.org/605748
02:48:13 openstackgerrit Brin Zhang proposed openstack/nova master: Add API samples test for volume_type https://review.openstack.org/606758
02:54:53 openstackgerrit Brin Zhang proposed openstack/nova master: Add API samples test for ``volume_type`` https://review.openstack.org/606758
05:55:13 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
06:04:32 openstackgerrit Brin Zhang proposed openstack/nova master: Add API samples test for ``volume_type`` https://review.openstack.org/606758
07:54:08 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
11:48:37 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031
12:00:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031
13:38:26 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge image_size extension response into image view builder https://review.openstack.org/606845
15:25:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Add hide server address tests in test_serversV21.py https://review.openstack.org/605435
20:50:04 openstackgerrit Merged openstack/nova master: Use INFO for logging no allocation candidates https://review.openstack.org/605765
23:08:06 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Update the CLI reference https://review.openstack.org/606871
#openstack-nova - 2018-10-01
00:43:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628
06:24:07 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
06:45:54 openstackgerrit Sam Morrison proposed openstack/python-novaclient master: Fix up userdata argument to rebuild. https://review.openstack.org/605341
07:01:30 bauzas good morning Nova
07:50:54 openstackgerrit Christoph Manns proposed openstack/nova stable/rocky: Fix stacktraces with redis caching backend https://review.openstack.org/606895
07:52:09 openstackgerrit Christoph Manns proposed openstack/nova stable/queens: Fix stacktraces with redis caching backend https://review.openstack.org/606896
07:53:46 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
09:07:47 openstackgerrit Matthew Booth proposed openstack/nova master: Raise error on timeout in wait_for_versioned_notifications https://review.openstack.org/604859
09:08:06 openstackgerrit Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733
09:08:06 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)
09:08:06 openstackgerrit Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846
09:11:58 kashyap testone nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_next_min_qemu_version_ok
09:12:47 kashyap Err, disregard that
09:13:14 openstackgerrit Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212
09:13:47 ttsiouts johnthetubaguy: I tried to refresh the spec ^
10:03:21 sean-k-mooney o/
10:04:17 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Update the contributor guide https://review.openstack.org/606926
10:05:40 sean-k-mooney stephenfin: when you get a chance can you run your eyes over https://review.openstack.org/#/c/604992/3. looks like an easy addtion to me.
10:05:48 stephenfin ack
10:06:23 sean-k-mooney stephenfin: i think we need to port other vif types from kuyr-kubernetes too but they can be in different patches
10:35:08 sean-k-mooney ralonsoh: hi are you around today?
10:35:47 ralonsoh sean-k-mooney: very well, fighting with DVR and FIP (and SRIOV)
10:36:52 stephenfin sean-k-mooney: Also, "i prefer explcit else clauses"
10:37:00 stephenfin sean-k-mooney: Just wanted you to know you're wrong
10:37:04 stephenfin sean-k-mooney: That is all.
10:37:05 stephenfin :)
10:37:17 sean-k-mooney ralonsoh: if you bussy i can respin https://review.openstack.org/#/c/605422/2
10:38:05 ralonsoh sean-k-mooney: sure, I'll do this now
10:38:40 sean-k-mooney stephenfin: hehe well we can agree to disagree on that. it can help prevent some types of errors when you are maintaing the fucntion in the future but also consumes indentation space so is not allways desireable
10:48:35 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Rewrite the console doc https://review.openstack.org/606148
10:48:35 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Allow 'nova-xvpvncproxy' to be called with CLI args https://review.openstack.org/606929
10:51:27 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IPTools deprecated implementation https://review.openstack.org/605422
10:51:45 ralonsoh sean-k-mooney: done
10:55:27 sean-k-mooney ralonsoh: thanks :)
11:11:38 openstackgerrit Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/554218
11:15:21 openstackgerrit Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212
11:28:36 openstackgerrit Merged openstack/nova stable/rocky: Use INFO for logging no allocation candidates https://review.openstack.org/605771
11:48:27 openstackgerrit Merged openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405
12:05:59 kashyap stephenfin: On that version bump thing for libvirt, I just looked at "what's involved" to clean up
12:06:15 kashyap stephenfin: I need to address at least 10 unit tests. Will get to it this week. Currently buried in something a bit more urgent
12:30:43 stephenfin kashyap: Yup, it's low priority and can be done anytime this cycle. No rush!
12:31:56 kashyap Yep.
12:32:40 kashyap Then, I might even wait until I'm through all the CPU modelling stuff that I'm looking into (until 27 Oct)
12:40:36 mdbooth bauzas: LM failure on https://review.openstack.org/#/c/604400/
12:41:05 mdbooth So, the exact failure is unlikely to be relevant. Chances are that this is just the first thing which touched that host, and the host was broken by the evacuate test we just ran.
12:41:35 bauzas mdbooth: so, you got a failure for req-dd88c142-4710-49d0-b6e9-594c6a1e89f6 (live-migrate action) because of http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_39_55_712058
12:41:52 mdbooth bauzas: Yes
12:42:01 bauzas rather http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_39_55_711740
12:42:07 mdbooth So, the question is: why does the computefilter think the host is down?
12:42:26 bauzas mdbooth: because the db record is old
12:42:45 bauzas mdbooth: I mean the updated_at filed
12:42:47 bauzas field*
12:42:52 mdbooth bauzas: Right, but in the compute logs I see the host joining the service group within the last 60 secs
12:42:58 bauzas https://github.com/openstack/nova/blob/master/nova/scheduler/filters/compute_filter.py#L44
12:43:12 mdbooth However, I note that it doesn't update the record at that point, just schedules it to be updated in 5 seconds
12:43:40 mdbooth So I then got to wondering why it doesn't seem to have been updated 5 seconds later.

Earlier   Later