Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-10-21
17:37:49 stephenfin let's see what CI thinks
17:37:53 gibi thanks
18:06:21 ganso melwitt, elod: Hi! If you have a minute could you please take a look at this stable/stein backport that is the same as the stable/train you previously: https://review.opendev.org/#/c/757886/ ? Thanks in advance!
18:08:52 openstackgerrit melanie witt proposed openstack/nova stable/train: Follow up for cherry-pick check for merge patch https://review.opendev.org/759118
18:09:13 melwitt ganso: ack, will look later
18:09:25 ganso melwitt: thanks!
18:16:04 openstackgerrit sean mooney proposed openstack/nova master: Fix unplugging VIF when migrate/resize VM https://review.opendev.org/751642
18:38:13 openstackgerrit sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.opendev.org/602432
18:39:11 lyarwood sean-k-mooney: if you're still around, did I see you talking about an ipv6 issue earlier today?
18:39:37 lyarwood sean-k-mooney: just noticed tempest-ipv6-only failing across a few of my changes
18:39:39 sean-k-mooney thats in the world dump script and limestone
18:39:52 lyarwood sean-k-mooney: kk, did you have a bug for that?
18:40:14 sean-k-mooney so https://review.opendev.org/#/c/759042/ is the fix for the worlddump script
18:40:28 sean-k-mooney and the limestone ci seam to be ipv6 by default
18:40:46 sean-k-mooney but there might be a dhcp/nat issue
18:40:59 lyarwood kk
18:41:06 lyarwood I'll leave my rechecks until the morning then
18:41:09 sean-k-mooney the vms are ment to get ipv4 private ips but devstack failed because it had not ipv4 adress atall
18:41:10 lyarwood thanks!
22:44:31 openstackgerrit Xav Paice proposed openstack/nova stable/ussuri: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/759151
23:16:12 openstackgerrit Xav Paice proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/759153
23:32:06 openstackgerrit Merged openstack/nova master: virt: Remove 'get_console_pool_info' API https://review.opendev.org/749311
#openstack-nova - 2020-10-22
01:12:42 openstackgerrit Xav Paice proposed openstack/nova stable/ussuri: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/759151
01:34:19 openstackgerrit Xav Paice proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/759153
04:49:09 openstackgerrit Merged openstack/os-vif master: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/745895
06:28:08 bauzas good morning Nova
07:11:46 gibi bauzas: good morning
07:11:59 bauzas :)
08:49:59 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/759114
09:05:19 gibi stephenfin: now I feel making o.vo fields typesafe is fultile. E.g. an o.vo IntegerField accept strings and coerce them to integeres with an int() conversion. So if I type an IntegerField to int then mypy will flag obj.int_field = '123' assignemnet which is valid in o.vo. Also if I map it to str then mypy will not flag obj.int_field = 'bob' which is invalid in o.vo
09:05:33 gibi s/flutile/futile
09:14:37 gibi which means it is very limited what we can typecheck
09:16:18 gibi List*Fields are complicated as they are coreced from Iterables but not from str or Mapping myobj.list_of_ints = {1, 2, 3}
09:16:36 gibi sorry I wanted to link https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L648
09:17:24 gibi btw myobj.list_of_ints = {'1', '2', '3'} also accepted by ovo as the elements of the iterable is coerced to int
09:27:27 stephenfin gibi: I hadn't considered coercion /o\
09:28:06 stephenfin Hmm, that does sound pretty futile alright. Damn :-\
09:29:49 stephenfin I had been thinking it might make sense to at least define the attributes and simply set their type to Any, so that we'd know you could define e.g. 'InstanceNUMACell.cpu_policy' but not '.cpu_policyyyy'. However, I guess we rely on that to store working data
09:32:48 gibi stephenfin: I can generate the fields with https://github.com/gibizer/ovo-mypy-plugin and map them to Any. I guess non o.vo fields defined via simply assigning value to them in nova code. So those can be typed in nova code and picked up by mypy
09:34:42 stephenfin gibi: I think there's probably value in that. I mean, it sounds like the best we can get unless we wanted to drop the coerce functionality
09:35:08 stephenfin which, tbc, I wouldn't be entirely against either; explicit is better than implicit and all that
09:36:12 gibi stephenfin: I don't know if the generic RPC code depends on coerce or not. But the specific nova usage of o.vo can be converted to not use coerce
09:37:13 gibi or at least to use explicit coerce
09:37:22 gibi instead of implicit during assignment
09:39:16 stephenfin Even if the RPC code did rely on it, type checking doesn't happen at run time so we could require explicit coercion for non-RPC cases
09:39:27 stephenfin i.e. get static type checking working
10:12:56 gibi stephenfin: so then we are back to defining IntegerField as int even though we know that o.vo is more forgiving than that at runtime?
10:41:21 openstackgerrit Merged openstack/nova master: virt: Remove 'post_interrupted_snapshot_cleanup' API https://review.opendev.org/749313
12:32:49 openstackgerrit sean mooney proposed openstack/os-vif stable/victoria: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/759255
12:33:06 sean-k-mooney lyarwood: can you reivew https://review.opendev.org/#/c/756406 and https://review.opendev.org/#/c/759255/
12:33:24 sean-k-mooney they are os-vif stable/victoria backports
12:35:28 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Kilo database migrations https://review.opendev.org/759256
12:36:43 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Kilo database migrations https://review.opendev.org/758396
12:40:41 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Kilo database migrations https://review.opendev.org/758396
12:54:12 lyarwood sean-k-mooney: ack will do
12:57:42 lyarwood sean-k-mooney: done, LGTM.
12:59:11 sean-k-mooney lyarwood:thanks
13:02:53 sean-k-mooney lyarwood: if you fell like doing other reviews https://review.opendev.org/#/q/topic:bug/1895220+(status:open+OR+status:merged) is related to a downstream escalation
13:03:19 sean-k-mooney i have just rechecked them but i would really like to get thos merged sooner rather then later
13:04:10 lyarwood sean-k-mooney: ack queued
13:12:58 openstackgerrit Merged openstack/nova master: fixtures: Handle binding of first port https://review.opendev.org/746949
13:13:06 openstackgerrit Merged openstack/nova master: Remove self.server dependency form _evacuate_server https://review.opendev.org/758209
15:47:57 gibi nova weekly meeting starts in 12 minutes in #openstack-meeting-3
16:07:25 openstackgerrit Merged openstack/nova stable/victoria: libvirt: Increase incremental and max sleep time during device detach https://review.opendev.org/757305
16:32:44 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Kilo database migrations https://review.opendev.org/758396
16:32:45 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Liberty database migrations https://review.opendev.org/758397
16:32:45 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Mitaka database migrations https://review.opendev.org/758398
16:32:46 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Newton database migrations https://review.opendev.org/759085
16:32:46 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Ocata database migrations https://review.opendev.org/759086
16:32:47 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Pike database migrations https://review.opendev.org/759087
16:32:47 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Queens database migrations https://review.opendev.org/759088
16:32:48 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Rocky database migrations https://review.opendev.org/759089
16:32:48 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Stein database migrations https://review.opendev.org/759090
16:32:49 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Train database migrations https://review.opendev.org/759091
16:33:31 openstackgerrit melanie witt proposed openstack/nova master: DNM test https://review.opendev.org/759294
16:39:44 openstackgerrit melanie witt proposed openstack/nova master: DNM test https://review.opendev.org/759294
21:27:42 openstackgerrit melanie witt proposed openstack/nova master: DNM test https://review.opendev.org/759294
#openstack-nova - 2020-10-23
04:09:53 openstackgerrit melanie witt proposed openstack/nova master: WIP Omit resource inventories from placement update if zero https://review.opendev.org/759348
07:42:16 openstackgerrit Jeffrey Zhang proposed openstack/nova master: Support inject-nmi action in watchdog https://review.opendev.org/741072
08:37:20 openstackgerrit Jorhson Deng proposed openstack/nova stable/rocky: optimize the shelve code flow https://review.opendev.org/759373
08:38:25 openstackgerrit Jorhson Deng proposed openstack/nova stable/ussuri: optimize the shelve code flow https://review.opendev.org/759374
08:39:26 openstackgerrit Jorhson Deng proposed openstack/nova stable/train: optimize the shelve code flow https://review.opendev.org/759375
08:39:54 openstackgerrit Jorhson Deng proposed openstack/nova stable/stein: optimize the shelve code flow https://review.opendev.org/759376
08:41:04 openstackgerrit Jorhson Deng proposed openstack/nova stable/queens: optimize the shelve code flow https://review.opendev.org/759377
08:41:34 openstackgerrit Jorhson Deng proposed openstack/nova stable/pike: optimize the shelve code flow https://review.opendev.org/759378
08:41:58 openstackgerrit Jorhson Deng proposed openstack/nova stable/ocata: optimize the shelve code flow https://review.opendev.org/759379
08:44:30 lyarwood ^ needs to start from stable/victoria btw, I've updated the stable/ussuri commit if you could take a look.
08:49:45 deke997 hey guys
08:49:53 deke997 I'm trying to get some updated info on multi-function pci passthrough
08:50:08 deke997 last I checked it was not really supported
08:50:09 deke997 anyone know more on this?
10:13:01 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Liberty database migrations https://review.opendev.org/758397
10:13:02 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Mitaka database migrations https://review.opendev.org/758398
10:13:02 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Newton database migrations https://review.opendev.org/759085
10:13:03 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Ocata database migrations https://review.opendev.org/759086
10:13:03 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Pike database migrations https://review.opendev.org/759087
10:13:04 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Queens database migrations https://review.opendev.org/759088
10:13:04 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Rocky database migrations https://review.opendev.org/759089
10:13:05 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Stein database migrations https://review.opendev.org/759090
10:13:05 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Compact Train database migrations https://review.opendev.org/759091

Earlier   Later