| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-28 | |||
| 16:20:25 | mdbooth | cfriesen: Apologies, Canada. Of course I knew that... | |
| 16:20:45 | mdbooth | I had it in my head you were in the UK for some bizarre reason. | |
| 16:21:43 | mriedem | ocata-em call to arms email sent | |
| 16:22:40 | cfriesen | mdbooth: no worries, we have the same queen. :) | |
| 16:22:48 | mdbooth | Incidentally, as a bonus the above patch will also correctly handle multiple builds from the same anti-affinity group sent to a single compute. Previously they could all be rejected, now it will always allow max_server_per_host to succeed | |
| 16:23:40 | mdbooth | That's builds in addition to rebuilds | |
| 16:31:26 | mdbooth | Is there some documentation somewhere of the query language used by logstash? | |
| 16:31:44 | mdbooth | This was what got me last time I tried to use this. | |
| 16:31:48 | cfriesen | mdbooth: do we still need the nested contexts given we're basically doing refcounting? I mean, it's an elegant way to ensure we call builders.remove(instance.uuid) but we could presumably also do that other ways. Just wondering if it'd end up being less code churn. | |
| 16:32:32 | mdbooth | cfriesen: I'd strongly prefer to keep the nested contexts | |
| 16:32:49 | mdbooth | Otherwise you'll end up having disjoint code, which is a maintenance nightmare | |
| 16:33:07 | mdbooth | and we have enough of it already | |
| 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 | |