Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-28
16:34:10 mdbooth cfriesen: Consider all the error cleanup edge cases you'd have to consider
16:34:17 cfriesen fair enough...just thought I'd mention it
16:35:46 mdbooth Incidentally, it also makes sense to do this test *before* taking a claim, as it's really scheduler take 2
16:37:54 cfriesen arguably both the claim and the affinity validation are both scheduler take 2.
16:38:37 cfriesen but sure, it'd be less wasted work to check affinity first
16:41:56 mdbooth afaict the syntax documented here is wrong: https://docs.openstack.org/infra/system-config/logstash.html#query-logstash
16:42:18 mdbooth I can't make any @message query return anything
16:43:10 mdbooth @fields.loglevel:"ERROR" returns nothing
16:43:23 mdbooth ...which is nice. We have no errors :)
16:45:22 mdbooth Similarly striking out here: https://www.elastic.co/guide/en/kibana/3.0/queries.html
16:48:40 mdbooth Ah, ha! Drop the @ seems to work
17:11:46 melwitt mriedem_lunch: roger copy
17:19:55 openstackgerrit Jack Ding proposed openstack/nova master: Handle missing marker during online data migration https://review.openstack.org/605164
17:37:06 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Rewrite the console doc https://review.openstack.org/606148
17:38:12 finucannot melwitt, mriedem_lunch: You two might be interested in that ^
17:38:55 melwitt ack
17:55:11 mriedem easy fix needs a +W https://review.openstack.org/#/c/605748/
18:12:05 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Null out instance.availability_zone on shelve offload https://review.openstack.org/606155
18:15:13 openstack Launchpad bug 1732199 in OpenStack Compute (nova) "test_extend_attached_volume fails with Unexpected compute_extend_volume result 'Error'" [Medium,Confirmed]
18:15:13 imacdonn mriedem: Fix merged in os-brick that will hopefully address https://bugs.launchpad.net/nova/+bug/1732199 - thing is, I'm not sure where zuul gets os-brick from - it appears to be using /usr/local/lib/python2.7/dist-packages/os_brick, so not sure if it needs a release
18:20:49 mriedem imacdonn: yes nova can't use it until it's released and upper-constraints has that new version
18:21:22 mriedem imacdonn: so probably work with jungleboyj to see when he's ready to do a new os-brick release
18:21:45 jungleboyj :-)
18:22:03 jungleboyj imacdonn: I can look at doing that.
18:22:16 jungleboyj It will cost you a beer though. ;-)
18:22:17 imacdonn mriedem jungleboyj: OK. We might want to let this soak a little bit ... I think Gorka was a bit apprehensive about the fix
18:22:58 mriedem wfm,
18:22:59 jungleboyj imacdonn: Ok, want to put it on next week's meeting agenda and we can see how things look then?
18:23:05 mriedem it's been a latent issue since queens
18:23:28 mriedem note that nothing will be using this in CI until it's released and in upper-constraints
18:23:33 mriedem so i'm not sure how you're going to soak it in
18:23:43 mriedem well, except os-brick CI itself
18:23:48 jungleboyj mriedem: Oh, good point.
18:24:00 imacdonn yeah, I was just pondering that too
18:24:38 imacdonn I just I'd at least want to give Gorka a chance to object before a release is made
18:24:44 imacdonn I guess*
18:25:05 jungleboyj I can propose a release and tag Gorka to review.
18:25:16 imacdonn OK, that works
18:25:50 jungleboyj Lets do that.
18:25:59 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Null out instance.availability_zone on shelve offload https://review.openstack.org/606161
19:38:17 melwitt zzzeek: hey, are you around?
19:38:25 zzzeek melwitt: heya
19:39:21 melwitt zzzeek: hey, I'm looking at how we can eliminate the warnings we're getting from our code, "OsloDBDeprecationWarning: EngineFacade is deprecated; please use oslo_db.sqlalchemy.enginefacade'"
19:39:51 melwitt I suspect they're coming from calls like this https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L137
19:40:25 zzzeek melwitt: probably
19:40:27 melwitt and I'm looking for the correct supported way in oslo.db to achieve the same thing
19:41:16 zzzeek melwitt: there might be a direct engine accessor at this point but the original idea was you wouldnt be using an "Engine" directly
19:41:48 zzzeek melwitt: looks like you can just drop the get_legacy_facade() part
19:41:49 melwitt hm, ok
19:41:53 zzzeek melwitt: there's a get_engine() now
19:42:17 melwitt zzzeek: I saw that, but it doesn't have the ability to specify use_slave, correct?
19:42:30 zzzeek melwitt: the "async reader" would be the "slave"
19:43:01 zzzeek so...allow_async.reader.get_engine() ? I guess
19:43:15 melwitt oh, I see. ok
19:43:48 melwitt I think that would do it then, thank you. looking to make an incremental progress
19:44:43 zzzeek melwitt: or maybe async_.reader.get_engine....sorry i havent looked at this in over a year
19:45:05 melwitt zzzeek: np, I can find it. just needed that hint
19:47:13 zzzeek melwitt: Ok i think "reader engine" is the slave, async or not
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

Earlier   Later