Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-17
20:17:08 mriedem superdan: do you know if we read deleted any of those?
20:17:21 superdan on compute nodes?
20:17:32 melwitt aggregates and pci devices also
20:17:38 mriedem i have like 1% knowledge of the pci_devices code
20:17:49 superdan definitely not pci_devices
20:18:32 superdan melwitt: if we did read aggregates, we'd generate on load: https://github.com/openstack/nova/blob/stable/newton/nova/objects/aggregate.py#L288
20:18:34 superdan so we're okay
20:18:57 melwitt ah, nice
20:20:38 mriedem we mostly read deleted instances
20:20:43 mriedem so yeah i think we're ok there
20:21:18 mriedem and flavors
20:24:23 mriedem just testing the test quick
20:24:28 mriedem do we want/need a release note for this?
20:24:32 mriedem since it's going to be backported
20:24:54 mriedem like even a "fixes" note or "upgrade" note? as i said earlier, if you hit this you can't get past 330 anyway
20:25:02 mriedem so this is more about fixing it before people hit it
20:25:23 mriedem last time we backported a db schema migration to stable and released it i got some flak from red hat
20:27:56 superdan this isn't backporting a new one,
20:28:00 superdan it's just tweaking an old one,
20:28:02 mriedem i know
20:28:03 superdan which we did in ocata too
20:28:06 mriedem melwitt: see my comment in the test
20:28:16 superdan but release notes are cool
20:29:03 mriedem so the test doesn't fail on the aggregate b/c you're using an object, which creates the aggregate in the api db
20:29:11 mriedem i don't know why it doesn't fail on the compute node when i remove your fi
20:29:13 mriedem *fix
20:29:15 melwitt oh, hm
20:29:21 superdan mriedem: it's because it has a uuid
20:29:24 superdan mriedem: if you reate it
20:29:27 superdan I'm thinking
20:29:27 mriedem ah
20:29:29 mriedem yup
20:29:35 superdan she'd have to use the db api
20:29:37 melwitt okay, let me fix this to put things in the old db
20:30:17 mriedem yup
20:30:24 mriedem pci device object one is fine
20:30:31 mriedem but use the db api directly for the compute notes and aggregates
20:30:44 mriedem fwiw i've put a hold on https://review.openstack.org/#/c/434928/ until we get this backported and merged
20:30:44 melwitt right
20:33:46 diana_clarke melwitt: Here's an easy review that fixes up the mismatch in metavars you noticed a couple of weeks back: https://review.openstack.org/#/c/431714/
20:34:06 diana_clarke Have a blast at the PTG, folks!
20:42:25 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402
20:42:50 diana_clarke melwitt: And here are some manual testing notes that should make that easier to review: http://paste.openstack.org/show/598292/
20:43:15 diana_clarke (anyhoo, no pressure, just cleaning up my TODO list)
20:46:18 melwitt diana_clarke: okay, thanks
20:46:30 diana_clarke lol, I just read "python-wsmanclient" as "python-womanclient"...
20:57:15 mriedem walmartclient
21:11:28 openstackgerrit Anusha Unnam proposed openstack/nova master: Stop using mox from tests/unit/test_service.py https://review.openstack.org/334395
21:24:56 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Find InstanceUnknownCell instances https://review.openstack.org/435600
21:35:41 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Find InstanceUnknownCell instances https://review.openstack.org/435600
21:38:04 openstackgerrit Sujitha proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_create_backup.py https://review.openstack.org/435610
21:38:59 openstackgerrit Sean Dague proposed openstack/nova master: Verify project_id when quotas are checked https://review.openstack.org/435010
21:38:59 openstackgerrit Sean Dague proposed openstack/nova master: Verify project id for flavor access calls https://review.openstack.org/435432
21:42:25 openstackgerrit Anusha Unnam proposed openstack/nova master: Remove mox from nova/tests/unit/objects/test_service.py https://review.openstack.org/435611
22:09:36 openstackgerrit Sean Dague proposed openstack/nova master: Verify project_id when quotas are checked https://review.openstack.org/435010
22:09:36 openstackgerrit Sean Dague proposed openstack/nova master: Verify project id for flavor access calls https://review.openstack.org/435432
22:09:45 openstackgerrit melanie witt proposed openstack/nova master: Skip soft-deleted records in 330_enforce_mitaka_online_migrations https://review.openstack.org/435546
22:24:20 melwitt mriedem, superdan: fixed the unit test ^
22:24:28 mriedem yup just pulling it down
22:35:58 jamielennox mriedem: you want auth_type, auth_section allows you to move auth options to a new ini section so you can reuse them
22:37:11 mriedem jamielennox: yup, thanks. turns out i don't need that check anyway
22:37:15 mriedem but good to know
22:39:47 mriedem melwitt: superdan: +2 on that fix now
22:39:56 mriedem melwitt: want to start the backport shenanigans?
22:40:10 melwitt mriedem: thanks. yeah, sure
23:04:25 superdan guh
23:04:31 superdan I see why the cells job is failing
23:06:30 mriedem what it be?
23:07:10 superdan that change makes us skirt around the cellsv1 replication
23:07:16 superdan so we can only target if we're non-cellsv1
23:07:21 openstackgerrit Dan Smith proposed openstack/nova master: Make servers API use cell-targeted context https://review.openstack.org/433362
23:07:22 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Find InstanceUnknownCell instances https://review.openstack.org/435600
23:07:34 superdan which we did in the get and list case, it was just a while ago and I kinda forgot about why that matters
23:08:05 melwitt ah, yeah
23:25:43 mriedem superdan: +2 on those backports https://review.openstack.org/#/q/I2285005098b7dab7753366f53667ff8d4532d668,n,z
23:25:54 mriedem and now i'm getting the f out of here
23:43:38 openstackgerrit Dan Smith proposed openstack/nova master: Make servers API use cell-targeted context https://review.openstack.org/433362
23:48:35 openstackgerrit Anusha Unnam proposed openstack/nova master: Remove mox from nova/tests/unit/objects/test_service.py https://review.openstack.org/435611
23:56:54 openstackgerrit Sujitha proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_create_backup.py https://review.openstack.org/435610
#openstack-nova - 2017-02-18
00:16:00 openstackgerrit Pushkar Umaranikar proposed openstack/nova master: Add debug logs to output host weights for weighers https://review.openstack.org/385636
12:16:19 openstackgerrit Tomi Juvonen proposed openstack/nova-specs master: Planned host maintenance https://review.openstack.org/310510
13:48:03 narasimha_SV https://docs.openstack.org/kilo/config-reference/content/list-of-compute-config-options.html
13:48:42 narasimha_SV apart fromsetting compute monitor to compute_monitors =nova.compute.monitors.cpu.virt_driver
13:48:51 narasimha_SV which gives metrics of CPU
13:49:20 narasimha_SV is there any other way to get host metrics
13:50:10 narasimha_SV I am not configuring snmp protocol in ceilometer
13:50:19 narasimha_SV while checking other options i found this
18:11:05 mriedem mdbooth: when you're done burning up the slopes, maybe you can remind me why the block device mapping list passed from nova-api to nova-conductor when building instances is basically worthless https://review.openstack.org/#/c/335358/38/nova/conductor/manager.py@543
18:11:46 mriedem oh right nvm, it's because nova-conductor pulls the bdms again and completely discards the bdm list passed into build_instances
18:30:26 mriedem anyone know why we don't check the volume AZ with the instance AZ when unshelving a shelved offloaded instance?
18:30:58 mriedem unshelving a shelved offloaded instance could put it in a different AZ from when it was originally created (or when the volume was attached) which could then put the instance and volume in different AZs after the unshelve
18:33:56 mriedem unless unshelving always takes into account the original AZ for the instance?
#openstack-nova - 2017-02-19
07:19:39 openstackgerrit MarkMielke proposed openstack/nova master: Disk usage miscalculation due to instance migration https://review.openstack.org/430091
07:21:44 openstackgerrit MarkMielke proposed openstack/nova master: Disk usage miscalculation due to instance migration https://review.openstack.org/430091
07:54:23 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Raise correct error instead of class exist in Placement API https://review.openstack.org/428612
16:51:46 openstackgerrit Jeffrey Zhang proposed openstack/nova master: simple_cell_setup should not return 1 when there is no compute hosts https://review.openstack.org/435792
16:55:32 openstackgerrit Jeffrey Zhang proposed openstack/nova master: simple_cell_setup should not return 1 when there is no compute hosts https://review.openstack.org/435792
#openstack-nova - 2017-02-20
01:48:33 openstackgerrit Jeffrey Zhang proposed openstack/nova master: simple_cell_setup should not return 1 when there is no compute hosts https://review.openstack.org/435792
02:22:46 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/426636
02:22:58 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237

Earlier   Later