Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-10
06:57:00 ramishra we've the option of skipping the tests though
06:57:27 stephenfin ramishra: Yeah, good point. I think the policy is to fast-revert anything that breaks stuff, so I've done that. Someone can investigate Monday
06:57:55 ramishra stephenfin: thanks!
06:58:29 stephenfin ramishra: No problem. I'm away next week but can you work with moshele to resolve these issues, please?
06:59:00 ramishra stephenfin: sure, thanks again for quickly approving it
07:28:55 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
07:39:05 openstackgerrit Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928
08:26:57 openstackgerrit Merged openstack/nova master: Fix instance lookup in hide_server_addresses extension https://review.openstack.org/467772
08:49:13 Tengu hello! small question: what might make the following command fail? "openstack -v --os-cloud c2c/admin compute service set --up controller-0.cloud.camptocamp.com nova-conductor" I get a "Failed to set service state to up"
08:49:33 Tengu in fact, all nova-related services are down on the "master" controller (3-controller setup, with HA)
08:49:53 Tengu and we're currently unable to start new instances :/
09:25:37 openstackgerrit Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871
09:43:45 openstackgerrit Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871
09:46:48 openstackgerrit Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871
09:47:44 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
09:50:49 openstackgerrit Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871
09:52:03 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
09:54:35 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
10:01:12 openstackgerrit Merged openstack/nova master: Revert "Don't overwrite binding-profile" https://review.openstack.org/518833
10:01:43 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
10:02:43 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
22:01:59 fried_rice leakypipes NRP series needs a rebase, it appears. You on that?
22:05:37 leakypipes fried_rice: no, might do it tomorrow morning.
22:06:52 fried_rice leakypipes Ight. I mean, it'll be another week before the Sydney hangover is mostly worn off, but there ought to be *somebody* around next week to approve it.
22:08:40 leakypipes fried_rice: :)
22:35:04 fried_rice leakypipes Anything you'd like included in this week's RP update email?
22:35:07 fried_rice It's a short one.
23:42:56 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix instance lookup in hide_server_addresses extension https://review.openstack.org/518996
23:43:12 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Fix instance lookup in hide_server_addresses extension https://review.openstack.org/518997
#openstack-nova - 2017-11-11
00:06:22 openstackgerrit Merged openstack/nova-specs master: Add spec for symmetric GET and PUT of allocations https://review.openstack.org/508164
00:20:58 openstackgerrit Matt Riedemann proposed openstack/nova master: Add microversion to allow setting flavor description https://review.openstack.org/515213
00:20:58 openstackgerrit Matt Riedemann proposed openstack/nova master: Add FlavorPayload.description for versioned notifications https://review.openstack.org/514782
01:50:32 openstackgerrit Merged openstack/nova master: Move last_bytes into the path module https://review.openstack.org/509417
01:50:38 openstackgerrit Merged openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818
01:55:09 openstackgerrit Merged openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819
01:55:15 openstackgerrit Merged openstack/nova master: Deduplicated instance.(un)pause notification samples https://review.openstack.org/452820
02:13:02 openstackgerrit Merged openstack/nova master: Deduplicate instance.power-off notification samples https://review.openstack.org/475860
02:13:08 openstackgerrit Merged openstack/nova master: Deduplicate instance.interface_attach samples https://review.openstack.org/515740
02:13:14 openstackgerrit Merged openstack/nova master: Deduplicate instance.live_migration notification samples https://review.openstack.org/515745
02:13:20 openstackgerrit Merged openstack/nova master: Deduplicate instance.reboot notification samples https://review.openstack.org/515751
09:44:30 openstackgerrit Michael Still proposed openstack/nova master: Move flushing block devices to privsep. https://review.openstack.org/519010
09:44:30 openstackgerrit Michael Still proposed openstack/nova master: Convert ext filesystem resizes to privsep. https://review.openstack.org/517516
09:44:31 openstackgerrit Michael Still proposed openstack/nova master: Start moving users of parted to privsep. https://review.openstack.org/519011
14:59:06 openstackgerrit Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
16:57:45 mriedem boooooya
16:57:57 efried Do tell
16:58:21 efried mriedem Back stateside?
16:59:09 mriedem rotting in LAX
17:01:38 efried Sorry to hear that.
17:03:21 efried mriedem You looking around for easy reviews to keep you busy?
17:03:27 efried <smiles sweetly>
17:03:58 mriedem nope
17:04:08 mriedem i'm going on about 1 hour of sleep so it wouldn't be helpful
17:07:10 efried mriedem What sacrifices need to be made to what gods to get a requirements bump backported to stable?
17:08:53 mriedem which one?
17:10:03 efried pypowervm. Found a bug that makes OVS-connected VMs drop their connections unrecoverably on reboot.
17:10:52 efried There's no patch up yet; still working through the fix on the pypowervm & nova-powervm side.
17:11:18 efried Just testing the waters to see if we're going to be able to backport the nova-powervm fix in its pretty form, or if we're gonna have to hack the crap out of it for stable.
17:15:57 mriedem1 sorry, wifi sucks
17:16:15 mriedem1 efried: in general we don't bump minim version in g-r
17:16:22 mriedem1 *minimum
17:17:08 efried Yuh. Are there cases for not-in-general?
17:17:34 mriedem1 yeah, cve's for one
17:18:07 mriedem1 we could maybe fudge this since it's an optional dependency,
17:18:18 mriedem1 but it is in requirements.txt so it kind of depends on how packagers handle requirements,
17:18:36 mriedem1 i know fedora/rhel strip out the requirements.txt and just handle package deps in the rpm spec
17:20:53 efried It's not really a security thing. Just a your-vm-will-break thing.
17:28:52 mriedem1 sure
17:29:13 mriedem one of the risks of putting a bunch of the driver functionality into a library, which makes things sticky
17:29:25 mriedem i will happily defer to his tonyb-ness on this
17:32:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Add FlavorPayload.description for versioned notifications https://review.openstack.org/514782
17:32:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add microversion to allow setting flavor description https://review.openstack.org/515213
17:52:04 mriedem efried: is this new? http://logs.openstack.org/73/513473/2/check/legacy-tempest-dsvm-cells/be1a0fa/logs/screen-placement-api.txt.gz#_Nov_10_01_14_05_230382
17:52:11 efried ...
17:52:56 efried mriedem It doesnae look good.
17:53:05 efried looking...
17:54:19 mriedem shit i don't think we index the placement api logs
17:58:32 clarkb we should, though indexing has been funky with the zuul v3 switch (slowly been improving its performance)
17:59:44 efried mriedem ensure_consumer_project_user is only a couple weeks old (https://review.openstack.org/#/c/500410/) but I don't see how the _ensure_project thing could fail unless timing (someone else came along and whacked the entry) or the insert failed.
18:00:51 openstack Launchpad bug 1731668 in OpenStack Compute (nova) "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Undecided,New]
18:00:51 mriedem https://bugs.launchpad.net/nova/+bug/1731668
18:00:54 efried oh. Or external_id ain't in there.
18:01:18 efried naw, cause that should insert it.
18:03:13 mriedem this is during scheduling, so not sure if it's a race or what - it must be a race otherwise we'd have a pretty high failure rate
18:03:23 mriedem 21 hits in 10 days
18:07:20 openstackgerrit Eric Fried proposed openstack/nova master: EXPERIMENTAL: Retry getting project ID https://review.openstack.org/519034
18:07:28 efried mriedem If it's racey, this'll fix it ^
18:07:38 efried But the real deal ought to have a limited number of retries :)
18:11:29 mriedem that seems pretty wrong, i mean, if the insert fails because of a duplicate entry
18:16:16 efried mriedem Depends how racey it is. Clearly the second SELECT failed, or we wouldn't be seeing the error. So we got to the exception clause, inserted a new one, then someone else came in and deleted it before we got to the return.
18:18:09 efried Sorry, let me try that again: The first SELECT returns nada. Then we try to INSERT, but someone had already inserted, so we get the exception. Then before we get a chance to run the second SELECT, someone comes along and deletes the entry.
18:18:25 mriedem true
18:18:54 efried That'd be a pretty wild race to be seeing so frequently.
18:26:04 mriedem in debugging this failed job, it looked like the scheduler was creating allocations twice for the same instance in the same scheduling run, but maybe it was moving the instance or something, i didn't dig into what the test does
18:26:49 mriedem actually this just failed in server create
18:28:26 efried mriedem Where do project IDs ever get deleted?
18:29:31 mriedem tempest cleanup
18:29:38 mriedem but that wouldn't be happening here,
18:29:47 mriedem and the project delete would be via keystone on test teardown in tempest
18:30:09 mriedem when we set allocations in placement, we first remove all of the existing allocations it looks like,

Earlier   Later